Add openapi.json to repo instead to have an unverionized one in market repo

ok for me, although I don't see the benefit right now :-)
I thought it will be accessible in the market anyway, as it is extracted at build time: https://github.com/axonivy-market/genderize-io-connector/blob/master/genderize-io-connector-product/pom.xml#L46

When this spec will change and we need to update the connector according to spec, then we don't know which spec matches to which connector. so it's versionized.
I'll remove all the special code handling for this unversionized stuff in the market website.

sorry I didn't know that this will be generated. I'll remove it again.
It was in the market repository, so I thought this will not be generated.

thanks; no problem. In the end this openapi was crafted in this repo ... so not big issue if it actually lives here as source.
while we are not adapting the openapi spec, both ways are ok.


Añadir Pausar temporizador

Hi @martinasaid,
I just wonder what this issue is about? Google translate said the title reads "Add Pause timer".
Can you provide more detail?
Thanks,
@billhung

Potential duplicated request of #43
https://github.com/DinoChan/OnePomodoro/issues/43

Hi, I used the app and I think that a good update would be can pause the
timer when one needs. This way you have a real productivity. I hope that
you understand the idea. Thanks for ask me.
El El lun, 5 de abr. de 2021 a la(s) 9:18 p. m., 熊ビル Bill Hung <
@.***> escribió:

Hi @martinasaid https://github.com/martinasaid,
I just wonder what this issue is about? Google translate said the title
reads "Add Pause timer".
Can you provide more detail?
Thanks,
@billhung https://github.com/billhung
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DinoChan/OnePomodoro/issues/63#issuecomment-813754618,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATPAHCL7W5D7A53MZQWVGBDTHJONFANCNFSM42DHLPDA
.

That's a good idea.


Handle KongIngress deprecation in admission webhook and in KongIngress validation rules

What this PR does / why we need it:
Alternative to https://github.com/Kong/kubernetes-ingress-controller/pull/5015. Deprecates KongIngress in the sense that it's still available for the time being, but results in logs instructing users to migrate rather than removing it immediately.
Which issue this PR fixes:
For https://github.com/Kong/kubernetes-ingress-controller/issues/4720 but not using auto-resolve since this isn't targeted to main.
Would need a follow-up "Remove KongIngress support" issue.
Notes:
I did leave in the checks for route and service even though CEL now blocks them. I figure it doesn't really hurt anything to have them there under normal circumstances, and we often had abnormal circumstances because users forget to update CRDs.
Added a Service webhook handler also. I don't really love having that in place for such a common resource, but it is at least only temporary: we'd remove it once we actually remove KongIngress.
PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

I've rebased the PR onto main and added CEL validations.

I had to rebase (conflict in go.sum), sorry @pmalek 😞


Python docs are bad for pulumi_azure.signalr.Service

See https://pulumi.io/reference/pkg/python/pulumi_azure/signalr/.

Something went bad with tfgen: https://github.com/pulumi/pulumi-azure/blob/master/sdk/python/pulumi_azure/signalr/service.py#L11-L19.
The Node docs are also not great: https://github.com/pulumi/pulumi-azure/blob/master/sdk/nodejs/signalr/service.ts#L30

This looks OK now, looks like something got fixed in TFGen.


Authentication credentials are missing or invalid.

Issue Description
We are seeing iOS build failures due to following issue:
Authentication credentials are missing or invalid.
  - Provide a properly configured and signed bearer token, and make sure that it has not expired.
    Learn more about Generating Tokens for API Requests https://developer.apple.com/go/?id=api-generating-tokens

I have researched it in this issue:

https://github.com/status-im/status-react/issues/13352

And it appears others are also experiencing it. It appears to have started today. Most probably the issue is on the Apple side.
Context
We are using most recent 2.205.2 version as are other people that reported this issue: @Dmytro-Mishchenko and @rborn.

As suggested in these issues and discussions:

https://github.com/fastlane/fastlane/discussions/18100
https://github.com/fastlane/fastlane/issues/19072

I - and others - have attempted to change the duration parameter for app_store_connect_api_key but both default 500 and maximum value of 1200 make no difference.

One red herring that I fond was an expired "Paid Application" agreement:

But accepting it has made no difference. It appears that something must have changed on the Apple Store Connect backend.

we are seeing that as well. Can it be an issues on Apple / AppStore Connect sides?

Yes, this would be my assumption. We have not changed anything in our Fastlane configuration, nor have we changed Fastlane version of renewed any certificates. The issues started happening today without any correlated changes.
My guess would also be that the problem is on the App Store Connect side.

The issue appeared around 4 hours ago for us. Reproduces every time.
In addition, AppStore Connect sometimes loads slow for me today, so I'd also assume that issue is on the Apple side.

The issue appeared around 4 hours ago for us too. Reproduces every time too.

We are also having this issue currently and we are getting error 500 errors on appstoreconnect.com. Their status page says App Store Connect & App Store Connect API is online/no problems... 😬

The issue started appear about 4 hours and it stays this way till now.

Our first build failed with this error at 09:23:51 UTC.

As noticed by @JencirJamal Apple finally updated their status page with info about Apple Store Connect API issues:

https://developer.apple.com/system-status/

We faced the same issue. But were managed to upload ipa via Transporter app as a tempo solution.

Looks like it's recovering, we had a successful build a moment ago.

Still not working for me, will update when it does, if 😅

same for me here

We're seeing the same issue.

It's working again for me. (No changes needed)

Same here when launching match lane. As a workaround we returned back to providing FASTLANE_SESSION env

Is this resolved for everyone? We're still having issues.

Had a few uploads go through just now successfully.

I Still have the issue here.

it seems, AppStoreConnect is back 💪 and to be honest, this issue is out of control from us (Unrelated to Fastlane scripts) so closing this issue...keep shipping! 🚀

Btw, just notice...AppStoreConnect is having issues again! Looks like, some WWDC 🆎 testing is going on... 😂
Let's wait!
https://developer.apple.com/system-status/

I created this issue mostly for visibility purposes,

You have a point...Re-opening this issue again because AppStoreConnect is again not behaving well and as you said, maybe helpful for some new users.
we can close it later when AppStoreConnect will become stable again.
Thank you.

Seem to be getting this again. Had a successful upload on 2nd June but now getting the same error. Updated fastlane to 2.206.2 and still occurring. Any one else have the same? Apple services seem to all green as well...

Same error with 2.206.2

@Vyazovoy - When I updated my xCode it seemed to fix it.

I get the error with fastlane 2.206.2 and 2.207.0

