feat: add C implementation for constants/int32/max

Description

What is the purpose of this pull request?

This pull request adds C implementation for @stdlib/constants/int32/max
Related Issues

Does this pull request have any related issues?

Removes blocker for gcd, requested by @aman-095.
Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

Can I put a ready to merge for my PR? XD

Hi, just noticed that the C macros are not there for float16 and some integer and unsigned integer constants in @stdlib/constants. Would these be needed for any present/future C implementations? If so, happy to write some macros!

@performant23 I think the most high priority constants would be for float32. See https://github.com/stdlib-js/stdlib/tree/develop/lib/node_modules/%40stdlib/constants/float32/half-pi for an example. Feel free to submit PRs for constants which seem most relevant from float64 (e.g., apery, catalan, etc, are not high priority).


Hi, Can i save a temp mp4 file and add image to it after app reboot ?

Thank you guys, this is an awesome project.
And I'm wondering if it's possible to save images to a temp mp4 file, and after my app reboot, I still can add images into this temp mp4 file?
Anything will help, thanks.

this is technically challenging but possible

If I understand you correctly you would be better off creating a folder and then save mp4 clips of 1s (or whatever is your fault tolerance threshold is) to it. Basically every time you have 25 (or 30) frames, just create a new muxer and close down the old one.
This way when your app restarts you will have chunks up to the fraction of the last second. Later if you need to you can export the frames out of all these small clips into one big MP4 movie.
Let me know if that is something you want to do.


Specify maintainer in Dockerfile

As seen in https://www.codacy.com/app/ivanrome/docker-elk/issues

Since the image is built by the end user and not the repo authors, shouldn't be/she the maintainer?

It's true dockerfile is minimal and just a FROM clause exists.
Maybe the mainteiner should be elastic.co

Yes, it is already:
https://github.com/elastic/elasticsearch-docker/blob/master/templates/Dockerfile.j2#L109

Yes, but that file is not in the project but in the source image.

I agree with @antoineco here, we provide simple wrapper around the official images in order to make the stack quite extensible. We don't really maintain any Docker images in here (nothing is hosted on any registry).
And if we wanted to add maintainers, I'd see my name as well as @antoineco's name (thanks again for the great job :+1:)

Closing this, but thanks for the proposal @ivanrome, hopefully it will come in handy for someone even if we don't merge it here.


Add option to filter out orphan channels

By default, the FreeAtHome class is going to include all possible channels, even those that are not placed on the FloorPlan (and hence, not usable).
Add a default option to the FreeAtHome class that'll filter out those orphan nodes. But leave the option exposed to be configurable in Home Assistant.

Discussed here https://github.com/kingsleyadam/local-abbfreeathome-hass/issues/29


Updated README.md with a summary of Zulip

Added a summary of the Zulip project with a link to the project homepage.

FYI - @hhirsch - this may be a starting point with with Issue #21

Thanks for writing this!  Can you sign the CLA?
I just pushed a change that will merge conflict with this; I'm happy to merge this once you've done that.

Done - apologies for merging an unrelated change.

No need to worry, you can fix it using git rebase and repush to the same branch to update the pull request.

Merged after a bit of editing, thanks for the contribution!


Reduce Docker image size

Currently, the size of the built Docker image is >1GB. It should be possible to reduce this quite significantly with some changes which will make spinning up new gateway instances faster and cheaper.
The size can be reduced in two ways by:

Making use of multi-stage builds in Docker and only packaging production node_modules in the final image.
Using a smaller Node image like the Node alpine images.

Dropping the dependency on ts-node brought in by knex would be great as well though I'm not sure how one could do that if migrations need to be ran in the production container.
Multi-stage builds: https://blog.logrocket.com/reduce-docker-image-sizes-using-multi-stage-builds/
Alpine and other Docker images: https://medium.com/swlh/alpine-slim-stretch-buster-jessie-bullseye-bookworm-what-are-the-differences-in-docker-62171ed4531d

Love the feedback @CDDelta these are high value adds we definitely want! :D


Add a rule to enforce extending React.PureComponent instead of React.Component?

Would it make sense to add a rule to enforce components to be pure i.e: extend React.PureComponent
This would be very useful in redux projects where you want to enforce all components to rely on shallow prop comparison.

We're working in a relatively large team and I want everyone to use PureComponent by default to avoid a lot of unneeded rendering. If a component is not pure in that sense (which we're trying to avoid), I want to make it explicit by adding a comment to disable this rule. Does this seem like a valid use case?

Airbnb has learned the hard way that overusing PureComponent actually hurts performance. Using PureComponent is a bad idea unless the component typically renders more than once with the same referentially equal props, because the shouldComponentUpdate logic incurs a cost.
I’d urge you to reconsider indiscriminately applying a very fine-trained optimization technique as the default.

Do you have a link to support the finding that PureComponents hurt performance?
I haven't tested, but it makes sense to me that PureComponents would provide a large performance boost by removing the need to rerender entire trees of components (other than potentially a single node somewhere that has changed. A shallow object comparison should not take significant time, and it can skip rendering for the entire tree of components inside.

@hedgepigdaniel not a link, just airbnb’s anecdotal experience.

