Fix content-type for bundles

Fix the Content-Type HTTP response header being overwritten with the simple "media type" default value, which meant the response was always application/json.
Now the Content-Type header takes precedence over that value if it is specified as an override, such as in an HTTP bundle.
Also updates the .NET Core SDK and NuGet packages to their latest versions.

Codecov Report

Merging #84 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #84      +/-   ##
==========================================
+ Coverage   98.94%   98.94%   +<.01%
==========================================
  Files          12       12
  Lines         947      950       +3
  Branches      150      151       +1
==========================================
+ Hits          937      940       +3
  Partials       10       10

Impacted Files
Coverage Δ

...ClientInterception/HttpClientInterceptorOptions.cs
98.17% <100%> (+0.02%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 972d85e...40390b6. Read the comment docs.


Fix collecting of EidosMentions

Adjust grounding name

@zupon, here's an update for string tests.  Let me know how close or far that is from what you need.

@zupon, @BeckySharp, to ground against simple text, I avoided some things in EidosOntologyGrounder.groundOntology because they involve use of the mention type.  I can add that as a parameter that can be passed in along with the text, though.  I do see that patterns are matched against odinMention.text while wordToVec gets run on the canonicalNameParts.  This was one of my concerns with regrounding.  If Ben calls it without a proper mention, it will probably not get the same result if it was a pattern that matched when grounded previously.

OK, makes sense, I wonder if we need to receive both when he calls
"reground".... or we can parse on the fly and repeat the canonicalization?
On Tue, Dec 3, 2019 at 2:51 PM Keith Alcock<EMAIL_ADDRESS>wrote:

@zupon https://github.com/zupon, @BeckySharp
https://github.com/BeckySharp, to ground against simple text, I avoided
some things in EidosOntologyGrounder.groundOntology because they involve
use of the mention type. I can add that as a parameter that can be passed
in along with the text, though. I do see that patterns are matched against
odinMention.text while wordToVec gets run on the canonicalNameParts. This
was one of my concerns with regrounding. If Ben calls it without a proper
mention, it will probably not get the same result if it was a pattern that
matched when grounded previously.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/709?email_source=notifications&email_token=ABJCPCK62MEPKTTGRDD2JBDQW3ID7A5CNFSM4JR46SV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF256OA#issuecomment-561372984,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJCPCJYPYXWFYHE4CLLP73QW3ID7ANCNFSM4JR46SVQ
.

I added some stuff to groundStrings in EidosOntologyGrounder so that we can match the regex patterns for the property branch without needing to invoke any kinds of mentions. I also changed around some of how I was checking the strings, and things seem to be working better. @BeckySharp  we can look at it when we meet tomorrow (12/5) and @kwalcock I'll keep you posted on what's going on.

Cool.  I was going to do something like that five minutes from now.  Instead, I'll wait and see what you came up with.

sounds great!
On Wed, Dec 4, 2019 at 10:55 AM zupon<EMAIL_ADDRESS>wrote:

I added some stuff to groundStrings in EidosOntologyGrounder so that we
can match the regex patterns for the property branch without needing to
invoke any kinds of mentions. I also changed around some of how I was
checking the strings, and things seem to be working better. @BeckySharp
https://github.com/BeckySharp we can look at it when we meet tomorrow
(12/5) and @kwalcock https://github.com/kwalcock I'll keep you posted
on what's going on.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/709?email_source=notifications&email_token=ABJCPCI6SYRG6VVHGYPMPVTQW7VHTA5CNFSM4JR46SV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF55TVQ#issuecomment-561764822,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJCPCJ6DXV5VXTIVGCR7W3QW7VHTANCNFSM4JR46SVQ
.

IIRC canonicalization requires things like POS which won't be available either.
 def isCanonical(lemma: String, tag: String, ner: String)

true, though we could parse them again, or at least POS tag them with
processors....
On Thu, Dec 5, 2019 at 8:40 AM Keith Alcock<EMAIL_ADDRESS>wrote:

IIRC canonicalization requires things like POS which won't be available
either.
def isCanonical(lemma: String, tag: String, ner: String)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/709?email_source=notifications&email_token=ABJCPCKBXXMRLOKSCIEKFLLQXEOHTA5CNFSM4JR46SV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGBEDSI#issuecomment-562184649,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJCPCK3UICJW5243HSOV4TQXEOHTANCNFSM4JR46SVQ
.

Wouldn't we need the entire sentence then?  That may not be in the mention text or canonicalName.

I think for POS tagging/NER/lemmatization we'd do decent with just the
mention text, but I can try to check empirically if you want
On Thu, Dec 5, 2019 at 9:37 AM Keith Alcock<EMAIL_ADDRESS>wrote:

Wouldn't we need the entire sentence then? That may not be in the mention
text or canonicalName.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/709?email_source=notifications&email_token=ABJCPCJACFGUF7U5RZ76PBDQXEUSZA5CNFSM4JR46SV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGBJ3OI#issuecomment-562208185,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJCPCO2EZDVYOASTPIZDP3QXEUSZANCNFSM4JR46SVQ
.


[MXNET-324] Use default nvcc when configured nvcc not present.

Description
This fixes an issue with the nvcc path resolution logic in the Makefile (#10553).  In prior releases the makefile would use $(USE_CUDA_PATH)/bin/nvcc for nvcc no matter what was specified in config. For some users this was actually a good thing if 'nvcc' was not on the path, as it would specify the binary with an absolute path.  However the behaviour in the prior release caused issues when cross compiling. You may have an arm and x86 cuda path specified, and you need to select the correct nvcc for your platform.
This PR checks the configured nvcc to ensure it is actually on the path.  If it isn't it will set the binary to the absolute path.  This should match the behaviour of the prior release for users without nvcc on the path, and also allow for cross-compilation.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Comments

Note on why this was missed in CI: we do not have any CUDA CI configurations that do not have nvcc on the path.

@cjolivier01 would you be able to take a look ?


Silent failure of "git push"

This code swallow any exceptions and leave repo in weird state where task reports that tag assigend successfully, but it's not.
https://github.com/karmann-dm/gradle-versions-plugin/blob/ba3c8812f4f9dae1dfb5d328e2d851e2e8e875ac/src/main/java/com/karmanno/plugins/tasks/AssignTagTask.java#L27-L29
It could fail with multiple reasons, e.g. authentication, lack of git config, lack of remote, etc
My proposal is to delete try catch, if it breaks, let the user know that task failed and why.
Therefore task will be in failed state which is expected

Mitigated in https://github.com/karmann-dm/gradle-versions-plugin/releases/tag/2.0.0 version, push functionality has been removed, only tagging remained. For now you can just do git push --tags manually or within you CI tool. Will be re-developed in 2.0.1


Prevent test for large IN running on JDBC

H2 is unable to handle the large NOT IN queries
cc @electrum

Looks good


Initial implementation of firewalld backend

Presently, we use one zone for all interfaces, with one policy for that zone to enable masquerading. Every time a zone or policy is added, we need to do a firewall reload, so we can only make one zone of each (the very first time netavark runs) unless we want to wipe out our port-forwarding rules every time we run. This differs a lot from my original architecture, but I think it's the best we can do given the constaints.
Presently compiles, but does not actually work. When I make the dbus calls manually via busctl is does work. I'm slowly tracking down why, but debugging dbus calls like this is painful.

It works now.

@baude @Luap99 @flouthoc PTAL, this is ready

LGTM.  @Luap99 @baude PTAL

/lgtm


[Docker swarm mode] No round robin when using service

What I tried
docker network create -d overlay frontend

docker service create --name traefik --network frontend -p 8081:8080 -p 81:80 --mount type=bind,src=/var/run/docker.sock,dst=/var/run/docker.sock traefik:camembert --web --docker.swarmmode=true --docker.domain=swarm.test --logLevel=DEBUG

docker service create --name whoami --replicas 5 --label traefik.port=80 --network frontend emilevauge/whoami

But every time I run:
curl -H 'Host: whoami.swarm.test:81' http://x.y.z.t:81/

I have the same container responding (instead of seeing different container ids).
That's strange since this should be distributed by docker itself...

EDIT: It seems like it switches to another container if no activity occurs. Is there a keep-alive between Traefik and the backend or something equivalent?

@vincentlepot This is kinda to be expected. Træfik in swarmmode points to the virtual ip of the swarm and thus, let the docker swarm mode internal load balancer work. So it really depends on how the swarm mode load balancer handle these.

Possibly related to https://github.com/docker/docker/issues/25325

The strange thing is that when using another reverse proxy - Interlock for instance - I don't have the issue with the same setup.

@mvdstam Does not really look the same since I don't have any error, just staying on requesting the same container. The other ones are untouched.
It is as if a connection is settled on a container and remains opened. Is there a way to tell Traefik not to use persistent connections on the backends in this case?

In this particular example the emilevauge/whoami http server sets Keep-Alive due to the default settings in Go HTTP server which is 30 seconds, so if you try to refresh in 31s it switches to another instance.
For me it looks that traefik obeys that which i'd not classify as bug. More of a feature which might need on/off configuration switch?
@vincentlepot try this (from Docker demos):
docker service create --name whoami --replicas 5 --label traefik.port=80 --network frontend instavote/vote

and it does the balancing on every request.

I agree with you, this is not a bug, more a configuration switch request.

Moreover, that sounds like a client side issue, not a traefik one. Hence, I close this one.


Static site build & components update

We are prepping our static site for production release again.
Need to update the following with ODI engineering changes and improvements (from drought, innovation, etc)
This effects:

Build system
Design system components
packages
11ty script

Old, done, closing


Uses catalog-production-alt by default

Uses catalog-production-alt by default in the rake task to update Solr config files since we are moving away from catalog-production
Updated Solr config files with current version of files in catalog-production-alt in pul_solr.

Coverage remained the same at 82.27% when pulling a8fae0129eca8331c0f80e292321e1bd43657191 on solrconf_update into 58b93154a69af9432861e14ca217bb685dbadaa4 on alma.


Cache the default region in location config

The default region is expensive to retrieve when one is not set by the user. Very often it needs expensive clouds calls (e.x. retrieve list of images).
Cache the value so it can easily be used by code that needs it. It can be used by customizers in starting from customizer(template) step or any other code after the first obtain call.

Closing because will not consistently set the property. When the location doesn't have a region set, but one is passed through flags (for example in provisioning.properties) we will not be able to obtain jclouds' default (as a non-default value is already passed in from flags).


Bumble Website Clone

Description
https://bumble.com/en-in/    I want to clone this website , I will make an exact clone of it and mobile responsive version.
I will make it in HTML,CSS and JS.
Code of Conduct

[X] I follow Contributing Guidelines of this project.

/assign

are you still working on this issue @StillAbeginnerr


Graph: fill type and line width support for area graph

Fill types
We should be able to configure the SVG paths by specifying whether to fill the paths with solid colors, gradients, or patterns.
Currently, the fill type is just solid colors. see this for example: https://codepen.io/apexcharts/pen/GQmoXP

This is supported via custom SVG defs, check out this example https://unovis.dev/docs/guides/tips-and-tricks#custom-fills-with-svg-defs

@rokotyan i'm trying to achieve this using the area but it doesn't seem possible. Do you have any example regarding this?

@rokotyan --vis-area-stroke-width: 5; doesn't seem to apply for area stroke width.

@dammy001 Try setting --vis-area-stroke-color along with --vis-area-stroke-width

I'm having this problem as well, settings stroke width and color produces not only the line stroke to be stronger. I tried other solutions but wasn't succeed

I made a work around by putting a VisLine after my VisArea and it worked well. I guess.
      <VisLine<DataRecord>
        x={useCallback((d: DataRecord) => d.x, [])}
        y={[useCallback((d: DataRecord) => d.y, [])]}
        curveType={CurveType.Linear}
        color="#049499"
      />
      <VisArea
        y={[useCallback((d: DataRecord) => d.y, [])]}
        x={useCallback((d: DataRecord) => d.x, [])}
        curveType={CurveType.Linear}
        color="url(#gradient)"
      />

      <VisLine<DataRecord>
        x={useCallback((d: DataRecord) => d.x, [])}
        y={[useCallback((d: DataRecord) => d.w, [])]}
        curveType={CurveType.Linear}
        color="#DE3563"
      />
      <VisArea
        x={useCallback((d: DataRecord) => d.x, [])}
        y={[useCallback((d: DataRecord) => d.w, [])]}
        curveType={CurveType.Linear}
        color="url(#gradient2)"
      />


feat: Support dyn_compare_scalar for Decimal256

Which issue does this PR close?
Closes #4083
Rationale for this change
What changes are included in this PR?
Are there any user-facing changes?
Yes

Could we add some tests showing what happens with values like i256::MAX, I think this will return an error which is fine. Longer term I have plans to fix this #3999, but we should make sure that we don't silently truncate scalars


[changed] Prevent Return in menu from submitting surrounding form

The default behaviour of hitting Return in a <input type=text> is to submit
the immediate surrounding form. I would hazard a guess that this isn't the
expected result when using Return to select an item in the autocomplete
menu. Since we don't expose the event object to the consumer via onSelect we
need to make sure we're making the most sensible choice by default.
Hitting Return a second time (after the menu has closed itself) will
result in the default, expected behaviour.

Looks good!


add extract from kaggle and save to duckDB

I see overlap between the two extraction scripts, please work together to identify commonalities and differences.
The end goal should be to populate a duckdb file with clean data extracted from the Kaggle website.
This duckdb file can then be stored on a cloud-based database service.
Applications will be connected to this cloud-based service.

One you agree on a final extract.py file structure, explore how to set up a cloud-based db with https://motherduck.com/
Explore fetching data from the cloud-based db afterwards.

Merging into development branch
the next step is to clean redundant scripts in development into main


CAN ID fix and writing

This is to fix an ongoing issue with CAN reading and implement a better sensor management system upon that being fixed.
According to typical CAN system designs, sensors should be pushing data constantly. Previously, we were using a call and response system. This pull request adds an easy function for sending data.
There was previously an error where, due to incorrectly initialized CAN ids on the Teensy side, most messages were being lost. In addition, the mask on the RoboRIO side had an error that was leading to further message loss, although this was introduced more recently.
The Teensy ID initialization is fixed here, and a function is added to allow for writing directly.

I would like to test this on Pinky before merging.

Regarding CAN IDs: The FRC system seems to use 28 to 32bit IDs. Talons use at least 26 bits. I will note here that setting the bitmask to include bits 29-32 inclusive on the RoboRIO led to almost no packet transmission, which implies that 28 bit IDs are use. Since FlexCAN takes in a 32bit number, we just set bits 29-32 to 0.


Refactor, get rid of global gs object, and fix naming

This depends on https://github.com/metarhia/globalstorage/pull/297

Landed in https://github.com/metarhia/globalstorage/commit/30b3387dff419254f5dd09b346c44fabec7e8510.


[Readme] Should use the same file name as default in Makefile

Makefile: WANDB_SECRET_FILE?=wandb-token.txt
readme: wandb-key.txt
in Readme should be same name

exactly!

done


String literal with link references not resolved

I have a string literal:
`[link] https://www.google.com

[Link][link] to google.
`

When using this a a source in my app using 2.5.0 of react-markdown the link ref is printed verbatim instead of being resolved. Moving the link ref definition to the end of the file also does not help.

Looks like the same issue as https://github.com/rexxars/react-markdown/issues/91.

Is this even valid markdown? I can't seem to get any other markdown parsers to render this, either?

It's described here: https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet#links
Or am I missreading the docs?

Should be something like this:
[Link to Google][some-identifier]

[some-identifier]: https://www.google.com

Well, the only problem with your example is the missing : after [link] in the first line :)


Trie in Python

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
I have implemented Trie using dictionary(adding new word & search of added words)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
closes #839
Fill up the DCP form
If you are a part of DCP 21, fill up this form after the PR is merged.
The links provided in the form will be checked thoroughly, if someone tries fooling around, strict actions will be taken against the individual.

ok

changes done

ok

Thank you this is my first merged PR


Open external links in new tab

Is your feature request related to a problem? Please describe.
As a user i want to easily go back to integreat after opening a external link
Describe the solution you'd like
External links should be opened in a new link
Describe alternatives you've considered
Keep it as it is
Additional context

this is a feature request from the target group task force
see additional information in open project: https://openproject.tuerantuer.org/notifications/details/3233/overview
this feature request was already discussed and approved by ui/ux (both Toni and Nikola)

Thinking out loud:
For any Link from react-router-dom or  tag just add target='_blank'

@ztefanie we had the same discussion a few years back, see https://chat.tuerantuer.org/digitalfabrik/pl/rj7ars9mmfbg8na47eyz78z5be.
In the end, we decided to keep opening external links in the same tab. Perhaps we should check if we actually think that we should change this now. Personally I agree that it probably makes sense though.

Thinking out loud: For any Link from react-router-dom or tag just add target='_blank'

@bahaaTuffaha Yup, perfect, do you want to take on this one? You'll have to figure out how to get the target into the RemoteContent :)

RemoteContent? 🤔 we either adjust the content (my guess is from cms)😇 or inject some JavaScript to webview 😈

The file is called RemoteContent :) We could ask the CMS team to adjust this but I think it's more of an App behavior.


