Migration error | Cannot add foreign key

Expected Behavior
php artisan orchid:install
Actual Behavior
Illuminate\Database\QueryException  : SQLSTATE[HY000]: General error: 1215 Cannot add foreign key constraint (SQL: alter table role_users add constraint role_users_user_id_foreign foreign key (user_id) references users (id) on delete cascade on update cascade)
Steps to Reproduce the Problem

php artisan orchid:install

Specifications

ORCHID Version: 5.8
Laravel Version: 3.11
PHP Version: 7.2
Database Version: MySql 5.7

Other Notes

Are you using InnoDB? We do not set this as default, please check it with you.

Thanks for the information, so i need to downgrade my Mysql version because the use of InnoDB is mandatory since MySQL 5.7

why lower? After all, innodb is recommended since version 5.1, perhaps by default you are using myislam and you just need to change this value.

I am using InnoDB but you said it's not set by default.
How can i change that ?

Create a new base as follows:
CREATE DATABASE "orchid" ENGINE = InnoDB

All my tables are already using InnoDB engine

Laravel 5.8 is using Big INT for user id column.
change the default create users table migration from
$table->bigIncrements('id'); to $table->increments('id');

Thank @mtsunu
I use "sqlite" for development, so I did not receive such errors. :anguished:
Got the same error when trying to upgrade to production.
Has released a corrective release.

Ah. I had created my users table a long time ago, so users.id was still just an unsigned int.


More sanity checking for --delta restores

The --delta restore can act like an rm -rf if used incorrectly.  There needs to be more sanity checking to make it safer:

Make sure that PG_VERSION exists in the correct place
Use backup.manifest as a marker file to indicate that a restore was started in case it is aborted while PG_VERSION is be rewritten
Make sure the tablespace dir has the correct name if doing a delta (if it does not exist it's OK to write it since this does not delete any files).

This should be combined with issue #19 since it will overlap heavily on 3.

3 was fixed in issue #19.


Automatically deploy mock server based on API specification.

I found a tool that can mock our API based on the specification.
This tool is called Prism. The only problem is that it uses NodeJs so I cannot upload it to our mock server (I think). If we could make it that our CI/CD pipeline would pick up changes to the specification file and run Prism to host that would make our lives a lot easier.
Quick start
Installation
Prism requires NodeJS >= 16 to properly work.
npm install -g @stoplight/prism-cli

# OR

yarn global add @stoplight/prism-cli

How to start mocking
Prism can help you create a fake "mock" based off an OpenAPI document, which helps people see how your API will work before you even have it built. Run it locally with the prism mock command to run your API on a HTTP server you can interact with.
prism mock https://raw.githubusercontent.com/SandorBalazsHU/elte-ik-msc-weather-cam/main/Documentation/WebApi/weather-cam-web-api.yaml

Learn more about how the mock server works.

While you guys sort this out I implemented the basic mock PHP endpoints and uploaded them to the server under mock.weather for a start. (related PR #12)
You can find more info here in the API wiki.

vm_weather@weather-cicd:~$ prism mock https://raw.githubusercontent.com/SandorBalazsHU/elte-ik-msc-weather-cam/main/Documentation/WebApi/weather-cam-web-api.yaml
Error: Cannot find module 'worker_threads'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:636:15)
at Function.Module._load (internal/modules/cjs/loader.js:562:25)
at Module.require (internal/modules/cjs/loader.js:692:17)
at require (internal/modules/cjs/helpers.js:25:18)
at Object.call (/usr/local/lib/node_modules/@stoplight/prism-cli/node_modules/micri/webpack:/micri/external node-commonjs "worker_threads":1:1)
at webpack_require (/usr/local/lib/node_modules/@stoplight/prism-cli/node_modules/micri/webpack:/micri/webpack/bootstrap:21:1)
at Object.call (/usr/local/lib/node_modules/@stoplight/prism-cli/node_modules/micri/webpack:/micri/src/with-worker.ts:8:1)
at webpack_require (/usr/local/lib/node_modules/@stoplight/prism-cli/node_modules/micri/webpack:/micri/webpack/bootstrap:21:1)
at Object.call (/usr/local/lib/node_modules/@stoplight/prism-cli/node_modules/micri/webpack:/micri/src/index.ts:36:1)
at nccwpck_require (/usr/local/lib/node_modules/@stoplight/prism-cli/node_modules/micri/webpack:/micri/webpack/bootstrap:21:1)


Allows ios extension to be built if using nme rebuild ios

The .a files weren't building for me when using the standard nme rebuild ios inside the project folder. Please test that after taking the PR that Openfl way to generate ios .a files still works.

Thanks @jgranick


Improve startup text of PDS and SecHub

Problem:
There is currently no way of knowing which default parameters for the PDS solutions are set and what the exact default value is.
Solution:
When starting a PDS solution it would be nice if all default parameters and their values would be shown.
Bonus:
It would be cool if there was some sort of PDS logo in ASCII art that will be shown on boot :)

A good idea, but I think the default parameters should be only logged at debug level. So available at development time.
And the logo ... nice ;-)

We will do

[ ] Add custom banner for SecHub server
[ ] Add custom banner for PDS
[ ] Each banner will show the application version and also the spring boot version
[ ] Show up PDS defaults (cannot be sensitive data) inside logs when PDS is started


Fix env var in Travis CI configuration to run CS check

@webimpress
Thanks for your contribution! 👍


Setting crossOrigin: "Anonymous" attribute on Images causes CORS issues on some platforms

We're testing a Lightning application on the PS4 and have run into an issue where images fail to load from some domains. Our image endpoints responses include the required Access-Control-Allow-Origin: * header for CORS but for some reason the PS4 browser still fails to load the images. We do not see this issue when running in any other browser, so we do not believe it's any configuration issue. It's just the PS4 browser security behaviour is unusual and unfortunately we don't have control over this behaviour.
We have verified that removing the crossOrigin attribute on images resolves the issue on the PS4 and so we're proposing that we add a stage option to disable adding this attribute for these problematic platforms. I will PR the changes.

Fixed with #231.


(fix) Upated YAML's in deploy folder

Signed-off-by: Rahul M Chheda<EMAIL_ADDRESS>
What this PR does / why we need it:
- Fixed approriate YAML's in deploy folder

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
- https://github.com/litmuschaos/litmus/issues/1393
Special notes for your reviewer:
Checklist:

[ ] Fixes #
[ ] Labelled this PR & related issue with documentation tag
[ ] PR messages has document related information
[ ] Labelled this PR & related issue with breaking-changes tag
[ ] PR messages has breaking changes related information
[ ] Labelled this PR & related issue with requires-upgrade tag
[ ] PR messages has upgrade related information
[ ] Commit has unit tests
[ ] Commit has integration tests

@rahulchheda can we sanitize the deploy folder:

Add the corresponding/mapped example in chaosexperiment CR ? i.e., the existing one seems to be pod-delete. We can either choose to go w/ pod-delete for both exp/engine or network-loss for both.
Probably worth including a sample rbac yaml
A README explaining very briefly on what is the recommended order of yaml application followed by a reference to litmus-docs for more information --- would complete the set?

An alternative approach is to keep CRDs here and just put out a readme asking users to refer litmus-docs :)

It would be great to index the deploy files for quick user evaluation. Say:
01-chaos-operator.yaml (w/ latest or ci tags)
02-nginx-deploy.yaml (an annotated one w/ litmuschaos.io.chaos: true)
03-pod-delete-chaosexperiment.yaml
04-pod-delete-rbac.yaml
05-chaosengine.yaml (w/o overrides, simplistic version)
The readme can include application of these and steps to view results/engine status


feat(attributes): add attributes for span and metrics

What type of PR is this?

feat:  add attributes for span and metrics
What this PR does / why we need it (English/Chinese):

Which issue(s) this PR fixes:

@CoderPoet 记得更新文档


stage1: app add, status didn't work with empty vols

Fixes #3398

@euank do you mind a pass on this PR and a check if there are some leftover issues at runtime?

semaphore failure is a flake

corrected, @euank PTAL!

LGTM


chore: support Windows CSE

Changes included in this PR:

Windows custom scripts

copied from the latest aks-release-v0.47.0-1 of aks-engine to parts/windows directory
changed WrapAsVariable resolved by ARM template to the GetParameter by go-template in agentbaker, this change is on only parts/windows/kuberneteswindowsfunctions.ps1 and parts/windows/csecmd.ps1
give up the use of parts/windows/csecmd.ps1 for now
for the other windows  custom scripts, they are mainly a copy-and-paste work, as they expose as function and the dynamic input variables is defined in parts/windows/kuberneteswindowsfunctions.ps1

pkg

support fetching Windows CSE command
add windows agentpool required functions in functionmap for later go-template parsing.

test

added UT tests in aks-engine master but lost in aks-engine aks branch
Added tests to generate testdata for Windows scenarios, these test data are generated and the compared with the result from the ones in the real cluster.  some of the variables like tenenatid is not real, for sharing across the tests and comparision.

AKSWindows2019+K8S116
AKSWindows2019+K8S117
AKSWindows2019+K8S118
AKSWindows2019+K8S119
AKSWindows2019+K8S119+CSI
AKSWindows2019+ManagedIdentity
AKSWindows2004+K8S119+hyperv
AKSWindows2019+CustomCloud
AKSWindows2019+CustomVnet
AKSWindows2019+EnablePrivateClusterHostsConfigAgent

ping @AbelHu for another look.

ping @AbelHu and @xuto2  for another look, thanks.


Test on images outside SUNRGBD dataset

Hey, I test the model and it works well in the SUNRGBD dataset! Thank you for sharing it. Could you please give any hint about how to apply the model to images outside SUNRGBD dataset? Should I generate a pickle file following sunrgbd_process.py? Do we need any input other than the RGB image, e.g., the camera intrinsics?