I doubt that PureComponent hurt performance as it is just shallow compare of props and state (https://github.com/facebook/react/blob/4fe6eec15bc69737910e5c6c749ebc6187d67be0/packages/react-reconciler/src/ReactFiberClassComponent.js#L259-L263). There are much more costly codes in js.
As a code reviewer/programmer, I prefer to reduce time on deciding where to use PureComponent to optimize things, it should be fine to use it as default

@foray1010 in fact it does, because the comparison itself is slow. Renders can happen thousands of times in a minute.
It's totally fine to selectively use, or not use, a PureComponent - which is why a naive rule such as this wouldn't serve anyone.
I'm going to close this unless other collaborators have a different opinion.

This would be very useful in redux projects

Actually, it's the other way around. Because the HOC produced by connect() with default options already impelements shouldComponentUpdate logic with shallow state props comparison. See docs. So making the wrapped component extend PureComponent will have no effect other than double props comparison.

Can't we inplement a lighter version of the suggested rule?
I mean, at least advice to use PureComponent, if no props.children is found.


Cleanup file conversions + OutList updates

I've updated the Write_CheckOutList files with the changes I had made to the code earlier (sept 2022), plus fixed a bug in the SubDyn output channel units.
This also updates some of the code for reading/writing OpenFAST input files. Note that the code is assuming that HD files will be updated to the format necessary for https://github.com/OpenFAST/openfast/pull/1008. Template files are also updated for that version of OpenFAST.
@ebranlard: I tried to update with your Octave compatibility changes, but let me know if I missed something. (Or if you want the template files reverted...)

Thanks, that looks good to me. Most likely, endsWith is also not available in Octave, but I can deal with it when I get to it!

@ebranlard , I had searched that endsWith is in Octave: https://docs.octave.org/latest/Searching-in-Strings.html
But it looks like that same documentation says it has startsWith, so it's probably an issue in the version you're using, too. Sorry about that!

My version is 6.4 which is a year old. I'm going to update it...


feat(cert): Certificates

Closes #12794
Tasks

[ ] Fix challenge thumbnails, they have legacy screenshots of the certificates

Wait until final QA

@jonathan-grah thanks a lot, and great work. I'll pull down to test too.
Some opinions, Let's not reactify this as of now. We may want these server rendered for better share-ability on social networks.
Not critical, but adding structured data (which are used to show cards, etc. Ex. og: tags) on the views would be rather more good to improve that.
But that can be separate from this PR and not mission critical at all.

@raisedadead, good ideas regarding reactification, makes sense.
I will look into having structured data, might have to ask you a thing or two about that 😉

@QuincyLarson :
Jonathan is correct. He can make the final update once we approve the look and feel.
@jonathan-grah :
Now coming to the review:
The Look and feel

Currently the certs look like so:

I have made some changes to the same to look like so:

The original looks a bit squared out and flat, vs the below that has subtle curves, in tune with our UI.
I think centered text looks better

The profile page
Until the profile page is in react, I think the static view (show.jade) should be updated.
You can see the all suggested code changes at my fork.
https://github.com/raisedadead/freeCodeCamp/commit/3cd200b78f58d40709b301d757e3a1c98ffa204a
@QuincyLarson if you agree we can make this changes, that I have proposed.

@jonathan-grah
I'll do the final testing of all of the other backend code, once we are though with the UX.

@jonathan-grah
Also, you need a rebase against the current staging, although there is no conflict yet. Certain routing is broken if this is PR is pulled down as it is. Its weird, but a rebase fixed it.

@raisedadead, thanks for taking the time. Your updated design was very similar to an early iteration, but it was decided against by @QuincyLarson and Michael (I can't remember the exact reasons). This was over a year ago, so @QuincyLarson, please give your opinion on this design update.

@jonathan-grah Yes - I agree that this looks better centered!
I like the flat green rather than the translucent image. It looks cleaner and more consistent with our "shapes-not-images" design philosophy.
One minor edit - can you change "Executive Director, freeCodeCamp" to "Executive Director, freeCodeCamp.org" - the .org simultaneously conveys that we are a website and a nonprofit.


Mark GROOVY_CLOSURE_CLASS as final and private

I had omitted this because it is a nested class inside the private static class DefaultFactory class. However I have no objection to this change.

FYI Some Checkstyle-like (internal) tool pointed out the missing final, just because of the all uppercase name.
The private may indeed be unnecessary technically, I just added that for consistency while I was at it (the tool did not).


Not getting the latest version of framework when installing through composer.(Getting Version 0.1.3)

Framework Version: 0.1.4

This is not an issue and could have been done on a single request on other channels.


Config of user management module doesn't get reflected immediately after changing it.

Fixed. Coming in v0.17.0


Allow alternative config file naming like Nlog does.

In the Classic desktop work it is common to deploy multiple applications in the same folder.
Project structure is then:

Project
-- WPF app1
-- WPF app2

The installer (like WIX) then takes the result of the build and places in the same folder, creates start menu items and shortcuts.
Usually, app1 and app2 are always deployed together and create only one entry in the "Add/remove" programs.
This is not a problem for NLog because:

The following locations will be searched when executing a stand-alone *.exe application:
- standard application configuration file (usually applicationname.exe.config)
- applicationname.exe.nlog in application’s directory
- NLog.config in application’s directory (Name sensitive; using docker dotnet core)
- NLog.dll.nlog in a directory where NLog.dll is located (only if NLog isn't installed in the GAC)

Basically, WPF_app1.exe.config solves the problem when two exes are deployed together with specific loggers for each one of those.
For the development purposes I have the following structure:

ApplicationInsights.config
ApplicationInsights.Deployment.config
ApplicationInsights.Test.config

VS uses the default and sends to debug listener only, during test, data is send to the separate instance (who want to see breakpoints) and when deployer, installer takes the deploy config.
Nothing new :)
However, when deploying two apps together there could be only one ApplicationInsights.config file for both apps.

Hi @Muhomorik, thanks for the interest in our SDK :)
Respectfully, this is not a feature that we're likely to support.
You may instead be interested in an in-code implementation of the config. I understand that may be less than ideal, but that should help achieve your scenario.
I'm going to close this issue for now, but if you have more questions feel free to re-open.
Thanks,


Unpredictable behavior for attached databases

If you attach a database(not account) then cosmosDB creates an account node and only under it you can see your database. If you want to add few databases from the same database account(in case if you have access to databases, but not for an account) then those databases are going to be in a different accounts tree item.
I think it's a bug and it would be good to put them under the same account node and its name should be like account3.documents.azure.com:10255, not account3.documents.azure.com:10255/database1:

The next thing: I added copy connection string buttons to attached accounts/databases and when I tried to copy connection string from the created account, I found that it contains database connection string, not the account one. The difference between them tiny and it's easy to get account connection string from database one.
So it would be good to have a correct account connection string for attached accounts that was created from database connection string.
When I create a database under the account it looks like everything work:

But when I copy connection string of database3 I can see that its the same as connection string of database2 and the same as account3.....database2, under which database were created. After tree was updated, I can see that the database created correctly under account3(with correct connection string), but it disappeared from the attached account node.

So, this behavior looks a little unpredictable for me and it can be improved in the way I mentioned before.

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


chore: set type to module in package.json

The type property of package.json may be used by frontend tooling (e.g Webpack, Vite) in determining whether the package's entry point is a conventional CJS or ES module. If no property is specified, the entry point may be treated as a CJS module. As an illustration, here is a warning that Vite shows when you import vaadin-usage-statistics:

@vaadin/vaadin-usage-statistics doesn't appear to be written in CJS, but also doesn't appear to be a valid ES module (i.e. it doesn't have "type": "module" or an .mjs extension for the entry point). Please contact the package author to fix.

Since Rollup converts vaadin-usage-statistics to an IIFE format, I would suppose the output can be treated as a regular ES module so that it shouldn't be an issue if we set the type property to module that is needed to avoid possible misinterpretation.
Related to vaadin/flow#12574
Type of change

[x] Internal

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


run_clm.py (+ other examples) output checkpoints in sharded format

When training with examples like run_clm.py and using TP, the checkpoints are saved in sharded format and require consolidation before they can be used with downstream tools.
For example, the following code can be added to the end of the training block in run_clm.py to perform consolidation and remove the shards:
    if (int(os.environ.get("RANK", -1)) == 0) and int(training_args.tensor_parallel_size) > 1:
        print("Converting sharded checkpoint to consolidated format")
        from optimum.neuron.distributed.checkpointing import consolidate_tensor_parallel_checkpoints_to_unified_checkpoint
        from shutil import rmtree
        consolidate_tensor_parallel_checkpoints_to_unified_checkpoint(
            training_args.output_dir,
            training_args.output_dir,
            "pytorch"
        )
        rmtree(os.path.join(training_args.output_dir, "tensor_parallel_shards"))  # remove sharded checkpoint files

Should this be the default behaviour? Or should we add a flag to automatically trigger this consolidation?

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!


[WIP] Remove providerID field from m3mdata

Remove providerID field from m3mdata
Reverts https://github.com/metal3-io/cluster-api-provider-metal3/pull/369

/test-v1b1-integration

on hold until proposal is approved

/hold

/test-v1a4-ubuntu-integration

/test-v1b1-ubuntu-integration

/hold cancel


feat(security-code): add support for security code for stored card me…

Description
Adds the option requireSecurityCode for embed to prompt for security codes on stored cards..
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own changes
[x] I have run yarn lint to make sure my changes pass all tests
[x] I have run yarn test to make sure my changes pass all linters
[x] I have pulled the latest changes from the upstream main branch
[x] I have tested both the react and the CDN versions on local and integration environments
[x] I have added the necessary labels to this PR in case a new release needs to be published after merging into main (e.g. release and patch)

Contribution guidelines
For contribution guidelines, styleguide, and other helpful information please
see the CONTRIBUTING.md file in the root of this project.

@cbetta This is the error you get locally when it is set to just 'dev'

:rocket: PR was released in v2.12.0 :rocket:


Documentation on all Pub pages is wrong

This issue was originally filed by<EMAIL_ADDRESS>
From the "File" menu, open a "New Application" (and make sure "Add Pub support" is checked).
'Add Pub Support" no longer exists as an option.
There are no instructions as to how to add something to the pubspec.yaml (which would involve the 'open as test' command). This should really be updated to the editor pane.
There is no longer a need to do a Tools -> Pub Install, that will happen automatically

This issue has been moved to dart-lang/pub#559.


Fix jank index loading

how is this worse than it was in 1.3.10....

mostly fixed in b63611ede80a8ee8c116e6764f9c7a0ea1ae2e29, turns out unzipping the zip file takes 30s alone. lol


Add Decorator design pattern (Java)

Title:
Decorator design pattern (Java)
Body:
Attach additional responsibilities to an object either statically or dynamically using the Decorator pattern
Are you working on this? - Yes

I am working on this. Please assign this issue to me.


Great version 1.2.2, but little problem with slow brightness adjustment.

Hello mrfaptastic,
I just updated your library to version 1.2.2 and it still works fine.
Since this version, the whites are perfect, and the colors are more contrasted.
I am using an ESP32 is two in-line RGB LED matrices (128x32)
And to get a good fluidity of the texts I increased min_refresh_rate to the value 1600 (I don't know the real limit)
Since this version 1.2.2, I just have a small problem with the "setPanelBrightness" command which no longer acts instantly.
I have a page an html page (in websocket) with a slider that allows you to adjust the brightness.
The slider value is read by setPanelBrightness each time the main loop is passed.
With the previous version the brightness can change immediately when I touch the slider.
Now I have to wait several loop (up to 5/10s) or a reset, for the brightness to change.
However if I display my instant value with Serial.print(myValue) this value changes well in same time with the cursor, but the setPanelBrightness command is much slower.
Did you change anything around the brightness setting?
Or do you have an idea?
Thank you for your work and your responses.

Hi @FrenchLab47. Glad you find the new library working well. I did make a change to the code to try and improve the 'drawPixel' performance quite a bit and the brightness check was a casualty.
There are two options to resolve:

Immediately after you call setPanelBrightness, do a clearScreen() and repaint.
Or on line 436 comment out  #define GO_FOR_SPEED 1 aka:

Turn

 #define GO_FOR_SPEED 1

to

// #define GO_FOR_SPEED 1

Above this #define you'll see what I was trying to get at. I think I might update the GIT so this is actually commented out by default. I got carried away trying to performance tune PatternPlasma to work smoothly on a  256x256 pixel matrix :-D!

On that note, you wouldn't mind sharing the code you for a) the websocket server & brightness call and b) the html / js required to do the scroller that does the websocket call? I would love to use this in one of my projects.

Thank you for this quick and precise answer, I confirm that it works.
I plan to share my code soon, with the web settings pages (mobile), the printed circuits, and the parts of the box in 3D printing.
I still have a few cosmetic bugs to resolve, a bit of cleaning up in the code, I'm about 95% of my job.
Best regards


[WIP] Deprecate callers to Address in Host

Wraps address in deprecated alias_attribute.

@miq-bot add_label wip

This PR is a replacement of https://github.com/ManageIQ/manageiq/pull/9204 as just removing all the callers to a method is hard, as there maybe some callers we don't yet know about.

@juliancheal There already is a mixin for deprecations at app/models/mixins/deprecation_mixin.rb  Can you reuse that file or does it already do what you need?

@juliancheal There already is a mixin for deprecations at app/models/mixins/deprecation_mixin.rb Can you reuse that file or does it already do what you need?

Ah thanks for the heads up. This looks good.
I think think we should log that a method is being deprecated. I'll update the PR.

Coolio, so we do log deprecations, but we list them as being removed in F release. What if we're going to depreciated something in the future in an unknown release? https://github.com/ManageIQ/manageiq/blob/master/lib/vmdb/deprecation.rb#L4 /cc @bdunne

@juliancheal When we cut the F branch, we change that to the next version

@Fryguy ha that was my guess.

Squashed commits to reflect this PR better.

@miq-bot remove wip

@miq-bot remove_label wip


Backslash is printed when escaping double quotes in cmd

Test configuration:
{
"name": "Test escaping characters",
"cmd": "echo "Hello"; echo 'world'; "
}
Output:
$ echo "Hello"; echo 'world';
"Hello"
world
Expected:
$ echo "Hello"; echo 'world';
Hello
world
Notes:
Same result using unicode \u0022 => {"cmd": "echo \u0022Hello\u0022; echo 'world'; "}
Environment:
MacOS 10.10.5
Shuttle v1.2.7
iTerm2 Build 3.0.8
Terminal.app 2.5.3 (343.7)

What are you wanting to accomplish? It seems that its working as expected.
If I manually type in terminal echo \"Hello\" I get the expected output of "Hello"
I get this because I'm escaping the double quotes and they print.
If I manually type in terminal echo "Hello" I get the expected output of Hello
I get this because I'm Not escaping the double quotes so they don't print

Probably I'm not fully understanding how escaping works. Let me give you a real life example.
I want to run the following command: ps -ef | grep java | awk '{print "Pid: "$2, $8, $9, $10}'
If I insert this in the shuttle.json config file I have to escape the double quotes of the print part otherwise it breaks the config file, so my cmd looks like this: "cmd": "ps -ef | grep java | awk '{print \"Pid: \"$2, $8, $9, $10}'"
But when I execute this through shuttle it runs it with the backslashes breaking the actual command:
$ ps -ef | grep java | awk '{print \"Pid: \"$2, $8, $9, $10}'
awk: syntax error at source line 1
 context is
       	{print >>>  \ <<< "Pid: \"$2, $8, $9, $10}
awk: illegal statement at source line 1

What am I doing wrong?

Btw, if I apply the same to name tag it works like I would expect:
Name tag is configured as follows:
"name": "ps -ef | grep java | awk '{print \"Pid: \"$2, $8, $9, $10}'"
And in the menu it shows up without the backslashes:

ps -ef | grep java | awk '{print "Pid: "$2, $8, $9, $10}'

@Franklin88 Awesome, Thanks for the real world examples. I'll look into a solution.

@Franklin88 Seems that we have a bug with the way double quotes are escaped.

Great! Thx

Not sure if this is the same issue or not, but I'm having an issue with wanting to use a backslash in my command and it not working. I try running it from the menu, and nothing happens (as soon as I remove the backslash, it works fine). It's the one right after "Mobile":
		{
			"name": "Run local site",
			"cmd": "cd /Users/myusername/Library/Mobile\\ Documents/com~apple~CloudDocs/Sites/mysitename && commandhere"
		}

I tried it with and without the double backslash, and neither works.

@WorldWideWebb I have confirmed that the bug is causing this issue as well. I'll try to have 1.2.8 for the weekend.

Anything new on this bug?

It's resolved on my dev system. Doing some more testing before I release. I can send you what I have if you want to test.

Should be fixed in v1.2.8. Let me know if its not working for you.

Works great! Thanks!

@WorldWideWebb Thanks for the update!

Hi! Thanks for the new release. The escaping works much better now. However I'm still experiencing some problems when the shuttle.json file is parsed.
In some cases escaping the semicolon when concatenating commands fails. For example:
Let's say I want to open Google and close the terminal after that (just as an example):
Configuration:
{
"cmd": "open https://www.google.es; exit",
"inTerminal": "new",
"name": "Google"
}
Output:
open https://www.google.es; exit
The file /Users/foobar/exit does not exist.
As can be seen a backslash is printed before the semicolon which breaks the command. As curiosity, if I put a space before the semicolon it works as expected:
"cmd": "open https://www.google.es ; exit"
However, I understand that it should work with and without a space like it does from the command line.

@Franklin88
I'll look into this... however you should not need the open directive try command:
"cmd": https://www.google.es"
this should open in the default browser.


Updating translateable config settings for Standard Campaigns and SMS Games.

Refs #5147
What's this PR do?
This PR updates the configuration variables for Standard Campaigns and SMS Games section of variables within DoSomething Config -> DoSomething Campaign.
Where should the reviewer start?
Just confirming that the code looks correct.
Any background context you want to provide?
The fields are translated both on the main site page, user-facing output and within the input fields in the CMS. They have all been testing to confirm that if a translation is available, the translation is used.
What are the relevant tickets?
#5147

@angaither

:+1:


Entity Type Update + Snapshot and Draft Refactors

Issue #: /bcgov/entity#10620
Description of changes:

Entity snapshot refactors
Unit test additions/fixes
Includes Business Type + Update tooltips
Implemented Placeholder for Change Summary
Refactors to handle save and resume for Change Draft Filings
UX Updates for My Business Section
Implements updated Fee Codes

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Codecov is consistently reported on untouched code.. so i'm not sure 🤷


/nickserv setusername

We got a report in #oragono from someone who doesn't like that the username of always-on clients is always ~user (after a server restart).
Let's provide an additional nickserv command to change the username (and send chghost). To avoid creating any incentive to use ident, this should always prefix the chosen username with ~.

I started doing this and realized I really don't want to. User/ident is cancelled.

Well it could simply be changed to the users actual nickname, so  Amiga600!~user@host - would become Amiga600!~Amiga600@host instead?

Non-ASCII nicks are invalid as user/ident fields.
We've actually gone even more aggressively in the opposite direction here with #1213 (an option to always coerce user/ident to ~user, even for non-always-on clients). I'm tempted to make that a recommended default, actually, since it makes all implementations of /kb function identically.

Well perhaps have logged in users show as Amiga600!Amiga600@host then? - Just forget using the ~ character perhaps?

It's more like, 姚明 is a valid nickname in the default configuration of oragono, but it's not valid as a user/ident (with or without a preceding ~).

Surely, that could be overcome with a simple check
ie. remove all non-ascii chars - if string has still illegal chars, set to "user" - if not set to "nickname"

To what end, though?
#1213 appealed to me so much because the people who asked for it reported that user/ident had become merely a source of confusion. I don't really see the point of the user/ident field anymore.

if the issue is aesthetics( / not including an english word) maybe just dan!~@localhost could look a lil nicer on the eyes than ~user everywhere~?

I wouldnt say its aesthetics its keeping to the IRC standards: https://en.wikipedia.org/wiki/Ident_protocol
Its hardly ever replied to for say Windows IRC Clients though, or people behind firewalls, as usually the IDENTD port is blocked by said firewall, in which case it is usually set to the nickname
For example on one irc server: Amiga600 is ~Amiga600@host
on a completely different one: Amiga600 is Amiga600@host
One simply ignores it, and passes my nickname onto IDENTD - the other correctly identifies me and puts my name without the ~ character there

Ident lookups are already disabled in our default configuration, though.

Yeah we're definitely aware of the standard way of doing things, the always-on way of working deviates from that a lot already though. In the 'default' config (the one where always-on, chat history, etc is all enabled), prioritising the account name (and making the nick and account name the same thing) is a pretty big part of sling's preference. The 'conventional' config's the one I like to keep as close to standard, traditional IRC though.
In something like the always-on mode, I can definitely see why sling'd prefer the ident field to never change. Letting users have their nickname as their ident would be nice for some users, but for people who aren't as used to traditional IRC they'll probably get confused as to why their nickname is used in two places.
In the channel an idea that was brought up was having an coerce-ident string instead of a suppress-ident bool, so the network admin can set the ident field to whatever string they wanted (with the default being ~ in the default config, and the conventional config not using it at all). For normal IRC, separate nick and ident fields are a given, but for the always-on side I reckon a change that de-emphasises the ident field makes sense, given that's also aimed as people less familiar with irc's intricacies.

I guess for simplicity then, perhaps have a config option, so the end user can then either set it to what they want?


PIN validation

Hi,
I wondered if you'd thought of adding pluggable validation rules when choosing a PIN, such as non-sequential numbers, and non-repeating numbers.
Or perhaps just a couple of new parameters that allow restricting the use of sequential numbers and all repeating numbers?
If not, would you accept a pull request?
Thanks.

Hi,
This is a really cool idea :). I'll give it a thought during the weekend.
Thanks!
Cheers
Jeremy

Hi,
Great idea. Are you likely to add this feature soon? Would you like a pull request for it?
--Tim

Hi,
Sorry for the delay @timt .
I started to work on this today. Will keep in touch.
Cheers,
Jeremy

Hi,
This is done in version 1.7.0 :).
I chose to go for the RegExp solution.
You can now use validationRegex property to check the PIN entered by the user and the titleValidationFailed one to tweak the title displayed if the check fails.
Thanks again for this idea :).
Cheers,
Jeremy


Support unpacking archives in the download task

I'd love to be able to download an archive and extract it as a single task or groovy command, like so
download {
    src url
    dest new File(baseFolder, filename)
    extractTo new File(baseFolder, folderName)
    onlyIfNewer true
}

where if you download a zip or a tgz file you can have the download command extract the archive to the specified folder
This would be equivalent to the above download followed by
copy {
    from tarTree(resources.gzip(new File(baseFolder, filename)))
    into new File(baseFolder, folderName)
}

Pretty good idea, but I think this should already be possible with existing means. Have a look at the following example:
https://github.com/michel-kraemer/gradle-download-task/blob/master/examples/unzip.gradle
Adding such a feature to gradle-download-task seems to me like reinventing the wheel. The copy task combined with zipTree or tarTree is very flexible and well tested. The only clean and maintainable solution IMHO would be to somehow auto-magically add a copy task succeeding the download task if extractTo is set. I'm not sure if this is possible in Gradle, but pull requests are very welcome! :smile:

My main issue with the unzip example is that because it's a distinct task, there's no way to short-circuit it if the file has already been downloaded and unzipped.  The the onlyIfNewer flag, I can skip the downloads, but I haven't seen anything that will let me skip the archive extraction if the target folder has already been created.  With a large set of dependencies, it's a pain to have extract run every time i do a build.

I just tried to reproduce what you are saying. I added overwrite false and onlyIfNewer true to the Download task and ran the example twice.
The first time it downloads the file as expected:
$ gradle -b unzip.gradle
Download https://github.com/michel-kraemer/gradle-download-task/archive/1.0.zip

> Task :downloadZipFile
Invalid cookie header: "Set-Cookie: logged_in=no; domain=.github.com; path=/; expires=Thu, 04 Mar 2038 07:28:21 -0000; secure; HttpOnly". Invalid 'expires' attribute: Thu, 04 Mar 2038 07:28:21 -0000

BUILD SUCCESSFUL in 1s
2 actionable tasks: 2 executed

The second time it says that everything is up-to-date:
$ gradle -b unzip.gradle

BUILD SUCCESSFUL in 0s
2 actionable tasks: 2 up-to-date

Isn't this what you expect?

If the file on the server side changes, it will re-download, but will the overwrite false prevent it from unzipping the new file?
My goal is to be able to download and decompress a bunch of pre-requisites for the build process such that if I've already downloaded the content, the tasks produce minimal overhead, but if the server side content changes, the new content is downloaded, replaces the old content, and on subsequent runs I get minimal overhead again, until such time as another server side file changes.

I'll take a look at the behavior when I'm back at work next week.  Thanks for the effort.


Unable to pull image

Unable to pull image from a session on https://labs.play-with-docker.com

I just tried this out and it worked fine. Can you please try to reproduce or close the issue?


Create Travis build

Steps to create appmaps:

added appmap.yml with minimal config required.
changed test_defs.bzl file to include jvm args for all tests. paths used in the jvm args are absolute paths
changed travis build file to run script bazel query //javatests/... | xargs -n1 bazel run .

Issues encountered :

bazel query //javatests/... | xargs -n1 bazel run is exited with 123 status once executed
PR build for appland branch is failing due to json issue in one appmap file.
PR builds for versions (2.28.1,2.28.2,2.28.3) are not able to upload appmaps via ruby script.
PR builds for oldest versions (2.28, 2.27) are not getting executed due to conflicts with master branch.

All the Branch builds are able to upload appmaps successfully.


Addon removed from Firefox

The link in the README (https://addons.mozilla.org/en-US/firefox/addon/duosu/) shows this:

Workaround
Manually edit firefox/manifest.json to add:
  "browser_specific_settings": {
    "gecko": {
      "id"<EMAIL_ADDRESS>    }
  }

because without an explicit extension id, the persistent storage API is not supported, which results in the extension crashing with error json is not defined because it cannot read or write storage. Then go to about:debugging in Firefox and load the manifest.json file as a temporary extension. It works as advertised.

Oh, it's just an outdated link. I guessed the correct one, I think: https://addons.mozilla.org/en-US/firefox/addon/duochrome/

Yep, had to rename it. Was the add-on force removed for firefox users?

wait I thought you were saying the firefox add-on like blew up or something
Sorry, I had to rename it from DuOSU -> Duochrome. I tried to rename everything in the README to point to Duochome but I forgot the firefox link (the old chrome one still points to the new one so I overlooked it). Sorry for the annoyance but glad you found it


Preliminary integration for API V2

Pending https://github.com/inaturalist/iNaturalistAPI/pull/196, and should be updated if/when we merge https://github.com/inaturalist/inaturalistjs/pull/31
Basically uses the alpha v2 API on obs detail when specified by URL param or by an opt-in test. Parity isn't complete yet, but the basics are there.

This all looks good to me. I fixed two small things

the identifiers weren't loading properly on obs show
there was an undefined constant in webpack/observations/show/ducks/controlled_terms.js which I presume was legacy from early in the implementation. The fetchAllControlledTerms method that contained it appears to only be used on Identify and ProjectCreate, so not yet using the V2 api anyway


#21 fix .widget-container size issue

I tried to resolve the css bug I mentioned in this issue: https://github.com/Wolox/react-chat-widget/issues/21

Hi! Thank you so much for contributing to this project! Now, I've tested what you change in this PR, but this renders something like this

I've seen that the problem is the height: 100%; in the widget-container's media query. It's no wrong but it shouldn't be there all the time, so the fix is to add it when the launcher is clicked :smile:

@mcallegari10 hi, I corrected it, could you please review it? thanks

Hey There. Curious if this fix was ever reviewed?

Fixed in #51


[doc] Update README.md

fix the sentence for more general hardware

Hi @mkbhanda  Kevin, please help to review


change test runner to ava and enzyme

lately I've explored the test runner avajs and it seems rather better than mocha, there is an option to explore it? step include enzyme?

@carloscuatin I've pushed the file structure changes. Earlier today, I saw your tweet. It's great that you started on implementing ava. :) Could you pull the v2.0.0 and continue working? It will be easier for you merge now than later.

@somus of course today finished implementing server tests

Hey @carloscuatin, how is ava implementation going? Could you include code coverage along with it too?

@somus if I send the changes as soon as possible and everything is ready

apologies week was very hard and could not send the PR but I'm preparing good vibes

@carloscuatin Hey Carlos! Hope you are doing good. Can you please give us an update on this? We are looking to release v2.0 towards end of this week (or early next week). If you can let us know the status, it'll be great!
In case you don't have much time, may be @somus can help you and finish this over the weekends (so that the release is not delayed).

@sandeeppanda92 quiet before the end of this week sends the update

Completed in v2.0.0 branch.


indices:admin/create requires a wildcard

It seems a wildcard is required for the indices:admin:create permission to work.
With this :
sg_myindex_admin:
  indices:
    'myindex*':
      '*':
        - ALL

Everything is fine. The users with sg_myindex_admin role can create and/or delete myindex.
But with :
sg_myindex_admin:
  indices:
    myindex:
      '*':
        - ALL

I get an error: no permissions for indices:admin/create.

This seems indeed to be a bug, will fix this, thx for reporting.


Nothing happen after approving the bluetooth pairing on both devices

I've tried tapping "accept" on the jade first, vise versa, both at once.
Never gets anywhere next.
Android 11

Have the same problem on new phone, also a samsung, S21

Is this still an issue ?

Closing as no comms for 18mo.


fix(autoreload): Reload invalid yaml files

When a YAML file is invalid, trigger auto-reload anyway so that user is aware that the configuration file is incorrect.
Failing to do so does not change the reload status in metrics and api.

Does this need to be backported to 2.55 ?


The NVIDIA driver was unable to open 'libnvidia-glvkspirv.so.510.60.02'.  This library is required at run time.

Describe the bug
When trying to launch elden ring on nvidia 3080 with the 510.60 drivers I get the following errors with a crash on launch: The NVIDIA driver was unable to open 'libnvidia-glvkspirv.so.510.60.02'.  This library is required at run time.
Steps To Reproduce
Steps to reproduce the behavior:

nix.pkgs.config.allowUnfree = true;
services.xserver.videoDrivers = ["nvidia" ];
hardware.opengl.enable = true;
programs.steam.eanble = true;

Expected behavior
The game should launch without crashing as it does on other distributions
Notify maintainers
@colemickens (just added this here because you said cc you)

cc: me. I have some commits to push related to nvidia, I'll update and see if I can repro this and address it.

cc: me. I have some commits to push related to nvidia, I'll update and see if I can repro this and address it.

are you getting this issue as well?

https://inbox.vuxu.org/voidlinux-github/20210727232846.qe2ZUqckbjL_9JyFeNsGMDlIT82dvCtva-DRkE_LeOk@z/T/
Might be some useful stuff in here, it's talking about running sudo ldconfig to fix a similar error, however filesystem is read only

Uh, I'm not actually sure, I might have to rope in my partner to login to Steam to fully test out the Elden Ring scenario. If there's an easier repro or another Vulkan? game or something that triggers it, please let me know.

@colemickens I wish, I just tried ghostrunner in dx12 mode and it ran just fine, so its not a dx12 to vulkan issue...

@colemickens I figured it out, I'm an idiot, I was installing steam as a package ontop of having programs.steam.enable = true;
having both created a system conflict!


Navigations_AllUsers

2019-07-25 Placeholder

This is going to take a little bit of collaboration with everyone. Need to get 75% agreement on the nav bar titles and then create a pseudo template so that it's consistent on every page.


Tournaments: Be less case sensitive

I've seen people having problems with this lately.

This is the wrong place for this code?

Like, this only affects display, not actual recognition.

Oh! You're right. I really should learn not to do this at 3:00 AM.

...Okay, maybe I'm still too tired to understand. Why doesn't this work?

This makes a rule that's capitalized incorrectly display as a rule.
It doesn't make a rule that's capitalized incorrectly actually work.

https://github.com/Zarel/Pokemon-Showdown/blob/8a4415f59f9c1bf7ef6c48b9b2838254713bee74/sim/dex.js#L489
is the line you need to change to be case-insensitive.

I really wish the github search function would work better :/

now normalize it somewhere, so you only need to do the case-insensitive check once. :p

Well, it functions properly with just one, but the other one is needed so that everything looks right in the chat, isn't it?

Oh my god I didn't even read that right. What is wrong with me?

Fixed in 0c94145c4074e6df55521aecd29c45d65c4598ff.


Unable to extend multiple feature layers

There appears to be a bug affecting multiple layers in full offline mode. We are seeing not all layers being written to indexdb.

It looks like a bug. The layer.extend() callback is returning before the database add/update is complete. It's very possible if you do multiple extend() operations that the current add/update may overwrite or interrupt the previous operation causing a skip to occur.
You wouldn't notice a problem if you are only using a single feature layer.
I'll see about getting a fix implemented asap.


device.relaunchApp was supposedly removed as of Detox 6, but still is defined

Per  the docs for device, relaunchApp was deprecated and supposed to have been removed as of Detox 6, but it is still defined

There are still some docs that reference relaunchApp as well

Hello,
It has not been removed because we do not wish to break existing users. Since the function does nothing but call launchApp, there is little maintenance upkeep to keeping it.
I remove the line from documentation that it will be removed. It remains deprecated, however.
If you'd like, please submit a PR with removing relaunchApp from the docs and test suite. It would be greatly appreciated.
Thanks


Default configuration not working in linux

Hello, thanks your vim 9 work!
Using the default configuration:

I get the following behavoiur.

Pressing enter on the search.

You have set grepprg to  grep -n $\* /dev/null. This is incompatible, but in a way I did not foresee this. So I removed the dependence on grepprg. It should be fixed now.
https://github.com/girishji/scope.vim/commit/caa05c1c80030b86aabf64ab8d12acb90aaa5590 and https://github.com/girishji/scope.vim/commit/61b48a989c05477e2977c0f58e3114f70fce42fc


Allow metadata

For required purchasing clips on certain subscriptions

Looks okay to me, and it's running in prod so I guess 🛒

How can it be in prod, if it's not merged here?
@dnahodil, would you mind taking responsibility for this, since Will is away?

How can it be in prod, if it's not merged here?

GemFile is pointing to the branch

@dnahodil, would you mind taking responsibility for this, since Will is away?

I... uh... um... okay.

GemFile is pointing to the branch

Correct, was just testing you.

I... uh... um... okay.

You don't have to, I just thought that you knew about this.  And you are the gopher.

You don't have to

I'll do it if I get time. 👍  It means merging this PR and updating the web-app GemFile.

Closed https://github.com/Biteable/shutterstock-ruby/pull/6


updating build timing types to fall in line with the ones in origin

@bparees ptal

[test]

@bparees fixed, ptal

[merge]


issue fixes #1 Capture and CaptureWithCGo leak file descriptors on OSX

symptoms after open file descriptors exhausted:
--- FAIL: TestCapture (0.03s)
Error:      	Expected nil, but got: &os.SyscallError{Syscall:"pipe", Err:0x18}
    Error Trace:    capture_wrapper.go:26

Error:      	Expected nil, but got: &errors.errorString{s:"fdopen returned nil"}
    Error Trace:    capture_wrapper.go:37
            	capture_test.go:13

fixes file descriptor leaks on Darwin & Linux:
tested on:
go version go1.7.4 darwin/amd64
go version go1.6.1 linux/amd64

@oberg-s: Sorry for taking ages, I was too busy. I will take a look at the PR now.

Pushed in your name using https://github.com/zimmski/osutil/pull/3


Fix up commits, misc changes

domd: update the commit for the kernel
Add disk images to .gitignore
Fix up all floating hashes and tags

Ack is added. Tested. Ready to be merged.


Updated for Swift 2.3/Xcode 8

Updated dependency libs with Swift 2.3 support
Fix SDK for Swift 2.3 ♥

Works well. I hope this is merged soon!

Thanks for the PR! A few things before I can merge this:

Can you sign the  Uber CLA?
I have a swift-2.3-dev branch up currently, think you could make this PR agains that branch instead? I want to make sure everything is working there for 2.3

I do not have anything new to add to the swift-2.3-dev branch. Thanks for creating the branch.

Oh sorry about that! Thanks for contributing! Were you able to successfully run tests? I'm running into some issues with the simulators & entitlements for xcode 8

I have tests breaking in OAuthTests, and also fatal error when KeychainWrapper object is being unwrapped to AccessToken.
Didn't see any issues on entitlements though.


core[patch], langchain[patch], vertexai[minor]: ChatVertexAI.with_structured_output

cc @lkuligin

Happy to help with this but should move to https://github.com/langchain-ai/langchain-google


A question about the 'go.norm' file.

My question is I don't know how to extract the 'go.norm' file from 'go.owl'. I'm not familiar about Ontology Language and groovy script language. In my conception, the files 'Normalizer.groovy', 'Corpus.groovy' and 'Axioms.groovy' were used to extract NF1~NF4 from go.owl file.And I don't know the data flow excatlly pass. I hope you could explain the data flow for me(the best way with a graph).Thanks for you very much.

Hi,
use Normalizer.groovy to generate go.norm file, you need to provide an owl file.


[Bug]: Extremly Slow and Stuck

Requirements

[X] I checked the troubleshooting section in the README to verify that I have the latest Mushroom version.
[X] I did a search to see if there is a similar issue or if a pull request is open.
[X] I have read the state of card mod support (in pin issues) and understand card mod support is not provided in this repository.

Current Behavior
HA: 2024.2+
Logs: Clear
Cache: clear
Chrome
Same problem on Phones
From new Update HA 2024.2+ every Card where is Customized with Card Mod extremly slow load i mean few seconds 5-20, i dont know is this problem in Card Mod or Mushroom, or after few minutes Dashboard it gets stuck and freezes
Expected Behavior
No response
Steps To Reproduce
No response
Context
No response
Environment
No response
Anything else?
No response

https://github.com/thomasloven/lovelace-card-mod/issues/359

I'll close this issue as it's card mod related. See https://github.com/piitaya/lovelace-mushroom/issues/1366


Link to api in plugin section is broken

Clicking the api id in a plugin, on the plugin section should bring the user to the edit api screen (at least that's what I'm guessing from the link), but the id is missing in the link, thus breaking it.

Hi @PauloGDPeixoto , can you clear your cash and try it again? I think this is a caching issue from a previous version (if you're using the latest commit)

I was still on 0.6.5, after pulling 0.7.1 the issue was resolved and I'm marvelled with the new features, I have a lot to explore! Sorry about the bogus ticket.

You could also try the next branch or if you're using docker
run pantsel/konga:next.
The version is almost ready and handles lots of things better.


6_JavaScriptClient has been moved to deprecated folder. My code is not working. Need help.

I am trying to use JS client along with Identiyserve4.
I was trying to find out a sample code for this approach.
But found out the folder in the 'Deprecated' folder.
Still I used it and was getting error on the 'Callback.html' - Error - OidcClient.processSigninResponse: No matching state found in storage.
Tried checking on stackoverflow, but did not get any resolution.
Need a working sample code for reference to test the javascript client. Can you please help.

Opening an issue in the JS lib would be more suitable.
https://github.com/IdentityModel/oidc-client-js


Dropdown menus don't get closed when selecting an option

[Tested on Chrome 21, Safari 6 and Firefox 14]
I've upgraded Bootstrap on a test-branch and dropdown menus no longer close automatically once an option is selected. User has to click outside for the menu to disappear.
The problem is reproducible in the official docs: http://twitter.github.com/bootstrap/javascript.html#dropdowns
Did I miss something on the changelog about this new behaviour? If this is a "WON'T FIX", is there any workaround to force them to close?

perfect marcelometal!!!


feat: add support for maps

Related to https://github.com/dylibso/xtp/pull/852

I need to study this a bit more. On the one hand, there is going to be some complication with this change and i get that. On the other, this is a lot of code, and moving some of the rendering to the helpers adds some indirection. I also feel like there is a lot of repetitive code in here and there might be some simpler ways to do this.

@bhelx

On the other, this is a lot of code, and moving some of the rendering to the helpers adds some indirection. I also feel like there is a lot of repetitive code in here and there might be some simpler ways to do this.

Yeah I agree, I went ahead and refactored the code a bit

Putting this feature on ice for a little bit

Closing in favor of #37


Content routes with markdown component not rendered statically in Angular v16

Please provide the environment you discovered this bug in.
Here's a sample blog app, statically built.
https://github.com/zawasp/analog-static-issues
Which area/package is the issue in?
content
Description
I deployed the repo at https://analog-static-issues.pages.dev/
The https://analog-static-issues.pages.dev/about Markdown page was properly prerendered statically:

However, the dynamic blog post entries at https://analog-static-issues.pages.dev/blog/post-1 and https://analog-static-issues.pages.dev/blog/post-2 are not:

The app-blog-post component is not rendered.
Please provide the exception or error you saw
The `app-blog-post` NG component appears in the HTML source, instead of its rendered content

Other information
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

This is an upstream Angular issue. Filed a bug here where there is a proposed workaround. Try it and see if it solves your issue
https://github.com/angular/angular/issues/50414

Thanks, will try shortly.

Yea, you also have to downgrade zone.js to 0.12.0 which seems to resolve it

Yeah, that works, thanks!

This has been fixed in https://github.com/angular/angular/pull/50552 and will go out with the next Angular release

This has been resolved with the latest release of Angular


pml document not containing a padv

l submit the PADV with state SUSPENDED however I see no effect in the PML afterwards,
(should it be removed? PADV states that The item shall not be part of the active treatment plan during set to suspended.)
I would have excepted that either it is removed or the PADV item is added to the PML?
Quentin: Ithink the PADV should have been added to the PML (it’s independent of the treatment status, except if you filter by treatment end date) but should have disappeared from the PMLC.
After re-reading the CMPD, PADV items are never included in the generated PML, so it’s working as expected. It’s also quite weird, because the PML is not very useful without.
Quentin: IHE Pharmacy Community Medication Prescription and Dispense specification, <IP_ADDRESS>.<IP_ADDRESS> (FindMedicationList) says:

Find the Community Medication List On-Demand Document (XDSDocumentEntry object) according to the Community Medication List (PML) Profile containing Medication Treatment Plan-, Prescription-, Dispense- and Medication Administration Items for a given patientID and other matching attributes. […] In this case, the Community Pharmacy Manager contains, or has access to, business rules to retrieve an existing list, or to create the patient’s medication list generated from the Community Medication Treatment Plan-, Community Prescription-, Community Dispense- and Community Medication Administration documents of the patient.

No reference to PADV documents here, but when listing possible values for the parameter $XDSDocumentEntryFormatCode (p.87), it’s said:

XXX Items shall be returned (and optional the related Community Pharmaceutical Advice documents related to them).

They’ve maybe forgotten to include it in the query description? I agree that a PML without PADV items isn’t useful.

PADV is added with latest release in the PML.

PADV is added with latest release in the PML.


Flashing error (IDFGH-10523)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
1.79.2 (Universal)
Operating System used.
macOS
How did you build your project?
VS Code IDE
If you are using Windows, please specify command line type.
None
Development Kit.
ESP32 WROOM 32D
Power Supply used.
USB
What is the expected behavior?
It should be able to pass the flashing part
What is the actual behavior?
I tired flashing the esp but i am getting this error
Steps to reproduce.
1.Build
2.Flash
Debug Logs.
A fatal error occurred: Unable to verify flash chip connection (Serial data stream stopped: Possible serial noise or corruption.).

More Information.
No response

Hello @uzunab07,
there seems to be an issue with the flash chip on your devkit. Esptool fails when it interrogates it for the first time.

Is there anything connected to your devkit? If there is, please disconnect everything.
Do you have other hardware to test with? Does the issue persist with other devkits?
Can you try flashing without the stub flasher? (Menuconfig -> Serial flasher config -> Disable download stub)

Closing this for inactivity. Feel free to reopen if needed.


No Pod Deployed on Kubernetes.

K8S Info:
Client Version: v1.26.0
Kustomize Version: v4.5.7
Server Version: v1.26.1
I copied the default values an added a service account name, and added the mistral model.
When I use the helm install command it creates the namespace, deployment, and secrets but no pod, jobs, or persistent volume.
I have GPU disabled as my cluster doesn't have any GPUs right now, but it will soon.
Is there something I'm missing? When I first tried to use it I had to create the namespace first as helm wouldn't do it.
This is also my first time using helm, I created yaml files to do it using kubectl but encountered the same issue, deployment but no active pods.

Hello @Greeley

Helm do not create namespace automatically if it doesn't exist in your cluster, you can pass the --create-namespace flag when you run helm install
For using persistant volume, you need a Persistant Volume Claim, you can set up a provisioner and use storageClass or create manually and use existingClaim, otherwise data will be deleted when container ends.

For the pod, what is the status of the ollama helm deployment (helm status --namespace <namespace> <name>) and the describe of the kubernetes deployment(kubectl describe -n <namespace> <deployment ressource>) ?

where should we be mapping persistent volume claims?

If you have defined manual PVC, you can set the PVC name:

persistentVolume:
# -- Enable persistence using PVC
  enabled: true
# -- If you'd like to bring your own PVC for persisting Ollama state, pass the name of the
  # created + ready PVC here. If set, this Chart will not create the default PVC.
  # Requires server.persistentVolume.enabled: true
  existingClaim: ""

If you have a dynamic provisioner in your cluster, you can just set the provisioner's class or leave blank if one storage class is set as default:

persistentVolume:
# -- Enable persistence using PVC
  enabled: true
# -- Ollama server data Persistent Volume Storage Class
  # If defined, storageClassName: <storageClass>
  # If set to "-", storageClassName: "", which disables dynamic provisioning
  # If undefined (the default) or set to null, no storageClassName spec is
  # set, choosing the default provisioner.  (gp2 on AWS, standard on
  # GKE, AWS & OpenStack)
  storageClass: ""

my question was what directories as we mapping with the PVC.

It's /root/.ollama as in the deployment.yaml:
          volumeMounts:
            - name: ollama-data
              mountPath: /root/.ollama


Allow workflow.startExecution to be called within the orchestrator

workflow.startExecution should issue a Command which starts the workflow and returns the execution Id to the workflow code

Duplicate of https://github.com/functionless/eventual/issues/124


How to draw a graph as beautiful as in README?

Hi, I am trying to find a package that can draw beautiful directed graphs. It seems the DOT files generated by draw package does not contains any predeinfed attributes. Does the images in README is drawed by DOT? What vertex/edge attributes is it using?
 You need the attributes colorscheme, color, fillcolor, and if you want, fontname. Additionally, you need the style attribute set to filled.
You can add these attributes using the VertexAttribute option. A graph like in the example could be created as follows (using the Brewer color scheme):
_ = g.AddVertex(1, graph.VertexAttribute("colorscheme", "blues3"), graph.VertexAttribute("style", "filled"), graph.VertexAttribute("color", "2"), graph.VertexAttribute("fillcolor", "1"));
_ = g.AddVertex(2, graph.VertexAttribute("colorscheme", "greens3"), graph.VertexAttribute("style", "filled"), graph.VertexAttribute("color", "2"), graph.VertexAttribute("fillcolor", "1"));
_ = g.AddVertex(3, graph.VertexAttribute("colorscheme", "purples3"), graph.VertexAttribute("style", "filled"), graph.VertexAttribute("color", "2"), graph.VertexAttribute("fillcolor", "1"));
_ = g.AddVertex(4, graph.VertexAttribute("colorscheme", "ylorbr3"), graph.VertexAttribute("style", "filled"), graph.VertexAttribute("color", "2"), graph.VertexAttribute("fillcolor", "1"));
_ = g.AddVertex(5, graph.VertexAttribute("colorscheme", "reds3"), graph.VertexAttribute("style", "filled"), graph.VertexAttribute("color", "2"), graph.VertexAttribute("fillcolor", "1"));

Rendering this will yield the following graph:

To arrange the nodes like in the example, you can use the osage engine for rendering the DOT file - I think this can be done by passing the -Kosage flag to the dot command. The DOT playground yields the following graph:

Now, all you need to do is to add the width and height vertex attributes, and set the fixedheight attribute to true.

@dominikbraun Thanks for your detailed answer, it is helpful to me. :D


Fix bug where UnrecognizedRtmMessage infinitely put on bus

While testing this with the pagare app, I found that this message would get stuck in an infinite loop on the RxBus because it extended RtmMessage, which the listener is checking for.

While I agree, and I commented on another PR I believe around this same issue with @skynet67 that the pattern I want is for:

the webapp to emit events the SDK doesn't know specifically about, they aren't "unrecognized" as this class name implies... they just aren't of interest to the SDK.

while not of interest to the SDK, the OEM may vary well be interested in them and MUST be able to subscribe to them.

They could do that with this code and listen to this unrecognized message, but that doesn't feel right to me.  And would be confusing to an OEM.

Yeah thats what I was thinking, and I was just asking sara about how our Queues are set up to ensure delivery of these events to the oem

I was thinking the same thing, but I didn't know if it would be more confusing that they would be filtering out all the RtmMessages, even the ones that the SDK is handling. But theoretically they would have the same approach of just listening for what they understand and ignoring what they don't.


HTTP(S) TagDB

This PR adds a TagDB that forwards all tag-related operations to an HTTP(S) server that implements the graphite-web tagging API.
It's designed to be used as a standard way to integrate with outboard tag databases, or with clustered graphite configurations.
I'm not sure what the best way is to go about writing tests for this, since you really need to have 2 graphite installations running for it to work.  Any suggestions appreciated!
I also fixed a minor bug in the remote reader, which wasn't properly formatting the error log entry if the remote returned a non-200 status code.

Codecov Report

Merging #2057 into master will decrease coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2057      +/-   ##
==========================================
- Coverage   71.78%   71.73%   -0.06%
==========================================
  Files          78       78
  Lines        7574     7577       +3
  Branches     1480     1480
==========================================
- Hits         5437     5435       -2
- Misses       1881     1886       +5
  Partials      256      256

Impacted Files
Coverage Δ

webapp/graphite/readers/remote.py
83.33% <100%> (+0.75%)
:arrow_up:

webapp/graphite/settings.py
76.37% <100%> (+0.39%)
:arrow_up:

webapp/graphite/tags/localdatabase.py
87.8% <0%> (-2.93%)
: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 b9a39bb...0480a15. Read the comment docs.

👍 ,  but I'm wondering should we include some documentation for Redis tagdb or HTTP tagdb in local_settings.py.example or documentation maybe?

For testing - yes, we can mock HTTP tagdb separately, but it will be not very easy and not sure if have any sense...

Good call, I'll see about adding some detail to the docs.

I figured out how to test it, will check out docs tomorrow


Pass path to serve-static middleware when using cloudflare module workers

I had a problem using the {path: "..."} option with the serve-static middleware when using cloudflare module workers. This should hopefully fix it.

Hi @raytracer !
Ah, I forgot it. Thank you for your help!
I'll release v2.0.2 including this fix later.


Update default last reading time in CachingDHT11 to be tz aware.

Issue #57

This change is

Coverage increased (+0.03%) to 86.23% when pulling adc444521c18ee940c9853579707b046cb1e0472 on dht11 into d2dd8b8f44b466e2a66762b509ea687a8a07fc31 on master.


File -> Exit should not close all instances

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.84.2
OS Version: Windows 11 22H2

Steps to Reproduce:

Open multiple projects in different VS Code windows/instances.
Close one of them with the Exit option in the File menu.

This causes all open instances to close, not just the one in which you opened the File menu, as users (at least this user) expect.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

close editor: closes the current editor
close folder: closes the currently opened folder
close window: closes the current window
exit: exit the vs code
exit in most all softwares i have used is used to like exit the software for good - which means closing all instances of it.

exit in most all softwares i have used is used to like exit the software for good - which means closing all instances of it.

I haven't done a large enough survey to say "most all software", but the applications I often use where I keep multiple windows open at once like SecureCRT or the SQLite db browser do not behave that way.  File->Exit just closes the instance you are focused on.
IMO any app doing it as you describe is doing it wrong from a UX standpoint, some sort of "exit all" would be a better name; users don't expect things they do in one window to affect any other window, generally speaking.

close window already does what you are saying, what is the point exactly?

close window already does what you are saying,

I know.  Honestly I don't need you to try to explain to me how it works, I do understand.  I just believe it's incorrect.

what is the point exactly?

That "file -> exit" should also do what I'm saying.
You disagree, that's fine.  No need to continue the back and forth that's going nowhere.

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Update solid package and playground to 1.6

Adds gitpod.yml file from #146 (still using gitpod)
Bumps the solid-js peerDependency to ^1.5.0 — #138 makes a fix targeting solid 1.5, which will cause the previous versions to have a wrong behavior
Updates @solid-primitives/props dependency
Updates dev deps
Adjusts tests for solid 1.6
Updates solid playground to latest solid-start

@mattgperry do you mind if we merge this?

I didn't see any issue with merging it myself. Wondering when the next release can go up :)

Attempting to publish but, when I run yarn test with these changes, the process just hangs?

solid runs fine for me
but it crashes on the dom package
also, it might be better to exclude /playgrounds from tests/build
"test": "turbo run test --filter=!./playgrounds/*"

Ah yeah that was a good call, it's all published now thanks!


Tokenizer returns an empty string

When the given text is an empty string, it should return an empty array. Instead, it returns an array with an empty string.
This gives an error when searching if 1) lunr pipeline is off and 2) doc[field.name] is an empty string.
this._fields.forEach(function (field) {
    var fieldTokens = this.pipeline.run(lunr.tokenizer(doc[field.name]))
    console.log(fieldTokens); // when body is empty, it gives [""], not [].

    docTokens[field.name] = fieldTokens
    lunr.SortedSet.prototype.add.apply(allDocumentTokens, fieldTokens)
  }, this)