Using tink 1.2.2 with java 7 fails with java.security.NoSuchAlgorithmException: Cannot find any provider supporting AES/GCM/NoPadding

We are using tink 1.2.2 with java 7 and it fails with java.security.NoSuchAlgorithmException: Cannot find any provider supporting AES/GCM/NoPadding. Do you have a suggestion on how we can bypass this issue. Our current setup does not allow us to easily migrate to a newer version of java that has support for the algorithm. We have tried adding the Bouncy castle provider but the error still occurs.
We are using the Hybrid Encryption primitive.
KeysetHandle privateKeysetHandle = KeysetHandle                 .generateNew(HybridKeyTemplates.ECIES_P256_HKDF_HMAC_SHA256_AES128_GCM)

Bouncy Castle should support that algorithm. Which version of BC did you use?
You can also try https://conscrypt.org.


Having a hard time compiling windows version. any doc about it?

Here are some information on how to compile OpenColorIO on Windows:
https://opencolorio.readthedocs.io/en/latest/quick_start/installation.html
The content is a little bit outdated but it should give good hints.


Disparity map???

HI, I am trying to put my own disparity map inside the OpenSFM dense.py.
There is a PATCH BASED STEREO which OpenSFM uses in creating dense map.
But I can't find the code inside the depth-map computation method that uses disparity map.
Does anyone know about this algorithm??

Hi @YangJae96 ,
You can find the dense depthmap computation here : https://github.com/mapillary/OpenSfM/blob/master/opensfm/src/dense/src/depthmap.cc
Yann


Maybe try a bit more before opening an issue claiming it doens't work. Also might want to specify what isn't working.


More examples would be great

Talking to friends who could probably use this in their jobs but aren't used to thinking of things this way, it seems like the mose useful introduction would be many small-but-interesting examples. A standard format for them might make them easier to parse, as well.

This has been split into separate issues for various examples, so I'll close it; if inspiration is needed later, refer to the lists above and create new issues!


ML-Enabler Staging Memory + Model.zip Upload

Staging is inconsistent if model.zip uploads fail or work. Sometimes they fail at 100%,  and other times they fail earlier in the upload process. The size of the model.zip file isn't the issue because these models are consistent with other model sizes, and a model that fails to upload to staging will upload successfully to prod. Sometimes a model will fail to upload, then a little bit later on after creating a new project version or re-trying the same project version upload will succeed.  I haven't really figured out a pattern.
The AWS logs indicate that that sometimes memory can't be allocated, so I think we should explore increasing memory allocations.
cc @ingalls

I've doubled the memory on both prod & staging and we haven't seen any issues since then. Feel free to reopen if this once again becomes an issue.


Pretty print output of notary key list.

The output is in sorted order by gun, by role, by key ID, and by location, and looks like:
    ROLE     GUN                                 KEY ID                                        LOCATION
-----------------------------------------------------------------------------------------------------------------
  root              4e7621b361d21344eaa4d4f0dff24cc3db4eb305dbe7f5009b3592e16908b0ad   yubikey
  root              95e4c4e2c65b5b4e088c64a284f89c505eee60e0c741c7e1788bb2239c8d1173   file (/tmp/xyzf/private)
  snapshot   repo   d7897f2e74403c9c4939dd8a975c61ac1031db5511ec5f31d06b6ed4772643ee   file (/tmp/xyzf/private)
  targets    repo   7cc91433422c0f3f1a2b3973955625c907ac79ef2621d4af9e329217c634a79b   file (/tmp/xyzf/private)

Fixes #273.

Wow, this looks great!

Assuming a message is added about no keys being found (or if I've just missed it and it's in there already) LGTM.


Fixing the bug where nested describe namespace was flattened.

This code handles both degenerate (zero-length) and normal cases
of array of test suite names.
Somewhere from 0.3.3 onwards there was a bug creeping in. This caused the
namespace to flatten, ie. only the first word of a namespace was included. This corresponds
to a flat namespace.
Nested describe's (per Jasmine language) were being flattened thus and overlaps of names
started happening.

CLA signed.

Thanks, will review doon. But why did you close the PR?