The inputs are only an RGB image and camera intrinsic matrix.
You should first detect 2D bounding boxes and feed the image patches and original image into LON and GGN to estimate the local objects and global geometry with attributes.
We may release the inference code in the future.

Hi, thank you for sharing this, it is a great project. Do you have any update on whether you will share the inference code?
Thanks


Add second API + graph deployment to cloud server with OpenNeuro currently

On graph-db-tester:

[X] Create new directory ~/projects_test containing some OpenNeuro data and a copy of the API repo
[X] Create second Stardog root containing license: ~/stardog_root_test
[X] Create new DB: test_data, uses default admin credentials
[x] Database should hold first 10 datasets from https://github.com/neurobagel/openneuro-annotations:ds000001.jsonld
ds000002.jsonld
ds000003.jsonld
ds000005.jsonld
ds000006.jsonld
ds000007.jsonld
ds000008.jsonld
ds000009.jsonld
ds000011.jsonld
ds000017.jsonld

[ ] ~Check if NGINX needs to be updated~

[x] No, we can just add a new OpenStack security group rule (Federation test rules) for the instance that allows traffic to the port we're exposing the API on

[x] Check if API can be accessed using IP + port: http://<IP_ADDRESS>:8888/

For reviewer: You can send a request to the IP address of the machine or check the stack running on the machine.

I have reviewed the new API and have confirmed that it is running beautifully 🎉 👍 🧑🍳


[v3.0.0] Server fails to start on node 5

I tried installing and running npm start with node 5 on Ubuntu 14.0.4 and got the following error
michael@charon
 ~/code/react-boilerplate> npm start

><EMAIL_ADDRESS>start /home/michael/code/react-boilerplate
> npm run start:tunnel

><EMAIL_ADDRESS>start:tunnel /home/michael/code/react-boilerplate
> cross-env NODE_ENV=development node server

module.js:341
    throw err;
    ^

Error: Cannot find module '../features'
    at Function.Module._resolveFilename (module.js:339:15)
    at Function.Module._load (module.js:290:25)
    at Module.require (module.js:367:17)
    at require (internal/module.js:16:19)
    at Object.<anonymous> (/home/michael/code/react-boilerplate/node_modules/caniuse-api/dist/index.js:19:32)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Module.require (module.js:367:17)
    at require (internal/module.js:16:19)
    at Object.<anonymous> (/home/michael/code/react-boilerplate/node_modules/postcss-cssnext/lib/index.js:11:19)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)

npm ERR! Linux 3.13.0-79-generic
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "start:tunnel"
npm ERR! node v5.10.1
npm ERR! npm  v3.8.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start:tunnel: `cross-env NODE_ENV=development node server`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start:tunnel script 'cross-env NODE_ENV=development node server'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the reactjs-boilerplate package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     cross-env NODE_ENV=development node server
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs reactjs-boilerplate
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls reactjs-boilerplate
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/michael/code/react-boilerplate/npm-debug.log

npm ERR! Linux 3.13.0-79-generic
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "start"
npm ERR! node v5.10.1
npm ERR! npm  v3.8.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `npm run start:tunnel`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'npm run start:tunnel'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the reactjs-boilerplate package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run start:tunnel
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs reactjs-boilerplate
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls reactjs-boilerplate
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/michael/code/react-boilerplate/npm-debug.log

If I swap to node v4.4.2 (npm 2.15.0) all is well

Ok, something else fishy might be happening here. It seems that the initial install fails no matter what version I'm on, but will succeed after an npm rebuild

I'm going to close this for now as I'm 90% sure it's an issue with my setup. I'll reopen if I find out otherwise.

Nice, thanks so much!

I see the exact same thing on windows, node v5.11.0.  npm rebuild does not solve it for me.

Me too. on osx 10.11.x

@morajabi Error: listen EADDRINUSE :::3000 tells that you already have process on 3000 port.
Maybe this link will help you
http://stackoverflow.com/questions/4075287/node-express-eaddrinuse-address-already-in-use-kill-server


Raw order picking config

requires https://github.com/ipa320/cob_common/pull/207
Actual configuration of RAW with UR arm

This has a merge conflict...as we did some bigger restructuring in cob_robots...please merge with latest cob_robots/indigo_dev, adjust accordingly and test updated branch on robot..

Your merge didn't go well....there is file which are already merged into ipa320/indigo_dev and you commited a lot of temp files from merge conflicts, e.g. *.orig or .BACKUP., .BASE., .LOCAL.

The masses of the UR10 arm should be reduced in order to get a stable simulation. The robot configuration  is now fine.

Has this been tested again with hardware?

Done. I icreased the batery mass to 50 kg and it worked

I have no idea how to fix these warning. Do you?

@ipa-bfb
I had to comment the ur_modern_driver part in https://github.com/ipa320/cob_robots/pull/601/commits/115c020ff201206dfd081e793505cea2c842e0a3 in order to pass the CI-Tests...
We will need to deal with this in a separate PR when you  did further tests...

Sure ;)


Paket pack fails if output type is executable.

This is probably a bit of an edge case, but: I have a project that produces an .exe file, but this .exe file is also used as a library, so I want to pack it as a nuget package.
Leaving aside the discussion on where in the package folder this .exe file should go, paket .pack assumes it should be looking for a dll file, and failes packing when the projectname.dll file is nowhere to be found. (It should be looking for projectname.exe).

(This is a project template btw, not a file type template)

/cc @mavnn

Attempted to work around this by not adding a template file to the executables => paket pack should include project binaries into package for packed dlls, same issue

Yes I assume this needs to be addressed in paket

It's holding up my progress, So I'll have a go at shaving this yak. (unless @mavnn  is already way ahead of me)

found the issue
Now, lets see if I can also get some tests covering it


(PUP-7732) Audit Acceptance Tests

This audits the second half of the puppet acceptance tests. It is a companion PR to https://github.com/puppetlabs/puppet/pull/6040  I will need to use the PUP ticket prior to merging.

ping @johnduarte

CLA signed by all contributors.

I'd would also like to get a run of the acceptance tests done locally before this is merged.

@justinstoller disappointed you didn't challenge some of my suggestions. :)

