replace @ignore with @hidden

@ignore is a JSDoc construct which we no longer use. This PR replaces it with TypeDoc's @hidden which exhibits the same behavior.

Looks good to me. Please do the following

resolve merge conflicts
wait for someone from Cosmos and Storage teams to acknowledge their parts of the code

Just curious, what is the benefit of using @hidden instead of @ignore? It looks like @ignore is an alias for @hidden, and neither are part of TSDoc, are they?

Just curious, what is the benefit of using @hidden instead of @ignore? It looks like @ignore is an alias for @hidden, and neither are part of TSDoc, are they?

@chradek this is for us to be consistent in using one over the other until we can rely on using TSDoc's@internal only (the docs team needs to upgrade their TypeDoc version for us to be able to do that). I went with @hidden because it is not part of JSDoc.

@chradek this is for us to be consistent in using one over the other until we can rely on using TSDoc's@internal only (the docs team needs to upgrade their TypeDoc version for us to be able to do that). I went with @hidden because it is not part of JSDoc.

I understand that the docs team using an older version of typedoc is forcing us to use either @ignore or @hidden since the older version does not respect @internal.

I went with @hidden because it is not part of JSDoc.

This part confused me a bit though... How does @hidden not being part of JSDoc makes us chose it over @ignore?

I understand that the docs team using an older version of typedoc is forcing us to use either @ignore or @hidden since the older version does not respect @internal.

I went with @hidden because it is not part of JSDoc.

This part confused me a bit though... How does @hidden not being part of JSDoc makes us chose it over @ignore?

This part confused me a bit though... How does @hidden not being part of JSDoc makes us chose it over @ignore`?

@ramya-rao-a Let me expand a bit. Ultimately it does not matter :) eventually we should have neither. For now, we should choose just one of them because

having two tags that do the same thing in our code is confusing (Initially, I was not sure which tag to use when).
otherwise we will need to whitelist two tags unnecessarily with TSDoc.

I chose @hidden because it is not part of JSDoc so we now no longer use any of JSDoc's tags.

This part confused me a bit though... How does @hidden not being part of JSDoc makes us chose it over @ignore`?

@ramya-rao-a Let me expand a bit. Ultimately it does not matter :) eventually we should have neither. For now, we should choose just one of them because

having two tags that do the same thing in our code is confusing (Initially, I was not sure which tag to use when).
otherwise we will need to whitelist two tags unnecessarily with TSDoc.

I chose @hidden because it is not part of JSDoc so we now no longer use any of JSDoc's tags.

I chose @hidden because it is not part of JSDoc so we now no longer use any of JSDoc's tags that are not in TSDoc.

I guess at the end of the day it doesn't really matter, except we need to educate the rest of the team that they should be using @hidden instead of @ignore now. I actually thought having the JSDoc @ignore would be preferable until we can get rid of it altogether since anyone that has worked on a JavaScript project using JSDoc would recognize it. It also seems that @ignore is used by many packages already (including cosmos) but @hidden is only used in cosmos, so it would have been easier to just use ignore instead.
Anyway, I wouldn't block this since you did the work and it won't change how documentation is displayed, I just don't understand the reasoning behind it.

I chose @hidden because it is not part of JSDoc so we now no longer use any of JSDoc's tags that are not in TSDoc.

I guess at the end of the day it doesn't really matter, except we need to educate the rest of the team that they should be using @hidden instead of @ignore now. I actually thought having the JSDoc @ignore would be preferable until we can get rid of it altogether since anyone that has worked on a JavaScript project using JSDoc would recognize it. It also seems that @ignore is used by many packages already (including cosmos) but @hidden is only used in cosmos, so it would have been easier to just use ignore instead.
Anyway, I wouldn't block this since you did the work and it won't change how documentation is displayed, I just don't understand the reasoning behind it.


Texture pack not loading

Describe the bug
I Have Adaptive Chestplate, leggings and boots but only the texture of the leggings are showing. I think its because my Chestplate and boots are fragged, but my leggings are not.
I'm using the Hypixel Skyblock pack v13.
Profile/Location
https://sky.shiiyu.moe/stats/Artinator11/Papaya
Expected behaviour
I Expected for the texture of my armour to load correctly
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: 10

Do the textures appear in-game?

Yes the textures do appear in game

Is this still broken or has it bean fixed? Artinator11/Papaya seams not to have this armor anymore can someone link to a profile that does

Pretty sure its still happening, once I had a chance to test it the profile didn't have the armor already

It's still happening
The leggings and bonzo staff for this profile aren't loading
https://sky.shiiyu.moe/stats/lv_oz2/Mango

Are any packs other than Hypixel Skyblock pack v13 effected?

Well only the Hypixel Skyblock pack v13 has a texture for Adaptive armour
But the bonzo staff on the profile loads the texture when using Furfsky reborn

can confirm still a problem

tested with https://sky.shiiyu.moe/stats/lv_oz2/Mango#Inventory


Create tidy Python library for loading data :)

With automated unit tests :)
User can easily say "I want this many historical timesteps; and this many forecast timesteps; and including these satellite channels, and these NWP params, and compute optical flow predictions based on the most recent pair of satellite images".

Started work on new library: https://github.com/openclimatefix/nowcasting_dataloader


feat: Improve logging & error handling

closes #112

For some reason, this is causing occasional crashes in Neovim. The error messages are also unclear. If anyone has insights into why this might be occurring, I would greatly appreciate your thoughts.

I was not able to find a workaround for the issue by far. It is most likely caused by interacting with Lua/Neovim from multiple threads, which explains the inconsistency of crashes.

One possible solution I can think of, is to connect to the pipe in non-blocking mode, which would remove the need for creating a separate thread, so we don't block the main thread. Rust doesn’t appear to offer a built-in way to achieve this on Windows without directly interacting with the C api, but this isn’t critical.


Streamline the backend process of trading commands

Trading items for items and coins for items should be integrated into a single command, with arguments in this command to indicate whether you are sending coins or items.
The item parameter is optional and when passed in assumes you are trading an item, else it assumes you want to send coins. The same applies for the what the invoker wants to receive from the other recipient in that sense.

See #136.


Price/odds don't update after trade succeeds after 5+ min.

trade succeeded: Object { nonce: "0x3", blockHash: "0xcc8479609206b6827b9497768026c7c6b…", blockNumber: "0x2a537", transactionIndex: "0x0", from: "0x2025b185239ba6c972f199a3b5eeb5bfc…", to: "0xcb41186d3fa3eae83aa828eb776a913ff…", value: "0x0", gas: "0x2fd618", gasPrice: "0xba43b7400", input: "0x7d9e76410000000000000000000000000…", 2 more… }```
Price should update instantly upon this log, but it doesn't

I put a 3 second delay on it (to give the cache a moment to update) -- does it not update at all, or just not update instantly?

N/m missed the 5+ minutes part.  Ha.  Troubleshooting now.

Fixed in 46b4fe842f925c37f54f060a85e0d33bba23afb4.


[FLINK-31917][table-planner] Fix the idempotence lost in JsonSerDe round trip for AggregateCall and RexNode

What is the purpose of the change
This PR aims to fix the FLINK-31917. The details can be found in the ticket's description.
Brief change log

Fix the issue that AggregateCallJsonDeserializer converts null to "null" for attribute name.

Fix the issue that RexNodeJsonSerdeTest#testTemporarySystemFunction registered the temporary catalog function by mistake.

Correct the behavior for RexNodeJsonSerializer#serializeLiteral.
Before
INT, BIGINT, and DOUBLE are serialized as STRING because the serializeLiteralValue receives literal type asDECIMAL. This is due to RexBuilder#makeExactLiteral using SqlTypeName.DECIMAL as the typeName (which is "An indication of the broad type of this literal", according to Calcite)
E.g.
 RexLiteral spec = new RexLiteral(3.14, TYPE_FACTORY.createSqlType(SqlTypeName.DOUBLE), SqlTypeName.DECIMAL);

is serialized to
{
    "kind" : "LITERAL",
    "value" : "3.14",
    "type" : "DOUBLE NOT NULL"
}

However, the deserialization will recognize the DOUBLE as the literal type and convert the JSON to the following object
 RexLiteral deserialized = new RexLiteral(3.14, TYPE_FACTORY.createSqlType(SqlTypeName.DOUBLE),
// the typeName has changed,  the reason why the current assertion works is that Literal's equals method does not compare typeName
SqlTypeName.DOUBLE);

If the deserialized object is serialized again, the result differs.
{
    "kind" : "LITERAL",
    "value" : 3.14,
    "type" : "DOUBLE NOT NULL"
}

After
serializeLiteralValue will use literal's type except for SARG. Meanwhile, for the FLOAT type, in case the precision loss, serialize it as string type.

Verifying this change
This change added tests and can be verified as follows:

Add a second round trip for JsonSerDeTestUtil#testJsonRoundTrip.
Add AggregateCallSerdeTest to verify the "null" string issue is fixed.
Add a SerDe round trip for TableTestBase#verifyJsonPlan and adapt the test output file.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@godfreyhe, would you mind sparing some time to review the fix?

@flinkbot  run azure

@flinkbot run azure

@flinkbot run azure

Just rebase master and squash commits to reinvoke CI...


Add trusted_task package

This new package allows verifying that a certain Task is trusted. It supports both the bundles and the git resolvers.
Eventually, this will supersede the attestation_task_bundle package.
Ref: EC-186

Opened as a draft because I'm still working out the details, but wanted to gather some initial feedback.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (d2fa0d7) 100.00% compared to head (a88c012) 100.00%.
Report is 10 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##              main      #885    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           89        93     +4
  Lines         4363      4574   +211
==========================================
+ Hits          4363      4574   +211

Files
Coverage Δ

policy/lib/refs.rego
100.00% <100.00%> (ø)

policy/lib/refs_test.rego
100.00% <100.00%> (ø)

policy/lib/tekton/trusted.rego
100.00% <100.00%> (ø)

policy/lib/tekton/trusted_test.rego
100.00% <100.00%> (ø)

policy/release/trusted_task.rego
100.00% <100.00%> (ø)

policy/release/trusted_task_test.rego
100.00% <100.00%> (ø)

Includes commit from https://github.com/enterprise-contract/ec-policies/pull/891.


Thai translation

I've fixed some Thai translation. I'm not sure why GitHub says it can't be merged.

Appreciate the PR! I'll get back to you on the merge, not sure why Appveyor is failing and Travis isn't running at all.
For future, I have a translation platform setup to handle this instead of PRs. :) I was going to add it to the README on the 1.0 release, but as people are paying attention I'll do it sooner. :)
https://dustinblackman.oneskyapp.com/collaboration/project/95440

@secretdataz I think the issue was you're trying to merge in to dev. Merge in to Master instead.

Done! I've put them on Onesky :)


login in native android module but can't get session in react module

Integrating with existing android app, but I can't get session in react module

Hey and thanks for reporting this!
There's an awesome place to ask question like this one: StackOverflow. It's the best system for Q&A. Many people from the community hang out there and will be able to see your question, you can vote on answers and mark question as answered etc. This lets us keep a list of bug reports and feature requests on github and especially Product Pains (again, with voting which is really nice).
I'm posting this here because github issues haven't been working very well for us and because StackOverflow is so much better. Thanks for reading! :)

@satya164 thanks


Performance Compared to RLottie

How is this library compared to RLottie in terms of performance? My main purpose is for usage in web as WASM, the WASM build for RLottie was around 300KB-400KB, but this library was around 600KB, so not superior in bundle size, but is it superior performance wise? it would be nice if someone provides benchmarks.

I guess that it also depends on the options you passed to thovg when building. which are they ?

@erfanmola Hello, the main reason for the size difference is that ThorVG supports WebP and Expressions, whereas rlottie does not. For a fair measurement, please see the size comparison if throvg disables these two modules:
rlottie: 500kb, thorvg: 300kb

With even this smaller size, I can assure that throvg lottie feature coverage / stability is much bigger than rlottie.
+ Please read this post on Canva team's benchmarking, where ThorVG's performance is superior to rlottie's.
https://lottiefiles.com/blog/working-with-lottie-animations/canva-enhances-ios-rendering-faster-and-efficient-with-thorvg
As the maintainer of both projects, I strongly recommend ThorVG. There are no benefits to using rlottie now.

@hermet also mention that by default thorvg is compiled with stripping and -s, so no optimization except for size. What about with -O2 ?

@hermet Thanks for the detailed answer, Then my decision is made up and i'll go with Thorvg, thanks for this amazing piece of software.
One thing I have noticed is the default wasm builder script generates a JS file that contains a base64 encoded version of .wasm binary in JS, which is considerably 2x in size, is there any possibility to load the wasm manually and not by inline base64?

@erfanmola please check this file, remove this option -sSINGLE_FILE=1
just so you know, for the compact lottie player, you can tweak build option as well in the build script
meson -Db_lto=true -Ddefault_library=static -Dstatic=true -Dloaders="lottie, png, jpg, webp" -Dthreads=false -Dbindings="wasm_beta" --cross-file /tmp/.wasm_cross.txt build_wasm

just for your information, on my computer, with software engine, with Lottie example:
debugoptimized (-Os) no simd : [  327]: elapsed time = 23ms (25ms)
debugoptimized (-Os) simd    : [  327]: elapsed time = 28ms (25ms)
release (-O3) no simd        : [  327]: elapsed time = 25ms (23ms)
release (-O3) simd           : [  327]: elapsed time = 20ms (20ms)

@vtorri Thanks. for now, the web bundle size might be a big concern, we need to compromise the option soon or later. (Probably, we can apply it to the default)


Pending requests on app.awork.io

@adguard-bot commented on Fri Feb 19 2021
Issue URL (Incorrect Blocking)
https://app.awork.io
Comment

I am using AdGuard on my Windows PC (AdGuard for Windows) and I recently found out that it has issues with one specific website (app.awork.io). This is a paid project management tool (with a trial). The issue is that some requests simply do not go to their servers when navigating the website. I already had a session with their support staff and we found out that as long is AdGuard is active, some requests will show in Chrome Developer tab as "400 Bad Request" but never reach their servers. With AdGuard disabled it worked as expected with no issues.
I already went ahead and whitelisted the hole site but and there is no block in the AdGuard filter protocol but still timeouts and 400 Bad Request will appear. Therefore I think its an issue with the internal proxy or something network driver related. Please tell me what informations you need from me to analyze the issue.
Here are some specifics about my setup:
Windows 10 Pro 2004 - 19041.804
AdGuard Version: 7.5.3 (build 3430, CL 1.7.150)
Browser: Chrome 88.0.4324.150
AV: Kaspersky Internet Security <IP_ADDRESS>0 (b)
I also already created a ticket (3212396)  with the support staff and tried the following:
AdGuard > Settings > General > Advanced settings and enable "Use driver in redirect mode", then click on "Save".
AdGuard > Settings > General > Advanced settings and disable options called "Intercept TCP connection requests" and "Filter local connections"
The filter log in AdGuard shows the HTTP response code 0 and and empty response header for this request.
If you need to my help, please contact me via the email assigned to the ticket number as I do not have an Github account.

Screenshots

Screenshot 1

Screenshot 2

System configuration

Information
value

AdGuard product:
AdGuard for Windows v7.5.3 (build 3430, CL 1.7.150)

Browser:
Chrome

AdGuard driver:
WFP

DNS filtering:
disabled

Filters:
Ad Blocking: AdGuard BasePrivacy: AdGuard Tracking ProtectionSocial Widgets: AdGuard Social MediaAnnoyances: AdGuard Annoyances, I don't care about cookiesOther: Filter unblocking search ads and self-promotionLanguage-specific: AdGuard German

Other extensions used:
Kaspersky Internet Security <IP_ADDRESS>0 (b)

@Alex-302 commented on Sat Feb 20 2021
On my end that requests made without errors.

@Alex-302 commented on Sat Feb 20 2021
Please write to<EMAIL_ADDRESS>

Logs available in helpdesk ticket 2488614.

Received additional HARs in helpdesk ticket Z501524.

Can't be reproduced anymore. Please reopen if it happens.


using post method instead of get to enable a runner

Hi,
This fix changes the enableRunner() method of RunnersApi to use the http POST method instead of GET to perform the operation.
Best regards,
Fouad

@fouadh
Thank you for your contribution.  This will be available in the next release which will happen in the next couple of days.


fix example by updating to 2.0.3

Fixes fatal runtime error Uncaught TypeError: middleware.listenForReplays is not a function.

That shouldn't matter, as the example aliases to whatever you have checked out above it in git. Check the webpack config.

I had an issue when using release 2.0.3 so i wanted to compare it to the
example at the 2.0.3 tag in the repo. This is where i encountered the issue.
If its relevant, i only npm installed the example not the main
redux-simple-router package.
Op 18 jan. 2016 15:06 schreef "Tim Dorr"<EMAIL_ADDRESS>
That shouldn't matter, as the example aliases to whatever you have checked
out above it in git. Check the webpack config.
—
Reply to this email directly or view it on GitHub
https://github.com/rackt/redux-simple-router/pull/213#issuecomment-172534471
.

Can you make it a caret range then? That should future-proof us against any updates until a major version, in which case there will probably be an updated example anyways.

Good idea, done.
Ideally the example gets tested prior to a release. I see the resolve code would devs to quickly use their latest version. Nice!

Ideally we could test our examples, just like Redux does. It's part of the test suite over there. I'd be really jazzed for someone to add those at some point :+1:


I try to add Google play game but nothing happens

I try to add Google play game but nothing happens, I followed instruction but it did straight up nothing, no game was added

Can you post the link you tried to add?


Other tags?

I see that both get_tag and tags are not public, is there any reason for this? I'd love to be able to query what other tags may be in the header.

This is the only discussion I could find regarding the visibility of tags
@phil-opp any updates regarding this?

I left that comment over 4 years ago :D. So I don't really remember why I was against new public items back then. I guess that I just wanted to be careful about safety, since e.g. the tag iterator type performs unsafe operations internally.
On a quick look, I think exporting the Tag type and get_tag function should be fine. The TagIter type in its current form seems unsafe so we shouldn't expose it, but I think we can export the tags() function after changing its return type to impl Iterator<Item = &Tag>.

Thanks @phil-opp! It's hard to believe it's been so long since the rust OS projects started
@drazisil feel free to submit a PR. If needed, I can probably work on it this weekend

Yeah, for me too!

@robert-w-gries I will try to find time to work on this. Someone else can tackle it first if they have time, I won't be offended :)

Probably we can close this, because we also closed #75
What do u think @IsaacWoods ?

Well, I completely forgot about it  I have no problem with closing it.


remove gzstream

In case it isn't clear, whats left after this removal are just the factory classes I added to automagically select between a normal ifstream/ofstream and a zipped variant. I am leaving that in with the expectation of resurrecting that functionality, and more, after this removal.


#53 ADD docs for psm_match.py

Sorry if I missed it, but are intensities normalized should this be done separately? LGTM!

Yes, normalization should be done separately, for example, in peptdeep.


Added the action property for posted task events

Codecov Report

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

@@             Coverage Diff              @@
##             master       #5      +/-   ##
============================================
+ Coverage     98.97%   98.98%   +<.01%
- Complexity       64       65       +1
============================================
  Files             7        7
  Lines           293      295       +2
  Branches          3        4       +1
============================================
+ Hits            290      292       +2
  Misses            1        1
  Partials          2        2

Impacted Files
Coverage Δ
Complexity Δ

...va/com/vliolios/eventposter/TaskEventListener.java
94.73% <100%> (+0.61%)
8 <3> (+1)
:white_check_mark:

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 488a099...4b8fe3e. Read the comment docs.


Add dag library to config.lib

Also replace all imports of dag.nix by the entry in config.lib.

Force-pushed updated code.

Apparently it also works to just use actual modules, for some reason I don't get recursion when I do this: https://github.com/Infinisil/home-manager/commit/35c422b0cd2154599b0bcda8f9b1e4fe90bf89fc
This also enables having the file-types.nix in config.lib and seems generally nicer

Hmm, when I switch
with config.lib;
with lib;

to
with lib;
with config.lib;

e.g. in systemd.nix I get
error: infinite recursion encountered, at /home/rycee/devel/nixpkgs-17.09/lib/modules.nix:62:71

with or without your changes. I tried to track this down but unfortunately couldn't find the cause.

Replaced all use of with config.lib to instead explicitly bind dag = config.lib.dag in the let. This seems to resolve all recursion problems.


Add metrics chart display recommendations

Adds two options for displaying charts on the service metrics page and some other small changes.
Adds .gitignore file
Closes #12

@jotak fyi, please review

@cshinn , @abonas I like the idea of grouping in/out charts by pairs in cards. Also, I wonder when they're grouped horizontally like this, we could maybe take the legend back (every chart wouldn't have the same height, but at least they would be aligned). 'Cause I fear removing the bottom legend has too many drawbacks: the bottom legend has the advantage to be interactive, makes it more immediate to understand the chart, also is probably more printing/screenshot-friendly.
It's great to be able to filter on histogram charts: it's a step in the direction of enhancing this screen, but we could also do more: see https://issues.jboss.org/browse/KIALI-936 (created a while ago but still valid). It would be nice to be able to filter out from label values (so, this is dynamic filtering as we don't know in advance what those values are).
Use Case Example: I want to compare metrics between v5 and v6 of my service. So I ask to show metrics by version, and get v1/v2/v3/v4/v5/v6. I'm only interested in the last two, others is noise. We could be able to remove v1 to 4 from the list of version values.

@jotak thanks for your inputs. I too like grouping into cards.
Regarding the legend, lets discuss with @cshinn what can be done

@jotak I think that's a good point. Having the charts paired could let us use the legend without messing up alignment. We should try that and see if it works well in unusual situations.

@cshinn just making sure I understand this correctly - this is waiting for some updates from your side on the paired charts legend?

@abonas I've updated the recommendations to reflect our discussion


refactor(src): change property into constant

By looking at the comments about the reason behind the usage of the capacity, seems that a constant is more fitting than a property.

Thanks for looking into the details. I think we'll leave it the way it is, since this would be a breaking change. Also, in the rare event someone needs to change the queue capacity, the property would enable setting that from the outside.


x/website: go.dev/doc/install is suggesting darwin/amd64 installer for M1 users

From @dmitshur's comment in https://github.com/golang/go/issues/47578#issuecomment-894457978 (thanks!)

To my knowledge, likely out of device fingerprinting concerns, browsers don't officially expose the underlying system architecture, so web pages typically look the same regardless of whether the user is visiting from a macOS device with an Intel or Apple CPU.
I'm aware there are some workarounds, like getting the device GPU vendor name and checking for a known substring that M1 devices have. An example of that being used is the VLC home page, https://www.videolan.org/, which changes its default download link using that heuristic.
So, I'm not sure that we should try to add automatic detection. If we make both downloads Featured, they should be equally visible and hopefully easier to pick the intended one.

I think detecting the underlying system may be difficult. Then, I think we should investigate alternatives

Offer "Universal" as the default, but also provide "Intel Chip" and "Apple Silicon" options.
Present both options for MacOS users: Either drop-down menu or two different download block. If we take this route, add first-time go user-friendly description to refer to each architecture (I rarely see darwin-arm64 or darwin-amd64 outside the go community)

Does "Universal" already exist, or are there plans to add "Universal" to https://go.dev/dl/ ?

@Deleplace There is currently no universal version. I expect that will involve more work from the release team's side, and I am not aware of any plan. I added this as a possibility :-)  And another consideration is the potential binary size.

To put a finer point on it: when a Apple Silicon mac user visits the download page, they are presented with this:

and that big download button is the wrong link.
They will probably install that package and everything will seem to work, because the emulation is very good. The programs they compile will be amd64 binaries and will run a little slower because they are running under Rosetta. They may only become aware that there is an issue when some deeper issue exposes the amd64/arm64 mismatch.
(This keeps happening at our company.)

I really think this needs fixing. For now, we can just have two download buttons to force the user to pick the appropriate one.
It also seems like the macOS installer should warn the user if they are installing the amd64 toolchain on an Apple Silicon machine.

As the issue of this title says, when this issue was originally filed, the "Download" button at https://go.dev/doc/install was quite misleading. CL 480955 made it point to the https://go.dev/dl/ page that lets users see all available downloads and select one appropriate for their needs.
We considered the universal installer path but decided against it (it's larger in size, creates an additional decision for users, and requires work to implement), and by this point it's less likely there will be new information that would tip the balance towards reconsindering that.
It doesn't seem like there's more to do here. @hyangah Are you okay with closing this issue as resolved?


Ability to add markers/selectors to a chart by end user.

What problem does this feature solve?
Ability to add multiple markers/selectors in the chart by the end user at a specific point similar to brush functionality , and moving them by dragging.  current brush functionality allows us to select only a range of points.
Example scenario of it .
https://stackoverflow.com/questions/75064009/customizing-echart-horizontal-select-brush-functionality-to-select-single-point
What does the proposed API look like?
Similar to brush functionality (Horizontal & Vertical selection )

A button in the toolbox to enable the selector.
after adding selectors/annotations to the chart, an event to capture selectors :

Look at this example.
By clicking twice on the chart, you will get two points pointInGridA and pointInGridB.  Then you could add a MarkArea between their X-coordinates.
It will look exactly like the yellow area in your example:


perf(core): do not run Entity constructors twice per field while discovering default values

I was investigating why my Entity constructors were begin called, both 1 - at startup, and 2 - at creation via em.getRepository(...).create() time despite doc saying

Internally, MikroORM never calls entity constructor, so you are free to use it as you wish. The constructor will be called only when you instantiate the class yourself via new operator, so it is a handy place to require your data when creating new entity.

I found that :

the constructor was indeed called twice for every prop in metadata, which this PR attempt to improve by instanciating entities twice only during metadata instanciation, and reusing those two instances for every property

Internally, MikroORM never calls entity constructor only refers to entities fetched from the DB, since entity constructor, as suggested in other pages, will indeed be called for every em.create().

The constructor will be called only when you instantiate the class yourself via new operator

this is wrong ?

📝 : this may just be a useless minor micro-optimisation; I opened it just in case

I really doubt this would be even measurable.

Internally, MikroORM never calls entity constructor only refers to entities fetched from the DB, since entity constructor, as suggested in other pages, will indeed be called for every em.create().

You can create managed entities this way too, so not every call. That sentence seems correct to me - the word internally is important, using em.create() is considered explicit usage, not internal. I don't mind improving the wording, but I woudn't call it wrong.

I really doubt this would be even measurable.

@B4nan yeah, I just had some logging happening in my constructor =>  that's what made me investigate this in the first place to understand why did I had those extra logs :)

the word internally is important, using em.create() is considered explicit usage

yeah, I now understand what you mean. I just didnt understand when I first saw it. I guess a small tweak could help here :)

I'll close this one as well, I dont think the perf-gain is there, not a measurable one, not worth complicating the code more.


How to remove this gem

How to remove this gem so i can install redactor2 gem thanks

what kind of problem do you have?
just remove on Gemfile or with command line gem uninstall not working for you @saroar ?


Updated INSTALLED_APPS for 'collectfast'.

Quote from https://github.com/antonagestam/collectfast:

For Django 1.7+, 'collectfast' should come before 'django.contrib.staticfiles'. For Django versions below 1.7, it should come after 'django.contrib.staticfiles'. Please note, that failure to do so will cause Django to use django.contrib.staticfiles's collectstatic.

Hey @ka7eh, the solution looks great! Let's not do any checks for Django version, we can always make the collectfast to be the first installed app and a comment on the top why it needs to be on the top[1].
Also, do make sure to add yourself to https://github.com/pydanny/cookiecutter-django/blob/master/CONTRIBUTORS.txt :blush:
[1] https://github.com/pydanny/cookiecutter-django/blob/master/{{cookiecutter.repo_name}}/{{cookiecutter.repo_name}}/config/production.py#L40-L46
closes #187

Hey @theskumar,  just updated the PR :)

Awesome! merging it in. Thanks a lot! :)


SSR without fetching data in server

Could you please give some solutions or examples of doing ssr without fetching data in server? (with apollo).
For our cases, the auto code spliting and ssr for page structure are great and enough. We want our server only to deliver the minimum app.
Thanks.

Fetch the data inside of componentDidMount() instead of getInitialProps().
The component doesn't mount until it's rendered by the browser.

You could also use  https://github.com/kadirahq/react-no-ssr

Or just don't call await getDataFromTree

Yep. I think these are the solutions.

Check this example app: https://github.com/zeit/next.js/tree/master/examples/progressive-render

Thanks for all your replies. I've built my app according to with-apollo example, and tried to implement this by

only creating a singleton apollo client and redux store on the server.
removing all the getInitialProps code of withData HOC.
and the client seems to act as I expect, but I'm still curious about
how should I set the ssrMode option of apollo client?
Will this way ensure that the apollo client on the server will not try to fetch data?
Must I set ApolloProvider on the server? I think it's only useful on the client.

I guess as long as you are client side, you can set ssr mode to false
If you don't call getDataFromTree, I guess the server code won't have the time to fetch any data.
Can try to remove it but probably the Apollo code will complain it doesn't find the provider. I guess.

@sedubois I have struggled out a simple structure to fulfill the requirmemt. see my app repo
I guess your 1 and 2 points are right. But when I removed the apollo provider on the server, it's the react on the client who complain that I've given different render code for ssr. So I have to mimic the components structure on the server, but I can confidently remove the data fetching code now.


Remove the dependency on Faraday

For our use case it is difficult to get an extra gem installed. At the moment, this means we cannot use the module. I would like to use at least the types (and providers) in this module, but they are 'hard wired' using faraday.
I've looked at the code and it seems you are using faraday mostly as an easy way to call NET:HTTP. I could try and make a PR removing Faraday from the provider. Is this beneficial? Am I missing something in the reasoning for Faraday?

We can remove faraday once we implement a proper redirect:
https://github.com/lostisland/faraday_middleware/blob/master/lib/faraday_middleware/response/follow_redirects.rb

In older issues I saw you also had ideas to remove faraday. Did you start with this?

I started working on #55, but got hanged up because of the redirect issue.

+1

I've been looking into this. I'm guessing issue #55 is caused by the standard Net::HTTP class and not by Faraday. It looks like Net::HTTP   is reading the whole response object into memory, causing memory problems when working with big files. So when we replace Faraday by our own solution, we also have to be careful not to keep the same problem.
Wouldn't it be easier to use wget. This would still introduce a dependency, but this is a rpm dependency that, most of the times is much easier. The advantage would be a battle tested en tried http/ftp and https implementation.
It supports a lot of stuff, for example:

retries
limit rate. This could be useful when downloading files on fragile networks
proxies
redirects

This module was originally written to handle both windows/linux. A native ruby solution worked better than being dependent on a package that I could not install in windows. Both wget/curl can be implemented as providers to supplement the default provider (or even default providers on Linux). The util portion can be moved to download method so the default provider can be the parent provider with the download method replaced with your preferred solution on platforms where it's suitable.

Sorry. Forgot about Windows for a moment ;-). I'll have my stab at making a provider for unix wget.


using for time

hi
i want using this package for time range selection.Have you try this for time?

yes, did you have any question?

tanks for you :)


Support More Preview/OnQueryChange Options

This hooks up the Preview functionality with Text/Command/Request/ValTown operations.
This also lets a page refresh (onQueryChange) with Command/Request/ValTown operations.

@pomdtr Thoughts?
I haven't updated the schemas yet.

Yeah definitely needed!


Swipe fix

Fling left, then immediately fling right on an item while the dismiss animation is going on. This will corrupt the state of the recycler view. This proposed fix disables touch right before animation begins, and re-enables it when animation completes.

Are you sure this corrupts the state of the recycler view? I tried on my devices and could not reproduce. The problem with this solution is that it blocks you from dismissing items fast, like this: https://dl.dropboxusercontent.com/u/8695288/demo.mp4.
I checked and the onAnimationEnd method from where the SwipeListener is called runs always on the UI thread, so there shouldn't be any problem with synchronism either.
What version of Android were you using?
Thanks for pulling, anyway.

Only when you swipe the empty space created by the first dismiss: https://www.dropbox.com/s/ov5z04wmyu8mw8f/swipe.mp4
I tried this on a One Plus One running KitKat, and a Xperia Z2 running 4.1.2. But you're right that the proposed fix blocks fast dismissal. Perhaps we can disable only the item being dismissed?

See https://github.com/brnunes/SwipeableRecyclerView/pull/3.


Add system_prompt

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?

I've updated the wrap_prompt method to accommodate an optional system_prompt parameter. This change allows the method to prepend a given system prompt to the user prompt, enabling more versatile usage scenarios. For instance, it can be used to set a context or provide instructions when generating prompts for AI models.
The inclusion of the system_prompt enhances the functionality of the wrap_prompt method without altering its existing behavior when no system prompt is provided. This backward-compatible enhancement makes the method more flexible and applicable in a wider range of situations, such as automated generation of prompts for various AI roles or tasks.

Hi @alexrudall did you had a look? Will this PR be merged?

Hey @deikka @mybuddyandrew , I'm not sure about this one. Could you share a simple scenario where it would be helpful? I can't quite see it.

They recommend using a system prompt when using Claude 2.1 (here) if using completions.

Thanks for this - closing as looks like the Text Completion endpoint is deprecated in favour of Messages.


fix: update chromedriver download URL, resolve test issues

This fixes a few things to get CI working for the repo again:

The chromedriver releases were previously 404ing, causing these tests to fail. The URL for where these are hosted was changed, so it's just a simple update to get this working again.
In later Cargo releases, the output format for the "Finished..." log line changed. I've fixed this by making matches_cargo accept both the old and new formats.
With MacOS 14 (and by extension, M1) runners becoming the macos-latest default, the current geckodriver setup fails for that platform. ~It's not entirely obvious to me why at this point. Downgrading to the macos-13 runner has these tests working again. This is probably the most.. questionable change as it'd be better to track down why the latest runners don't work.~ Edit: It seems like the only thing preventing macOS from working was that I needed to manually install Firefox. I'm guessing this was available in the previous macOS runners by default.

Make sure these boxes are checked! 📦✅

[x] You have the latest version of rustfmt installed

$ rustup component add rustfmt

[x] You ran cargo fmt on the code base before submitting
[x] You reference which issue is being closed in the PR text

✨✨ 😄 Thanks so much for contributing to wasm-pack! 😄 ✨✨
Closes #1390

Seems like there's some unfortunate caching issues I'll have to dig into.

Seems like there's some unfortunate caching issues I'll have to dig into.

Thank you for looking into this! Appreciated!

Thank you for looking into this! Appreciated!

This issue seems to be resolved now for the sake of this PR. However, in general I've found some consistency issues with trying to determine what paths to cache when manually in many of my other projects. Are you open to switching to something like Swatinem/rust-cache or mozilla-actions/sccache-action? I've used both successfully quite a bit, though recently all of my projects have moved to the latter. If so I can open up another PR after this is cleared to merge to switch out the caching.


Laravel 8.77.1 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v8.77.1 branch.
Before merging, you need to:

Checkout the shift-ci-v8.77.1 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:information_source: The following Laravel packages were updated as they are known to be compatible with Laravel 8.77. Nonetheless, you should review the CHANGELOG to ensure no additional changes are needed.

[ ] thujohn/twitter

:warning: The Shift build servers run PHP 8. If you are still running PHP 7.4 or lower you should run composer update locally to ensure you have the proper versions of your dependencies to reflect your PHP 7 environment.


NuxtErrorBoundary does not catch all client-side errors

Environment
nuxt 3.10.2
Reproduction
Repository: https://stackblitz.com/edit/github-tobvzw-jwk7ax
Just load the application, you will be presented with the "500" error view of nuxt. The expected behavior is that the text "This is the error rendering" is rendered.
If one would comment out the onMounted hook in my-broken-component.vue and click on the rendered paragraph, the NuxtErrorBoundary works as expected.
Describe the bug
It seems that client-side thrown errors will pass through the NuxtErrorBoundary component if they are thrown right after being mounted.
In the stackblitz example, the error is thrown in the onMounted lifecycle hook. This is also the case if one would watch a value coming from useAsyncData (with {server:false, immediate:true} as configuration) and throw based on that. So it appears to be a timing issue when throwing errors in the client too early.
This only happens when SSR is enabled.
Additional context
No response
Logs
No response

Incredibly frustrating - I can't find a single example of NuxtErrorBoundary being respected in this and similar scenarios - seems to be no documentation on addressing this either.


Improve CSV File Handling

This PR aims to improves handling CSV file creation and improve code quality.

Adds class for CSV file handling, includes related CSV features.
Adds Save file mode allowing to save CSVs next to timelines or inside the Translation file path.
Fixes Update CSV files not finding all timelines outside of Translation file path.
Adds Save file mode setting to signal receivers saving settings.
Disables Update CSV files if Translation file path is required and invalid.
Updates related tooltips.

Tasks left to do:

[x] Disable Update CSV files button, if Translation file path is empty and a project CSV file is used or the save file mode requires it.
[x] Further testing.

Lgtm. Thanks!


Fix bugs on MSEdge

About
resolve #5
thanks! @CookiesEater and sorry for the delay 🙇

🤣
https://qiita.com/tonkotsuboy_com/items/7b36bdfc3a9a0970d23b#web制作者が注意すべきこと


Can't add or retract GP after new drop-down menu

Describe the bug
Can't add or retract GP after new drop-down menu
When adding or retracting GP manually, it instead adds or retracts EP
To Reproduce
Simply try and add GP manually (see screens below)
Expected behavior
GP was expected to be added - not EP
Screenshots

EPGP before:

EPGP after:

AddOn and Game Version:
v4.1.1
WotLK Classic

Hi
Thank you for the report. This is an unfortunate regression and the UI changes shouldnt have been released

All good, thanks :)
Rolled back to v4.1.0 for now and all works again.


API Token use cases integration for account page

What this PR does / why we need
This PR integrates the API Token account page UI with the new API token management use cases, ensuring seamless interaction between the UI and backend functionality.

Read current API token
Recreate API Token
Revoke API Token

Which issue(s) this PR closes:

Closes #505 #506

Special notes for your reviewer:
waiting until this PR is reviewed.
Suggestions on how to test this:
Step 1: Run the Development Environment

Execute npm i.
Navigate with cd packages/design-system && npm run build.
Return with cd ../../.
Ensure you have a .env file similar to .env.example, with the variable VITE_DATAVERSE_BACKEND_URL=http://localhost:8000.
Navigate with cd dev-env.
Start the environment using ./run-env.sh unstable.
To verify the environment, visit http://localhost:8000/ and check your local Dataverse installation.

Step 2: Test the feature

Ensure the application is running on http://localhost:8000/.
Navigate to http://localhost:8000/spa/account?tab=apiToken to begin testing.
Confirm that you are logged in and have appropriate permissions.
Press the "Recreate Token" button on the page.

A new API token is displayed and an expiration date is shown in the correct format. YYYY-MM-DD
Reload the page by refreshing the browser, ensure the same API token and expiration date remain

Press the "Revoke Token" button on the page.

The API token is revoked (disappears from the page).
"API Token for Dataverse Admin has not been created." and a "Create Token" button is shown.
Reload the page by refreshing the browser, ensure the same API token and expiration date remain

Press the "Create Token" button on the page.

A new API token is displayed and an expiration date is shown in the correct format. YYYY-MM-DD.
Reload the page by refreshing the browser, ensure the same API token and expiration date remain

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:
Additional documentation:

coverage: 97.486% (-0.03%) from 97.518%
when pulling c60e345cc12fc15921107b39a42c0a64c016948a on api-token-usecase-integration
into 82e4779703c5abe4c5df5165b256a84da040aa61 on develop.

Hello @ChengShi-1,
Had an observation on this ticket.
Copy to Clipboard isn't working when I Recreate token. Can be reproduced in storybook and local.
Steps to reproduce:

Go to http://localhost:8000/
login
Now go to http://localhost:8000/spa/account?tab=apiToken
Copy API Token to clipboard
Paste API token anywhere to verify copying worked.
Recreate token
Copy to clipboard
Paste API Token

Here is where the issue lies, the recreated token isn't getting copied.

Hello @ChengShi-1,
Had an observation on this ticket.
Copy to Clipboard isn't working when I Recreate token. Can be reproduced in storybook and local.
Steps to reproduce:

Go to http://localhost:8000/
login
Now go to http://localhost:8000/spa/account?tab=apiToken
Copy API Token to clipboard
Paste API token anywhere to verify copying worked.
Recreate token
Copy to clipboard
Paste API Token

Here is where the issue lies, the recreated token isn't getting copied.

Thank you, Omer! one variable was wrong in the copy function, I have a new commit to fix it. Also, it may need to be reviewed and approved again @g-saracca Thanks!!

Thanks for the fix @ChengShi-1 looks good now. Merging PR.
Testing of 531.docx


fix(versions): update Activiti/activiti-cloud-infrastructure-chart versions

UpdateBot pushed helm dependency: activiti-cloud-gateway to: 1.0.7

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-gateway 1.0.7

/test this
/approve


setup_jenkins

Why does it deletes other KeyChain such as Login?

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

Complete output when running fastlane, including the stack trace and command used

[INSERT OUTPUT HERE]

Environment

[INSERT OUTPUT HERE]

@fastlane-bot there is not output of fastlane env! It just deletes login keychain that we can see on keychain access app

@skathayat It shouldn't be deleting the login keychain. Can you provide what parameters you are passing to setup_jenkins, if any?

@joshdholtz I have tried without any parameters, and it deletes login keychain. The name of my keychain is different from login.


Setting default Indent type to TABS

without this indents defaults to space

@bojidar-bg Indent checks are done against a String "Tabs"
use_space_indentation = EditorSettings::get_singleton()->get("text_editor/indent/type") == "Tabs" ? 0 : 1;

Superseded by #8544.


Add Truth plugins to fuse

This PR will add truth output to fuse. The logic differs from the one used in WFSim so one has to be a bit careful with the interpretation of the output.
The code in the PR works as follows:

The cluster_id from the microphysics clustering will be propagated up to the propagated_photons level. For each photon, it is now possible to tell which cluster it came from.
The PMTResponseAndDAQ puts these photons into waveforms and splits the waveforms into raw_records.  It will now save an additional output called contributing_clusters. This output contains the information which cluster contributed with how many photons to a given raw_record.
In the next step a new PeakTruth plugin evaluates which clusters contribute to a peak and calculates truth data for each peak. One of the provided columns is the observable_energy_truth (we can rename columns if needed), indicating how large the energy was that went into a certain peak. Some examples:

Let's assume there would be a cluster of 100 keV creating 100 S2 photons. If just 95 photons make it into a peak, the observable energy for this peak will be given as 95 keV.
If strax would split the S2 originating to this cluster in two peaks each with 50 photons, the observable energy for each peak would be listed as 50 keV.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-113 of 209 (45.93%)  changed or added relevant lines in 13 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-1.1%) to 65.299%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fuse/plugins/truth_information/surviving_clusters.py
12
26
46.15%

fuse/plugins/truth_information/event_truth.py
12
30
40.0%

fuse/plugins/truth_information/peak_truth.py
11
48
22.92%

fuse/plugins/pmt_and_daq/pmt_response_and_daq.py
36
80
45.0%

Files with Coverage Reduction
New Missed Lines
%

fuse/plugins/pmt_and_daq/pmt_response_and_daq.py
1
47.86%

Totals

Change from base Build<PHONE_NUMBER>:
-1.1%

Covered Lines:
1703

Relevant Lines:
2608

💛  - Coveralls

Hi @HenningSE , I would say the ap_photon_number_truth is a bit misleading. If I understand correctly, it is actually how many photons creates afterpulses, right? But the name seems to indicate the magnitude of afterpulses in unit of PE. Maybe we should rename it, or give the magnitude (Like the sum of photon_gain divided by PMT gain, which doesn't need to be an integer. Maybe this is technically challenging.) instead.

Hey @HenningSE . Just for my more clear understanding, what is the plan for s2_tag after this PR? Is the s2_tag already in this PR?

I added a draft for a cluster tagging plugin with the last commit. The plugin should evaluate if a certain cluster contributed to the main or alternative S1 or S2. Can you check if this fits your need @dachengx @mhliu0001 ?

@HenningSE Thanks for writing this draft! Yes that is close to what appletree needs. However, there are two problems here:

As you said before, a cluster may contribute to both main and alt S1/S2. Currently appletree separates the simulation for main/alt S2 simulation, as if they are two events. If this is the case, it doesn't make sense if appletree:

either double-count the energy deposition in main S2 simulation and alt S2 simulation;
or it only counts the edep in the main S2 simulation.

There is an upper limit to how many clusters do contributing_clusters contain. Although the number of clusters stored can be configured, it is hard to determine what is the correct number to account for neutron multiple scatter events.

So my suggestion is, instead of tracking the peaks, can you track the photons instead? For example, in photon_summary you have the time and the cluster_id that creates this photon. By time overlapping it is easy to determine whether these photons contribute to main/alt S1/S2, and it is impossible for photons to contribute to both main and alt S1/S2. After that, you can just count: N1 = how many photons are created by a certain cluster; and N2 = among these photons, how many contribute to main/alt S1/S2. Then a ratio can be stored by N2/N1.
This way, appletree can separate the main/alt S1/S2 simulation more easily, and avoid double-counting any energy deposition by multiplying the edep by the ratio.

Thank you very much for your feedback @mhliu0001! I updated large parts of the PR today to follow your suggestion of using the propagated_photons directly to map the clusters to the peaks (and vice versa). Let me know what you think.

Great! I think that is what we need. I will test it with SR0 AmBe fit.

@mhliu0001 how was the "ratio" used in SR0's "S2 tagging"? There was no such a ratio, right?

In general, I understood the necessity of ClusterTagging as a replacement for the so-called "s2_tagging"(we might want to call it cluster tagging from now?). What are the applications of EventTruth, PeakTruth, and RecordsTruth. This is not an objection to these plugins, just to a better understanding.

I was thinking of studies that would need truth data like for example a new 3D position reconstruction just based on the S2 peak information. With these truth plugins you have a simple method at hand to match the positions with the peaks. Right now they are more like a "draft" showing how we could match this data. We can then later add more information to the output of these plugins if this is required for some studies and the users tell us what they need.

@mhliu0001 how was the "ratio" used in SR0's "S2 tagging"? There was no such a ratio, right?

No, there was not. But I think I can use that when a cluster contributes to both main and alt S1/S2 (rare case, but might happen). With the current appletree framework, I can either:

Split the cluster to two different clusters, one contributing to main S2 and the other contributing to alt S2, with different edep according to the ratio;
Depending on the number of photons, assert that the cluster contribute to the larger of them.

I will now bump the plugin versions & update the documentation. When merging this PR we should also bump the overall fuse version to 1.1.0.

@HenningSE In tagged clusters, can you also include how many S1/S2 photons created by a certain cluster are detected? Just by counting the propagated_photons, should be easy. Maybe call them photons_s1 and photons_s2?


Stop sysdig/falco failed when restart dagda server.

Short description
If Dagda server is started twice, it will fail like issue48.
Reproduction steps

start dagda server
python3 dagda.py start -p 5003 -d
stop it with CTRL + C
start another dagda server
python3 dagda.py start -p 5000 -d

Expected results
I expected that the dagda server start successfully both.
Actual results
<2019-03-12 15:09:41,921> <DEBUG> <DagdaServer> <sysdig_falco_monitor> <pre_check:95> <737948e77328251a8c0555dcf9d6767ba0eea108c984e860aeab54e2c07bfc44>
<2019-03-12 15:09:41,921> <DEBUG> <DagdaServer> <docker_driver> <docker_stop:110> <7>
<2019-03-12 15:09:41,926> <ERROR> <DagdaServer> <dagda> <main:46> <500 Server Error: Internal Server Error ("Multiple IDs found with provided prefix: 737948e77328251a8c0555dcf9d6767ba0eea108c984e860aeab54e2c07bfc44")>
Traceback (most recent call last):  File "/home/sxue/project/dagda/dagdaenv/lib/python3.7/site-packages/docker/api/client.py", line 246, in _raise_for_status
    response.raise_for_status()
  File "/home/sxue/project/dagda/dagdaenv/lib/python3.7/site-packages/requests/models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: http+docker://localhost/v1.37/containers/7/stop

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "dagda/dagda.py", line 35, in main    r = execute_dagda_cmd(cmd=cmd, args=parsed_args)
  File "/home/sxue/project/dagda/dagda/cli/dagda_cli.py", line 43, in execute_dagda_cmd
    ds.run()  File "/home/sxue/project/dagda/dagda/api/dagda_server.py", line 101, in run
    self.sysdig_falco_monitor.pre_check()
  File "/home/sxue/project/dagda/dagda/analysis/runtime/sysdig_falco_monitor.py", line 98, in pre_check
    self.docker_driver.docker_stop(container_id)
  File "/home/sxue/project/dagda/dagda/driver/docker_driver.py", line 111, in docker_stop
    self.cli.stop(container=container_id)
  File "/home/sxue/project/dagda/dagdaenv/lib/python3.7/site-packages/docker/utils/decorators.py", line 19, in wrapped
    return f(self, resource_id, *args, **kwargs)
  File "/home/sxue/project/dagda/dagdaenv/lib/python3.7/site-packages/docker/api/container.py", line 1134, in stop
    self._raise_for_status(res)
  File "/home/sxue/project/dagda/dagdaenv/lib/python3.7/site-packages/docker/api/client.py", line 248, in _raise_for_status
    raise create_api_error_from_http_exception(e)
  File "/home/sxue/project/dagda/dagdaenv/lib/python3.7/site-packages/docker/errors.py", line 31, in create_api_error_from_http_exception
    raise cls(e, response=response, explanation=explanation)
docker.errors.APIError: 500 Server Error: Internal Server Error ("Multiple IDs found with provided prefix: 737948e77328251a8c0555dcf9d6767ba0eea108c984e860aeab54e2c07bfc44")

I printed the container_id  to log, you can seen that the container_id is only the first number when stop container sysdig/falco.
<2019-03-12 14:39:18,563> <DEBUG> <DagdaServer> <docker_driver> <docker_stop:110> <7>                                                                                [3/1822]

I have open a PR to fix this issue，https://github.com/eliasgranderubio/dagda/pull/52

I have reviewed the PR #52 and I agree with you but that PR mixes issue fixing with documentation improving.
My commit includes your suggest and it replaces the 'equal' by 'append' and now all look like works fine. Please, review it and give me feedback if the error persists.
On the other hand, I have seen the SVG and it is not entirely true. Feel free to open me another issue for improving the documentation about the internal work flow and I will include the complete flow ;-)

All look like works fine. And I will open a new issue about work flow


There is no "tune_dirichlet_nn.py" in the repo

In the readme, there is calibration commands including tune_dirichlet_nn.py. But I cannot find them in the repo.

Hey! Thanks for pointing out the issue. There was a typo in the ReadMe, instead of "tune_dirichlet_nn.py" there should be "tune_cal_odir.py".
Let me know if there are any more questions! :)


Students with fully marked assignments sometimes show up as "---"

Students in the MarkingTab sometimes show up as ungraded even when they are completely graded.

This is now fixed, was a mistake with an inverted logical expression.


Static build get some warnings about glibc

dlopen getaddrinfo getgrouplist getgrgid_r getgrnam_r getpwnam_r getpwuid_r
Temporally change running docker image to glibc version alpine in this commit https://github.com/CovenantSQL/CovenantSQL/pull/142/commits/ace6fa58872b71b6cbccdfc1be9917c1a0622611

dlopen in sqlite should be disabled

Fixed in e40caf2fa8c2d1e57a3623fdee2a748bebb36aab


Created a text animation for the title when the page load

A typing animation created in the title when the page loads
screen-capture (1).webm

Fab, thanks very much


Add infrastructure to longhaul nightly to run against release branches

Right now we have to manually run this through my foirk updating to a certain release, we should have infra where it picks up release branch and runs longhaul tests against it.

/assign


Emojis and double byte characters handling incorrect

Issue Type: Bug

Insert US flag emoji (🇺🇸)
Move left once (left arrow)
Delete (or backspace)
Flag disappears, but only part of the encoding pair is removed and you can see the rest in the editor (e.g. 🇸)

This is counter-intuitive (as well as different from all other editors).
It also affects selecting text, clipboard operations and in case of editing can distort nearby characters.
Possibly related to: https://github.com/Microsoft/vscode/issues/34497
Thanks guys!
VS Code version: VSCodium 1.31.1 (1b8e8302e405050205e69b59abb3559592bb9e60, 2019-02-13T17:56:31.816Z)
OS version: Darwin x64 16.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-5557U CPU @ 3.10GHz (4 x 3100)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
4, 3, 3

Memory (System)
16.00GB (0.07GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

vscode-languagetool
ada
3.8.0

vscode-languagetool-en
ada
3.8.0

xml
Dot
2.4.0

copy-markdown-as-html
jer
1.1.0

code-spell-checker
str
1.6.10

Came here for a related issue but I can confirm the same behaviour for the US flag.
I'm also seeing that the England ( 🏴), Scotland( 🏴) and Wales ( 🏴) flags render as a black flag when pasted in to the editor, and it takes 7 backspaces to remove those black flags from a .js file.
Version: 1.38.1
Commit: b37e54c98e1a74ba89e03073e5a3761284e3ffb0
Date: 2019-09-11T13:31:32.854Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 18.7.0


url parameters casting bytes to str

Hi there! I noticed a change in behavior in werkzeug 0.15.x whereby url parameters used to be safe to send as bytes (on python 3.6).
I apologize for not showing a more direct example in werkzeug, but like most users of werkzeug - I don't use it directly - I use it through flask.
Demonstration (using Flask)
from flask import Flask, url_for

app = Flask(__name__)
app.config['SERVER_NAME'] = 'www.foo.com'

@app.route('/<string:foo>')
def hello(foo):
    return 'Hello, World!'

with app.app_context():
    print(url_for('hello', foo=b'bar'))

With werkzeug<0.15, this will output:
http://www.foo.com/bar

However as of werkzeug>=0.15, it now returns this malformed url:
http://www.foo.com/b%27bar%27

I would expect that if werkzeug isn't going to decode bytes for you anymore, that it ought to raise a TypeError indicating that bytes is no longer safe to pass in.
I hope this example makes sense! Thanks for reviewing my issue - I'd be happy to (attempt to) submit a pull request if y'all agree that this is not desirable behavior.

I'll have a look at this. We (evidently) don't have tests for this, so I suspect accepting bytes was never intended behaviour.

Seems like fixing this is as easy as catching it: https://github.com/edk0/werkzeug/commit/794eb16b23096db9e51c7cd58a7ded9086f8c930 so I'm inclined that way.

@edk0 I don't think you changed that code in your PR, was there a related change that cause this?
I'm not opposed to this, but it seems a little weird to be passing bytes directly in a URL. @fuhrysteve Under what circumstances do you have UTF-8 bytes and not the text? If you're really passing bytes, and not text, I'd expect to pass them as Base64, which would require a custom converter.

@davidism in my case it was a json web token. PyJWT returns bytes by default.
In [1]: import jwt

In [2]: jwt.encode({'some': 'payload'}, 'secret', algorithm='HS256')
Out[2]: b'eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJzb21lIjoicGF5bG9hZCJ9.Joh1R2dYzkRvDkqv3sygm5YyK8Gi4ShZqbhK2gxcs2U'

@davidism I believe it was an unintended consequence—I swapped out the implementation of to_url for a faster one that pre-baked but needed bytes as input; the usual behaviour is to convert to text_type and then encode, so I did that, and didn't notice the change in behaviour for bytes.
I do wonder if it might be a better idea to add a bytes converter and have BaseConverter throw an error.

@edk0 isn't everything passing through UnicodeConverter here, which is the "default converter"? At a glance, I'd suspect it ought to go there and have BaseConverter or something beneath it raise TypeError prior to any conversion to a string.
I share your hesitation to make utf-8 assumptions in BaseConverter

It's not actually making UTF-8 assumptions, just URL encoding the bytes as-is, but it does seem wrong.
I'm torn because we did special-case bytes in the past. Maybe we should add a converter that passes bytes as-is, and have BaseConverter do the same for now, but with a warning.

Same problem here. I use itsdangerous to generate JWT token for user confirmation in this way:
from flask import current_app
from itsdangerous import TimedJSONWebSignatureSerializer as Serializer

def generate_token(user, operation, expire_in=None, **kwargs):
    s = Serializer(current_app.config['SECRET_KEY'], expire_in)

    data = {'id': user.id, 'operation': operation}
    data.update(**kwargs)
    return s.dumps(data)

The return value will be in bytes type, like b'eyJhbGciOiJ...YmQ'. Then I generate the confirmation URL with the token returned in email template:
{{ url_for('auth.confirm', token=token, _external=True) }}

With Werkzeug >= 0.15, It end up to something like this (Notice the unnecessary b%27 and %27 part in the URL):
http://<IP_ADDRESS>:5000/confirm/b%27eyJhbGciOiJ...YmQ%27

Here is the view function that handle token confirmation:
@auth_bp.route('/confirm/<token>')
@login_required
def confirm(token):
    print(token)  # b%27eyJhbGciOiJ...YmQ%27

The correct token value should be just eyJhbGciOiJ...YmQ, so the token confirmation will be failed finally.


Correct arity in error: cannot reverse migration

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Pin pino to 8.16.0

Description
Possible regression with 8.17.0 https://github.com/pinojs/pino/issues/1864
Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change

Related issues

Fix #1

Migration notes

Describe migration notes if any

Extra info

Add any additional information

Does this fix the logging INFO issue?

No that's an Airbyte platform issue.


[Sublime Text] Lua.workspace.checkThirdParty is not allowed

I think this is an issue of Sublime

Maybe this option is not allowed in the user config just because it is not in the default config?

Sorry, I just noticed that the Sublime plugin has its own repository. I should report this problem there.


Create continuous delivery to the staging environment using OpenShift

[x] For every commit (merged pull request) to the main branch, we'll deploy to the staging environment.

The pipeline must be green (earlier pipeline steps pass: no linter or test erros)

CI/CD to staging complete. Production yet to be configured.

Production deployments will require a manual github tag/release, keep production deployment "manual" (couple commands, will be documented) for now.


Update to the "Concepts" Page in Documentation

I've started work on summarizing the original hls4ml paper in the concepts page, mostly speaking to the motivation for the project thus far. I plan on adding more detailed information about the jet substructure application.

Thanks, let me know if you need help. I think we should aim to make it very introductory & concise.

Okay. I'll just wrap up with a paragraph or two explaining the neural net implementation described in Figure 2 of the paper.

Hey, I think what I have in mind is something like this -- A step-by-step introduction with concise & intuitive sections.

I'll make some changes to get something more along those lines.

Yes, this is what I was thinking of!! We might want to fix some parts but the outline is there!😃

Thanks for contributing this! It looks like the math formatting isn't working to me?

I made the requested changes. It seems that the LaTeX plugin for GitBook no longer works, and I tried a few other ways of loading the MathJax script that did not work. As such, I just redid the equations in html, which isn't ideal but doesn't look too terrible.

What is the status of this? Was it included in the new sphinx docs?

Yes, sorry forgot to close this.


Fixed bug in backward hooks

Also fixed bug in generated hook_dict (with a very, but less, janky fix)


Add slow fontmake compile time projects for testing

Thread to collect information on open projects that have slow fontmake-based compile times so that we can reproduce the compile during diagnostic and optimization tests.
Let's consider "slow" to be over 10 minutes wall clock time to start.  Please feel free to suggest a different time if this is not a good arbitrary target to begin. (We're going to examine the distribution of Fonts compile times to better define what is "slow", #25)
Please include the following details:

repository URL
testing commit
project requirements.txt file with Python dependency versions
any information that you have on why the project might have slow source to font compile times

cc @marekjez86 @madig @simoncozens @belluzj

https://github.com/googlefonts/roboto-serif
@davelab6 mentioned to me today that this is might be a good example based on feedback from @m4rc1e.
154 master, 4 axis VF design space


Fix plots for TensorNetworks with hyperedges

From issue #21
Summary
In this PR we fix the plot functions: now these can plot TensorNetworks with hyperedges. The hyperedges are visualized with a grey diamond-shaped COPY tensor, with grey indices coming from it.
Example
julia> using Tenet
julia> using GLMakie
julia> using Makie
julia> tn = TensorNetwork([Tensor(rand([2, 2, 2]...), tuple([:i, :c, :k]...)), Tensor(rand([2, 2, 2, 2]...), tuple([:i, :k, :s, :p]...)), Tensor(rand([2, 2, 2]...), tuple([:i, :s, :c]...)), Tensor(rand([2, 2]...), tuple([:p, :t]...))])
TensorNetwork{Arbitrary}(#tensors=4, #inds=6)
julia> plot(tn; labels=true)
(Scene (800px, 600px):
  0 Plots
  1 Child Scene:
    └ Scene (800px, 600px), Axis (1 plots), Combined{GraphMakie.graphplot, Tuple{Graphs.SimpleGraphs.SimpleGraph{Int64}}})
julia> [tensor.labels for tensor in tn.tensors]
4-element Vector{Tuple{Symbol, Symbol, Vararg{Symbol}}}:
 (:i, :c, :k)
 (:i, :k, :s, :p)
 (:i, :s, :c)
 (:p, :t)

PD: Still working in 3D.

Okey, now it looks something like this:


Remove recommendation for creation of labels

This PR removes the recommendation to add labels to PRs and issues in CONTRIBUTING.md because community members do not have privileges to create labels in this repository (text adapted from here)
This has been done

in the Android repository: https://github.com/corona-warn-app/cwa-app-android/pull/2122
in the Website repository: https://github.com/corona-warn-app/cwa-website/pull/652

@Ein-Tim ok, I see. There are certain labels the community can use. Maybe it would be nice to include a sentence to use one of the available labels (bug, question, enhancement, ...) ?
Removed sencence:

Please apply one or more applicable labels to your issue so that all community members are able to cluster the issues better.

@dsarkar
Yes we could for sure go that way, I took the PR from @heinezen back in the website repo as basis, he removed the suggestion completely, so I also removed it.
Just let me know what you want 👍

@dsarkar

There are certain labels the community can use. Maybe it would be nice to include a sentence to use one of the available labels (bug, question, enhancement, ...) ?

Community members do not have the privileges to add or remove labels. Labels like "Bug" are added through one of the files in ISSUE_TEMPLATE. The label depends on which issue type the user chooses when they open a new issue and it can't be changed by the user subsequently. That is only something that somebody with more privileges can do. I can't see how that works exactly because I am only a Contributor so I don't have view access to the Settings of this repository!
@Ein-Tim's PR looks good; completely removing the sentence as was already done in the other repositories.

@MikeMcC399 @dsarkar @Ein-Tim
Mike is correct in that non-members cannot assign or remove labels by themselves. The chosen template decides the initial labels of an issue.


Misc typo

attributes is duplicated
If you set this variable to true, then XML elements will include the following attributes attributes:

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

ID: 2f5fa09e-7f51-7efa-9017-d24c23cbc88f
Version Independent ID: 2edba48f-c57d-45e7-a249-52a41aa05c35
Content: WordXmlPart Method - Business Central
Content Source: dev-itpro/developer/methods-auto/report/report-wordxmlpart-method.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks! The fix will be published soon.


Fix RFC7230

See: https://hypothes.is/a/ZIoKvGXDQpehbcAXc1Y5iw

@azaroth42 @csarven @melvester your thoughts on how best to approach fixing RFC7230's userinfo :bug: would be most helpful. :smile:

Hmm. The last paragraph of 2.7.1 is pretty explicit that userinfo is now an error in HTTP 1.1 URIs
[Semantic versioning? What's that?]
Looks like an uphill battle, I'm afraid. Not sure I have any good ideas :(

Yeah...looks like I should've gotten into specification discussions a bit earlier. :frowning:
Well, despite the "overreach" I think the use case(s) are compelling enough to soldier on and see how far we get.
They're still valid URI's. It's just currently invalid for an HTTP to parse them into a header. Once there's a reasonable HTTP header to parse just the user part into, then maybe it'll go down easier. :wink:
We'll see. :eyeglasses:


How to build from YML file?

Firstly, I want to say a massive thank-you for building this project.... I have been flighting with SDR software and VAC issues for years and years. I am very very excited to try out this project.
My current issue is that I am not sure what sort of tool set I need to install in order to build either Windows or Linux files from the YML files.
Your readme.md does not mention how to build the code?
Thanks.

Hi! There should be windows and linux binaries under the releases page (https://github.com/jeroenbeijer/SDRReceiver/releases) and then assets. I have not read up on the proper release process etc yet but will do so soon. Anyways those should get you started. You can run the windows build locally from an msys command line. From ubunti and also raspi (jonti has confirmed) it should work like this with git already installed:
git clone https://github.com/jeroenbeijer/SDRReceiver
cd SDRReceiver
./ci-linux-build.sh
There are probably some issues to iron out still but hopefully the basics are working.

Ah, my bad. Somewhat inexperienced in the way of the GitHub. Thanks for pointing that out.
I will be testing on Windows to start, but am very interested in running up a Pi 4 to see how it goes there.
Will be writing up how-to notes etc on my pages: https://thebaldgeek.github.io/
Thanks again for pointing out where to find the builds.


add .zenodo.json file?

On a related note to the mailmap PR (#485), we could potentially also add a .zenodo.json so that Zenodo will get author names and affiliations right.
I have seen some other projects do this. For example:  https://github.com/nipy/nipype/blob/master/.zenodo.json
The bulk of contributors from non-academic positions probably have little interest one way or another, but it might be nice to support it for those that want to add their affiliations here rather than having to edit that info manually over at the Zenodo site.

interesting, I had never seen that before. yes, it sounds fine with me. let's just add a comment in it though that this is for every contributor to maintain for themselves - if they move, then they need to update it via a PR or it will remain out of date


Lobby/party system

Hey man,
Tried the backend system, and it works only for "Solo" mode. Tried forking it and make a lobby/party system and it's kinda hard.
Do you have any ideas?
Thanks

Hey,
I understand the problem. When I started this project my initial case was just to be able to setup a simple matchmaking system by filling up queue's.
But after a while I realised how difficult a matchmaking system is.
Because to be able to create a Lobby/party system you should remake a large part of Steam.
Since you have to be able to add Friends, invite them to a lobby. And then fill up the remaining places with others.
Since I know that other Matchmaking systems are already there (Faceit, ESEA) I don't think a lot of people will use this. So if you want to add such a feature please do so. I am available for support if needed but I think that this basic setup is what it is for now.


[ADDRESS-META] SonicDiamond

SonicDiamond
What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
kind api-change
/kind backend-change
/kind ui-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
Includes new metadata for SonicDiamond (SDI) Token
Which issue(s) this PR fixes:

Fixes #
Adds SDI Info
Special notes for your reviewer:
Follow up to iotex-token-metadata pull request
Does this PR introduce a user-facing change?:

SDI data will now be available to view.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Error: Function not rcognised

The tool worked perfectly until around 17.00 yesterday, and has been down since.

Most probably you have exceeded your daily quota for URL Fetch calls that the plugin uses internally to communicate with the Wikipedia and Wikidata APIs. You can see the quota here:
https://developers.google.com/apps-script/guides/services/quotas#current_quotas.

Thnaks! Might be for me, but not for two other users who hadn't used it yesterday. They get the same message.

Can you try the solutions from this answer: https://stackoverflow.com/questions/32077904/custom-function-not-recognized. The Google Sheets backend occasionally gets stuck :-(

No. Tried 4 diff browsers.
Do they work for you? Try =WIKIDATAQID(CONCAT("en:",B2)) in B1; enter 'London' or 'Madrid' in B2. Should return the Qcode (Qid).

It worked. One tip is to open the documentation, this helps "kick" the loading of the add-on. See the screenshot below for how to do this:

It worked! Reminds me of starting our old Morris Oxford by hand, back in the early 70s! But it worked!
Thanks for your time and patience!

Happy it worked. I also looked up Morris Oxford and was not disappointed. What a diamond!


Cannot access class CSVFormat on writing into csv file

Project declares only following dependency to this project:
implementation("org.jetbrains.kotlinx:dataframe:0.8.0-rc-2")

When I try to compile a file that has following line:
dataFrame.writeCSV(benchmarkCsvFile)

Compilation fails with following message:
Cannot access class 'org.apache.commons.csv.CSVFormat'. Check your module classpath for missing or conflicting dependencies

Adding compileOnly("org.apache.commons:commons-csv:1.8") dependency fixes compilation issue.

It was fixed two days ago: https://github.com/Kotlin/dataframe/commit/5a0668c4bc28533c90ef8baf1e12ea344dd29c14
Should be present in the nearest upcoming release

Fixed in 0.8.0-rc-3


Update dlp-policy-tips-reference.md

Updated  a note on Policy Tips in older versions of Outlook
Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/9393

@JohanFreelancer9 Please copy review

Docs Build status updates of commit 58f4c89:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/compliance/dlp-policy-tips-reference.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 906ec1e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/compliance/dlp-policy-tips-reference.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 0bcce1c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/compliance/dlp-policy-tips-reference.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@chrfox Please merge this PR. Thanks

reviewing with engineering

Docs Build status updates of commit 580709e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/compliance/dlp-policy-tips-reference.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


New Snippet example for Shared Image Gallery

Adding new Code Snippet for Shared Image Gallery Resource.
Contributing a Pull Request
If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check.
Contributing to documentation

[ ] The contribution does not exist in any of the docs in either the root of the docs directory or the specs

Contributing an example

[ ] I have checked that there is not an equivalent example already submitted
[ ] I have resolved all warnings and errors shown by the Bicep VS Code extension
[ ] I have checked that all tests are passing by running dotnet test
[ ] I have consistent casing for all of my identifiers and am using camelCasing unless I have a justification to use another casing style

Contributing a feature

[ ] I have opened a new issue for the proposal, or commented on an existing one, and ensured that the Bicep maintainers are good with the design of the feature being implemented
[ ] I have included "Fixes #{issue_number}" in the PR description, so GitHub can link to the issue and close it when the PR is merged
[ ] I have appropriate test coverage of my new feature

@fberson - thank you for another contribution! I will review once https://github.com/Azure/bicep/pull/2307 is completed.


Memory leak in uikit pa11y test

Bug Report

[/] I’ve read and understood the Contributing guidelines and have done my best effort to follow them.
[/] I’ve read and agree to the Code of Conduct.
[/] I’ve searched for any related issues and avoided creating a duplicate issue.

What happened
Running npm run test is getting a memory leak. It is also failing in CI.
What I expected to happen
Should not create a memory leak.

This was related to an update to puppeteer, locally we got a memory leak and circle was failing to finish creating pages.
We could not quickly resolve the memory leak so we decided to stay with the current version of puppeteer for now. Hopefully this issue resolves itself 🙏


State and cities drop down not working after selecting the country

Describe the bug
I followed your documentation and try to use it but i am unable to click on states and cities after i select on country. I followed the same example shown on pub my code looks like
/// Variables to store country state city data in onChanged method.
String countryValue = "";
String stateValue = "";
String cityValue = "";
String address = "";
@override
Widget build(BuildContext context) {
return Scaffold(
appBar: AppBar(),
body: Center(
child: Container(
padding: EdgeInsets.symmetric(horizontal: 20),
height: 600,
child: Column(
children: [
CSCPicker(
showStates: true,
showCities: false,
flagState: CountryFlag.DISABLE,
dropdownDecoration: BoxDecoration(
borderRadius: const BorderRadius.all(
Radius.circular(20),
),
color: Colors.white,
border: Border.all(color: Colors.grey.shade300, width: 1)),
disabledDropdownDecoration: BoxDecoration(
borderRadius: BorderRadius.all(Radius.circular(10)),
color: Colors.grey.shade300,
border: Border.all(color: Colors.grey.shade300, width: 1)),
            ///placeholders for dropdown search field
            countrySearchPlaceholder: "Country",
            stateSearchPlaceholder: "State",
            citySearchPlaceholder: "City",

            ///labels for dropdown
            countryDropdownLabel: "Country",
            stateDropdownLabel: "State",
            cityDropdownLabel: "City",

            ///Country Filter [OPTIONAL PARAMETER]
            countryFilter: [
              CscCountry.India,
              CscCountry.United_States,
              CscCountry.Canada
            ],

            ///selected item style [OPTIONAL PARAMETER]
            selectedItemStyle: TextStyle(
              color: Colors.black,
              fontSize: 14,
            ),

            ///DropdownDialog Heading style [OPTIONAL PARAMETER]
            dropdownHeadingStyle: TextStyle(
                color: Colors.black, fontSize: 17, fontWeight: FontWeight.bold),

            ///DropdownDialog Item style [OPTIONAL PARAMETER]
            dropdownItemStyle: TextStyle(
              color: Colors.black,
              fontSize: 14,
            ),

            dropdownDialogRadius: 10.0,

            ///Search bar radius [OPTIONAL PARAMETER]
            searchBarRadius: 10.0,

            ///triggers once country selected in dropdown
            onCountryChanged: (value) {
              setState(() {
                ///store value in country variable
                countryValue = value;
              });
            },

            ///triggers once state selected in dropdown
            onStateChanged: (value) {
              setState(() {
                ///store value in state variable
                stateValue = value!;
              });
            },

            ///triggers once city selected in dropdown
            onCityChanged: (value) {
              setState(() {
                ///store value in city variable
                cityValue = value!;
              });
            },
          ),
          TextButton(
              onPressed: () {
                setState(() {
                  address = "$cityValue, $stateValue, $countryValue";
                });
              },
              child: Text("Print Data")),
          Text(address)
        ],
      ),
    ),
  ),
);

To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
I must be able to select the cities and states as well after i select on the country
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Is the problem solved yet as still it is not working

I'm also facing the same issue... after selecting country there is no dropdown for state and city!!

any solution??

The value of stateValue and cityValue are nullables and hence we need to use null-aware operation and assign a default value. Replace value with value ?? "".
Example:
onStateChanged: (value) {
setState(() {
///store value in state variable
stateValue = value ?? "";
});
},


Gaussian process for tracking and bayesian optimisation for sensor managment

Code from the University of Sheffield.

A couple comments:

there are multiple hundred FLAKE8 formatting errors that need fixing (my rough estimate is close to 1000). See https://app.circleci.com/pipelines/github/dstl/Stone-Soup/3275/workflows/e8f186bb-11fe-4392-939e-1982b08aa166/jobs/17963/tests
Some of the file names need changing. Examples such as "visualization_2" are not descriptive as to what is happening. There is also a typo in another file name.
The tutorial is 23,00 lines long which is too big to even render on GitHub and bigger than all the tutorials combined. It is also in the same directory as the introductory tutorials and is not in the correct python format. You have a jupyter notebook but they are .py sphinx files.

Since this PR is a long way from being ready for a formal review, I recommend getting a couple of files at a time into the correct format for Stone Soup so the docs successfully build locally and all tests pass. Ensure the files have appropriate names and are in the correct directories/sub directories. Make a PR for those few files and then repeat so there are multiple, smaller PRs which can be more easily reviewed.


New blank project failed to start scafolding

Not sure if going back a version would do the trick
PS C:\Users\emmanuel.auffray\Documents\Personal\ionic\recorder> npm install -g cordova
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
C:\Users\emmanuel.auffray\AppData\Roaming\npm\cordova -> C:\Users\emmanuel.auffray\AppData\Roaming\npm\node_modules\cordova\bin\cordova
C:\Users\emmanuel.auffray\AppData\Roaming\npm
`--<EMAIL_ADDRESS>PS C:\Users\emmanuel.auffray\Documents\Personal\ionic\recorder> npm install -g ionic@beta
C:\Users\emmanuel.auffray\AppData\Roaming\npm\ionic -> C:\Users\emmanuel.auffray\AppData\Roaming\npm\node_modules\ionic\bin\ionic
C:\Users\emmanuel.auffray\AppData\Roaming\npm
`--<EMAIL_ADDRESS>PS C:\Users\emmanuel.auffray\Documents\Personal\ionic\recorder> ionic start ionic2-recorder blank --v2
Directory already exists: C:\Users\emmanuel.auffray\Documents\Personal\ionic\recorder\ionic2-recorder
Would you like to overwrite the directory with this new project?
(yes/no): yes
yes
Creating Ionic app in folder C:\Users\emmanuel.auffray\Documents\Personal\ionic\recorder\ionic2-recorder based on blank project
Downloading: https://github.com/driftyco/ionic2-app-base/archive/master.zip
[=============================]  100%  0.0s
Downloading: https://github.com/driftyco/ionic2-starter-blank/archive/master.zip
[=============================]  100%  0.0s
Installing npm packages...
Error with start undefined
Error Initializing app: There was an error with the spawned command: npminstall
There was an error with the spawned command: npminstall
Caught exception:
undefined
Mind letting us know? https://github.com/driftyco/ionic-cli/issues

Fixed by upgrading to node 6.5

@ManuInNZ After upgrading still same error
Ionic<EMAIL_ADDRESS>Node: 6.5.0


feat: Provide support for explicitly pausing autoscaling of ScaledJobs

Introduce paused annotation for scaledJobs
Checklist

[x] When introducing a new scaler, I agree with the scaling governance policy
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added
[x] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[x] A PR is opened to update the documentation on (repo) (if applicable)
[x] Changelog has been updated and is aligned with our changelog requirements

Fixes #3303

We are going to release KEDA v2.9 on Thursday. Do you think you can complete the open work by Wednesday @keegancwinchester?

@tomkerkhove apologies for late reply. will have this done by end of week

Any update on this?

What is the current status of this PR? I thought it would be in 2.10.0

Hi @keegancwinchester
Are you still willing to contribute with this? We can unassign the issue if you want

When can it be expected? It would be advantageous to have this functionality.

Hey, any update on this? My org is adopting Keda for it's k8s environments and this is a key piece of functionality for deploying.
I am keen to pick up the work if needed.

I think that this PR is abandoned :(
There hasn't been any answer for 2 months, so I'll close it. @flux-benj go ahead if you are willing to do it


Device provision service in React Native.

I want to do Device provision for Azure IOT hub in React Native.

Device provision with symmetric key
Send telemetry message
Device provision with certificate and key

How can I do that with React Native using SDK.

We have a basic example of using an SDK with React Native here: https://github.com/Azure/azure-sdk-for-js/tree/main/samples/frameworks/react-native/appconfigBasic
Is there a specific issue you are running into with IOT Hub, @Shahanshah-TA ?

Hi @xirzec I am looking for equivalent of x509 register functionality from azure-iot-sdk-node in sdk-for-js to use it in React Native but I am not able to find it, as I can not use azure-iot-sdk-node

Hi @xirzec any update on this?

@Shahanshah-TA I'm confused about what package you are using that you are having trouble with. I see there was a previous issue opened here: https://github.com/Azure/azure-sdk-for-js/issues/29142
Is there an @azure/ package in this repository you need help with? We cannot provide support for packages outside of this repository.

Hi @xirzec I was initially using azure-iot-sdk-node and I found out that azure-iot-sdk-node is specifically for node and doesn't support React Native but later I found out that Azure-sdk-for-js support React Native, so I switched Azure-sdk-for-js only. But now I am not able to find the functions which I need.
Requirement:
My requirement is to Provision a device on IOT hub using symmetric key, after registration send telemetry message to IOT hub assigned, and Provision device with x509 certificate with React Native.
For reference:
Provision with symmetric key: https://learn.microsoft.com/en-us/rest/api/iot-dps/device/runtime-registration/register-device?view=rest-iot-dps-device-2021-10-01&viewFallbackFrom=rest-iot-dps-device-2021-06-01
Send telemetry message with MQTT: https://learn.microsoft.com/en-us/azure/iot/iot-mqtt-connect-to-iot-hub
Provision device using x509 certificate: https://learn.microsoft.com/en-us/azure/iot-dps/how-to-control-access#certificate-based-authentication

@Shahanshah-TA as far as I know, we only have one management package related to iot hub in this repo https://github.com/Azure/azure-sdk-for-js/blob/ea4c874241b10e2bd72ea864ae362e14379d7e2c/sdk/iothub/arm-iothub/README.md#L9.
You probably would need to ask for help in the azure-iot-sdk-node repo.

Hi @jeremymeng
Thank you for response
azure-iot-sdk-node doesn't have React Native or browser support, anyway thanks for response.
If I would have even 1% chance to switch to any other platform like AWS, I would have done that because documentation of Azure is so so bad that I can't even explain, you guys doesn't have browser supported package, you guys doesn't have iOS supported package. And luckily if you will find any package which support browser or iOS then you will not find all the functionality needed.


Add Build Step for llhttp

Adds a simple step that clones the llhttp repo, builds it into wasm, turns it into b64, and then cleans up
Note
requires this kaito llhttp pr to be merged before this will work (since the build step will fail otherwise)

COOOOOOOOOOOOOOOOKKKKKKKKKKKKK 🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳🧑🍳