Sorry, I thought this was part of the workflow. :)
What I did was try to now put a new pull request to fix my typos per Javascript (I used semicolons etc.)
Can you see the latest, as passing the automation on syntax?

[Clicked] It's open. :) As said, this is very educational. Getting the hang of it.

Hygienics done!
Really excited to see what comes. I'll be here all night along.

@Dignifiedquire Done refactoring.

If you want any comments etc feel free. I'm at it. :)

Sorry for not responding earlier. This looks good to me. You mentioned something about testing this, that would be nice but I'm not sure how to do that as I can't reproduce the error myself at the moment.

It would require having the reporter be run on three cases I think:

flat describe structure (just plain Jasmine, with no nesting, just a flat describe() and with it normal it() specs.

level nested describe (having one level of nesting)

level nested describe (kind of 'checking the iteration beyond obvious')
Where could I try these?

@jpaulin I would suggest adding those into this repo, so we always run them, it's okay if the we have to manually review the output for now.

I'll start doing the change now. Thanks for getting back on this.
I'll check the Travis part more, since that's kind of new for me. Travis is used automatically in all Github projects as the CI engine. I generally know CI systems quite well, but it's the particulars that make me first read the introduction.
Btw, any chance I can live with current devDependencies and **not **add any new?

I'll check the Travis part more, since that's kind of new for me. Travis is used automatically in all Github projects as the CI engine. I generally know CI systems quite well, but it's the particulars that make me first read the introduction.

It's pretty straightforward it'll just give a linux env executing the commands you give it. Also it gives a Firefox to run karma in a real browser which is nice. Checkout out this .travis.yml from karma-mocha for some details. Also the travis docs are pretty decent: https://docs.travis-ci.com/user/getting-started/

Btw, any chance I can live with current devDependencies and *not *add any new?

Not sure what you mean with that

Not sure what you mean with that

I'm trying to design a test that fails in case this PR code wouldn't solve what it was meant for. So the initial idea is to have at least 2 Jasmine flavor test file (first: "depth one", ie. use of 'describe' inside another 'describe') and the second for depth two. If the new code now under work fails to produce output, the output would either be ~ non-existent, or still be flattened. That would be the contents of the test.
The question I have is that does Travis basically contain already all that's required?

Just to clarify travis just runs what we have in this repo. At the moment there is no real test setup in this repo, aka there is no karma started with the reporter and executed. So for this you would have to set this up, you can take a look karma-mocha for an example how this is done. So the idea is that this test runs on your local machine on calling grunt like it does linting at the moment. And if that works they should work on travis as well.

Very good tip, and probably just saved tons of my forehead bumping to wall! I would've been looking for that ghost runner for a looong time. Now will go for local first.

I think this PR can be closed, superseded by recent activity in PR #85

#85 was merged so closing this one.


Package request: Geary and Shotwell

With the recent continuation of development of Geary and Shotwell, it is worth packing them into AppImages.
Although I should mention that they probably will require at least Centos7 as a base.

I think these apps depends heavily on latest versions of many libs, so, nevermind


Enabled? -> Activated?

@thieman the command is dusty activate so it makes sense that afterwards, the bundle would be activated instead of enabled.

So happy you changed the docs too, literally just got done working on those :-P

Thats where I noticed it


求代理功能啊

天朝开发
求代理配置方法

First of all, use English in the open source world.
If you have a PyPI mirror deployed in the intranet, use --pypi-mirror or replace the [source] section in your Pipfile.
Otherwise, if you need proxies, HTTP_PROXY and HTTPS_PROXY env var should just work for you.

@frostming
while i set HTTP_PROXY and HTTPS_PROXY env var,
some other soft(e.g. poco service) cannot work (conflict)
so i need pipenv proxy feature like pip install some_package --proxy=<IP_ADDRESS>:port


Equilbrium not working

When you start a game using the in client Equilibrium game mode you are unable to place buildings or build units from buildings. The UI within the game mode is completely broken.

That's related to the last FAF patch, not to the client. I'm transferring this issue to the repository.


ldgallery: init at 2.0

Motivation for this change
This is an attempt at packaging ldgallery, a static gallery generator.
It's made out of three parts:

"ldgallery-viewer" which requires NodeJS to be built
"haskellPackages.ldgallery-compiler" which is an Haskell app
the "ldgallery" package that assembles the two previous ones

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@pacien Sorry about basically forgetting about this.  Are you still hoping to get this into nixpkgs?
I guess this is just ready for a final review and hopefully a merge-in?

@thiagokokada wrote:

Code looks good, but build is failing (nixpkgs-review):
error: anonymous function at /home/thiagoko/.cache/nixpkgs-review/pr-86599-1/nixpkgs/pkgs/development/node-packages/node-env.nix:3:1 called without required argument 'util-linux', at /home/thiagoko/.cache/nixpkgs-review/pr-86599-1/nixpkgs/pkgs/tools/graphics/ldgallery/viewer/node-composition.nix:8:13
(use '--show-trace' to show detailed location information)
nix eval --json (import /nix/store/rlqba15z5wcnh7msqq94jddxjswk2032-nixpkgs-review-2.4.2/lib/python3.8/site-packages/nixpkgs_review/nix/evalAttrs.nix /run/user/1000/tmp6b8iseyh) failed to run, /run/user/1000/tmp6b8iseyh was stored inspection
https://github.com/NixOS/nixpkgs/pull/86599 failed to build
$ git worktree prune

This seems to be due to https://github.com/svanderburg/node2nix/issues/213.
I modified the generate.sh script accordingly to fix the package in the meantime.
@cdepillabout

Are you still hoping to get this into nixpkgs?
I guess this is just ready for a final review and hopefully a merge-in?

Yes, it builds and should now be ready.

@thiagokokada wrote:

Code looks good, but build is failing (nixpkgs-review):
error: anonymous function at /home/thiagoko/.cache/nixpkgs-review/pr-86599-1/nixpkgs/pkgs/development/node-packages/node-env.nix:3:1 called without required argument 'util-linux', at /home/thiagoko/.cache/nixpkgs-review/pr-86599-1/nixpkgs/pkgs/tools/graphics/ldgallery/viewer/node-composition.nix:8:13
(use '--show-trace' to show detailed location information)
nix eval --json (import /nix/store/rlqba15z5wcnh7msqq94jddxjswk2032-nixpkgs-review-2.4.2/lib/python3.8/site-packages/nixpkgs_review/nix/evalAttrs.nix /run/user/1000/tmp6b8iseyh) failed to run, /run/user/1000/tmp6b8iseyh was stored inspection
https://github.com/NixOS/nixpkgs/pull/86599 failed to build
$ git worktree prune

This seems to be due to https://github.com/svanderburg/node2nix/issues/213.
I modified the generate.sh script accordingly to fix the package in the meantime.
@cdepillabout

Are you still hoping to get this into nixpkgs?
I guess this is just ready for a final review and hopefully a merge-in?

Yes, it builds and should now be ready.

Result of nixpkgs-review pr 86599 1

2 packages built:

haskellPackages.ldgallery-compiler
ldgallery

Result of nixpkgs-review pr 86599 1

2 packages built:

haskellPackages.ldgallery-compiler
ldgallery

@pacien Looks good, thanks for fixing this up!

@pacien Looks good, thanks for fixing this up!


Top level component for cron and notifications

What's the best place / way to run a cron job for heartbeat? I need to check if user session is still valid. I tried to keep it in Layout component, but I get many instances of cron.
Secondly, I would like to have growl type of notifications / messages, but when I change page they disappear.
Many thanks for all help.

Set a timeout in componentDidMount and clear it on componentWillUnmount then you won't have multiple instances 👍

Secondly, I would like to have growl type of notifications / messages, but when I change page they disappear.

I guess you can use React portals for this 👍


How to deactivate or customize spellcheck

I don't know how, but I activated spell checking. The problem is that I write in italian and all not english words are signed as wrong. What are you using for spell checking and how can I add another language dictionary? Or how can I deactivate it?

This is the effect writing an Italian Markdown file

first run this to make sure what is the spell status for you
:verbose set spell?

if that outputs nospell
then try disabling vale by commenting these lines
https://github.com/abzcoding/lvim/blob/31b1620ce6f40ccfe4343e74e7c4a29dd50cb137/lua/user/null_ls/init.lua#L86-L89

closed due to inactivity


Moving cells around should recenter the notebook on that cell

When you move a cell up or down, the screen doesnt recenter around the cell that you are moving, so if you have to move the cell multiple spots, you need to keep moving your mouse/scrolling in the notebook

Thanks for your feedback. This issue is addressed with new native support for editing notebooks starting with v1.59 of VS Code. If this issue continues to occur in the latest build of VS Code, please reopen or enter a new issue.


Fix stitching serialization invalid json #2024 #1972

Same as https://github.com/ChilliCream/hotchocolate/pull/2025 without Convert.ToInt32

Hi, thanks for the effort on this one.

@erwan-joly  are you on version 10 or 11?

/AzurePipelines run

@erwan-joly  are you on version 10 or 11?

I’m on version 10.4.3

@erwan-joly The master branch is version 11.
Can you also do this fix on version 10, which is in branch "version_10_0_0_master"? The structure has changed so much that we cannot merge the change into version 10 automatically.
For version 11, I will merge it now. However, we will, I think, entirely rework this part for 11 since it is not ideal and clean to do it like that. With that, I do not mean the fix itself but how we handled this part in the first place.
Thanks for your help with this.

@PascalSenn fyi


This is definitely bug.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Browser details (please complete the following information):

OS: [e.g. OS X, Windows]
Browser [e.g. chrome, safari]
MetaMask Version [e.g. 5.0.2]

Additional context
Add any other context about the problem here.

I would like to get any updates for this issue.
Thanks guys


Enhancement of Ping Graph Scaling for Improved Data Visibility

Description of the feature, modification, idea or suggestion
Ping graph is really useful feature, but it has one drawback when using with fast targets. The graph’s default scale is quite large, capable of displaying up to 1000 ms or more. This causes all relevant data to be compacted at the bottom, making it difficult to analyze and interpret the ping data effectively.

Proposed implementation details (optional)
Here are a few suggestions on how this could be improved:
Adaptive Scaling: Implement adaptive scaling where the y-axis adjusts dynamically based on the range of ping values recorded. This will ensure that data is spread across the graph for better visibility and analysis.
Customizable Scale: Allow users to manually adjust the scale of the y-axis. Providing options to set maximum and minimum values can help in focusing on specific data ranges.

Options > General > Enable fixed graph scaling (experimental) or alternatively configure ProcessHacker.NetworkTools.PingMinScaling with the maximum scaling as a hexadecimal value (e.g. c8 == 200ms)

Thanks! Both methods offer so needed solution. I guess now what's needed is a way to bring these settings closer to a user rather then buried deep into options, but that's a topic for another issue.
Just out of curiosity: why does "Enable fixed graph scaling" option seems to do exactly opposite, namely it disables fixed graph scaling and enables adaptive graph scaling? Is it on purpose, or by accident?


221250 - Fix broken links from OPS report 5/20

@v-hearya : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Automated cherry pick of #8059: fix: avoid serial vm creation

Cherry pick of #8059 on release/3.3.
#8059: fix: avoid serial vm creation

/lgtm
/approve


Crashes without error when executing user32.SetWindowLongPtrW() with a WindowProc Callback

Trying to migrate an application from ffi-napi and everything works fine except for user32.SetWindowLongPtrW()
When it executes it crashes without any error message.
I try to set an Electron created window as a taskbar for Windows with that
Here are some code snippets
// type defs
const HANDLE = koffi.pointer(koffi.opaque('HANDLE'))
const HWND = koffi.alias('HWND', HANDLE)
const UINT = koffi.alias('UINT', 'uint')
const WPARAM = koffi.alias('WPARAM', 'uintptr')
const LPARAM = koffi.alias('LPARAM', 'longlong')
const WNDPROC = koffi.proto('WindowProc', LRESULT, [HWND, UINT, WPARAM, LPARAM])

const SetWindowLongPtrW = user32.func('SetWindowLongPtrW', koffi.pointer(WNDPROC), [HWND, INT, koffi.pointer(WNDPROC)])

const registerWinPosChangeHook = (windowHandle: bigint, posChangeCallback: (browserWindowHandle: bigint) => void) => {
  let origWindowProc: unknown

  // DModel types are leftover from old lib, but they all map to number or bigint anyway
  const hookCallback = (hWnd: DModel.HWND, msg: DModel.UINT, wParam: DModel.WPARAM, lParam: DModel.LPARAM) => {
    switch (msg) {
      case WM_MOVE:
      case WM_WINDOWPOSCHANGING:
        posChangeCallback(windowHandle)
        return 1
      default:
        // call default WindowProc in case we don't have any custom logic for it.
        const windowProc = user32.CallWindowProcW(origWindowProc, hWnd, msg, wParam, lParam)
        return windowProc
    }
  }

  const callback = koffi.register(hookCallback, koffi.pointer(WNDPROC))

  origWindowProc = user32.SetWindowLongPtrW(windowHandle, GWLP_WNDPROC, callback)
}

posChangeCallback  executes setTaskbarPos()
// hWnd belongs to a newly created window that we created in a previous function with SHAppBarMessage(ABM_QUERYPOS, appBarData)
const setTaskbarPos = (hWnd: bigint) => {
  const appBarData = {
    cbSize: 0,
    hWnd: hWnd,
    uEdge: 0x3,
    rc: {
      left: 0,
      top: height - actualTaskbarHeight,
      right: width,
      bottom: height,
    },
  }

  shell32.SHAppBarMessage(ABM_QUERYPOS, appBarData)
  shell32.SHAppBarMessage(ABM_SETPOS, appBarData)
  user32.SetWindowPos(appBarData.hWnd, HWND_TOP, 0, taskbarBottom, taskbarWidth, actualTaskbarHeight, SWP_NOACTIVATE | SWP_FRAMECHANGED | SWP_NOSENDCHANGING)
}

If I try to get an error with kernel32.getLastError() it gives me a 203 ERROR_ENVVAR_NOT_FOUND after I execute user32.SetWindowPos()
setTaskbarPos() on its own works fine without any crashes or errors and also does what it's supposed to do, but inside the WindowProc Callback it crashes, can't figure out what's wrong, so maybe I missed something or it's a Koffi issue or the typing for user32.SetWindowLongPtrW and/or user32.CallWindowProcW are wrong?

Win32 callbacks typically use the non-default __stdcall convention on x86. If you get it wrong, you'll get a crash in Node.js 32-bit, maybe that's it?
const WNDPROC = koffi.proto('__stdcall', 'WindowProc', LRESULT, [HWND, UINT, WPARAM, LPARAM])

Also, this callback will be called after the initial SetWindowLongPtrW() call has returned so you need to register it, as documented here:
https://koffi.dev/callbacks#registered-callbacks

I already tried to delcare the functions with __stdcall, but that also didn't work out, besides that I'm using a 64bit system and also compile it to just 64bit
I also registered the callback like that, but it still doesn't work
const callback = koffi.register(hookCallback, koffi.pointer(WNDPROC))

I have the same issue, the error is:
[10808:0202/174347.734:ERROR:node_bindings.cc(158)] Fatal error in V8: v8::HandleScope::CreateHandle() Cannot create a handle without a HandleScope
Electron: 28.2.1
Koffi: 2.7.3
I was able to successfully use EnumWindows callback though.
And this also work (change and immediately return old wndProc):
  baseWndProc = SetWindowProc(handle, GWL_WNDPROC, wndProcCallback);
  SetWindowProc(handle, GWL_WNDPROC, baseWndProc);

i.e. I suppose that the method is invoked and the wndProc is correctly set.
I've attached the sample repo - basically the vanilla Electron Filddle repo with subclass related stuff.
koffi-subclass.zip

I've made some changes that should fix this, at least for @estoykov, but if it's the same problem that'll fix both.
I've published Koffi 2.7.4-beta.1, install with npm install koffi@beta and test. Please report to me if it works and I'll make a proper release.

(sorry, closed by mistake)

It's working now ;)
    "koffi": "^2.7.4-beta.1"
And btw, sorry, but I forgot to include page loading in the repo:
  mainWindow.loadFile('index.html')

  const handle = mainWindow.getNativeWindowHandle().readInt32LE();

Anyway, thank you for the quick response!
I'm working with Windows API for almost 30 years, but never imagined that I will have an ability to subclass a window procedure at so "high level" :)
Cheers and keep the good work!

Thank you for the report!
I've made a sample based on the snippets of @Spookyfan, and it seems to work too.
const koffi = require('koffi');
const { app, BrowserWindow } = require('electron');

const HANDLE = koffi.pointer('HANDLE', koffi.opaque());
const HWND = koffi.alias('HWND', HANDLE);
const WNDPROC = koffi.proto('__stdcall', 'WindowProc', 'intptr_t', [HWND, 'uint', 'uintptr_t', 'intptr_t']);

const GWLP_WNDPROC = -4;
const WM_MOVE = 0x0003;
const WM_WINDOWPOSCHANGING = 0x0046;

const user32 = koffi.load('user32.dll');
const SetWindowLongPtrW = user32.func('SetWindowLongPtrW', koffi.pointer(WNDPROC), [HWND, 'int', koffi.pointer(WNDPROC)]);
const CallWindowProcW = user32.func('CallWindowProcW', 'intptr_t', [koffi.pointer(WNDPROC), HWND, 'uint', 'uintptr_t', 'intptr_t']);

const registerWinPosChangeHook = (hWnd, posChangeCallback) => {
    let origWindowProc;

    const hookCallback = (hWnd, msg, wParam, lParam) => {
        switch (msg) {
            case WM_MOVE: {
                let x = lParam & 0xFFFF;
                let y = (lParam >> 16) & 0xFFFF;

                posChangeCallback(hWnd, x, y);
            } break;
        }

        const windowProc = CallWindowProcW(origWindowProc, hWnd, msg, wParam, lParam);
        return windowProc;
    };

    const callback = koffi.register(hookCallback, koffi.pointer(WNDPROC));
    origWindowProc = SetWindowLongPtrW(hWnd, GWLP_WNDPROC, callback);
}

function createWindow () {
    const mainWindow = new BrowserWindow({
        width: 800,
        height: 600
    });

    const hWnd = mainWindow.getNativeWindowHandle().readInt32LE();

    registerWinPosChangeHook(hWnd, (hWnd, x, y) => {
        let msg = `Moving: ${x}x${y}`;
        console.log(msg);
    });
}

app.whenReady().then(() => {
    createWindow();
});

app.on('window-all-closed', function () {
    if (process.platform !== 'darwin')
        app.quit();
});

So I've published Koffi 2.7.4 with this fix and a couple other minor things.


magika_node missing from exports in package.json

Currently this package is not usable in typescript via node as the node export is missing from the package.json

ps: the labels were automatically added and I cant remove them

nvm, my mistake

+1

Extremely good point. It should be fixed now. Let me know if it looks good to you

Ignore last comment. For anyone else running into the resolution issue above, make sure to change tsconfig to use node16 or nodeNext as the moduleResolution strategy:
package.json exports

Any resolution on this? I'm trying to import MagikaNode from a nextjs server side component.  It works even though there is a linting error, however the app won't compile.
Module '"magika"' has no exported member 'MagikaNode'.ts(2305)

I found that to be able to import in CJS where I just cannot simpl migrate to ESM I need to modify magika package.json exports to (prepend both with ./) otherwise I am not able to import from 'magika/node':
    "exports": {
        "./node": {
            "import": "./dist/mjs/magika_node.js",
            "require": "./dist/cjs/magika_node.js"
        },
        "./default": {
            "import": "./dist/mjs/magika.js",
            "require": "./dist/cjs/magika.js"
        }
    },


XBE Browser shows DVD files, FTP fails to retrieve directory listing

Inserted a DVD, system info shows media detected, xbe browser is able to view the files but FTP client fails to retrieve the listing.
Command:	LIST
Response:	550 Can't open directory
Error:	Failed to retrieve directory listing

Fixed in 8f9953b


Credential input

Hello. It seems I type in my credentials every time, but as soon as i start nadeko it says it cant find the credential info and I click on the credentials .jnson file and they are completely wipe clean and i input all over again and the process restarts? Anyone know anything about this?

You are doing something terribly wrong.
Make sure you've enabled file extensions for known file types. Next to your NadekoBot.exe you must have credentials.json file (make sure you spelled it correctly).

So you dont edit credentials_example.json?

you rename credentials_example.json to credentials.json and edit that.

Woah i feel dumb. Haha thanks i got it :D


refactor(cli): add fallback with husky git params to deprecation handling

Description
This adds HUSKY_GIT_PARAMS fallback to the existing GIT_PARAMS deprecation handling, as described in their docs.

Note, I've created two solutions for this problem. This is one, and the other one is to remove (and later reimplement). Please pick one, I'll close the other.

Edit, I prefer this one since it's now backward compatible.

Motivation and Context

Usage examples
How Has This Been Tested?

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[X] All new and existing tests passed.

There we go! Now it shouldn't be backward incompatible and doesn't require a major update 😄

Ok, now it works fine I think. If GIT_PARAMS is used, but not defined, it falls back to the HUSKY_GIT_PARAMS check. If that is also not defined, the error is thrown. 😄

I prefer to use this one over #499 because it's backward compatible.

If this works fine I would also use this over #499 👍

OK, let's do this! :)


[hevce] prevent HRD parameters submitting for CQP cases

HRD parameters cause switching to CBR in media driver for HEVCe FEI

@lakulako AddVaMiscHRD just sets HRD parameters depending on CQP. But we need to avoid submitting of such buffer to driver for CQP case.

@vilichev, may be it would be more correct not to switch to CBR if HRD buffer is attached. Anyway CBR is not the only mode that uses HRD.

@vilichev, may be it would be more correct not to switch to CBR if HRD buffer is attached. Anyway CBR is not the only mode that uses HRD.
@lakulako I think we should fix it in MSDK too, since HRD does not intend for CQP/ICQ cases.


Support building against system libraries

Hi Branimir,
My MAME package in Debian just got this bug report:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=838052
It is common in Debian and Fedora to build against system libraries if possible, and I'd be glad to be able to do so with the embedded copy of bgfx. Is there any way this can be done currently, or if not, could you consider adding the possibility for at least the ones present in the usual Linux distributions.
At least, freetype, libsquish would be nice to support.
Thanks!

I have some modifications in libsquish. The problem with libsquish is that it's not supported at all by original author, so there is no way to push changes upstream so it sometime get to Debian repo.
FT2 is simpler, I just need to remove amalgamated version of library.
 While getting libsquish packaged, Wookey looked at all the forks of the library already present in unrelated packages like MAME, looked at the differences and fed patches upstream. The result is the new upstream release that recently got published.
In particular, I believe your changes are here: https://sourceforge.net/p/libsquish/code/69/
Sure, distributing the lib in 3rdparty is very fine, I just want a way to tell the build system to not build that and link against the available shared library from the system instead.

Other relevant links:

https://tracker.debian.org/news/798208 → Initial upload incorporating your changes
https://tracker.debian.org/news/800011 → New upstream release including them

Where is main repository?

What do you mean with main repo? You can download libsquish here:
http://debian.mirror.constant.com/pool/main/libs/libsquish/
This directory includes .debs and the source tar.gz: http://debian.mirror.constant.com/pool/main/libs/libsquish/libsquish_1.14.orig.tar.gz

@jmallach On a side node, why it's important to be able to build against system libraries in a first place? Because usually in gamedev all 3rdparty code is bundled in.

On the principle that we are distributing bgfx via MAME, and that in turns distributes a copy of libsquish (and freetype). If a security issue arises on any of these libs, the security team not only needs to patch and create advisories for say freetype, but for all the unrelated packages that embed a copy.
Fedora, Debian and other distros have a policy of using the system libraries when possible. So the default would be to still use the included bundles, but having some way to tell the build system to use system libraries would be nice for downstream distros.

I could add option --with-syslibs that would in those cases prefer using system libraries, but in case I fix some bug in for example libsquish how that's going to get to system library?
Also these things are non-issue for MAME, since they are just using library itself, not examples and utilities.

Honestly, what we do in those cases is send patches upstream.
Do you add functionality to libsquish often? Or, now that the changes are merged , do you envision needing changes to libsquish in the near future? It seems that the upstream maintainer is now quite open to adding bits, as he merged all the forks in one go.

The problem I had was that I wanted to use a different version of freetype.h, but couldn't because 3rdparty directory is included by default (so bgfx/3rdparty was used instead of the new one). Freetype is used only in examples, so I guess there should be a 3rdparty/examples/ include where all example (and non run-time) includes etc would be.


Create Entity With Predefined Rows

Hi,
I'm using Requery with Kotlin in an Android App, so i wonder if its any way to create an Entity with a predefined values
Thanks

Hi there,
are you speaking about default values? Then maybe this will help @Column(value = "...")

Yup, or you can also just define a factory method to create an object with the values you want.

I'm talking creating records before all, I have a Users table and i want to create 3 users when the table are created, i already tried coping the kotlin-android-example but it doesnt work


Feature request: Make categories configurable

I didn't like the predefined categories back in the days in Splitwise and I still do not like them. So I went on and finally changed them in my instance in the database.
I also added a column pointing to the name of the icon to be used for that category (see Lucide icon set).
The next obvious step would be to implement a UI to make categories really configurable.
More ideas

define a background color for each category group
probably we would want to keep the two standard categories "General" and "Payment"
when a category is about to be removed, warn if there are expenses referring to it. We could move them to "General" or let the user choose

Going further...
How about being able to use a different set of categories for each group?
What's your opinion about it?

I agree a UI would be really helpful, but in lieu of that for now, can you describe how you changed the categories on your instance?

Here is my Category database:

I could add a PR. But due to inactivity I implemented what I needed and thus my fork somehow diverged from this repo. You are of course free to look at my code:
https://github.com/shynst/spliit

This would be great. I had written a list of categories I felt were missing, but found this in the issues and think it would be a much cleaner fix for my problem.


pgbouncer stats blank

On the pgbouncer tab of the pgbadger reports we have no dataset for the following:
Histogram of session times, sessions per database, sessions per user, sessions per host and most used reserved pools. Is there something that needs turning on in our logs somewhere or is this a known issue with pgbadger? Any help would be much appreciated!

Sorry for the response delay. In your pgbouncer log file you should have lines like:
2019-04-11 07:10:21.827 1525 LOG Stats: 580 req/s, in 136760 b/s, out 64757 b/s,query 21449 us

if there is you should have a report. Which pgbadger version are you using and have you tried with latest development code?

I guess that you was using pgbouncer >= 1.11? In this case please use latest development code it has been fixed in commit 98d4c99.


UHF-9833: Support services

Codecov Report
Attention: Patch coverage is 98.85057% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 88.61%. Comparing base (253b206) to head (b531912).
Report is 1 commits behind head on main.

Files
Patch %
Lines

src/Environment/Environment.php
42.85%
4 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #169      +/-   ##
============================================
+ Coverage     87.26%   88.61%   +1.35%
+ Complexity      533      519      -14
============================================
  Files            76       76
  Lines          1782     2055     +273
============================================
+ Hits           1555     1821     +266
- Misses          227      234       +7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Segmentation fault on testnet.rb delegate node. unknow reason

run
./testnet.rb --create

Please be patient, this will take several minutes...
Replaying blockchain... Approximately 0.00% complete.Process 4877 stopped
* thread #1: tid = 0x9cccb9, 0x00007fff97ff7172 libsystem_c.dylib`strlen + 18, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
    frame #0: 0x00007fff97ff7172 libsystem_c.dylib`strlen + 18
libsystem_c.dylib`strlen + 18:
-> 0x7fff97ff7172:  pcmpeqb (%rdi), %xmm0
   0x7fff97ff7176:  pmovmskb %xmm0, %esi
   0x7fff97ff717a:  andq   $0xf, %rcx
   0x7fff97ff717e:  orq    $-0x1, %rax
(lldb) quit

To check whether this fix already address this issue.

Fixed in https://github.com/bitsuperlab/bitshares_play/commit/ade92ac2b5c908ae4781dfa5ae3e981683bd8c26


Have you considered adding a proxy function?

As you know, OpenAI is not available in some regions, so maybe some users need to set up a proxy for it.

Yup, see #105! Should land soon

Hmmm, actually, I mean, is it possible to have a socks or http proxy function? not "Custom OpenAI API address"

Released in v0.5.1!

Released in v0.5.1!

How should i use the proxy ?

Released in v0.5.1!

How should i use the proxy ?

See https://github.com/zurawiki/gptcommit/releases/tag/v0.5.1

Released in v0.5.1!

How should i use the proxy ?

See https://github.com/zurawiki/gptcommit/releases/tag/v0.5.1

I had already set the proxy config with the  gptcommit config set openai.proxy "http://<IP_ADDRESS>:8899"
But it didn't work

And I can't see any request in my proxy tools


⚠ ClamAV has degraded performance

In 676e3ee, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in b8ee144.


pg_dumpbinary does not tolerate partitionless partitioned tables

Running pg_dumpbinary against a database/schema with partitioned tables, I got:
-> pg_dumpbinary -d [source-db-name] -p $PGPORT -n [my-schema] -T 'pub*' -T 'wrk*' --load-via-partition-root -j 2 [directory-name]
Database [source-db-name] dump created at 2023-01-20T12:59:58
Dumping pre data section at 2023-01-20T12:59:58
Dumping data at 2023-01-20T13:00:17
psql:/tmp/pg_dumpbinB8AH.tmp:38: ERROR:  cannot copy from partitioned table "[table-name]"
HINT:  Try the COPY (SELECT ...) TO variant.
psql:/tmp/pg_dumpbinB8AH.tmp:42: ERROR:  current transaction is aborted, commands ignored until end of transaction block
psql:/tmp/pg_dumpbinB8AH.tmp:46: ERROR:  current transaction is aborted, commands ignored until end of transaction block
...
and I discovered that the table named in the error message was partitioned, but had zero partitions.
I was able to work around this (for now) by running a script to locate such "partitionless partitioned tables", and create a dummy partition for each. Once I did this, pg_dumpbinary ran without complaint.
Would it be possible to make pg_dumpbinary tolerant of this condition? I can't be sure I will always be able to use this workaround.
(pg_dump ran OK with tables in this condition.)
pg_dumpbinary Version: v2.7
PostgreSQL 14.5 on x86_64-pc-linux-gnu, compiled by gcc (GCC) 8.5.0 20210514 (Red Hat 8.5.0-10), 64-bit
Thank you,
Mike Tefft

Commit d449f26 might fix this issue.

Probably next week, I want to add regression test for the two fixes but it can takes more time, I will notice you in this issue.

Is this fix still present in pg_dumpbinary Version: v2.10 ? I seem to be hitting this issue again.

Yes, the fix is present. Can you report the error messages?

ah, sorry, there must be something else going on. I got errors on only one of several databases that have partitionless tables. Someone probably did something goofy to that database.


Util Hotkeys

hotkeys that wide hardo, open to lan, locate mapless, go to nether.
in specnr macro

These are can be added using the recently commited Julti Scripts system
However a command needs to be added to Julti to support window manipulation to support widehardo or eye zoom functionality

Util hotkeys are basically added, anything that can't be done is a result of scripting limitations. More scripting and commands stuff can be suggested here: #61


Documentation comments duplicated

Given the following schema:
schema.json
Document comments are emitted twice.

Version: 4.0.0-preview.17

Resolved in v4.0.0-preview.19


Path from local Broker (Memory or Redis) to dumping to K8s

Lets try to paint a picture of what the UX will be like once we've implemented all the possible paths, then figure out what the first steps should be.
Configuring MemoryBroker and RedisBroker
As a user, I want to be able to create either a MemoryBroker or a RedisBroker on my machine, depending on my requirements (e.g. mission critical or not), and depending on my flow as I test the product (happy to start with in memory first to keep it simple, then evolve to persistence with Redis).
tmctl create broker --type MemoryBroker
tmctl create broker --type RedisBroker

This means a Broker in tmctl is always defined as either one of these types. We don't necessarily allow for switching a Broker from one type to the other either (we could consider them to be completely distinct components).
Omitting type could default to MemoryBroker, because it is the lowest-friction path during product onboarding.
In the future, each of these broker types  might have different parameters, giving a dumb example here:
tmctl create broker --type MemoryBroker --maxMemory 1000
tmctl create broker --type RedisBroker --maxDisk 1000

Dumping to K8s
As a user, I'm going to dump my configuration to K8s YAML so I can apply to my Kubernetes Cluster. In doing so, I expect the dump command to do its best to carry over my local configuration to the Kubernetes cluster.

If I'm using a MemoryBroker locally, then tmctl dump should produce MemoryBroker in the YAML
If I'm using a RedisBroker locally, then tmctl dump should produce RedisBroker in the YAML

When using the tmctl dump --knative option, if possible, regardless of the broker type we should output a knative InMemoryBroker in the output.
If this becomes tricky or unclear e.g. due to additional parameters being specified for the RedisBroker, then tmctl dump --knative could e.g. produce an error if the local broker type is RedisBroker.
Short term
In the short term, we could keep only the following command with no option for Broker type:
tmctl create broker and this would default to MemoryBroker (as would be the case in the future).
When I dump to K8s, it would dump a MemoryBroker.
From there, the user could modify the manifest to switch it to a RedisBroker if they want.
We could also imagine an option like tmctl dump --convertMemoryToRedis true but I'm not sure if we need it.
If we're able to support tmctl create broker --type MemoryBroker and tmctl create broker --type RedisBroker before release thats a bonus.

Outcome from today:
We can keep the current behaviour for the release, in which locally memory broker is used, and dump produces a redisbroker. we'll iterate from there later.


Modal closing is handled inconsistently

If you defines actions via shorthand, the modal will close automatically when the action buttons are clicked However, if you want to define rich content in the modal (instead of simple text via the text prop), you have to define the children, and cannot use the actions shorthand. This means you need to define the actions via the Modal.Actions component. However, actions defined this way do not automatically close the modal when clicked. This means you must create a controlled modal for any case where you want to define more than simple text content. I realize that I can send a component to the content prop, but that starts to get pretty messy.
Reproducible test case: https://codesandbox.io/s/20p1n92rjr
Closes Automatically
<Modal content="Close this?" actions={[{key: 'ok', content: 'Ok', positive: true}, {key: 'cancel content: 'Cancel positive: false}]} />

Won't Close Automatically
<Modal>
  <Modal.Content>
    Close this?
  </Modal.Content>
  <Modal.Actions actions={[{key: 'ok', content: 'Ok', positive: true}, {key: 'cancel', content: 'Cancel', positive: false}]}
</Modal>

Possible solution would be to pass a close callback into the data object that is passed to onActionClick.
Version 0.81.1

You cannot use shorthand API and children API simultaneously in such manner. It's the expected behavior.
The shorthand API is powerful and allows complex markup:
<Modal content={{ className: 'foo', content:
complex
, anotherProp: true }} />

I understand that mixing them is not good, but the issue still stands. The actual functional behavior is different between the shorthand and child api. Shorthand closes the modal automatically when clicking a button, child api does not.

Shorthand closes the modal automatically when clicking a button, child api does not.

You're describing the correct behavior.
We cannot handle everything, if you need something really complicated use children API and control your state manually.


S3 bucket prevents easy contribution

I'm working on adding support for dep to the buildpack. I need to pull down two new binaries, dep and tomlq. I've added them to the files.json, but the sync-files command requires access and auth keys to a private Heroku S3 bucket.
We should change the buildpack to accept an environment variable (HEROKU_GO_S3_BUCKET) to configure the bucket and aid contribution.

What I'd like to do is to figure out a way to pass a something like a HEROKU_GO_FILES_LOCATION env var that could also be a files:/// url so that when running the tests the files would be "downloaded" from the local file cache instead. This could then also be used for hosting your own set of binaries in the S3 bucket (or other location) of your choice.

Master now uses a file:// url and files in test/assets, which is populated with the files in [data.json](https://github.com/heroku/heroku-buildpack-go/blob/8fe505ee4193bb30520567c4add9fdcd50a756d1/data.json#L61)'s test.assets array (references the files in files.json) by the [bin/fetch-test-assets](https://github.com/heroku/heroku-buildpack-go/blob/8fe505ee4193bb30520567c4add9fdcd50a756d1/bin/fetch-test-assets)` script.
This should allow mostly local testing. There are still cases though where dep/govendor/glide or others may fetch things from the internet. But the buildpack itself will use the file:/// URL, except when using a devel version of go (the buildpack then fetches go from github and checks out the specified SHA; there is a test for this).

PS: tests are also faster because of this as the test don't download the same version of Go over and over again for each test.


Failure with URL on Apple Watch (real device)

Hi,
first of all, great job!!!
I have encountered a problem that occurs only when I try to parse an RSS feed on Apple Watch device (on simulator it works).
the feed has the following url:
http://www.servizitelevideo.rai.it/televideo/pub/rss101.xml
FeedKit always fail on this try:
data = try Data (contentsOf: sanitizedSchemeUrl)
If run the exact same code on iPhone it works. Of course, even if I download the Data with Alamofire and then I pass it to FeedKit

I ran into a similar problem. I read somewhere that Data(contentsOf: ) under the hood works with something not watch friendly. Sure enough, obtaining my Data from URLSession instead worked for me. This way you can use the FeedParser(data: Data) initializer. It would be nice if the FeedParser(url: URL) initializer had a fallback to URLSession on watchOS though

I remember encountering this issue a long time ago, and it wasn’t limited to just watchOS. We even reported it to Apple, but they couldn’t replicate it, despite the fact that we had three machines where the problem occurred consistently.


Fix store unable to seal replicas in second cluster issue

Fixed a issue that store didn't seal replica in second cluster because
in-mem replica state is already sealed (the state comes from first cluster)
Assigned a dedicated scheduler to participant consistency checker as there
is contention to get thread from regular thread pool(size = 10).

Codecov Report

Merging #1543 into master will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1543      +/-   ##
============================================
+ Coverage     73.62%   73.66%   +0.04%
- Complexity     8124     8128       +4
============================================
  Files           572      572
  Lines         44611    44617       +6
  Branches       5634     5637       +3
============================================
+ Hits          32845    32869      +24
+ Misses        10110    10098      -12
+ Partials       1656     1650       -6

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com/github/ambry/server/AmbryServer.java
82.06% <100.00%> (+0.09%)
19.00 <0.00> (ø)

...rc/main/java/com/github/ambry/store/BlobStore.java
88.23% <100.00%> (-0.14%)
123.00 <0.00> (+2.00)
:arrow_down:

.../main/java/com/github/ambry/router/EncryptJob.java
85.71% <0.00%> (-8.17%)
6.00% <0.00%> (-1.00%)

...n/java/com/github/ambry/clustermap/MockDiskId.java
80.00% <0.00%> (-5.72%)
12.00% <0.00%> (-1.00%)

...va/com/github/ambry/router/TtlUpdateOperation.java
90.66% <0.00%> (-0.67%)
50.00% <0.00%> (-1.00%)

...in/java/com/github/ambry/store/BlobStoreStats.java
69.42% <0.00%> (-0.62%)
101.00% <0.00%> (-2.00%)

...a/com/github/ambry/clustermap/ClusterMapUtils.java
92.57% <0.00%> (-0.50%)
48.00% <0.00%> (ø%)

...in/java/com/github/ambry/network/SocketServer.java
85.24% <0.00%> (-0.41%)
15.00% <0.00%> (ø%)

.../ambry/clustermap/DynamicClusterChangeHandler.java
90.76% <0.00%> (-0.39%)
62.00% <0.00%> (-1.00%)

...va/com/github/ambry/rest/NettyResponseChannel.java
90.83% <0.00%> (-0.32%)
134.00% <0.00%> (+3.00%)
:arrow_down:

... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4ad0b16...7784391. Read the comment docs.


UseDeveloperExceptionPage() doesn't work if exception is thrown after starting to write response

The following user code shows the exception page as expected:
app.UseDeveloperExceptionPage();
app.Run(async (context) =>
{
    throw new Exception();
    await context.Response.WriteAsync("hello");
});

However, the following user code just gets an incomplete response and no exception page:
app.UseDeveloperExceptionPage();
app.Run(async (context) =>
{
    await context.Response.WriteAsync("hello");
    throw new Exception();
});

In ASP.NET 4.6, I believe we buffer responses (at least by default), so developers get a nice error page almost anywhere an exception is thrown.  We may want to do the same in ASP.NET Core, perhaps only when running in debug mode or only when UseDeveloperExceptionPage() has been called.

This is by design. Buffering by default breaks a number of scenarios. Buffering only in Development (or when the middleware was present) would make your app function very differently, such that it would likely break when published.

Hit the issue when an exception is being thrown at var cookieValue = Options.TicketDataFormat.Protect(ticket, GetTlsTokenBinding()); in Microsoft.AspNetCore.Authentication.Cookies.CookieAuthenticationHandler.FinishResponseAsync().

@cnblogs-dudu that's a weird place for an exception. What was it?

@Tratcher use the following TicketDataFormat:
public class FormsAuthTicketDataFormat : ISecureDataFormat<AuthenticationTicket>
{
    private string _authenticationScheme;

    public FormsAuthTicketDataFormat(string authenticationScheme)
    {
        _authenticationScheme = authenticationScheme;
    }

    public string Protect(AuthenticationTicket data)
    {
        throw new NotImplementedException();
    }

    public string Protect(AuthenticationTicket data, string purpose)
    {
        throw new NotImplementedException();
    }

    public AuthenticationTicket Unprotect(string protectedText)
    {
        throw new NotImplementedException();
    }

    public AuthenticationTicket Unprotect(string protectedText, string purpose)
    {
        //Get FormsAuthenticationTicket from asp.net web api
        var formsAuthTicket = GetFormsAuthTicket(protectedText);
        var name = formsAuthTicket.Name;
        DateTime issueDate = formsAuthTicket.IssueDate;
        DateTime expiration = formsAuthTicket.Expiration;

        //Create AuthenticationTicket
        var claimsIdentity = new ClaimsIdentity(new Claim[] { new Claim(ClaimTypes.Name, name) }, "Basic");
        var claimsPrincipal = new System.Security.Claims.ClaimsPrincipal(claimsIdentity);

        var authProperties = new Microsoft.AspNetCore.Http.Authentication.AuthenticationProperties
        {
            IssuedUtc = issueDate,
            ExpiresUtc = expiration
        };
        var ticket = new AuthenticationTicket(claimsPrincipal, authProperties, _authenticationScheme);
        return ticket;
    }
}

What was the exception?

So it's throwing NotImplementedException?

@Tratcher yes, the NotImplementedException was printed in console and the DeveloperExceptionPage was not shown.
System.NotImplementedException: The method or operation is not implemented.
   at FormAuthTicketService.Protect(AuthenticationTicket data, String purpose) in /data/identity/src/CNBlogs.Identity.Application.Services/FormAuthTicketService.cs:line 27
   at Microsoft.AspNetCore.Authentication.Cookies.CookieAuthenticationHandler.<FinishResponseAsync>d__14.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Authentication.AuthenticationHandler`1.<FinishResponseOnce>d__55.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Authentication.AuthenticationHandler`1.<OnStartingCallback>d__54.MoveNext()

But now i can't reproduce the NotImplementedException.


Docu Evaluation

Theoretically applicable to all strategies (currently only implemented for OfflineEnvironment)
Textual description of the graphic
Highlight the difference in applicable metrics depending on the strategy

(Gitlab Issue 123, created by @newLabAspect)

We have a description of all learning strategies in our User Guide.


Use more targeted approach for README conversion check

With this change we use the somewhat dedicated approach for checking for a correct .rst -> .html conversion before packaging. Seen at https://github.com/jazzband/pip-tools/pull/608.

Was sick in bed the last week, so only looking at it now. Looks good! I'll just merge.

Hope you're well again now. Have a great weekend!


Potential response time issue

We are having problems with searches taking a very long time to complete on /Flight-Search and even on the generic /flights. It seems that any request that would return data is very slow.

We understand your concerns. We recently done some database refactoring so it might cause this unexpected issue. We will try to check the commit and fix the problem.

Issue got fixed. Please help us check. Sorry for the inconvenience

I think this got fixed

After the fixes in #6, this seems to be an issue again

What APIs did you use? I can get flights, purchase history and order new ticket.

I'm using the new endpoints at http://<IP_ADDRESS>/api/v1.2.1/

Not sure what happened on your side but all members in our team can access all endpoints within a reasonable response time. I have checked and couldn't found any issue.

I'm not sure either. It could be a google cloud thing.


Result: when referencing 2 different nugets, one depending on the light bc nuget, the other on the full bc nuget, the C# compiler fails to resolve at runtime one or the other assembly with an error in the fusion log.
I lost a full day finding this incompatibility issue with these nugets:

GoogleApis.Auth
MailKit

When using a method on mailkit while having GoogleApis.Auth referenced, the app crashes with a fusion log error on bounty castle.
I fixed the issue by using the GoogleApis.Auth sourcecode instead of the nuget, and updating the dependancy on Bounty Castle from it, so it uses the same bc nuget as MailKit.

I second this, even version 1.7.0 and 1.8.1 are both on the same NuGet package but using 2 different public keys. That makes it impossible to reference libraries using the older assembly.

It is really difficult to tell which packages are official packages and which are maintained by unrelated people. There's actually two unanswered questions on the mailing list (One from 4 months ago and another from 10 months ago) about people being confused about which packages to use, or package differences. None of the BouncyCastle package owners for what I think is the main Nuget package are on the website's contributor list. There might be more queries, but the mailing list search function was broken. There are people reporting here and on the JIRA page.
I spent over half an hour combing through pages, searching issues and hunting to finally make the connection via @peterdettman on the JIRA tracker, which lead me to this mirror, before I could feel vaguely confident that the package was vaguely blessed by core contributors.
Near as I can tell, there's also the Portable.BouncyCastle library being maintained by Oren who seems to make significant efforts to keep BouncyCastle usable for newer xplat deployments.
It would be very much appreciated if you'd put a page up on your site telling people which Nuget packages are approved, and worked with all the various forked package authors to try and clean up the single-function packages. Additionally, it would be useful if you updated some of the older sections of the site, or at least marked them as unmaintained sections and guided people towards the areas that receive attention. It's a shame how difficult it is to find the right package for a cryptography library. People should know which, if any, of the builds are from trusted personnel. Especially given the historical importance of this one, and how often it ends up being a dependency.

FWIW, I'm updating the build pipeline for Portable.BouncyCastle to gear up for an updated release. It's still up-to-date with the bc-git master.
It will target the following platforms:

net45
netstandard1.0
netstandard1.3

This should make it work everywhere. It has a different public key than the main BC dist because the main snk was never checked in...despite the SNK not being a security mechanism.
That said, I'm also going to be signing my binaries with an Authenticode cert containing my name, staring with this coming update.

For anyone interested, Portable.BouncyCastle <IP_ADDRESS> is now on NuGet. No real changes; it's a fresh port of master using the VS 2017 cross-compile project types.
The binaries are also Authenticode signed with my name on them.

I second this too. I guess I am going to have to have a local build of this stuff as the nuget packages are either behind or unmaintained.

@FrankSzendzielarz My packages are up to date. Please use the Portable.BouncyCastle for now. It's a drop-in replacement if you need more than just .NET support.

@onovotny  Nazdar :-)  Is that correct? Last I looked on NuGet the update was 6 months ago and I saw some recent updates I wanted in the main build (particularly ConcatenationKdfGenerator). I might be misreading/misunderstanding which version of what is where so if you could let me know how up to date your packages are and from where I should get them?