Watch what you wish for (;

So running this PR I get:
              Attempted: 339
                 Passed: 271
                 Failed: 6
                Errored: 8
                Skipped: 53
                Pending: 1
                  Total: 339

But I get those same results from running the current head of master: b76616f9ba9794d47bd986979e62dcfbc24eba17 ... So, I guess this is fine?

@justinstoller this PR should be targeted at the acceptance-audit branch. I changed the base which brought in a couple of extraneous commits. Can you rebase?

rebased


Expose CF_DIAL_TIMEOUT and other CF specific env vars to CLI plugins

What's the user value of this feature request?
Plugin creators will have easy access to CF specific settings and will be more aware of their existence.
Who is the functionality for?
CLI plugin authors
How often will this functionality be used by the user?
At least once for any CLI plugin that talks to a CF component other than CAPI.
Who else is affected by the change?
Should be non breaking
Is your feature request related to a problem? Please describe.
Sort of... We write CLI plugins and got complaints that we weren't respecting CF_DIAL_TIMEOUT. We can get the variable from the environment but it would really help if there was a built in function so that CLI plugin authors would know about it before a customer has an unexpected experience.
Describe the solution you'd like
Either a function added to the cli connection to get dial timeout specifically. Or a function that returns some configuration struct with things like CF_DIAL_TIMEOUT and CF_COLOR on it.

Hi @kitemongerer thanks for reaching out, apologies for taking so long to respond. We don't currently have plans to support this unfortunately. I'm closing this issue but feel free to get in touch if you have any comments or feedback.

Hi @abbyachau I think this would be a very useful feature to create a uniform experience between individual plugins and the main CLI. I'd love for this to stay open until we solve the underlying issue

Hey @kitemongerer thanks for the feedback. Could you elaborate on why getting the variable from the environment is insufficient/painful? Thanks.

Yes. It is insufficient because there are several important environment variables for plugin authors to be aware of and there is no easy way to find out about them or what their defaults should be if they are not set. This means that every plugin author needs to learn about these variables through experience and then handle getting them from the environment and picking reasonable defaults (which are potentially different from the main CLI). If there were a uniform place to get important environment variables from with reasonable defaults chosen, it would create a more unified experience for plugin users because authors would be aware of the knobs users expect to be able to turn.

Thanks, reopening so we can track your concern.

Thank you for your submission. If this issue is still relevant with the latest CLI v8, please open a new ticket with the necessary information mentioned in our issue template.


fixed checkIpForward rule (gcp)

had to make sure it is backwards-compatible (name field is required in the template)


Distributed Configuration : support for serving plain text

Hy there,
I'm using spring cloud consul for distributed configuration. It works very well for the spring environment.
But I would like to serve plain text files such as logback.xml for example (many other needs in my company).
It exists in the spring cloud config, here is the link to doc: http://projects.spring.io/spring-cloud/spring-cloud.html#_serving_plain_text
After carefully reading your spring cloud consul documentation I've not found such a feature. I've made a bunch of tests. All unsucessfull.
Do you have any plan to add such a functionnality ?
See you,
Nicolas

So the spring cloud config functionality serves up text files using an http api. Consul already has a KV interface where you can put anything you want. I'm not sure this project needs to do anything. You can autowire a ConsulClient and use that.

Hy Spencer, thanks for your reply.
Is there any way that those plain text files added to the consul KV might be found in the classpath during the bootstrap phase of spring boot ? (I still have my logback.xml file in mind as an example)
Thanks,
Nicolas

Not automatically. It's not a feature of spring-cloud-config client either.

Too bad. It would be an awesome feature !
How hard the implementation could be ?

Difficult enough that we haven't done anything in 3 years.


Fix logo position on rate app dialog

Please fix for all languages (if it depends on it)

App version: Testnet 3.1.8
Phone's model: iPhone XR
Some languages have problems with this dialog:

Finnish - defect of the bottom button + logo is too close with text area

Korean - logo is too close with text area

Vietnamese - defect of the bottom button + logo is too close with text area

checked.
app version: testnet 3.1.9
phone's model: iPhone XR


Automated creation of use_foo_query for server fns

If I have a server function like,
#[server(GetNixInfo, "/api")]
pub async fn get_nix_info() -> Result<NixInfo, ServerFnError> {

would it make sense to extend it as:
#[server(GetNixInfo, "/api")]
#[query(options=QueryOptions::default)]  // <- Provided by leptos_query
pub async fn get_nix_info() -> Result<NixInfo, ServerFnError> {

such that the user does not have to manually write code like this?
/// Like [QueryResult] but used when calling Leptos [server] functions
type ServerQueryResult<T, R> = QueryResult<Result<T, ServerFnError>, R>;

fn use_nix_info_query(cx: Scope) -> ServerQueryResult<NixInfo, impl RefetchFn> {
    leptos_query::use_query(
        cx,
        || (),
        |()| async move { get_nix_info().await },
        QueryOptions::default(),
    )
}

I imagine that the bulk of this module can be removed, as a result:
https://github.com/juspay/nix-browser/blob/2357d0548a9a51734d654f53104d5306a878abd8/src/query.rs

I think a better approach is to write generic functions instead of macros. Something like:
/// Like [use_query] gut for server functions
pub fn use_server_query<S, V>(cx: Scope, k: ReadSignal<S>) -> ServerQueryResult<V, impl RefetchFn>
where
    S: Hash + Eq + Clone + 'static,
    ServerQueryVal<V>: Clone + Serializable + 'static,
    S: Clone + ServerFn,
{
    let action = create_server_action::<S>(cx);
    leptos_query::use_query(
        cx,
        k,
        |k| async move {
            action.dispatch(k);
            action.value()
        },
        query_options::<ServerQueryVal<V>>(),
    )
}

type ServerQueryResult<T, R> = QueryResult<ServerQueryVal<T>, R>;
type ServerQueryVal<T> = Result<T, ServerFnError>;

(This code ^ does't compile, I'm still working on it)

I think the macro will be very difficult because server functions can have variable arguments.
As far as you second point, I think there is some sort of conflation between Server Actions, and Server Functions.
Server actions are for mutative operations. These operations are still async, but most of the behavior does not fall under the domain of use_query. Queries are more like enhanced leptos resources.
Tanstack Query does include a useMutation function that tries to cover this, including callbacks for onSuccess, onFailure, and other niceties. Do you think this is something worth adding to leptos_query? Perhaps as an optional feature? I am not opposed to it.
https://tanstack.com/query/v4/docs/react/guides/mutations

@nicoburniske I wasn't talking about mutative operations (which doesn't belong to leptos_query). Actually I have a more precise request; will open a new issue to talk about it.

contd. #8


System.IndexOutOfRangeException When Using PowerShell on Windows Server 2012R2

When using AppInsights in PowerShell as described at https://vnextengineer.azurewebsites.net/powershell-application-insights/, everything works fine on Windows 10 but not on Windows Server 2012R2:
PS C:\Users\ghazan\Documents> [Reflection.Assembly]::LoadFile("C:\Users\ghazan\Documents\2.1.0-4.5\Microsoft.Application
Insights.dll")

GAC    Version        Location
---    -------        --------
False  v4.0.30319     C:\Users\ghazan\Documents\2.1.0-4.5\Microsoft.ApplicationInsights.dll

PS C:\Users\ghazan\Documents> $telClient = New-Object "Microsoft.ApplicationInsights.TelemetryClient"
PS C:\Users\ghazan\Documents> $telClient.InstrumentationKey = "secret"
PS C:\Users\ghazan\Documents> $telClient.trackEvent("test")
Index was outside the bounds of the array.
At line:1 char:1
+ $telClient.trackEvent("test")
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (:) [], IndexOutOfRangeException
    + FullyQualifiedErrorId : System.IndexOutOfRangeException

Full stack using https://blogs.msdn.microsoft.com/powershell/2006/12/07/resolve-error/:
Message        : Index was outside the bounds of the array.
Data           : {System.Management.Automation.Interpreter.InterpretedFrameInfo}
InnerException :
TargetSite     : Int32 CompareOverloadCandidates(OverloadCandidate, OverloadCandidate, System.Object[])
StackTrace     :    at System.Management.Automation.Adapter.CompareOverloadCandidates(OverloadCandidate candidate1,
                 OverloadCandidate candidate2, Object[] arguments)
                    at System.Management.Automation.Adapter.FindBestCandidate(IEnumerable`1 candidates, Object[]
                 arguments)
                    at System.Management.Automation.Adapter.FindBestCandidate(IEnumerable`1 candidates, Object[]
                 arguments, PSMethodInvocationConstraints invocationConstraints)
                    at System.Management.Automation.Adapter.FindBestMethod(MethodInformation[] methods,
                 PSMethodInvocationConstraints invocationConstraints, Object[] arguments, String& errorId, String&
                 errorMsg, Boolean& expandParamsOnBest)
                    at System.Management.Automation.Language.PSInvokeMemberBinder.InvokeDotNetMethod(DynamicMetaObject
                 target, DynamicMetaObject[] args, BindingRestrictions restrictions, MethodInformation[] mi, Type
                 errorExceptionType)
                    at
                 System.Management.Automation.Language.PSInvokeMemberBinder.FallbackInvokeMember(DynamicMetaObject
                 target, DynamicMetaObject[] args, DynamicMetaObject errorSuggestion)
                    at System.Management.Automation.PSObject.PSDynamicMetaObject.BindInvokeMember(InvokeMemberBinder
                 binder, DynamicMetaObject[] args)
                    at System.Dynamic.DynamicMetaObjectBinder.Bind(Object[] args, ReadOnlyCollection`1 parameters,
                 LabelTarget returnLabel)
                    at System.Runtime.CompilerServices.CallSiteBinder.BindCore[T](CallSite`1 site, Object[] args)
                    at System.Dynamic.UpdateDelegates.UpdateAndExecute2[T0,T1,TRet](CallSite site, T0 arg0, T1 arg1)
                    at System.Management.Automation.Interpreter.DynamicInstruction`3.Run(InterpretedFrame frame)
                    at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame
                 frame)
HelpLink       :
Source         : System.Management.Automation
HResult        : -2146233080

I’ve tried both https://www.nuget.org/api/v2/package/Microsoft.ApplicationInsights/1.2.3 and https://www.nuget.org/api/v2/package/Microsoft.ApplicationInsights/2.1.0 with versions 4.0 and 4.5.
Windows Server 2012R2 (not working):
$PSVersionTable
Name                           Value
----                           -----
PSVersion                      4.0
WSManStackVersion              3.0
SerializationVersion           <IP_ADDRESS>
CLRVersion                     4.0.30319.34209
BuildVersion                   6.3.9600.17400
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0}
PSRemotingProtocolVersion      2.2

Windows 10 (working):
$PSVersionTable
Name                           Value
----                           -----
PSVersion                    5.0.10586.494
PSCompatibleVersions      {1.0, 2.0, 3.0, 4.0... }
BuildVersion                  10.0.10586.494
CLRVersion                    4.0.30319.42000
WSManStackVersion             3.0
PSRemotingProtocolVersion     2.3
SerializationVersion           <IP_ADDRESS>

Could you please advise how this issue can be resolved?

Oh, judging by the name CompareOverloadCandidates I'd guess that you just need to do TrackEvent rather than 'trackEvent' (note upper case).

Unfortunately I am getting the same error:
$telClient.TrackEvent("test")
Message        : Index was outside the bounds of the array.
Data           : {System.Management.Automation.Interpreter.InterpretedFrameInfo}
InnerException :
TargetSite     : Int32 CompareOverloadCandidates(OverloadCandidate, OverloadCandidate, System.Object[])
StackTrace     :    at System.Management.Automation.Adapter.CompareOverloadCandidates(OverloadCandidate candidate1,
                 OverloadCandidate candidate2, Object[] arguments)
                    at System.Management.Automation.Adapter.FindBestCandidate(IEnumerable`1 candidates, Object[]
                 arguments)
                    at System.Management.Automation.Adapter.FindBestCandidate(IEnumerable`1 candidates, Object[]
                 arguments, PSMethodInvocationConstraints invocationConstraints)
                    at System.Management.Automation.Adapter.FindBestMethod(MethodInformation[] methods,
                 PSMethodInvocationConstraints invocationConstraints, Object[] arguments, String& errorId, String&
                 errorMsg, Boolean& expandParamsOnBest)
                    at System.Management.Automation.Language.PSInvokeMemberBinder.InvokeDotNetMethod(DynamicMetaObject
                 target, DynamicMetaObject[] args, BindingRestrictions restrictions, MethodInformation[] mi, Type
                 errorExceptionType)
                    at
                 System.Management.Automation.Language.PSInvokeMemberBinder.FallbackInvokeMember(DynamicMetaObject
                 target, DynamicMetaObject[] args, DynamicMetaObject errorSuggestion)
                    at System.Management.Automation.PSObject.PSDynamicMetaObject.BindInvokeMember(InvokeMemberBinder
                 binder, DynamicMetaObject[] args)
                    at System.Dynamic.DynamicMetaObjectBinder.Bind(Object[] args, ReadOnlyCollection`1 parameters,
                 LabelTarget returnLabel)
                    at System.Runtime.CompilerServices.CallSiteBinder.BindCore[T](CallSite`1 site, Object[] args)
                    at System.Dynamic.UpdateDelegates.UpdateAndExecute2[T0,T1,TRet](CallSite site, T0 arg0, T1 arg1)
                    at System.Management.Automation.Interpreter.DynamicInstruction`3.Run(InterpretedFrame frame)
                    at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame
                 frame)
HelpLink       :
Source         : System.Management.Automation
HResult        : -2146233080

The only mention of this issue I found so far is in this PR: https://github.com/Microsoft/vsts-rm-extensions/commit/e563501484e209550a298c74553c2e9d357cdd91
Can you please try TrackEvent("test", $nil, $nil) or whatever null is in powershell.
I suspect it found this override and failed. Maybe a hint to use a different override will help.
I'm speculating here

I've upgraded .NET to 4.6, ran the script as an administrator, added nulls to the method's signature and modified the script as following:
$AppInsightsInstrumentationKey = "<secret>"
$ES = "$PSScriptRoot\2.1.0-4.5\Microsoft.Diagnostics.Tracing.EventSource.dll"
[Reflection.Assembly]::LoadFile($ES)
$AI = "$PSScriptRoot\2.1.0-4.5\Microsoft.ApplicationInsights.dll"
[Reflection.Assembly]::LoadFile($AI)
[Microsoft.ApplicationInsights.Extensibility.TelemetryConfiguration]::Active.InstrumentationKey = $AppInsightsInstrumentationKey
$TelClient = New-Object "Microsoft.ApplicationInsights.TelemetryClient"
$TelClient.InstrumentationKey = $AppInsightsInstrumentationKey
$TelClient
$TelClient.TrackEvent("hello", $null, $null)

There are no errors now and the output looks right:
GAC    Version        Location
---    -------        --------
False  v4.0.30319     C:\Users\ghazan\Documents\2.1.0-4.5\Microsoft.Diagnostics.Tracing.EventSource.dll
False  v4.0.30319     C:\Users\ghazan\Documents\2.1.0-4.5\Microsoft.ApplicationInsights.dll

Context            : Microsoft.ApplicationInsights.DataContracts.TelemetryContext
InstrumentationKey : <secret>

The problem now is that the event does not show app in App Insights.
If I run the same code from a Windows 10 machine, the event does show up.
Any ideas what may be going wrong here?

Try to run: $TelClient.Flush() Maybe powershell wouldn't wait for event to be send and kill some threads we are running

BTW, did just passing of argument helped or another framework and EventSoruce fixed the issue?

This is getting interesting. If I run $TelClient.Flush()  and interrogate the last exception, it reveals that more dll's need to be loaded. I'll try that next.
Message        : Could not load file or assembly 'Microsoft.Threading.Tasks, Version=<IP_ADDRESS>, Culture=neutral,
                 PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.
FileName       : Microsoft.Threading.Tasks, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
FusionLog      : WRN: Assembly binding logging is turned OFF.
                 To enable assembly bind failure logging, set the registry value [HKLM\Software\Microsoft\Fusion!EnableLog] (DWORD) to 1.
                 Note: There is some performance penalty associated with assembly bind failure logging.
                 To turn this feature off, remove the registry value [HKLM\Software\Microsoft\Fusion!EnableLog].

Data           : {}
InnerException :
TargetSite     : Void DequeueAndSend()
StackTrace     :    at Microsoft.ApplicationInsights.Channel.InMemoryTransmitter.DequeueAndSend()
                    at Microsoft.ApplicationInsights.Channel.InMemoryChannel.Flush()
                    at CallSite.Target(Closure , CallSite , Object )
HelpLink       :
Source         : Microsoft.ApplicationInsights
HResult        : -2147024894

Loading the EventSource and passing additional arguments helped with the previous problem.

Are you sure you are using 4.5 version of Application Insights dll?

I think so, although anything is possible. I am not a .NET developer.

My question is where did you take this assembly from:
$AI = "$PSScriptRoot\2.1.0-4.5\Microsoft.ApplicationInsights.dll"
[Reflection.Assembly]::LoadFile($AI)

I took it from lib\net45 in https://www.nuget.org/api/v2/package/Microsoft.ApplicationInsights/2.1.0

Finally, it looks like it started working for me. I've started using dll from lib\net46.

Thanks for help, Sergey!


Add extracurricular activities

Adds config to add extracurricular activites in education degree.

Thank you @BernatBC. Can you please rebase this PR and fix the conflicts?


Fail in test_autocast_xpu::TestAutocastGPU

🐛 Describe the bug
Failure in pre-ci,
https://github.com/intel/torch-xpu-ops/actions/runs/9413187922/job/25929444890?pr=366
test_autocast_xpu.py::TestAutocastGPU::test_cast_cache_is_global FAILED  [ 25%]
Guilty commit,
https://github.com/pytorch/pytorch/pull/127741
Versions
Latest PyTorch.

fixed in https://github.com/pytorch/pytorch/pull/128383. Close this ticket.


clean up ec2 instances after each build job

Issue #, if available:
Description of changes:
Clean up the e2e test instances after each test run. Follow up to this is going to be tagging instances based on the build ID and using that tag for the cleanup.
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/override eks-anywhere-release-tooling-test-presubmit

this is stale, closing


docs(guides) Improve documentation about glob support in sideEffects flag

fixes webpack/webpack#11779

Thanks!


PRs to fix patches in java-buildpack upstream project

./api/java-buildpack/jobs/create_bosh_pre_start.sh
./api/java-buildpack/ig_resolver/patch_job_mf.sh

relates with #12


Add "Open Terminal Here" to context menu

As with today's command line or powershell, it is possible to go to the address bar in the file explorer and type 'cmd' or 'powershell' and it would open in the current directory location.

This feature request is about allowing the same behavior by typing 'terminal' - which will open the terminal with the default profile in the directory location.

In addition please also allow a context menu entry for terminal that would also open the default profile in the current directory.

Please make sure you search for existing issues before you file a new one.

1. This feature request is about allowing the same behavior by typing 'terminal' - which will open the terminal with the default profile in the directory location.

Use wt


Access to auxiliary variables in Tseytin transformation

Hello,
The auxiliary variables in Tseytin are non-independent variables, i.e. they are composed of the originals. How can one get access to this composition?
In other words, how can one verify equisatisfiability? (https://en.wikipedia.org/wiki/Tseytin_transformation#Approach)
Thanks.

I need to understand this better.
Let's just create a Tseying transformation, and look at it closely:
# auxiliary variable context
>>> aux = Context()
>>> w, x, y, z = map(bx.get_var, 'wxyz')
>>> f = bx.xor(w, x, y, z)
>>> g = f.tseytin(aux)
>>> list(g.iter_sat())
[{a_0: 1, w: 1, x: 0, y: 0, z: 0},
 {a_0: 1, w: 1, x: 1, y: 0, z: 1},
 {a_0: 1, w: 1, x: 1, y: 1, z: 0},
 {a_0: 1, w: 0, x: 1, y: 0, z: 0},
 {a_0: 1, w: 0, x: 0, y: 0, z: 1},
 {a_0: 1, w: 0, x: 0, y: 1, z: 0},
 {a_0: 1, w: 1, x: 0, y: 1, z: 1},
 {a_0: 1, w: 0, x: 1, y: 1, z: 1}]

So you can see that the 'a_0' variable is the auxiliary variable. It is not in the same context as variables w, x, y, z. In this particular example, equisatisfiability looks fine. But for more complicated examples you may need to do some filtering based on the variable names in the sat solutions.

Thanks for your response!
I will provide more detail.
To restate what I’m trying to do:
Let’s say we assign particular values to: w,x,y,z. Let’s say the value of original equation is 1 with this assignment.
How do we find out the aux values in corresponding tseytin equivalent ?  One way is to iterate all points from tseytin.iter_sat() until we find the same w,x,y,z assignment. Then we can get a satisfying assignment for the remaining aux vars. But this technique does exhaustive iteration.
On the other hand if we know the equations for aux in terms of the original , we can simply find the aux values using restrict, for each aux. Does the question make sense?
Thanks.

Sorry for delayed response.
If you already have a satisfying assignment, I'm not sure what you need the auxiliary variable values for, but I don't think it's a problem. Here is what I would do:
>>> import boolexpr as bx
>>> ctx = bx.Context()
>>> w, x, y, z = map(ctx.get_var, 'wxyz')
>>> F = bx.xor(w, x, y, z)
>>> G = F.tseytin()
# One SAT model of 4-input XOR is {0, 0, 0, 1}
>>> G.restrict({w: 0, x: 0, y: 0, z: 1})
a_0

So you restrict the Tseytin transform for the particular known solution, and it will give you the assignments to all auxiliary variables. In this case, a_0=1.

I am going to close this as "not a bug". Feel free to email me or ask more questions on StackOverflow.
On a related note, I agree that the current methodology for dealing with auxiliary variables is not sufficient. The current idea of a "context" seems slightly wrong. What we really need is nestable namespaces. All variables belong to a namespace, which all have parents up to a "null" namespace at the root of the tree. This might make it easier to deal with things like auxiliary variables. Unfortunately, it will have to remain just an idea for a while, as I have been too busy for hobby coding lately.


moto server dashboard 500 from data.json resource

How to reproduce the bug: run moto docker in server mode and go to http://localhost:5000/moto-api/.
Expected: dashboard page with service status and available AWS resources (e.g., S3 objects).
What happens: visit http://localhost:5000/moto-api/ and get a mostly empty page (except for tabs at the top). Inspecting the page shows that /moto-api/data.json returns a 500. curl GET against that resource also returns a 500.
Version: Server mode using https://hub.docker.com/r/motoserver/moto latest (digest ef4e4172daf2) using docker-compose on the default port (5000). Passing MOTO_SERVICE="ec2 sns sqs s3"

Hi @wallowa, can you share you docker-compose file? I can't seem to be able to reproduce it
I did find a bug where the data.json call returns a mostly empty list, but it does return a 200 for me.

Thank you for the quick response. It turns out it was triggered by passing a bad policy json when subscribing an sqs queue to sns, though using a policy appears to be unnecessary in moto so I just removed it.


I am getting an error while building a image

When i am trying to run command like
docker build -t app .

Please help me to resolve this

@harshadaju is your dockerfile named Dockerfile or dockerfile?

closing from lack of reply, please comment if it's still an issue


[HW] Saurabh Malviya

Name: Saurabh Malviya
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/saurabh-malviya-34763616/
Attach the homework step I screenshots below:

Adding a content from Ania's tutorial

Attach the homework step II screenshots below:

Awesome job, Saurabh! Thanks!

Hey Saurabh Malviya, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/IxAeMH-MRPuB_6s1iGnAag


How to setup a transaction manager?

Hello, is it possible to configure a transaction manager to be accessible by JNDI?
I tried this, but didn't work, once each time InitialContext is created, it reads from the properties files and replace all previously binded resources:
            InitialContext jndiCtx = new InitialContext();
            jndiCtx.bind(JNDI_TRANSACTION_MANAGER, createTransactionManager());
            jndiCtx.bind(JNDI_DATASOURCE_NAME, createDataSource());

Do you have any idea no I can have my TransactionManager binded too?

I debugged Hibernate and discovered that he calls the context.close().
Using the flag org.osjava.sj.jndi.ignoreClose=true apparently solves my problem.
I am glad all of this is open source, so I could debug it all to understand how it works.

Thanks for your feedback


jdk14 openliberty-mp-tck does not start on jdk14

The following messages appear when Test_openjdk14_j9_sanity.external_x86-64_linux_openliberty-mp-tck  is started by the openjdk14 pipeline
15:53:45  clean_image:
15:53:45       [exec] Error: No such image: adoptopenjdk-openliberty-mp-tck
15:53:45
15:53:45  build_image:
15:53:45       [exec] Sending build context to Docker daemon  12.29kB
15:53:46       [exec] Step 1/21 : ARG SDK=openjdk8
15:53:46       [exec] Step 2/21 : ARG IMAGE_NAME=adoptopenjdk/$SDK
15:53:46       [exec] Step 3/21 : ARG IMAGE_VERSION=nightly
15:53:46       [exec] Step 4/21 : FROM $IMAGE_NAME:$IMAGE_VERSION
15:53:47       [exec] pull access denied for adoptopenjdk/null, repository does not exist or may require 'docker login'
15:53:47
15:53:47  BUILD FAILED

Example jobs:
sanity.external
https://ci.adoptopenjdk.net/job/Test_openjdk14_j9_sanity.external_x86-64_linux/25/
Downstream job: https://ci.adoptopenjdk.net/job/Test_openjdk14_j9_sanity.external_x86-64_linux_openliberty-mp-tck/15/
@sophia-guo - any idea where the problem might be?

These test images base FROM AdoptOpenJDK/openjdk${jdkVersion} images.  Looking at hub.docker, there are none currently being produced for jdk14, so we fail to be able to build a test image that is based on a jdk14 base image.
https://hub.docker.com/search?q=AdoptOpenJDK&type=image
If there is not already an openjdk-docker issue for producing AdoptOpenJDK/openjdk14 and openjdk14-openj9 images, then should raise one.

Also, https://github.com/OpenLiberty/open-liberty/issues/9156 - so the playlist should get updated to restrict running on jdk14 until its supported.

Thanks @ShelleyLambert - I've raised https://github.com/AdoptOpenJDK/openjdk-docker/issues/294

Closing as https://github.com/AdoptOpenJDK/openjdk-tests/issues/1858 is a more current reflection of state of these tests.


[#133751193] Upgrade terraform container to 0.7.10

#133751193 Enable datadog BOSH healthmonitor integration
What?
We want to upgrade terraform to use a patched version of the datadog provider.
Latest version is 0.7.10. There is no need to use our forked version of terraform anymore, so we go back to the code that downloads the official binary.
We were forced to upgrade the gem for docker-api for our tests, as the docker API has changed and the tests fail in the latest docker versions (~>1.12)
How to test?
Check Travis should pass.
You can optionally run rake build:terraform spec:terraform.
Who?
Anyone but @keymon or @saliceti

works fine...


Generate types for environment variables

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


An OverlayRenderer that displays a color-bar for the FeatureColorModel on Mastodon views.

The color-bar looks like this on TrackScheme, when a tag-set is selected in the coloring menu:

Zooming a bit more:

For feature-based coloring, when the parameters are identical for vertices and links:

And when they are different:

A sub-menu of the view menu, just under the coloring menu, controls the visibility and position of the color-bar:

Integrated since beta-21 and superseded for bete-22


fix boilerplate in mixer

fix for #8996

@fisherxu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
8fe26374ff5c142cdc3d90c3422910aac02fb2c8
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests

@fisherxu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
8fe26374ff5c142cdc3d90c3422910aac02fb2c8
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests

@geeknoid Sounds good, let's go with the flow :-)


Auto-update or quick refresh button (please add! 🙂)

To aid with updating the map more quickly when searching along distance arcs. 🙂
Ideally re-centres map on same spot, where it was zoomed into.

There are two main ways to go with this:

for browsers, the new Filesystem Access API is required.
or the maptool could be ported into, for example, Electron desktop app.

There are downsides to both.
Filesystem Access API only works in Chrome and Edge, as other browser vendors deem it a security and privacy risk currently. It would have to be made into an optional feature, and fall back to the old file input.
With a desktop app the downside is having to download and somehow update the application.

Aside from allowing reloading and watching files for changes, the Filesystem Access API also allows to "open folder", which could be used to make it easier for users to browse their saved maps.

Mz added some updates to keep the pan/zoom.  I just finished implementing the file api and it appears to work!  enjoy.


Add go report card badge

Description
Add go report card badge
Type of change

[ ] :sunflower: Feature
[ ] :bug: Bugfix
[ ] :world_map: Documentation
[ ] :robot: Test
[ ] :computer: CI/Deployment
[x] :hamster: Trivial/Minor

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


What the loss should be when the model converge normally？

In your release model, the final loss is about 25 on training data, is that anything wrong? What the loss should be when the model converge normally？

Hi,
Thanks for asking.
I changed the original GAN loss into the non-saturating loss (i.e. min log (1-D(G(z))) -> max log D(G(z)), see the GAN paper) so it becomes more stable. Now the adversarial loss is a positive term and has larger value.
I've run it multiple times, and it does converge to ~20 reconstruction loss + ~5 adversarial loss. It seems that when the reconstruction loss is very small it's hard to align the hidden states and the model behaves like a pure auto-encoder, not changing the source sentence after flipping the style. There're definitely learning and optimization problems here, but I haven't found a better way to this.


Line direction is reverted at merge

When you merge two lines the direction is reverted but only if you select the last line first.

Ah yeah, iD strongly prefers to generate a forward path..
The same code that joins ways also joins relation members.  This comment is right before the part that does the reversal.
https://github.com/openstreetmap/iD/blob/00c4c57fc1d932625f8eb5a38fa6274d55a77a1d/modules/osm/multipolygon.js#L131-L135
I'm going to close here, since it doesn't really affect anything..

@bhousel It seems there is something wrong there. The direction matters for waterways and one-way highways. Waterways can be very wandering and you can easily lose track of what direction the current segment is supposed to have. It has already occurred to me to save such incorrect reverting, but I was lucky to notice the mistake later on. With one-way highways such mistakes are especially dangerous due to usage in navigation.

Ok, I just tested this - the reversing doesn't actually affect oneway roads (the tag gets reversed to oneway: -1), but it does affect waterways.
I'll see if this is something we can fix.  The main issue is if those ways also belong to a relation that requires the members ordered in a forward direction.  Maybe I can skip that reversal code in most cases.

the reversing doesn't actually affect oneway roads (the tag gets reversed to oneway=-1)

Ok, the tag secures the navigation. But is it really correct to revert the line direction and then fix it by means of the tag in this case? Let's take for example this one-way highway which is part of a route
https://www.openstreetmap.org/edit?way=314718034#map=17/52.96646/29.87338
The direction of this way is correct and conincides with the direction of the proceeding and following ways.
If you cut this way and merge it back, its direction will be reverted and the oneway tag will get changed from 'yes' to '-1', whereas the proceeding and the following ways will reserve their initial correct direction.

I'll see if this is something we can fix. The main issue is if those ways also belong to a relation that requires the members ordered in a forward direction. Maybe I can skip that reversal code in most cases.

☝ I did this  - now iD won't reverse the ways when it doesn't need to..

Thanks, @bhousel !


fix bug in vm pxe provision from rhevm

This pull request is for fixing bug in vm pxe provision from rhevm. The source code i use is release darga-2 in branch darga.
When i am trying vm pxe provision, I found that the steps that generate ks.cfg and pxe configuration files ware not called during the provision through reviewing the evm.log. And this is the reason why i can not provisoning a vm with os determined by pxe config and custom script.
So, i add a function customize_destination in module ManageIQ::Providers::Redhat::InfraManager::ProvisionViaPxe::StateMachine. After this, pxe provision works fine for me.

Closing in favor of PR #11606 which is exactly the same minus the trailing whitespace issues.


Readme PoAssets

Motivation
Crate Readme PoAssets
Have you read the Contributing Guidelines on pull requests?
YES
Test Plan
Not required

/land


Player, when smaller, takes damage from lava just from being near it

Description
When the player is at size 0.2 or smaller he takes damage when being near lava or when being near lava with a block above the lava.
To reproduce
To reproduce it just set your size at 0.2 or smaller and go near lava.
Expected behavior
You should be able to stay near lava without dying (like you can when you are normal size).
Video(Because you can't really tell with a screenshot)
Here's a small video about this bug: https://youtu.be/6Qom2ftJFVA
Additional information:

Forge version: 1.12.2-<IP_ADDRESS>47
Mod version: chiseled-me-1.12.2-<IP_ADDRESS>

If you need more infos, don't hesitate to ask!

confirmed in mixin-fixed version, if it can help, the exact size from which you take damage is 0.222222216427 (yes, I have some free time lol and yes I love this mod)

I dont think that the exact size would help me much but thanks for the effort :)


Added --generate-sources-and-exit option

When specified, TensileCreateLibray generates asm and source kernels
and exists without compiling them. Assembly sources will be placed in
the output directory.
Also added -v and -j to control output verbosity and the number of
threads launched by the script.

I didn't test the patch with the master branch, so I'm not sure if it works there.
Original version of the patch is on rocm-3.7.x branch and can be found here:
https://github.com/Artem-B/Tensile/commit/6893c8dedcbc30f282b04fe23301098e286692bb


Admin view not working

After putting in my API credentials, I'm getting errors from dependencies not declared i.e. "firebase is unknown." This comes up after I ran node main.js.

/cc @stephenmcd

The admin view is just a static HTML page plus client-side JavaScript, not a Node app.


constructor not working

Hi.
I see the idea you had with your constructor using default values. On the other hand, if you use CCS811() as a constructor call, this will call the default constructor, because it is still there and fits better than the constructor you created.
So just a hint, that it might not work the intended way...
  public: // Main interface
    CCS811() {
		CCS811(-1,CCS811_SLAVADDR_0);
	}
    CCS811(int nwake=-1, int slaveaddr=CCS811_SLAVADDR_0);

Hi - sorry for the very late reaction.
I didn't known that github projects have active Issues list be default.
I tried to understand you comment, but I don't.
Maybe you can give me some more help.
I tried the attached test-cons.ino script, it gave the attached test-cons.txt output.
And that seems to do what I want.
Thanks for the feedback.
test-cons.ino.txt
test-cons.txt

Shall I close the issue?

yes please.
as far as I see, I'm wrong...


get a maintainer.

This lib is dying. Transferring it to OER was a mistake.

+1 I agree. Need to fork this just to get https://github.com/openexchangerates/accounting.js/pull/106 in, which enables automated builds in other projects.
bower + npm

Hey guys, forgive the silence. Updates and PR merges coming within the next few weeks.

@josscrowcroft sounds good :+1:

Its now October hmm.


Add benchmark for String.indexOf(String)

Add benchmark for String.indexOf(String) API
Signed-off-by: Nigel Yu<EMAIL_ADDRESS>

Updated the benchmark to use a human-readable string as its default source string, based on which sub-strings are generated.


Add cmua.

Carnegie Mellon University in Australia community, we have 300-500 active internal users discussing various topics in our CMUA community, social events, technical topics, questions about academics, etc.
The icon of this community can be downloaded from this link:
CMU Square

Hello, can you help me to point out why these checks failed? I followed exactly the format. Much appreciate.

It's missing the icon for the site. I can add it. Thank you!

Actually, I just realized the forum is private. That's why pieces of my script is failing. We can't add private forums, only public.


Honor serial numbers of probes

This PR introduces handling of serial numbers when listing probes and when connecting to probes.
What I am not sure yet is how we best say "connect to probe with serial number X". Because right now we always pass a DebugProbeInfo struct which also needs some other info. We could either just construct a partial info struct or introduce another struct pendant to TargetIdentifier but for probes.
What do you think?

Ok so this is ready for merging featurewise. I even tested this with cargo flash where i made a branch for this. If zou eanna test it you cna check out that branch :)


Make combat directional

You can currently attack anyone within a certain radius. Instead, your attack area should be dependent on the direction your character is facing.

13de0135bd0125901ead4cc95867bd22ee9836bc


Add view parameter, remove adminDistrict type

Data Plane API - Pull Request

API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue:

Is this review for (select one):

[ ] a private preview
[ ] a public preview
[ ] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.
Note: This api version is unpublished and will not create breaking change.

Add view parameter for Get Reverse Geocoding and Get Reverse Geocoding Batch.
Remove adminDistrict.type property in address definition.

Design Document:
Previous API Spec Doc:
Updated paths:

❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

Hi @gigigoo0, one or multiple breaking change(s) is detected in your PR. Please check out the breaking change(s), and provide business justification in the PR comment and @ PR assignee why you must have these change(s), and how external customer impact can be mitigated. Please ensure to follow breaking change policy to request breaking change review and approval before proceeding swagger PR review. Action: To initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki. If you want to know the production traffic statistic, please see ARM Traffic statistic. If you think it is false positive breaking change, please provide the reasons in the PR comment, report to Swagger Tooling Team via https://aka.ms/swaggerfeedback. Note: To avoid breaking change, you can refer to Shift Left Solution for detecting breaking change in early phase at your service code repository.

@tjprescott This is not a breaking change as this api version is not yet published. The added view parameter to reverse geocoding aligns it with geocoding and Azure Maps localization support. The removed adminDistrict.type was introduced in this api-version and will nolonger be needed.

@tjprescott please let me know if this PR can move forward.

This change needs to be approved by the API stewardship board.


Validate ENV/LABEL Change options in varlink

If you pass in an invalid CHANGE ENV or LABEL option without the "=" character
podman crashes.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

Closes: https://github.com/containers/libpod/issues/2869

LGTM

LGTM once we clear our lint issues

@rhatdan Rebase to fix lint and I'll merge

Please merge this when it finishes.

Closes: https://github.com/containers/libpod/issues/2869

Still unhappy tests @rhatdan

@mheon @baude @vrothberg @TomSweeneyRedHat @giuseppe Tests pass now can someone give this a /lgtm

/lgtm


[Feature]: Plex Search results and "parsing" output should be cached and reused

Is your feature request related to a problem? Please elaborate.
I'm running a series of overlays; most of them are based on plex: all with filepath filter.  This means that for every overlay PMM "parses" the same list of 884 IDs; this is happening 12 times in my case at a couple minutes each time.  I imagine that work is the same every time, so it seems like it could be done once instead of a dozen times, taking 4 minutes instead of close to 40.
For a larger library [like my 3000-item 4K library] this will be even longer, probably near 2 hours.
Describe the solution you'd like
I'd like the output of the "plex search + parsing" to be cached and reused during a single run of PMM, similar to the way that builders can be cached.  I don't necessarily see a reason to cache them for longer than the duration of a single run.
The output of the "search+parse" operation as a whole could be cached, perhaps keyed on a hash of the search definition, or the parse output from individual items could be cached.  I haven't looked at the code yet, but I imagine that when PMM parses ID 12345 as part of one plex-search it does the same thing as for another plex-search, so if individual parse output were cached, that could benefit all subsequent searches.
The overlay case is the egregious offender, though, in my mind, since by definition it will often or likely involve repeated identical searches in close temporal proximity to one another.
Additional Information
No response

Added in 1.17.0


Fix Wrange-loop-analysis warnings in SPIRV-Tools.

Part of https://bugs.chromium.org/p/chromium/issues/detail?id=1039697

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

Thanks! Can you hit "merge" as well?


Io checks

The test runs in <1 second when you go into the test folder, but I noticed this when I ran in the (incorrect) top level folder (where the files dont exist):
$ time psinterp --outputfile interp.int 20180115_20191106.int psfile 600 600
Processing with options:
{'ifgfile': '20180115_20191106.int', 'psfile': 'psfile', 'nrow': '600', 'ncol': '600', 'outputfile': 'interp.int', 'N': 20, 'rdmax': 101, 'alpha': 0.75}

real	0m11.964s
user	0m11.990s
sys	0m0.139s

it just made a blank 0s file and took a lot longer.
Also added the import numpy as np which was missing for the interp(ifg,... function.

Thank you, Scott! I didn't see this pull request until you sent me the review request. Not sure how could the code run properly without finding the input files.


FWI-1614 - Allow disabling TLS for skopeo and docker in trivy plugin

This PR fixes #
Checklist

[x] I have signed the CLA
[x] I have updated/added any relevant documentation

Description
What's the goal of this PR?
Added SKOPEO_ARGS to make skopeo arguments more flexible
What changes did you make?
What alternative solution should we consider, if any?
Internal Ticket FWI-1614

@makoscafee looks like there's a TRIVY_INSECURE flag too: aquasecurity/trivy#757

@rbren  nice catch, this can be just added to the trivy env variable in the chart, I think.


Placeholder for Maven repo inside Iglu Server

To host POJOs, Scala case classes, Clojure Schemas auto-generated from JSONs, Thrifts, Avros etc

Initial draft
Not exact Maven repo, but rather "Creating Scala classes from JSON Schemas", but it was referenced from internal issues tracker.
Few projects to explore:

SBT Datatype
Json4CaseClasses (does not use JSON schema, GPLv3)
Scraml supports JSON schema inside the API spec and generates case classes
jsonschema2pojo (Java only)
JSON Schema Codegen
Salat

For me SBT Datatype looks like most promising (especially taking in account its origin). I think I will explore what can we do with it. Others libs listed here just for some technical details.
Definition generation
It uses format other than JSON Schema, but fairly straightforward. We can generate it the same way as we do with Redshift DDL. So, for example, having following JSON Schema (for iglu:com.acme/example/jsonschema/1-0-0):
{
  "type": "object",
  "properties": {
    "firstLevel": {
      "type": "integer"
    },
    "nested": {
      "type": "object",
      "properties": {
        "nestedInt": {
          "type": "integer"
        }
      },
      "additionalProperties": false
    }
  },
  "additionalProperties": false
}

We can generate following SBT Datatype definition:
{
  "types": [
    {
      "name": "Example$Nested",
      "type": "record",
      "target": "Scala",
      "fields": [
        {
          "name": "nestedInt",
          "type": "Int"
        }
      ]
    },
    {
      "name": "Example",
      "type": "record",
      "target": "Scala",
      "fields": [
        {
          "name": "firstLevel",
          "type": "String"
        },
        {
          "name": "nested",
          "type": "Example$Nested"
        }
      ]
    }
  ]
}

Adding $ to class name should help us avoid namespace collisions (nested can be defined on many objects). Following should in the end generate case class-like classes (with toString, hashCode, companion object etc, but without unapply):
final class Example$Nested(val nestedInt: Int) extends Serializable
final class Example(val someInt: Int, val nested: Example$Nested) extends Serializable

So we could write following in type-safe way:
example.nested.nestedInt

Still need to make many decisions about dynamic-json-to-static-scala correspondence, but some simple cases should work.
Iglu integration
So, assuming above will work, we need to:

Include registry into sbt project
reference generated classes in code in some Iglu-compatibe way (optional)
parse plain JSON into generated classes

I'm trying to design it assuming as few non-existing features as I can in SBT Datatype. So I'm going to mark everything we cannot do with it (we can fork or PR of course, but not sure they're going to include anything Iglu-specific)
Including into project
Let's assume we want to create SqlQueryEnrichmentConfig class from Iglu Central in Scala Common Enrich.
a. Enable SBT Datatype plugin in SCE's plugins.sbt
b. Run igluctl against Iglu Central JSON Schemas to generate SBT definitions in sbt-datatype directory (along with schemas, ddl etc) in Iglu registry
c. Release Iglu registry to Maven repository
d. Include it as a dependency: "com.snowplow" %% "iglu-central" % "58", so it can be embedded registry.
e. Set datatypeSource in generateDatatypes := file("resources/sbt-datatype"). Not sure if SBT Datatype can do it for third-party projects.
For now I'm really unsure only about last one, everything else should work on this step.
Reference and parse JSONs in Iglu-compatible way
It is a bit trickier. It's definitely possible to create some macro flavor to access it using Iglu URI as a string (but it looks like overcomplicated way):
import com.snowplowanalytics.iglu.registry // macro for access to class by string and containing serializers for parse

val json: JValue = ???
val enrichment: Either[String, SqlQueryEnrichmentConfig] = registry.schema("iglu:com.snowplowanalytics.snowplow.enrichments/sql_query_enrichment_config/jsonschema/1-0-0").parse(json) // jsonschema? Or sbt-datatype?

/cc @alexanderdean

But actually, original idea with other way round (with Maven inside Iglu, not Iglu on Maven) has its own clear benefits.

Very interesting approach @chuwy ! Looking forward to mulling it some more...

Having thought about it some more: while the idea of making e.g. Iglu Central embeddable inside an app is interesting, one of the flaws is that it depends on a versioning scheme for a registry which doesn't really exist: "com.snowplow" %% "iglu-central" % "58". The R58 there has no semantic meaning - it's just an artifact of the fact that we are using git with formal "releases" to back Iglu Central. The versioning inside an Iglu registry is all at the schema level, so really a developer would want to pull in a dependency like:
"com.mandrill" %% "message_opened_1" % "0.0.0"

This is the closest we have to this currently:
https://github.com/snowplow/snowplow/blob/master/2-collectors/scala-stream-collector/src/main/scala/com.snowplowanalytics.snowplow.collectors/scalastream/sinks/AbstractSink.scala#L34

Releasable Registry with explicitly defined milestones is more or less proven against current patching approach.
We can use it without milestones only if we're going to abandon patching after Open Versioning is embraced. I cannot see if open versioning can really help us with patching.

To elaborate:
case class Example(foo: Integer)

After patching can easily become
case class Example(foo: Option[Integer])

Which is a huge problem for both releasable and unreleasable, but having explicit milestone we can at least see how exact Schema was look like in some milestone.

The problem is that releasable registries is just a convention, it's not an intrinsic part of Iglu - a GitHub tag is not first class in any way in an Iglu registry. Even if it were, it's a very clunky level of indirection - "I want to reference Mandrill schema blah in my app, which GitHub tag do I need to cite to get that?"
Schema patching is easily handled like this:
"com.mandrill" %% "message_opened_1" % "0.0.4"

where this corresponds to com.mandrill/message_opened/jsonschema/1-0-0, 4th patch of the schema.

Patch approach looks good for me. Not that I really like idea that minor version can introduce source/binary incompatibilities, but for now it is probably best we have.

And where patch is defined? If we're going to do it manually - we'll need some sort of release as well?

It feels like we are going to have to make patches first class inside an Iglu registry - i.e. for a given schema you can see which patch release it is currently...

Yep, feels like that was going to happen anyway. These patches can be too important sometimes to just drop this information.


Document whether javascript: links are acceptable

Browsers accept and can act on links that look like <a href="javascript:alert('foo')">. However allowing users to submit links that look like this can lead to XSS and/or other problems.
It would be nice if linkifyjs documented somewhere that these links are not parsed/translated (at least as far as I can tell). I checked the README and the options page, and neither mentioned how javascript: links are treated.

Currently linkify only supports links with http(s): and ftp(s): protocols. javascript: is not a valid protocol from the plugin's perspective, so linkify will not attempt to create <a> tags for text containing javascript:.
For more information, I've added this doc to go over what kind of XSS prevention this plugin provides.


Configurable task failures in integration tests

Configurable max task faiures for integration tests
Running integration tests with the fault injection tool requires
tolerating falures.
Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

build


Use setup-php tools in php-linter.yml

setup-php action can provide parallel-lint.
source

Thanks 🙏


Turn peribolos into a GHA or APP

@evankanderson  pointed out that https://github.com/uwu-tools/peribolos as an example of this.
Side note:
Research has been done for alternatives to peribolos; https://github.com/knative/test-infra/issues/3618 however they don't seem mature as of yet.

/remove-lifecycle stale


Add another workaround for darwin SDL blank window bug

Pull Request Template
Description
As for me it's much better then moving window at one pixel
(Based on the https://github.com/Noofbiz/glfwMojaveFix)
Fixes #108
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Run aminal look at window content it should be appeared istead of black screen

Test Configuration:

OS: macOS
OS version: 10.14.1
Go version: 1.11.2

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Unfortunately on the CircleCI there is no binaries to build native code which injected to go source at build machine... Locally everything is ok, and also from 10.14 NSOpenGLContext marked as deprecated... but it's only warning... I hope when it will be removed such workaround will not needed
If you want to leave fix with moving I would like to propose to move +1 pixel and return to initial position move again to -1... To make sure that the position is initial what we expected

It seems need problem in the build machine https://stackoverflow.com/questions/11912878/gcc-error-gcc-error-trying-to-exec-cc1-execvp-no-such-file-or-directory/45403093

That looks good, nice one 👍

Found how to resolve issue with "build for darwin only"... See latest commit, with all compiler configuration and files for different oses

Ohhh... so quickly ))) Thanks!

@liamg-form3 it will be executed only at first swap buffers operation since is used global variable (counter) and we check if it have been done 2 times... Thus it will be executed only before first swap buffer

@liamg-form3 thank to pointing to build configuration, I've don't see it when look at the code =))) I'm always ignore comment when look at the code ( Thus I've miss that... But google helps me how to do that.
BTW go it's something new for me... and as Java developer I can say that it's very similar of combination:

Java
Pascal
C

And have full power of c++, great language as for me )


Moonpay in Desktop no longer works

To Reproduce
Steps to reproduce the behavior:

Go to /$/wallet
Click on Buy
Select Denmark
Click continue

The system is now waiting infinitely.
Expected behavior
I have earlier bought LBC thru Moonpay thru the app. I had expected I could do this again.
System Configuration

About
App	0.50.2 -
Changelog
Daemon (lbrynet)	0.92.0
Connected Email	none ￼
set email
Reward Eligible	No
Platform	Linux (Linux-5.4.0-72-generic-x86_64-with-Ubuntu-20.04-focal)
Installation ID	AMo4bUQT7W5cpb6DdFDuHB7DrRmVtu353BKmdrehYZrUxtTM8BPEHWaN1LVHgJFepp

{
  "jsonrpc": "2.0",
  "result": {
    "build": "release",
    "desktop": "KDE",
    "distro": {
      "codename": "focal",
      "id": "ubuntu",
      "like": "debian",
      "version": "20.04",
      "version_parts": {
        "build_number": "",
        "major": "20",
        "minor": "04"
      }
    },
    "lbrynet_version": "0.92.0",
    "os_release": "5.4.0-72-generic",
    "os_system": "Linux",
    "platform": "Linux-5.4.0-72-generic-x86_64-with-Ubuntu-20.04-focal",
    "processor": "x86_64",
    "python_version": "3.7.10",
    "version": "0.92.0"
  }
}