It looks like that lunr.trimmer contains the logic that removes empty tokens. My guess is that the lunr.trimmer pipeline function gets removed due to non English use cases?
It seems that the tokenizer is probably the right place for this check, since it will always be performed while the trimmer may not.
On a slightly related note, it seems weird now looking at this that the tokenizer and pipeline are different objects. From what I remember they always are used together, in which case having them separate might not be the right split.
If you can put together a PR that moves the check for empty tokens from lunr.trimmer to lunr.tokenizer I think we can fix this for now, and I'll have a think about merging the pipeline and tokenzier.

Doh, I see you already did open a PR will take a look.

I've now pushed a fix for this in the version 0.6.0, please take a look and let me know if you are still seeing issues.


Support using wasm2c modules built with exported memories

It would be nice if rlbox_wasm2c_sandbox supported sandboxing with wasm2c modules that have been built with one or more of the exported memory compile options(i.e. --export-table and --export-memory).
Thread for context : https://github.com/emscripten-core/emscripten/commit/06825ec9e4bbee3ecb6c13111146d61bdc0a522b#r138497857

acknowledged. Investigating how to support this!

Done in #13


Following readme instructions does not display the branch switcher in the tool bar

I think I might be missing a step or not understanding the usage correctly. When I run storybook locally after following the read me instructions no branch switcher appears in the storybook tool bar.
I have two branches master and beta on GitHub
Steps I followed while in beta branch