@FrankSzendzielarz In general, I was following BC's releases, though I can easily push out an update if there's relevant updates post the <IP_ADDRESS> version.

@onovotny One second, I might be misreading, please can you check this and see if you have it in your lib? https://github.com/bcgit/bc-csharp/commit/86a7490aaf8f0d9b35cc2fb094efae8409c095a6#diff-60cdc397256fad26647be3d61b6c1e23
Basically I want the stuff to support the Ethereum NIST SP 56 (section 5.8.1) stuff so I don't have to port their god awful GoLang garbage.

Looking at the commit graph, it's not there yet, but I can release an update in the next couple hours that has it. It'll be <IP_ADDRESS>.

@onovotny Wow. That would be fantastic! There's a cold (original) Budweiser waiting for you when you need it. Let me know!

@FrankSzendzielarz Just uploaded, should be visible in the next 10-15 min https://www.nuget.org/packages/Portable.BouncyCastle/<IP_ADDRESS>

@onovotny Top Class. You saved me a ton of time. Many thanks for that. Like I said, if you are my way stop by and the lagers are on me.


Printing QAOA ansatz circuit with optimized parameters

Hi XACC developers,
I'm currently working on a project where I'm trying to print the ansatz circuit for QAOA (at the moment, in xasm format) and get a generic circuit with gamma0, beta0, etc.... I have a set of optimized theta values from QAOA that I'd like to replace with the generic gamma and beta values.
However, the issue is that for a circuit with say 12 optimized theta values, I get 6 generic values in the order of gamma0, beta0, gamma1, beta1, gamma2, beta2 all having two occurrences of each. How do I know, in which order I'm supposed to replace these generic values with the optimized theta values.
Thanks for your patience!