Thanks for filing. This is covered under https://github.com/lbryio/lbry-desktop/issues/5097 . Right now we'd suggest using lbry.tv or odysee.com to proceed.


Javascript tracker custom variables on trackPageView

HI,
Is it possible to add custom variables on trackPageView, i can't find infos about it in the doc ?
Why is it not possible to do snowplow('setFoo', 'bar'); ?
thx

Hi @gmpetrov! Please direct support questions to our forums:
http://discourse.snowplowanalytics.com/


Enable LTO for binary releases

Locally, running with
[profile.release]
incremental = true
debug = 0 # Set this to 1 or 2 to get more useful backtraces in debugger.
lto = true
codegen-units = 1

gives nice 20% speedup (but takes ages to compile).
I think we should setup CI to run that on release (both for nightly and weekly). I think cargo now supports setting profile values via env variables. So we should add --lto flag to xtask dist, whichs sets the vars before compilation. We should explicetly enable this flag on CI. Locally, --lto should not be run. For metrics, we should avoid LTO as well.
Separately, it might be useful to try to find some missing #[inline]s.

Also maybe mimalloc?

Maybe? Last time that didn't work out due to cross-compilation, but, if we are already using containers for CI, we might as well figure that out.

I think rust-analyzer shares dependencies with other crates like cargo/clippy/rustfmt in the rustc repo and in order for these changes to have effect, all of them would have to be compiled with 1 cgu and monolithic lto..?