Installed addon via npm i --save-dev storybook-branch-switcher

Added storybook branch switcher to my main.js

const config = {
  stories: ['../src/**/*.mdx', '../src/**/*.stories.@(js|jsx|ts|tsx)'],
  addons: [
    '@storybook/addon-links',
    '@storybook/addon-essentials',
    '@storybook/addon-interactions',
    '@storybook/addon-toolbars',
    '@storybook/preset-create-react-app',
    'storybook-branch-switcher'
  ],
  framework: {
    name: '@storybook/react-webpack5',
    options: {
      builder: {
        fsCache: true,
        lazyCompilation: true
      }
    }
  },
  docs: {
    autodocs: 'tag'
  }
};
export default config;

Created .branches.json in .storybook directory

{
  "from": "storybook-static",
  "to": "storybook-bundle",
  "default_branch": "master",
  "default_root": true,
  "script_name": "build-storybook"
}

Ran sb-branch-switcher which successfully created storybook-bundle>beta

Ran npm run storybook locally and nothing appeared in the tool bar am I missing something here?

@utarwyn any advice?

Hello @jessepinuelas, sorry for the response delay.
The script extracts branches only via one of the two providers mentioned in the README (Bitbucket PRs or GitHub PRs) otherwise it will take only the current branch. It looks like you can't search for all the existing Git branches.
Is this what you need? Or would you prefer to specify the branches in the configuration file?

@utarwyn When I run storybook locally after following the read me instructions no branch switcher appears in the storybook tool bar.


Cannot add Kepware server application when it is remote from the IoT Edge Runtime

Describe the bug
When Kepware (TKS) is installed on a Windows 10 VM (same network / resource group as IoT Edge RT VM) then "add application" fails.
To Reproduce

Deploy the IIoT 2.6 stack, including an edge with 2.6 modules
Install Kepware (TKS) on a Windows 10 VM in the same resource group
Update the Kepware configuration to expose a discovery URL to the network
Use Postman (or the Swagger client) to invoke AddApplication.
The Kepware TKS server never appears in the list of applications.

Expected behavior
The remote TKS server can be added
Screenshots
This illustrates a connection config using UA Expert:

This shows UA Expert connected using this configuration. UA Expert is running on my host in this example (the TKS server is running on an Azure hosted Windows 10 VM):

Here is the Kepware discovery URL configuration. Note the hostname is not fully qualified:

Here is Postman initiating the AddApplications call using "no security":

Finally, here is the discovery module output:

Hi @pcameron-ptc, would it be possible for you to send us the ARM template that you used to deploy the VM and IoT Edge VM or a description of the network configuration for both VM?   If the IoT Edge VM and cannot resolve the name or the resolved IP is not reachable then this might be happening.   We will see if we can replicate such a setup in our simulation environment.