Hi,
The variable/parameter order should be the same as the order that these variables first appear in the circuit printout.
For QAOA, it should be as you saw {gamma0, beta0, gamma1, beta1, gamma2, beta2, etc.}.
Only unique variables are added to the list (repeated variables don't count).
Hope that it helps.

@tnguyen-ornl Thank you for the quick response! If I'm understanding this right, suppose my optimized theta are {1.0, 2.0, 3.0, 4.0} and I get the corresponding generic values as {gamma0, beta0} (each entry occuring twice). So I just replace the first entry of gamma0 with 1.0, second entry of gamma0 with 2.0, first entry of beta0 with 3.0 and so on?

I don't quite understand the 'optimized theta' array that you're referring to.
If the set of parameters are {gamma0, beta0} then there should only be two double values, e.g. all occurrences of gamma0 in the circuit should be replaced with a single value.
In this example, you should only need an array of size 2, e.g. {1.0, 2.0}, all entries of gamma0 -> 1.0 and all entries of beta0 -> 2.0.
In case you want to parameterize each angle independently, the variable names should be different.
If you have a concrete example in XACC, feel free to share it here as well.

So, using the qaoa implementation in xacc, I'm printing the generic ansatz circuit in xasm format and also the optimized theta values as below:
    Optimized theta: {2.03391, 2.35685, 1.58292, 2.35503}
Sample circuit with replaced vals:
__qpu__ void qaoa(qbit q) {
H(q[0]);
H(q[1]);
Rz(q[0], -1.000000*gamma0);
CX(q[0],q[1]);
Rz(q[1], 2.000000*gamma0);
CX(q[0],q[1]);
H(q[0]);
Rz(q[0], 2.000000*beta0);
H(q[0]);
H(q[1]);
Rz(q[1], 2.000000*beta0);
H(q[1]);
Measure(q[0]);
Measure(q[1]);
}

The question is, which two theta values out of the four should I replace with {gamma0, beta0}.

@tnguyen-ornl btw the above circuit was created using qaoa_ansatz = xacc::createComposite({...}); and I understand that there is a qaoa_ansatz.eval([...]) function which takes in the optimized theta values as arguments and prints out the circuit.
Possibly with that I could interpret the ordering of the angles used in the circuit. However, it's not clear to me about the syntax of the function (couldn't find it while reading the docs). Could you help me with it? Thanks.

Hi,
In XACC, in order to get the parameterized QAOA ansatz circuit, you can use createComposite('qaoa') and expand it with appropriate parameters.
Looking at your example circuit, I think the Hamiltonian is equivalent to -0.5 Z0 + Z0Z1. So this is an example of using QAOA in the 'Standard' and 'Extend' mode:

Standard:

qpu = xacc.getAccelerator('qpp')
ham = xacc.getObservable('pauli', '-0.5 Z0 + Z0Z1')
buffer = xacc.qalloc(2)
nbTotalParams = 2
# The optimizer: nlopt
opt = xacc.getOptimizer('nlopt', { 'initial-parameters': np.random.rand(nbTotalParams)} )

# Create the QAOA algorithm
qaoa = xacc.getAlgorithm('QAOA', {
                        'accelerator': qpu,
                        'observable': ham,
                        'optimizer': opt,
                        'steps': 1,
                        'parameter-scheme': 'Standard'})
# Run
result = qaoa.execute(buffer)
print('Min value = ', buffer.getInformation('opt-val'))
print('Opt-params = ', buffer.getInformation('opt-params'))

# Get the circuit
qaoa_ansatz_std = xacc.createComposite('qaoa')
qaoa_ansatz_std.expand({'nbQubits': 2, 'nbSteps': 1, 'cost-ham': ham, 'parameter-scheme':'Standard'})
print('Standard parameterized QAOA circuit:')
print(qaoa_ansatz_std)