We received the error today as well (and spent some time chasing a secondary error which it turns out has been reported in Fastlane multiple times "Error finding relevant GitHub issues: undefined method `map' for nil:NilClass" - #13877, #12587, #5817, #7274...).

Same error with https://github.com/fastlane/fastlane/issues/20645
...
Waiting for App Store Connect to finish processing the new build (1.0.0 - 1) for IOS
Waiting for App Store Connect to finish processing the new build (1.0.0 - 1) for IOS
Waiting for App Store Connect to finish processing the new build (1.0.0 - 1) for IOS
...
Authentication credentials are missing or invalid. - Provide a properly configured and signed bearer token, and make sure that it has not expired. Learn more about Generating Tokens for API Requests [https://developer.apple.com/go/?id=api-generating-tokens](https://developer.apple.com/go/?id=api-generating-tokens%1B[0m)
...
fastlane finished with errors
...

fastlane version: 2.194.0

@crazymanish this error is back again.

@crazymanish this error is back again.

I've seen this happen on and off for a few weeks. Sometimes it happens and sometimes it doesn't even with the duration for the key set to 1200 .

OMG, Just got this error today :(

It still happens…… #21160

Happens on my side sometimes.
[15:21:25]: Authentication credentials are missing or invalid. - Provide a properly configured and signed bearer token, and make sure that it has not expired. Learn more about Generating Tokens for API Requests https://developer.apple.com/go/?id=api-generating-tokens

It fails on upload_to_testflight step.
But app_store_connect_api_key step is not generating any errors.

Also seeing this behavior @pavlobu.

setting FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD didn't solve the problem in my case


pre-commit

Added .pre-commit config yaml file, which is a copy of the Pithia Foundations pre-commit config
Ran pre-commit on existing files in the cookbook template

cc @brian-rose

Hey there @brian-rose, thanks for the feedback!
I'm fine with any of those. Waiting until prettier 3.0.0 seem fine, definitely don't want to create more tedious cleanup work!
As for exploring option #2, I'm not sure if it works, but can we pin the prettier version?
Something like is mentioned in this issue: https://github.com/pre-commit/pre-commit-hooks/issues/332
  - repo: https://github.com/pre-commit/mirrors-prettier
    rev: v2.7.1
    hooks:
      - id: prettier
        additional_dependencies<EMAIL_ADDRESS>

As for exploring option #2, I'm not sure if it works, but can we pin the prettier version?

I don't know if it will work either, but let's give it a try! We can try it out here in the template repo for a week or so and see what autoupdates get generated. If we're happy these changes, then we can migrate them to all the existing cookbook repos.

Thanks for your contribution @norlandrhagen !

Thanks @brian-rose! Not sure if it'll work, but fingers crossed.


numpy.clip accept numpy.array for a_min, a_max

Closes https://github.com/numba/numba/issues/7521
I have used np.broadcast_arrays to support arrays in a_min and a_max arguments of np.clip. Note that this doesn't deal with supporting scalars in a argument. However, I can do that if required.
cc: @guilhermeleobas @gmarkall @Twomem

Closes #7521
I have used np.broadcast_arrays to support arrays in a_min and a_max arguments of np.clip. Note that this doesn't deal with supporting scalars in a argument. However, I can do that if required.
cc: @guilhermeleobas @gmarkall @Twomem

Hi @czgdp1807 , just checking, is the plan still to also add similar support for numpy.minimum? Not sure if this is in this pull request or it would be better to do in a separate request. Thanks!!

is the plan still to also add similar support for numpy.minimum?

I can do that, just waiting for reviews on this PR.

@guilhermeleobas I have addressed your latest reviews to inline jitted functions which are called only once. Please let me know if there is anything else to be done here.

Hi. @sklam @stuartarchibald @esc Is it ready for merging? Please let me know if anything is left to be done here. Thanks.

Hi. I am starting my work on supporting arrays in np.minimum as this PR has received one approval and no objections. So, I think there is support for this work. Please let me know if the case is opposite. Thanks.

Hi. I am starting my work on supporting arrays in np.minimum as this PR has received one approval and no objections. So, I think there is support for this work. Please let me know if the case is opposite. Thanks.
Edit - Seems like np.minimum already supports arrays as inputs (may be because its a ufunc). @Twomem Please let me know if you have any function in mind which should support arrays as inputs but currently doesn't. Thanks.

Thanks, @czgdp1807 -- I think if all the ufuncs are supported that would be the main thing as from what I can see, the biggest performance gains can be had using vectorise / parallel / fast math and GPU acceleration all benefit most from the ufuncs. What I don't get is whether np.clip is a ufunc -- in my mind it should be but the docs don't seem to list it


Crash when running tf.random.fixed_unigram_candidate_sampler

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
No
Source
binary
Tensorflow Version
2.10.0
Custom Code
Yes
OS Platform and Distribution
Ubuntu 22.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
A bug happened!

Standalone code to reproduce the issue
https://drive.google.com/drive/folders/1CAWUAcGHbpmwRCSM9dwRAhEZwMkilyVM?usp=share_link

Relevant log output
No response

Not actionable, closing. Please report using the right channels.


ARROW-16168: [C++][CMake] Use target to add include paths

We can remove "include_directories(SYSTEM)" by this.

@github-actions crossbow submit -g nightly

@github-actions crossbow submit amazon* ubuntu-18.04 ubuntu-20.04

@github-actions crossbow submit ubuntu-18.04

@github-actions crossbow submit verify-rc-source-cpp-linux-ubuntu-18.04-amd64

@github-actions crossbow submit verify-rc-source-cpp-linux-ubuntu-*-amd64

@github-actions crossbow submit -g nightly-tests -g nightly-packaging -g nightly-release

@kou Should this be updated for git master? Is there anything left to do otherwise?

Should this be updated for git master?

Yes.

Is there anything left to do otherwise?

I need to confirm that the current failures are fixed by rebasing on the master. If there are unknown failures, I need to fix them in this pull request. (I prioritized 8.0.0 release related tasks over this.)

@github-actions crossbow submit -g nightly-tests -g nightly-packaging -g nightly-release

@github-actions crossbow submit -g nightly-tests -g nightly-packaging -g nightly-release

@github-actions crossbow submit almalinux-8-amd64

@github-actions crossbow submit almalinux-8-amd64 ubuntu-bionic-amd64 test-ubuntu-20.04-cpp-thread-sanitizer wheel-windows-cp310-amd64

@github-actions crossbow submit -g nightly-tests -g nightly-packaging -g nightly-release

This is ready to merge.
I'll merge this tomorrow if there is no objection.

@kou Is the AppVeyor failure unrelated?

@kou Is the AppVeyor failure unrelated?

Yes. It should be fixed by https://github.com/apache/arrow/pull/13101 .

@github-actions crossbow submit -g nightly-tests -g nightly-packaging -g nightly-release

+1


Add recent commits to legacy3.1.x branch

This adds three of the commits. The preferences window is a bit trickier, because it brings all the intermediate changes to MainMenu.xib as well. Is there are way to do this painlessly without having to essentially re-apply the change manually?
@josh64x2 What did you mean with help bundle?

@Eitot oh for the help book one I meant the vienna-rss.org -> vienna-rss.com conversion :)
thanks!

@barijaona is this granular enough or do you want the commits broken down into individual pull requests?

There is much less granularity issues on legacy branches : we just cherrypick commits which have already been reviewed and tested before being merged into master, so the review can be lighter.


Unable to use self assigned strings as _id

Hi, Mongo allows usage of any arbitrary strings for the _id field. However monk throws an error as follows when trying to insert a document with a custom string in the _id field.
Error: Argument passed in must be a single String of 12 bytes or a string of 24 hex characters

I feel the code in helper.js on line number 13 is a little weird.
return typeof str === 'string' ? ObjectId.createFromHexString(str) : str

Based on this code, it always tries to create an ObjectId object, and a custom string id always throws an error. All works well with the native mongo driver.

There is a castIds option the you can set to false to use arbitrary strings for _id


Improve body/body_path documentation

README.md:

body: Text describing the contents of the release. Optional, and not needed if using body_path.
body_path: A file with contents describing the release. Optional, and not needed if using body.

Of course body is not needed if using body_path and body_path is not needed if using body, since they are optional according to the documentation. What would be useful, however, is to know what happens when both or neither of them is defined.

How about something like:
`body`: Text describing the contents of the release. If `body_path` is given and the file can be read then its contents will take a precedence. If neither `body` or `body_path` is given, the release will be created with an empty body.
`body_path`: A file with contents describing the release.

Please also add how should we address the file for body_path, absolute or relative, and relative to where


Add usage docs or readme section

There should be something explaining how I can start your project. The first issue I had was I needed to install dependencies before running the server, but I didn't find the dependencies list anywhere. Fortunately, I had to install just one library, but there may be more of them in future, so it's not nice. The second problem is, installing the dependencies is not enough! Looks like I have to do something else to use the project because when I start it and visit the webpage from my browser, I get the not found error on the http://localhost:8080/lib/saves/names.json URL. In fact, I don't even want to have saves, I want to just view my particles live (probably it has something to do with #15)
By the way, why are saves stored inside of lib?! It might be worth opening a separate issue

So, installing dependencies is fixed now, will fix everything else now

So, installing dependencies is fixed now, will fix everything else now

Now, this is not fixed unless it's in master. And the master's README still contains no usage instructions

Now, this is not fixed unless it's in master. And the master's README still contains no usage instructions

Now this is fixed (by #21)

Now this is fixed (by #21)


Consider translating documentation to markdown

Consider changing the documentation format https://pyscesdocs.readthedocs.org to Markdown. We would then probably also need a Markdown --> ReST converter for Sphinx.

Not sure why you would want this? The documentation is all there and written. Sphinx needs ReST AFAICT. RTD runs Sphinx on the ReST source it automatically pulls in from GitHub.
Pandoc is quite good at interconverting all these formats, but for the docs I'd prefer to have them in ReST since that is what Sphinx uses, and we would have to have to check in any case if the conversion MD->ReST went without error. So the "authoritative version" should be ReST as that is what Sphinx uses.
If a new set section/chapter docs has to be written and you prefer writing MD, I'd rather suggest writing the first version in MD and then using pandoc to convert it to ReST.
I'd rather w

I've had good experiences with Pandoc and markdown to other things . The advantage is that all documentation could then be be in Markdown and we can do fancy things with Pandoc (like includes).


[WIP] Add a "Web seed"

Fixes #51 - this PR is for your visibility and I don't think is something that should be merged yet.

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to documentation are included in the PR

This PR get's the breadboard and llm-starter ready for running in a browser. The summary of changes are:

Remove direct nodejs imports (and make them dynamic)
Change secrets to pull from the user-context vs put on the process.env - i.e the graph runner will ask your user-space code for a bag of things to resolve the secrets.
Add a web-demo so that we can play with different parts of this project.

We are unable to resolve "npm:" url's for kits right now, so a graph file has to have "npm:https://esm.sh/@google-labs/llm-starter"

We don't yet have a node editor in this.

The UI currently has

A Palm-lite tester.
A "completion" test harness that uses the imperative Board API.
A breadboard viewer

We are unable to resolve "npm:" url's for kits right now, so a graph file has to have "npm:https://esm.sh/@google-labs/llm-starter"

Interesting! We don't load npm packages in Node. They have to be explicitly installed as dependencies, and the kits reference only attempts to import them. With the esm.sh, the meaning changes, since now we're actually installing them. Is there a way in vite to say: "hey, I will dynamically import this package later, please keep it around"?

They have to be explicitly installed as dependencies

At least for how I envisage using graph files (sharing etc), we have to have a way to load kits without knowing about them ahead of time. If I share a graph with you, you then have to parse the json file, find the kits, add them to your project as dependencies...urgh. This might be fine if it's a static project, but for sharing it doesn't really work. If Kits have to be hard dependencies, then it feels kinda weird that we would have a definition defined in a JSON file at all and then have it hardcoded in the code that loads and runs the graph.
Right now you dynamically load models through the KitLoader, I'd probably just do Board.load("path to graph", {kits: [LLMStarter]}). Or something like that.

With the esm.sh, the meaning changes, since now we're actually installing them. Is there a way in vite to say: "hey, I will dynamically import this package later, please keep it around"?

We're not installing them, it's all dynamically downloaded and imported at runtime when the we call Board.load. For the viewer that I was building there is no way to know that for any given graph file what kits are included. If the Kit isn't accessible I can't even render the graph, let alone run it.
To test these changes for example, I've uploaded my own kit. https://www.npmjs.com/package/@paulkinlan/llm-starter - I thought that was neat.

My hope is to turn boards to Kits. The basic idea is that Kits are just boards without wires. I'd like to eliminate the concept of boards as npm packages altogether.
See: https://github.com/google/labs-prototypes/issues/36#issuecomment-1664799621

My hope is to turn boards to Kits. The basic idea is that Kits are just boards without wires. I'd like to eliminate the concept of boards as npm packages altogether.
See: #36 (comment)

Also see #58 for a previous discussion of loading npm packages.
Pragmatically I think we want to

not ship this with a giant security hole, so at least need an allowlist or so as a temporary protection (which having to manually install a package is effectively in the node environment)
be able to use the web interface while developing a kit, so avoid having to publish a package for every iteration

Since we'll want to move from npm to graphs-as-kits for kits anyway before sharing graphs with new kits is an actual thing, is there something quick we can do for now that unblocks this? Let's assume everyone builds from the repo for example.

I will close the PR and open a new one that has all the commits squashed so it doesn't poluate all your histories.


Implement objpict

Hi @gmischler ,
I started writing objpict and ran into a couple of issues.

Piping doesn't seem to work with oconv. If you uncomment the call_two here and run you'd get an idea.
Secondly, my views don't seem to be capturing the object correctly.

I am still working on this one as I haven't implemented the dry run and verbose options yet. So, the file is currently on my fork. Once I have made the correction I will send a pull request.

Oconv needs to know that it's supposed to read from stdin. That's what the '-' argument does:
octreeCmd = ['oconv', self.testRoom, '-']
The Perl version ads a -t .5 .5 .5 to the transform (haven't checked the logic of that).
Btw: List comprehensions are often much more practical to use than the outdated map().

Thanks. That fixes it.
Thanks for the tip about list comprehensions. I do use for dict and list comprehensions but didn't really think about them from an efficiency perspective. I did some reading on that and it makes sense to employ comprehensions instead of map. Although I'd like to use them, I still haven't found excuses for using generators till now as all these wrapper type scripts don't seem to require them.
Since I didn't find them in your code, are you okay with me using lambdas? I tend to write a lot of code in PyQT for my assistantship work at my university and have gotten used to employing lambdas to pass additional variables to methods.eg. something like self.btnSelectColorLowerMask.clicked.connect(lambda:self.spMaskSettingsActivated(False)) where the ..clicked.connect() needs a callable object as input. I know that I could get the same thing done through functools.partial but it just seems like an overkill if lambdas can do the same for me. I have used them in a couple of places here to take shortcuts.

Python has many advanced functional features (list comprehensions, generators, generator expressions, lambdas, etc.), each of which has its uses. That doesn't mean you have to use all of them, just take the one that solves the problem most easily.
I also don't see a need for generators very often, they are most helpful for creating largish sets of data on the fly (I have a reimplementation of vwrays that uses them).
My most common use of lamdas is also for "currying" argument values in callback functions as you describe. Using one for just three explicit applications of os.path.join() seems a bit of an overkill though, and doesn't really help to simplify the code.

@gmischler Thanks for you insights. If you are okay with it, I am also going to implement Ltpict and Ltview then send a combined pull request in the coming weeks. I was tinkering with the objline c-shell script yesterday on my windows machine and wasn't able to get the part with targa files to work. Do you think it's possible to implement objline in a windows environment? Although the c-shell script has been around for years, I haven't seen a windows compatible script anywhere.

 Of course I'm happy to see the full set of preview scripts implemented!
Ibjline.csh doesn't seem to handle any targa files. It produces "meta" files as output and "mgf" files as intermediate format.
The manpage gives an example of how to convert the result to an image via a targa file. Are you talking about this? Does meta2tga or one of the other programs mentioned there not work on Windows?

Hi @gmischler, I thought of letting you know about my intention of eventually getting to Ltpict and Ltview as I didn't know if you were planning to add those scripts from your end.
Regarding, the targa files, I get a fatal -cannot open file "c:\tmp\vchars.mta", mode "r" error. I literally read between the lines of the obline csh script to check if the workflow works without any glitches. rad --> mga --> meta works fine but the conversion from meta to targa fails.

@sariths, I've seen your previous announcement about ltpict and ltview. I'm not working on those myself.
Your error message comes from the command pexpand, which again gets invoked by both plot4 (in the script) and meta2tga (not in the script).
The file "vchars.mta" is a meta file included with Radiance as a support file to be used by other meta files. It is installed in the "meta" subdirectory of the Radiance library. Well, at least if you have installed everything. I found the following explanation from Greg in the mailing list archives:
http://radiance-online.org/pipermail/radiance-general/2015-October/011285.html
If that doesn't help, we'll have to check further.

I guess the meta subdirectory does not get shipped with the NREL x64 bit version that I got from here. Thanks pointing me to the root of the problem.
I don't want to bother too many people with these hobby projects of mine as objpict and the other scripts work fine on my linux machine. I have tried building Radiance on Windows with cmake with mixed results in the past. I saw some discussion on Scons in the last few weeks but haven't really wrapped my head around the concept. Perhaps the meta directory might get built if I go the Scons route. I think I will close this issue for now and ping you back in a few weeks with the new scripts.

Those files don't get built. They get installed from the support file package rad5R0supp.tar.gz (or in rad5R0all.tar.gz as mentioned in Greg's reply).
If the NREL distribution does not include them, but includes the executables that require them, then that's obviously a bug that should get fixed. I'll ask Rob about it.
This may be a "hobby project" to you. To everybody else it simply means that you're contributing improvements to Radiance. It's perfectly acceptable to "bother" the other developers if you run into any difficulties doing that.
In case you didn't follow that discussion: Based on our mutual finding that we couldn't get glrad to work, I've asked Greg on the dev list about it. His response was to fix it, so that the full functionality of objview.py is now available on the platforms that support X11, including Linux. This fix should already be included in the latest NREL release 5.0.a.10 (other than Windows). Of course it's also in the HEAD source release, but not yet in the "official" source release.
Personally I think that the SCons system is easier to use and more reliable than CMake, but that may just be because I created it myself... It's disadvantage is that it doesn't currently build qtrvu. Also note that you'll need Visual Studio 2013 to build with SCons (the community edition is ok). Visual Studio 2015 currently still has a show-stopping bug in its libc, which Microsoft will hopefully fix with the upcoming "anniversary update" to Windows 10 this summer.


fix: change decimals of wbtc and usda

Refer to https://github.com/alexgo-io/alex-v1-app/issues/237.
closes #313

I've pushed the commits for the test cases of YTP, FWP and CRP for you to review the numbers.

Hi @SaadTahir28 , I reviewed the numbers over the weekend, and am coming to  conclusion that it is actually very hard to support zero decimals, primarily due to that we use 8-digit fixed notation for most of the calculation and, either the decimals get truncated (not something we want) or extra care needs to be given to properly handle this (not something I think that is worth pursuing).
I will ask you to help me on https://github.com/alexgo-io/alex-v1/pull/349 instead.

Yeah exactly. I was also having these issues and thought of that it is certainly not possible because of decimal places. I will start looking into this issue today and understand what is going on.


fix error of metro file checker, update error messages

fix error of metro file checker
update error messages
update test coverage

Codecov Report

Merging #39 (701fe84) into master (a3b115e) will decrease coverage by 0.29%.
The diff coverage is 65.21%.

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
- Coverage   76.94%   76.64%   -0.30%
==========================================
  Files          17       17
  Lines        1739     1760      +21
==========================================
+ Hits         1338     1349      +11
- Misses        236      246      +10
  Partials      165      165


How do I configure to scan files for my own custom funtions

This question is similar to https://github.com/php-gettext/Gettext/issues/128 but relates to the current version. In my code I use a global function call \Trans::late('text to translate goes here'). I want to scan all my templates and create a *.po file from it. I can not figure out, how I configure the tool to use my \Trans::late() function to get all texts to translate. Is that possible?

You have to install the PHP Scanner package https://github.com/php-gettext/PHP-Scanner/ that scan the code and search for the defined functions.
To configure the functions, use the setFunctions function.
The default values are here: https://github.com/php-gettext/PHP-Scanner/blob/master/src/PhpScanner.php#L16
//Create a new scanner, adding a translation for each domain we want to get:
$phpScanner = new PhpScanner(
    Translations::create('domain1'),
    Translations::create('domain2'),
    Translations::create('domain3')
);

//Set a default domain, so any translations with no domain specified, will be added to that domain
$phpScanner->setDefaultDomain('domain1');

// Configure the functions
$phpScanner->setFunctions([
  // Your functions here
]);


Mempool - dockerize and deploy Mempool app

The Mempool app source code is located here https://github.com/block-core/mempool.

This issue has not been completed and the docker-compose with relevant configuration has been added to the forked repo.

you mean it has been completed?


Should allow +c +o interfaces

We currently allow interfaces that are both +c and +j (see TokenTest). Need to do the same for ObjC.
Right now we generate classes for +c and protocols for +o. An interface that's +c +o would be a protocol with an old-style CppProxy implementation.

So should +c +o create @protocol or @interface? I didn't find a conslusive answer to this myself. I tend towards @interface since @protocol is just a bit weird in general, especially since class (static) methods don't have implementations, which was one of the reasons users previously had to include the CppProxy headers (very awkward).

I was thinking @protocol, and will probably just disallow static methods on +c +o.

That creates a strange asymmetry if they are allowed for +j

Yep, they shouldn't be allowed on +c +o or +c +j +o either (I think they currently are, but will probably generate broken code).

+c +j rather.

Well, I guess it is doable. The CppProxy has to implement the static method like usual and clients have to invoke it from Objective-C using
id<Thing> thing = [ThingFactory createThing]; // returns ThingCppProxy
[[thing class] doThing];

You cannot, however, make the factory method a member of the +c +o interface itself as you require a runtime object to get the class from in the first place. So it's fine except for this one (very frequent and necessary) use case.

Gave this some more thought and I think you're right. The point of a static method is it exists only once. However any combination of more than one +x is ambiguous about which language implements the static method, so it probably shouldn't be allowed at all.

I've pushed f0aec36d99764ca62903ba58fbb9c4af9c886c5 to a branch to let it bake for a bit before merging into master. The Carousel team is working on some refactoring to use this - if they don't run into any showstoppers then I'll merge it in and call it done.

You know this makes me think whether +c is even necessary at all. On the C++ side whether +c is present or not makes no difference, there's always just a plain C++ abstract base class clients can do whatever with.
It only makes a difference for the glue code and I was wondering: would it make big of a difference to always imply +c? Because the generated C++ code doesn't distinguish whether +j/+o is there or not why even bother putting the +c mark in there.
In the glue code if the dynamic_cast<Proxy*> fails one received some other T derived from the interface. Theoretically one could wrap that inside a new proxy and let the Java/Obj-C code interface with it instead.

I think it's useful to limit the size of the generated code (on both the Java and JNI side), since code size and Java method count can be concerns.

Whether +c exists or not does not impact the generated Java code. And in any real world app the size of the Djinni glue code should be insignificant compared to the rest of the application.
I'm still not certain if it is possible to drop the +c and make it work in all cases, that requires some more experimentation.

OK, correction. There is a difference in Java generation. I just wonder whether it's really necessary...

Done as of f0aec36d99764ca62903ba58fbb9c4af9c886c54


Separate the collection controller into it's own image

Fixes #58

Thanks for the review Tim. I updated the names and made it look less operator-ish. Let me know if there is something else that needs to be updated.


Raincatch 851interface

https://issues.jboss.org/browse/RAINCATCH-851
read, update and delete to the storeapiservice. no actual implementation just an interface

@austincunningham Introduced changes do not make sense without extending store interface.


Update ns3 backend to the most recent 3.36 version

Summary
Update NS3 backend to the most recent 3.36 version (from 3.17)
Key changes

We no longer need python 2.7 or gcc 5 to build AstraSim with NS3
The new ns3 repo is a fork of the main ns3 repo (instead of a simple copy/paste), allowing us to track future ns3 updates.
For now we create the ns3 repo in a personal github account, but will move to the astra-sim group in the future.
https://github.com/jinsun-yoo/ns-3-dev

For details on changes made to the ns3 backend itself, please refer to the repo.
Test Plan
Clone Astra-sim repo to a fresh directory & build
Additional Notes
Include any other notes or comments about the pull request here. This can include challenges faced, future considerations, or context that reviewers might find helpful.

Just move ns3 repo into astra-sim org
Also, make it public


Suggestion: allow path to optionally be an array.

It would be really nice to be able to do something like:
reply(content).push([
    somefile.js,
    someotherfile.css
]);

Rather than repeatedly doing .push calls. This would also allow easily setting the same headers for those resources (if required).
Thoughts? Criticisms? Reactions of pure unabated horror?

I can see the value in that!  Would certainly take a PR for it.


wallet: signing support for Miniscript descriptors

Based on #5.
This makes the Miniscript descriptors solvable. Note this introduces signing support for much more complex scripts than the wallet was previously able to solve, and the whole tooling isn't provided for a complete Miniscript integration in the wallet (especially the Miniscript <-> PSBT interaction).

I've replicated the Miniscripts used in the unit tests with the Rust implementation: https://github.com/darosior/rust-miniscript/tree/bitcoind_miniscript_pr.

Added some more do comments.


Add linux to line about gradlew

I thought that because of how opensource oriented terasolgy is that it would be only fair to have The opensource OS that is linux.

LGTM!


[bug]: llama_decode error when send the same prompt twice

Hi, I've found that when completion using useCache: true, and pass the same prompt, for example [{role: 'user', content: 'hello'}] to the same instance one by one, at the second time it'll get error llama_decode: failed to decode, ret = -1.

To be more clearly, If I ask hi, ask hello and ask hi again, everything is fine. But if I ask hi, wait the response, ask hi again, it'll output this error.
I'm sure it is related to the useCache attribute but I'm wondering what is happening?

Yes there should be an error in the useCache option. It will be fixed soon.


Ledger nano S import doesnt work

I'm submitting a…

[ ] Regression (a behavior that used to work and stopped working in a new release)
[X] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request

Expected Behavior
Extended Public key to be imported
Current Behavior
When clicking import extended public key error appears saying unable to reset the device
Steps to Reproduce (for bugs)

Open caravaan app and click wallet
select for extended public key 1 ledger device
Connect ledger nano S
Unlock ledger nano S and start bitcoin app
Click Import extended public key

Environment

Where are you running caravan:  unchained-capital.github.io
Operating system: Windows
Browser and version: Chrome 87

Chrome v87+ enables a "new usb backend" by default:
We are rolling out enabling chrome://flags#new-usb-backend by default in M-87
from this chromium dev link.
This doesn't work well with Ledger.  A workaround:
Disable the new-usb-backend and use the old version:

Paste this in the Chrome URL bar: chrome://flags#new-usb-backend
Change it to be Disabled if it shows up as Default enabled.
Chrome will refresh

Confirmed it works, it would be good if caravan had a warning or notice to warn users if the ledger import fails and the user agent is greater then chrome 87

Confirmed it works, it would be good if caravan had a warning or notice to warn users if the ledger import fails and the user agent is greater then chrome 87

See https://github.com/unchained-capital/caravan/issues/187


Unsupported type for timedelta seconds component

I'm getting the following error when updating to version 5.1.0. Version 5.0.0 works without problems:
TypeError: unsupported type for timedelta seconds component: datetime.timedelta

Settings:
SIMPLE_JWT = {
    'SIGNING_KEY': SECRET_KEY,
    'ACCESS_TOKEN_LIFETIME': timedelta(minutes=30),
    'REFRESH_TOKEN_LIFETIME': timedelta(days=15),
    'LEEWAY': timedelta(minutes=2),
    'AUTH_HEADER_TYPES': ('JWT',),
    'USER_ID_FIELD': 'id',
    'USER_ID_CLAIM': 'user_id',
    'ROTATE_REFRESH_TOKENS': True, # Rotate the tokens to allow infinite refresh lifetime.
    'BLACKLIST_AFTER_ROTATION': True, # Invalidate refresh token when it is used and rotated.
}

Not using a seconds unit anywhere...

Hi it would be great to see some traceback!

It seems that the leeway value currently only supports int type. @Andrew-Chen-Wang  @toniengelhardt

Sorry about the missing traceback and thanks for fixing @2ykwang 🙏🏽

thanks for catching this guys!!

Hey @2ykwang, when will the fix be released?

apologies, I should probably make a release with all these fixes. I've been swamped lately, so either this weekend but mostly likely next. Sorry and thanks for the patience!

No problem @Andrew-Chen-Wang, thanks a ton!


Support OpenBMC based systems

We currently assume IPMI, which is not the case (at least currently) for OpenBMC based systems.

We have very preliminary support as of https://github.com/open-power/op-test-framework/commit/738db7943eb0f090fa3db07c5200df536d0e3a4a but we need more complete support, notably power on/off/set boot device.

In the class OpTestOpenBMC(), shoudn't the ssh port number be 22 (rather than 2200) ?

@philippe56 OpenBMCs have 2 SSH ports: 22 is for logging into the BMC itself, and 2200 is for getting access to the host console

b19d231096ac27fa137eaf07b22c6deb61032e24 should mostly fix this now that OpenBMC has IPMI.
However we still likely have bugs, as the IPMI support in OpenBMC certainly does.


💡 Add ULID support

I have an idea!
Basically we use UUID for our applications, but new ULID identifier a quite bit faster and ensures that the IDs can be monotonically ordered and can be sorted even when generated within a millisecond.
Here is the ULID specs:

128-bit compatibility with UUID
1.21e+24 unique ULIDs per millisecond
Lexicographically sortable!
Canonically encoded as a 26 character string, as opposed to the 36 character UUID
Uses Crockford’s base32 for better efficiency and readability (5 bits per character)
Case insensitive
No special characters (URL safe)
Monotonic sort order (correctly detects and handles the same millisecond)

FYI: We have supporting for UUID v7 in https://github.com/cycle/entity-behavior-uuid

I use Ulid form almost all entities PKs wrapped in ValueInterface, it works great. I'm thinking about switching to a entity-behavior, but turns out I actually like the control I have over values assignment, so I'm used to setting PKs manually.

There's ought to be another plugin entity-behavior-uid that uses Symfony Uid package. 🙈 (Symfony Uid also supports Ulid among regular UUIDs)

I use Ulid for almost all entities PKs wrapped in ValueInterface, it works great. I'm thinking about switching to a entity-behavior, but turns out I actually like the control I have over values assignment, so I'm used to setting PKs manually.

The same. I prefer to generate UUID in a separated service.


Keyboard Shortcuts Master List

What should the list of keyboard controls be for 1.2?
Currently, we have

KEY_SPACE

Toggle Flip_texture property of highlighted vertex

KEY_SHIFT

Add control points to a vertex for bezier curves

KEY_ALT

Grab Edge (Moves two control points at once)

Mousewheel

Toggle Flip_texture property of highlighted vertex and pressing ALT (only when using mousewheel down... I think this is a bug)
Cycle through texture indices of a vertex

We currently don't have a way to adjust the width of a vertex using shortcuts.
I also removed editing the vertex property arrays manually. Should this be restored?

Could maybe use the control key with a mouse drag to adjust the widths.  Or maybe Ctrl and mouse wheel to adjust the widths.
I've never had a need to manually adjust the vertex arrays.  I may have simply added them at the time so I could see what was going on.
I'm perfectly fine with removing them.

Ok, I'll set things up so that the controls will be the following

KEY_SPACE

Toggle Flip_texture property of highlighted vertex

KEY_SHIFT

Add control points to a vertex for bezier curves
Grab Edge (Moves two control points at once)

Mousewheel

Cycle through texture indices of a vertex
KEY_SHIFT

Increase/Decrease texture width property


Ensure message headers are still mutable after interceptor is called

Some components in Spring assume (perhaps wrongly) that a channel
interceptor will not convert a mutable message into an immutable one, and
they continue to modify the headers dowstream. We can dosge the issue of
whether this is right or wrong by keeping the message headers mutable, just
in case.

Wrong base branch (should be 1.0.x)


How to find the maximum utilization of compute units

While running a application how to find the u percentage of compute power is getting utilized?
How many compute units are loaded ?

profilers or performance counters

@FalconBsp ROCm-SMI samples the power from the GPU not total board level power aka TGP


SFunction: recive all connected clients user names

Implements in function in the server as Request code for receive back a list of all connected clients
##Technical information:

function code: 201
return a list

##Request Response:
{"STATUS": 201 ,"CONNECTED_CLIENTS": [ ] }
##Request format:
{"REQUEST_CODE": 201}

waiting #6 be resolved


Adding README file

Dear Nagham I found this errors but I couldn't  comment it because you did not open a pull request for the others to be able to review
Error: README.md:1 MD022/blanks-around-headings/blanks-around-headers Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: "# 404s -Turning Errors Into Excellence"]
Error: README.md:2:81 MD013/line-length Line length [Expected: 80; Actual: 112]
Error: README.md:2 MD036/no-emphasis-as-heading/no-emphasis-as-header Emphasis used instead of a heading [Context: "🌟 Welcome to the 404s Reposit..."]
Error: README.md:4:81 MD013/line-length Line length [Expected: 80; Actual: 103]
Error: README.md:13 MD012/no-multiple-blanks Multiple consecutive blank lines [Expected: 1; Actual: 2]
Error: README.md:14:62 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
Error: README.md:22 MD012/no-multiple-blanks Multiple consecutive blank lines [Expected: 1; Actual: 2]

Great Nagham!
I have suggestion that if you add some hierarchy in the file using headings from (#) to (####) for example it would much better.

Thanks . I'll try what you suggested.

Hello Nagham, I tried adding a conclusion on the README file but I got the same errors Aziz talked about but it's like the changes have been implemented.
If not here is what I was suggesting could be added at the end:
💡 Innovation through Collaboration
Every detour presents a chance for discovery, and each 404 invites us to innovate.
By leveraging our diverse skills, we foster a culture where:

Ideas Flow Freely: Open discussions lead to unexpected solutions.
Feedback Loops: Continuous feedback refines our approaches.

Together, we transform challenges into opportunities for growth and excellence!


Task 7: Wireshark Task

Issue: #216
Captured live traffic with Wireshark

Hey @shrut2606
Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm.
Thanks for contributing in OpenCode'24 ✨✨!

Hey @shrut2606
Your PR has been merged 🥳🥳 and you have earned 20 points.
Thanks for contributing in OpenCode'24 ✨✨


Added my Pizza Corner Website

A menu card website for my restaurant Pizza Corner with the description of the pizza along with the price tag.

Please merge my pizza corner website. issue #222

@ilhamansari please mention the issue number that is #201 in this case and not the PR number that is #222.


Patients contacted

I rule and have completed some work on Case Manager that's ready for review!
Builds on the cooking with coolio work by @agkramer in #715
This pull request makes the following changes:

So far just adding the feedback from @colinxfleming

It relates to the following issue #s:

Fixes #712

Codecov Report

Merging #852 into master will decrease coverage by -1.21%.
The diff coverage is 12.5%.

@@            Coverage Diff             @@
##           master     #852      +/-   ##
==========================================
- Coverage   98.48%   97.27%   -1.21%
==========================================
  Files          35       35
  Lines         727      735       +8
==========================================
- Hits          716      715       -1
- Misses         11       20       +9

Impacted Files
Coverage Δ

app/models/patient.rb
91.83% <12.5%> (-4.57%)
:x:

app/controllers/users_controller.rb
94.59% <ø> (-5.41%)
:x:

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 1f5a98b...b0d6617. Read the comment docs.


Please fill the requirements.txt

I want to test this code, but requirements.txt is empty and I can't guess the dependencies and their versions by just looking at the imports.
Please run pip freeze > requirements.txt to update the file an then push it to either main or dev.
This will allow anyone to run the code on their own environments :)
Also, which Python version are you running this project on?
If you just run python3 core/example.py, then you can know the version by running python3 --version

Requirements are updated. Also, Python version is now showed on README. I'm closing this issue.


Dispatcher error reporting

Remove an error that should always be replaced by a result or different error later.

Codecov Report

Merging #459 (bedce69) into master (ba3d9ad) will decrease coverage by 0.11%.
The diff coverage is 89.28%.

@@            Coverage Diff             @@
##           master     #459      +/-   ##
==========================================
- Coverage   39.65%   39.54%   -0.12%
==========================================
  Files          34       34
  Lines        5238     5263      +25
==========================================
+ Hits         2077     2081       +4
- Misses       3161     3182      +21

Impacted Files
Coverage Δ

assemblyline_core/dispatching/dispatcher.py
76.66% <89.28%> (-0.01%)
:arrow_down:

...mblyline_core/scaler/controllers/kubernetes_ctl.py
11.42% <0.00%> (-0.28%)
:arrow_down:

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 ba3d9ad...bedce69. Read the comment docs.


initial implmentation

Moved from core's PR.
The default branch doesn't seem to be master right now.. can we change it?

It is public. Maybe @algrs set that.

Left a few more notes. Let's touch base quickly about the module names just before you merge and publish.

tested pip installwith git+ssh and git+git both works good.

Looks good! 👍

@paulmelnikow Thanks!


Cloudfoundry autoconfiguration for cloudsql is broken when java-cfenv is on the classpath

Hi,
when using spring-cloud-gcp in cloudfoundry in combination with java-cfenv, the integration is broken and no connection to bound cloudsql instances can be established.
The root cause is that GcpCloudFoundryEnvironmentPostProcessor is not setting spring.datasource.url, which means this property is set by cfenv.
Cfenv does not know about the special format the cloudsql socketfactory needs, so no connection can be established.

Is it possible to override spring.datasource.url to be an empty string with java-cfenv so that it's ignored?
How does it determine spring.datasource.url anyway?
@artembilan Anyone working on CF can advise us here?

cfenv-java is parsing VCAP_SERVICES, which contains the url to the instances public ip address.
java-cfenv is working on a feature to disable certain bindings, but that is yet to be merged.
Setting spring.datasource.url via the property source in GcpCloudFoundryEnvironmentPostProcessor would be the correct thing to do IMHO, since @ConfigurationProperties are populated from the environement's propertysources.

I also don't see any problems with overriding it with the GcpCloudFoundryEnvironmentPostProcessor. Not sure though why VCAP_SERVICES doesn't provide a proper JDBC url in GCP environment...
I see there is an AbstractJdbcUrlCreator abstraction. So, maybe in case of GCP we need to introduce our own implementation and expose it as a services?..
See this module: https://github.com/pivotal-cf/java-cfenv/tree/master/java-cfenv-jdbc

Our auto-configuration for Cloud SQL works on the assumption that if spring.datasource.url is provided by the user or whatever is upstream, in this case CF, that's what the application wants to use to connect.
I think the onus is on whoever is specifying the URL to get it right, or just not specify it.
I'm confused as to why the URL specified doesn't work. If it's a public IP, it should just work. You don't need anything else. In fact, you don't need this module at all in that case. I would set spring.cloud.gcp.sql.enabled=false.

@KamikaZeeFu Can you please comment on why the URL with the public IP doesn't work? Or, if you've resolved the problem, please close this issue.

Hello, sorry for replying so late. In order to connect through the public IP one has to configure the cloudsql instance to allow the source IP range. Since IP addresses are not known beforehand in a dynamic platform such as cloud foundry, the range if possible addresses would be quite large.
That's why so far, we've been using spring-cloud-gcp resp. the cloudsql socket factory

@KamikaZeeFu I opened a PR that should resolve your issue and applies a more general change to the behavior of Cloud SQL auto-config.


typo

README.md, plautopatch.tex: which are aware of -> which are unaware of
これは酷い…

2018-08-22 を出した後にまた気づいた… ordinally → ordinary


conf.yaml options vs. run.sh options

In the WHAM ConvTasNet scripts, you can set some options in conf.yaml and some options in run.sh. The run.sh ones seem to have precedence over the conf.yaml ones.
To me it's confusing since I do not see the reason for two places to specify these things. In practice, I never use the run.sh ones since I want to keep multiple model configurations anyways, so I'll end up having multiple conf.yaml files.
My suggestion is to remove the options from run.sh and add a new flag to run.sh, say --conf, that is a path to a conf.yaml file. This way it's obvious where the config is coming from and also you can easily switch between multiple configs.

Me and @mpariente discussed about this since the beginning of Asteroid.
Basically now it is super flexible, but in my opinion it is also confusing.
In order to run the full recipe you have however to pass also paths to where.json data is saved which can be modified by the user in run.sh.
My suggestion is to keep the paths in run.sh and every hyperpar in the yaml file. Paths will be common to multiple runs so one just specify them in run.sh and then moves to the yaml to modify hyperpars for successive runs.

Yes, I agree that this is still confusing and not optimal and we discussed it several times..
One of the most important feature to have IMO is to be able to change the hparams of an experiment from the command line so that you can use a for loop to launch experiments with different hparams, without needing to duplicate your conf.yml file.
I personally never duplicate conf files, but we can change the logic to make this simpler.
@jonashaag can you have a look at this PR (#104) and tell me what you think?
I think this is close to what I'd like:

There are no duplicates between bash and yaml
Every yml field can be modified from the command line
You could easily pass a config file to start from I guess

The only thing that holds me to do that is that it's "complicated" and the code can be confusing..

Looks good to me, but I wonder why roll our own? We could use something like https://github.com/bw2/ConfigArgParse instead (never used it but looks good).
For my use cases, I think best case would be something along the lines of:

Have a single "model command runner" Python file that knows how to preprocess, train, eval, infer for a certain model. So, more or less a mixture of train.py and eval.py with the addition of inference (e.g. directly from .wav files).
The runner file reads a model YAML file (like conf.yaml but without the strictly "localhost" things like data paths). The values read from the YAML file can be changed using CLI args.
The runner file also parses "localhost" things like data paths from CLI args. These do not make sense in the model YAML file.
Get rid of run.sh entirely, since then everything is covered by the Python script, and this way we also do not have to deal with arg parsing that is consistent in Bash and Python code.
(Optional: Move dataset preprocessing code to the dataset code itself, since it's duplicated for any model/dataset combination at the moment)

Also thought about removing the run.sh file completely and replace it by an omnipotent Python file.
And yes, in this case, argument parsing gets much simpler.
It's also related to #183, because unit testing python files would be easier. And we could refactor the evaluation to be part of the lightning module.
Would you be willing to put some time into this @jonashaag ?
There are lot of things to improve on asteroid's code, particularly things which are recipe-related IMO and your feedbacks are well appreciated thanks !
@popcornell What would you say about moving full python?

You would think about something like this?
python run.py dataprep  # run dataprep train and eval (default)
python run.py train --someargs  # run train and eval
python run.py eval --otherargs  # run eval only
python run.py predict  --files_glob # perform inference

IDK for simple things full python is nice. But what about if we have to check some dependencies in one recipe and pip install them. An advantage of going full python is debugging. Easier especially with an IDE.
Sure there are workarounds but they are not immediate as they are in bash. Also data download is not immediate. You have always to use subprocess.
And this comes from one who is not a bash super fan. I prefer to avoid it when I can but for some things it is the best tool.

@popcornell is right, parts of data preparation is a mess in python, how about?

Centralized bash script for each dataset, registered as a package script (accessible everywhere)
Python wrapper to that script, with subprocess, inside asteroid.data.dataset_name
Recipe in python just calling asteroid.data.librimix.dataprep in the dataprep mode

If we want to rewrite everything in Python for the dataprep later, we also can.

I will try to spend some time into understanding the similarities and differences of the models and datasets and come up with a concept


Cancel previous ongoing runs in the PR/branch when you push new commits

Summary:
It makes sense to only build the latest commit and cancel any previous ongoing builds when you push new commits to a PR or branch. This will ensure that we don't run out of free build mins.
Link for help https://stackoverflow.com/questions/66335225/how-to-cancel-previous-runs-in-the-pr-when-you-push-new-commitsupdate-the-curre
Would you like to work on the issue?
No

@madhurgupta10 can you please review if my approach is right?


Triple fault after triggering stack overflow

We're still seeing this issue.  Needs more investigation

With new bootloader, the stack overflow kills qemu

I think we need to allocate a stack for the TaskStateSegment.interrupt_stack_table
https://github.com/rust-osdev/x86_64/blob/master/src/structures/tss.rs#L29

Fixed in #74


Disabling the HTTPS

I've installed the zulip on ubuntu server 18.04 from Git and now I need to disable the HTTPS on it.
I checked here and online but didn't find a way to disable the https.
Already tried putting bellow lines in settings.py, all together or each one separately but no luck.
DISABLE_HTTPS = 'true'
HTTP_ONLY = 'true'
EXTERNAL_URI_SCHEME = 'http://vm-zulip'
EXTERNAL_HOST = 'vm-zulip:80'

Zulip version: 1.9.0-rc2+git

try
DISABLE_HTTPS: "True"

I've used that in "settings.py" but no changes.

@ITMaya you need to run scripts/zulip-puppet-apply after changing zulip.conf for that sort of settings change to take effect.  Are you using the Docker image, or the normal Zulip installer?  The DISABLE_HTTPS type settings names are only a feature for the Docker environment.  The zulip.conf thing is what you need for a normal Zulip server.

how do we let the installer go through without ssl certificates? I'm running zulip on AWS behind a loadbalancer using ssl termination, but the installer won't finish with
"No SSL certificate found. One or both required files is missing:"
/etc/ssl/private/zulip.key
/etc/ssl/certs/zulip.combined-chain.crt
I'm guessing, the [applicaion_server] http_only = true option only goes for after the installation? would the solution then be use --self-signed-cert option and add that to the zulip config once it's generated?


always call connection logQuery function to broadcast QueryExecuted event

fixes https://github.com/jenssegers/laravel-mongodb/issues/2068
Queries will only be added to the in memory connection query log if
the query log is enabled using DB::connection('mongodb')->enableQueryLog();

Codecov Report

Merging #2069 into master will decrease coverage by 0.00%.
The diff coverage is 81.81%.

@@             Coverage Diff              @@
##             master    #2069      +/-   ##
============================================
- Coverage     86.90%   86.89%   -0.01%
+ Complexity      672      671       -1
============================================
  Files            31       31
  Lines          1558     1557       -1
============================================
- Hits           1354     1353       -1
  Misses          204      204

Impacted Files
Coverage Δ
Complexity Δ

src/Jenssegers/Mongodb/Collection.php
88.88% <81.81%> (-0.59%)
5.00 <0.00> (-1.00)

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 0bf3f03...baa7332. Read the comment docs.

Thanks!


resolve and reject return a promise (with tests)

I love how you came up with the same pun for the same feature as me 😉 and implemented  it basically the same way I did.
I've updated this to return the original promise when resolving or rejecting, helps with some types of chaining and improves compatibility with async/await. New tests written and passing (along with the old ones of course).
Note: I did not touch package.json myself. Npm edited it during install. I guess they have changed their formatting a bit.

I'm curious why you think an empty promise "[in] general, should be avoided.". I see this as a vast improvement over native API if used correctly, because you can pass around promises as simple tokens instead of state-machines.

Haha! Great minds think alike I guess 🤔  :)

I'm curious why you think an empty promise "[in] general, should be avoided.". I see this as a vast improvement over native API if used correctly, because you can pass around promises as simple tokens instead of state-machines.

I just didn't find myself in many situations where it was useful except for when mixing promises with other async paradigms (e.g. events, streams). The native promise API is generally sufficient to wrap "nodeback" style code, e.g.:
const promisifiedFunction = (...args) => Promise((resolve, reject) => {
  someNodebackFunction(...args, (err, result) => {
    if (err) return reject(err)
    resolve(result)
  })
})

Published as<EMAIL_ADDRESS>

Thanks for the update.
I keep putting variations of this function at the top of my many of my js files or in some imported util file. I finally went to turn it into an npm plugin today. I was irked that someone took my name, until I looked it up and found almost exactly the same code I had written, but with a mini test suite and a build system in place. Hahahaha
I think you should give empty promises a second chance. The improved chaining ability from this update might also open up some new use cases for you.
I've got quite a bit more to say on the topic of 'usefulness' of this module, but I'm gonna plan to open it as an Issue proposing a change to the readme, so the conversation can be more public.
Personally I think if this were popular, it could potentially change the landscape of writing code with promises. I don't have a popular blog to use for evangilizing it, but I can at least make a public case to you that it deserves a highly promotional "you should use this everywhere!" readme with good tips for how to use it, instead of a mostly derogatory "you shouldn't use this very often, but it can help with edge cases" readme.


Update installation.rst

I only found out later that feincms3 was a separate project. All though I fully understand that you have started this as a separate project, still it merits a mention (a bit high before you actually start installing feincms).

Thanks!


s3_file - add support for virtualHost

Looking to use S3 behind a virtualHost:
i.e. having https://my_bucket.corp.com.eu.s3-eu-west-1.amazonaws.com
I want to be able to access it using my_bucket.corp.com.eu
https://docs.aws.amazon.com/AmazonS3/latest/dev/VirtualHosting.html
This is done using virtualHost:true param when creating presigned_url in s3_url method:
https://www.rubydoc.info/github/aws/aws-sdk-ruby/Aws/S3/Object#presigned_url-instance_method

Closing this now that the code is merged


Manager30 merge

depends on:
https://github.com/moio/sumaform/pull/25
https://github.com/moio/sumaform/pull/36


Throwing errors in console about depreciation

Look I'm not sure what this is exactly, but it's loading on my page(It appears to be part of base.js from youtube?), and it's throwing this error:
SVG's SMIL animations (< animate >, < set >, etc.) are deprecated and will be removed. Please use CSS animations or Web animations instead.

I just took a look and I can see the message you're talking about, but it appears to be in YouTube-specific code, not anything from Caja, so there's nothing for us to do here. Sorry!


REMS-173: Preload keycloak users

Add Alice, Jane Doe, Jon Snow, as preloaded users. The logins are:
Alice: nurse
Jane: practitioner
Jon: patient
All their passwords are just their first names, all lowercase.
Note that they all just have the default role. It was not clear to me if/which other roles should be assigned.
Corresponding PR which includes the docker compose update: https://github.com/mcode/REMS/pull/23

can yup update the Keycloak docker image file as well according to : https://mitrecorp.slack.com/archives/C02D3LDM3PU/p1654618708644289?thread_ts=1654614659.581139&cid=C02D3LDM3PU

Done


Rule implementation

In this PR, I'll finish functions of serialization.

This change is

@rum-yasuhiro How about now?


Allow extra props to be passed transparently through the CellMeasurer

Firstly the CellMeasurer is a great tool!!, however I have a use-case where I need the props past into the CellMeasurer to be forwarded onto its Child. This is because the props are added via a higher order component, in the parent component's render function (out of my control). Having the ability to transparently pass the props though would solve this problem.
Thanks
✅  Tests Pass Locally
✅  Lint Pass Locally

Hi @bpneal. I appreciate the pull request. I don't want to introduce additional destructuring and clone steps into an already performance-sensitive area though- particularly for something that seems a pretty unusual use-case.

This is because the props are added via a higher order component, in the parent component's render function (out of my control).

I don't really understand how something out of your control is passing params to your cellRenderer function. But I think you could solve this in your application by adding an intermediary/wrapper component that just reads the added props and passes them through to your cellRenderer child.


Added 'Content-Type' header

Closes #3.
Once this gets merged, the following image should show up correctly:

All of the wows. :shipit:

Boom! :boom: Working.


Test Connect Sql Server dotnet6 'SNIAddProvider' exception

Describe the bug
local SQL Server DB, using grate.exe (64bit self contained) throws
To Reproduce
1 Download, unblock, unzip: https://github.com/erikbra/grate/releases/download/1.4.0/grate-linux-x64-self-contained-1.4.0.zip
2 run local example with winget commented out, and using local grate.exe
Expected behavior
SNIAddProvider exception
Screenshots
console out:
Unhandled exception: System.TypeLoadException
   at Microsoft.Data.SqlClient.SNINativeMethodWrapper.SNIAddProvider(SNIHandle pConn, ProviderEnum ProvNum, AuthProviderInfo& pInfo)
   at Microsoft.Data.SqlClient.TdsParserStateObjectNative.EnableSsl(UInt32& info, Boolean tlsFirst)
   at Microsoft.Data.SqlClient.TdsParser.EnableSsl(UInt32 info, SqlConnectionEncryptOption encrypt, Boolean integratedSecurity)
   at Microsoft.Data.SqlClient.TdsParser.ConsumePreLoginHandshake(SqlConnectionEncryptOption encrypt, Boolean trustServerCert, Boolean integratedSecurity, Boolean& marsCapable, Boolean& fedAuthRequired, Boolean tlsFirst)
   at Microsoft.Data.SqlClient.TdsParser.Connect(ServerInfo serverInfo, SqlInternalConnectionTds connHandler, Boolean ignoreSniOpenTimeout, Int64 timerExpire, SqlConnectionString connectionOptions, Boolean withFailover)
   at Microsoft.Data.SqlClient.SqlInternalConnectionTds.AttemptOneLogin(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, Boolean ignoreSniOpenTimeout, TimeoutTimer timeout, Boolean withFailover)
   at Microsoft.Data.SqlClient.SqlInternalConnectionTds.LoginNoFailover(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString connectionOptions, SqlCredential credential, TimeoutTimer timeout)
   at Microsoft.Data.SqlClient.SqlInternalConnectionTds.OpenLoginEnlist(TimeoutTimer timeout, SqlConnectionString connectionOptions, SqlCredential credential, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance)
   at Microsoft.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, SqlCredential credential, Object providerInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling, String accessToken, DbConnectionPool pool)
   at Microsoft.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
   at Microsoft.Data.ProviderBase.DbConnectionFactory.CreatePooledConnection(DbConnectionPool pool, DbConnection owningObject, DbConnectionOptions options, DbConnectionPoolKey poolKey, DbConnectionOptions userOptions)
   at Microsoft.Data.ProviderBase.DbConnectionPool.CreateObject(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
   at Microsoft.Data.ProviderBase.DbConnectionPool.UserCreateRequest(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
   at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, UInt32 waitForMultipleObjectsTimeout, Boolean allowCreate, Boolean onlyOneCheckConnection, DbConnectionOptions userOptions, DbConnectionInternal& connection)
   at Microsoft.Data.ProviderBase.DbConnectionPool.WaitForPendingOpen()
--- End of stack trace from previous location ---
   at grate.Migration.AnsiSqlDatabase.Open(DbConnection conn)
   at grate.Migration.AnsiSqlDatabase.OpenNewActiveConnection()
   at grate.Migration.GrateMigrator.CreateGrateStructure(IDbMigrator dbMigrator)
   at grate.Migration.GrateMigrator.Migrate()
   at grate.Commands.MigrateCommand.<>c__DisplayClass0_0.<<-ctor>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.NamingConventionBinder.CommandHandler.GetExitCodeAsync(Object returnValue, InvocationContext context)
   at System.CommandLine.NamingConventionBinder.ModelBindingCommandHandler.InvokeAsync(InvocationContext context)
   at System.CommandLine.Invocation.InvocationPipeline.<>c__DisplayClass4_0.<<BuildInvocationChain>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass17_0.<<UseParseErrorReporting>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass12_0.<<UseHelp>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass19_0.<<UseTypoCorrections>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<UseSuggestDirective>b__18_0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass16_0.<<UseParseDirective>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<RegisterWithDotnetSuggest>b__5_0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass8_0.<<UseExceptionHandler>b__0>d.MoveNext()

Desktop (please complete the following information):

OS: Windows10
win64 version of grate.exe

Additional context
1 dotnet tool install grate, installs 1.4.0, if reference that version of grate, then no errors.
2 Not sure if this is related and it is my local system causing problems, but I tried building the source code locally but with a global.json specifiying to use my SDK of 6.0.400, the buildtime exception for
dotnet publish -c Release -r win-x64 -p:PublishReadyToRun=true --self-contained
is:
grate\Commands\MigrateCommand.cs(41,23): error CS0103: The name 'CommandHandler' does not exist in the current context

Are you really downloading the linux package and running on Windows 10?
I just ran both the local example as committed and it worked fine.
I then winget uninstall erikbra.grate, downloaded the winx64 self-contained package, extracted and unblocked the exe to the example folder, updated the ps1 to remove the winget install, and changed grate to .\grate to appease the PS gods, and that worked fine too.
If I've missed a step or something feel free to zip up a repro folder and I can try and run it on other machines?

@wokket I did actually download the 'grate-win-x64-self-contained-1.4.0' zip. I copied the link in the issue incorrectly.
I think you also need to change the grate to .\grate.exe to really appease the PS gods.  Maybe uninstall winget version and dotnet tools version so you can be sure it's not 'falling back' to them.

I had uninstalled the other versions mate.  I have no doubt there's an issue of some sort there, but if I can't repro it there's not much I can do :(

perfectly understandable. maybe related to my inability to build the source code locally too.

Hi, @OzBob , it's not good to know if this is a problem with your computer or grate. As you say, we do trim the assemblies, and there might be corner cases where the trimming trims a bit too much.
If you want to build it properly "self-contained" yourself, you should also set the -p:SelfContained=true too, as in:
https://github.com/erikbra/grate/blob/main/.github/workflows/build.yml#L93
there are a couple of extra properties that need to be set, in addition to the --self-contained command-line flag, and I've handled this with conditional properties in the .csproj, using the SelfContained msbuild property:
https://github.com/erikbra/grate/blob/main/grate/grate.csproj#L47
On what form is your connection string when you try to run it? Are you using any special features? Could it be that you are using some form of authentication, etc, that could make it fail with the SNIAddProvider exception?

There are some SNI related troubleshooting tips here, I don't know if they are applicable, but at least is looks like SNI (Server Name Indication, I only just learnt that) is dependent on some native DLLs:
https://learn.microsoft.com/en-us/sql/connect/ado-net/sqlclient-troubleshooting-guide?view=sql-server-ver16#issues-in-net-core-applications

I am not using grate but received the same error when using "Trim unused code" in my build. I think @erikbra is on to something. The error disappeared when I quit using "Trim unused code", which is apparently too aggressive, at least in this case.

build without -p:PublishReadyToRun=true --self-contained and the grate.exe does not have the TypeloadException. SNINativeMethodWrapper.SNIAddProvider error

I found this project today and was looking to compare it to DBUP. I installed it with winget and I recive the same error as OzBob. I thought I had the connectionstring wrong.
Will uninstall it with winget anddownload the selfcontained package so that I can test this project

I am having the same issue when using v1.4.0 (even with the selfcontained package). I have reverted to using v1.2.0 for now until this is resolved.

Fixed by #326


style: fix style issues

eslint has complained about this for some weeks. This commit installs the missing package.

Does npx eslint --fix --ext .js,.ts,.jsx,.tsx ./ work now?

Rebase and squash into one commit and we are good to go!

Thanks!


Add func in CacheGuardian, refactor 2FiberData -> cache

What changes were proposed in this pull request?

create a releaseFiberCache in CacheGuardian
2FiberData / fiberToData -> cache

How was this patch tested?
Existing tests.

@yueawang

1 is a TODO
it's more clear this way, take fiber2Data as an example, it applies to a Fiber, so fiber is redundant, data is ambiguous. More important, what we actually do is putting data/index to OAP cache and get the memory reference(FiberCahe), so it's pretty nature to name it in the new way, it now reads like a sentence. : )

@carsonwang I got an offline LGTM from @linhong-intel ...


catch teamEK fetching errors to avoid hosing chat

r? @mmaxim

Gonna ram this one through without waiting for CI. Gods forgive us.


Error add firebase plugin

What's going on?
I have got below error, when add firebase plugin
DEPRECATION WARNING: addPluginComponentExample called with 'FirebaseExample.js' and no .ejs extension. Add .ejs to your template filename when calling this function.
✖ Cannot read property 'includes' of undefined
➜  MyApp git:(master) ✗ react-native start

Steps to reproduce
ignite add firebase
ignite doctor results:
System
platform           linux
arch               x64
cpu                4 cores                           Intel(R) Core(TM) i5-6500 CPU @ 3.20GHz
directory          /home/jean/Projetos/MyApp
JavaScript
node               9.6.1        /usr/local/bin/node
npm                5.6.0        /usr/local/bin/npm
yarn               1.5.1        /usr/local/bin/yarn
React Native
react-native-cli   2.0.1
app rn version     0.50.4
Ignite
ignite             2.0.0        /usr/local/bin/ignite
Android
java               1.8.0_161    /usr/bin/java
android home       -            /home/jean/Android/Sdk

You should open this issue in the https://github.com/stoneman1/ignite-firebase repository.
Note that the readme says:

THIS PLUGIN IS STILL IN VERY EARLY STAGE. USE AT OWN RISK.


feat(client): add engula_client

This closes #765.

You can run for debugging with:
cargo run start --db <db-path> --init # start server

cargo run --example kv

Thanks for your review! Merging...
I'll verify #783 now.


Improvements from NIP-183 investigation

Added some useful instrumentation & ops improvements while investigating NIP-183.
And one important fix.

test this please


Need a way to inject the nextTick implementation

I am using Q in mozilla addon sdk which parses for static requires in order to authorize loading a module dependency from a module.
Problem is serverSideRequire("event-queue") which is not detected (as indended).
The best solution that does not interfere with the current requirements seems to be injecting the nextTick implementation.
Could we add something like Q.nextTick which I can set externally and use that over fallbacks?

This makes sense. Just to make sure, neither the MessageChannel nor setTimeout implementations are sufficient for your purposes? I would imagine the former is both available and sufficiently fast.
In any case, there is actually already a Q.nextTick, but we'd need to change all the internal usages to reference exports.nextTick instead of just nextTick if we wanted to make setting it useful.

setTimeout() is not available as a global in the mozilla addon environment.
serverSideRequire("event-queue") will return a module I am providing BUT the addon sdk loader will not allow me to load the module without finding it declared in a static analysis pass (looking for require('<id>')).
If I can set exports.nextTick externally I can bypass all the current detection logic.

A settable nextTick makes sense to me. I'm just wishing there was a way to make it work out of the box. Let me know if you think of anything.
I'll wait for @kriskowal to weigh in before committing anything though, since I don't understand all the security guarantees Q provides enough to know whether a settable nextTick would break them.

Unfortunately there is no clean way to detect the addon sdk environment. Since setTimeout() nor checking for serverSideRequire("timers").setTimeout (due to not finding require('timers')) will work I think the only solution is setting nextTick externally.
The function could be cached on first use to avoid security implications if that helps.

I would prefer for the nextTick underlying implementation to be injected somehow, rather than patched on post-instantiation. There are no securability issues if it’s injected, but there are security implication if we allow it to be replaced.
Bear in mind that this implementation of the Q interface does not make security guarantees. It just strives to behave identically to a hypothetical secure implementation like ref_send or makeQ.  It is intended to pave the way, not be the way.  That is, while it does make a lot of guarantees about the order of events (which are good for robustness in general), it does not guarantee that mutually suspicious programs could share the Q interface or share promise objects without fear of interference.  To do so, it would have to freeze a lot more.
This is a tricky situation. @cadorn, does the MessageChannel implementation of nextTick not work in the Addon SDK?  I am thinking that the event-queue should be entirely subsumed by Q at this point.

@kriskowal typeof MessageChannel: undefined
injection is indeed the way to go and that will be available in future as we have more intelligent loaders but Q is such a low-level module that it may be used without an injecting loader quite frequently (or even to implement the loader as in my case). I don't think the addon-sdk is alone when it comes to striving to provide a clean (minimal globals) module environment.
You can say; just patch Q when you import it into your project; but that is not consistent with the current attempts to subsume the event-queue and provide an out-of-the-box experience.
The addon-sdk exposes require("timers").setTimeout() but if used as serverSideRequire("timers").setTimeout it will not allow access to the timers module as the static analysis fails. (I currently implement an event-queue module and am adding require('event-queue'); to the top of q.js).
To prevent replacing the nextTick implementation it should be cached on first use. I think this is a fair workaround for the lack of injectability.
There are also cases when you want more control over nextTick than Q currently offers. In the addon-sdk the timers implementation respects the lifecycle of the program and test runner. See: https://github.com/mozilla/addon-sdk/blob/master/packages/api-utils/lib/timer.js
I suppose if we decide against a one-time settable nextTick I can look into teaching the addon-sdk loader the ability to allow loading of the event-queue module (since it cannot find it by static analysis) by configuration from package.json. This is a use-case not unique to this scenario but was hoping to hold off on for now.

@cadorn Does the addon runtime provide a reuqire.async(id, cb, eb) hook?


Fix codeblock format issue and refresh language.

Will close #2595

Hi @andreas-kupries, I want to check. It says all checks have failed. Should I still merge? It's only the README so not related I assume.

:eyes: ... Looking at the check it fails to log into docker.
Does not have user/password.
Fully unrelated to documentation.
It actually should not have triggered, IMHO, the README is in the list of ignores.
Hm. ... The ignores might be limited to pushes on main, not for PRs :(
Merging it now.


existing_key_id is mandatory when use_encryption is set to false

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version and Provider Version
Terraform v1.0.4
on darwin_amd64

Provider version: 3.0.0

Terraform Configuration Files

module "oke" {
  source  = "oracle-terraform-modules/oke/oci"
  version = "3.3.0"

  compartment_id = var.compartment_id
  tenancy_id     = var.tenancy_ocid
  user_id        = var.user_ocid
  api_private_key_path = var.private_key_path
  api_fingerprint = var.fingerprint

  bastion_enabled = false
  use_encryption = false
  ssh_public_key_path = ""

  label_prefix = var.label_prefix
  region       = var.location

  vcn_dns_label = module.vcn.vcn_all_attributes.dns_label
  vcn_name      = "oke-vcn-liftpoc"

  operator_timezone = "Europe/Amsterdam"

  metricserver_enabled   = true
  # vpa                    = true
  dashboard_enabled      = true
  create_service_account = true
}

Expected Behavior
existing_key_id should be optional.
Actual Behavior
Getting an error that I should supply existing_key_id even when use_encryption is set to false
Steps to Reproduce

terraform apply

Hello,
Thanks for logging this issue.
We'll look into this.

Hi,
Please set existing_key_id = "" in your terraform.tfvars file. We've fixed this in next major release.


on realize route logic - need transaction net value of address for book value accuracy on reporting completey sold xtz position

realize all staked xtz and remaining rewards need to add net transactions of address to get accurate most accurate book value

get transaction net value object synchronized with analysis book value object

resolved on analysis route with getTransactions function


InstallShield Limited Edition Project

InstallShield Limited Edition Project is not available for vs 2017. Please include the steps for 'How to use the existing setup project for office solution deployment'

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

ID: 283a3669-9f0e-c872-028d-9eac66fc221b
Version Independent ID: bdfea87a-3b7d-80ef-51f2-ceadb1ba26bb
Content: Deploy an Office solution by using Windows Installer - Visual Studio
Content Source: docs/vsto/deploying-an-office-solution-by-using-windows-installer.md
Product: visual-studio-dev15
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@manimaran66 Thanks for your feedback! We're aware that this page needs updating. It's on our backlog to fix.

@TerryGLee - Thanks for your quick response. :)

@manimaran66 You might check out this section and see if it helps you. https://docs.microsoft.com/en-us/visualstudio/deployment/deploying-applications-services-and-components?view=vs-2017#create-an-installer-package-windows-client

Please see the updated document: https://docs.microsoft.com/en-us/visualstudio/vsto/deploying-a-vsto-solution-by-using-windows-installer?view=vs-2019


⚠ Bofrost.si has degraded performance

In 7dc3d23, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 2483 ms

Resolved: Bofrost.si performance has improved in 4c28d6f after 3 minutes.


Helm installation broken

Helm installation is not working due to the recently added overlays removing rules from the open-feature-operator-manager-role. The operator is now not able to read openfeature.dev/* custom resources when installed with Helm:
failed to list *v │
│ 1beta1.InProcessConfiguration: inprocessconfigurations.core.openfeature.dev is forbidden: User "system:serviceaccount:open-featur │
│ e-operator-system:open-feature-operator-controller-manager" cannot list resource "inprocessconfigurations" in API group "core.ope │
│ nfeature.dev" at the cluster scope

@thisthat @bacherfl When I checked the output charts in https://github.com/open-feature/open-feature-operator/pull/647, the role-binding looked fine, but I never checked the role which is now more messed up :sweat_smile:
I will see how far I can get with this, but I wonder if we should change the patch to not be a delete and instead add the additional rules we need for ingress creation?

I started down the path of maintaining 2 roles and 2 rolebindings, one that controls the extra stuff we need when managerConfig.flagdResourceEnabled is enabled, but after running through some proposed tutorials and dealing with this additional complexity, I honestly think the entire idea of managerConfig.flagdResourceEnabled is not worth it. My reasoning is that it's easy to restrict the creation of flagd CR instances entirely (or instances with ingresses) through many different tools and policies, since these are not created automatically by annotations. I just don't think we need to control them at the OFO install level.
Sorry @bacherfl for the wasted time here, since this was initially my idea. :pensive:


Show the error message symbols

Show the error message symbols.
This makes it easier to add pylint disable pragmas that are more readable.
E.g.
# pylint: disable=broad-except

rather than
# pylint: disable=W0703

Hey. @jzoldak. Thanks for the quick PR. I have a few concern that the code seems to contain some regular expression pattern. Do we need to change it accordingly?

Hi @XericZephyr
Thanks for looking at this PR.
I'm not sure what you meant by regular expression pattern. Maybe the parentheses are confusing?
I was using them to visually separate the symbol from the message.
E.g.
BEFORE: x:y:W0703: Catching too general exception
AFTER: x:y:W0703: Catching too general exception (broad-except)

No problem, @jzoldak . I just want to make sure the change will not break other parts of the code. This PR looks good to me.
Hope all your guys have a good thanksgiving.

Hi, @jzoldak, any update on this PR? How long should we wait before this PR could be merged?

LGTM


Encoding Problem - Windows Console

If I run wiremock in windows in maven there is an encoding problem.
Wiremock is used for our tests. We set our maven encoding to "UTF-8". (set MAVEN_OPTS= -Dfile.encoding="UTF-8")
I think wiremock will start without that encoding parameter, so it trys to convert the current UTF-8 encoded file again, because it use the standard windows encoding.
Is it possible setting the encoding parameter of files while wiremock configuration?
When I run the test in the IDE there is no problem (Because the Encoding matchs). Just if I run them in the console there are errors.
Here the jUnit Error:
org.junit.ComparisonFailure: expected:<...------
{
  "text": "[üöäåÅש؁ж໗🛀⟰п]"
}
----------------...> but was:<...------
{
  "text": "[Ã¼Ã¶Ã¤Ã¥Ã…×©Ø�Ð¶à»—ðŸ›€âŸ°Ð¿]"
}

WireMock is hard-coded to UTF-8 I'm afraid. Might be better to convert your files to UTF-8.

Our files are UTF-8 encoded. I guess wiremock will be started with the iso-code converts the files twice.

The files are UTF-8 encoded. Maybe the Tests in Maven will run Wiremock with the windows encoding, so wiremock "think" the file is not utf-8 encoded, so the file will be double utf-8 encoded or something like that. If the files wouldn't be encoded the direct tests via JAVA command would end in error too.

WireMock forces to UTF-8 when reading files so I wouldn't have thought it would matter what the file.encoding property was set to.
Not totally sure I understand your last statement, did setting the property explicitly fix your problem?

Ok - I try to explain it again:
I use wiremock for the unit-tests. If I run the single test via java and encoding parameter it works. If I run mvn verify in my windows console there are encoding errors. At the Mac / Unix console works all fine.
Now I fixed the windows console problem with the MVN_OPTS Parameter, so the expected result in unit test is fine. But the return value of wiremock is now incorrect.
The files are all UTF-8 Encoded.
I think, when wiremock start at the maven verify command it try to convert all files to UTF-8 again. Is there any parameter to configure this or do you have another idea why this could happen?

Hi,
I ended up in the same situation: tests running perfectly fine on IDE (IntelliJ), but when using mvn verify on Windows I got all kinds of encoding issues, as I have some special non-ASCII characters in the WireMock responses. However, one difference from @nilsmarkmeyer is that I am not reading any file: I encode the WireMock response directly as a Java string in my test cases. Somehow, when run from Maven, WireMock sends the http response in ISO-8859-1 instead of UTF-8, and that does crash the XML parser on the other side (ie the system I am testing) as it expects UTF-8. I tried to debug what is going on, but unfortunately with no result :(  in my case, my workaround was to get rid of all the non-ASCII characters (was just one) in my tests, as anyway I did not really need them

If you could post your build and test code (or at least examples) that would be incredibly helpful.
Incidentally, have you only tried it on Windows, or is this a problem running from Maven generally?

unfortunately, it is proprietary code that I cannot share :(  I only tried from Windows, so not sure if it is a general issue with Maven. I can try to reproduce the issue in a standalone project, once I manage to find some time for it :) however, I just wanted to report it as it looks like the issue was independent on whether the response body is loaded from file or directly encoded in the Java tests

OK, I'll take a look when I can.

I had the same issue and could fix it by explicitly setting UTF-8 encoding in the Maven Surefire plugin.
<plugin>
    <artifactId>maven-surefire-plugin</artifactId>
    <configuration>
        <argLine>-Dfile.encoding=UTF-8</argLine>
    </configuration>
</plugin>

I think the problem is in com.github.tomakehurst.wiremock.common.Strings.bytesFromString(String) where the call to str.getBytes(); depends on the current platform encoding.
The correct call should str.getBytes(UTF_8);.

This seems to be fixed in 2.10.1 (and possibly in some earlier version too). Maybe this issue should be closed?


<EMAIL_ADDRESS>

Welcome! Please use this template for reporting documentation issues on https://developers.google.com/web/.
For questions about Chrome DevTools, checkout the
google-chrome-devtools tag on StackOverflow.
For questions about Chrome, check the google-chrome
tag. If you think you've found a Chrome bug, check out our
feedback page for more details on how to file it.
Page Affected: https://developers.google.com/web/TODO
What needs to be done?
Please provide description of what's needs to be done here. Please be sure to include the link to the original page
and any supporting links/data.

[ ] Appropriate type-something label has been added.
[ ] Appropriate section-something label has been added.

เเก้ไขอย่างไร