@pcameron-ptc I managed to replicate the issue and we're working on a fix now. it seems the issue is that the dns resolves the hostname to an ipv6 address and the UA stack cannot properly cope with it. As a workaround til we fix it, you can try using the IPV4 address of the host in the discoveryUrl?

I'm still seeing this issue after verifying that the 2.6.116 version of the modules have been deployed to our IoT Edge Runtime.
This screenshot captures the network configuration of the IoT Edge Runtime system:

This screenshot captures the network configuration of the Windows (TKS) system. Note that it shares a virtual network with the IoT Edge system:

Here is the Iot Edge system successfully connecting to port 49320 on the TKS system using telnet. Note that the DNS name "denali-vm" is resolved to the private IP <IP_ADDRESS>

Here is the TKS system successfully pinging the IoT Edge system:

Here is the configuration of the "add-application" call in Postman

Finally, here are the results from tailing the discovery module logs:

the fix was pushed to master


ImportError: cannot import name 'python_2_unicode_compatible'

I get this error when i am using it with django 3.x version.
File "/home/user/CASTWO/venv/lib64/python3.6/site-packages/mama_cas/models.py", line 13, in
from django.utils.encoding import python_2_unicode_compatible
ImportError: cannot import name 'python_2_unicode_compatible'
Is there any working solutions ? Thankyou

The latest commit solved it. Thanks

The latest commit solved it. Thanks


display images in fullscreen

Allow user to click on a step image to open in full screen

Not sure if you’re already using this package but it has fullscreen mode ability:
https://www.npmjs.com/package/react-image-gallery


feat: test and support for Python 3.13

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

Project coverage is 74.40%. Comparing base (6b39973) to head (d48872a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   74.40%   74.40%
=======================================
  Files          29       29
  Lines         969      969
  Branches      187      187
=======================================
  Hits          721      721
  Misses        201      201
  Partials       47       47

Flag
Coverage Δ

unittests
74.40% <ø> (ø)

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

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


Option to turn on/off eager failure in connecting

If we turn off eager failure in connecting, then there will be no NoHosts error but only Timeout.

This is crucial if the whole zookeeper cluster is only accessible through a single VIP, so we can tolerate node failures in session establishment.


Gradients are always zero during training

I have implemented YOLO algorithm in tensorflow and face with an issue during training: all my gradients are equal to zero from the very beginning of training. I also noticed that with some weights initialization gradients become non-zero at first several iterations and then stick to zero. Thanks in advance for any help!
Environment info
Operating System: Ubuntu 16.04 lts
Tensorflow version: 0.8.0

The issue has been fixed by itself.

Can you explain a bit about how you fixed it ?

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain mazecreator.com has exceeded the max emails per hour (28/25 (112%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext6.iad.github.net ([<IP_ADDRESS>]:51228 helo=github-smtp2b-ext-cp1-prd.iad.github.net)
by server2.lowesthostingrates.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.89)
(envelope-from<EMAIL_ADDRESS>id 1demwa-0005nS-Ff
for<EMAIL_ADDRESS>Mon, 07 Aug 2017 13:40:30 -0500
Date: Mon, 07 Aug 2017 11:50:53 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1502131853;
bh=wnmIVO8laFF0/QYZjd0HsbZDsIKi1MP3qQarBGXHvo4=;
h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=ojwXFYfc7BzmEQlBPlWzcORr6EGUAPGYzbPOJJNqWanb+B7ecIPDoxeGNxUvaW92x
tJ7vYUNoUXaVAPBO/5O8UGQv9JWh6Gveg8bxEkc9wXZFa+XW7twYtIFj23skNjBTuA
rIeZFIvwcnuRkgxYY+dT5/pA7bxCE8LExK99pLNk=
From: samrat1997<EMAIL_ADDRESS>Reply-To: tensorflow/tensorflow<EMAIL_ADDRESS>To: tensorflow/tensorflow<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [tensorflow/tensorflow] Gradients are always zero during training
(#3170)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5988b68d84af9_53e13fb096101c34317762";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: samrat1997
X-GitHub-Recipient: Mazecreator
X-GitHub-Reason: subscribed
List-ID: tensorflow/tensorflow <tensorflow.tensorflow.github.com>
List-Archive: https://github.com/tensorflow/tensorflow
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/ARjzoBlP33jLOO83253dVH28h6LOx53Yks5sV1yNgaJpZM4JD3wR
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-Spam-Status: No, score=
X-Spam-Score:
X-Spam-Bar:
X-Ham-Report:
X-Spam-Flag: NO
----==_mimepart_5988b68d84af9_53e13fb096101c34317762
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Can you explain a bit about how you fixed it ?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/tensorflow/tensorflow/issues/3170#issuecomment-320748207
----==_mimepart_5988b68d84af9_53e13fb096101c34317762
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Can you explain a bit about how you fixed it ?
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5988b68d84af9_53e13fb096101c34317762--


Any examples for numerical features?

I don't find any examples for numerical + category + sequence dataset
maybe you can provide it

This example demonstrate category + sequence + pretrained_embs  with DIN model. https://github.com/openbenchmark/BARS/blob/main/ctr_prediction/benchmarks/DIN/DIN_microvideo1.7m_x1/DIN_microvideo1.7m_x1_tuner_config_06.yaml

We do not have numerical + category + sequence dataset by now for the demo.

thanks, maybe you can make a demo for numerical + category + sequence for many real world datasets

And I think Avazu dataset maybe use time-series split is better than random
split
xpai @.***> 于2023年5月29日周一 11:55写道：

Closed #56 https://github.com/xue-pai/FuxiCTR/issues/56 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/xue-pai/FuxiCTR/issues/56#event-9364894574, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFMF3ZCLLXBWQ7T6GPHXVCDXIQM2LANCNFSM6AAAAAAYO7H7RA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Is it ready for Nuxt?

Added as plugin for Nuxt, got the following error: "document is not defined" in the following line:
var styleElement = document.querySelector('style[data-vue-ssr-id~="' + obj.id + '"]')

Seems like it doesn't work properly in SSR process.

Can you give some more details? I used it with nuxt and it's working.

there is alot of bad practice , can you explain why are you using id inside a component ? and instead of using const you keep using let.

and in the index.js part you are using != and suddenly you used === , I don't see any reason to suddenly use triple equals in this case.

:( It's my fault. I will remove id from component and it will add prop.

Hi,
to make it work in Nuxt with SSR you need to add this plugin as client-side only. Just create a file plugins/vue_confirm_dialog.client.js (the 'client' part is neccessary), put there
import Vue from 'vue'
import VueConfirmDialog from 'vue-confirm-dialog'

Vue.use(VueConfirmDialog)
Vue.component('vue-confirm-dialog', VueConfirmDialog.default)

and add proper entry to nuxt.config.js.
Then in your layout layouts/default.vue you need to wrap the component from the plugin with client-only directive:
<client-only>
  <vue-confirm-dialog />
</client-only>

If you don't wrap the component with client-only you receive an error about the difference between DOM from  the server and DOM in the client side.
Nuxt version: 2.14.6

Hi,
to make it work in Nuxt with SSR you need to add this plugin as client-side only. Just create a file plugins/vue_confirm_dialog.client.js (the 'client' part is neccessary), put there
import Vue from 'vue'
import VueConfirmDialog from 'vue-confirm-dialog'

Vue.use(VueConfirmDialog)
Vue.component('vue-confirm-dialog', VueConfirmDialog.default)

and add proper entry to nuxt.config.js.
Then in your layout layouts/default.vue you need to wrap the component from the plugin with client-only directive:
<client-only>
  <vue-confirm-dialog />
</client-only>

If you don't wrap the component with client-only you receive an error about the difference between DOM from  the server and DOM in the client side.
Nuxt version: 2.14.6


map blinks through scroll view when scrolling

From @1aurabrown on July 9, 2014 22:30
not sure what causes this flash of the map through the scroll view -- pay attention at the very beginning of the gif:

(also the scroll view shouldn't be so long given its content.)
Copied from original issue: artsy/eigen-private#1853

This is easily reproducible, you need enough content to be able to hit a 0 height maps, then scrolling past that flickers to something big, maybe when it's size first hits 0 something happens?

My guess, is that at 0 height it's using the intrinsic content size, then we go back to 1 and minus 1

Almost – precisely -0.5.


[BUG] Docker image not working as expected

Is there an existing issue for this?

[X] I have searched the existing issues

Are you using the latest docker image for BTPSA?

[X] I'm using the latest docker image for BTPSA.

Which area is mainly impacted
Docker Image (Build or Run)
Current Behavior
There is an issue with the docker container run command that is the prebuilt image :
" docker container run --rm -it --name "btp-setup-automator" "ghcr.io/sap-samples/btp-setup-automator:latest" "
The above image is not pointing to the latest updated automator code.
If the docker image is built locally by following the below guide :
"https://github.com/SAP-samples/btp-setup-automator/blob/main/docs/BASIC_SETUP.md#option-3-start-docker-container-with-self-built-image"
then it works fine.
Due to this issue, the mission validation for the following missions :

Mission : Develop a Multitenant Extension Application in SAP BTP, Kyma Runtime
Mission Link : " https://discovery-center.cloud.sap/missiondetail/3683/3726/ " and
Mission : Extend a Kyma Based Multitenant Application With Day 2 Operations
Mission Link : https://discovery-center.cloud.sap/protected/index.html#/missiondetail/3999/4206/?tab=overview

are failing in the initial steps of preparing the development environment.
Expected Behavior
The prebuilt image should be in sync with the latest updated btp-setup-automator repository code.
Steps To Reproduce
No response
Logs and configuration files available?
No response
Anything else?
No response

@yashika-Gulati : What exactly is missing in the pre-built image or what error messages occur when executing the image?
Be aware that the image tagged with lates is referring to the latest stable release built upon the main branch of this repo.
The dev branch might have additions that are not yet part of the latest stable image.
Can you check if the error also occurs in case of using the dev image via
docker container run --rm -it --name "btp-setup-automator" "ghcr.io/sap-samples/btp-setup-automator-dev:dev"

I will check it and get back if it is working or not.
Thank You

Hello @lechnerc77,
So I tried following the steps again for my mission using the latest image, but I was getting the following error:
During the authentication step, I visit the url : http://localhost:18000 manually, and this url is not working and the local host refused to connect. Here are the screenshot of the errrors faced:

Hi Yashika,
this is not an error of the BTP Setup Automator, but is an issue/limitation with the handling of the logon flow for Kyma. This causes issues wrt to the callback of the OIDC flow. You probably need a service user in Kyma to execute the login flow in your setup as described here
Best, Christian

@lechnerc77
It is working and the main branch is in sync with the latest update.
Thank You


Automated cherry pick of #9891: Repair of abnormal login notification

Cherry pick of #9891 on release/3.7.
#9891: Repair of abnormal login notification

Codecov Report

Merging #9892 (b31f699) into release/3.7 (0ff3997) will increase coverage by 2.07%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           release/3.7   #9892       +/-   ##
===============================================
+ Coverage         5.98%   8.05%    +2.07%
===============================================
  Files              848     107      -741
  Lines           131795   12991   -118804
===============================================
- Hits              7887    1047     -6840
+ Misses          123206   11824   -111382
+ Partials           702     120      -582

Flag
Coverage Δ

aFlag
8.05% <0.00%> (+2.07%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/auth_totp.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/idp.go
1.84% <0.00%> (ø)

pkg/apigateway/handler/misc.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/resource.go
0.00% <0.00%> (ø)

pkg/hostman/diskutils/nbd/driver.go
0.74% <0.00%> (-0.01%)
:arrow_down:

pkg/hostman/diskutils/nbd/lvmutils.go
4.70% <0.00%> (-0.24%)
:arrow_down:

pkg/monitor/alerting/eval_context.go
34.25% <0.00%> (-2.75%)
:arrow_down:

pkg/util/version/version.go

pkg/hostman/monitor/monitor.go

... and 702 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #9892 (b31f699) into release/3.7 (0ff3997) will increase coverage by 2.07%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           release/3.7   #9892       +/-   ##
===============================================
+ Coverage         5.98%   8.05%    +2.07%
===============================================
  Files              848     107      -741
  Lines           131795   12991   -118804
===============================================
- Hits              7887    1047     -6840
+ Misses          123206   11824   -111382
+ Partials           702     120      -582

Flag
Coverage Δ

aFlag
8.05% <0.00%> (+2.07%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/auth_totp.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/idp.go
1.84% <0.00%> (ø)

pkg/apigateway/handler/misc.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/resource.go
0.00% <0.00%> (ø)

pkg/hostman/diskutils/nbd/driver.go
0.74% <0.00%> (-0.01%)
:arrow_down:

pkg/hostman/diskutils/nbd/lvmutils.go
4.70% <0.00%> (-0.24%)
:arrow_down:

pkg/monitor/alerting/eval_context.go
34.25% <0.00%> (-2.75%)
:arrow_down:

pkg/util/version/version.go

pkg/hostman/monitor/monitor.go

... and 702 more

Continue to review full report at Codecov.

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


Include complete copyright and rights in data information

Each source tree that might plausibly be downloaded independently from the entire VE tree should contain a COPYRIGHT file that clearly asserts ownership and rights in data conferred under contract, and acknowledges earlier copyrighted material used under license in developing the current material, and also acknowledges "rights in data" clauses for work performed under contract (i.e. if the contract supports it, the work may be copyrighted by the developer, but will explicitly establish rights to distribution and use granted to the contracting agency).
A LICENSE file should exist that explicitly states the license under which the code is released, and that demonstrates compatibility with the licenses for all incorporated work.  Consistency with the COPYRIGHT file should be verified.
In addition, each source file should contain a comment header at the beginning of the file recapitulating the copyright information applicable to that file (reference to the external COPYRIGHT file is acceptable).  Such a header should be included in any file that supports internal comments, but not .csv files or the R package DESCRIPTION which is itself one long comment.

Any suggested examples? One thing that confused me is how to distinguish what copyright goes with what code; or if that's even necessary. All of the original GreenSTEP  and RSPM code were copyrighted by ODOT. The original visioneval code was also developed under contract with ODOT. Substantial changes were made to the visioneval code under the current contract, but I don't have any enumeration of what code is original and what is new. Likewise with the RSPM code changes to fit the new framework. Another complication is that I make changes to various parts of RSPM under different contracts or pro bono work. Some of these are smaller fixes and some add functionality. Is it necessary in the copyright notice to identify what the copyright covers? Is there some shorthand way to acknowledge all the copyright holders?

See examples in the VESimHouseholds module. Copyright notice contained at the top of the CreateHouseholds.R script. Copyright information added to package NOTICE file. Copyright in package DESCRIPTION file as well.

The DESCRIPTION file did not pass package checks. Removed 'Authors@R' field and added 'Author', 'Maintainer', and 'Copyright' fields. Edited NAMESPACE file to add visioneval import.

I think we need to write “Copyright contributing authors” at the top of each file since it’s impossible to identify owners of specific resources.

We decided:

License and copyright will be in the DESCRIPTION file.  There needs to be a LICENSE file as well.
We will remove it from source files if present.
We will add a DESCRIPTION file to VEGUI.
We will set the copyright as AASHTO for now.
We will request that we change the copyright to "Contributing Authors".

I updated this page as well - https://github.com/gregorbj/VisionEval/wiki/Developer-Orientation#understand-licensing


Adding support for custom rule in EachValidator

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
should

Fixed issues
#18051

This changes a bit how the InlineValidator is created in case of model-method-based inline validators. Instead of just passing a method's name now there is an array with an instance of Model in question and the method's name. This works thanks to the fact that call_user_func can take such array as first parameter so there should be no BC change.
Since EachValidator creates a new instance of DynamicModel internally this change allows inline validators defined in the original model to be used.
I haven't checked all possible drawbacks here. Definitely now method that is inline validator cannot have limited visibility but I haven't checked if it could be the case before this change.
Please take a look and see if this can be the solution to the issue. If so, I will add changelog.

I haven't checked all possible drawbacks here. Definitely now method that is inline validator cannot have limited visibility but I haven't checked if it could be the case before this change.

I think it was allowed before.

As I can see the validator is doing the same thing when $type is just a string - it uses [$model, $type] for call_user_func but then the $model is the original one. So I think limited visibility methods were no good previously as well.

Hm, I also thought it always worked

Added :heavy_check_mark:

👍