Extend:

qpu = xacc.getAccelerator('qpp')
ham = xacc.getObservable('pauli', '-0.5 Z0 + Z0Z1')
buffer = xacc.qalloc(2)
nbTotalParams = 4
# The optimizer: nlopt
opt = xacc.getOptimizer('nlopt', { 'initial-parameters': np.random.rand(nbTotalParams)} )

# Create the QAOA algorithm
qaoa = xacc.getAlgorithm('QAOA', {
                        'accelerator': qpu,
                        'observable': ham,
                        'optimizer': opt,
                        'steps': 1,
                        'parameter-scheme': 'Extend'})
# Run
result = qaoa.execute(buffer)
print('Min value = ', buffer.getInformation('opt-val'))
print('Opt-params = ', buffer.getInformation('opt-params'))

# Get the circuit
qaoa_ansatz_ext = xacc.createComposite('qaoa')
qaoa_ansatz_ext.expand({'nbQubits': 2, 'nbSteps': 1, 'cost-ham': ham, 'parameter-scheme':'Extend'})
print('Extend parameterized QAOA circuit:')
print(qaoa_ansatz_ext)

The number of parameters returns by the algorithm must match that of the circuit.
I think you may have mixed different modes of QAOA which have a different number of variational parameters.

@tnguyen-ornl Thank you for the example codes! I ran the 'Extend parameter-scheme code and got an output like this:
Min value =  -5.999999473288454
Opt-params =  [2.7283658364289045, 0.7857359705958752, 1.5693758224110206, 0.7839685760905409]
Standard parameterized QAOA circuit:
H q0
H q1
Rz(-1.000000*gamma0) q0
CNOT q0,q1
Rz(2.000000*gamma1) q1
CNOT q0,q1
H q0
Rz(2.000000*beta0) q0
H q0
H q1
Rz(2.000000*beta1) q1
H q1

