BUG: datadog/notification-api Containers constantly exiting

Value statement:
As a VA Notify Engineer
I want to stop our containers from constantly restarting
So that our system remains stable
As a Product Stakeholder
I want our service to be stable for our clients
So that we consistently provide the best service possible
Description
Our Datadog and notificatino-api containers are constantly exiting out (see images).
Additional Info and Resources

This may be to a task definition configuration or an issue with how the forwarder is configured, and I would suggest looking into container configuration and the latest image pushed to ECR first before looking at the forwarder configuration
Engineering Checklist

[x] Open support ticket with DD when ticket is picked up
[x] Investigate Datadog image and task definitions; considering environment variables and ddtrace
[ ] Investigate when these exits started and if its related to the updated datadog image
[ ] Consider whether this is related to the forwarder

Steps to Reproduce

This is an active problem that can be seen here:

Impact/Urgency

This should be considered medium priority.  There seems to be no impact to our datadog currently, but because our containers are actively exiting and this is a critical tool for our SRE, this should be addresses sooner rather than later.
Expected Behavior

[ ] Datadog containers do not exit, and this is reflected on the containers page in Datadog.

Additional Info & Resources

Investigated this and was unable to find any evidence the API containers were actually exiting. There are no logs showing gunicorn restarting. There are no logs in ECS I could find showing that the containers were restarting. The exit status that Datadog is reporting for API containers comes and goes.
The Datadog agent is 100% having issues though, and does need investigation. Possibly an upgrade to ddtrace.

Hey team! Please add your planning poker estimate with Zenhub @cris-oddball @EvanParish @k-macmillan @kalbfled @ldraney @MackHalliday @mchlwellman

@ldraney to reach out to DD, update ticket with response info...will inform next steps re: working ticket.

Sent ddog a message, we'll see how they respond!  I'll then update this ticket comment and maybe additional info based on that response.

Got a response from DDog team:

Hi Lucas,
Can you confirm if your Agent deployment configuration has the following?

Name: DD_PROCESS_AGENT_ENABLED
Value: "true"

If the above is set, you will need to set the pidMode parameter to task in the task definition for us to properly execute according to our doc here.
Sincerely,
Spencer Phillips | Solutions Engineer | Datadog | Business Hours: 10 a.m. to 10 p.m. ET, Monday to Friday

From the AWS documentation for PID mode:

If task is specified, all containers within the specified task share the same process namespace.

