Android - Accept request and choose ID button in sharing screen should be changed to Accept request and share

Describe the bug
Accept request and choose ID button in sharing screen should be changed to Accept request and share
prerequisite: Vc is downloaded and stored in the app
Device A  : Requesting
Device B  : Sharing
To Reproduce

unlock the app
Open qrcode in Device A
Open scanner in Device B
scan qrcode from Device B
Select VC in sharing screen

Current behavior
Select VC and see that "Accept request and choose ID " button is enabled . As we VC is selected already the word Choose ID in the button is not giving the right meaning rather it is making user to assume that VC should be selected in Next step which we selected already.
Expected behavior
Accept request and choose ID button in sharing screen should be changed to Accept request and share
Screenshots

Smartphone :

Device: PIXEL 6
OS: Android 13
MOSIP Version: <IP_ADDRESS>
MOSIP server: qa5.mosip.net
APP link; https://mosip.atlassian.net/browse/DSD-1816

This issue is not valid as new label changes are requested and picked up and merged as part of 28-02-2023 release in issue #589 - https://github.com/mosip/inji/issues/589.
The new changes are,
--> "Accept request and choose ID" to "Share".
--> "Accept request and verify" to "Share with Selfie".

@santhoshsunder - Re-assigning it to you, as this defect isn't applicable you are free to close.

The issue is not applicable in the new build, the button has been removed, hence closing the bug


Handle a scenario where a user tries to remove a trackable that's still being added

There can be a situation in which a trackable is being added and a moment later it is being removed. That's because both add() and remove() operations are async ones. What happens now is not predictable as the result depends on whether the trackable adding operation was completed before the removal began. What should happen is that the trackable should always be removed if someone called the remove() method. So if the trackable that's requested to be removed is currently in the add process we should stop that process, revert all operations that were performed until now and then notify that the removal was successful.
On Android, we have a TrackableRemovalGuard that's used to mark trackables for removal if they're still during the add process. Then each step of the add process checks if the trackable was marked for removal and if it was they revert the add process.

This should be done alongside #341.

@KacperKluka What should be the result of the in-progress add operation? Should it fail?

Yes, the in-progress add operations should fail with a special exception (here's how it works on Android).

The corresponding Jira issue is https://ably.atlassian.net/browse/SDK-2594


GCP PS1 Prompt Support for GCP Accounts and Projects

Feature Request
I know that Geodesic tends to lean more into the AWS landscape, but would y’all consider offering some GCP PS1 prompt support for GCP accounts like:
function role_name() {
    local role_name
    # Could be a primary or assumed role. If we have assumed a role, cut off the session name.
    local current_role=$(gcloud auth list --filter=status:ACTIVE --format="value(account)" 2>/dev/null | cut -d '@' -f1)
    if [[ -z $current_role ]]; then
        unset ASSUME_ROLE
        return 0
    fi
    export ASSUME_ROLE="$current_role"
}
role_name

From my understanding, GCP has a few different authentication flows, but when it comes to the credentials used by SDK tools like gcloud or gsutil, they are stored in a DB file located at ~/.config/gcloud/credentials.db. Whenever someone runs gcloud auth login, it appends an entry to this file. I also found credentials inside of access_tokens.db, and was able to extrapolate them using Sqlite.
When I look in ~/.config/gcloud/configurations/config_default :
I see:
[core]
account = [michael<EMAIL_ADDRESS>project = projectexample

I would be in favor of supporting GCP prompts along the same lines as we support AWS prompts. Standing in the way:

Cloud Posse does not have a sponsor to support the work. This is not always a requirement, but it definitely helps, both in providing resources and in providing feedback on proposed solutions.
Cloud Posse does not have a GCP workflow or authentication scheme in its reference architecture, making it unclear what GCP support should actually look like. The AWS support is quite complicated because of the various sources of identity and the numerous reasons to assume different AWS IAM roles and the differing ways to assume them. It is extra complicated because of legacy support for deprecated and obsolete workflows, which should not be carried forward to GCP support.
The existing prompt support for AWS roles is very complicated. I would not expect a contributor to understand all the intricacies and would not trust a contributor PR to provide GCP support.
Cloud Posse does not have any kind of established way to test the prompts. The AWS prompts were developed through continuous feedback from customers, but we do not have established relationships with GCP users to provide adequate feedback.

I will leave this issue open and we may at some point undertake to provide the feature, but I would not estimate it happening anytime soon. My best recommendation for now is that GCP users look at the existing AWS support, ignoring in particular the support for aws-vault and the expectation that a primary use case is launching a sub-shell with a different assumed role, and use the existing customization options to craft your own solution.
If you feel you have something robust enough to share, I will consider a PR, but you should craft your PR in a way that other GCP users can adopt it into their custom Geodesic images before the PR is merged and without having to fork the repo.


Extending space and org defaults to include all options

Issue
When doing  export-config or add-org-to-config or add-space-to-config it would be useful to be able to have defaults set for fields such as "enable-remove-users" for every newly created orgConfig.yml or spaceConfig.yml. At the moment we use a custom bash script to check for these settings and change them (we almost always want them set to false).
Proposed Change
Allow orgs.yml (or a new orgDefaults.yml) and spaceDefaults.yml to act as templates for each, have fields to set the defaults for all newly created orgConfig.yml and spaceConfig.yml files.
For orgConfig.yml this would include fields :

enable-remove-users
enable-remove-private-domains
enable-delete-spaces
memory-limit
instance-memory-limit
total-routes
total-services
paid-service-plans-allowed
total_private_domains

For spaceConfig.yml this would include fields:

allow-ssh
enable-space-quota
memory-limit
instance-memory-limit
total-routes
total-services
paid-service-plans-allowed
enable-security-group
enable-remove-users
total_private_domains
total_reserved_route_ports
total_service_keys
app_instance_limit

Sounds like a beneficial feature.  Will look at scoping this in soon.

We sure are interested in this one too !

@srothwell01 @metskem Take a look at the new cf-mgmt-config cli.  Just released the ability to set all values on creation of org or space.
Does this give you the feature/capability that you would want?
The challenge with spaceDefaults is those are used at runtime currently not at creation of the configuration files.
If not, let me know and can see how we can still work this feature to allow configuration of the defaults when you first add a space or org.

Thanks for the update.
This new command with all new flags would allow for generating orgConfig.yml and spaceConfig.yml with the requested default settings. So I guess this would be beneficial if you use this command as your primary interface.
Our current way of working is using the cf-mgmt command to generate the org/space structure and then update the spaceConfig.yml manually (or copy mostly from an existing one).
So in our case it could be slightly beneficial, it would help us more if the cf-mgmt-config command would take the value for each option that is omitted from the spaceDefaults.yml file :-) .

Thanks for the response Caleb.
This works a bit nicer for us. From a scripting point of view it's much nicer to be able to do this than using a script to go and sed the files or doing it manually on creation of the default.

Ok... I'm going to go ahead and close this specific issue.  Will add something to backlog for config defaults (other than the current spaceDefaults) so that you can setup defaults for org/space when a create-org/space is executed it will take the defaults if you don't specify on the command line that option.


Are you seriously recommending Wingdings for more icons?!?

> You don't have to use the Segoe MDL2 Assets font--you can use any font the user has installed on their system, such as Wingdings or Webdings.
Cmon. You guys just previously said
> Don't use an icon if its meaning isn't clear, or if making it clear requires a complex shape.
Conflicting logic in the documentation

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

ID: 090a5bbe-1b89-c80b-4ad8-a932d38246ab
Version Independent ID: 5166ef46-fb83-98fb-b852-f16c5ac3ef8e
Content: Icons - UWP app developer
Content Source: windows-apps-src/design/style/icons.md
Product: windows
GitHub Login: @mijacobs
Microsoft Alias: mijacobs

You don't have to use these fonts if you don't want to, this is just guidance.


memory profiling feature request

I've noticed my app with crystal 0.32.1 seems to "monotonically increase" in the amount of RAM it uses over time.  I can use valgrind to find "real C level memory leaks" but if the source is in crystal land (something hanging onto too many references) it's difficult to detect the culprit.
I wish there were a way to have an object graph showing relationships between objects and which ones are holding onto other objects.  So similar to java jhat's "browse the memory space" functionality (which shows classes, references).
Or, barring that, a count of each class instantiated so I could narrow down the search.  So similar to jhat's "histogram" of allocated classes, or ruby's ObjectSpace.count_objects.
I realize this might require a new GC or what not, but eventually it would be really useful for debugging, though possibly not the end all for my particular problem.  Thanks.

Somewhat related: #5271

Yes, I'd love this so much right now. related. I saw Benchmark has a memory method, but it doesn't quite give enough information when your app starts to just run out of memory. And no one on my team knows C, so looking at something like valgrind becomes quite the task. I think starting with something super simple that just says "You instantiated X amount of objects for a total of 29203kb" would be a great start. Then we could throw that in to web frameworks for debugging to see if anything is leaking, or getting out of control with memory usage.

See also #3997 somewhat related

This will require either a new GC option (naive mark and sweep perhaps), or some significant improvements to bdw, I don't see much of an option there for it at present...

I wonder if one low ball way to get this would be "in debug mode" count how many of each type of object are currently allocated (decrement some counter in a finalizer).  Just a thought...

Yeah, I think we just need an initial anything. We don't need to start out with everything under the sun. Just something super basic to get people started, and build on top over time. That would be great for all these frameworks out there to see something like "oh, we're allocating 4 copies of this, when we should only be doing it once" type of stuff.

bdw has some options, for the adventurous: https://github.com/ivmai/bdwgc/blob/master/doc/debugging.md gc_dump, and maybe https://github.com/ivmai/bdwgc/blob/de8e2e986039495ed4416dcb61fe4fb34026a2dc/doc/porting.md gc_save_callers

I wish there were a way to have an object graph showing relationships between objects and which ones are holding onto other objects. So similar to java jhat's "browse the memory space" functionality (which shows classes, references).

crystal-lang/perf-tools#5 will probably achieve this.

Or, barring that, a count of each class instantiated so I could narrow down the search. So similar to jhat's "histogram" of allocated classes, or ruby's ObjectSpace.count_objects.

This should now be possible with PerfTools::MemProf.log_object_counts from that repo.


Dependency Injection

Ready for review

@rynowak @davidfowl @Eilon @dougbu Please review

@ardalis can you send me this is a word doc?

@ardalis  a word doc pulls in the source and doesn't require me to compile the source - I'll add comments here - added bonus, you can run spell check

@Rick-Anderson I'm not sure what you mean by "pulls in the source". And yes, it would catch a typo here or there, but so does our review process without the need for migrating the document to/from another tool. If word has native support for reStructuredText that's news to me...

@ardalis The built output is much easier to read as it pulls in the referenced source file lines. We shouldn't depend on reviews to do the job of a tool (spell check). It's not uncommon to request a word doc (or even a html file) - feel free to give me an HTML file, I just want to see the processed build. To get a word doc copy/paste the built HTML file.

@dougbu I can trim out some parts of the template that aren't needed for the demo. If you have feedback on the template itself, I would direct it here:
https://github.com/aspnet/templates

I can trim out some parts of the template that aren't needed for the demo.

Sounds good to me.

:watch:

Great offline suggestion from @lodejard that I'm likely misquoting: Reorder the progression here to start with the general example -- add a repository and use it in your controller. Then move to minutia of Startup class. Finally cover the lower-level service lifetimes example.

We get questions sometimes about depending on a service that may or may not be present. This article should mention IEnumerable<TService> constructor parameters as a solution for that case. (Don't mention services that are actually registered multiple times, that's rarely of interest outside framework components. Even those creating custom services of these types shouldn't care.)

:watch: and suggest @davidfowl or @lodejard comment on the overall positioning.

Hopefully good to go now. Thanks, @dougbu. Ship it ?

:shipit:

37bef203d17fb4d82ae1d39bd2700d94a3455748


--match-filter Privated, Deleted or Premium videos?

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Using yt-dlp version 2021.08.10 (zip).
I have a list of youtube URLs. I only need to get the IDs of videos that are either Private or Deleted or Premium.
I know that in order to only get unlisted URLs from the list I could use
yt-dlp --match-filter "availability = 'unlisted'" --batch-file list_of_youtube_links.txt --get-id

What should the --match-filter be to only get the ids of

Privated,
Deleted,
Premium videos?

I'm especially looking for a way to only --get-id of URLs that give this error:
ERROR: Video unavailable. This video is no longer available because the YouTube account associated with this video has been terminated.

Thanks!

There is no single-step way to do this since we cannot extract any metadata from unavailable videos.
You will need to first run yt-dlp --download-archive archive -s --force-write-archive -a batchfile. Now all videos that can be downloaded will be recorded in the archive. Then you can compare it against your batchfile to get the videos that aren't downloadable

@pukkandan commented on Aug 14, 2021, 6:07 PM UTC:

You will need to first run yt-dlp --download-archive archive -s --force-write-archive -a batchfile. Now all videos that can be downloaded will be recorded in the archive. Then you can compare it against your batchfile to get the videos that aren't downloadable

It's a little tedious but with sort | uniq -u that works! Thanks!

There is no single-step way to do this since we cannot extract any metadata from unavailable videos.

That is unfortunate, since I have ~10-20K videos saved, and am just trying to ascertain which ones among those have been removed from public youtube, so it would be great if there were also an --archive-errors errored-ids.txt that saves errored IDs in addition to good ones.

...so it would be great if there were also an --archive-errors errored-ids.txt that saves errored IDs in addition to good ones.

I have had a feature like this in mind as well, but for a different reason. I regularly re-crawl large playlists where most of the videos are already present in my download archive. yt-dlp quickly filters these out by pre-checking the archive, but that leaves mostly deleted videos which yt-dlp attempts to extract every time. With enough such videos, this becomes a significant bottleneck.
With some types of errors, we can be pretty certain that a video won't become available again. By keeping track of these in a separate "error archive", yt-dlp can save the time it takes to rediscover that the same videos have been deleted every time. Determining which errors should be considered permanent and thus suitable for the archive vs user-specific (age-gated) or temporary (private videos which could potentially be made public again) might require dividing them into categories as described in https://github.com/yt-dlp/yt-dlp/issues/457, though.

I've also wanted this, but can't figure out how to implement it 😅
The exceptions are thrown from all over the place and by the time they are caught, it is no longer known what the  URL/video_id that threw the error is. I can probably make a dirty hack by saving the last processed URL in a class variable, but this is not a good practice and definitely won't be easy to maintain in the future

From https://github.com/yt-dlp/yt-dlp/commit/1151c4079a15939c981f3e30129b82dea1b1bc6d, the error message will show the extractor and the video id. I know it isn't exactly what you requested, but this can help you find errored videos much easier than before
$ yt-dlp yZIXLfi8CZQ qEJwOuvDf7I -q
ERROR: [youtube] yZIXLfi8CZQ: Private video. Sign in if you've been granted access to this video
ERROR: [youtube] qEJwOuvDf7I: This live stream recording is not available.

Note that for extractors other than youtube, the ID shown may not be the actual id (that is used in archive). This happens since we may not know the actual id when the extraction fails

btw, please close this issue if your original question has been fully answered.

it would be great if there were also an --archive-errors errored-ids.txt that saves errored IDs in addition to good ones.

if you still want this, open a feature request

Closed due to inactivity


Added count.posts and count.members to the newsletter resource

refs https://github.com/TryGhost/Team/issues/1524

Follows the count.x convention used on other resources
This powers the ghost admin so that the admins can get a quick overview of the newsletter total members and posts.

Note: for now I wasn't able to see the count.x properties in the newsletter resource, there must be a missing piece.

Codecov Report

Merging #63 (a30a11f) into main (9799b44) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #63   +/-   ##
=======================================
  Coverage   89.92%   89.92%
=======================================
  Files          46       46
  Lines        2919     2919
  Branches      406      406
=======================================
  Hits         2625     2625
  Misses        290      290
  Partials        4        4

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 9799b44...a30a11f. Read the comment docs.


Use host in windows mount script

Resolves #3518
What is being addressed
The mount script fails to connect to shared storage.
How is this addressed

Use the Host instead of the Endpoint when calling the mount command as it expects a FQDN and not a URL.
Upgrade versions (Terraform and providers)
Use the native templatefile function instead of the template_file provider

/test-extended

/test-extended

/test-destroy-env


Harmless "attempt to register" severe alert is thrown during load when DontUseAsyncChat is enabled (Tekkit Classic 1.2.5)

When loading with DontUseAsyncChat the plugin throws the following.  It appears to be harmless.  It might not be possible to make the alert disappear.  As it stands it doesn't ignore the events at all as the plugin still functions as expected when this alert shows up.
[SEVERE] Plugin FactionChat is attempting to register event org/bukkit/event/player/AsyncPlayerChatEvent, which does not exist. Ignoring events registered in class nz.co.lolnet.james137137.FactionChat.FactionChatListener

Closing issue


Nuxt.js hot reloading doesn't work.

"create-nuxt-app" project won't hot-reload.  Looked at these: this, this, this.
Clean nuxt app, nothing added, pc is newly formatted, no other packages or any external code other than pure create-nuxt-app.
Tried:

Try different browsers
Change directory
Update npm packages
Install WSL 2 with ubuntu(Just in case
Make a new project without changing anything.
Try ssr, static, server, content, jamstack etc options while creating the app

Versions

nuxt:  2.14.12
npm: 6.14.10
node: 14.15.4
windows 10: 10.0.19042 Build 19042
vscode: 1.53.0

Reproduction
No need for reproduction as you only need pure nuxt project without any code or extensions.
Steps to reproduce
npx create-nuxt-app
npm run start
What is Expected?
Need to reload automatically.
What is actually happening?
Won't reload. No errors or anything.

Couldn't find what was the problem but I reinstalled Node.js deleted AppData/Roaming/npm&npmcache.
Would still love to know the problem.

@danielroe Thanks a lot for the reply. Is it written in the official docs? If not it should be. I didn't know this.

You can find out more at https://nuxtjs.org/docs/2.x/get-started/installation and https://nuxtjs.org/docs/2.x/get-started/commands
I hope this helps.

Entirely my bad. I didn't pay enough attention. Thanks a lot!


Blocking Mode for pattern

Urgent review!

Variable Names
Localization Names
Method Description
Checkbox position (Grid checkbox is awfully placed!)

Updated with a new approach. I'm have encountered a bug, but I don't know how to fix or what I have done wrong. When I restart the client the system doesn't recognize the pattern.
Picture of the result:

PD: I don't think the pattern grid blocking feature is needed.

Yeah, I agree. Pattern grid doesn't need it I think. So, 2 questions:

What do you mean with "When I restart the client, the system doesn't recognize the pattern."
Does it work if a pattern that isn't blocked needs an item that IS blocked. Does that work correctly in the chain?


Multiple queries?

How can I make multiple queries to the Github API? I have a simple query that I want to repeat for two separate orgs and merge the results. Not sure how I can do it with this plugin.
I have (naïvely) tried to repeat the plugin usage but that will not work. Here is my code:
https://github.com/lyft/lyft.github.io/pull/20

Never mind, found a GraphQL way of fetching multiple orgs with Github GraphQL API:
https://github.com/lyft/lyft.github.io/pull/20/commits/a32e5658901fcf4c89daabf9185faeac9ac95804


RECOVER BTC AND MONEY SENT TO SCAMMERS HIRE A HACKER TECHNOCRATE RECOVERY

When my cryptocurrency wallet was suddenly emptied, leaving me with a loss of $145,000, I was engulfed in shock and disbelief. The devastation was compounded when I uncovered the truth: my ex-girlfriend, whom I had recently discovered was unfaithful, was behind the theft. The realization that someone I once trusted had betrayed me so deeply was crushing. The emotional turmoil left me desperate for a solution, so I began searching for recovery services. During this search, I came across a Telegram group where someone recommended TECHNOCRATE RECOVERY. Their glowing reviews provided a glimmer of hope, and reaching out to them turned out to be a pivotal decision. From our very first conversation, TECHNOCRATE RECOVERY exhibited a level of professionalism that immediately reassured me. They didn’t just sympathize—they acted, quickly launching an investigation into how the theft occurred. Knowing that this betrayal came from someone close to me, their careful, methodical approach was exactly what I needed. They were meticulous in their investigation, tailoring their efforts to the unique aspects of my case, which was vital given the complexities of cryptocurrency recovery. Their team was both knowledgeable and empathetic, fully understanding the emotional weight of my situation. Throughout the process, I experienced an emotional roller coaster. The initial despair slowly gave way to hope as the team at TECHNOCRATE RECOVERY consistently communicated with me. They were transparent, explaining each step in a way that was easy to understand, which helped alleviate some of the anxiety I felt. Their regular updates were a lifeline, ensuring I was never left in the dark about the progress being made. Their ability to break down complex technical details into something manageable was a testament to their commitment to client care. TECHNOCRATE RECOVERY’s use of advanced forensic tools and blockchain expertise was nothing short of impressive. They tracked the movement of my stolen assets with precision, leaving no avenue unexplored. I was particularly impressed by their collaboration with other experts and law enforcement when necessary, demonstrating their dedication to recovering my funds. Their transparent fee structure was another standout feature. While many services charge hefty fees or demand significant upfront payments, TECHNOCRATE RECOVERY provided a clear breakdown of costs from the outset. This transparency built trust, making the financial aspect of the process far less stressful. Their focus on results over profit was evident and greatly appreciated. Ultimately, TECHNOCRATE RECOVERY successfully recovered all my stolen assets, bringing a sense of closure to this painful chapter. But what truly set them apart was their post-recovery support. They provided invaluable advice on securing my cryptocurrency wallet against future threats, arming me with the knowledge I needed to prevent such a breach from happening again. If you ever find yourself in a similar situation, know that there’s a way back from the brink. TECHNOCRATE RECOVERY isn’t just a service—they’re a lifeline, guiding you through the darkest moments and helping you regain control over what you thought was lost forever.
Web Info: ww w.technocraterecove ry.site
Contact Email: technocratrecovery (@) contractor. net

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

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

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

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

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

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

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

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

to fix your trouble try download this fix, i see it in another issue,
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/6mkbh6xhau31g
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/6mkbh6xhau31g
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/6mkbh6xhau31g
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/6mkbh6xhau31g
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

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

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


Introduce new field: detectedTypes.

Added a few comments, looks fine to me. Though the name scopes is not that descriptive -- I'd prefer searchTerms or something similar.

@isoos
Is this CL still waiting for something or should it be merged}
About the original TBDs:

the field's name (scope)

Would prefer a more descriptive name (tags, searchTerms, ...) - though since I don't have a strong opinion, you can stick with scope.

the values (e.g. flutter vs flutter_plugin)

It's just a marker name, right? So we can use whatever we want. I'd go for flutter_plugin if it is a plugin.

whether the Package instance should also have this field (e.g. whether we will need it for querying the Datastore)

It would be preferred to not have duplication if it is not necessary: If all places where we query the package (e.g. for rendering the package page template) we also query the latest version, then it should be enough to have it on the latest version I think. It's always possible to add it later, if need be.

This is still pending, I'm waiting on @kevmoo's feedback on the search choice.
e.g. if we go ahead with using the current custom search API with additional attributes, than "listing the flutter plugins" page will need to have the duplicated values on the Package object, otherwise we may not need it there.

Updated the value (flutter -> flutter_plugin).

@mkustermann please do another round of review, this version should be ready to merge.
I've renamed scope to detectedTypes, and tried to be keep the other parts consistent with it.(I'm open for naming suggestions thought.)
Talked with Kevin, and we concluded that we start with the custom search API, and for the "list of flutter plugins" page we'll need that field on the Package object too.

@mkustermann cool w/ this change?

Looks fine with me.


Update kafka to 2.4.0

Updates org.apache.kafka:kafka from 2.3.1 to 2.4.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.kafka", artifactId = "kafka" } ]

labels: semver-minor

superseded by #126


[SPARK-24478][SQL][followup] Move projection and filter push down to physical conversion

What changes were proposed in this pull request?
This is a followup of https://github.com/apache/spark/pull/21503, to completely move operator pushdown to the planner rule.
The code are mostly from https://github.com/apache/spark/pull/21319
How was this patch tested?
existing tests

cc @rdblue @gengliangwang

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/168/
Test PASSed.

Merged build finished. Test PASSed.

Test build #91885 has started for PR 21574 at commit 1aeb6ab.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4059/
Test PASSed.

Test build #91885 has finished for PR 21574 at commit 1aeb6ab.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91885/
Test FAILed.

retest this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/209/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4100/
Test PASSed.

Test build #91937 has started for PR 21574 at commit 1aeb6ab.

Does this move actually make sense? It'd destroy stats estimation for partition pruning.

According to the dicsussion , we agree that it's better to remove the hacky workaround for mutable DataSourceV2Relation and not having stats estimation for partition pruning. For builtin data source like file source, we can still keep the optimzer rule PruneFileSourcePartitions to make them work.

@rxin, we can also add the second pushdown (in the stats visitor) to get better stats with a property to turn it on or off. We're going to add it back in our branch anyway.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/286/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4180/
Test PASSed.

Test build #92041 has started for PR 21574 at commit 13c91df.

+1 (non-binding) assuming that tests pass.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/294/
Test PASSed.

Test build #92041 has finished for PR 21574 at commit 13c91df.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92041/
Test PASSed.

Test build #92049 has finished for PR 21574 at commit 4471422.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92049/
Test PASSed.

thanks, merging to master!


Improve stub templates

Right now, the're essentially empty, even though the use case is consistently the same. Some error-prone* typing could be spared.
'* forgetting about [Inject] above constructor is a classic.

It could be a good case to actually select, which parts of stub user wants generated (constructor? some method override?) as part of #16 . For now, this could be provided using attributes.


Drag & drop with multiple folders, some of which are compact folders, always moves the leaf folders of the compact folders

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

VS Code Version: 1.57.0-insider fe1eafc80c07608de52a2f9c99fee103dea77f5e
OS Version: Windows_NT x64 10.0.19043

Steps to Reproduce:

create a folder structure as below
open/close folders, such that projects shows sub1, sub2\subA, sub3
select sub1 till sub3 and drag & drop them into `new-projects

Folder structure:

new-projects
projects

sub1

foo.txt

sub2

subA

bar.txt

sub3

baz.txt

Expected: the whole subtree of projects is moved into new-projects (note that this is the behavior when doing Cut/Paste instead of using drag & drop)
Actual: instead of moving sub2, its subdirectory subA is moved
Demo:

So you would like the drag and drop of a compact folder to drag the upmost parent..

Imho this is a bug, not a feature request, because when I want to move everything in projects into new-projects using DnD, the intuitive course of action (click first folder, Shift+click last folder, do DnD) has an unintuitive outcome. The presence of a compact folder is an accidental detail that shouldn't influence what's moved (even more so because I didn't explicitly select neither sub2 nor subA).
In fact, there's no way to do what I want with DnD at all because, as the demo below shows, even if I use Ctrl+click to explicitly select foo, DnD still attempts to move bar. So DnD doesn't honor the folders that were selected by the user & just copies the leaf folder no matter what.

@isidorn please consider classifying this as a bug. Imho the mere fact that DnD is inconsistent with cut/paste is sufficient argumentation to do so


[Task]: Setup cloud monitoring and email alerts for ToB

What needs to be done
Setup monitoring and email alerts for ToB endpoints which cover:

bad http codes
internal service errors: panics, cold restarts

Consider implementing as terraform script
Issue Priority
Priority: 3
Issue Component
Component: tour-of-beam

.take-issue


Improve / implement a UI to switch between inputs once in VR mode.

I think we need a UI in the boilerplate to pick between VRDevices. Related: https://github.com/borismus/webvr-polyfill/issues/4

I took a stroll through the code with this issue in mind.
Just to clarify, the idea is to enable a user to switch between all supported inputs (mouse/kb, touch, hmd) including when they have multiple hmds connected, right?
Some thoughts on the UX/Design side:

When should an input menu be displayed?

if no hmd (ex: keyboard/touch), don't display any menu; enter immersive on any action.
if 1 hmd and canvas selected, take user straight into VR.
if 1 hmd and WebVRButton selected open input select menu
if N hmds and WebVRButton selected display menu.
if N hmds and canvas selected... display flyout menu?
Have a separate button entirely for selecting inputs.

Should a user be able to switch inputs once in VR?

I'd say probably not. User can only exit Immersive/VR mode. Inputs are only selected via touch or mouse/keyboard.

Basic work items seem something like:

Detect and return multiple hmds.

Requires updating VREffect and a fair amount of WebVRManager.

Implement an input select menu (showing/hiding/animating/etc).
Launch Immersive or VR Mode based on input selected.
Bake in some sane defaults to avoid showing an input menu unless we must. ^ Design dependent.

If this issue is still of interest, any thoughts?

John, thanks for your interest :)
Overall, yes the issue is disambiguating between HMDs and inputs. However I think the general problem is really hard, and I'd like to tackle a real world issue first: disambiguating between different types of Cardboard-like viewers. This situation is especially tricky since Cardboard-style VR doesn't have any way to convey data about the optical params of the HMD.
So I think there are a couple of possibilities: either we enumerate the top N popular Cardboard-like devices (eg. durovis dive, etc), and/or we do what WWGC does, and support scanning of QR codes to pick the right device params which are encoded therein.
I do think that it may be worthwhile to actually allow switching Cardboard params within VR mode. In VR mode, there could be a little configuration gear (similar to native Cardboard app) that you tap to reveal a UI to scan a QR code or pick a popular device. Outside of VR mode I think it's a lot to ask users to specify which configuration they prefer to use.
A big problem is that this config can be persisted per-domain only, unless we want users to login to get their preferred Cardboard-like device params :\
So more specifically, I think implementing this in-VR UI and QR scanner would be an excellent start.

Currently have a UI for picking viewer profile which works on both iOS and Android. Eventually aiming for a QR picker.

The URLs encoded in the official WWGC QRCodes do not directly encode the parameters, but are shortURL links hosted by Google redircting to the real (much longer) ones. This way they can chop of functionality of cardboard viewers they don't like any longer. i.e the ones using head straps - my favs.


TaskItemComparer.Compare method has misprint in it.

When comparing collection length it uses the same parameter 'x' twice:
if (x.CloneCustomMetadata().Count != x.CloneCustomMetadata().Count)

@akoeplinger, could this be by design:
https://github.com/Microsoft/msbuild/blob/82177a50da735cc0443ac10fa490d69368403d71/src/XMakeBuildEngine/UnitTests/BackEnd/TaskItemComparer.cs#L51?
I think @serjic alone has caught some very fundamental bugs in msbuild code base: https://github.com/Microsoft/msbuild/issues/created_by/serjic which need attention :smile:

@jasonwilliams200OK it looks like a bug to me, I hope the msbuild team takes a look :)


Coway API error with Airmega 250s

Hey,
Thank you for this great integration.
When I'm trying to connect my new Airmega 250s, I get the following error:
"Failed setup, will retry" and "Coway API error: Coway server failed to return purifier quality status".

Are there any logs that you can share?
The "failed to return purifier data" just means that the coway servers returned an error when calling the purifier endpoint. This can happen intermittently.

 Was waiting till I had time to properly document the issue, but since you posted this, I'll hop on board.
Procedure:
Taken from a running system. I click on "Enable debug logging" in the supervisor for the coway custom component, reload the component, wait to go from "Initializing" to "Failed setup, will retry", and turn off debug logging.
This appears in the resulting log in what appears to be the system startup logs:
'''2024-03-12 11:44:47.626 WARNING (MainThread) [homeassistant.bootstrap] Waiting on integrations to complete setup: coway'''
This appears in the log at the end:
2024-03-13 18:38:46.718 WARNING (MainThread) [homeassistant.components.rest.util] JSON result was not a dictionary or list with 0th element a dictionary
2024-03-13 18:38:55.570 DEBUG (MainThread) [custom_components.coway] Finished fetching coway data in 25.035 seconds (success: False)
2024-03-13 18:39:01.725 WARNING (MainThread) [homeassistant.components.rest.util] JSON result was not a dictionary or list with 0th element a dictionary
2024-03-13 18:39:16.720 WARNING (MainThread) [homeassistant.components.rest.util] JSON result was not a dictionary or list with 0th element a dictionary
2024-03-13 18:39:31.559 DEBUG (MainThread) [custom_components.coway] Finished fetching coway data in 30.603 seconds (success: False)
2024-03-13 18:39:31.725 WARNING (MainThread) [homeassistant.components.rest.util] JSON result was not a dictionary or list with 0th element a dictionary```

Recently received my 400S and I'm having the exact same issue as above.  Everything so far is working fine in the Mobile app.  If there's any other data I can provide that's not already mentioned above, just give me a shout.  Happy to help where I can.

On vacation so I won't be able to do anything until I'm back. Looks like they have made some changes to the API as indicated by the forced requirement to update the mobile app to continue using:

I have investigated the cause of the current issue and it is due to Coway completely changing the API. This will require a completely rewrite of the backend library, which I'll get to once I have some time. I will add a message to the readme of this repo to indicate the state - this message will be removed when the rewrite has been completed.

Hi,
I bought a 300s unit last week and I had the same issue until now.
I disabled the 'skip password change' and like magic, everything seems to work. Maybe this can work for other people too?
Cheers!

@ZappavdV Lucky you. Perhaps your device is old enough that your Coway app and account are pinned at an old version?

Hi,
I bought a 300s unit last week and I had the same issue until now.
I disabled the 'skip password change' and like magic, everything seems to work. Maybe this can work for other people too?
Cheers!

The most likely reason this was your issue, and it was resolved by changing this option, is because your Coway account is brand new - With your account being new, Coway won't present you with the password change prompt during login. Possibly, at some point, Coway will start serving you with a prompt to change your password. At that time, the integration will fail to work, and you'll need to adjust the integration's options to have the "skip password change" option enabled.
For anyone that is experiencing issues with purifiers working and then the integration not working when a newly purchased purifier is added to the account, I'll need to investigate if Coway is doing something different with a new firmware version on the purifiers  / what happens in the backend library to cause the integration to break. The only way I can investigate this is by having anyone with such an issue reach out to me on Discord (RobertD502) to temporarily grant me access to their Coway account.
@kenblu24 It looks like something funky is going on with your account....Just like the scenario above, the easiest way to get to the bottom of this is to grant me temp access to your account (to debug what is going on) by reaching out to me on Discord (RobertD502).

For anyone that is experiencing issues with purifiers working and then the integration not working when a newly purchased purifier is added to the account, I'll need to investigate if Coway is doing something different with a new firmware version on the purifiers / what happens in the backend library to cause the integration to break. The only way I can investigate this is by having anyone with such an issue reach out to me on Discord (RobertD502) to temporarily grant me access to their Coway account.

This is exactly my situation. Integration worked great right up until I added a new 250S to my account a few days ago.
@RobertD502  Is there a way I can provide debugging info without sharing account credentials?

Unfortunately, I won't be able to get far with just a debug from the backend library - I'd need to use an affected account with an older version of the IoCare app (to use the older API) and MITM the connection in order to see what it is doing. For anyone concerned, you can change your password after I am done. I don't have any nefarious intentions (I have users provide accounts all the time for adding new devices for my other integrations).
The only other solution is for any affected user to do the work themselves and then open a PR that fixes the issue in the cowayaio backend library.

As a test, I removed the new 250S from my account (via the iPhone app) and reloaded the Homeassistant integration and it initialized correctly and I had access to the previous 400S again. Re-registered the 250S and reloaded the integration and it fails again.
Deleting the older 400S and leaving the newer 250S and then restarting the HomeAssistant integrations fails. Adding the older 400s back also continues to fail.
@RobertD502 I'm pinging you on discord about credentials.

@RobertD502 if it helps, I made an account with Coway on 9th of april, so quite recently, and use an AP-1512HHS.
If you need credentials, tell me about the best way to send them to you. 😃

0.4.0 has been released which should take care of the issue caused by newer purifiers not using the old quality status endpoint.
Thank you to @randomstring for giving me temporary access to his account which allowed me to track down this issue and add some extra goodies for 250S users.

I can confirm this works with my new 400S.  Many thanks!

Can confirm as well, works with my AP-1512HHS.
Thanks a lot for this fix 👌😀

My previously not-working 250S is now working as well.  @RobertD502 Thank you!

My previously not-working 250S is now working as well. @RobertD502 Thank you!

Same here.


Document how to restart from checkpoint

AFAIK, the docs don't explain how to restart a training run. An example should be added, e.g.
To restart a training run from checkpoint replace the --settings_file ... to dingo_trainby --checkpoint YOUR_LAST_NETWORK.pt.

Two things:

Add this to the tutorial
Improve the docstring to indicate that settings_file and checkpoint are mutually exclusive.


Configuration file and function

Create a JSON that allows for environmental configurations
Create functions that will query the JSON and return the objects

Not .psd1? 🙂

What configurations would be needed in there? I would work on this issue, my use case is to define the emergency access account(s) for the tenant.
What format should be preferred? JSON would be a possibility for sure. Personally i have a preference for YAML. PSD1 would also be a suitable format.
@merill maybe?

@FriedrichWeinmann's PSFramework has a really good configuration system. https://psframework.org/documentation/documents/psframework/configuration.html
It already supports exporting/importing to .psd1, clixml and json. https://psframework.org/documentation/documents/psframework/configuration/persistence-serialization.html
I'm thinking we can leverage that instead of re-inventing the wheel.


fix flink-test

What changes were proposed in this pull request?
Why are the changes needed?
Does this PR introduce any user-facing change?
How was this patch tested?

@zhongqiangczq Is this a bugfix? Does it need to be merged to 0.3.1?


fix: undefined reference errors in ToLinkMap callback (#1130)

Problem
Merges in a commit from a patch branch
This one: https://github.com/aws-amplify/amplify-codegen-ui/pull/1130
Solution
merges in a previously approved commit
Additional Notes
none
Links
Ticket

GitHub issue:
Other links
Verification
Manual tests

Automated tests

[x] Unit tests added/updated
[ ] E2E tests added/updated
[ ] N/A - (provide a reason)
[ ] deferred - (provide GitHub issue for tracking)

Housekeeping

[x] No non-essential console logs
[x] All new files contain license notice

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

We're currently processing your upload.  This comment will be updated when the results are available.


Roll Flutter Engine from 8567d96993ed to f03b123740ac (1 revision)

https://github.com/flutter/engine/compare/8567d96993ed...f03b123740ac
2023-01-27<EMAIL_ADDRESS>[fuchsia] Migrate to new RealmBuilder API (flutter/engine#39205)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Nested tree not load async nodes with AOT

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Building the app with aot and click item icon, it will load children and update view with children of the node.
What is the current behavior?
Building the app with aot and click item icon, it will load children and not show children of the node.
What are the steps to reproduce?

Clone the repo: https://github.com/DocLM/nested-tree-example
Install dependencies
Run with ng serve --aot
Open the page and click on the > icon.

Live example (here it work): https://stackblitz.com/edit/angular-olseyg
What is the use-case or motivation for changing an existing behavior?
It work without aot and not work with aot
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Browsers: Chrome 65 on MacOS 10.13.3
Material: 6.0.0-beta.4
Angular: 6.0.0-beta.7 (also tested with 5)
Typescript: ~2.5.3

I tested the fix with rc.0 version but nothing changed.
I tested it with ng serve --prod with @angular/cli 6.0.0-beta.8
https://github.com/DocLM/nested-tree-example

@DocLM This could be a bug in angular. We will have a workaround #10635


ENH: Add itk::Copy(const T & original), which simply returns a copy

Intended to be used to make the act of copying explicit, and to avoid warnings like:

Microsoft Visual Studio 2022, IntelliSense Code Linter [lnt-accidental-copy] "auto doesn't deduce references. A possibly unintended copy is being made" (https://learn.microsoft.com/en-us/cpp/ide/lnt-accidental-copy)
Microsoft Visual Studio 2022, Warning C26820, "This is a potentially expensive copy operation. Consider using a reference unless a copy is required (p.9)" (https://learn.microsoft.com/en-us/cpp/code-quality/c26820?view=msvc-170)
Synopsys Coverity 2023.6.2, "CID 331225:  Performance inefficiencies  (AUTO_CAUSES_COPY)", "Using the auto keyword without an & causes the copy of an object...".
SonarSource static code analysis, "Unnecessary expensive copy should be avoided when using auto as a placeholder type", https://rules.sonarsource.com/cpp/RSPEC-6032/

For your information, a similar warning is produced by SonarSource, https://rules.sonarsource.com/cpp/RSPEC-6032/ as mentioned by Dvir Yitzchaki at the ACCU mailing list. Dvir posted an example at https://godbolt.org/z/ETdsYT3Ph

/azp docs/readthedocs.org:itk

I think this PR is ready, only docs/readthedocs.org:itk is still "Pending — Read the Docs build is in progress!"


Allow anonymous requests for putObject operations, circumventing mult…

…ipart

https://github.com/aws/aws-sdk-js/issues/512 - referencing similar aws-sdk-js issue here.


format content while validating

Is there a way to force jsonschema to actually parse and change the content of a json while validating it?
for example:
from jsonschema import validate
schema = {
    "type": "string",
    "format": "date-time"
}
data = "1937-01-01T12:00:27.87+00:20"
data_fmt = validate(data, schema)

with data_fmt containing a datetime object instead of a string.
Thanks

The current spec only deals with validation, not coersion.
https://github.com/Julian/Seep was / is an exploratory place for doing some coersion, which you might want to have a look at.
Happy to accept PRs / improvements there certainly as well, if you find something you'd like to add.


Registering Covalence commands with config value

I've been allowing users of my plugins to customize their commands through the config, but Covalence does not support this as it can't see the config values when it is loaded.
How I normally do it:
    command.AddChatCommand(settings.Command, self.Plugin, "ChatCommand")
    command.AddConsoleCommand("global." .. settings.Command, self.Plugin, "ConsoleCommand")

How I tried to do it:
Command{ settings.Command }
Command{ "global." .. settings.Command }

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/23069800-registering-covalence-commands-with-config-value?utm_campaign=plugin&utm_content=tracker%2F6855523&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F6855523&utm_medium=issues&utm_source=github).

Closing for now, not something that is currently easily doable with Covalence.


CI Improvements

[x] Fix emulator for instrumented tests
[ ] Setup cache #95

Implemented in #105


send_join: clarify auth_chain description

The auth_chain field should contain all of the auth events required to auth the
state events, as well as those required to auth the join event itself.
(cf https://github.com/matrix-org/synapse/blob/e148438/synapse/handlers/federation.py#L1076-L1077).

better now?


Yang enumerations

compile_data_for_schema (lib/yang/binary.lua) breaks if YANG data contains an enumeration. The problem is that the code expect enumerations to have a ctype. Enumerations should be treated as strings.

Since you are working in this area, would you mind also adding a verifier for enumerated values, as we did for identityrefs?

Since it's optional for enumerated values to have associated integer values, I guess it makes sense to represent enumerated values as strings.  It would be possible to represent enumerations where all values have an integer as int32 (see https://tools.ietf.org/html/rfc6020#section-<IP_ADDRESS>), which would allow for more efficient representation of lists containing enumerated values -- basically we could allow them to compile to the cdata representation.  But perhaps this isn't important and we can avoid doing that.  In summary I think going with strings is the right answer, yes.

wingo: Addressed comments and added an enum validator. I added an enum initializator as well to collect all the properties defined in https://tools.ietf.org/html/rfc7950#page-150. The validator only checks that a schema instance refers to an enumeration defined in the schema.
OTOH, according to RFC7950, an enumeration definition should comply several constraints (value max be a signed int 32 value, enumeration identities must not be duplicated). I can add these constraint validations in the schema. WDYT?

@wingo right, the enums validation was not working as expected. I fixed it in the last commit.


Simplify locating the artifacts for the most recent successful build of a project

Merging these changes will add a faux build ID named latest_successful for projects that when used will trigger a redirect to the build ID for the given project that was most recently completed successfully.  This redirection works for any sub-paths of the build, including artifacts.  404 results are returned for invalid projects and projects without any successful builds.
This makes it easy to share a static link with others that will always point them to the latest build for a project so that they can make use of any relevant artifacts.

 Thanks for the code, and for including tests!


adding libjemalloc2 to docker images

Fixes #128
Checklist

[x] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

As per our discussion, this is good but didn't it need additional configuration in startup scripts?

As per our discussion, this is good but didn't it need additional configuration in startup scripts?

yes, but the change needed is to the starctl script which will require PR on stargate repo.

As per our discussion, this is good but didn't it need additional configuration in startup scripts?

yes, but the change needed is to the starctl script which will require PR on stargate repo.
stargate/stargate#2404

Ah. Sorry, missed the repo this was for.

will this also be copied to the https://github.com/stargate/stargate/blob/main/coordinator/Dockerfile?

will this also be copied to the https://github.com/stargate/stargate/blob/main/coordinator/Dockerfile?

yes, #2402


[users] add demo account

what

Demonstrate adding a user with pull requests

why

Example use-case for running atlantis in geodesic to achieve operations by pull request

atlantis plan -p users


Tutorial for neural adaptive smc

Neural Adaptive SMC, Gu etc. is a nice framework that allows us to train proposals for non-linear state space models. We can use forward KL in a nested variational inference scheme because both derivations provide similar grad estimations.
For state space models, we typically don't have reverse kernel because the state dimension grows over time. This example will greatly illustrate how to deal with growing-dimensional variables in JAX. The trick will be to prepare a full dimensional variable and perform index update in each smc step.

Hi @fehiepsi,
I'd like to take a stab at this, but could use a little help getting started. Given the model from 5.1 in the paper
def ssm(xs = None, T_max = 1000):
    z_0 = numpyro.sample("z_0", dist.Normal(0, 5))
    z_t_m1 = z_0
    for t in range(1, T_max):
        z_t_loc = z_t_m1 / 2 + 25 * z_t_m1 / (1 + z_t_m1 ** 2) + 8 * jnp.cos(1.2 * t)
        z_t = numpyro.sample(f"z_{t}", dist.Normal(z_t_loc, jnp.sqrt(10)))
        x_t = numpyro.sample(f"x_{t}", dist.Normal(z_t ** 2 / 20, 1), obs=xs[t - 1] if xs is not None else None)
        z_t_m1 = z_t
    return x_t

I figure what needs to be implemented is an LSTM-based mixture density network which parametrizes q(z_t | z_1:t-1, x_1:t) (or q(v_t | z_1:t-1, x_1:t, f(z_t-1, t)), since that works better according to the paper). Then make a list of proposals, one for each z_t, each of which is sampled and used to update the full dimensional variable using zs.at[t].set(sample)  ? Would the targets simply be the model above, conditioned on x_1:t ?
I will try to code something up, but some guidance would be very helpful!

Great to hear that you are interested in this issue, @deoxyribose! The main theme of using coix is to define subprograms, then combine them together. Each subprogram is modelled by using a PPL, e.g. numpyro.
Your model is already in the form of a "combined" one. You can factor it out by creating subprograms: init_proposal, proposal_t, target_t. Here target_t is your body function of your for loop. proposal_t is your lstm-based model. target_t defines the joint distribution of p(z_t,x_t|...) while proposal_t is q(z_t|...). Let's walk through this step first.
The next step is to combine those programs together. You can use the algorithm in coix.algo.nasmc or even better, combine them in your own way. Let's discuss this later.

Thanks @fehiepsi! I've tried to do what you suggest here: https://github.com/deoxyribose/nasmc/blob/master/nasmc.ipynb, but I don't have a good handle on how it's supposed to look like yet. I'm not sure whether init_proposal and z_0 sampling should be separate from ssm_proposal and ssm_target respectively. In any case, I don't know how to progress from this current error message, but I figure I probably have some misconceptions apparent from the code which you could clear up :)

Sorry for the late response, @deoxyribose! I'll look into your notebook tomorrow.

Hi @deoxyribose I addressed several issues in your notebook to match the paper; it seems to work: https://gist.github.com/fehiepsi/b7def6a77bf9ca150cf2f17f2ba1a2b5

using num_particles=100 like in the paper
using batch_size=10 (not sure what is the mini-batch size that the authors used; they mentioned "Adaptation of the proposal was performed on 1000 samples from the generative process at each iteration.")
using T_max=100 for training and 1000 for evaluation
using Normal(0, 5) for the initial proposal
using LSTMCell like in the paper
simulate ssm by composing the targets together
fix a bug in proposal to use z_prev instead of the current z_t
keep track of LSTM's carry during the run
use different seeds for data generation and nasmc
support chain=True in nasmc (to allow output from proposal to be used in the target)

Let me know if something is unclear.

Thanks so much @fehiepsi! I got busy, but will have plenty of time to finish this in 2 weeks, if not before.


Raycast example without identity mat?

example shows how to raycast with an identity matrix. When I take this and supply my own matrix I receive compile errors.
error: the trait `nalgebra::traits::geometry::Transform<nalgebra::structs::pnt::Pnt3<f32>>` is not implemented for the type `nalgebra::structs::mat::Mat4<f32>` [E0277]

I don't understand what instead I should supply other than a Mat4, or Iso3.to_homogeneous() which is how I derive my Mat4. Please advise, thank you

You should use the iso3 directly instead. A Mat4 does not have enough geometric properties to allow an efficient ray-cast implementation for most type of objects (e.g. a Mat4 could contain things like shearing or be a projection which modifies significantly the transformed object original shape). The Iso3 on the other hand is guaranteed to be a composition of rotation + translation only.

So I tried that originally, I get this error:
error: the trait `nalgebra::traits::geometry::Transform<nalgebra::structs::pnt::Pnt3<f32>>` is not implemented for the type `&nalgebra::structs::iso::Iso3<f32>` [E0277]

How ever I just tried again, as Iso3 and not &Iso3 and it works. Thanks! I should have looked at the definition more closely


Implement lists

As I see, recursive data types are supported now. So I guess we need to implement in-language support for lists?
For example, [1, 2, 3] needs to be automatically converted to Roy list.

I'd prefer for that notation to not convert into ADT lists.
We should compile into JavaScript arrays and use that. For a few reasons:

Performance
Debugging
Readability

We can still write a good standard library but we just have to come up with a good definition of JS arrays. At the moment I've done two things:

Tuples: (1, true, "three") is typed as (Number, Boolean, String) and output to [1, true, "three"]
Arrays: [1, 1, 1] is typed as [Number] and output to [1, 1, 1]

This creates the problem of requiring JavaScript definitions of core functions (head, tail, map, etc) to be written for arrays. Still not sure how we should do that.
What do you think?

I think this issue needs more research.
I've benchmarked Roy lists and they seem to be 10 times slower on 10 elements. This is a no-go, because for 100-element lists they would be 100 times slower.


Bump all Howatworks packages to 0.10.5.

Codecov Report

Merging #142 (5da07c3) into master (2055b62) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #142   +/-   ##
=======================================
  Coverage   23.17%   23.17%
=======================================
  Files          36       36
  Lines         725      725
=======================================
  Hits          168      168
  Misses        557      557

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[CARBONDATA-2770][BloomDataMap] Optimize code to get blocklet id when rebuilding datamap

we should get exactly number of blocklet id from blocklet scanned result instead of building it ourselves.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7397/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/6156/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/6163/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7406/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5967/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5970/

retest sdv please


Update fuse interface to use io.ReaderCloser

Allows interface to be implemented from outside the continuityfs package.

LGTM


trying to fix CI: pin external action to the specific git hash

Summary
trying to fix CI: pin external action to the specific git hash
according to: https://infra.apache.org/github-actions-policy.html
Impact
Testing
none

this should works, mynewt uses something like this: https://github.com/apache/mynewt-core/blob/259b76ded76313e28014ab8f68818f23108fa190/.github/workflows/build_blinky.yml#L37

for some reason this doesn't work. I'm closing


Syncthing executable upgrade corrupts application bundle signature

This is a followup of #27 to make sure Gatekeeper will stay happy when syncthing executable upgrades itself. The problem is that the signature of the application bundle changes after distribution and Gatekeeper will not allow that.
The solution is:
During start of the bundle we need to check if the syncthing executable doesn't exist. E.g copy it (see configuration doc):
/Applications/Syncthing.app/Contents/Resources/syncthing/syncthing -> $HOME/Library/Application Support/Syncthing/syncthing

Gatekeeper only blocks the app when it's in quarantine (downloaded from an external source). After first run, it should not complain even if the app signature becomes invalid.
It's still better to keep the signature valid, though.


GEODE-5860: Do not compile jar in test code.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[n/a] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Some overlap with #2593 in gradle files, but either identical or easy to merge.

-1
Please don't do this. This is explicitly tying the test artifact creation to the build process. Future devs will scratch their heads wondering where how this artifact is being created. The test should be as self-contained as possible.


added 3 DIAS and on product

maybe all four DIAS could be listed separately?

maybe all four DIAS could be listed separately?

Would make sense, since they are (semi-)official ESA-comissioned services for bringing compute to the data.

Good suggestion - I added the DIAS' as separate category.
Thanks for the PR @mikrup. The next time please make sure to also add the relevant information if you add new entries, instead of copying the info from other entries - i.e. in your PR all DIAS had the same information and that was specific to the Mundis DIAS.


Beam search

Beam search is generally not supported by our models, though the flag exists. It appears to be supported in lstm; it is unclear to me whether it's supported by feature_invariant_transformer, transformer and transducer (if any of these silently ignore the flag, which is what I think happens, this is a serious but easy-to-fix bug), and it's explicitly not implemented in pointer_generator_lstm.

Going to assign to @Adamits since I think he may be able to bring this back.
Summarizing some discussion we had in person, the heapq in CPython is actually a pretty snappy implementation (it delegates to a C-based impl called _heapq which is going to be widely available) so it's probably fine as is. (If you want you can write a wrapper class to make it cuter.) It just needs to be resurrected, tested, and documented.


Fix previews in actions ("/me") after page reload

Fixes #1346
Took me quite a while to figure this one out!
What happens essentially is that renderPreview adds preview templates (toggle button and content) with msg.find(`.text a[href="${preview.link}"]`) and msg.find(`.preview[data-url="${preview.link}"]`). At that point, msg contained $(templates["msg_action"](data.msg)) but the content of the potential "actions" templates.
Switching those makes sure that when the previews get added, the content of the generated message is fully loaded before adding the previews.

Nice catch.


[#13332] Allow feature/* branches to be tested

Closes #13332
@rigazilla Are there any actions that I have missed? We're only interested in running tests on these branches, other automations are not applicable.

I think it's fine without either of those, as we will regularly be rebasing on main and eventually we will create a PR to merge the feature branch into main so we can detect those failures then


Create types module

As suggested by @oerc0122, we should create a types module for TypedDicts, detailing results and sensible aliases for frequently used things e.g. Union[str, Path] and Union[T, list[T].
Would it make sense to import this into aiida-mlip, or create something similar there too?

yes please


cmd/compile: cant atomic access of first word of allocated struct occasionally?

What version of Go are you using (go version)?

$ go version
go version go1.13.7 darwin/amd64
$ uname -a
Darwin newbmiaodeMacBook-Pro.local 17.7.0 Darwin Kernel Version 17.7.0: Sun Jun  2 20:31:42 PDT 2019; root:xnu-4570.71.46~1/RELEASE_X86_64 x86_64

Does this issue reproduce with the latest release?
Not sure
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/newbmiao/Library/Caches/go-build"
GOENV="/Users/newbmiao/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/newbmiao/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.13.7/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.13.7/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/m_/hlhv5wl17vg368420bbxszhc0000gn/T/go-build908799930=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
This issue about test the spec below, but got panic:

https://golang.org/pkg/sync/atomic/#pkg-note-BUG
The first word in a variable or in an allocated struct, array, or slice can be relied upon to be 64-bit aligned.

I use code below to test, and will panic occasionally
(Test it on my 64-bit mac: GOARCH=386 go build -ldflags=-compressdwarf=false -o test test.go)
But if i remove the fmt.Println() inside of the test function, it will pass.
I also debug it use gdb (append info below), x escape to heap, and use tinyallocs to alloc it
And i notice thatc.tinyoffset is 2.
But when no fmt.Println() it will be 0 or 16， Or just nextFreeFast to get a new object.
So i am confused, is that spec work in this situation?
Or maybe i misunderstand with this spec?
Or a issue cross compile under macOS?
May related with this issue: https://github.com/golang/go/issues/599
Code:
package main
import (
	"fmt"
	"sync/atomic"
)

// On ARM, x86-32, and 32-bit MIPS, it is the caller's responsibility to arrange for 64-bit alignment of 64-bit words accessed atomically.
// The first word in a variable or in an allocated struct, array, or slice can be relied upon to be 64-bit aligned.
func test(){
	x := struct {
		v uint64
		y uint32
	}{}

	fmt.Println()  //if comment this line, it will pass on 32-bit arch
	atomic.AddUint64(&x.v, 1)
}
func main() {
	test()
}

gdb info
$ GOARCH=386 go build -ldflags=-compressdwarf=false -o test test.go
$ gdb ./hello
(gdb) b main.main
Thread 2 hit Breakpoint 1, main.main ()
    at /Dig101-Go/test.go:19
19      func main() {
(gdb) s
20              test()
(gdb)
main.test () at /Dig101-Go/test.go:10
10      func test(){
(gdb)
11              x := struct {
(gdb)
runtime.newobject (typ=0x9d280 <type.*+93280>, ~r1=<optimized out>)
    at /usr/local/Cellar/go/1.13.7/libexec/src/runtime/malloc.go:1150
1150    func newobject(typ *_type) unsafe.Pointer {
(gdb)
1151            return mallocgc(typ.size, typ, true)
(gdb)
runtime.mallocgc (size=<optimized out>, typ=0x9d280 <type.*+93280>, needzero=true,
    ~r3=<optimized out>)
    at /usr/local/Cellar/go/1.13.7/libexec/src/runtime/malloc.go:877
877     func mallocgc(size uintptr, typ *_type, needzero bool) unsafe.Pointer {
(gdb)
878             if gcphase == _GCmarktermination {
(gdb) until 945
runtime.mallocgc (size=<optimized out>, typ=0x9d280 <type.*+93280>, needzero=true, ~r3=<optimized out>)
    at /usr/local/Cellar/go/1.13.7/libexec/src/runtime/malloc.go:945
945             if size <= maxSmallSize {
(gdb) n
946                     if noscan && size < maxTinySize {
(gdb)
976                             off := c.tinyoffset
(gdb)
978                             if size&7 == 0 {
(gdb)
980                             } else if size&3 == 0 {
(gdb)
981                                     off = round(off, 4)
(gdb)
985                             if off+size <= maxTinySize && c.tiny != 0 {
(gdb) p off
$1 = 4
(gdb) p c.tinyoffset  <========= notice this line
$2 = 2
(gdb) n
987                                     x = unsafe.Pointer(c.tiny + off)
(gdb)
988                                     c.tinyoffset = off + size
(gdb)
989                                     c.local_tinyallocs++
(gdb)
990                                     mp.mallocing = 0
(gdb)
991                                     releasem(mp)
(gdb)
992                                     return x
(gdb)
main.test () at /Dig101-Go/test.go:16
16              fmt.Println()
(gdb)

17              atomic.AddUint64(&x.v, 1)
(gdb)

Thread 2 received signal SIGSEGV, Segmentation fault.
runtime/internal/atomic.Xadd64 () at /usr/local/Cellar/go/1.13.7/libexec/src/runtime/internal/atomic/asm_386.s:105
105             MOVL    0, AX // crash when unaligned
(gdb)
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x2f7c]

goroutine 1 [running]:
runtime/internal/atomic.Xadd64(0x1141c0b4, 0x1, 0x0, 0x0, 0x0)
        /usr/local/Cellar/go/1.13.7/libexec/src/runtime/internal/atomic/asm_386.s:105 +0xc
main.test()
        /Dig101-Go/test.go:17 +0x72
main.main()
        /Dig101-Go/test.go:20 +0x11
[Inferior 1 (process 68214) exited with code 02]

What did you expect to see?
go run  successfully
What did you see instead?
Panic sometimes:
$ ./test  # success

$ ./test # failed

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x2f7c]

goroutine 1 [running]:
runtime/internal/atomic.Xadd64(0x1147e004, 0x1, 0x0, 0x0, 0x0)
        /usr/local/Cellar/go/1.13.7/libexec/src/runtime/internal/atomic/asm_386.s:105 +0xc
main.test()
        /Dig101-Go/test.go:17 +0x72
main.main()
        /Dig101-Go/test.go:20 +0x11

CC @randall77 @josharian

Maybe I'm missing something, but this looks like a malloc issue, not a compiler issue.
cc @mknyszek @aclements

@aclements Looks like the tiny allocator doesn't uphold the guarantee in the spec. The lines are:
			// Align tiny pointer for required (conservative) alignment.
			if size&7 == 0 {
				off = round(off, 8)
			} else if size&3 == 0 {
				off = round(off, 4)
			} else if size&1 == 0 {
				off = round(off, 2)
			}

In this case size is 12, and so we align to 4 (as the trace in the original post at the top shows), but it should be 8, because it contains an 8-byte field (right?). I don't think this is even just a 386 problem; we could end up misaligning in any case.
Maybe the fix is to just align to typ.align?

@mknyszek would that make the tiny allocator much less efficient, by increasing the number and size of gaps? And if so, are there mitigations?

@mknyszek would that make the tiny allocator much less efficient, by increasing the number and size of gaps? And if so, are there mitigations?

Maybe, but given that we don't have atomic types and instead have atomic operations, it's impossible to know whether we really need the full alignment, and so we must conservatively always assume we need it. In that sense, I can't think of any mitigations besides waiting for #36606 which may allow users to set a custom alignment (thereby signaling the alignment they actually need).
It might also be slower because we have to touch the type whereas we didn't have to before.

typ.align doesn't help in the current way. On 32-bit architectures it is 4 for int64. This is what #36606 is for.

I think this can only happen for a size 9-15 object on a 32-bit architecture. Maybe we round up for only this case. I don't think it is too bad.

Since a type's size is rounded up to its alignment, I think this specific case with a 12 byte struct on 32-bit is really the only way that this can happen, and it's a direct consequence of the underalignment of uint64.
/cc @danscales for an interesting consequence of the current alignment rules

@aclements @cherrymui Got it. I missed the fact that the size is rounded up. That makes sense, and I was surprised that this wasn't more broken...
@josharian It looks like what I was proposing is already there, so there shouldn't be any packing problems.
It looks like this problem is wholly #36606 as @cherrymui mentioned, so we might want to just close this as a duplicate?

It looks like this problem is wholly #36606 as @cherrymui mentioned, so we might want to just close this as a duplicate?

Not exactly. Fixing #36606 will make this go away, but we can also fix this separately. This issue is specific to the first word of an allocation, which we try to make it still 8-byte aligned, as mentioned in https://golang.org/pkg/sync/atomic/#pkg-note-BUG . Currently it is mostly aligned, except for tinyalloc (and sbrk mode).

@cherrymui I am thinking, in the spec: allocated struct, array, or slice (https://golang.org/pkg/sync/atomic/#pkg-note-BUG), the struct need be declared as a global type, not local type? Which will not have this issue. IMHO

This didn't get fixed.  Rolling forward to 1.16, and marking as release blocker since it does seem like a serious bug, albeit one that is hard to hit.

Given that #36606 is stalled, I think we need to revisit this. It seems like we need to special-case 12 byte allocations on 32-bit to by 8-byte aligned. Are there any other solutions on the table?

I think we should just special case these 12-byte allocations as you and @cherrymui suggested. I can work on this now.


[DUOS-1540][risk=no] Clean up automation DAR calls

Addresses
https://broadworkbench.atlassian.net/browse/DUOS-1540
Automation test cleanup

Have you read CONTRIBUTING.md lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

What is the significance of the "Chain" files?

Great question @quazi-broad. You're right that a "chain" represents a discrete block of functionality that can be re-used in different scenarios. All of this code is "test" code so the idea behind all of this is to simulate what happens in the UI when a user follows a particular path. These were constructed with the idea that we would be able to load test a particular user journey simulating N users across a limited time frame.


Consider if asserts should/can have debug messages

We can consider if we should attach messages to asserts that explain the likely problem if it fails and why it should not fail. This can help clients save time, and client developers will need a lower degree of understanding of the internals of the conformance suite to use it; They won't need to know how we develop our tests to understand a failure.

Yeah this would make sense at least in cases where we currently do something like this
# comment explaining why we assert x
assert x

this could be just
assert x, "explanation here"

I assume pytest would show that in a reasonable way


Fix NetworkNotFoundError and EndpointNotFoundError message strings

It looks like it had its logic backwards. Added some test coverage and tried to make the test failures more helpful as well

 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.:x: benmoss sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@benmoss - If you sign the CLA I will merge this :)

This also looks good

Sorry, my company's legal department seems to have sent me into a black hole. Let me try to get approval again :)


Add selenium tests

This PR mostly adds selenium tests and ways of executing them in cicd.
However, when working on this, some minor issues have been fixed in code.
Closes #87

Note that since running Selenium is somewhat unreliable in github cicd, for now I'm gonna skip the requirement of that step. At some point I'd like to bring it back, but I see no value in spendin many hours trying to make this run always pass on GH, if you can easily confirm whether it's working or not locally.


Assign categories to nodes and edges

Node category counts now:
$ cut -f2 data/transformed/phenio/phenio_node_sources_nodes.tsv | sort | uniq -c | sort
      1 category
     38 biolink:LifeStage
    242 biolink:Gene
    320 biolink:MolecularEntity
   1101 biolink:Protein
   1775 biolink:Cell
   1854 biolink:OrganismTaxon
   4498 biolink:CellularComponent
   5616 biolink:ChemicalEntity
  12576 biolink:MolecularActivity
  16471 biolink:NamedThing
  27870 biolink:Disease
  30655 biolink:BiologicalProcess
  62542 biolink:AnatomicalEntity
 124131 biolink:PhenotypicFeature


Kernel startup is ~8 seconds: on building and using a "system image"

Thank you @mauro3 for this! This helped me centralized an installation of Julia on TLJH using the julia_install.sh script! :heart: :tada:
I think this issue may be more suited for JuliaLang/IJulia.jl than this project, but I'm not sure. I'll go with posting it here as I'm such a beginner with these matters and don't really know at this point =/
Issue summary
Using julia in a terminal, you get started very quickly (<1 sec). But, if you are starting a jupyter notebook with a IJulia kernel, the startup time is ~8 seconds for me.
During these ~8 seconds, I observe a julia process is started and that it hogs 100% of a CPU core. Could it be because its stuck compiling some Julia package like IJulia?
Solution idea seed
Perhaps something like sudo julia --eval 'using Pkg; Pkg.instantiate(); Pkg.resolve(); pkg"precompile"' can be used? Note that I don't understand this code, it is just a snippet that I think could be relevant.
Confusion points

If we have centrally installed julia like with julia_install.sh, can you install packages and precompile them centrally for all users, or does each user have their own standalone environment that needs their standalone installed packages and precompilation?
How to debug the kernel startup or similar to better understand what makes it take ~8 seconds to start a kernel?

Related

Inspecting /opt/julia/1.6.4/local/share/julia is relevant. I find a compiled folder there for example.
Doing jupyter kernelspec list gives me /opt/tljh/user/share/jupyter/kernels/julia-1.6
cat /opt/tljh/user/share/jupyter/kernels/julia-1.6/kernel.json contains what's listed below.{
  "display_name": "Julia 1.6.4",
  "argv": [
    "/opt/julia/1.6.4/bin/julia",
    "-i",
    "--color=yes",
<EMAIL_ADDRESS>    "/opt/julia/1.6.4/local/share/julia/packages/IJulia/e8kqU/src/kernel.jl",
    "{connection_file}"
  ],
  "language": "julia",
  "env": {},
  "interrupt_mode": "signal"
}

cat /opt/julia/1.6.4/local/share/julia/packages/IJulia/e8kqU/src/kernel.jl contains what's listed below. I'm didn't find anything imported that wasn't IJulia, so does that mean only that needs to be precompiled?import IJulia

let
    ijulia_kernel_file = joinpath(dirname(pathof(IJulia)), "kernel.jl")
    this_kernel_file = @__FILE__
    if ijulia_kernel_file != this_kernel_file
        @warn "this kernel.jl is different from the one provided by IJulia" this_kernel_file ijulia_kernel_file
    end
end

# Load InteractiveUtils from the IJulia namespace since @stdlib might not be in LOAD_PATH
using IJulia.InteractiveUtils

# workaround #60:
if Sys.isapple()
    ENV["PATH"] = Sys.BINDIR*":"*ENV["PATH"]
end

# the size of truncated output to show should not depend on the terminal
# where the kernel is launched, since the display is elsewhere
ENV["LINES"] = get(ENV, "LINES", 30)
ENV["COLUMNS"] = get(ENV, "COLUMNS", 80)

IJulia.init(ARGS)

let startupfile = !isempty(DEPOT_PATH) ? abspath(DEPOT_PATH[1], "config", "startup_ijulia.jl") : ""
    isfile(startupfile) && Base.JLOptions().startupfile != 2 && Base.include(Main, startupfile)
end

# import things that we want visible in IJulia but not in REPL's using IJulia
import IJulia: ans, In, Out, clear_history

pushdisplay(IJulia.InlineDisplay())

ccall(:jl_exit_on_sigint, Cvoid, (Cint,), 0)

println(IJulia.orig_stdout[], "Starting kernel event loops.")
IJulia.watch_stdio()

# workaround JuliaLang/julia#4259
delete!(task_local_storage(),:SOURCE_PATH)

# workaround JuliaLang/julia#6765
Core.eval(Base, :(is_interactive = true))

# check whether Revise is running and as needed configure it to run before every prompt
if isdefined(Main, :Revise)
    let mode = get(ENV, "JULIA_REVISE", "auto")
        mode == "auto" && IJulia.push_preexecute_hook(Main.Revise.revise)
    end
end

IJulia.waitloop()

This is one  (only? ;-) of Julia's pain points, it goes by the name "time to first plot" (as this problem is often very apparent there).  In essence, more stuff needs to be compiled before package-code is running than what is "pre-compiled", i.e. pre-compilation is only a part of the whole compile.
There is one way to make this better, which is making a "system image" with the required packages compiled into (this allows full compilation).  There is a package for this: https://julialang.github.io/PackageCompiler.jl/dev/.  I think this could be used in this setup as well, to bake the system-wide packages into the used sys-image.  However, I haven't gotten around to do this (yet?); it is listed under the TODO of the README.

Ah, thanks for helping clarifying this situation and your work to help me get this far @mauro3! It sounds like this issue could make sense to retain in this github repo to represent that todo item.
Btw, I linked this issue from https://github.com/jupyterhub/the-littlest-jupyterhub/issues/728 as I figure this could be relevant to make a note about in the TLJH documentation in the future.

Yes, keeping this open sounds good.
And yes, I saw the other issue.  I will try to keep this repo/description functioning (although often I only work on this when I need it, which varies over the academic year), so linking to this would be fine.


Fix RedHat DoNotUseTerms validation issues for Red{nbsp}Hat

Updated the regex to not match Red{nbsp}Hat.
Currently it shows a suggestion:
Do not use colors to describe something unless it is also described non-visually,
for example, with the name of the item.

Need to merge this one. Can you fix the tests and we'll get it in.

Thank you for reviewing @aireilly Your comments make sense. Another thing we can try is to add Red{nbsp}Hat in the dictionary instead of this. I'll try it locally.

Felt this was important enough to complete so did it here: https://github.com/redhat-documentation/vale-at-red-hat/pull/780
Thanks for raising the PR


How can I use a different drawflow parent other than app in vue3

Hi, I'm trying to upgrade our app from vue2 to vue3. In vue2, we can set drawflow's parent to whatever component we feed it through the last parameter of the constructor. But in vue3, it seems from we can only give appContext as parent.
This troubles us because we're trying to use provide and inject so drawflow nodes can have reactive data changed elsewhere in the app. Now since the parent can only be app, provide and inject no longer works in our setting.
Is there a way we can set a different parent in vue3 other than appContext
Thanks for for this amazing library. I hope to contribute to it but I don't know much about vue's internals to make a PR here.

You can see in the vue3 example. How do we pass the property df.

https://github.com/jerosoler/drawflow-vue3-example/blob/dbd08ca8156a4aae7a8c5d4f9fe5f29a0f5b0a58/src/components/drawflow.vue#L82C8-L82C8

To access from the nodes.

https://github.com/jerosoler/drawflow-vue3-example/blob/dbd08ca8156a4aae7a8c5d4f9fe5f29a0f5b0a58/src/components/nodes/node2.vue#L56C7-L56C7

I don't know if this will help you.

🤔 Creating standalone apps? With "createApp"?


add bash script to run tests

this is a start at a test script we could run on a regular basis! https://highfidelity.atlassian.net/browse/HIFI-565

I'm actually going to close this and think about another way of doing it. :)


Qdrant filter integrations

PRs:

https://github.com/code-kern-ai/refinery-ui/pull/150
https://github.com/code-kern-ai/refinery-gateway/pull/142
https://github.com/code-kern-ai/refinery-submodule-model/pull/49
https://github.com/code-kern-ai/refinery-neural-search/pull/31
https://github.com/code-kern-ai/refinery-embedder/pull/52

Should I look into this one here too?


Rework fin_control diagnostics

This pull request drops rate checks for fin_control angle reporting topic, as it is disabled during missions.
Additionally, it ensures diagnostic updates occur even if angle reporting is disabled.

Are we gonna maintain the FIN_DATA_STALE constant in health_monitor/msg/ReportFault.msg?

Good question. @ChrisScianna thoughts?

@patapufate do you agree with moving forward with this PR as-is. We can later revisit and drop that (now unused) fault code. WDYT?


Replace noto sans with smaller font

Apparently used only when label exists
17M	vendor/endroid/qrcode
16M	vendor/endroid/qrcode/assets/noto_sans.otf

Hi @tonybolzan I could not find a smaller font containing also Japanese, Chinese and Korean characters. If you (or anyone else) can provide one, I am happy to replace it.
Thank you.

Please let me know if you find a smaller font compatible with above languages. You can always use your own font using the setFontPath method.

Hi @endroid , I understand your logic and 15MB is nothing these days if it means having a global support.
Just out of interest; the Noto Sans is the Noto Sans from Google font, right? Because the ttf files are just shy of 500kb and when I use 'some online ttf to otf convertor', the otf also is about 1 MB.
I have no idea how to test if Chinese and Korean still works (I'm from Belgium, so I'm limited to English, French and Dutch), but you might give it a go?
You can find the ttf and otf files here

https://github.com/satbyy/go-noto-universal/releases


Firmware broken on Crazyflie 2.0 hardware -- unable to connect to the nsh console using mavlink

Describe the bug
Unable to connect to the nsh console of crazyflie 2.0. I did a bisection on the Firmware commits and found that the connection problem started from this merge. I was able to compile and flash the codes, but I was unable to connect to the nsh console using mavlink afterwards. A similar issue was reported before but was not resolved.
To Reproduce
Steps to reproduce the behavior:

Compile the default PX4 firmware for the crazyflie 2.0.
Connect your computer to the vehicle via a usb port and flash the vehicle.
Try to connect to the nsh console using the command ./Tools/mavlink_shell.py /dev/ttyACM0 (You may use dmesg to see if the crazyflie 2.0 corresponds to /dev/ttyACM0 or some other path.)
The connection get stuck forever after displaying "Connecting to MAVLINK..." and no error message was displayed.

Expected behavior
The connection succeeds and the console displays nsh>

Should be fixed in https://github.com/PX4/Firmware/pull/15421.


fix disk tutorial after merge

not sure what is raising the problem, but after moving the text below the heard to another cell, ~seems to be fine the disk tutorial~
For the nbs/color_yuv420_to_rgb.ipynb just fix the header levels

in favor of #51


Document list page

Document List page created
Info ,group and comment tabs created
Image component basing document type

Closing since this PR will be superseded by another PR that is up to date with master branch


Allow a stub to return a given value

Removes stub.is_stub as it wasn't being used and the stubfunc is now replaced with a dynamic function
Users can provide a return value for their stubs now.

stub(test, "key", "my return value")

It looks like this breaks the build, mind investigating?

Sure thing, I'll take a look.

It breaks because busted 2.0-rc-6 is incompatible with recent version of lua_cliargs. .travis.yml should be updated to install scm version of busted.

Is that as simple as changing the .travis.yml from:
install:
  - sudo luarocks make
  - sudo luarocks install busted

to
install:
  - sudo luarocks make
  - sudo luarocks install https://raw.githubusercontent.com/Olivine-Labs/busted/master/busted-scm-0.rockspec

busted 2.0-rc-7 should fix this, just amend your last commit and re-push.

Great, thanks @o-lim

:+1: thanks!

Uploaded as v1.7.5.


Fix nil donations path

Sorry @ad510 :heart:.

No problem. I'm happy you ran into this bug with me rather than someone else.

<3
ᐧ
On Thu, Jul 13, 2017 at 6:02 PM, Andrew Downing<EMAIL_ADDRESS>wrote:

No problem. I'm happy you ran into this bug with me rather than someone
else.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hackclub/website/pull/324#issuecomment-315241242, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA8j-K9A4c8WRV442-0oNWwS7YfZl8VRks5sNr4ggaJpZM4OXsZm
.

--
Zach Latta
Hack Club |<EMAIL_ADDRESS>| Donate https://hackclub.com/donate