This is mostly about release artifacts in this repo, and not the rustup binaries.

Have you considered lto = "thin"? It's a decent middle ground that shouldn't take too much longer to compile, while still apparently providing most of the benefits of LTO.
It doesn't look this this is already enabled, so it might be worth giving a shot.

Also there are cases when ThinLTO gives better performance boost than Fat LTO.

regarding cross-crate thinLTO (per crate thin lto is on by default afaik) I came across https://bugzilla.mozilla.org/show_bug.cgi?id=1640982 when trying to enable it, the solution to get it to work seems to be set -x CARGO_PROFILE_RELEASE_LTO thin which is what I'm using for all builds at the moment.

I don't think time to produce release artifacts really matters, so lto=true would be ok. For local builds, we should just stick with what are default cargo settings.

locally, lto = true changes the build times from 1 minute to 4 minutes, which is quite OK I think. On CI, the difference will be smaller as it is not as parallel to start with.


Can not find the pretrained model

Thanks for the nice repo you provided, but sadly I can't find the LA_disco.pth mentioned in test_LA.sh, hope you can share the model so that it will be easier to reproduce the result you say in the paper :)

There are DiscoNet and SyncDisocNet examples of checkpoints when compress_level is set to be 2 in version 2.0 of V2X-Sim.
link: https://pan.baidu.com/s/1BdMg4LOHQ1tgjnB1Zblbyw?pwd=3vpq code: 3vpq