Which would need to be set up via the task definition here:
  "containerDefinitions": [
    {
      "name": "notification-api",
      "image": "{will-be-replaced-by-ci}",
	  "pidMode": "task",

Okay, updated a branch and preparing to deploy.  Before I need to get a little setup in Datadog to verify the containers are still failing BEFORE/AFTER my deploy to dev

I can't tell how Kyle got the "exited at" column in the above screenshot.
Compared to what I see:

But the logs still clearly show exiting containers, which should be enough for now:

deploying to dev

Had to fix task definition based on AWS doc

The errors kept coming.  It may have been from the other tasks, so I will try pidmode on those as well.
But first I want to make sure this is related to the DD_ PROCESS_AGENT_ ENABLED variable, as support suggested.
I'm deleting that variable for now and redeploying to see if the errors stop.
Then, I'll try PIDmode one more time.  Getting our system processes in fargate seems like a good idea, so I'll try to get that working while I'm here (not outside of scope - should have been the original functionality); but if it doesn't work we'll just consider bug fixed for now, I suppose.  Still thinking about it.

 Successful deploy between 1:27-1:29 MST (celery finished deploying last), seeing outcome...

The logs did stop!  Confirmed bug is related to DD_PROCESS_AGENT_ENABLED.

Next step, try readding this with pidMode: task.  If not, this is an appropriate time to continue conversation with Datadog.

Successful deploy with pidmode: task
and logs seem promising!  You can see errors have stopped for the last two deploys so far:

I'm going to give it a little time, and also see if we are getting process information in Datadog as the ddog documentation indicates we should.

We are indeed getting new process information:
Before

After

Things are looking good!  I'm going to adjust the other environments, check on these pages again to confirm containers have stopped exiting, and open the PR with appropriate links in the "tests" section

Having a conversation with Kyle related to the risks of this feature.
Perhaps more notably, I noticed that the datadog container is marked as "essential", where it should actually be the notification-api container that is essential!  That should be addressed asap, so that datadog failures don't affect notification-api containers and tasks.
https://docs.aws.amazon.com/AmazonECS/latest/developerguide/task_definition_parameters.html#ContainerDefinition-essential

Kyle wants a more robust investigation to PidMode, Essential, and how we transfer PID to our application.

This page has been the most useful
A useful and relevant TLDR section is at the end, Caveats of sharing a process id namespace:
While a process id namespace can now be shared by containers in the same ECS task on AWS Fargate, there a few things to be aware of. We’ll walk through those caveats in the context of the application and sidecar ECS task defined previously:
A process in the sidecar container can observe, stop, or restart a process in the application container.
When sharing a process in an ECS task, a new pause process runs as PID 1 for the whole task.
So, that explains the pause task we see now.
It also means that, as long as the datadog container has no functionality that can stop the notification-api container (I would be greatly surprised if it did), that notification-api should run smoothly, regardless of the status of the container.
To address your point earlier that things have crashed due to datadog, this should be moreso attributed to our ddtrace setup, and I'm in communication with the datadog team on this.  This is because Fargate is essentially running docker-compose, each container has its own isolated resources, so even if one container crashed, it will not affect the other container unless there was a dependency relationship (e.g. notification-api depending upon datadog, which it doesn't).
I'm also asking Datadog about their guidance on essential, as their own guide declares essential on both containers in the task, which doesn't make sense to me.

From AWS customer support:

Setting pidMode to task would not cause all containers to fail by default. If, for example, you had a task with 3 containers with pidMode set to task where 2 of the tasks did something small like download a file and then exit. Having those containers exit would not automatically cause the whole task to fail even if they exited with a non-zero exit code.

AWS support is asking for more information on how we use PID, though assumes we are using it for graceful shutdown with ECS.  Confirming this with Kyle.

I see related PID settings in gunicorn_config.py

Most useful documentation so far:
Announcing additional Linux controls for Amazon ECS tasks on AWS Fargate, which addresses some of our main concerns.

Also, Datadog responded to me about essential:
TLDR: we should have it on both containers, as their guide suggests:

Hi Lucas,
We mention specifying both containers as essential as the application container could still start without the Agent but this would cause the loss of monitoring. Marking the Agent as essential within the same task definition guarantees that the Agent is also started as a sidecar to the application. AWS mentions that at least one container should be marked as essential but this does not preclude additional containers within the task from being marked as such.
Sincerely,
Spencer Phillips | Solutions Engineer | Datadog

I continue to wait for AWS next response.  Meanwhile, I'm engaging in some DevOps study.

I have been helping Mack for the last little bit.
AWS support still hasn't responded.  Sent them a message:

This is great!  We're awaiting your response.  I appreciate the details you can provide for our own documentation on this situation!

AWS response as of ten minutes ago

successful deployment as of 2:25 MST.  Checking logs

so far looking good

Preparing for a sync with Kyle on this

I'm going to remove essential from my PR and open this tomorrow morning for review

Merged!  Moving this to the QA lane.

I found the logs most helpful for development.  the above log search query with pool and shutdown proves the pidmode didn't interfere with graceful shutdown.
or exit in the same dev-notification-api-log-group, with a timespan of a week, will show these errors coming and going as I messed with deployments, and ultimately gone away since the deploys today.
Also, the containers are clearly not exiting anymore on the containers page; if the containers were exiting, this would be apparent in more recent start times for some failing containers, but since they are all the same since the last deployment, the fix appears to be working. :


Couldn't include file 'includes/modules/bromsock.lua' (File not found)

When i do require("bromsock") it fails.
Looks like there's a missing Lua library or something but i couldn't find it. Running Arch Linux x86_64

Nvm solved it


Using Phaser-Input in Phaser 3 Has An Error

Phaser version: 3.10.1
phaser-input version: 2.0.5
webpack version: 4.15.1
When I tested in Chrome, I got a error from Chrome Developer Tool:

Uncaught Error: Module build failed (from ./node_modules/exports-loader/index.js):
no such file or directory, open<EMAIL_ADDRESS>
If phaser-input not support Phaser 3?

I got the same error

It's not available for v3 (yet),
Also based on your error I'd say you are missing a slash between node_modules and @orange-games

Will phaser-input support v3?

I want it too, unfortunately I can't give an ETA


[Bug] Fix the chatbot close button

Description
The Head Section Of Chat Bot Is behind The Navigation Bar, and There Is No Option For Cancelling The Chat Bot Field If The User Wants To.
Please Assign This Task Under Me For GSSOC'24!!!
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[x] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Hey I would love to fix this

Issue 621 is already open for this, pls create new issues @Anupsimha


fix: version notation in requirements/base.txt (#28)

Closes #28

@your-highness please add a noop commit for conventional pull requests
git commit --allow-empty -m 'noop'
git push

I will actually fix this in #32. Thank you for the report and PR.


Using private struct's fields

Is there any way to use private fields in structs to insert/get them as usual?
I. e. :
type User struct {
	SteamID      string
	ID           string `gorethink:"id,omitempty"`
	money        int    `gorethink:"Money"`
}

By inserting this struct to database there is no field "Money".
gorethink uses reflection anyway, so why doesn't it scans private fields, especially when they have gorethink tags?
I need this feature because User object is cached in-memory in my app, but I wanted to get user's balance directly from database, by exposing User.Balance() method, or something like this.

In other words, I would ask: How do I create User object with Money field in database within inserting a struct without such field?
I don't see a way to do this, except creating some anonymous struct upon inserting.

Hey @dobegor, Unfortunately I dont think I can support the use of private fields even if it is technically possible. Looking at other similar libraries they all do the same thing.

@dancannon that's ok, but how about some new tag to create fields?
like [this] (https://github.com/alexflint/go-restructure) library does with fields named "_" with tags.
something like:
_ int `gorethink:"Money"`


Readme Errors

Hi,
just want to highlight two minor Readme Errors:

2nd Example for the serve command: It does not have a mode parameter
Examples for the watch command are using the serve keyword

Regards

Thanks for highlighting these.

@muenzpraeger The merge command example still have the wrong flags


Need a recovery program from a power cycle

A few times, an experiment has been prematurely halted due to a power cycle on a Pi. There are a few situations to consider:

Power cycle on a leader also a worker.
Power cycle on a worker only.
Power cycle on both the leader and worker.

Ideally, a power cycle occurs, and the machine is in the same, or near-same, state as previous to the cycle. This isn't always wanted though, so maybe it's configurable (i.e. I pull the plug to disconnect some bad action, I don't want that action to start again when I turn it back on).
Related to #107

Alternatively, there is an alerting system when the RPi cycles unexpectedly mid-experiment, so the user can manually intervene.

The current position is not good: when the leader goes down, the broker looses all the LWT (whyyyyy), so we need a way to correct this.  Solutions:

[x] monitor does a correction: gets latest experiment, looks for jobs with state="ready", and compares against what is currently running on the machine. Differences get a "lost".
[ ] watchdog does some sort of ping to all MQTT-active jobs?

Using a script like the bottom of this page: https://ubuntuforums.org/showthread.php?t=1621039, allows us to know if the shutdown was graceful or not.

monitor does a correction: gets latest experiment, looks for jobs with state="ready", and compares against what is currently running on the machine. Differences get a "lost".

We removed this, but I think we should add this functionality back?

When a leader/worker cycles
Here's an idea that would help users avoid the "UI state doesn't match system state" that is caused by MQTT: don't persist MQTT messages to disk. What are the current topics / messages that need to be persisted - and move that data to another storage.
Not persisting messages means future developers need to find another solution (like local_persistant_storage) which is a good pattern.


SAML Signout url

I'm working on an App that use Azure AD B2C if the identity provider.
I connect to it with SAML and this all word fine.
Only the signout, currently the app only clears the cookies so the user can sign in again without credentials.
I found this Doc:
https://docs.microsoft.com/en-us/azure/active-directory-b2c/session-behavior?pivots=b2c-custom-policy#single-sign-out
But I can't find the URL where I should point my app to in order to sign out properly.
If I try the following URL:
https://tenant.b2clogin.com/tenant.onmicrosoft.com/oauth2/v2.0/logout?post_logout_redirect_uri=https.....p=B2C_1A_signup_signin_saml
I get the next errors.
We can't sign you in
Your browser is currently set to block JavaScript. You need to allow JavaScript to use this service.
To learn how to allow JavaScript or to find out whether your browser supports JavaScript, check the online help in your web browser.
Sorry, but we're having trouble signing you in.
We track these errors automatically, but if the problem persists feel free to contact us. In the meantime, please try again.

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

ID: 9627f2fa-6005-9c15-2824-03b7bcd4aae9
Version Independent ID: 619e1eea-ed67-994b-7eee-469123e9b7ef
Content: Configure session behavior - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/session-behavior.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@WimSchuiteman Thank you for your query. We will investigate and update the thread.

@WimSchuiteman Thank you for your query. We will investigate and update the thread.

SAML logout endpoint URL can be found in the SingleLogoutService element of the Azure AD B2C SAML identity provider metadata.

SAML logout endpoint URL can be found in the SingleLogoutService element of the Azure AD B2C SAML identity provider metadata.

@WimSchuiteman Let us know if you need any further help.

@WimSchuiteman Let us know if you need any further help.


Subjects on mobile not clickable

@JonasBley in the serlo-home file there are more closing div tags than opening ones thus closing the .content div too early at https://github.com/serlo-org/athene2/blob/94adf55a1026dcc3301f7d4b0040994e2a878d30/src/module/Ui/templates/layout/de/serlo-home.phtml#L198
I think you should just remove this, however this causes your padding in the home-calls-wrapper to look different, so maybe you want to change some styles too. Please take a look at this, as the landing page is really broken on mobile...

That should already be fixed and merged in #494
I think it is not deployed yet though.

Unfortunately not. That PR is already deployed :/

Oh i see. That is weird as it worked for me when I tested it. I will look into it asap.

That PR was fixing the links in the home-calls-wrapper, but the subject icons still aren't clickable. Also the changes there are somewhat weird, as they normally shouldn't change anything on mobile^^ That's why I looked into it. I think removing the closing div tag mentioned above should solve those issues.

The change that I though would fix it was the col-sm-12 to col-xs-12 class change for the headlines. I thought the issue was that, for screen sizes smaller than md, the div around the title Serlo.org ist die Wikipedia fürs Lernen just wrapps around the whole page.

It makes sense that the closing div tag is the problem


work with existing viewcontrollers in storyboard?

Is there a way to integrate EZSwipeController with existing view controllers in storyboard? I tried to do this but received an error when EZSwipeController tried to access view controllers. error : fatal error: unexpectedly found nil while unwrapping an Optional value

Can you show me which line it was at?

It was at the very first few lines of my first view controller.         captionTextField.delegate = self

I don't know what a captionTextField is. I need more info, this doesn't seem to be related to EZSwipeController

Everything in EZSwipeController seems to work fine, but the error was thrown when it tried to access my existing view controllers. captionTextField is a UITextField in one of my existing view controllers, as soon as I comment all IBoutlets everything works fine. I was just wondering if there was a way to integrate existing view controllers in storyboard with EZSwipeController. In the demo project, it looks like everything was added programmatically, but can i use storyboard tho?

 You can't use storyboard without making some edits but you can call from a button that is inside the storyboard

Hello, have you been able to solve this problem?

No, I just used a UIPageViewController instead

:cry:
If there is a way to solve the problem you had, let me know or send a pull request please

I will try!
On Jan 23, 2016, at 10:59 PM, Goktug Yilmaz <notifications<EMAIL_ADDRESS>wrote:
[:cry:]
If there is a way to solve the problem you had, let me know or send a pull request please
—
Reply to this email directly or view it on GitHubhttps://github.com/goktugyil/EZSwipeController/issues/5#issuecomment-174245106.

http://stackoverflow.com/questions/tagged/ezswipecontroller
You can ask your question here if you want to. If you solved the problem please send a pull request. Thanks!

I had run into this issue as well. I believe it is because Jackson and I both have previously made ViewControllers in a Storyboard file, and are now trying to use them in the EZSwipeController. However, if you have objects that were outletted to the ViewController class, since it hasn't been instantiated from the storyboard, it is nil and therefore causes a crash.
To fix this, in my custom subclass of EZSwipeController's setupView() function, I instantiate the three ViewController's I need from the storyboard (using storyboard.instantiateViewControllerWithIdentifier), save it to my custom subclass of the EZSwipeController and then reference that array of ViewControllers in the viewControllerData() extension func.

@blazierkyle  can you provide code example? Thanks

@otymartin Happy to hear you got it working!

@blazierkyle Thanks for the solution, i was so happy to finally get it done. Im curious though if you ever managed to get rid of bounces in first & last VC. Thats what im struggling with now in #38


Default Notification sound playing instead of custom notification in android.

This is a sample code for it.
`
messaging().onMessage(async message => {
await notifee.createChannel({
id: message.messageId,
name: 'message',
sound: 'hollow',
importance: AndroidImportance.HIGH,
vibration: true,
vibrationPattern: [300, 500],
});
await notifee.displayNotification({
title: message.notification.title,
body: message.notification.body,
id: message.messageId,
android: {
channelId: message.notification.android.channelId,
sound: 'hollow',
importance: AndroidImportance.HIGH,
vibrationPattern: [300, 500],
},
ios: {
sound: 'notification_tone.wav',
critical: true,
criticalVolume: 0.9,
},
});
});`

Please help me figure out what I'm doing wrong

could it be the wrong channel id? When you create the channel you use message.messageId, but when you display the notification you use message.notification.android.channelId?

Thank you Helenaford, it worked when I made the id and the channelId the same value.


requirements file fixes

requirements file fixes

Please rebase on the most current version of the winpanda branch and update PR.


can't pass list of colors in set_color() fixed

Motivation

Trying to fill gradient into a VMobject was returning error because of this commit
Overview / Explanation for Changes

The change in the commit mentioned above didn't take care of lists passed to set_color(). So I added a check to avoid that and gradient works correctly now.
Example:
Code
class gradient(Scene):
    def construct(self):
        circle = Circle(fill_opacity=1).set_color(color=[YELLOW, GREEN])
        self.play(ShowCreation(circle))

Before:
File "/home/user/.local/lib/python3.9/site-packages/colour.py", line 664, in web2hex
    if web.startswith('#'):
AttributeError: 'list' object has no attribute 'startswith'

After

Oneline Summary of Changes

- Can't fill gradient bug fixed (:pr:`990`)

Acknowledgements

[x] I have read the Contributing Guidelines

Reviewer Checklist

[ ] Newly added functions/classes are either private or have a docstring
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings
[ ] The oneline summary has been included in the wiki

Looks good to me! Could you take the code you have added above, replace self.play by self.add, and make it a graphical unit test?
We should make sure that we don't break this again in the future.

Question: why nit making set_color only accepts one color, and using set_color_by_gradient when one wants to create a gradient ?

idk but Circle().set_color_by_gradient(BLUE, GREEN) doesn't set a gradient for me

yes, that's kind of the problem. I think this PR is the right place to fix this, don't you think ? (it shouldn't be long, only a few lines)

I think Mobject.set_color_by_gradient has its own right to exist, I would not want to remove that functionality here.

yes, but it should be renamed. It's very confusing, in my opinion at least.

Deprecating the old name and switching to a new one (together with adding some documentation maybe :-)) is a good idea. I don't see a reason to block this PR over it, though.

If you say so. Then, SGTM

thanks for the contribution @sparshg :D


add server and webpack bundler

Goal here is to add a simple express server that serves a single html page and a .js bundle generated via webpack.
Ideally the bundle will only include the modules necessary to run the client (ex: no need to provide anything pertaining to midi).

useful article explaining how to exclude modules from bundle:
http://remarkablemark.org/blog/2017/02/25/webpack-ignore-module/


support for running on push in branches

Setup
I want to run CI on every push to my feature branch.
on:
  push:
    branches-ignore:
      - 'main'

Am I the first person to try this? It's strange to me that it doesn't work.
Expected
I would expect nx-set-shas to execute the git merge-base strategy.
Actual
nx-set-shas chooses the "last successful workflow" strategy for setting a BASE. Then it fails to find one and defaults to HEAD~1, with inconsistent outcomes for the robustness of the work on my branch.
Ideas

Can nx-set-shas choose the git merge-base strategy whenever the current ref is not mainBranchName?
Can nx-set-shas allow me to configure the strategy I want it to use? I have a different action for releasing on pushes to main.

Workaround
In my use case, it appears that nx-set-shas can be replaced with simply:
- name: set HEAD and BASE for nx affected
  run: |
    echo "NX_HEAD=$(git rev-parse HEAD)" >> "$GITHUB_ENV"
    echo "NX_BASE=$(git merge-base origin/main HEAD)" >> "$GITHUB_ENV"

Adding a fallback SHA does not address this need. My problem is that nx-set-shas chooses the wrong strategy for running tests on my branch. It chooses the strategy intended for the main branch (deploys) rather than the strategy intended for feature development (pull requests).
I believe the problem is that nx-set-shas depends entirely on eventName to pick its strategy. This approach fails for events like push that can happen in multiple contexts.
I don't currently feel comfortable submitting an implementation change because it seems like a very delicate thing to change. This appears to be a deep assumption in the package.


Adding support for 527K - Dyson Pure Hot+Cool Formaldehyde

This adds support for the Dyson Pure Hot+Cool Formaldehyde (527K).
This depends on https://github.com/shenxn/libdyson/pull/36.
This solves https://github.com/shenxn/ha-dyson/issues/167

I've added a second commit fixing the class and unit of the formaldehyde sensor, which were not being seen by HA as a measurement class. Feel free to reject that second commit if that was setup that way on purpose, but I suspect it wasn't intentional.

Great work! Hoping to see this added soon. Been waiting for Formaldehyde sensor support for my TP09 :)

It should be easy to apply this (and related PR to libdyson) by hand on your system in the mean time, if you don't want to wait

It seems I get the following. I've seen others with this in the issues section.
Platform not found (cannot import name 'DysonPureHotCoolFormaldehyde' from 'libdyson' (/usr/local/lib/python3.10/site-packages/libdyson/init.py)).

Yeah you need to patch your libdyson with the content of that PR first, the code here depends on the lib already supporting the fan

Yeah you need to patch your libdyson with the content of that PR first, the code here depends on the lib already supporting the fan

Gotcha, thank you. I made that change and the error is gone, but I do not see the HCHO sensor appear. I made sure the changes I added from your diff are in my sensor.py and that manifest.json points to the new lib like your previous comment. I've tried reloading the device and rebooting HA. Maybe I missed something.

Ah yes, that's a different device.
If you know what code it is (get_devices.py says it) we could always add it to the PRs, should be easy enough

@Ulrar , I get libdyson.exceptions.DysonInvalidAuth when trying to use get_devices.py, similar to this post. I have a Dyson Pure Cool Formaldehyde TP09.
Modifying your changes to the following allow me to see the HCHO readings in HA that match my TP09
from libdyson import (
    Dyson360Eye,
    Dyson360Heurist,
    DysonDevice,
    DysonPureCoolLink,
    DysonPureHumidifyCool,
    DysonPurifierHumidifyCoolFormaldehyde,
    DysonPureHotCoolFormaldehyde,
    DysonPureCoolFormaldehyde,
)

if isinstance(device, DysonPurifierHumidifyCoolFormaldehyde) or isinstance(
            device, DysonPureHotCoolFormaldehyde) or isinstance(
            device, DysonPureCoolFormaldehyde):

I'm not sure what the best way to add it is since you have a PR pending. I can branch from your branch, or you can add, either way :P

Ah sorry, completely missed this.
Looking here it sounds like the product code might be 438E, does that sound right ?
Ideally you wouldn't just use this PR as is, since your device doesn't have heating, but we can add another instance without heating for that product code, assuming that's the right one.

@Ulrar i have a HP09, got the serial and the credentials via libdyson.
I've downloaded dyson-local, did the changes to the sensor.py manually, restarted HA.
When i try to add the device i get "Failed to connect".
How can i solve that?

Have you made the changes to libdyson as well?
There's two PRs, one for the HA integration and one for the underlying libdyson, both are needed

Ah sorry, completely missed this.
No worries.

Looking https://github.com/lukasroegner/homebridge-dyson-pure-cool/issues/225 it sounds like the product code might be 438E, does that sound right ?

Yes, that is correct. I've been using the changes I mentioned above without issue for a couple weeks now.

@Ulrar @dustingrady do you also use HomeAssistant? Cause i cant find the libdyson folder in HomeAssistant, i've done the changes in dyson-local files.

Yeah but I don't use Hass OS so I can't talk to it, I just use the docker container.
Have a look in the other PR, there's discussions about this

Is the main project we're branching from still supported? I haven't seen any merges recently by the author.

Support for both 527K devices (Formaldehyde and non-Formaldehyde) has been implemented on https://github.com/libdyson-wg/ha-dyson and https://github.com/libdyson-wg/libdyson-neon
@dustingrady sadly, while we're grateful for shenxn's building of the foundation, they are no longer active. the ha-dyson fork has been created to continue development of these projects.

Hope this gets merged in one day soon :)


More info on meal cost calculation in summary

So that cooks can adjust if there are late signups, etc.
Originally created by Tom Smyth at 2015-10-03 19:59, migrated from redmine-#3265

Estimate set to 1.00 h

By Tom Smyth at 2018-04-21 18:30


Reevaluate behavior when cookie parsing fails

Related to issue #390 but this topic is more about what the behavior should be when you're in this state.
I ended up with request cookies like the following for http://localhost:5000/:
__RequestVerificationToken=xhhkbQ7Vw0bBpyKt924bA4mEwn-xqrEj8w_Icr5Z2LQnXtugI8qt_glxLRAP3KPQR8RGBUukjfQzPOFCO2sffThjcUmtdgSkzn2iZ4NoObo1;
rm3EK3lo-DQ=CfDJ8DZcqGjS2eNPqurEN8WnNPUoyWWl9b3SROeJgbx_QsBsWViYvx7lI8jIQf5nv9_3B8zVdvmQdTYuqACH5p9Lx07aH8sctN-kt6CJTA4pp39O2Yymvs7M2IBx8eeTqeEn5uXh9_yV66f6dqK4l78r7dQ;
Sample=36;
ipt={"v":{"L":3},"pt":{"d":3},"ct":{},"_t":44,"_v":"2"};
.AspNet.Session=f5b61640-9efc-a9f8-32c1-6291f903f795;
BQFxCZqo5n4=CfDJ8GVk6iB7ymZKmY1csOqk7Cds8G0Hd8XFeWgB9u4RB0r73u2gJvYuQmjGY2vn8WtGvFFKwUOLS3LVlkIxH9YmvQBopm69anm0VuXYBePCzYMYRruxRQ9ma34AAZSI76o3eV6cV0xsU3MYytMgTkC_f1U;
eZYsIZozZe4=CfDJ8GVk6iB7ymZKmY1csOqk7CeNhzXg8fanKIOAcxGcBWXzm5AYBDlOgfSNHG8dLtWG2bFucr_PNhmjyEybiZQgGrdL2CEmvrhxR4MIoh6FgXdcx7N6OHL0tgDn520ud3i-ODYlXGr3aYT-5Eg3M8ulwvc;
XSRF-TOKEN=CfDJ8GVk6iB7ymZKmY1csOqk7CfYDif53fpXVX_mfM1SVKdA4fDDMi3rx98lBVks-zLiGjMv3j6SgAEHMe4VwKqMw5WCdJObXVgF1kTLA62U8SQJ2eLK2cv4vhcmlixQN3dhtOt_yeiPYtAkipDlhPUWiAg

and I was mystified about what to do when antiforgery wasn't working. At some point in my otherwise squeeky-clean past, I was hacking on something that added ipt={"v":{"L":3},"pt":{"d":3},"ct":{},"_t":44,"_v":"2"}; to my cookie store, and it effectively 'poisoned' local development for me with anything that relies on cookies in our stack.
What happens when you get into this state is that there's no indication that the cookie header was consider invalid, the cookie collection is empty, and no exception is thrown.

Note: because cookies are domain specific you would only see this kind of cross app contamination on a dev machine (e.g. localhost).

Now that you mention it - you could also see cookie contamination when your web app is part of a compound site behind a layer 7 network load balancer

Or similarly on IIS with vdirs

I am in favor of changing this behavior. I am experiencing an empty cookie collection from  cookies that are created from a 3rd party web site that we use regularly within our company. If the 3rd party web site is run I loose the ability to get any cookies in our web site.

just a note: I see the cookie contamination in other environments - not just localhost.

Moving this to Backlog as we will be in RC2 ask mode very soon.  If you feel strongly about this issue, please ping me.

This is a pretty big issue for us. We have a large domain with many applications some third party like the one that is crashing the cookie parser. In our case we are using a cookie to authentication tokens and not having it available reliably means we can’t authenticate our web api calls.
Thanks.
Jeff
From: Murat Girgin<EMAIL_ADDRESS>Sent: Thursday, January 21, 2016 5:18 PM
To: aspnet/HttpAbstractions<EMAIL_ADDRESS>Cc: riegelj<EMAIL_ADDRESS>Subject: Re: [HttpAbstractions] Reevaluate behavior when cookie parsing fails (#515)
Moving this to Backlog as we will be in RC2 ask mode very soon. If you feel strongly about this issue, please ping me.
—
Reply to this email directly or view it on GitHub https://github.com/aspnet/HttpAbstractions/issues/515#issuecomment-173731414 .  https://github.com/notifications/beacon/ADvawtFD3utZ6qG9Oq5qAnEVK2OyJjYlks5pcVCKgaJpZM4G2_r6.gif

Please share sample (redacted where needed) cookie headers containing a mix of valid and invalid values you've observed. I'll see how forgiving we can make the parser.
In the case of cookie1=valid1; cookie2=invalid2; cookie3=valid3 it's easy enough to return cookie1 but returning cookie3 can be a challenge. Should we even try to return cookie2?

I sent fiddler loges to the support agent SR number<PHONE_NUMBER>08104 which he used to reproduce the problem hopefully they should still be attached. My need would be if cookie2 is bad we should still get cookie1 and cookie3

Thanks @riegelj, I was able to retrieve your sample and successfully parse it with my changes. I still skipped the invalid value though. Having spaces in the value is a whole new flavor of invalid...

Roger that, Thanks!


[Question] How can I retrive the queued job by its id?

Environment

Laravel Version: 5.4
PHP Version: 7.1
Queue Driver: Redis

Description
I want to:

Queue the job into Redis.
Get its id.
Save the id with the related user information into Redis or MySQL.

Then:

Retrive the job by its id.
Cancel the job!

queue - How to get the queued job from job ID in Laravel? - Stack Overflow
I've also got stuck in the same situation, but there are still no answers...

Please ask on the firums.

Oops, sorry

Reposted: How can I retrive the queued job by its id? - Laravel.io - The Laravel Community Portal


Phaser.Loader.  Errors in getAudioURL() and getVideoURL() using BLOB: or DATA:

I'm trying to load sounds using data-uri scheme and I've found a small bug when using {uri: .., type: ..} pair.
In the preload() function of a state I add my audio like this:
game.load.audio('Music', [{uri: 'data:audio/mp3;base64,SUQzAwAAAAAAFgAAAAAAAAAAAAAAAAAAAAAAAAAAAAD/+3BAAAAB/Rg.....', type: 'mp3'}] );

But when the audio gets loaded I get the error:
Phaser.Loader - audio[Music]: error loading asset from URL false (No supported audio URL specified or device does not have audio playback support)
I've tracked the error and is easily fixable, the problem is inside the getAudioURL() method of the Loader manager: When using url-type pairs, the url property gets overwritten before we can set the audioType property.
Currently the code looks like this:
            var url = urls[i];
            var audioType;

            if (url.uri) // {uri: .., type: ..} pair
            {
                url = url.uri;
                audioType = url.type;
            }
            else

And should be like this:
            var url = urls[i];
            var audioType;

            if (url.uri) // {uri: .., type: ..} pair
            {
                audioType = url.type;
                url = url.uri;
            }
            else

Also at the end of the for-loop, if we defined the audio with a {uri: .., type: ..} pair the code returns the url object that's inside the urls array instead of the url with the 'data:'
         if (this.game.device.canPlayAudio(audioType))
            {
                return urls[i];
            }

... so it breaks in the xhrLoad line of the loadFile() method when this.transfromUrl() gets executed because file.url is an Object instead of an url:
this.xhrLoad(file, this.transformUrl(file.url, file), 'arraybuffer', this.fileComplete);

To fix this, getAudioURL() should be like:
         if (this.game.device.canPlayAudio(audioType))
            {
                return url;
            }

We'll find these bugs in getVideoURL() too.

I made a PR (#2236) to fix this, just making your changes in the Phaser.Loader source.

Excellent, thanks!, I'm still new to all things related to git.

Thanks both, fixed via PR merge.

I user v2.5 but I same error
Phaser.Loader - audio[s-start]: error loading asset from URL false (No supported audio URL specified or device does not have audio playback support)

@lilijreey if you want this looked at, it needs opening as a new issue, with a demonstrable test case. Comments to old / dead threads don't ever progress beyond that I'm afraid.

how to load a BASE64 image in phaser 3


kmod: add patch to correct behaviour with --field

#96123 targeted at staging

This came up in https://github.com/NixOS/nixpkgs/pull/96008.
Without this patch modinfo always prints the module name for builtin
modules, even if an explicit --field is passed.
(I also posted this patch upstream, waiting for feedback there)
Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Upstream submission: https://lore.kernel.org/linux-modules/20200823215433.j5gc5rnsmahpf43v@blumerang/T/#u


Пропустить рекламу

No description in the issue!!

It worked at first, now it doesn't. opera browser

Is it not skipping ads or like what main issue are you facing. Can you please elaborate so that I can help you further

Is it not skipping ads or like what main issue are you facing. Can you please elaborate so that I can help you further

Now everything seems to be fine. I reinstalled the extension. The problem was that the advertisement was not skipping. Thank you.

@andoem I fixed this issue and also released a much more advanced version of it AdBlitz. Can you please switch to it and see if issue still persists.

sample code in console && profit
setInterval(()=>{document.querySelector('.ytp-ad-skip-button-modern.ytp-button').click()}, 10000)

setInterval(()=>{     let button = document.querySelector('.ytp-ad-skip-button-modern.ytp-button')     if(button !== null) button.click() }, 3000)

sample code in console && profit setInterval(()=>{document.querySelector('.ytp-ad-skip-button-modern.ytp-button').click()}, 10000)

Брат, столкнулся с такой же проблемой, но установил его новое расширение и оно уже корректно работает. Попробуй


gist:OrderedCollection has no ordering mechanism ?

I do not see that gist:OrderedCollection defines a mechanism for specifying an order. Does that mean that it is left up to the implementation to define how to establish order? Or am I missing something?
From a Data Centric Architecture point of view, an ordered collection is a bit useless without a defined ordering mechanism.
Why is there an ordering mechanism on gist:OrdinalCollection but not gist:OrderedCollection?
To put an ordering mechanism in place I can think of two existing examples:

use rdf:Seq and/or rdf:_1 in some manner

move the ordering mechanism of gist:OrdinalCollection up to gist:OrderedCollection? Meaning gist:directlyPrecedes and gist:directlyPrecededBy have owl:someValuesFrom a new class gist:OrderedMember and deprecate gist:OrdinalMember (or make it a rdfs:subPropertyOf gist:OrderedMember)??

The discussion on issue #112 is also relevant.

Does this issue include anything that's not in #112? If not, I suggest closing it.

I agree. Just make sure to point back to this issue (114).

Duplicate of #112, which is broader.


Elaborate / emphasize that Kedge is an "extension" of Kubernetes.

One of the best features of Kedge is the fact that you are able to add Kubernetes constructs easily to the Kedge file.
For example:

The entire Kedge file is an extension of PodSpec.
Most of the Kedge objects that we have are also extendable (for example, service is extended from serviceSpec).

We need to emphasize this in the README as well as the documentation that the Kedge-specific keys are there to help and make your life easier, however, you can still add Kubernetes-specific constructs to your file.

Fix in https://github.com/kedgeproject/kedge/pull/437


[Designer] Collapse toolbox on ESC

Related Issue
Fixes VSO 30863472
Description
The expand/collapse widget in the designer should respect an ESC keypress to trigger collapsing a toolbox:

How Verified

local build, devtools

Microsoft Reviewers: Open in CodeFlow

Why?

Why?

it's a fair question -- the bug has details. I'm not sure I necessarily agree with the requirement, but I figure that the change is simple and innocuous enough that it's easier to take the change rather than argue about it

it's a fair question -- the bug has details. I'm not sure I necessarily agree with the requirement, but I figure that the change is simple and innocuous enough that it's easier to take the change rather than argue about it

I must say this "bug" doesn't make much sense to me. It's a nit, not an actual issue (let alone an accessibility issue), pretending that it would make users more efficient without demonstrating it would.
In the tree view in particular, ESC would be a lot more useful if it selected the parent of the currently selected element (which for that matter is already implemented on the design surface itself). This would allow for much faster and predictable navigation.
The code change itself is fine, but I really don't think it's necessary.

I must say this "bug" doesn't make much sense to me. It's a nit, not an actual issue (let alone an accessibility issue), pretending that it would make users more efficient without demonstrating it would.
In the tree view in particular, ESC would be a lot more useful if it selected the parent of the currently selected element (which for that matter is already implemented on the design surface itself). This would allow for much faster and predictable navigation.
The code change itself is fine, but I really don't think it's necessary.

I must say this "bug" doesn't make much sense to me. It's a nit, not an actual issue (let alone an accessibility issue), pretending that it would make users more efficient without demonstrating it would.
In the tree view in particular, ESC would be a lot more useful if it selected the parent of the currently selected element (which for that matter is already implemented on the design surface itself). This would allow for much faster and predictable navigation.
The code change itself is fine, but I really don't think it's necessary.

feel free to push back in the bug -- I just don't think the argument is worth the effort

I must say this "bug" doesn't make much sense to me. It's a nit, not an actual issue (let alone an accessibility issue), pretending that it would make users more efficient without demonstrating it would.
In the tree view in particular, ESC would be a lot more useful if it selected the parent of the currently selected element (which for that matter is already implemented on the design surface itself). This would allow for much faster and predictable navigation.
The code change itself is fine, but I really don't think it's necessary.

feel free to push back in the bug -- I just don't think the argument is worth the effort

I just don't think the argument is worth the effort
Agreed...

I just don't think the argument is worth the effort
Agreed...


Improve verb tense in IAM v2 Overview doc

Signed-off-by: Mary Jinglewski<EMAIL_ADDRESS>:nut_and_bolt: Description: What code changed, and why?
These edits implement a simple present verb tense in IAM v2 Overview documentation.
:chains: Related Resources
:+1: Definition of Done
:athletic_shoe: How to Build and Test the Change
:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[x] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

any chance this could merge into force-upgrade instead of master? otherwise, i think we are going to have conflicts to resolve.. maybe?

looks like weve got double spaces where the v2s were removed


Allow numeric boolean parsing in kml styles

Adds a KmlBoolean class that encodes the boolean parsing method so that
both 'true' and '1' will be interpreted as a boolean true value, all other values
will return false.
Modify the KmlStyleParser to use this method for parsing booleans instead of
using Boolean.parseBoolean(string).
Adds two unit tests, and sample KML data files that implement this handling.

LGTM - thanks!


Events not appearing in any index

See:

http://discourse.snowplowanalytics.com/t/events-sent-to-snowplow-mini-not-appearing-in-any-index/991
https://snowplow.zendesk.com/agent/tickets/4800 (private)

We don't know if this is going to be an easy fix (maybe e.g. a char encoding glitch with Unix pipes), or is just random loss and will only be fixed when we do the NSQ re-write...

Pushing back...

Do we push back @BenFradet ?

Closing as we migrated from unix pipes a long time ago


Improve Logging on State Transitions

As someone trying to debug why a transition is not occurring, I'd like to see additional logging of what that state was that it was trying to transition from -> to in addition to the logs including which SM ID and event IDs are involved.  For example, unexpected event transition from open to re-open [....

@jwang-copilot @YulyTenorio @massenz
Looking to resolve this request through https://github.com/massenz/go-statemachine/pull/46
An example changes from this PR and from other recent commits:
2022/10/04 17:47:21 listener.go:59: Listener[DEBUG] Received request event_id:"39e176cd-4856-418f-a287-fee93d33029e" timestamp:{seconds:1664905641 nanos:744012453} transition:{event:"fail"} originator:"statemachine-lambda-client"
2022/10/04 17:47:21 listener.go:107: Listener[INFO] Event `fail` transitioned FSM [b9cad624-bf5a-43c5-8fb8-27bb64de18e8] to state `failed` from state `ingested` - updating store

Fields tracked:

State transitioned from (info)
SM ID (info)
Event ID (debug)


Unicode characters inside templates don't get encoded when plugin is enabled

When the plugin is enabled, Unicode characters that I can otherwise type as HTML codes do not get encoded into their corresponding characters in the final output.
This means, for instance, that typing © into my footer's Nunjucks partial template means that the character never gets converted to a copyright symbol. I have to explicitly type © into my template in order for it to work.
I'm running @11ty/eleventy ^0.11.0 and eleventy-plugin-images-responsiver ^1.5.3. I've detected the issue by turning all other plugins and imports inside my .eleventy.js file. I can reproduce the issue regardless of what is present in my images-responsiver config.
I've moved on to a custom shortcode component, but the relevant files are referred to in this commit: https://github.com/fabiomrbarbosa/fabiomrbarbosa.com/commit/caae0df2191181036c0fbaf9ebb80af31292f68f
Thanks in advance.

Hi @fabiomrbarbosa , I also had this issue but never thought it was because of this plugin. I'll investigate, thanks for the issue.

It looks like it could come from htmlparser2, which is used by BasicHTML, which is used by images-responsiver.
There was this issue: https://github.com/fb55/htmlparser2/issues/169
I'll try to build a reduced test case.

I'm having the same problem. I added a copyright to my website.
When the plugin is active, it becomes ©


[Feat:] Added Back to Home Page functionality

Description
Provided a back button functionality in the sidebar of dashboard, which redirects to home page
Method Adopted:

Modified the src/app/dashboard/page.jsx file.
Used a Link to direct from Dashboard page to Home page.
Used "ArrowBackIosNewRoundedIcon" from Material UI as a displaying Icon.

Here is how it looks now:

Also fix the merge conflicts


Added .vscode to gitignore for people editing with VSCode.

I use VSCode to program as I'm sure others do and noticed that the VSCode file wasn't ignored so I thought it would be helpful to add it.

Visual Studio Code is your editor of choice. There are several other editors out there. Everything that is not specific and directly related to the project should be kept in a separate, global ignore file, e.g. $HOME/.config/git/ignore. See man gitignore | grep -A5 'all situations':
   o   Patterns which a user wants Git to ignore in all situations (e.g.,
       backup or temporary files generated by the user's editor of choice)
       generally go into a file specified by core.excludesFile in the
       user's ~/.gitconfig. Its default value is
       $XDG_CONFIG_HOME/git/ignore. If $XDG_CONFIG_HOME is either not set
       or empty, $HOME/.config/git/ignore is used instead.

I understand, wont happen again


Version 0.0.1

Why only version 0.0.1 available ?
i can't add version 0.15.0
and with version 0.0.1 it's shown error like this

sorry Typo
it should be vue3-google-map and not 'vue3-google-maps'


typo: time > item

@crowcoder : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Decide on ' or " in cases where literal_eval is used

In most C-like languages ' and " have different semantics; '  being single character code / an int.
We should probably restrict usage to one.

@aothms @Ghesselink is this still valid or obsolete?


try conda pack

conda-pack will allow us to properly activate the environment instead of simply adding it to our $PATH

we could also consider switching to pixi and pixi-pack
https://github.com/Quantco/pixi-pack?tab=readme-ov-file#pixi-pack-unpack-unpacking-an-environment


Fix optional argument is listed as mandatory

[x] I've completed this task

I fixed a bug #76. The cause of the bug was that the option of ArgumentDefinition.Help was not correctly given only for init(name:, default:, parsing:, help:, transform:).
I added tests HelpGenerationTests and UsageGenerationTests.
Checklist

[x] I've added at least one test that validates that my change is working, if appropriate
[x] I've followed the code style of the rest of the project
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary

Looks great, @griffin-stewie — thanks!


Update macOS system requirements

Proposed changes

Emphasized the macOS requirements for clarity
Explicitly listed the current versions of macOS supported on Docker Desktop.

@nebuk89 Could you PTAL?


style: fixed broken UI when switch diff tabs with long list

Following the conversation at https://github.com/diiq/atomatigit/issues/90, i do understand there is fix in  https://github.com/diiq/atomatigit/pull/107 to make list scrollable which is very similar to the fix that @lordofthelake proposed.
But I still encounter the broken UI as stated in https://github.com/diiq/atomatigit/pull/107#issuecomment-164459716
But after incorporated and edit a little the fix that @lordofthelake proposed, the issue seems to be fixed, thus submitting a PR.
credits to @lordofthelake

:+1:


What is "MY_EXTERNAL_IP_RANGE"?

[Enter feedback here]
Reading through the "Restrict inbound traffic to specific IP ranges" section...
It would be nice to have a tiny explanation of the MY_EXTERNAL_IP_RANGE.  I first thought it might be a global or env variable, but I think it is probably just a placeholder for a written IP range... In that case, how should it be written/formatted in order for this to function properly.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 60a7a0a8-97e7-0fda-763c-1a9972f4e9bc
Version Independent ID: 82b46441-43fc-fe48-97e2-0f3fca3d6eab
Content: Use a Public Load Balancer - Azure Kubernetes Service
Content Source: articles/aks/load-balancer-standard.md
Service: container-service
GitHub Login: @palma21
Microsoft Alias: jpalma

@RyanMannix You are correct that this is a specified external range, it should be specified using CIDR notation.
Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@RyanMannix You are correct that this is a specified external range, it should be specified using CIDR notation.
Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@TravisCragg-MSFT I figured it out after a little trial and error.
The format that worked:
loadBalancerSourceRanges:      - "<IP_ADDRESS>/32"
Probably no need to update the docs, I'm just new to kubernetes
Feel free to close the issue.

@TravisCragg-MSFT I figured it out after a little trial and error.
The format that worked:
loadBalancerSourceRanges:      - "<IP_ADDRESS>/32"
Probably no need to update the docs, I'm just new to kubernetes
Feel free to close the issue.

@TravisCragg-MSFT Probably no need to change the docs.. I figured it out after a little trial and error.

@TravisCragg-MSFT Probably no need to change the docs.. I figured it out after a little trial and error.

@RyanMannix Sounds good, I will go ahead and close the issue.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@RyanMannix Sounds good, I will go ahead and close the issue.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


bash: llamafactory-cli: command not found

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
按照readme进行clone项目，以及pip install -r requirement.txt 安装依赖，执行到llamafactory-cli 出现bash: llamafactory-cli: command not found
Expected behavior
No response
System Info
No response
Others
No response

我也是

cd LLaMA-Factory
pip install -e .[metrics]重新安装依赖

cd 你路径下的\LLaMA-Factory
重新安装依赖
pip install -e .[metrics]


The ADD ID getting stacked down by the VC card

Describe the bug
The ADD ID button getting stacked down by the VID card
To Reproduce

Open the app
Authenticate it and enter to the home screen
Download more than four VC cards

Current behavior
The VC cards stacked one by one bellow and making the ADD id butto go to the bottom of the screen, to access the button it takes us to scroll down to access it
Expected behavior
as from the previous version of the app the ADD id button used to hover over the screen, so it wont get stacked down
Screenshots
https://user-images.githubusercontent.com/102220709/200237741-6c39ad60-b1ab-42fa-9faf-1b36a260fcaa.mp4
Smartphone (please complete the following information):
Device: android
MOSIP Version: 121
MOSIP server: qa-121.mosip.net
APK link : https://github.com/mosip/inji/actions/runs/3287270987

@kyanthony can we get Walid's input on this?

Tested on:
Android: QA4_io.mosip.residentapp-0.4.1-beta.1-10-d68f577-feature_mosip-55-face-matching-newlogic_20221115_1441.apk
STEPS

Open the app
In the right top click on language icon
select Arabic language

WORKS AS EXPECTED
@kyanthony @pmigueld @danicaerediano
Devices:
Samsung Galaxy A23 - Android 12
Screenshot
https://user-images.githubusercontent.com/102940764/202134692-d4607a9e-8960-4129-81aa-f47721a0fb29.mp4

Tested on:
Android: QA4_io.mosip.residentapp-0.4.1-rc1-newlogic_20221123_1723.apk
WORKS AS EXPECTED
@kyanthony @pmigueld @danicaerediano
Devices:
Samsung Galaxy A23 - Android 12
Screenshot
https://user-images.githubusercontent.com/102940764/203737335-a0a093a1-28d8-496d-99a4-9ab21976000b.mp4

verified this issue in qa5 env, the ADD ID is hovering over the screen, and working as expected, iam attaching the screen shot bellow, and closing this bug
screenshot :

Smartphone :

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


Installation Requirements should be mentioned

An important detail is missing in the comparison of the distribution modes:
The evergreen runtime requires elevation to install (not all users have those privileges), while the fixed version can be used for per-user installations.
It is also nowhere mentioned which distribution mode should be used for packaged apps.
Reference for "requires elevation": https://github.com/MicrosoftEdge/WebView2Samples/issues/145

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

ID: 8a805f15-55ba-4ef4-aeda-22188a040c66
Version Independent ID: 88de0955-526f-a4c9-efcc-d952281850e1
Content: Distribute your app and the WebView2 Runtime - Microsoft Edge Development
Content Source: microsoft-edge/webview2/concepts/distribution.md
Product: microsoft-edge
Technology: webview
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

AB#47625979

Thank you for your feedback. Adding to our backlog.

From my understanding, the Evergreen WebView2 runtime does not require elevation, and by default installs as user.
May I understand what you mean by:

It is also nowhere mentioned which distribution mode should be used for packaged apps.

From my understanding, the Evergreen WebView2 runtime does not require elevation, and by default installs as user.

Saying "Elevation" was probably a bit misleading. I meant privilege elevation from the perspective of a packaged and containerized app like UWP (and I think also WinUI3 without 'runFullTrust').  Those apps don't even run under the regular user account and are not able to install the WebView2 evergreen runtime.
That's the context of my sentence which you have quoted: How am I supposed to install the WebView2 runtime from a Windows Store app (which doesn't run with fullTrust) ?
This is nowhere mentioned, and from my understanding, the only possible way is to ship the app with a fixed runtime (which is what we are doing).

@victorhuangwq ping on the above response.


Zitadel - be able to run with localhost:8080 / local instance during development

Would it be possible to support this when running say in development?

Just change the redirect uri in the config and in the Zitadel UI.

Doesn't work currently because nuxt-oidc requires https:// and by default in the compose setup Zitadel uses http on :8080

You can just change it in the config there is no https requirement:

Just tested:

Hi, yeah but that not gonna work locally due to https://github.com/itpropro/nuxt-oidc-auth/blob/main/src/runtime/providers/zitadel.ts#L39 no ?

My screenshot is from my local machine in http ;)
The line you are referencing is just making sure that if you use token validation (which is only used when the audience is set correctly, it is turned on and it is possible to resolve the openid config) that the well-known config of the provides (Zitadel in our case) is requested from a https url.

Ah, ok thank you!
Am I assuming btw that I should be able to enable Zitadel config purely via ENV vars like having a empty
  oidc: {
    session: {
      // expirationCheck: true,
      automaticRefresh: true,
      expirationThreshold: 3600,
    },
    middleware: {
      globalMiddlewareEnabled: true,
      customLoginPage: false,
    },
    providers: {}
  },

and then providing the following in my .env or in a production built deployment?
NUXT_OIDC_PROVIDERS_ZITADEL_CLIENT_ID=282949336457872899@tradero
NUXT_OIDC_PROVIDERS_ZITADEL_CLIENT_SECRET=""
# NUXT_OIDC_PROVIDERS_ZITADEL_AUDIENCE=282949336457872899@tradero
NUXT_OIDC_PROVIDERS_ZITADEL_BASE_URL=http://localhost:8080
NUXT_OIDC_PROVIDERS_ZITADEL_REDIRECT_URI=http://localhost:3000/auth/zitadel/callback
NUXT_OIDC_PROVIDERS_ZITADEL_LOGOUT_REDIRECT_URI=http://google.com
NUXT_OIDC_PROVIDERS_ZITADEL_AUTHENTICATION_SCHEME=none

The way how nuxt work you still have to have all of the entries that you want to set via env variables in your nuxt.config.ts as empty strings, but the rest is correct.

Well then that is weird with what you say about audience because I don't have it set.

Well then that is weird with what you say about audience because I don't have it set.

If the clientId or the audience field is in the aud claim or the accessToken or idToken, token validation takes place. Zitadel doesn't hand out parseable accessTokens by default, so the accessToken is definitely not validated. The idToken is then validated, if the aud field of the idToken matches clientId or audience config if set.
So the only token that could be validated is the idToken.

I would close this issue for now, if you have a separate question or another issue feel free to open a new one for that.


bind visible inputs to change to allow 3rd party javascript libraries to update bindings

3rd party libraries that trigger a change in a textarea or text input field with
$("textarea").val("attr");

do not trigger the event ''input" on the field hence the volt binding does not update. This fix allows for libraries that provide functionality such as autocompletion to update volt bindings.
This fix would probably require refactoring and as such is just a proof of functionality.

If we do this, are we getting two events for each keypress?

Unfortunately two events do indeed fire for each keypress. One on the actual input and one when the field blurs. With my limited knowledge of javascript, I am not sure how to get around that problem. Could we use .watch! as is done for the hidden input fields?

While this is still an issue, I have closed this PR for the moment because of changes to the structure of my fork of volt.


Segment paging showing NaN

How are you running Flagsmith

[ ] Self Hosted with Docker
[ ] Self Hosted with Kubernetes
[X] Self Hosted at flagsmith.com
[ ] Some other way (add details in description below)

Describe the bug

Steps To Reproduce
Generate a lot of segments. Go through the paging.
Expected behavior
Paging shows correctly.
Screenshots

@kyle-ssg can you check all implementations of paging here please? I've noticed this is the case also on the Audit Log page.


ECS hover inset color looks wrong; buttons wrong style, should be links, text should be white

Inset color, text and spacing fixed.
Button needs to be separate issue.

Resolved with PR 389


Implementation of ReactiveStreams for RX/IX

Hi,
Does anyone know of an implementation of ReactiveStreams for RX/IX?
Any help appreciated.
Cheers
Sean.

@seanFarrow not that I know, RX.java has implemented the Reactive Streams interfaces but RX.net has not.
See also https://github.com/Reactive-Extensions/Rx.NET/issues/19

I have started Reactor-Core.NET a while back which does use RS.NET and is Rx style. It's incomplete as I have now other demanding responsibilities.

Should we look at doing something. I’m happy to take this on, as I’m using both RX and Akka.streams.
If someone could create a repo, then I can work on this.
Cheers
Sean.
From: David Karnok<EMAIL_ADDRESS>Sent: 21 September 2016 08:20
To: reactive-streams/reactive-streams-dotnet<EMAIL_ADDRESS>Cc: Sean Farrow<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [reactive-streams/reactive-streams-dotnet] Implementation of ReactiveStreams for RX/IX (#24)
I have started Reactor-Core.NEThttps://github.com/reactor/reactor-core-dotnet a while back which does use RS.NET and is Rx style. It's incomplete as I have now other demanding responsibilities.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/reactive-streams/reactive-streams-dotnet/issues/24#issuecomment-248531317, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABY1fk62k-MUy1ucYuA0nqRBr863_e9gks5qsNqMgaJpZM4J_hGB.

I’m happy to take this on

What exactly would you like to do? RS requires a different architecture and I don't think you can change Rx.NET gradually to RS. (I have started porting RxJava even longer ago but that died off as well and uses 3rd generation architecture only.)

I’m looking to implement the ReactiveStreams interfaces for RX.net.
From: David Karnok<EMAIL_ADDRESS>Sent: 21 September 2016 08:47
To: reactive-streams/reactive-streams-dotnet<EMAIL_ADDRESS>Cc: Sean Farrow<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [reactive-streams/reactive-streams-dotnet] Implementation of ReactiveStreams for RX/IX (#24)
I’m happy to take this on
What exactly would you like to do? RS requires a different architecture and I don't think you can change Rx.NET gradually to RS. (I have started porting RxJavahttps://github.com/akarnokd/RxAdvancedFlow even longer ago but that died off as well and uses 3rd generation architecture only.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/reactive-streams/reactive-streams-dotnet/issues/24#issuecomment-248536672, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABY1foWaD4HLzDJigoooSFg_IHLEzuV0ks5qsODcgaJpZM4J_hGB.

Why not contributing to Reactor Core .NET, we're looking for core contributors. We already are struggling with some semantic discussions between Rx and Reactor, however it seems audiences increasingly agree on an explicit separation between backpressure driven flows as implemented by Flux (/Mono) and non backpressure flows as implemented by Observable etc.

Hi,
Could you provide a link to the repo?
Cheers
Sean.
From: Stephane Maldini<EMAIL_ADDRESS>Sent: 21 September 2016 12:47
To: reactive-streams/reactive-streams-dotnet<EMAIL_ADDRESS>Cc: Sean Farrow<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [reactive-streams/reactive-streams-dotnet] Implementation of ReactiveStreams for RX/IX (#24)
Why not contributing to Reactor Core .NET, we're looking for core contributors. We already are struggling with some semantic discussions between Rx and Reactor, however it seems audiences increasingly agree on an explicit separation between backpressure driven flows as implemented by Flux (/Mono) and non backpressure flows as implemented by Observable etc.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/reactive-streams/reactive-streams-dotnet/issues/24#issuecomment-248587837, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABY1fqQyPCJPO7aW_x7thcSQAQMUPte4ks5qsRksgaJpZM4J_hGB.

https://github.com/reactor/reactor-core-dotnet

Hi,
Thanks, what problems/discussions are you having currently?
I’m trying to work out how I could contribute.
From: Stephane Maldini<EMAIL_ADDRESS>Sent: 21 September 2016 12:49
To: reactive-streams/reactive-streams-dotnet<EMAIL_ADDRESS>Cc: Sean Farrow<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [reactive-streams/reactive-streams-dotnet] Implementation of ReactiveStreams for RX/IX (#24)
https://github.com/reactor/reactor-core-dotnet
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/reactive-streams/reactive-streams-dotnet/issues/24#issuecomment-248588382, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABY1fvMAGbskV4ayELIzhGzgHdPIOn8dks5qsRnJgaJpZM4J_hGB.

@SeanFarrow the lack of time. What we want and how we want it is already set, the utility/tool support is also there. We'd like to have these added:
https://github.com/reactor/reactor-core-dotnet/blob/master/Reactor.Core/Mono.cs#L67
~50% of the operators are missing.
https://github.com/reactor/reactor-core-dotnet/blob/master/Reactor.Core/Flux.cs#L2881
~10% of the operators are missing.
They can be adapted from https://github.com/reactor/reactor-core/blob/master/src/main/java/reactor/core/publisher/Flux.java and https://github.com/reactor/reactor-core/blob/master/src/main/java/reactor/core/publisher/Mono.java mostly in a straightforward manner.

Ok, let me take a look and see what I can do!
Are there any examples of putting this together with rx?
From: David Karnok<EMAIL_ADDRESS>Sent: 21 September 2016 13:20
To: reactive-streams/reactive-streams-dotnet<EMAIL_ADDRESS>Cc: Sean Farrow<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [reactive-streams/reactive-streams-dotnet] Implementation of ReactiveStreams for RX/IX (#24)
@SeanFarrowhttps://github.com/SeanFarrow the lack of time. What we want and how we want it is already set, the utility/tool support is also there. We'd like to have these added:
https://github.com/reactor/reactor-core-dotnet/blob/master/Reactor.Core/Mono.cs#L67
~50% of the operators are missing.
https://github.com/reactor/reactor-core-dotnet/blob/master/Reactor.Core/Flux.cs#L2881
~10% of the operators are missing.
They can be adapted from https://github.com/reactor/reactor-core/blob/master/src/main/java/reactor/core/publisher/Flux.java and https://github.com/reactor/reactor-core/blob/master/src/main/java/reactor/core/publisher/Mono.java mostly in a straightforward manner.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/reactive-streams/reactive-streams-dotnet/issues/24#issuecomment-248594588, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABY1fuqJHcxiNnKfrrn0s3Cskwkt1troks5qsSDcgaJpZM4J_hGB.

Given an IObservable we have a ToFlux extension method to get in and ToObservable to get out. We don't depend on Rx.NET but only on standard C# IObservable and IObserver.

Ok, and is it possible to convert to a ReactiveStreams publisher? That’s what I’m after!
From: David Karnok<EMAIL_ADDRESS>Sent: 21 September 2016 14:13
To: reactive-streams/reactive-streams-dotnet<EMAIL_ADDRESS>Cc: Sean Farrow<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [reactive-streams/reactive-streams-dotnet] Implementation of ReactiveStreams for RX/IX (#24)
Given an IObservable we have a ToFlux extension method to get in and ToObservable to get out. We don't depend on Rx.NET but only on standard C# IObservable and IObserver.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/reactive-streams/reactive-streams-dotnet/issues/24#issuecomment-248608114, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABY1fpXiHvPoL3mFEwORGCTWSqxyI3yoks5qsS1tgaJpZM4J_hGB.

IFlux is an IPublisher and you can use Wrap to make an arbitrary IPublisher into an IFlux. We use this to avoid bloating the extension method space of IPublisher.


Better Handing of HD Public Keys in Multisig TXs

Add smart understanding of xpubs in PSBTs, so that output descriptors are not required to sign when these are properly supplied. Update multiwallet with this UX.
Modularize input and output description in PSBT class so that this can be applied to other script types (especially p2sh) in the future
(re)generalize from_hd_pub and from_hd_priv path handling (don't falsely specify root_path when xpub use-cases are base_path
Helper method to create a PSBT with hd_pubs and other metadata
More test coverage
Minor cleanup of previous codebase

This fixes https://github.com/buidl-bitcoin/buidl-python/issues/42


iOS server selection isn't saving

The iOS app is working great for me but there is one issue I've noticed. I’m able to deselect “select server automatically” and connect to my own server, but it isn’t saving the server. When I re-opened the app it had "select server automatically" re-checked and a different server selected. Am I missing a “save” button or something? My server is sometimes listed under “connected node, height” but “select server automatically” was back to enabled and a different wallet server was selected.

Yeah this is likely an oversight on my part when implementing the UI.
I'll work on this as it's definitely a bug.  I'll either add a save button or implicitly save the server upon editing (probably the more likely approach is the latter).
Sorry!  Thanks for opening the issue by the way!

Hey thanks for reporting this! I have fixed this bug.  I think I may submit an update to Apple sometime this week so perhaps next week at the latest you'll see it pushed "live".
Thanks again!


🌱 Add kubesec for IPAM

Kubesec is a security risk analysis tool for Kubernetes manifest files and this PR is going to add a github action script to IPAM. All the manifest files is going to be scanned by Kubesec once in a week.
Github action test in Nordix
Test result in Nordix

/test unit

/test-v1b1-ubuntu-integration

/hold

/retitle 🌱 Add kubesec for IPAM
/hold cancel

/test-v1b1-ubuntu-integration

/lgtm

/test all

/test unit

/hold

/hold cancel

/hold

/hold cancel

Just for testing, to check how prow reacts after removing hold

/hold


Disable data sampling on demande via the UI

Disable data sampling on demande via the UI
description:

In some cases, such as data sources known to be bad candidate for data quality assessment, it might be require to disable on demande the data sampling feature
For this purpose, provide a UI based feature to achieve this on demand

In version 1.2.25.


Problems with paths and dependencies

In my effort to design a collection phase for the compiler I've been trying to sort out exactly how dependencies and name-resolution work in sway. The current design has some issues with things being either broken or under-specified, in no particular order:

A package's transitive dependencies appear in the namespace while the package is being compiled. eg. If my library lists a dependency on dependency_library in its Forc.toml, and depency_library lists a dependency on transitive_dependency_library in its Forc.toml then library will be allowed to refer to transitive_dependency_library in its code. This breaks encapsulation by exposing the implementation details of dependency_library to library.

It's possible to have two dependencies which depend on two different transitive dependencies with the same name (but different file-system location). Rather than causing an error this currently makes the compiler very confused.

The name of a library, as specified in the library name; line of its root module, doesn't have to match the name specified in the package's Forc.toml. When the package is used as a dependency it seems that only the name given in Forc.toml is actually used. The other name is completely ignored - except that if it's omitted (eg. because the package is a script) then the compiler will crash.

Similarly but oppositely, the name of a submodule dependency given in a dep some_submodule; declaration doesn't have to match the name that the submodule gives itself. In this case the name that the submodule gives itself in its library name; line is used as the submodule's name whereas the name given in the dep declaration is only used to determine the file name.

Two submodules can have the name same but different file names. They can both be imported along-side each other, causing weird name-resolution issues.

A submodule of a package's root module can have the same name as a dependency without causing an error and with no way to disambiguate between them.

Given all of these above, these are the fixes that I'd like to see:

A package's transitive dependencies should not be visible to that package. This fixes (1) and (2).

dep declarations should be replaced with mod declarations a.la. rust. The name given in the declaration should act as the canonical name of the module and should also be used to find its source file/directory. This fixes (5).

Rather than a source file specifying what type of program it is with a script/library foo/contract/predicate line this should be specified in Forc.toml. This fixes (3) and (4).

To fix (6), we should decide on some concrete rules for how name resolution works - ie. how to refer to things from external libraries, from under the root of the current package, and from under the current module. We shouldn't repeat rust's mistakes of having to change the rules in a later edition, having the rules work differently between use statements and everywhere else, and having the syntax be ambiguous with special rules that are only invoked to resolve ambiguous edge-cases. My proposed rules are: paths beginning with :: must be followed by the name of a library. You'd refer to foo in the std library by writing ::std::foo, if you want to just write std::foo then you'd need to have a use ::std declaration in scope. You'd refer to foo under the root of the current package by writing ::name_of_this_package::foo, not ::foo (since ::foo would refer to an external library named foo). Optionally, we could also allow ::self to refer to the current package, similar to how rust uses crate for this purpose. Paths that don't start with :: are always relative to the current module, making paths beginning with self unnecessary. Paths should work exactly the same in use declarations as they do everywhere else.

What are other people's thoughts on all this? @sezna, @otrho, @emilyaherbert, @mitchmindtree?

Yep, these are obviously issues which need to be resolved properly ASAP.  I think all your proposals sound great.

Not sure why Sway uses dep instead of mod, @sezna can answer that.
Putting script or predicate or contract or library in Forc.toml makes sense I think.  But only if the current dep -> library thing is dropped and replaced with a mod equivalent.  But that's what you're suggesting.
So using ::crate doesn't fit (Sway isn't Rust) but I don't really like ::self much.  We could use ::pkg or perhaps as you suggest only allow the explicit package name.

A package's transitive dependencies should not be visible to that package. This fixes (1) and (2).

Good catch! I think this is just a bug in forc-pkg. I've opened #2136 which should address points 1 and 2.
I totally concur with everything else!


Store the Docker tags as build artifacts?

This seems like a good idea, as there is no easy way in Docker to find all the tags for an image.

Would just be a case of writing the Docker tags to a file at the same point as tagging them, and then using CircleCI’s store artifacts functionality.

Closed by #18


Playground eval

Playground "Run" button now supports:

Running code like 1+2 now outputs [EVAL] 3, without the need for wrapping in console.log.  Like most REPLs, outputs nothing if the eval'd value is undefined.
console.error, console.warn, etc. (prefixing output with [ERROR] , [WARN] , etc.). console.timeEnd etc. don't currently work well (but they still work in the actual console); this could be fixed later.
console commands now work from async functions too (fixes #938). This is done by modifying the output AST to replace all console identifiers with civetconsole. (Note: if you run async code twice in rapid succession, you may get output from the first run in the second run.)
Top-level await, by wrapping the Civet code in an async do block (or do -> block in CoffeeScript mode). I did this at the Civet level so that we still get implicit return of the last value, which (once resolved) gets printed with [EVAL] prefix like the sync case.

Ugh, yarn test fails with my attempt to export lib from the main Civet bundle. The issue is:

parser.hera and lib.civet are in CJS. I think this is necessary until Hera is fully ESM?
main.civet is ESM, and wants to export from lib.civet

esbuild can build these fine, but running in actual ESM mode as in yarn test fails, because ESM isn't supposed to import CJS. Any ideas for a workaround?

I think this branch in Hera is maybe 90% of the way to emitting ESM. I recall leaving it though because I encountered an issue with the latest NPM loaders API and I couldn't find a way to actually get Node to load ESM using the new API. Maybe the examples and documentation are better now and I can continue it soon.

#942 fixed the build here! So this is ready for review.


Bug with Get-InstalledScript and Uninstall-Script

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest version
[X] Search the existing issues.

Steps to reproduce
Install-Script -Name Install-VSCode -Repository PSGallery -NoPathUpdate -Scope AllUsers -verbose -debug
will install at AllUsers scope (as expected).
Expected behavior
Get-InstalledScript Install-VSCode
should be able to find the script installed.  PowerShellGet 2.2.5 functionality checks AllUsers first and if it's not there, checks CurrentUser scope.
Actual behavior
Get-InstalledScript
does not have a -Scope parameter so it does not pass a -Scope parameter onto Get-InstalledPSResource.   Get-InstalledPSResource just uses the default value of CurrentUser and never checks AllUsers.
Note:  This is also happening with Get-InstalledModule, Uninstall-Script, and Uninstall-Module.
One proposal is to have PSResourceGet's ScopeType be a flagged enum so that multiple values can be passed (this would be a breaking change in PSResourceGet).
Version
3.0.23-beta23

The plan is to adapt PSResourceGet so that it checks first currentuser scope and then alluser scope by default


Use Bayesian Optimization for faster Hyper Parameter Optimization

Grid search for hyperparameter optimization can take too long as every combination of parameters is being tested, even the ones that are obviously bad. Using Bayesian Optimization (BO), we could speed up the search because BO can find the best combination using multiple intelligent steps, exploring the search space.
Refs:

https://machinelearningmastery.com/scikit-optimize-for-hyperparameter-tuning-in-machine-learning/
http://hyperopt.github.io/hyperopt/

Use Optuna: https://optuna.readthedocs.io/en/stable/index.html


Support Video Capturing?

Great work! But where's video capturing? I'm missing it or you've not added yet. :)

=(

+1


--reuse-window and OS X fullscreen window switching

VSCode Version: 1.1.1
OS Version: OS X 10.11.5 (15F34)

Steps to Reproduce:

Open VSCode and Terminal
Make them both fullscreen (either split screen or same screen)
In the fullscreen Terminal, run code --reuse-window
Observe that OS X snaps you back to the original Desktop that you opened the two apps in, then back to VSCode.

I can reproduce. Weird!

My feeling is that this happens because the main process of VS Code is associated with the first desktop and the second window talks to the main process to bring up the window. I am not sure how to change this.

@mwillsey I think I cannot reproduce this anymore with 1.8. Can you try?

@bpasero Me neither. I guess it got fixed!


nuScenes-Based Lidar Examples

What
Adds two new examples based on the nuScenes dataset. One example visualizing most of the data (lidar, radar, camera, and 3D bounding boxes) in the nuScenes dataset, and a more minimal example that only visualizes the lidar data in the lidar reference frame.
Lidar example

nuScenes example

The download module is included in both examples for now; if we have a place so this could be shared across examples, that would be nice.
Closes #2510.
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested app.rerun.io (if applicable)
[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG

PR Build Summary
Docs preview
Examples preview
Recent benchmark results
Wasm size tracking

Actually, one more question after playing with it: is the nuScenes dataset missing an annotation context?

I just tried adding an annotation context, but the labels are taking a lot of space. Is there any benefit in having the annotation context with just the ids?

Hm, that's strange. I guess pip install nuscenes-devkit fails for you?

Oh yeah, I was pip installing the wrong requirements.txt file 🤦  Now it seems to be working (downloading dataset…)


How difficult is to make vue-router-next agnostic, and used with React?

What problem does this feature solve?
I don't know if this is the best place. Sorry, but...
Looking at the code, I saw that there are few points of coupling with Vue.

dependency with @vue/devtools-api
router-link and router-view component
called app.use(router)

Are these just touch points really? How complex is it for me to open a fork to adapt for react?
What does the proposed API look like?
Not applicable

I found more links to the vue in the file src/router.ts.  It looks complex.

It's doable and I started a project on the side to provide an agnostic client-side router based on vue-router but it will take time. Feel free to fork it and try yourself 🙂

@posva, is this side project open? Can you share the link?
Anyway, I will try to start a fork project. Tks!

It's not open yet


path to data directory

Hi, thanks for sharing.
I am quite confused at the path to the directory of AMR dataset. During data preprocessing, I put my data under gtos/generator_data/data/AMR/LDC2017T10, things go well. However, during, Vocab & Data Preparation, I need to move the data directory to gtos/data/AMR/... to make sh prepare.sh work.
Below is the log when I execute sh prepare.sh under gtos/generator.
I am running experiments on a 4*P100(16G per GPU) server. By default, --train_batch_size 66666 and --dev_batch_size 44444. Is unusual to see such a large batch size, is this a random number or the correct parameter to reproduce the result?

@sysu-zjw Hi, thanks for your interests.
The printed log seems good! Actually, you can also choose to change the dataset path config in prepare.sh rather than moving the data.
https://github.com/jcyk/gtos/blob/e81578b7cf90dbab1b5d9a70e0c3cd00ab14a6e0/generator/prepare.sh#L1
The default settings in this repo should reproduce our experiments. For the unusual batch size, it is counting sum(n+m^2), where n is the number of tokens and m is the number of nodes. Therefore, it seems quite large. You can learn more details in data.py, more specifically, in the following line. https://github.com/jcyk/gtos/blob/e81578b7cf90dbab1b5d9a70e0c3cd00ab14a6e0/generator/data.py#L300

Hi, @jcyk, thx for your reply!
It seems I cannot train such batch size on my device (16G GPU * 4). May I ask what kind of device you are training with.

hi @sysu-zjw
Our AMR experiments are conducted using a single GPU with 24GB memory.
I think you may need to adjust the batch size to your devices. A hint for you is that in the master GPU needs more space when using multiple GPUs.

also, I recall that the maximum memory cost is much larger than the average cost. a possible workaround is to identify and split those very memory-consuming batches.

let me know if you have further questions.


Koa2: setting ctx.status prior to ctx.redirect call does not yield any effect.

Is this a bug?
per documentation https://github.com/koajs/koa/blob/v2.x/docs/api/response.md#responseredirecturl-alt
if i set ctx.status prior ctx.redirect call my response status should be set to what ever is i specify it to, but this does not happen in my case. I have middle wear that redirects visitor for any path that does not fall under exception if they do not have authenticated session.
        if (authRequired && !ctx.isAuthenticated()) {
            ctx.status = authConfig.authRequiredRedirect.statusCode;
            ctx.redirect(authConfig.authRequiredRedirect.path);
        }

Am I missing something?

do you mind making a PR with a failing test case?

I will after i test few more things.

@jonathanong is there a guide how collaboration should be done?
or this is good enough?

yes, this a great example of making a failing test-case :)
however, i'm not sure why you're using redirect() to set a non-redirect status code. you might as well just set the location header

Maybe i am doing this wrong. Possibly my understanding of this subject is off. let me consult web for a bit.
Original goal was to respond to none-auth user with 401 and redirect them to login page.

yeah, i am doing it wrong. i will close both : ) thanks for help


The directory HAIS/lib/spconv does not exists in the repo.

Hi, thanks for uploading the code. I am following your guideline to install spconv. At this step, The directory HAIS/lib/spconv` in the directory does not exist. Is the spconv version used in HAIS the same as in PointGroup (spconv @ 740a5b7)
cd HAIS/lib/spconv python setup.py bdist_wheel

Yes, I included --recursive when I clonning the code but spconv was not downloaded.
I cloned PointGroup repo and --recursive works with PointGroup repo but not with Hais repo. Maybe link to spconv was not correctly pushed in Hais repo?