So, in the actual circuit, is it safe to say that I replace 2.728 with gamma0, 0.785 with gamma1, 1.569 with beta0 and 0.783 with beta1?


Add an enumerator for Dictionary values

I added a method on a ClrObject that allows to enumerate the child values when the object is a generic dictionary. This can be very helpful when debugging a crash dump where each object has dictionaries and we want to get its content.

Thanks for the contribution!


Review translation from Resi

Major changes, especially "berkas" which was changed into "file".
Reasons:

Most people don't know what "berkas" means in the digital sense (although I use "berkas" in academic stuff)
Google Chrome uses "file" too

For maximum consistensy, please re-review again

Gonna delete/refork my repo cause git clone took a long time


Old PowerToys icon

There's a new PowerToys icon. This page still shows the old one.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7779352b-7b4e-dad8-7c1b-b9aba2c5e561
Version Independent ID: a5b81b80-87a1-b6e2-8936-baf6c1a0b9c5
Content: Set up your development environment on Windows 10
Content Source: hub/dev-environment/overview.md
Product: dev-environment
Technology: windows-nodejs
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

Thanks @niels9001 - found the new icon and fixed it. 👍
https://github.com/MicrosoftDocs/windows-uwp-pr/commit/1e06168ada5ce6013b1d07c428548f084464a286


Remove pry reference and bump version

This causes production environments (e.g., set up w/ bundle install --without development test) to fail with:
LoadError: cannot load such file -- pry

:+1:  Thanks

No prob!


Build arrow for Python and Java simultaneously

What do these changes do?
Now, Ray's build dir has been unified for Python and Java to ray/build, see https://github.com/ray-project/ray/pull/2171. We can execute command make in the same dir to build Ray.
However, Ray's build depends on the third party lib: Arrow, and now when we build Ray for Python, we can only generate Python client, then if we want to build Ray for Java, Arrow will not be built again and Ray's build will fail. (BTW, if we build Ray for Java first, we will also generate arrow's Python client)
So, to solve the problem, we can generate both Python and Java client simultaneously when building Arrow. I know If user only wants Python client and we generate Java client simultaneously, that's useless, but Arrow as a third party lib, it only need to be built once. So generating Arrow's Python and Java client simultaneously is a good idea.

@raulchen @imzhenyu @eric-jj Please help to review.

Does building arrow for Java require JDK?
If so, I'd suggest not to do it by default. Because some users may only want to build Ray for python and doesn't have Java installed.

@raulchen Yes, you're right. I'll figure it out later.


refactor(Cyclic): redefine, move definition

Redefine IsCyclic without Subgroup dependency, move the definition to Algebra/Group/Defs.

I'm going to prove that certain groups aren't cyclic, and the relevant file doesn't import SpecificGroups/Cyclic.
I added import Mathlib.Logic.Function.Defs to Algebra.Group.Defs to have Surjective.

@semorrison what was the rationale for assert_not_exists Function.Injective?

It was added in #9606 while removing import Mathlib.Logic.Function.Basic which is a bit heavier than Mathlib.Logic.Function.Defs.

We've worked hard to get the dependencies for Algebra.Group.Defs very low. While I agree Logic.Function.Defs is not heavy, it's relative weight for such a low-level file is still relatively high. So, personally, I'd prefer to have another solution here.
Maybe expand the definition of Function.Surjective locally in this file, and add the actual Function.Surjective lemma in a later file?

@j-loreaux
What if I move everything but the definitions from Logic/Function/Defs, so that it becomes a very light dependency?

Is there harm in just making a new file?

BTW, the only new Mathlib imports that come with Function.Defs are Tactic.AdaptationNote and Attr.Register. Probably, the script reports Batteries imports too.

Is there harm in just making a new file?

What do you suggest to move to a new file?

IsCyclic/IsAddCyclic to Algebra.Group.IsCyclic?

I would expect that Algebra/Group/IsCyclic is a file with basic properties of IsCyclic, not a file with the definition.

Another option is to move everything but definitions from Logic/Function/Defs to a new file (or to Function/Basic if every file that needs them imports Basic).

@j-loreaux What are the benefits of not importing Function/Defs in Algebra/Group/Defs?

Ah, I didn't see your earlier comment. I think I'm okay with the new imports pulled in by Function.Defs, but I would like other input.
My goal here is just to be very stingy with respect to imports of low-level files so that we don't end up slowly migrating back to the old paradigm.

Whoever looks at this, tha main question is should we import Logic.Function.Defs into Algebra.Group.Defs. Ultimately, I'm okay with it, but @mattrobball suggested moving to another file, which I also think is a good option. Please weigh in or merge.
maintainer merge

bors merge


Dashboard & Web Installer Dark Theme

Dark Mode / Theme for both Dashbord & Web Installer as they both use the same base CSS

I know there is an older PR trying to do the same, but as the web installer and dashboard share css and some components it had to be done for both, here are some screenshots:

@grahambrown11 also see this suggestion: https://github.com/esphome/feature-requests/issues/1003 (maybe for a later PR).

@grahambrown11 also see this suggestion: esphome/feature-requests#1003 (maybe for a later PR).

Currently as the dashboard is not aware of HA that'll be a complex change to get the users HA theme...
I'm hoping this'll work for most ppl who prefer dark mode they'll have it set in their OS...
Maybe allowing the user to chose a light / dark theme like some websites do and default to OS prefered mode would offer more flexibility... I'll see what I can do

As I said, don't stress yourself on this, I just linked it to see some of the (eventually upcoming) user demands...

Added a user toggle for the theme incase the user wants to specify over the browsers refers-color-scheme

Found a bug, this button is purple now:

Ah nevermind, Jesse found out the CSS was cached for ESPHome Web. All good 👍

The dropdown for picking wifi network is not styled. However, I think it's because it's actually the Improv Wi-Fi SDK that's embedded in the ESPHome Web that shows this dialog, not a big deal.

Feedback on discord is to not have this button

Where? Didn't see that...
I would have voted to have it.

Thank you @balloob for your review and helping getting this over the line 🎉
I and the DrZzs community will like not having our retinas burnt when the doc goes to ESPHome now on his live streams 😺

Hi @grahambrown11
I can't seem to create an issue in this repo, but I think you forgot to check the login screen, all the text and icons are dark on a dark background, see next picture:

I think you forgot to check the login screen

Oh sorry, I will look into that now... I only use this in HomeAssistant, so have not seen that login mode...

@golles hoopefully my new PR will be accepted - this should sort it https://github.com/esphome/dashboard/pull/500/files#r1240875893

Can we move the search into the top toolbar, just icon only, and have the search overlay the toolbar when active. That way it doesn't take up a lot of space when not used

Search icon could still be shown next to the overflow menu icon on mobile.

~Added a user toggle for the theme incase the user wants to specify over the browsers prefers-color-scheme~
Feedback on discord is to not have this button

@grahambrown11 What was the feedback, and for what reason?
As the web adopts standards for adapting to users systems preference, while still maintaining a mean to choose, I find it really strange (and not in a good way) to specifically choose not to follow standards.

As the web adopts standards for adapting to users systems preference, while still maintaining a mean to choose, I find it really strange (and not in a good way) to specifically choose not to follow standards.

@cenobitedk The decision was not mine it was feedback from the maintainers of HomeAssistant & ESPHome. As for the standards, I'm not sure what you're talking of - there are no standards when it comes dark theme switches or the web in general, some websites chose to have a visible switcher some put it in user preferences but the majority don't even support it at all...

@grahambrown11

The decision was not mine it was feedback from the maintainers of HomeAssistant & ESPHome.

Yeah I know, I was curious about their reasoning.

As for the standards, I'm not sure what you're talking of - there are no standards when it comes dark theme switches or the web in general, some websites chose to have a visible switcher some put it in user preferences but the majority don't even support it at all...

You're right that implementation various a lot, but there certainly is support for it on many many sites, and that is setting a de facto standard for other sites to follow.
There's no standard for login flow either, yet a billion sites follow the samme approach based on best practices in regards to security and UX.
Since we have wide spread browser support (https://developer.mozilla.org/en-US/docs/Web/CSS/@media/prefers-color-scheme) I doesn't make sense to go against the way other sites have implemented client support.

Yeah I know, I was curious about their reasoning.

There was no reasoning (no democratic opinioning either) a question was raised, an a HA dev responded negative, in a short while that's it. No other users were listened. As soon as that single response came, this was done.

That's just sad that a single dev get's to decide what's good user experience. :-( and it's in opposition to how it's done in HA:

I think this deserves a revote. I think the good work @grahambrown11 has done deserves to be in the dashboard so that people can choose for themselves rather than what a single dev thinks.