Quick draft "Result::expect" rfc

See also https://github.com/rust-lang/rust/pull/25359 for implementation
Rendered

Seems logical to me. After you get to know .expect() on Option, you'll expect to find the method on Result too.

@rust-lang/libs This needs a sheriff

This would be great for quick and dirty scripts. It would also make one of the first examples in the rust book easier to read:
   io::stdin().read_line(&mut guess)
        .ok()
        .expect("Failed to read line");

https://doc.rust-lang.org/book/guessing-game.html

I think .expect("Failed to read line") mean to be "expect to Fail..." which is logically inverse to what it is done. I think .on_error("Failed to read line") should be more logical. Am I the only one thinking this way?...

@grissiom Possibly, this RFC just brings the avaliable methods into line with Option's methods.

@grissiom, I completely agree. I think this should at least be discussed before accepting this RFC so we'd only have to deprecate one method, not two, if this were accepted.

@grissiom @gsingh93 The .expect() method is part of the stable API for Option, and thus cannot be changed without incrementing the language's major version.

Yes, but it can be deprecated.

@gsingh93 @thepowersgang OK, when I have time, I will open an other RFC to discuss Option::expect and reference this RFC. Thanks.

This RFC is now entering the week-long final comment period.

I still REALLY think that this should take msg: Display, but that can be added after the fact. I'm happy with this RFC in its entirety (including the format string).

I don't think the point about Debug formatting for errors being inferior to Display has been addressed.
Which of these are more helpful?
Error { repr: Os(2) } vs No such file or directory (os error 2)
JoinPathsError { inner: JoinPathsError } vs path segment contains separator ':'
Maybe this issue should be raised elsewhere though because it's not the problem with expect per se.

Maybe this issue should be raised elsewhere though because it's not a problem with expect per se.

I suspect this is the right way to think about it. For better or worse, Option::unwrap, Option::expect, Result::unwrap and Result::unwrap_err are all bounded with Debug. Making Result::expect use Display instead seems pretty surprising to me.

The text as written now is a bit unclear. I'd really like it to explicitly provide the signature and implementation (since it should be trivial to provide, and is the entire point of the RFC).

@gkoz That should be raised somewhere, probably on the offending error type, making Debug include the error text as well as the number.

Maybe this issue should be raised elsewhere though because it's not a problem with expect per se.

Agreed. We could change the conventions for error types to have more informative Debug output, or, when specialization is implemented, use Display output for objects implementing Debug+Display and Debug output for objects only implementing Debug as a couple of options.

@burntsushi: I think the cases are different. The Debug bound is on the value inside the Result. The Display bound would be on the message you pass to .expect(). Display is a (natural?) generalization of passing just a &str for a message. Debug is not a generalization of passing a string at all.
Also, we can demand a rarer trait like Display when the user must conciously choose to pass in a fitting message. We need Debug for the other cases to be able to work with a broader range of values contained in the Result.

@bluss but that reply was about the value inside the Result, not on the message.

@gkoz Oops! I'm sorry for the noise.

The consensus among the libs team is to accept this RFC now. The matter of generalizing the argument of expect to T: Display can be handled at a later date (and should also affect Option as well). Additionally, using Debug instead of Display for the error type in the result follows the precedent of the unwrap method, which while without downsides covers the most possible types today.
Thanks for the RFC @thepowersgang!


Feature/spa 1394

 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.

I wrote some doc about setting the "Add to home screen" in a blank application with steps. The full setup is explained (even the changes in angular.json).
I would create new issue for integrating ServiceWorkerModule into our StorefrontModule. It is not that easy. Moving it from this place to StorefrontModule doesn't work and I need some time to figure it out.
Also changed the icons in our demo app :)
Let me know what you think @LTiger14

Docs proposal:
Configuring PWA features
One of the project goal is to make good use of PWA features in modern browsers for even better user experience.
We are currently looking for the best approach to implement this features with good defaults and easy way to customize.
While we work on better approach you can already leverage "Add to home screen" feature.

Create and configure manifest.json file beside your index.html (example: projects/storefrontapp/src/manifest.json).
Provide icons for you site. You can find more about configuring manifest.json on Google Developers site

Don't forget to add relative paths for manifest.json file and other files referenced in it to assets section (projects>name-of-the-project>architect>build>options>assets) of your angular.json file.

Reference manifest.json file in index.html file
<link rel="manifest" href="manifest.json">

Create ngsw-config.json file beside index.html and configure it. (example with good defaults: projects/storefrontlib/src/lib/pwa/ngsw-config.json)

Set serviceWorker: true in configurations of builder in angular.json (example: angular.json)

Set "ngswConfigPath": "relative path to ngsw-config.json" in options of builder in angular.json (example: angular.json)

Import ServiceWorkerModule from @angular/service-worker in project entry module.

Add to your projects entry module imports
ServiceWorkerModule.register('/ngsw-worker.js', {
  enabled: true
}),

Set pwa/addToHomeScreen config to true in StorefrontModule configuration
StorefrontModule.withConfig({
  server: {
    baseUrl: 'https://my-custom-backend-url:8080'
  },
  pwa: {
    addToHomeScreen: true
  }
}),

Build your app

Visit created app and in mobile menu you should be able to see Add to home screen button. Clicking it will invoke "Add to home screen" prompt.

Created GH-279 to update the documentation with the input provided above. This ticket can now be moved to QA.
@marlass Excellent doc input, by the way! Thank you! 😄


Bug fix: Admin changes are not reflecting in proxy

Summary
In case of admin ip changed from default.
The admin api changes are not being pushed to serf.
As the default ip (<IP_ADDRESS>) is hardcoded in the script.
Full changelog

Set the new admin ip if changed from default.

Issues resolved
Fix #1509
Setting the admin IP if changed from default.
Fix for issue: https://github.com/Mashape/kong/issues/1509

@dheerajdwivedi can you rebase on the next branch?

Done !
Message while creating pull request is confusing.
**Note: new features are to be opened against next, hotfixes against master.**

One last thing: can you use double quotes " instead of single quotes ' ? @dheerajdwivedi

Done.
But build failed, is this usual failure ?
[  FAILED  ] 1 test, listed below:
[  FAILED  ] spec/02-integration/01-cmd/02-start_stop_spec.lua @ 156: kong start/stop dnsmasq recovers from expired dnsmasq.pid file

@thefosk Build failure is inconsistent, there has not been any change in logic only cosmetics changes are done after last successful build.
Build environment is not stable.

Please re-run the build.

@dheerajdwivedi it's okay - those errors are related to something else.


8.2 as requirement?

Hi, just wondering what the reasoning is for version 4 and beyond requiring PHP 8.2 as minimum given that it only had a general release a few months ago and the majority of applications won't be moving to it for at least another year or so. Those versions seem to add a lot of good features and crucial bug fixes, so it seems a shame not to have these available to the majority of PHP applications just for the PHP 8.2 requirement.

i belive the correct php version is 7.4 looking the code

Sorry for the late reply, you might use this library with earlier versions of PHP such as 7.4 but due to further development, compatibility is not guaranteed.


https://github.com/SAP/open-ux-tools/issues/673

Fix for: https://github.com/SAP/open-ux-tools/issues/673

Fix looks good, however, I don't know how to properly test it. Any idea how to put this in a unit test?
If not, I am also ok with it.

Possibly by mounting the test fs to include a path with a dot. Will create a follow up PR if thats ok since this fix is needed as priority?

@IainSAP Does this need to be applied to all copyTpl calls to be safe?

I dont know as that requires more extensive testing and the fix is urgent. The bug is fixed by this PR for now. A follow-up may be required with additional testing.

Follow up item created: https://github.com/SAP/open-ux-tools/issues/675


consider simplifying PROMPT_COMMAND handling

We should investigate how starship can get away with just doing this
https://github.com/starship/starship/blob/03343272b778260016216266facd190936f9e7d3/src/init/starship.bash#L53
meanwhile we have
https://github.com/microsoft/vscode/blob/4dc90628dfbcfa76b629399b75bef191bd0af2f1/src/vs/workbench/contrib/terminal/browser/media/shellIntegration-bash.sh#L128-L143

don't think they support PROMPT_COMMAND of the formats that we do


Compatibility with file-loader 3.0.0

Version 1.1.2 when used with file-loader version 3.0.0 causes the following failure:

Module build failed (from ./node_modules/url-loader/dist/cjs.js):
ValidationError: File Loader Invalid Options
options should NOT have additional properties
at validateOptions (node_modules/schema-utils/src/validateOptions.js:32:11)
at Object.loader (node_modules/file-loader/dist/index.js:21:28)
at Object.loader (node_modules/url-loader/dist/index.js:76:19)

I believe that's because url-loader passes down its own options to file-loader. In my case removing limit: 8192 from options fixed the issue.

Already fixed in master and release will be near future

At version 5.x, url-loader is using esModules however file-loader is using esModule. Is that a mistake or something else?

Here is the commit from file-loader 9eada2d06

@fwh1990 i am working on new major release for url-loader, they will have esModule option


Fix tests

Hopefully with this PR we will make all tests pass
Depends on #81

Rutroh. I think I accidentally committed a WIP branch to master? Let me back track later today to make sure your fixes are necessary.

See #81, I had a bad commit of a WIP branch to master.

@zmully yea looks like a WIP got merged.
IMO would be good to see the fixes on a PR like this, instead of pushing revert commits directly to master.

@rodowi I'm not going to have time today to revert this, so if you want to do it, please go ahead. The commit I meant to merge was the lambda-cfn version update in package.json.

@zmully lambda-cfn upgrade? the latest change in package.json is this https://github.com/mapbox/patrol-rules-aws/commit/a7e4182f6f1134afb7fe7abf07762fa077077895, but I can't find the WIP you mentioned here: https://github.com/mapbox/patrol-rules-aws/pull/82#issuecomment-224587611
what about ../lib/policyProcessor? that file is missing in the repo and is invoked by two rules.

@rodowi  revert master to 2889d898bd43673ef21b9e0617b449b948926a78, then apply only the change to the package.json in a7e4182. Everything else is from a local branch I've not yet pushed.

@zmully gotcha, I'll introduce that revert commit in this same branch so we leave a mark in the history about this PR.

Well, if I hadn't bungled the commit, this PR wouldn't be necessary, so I'm not sure what we're preserving here unless I've missed something in this PR that was broken before my bad commit.

Well, if I hadn't bungled the commit, this PR wouldn't be necessary

true, I'm closing this PR and opening a new one with just the revert action and applying only the lambda-cfn upgrade, makes sense?

:+1:


Support health check for harbor bosh release

Add health check logic and scripts

No efforts to cover this item, move it to  later sprint

Add status checking via cron.

check status of all harbor containers
check the api endpoint of systeminfo
check docker login

Done, close this issue


update to node22

Description
Starting to see some packages throughout the ecosystem move over to node22. Instead of having to polyfill any of the new functions that exist in it we should just bump our node version so we're already ready for when node18 support gets dropped in April

netlify doesn't have full support for 22 yet so going to leave this hanging around


Remove Google fonts from installer

During install, I see in uBlock Origin that the domain gstatic.com is contacted for Google fonts. Perhaps third-party fonts are not needed, and can be removed, to respect user privacy?

Noto is the official Drupal branding font. We can bundle it locally rather than using a third party site.

I did check drupal.org, and didn't see Noto, but you probably have a point, that external sites may need to use an official branding font.
Bundling the Noto Sans font would work fine, though I am not sure how to via Github (I only ever use the GUI on Github ...) but this web site is great for downloading, and getting the paths generated:
https://gwfh.mranftl.com/fonts/noto-sans
It looks like only these two numerical weights are used in the wasm-drupal.mglaman.dev/styles.css file, where bolder, as I understand it, is a relative value:

font-weight:500
font-weight:600
font-weight:bolder

I was looking up the new branding announced at DrupalCon Portland: https://drupal.widencollective.com/portals/gfvztttq/BrandPortal
Thanks for identifying the specific weights, I'll add the files locally to the app

You're welcome, and thanks for the link to the Brand Portal, I see they even have a Noto Sans font pack. Very nice.
Including these fonts should do it, I would think. I'll try to verify it, when a new version is ready for testing, regarding fonts.

regular
italic
500
500 italic
600
600 italic

Once https://github.com/mglaman/wasm-drupal/actions/runs/9810831878 is done, it should be fixed

Perfect, I am getting Noto Sans, directly from the page, thanks!


Entity linking ?

Currently, I'm connecting Keys and Doors by having an integer field of the same value. are there plans to allow linking directly to another entity?

Yes, it's planned :) Related: #33


Awake - show current state in tooltip

Description of the new feature / enhancement
Awake has potential, however it is not yet usable for me and I keep going back to Caffeine.
It needs in my opinion:

show current state ([not] active + how long remaining) in the tooltip, maybe even with a different Icon (full/empty cup)

Scenario when this would be used?
I like to be able to see if is active looking at the icon and be able to see how long it will remain active by hoovering.
Supporting information
No response

Thank you for the suggestion, @miguelanjo - this is a good one!

Any update?

@Jay-O-Way working on this alongside the enhancement that will enable us to track the proper icon for the state.

This seems party/mostly a dupe of #11996, with the addition of a tooltip. @dend I'm glad it's on the list.

Hi, I would love to see the changing icon.
I believe that the icon should change even with the "unlimited awake" period.

Aiming to get this done in #32378


K9 dos not delete message from Server

Expected behavior
deleting a message on K9 shoudld elete the message from the server as is configured in the accounts Fetching mail options
Actual behavior
messages are not deleted from the server but remain there. the local message is deleted
This occurs only with my google account
Environment
K-9 Mail version: 5.203
Android version: 6.0.1
Account type (IMAP, POP3, WebDAV/Exchange):
IMAP

I don't know if this is opened in another issue, anyway:
Expected
Deleting a message should move it to [Gmail]/Trash folder
Actual
K-9 created [Imap]/Trash folder and moved the message there.
Environment
K-9 Mail 5.203
Android 6.0

Probably caused by #752

@nanuit: Please record a debug log while trying to delete a message. https://github.com/k9mail/k-9/wiki/LoggingErrors

k9-log.txt
This log should cover the deltetion process
The error does only occur with one of my gmail accounts. The other (gmail and others) are working fine.
When i delete a mail in my desktop mail client it is deleted in the server also

The log ends right before K-9 Mail is attempting to move the message to the Trash folder. The important part is missing.

the command "adb logcat -d -v time k9:V '*:S' AndroidRuntime:E > k9-log.txt" is stop writing at this point and exits.
But i found the problem: the trash Folder (also the other Span, ...) was set to Trash and there is no corresponding Label in my google account. I set it to [Google Mail]\Bin and now it works
Maybe it has to do with the fact, the this is a google apps account and not a gmail
In my Gmail Account there is the [Gmail]\Bin Folder set as Trash and there was no problem


making only direct children of slide block elements

All slide elements are becomming block elements, this is causing problems with elements like <i>, <a>, <span>, <sup>, etc.
I think just the direct children of a slide should be block elements and leave the grandchildren elements with their default displays.
Then i suggest the rule .crellyslider > .cs-slides > .cs-slide * {, at line 54, should be .crellyslider > .cs-slides > .cs-slide > * {.

The children are displayed as a block to be responsive. I'll look at the code and see if there's a way to change it without damaging the responsiveness feature.


DCD installation and spawning issues

There are a couple of issues with DCD:

[x] If the files are missing serve-d asks to install dcd itself, but there is no notification to the user to notify them if the installation is done. -> The solution to this is to bundle DCD binary like how we do for seve-d.
-> Fixed in #43

[ ] The connections are disposed, refused, or other similar errors. I had this issue in this comment, and I thought the issue is fixed by copying dll files, but after releasing a new version, I am getting similar errors again. Not sure what has changed with DCD!
-> Apparently Bypassed in #43 ??

[ ] When dcd does not work properly as described above, if Atom is closed, dcd-server continues to run in the background.
-> Apparently Bypassed in #43 ??

[ ] Sometimes DCD seems to be spawned, but the datatips, autocomplete, etc don't return anything like in this comment
The sent URI is like this:

"file:///C:/Users/aminy/Documents/GitHub/JavaScript/atom-ide-d/spec/fixtures/source/app.d"

[ ]  DCD uses a lot of CPU for some reason.

Also there is an issue with dub initialization when Dplug repo is opened.

@WebFreak001 Are you able to set up Atom and help me with these? These are related to the internals of serve-d, and I am not very familiar.
There is guide for development in CONTRIBUTING.md

OK. I found the issue with serve-d. Serve-d release files are built for win32 while my system is win64. When I build from the source it works, but when I use the release files from serve-d repository, they don't work!


Move PORTUS_PUMA_HOST export to an ENV variable

This commit moves PORTUS_PUMA_HOST variable set into the Dockerfile.
This way it can be easily configured at runtime.
Fixes #98

@mssola I think moving the variable to Dockerfile makes sense, however I am not convinced about the other exports form init. I think is just fine to have a fixed RACK and RAILS configuration to production.

Why would anyone want to change PORTUS_PUMA_HOST ? If you are worried about the port being taken, you can always do port mapping. And puma should always be on localhost in containerized deployment, so I don't see why someone would want to change this.

I am not convinced about the other exports form init

Well, the other environment variables are either implementation details (e.g. the gem path), or values that should never be changed (e.g. rails_env).
Also, note that one of the goals for this image is to have a minimal Dockerfile, so all the logic was moved into the init script.

And puma should always be on localhost in containerized deployments, so I don't see why someone would want to change this.

Well, agree that host should not be changed. I still think it could be handy setting the port, but you are right that there are simple and multiple options to handle ports.
Feel free to close this PR and close #98 issue if you prefer to leave it as is.
Thanks


docs(operator): document the automatic release upgrades

What this PR changes
This commit adds documentation on how to use the experimental capability to automatically upgrade the Helm chart releases.
It moves the documentation from our Helm chart’s README to the kubescape.io website, as discussed with Ben and David.

@vladklokun what is the status of this PR?


iOS DX: Automagically find packager's IP address

I've been thinking a lot about the pain of configuring packager's IP address on the iOS side of React Native project. This is option 1 of 2.
Seems like the most common flow is to use packager for simulator and device builds in development and pre-bundled file in production. The most reliable option of distinguishing development/production environment is inspecting DEBUG in Xcode, it is defined for "Debug" build configuration and not defined for "Release" (we already rely on this in a bunch of other places). I think it's safe to assume this is the case and add #ifdef DEBUG into generated AppDelegate.m by default. Important note here is that it doesn't hide this deep inside framework, just makes the most common case easier. Users can still rewrite  logic around jsCodeLocation to match their use case (e.g. CI builds).
The second part of this diff is about automatically detecting packager's IP address. There is a script that runs as a part of every Xcode build called react-native-xcode.sh. We can use it to store developer's computer IP address to a file inside the app. We only need to do this for development builds of RN app for device. I've included a small helper function to read the IP address from that text file. This function can be helpful for figuring out jsCodeLocation.
And the last part – this PR makes RCTWebSocketExecutor use the same host/port as the corresponding bridge bundle URL. This means that by only setting jsCodeLocation to point to correct location we can get Chrome debugger to work seamlessly without any additional hacks.
Let me know if you'd prefer me to split these into 3 chunks, but I believe getting these 3 changes in will improve DX on iOS a lot.
Test plan
Patched react-native-cli on my machine to point to my local react-native checkout, react-native init TestProject. Ran it on simulator and device, confirmed loading and Chrome debugging worked in both environments without andy additional configuration. Made sure pre-bundled file is used when Xcode "Build Configuration" is set to "Release".

Damn, we're also working on this. But we choose a different approach (pass host and port thru the xcodebuild command). Maybe it's worth to take a look on our implementation also? We'll compose a PR in an hour.

Well, seems we can't compose a PR right now :( We'll still make a proposal when we finish, but it definitely shouldn't block this PR.

Let's continue with #6362

Did you ended up with a solution to this. Connection to the packager is very unstable on<EMAIL_ADDRESS>It can work for days and then fails randomly, it happens often and it's quite frustrating. Here is the complete Xcode log.

Note:
While I was writing this message my phone finally found the packager. (?)
That was during the same run, I added the end of the log in case it helps.

I have tried so many things I can't even remember all of them, here is what I've done:

npm start  -> Running Metro Bundler on port 8081.
Simulator works (manage to get bundle).
pings phone <-> laptop ok.
DNS are set to <IP_ADDRESS> and <IP_ADDRESS> on both my laptop and my phone.
Phone in airplane mode.

I saw that you talked about a file "$DEST/packager-ip.txt", my react-native-xcode.sh file has a different destination for the IP:
DEST=$CONFIGURATION_BUILD_DIR/$UNLOCALIZED_RESOURCES_FOLDER_PATH

if [[ "$CONFIGURATION" = "Debug" && ! "$PLATFORM_NAME" == *simulator ]]; then
  IP=$(ipconfig getifaddr en0)
  if [ -z "$IP" ]; then
    IP=$(ifconfig | grep 'inet ' | grep -v ' 127.' | cut -d\   -f2  | awk 'NR==1{print $1}')
  fi

  echo "$IP" > "$DEST/ip.txt"
fi

The command used here do work on my bash and returns <IP_ADDRESS>.
But if I do find . -name "ip.txt" I don't find the ip file.
I don't know if it's related to this issue, but it happens from time to time that when I shake my phone, two menus are showing (still happening now that the app is running).
Another strange warning just appeared today (even though I didn't change any package in my react-native project): [warn][tid:main][RCTBridge.m:119] Class RCTCxxModule was not exported. Did you forget to use RCT_EXPORT_MODULE()?.
One last thing, my application is using web socket, and my app just crashed before and some socket may not have been closed properly (once again, I don't know if this has any influence but I prefer to add too much information in here, just in case).

I finally found the issue, it comes from my Firewall which for some reason started to block connection to my packager.
I realised that when tried to manually get the bundle from my phone web browser using the address http://<IP_ADDRESS>:8081/index.bundle?platform=ios&dev=true and it didn't work (timeout) even though it was working from my laptop.
I feel like this happens because of a more complex rule than "blocking a port" or blocking an application", indeed, I restarted my Firewall and relaunched the packager and now it works (in the mean time I also restarted my packager as super user, not sure if this had any effect tho).


Windows is not refreshed on linux

Hello
When a mail is received the windows content is not refreshed (at least on linux running oracle jdk 1.8).
Only way to force the refresh is to resize the window.
Looking at the code I see that the gui operations are not done in the dispatch thread as it should be from what I remember in swing, not sure how to fix this cleanly in the meantime I'm forcing a repaint of the mainpanel (ugly) but that fixes the issue.

Thanks, which window/desktop manager are you using?

Cinnamon on Linux Mint 17.2

Is this still an issue?  Out of curiosity I built this on a similar OS - Mint 17.3 (64 bit), Cinnamon 2.8.8, Java 1.8.0_92. Ran an integration test sending 5 emails, with and without delay.  The window seems to display emails as they're being processed; I'm not sure if I'm duplicating the scene.

To be honest I only run my forked version with my fix ...
But in any case I believe your code is incorrect because as I said you're not doing the repaint operations in the event dispatch thread.

you can have a look to   https://github.com/ymartineau/peers/pull/13


No scrollbar in the items list.

Many changes was occured, but i can't understand this. Yes, here exist search by name, but it's not always convenient. Plus now i should scroll trough all block parts (pistons in all positions, water of all levels, etc.). Founded in version <IP_ADDRESS> and <IP_ADDRESS> (just not check earlier)

I don't think there was ever a physical scrollbar, but if it helps page up/down scroll it a lot faster.

Right, there never was any scroll bars, and I confess this feature is (still) missing...

Done in commit 9b9b8f88b5cab516b883c6656a3eb3cc5bcf575b!

That's right, but problem is here again.
1st - I cannot grab scrollbar by mouse in blocks/items list, he's just too small.
and 2nd - he moves with little delay, but this is secondary.


Chore: Fix typo in migrating-to-8.0.0.md

s/Three new rules/Four new rules/

Thanks. This will only fix the guide on the live website, can you also submit a pull request for: https://github.com/eslint/eslint/edit/master/docs/user-guide/migrating-to-8.0.0.md

https://github.com/eslint/eslint/pull/14985


Stats are not accurately displayed.

Hello, a few people have been complaining recently that the stats dont seem to make sense.
When looking at the database, everything is fine, so can only assume its something to do with how the stats are being read and summed up on the front end.
EG we have some players where it is reported that they have broken no stone blocks, but yet still have 250+ iron ore breaks. Looking inthe DB we can see that stone breaks have been recorded.
**** putting this here as a memory jog for me  - Another admin on my server has suggested that possibly we should sum up the block break values before grouping them.   -  I will look into this when i have some spare time ****

This query seems to be the issue:
SELECT *, Stats_blocks_broken.name as 'name', Stats_blocks_broken.value as 'broken', Stats_blocks_placed.value as 'placed' FROM Stats_blocks_broken INNER JOIN Stats_blocks_placed on Stats_blocks_broken.uuid = Stats_blocks_placed.uuid and Stats_blocks_broken.name = Stats_blocks_placed.name and Stats_blocks_broken.world = Stats_blocks_placed.world	WHERE Stats_blocks_broken.uuid = 'UUID' GROUP BY Stats_blocks_broken.name, Stats_blocks_broken.world;
Comparing above with the below shows missing blocks being returned by the above statement
select * from Stats_blocks_placed where uuid = 'something'

removing the INNERJOIN allows me to see blocks broken with values that i would expect (obviously doesn't bring back any placed blocks though).

This is actually rather difficult as MySQL doesn't support full joins. Due to a player sometimes only placing a block but not breaking it, it's even more difficult as Lolmewnstats only registers stats for blocks which have been broken/placed by that player.
Using two separate tables would be much easier for now but it wouldn't be as nice as having just one table. All help to fix this would be appreiciated

LEFT JOIN seems to get what we want, but returns null values instead of 0.
I'm not sure if it introduces any other issues though.

SELECT *, Stats_blocks_broken.name as 'name', Stats_blocks_broken.value as 'broken', Stats_blocks_placed.value as 'placed' FROM Stats_blocks_broken LEFT JOIN Stats_blocks_placed on Stats_blocks_broken.uuid = Stats_blocks_placed.uuid and Stats_blocks_broken.name = Stats_blocks_placed.name and Stats_blocks_broken.world = Stats_blocks_placed.world	WHERE Stats_blocks_broken.uuid = 'UUID' GROUP BY Stats_blocks_broken.name, Stats_blocks_broken.world;

Handles NULLS:
SELECT Stats_blocks_broken.world, Stats_blocks_broken.name, Stats_blocks_broken.value as 'broken', IFNULL(Stats_blocks_placed.value, 0) as 'placed' FROM Stats_blocks_broken LEFT JOIN Stats_blocks_placed on Stats_blocks_broken.uuid = Stats_blocks_placed.uuid and Stats_blocks_broken.world = Stats_blocks_placed.world and Stats_blocks_broken.name = Stats_blocks_placed.name WHERE Stats_blocks_broken.uuid = '1d439936-b7b8-4812-8080-5235eb224291' GROUP BY Stats_blocks_broken.name, Stats_blocks_broken.world;

I guess the issue with this is that if a block has been placed, but not broken, it is missed by the query.

What about
SELECT Stats_blocks_broken.world, Stats_blocks_broken.name, Stats_blocks_broken.value as 'broken', IFNULL(Stats_blocks_placed.value, 0) as 'placed' FROM Stats_blocks_broken LEFT OUTER JOIN Stats_blocks_placed on Stats_blocks_broken.uuid = Stats_blocks_placed.uuid and Stats_blocks_broken.world = Stats_blocks_placed.world and Stats_blocks_broken.name = Stats_blocks_placed.name WHERE Stats_blocks_broken.uuid = 'UUID' GROUP BY Stats_blocks_broken.name, Stats_blocks_broken.world union all SELECT Stats_blocks_broken.world, Stats_blocks_broken.name, Stats_blocks_broken.value as 'broken', IFNULL(Stats_blocks_placed.value, 0) as 'placed' FROM Stats_blocks_broken RIGHT OUTER JOIN Stats_blocks_placed on Stats_blocks_broken.uuid = Stats_blocks_placed.uuid and Stats_blocks_broken.world = Stats_blocks_placed.world and Stats_blocks_broken.name = Stats_blocks_placed.name WHERE Stats_blocks_broken.uuid = 'UUID' GROUP BY Stats_blocks_broken.name, Stats_blocks_broken.world;

How about something to the tune of:
SELECT a.id, a.world, a.name as 'name', IFNULL(a.value, 0) as 'broken', IFNULL(b.value, 0) as 'placed' FROM Stats_blocks_broken as a LEFT OUTER JOIN Stats_blocks_placed as b on a.uuid = b.uuid and a.world = b.world and a.name = b.name WHERE a.uuid = 'UUID' GROUP BY a.name, a.world union all SELECT c.id, c.world, c.name as 'name', IFNULL(c.value, 0) as 'broken', IFNULL(d.value, 0) as 'placed' FROM Stats_blocks_broken as c RIGHT OUTER JOIN Stats_blocks_placed as d on c.uuid = d.uuid and c.world = d.world and c.name = d.name WHERE c.uuid = 'UUID' and c.name is null GROUP BY c.name, c.world;

$sql = "SELECT a.world, 					a.name as name, 					ifnull(a.value, 0) as 'broken', 					ifnull(b.value, 0) as 'placed' 				FROM {$this->prefix}blocks_broken as a 				LEFT OUTER JOIN {$this->prefix}blocks_placedas b on 					a.uuid = b.uuid 					and a.name = b.name 					and a.world = b.world 				WHERE a.uuid = ? 				GROUP BY a.name, a.world 				UNION ALL 				SELECT c.world, 					c.name as name, 					ifnull(c.value, 0) as 'broken', 					ifnull(d.value, 0) as 'placed' 				FROM {$this->prefix}blocks_broken as c 				RIGHT OUTER JOIN{$this->prefix}blocks_placed as d on 				c.uuid = d.uuid 					and c.name = d.name 					and c.world = d.world 				WHERE c.uuid = ? and c.value is null 				GROUP BY c.name, c.world";         if ($stmt->prepare($sql)) {             $stmt->bind_param("ss", $player,$player);

OK yes, i agree, this is quite an annoying problem.

I think this might be working, but ive been wrong several time tonight already!
$stmt = $this->mysqli->stmt_init();         $sql = "SELECT a.world, 					a.name as name, 					ifnull(a.value, 0) as 'broken', 					ifnull(b.value, 0) as 'placed' 				FROM {$this->prefix}blocks_broken as a 				LEFT OUTER JOIN{$this->prefix}blocks_placedas b on 					a.uuid = b.uuid 					and a.name = b.name 					and a.world = b.world 				WHERE a.uuid = ? 				UNION ALL 				SELECT d.world, 					d.name as name, 					ifnull(c.value, 0) as 'broken', 					ifnull(d.value, 0) as 'placed' 				FROM {$this->prefix}blocks_broken as c 				RIGHT OUTER JOIN{$this->prefix}blocks_placed as d on 				c.uuid = d.uuid 					and c.name = d.name 					and c.world = d.world 				WHERE d.uuid = ? and c.value is null 				GROUP BY d.name, d.world";         if ($stmt->prepare($sql)) {             $stmt->bind_param("ss", $player,$player);

Thanks @smc97. That query seamed to work flawlessly!

:-)


View returns to home after reload due to being offline

If my computer is offline and goes back online, Zulip reloads. However, instead of returning to the narrow I was looking at before, it shows me the home view. The original narrow shows up in the URL as oldhash but this seems not to do anything. I'm reasonably sure this wasn't happening ~6 months ago.
Observed on recurse.zulipchat.com. Example URL:
https://recurse.zulipchat.com/#reload:send_after_reload=0+csrf_token=<deleted>+oldhash=narrow%2Fis%2Fmentioned

I wind up at the home view instead of looking at my mentions as I was before the reload. This also makes opening topics in new tabs kind of useless.

Hrm, definitely sounds like a regression.  @brockwhittaker I'm guessing this was introduced by your recent work on hashchange.js; can you investigate?

I figured this out today; turns out to be a complex consequence of our upgrading Django a few months ago.  Sigh.  Anyway, should be closed shortly.


Multi-Column Per Line

Describe the solution you'd like
It would be helpful that it can show, say, 2 column of selected files side by side:
File           File
file#1       file#2
file#3       file#4
file#5       file#6

Can you provide an example with some other rows? I do not understand the request.

Do you mean the metadata or anything?
Now the Dataview can just show one sample per row:
File     Modified Time
file#1    time#1
file#2    time#2
file#3    time#3
file#4    time#4

Is there any way to change to two samples per row:
File     Modified Time    File     Modified Time
file#1    time#1        file#2    time#2
file#3    time#3        file#4    time#4

You can achieve this using DataviewJS and manually splitting the result list as desired to make as many columns as you want; for regular Dataview, I think I would need to add some extra context to GROUP BY to enable this specific behavior (i.e., allow grouping on indices and then group on index / 2).


refactor: change function calls when arity is one

When a function only takes one parameter (f.ex. (string) => capitalize(string)), you can replace it with capitalize completely, achieving a point-free function call and a less verbose code.
Where applicable (function only takes one parameter and is no built-in function), I've changed the calls.

Codecov Report

Merging #3563 into dev will decrease coverage by 0.53%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #3563      +/-   ##
==========================================
- Coverage   99.19%   98.65%   -0.54%
==========================================
  Files          18       18
  Lines        1116     1115       -1
  Branches      300      300
==========================================
- Hits         1107     1100       -7
- Misses          9       13       +4
- Partials        0        2       +2

Impacted Files
Coverage Δ

lib/common/options.js
100% <ø> (ø)
:arrow_up:

lib/builder/builder.js
96.7% <100%> (-3.3%)
:arrow_down:

lib/common/utils.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@manniL There is some problems with this pr, it may change the context of some member methods in class.
class A {
  constructor() {
    this.c = 'I amm c';
  }
  b() {
	console.log(this.c)
  }
}

const obj = new a()
obj.b() // this is fine

const method = obj.b
method() // this has error

I'll revert some codes.


Publishing to live links to MSLearn

Docs Build status updates of commit b45350f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/index.yml
:white_check_mark:Succeeded
View

articles/welcome.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Turbo: improve docs for forms

Replace occurence of renderForm with the new handleForm method, and make use of HubInterface instead of PublisherInterface since I always get an error locally whenever I try to use PublisherInterface.

LGTM except my minor comment.

Careful, things just moved again to renderForm :) https://github.com/symfony/symfony/pull/41190

Hey @mercuryseries - could you update the docs again for the latest renderForm() version - symfony/symfony#41190
Thanks!

Hey @mercuryseries - could you update the docs again for the latest renderForm() version - symfony/symfony#41190
Thanks!

Done @weaverryan ✅

Thanks @mercuryseries! Apologies for the slow merge :)


Add tex2tast command

Changes

Add commander module as optional dependency.
Add tex2tast command.

It named from "TeX to textlint AST".

This command is a development utility. We can confirm the output of our plugin easily.
# Link current package if you want to tex2tast inside textlint-plugin-latex2e project.
$ npm link
$ npx tex2tast
Usage: tex2tast [OPTIONS] FILE

Options:
  -l, --location  enable location and range
  -r, --raw       include raw
  -h, --help      display help for command

A sample source is as follows.
\section{title}

sentence

$ npx tex2tast sample.tex
{
  "type": "Document",
  "children": [
    {
      "depth": 2,
      "type": "Header",
      "children": [
        {
          "type": "Str",
          "value": "title"
        }
      ]
    },
    {
      "type": "Html",
      "value": "\n\n"
    },
    {
      "type": "Paragraph",
      "children": [
        {
          "type": "Str",
          "value": "sentence"
        }
      ]
    }
  ]
}

NOTE
Current lockfileVersion in package-lock.json is 2. This change was introduced in 48cfeaa553afd15f879d1ca2aec204afe927440e .
@tani
Maybe, is the version of your npm command v7.x? npm@v7 uses lockfileVersion: 2.
https://blog.npmjs.org/post/626173315965468672/npm-v7-series-beta-release-and-semver-major.html
We can use either v6 or v7, but we want to have an agreement on which one to use.
I think that we should use v6 since the npm command installed on CI is v6 now.
https://github.com/textlint/textlint-plugin-latex2e/runs/1939827921?check_suite_focus=true#step:3:7
https://github.com/textlint/textlint-plugin-latex2e/runs/1939827904?check_suite_focus=true#step:3:8

This is the nice what I want! Yes you're right, I updated npm in the local environment. As you said, we should follow to the test environment.

After the upload to npm, we can use npx -p textlint-plugin-latex2e tex2tast <file>

I've tried, but I'm not sure if I'm doing it right.

Thank you for your effort.


Improve error message when extension has the wrong value type

Describe the bug
FHIR Server returns a vague error message when extension has the wrong value type
To Reproduce
With information about what we did:

define an extension with value type decimal and register it with the server; then
create and instance with an extension that has this url but of type string
invoke the server's $validate endpoint with this instance
Note : I am using FHIR Server 4.2.3.

Server response includes the below error messages
{
"severity": "error",
"code": "invariant",
"details": {
"text": "generated-communication-priority-sequence-1: Constraint violation: url = 'http://ibm.com/fhir/cdm/StructureDefinition/communication-priority-sequence'"
},
"expression": [
"Communication.extension[3]"
]
},
{
"severity": "error",
"code": "invariant",
"details": {
"text": "generated-ext-1: Extension must conform to definition 'http://ibm.com/fhir/wh-engagement/StructureDefinition/communication-priority-sequence'"
},
"expression": [
"Communication.extension[3]"
]
},
NOTE : Expecting a more useful error message.

I think there was something else going on here.
note how the definition url doesn't match the expected url:
http://ibm.com/fhir/wh-engagement/StructureDefinition/communication-priority-sequence
http://ibm.com/fhir/cdm/StructureDefinition/communication-priority-sequence
We ran our own test and while the error message isn't super friendly, I think its sufficient to point a technical user in the right direction in terms of what to check:
{
    "severity": "error",
    "code": "invariant",
    "details": {
        "text": "generated-us-core-ethnicity-3: Constraint violation: extension('text').where(value.as(string).exists()).exists()"
    },
    "expression": [
        "Patient.extension[0]"
    ]
}
{
    "severity": "error",
    "code": "invariant",
    "details": {
        "text": "generated-ext-1: Extension must conform to definition 'http://hl7.org/fhir/us/core/StructureDefinition/us-core-ethnicity'"
    },
    "expression": [
        "Patient.extension[0]"
    ]
}```


To be possible to turn on / off battery banks

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
Right clicking a battey bank, allow the player to turn on / off the battery bank. Sometimes you are going to be a long time away from your base and don't need the batteries to be used or charged, so you can conserve them.
Describe alternatives you've considered
For now, i just remove the batteries from the banks.
Additional context
None.

This is a good idea, I will take a look, shouldn't be too hard to implement.


void-variable EMACS27+ error

What did you expect to happen?
Help with error
What actually happened?
After upgrade to latest version doom I have that error
Rrror in private config: modules/editor/evil/init.el, (void-variable EMACS27+)

Checking Doom Emacs...
x There was an unexpected error
Message: Symbol's value as variable is void
Data: (void-variable . EMACS27+)
Backtrace:
(if EMACS27+ '(thread))
(append (if EMACS27+ '(thread)) '(tide timer-list transmission trashed typ
(cons 'tetris (append (if EMACS27+ '(thread)) '(tide timer-list transmissi
(cons '(term term ansi-term multi-term) (cons 'tetris (append (if EMACS27+
(cons 'tar-mode (cons '(term term ansi-term multi-term) (cons 'tetris (app
(cons 'tablist (cons 'tar-mode (cons '(term term ansi-term multi-term) (co
(cons 'speedbar (cons 'tablist (cons 'tar-mode (cons '(term term ansi-term
(cons 'sly (cons 'speedbar (cons 'tablist (cons 'tar-mode (cons '(term ter
(cons 'slime (cons 'sly (cons 'speedbar (cons 'tablist (cons 'tar-mode (co
(cons 'simple (cons 'slime (cons 'sly (cons 'speedbar (cons 'tablist (cons

Describe your attempts to resolve the issue
No response
Steps to reproduce

Open Emacs

System Information
https://pastebin.com/mBXb27RA

EMACS27+ no longer exists in Doom's codebase on the latest commit. The EMACS27+ instance it's complaining about was removed in https://github.com/hlissner/doom-emacs/commit/34fae1c01ce6800cb7ed8830173cc644988af3e8#diff-627b3102c690d40dd1464ed2851e1aa37c51579a3b5cee29dff1272c1aaf9045R214 3 weeks ago.
Clearly something is wrong with your installation, and judging from your module list, you appear to be upgrading from a very old version of Doom. Your best bet is to delete ~/.emacs.d/.local/ by hand and run doom sync to start from scratch.
As this is not a Doom issue I will close this.

It's not working I try to download fresh brnch and recreate.

Some modules don't run too.

Sounds like you're experiencing errors. Check *Messages* for warnings or errors (C-h e to open it).

No errors
Doom loaded 382 packages across 64 modules in 5.888s
Package cl is deprecated

I can't help without more information. Try asking on the Discord instead.
You could also try again from the start.

Delete ~/.emacs.d
Delete ~/.doom.d
git clone https://github.com/hlissner/doom-emacs ~/.emacs.d
~/.emacs.d/bin/doom install`

Please keep an eye out for errors during this process.
Because this works without problem on more than 12 test systems, this is clearly a local issue.

I don't know what was wrong. I recreate all emacs and now is OK.


Attempting to edit a date field with a cleared date results in invalid date error

What happened?
This first happened to me when attempting to edit a field with an existing date in the board editor view. Clicking away from the date field caused the date to go blank, and then attempting to add a new date did not add anything—this eventually broke the Table view and "Include Fields" selector on the board view, and caused the date associated with the note to be "Invalid Date."
With more digging, this seems to happen when editing a date that has been cleared, either manually or by clicking away in the board tab editor. When attempting to set a date from a date that has not been cleared, everything works fine.

When clicking away from a date field in the Board editor view, the date field is not cleared
When attempting to set a new date from a date that has been cleared, the date is set successfully

How can we reproduce it (as minimally and precisely as possible)?
In Table view

Create or use a note with a date field
Clear the date from the field
Attempt to set a new date

https://github.com/user-attachments/assets/cb07f1ae-f7e3-4bbc-af12-9422f19b6385
In Board view

Click on a note with a date field
Click on the date field, then click away - this clears the date
Attempt to set a new date
To reproduce Uncaught RangeError, include date field in card view

https://github.com/user-attachments/assets/0882e063-1e1c-4153-ba6e-3ebe2e02ff92
Anything else we need to know?
No response
Plugin version
1.17.4
Obsidian version
1.6.7
OS
macOS

Yeah I've just noticed it and pushed a fix in #931. Will deploy soon.

That's great to hear, thanks so much!


Arduino IDE delivers buggy version of this library

Hi, Arduino IDE library manager only knows about the buggy 2018 version of this library.  Thats the version tagged 0.1.0.
I presume that if you apply a tag to the latest (not buggy) commit, then that will be available by default through arduino IDE???

@DuncanAtISIS Thank you! I just remembered that I had to release the new version. Released v0.1.1
https://github.com/hideakitai/MCP4728/releases/tag/v0.1.1


Prefer find_by to send("find_by_#{foo}")

There was just one gotcha here - the tags controller was leveraging this
method to find categories by name, which just so happened to implement
the .find_by_name method. Since it's preferable to use find_by for
real columns, the tags controller can be updated to call what it
needs directly instead - Category.find_by_name.
Thanks @gtanzillo for helping me debug this.
@miq-bot add-label api, refactoring
@miq-bot assign @abellotti

👍 LGTM!!


Issue with registering the services is throwing boolean expression must not be null

Upgraded Flutter to 1.22.1.  However when I try to run
flutter pub run build_runner build --delete-conflicting-outputs

it is throwing an error
[SEVERE] injectable_generator:injectable_builder on lib/services/third_party_service.dart:

Failed assertion: boolean expression must not be null

third_party_service.dart file is similar to what is mentioned it the readme
import 'package:injectable/injectable.dart';
import 'package:stacked_services/stacked_services.dart';

@module
abstract class ThirdPartyService {
  @lazySingleton
  NavigationService get navigationService;

  @lazySingleton
  DialogService get dialogService;

  @lazySingleton
  SnackbarService get snackbarService;
}

info pertaining to pubspec.yaml
dependencies:
  get_it: ^4.0.4
  injectable: ^1.0.4

dev_dependencies:
  auto_route_generator:

  build_runner: ^1.10.3

  flutter_test:
    sdk: flutter

  injectable_generator:

  json_serializable: ^3.4.0

dependency_overrides:
  analyzer: '0.40.4'

Where is that error being thrown? can you see what it's trying to do. I haven't seen this.
Make sure you run flutter clean and then flutter pub upgrade.

I am no longer running into this issue with Flutter 1.22.2.  Closing this issue.


xfpga: fix impl. of ObjectGetSize

Syncing an object is only applicable to objects that are backed by sysfs
files (and not directories or containers). This fix ensures that
sync_object is called only for file objects.

Coverage decreased (-0.02%) to 79.989% when pulling aa70b81dd6660a98bfe06de8f9af1562e89a4196 on rrojo/sysobject_getsize into 842aecfb0ee07ab74d178ae0e4f19134329f3dd1 on master.


warning: already initialized constant JsonbAccessor::JAPerson::JAExtras

Hi! This gem is awesome!
I am getting, any idea?
/Users/gaston/.rvm/gems/ruby-2.2.1/gems/jsonb_accessor-0.3.0/lib/jsonb_accessor/class_builder.rb:40: warning: already initialized constant JsonbAccessor::JAPerson::JAExtras

/Users/gaston/.rvm/gems/ruby-2.2.1/gems/jsonb_accessor-0.3.0/lib/jsonb_accessor/class_builder.rb:40: warning: previous definition of JAExtras was here

This could be caused by having 2 nested fields that are both named extras like this:
class Foo < ActiveRecord::Base
  jsonb_accessor :jsonb_column_1, extras: { nested: :value }
  jsonb_accessor :jsonb_column_2, extras: { nested: :value }
end

Mmm I was using jsonb_accessor along side with store_accessor, was that?

I don't think so. Could you provide an example of the code that is causing this problem?

@imton Just checking in. A complete example would really help us diagnose this.

I had the same, but it was because I had miswritten the jsonb column name.

Closing this until a more detailed example is available.


Native Auctions Views

We're turning the martsy view into native code in prep for live integration. There are lots of small pieces, so let's break it down:

[x] Auction banner view #974

https://github.com/artsy/eigen/pull/1022
https://github.com/artsy/eigen/pull/1023

[x] Redesigned countdown view #1034

https://github.com/artsy/eigen/pull/1044

[ ] Refine UI & capabilities #971

#1059

[ ] Lot views (list & masonry) #972
[ ] Parallax-ish scrolling behaviour on banner #973
[ ] Show registration status of user #975
[ ] Native auction registration flow #997 (bumped)

Thought this was "due" Feb 15 (didn't think it'd take that long) but it's due Feb 1 instead. Bringing in @orta for some assistance. /cc @katarina

What does “due” mean? Like, it should be able to go live on Feb 1? Should it be in the store by Feb 1?

I had many of these updates slated in our milestone that passed on January 17th, which might not have been a clear way to flag the due date (due date being the day we submit to the store). I figured using milestones were a natural way to mark these due dates but if they are not let me know what system would be preferable.

I believe "due" meant "implemented", maybe also tested.

Is it fair to ask us to all come around a defined definition of due, perhaps ship (ship being when we submit to the store)? And tie the ship date to our milestone? Put all issues in the milestone that need to be shipped by that date...thoughts?

@katarinabatina Agreed, I think “due” should be the date that it can be submitted to the App Store (so developed, and tested).

:+1:

@katarinabatina With that definition, is the “due” date still Feb 1?

(@ashfurrow Once this discussion about the due date is completed, maybe summarise it and delete all the comments.)

@alloy yes, let's make this due Feb 1

Looks like we're tracking this in Trello too: https://trello.com/c/FDH2rT7J/241-eigen-auction-sale-page-updates
I think based on the Slack conversation we had that we were going to track these in Trello (one source of truth). If that makes sense, we should consider closing this issue.

@ashfurrow agreed we should close this and use the milestone as a marker for the card in trello.

OK, gonna close this then.

There needs to be a simple way to traverse these separate tickets up to their high-level requirements. I’m fine with these umbrella overviews moving to Trello cards, but right now (without remembering this ticket) I can’t easily do that, so please state on the top of each ticket that they are part of a Trello card and a link to the card.

Yeah, agreed. Some of that is on me, but to be honest I'm a bit tired of playing liaison between Trello and GitHub. The eigen team prefers using GitHub issues, and I personally prefer them to Trello cards as well. If it's OK with @devangt and @acjay I'd like to move to tracking auctions-stuff in GitHub milestones, which we can sync with a Trello card. This'd let the team have maximum visibility into what's going on while requiring the least overhead from me. Sound OK?

That’s fine with me. To be clear, I’m fine with the umbrella issue being in Trello or wherever, I just want to easily find it without having to remember anything :)

Nope, no one has said tracking her is bad, so I'm re-opening ¯\(ツ)/¯
Will go through and add details from PRs I've done in-between.

@ashfurrow @alloy I'm in support of staying in Github

Sorry for the delay. @acjay and I just chatted and agree with you @katarinabatina. Since the iOS team has all of their work in GitHub it makes sense to have all issues and tickets in relevant milestones in the eigen repo as you already have!
For each sprint we will have a high level placeholder on Auctions Trello to call out the high level iOS work (but note that will only be a placeholder and most work and progress will happen here in Git).
The only other thing we will track in Trello are show stoppers that pop up time and again so we can track timely issues to closure in the auction season.
Note: We are also working on synthesizing the thoughts around the project management workflow given some of our discussions and will call for a quick sync/update by mid week. So stay tuned.

👍👍👍👍👍

@ashfurrow Cool, thanks :+1:
@katarinabatina @devangt Great, thanks :+1:

#1175 should finish this up, waiting on confirmation from Orta and then we can close this and start design QA :tada:

Yep, only things we need in for shipping are, analytics + spotlight support

Closing this,


gocode.exe not working on Windows 10

Hi.
I am developing in go under Ubuntu but at home I wanted to give it a try on my home Windows machine. It seems everything is working properly except gocode.exe - running any of gocode.exe set, gocode.exe set autobild, ``gocode.exe set autobild true` nor the autocompletion for go work (no output at all for commands in console nor in console running as administrator). I tried running them in cmd.exe as well as in PowerShell (and PowerShell Administrator ISE) - no luck getting any output nor autocompletion (in Atom) to work.
Windows 10, 64 bit, Go 1.7 (after running go version in cmd got this output go version go1.7 windows/amd64).
One weird thing I have just now experienced:

I navigated to c:\gowork\github.com\nsf\gocode
I ran go build and go install
then after running gocode set I finally got some output
then I cd c:\gowork\ and gocode set is again not working... WTF?!

Is C:\gowork\bin in your PATH environment variable?

Of course it is. Running PATH in cmd returns me my PATH with c:\gowork\bin included.

I don't know here, hard to tell what's going on.

@nsf Same issue for me, Win 10 64 bit, Go 1.7.1. I can get auto completion to work occasionally but the performance is unreliable. No log with any gocode commands and have everything correct in my path.

I'll try it on windows 10 tomorrow.

Installed Go 1.7.1 on my windows 10 machine (official Go installer), it added GOROOT env var and modified PATH env var.
Installed Git 2.1.0 (official installer), asked it to expose itself on PATH.
Added GOPATH env var pointing to C:\waytogo.
go get github.com/nsf/gocode
go get github.com/nsf/godit

Tried editing files in godit, autocompletion works perfectly, haven't noticed any lags.
Could you guys try editing stuff in godit? It's a simple emacs-like text editor written by me, it works on windows just fine. Just open some random Go file in it (godit file.go), make sure it's saved on disk (otherwise gocode won't be triggered), and use Ctrl-x Ctrl-a for autocompletion. Ctrl-x Ctrl-c for exit.
If it works well in godit, then text editors are to blame. Godit can be seen as a reference gocode integration.

Oh, just wanted to add. My windows 10 has automatic updates disabled, hence its state may not match yours. So, there is a chance that some windows 10 update breaks things, but it seems unlikely.

I will try goedit later today if I have time. @shadyyx Removing a factor hopefully, don't have avast slowing down your gocode do you?

It works fine here on Windows 10 (using it with VSCode) but the performance is atrocious (a few seconds for code completion). Running gocode in debug mode I noticed that all packages are rebuilt every single time a request comes in o_O This is something I noticed with go build if you don't use -i.
@nsf any clues on what might be happening?

Gocode's "autobuild" feature is experimental and have never been tested properly. Maybe for some reason it fails on windows. Unless vscode forces it on, I recommend disabling it.

@nsf VSCode doesn't do anything special, it starts the process and exposes it on a TCP socket, then spawns a new process to ask for auto completion of the file at a specific cursor position.
How can I try to disable that feature to see if it helps?

Well, in this case autobuild should be disabled by default. You need to find the gocode binary and exec it via cmd.exe or something. gocode set will print the gocode config. There make sure autobuild is set to false. If it is, then it's not gocode who's building stuff. Also keep in mind that third party plugins may invoke third party tools and who knows maybe it's not gocode's fault.

gocode.exe set
propose-builtins true
lib-path ""
custom-pkg-prefix ""
custom-vendor-dir ""
autobuild true
force-debug-output ""
package-lookup-mode "go"
close-timeout 1800
unimported-packages false

Autobuild is set to true o_O
This is how I started gocode: gocode.exe -s -sock tcp -addr <IP_ADDRESS>:37373
VSCode spawns a process and send a command similar to gocode.exe -f=json autocomplete "c:\Users\Matt\Go\src\github.com\pkg\my_file.go" c4706

Vscode plugin might invoke "set" command and set it to true. Find a way to force it to false. Plugin should be open source. gocode set autobuild false will set it to false, I don't remember where gocode's config is on windows. Somewhere in there: https://github.com/nsf/gocode/blob/master/os_windows.go#L40-L48

I did find out that the VSCode Go extension does have a flag to autobuild:
let gocode = getBinPath('gocode');
			let autobuild = vscode.workspace.getConfiguration('go')['gocodeAutoBuild'];
			cp.execFile(gocode, ['set', 'propose-builtins', 'true'], {}, (err, stdout, stderr) => {
				cp.execFile(gocode, ['set', 'autobuild', autobuild], {}, (err, stdout, stderr) => {
					resolve();
				});
			});

I'll turn that off this way (in the config) and confirm that it fast again. Funny enough this feature works well on Mac but I'll investigate that separately.

I think the feature invokes the go tool. So if you have some unconventional setup or something it might fail. Also I believe go tool uses modification timestamps to rebuild out of date things. Not sure how well it would work on windows. On linux and mac it's solid because timestamps were used by "make" for ages.

I can confirm that when turning off this feature it is fast again. The VSCode Go extension have that feature turned on by default, probably because most extension developers are on unix systems.
I'll look at what gocode/go tool does to determine if the packages should be rebuilt. I've only recently started developing on Windows from time to time and I did notice that go build recompiles everything if I don't use the -i flag. So there might be a simple workaround.
Thanks again for gocode, it's a very convenient tool I rely on daily :)

go build should not be used most of the time. Gocode's autobuild feature runs go install. Go build is useless because it's a blank build, it does nothing. go install builds and saves binaries.
Code for reference: https://github.com/nsf/gocode/blob/master/declcache.go#L189-L252
It's a very simple feature.

perfect, I'll investigate and see what's causing the issue under Windows.

After further investigation... I have no idea what was wrong. I removed the executable, added some debug statements, rebuilt it and copied it (for some reason go install wasn't moving the file properly). After that, autobuild worked as expected on Windows 10, VSCode, autobuild on, Go 1.8.
I can only assume something was wrong with my system and my coworkers.


Delete SharePoint Site

When an admin tries to delete a site or subsite that is included in a retention policy they receive following message: "This site cannot be deleted because it is included in an eDiscovery case or retention policy". This seems like expected behavior based off this statement from this article: "Note that a user receives an error if they try to delete a library, list, folder, or site that's subject to a retention policy." Please confirm this is the case. Also, please provide a solution. If we don't want a site to show any longer in SPO there should be a way of deleting it, or at least removing it from view. Why does it not go to the 'Preservation Hold library' like other objects? Please advise.

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

ID: 30d08974-4cd1-4fab-7a59-17e5f8517566
Version Independent ID: 93e7f78f-6a94-1b90-1adf-70d2e266ce51
Content: Overview of retention policies
Content Source: microsoft-365/compliance/retention-policies.md
Service: o365-seccomp
GitHub Login: @LauraWi
Microsoft Alias: laurawi

@toddjohnson39  - Thank you for submitting feedback.
I will get this issue over to the Microsoft 365 writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Hi @toddjohnson39 Yes, the behavior you note is expected. This section explains how retention works for in-place items: https://docs.microsoft.com/en-us/microsoft-365/compliance/retention-policies#how-a-retention-policy-works-with-content-in-place
If content is modified, a copy of the original is placed in Preservation Hold, but the content remains in place until the retention period expires. If your goal is to remove a site from view in SPO, you can explore removing links to the site in navigation and changing search settings to make it less discoverable.

Closing issue.


Barrier API documentation is not informative

What should we add?
A Google search for "qiskit barrier" leads one to its entry in the API reference as the first hit.  Unfortunately, this page contains only a circular definition, and does not even explain what a barrier does:

Barrier instruction.
Create new barrier instruction.

There is a much more useful description by @chriseclectic on StackExchange, with tens of upvotes.  I believe a similar explanation should be incorporated into the Barrier docstring(s).

For reference, this is Chris's answer on stackexchange.  The relevant docstrings that need updating are the QuantumCircuit.barrier method and the Barrier class.

How is it going with this issue? I am interested to learn more and contributing.

@faraimazh Do you want to work on this issue? I can assign you.

Hi @HuangJunye , if it's okay, may I work on this?

I'd like to work on this

The PR #9015 is ready, please let me know if there is anything else needed from me


added hasher

Thanks for the PR! 👍 Build is failing because of Swinejct 2.7 no longer supporting Xcode 10.2 - we could fix this by bumping Xcode version in travis.yml

I need this 👍

Any updates to merge into a version?

Can this be solved? https://github.com/Swinject/SwinjectStoryboard/pull/151

Thank you for making the PR😃
It looks the change is covered by PR #135, so I'm going to close this issue.


Recommendations for proper versioning of the plugin

Want to make sure we don't have inconsistencies between the published
versions and the commits in git.
Issue #97 and #101

@brunobowden ptal

Looks good


Enable service fabrik integration with credhub

IAAS credentials need not be stored as part of the service. Supports credhub integration.
Passes credhub co-ordinates during service deployment.

Pull Request Test Coverage Report for Build 1142

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 88.903%

Files with Coverage Reduction
New Missed Lines
%

broker/lib/fabrik/DBManager.js
1
98.85%

Totals

Change from base Build 1131:
-0.01%

Covered Lines:
6625

Relevant Lines:
7222

💛  - Coveralls


Got an error after performing npm run dev

Hi everyone, I'm still new to Astro in general and came to the following issue. I downloaded the code in Zip file, extracted them and ran the following as instructed in the README.

npm install
npm run dev

And got the following error:
><EMAIL_ADDRESS>dev
> astro dev

dyld: lazy symbol binding failed: Symbol not found: _pthread_jit_write_protect_supported_np
  Referenced from: /Volumes/DATA/Astro/astrowind/node_modules/sharp/build/Release/../.././vendor/8.13.3/darwin-x64/lib/libvips-cpp.42.dylib
  Expected in: /usr/lib/libSystem.B.dylib

dyld: Symbol not found: _pthread_jit_write_protect_supported_np
  Referenced from: /Volumes/DATA/Astro/astrowind/node_modules/sharp/build/Release/../.././vendor/8.13.3/darwin-x64/lib/libvips-cpp.42.dylib
  Expected in: /usr/lib/libSystem.B.dylib

/var/folders/p1/k78b7zpd401fqcvhv6jx8w2r0000gn/T/dev-f33a7221.sh: line 1: 10857 Abort trap: 6           astro dev

Though it went smoothly on the previous release, wondering if anyone could point me on how to fix them.
Thanks,

Hi @prototypa,
I'm on node v16.17.1 running on MacOS Mojave 10.14.6. While the previous version of the theme was running smoothly on my local system, but once I've downloaded the latest one, it kept giving me the above errors.
Thanks,

As it appeared couple of similar issues have been reported previously and apparently, it is a Mac OSX specific issue, rather on the theme itself. So for those of you whose coming from a Google Search and having similar issue, here's the solution that actually worked for me:

npm uninstall sharp and
npm install<EMAIL_ADDRESS>
And, they're back again working as intended.
Thanks,


Update README.md

Fixed chart, added Chef-Partners maintaining.

Do'h i guess README through Github editor doesn't count for DCO 😢 🐼


[Refactor][UI] Refactor task group queue using NSpace component.

Purpose of the pull request
ref #10844

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #11341 (0cece0c) into dev (0fdec2c) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #11341      +/-   ##
============================================
- Coverage     40.28%   40.24%   -0.05%
+ Complexity     4846     4840       -6
============================================
  Files           974      974
  Lines         37323    37321       -2
  Branches       4141     4137       -4
============================================
- Hits          15036    15019      -17
- Misses        20739    20753      +14
- Partials       1548     1549       +1

Impacted Files
Coverage Δ

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
48.64% <0.00%> (-7.21%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...erver/master/processor/queue/TaskEventService.java
75.00% <0.00%> (-5.36%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.00% <0.00%> (-2.78%)
:arrow_down:

...pache/dolphinscheduler/common/utils/DateUtils.java
74.83% <0.00%> (-0.17%)
:arrow_down:

.../dolphinscheduler/plugin/task/datax/DataxTask.java
0.00% <0.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Fix a bug that cell functions may be skipped when two columns shares the same property

When data contains nested objects, it is useful to display two columns with the same property settings, and provide a cell function that reads from the nested object (see the test for example)
However, currently <Table> 's render() is flawed. The bug may cause a column's calculation result leaked to another column, which leads to the columns that share the same property settings renders incorrectly.
This pull request addresses the data leakage among columns and thus fixes the bug.

It's alright as a separate test. This way it's clear what it's more explicit (good thing). :+1:

I published 0.11.0 with the fix. As I updated the project deps, I went to a new series just in case.


[REQUEST] Fake Votes

It should be as simple as a fake ban right? Just a name change.
Now I am a developer myself but I've never touched on CS:GO cheats but I do assume it can't be that hard.

Fake vote got patched.


Add key public and key inspect cli commands

Context
https://jira.iohk.io/browse/ADP-81
We need a command to derive public keys from private keys.
We also need an inspect command that splits the 32-byte chain code  from the 64-byte extended private key. It should also do the corresponding thing when inspecting public keys.
The CLI should also ultimately support soft derivation using public keys. This may need some thought to do properly.
Decision
Implement quick, simple versions of key inspect and key public.
Don't bother with adding support for inspecting public keys. That would be best handled later when we add public key derivation support in another U/S.

Acceptance Criteria

We must implement key public
We must implement key inspect
The commands should conform to the usage:

$ key public 5bd164a28...1ae275939
  10bf8a5a8...fbab14ea9

$ key inspect 5bd164a28...1ae275939
  private-key: ...
  chain-code : ...

Development

Quick implementation of key public #1393
Quick implementation of key inspect #1397

QA

Added one golden test each for public and inspect.
Negative scenarios (relating to decoding xprvs) should be covered by the tests for key child.

With the current CliKeyScheme, it might be as well to implement public key derivation at the same time.
We could also probably be "fine" with a Text -> m Text implementation, and a golden test or two. There really shouldn't be any corner cases with converting a XPub to hex Text. (Unlike the intricacies of going from hex to XPrv).

lgtm.


RSpotify de-authenticates after some time

Describe the bug
Spotify-player will 'refuse' to play any songs after a random period of time(?)
To Reproduce
You can't, and i can't.
This is completely random; but what i did to produce it:

Play some music
Pause
Go take a shower
Can't play any more music

Expected behavior
I can play any more music.
Log and backtrace
Got a panic: PanicInfo {
    payload: Any { .. },
    message: Some(
        RSpotify not authenticated,
    ),
    location: Location {
        file: "/home/sewn/.local/share/cargo/registry/src/github.com-1ecc6299db9ec823/rspotify-0.11.6/src/clients/base.rs",
        line: 99,
        col: 14,
    },
    can_unwind: true,
}

Stack backtrace:
   0: <unknown>
   1: <unknown>
   2: <unknown>
   3: <unknown>
   4: <unknown>
   5: <unknown>
   6: <unknown>
   7: <unknown>
   8: <unknown>
   9: <unknown>
  10: <unknown>
  11: <unknown>
  12: <unknown>
  13: <unknown>
  14: <unknown>
  15: <unknown>
  16: <unknown>
  17: <unknown>
  18: <unknown>
  19: <unknown>
  20: <unknown>
  21: <unknown>
  22: <unknown>
  23: <unknown>
  24: <unknown>
  25: <unknown>
  26: <unknown>
  27: <unknown>
  28: <unknown>
  29: <unknown>

Above is spammed each time i want to play a song in this state.
2023-03-02T12:16:36.893454Z  INFO client_request{request=GetCurrentPlayback}: spotify_player::client: successfully handled the client request, took: 399ms
2023-03-02T12:16:37.039757Z  INFO client_request{request=GetCurrentPlayback}: spotify_player::client: successfully handled the client request, took: 546ms
2023-03-02T12:21:29.444037Z  INFO spotify_player::streaming: Got an event from the integrated player: Paused { play_request_id: 93, track_id: SpotifyId { id:<PHONE_NUMBER>07603666607343732754352089375, audio_type: Track }, position_ms: 292181, duration_ms: 517693 }
2023-03-02T12:21:29.986013Z  INFO client_request{request=GetCurrentPlayback}: spotify_player::client: successfully handled the client request, took: 541ms
2023-03-02T13:13:04.854185Z  INFO client_request{request=Player(StartPlayback(Context(Playlist(PlaylistId("6yYlmM3Xuq79OXZoCdKWcW")), Some(Uri("spotify:track:00nv6xVzlNwvxuRxyjT5hn")))))}: spotify_player::token: Getting new authentication token...
2023-03-02T13:13:05.151246Z ERROR client_request{request=Player(StartPlayback(Context(Playlist(PlaylistId("6yYlmM3Xuq79OXZoCdKWcW")), Some(Uri("spotify:track:00nv6xVzlNwvxuRxyjT5hn")))))}: spotify_player::client::spotify: Failed to get access token: failed to get token: MercuryError
2023-03-02T13:13:29.821318Z  INFO spotify_player::client::handlers: Current context ID (None) is different
from the expected ID (Some(Playlist(PlaylistId("6yYlmM3Xuq79OXZoCdKWcW")))), update the context state
2023-03-02T13:14:07.826981Z  INFO spotify_player::client::handlers: Current context ID (None) is different
from the expected ID (Some(Playlist(PlaylistId("6yYlmM3Xuq79OXZoCdKWcW")))), update the context state

As you can see, it begins at 13:13:04.854185Z.

I know restarting helps, but this did not use to happen on clients such as the official one and spotifyd

I've thought about making the app more robust to these kinds of errors (e.g networking). It's not trivial though and there is always a workaround, which is to restart the app shrug

My problem is, that I have to re-enter user and password every time this happens. I have a really unstable connectios, so this happens often.


Change types of structs to public

Review structs and make sure the fields are public so can be used by external systems.

Becomes part of v6 release


add openwrt support

add cmake files for openwrt,
and change in uv_unix_async.c, without this change, calling to uv__eventfd2 will return "Invalid argument" error on openwrt board.
I think it's  caused by non zero flags passed when make __NR_eventfd2 system call.
Refer to http://linux.die.net/man/2/eventfd2, I find this "In Linux up to version 2.6.26, the flags argument is unused, and must be specified as zero", so I make change using fcntl to set the flags.

Please add signed-off message, read this link

"In Linux up to version 2.6.26, the flags argument is unused, and must be specified as zero"

So you are using kernel with lower version?

ok, I will add signed-off later
I using linux 3.10.49

If using 3.10.49, then parameters for __NR_eventfd2 are sued. Please explain if I don't understand correctly.

Sorry, maybe something wrong on Openwrt, below code work correct on PC with kernel 3.16, but failed on Openwrt with kernel 3.10,
I will figure out what going wrong.
#include <sys/eventfd.h>

int main()
{
    int fd = eventfd(0, EFD_CLOEXEC | EFD_NONBLOCK);
    printf("%d\n", fd);
    return 0;
}

it turns out that, on mips kernel, the EFD_NONBLOCK(equal to O_NONBLOCK) flag is different from others.
arch/mips/include/uapi/asm/fcntl.h
#define O_NONBLOCK      0x0080

but in libtuv, the UV__O_NONBLOCK is hard coded to 0x800, to this cause the __NR_eventfd2 system call return "Invalid argument" error.

ok. thank you.

oh, I see the newest version of libtuv has fix this problem,
but the iotjs still use the old version of libtuv with that problem,
so I will try to change the iotjs to use the latest version of libtuv

there's a code in uv_linux_syscall.h
#if defined(__alpha__)
# define UV__O_NONBLOCK       0x4
#elif defined(__hppa__)
# define UV__O_NONBLOCK       0x10004
#elif defined(__mips__)
# define UV__O_NONBLOCK       0x80
#elif defined(__sparc__)
# define UV__O_NONBLOCK       0x4000
#else
# define UV__O_NONBLOCK       0x800
#endif

can you check with _mips definition?

hi, I don't know how to make this pull request clean here, so I make a new pull request with all changes combined and signed-off added, please close this one.

ok, moving to #21, you can close but I'll just close this.


Converting the output of the python client filter function back to UAST

I'm using python client to parse a java source code file and use bblfsh.filter to check a condition of a rule.
The return type of this filter function is an iterator. I want to convert back to UAST.
how to do it ? please guide

You can apply filter over any node in the UAST. Technically, there is not a UAST format (well, there is, but is binary and internal to the protocol), but for example in Python any Node object could have a filter function applied to it.

Thanks for the reply. I have a different request. Can I save the UAST returned from the parse function i.e. response.uast into a plain text file ? If yes how to do that? I have tried to convert response.uast into list by list(response.uast) and try to iterate it says -  'Node' object is not iterable

@juanjux  - thanks for solution for the original filtering UAST over and over again. It worked.
Also it helped me export the USATto text file. Thanks a lot for your help

Happy to help!


Washington State College of Ohio

https://www.wsco.edu/academics/business/cyber-security-investigation/
thank you very much in advance

@braxthegen Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Added Table of Contents

Added Table of Contents for better navigation and accessibility.

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@BeastMaster07 Awesome! Thanks! Congrats on your first open source contribution to this repository!


Cannot Share, command not found. Dot net runtime error

Issue Type: Bug
Open VS Code and try to start a live share session.
It says
Command 'Live Share: Start Collaboration Session (Share)' resulted in an error (command 'liveshare.start' not found)

In the console, there is another error that seems related
/opt/visual-studio-code/resources/app/out/vs/workbench/workbench.desktop.main.js:1734 Activating extension 'ms-vsliveshare.vsliveshare' failed: Agent terminated with exit code: 0 and signal null: Error:
  An assembly specified in the application dependencies manifest (Microsoft.NETCore.App.deps.json) was not found:
    package: 'runtime.osx-x64.Microsoft.NETCore.App', version: '3.1.1-servicing.19608.4'
    path: 'runtimes/osx-x64/native/System.Globalization.Native.dylib'.

I've installed the dot net runtime, and I'm not on osx, so I'm not sure where to go from here.
Extension version: 1.0.3784
VS Code version: Code 1.53.2 (622cb03f7e070a9670c94bae1a45d78d7181fbd4, 2021-02-11T11:48:44.518Z)
OS version: Linux x64 5.10.15-1-MANJARO

System Info

Item
Value

CPUs
AMD Ryzen 9 3950X 16-Core Processor (32 x 2876)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
31.33GB (25.91GB free)

Process Argv
--no-sandbox

Screen Reader
no

VM
0%

DESKTOP_SESSION
gnome

XDG_CURRENT_DESKTOP
GNOME

XDG_SESSION_DESKTOP
gnome

XDG_SESSION_TYPE
wayland

Closing this as resolved, since we believe we've fixed this issue and we haven't heard it reported it in a while. Please re-open this if you're still encountering a problem here. Thanks!

I gave this a try again and can confirm it does work now. Thanks for the great work on this!


Wait: turn the debug logs into stdout print

What this PR does / why we need it:
if --wait flag for helm install/upgrade is specified, Helm will log to standard output (stdout) the wait informations.
Before: it was logged only with --debug flag to stderr.
Solve https://github.com/helm/helm/issues/7760
Example output:
using a sample values.yaml containing:
persistence:
    enabled: false
replication:
    enabled: false
    slaveReplicas: 3

bin/helm upgrade --install test stable/postgresql --wait --values values.yaml
Release "test" does not exist. Installing it now.
 In fact, it would be nice to use log everywhere without short file name and make its format configurable.
My current concern is about backward compatibility if people used output in batch scripts.

If someone has a complicated chart and uses the --wait flag this would add a lot to the output. Should the extra output be hidden behind a flag?

The log would not be that huge since helm only outputs "XX is not ready" for a single object a a time (i.e in the case it is waiting for 2 Statefulset, it will arbitrarily stick for one of the two until it is ready, and print status of the second only if it is not ready).
In the given example, see how it did NOT print about "default/test-postgresql" StatefulSet.

Why stdout? stderr is the right stream. The issue that this PR is trying to solve (#7760) is that there's no feedback at all when --wait is used without --debug. And --debug isn't the right feedback. A few spinners would do the trick. I'd love to have a UX similar to kind create cluster. See screenshots below:

closing as this does not solve the original issue in the way it was proposed. This PR has been stale for over 2 months with no activity after reviews.
Please re-open when you have addressed the comments given above. Thanks.


Support TEC Garden type

This epic is to keep track of the tasks that still need to happen to fully support the TEC Garen type.

[ ] Update the UI to detect if the request token is different from the Garden token.

@sembrestels would you mind filling the blanks here?


nice work!

Hi there,
I think the best way is to return x at this point.
https://github.com/YuanGongND/ast/blob/9e3bd9942210680b833b08c39d09f2284ddc4d1d/src/models/ast_models.py#L184
Note - x is a sequence, in time-frequency order (i.e., not in time, and not in frequency order), starting with two [cls] token. If you don't care sequence, just return after x = (x[:, 0] + x[:, 1]) / 2, else return before x = (x[:, 0] + x[:, 1]) / 2 and do your desired operation.
-Yuan

To save, you can just convert x to a numpy by x=x.detach().cpu().numpy(), and than np.save(x).


Fix Deserializer of serde_test to correctly deserialize unit variants from integers/strings/bytes

This is actually possible, but Deserializer of serde_test returns an error instead of deserialization

It is difficult to remember original motivation because so many time gone, but I will try. Most probably, I tried to create a full set of tests for quick-xml crate and looking for a common ways to testing serializer and deserializer. During that I've found, that serde itself has very disparate and weakly structured tests. It is often difficult to understand which parts are tested and which are not, some parts, on the contrary, are tested many times in different places. So I've started with structured approach and tried to implement exhaustive tests for some parts, enums particularly.
I've found, that some token streams these either produced by the enum serializers or just supported by deserializer and tested implicitly (via deserialization, see additions in test_suite/tests/test_value.rs cannot be tested by serde_test deserializers because of panic in serde_test about unexpected data.
So I've fixed serde_test's deserializer to accept tokens that de-facto accepted by real deserializers and add corresponding tests in test_suite/tests/test_de.rs).


Activate nightly feature for futures-stable

Fixes https://github.com/alexcrichton/futures-await/issues/76

After this the project still won't compile but that particular issue will have been fixed.

I've just noticed that https://github.com/alexcrichton/futures-await/pull/73 already takes care of this, among other things.


Deprecation of Panel4D/PanelND

Related to https://github.com/pydata/pandas/issues/13563 (deprecation of Panels), we could/should also deprecate Panel4D / PanelND.
(xref https://github.com/pydata/pandas/issues/5049)
Deprecating Panel4D/ND could probably be done sooner than deprecating Panels (e.g. actually deprecating in 0.19.0 ?)

cc @pydata/pandas

I'm very in favor of this. I will leave some comments on #13563

Let's do this in 0.19.0 then


BUG: Buy now button on collection page doesn't work

When I click this button, nothing happens.

@georgeroman This endpoint is supposed to return token data for the floor: https://rinkeby-api-v4.reservoir.tools/stats?collection=forgottenruneswizardscult
https://mainnet-api-v4.reservoir.tools/documentation#/stats/getStats
This is causing this error. Should I rely on this endpoint for this function?

@diegoramosxyz What do you mean by "return token data for the floor"? Does the response miss anything that it previously returned in an old version?

@georgeroman This is what the docs say the response from this endpoint should be:
{
  "stats": {
    "tokenCount": 0,
    "onSaleCount": 0,
    "sampleImages": [
      "string"
    ],
    "lastBuy": {
      "value": 0,
      "block": 0
    },
    "market": {
      "floorSell": {
        "hash": "string",
        "value": 0,
        "maker": "string",
        "validFrom": 0,
        "validUntil": 0,
        "token": {
          "contract": "string",
          "tokenId": "string",
          "name": "string",
          "image": "string"
        }
      },
      "topBuy": {
        "hash": "string",
        "value": 0,
        "maker": "string",
        "validFrom": 0,
        "validUntil": 0
      }
    }
  }
}

On the example call I provided: https://rinkeby-api-v4.reservoir.tools/stats?collection=forgottenruneswizardscult
"token": {
          "contract": "string",
          "tokenId": "string",
          "name": "string",
          "image": "string"
        }

is missing. I need that token object to have a buy now button that updates based on filtering.

Will add that today


Remote memory write corruption bug

eg. "/data/local/tmp/libhook.so" = 26 bytes + (align remaining bytes) = malloc(28)

malloc does not initialize the bytes to 0, therefore when you (https://github.com/evilsocket/arminject/blob/master/jni/injector/traced.hpp#L146):

/*
You copy 26 bytes, the remaining alignment bytes bytes are junk (not \0\0),
PTRACE_POKETEXT will write those 2 junk bytes from malloc(28)!
*/
memcpy(ptr, buf, blen);

The remote write then becomes "/data/local/tmp/libhook.so??" instead of "/data/local/tmp/libhook.so\0\0"

Finally, when the remote dlopen is called, it's looking up a bad filename and thus fails
to inject

Solution:
Use calloc() instead of malloc() (https://github.com/evilsocket/arminject/blob/master/jni/injector/traced.hpp#L144)
char *ptr = (char *)calloc(blen + blen % sizeof(size_t),1);

PS. Sometimes you'll get real lucky and malloc actually returns 0 initialized bytes in the alignment area, masking this elusive issue! ;)
Cheers!
Martin Alexander

Or alternatively:
write( mem, (unsigned char *)s, strlen(s) + 1); //+1 to capture the \0


setup - allow absolute paths to setup folders

ob: why l99 use requires the config to be under setups/ folder? When I use absolute path to the config folder, it should not prepend setups/ which results in setups//some/absolute/path.
bb: the idea was to just keep all configs in the setups folder - but will add 'allow absolute paths' to the tasklist

Where do the changes need to happen?

anywhere the .l99_setup file is referenced - currently it just stores the setup name, eg 'oxbox', not a path
eg
https://github.com/Ladder99/ladder99/blob/main/shell/commands/use#L30-L31
https://github.com/Ladder99/ladder99/blob/main/shell/commands/start#L42-L44

@bburns, how about we check if a particular env var is set? If it is not set, we use $PWD.
Something like this:
L99_HOME="${L99_HOME:-$PWD}"


Fixed new_order call for market order resulting in server error

Bugfix.
price = 0.00001 was coerced into "1.0e-05" string resulting in server error.

Good catch.
Merged