How to get the the Gumbel softmax distributions over vocabulary inside GumbelSoftmaxEmbeddingHelper

            helper_2 = tx.modules.GumbelSoftmaxEmbeddingHelper(
                # embedding=self.encoder.word_embedder,
                start_tokens=torch.tensor([101]*2).long(),
                end_token=102,
                tau=1,
                straight_through=1.)
            outputs, _, sequence_length = self.decoder(
                memory=outputs_enc[0],
                memory_sequence_length=batch['source_length'],
                max_decoding_length=11, helper=helper_2)

GumbelSoftmaxEmbeddingHelper provide a very easy way to feed Gumbel softmax distributions over vocabulary to the next step. How can I get the Gumbel softmax distributions over vocabulary at each time step?

Thank you for your interest in texar-pytorch!
To the best of my knowledge, outputs here contains the outputs at all time steps.

If you set straight_through to be False, the  Gumbel softmax distribution is fed to the next step at each time. You then can access to all the distributions from outputs.sample_ids.

If you want to set straight_through to be True, which means a single token with the highest probability is fed to the next step at each time. I do not think we store the distribution information in the case.

Please correct me if I am wrong. @huzecong

Thanks for your quick reply.
I checked outputs.sample_ids and found that if I set straight_through=True. The outputs.sample_ids is one-hot vector.
What I want to do is to pass this one-hot vector to another model D (saying a discriminator).
Can gradient flow through this one-hot vector from model D?

If you set straight_through=True, outputs.sample_ids then stores the tokens with the highest probability at each time step. So the one-hot vector is expected.
In this case, sample_ids is computed by (sample_ids_hard - sample_ids).detach() + sample_ids (here), where sample_ids_hard is the token index (one-hot vector), and sample_ids is the distribution tensor. I think the gradient will flow through the distribution tensor.
You might also want to wait for the feedback from @huzecong who wrote this part of the code.

@gpengzhi explained it right. The definition of "straight-through" Gumbel-softmax is to use the IDs with maximum probability (hard IDs) during the forward pass, but use the softmax distribution (soft IDs) during the backward pass. Our implementation follows this behavior, so the gradient can flow through this one-hot vector.
The trick here is to use detach. As @gpengzhi pointed out, we compute the straight through sample IDs by (hard_ids - soft_ids).detach() + soft_ids, where detach removes the tensor from the computation graph so gradients are not computed for them. Thus, in the forward pass, the value would be hard_ids - soft_ids + soft_ids = hard_ids, while in the backward pass, only the non-detached soft_ids get used in gradient computation.

Thanks for @gpengzhi @huzecong help! I think my question is well answer and I will close this thread.
One suggestion is that the document can be further improved if possible to give users more information.
For example, right now, the definition of sample_ids is

The sampled results (at each step/of all steps). For example, in AttentionRNNDecoder with decoding strategy of "train_greedy", this is a torch.LongTensor of shape [batch_size, max_time] containing the sampled token indices of all steps.

I thought sample_ids is just the index of tokens when I first read this paragraph. However, it turns out that it can also be a one-hot vector which has gradient if I use gumbel helper.
I appreciate all you guys' help and good work.  Texar is really awesome. 👍

@XinDongol Thank you for pointing this out! We will update the documentation accordingly for the helpers. Thank you for supporting the Texar library!


Build error while trying to run tests locally

Versions:
ember-cli: 5.12.0
node: 20.18.0
pnpm: 9.12.3
os: darwin arm64 (MacOS Sequoia 15.1)
In package test-esa:
[gaurav.munjal@MacBookPro test-esa (gaurav/feat/support_partitioned_cookies)]$ ember test
Environment: test
⠋ BuildingTorii is installed but not configured in config/environment.js!
cleaning up...
Cleanup error.
⠋ cleaning upCannot read properties of undefined (reading 'cleanup')

Stack Trace and Error Report: /var/folders/5n/x1_c935x0sq7lnqzfpsjkt080000gn/T/error.dump.873624881ff2ed0e4589d628770e0a6b.log
ENOENT: no such file or directory, stat '/Users/gaurav.munjal/dev/work/mainmatter/ember-simple-auth/packages/ember-simple-auth/node_modules/.bin/pnpm'

Stack Trace and Error Report: /var/folders/5n/x1_c935x0sq7lnqzfpsjkt080000gn/T/error.dump.9fa6036dd8b668dd27bb4d3ea3bda321.log
[gaurav.munjal@MacBookPro test-esa (gaurav/feat/support_partitioned_cookies)]$ node -v
v20.18.0
[gaurav.munjal@MacBookPro test-esa (gaurav/feat/support_partitioned_cookies)]$ npm -v
10.8.2
[gaurav.munjal@MacBookPro test-esa (gaurav/feat/support_partitioned_cookies)]$ pnpm -v
9.12.3
[gaurav.munjal@MacBookPro test-esa (gaurav/feat/support_partitioned_cookies)]$ ember -v
ember-cli: 5.12.0
node: 20.18.0
os: darwin arm64

Note that I get the same error running:
pnpm run --filter test-esa test
pnpm run --filter test-esa test:one ember-default

It looks like adding
hoist=true
hoist-workspace-packages=true

to .npmrc and rerunning pnpm i solves this problem. Would you like me to open a PR with this change?

Thanks for vigilance!
We don't want to enable hoisting or any additional configuration.
It seems like the issue might've been a bloated lockfile, usually renovate mitigates the problem but we might've missed a lockfile maintenance PR. Everything should be fine now I think.
I'll close this issue but feel free to re-open if you have this problem again.

Still having this issue, but I'm unable to reopen.

Have you tried on a completely fresh clone?
or
rm -rf node_modules/ pnpm-lock.yaml packages/**/node_modules
pnpm install

Not sure, but tried it now. Still occurs.


Yet another crash at launch

Fatal error
A fatal error was caught by Alias Isolation, and the current process will terminate.
A crash dump has been written to C:\Users[REDACTED]\AppData\Local\Temp\aliasIsolationCrashDump.dmp.
Press Ctrl+C to copy the contents of this message to the clipboard.
DebugBreak
aliasIsolation_hookableOverlayRender
aliasIsolation_hookableOverlayRender
aliasIsolation_hookableOverlayRender
aliasIsolation_hookableOverlayRender
aliasIsolation_hookableOverlayRender
aliasIsolation_hookableOverlayRender
aliasIsolation_hookableOverlayRender
aliasIsolation_hookableOverlayRender
aliasIsolation_hookableOverlayRender
aliasIsolation_hookableOverlayRender
??
??
??
??
??

OK
This happens 9/10 launches. I have been able to get it to run /sometimes/ through EITHER the GUI or regular injector versions.
One thing I noticed is pretty much every time I launch through EdgeTile on the start menu, I get this error, even though it was working fine a few weeks ago.
Already tried numerous fixes mentioned by others before:

Launch from desktop shortcut instead of start menu (deal-breaking non-permanent solution for me - I don't like clutter on my screen)
Launch injector.exe/injectorGUI.exe directly from its folder (simply not doing this every time I launch the game, period.)
Launch from taskbar (see #1)
Run as admin (always have been)
Make sure Steam is not running before launching injector (also always have been)
Delete all files, reinstall game, reinstall Alias Isolation

Anyway none of that should matter because it used to work fine and I changed literally nothing about my method of starting it up. Game runs fine when not using the mod. Something changed on its end or on Steam's, and only within the last month.

INB4 yes I launch it in windowed borderless. Can't really imagine why anyone would do otherwise.


”Florești” is listed in the drop-down menu

Description:
"Florești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Florești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Florești, MEHEDINȚI".


Building ios failed.

3 issues
/Users/zhanghong/dev/newapp/.meteor/local/cordova-build/platforms/ios/newapp/Plugins/com.dooble.phonertc/Session.swift:47:24: 'URLWithString' is unavailable: use object construction 'NSURL(string:)'
/Users/zhanghong/dev/newapp/.meteor/local/cordova-build/platforms/ios/newapp/Plugins/com.dooble.phonertc/Session.swift:52:24: 'URLWithString' is unavailable: use object construction 'NSURL(string:)'
/Users/zhanghong/dev/newapp/.meteor/local/cordova-build/platforms/ios/newapp/Plugins/com.dooble.phonertc/Session.swift:200:48: 'regularExpressionWithPattern(_:options:error:)' is unavailable: use object construction 'NSRegularExpression(pattern:options:error:)'

In case anyone else finds this,
Ensure that your build target is an actual iPhone or iPad running on the arm7 architecture. The iPhone and iPad simulators are not emulators, and only run on i386. The compiled RTC libraries for ios have been built for arm7.

Hi there,
I followed all instructions or iOS but I have 25 errors most of theme are related to the "try" statement:
let json = (try! NSJSONSerialization.JSONObjectWithData(message,
options: NSJSONReadingOptions.MutableLeaves)) as! NSDictionary
==> Expected ',' separator
==> Use of unresolved identifier 'try'
do {
data = try NSJSONSerialization.dataWithJSONObject(json,
options: NSJSONWritingOptions())
} catch let error as NSError {
jsonError = error
data = nil
}
==> Consecutive statements on a line must be separated by ';'
Something wrong in my config ?
Thx


Color texture format on WebGL and GL ES < 30

For glTexImage2d, WebGL follows GL ES 2.0: https://www.khronos.org/registry/webgl/specs/latest/1.0/#TEXIMAGE2D_HTML which states that format "must match internalFormat" (https://www.khronos.org/opengles/sdk/docs/man/xhtml/glTexImage2D.xml)
For example: RGBA8 should have internal format GL_RGBA and format GL_RGBA instead.
Not sure what's the reason for this check:
https://github.com/bkaradzic/bgfx/blob/master/src/renderer_gl.cpp#L1349
Without that loop, m_supported won't be populated and RGBA8 won't work on WebGL

Not sure what's the reason for this check:

Firefox throws exception when there is texture format it doesn't recognize.

As of firefox-developer 39.0a2 and chromium 42.0.2311.135 on emscripten-<IP_ADDRESS>.g5ec57b8 they only emits warnings but firefox-developer decided to squelch warnings after too many invalid operations which can be a problem.
After commits https://github.com/bkaradzic/bgfx/commit/8da183a2f14027cdc92af842411a62e8f4e12017, I got the old behaviour again when RGBA8 is not detected and bgfx decided to swizzle R8 (the only thing that works based on GL_LUMINANCE) into it which makes everything monochrome.
I'm not sure what's the correct way to fix this. How about manually check for 4 standard texture types that I corrected with setTextureFormat if emscripten is enabled? That way, I still have something to work with (esp RGBA8) and still will not trips some browsers.

I just tested with different setup and it works there. Not quite sure if it's FF or emscripten that causes issue. Setup that had problem has older emscripten and newer FF, and this setup where it works it has newer emscripten but older FF. Anyhow if it's FF or emscripten issue probably I should file bug there.


Missing visionOS support for SPM

As the title explains, AgoraRTM cannot be used in a native visionOS app. I can imagine this would be as simple as adding .visionOS(.v1) to the platform field in the Package.swift.

CoreTelephony dependency should be solved too, since it is not available for visionOS. Is it needed?

hi @damiantrimboli , we will support visionOS in RTM version 220, which is scheduled to release by end of March. For more details please contact agora support team.

hey @plutoless! any updates about this?


target: mbedos5: Added DigitalIn & Serial js bindings

This is now working:
var a = new DigitalIn(BUTTON1);
print(a.read());

Also added the other missing class constructors to lib_drivers.h as commented out lines. Lets get them all done!
JerryScript-DCO-1.0-Signed-off-by: Rouan van der Ende<EMAIL_ADDRESS>

@janjongboom please check this patch

Thank you @zherczeg I've also now done:
Serial(tx,rx,baud)
Serial.baud()
Serial.putc()
Serial.getc()
Serial.writable()
Serial.readable()

Will do a commit now.

I've been struggling to get Serial.attach() to work.
As soon as I send a byte to the board to trigger the interrupt I get this error:
CMSIS-RTOS error: ISR Queue overflow (status: 0x2, task ID: 0x0, object ID: 0x20001240)
If anyone can add this function it would be much appreciated.

@fluentart a technical note to the pull request: it seems that you have opened the PR from your master which you also use for development. Now, there are already 4 commits stacked on each other with somewhat independent changes. The suggested way for contributing to the project is to have independent well-scoped features or pull requests in separate PR-s 1 commit each. Therefore it is a good practice not to use your master to open PR from but to have separate branches for the PRs. This way, each branch can evolve in its own pace as reviews start flowing in. And you can also keep your master in sync with the project's master easier.
The second thing is that you'll have to sign all your commits by adding a line at the end of the commit message:
JerryScript-DCO-1.0-Signed-off-by: Your Own Name<EMAIL_ADDRESS>
All this is described in the contribution guidelines at https://github.com/jerryscript-project/jerryscript/blob/master/CONTRIBUTING.md . And you can also take a look at existing commits for signature lines at https://github.com/jerryscript-project/jerryscript/commits/master .
Note to ourselves: Travis should have warned about missing DCO signoff but Travis hasn't been running at all for any of the commits.

Thanks @akosthekiss it wasnt clear if this message:
JerryScript-DCO-1.0-Signed-off-by: Rouan van der Ende<EMAIL_ADDRESS>should have been on the git commit or on the PR comment. Well, I've now moved on to Zephyr for now, you are free to use the code or not.

@fluentart if you are abandoning the PR, please close it. "you are free to use the code or not" is not how the project works. As per project governance rules in place Code without a proper signoff cannot be merged into the mainline (see: https://github.com/jerryscript-project/jerryscript/blob/master/DCO.md). If you will not add DCO sign-off and will not follow up reviews, we will not be able to use your contribution.


preserve playlist order

What is the purpose of your issue?

[ ] Bug
[X] Feature Request
[ ] Question
[ ] Other

Description

spotdl should put a number, like 01 or 001, in front of the first track of a playlist, and each successive track, so the original order of the playlist is preserved, or at least have that option, like spotdl -ln could include leading zero numbers in front of the file names

+1 for this feature.

See https://github.com/ritiek/spotify-downloader/wiki/Advanced-usage#set-custom-filenames, track_number. @mrevjd please use GitHub's reaction feature (i.e. add a :+1:) if you just want to communicate agreement with a comment.

@linusg - It doesn't preserve the number ordering from the playlist but uses the track number from the meta data. I would like to preserve/use the playlist order as the track number.
Cheers

Ah, right. Still can't do this, see https://github.com/ritiek/spotify-downloader/issues/417#issuecomment-448672193 for an explanation.


Token request failed - [nuxt-oidc-auth]: [POST] "https://github.com/login/oauth/access_token":  fetch failed

Hello guys,
I'm having this error when trying to implement authentication in my webapp. I tryed the playground with github just to see if i could understand it better (like if i got a working example i could try to reproduce it in my app) but unfortunately i have the same error using github.
When i try to fetch the token with postman it works...
I also have this errors in vs-code:

This is what i have in my nuxt.config:

My .env:

Im redirected to github and 'im able to autenticate. The problem is while trying fetching the token (as far as i understand). I also printed in the console what is really being fetched and everything seems correct.
Can you help me please?
EDIT: I also tried with different different tokenRequestType and nothing.
UPDATE: What i found out is that you can't use a proxy while using this module because this is using ofetch without that option. I've downloaded the source code and changed the ofetch request in oidc.ts and it worked, now it can make the request correctly.
Still have a problem with the session. I'm using nginx as a reverse proxy to redirect the redirect_uri used in the idp and when it arrives to the app the module is not able to retrieve the session.
Thanks a lot!
Diogo

Hi Diogo,
would it help if we add proxy support with ofetch?

Hi Diogo, would it help if we add proxy support with ofetch?

Hello Itpropro!
It would be awesome if you had proxy support.
I'm still struggling with retrieving the session when being redirected to my app. Using PKCE im not able to fetch the token because i loose the code verifier.
Do you have any tips?
Thanks a lot,
Diogo

Auth is generally hard to handle with proxies. I am currently looking into adding proxy support to the config to hand it over to ofetch.
Any chance you can exclude the traffic between the backend and the IdP from your proxy?

For now i'm using your source code with some changes in the ofetch call to
include the proxy.
I can't exclude the trafic because i have security restrictions in my
company.
Regarding the session problem, do you want me to open another issue?
Thanks a lot!
A quarta, 4/09/2024, 19:29, Jan-Henrik Damaschke @.***>
escreveu:

Auth is generally hard to handle with proxies. I am currently looking into
adding proxy support to the config to hand it over to ofetch.
Any chance you can exclude the traffic between the backend and the IdP
from your proxy?
—
Reply to this email directly, view it on GitHub
https://github.com/itpropro/nuxt-oidc-auth/issues/44#issuecomment-2330425431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOQNUHVYWVYLWBG2BCXES7LZU6XYTAVCNFSM6AAAAABL4EFLACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZQGQZDKNBTGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi again itpropro,
Just sending a mensage because i didn't understand if in the end you were talking about the reverse proxy i'm using (Nginx) or the company proxy used in the ofetch to make the requests from my app.
Regarding the second, the problem was solved changing the ofetch call in the source code.
The first i'm still struggling with the session. At this moment i don't even understand if the issue is in the nginx configuration or in the module itself. Did you tried something similiar? (or anyone else)
Thanks a lot for all your help!
Diogo

Hi again itpropro,
Just sending a mensage because i didn't understand if in the end you were talking about the reverse proxy i'm using (Nginx) or the company proxy used in the ofetch to make the requests from my app.
Regarding the second, the problem was solved changing the ofetch call in the source code.
The first i'm still struggling with the session. At this moment i don't even understand if the issue is in the nginx configuration or in the module itself. Did you tried something similiar? (or anyone else)
Thanks a lot for all your help! Diogo

Hey,
Can you maybe share the changes you did with the ofetch call? That way I could make it configurable in case other people run into a similar issue.
It would be great if you could open the session problem in a separate issue so we can track that accordingly.

Hi again itpropro,
Just sending a mensage because i didn't understand if in the end you were talking about the reverse proxy i'm using (Nginx) or the company proxy used in the ofetch to make the requests from my app.
Regarding the second, the problem was solved changing the ofetch call in the source code.
The first i'm still struggling with the session. At this moment i don't even understand if the issue is in the nginx configuration or in the module itself. Did you tried something similiar? (or anyone else)
Thanks a lot for all your help! Diogo

Hey, Can you maybe share the changes you did with the ofetch call? That way I could make it configurable in case other people run into a similar issue. It would be great if you could open the session problem in a separate issue so we can track that accordingly.

Hello!
Off course!
So i just changed in the server/handler/callback.ts (in the 0.17.0 version was in the server/lib/oidc.ts, other versions i don't know) the ofetch calls to retrieve the token (using the code previous sent by the idp) and also in the ofetch of the openIdConfiguration (if you have the token validation).
The change:

You will need the import:

The requestTls also could be a nice feature, so if you are in development mode, using self-signed certificates, making the rejectUnauthorized: false, will bypass the certificate verification.
If you need anything else please let me know.
Thanks,
Diogo


templates/testing: Add VNC_GEOMETRY parameter + minor changes

Also removing an old template that's not being used anywhere.

HMS-3020


Install instruction for using Android BOM artifact

@calebk1  https://github.com/AdobeDocs/aep-mobile-sdkdocs/blob/main/src/pages/documentation/manage-gradle-dependencies.md found it. It seems that we used the wrong links for each extension., I will fix it in this PR. Thank you for pointing it out.


Error running coinstac-simulator with sample computations.

Problem
I am getting an error running the coinstac-simulator for the coinstac-local-test and coinstac-decentralized-test.
System is KUbuntu 18.04 running in VirtualBox on a windows 10 host.
coinstac-simulator installed with "sudo npm install -g coinstac-simulator"
I cloned the MRN-Code coinstac repo and then ran coinstac-simulator in the coinstac-local-test directory and the coinstac-decentralized-test directory.  Error messages for each are attached.
Since the decentralized test error message listed an error re: a container, I did verify that the docker hello-world container can run properly.  The command "docker run coinstac/coinstac-decentralized-test" seems to fail while downloading the container. The local test doesn't mention anything related to docker that I can see, so that seems like a separate issue.
I'm thinking that this is a problem with my local environment but I did not see anything in the documentation that pointed me in the right direction to resolve this.  Any ideas what might be going wrong?
coinstac-decentrailized-test.error.txt
coinstac-local-test.error.txt
Tasks

Got this resolved with help from @rssk who put togther this fine Developer Guide.
I was missing the step of pre-building the docker image for the simulation:
cd ~/coinstac/packages/coinstac-images/coinstac-decentralized-test
sudo docker build  -t coinstac/coinstac-decentralized-test   .
sudo coinstac-simulator


test: Fail ginkgo tests on warnings

This pull request makes the ginkgo tests fail on any level=warning agent or operator log. See commits for details.
I had to allowlist a bunch of warnings so we can at least stop the bleeding before we fix them. I ran the full ginkgo suite 12 times without any new warnings popping up so we should be good to go.

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/ci-ginkgo

/test

I don't have all the context to review all warning messages (except for the one related to envoy/gateway api and ingress).

Me neither (except the ones related to datapath :sweat_smile:). I opened issues for all of them if they required further investigation. My goal for now is really just to stop the bleeding, not to make a judgement call on whether any of these are legitimate or require fixes. That will take more time.


switched to new omfit_classes package and modified user agreement

Switch Aurora to use new omfit_classes package and specify that users cite OMFIT when using Python tools in Aurora.
Outstanding issue before this is merged into master: omfit_classes currently prints to screen all the PYTHONPATH during the import. This is pretty annoying and can probably be avoided. @orso82 could you please have a look?
Also, did I get the OMFIT reference right in the Aurora user agreement? I took it from the OMFIT user agreement.

thanks @orso82 for the update to omfit_classes. Everything seems to be fine now, so ready to merge.


Change yaml parser 'yamljs' to 'js-yaml'

Change yaml parser 'yamljs' to 'js-yaml'
Fix #142

yamljs is not being maintained.
yamljs not support Flow Scalars.

See logs bug version here

See logs patched version here

See source code here

@all-contributors please add @srealmoreno for their code contribution


incompatible with android 13

i am using android 13 and android auto 8.9, and i cannot add carstream to the autorized applications.
i tried with android auto 8.8 and it does not work also.

no new notifications message here. may just jump off a cliff later

Curious if there's a path forward with android 13?
Having the same issue

Have you seen this? This worked for me in aaad. https://github.com/shmykelsa/AAAD/issues/247

https://github.com/shmykelsa/AAAD/issues/247


async* infinite loop may result in unbounded memory increase and eventual VM crash

Dart SDK Version: 2.12.0
The following (accidental) stream with async* resulted in an unbounded memory increase, and a VM crash without any trace or indication what the issue was.
Stream<int> loop() async* {
  yield* loop();
}

In contrast, inline recursive calls will get a Stack Overflow soon, while the following seem to be optimized by something to return without issues:
Iterable syncLoop() sync* {
  yield* syncLoop();
}

It's what I would expect to happen for something which recursively spawns asynchronous operations.
I guess the out-of-memory could be reported better.
I expect something similar from:
foo() async {
  await 0; // avoid stack-overflowing, `async` functions start synchronously.
  await foo();
}


ExpatError

Hello,
I got an error when I ran the 'Fig8.SequentialKL.ipynb'. I not sure how to resolve it.
Thanks,
Si

can you check that nibabel is imported correctly? seems to be an issue loading the surface file. Potentially check the route of the file as well?

can you check that nibabel is imported correctly? seems to be an issue loading the surface file. Potentially check the route of the file as well?

Hello,
There is no issue when nibabel is imported. The '.gii 'file is also in correct route.
I also got the same error when I load your '.gii' file using 'nib.load()'.

odd... I'm thinking it might be a permissions error. Could you try loading it in fsleyes or wb_view?

I got an error when I loaded 'L.rhum.inflated.surf.gii' in fsleyes.

Ok, I'll change permissions and reupload. can you open up the pig surfaces alright?

Ok, I'll change permissions and reupload. can you open up the pig surfaces alright?

I could not open the pig surfaces in fsleyes, either. Thanks for your help!

Hi there, I've changed the permissions on the files.  See if you can pull or clone fresh and try opening again please :)

you can also try doing  a checkout to the master branch

So it appears it was an issue with git large file storage. I've since uploaded a new version to the master branch. try doing a pull and if that doesn't work clone it new. let me know how it goes please and if It works I'll close the issue.

So it appears it was an issue with git large file storage. I've since uploaded a new version to the master branch. try doing a pull and if that doesn't work clone it new. let me know how it goes please and if It works I'll close the issue.

Appreciate it! It working now.
I have another question about the normalized colorbar. How did you get the R_hmax value for the vmax argument?
Thanks,
Si

oops, that's just a bit of left over code from when I ran the right hemisphere. you can ignore it.


DBZ-3107 Fix timing issue in test

https://issues.redhat.com/browse/DBZ-3107

@jpechane, no way, the PG testing issue is solved?


Upgrade UniFlow to Polymer 2.x

The goal of this pull request is to make the necessary changes to the source code/documentation in order to make UniFlow compatible with Polymer 2.x.

I signed it!

Hi Motamedi,
Thanks a lot for your contribution! I will review your changes within next
couple of weeks and let you know whether we can merge this pull request.
Best,
Andrei
On Fri, Sep 15, 2017 at 12:31 PM, Motamedi<EMAIL_ADDRESS>wrote:

The goal of this pull request is to make the necessary changes to the
source code/documentation in order to make UniFlow compatible with Polymer
2.x.
You can view, comment on, or merge this pull request online at:
https://github.com/google/uniflow-polymer/pull/12
Commit Summary

initial polymer 2 upgrade
Some cleanups
Update documentation
Merge pull request #1 from urdeveloper/polymer-2

File Changes

M README.md
https://github.com/google/uniflow-polymer/pull/12/files#diff-0 (119)
M action-dispatcher.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-1 (160)
M action-emitter.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-2 (79)
M application-state.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-3 (277)
M bower.json
https://github.com/google/uniflow-polymer/pull/12/files#diff-4 (22)
M index.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-5 (2)
M list-view.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-6 (246)
M model-view.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-7 (471)
M state-aware.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-8 (222)
M state-mutator.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-9 (63)
A test-all.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-10 (16)
M test/action-dispatcher_test.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-11 (69)
M test/action-emitter_test.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-12 (59)
M test/application-state_test.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-13
(113)
M test/list-view_test.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-14 (79)
M test/model-view_test.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-15
(117)
M test/state-aware_test.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-16 (32)
M todomvc/README.md
https://github.com/google/uniflow-polymer/pull/12/files#diff-17 (2)
D todomvc/bower.json
https://github.com/google/uniflow-polymer/pull/12/files#diff-18 (21)
M todomvc/elements/todo-action-dispatcher.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-19 (83)
D todomvc/elements/todo-action-dispatcher.js
https://github.com/google/uniflow-polymer/pull/12/files#diff-20 (74)
M todomvc/elements/todo-app.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-21 (71)
D todomvc/elements/todo-app.js
https://github.com/google/uniflow-polymer/pull/12/files#diff-22 (32)
M todomvc/elements/todo-footer.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-23 (54)
D todomvc/elements/todo-footer.js
https://github.com/google/uniflow-polymer/pull/12/files#diff-24 (32)
M todomvc/elements/todo-header.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-25 (84)
D todomvc/elements/todo-header.js
https://github.com/google/uniflow-polymer/pull/12/files#diff-26 (33)
M todomvc/elements/todo-item.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-27
(100)
D todomvc/elements/todo-item.js
https://github.com/google/uniflow-polymer/pull/12/files#diff-28 (66)
M todomvc/elements/todo-items.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-29 (62)
D todomvc/elements/todo-items.js
https://github.com/google/uniflow-polymer/pull/12/files#diff-30 (19)
M todomvc/elements/todo-styles.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-31
(363)
M todomvc/index.html
https://github.com/google/uniflow-polymer/pull/12/files#diff-32 (14)

Patch Links:

https://github.com/google/uniflow-polymer/pull/12.patch
https://github.com/google/uniflow-polymer/pull/12.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/uniflow-polymer/pull/12, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGGGYeFkPRu67ceijm4hso_f60iuSu1Aks5sitCkgaJpZM4PZapV
.

Thanks again @urdeveloper for your contribution. I reviewed the code and merged your pull request with some minor additions/corrections. Good quality work, really appreciate your help with the project!
-Andrei


Write tests for data_manager

Add basic tests for data_manager view.

Unit tests for data_manager are written in commit 3fd8a9a87ca94a366f241d400dec1c16274b9e38


fix(bberg): update aggregation_state usage to match bberg changes

Description
Blocked by https://github.com/AztecProtocol/barretenberg/pull/414
Checklist:

[ ] I have reviewed my diff in github, line by line.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to the issue(s) that it resolves.
[ ] There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
[ ] The branch has been merged or rebased against the head of its merge target.
[ ] I'm happy for the PR to be merged at the reviewer's next convenience.

bberg changes are being reverted


Better bulk uploading (NFT.storage)

This uploads the images directory in one shot, but still needs to process each metadata file to assign the images. Then NFT.storage lib seems to have this capability automatically, but I could only find it when storing a single image at a time. I haven't yet tested in bulk

Thanks for this!
This still has the issue with sort order... see #110 for context. Also, in #110, I switched to uploading the images directory in one shot. Have a look and let me know what you think @a-mummey.


Feature/elasticsearch last

Elasticsearch bug is fixed, now you can save data outside of localhost

Remove changes in the policy of ElasticSearch. You should use your own config files.

Looks fine!! merged


Add iand675 packages

There are a few upper bounds problems. Let me know if you have any questions

OK, I've got a new version up for the serf & flowdock packages that work with the stackage nightly config. Is there anything I need to do now?


E_DEPRECATED errors in PHP 7

What steps will reproduce the problem?
Call a method inside another controller statically
I have two controllers A and B. Class B extends Class A. My script calls self::actionFunc($id); Which is a non-static method. I cannot make it static because Yii does not support action methods to be static. Also I can't initialize the object. I get parameter error if I do ((new B())->actionFunc($id);
What is the expected result?
It should return the response from B::actionFunc($id); method.
What do you get instead?
E_DEPRECATED error
Additional info

Q
A

Yii version
2.0.8

PHP version
7

Operating system
CentOS

I unsure what you traying to do.
Why you calling action method as static?
Just use action to handle request.

there are better ways to do it. Make an action class and use it on both controllers
http://www.yiiframework.com/doc-2.0/guide-structure-controllers.html#standalone-actions

@Faryshta suggested right thing


Market analysis takes the lowest market rate and does not ignore dust orders

Market analysis takes the lowest market rate and does not ignore dust orders
Seemingly it keeps the BTC popping up dust lending orders at 0.01xxxx but just less than 0.1 bitcoins above that, people are lending at 0.09
see attached

also, really can't figure out what a percentage of YOUR balance means. Could the explanations add a practical example for ease of understanding?

I can confirm some weird market analysis behavior - I'll look into that.
Didn't get your 2nd comment is it related to the issue?

Thanks for the updates. I've been having issues putting the gapbottom and gaptop functions into practical application. Pasting the instructions here for reference:

gapbottom is how far into the lending book (in percent of YOUR balance for the respective coin) the bot will go to start spreading lends...
--  10% gapbottom is recommended to skip past dust at the bottom of the lending book, but if you have a VERY high volume this will cause issues as you stray to far away from the most competitive bid.

gaptop is how far into the lending book (in percent of YOUR balance for the respective coin) the bot will go to stop spreading lends.
--  This value should be adjusted based on your coin volume to avoid going astronomically far away from a realistic rate.

Writing out a practical scenario of what it might have been programmed to function:

A man has 3 BTC. 1 in exchange, 1 in margin, and 1 in Lending.
Transfer is set as off.
The bot takes the 1 BTC as lending as the "balance"
gapbottom is set at 10, gaptop is set at 200
The bot calculated values for gapbottom and gaptop are 0.1 BTC and 2 BTC respectively
The 'spreadlend' is set at 3
The bot ignores the first 0.1 BTC. In the above sceenshot, it would have ignored the first two offers. The effective calculated gapbottom loan rate is now 0.100000%
The bot looks for the gaptop. As the gaptop value is 2 BTC, it deducts alls the loans from the bottom up, to reach the 2 BTC mark. In the above screenshot, because someone is trying to lend 22 BTC at 0.1000, the 2 BTC mark would be calculated as 0.100000%
The spreadloans take effect. As the gapbottom and gaptop values are 0.100000%, the three loans are placed at 0.100000%

Does that sound about right?

Yes, that's sounds right.
Therefore my config values are:
gapbottom = 100
gaptop = 3000

@fireburndread can we close this issue?


Training command

The training command mentioned in README.md has an issue.
python preprocessing/preprocessing_utk.py --data_dir '/path/to/cacd/images' --output_dir 'path/to/save/processed/data'
Has to be preprocess_utk.py. The file is misnamed.

Thanks for the suggestion, I will push a fix

Fixed


Update supabase.py, add filter to query (matches latest supabase docs & js)

Description: Update supabase to support optional filter argument (if present, used, if not, doesn't break things)
Tag maintainer: @rlancemartin, @eyurtsev

ok, black/mypy are clean.

Oh dang, I was just about to write this actually! Thank you for this!!


Style landing page

A draft for landing page is done..... open to feed back for improvements.  The button does work, taking user to the form page.


change mkdir --parent to mkdir -p

Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock
[ ] Style conformance: stylish-haskell

Jenkins: test this please


Update LICENSE file

Ok, last PR on license/copyright stuff today, promise! Turns out the copyright line belongs in the LICENSE file and that should be the single source of truth for this info.

          1 Warning

:warning:
Please assign someone to your PR

  Generated by :no_entry_sign: Danger


Installer Issue

I cant connect to the database. I think its because the "-" in the database name.
I cant change the name of the database.

PHP: 8.3.14
MariaDB: v10.11.10
Thanks

Hello @JnsGSK
Thx for this report. I fixed that in dev branch. Will be in the next release. Either you use dev branch or wait for the next release.

For documentation: Fixed with 789b9af52375b4a7cbf50420bfbbe26bf0943c0d

Works perfect!
Thank you for the fast Support!


Variant beta filter does not work on current main

Describe the bug
Load variant filtration page. The quick presets don't work.
To Reproduce
Steps to reproduce the behavior:

Got to beta filter
See that the presets are not loaded correctly.

Expected behavior
The presets should be loaded.
Screenshots

Additional context
N/A

Root Cause Analysis
We need the following patch.
diff --git a/variants/vueapp/src/stores/filterQuery.js b/variants/vueapp/src/stores/filterQuery.js
index 408d1fb..28c939d 100644
--- a/variants/vueapp/src/stores/filterQuery.js
+++ b/variants/vueapp/src/stores/filterQuery.js
@@ -37,7 +37,7 @@ const fetchPresets = async (
       ] +
         ['frequency', 'impact', 'quality', 'chromosomes', 'flagsetc'].map(
           (category) =>
-            variantsApi.fetchCategoryPresets(csrfToken).then((presets) => {
+            variantsApi.fetchCategoryPresets(csrfToken, category).then((presets) => {
               categoryPresets.value[category] = presets
             })
         )


[bigtable] bump client version to 0.9.3

Current coverage is 45.02% (diff: 100%)

Merging #109 into master will not change coverage

@@             master       #109   diff @@
==========================================
  Files           596        596
  Lines         15377      15377
  Methods           0          0
  Messages          0          0
  Branches       1581       1581
==========================================
  Hits           6924       6924
  Misses         7992       7992
  Partials        461        461

Powered by Codecov. Last update 2a16276...577aeb9


feat: auto-merge minor versions

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Language cleanup

The random_sentence() function used for benchmarking in the "inference/inf2-bert-on-sagemaker/inf2_bert_sagemaker.ipynb" notebook contains some potentially offensive random terms. Can we please replace those?

Thanks @brianloyal . Will work on this.


Setting icon in our CSV

Since the project doesn't have it's own logo,
the generic Openstack logo will be used instead, as it is by openstack-operator.
Implements: https://issues.redhat.com/browse/OSPRH-8225

recheck

/cherry-pick 18.0.0-proposed


Update 2019-04-01 API as changes from 2019-06-01.

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Copied changes from https://github.com/Azure/azure-rest-api-specs/pull/6894 to older API version
ARM has already signed these changes off

Automation for azure-sdk-for-go

CI failed due to SDK Java getting timed out after 2 hours
@tjprescott could you please take a look at this PR? This is a copy of changes from another API version, ARM has already signed off them (https://github.com/Azure/azure-rest-api-specs/pull/6894)

/azurepipelines run

Restarting CI. I can't merge PRs unless all required checks pass.

@sergey-shandar this keeps failing on the Java step, preventing me from merging.

/azurepipelines run

/azurepipelines run

reopen PR to trigger CI. prevoius CI failed due to SDK java 2  hours timeout


Add missing header text

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

Project coverage is 48.87%. Comparing base (e21f499) to head (634862b).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #144      +/-   ##
===========================================
- Coverage    48.88%   48.87%   -0.02%
===========================================
  Files         1188     1188
  Lines        98305    98305
  Branches     14745    14746       +1
===========================================
- Hits         48060    48046      -14
- Misses       50245    50259      +14

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


Update kubevirtci default to 1.25

What this PR does / why we need it:
The k8s-1.23 kubevirtci provider no longer exists - update the default to k8s-1.25
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Issue with the periodic nightly push - k8s-1.23 is not found
https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/logs/periodic-containerdisks-push-nightly/1625676304881815552#1:build-log.txt%3A140
Special notes for your reviewer:
/cc @0xFelix @lyarwood
Release note:
NONE

Sorry @brianmcarey I didn't see this before posting a slightly different version under https://github.com/kubevirt/containerdisks/pull/46, would you mind if we closed this?

Yes sure.
/close


Update index.tsx

I have added code for custom icons

@Darshan292
Thanks for the creating this pull request to add an example of how to use an svg. I see now that the StaticCloud on ln 16 has a custom img tag.
Perhaps you can update the StaticIconCloud to include another  who's img is a hosted svg?


2018f

[x] Get original script working with current OSM data
[ ] #46
[ ] #42
[ ] #41
[ ] #40

does 2018f tzdata contain real changes to zones?

The timezone database release didn't create any new zones.  However, this release of timezone-boundary-builder will include a lot of changes to various zones.

Two zones in Greenland will be switch to fix an error (#40)
Many new zones in Antarctica will be added to this project (#42)
A very small fix to correct the border along America/Argentina/Catamarca and America/Argentina/Rio_Gallegos (#46)
Existing zone boundaries will be slightly updated due to changes in OpenStreetMap.
Some zones will now overlap in areas where two territories claim the same area or in areas where there are two or more different timekeeping methods used by different people in the same area (#41)

Looks like 2018g has just been released, so the work being done in here will be released under that name... unless yet another release is done before this is finished.

Just a quick update, I've been involved with other things so haven't had time to complete this update just yet.  But, I'll definitely have a new release by the end of next week.

Release 2018g is now available!


Is there a transpiled version (or older version) without ES5 code?

I am using this library with create-react-app, but it causes a problem when I tried to compile my code into a production build, because uglifyjs in used by create-react-app doesn't work with ES5 code. See more discussion on
https://github.com/facebookincubator/create-react-app/issues/1125

Until then I recommend to use libraries that take care of transpilation step.

I am mainly looking at the Matrix and PCA functionality currently.
Or if you have idea on transpilation (e.g. with babel), please provide some guidance, and I could try it  myself.

I don't use create-react-app but if you can play with the webpack config, I suggest you to replace uglifyjs with babili. That's what we use to minify our applications.
https://github.com/webpack-contrib/babili-webpack-plugin

Thanks for the tip. Babili seems promising, but for now, I reimplemented the few functions I needed (matrix transpose and PCA) myself.


Issues with image_picker library support for web

In PR https://github.com/flutter/plugins/pull/2767 it is mentioned that it has been fixed for web but i do not see any code for web added in the library documentation. Is it really fixed? if so can someone provide me with examples of using it on web as well as mobile.

Mentioned PR is closed for https://github.com/flutter/plugins/pull/2802 and it is still open
Please wait for it to be merged and new release of image_picker Reference
if you disagree please write in the comments and I will reopen it.
Thank you


Does it support android? I can't receive location on android.

Hi, what version of RN are you using? are you with the last version of master?

I'm going to close this issue due I need more information and I get no response


Feat/add algo cma me

Related to issue #80
Done:

first implem of CMA-ME (improvment emitter)
example notebook
reproduce results on ragistrin (results from DQD paper)

Remaining tasks:

add the random emitter
add the opt emitter
double check perfs
refactor CMA-ES
refactor CMA-MEGA
refactor CMA-ME
add tests
update documentation

Codecov Report

Merging #86 (f20d645) into develop (82262e5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #86   +/-   ##
========================================
  Coverage    89.60%   89.60%
========================================
  Files           67       67
  Lines         3914     3914
========================================
  Hits          3507     3507
  Misses         407      407

Impacted Files
Coverage Δ

qdax/core/emitters/cma_mega_emitter.py
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix Typo for HEIMAN HS1CG-M

Fix typo in zigbeeModel: GasSensor-EM
Tested with actual hardware

This zigbeeModel is already used by another device: https://github.com/Koenkk/zigbee-shepherd-converters/blob/045145cd1c5e1716b87ce11161caadf33c8251de/devices.js#L5563. So it should already be supported in zigbee2mqtt 1.6

Several observations:
git clone https://github.com/Koenkk/zigbee2mqtt.git
npm install
grep -r GASSenso *
node_modules/zigbee-shepherd-converters/devices.js:        zigbeeModel: ['GASSensor-N'],
node_modules/zigbee-shepherd-converters/devices.js:        zigbeeModel: ['GASSensor-EN'],
=> It is not bringing up the latest devices.js version and does not contain the Piri sensor you are referring to.
Sorry that I didn't compare to the latest git status.
Further observations are all based on the above pull.
I have an Heiman HS1CG-M no Piri. I see this in the logs:
{"ieeeAddr":"0x000d6f0013c8bf90","type":"Router","model":"GASSensor-EM","friendly_name":"0x000d6f0013c8bf90","nwkAddr":15106,"manufId":4619,"manufName":"HEIMAN","powerSource":"Mains (single phase)","modelId":"GASSensor-EM","hwVersion":16,"dateCode":"2018.3.26"}]}
Without my change I get:
info: 0x000d6f0013c8bf90 (0x000d6f0013c8bf90): unkown - undefined unknown (Router)
with the change:
info: 0x000d6f0013c8bf90 (0x000d6f0013c8bf90): HS1CG-M - HEIMAN Combustible gas sensor (Router)
I have not tried with the latest version of devices.js. Let me know if that is helpful?
But the questions I have are:
Is there such a thing like a Heiman HS1CG-M which is called GASSensor-EN? Or is it still a typo?
I don't understand the details yet, could there be such a thing like 2 times the same zigbeeModel from 2 vendors?

GASSensor-EN is a valid device (at least I'm quite sure, otherwise it never worked for somebody). And 2 times the same zigbeeModel from 2 vendors isn't exceptional, probably the device is exactly the same, only to logo on it is different.
I just checked and your device should be supported in the latest zigbee2mqtt dev branch.


fix typo

First of all thank you for this kind of condensed knowledge about go, very useful 👍
I just corrected some insignificant typo mistakes 🎃

Thanks for the catch!


Grunt Templates support

I tried to use Grunt Templates in src but it does nothing:
dist: {
	options: {
		…
	},
	files: {
		'<% yeogurt.dist %>/styles/main.css': '<% yeogurt.client %>/styles/*.styl'
	}
}

grawl@devastator ~/S/g/grunt> grunt stylus:dist --verbose
Initializing
Command-line options: --verbose

Reading "Gruntfile.js" Gruntfile...OK

Registering Gruntfile tasks.
Reading index.json...OK
Parsing index.json...OK
Loading "Gruntfile.js" tasks...OK
+ build, default, serve, test

Running tasks: stylus:dist

Loading "grunt-contrib-stylus" plugin

Registering "/Users/grawl/Sites/grawl.ru/grunt/node_modules/grunt-contrib-stylus/tasks" tasks.
Loading "stylus.js" tasks...OK
+ stylus

Running "stylus:dist" (stylus) task
Verifying property stylus.dist exists in config...OK
Files: [no src] -> /styles/main.css
Options: banner="", compress, sourcemap=false, paths=["client/bower_components","client/styles/"]

Done, without errors.

Without Grunt Templates it does:
dist: {
	options: {
		…
	},
	files: {
		'dist/styles/main.css': 'client/styles/*.styl'
	}
}

…
Files: client/styles/main.styl -> dist/styles/main.css
Options: banner="", compress, sourcemap=false, paths=["client/bower_components","client/styles/"]
Reading client/styles/main.styl...OK
Writing dist/styles/main.css...OK
File dist/styles/main.css created.

Done, without errors.

As you can see in your logs, they works fine: Files: [no src] -> /styles/main.css. You need to print variables: <%= bla %>.

 Thank you.


CAMEL-11623: Catch throwable

Fix for issue discussed in:
https://issues.apache.org/jira/browse/CAMEL-11623
Pretty straightforward, no additional changes.

Did you read the last comment in the JIRA issue? Can you also add the issue number in the commit message? Thanks!

Made the change as you requested.


Update pto-request-approval-process.md

URL of RTD document: https://civicactions-handbook.readthedocs.io/en/hhartingrex-patch-2/

Suggested adjustments, as per the Project Management Practice Area. Thank you!

NOTE: This page conflicts with https://handbook.civicactions.com/en/latest/040-employee-handbook-us/benefits-and-holidays/#holidays
Are we able to only have one page?


chore: use the latest JWA image (#346)

chore: use the latest JWA image

canonical/kubeflow-rocks#81 introduced changes in the rockcraft project that improved readability and set a standard. This commit uses the version of the container image that corresponds to that PR.

Re-running the CI as the Integration tests step ran for more than 2h. If this happens again, let's check the CI.

I tried to use this image and run it with upstream KF 1.8. The pod becomes ready (because pebble doesn't crash) but I see the following logs:
2024-04-18T12:26:13.881Z [pebble] Started daemon.
2024-04-18T12:26:13.894Z [pebble] POST /v1/services 6.295847ms 202
2024-04-18T12:26:13.898Z [pebble] Service "jupyter-ui" starting: /bin/bash -c gunicorn -w 3 --bind <IP_ADDRESS>:5000 --access-logfile - entrypoint:app
2024-04-18T12:26:13.985Z [jupyter-ui]
2024-04-18T12:26:13.985Z [jupyter-ui] Error: No application module specified.
2024-04-18T12:26:14.001Z [pebble] Change 1 task (Start service "jupyter-ui") failed: cannot start service: exited quickly with code 1
2024-04-18T12:26:14.010Z [pebble] GET /v1/changes/1/wait 116.367625ms 200
2024-04-18T12:26:14.011Z [pebble] Started default services with change 1.

After ssh-ing into the pod and trying to run the command printed above from the /src/ directory I get the same error

Error: No application module specified.

After looking closer at the upstream command I see the following differences:
ROCK
https://github.com/canonical/kubeflow-rocks/blob/main/jupyter-web-app/rockcraft.yaml#L19
"/bin/bash -c gunicorn -w 3 --bind <IP_ADDRESS>:5000 --access-logfile - entrypoint:app"

Upstream Image
https://github.com/kubeflow/kubeflow/blob/v1.8-branch/components/crud-web-apps/jupyter/Dockerfile#L59
ENTRYPOINT ["/bin/bash","-c","gunicorn -w 3 --bind <IP_ADDRESS>:5000 --access-logfile - entrypoint:app"]

If I instead run /bin/bash -c "gunicorn -w 3 --bind <IP_ADDRESS>:5000 --access-logfile - entrypoint:app" inside the container then the command succeeds.

I have fixed the issue here. The reason why the charm is working is because the charm code sets the correct command in the code. I will close this PR as the image that we need to use will be different.


[Storage] Make StorageIT easier to setup with new projects.

This PR is an attempt to fix #3594.

@andreamlin could you do a pass?

Thanks for the review @andreamlin, could you elaborate more on what you mean by "another PR for the implementation side"?

Is this just a test fix, and/or an example to users for how to set up KMS
for Storage?
On Tue, Aug 28, 2018 at 8:33 AM Frank Natividad<EMAIL_ADDRESS>wrote:

Thanks for the review @andreamlin https://github.com/andreamlin, could
you elaborate more on what you mean by "another PR for the implementation
side"?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-java/pull/3608#issuecomment-416632495,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD1KsgmvcdSF8xpNxY4dLpB-QnQgy8TCks5uVWKsgaJpZM4WLzth
.

It's a test fixes to make them easier to setup on a new project.

Ok, this looks good to me then.
On Tue, Aug 28, 2018 at 9:28 AM Frank Natividad<EMAIL_ADDRESS>wrote:

It's a test fixes to make them easier to setup on a new project.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-java/pull/3608#issuecomment-416652239,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD1KsoTV-8lJru7smQHiLcYIXG3Wv-oKks5uVW-vgaJpZM4WLzth
.

Thanks @andreamlin, @garrettjonesgoogle PTAL when you have a moment. Thank you!

Thanks @garrettjonesgoogle


python310Packages.bthome-ble: 2.4.0 -> 2.4.1

Changelog: https://github.com/bluetooth-devices/bthome-ble/blob/v2.4.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 209214 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.bthome
python310Packages.bthome-ble
python39Packages.bthome-ble


Dependencies not found running vue serve

Linux Mint 19.xx
npm 6.13.1
vue 4.1.1
These dependencies were not found:

core-js/library/fn/array/from in ../node_modules/bootstrap-vue/esm/utils/array.js
core-js/library/fn/array/is-array in ../node_modules/bootstrap-vue/esm/utils/array.js
core-js/library/fn/object/assign in ../node_modules/bootstrap-vue/esm/utils/object.js
core-js/library/fn/object/is in ../node_modules/bootstrap-vue/esm/utils/object.js

Do not run npm run serve from src folder.


Remove unneeded heartbeat logic with sketchy time addition

Tentatively approving pending the lint fix, but I also have a question - is the heartbeat not necessary anymore? What is the consequence of removing this code?

The expiry duration is static for v2, so it is no longer needed.


Add in reusability and new primitives

📚 Context
Please describe the project or issue background here

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Other, please describe:

🧠 Description of Changes

Add bullet points summarizing your changes here

[ ] This is a breaking API change
[ ] This is a visible (user-facing) change

Revised:
Insert screenshot of your updated UI/code here
Current:
Insert screenshot of existing UI/code here
🧪 Testing Done

[ ] Screenshots included
[ ] Added/Updated unit tests
[ ] Added/Updated e2e tests

🌐 References
Does this depend on other work, documents, or tickets?

Issue: Closes #XXXX

Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

This still needs to refactor the list of sequential continuous and discrete categorical colors. Maybe that can be done in another pr but there's definitely some duplication here. The only thing is that how plotly and altair accepts those colors is different (plotly is an array of arrays, while altair is an array).
I will think more about this and try to commit a first attempt of refactoring later.


Encryption functionality

First of all, awesome lib you have here.
Does the library provide encryption functionality after getting the Rsa public key from a X509 certificate?
Something like:
val cipher: Cipher = Cipher.getInstance("RSA/ECB/PKCS1Padding")
cipher.init(Cipher.ENCRYPT_MODE, rsaPublicKey)
val encryptedPayload: ByteArray = cipher.doFinal(initiatorPasswordByteArray)

We're working on it (see draft pull request). but it is going to take a little, since we want to support attestation / deviceCheck and the possibility for remote crypto providers out of the box. You can still use the platform functionality in the meantime

To be more precise: Once signing is there, asymmetric encryption comes easy-peasy. Symmetric encryption is planned for later. Since you are asking about asymmetric encryption, some time in September might work out for a release with that feature set. That depends on how fast we get signing going, though

Signing is progressing nicely. ECDH key agreement and symmetric encryption is planned next. Asymmetric encryption might be possible in parallel. Out of curiosity: why do you want to RSA encrypt?


Images

Ensure the image elements are accessible and have alt attributes that allow non visual users to use the website.

Image elements have had "alt" attributes added. In addition "title" attributes have been added to the 3 classes within "Content".
For class "hero" a "title" element has been added since CSS formatting creates background image.


Question with smatch installation

Comment:
I am attempting to use the SSI code found here: https://github.com/LSSTDESC/ssi-cosmodc2/ which requires the smatch package to be installed. However, I am coming across the following error:
ImportError: cannot import name '_smatch' from partially initialized module 'smatch' (most likely due to a circular import)
Here is the traceback:
`Traceback (most recent call last):
File "process_tract_patch.py", line 15, in <module>

    from ssi_tools.matching import do_balrogesque_matching

  File "/global/u2/j/jjeffers/SSI/2020_08_02_coadd_sep/ssi_tools/matching.py", line 2, in <module>

    from smatch.matcher import Matcher

  File "/global/u2/j/jjeffers/SSI/2020_08_02_coadd_sep/smatch/__init__.py", line 2, in <module>

    from . import smatch

  File "/global/u2/j/jjeffers/SSI/2020_08_02_coadd_sep/smatch/smatch.py", line 4, in <module>

    from . import _smatch`

Any help appreciated!

Can you post the installation procedure you used plus the other information (conda list and conda info)?

Ohhh. Install from conda using conda forge.

So I deleted the package and reinstalled as you had said, however now I am getting a new issue
ModuleNotFoundError: No module named 'smatch.matcher'; 'smatch' is not a package
I did conda list and smatch is now listed under the conda-forge channel--I'm not sure why the script can't seem to find it.

Try conda install spherematch

Actually conda install -c conda-forge spherematch


[AKS] az aks create/update: Update AMW resource.location to remove extra characters

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

AKS

Add @FumingZhang to help review

Queued live test to validate the change.

test_aks_create_with_azuremonitormetrics
test_aks_update_with_azuremonitormetrics


Dashboard web / docker image

I'm a big fan of desktop applications, but it would be really cool if your dashboard implementation could be made available in a web interface, so we could make it available via Docker and make it easier to deploy in home environments.
It would be a huge advantage to be able to manage my relay from anywhere.
I'd be happy to help with testing and building images (arm64v8 and amd64) if this suggestion is viable.

That's cool! I had a similar idea before, but I couldn't think of an easy way to implement it. If you have any ideas, PRs are always welcome 🤗


Metrics Gradually Trending Towards Zero

Why
Metrics that trend towards zero over time could indicate several potential problems, such as data loss, issues with data collection methods, inaccuracies in metric calculation, or even underlying issues within the system being monitored. This trend could lead to misinformed decisions, overlooked system issues, and a general lack of trust in the data being reported. It's crucial to identify whether this trend is due to expected system behavior or if it signifies a bug or flaw in the metric collection and reporting process.

mainnet_weird.txt


[array: unnest] crash & no solution

SELECT value
FROM unnest(array(0,1,2,3,4,5))
WHERE value < 3;

Parse error: no query solution
(not sure what the behavior should be. I hoped for an other result)

SELECT value
FROM unnest (1);

crashes SQLite without error

This one took some time 😅 Fixed now.
The first one returns three rows:
SELECT value
FROM unnest(array(0,1,2,3,4,5))
WHERE value < 3;

0
1
2

The second one returns an error, because the argument to unnest is not a valid array:
SELECT value
FROM unnest(1);

Runtime error: SQL logic error


Allow setting the tag in the configuration

Sometimes helpful, our project uses tags based on git revision for example.

#5 is better, missed that...


Improve CMP HTTP test server

This is a further excerpt from #15053 to ease reviewing.

CMP test server: Extend error reporting on cert rejected for revocation
HTTP test server: Improve connection management and logging
cmp_server.c: Improve transaction management and logging
OSSL_CMP_SRV_process_request(): Log any error queue entries, which also increases stability

This PR is meanwhile superseded by #15053, which has been approved as well.
So closing this excerpt, which was provided to simplify reviewing, has become obsolete.


Added ChangeEvent to exports

Seems that ChangeEvent was missing from the import-to-export list of event types. I've added it. Hope nothing else is missing.

The onchange handler actually exposes a callback over ChangeData, not ChangeEvent. So perhaps this pull request is redundant, and is fixing something that rather than being an oversight is a design decision.

Thanks for following up, closing


Bug: Ignore Max Player field is set to True automatically on service restart

I manually set them to false in Player Manager but they revert back to true on service restart.
Version 2.6.RC2

I will check this out! Thanks again!

Confirmed! Simple fix, will be shipped shortly! Thanks man!

Confirmed issue is now fixed. Closing issue.


Add hacktoberfest tag

I can't add tags to the repository.

@leachim6 Can you add the hacktoberfest tag to this repository?

I guess only the owner can do so. Steps To Do So

Yeah, only repo owners or collaborators can add labels to issues unless it's a template

Hey sorry all, I didn't realize you folks needed me to do these things and I had the email notifications switched off!
I'll be more wary of this in the future!

Hasn't been added?

I'm not really clear what adding this tag does, would adding "hacktoberfest" in Februrary 2022 sorta miss the boat? What am I missing here

It's in anticipation for October

Adding the tag is a thing we should just set and forget right now

@Richienb thanks for the explanation, tag's been added!


GetNumConformers() fails to generate conformers on bigger molecules

Describe the bug
When calling GetNumConformers() on bigger molecules loaded from PDB (repaired in Maestro so CONECT records are correct), no conformers are generated. This happens to two types of molecules out of 11 I am testing.
To Reproduce
I am attaching two PDBs to illustrate the issue (with *txt extension so GitHub accepts it):
structure.pdb.txt
structure2.pdb.txt
Call the following simple code with the molecule pdb.txt file as a parameter (or hardcode the names):
import sys
from rdkit import Chem
from rdkit.Chem import AllChem

rdmol = Chem.MolFromPDBFile(sys.argv[1], removeHs=False)
cids = AllChem.EmbedMultipleConfs(rdmol, numConfs=1)
confs = rdmol.GetNumConformers()

print(cids, confs)

Expected behavior
One conformer should be generated (confs variable should be 1).
Configuration (please complete the following information):

RDKit version: 2020.09.3        py37h400b6df_0    conda-forge
OS: Linux RedHat
Python version (if relevant): 3.7.8
Are you using conda? yes
If you are using conda, which channel did you install the rdkit from? conda-forge

Additional context
This affects other tools using RDKit like OpenFF (see discussion here: https://github.com/openforcefield/openff-toolkit/issues/822).

Thanks @greglandrum ! useRandomCoords=True seems to help!
Do I understand correctly that useRandomCoords somehow translates&rotates so the coordinates are different comparing with the input PDB, but the conformer is generated based on input topology?

Thanks @greglandrum ! useRandomCoords=True seems to help!
Do I understand correctly that useRandomCoords somehow translates&rotates so the coordinates are different comparing with the input PDB, but the conformer is generated based on input topology?

Not exactly. The original coordinates are not used in any of the RDKit's conformation generation approaches.
useRandomCoords replaces steps 3 and 4 described here:
http://www.rdkit.org/docs/GettingStartedInPython.html#working-with-3d-molecules
Instead of picking a particular distance matrix and embedding it, the useRandomCoords option picks a random set of coordinates for the atoms and then minimizes their positions using the "distance geometry force field" (step 5 on the page above).

Not exactly. The original coordinates are not used in any of the RDKit's conformation generation approaches.
useRandomCoords replaces steps 3 and 4 described here:
http://www.rdkit.org/docs/GettingStartedInPython.html#working-with-3d-molecules
Instead of picking a particular distance matrix and embedding it, the useRandomCoords option picks a random set of coordinates for the atoms and then minimizes their positions using the "distance geometry force field" (step 5 on the page above).

Thanks @greglandrum!

Thanks @greglandrum!

I'm closing this because it looks like the question is answered. Please reopen if necessary.

I'm closing this because it looks like the question is answered. Please reopen if necessary.


+[ZWATCH_NOW√18+!] one girl one frog Viral Leaked Video 2024 Original NOW On Social Media X Telegram [Trending√S.E.X.70K]

11 sec ago one girl one frog Leaked Video Viral On Social Media. +[ZWATCH_NOW√18+!] one girl one frog Viral Leaked Video 2024 Original NOW On Social Media X Telegram [Trending√S.E.X.70K]

one girl one frog's newest video has gone viral, capturing the attention of internet users and sparking extensive conversations across social media. This emerging digital creator, known for her engaging content, particularly her recent Spiderman-themed video, has seen a significant rise in interest from fans eager to find out where they can watch the full version. Many are turning to platforms such as Instagram, X (formerly Twitter), and Discord to access her content.
As the excitement builds, fans are actively sharing their reactions online, showcasing the powerful influence of social media in elevating creators like one girl one frog into the spotlight. With her unique style and creative flair, she has quickly established herself as a prominent figure in the digital landscape, leaving many curious about her upcoming projects and content.
[Trending√_S.E.X√NOW!] one girl one frog Viral Leaked Video 2024[FULL70K]

thank you

fg

mh

jf

gh

dgh

syfhgsdf
dgdkdsg

FG

.
.
.

vf
.
.

.

.
.


implement changes to service review API

Fixed https://github.com/eclipse-xpanse/xpanse/issues/2107
Register service template:

Get service template requests pending to review:

Review the register request:

Update the approved service template:

Review and approve the update request:

After the update request approved, the Ocl copied from  table service_template_request_history to table service_template:

Unregister the service template:

No requests pending to review, because of the unregister request always is approved.
Re-register the unregistered service template:

Review and reject the re-register request:

LGTM


Tilde in SDK path isn't supported

If a user uses an SDK path like ~/path/to/dart then the plugin won't activate.

So tilde's are supported in the input field? Cool!

Looks like no, from testing it. Also not env variables. But we do do a much better job of letting users know about failing to locate an sdk; hopefully the user will have better tools to diagnose the issue.

Reopening as we might want to support tilde one day. Glad we have better messaging, though.

Fixed... https://github.com/dart-atom/dartlang/commit/1de5949f2357385985591cfad05c175cd77e8642


Doesn't work for youtube tv

Homework

[x] I looked at nativefier --help and https://github.com/nativefier/nativefier/blob/master/API.md
[x] I checked the Troubleshooting section of the README.
[x] I searched existing issues, open & closed. Yes, my bug is new.
[x] I'm using the latest version available at https://github.com/nativefier/nativefier/releases

Bug description
A clear and concise description of what the bug is.
Steps to reproduce
nativefier --name "TV" 'https://tv.youtube.com/'

Expected behavior
youtube tv would work signed in
Actual behavior

Google sites block login on electron apps as they not real chrome, so you must change the user agent to something else. Build with --user-agent "safari" or --user-agent "firefox". Login will probably still fail at first attempt, so click 'try again' and give it a few (up to 5) shots.

I will try that. I mainly asked this here to aid others that may try YouTube TV in the future. This post should be useful for the next person with the same issue searching google. Thanks for your response.


Links not clickable on Samsungs Galaxy S3 native browser

Hi there,
first of all: thank you for this great plugin!!
This is just what I needed and it works perfectly for me.
Except for one thing...
I made a website responsive with sidr, the navigationbar works great in the browsers I tested...but I did not test with the native internet browser of my Samsung Galaxy S3, running Android 4.3.
The links of  the navigation-bar are not clickable in that browser.
Like there is some overlay or something.
Has anyone the same problem and/or any solution to this?

Is this still happening?


feat: adds custom app download banner in place of smart banner

Closes DIA-733, DIA-734
This PR adds a custom app banner that rotates through a few bits of copy, in place of the native smart banners
Will keep on draft until I know we can launch this separately from the footer popup.

Closed in favor of https://github.com/artsy/force/pull/14256


These new approaches seem to work:
python -m vandal.cli.disassemble -h
python -m vandal.cli.decompile -h

Whereas the version I was going for still needs a little work to support nargs properly:
python -m vandal -h

Want me to recreate these changes so it's ready to merge, or did you want to wait until that^ version is ready?

Closing due to inactivity


do starter not lazily load bundles?

I just cloned this repo, made npm i and run npm run build:prod then npm run server:prod. When I open it in browser, console shows me following logs:
Initial App State Object {}
main.5bf4ea0….bundle.js:1 hello `Home` component
0.993da87….chunk.js:1 `Detail` component loaded asynchronously
0.993da87….chunk.js:1 `Detail` bundle loaded asynchronously
1.2f878f0….chunk.js:1 `Barrel` component loaded asynchronously
1.2f878f0….chunk.js:1 `Barrel` bundle loaded asynchronously
5.fe560f6….chunk.js:1 `ChildDetail` component loaded asynchronously
5.fe560f6….chunk.js:1 `Detail` bundle loaded asynchronously
6.aab8402….chunk.js:1 `Barrel` component loaded asynchronously
6.aab8402….chunk.js:1 `ChildBarrel` bundle loaded asynchronously

However I didnt navigate throw the different tabs. So, the question is, does lazy load works correctly right now?

or probably its just a false alarm and routing preloading does this thing....

it's preloading

This seems to be caused by the change from angular-router-loader to ng-router-loader. Is it bad that it is preloading?

if its preloading then it is expecting behaviour

angular2-router-loader TO ng-router-loader for preloading.
At the first page the site call many chunk files. so is it Good?
in version 2.2 i saw first page just has 1 chunk and go to sub page site will call new chunk.
*version 2.4: A site maybe up to 6.2MB (without Gzip) - AoT Mode
*version 2.2 : A site is about 2.7MB (without Gzip) - Prod Mode
I have a little bit confusing in this problem. Could u tell me which one is better?

@pleerock
export const ROUTES: Routes = [
  { path: 'detail', loadChildren: () => '../my-ng-modules/details#DetailModule?loader=sync' },
];

if you want to make lazy load Module,  pls check this https://github.com/shlomiassaf/ng-router-loader#synchronous-loading


infura-refresh: detect when the ASG has instances in standby

If a previous deployment doesn't go well, it can happen that some instances are left for the ASG in "standby" state, meaning that they are still managed by the ASG but not actively serving traffic. They are left around for debugging.
In this situation, the infura-refresh strategy get confused and wait for new instances that will never come online. Eventually it will trigger the timeout and revert the deployment.
This strategy should at the minimum detect that situation and refuse to proceed. Ideally it would handle this case and successfully execute the refresh.

After more investigation, it turns out that this tool handle this situation just fine.


setHeight accepts invalid values

would it make sense to prevent Settings::setHeight to accept invalid values?
public function setHeight(int $height): self
    {
        $this->height = $height;
        return $this;
    }

by changing it into
        $this->height = max(1, $height);

That makes sense, fixed.


review suggestion: move types to internal/dataplane/config

Review suggestion to #5092.

Moved DBMode and RouterFlavor enums from internal/util/dataplane to internal/dataplane/config.
Added a linter rule for internal/dataplane/config alias to be enforced to dpconf
Changed DBMode's helper functions to methods

Thanks, will put into my PR.


feat!: clean up deprecated Databases and DataTypes

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Breaking change/chore

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

This PR cleans up old migrations and old typings for Collections and data types (old, meaning >= 12 months). This means that direct data migrations will now only be supported in a 12 month window. "Structural" migrations from old releases have been moved to the "base" migration file 0_1_0.ts, where other such structural migrations already existed.

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report

Merging #713 (5b92f41) into release42 (aa685a2) will increase coverage by 5.49%.
The diff coverage is n/a.

:exclamation: Current head 5b92f41 differs from pull request most recent head 0030349. Consider uploading reports for the commit 0030349 to get more accurate results

@@              Coverage Diff              @@
##           release42     #713      +/-   ##
=============================================
+ Coverage      69.76%   75.25%   +5.49%
=============================================
  Files            306       94     -212
  Lines          34677    24096   -10581
  Branches        4703     2345    -2358
=============================================
- Hits           24191    18134    -6057
+ Misses         10036     5956    -4080
+ Partials         450        6     -444

Impacted Files
Coverage Δ

meteor/server/api/snapshot.ts

meteor/server/migration/0_1_0.ts

meteor/server/migration/1_18_0.ts

meteor/server/migration/1_19_0.ts

meteor/server/migration/1_32_0.ts

meteor/server/migration/1_33_0.ts

meteor/server/migration/1_34_0.ts

meteor/server/migration/1_35_0.ts

meteor/server/migration/1_36_0.ts

meteor/server/migration/1_37_0.ts

... and 171 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 aa685a2...0030349. Read the comment docs.


Update the reference file for regression test of acmart

Since the result of acmart depends on \the\year, the regression test for acmart fails now, although the content of apxproof.sty hasn't changed.
This pull request substitutes the file by the result of the following:
cd examples/
make acmart/test.pdf
rm acmart/reference.pdf
mv acmart/test.pdf acmart/reference.pdf

I changed the test file to fix the year to 2019, to avoid the issue. Sorry for that.


Remove limit to 4000 datapoints for local backtest

There is no need to limit the number of datapoints in the chart data when a local backtest is run.
Please remove this limit, or at least provide config setting for it when a local backtest is run.

Graphic cards / js browsers can only handle so many points and so much RAM before they choke and crash. We've experimented and found roughly 4000 is a good mid-ground which survives on the majority of browsers. There is no sufficient data-store for high resolution charting so everything is done in ram.
In the web platform we're working on allowing zooming; but this won't be available on the desktop as the data isn't available.
This could be made a config parameter with a default setting of 4000. As its hard coded in the series class this would probably mean removing the check in series and putting it into the result handlers who pull out the data.

Yes, please provide a config param when running desktop backtesting. I am working on my own charting solution for Lean engine using a charting library I am sure can handle much more than 4000 datapoints.

+1


scaling and addition of operators

Ok so in this branch, I have defined constructors, traits, and functionality for the following:

ScaledDiffEqOperator which implements (a * A)(u) = a * A(u) (based on DiffeqScaleOperator)
AddedDiffEqOperator which implements (A + B)(u) = A(u) + B(u) (based on DiffEqCombination)

This branch depends on https://github.com/SciML/SciMLOperators.jl/pull/7 which defines some basic operations. together these form building blocks for doing algebra on operators, and applying them to vectors.
I have also defined new traits, issquare, iszero, and added more functions than earlier to short-circuit operations when possible. I have also moved DiffEqArrayOperator, and FactorizedDiffEqArrayOperator to diffeq_operators.jl, and ScaledDiffEqOperator and AddedDiffEqOperator to basic_operators.jl placing building blocks together and user facing operators together made more sense to me.
the next building block left to clean up and implement is DiffEqOperatorComposition which I can do in the next PR. Then can move on to clearing up the abstract tree and figure out what depends on what
cc @ChrisRackauckas @xtalax

tests passing


README examples & Tools

Look in the documentation standard before starting
Documentation standard
TODO:

[ ] Add some examples of how the app looks like, how it behaves and whats expected in the console when sending code to the server and then what's Pepper expected to do under Examples in the README.
[ ] Add a short descriptions of the tools and frameworks used (already listed) in the README.
[ ] Create an MD-file with detailed description for every tool and framework explaining why and how it is used.
[ ] Add links to the tools and frameworks own detailed description and if there is no need to give additional information in the form of commands, flags, settings etc then just the tools and frameworks own API documentation may work for now. Aim for creating our own documentation as described in the bullet above this one.

Redo all of ti with new information and remember to follow the code standard declared in the docs repo, link in description of this issue.

Isn't this done already? @casperlundberg


Fixed SqliteDatabaseDriver with custom DatabaseFilesProvider

When opening a table the DatabaseFilesProvider is first checked for the table then if not found there we fall back to the context.getDatabasePath() call.
If a custom DatabaseFilesProvider is used then the context.getDatabasePath() won't find those databases.

Bugs with 'DefaultDatabaseFilesProvider'
On DefaultDatabaseFilesProvider, the database file was added by
databaseFiles.add(new File(filename));

It created a invalid file on directory / instead of database dir , so, when you traversal mDatabaseFilesProvider on findDatabaseFile, it will return the invalid file.

I just ran into this bug while trying to read my local WebView's cookies database located in /data/data/com.example.app/app_webview/Cookies and /data/data/com.example.app/app_webview/Web Data which were not being picked up by the Context#databaseList() call.
Could one of the senior members like @jasta look into merging this?
Thanks.

Hmm, there appear to be deeper issues here than I initially realized.  This solution isn't incorrect by itself, but highlights a greater issue whereby our databaseId is the file name itself failing to recognize that it could create a conflict with multiple database names provided across the system.  We appear to need a disambiguation strategy and a proper mapping between databaseId and location.

This fix looks locally correct and for that reason we can merge it in but I need to do some work to correct the larger problem here...


"find call hierarchy" feature

Hey!
In Xcode, you can right click on a function > "find call hierarchy"
it displays a tree showing up all the function calling it, then all the function calling these ones, and on and on. You can super easily see how a function is truly used.
Right now with CTags I can only find the definition/declaration of a function, but not its usages.

ST doesn't provide an API to display custom tree items. A text based solution in a view or panel doesn't seem viable.


Validate version mismatch

When the user has an old version of makes and attempts to run a command from a newer one
The proposal is to handle this case, showing a clear error on why it happened and how to update makes

I think the best way to do this would be by checking if both the CLI and the framework of Makes are pointing to a commit within the same version.
if not same_version(makes_cli, makes_framework):
   warn(
      f"You are currently using Makes CLI v{makes_cli.version}, "
      f"but have Makes framework v{makes_framework.version} pinned on your makes.lock.nix file. "
      f"This may cause incompatibility issues."
   )

We can also forward here: https://makes.fluidattacks.com/versioning/?h=#compatibility-information


Add an option to reject a message

Currently we can only respond to a message, remotely resolving a promise awaiting in the other frame. We'll find it useful if we could also reject a message like this:
talker.onMessage = function(message) {
  message.reject(reason);
};

Then, the reason string will be send to a talker in other window and the awaiting promise will be rejected with the reason.

Sounds like a great feature! PRs accepted :)


added guess_word_game

guess the word game and #2533

Close #2533
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

@MohitGupta121 kindly review this PR


Cherry-pick commtis in #4196 from master to stable branch

Merge pull request #4196 from CyberShadow/pull-20160413-210023
std.datetime: Update Windows time zones for KB3148851

Auto-merge toggled on


Fixing:

make it compile for redhat which use apr version that is less then 1.5.0 and doesn’t have the function apr_sockaddr_is_wildcard
make the module to not drop packets even if there is no proxy header. So users could access the web directly without proxy_protocolw.

Thank you for your contributions, and apologies for the late reply.
A few things. First of all, there are two completely separate things being mixed into one commit, so at the very least this needs to be split up: one commit addressing the compatibility with apr < 1.5, and one commit making the header optional; also, that second log-level commit should probably be merged  with the header-optional commit.
Having said that, I doubt I would accept the header-optional patch at all for security reasons. In fact, the protocol specification explicitly forbids doing that - see http://www.haproxy.org/download/1.5/doc/proxy-protocol.txt, Section 2, 2nd to last paragraph. The way to handle the situation where both direct and indirect access is allowed is to listen on two ports, one configured to expect the proxy-protocol header and which is limited to accepting connections from the load-balancer/gateway/etc, and one which is configured to not expect the proxy-protocol header and which is open to everybody else (or whatever limits you want). E.g. something like the following:

@mamanguy I have now pushed a fix for apr < 1.5 in commit 84d3f3b3b57a598d109554857f27792deefac6f8. It's done a bit differently, and more completely. But thanks for your contribution.
Regarding the option to allow missing proxy header, as I noted above, I don't think this is acceptable from a security standpoint.


P22 [Q1-24"] Expand Moderator incentive program (e.g. Give moderator badge and NFTs for training or report on profile)

Backlogged as discussed here https://docs.google.com/document/d/1jNw0qFMLzi_3xKLmKZDEINxY8omzXF5VedcXCeHh7aQ/edit


WIP: Fix layout exception with shrinkwrap

Needs test

Please ping me when you've added the test.

@knopp can we get some tests added to this?

@matthew-carroll, sorry for the delay, added test.

@angelosilvestre can you cherry pick this?


feat: go to implementation on trait functions

Fix #8537.
GIF:

bors r+

bors retry

CC https://github.com/actions/virtual-environments/issues/3418


OAuth 2.0 token exchange. Allow multiple resource parameters

OAuth 2.0 token exchange. Allow multiple resource parameters

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Attention: Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.

Project coverage is 41.16%. Comparing base (e2adc87) to head (6985315).

Files
Patch %
Lines

credentials/options.go
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1366      +/-   ##
==========================================
- Coverage   41.72%   41.16%   -0.56%
==========================================
  Files         348      348
  Lines       35104    29677    -5427
==========================================
- Hits        14646    12216    -2430
+ Misses      19979    16978    -3001
- Partials      479      483       +4

Flag
Coverage Δ

go-1.20.x
42.96% <100.00%> (-0.63%)
:arrow_down:

go-1.21.x
?

go-1.22.x
41.12% <81.81%> (-0.54%)
:arrow_down:

macOS
41.16% <81.81%> (-0.07%)
:arrow_down:

ubuntu
?

unit
41.16% <81.81%> (-0.56%)
:arrow_down:

windows
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Revert "remove upload photo step if using default avatar"

Reverts Expensify/App#44702
Testing if this revert will fix:

https://github.com/Expensify/App/issues/45842
https://github.com/Expensify/App/issues/45824
potentially https://github.com/Expensify/App/issues/45838

Fixed Issues
$ https://github.com/Expensify/App/issues/45838
Tests
First set of tests:

Open the app.
Click on the (+) plus icon and choose "Start Chat"
Select multiple people to add to the group.
Click on "Next"
Click on the avatar and upload an image.
After uploading, click on "View photo"
Remove the image and upload it again.
Create the group and check the header.
Verify that the group name in the header does not say "Chat report"

https://github.com/user-attachments/assets/461705b3-bdce-4c7a-b72e-3c5178ec2f16

Second set:

At the end it seems like the revert did not fix any of those


MacOS (M1, Ventura): Try manual download errors out and does nothing

Pressing "Try Manual Download" doesn't do anything but print this error into the stdout:
Exception in Tkinter callback
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/tkinter/__init__.py", line 1921, in __call__
    return self.func(*args)
  File "/Users/igmn/uvr/UVR.py", line 2979, in menu_manual_downloads
    manual_downloads_link_Frame = self.menu_FRAME_SET(manual_downloads_menu, thickness=5)
TypeError: MainWindow.menu_FRAME_SET() got an unexpected keyword argument 'thickness'

Replacing the line 2979 in UVR.py with:
        manual_downloads_link_Frame = self.menu_FRAME_SET(manual_downloads_menu)

fixes it


Calibration missing for human demonstrations (data tried cfg3)

Great work! When trying to run the visualize.py script using the --preprocess flag with human demonstration data, I get the following:

FileNotFoundError: [Errno 2] No such file or directory: 'rh20t/RH20T_cfg3/calib/1640674052372/intrinsics.npy'

It seems to run fine for robot collected data. I indeed see in calib folder there is no corresponding folder named<PHONE_NUMBER>372. I have also tried it for various human generated data in cfg3 downloaded data and it doesn't seem to find any of those in the calib folder since they don't exist in the data I downloaded and extracted. Can you please share these folders or provide some insights on how can I access them? Thanks!

Hi! I think some of the folders including<PHONE_NUMBER>372 are somehow lost, or incorrectly included in timestamps.npy file. You may try using an earlier calibration folder by removing<PHONE_NUMBER>372 (and all others that do not exist in the calib folder) from the list in rh20t/RH20T_cfg3/calib/timestamps.npy. We will check on this in the provided dataset files.


update docker daemon initialization to use subcommand vs -d flag

this is kind of a nit change, but there was a switch from docker -d to docker daemon in base docker.

Thanks @MHBauer! Yes we should stay in line with the docker engine doc ;) LGTM

/cc @thaJeztah @moxiegirl :palm_tree: :tropical_drink:

Yes, LGTM (but be aware that this will only work on docker 1.8; not sure what the current requirements for Swarm are)

@thaJeztah Good point! We should probably add a note instead saying that -d is deprecated in favor of daemon on engines > 1.8 then?

Might be worth adding. In Docker we only document the current version, but for Swarm it could well be that users are still on a slightly older version.

Change it back and add a note?
Or keep the changes and add a note like:
"versions of docker prior to 1.8 used the deprecated -d flag instead of the docker daemon subcommand."

LGTM

thanks @MHBauer!


gRPC C# nuget package fixes

Put tools into a separate package Grpc.Tools. Because of http://nuget.codeplex.com/workitem/595,
Grpc package itself cannot contain tools.
Fixes #1813

LGTM. Waiting for Travis.


Incorrectly worded test text for overriding class declarations with inline styles

Challenge Name

Override Class Declarations with Inline Styles
Issue Description

When trying to resolve the test cases, I misread the question. It said to make the h1 element white with an inline style and yet I made it green (following the example given). This still passed one of the tests when the wording suggested it should not have.
Browser Information

Browser Name, Version: Chrome 65
Operating System: Windows 10
Mobile, Desktop, or Tablet: Desktop

Your Code

Hello World!

Screenshot

Ha. Just went to the repo to fix it and its already been done! That'll teach me not to search before raising. Apologies.


Customizable UploadDropzone component

Hi @hardcommitoneself !
Please can you provide a little more detail, with code examples?
Thanks!

Hi, @ljwagerfield
Can we implement the following features?

customizable upload button(we can change the button style, name etc)
trigger the upload button click event from outside of the drop zone component.

we can change the button style,

You can currently change the colour through styles.colors.primary -

name, etc

You can also change the text by providing a custom localization object.

trigger the upload button click event from outside of the drop zone component.

 (Since this is what the button's click event does.) If so, yes, we can think about adding this :)

Added support for a className prop in 3.4.0:
https://github.com/upload-io/react-uploader/releases/tag/v3.4.0


PowerShell Language Support

Hi CDK team, know. Know you support .NET but would love to see support extended to PowerShell Core.

For those looking for a workaround, here is a sample repository showing one way to author CDK with PowerShell via the .NET assemblies for CDK: https://github.com/aws-samples/aws-cdk-powershell