Thanks for your reply!


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.15.24",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=43ae0ccc4d2722b833fb59b905af129428e06d03"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "43ae0ccc4d2722b833fb59b905af129428e06d03",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "powerpc/fixmap: Fix VM debug warning on unmap\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.24 by commit 43ae0ccc4d2722b833fb59b905af129428e06d03. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000416


Upgrade dependencies

This PR upgrades the dependencies to their latest versions.

Is this pr still gonna see the daylight? Because the sqlite3 dependency is already outdated again 👀
Dependabot or RenovateBot might help

This should be merged after https://github.com/jmoiron/sqlx/pull/922.
@ardan-bkennedy

Is this pr still gonna see the daylight? Because the sqlite3 dependency is already outdated again. It's pending for over a month now again 👀 Dependabot or RenovateBot might help

Hi. I don't think we need the noise from bots for now. We'll try to upgrade the dependencies before package releases, but this is something @ardan-bkennedy and I haven't talked about yet.


Split "set rtt to blocking-mode" into function

Firstly this shortens the fn start_program,
secondly it makes the "what" of this block of code more apparent.

bors r+


[Bug] File picker keeps in loading state when File select property is set

Description: If you set a "On File Selected" property and upload any image the loader on filePicker widgets keeps in loading state
Steps to reproduce the behaviour:

Drag and drop file picker widget
Set "On file selected property"
Now try uploading the file
Observe that after upload the loader on the widget keeps in loading state

Version: Cloud
OS: MacOSX
Browser: chrome
Environment: production

https://www.loom.com/share/dcc6387e38e24a80a243de0439e798c2

@vivekverma2312 does the file get uploaded regardless of the loader?

@somangshu Yes the files are getting loaded but the loader keeps in loading state. Due to this the file picker widget becomes disable for 2nd upload.


Add environment variable for Ankaios address

Description

Currently if the user changes the Ankaios server socket to another address and/or port the Ankaios user must provide the --server-address http://<ip>:<port> all the time when starting an agent and when using the Ankaios CLI.
This is a bit redundant and can be simplified with adding the environment variable for the Ankaios server socket.
Goals

Introduce an environment variable for setting the Ankaios Server socket address.
Final result
Summary
To be filled when the final solution is sketched.
Tasks

[ ] Task 1
[ ] Task 2
[ ] ...

I think different environment variables for cli, agent and server might be better to have a clear relation between variable and tool but for the way to connect to the server a URL should be used that includes the scheme(protocol like http or https) and the other parts like domain etc.

I think different environment variables for cli, agent and server might be better to have a clear relation between variable and tool but for the way to connect to the server a URL should be used that includes the scheme (protocol like http or https) and the other parts like domain etc. The URL structure is pretty flexible (see https://en.m.wikipedia.org/wiki/URL) and various schemas have been defined.

I am not sure if three different environment variables have a real benefit. Ankaios CLI and Ankaios agent are using the URL format. So we can have an env variable for the URL format and the socket address for the server.

Can I also make the support of the env variable in the requirements generic and link it in the clap argument structure in our code? For prioritizing the cli arg over the env var I have created a generalized requirement, but for the env variable I was not sure, because maybe some arguments in the future shall not be supported as env var and then I was not sure if it makes sense. What do you think?

Make sense to me to also support this setup you have described. Then the better use-ability is with separate variables. @krucod3: We have first said that two env variables are enough. Is it also fine for you to introduce separate ones? We have also thought about only adding the env variable to the Ankaios CLI, not to agent and server because they are not started very often. This would reduce custom code until clap has fixed the issue. For now, a single "ANK_SERVER_URL" might be enough. Do you both agree with the naming of the key? Shall we add a system test for the env variable?

Fine for me. I am not sure if we need a system test for that.

Maybe we should really go for 3 variables as proposed by @windsource. This way we can support the use-case where all 3 executables are running on the same machine, but you want to connect the CLI an/or the agent to a different server. It's a funny use-case, admittedly, but the variables would at least be mapped one-to-one with the input arguments

Maybe we should really go for 3 variables as proposed by @windsource. This way we can support the use-case where all 3 executables are running on the same machine, but you want to connect the CLI an/or the agent to a different server. It's a funny use-case, admittedly, but the variables would at least be mapped one-to-one with the input arguments

Yes I understand. The reason was that the Ankaios server and agent is not started very often, so the improvement is more on the side of the Ankaios CLI. But three variables are also fine for me. @windsource: Can I count your reaction on @krucod3 post as accepted for implementation?

So with 3 variables would mean:

ANK_SERVER_URL
ANKAGENT_SERVER_URL
ANKSERVER_ADDRESS

Like that?

Fine for me, I just thinking about the last one ANKSERVER_ADDRESS. I know that the current cli arg also is named address but should we include the "socket" also in the naming ? => ANKSERVER_SOCKET_ADDRESS

Fine for me.

Ok then I will implement three variables with the accepted names above.

As discussed with @krucod3: There are no stests added but claimed in the requirements. This will be done later, because of current other priorities.


Update main landing page doc

Update section "Deploy Azure Virtual Desktop with the Landing Zone Accelerator" to add the link to custom image deployment and future terraform deployment option (Contact @jensheerin for more info on terraform timeline)
Link: https://docs.microsoft.com/en-us/azure/cloud-adoption-framework/scenarios/wvd/enterprise-scale-landing-zone

Updates applied and PR submitted


Add support for EXT-X-SESSION-DATA

Is it possible add support to EXT-X-SESSION-DATA?
https://tools.ietf.org/html/draft-pantos-http-live-streaming-23#section-<IP_ADDRESS>
The EXT-X-SESSION-DATA tag allows arbitrary session data to be  carried in a Master Playlist.
In this example, only the EXT-X-SESSION-DATA is shown:
#EXT-X-SESSION-DATA:DATA-ID="com.example.lyrics",URI="lyrics.json"
#EXT-X-SESSION-DATA:DATA-ID="com.example.title",LANGUAGE="en", VALUE="This is an example"
#EXT-X-SESSION-DATA:DATA-ID="com.example.title",LANGUAGE="es", VALUE="Este es un ejemplo"
Usage sample (Chaptering and Thumbnails): https://developer.apple.com/documentation/avfoundation/media_assets_playback_and_editing/javascript_object_notation_json_chapter_format_for_http_live_streaming

@joeyparrish Are we going to add this?

@avelad What do you see us doing with this information?  It looks like what we would do is fire an event when we see that tag and let the app do something with it.

@TheModMaker , We will use this information for chaptering and thumbnails using this "standard" json https://developer.apple.com/documentation/avfoundation/media_assets_playback_and_editing/javascript_object_notation_json_chapter_format_for_http_live_streaming
Fire a event with all information(DATA-ID, VALUE, URI, LANGUAGE) is enough for me.

Any updates on this? I would also like this for chaptering.

I don't see any reason why we couldn't parse it and send an event.  When would you expect the event to be fired?  When the master playlist is parsed?

Yes, I expect the event when the master playlist is parsed. Another possibility is provide a method in the player for get this information.


Updater

Work related to:

https://github.com/easyScience/easyDiffractionApp/issues/28
https://github.com/easyScience/easyDiffractionApp/issues/81
https://github.com/easyScience/easyDiffractionApp/issues/58

Extra:

easyAppGui and easyAppLogic are now merged into easyApp
Redesigned application preferences dialog

When running the update, the component display of items to be updated is less than optimal: