Separate SOW & PAF form editing

Fixes #4204. Moves the SOW out into it's own form when editing. This also removes the user_has_updated_details attribute in favor of using a property for both the project form & SOW that checks the field_data on the respective form. This allows for tracking of the project form & SOW independently.
Also a few small aesthetic changes bundled in, like margin additions & hiding of submission attachments sidebar when there's none.
Screenshots
appearance when first creating a project

appearance after editing both forms

Test Steps

[ ] When viewing the details of a project with both a Project Form & SOW, ensure that both appear under Project documents.
[ ] Confirm that both can be edited, saved, and viewed on their own
[ ] Confirm that when creating a project from a fund that has a Project Form & SOW, a todo list item is created for both

all that left is getting these tests passing - a bit of a battle because my flag uses the project's form_data, but that won't get written unless a valid FormFieldBlock wrapped in a StreamValue is passed in the save method kwargs. it's giving me a hard time when I try to mock it
https://github.com/HyphaApp/hypha/blob/acdbea36177c90adb5cf24a7a61dd6d527a33f9e/hypha/apply/projects/forms/project.py#L137-L143

@wes-otf How vital is the test? We could remove it.

@frjo Yeah that sounds good - I don't think it's vital as it'd really just be checking if form_data is an empty dict. I tossed it and added your suggestions

grabbed some final bug fixed but should be ready to be deployed to test now @frjo

@wes-otf When I download a SOW at http://apply.hypha.test:9001/apply/projects/4/sow/ no fields are added to the exports. I guess something changes regarding getting the fields?
When I test the main branch locally it seems to work as it should.

@frjo great catch! my logic was copy & pasted, checking self rather than project - also removed some mentions of the SOW in the PAF export.

Deploying the latest to test now.

Can confirm that the bug in sow exports are now fixed. Looks all good to me.

@frjo thanks for testing - I think it's ready to go then! I did more testing on my end too and didn't see anything


SARIF: region should be omitted if result refers to the entire artifact

Rules like opa-fmt that refer to the entire file (known as an artifact in SARIF) should omit the region block per guidance in §3.29.4.
Currently for rules that apply to the entire artifact the region block is set to the following:
{
    "region" :{
        "startLine": 0,
        "endLine": 0
    }
}

This ends up violating the JSON Schema which requires a minimum value of 1 for both startLine and startColumn.

Thanks! That should be an easy fix. I'll take a look.


Middleware running at CDN vs origin

NextJS version: 13.0.3
@netlify/plugin-nextjs: 4.29.1
We recently upgraded to the latest version of the plugin (at the time, that was 4.29.1), as instructed by the deployment log on Netlify. This resulted in a change to how the middleware is run (CDN vs origin).
We run a cloudfront distribution that distributes traffic from our public URL to our netlify URL.
We use the next middleware to add a cookie and then redirect requests to a particular route in our application /beta back to the origin, which is our public URL.
Since upgrading to the latest version of the plugin, this functionality is broken because the host and origin on the request coming into the middleware is no longer the public URL, but rather the netlify URL. So users hit /beta and are redirected to a Netlify URL.
Is there a way to fix this?

If you'd like to opt-out of the middleware running at the edge just to get things working again, you can do so by setting the NEXT_DISABLE_NETLIFY_EDGE environment variable.
But assuming you'd like to try and use middleware at the edge and maintain the existing logic of the site, we'd need to see an example of the code that you're using to achieve what you've outlined above to make sure we're not misunderstanding the implementation of what you have in your codebase before making any suggestions (could be a contrived example).
Also - based on your description above, can I confirm you're using host-based or origin-based routing for the distribution of traffic?

Hi. I'm closing this because we've not heard back for two weeks. Please open a new issue with the extra details if you still have the problem.


Date-field resets when getting to year section

I know. It sounds weird.

Not sure why it's happening unfortunately, I have a clean repo with the minimum needed to reproduce it.
https://github.com/lcpriest/ember-form-for-date-field-bug
I'll add notes to this issue as I find stuff.

Looks like it's being rolled back due to format issues

@martndemus any tips on how to get around this?

@lcpriest First I thought the problem is with ember-one-way-controls update method and the browsers local format. But it turns out to be a htmlbars AND jQuery problem.
To fix the jQuery side you can add a valHooks:
jQuery.valHooks.date = {
  set(elem, value) {
    let [year, month, day] = value.split('-');

    if (year.length === 4) {
      elem.value = value;
    }

    return `${("0000" + year).slice(-4)}-${month}-${day}`;
  }
}

To fix the htmlbar side we need to change setPropertyStrict. I'll open a PR there.

I put up an initializer that is a workaround:
https://gist.github.com/fsmanuel/6abe960d131a099a2ef3628ca301d53f

Is there a workaround for the latest Ember (2.11), as the above workarounds seem not to work anymore (glimmer 2 in emberjs 2.10)?


Property 'withResolvers' does not exist on type 'PromiseConstructor'.

Search terms

Question
Hello, could you tell me how to deal with this kind of problem? thanks

That error is coming from typescript, and is caused by not having a recent enough ES version in your tsconfig.json's "lib" option

Thanks for replying, I actually have the latest version of typescript and "lib" is set to "ESNext". There are no errors in my project, this only happens when I do typedoc.

oh ,sorry! my fault.  I'll close this.


Removing "accept" header from cache/originRequest policy when AutoWebP is disabled.

There's no need to include the accept http header in the cloudfront's cache key when AutoWebP is disabled.
These changes will improve the cloudfront's cache hit ratio when AutoWebP is not used. Also stops sending unnecesary header (in such case) to origin (resizing backend).
Issue #, if available:
#304
Description of changes:
Infrastructure generation code was modified. Added a cloudformation's boolean condition (CfnCondition) based on AutoWebPParameter value, and evaluated it (using a cloudformation's Fn::If instrinsic function) inside cache/originRequest policy code.
Had to switch to "Cfn", aka L1, versions of cache/originRequest policy. Please read this issue.
Tests were updated to reflect changes.
Checklist

[x] :wave: I have run the unit tests, and all unit tests have passed.
[ ] :warning: This pull request might incur a breaking change.

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

Notice: There's another PR that tries to address the same task. Current PR (mine) aims to replace it.

Also related: #32 and #59
But this PR addresses (successfully I think) the removal of accept header from whitelist (cache key & sending to image handler lambda) when AutoWebP is disabled.

Hi!
Added a commit to preserve logical IDs of cachePolicy and originRequestPolicy resources (now all pre-existing logical IDs will be preserved).
--
What I did was extracting the code that creates that resources into classes imitating L2 classes CachePolicy and OriginRequestPolicy behavior (the new CustomBackEndCachePolicy and CustomBackEndOriginRequestPolicy classes) but adding the selection of proper headers.
Notice for code reviewers: github's diff is not smart enough when showing only the second commit, and is displaying as removed / added a lot of lines that hasn't been modified. I would recomend using the "all commits" view which shows the differences properly.

This pr has not received a response in a while. If you want to keep this issue open, please leave a comment below and auto-close will be canceled.

Keeping it open.

Updated with current main branch (using rebase). Conflicts were resolved. Also fixes eslint issues on my PR.

Hi @fvsnippets,
Overall I like this change, my only concern is that this will remove the ability to enable AutoWebP through the Lambda environment variables after the fact. Some changes to the documentation may overcome this, though I'm concerned it may still confuse users when modifying the environment variable doesn't do anything.
Simon

Hi @simonkrol
Thanks for taking a look into this PR.
I do not think that (manual) modification of this particular variable after deploy (in the lambda) is the best choice for consumers of this solution, but yes, you are right.
I'ven't worked with AWS in a while, but if I recall correctly you can also (manually) modify CloudFront to re-include the (now missing) header "accept", right?
So, as an alternative, if you are going to (manually) enable that variable after deploy, you can also (manually) add the header "accept" in CloudFront. Documenting this procedure is an alternative.

Hi @fvsnippets,
Yep, ideally manual modification of the environment variables doesn't happen too often, redeploying an updated stack with those new values would be better, but those modifications are something documented in the Implementation Guide that some users may rely on.
We can definitely improve the documentation there, my concern is mostly around having exceptions like these, where the IG indicates "you can change any of the env variables in the same way, except AutoWebP, which requires that you read this other section and perform an additional step". It may be wise that we rework that section of the document, aiming instead to have users update their stacks with the updated parameters, which would handle all of these issues for us. I'll chat with the team some more, and hopefully we can come to a decent solution.
Thanks for your contributions here :)

Hi @fvsnippets,
Just an update here, we've decided to implement this functionality and will be updating the IG to reflect the change as far as updating Environment variables goes. While our internal processes have us make the change manually, once this is released, you'll see this PR and your Github profile mentioned in the External Contributors section towards the bottom of the readme and this PR will be closed.
Thanks again for your contributions to SIH,
Simon


Cache changes location

I am posting this issue here because it seems to me that no one from the developers is reading user reviews at Android Market. There are many users having this serious issue.
I noticed this problem first time on 4/10/2011. Sometimes I have to refresh the cache for several times to get a correct position. I could say I having this problem with every 4-5 cache. I noticed differences in distance from 3 to 50m. It is very annoying.
It might have a connection with batch storing of caches from the map. I always had this issue when I was using stored caches. Fix it, please.

It's simple. Go to settings and enter correct username & password.

Is this some kind of joke? Of course I am using correct user name and password.

Send log. How to is on our faq page... Faq.cgeo.org

Sent

I had problem mostly with stored caches (stored using wireless internet connection with excellent signal at home). At the cache location I am downloading maps only so problem could be with maps?

After that I found something interesting:
500 (1036 ms) POST http://www.geocaching.com/default.aspx
This is a problem that needs to be investigated #684
You managed to login 3 times (that should not happen, cause you should login once and then stay logged in)
You are using go4cache and as far as I can see you did not open map

Last time I've had this issue was 13/10. If you tell me the exact time when this triple login had happened, I'll tell you what I was trying to do.

10-18 14:02:37.489

Ok. I was at home connected to my wireless network. I think I was just exploring the map. No downloading caches or so.
For the issue above check the date 14/10 at 12:40 to 13:10
or 04/10 around 13:30 and 15:50-16:30

Log contains only last 10 or 15 minutes of data.

That's bad. I didn't know that. Next time I'll have this issue I'll send you the new log.

I tried to replicate this nasty bug today, but with no luck. I had only one cache in my vicinity. I will continue tomorrow.
But I noticed one interesting message. It said that c:geo connection was lost and downloaded coordinates might be wrong or something like this.
I don't understand this message. I would understand if the coordinates were downloaded, then they are correct or weren't downloaded then they are none. How can application can download wrong coordinates?

If you are not logged in geocaching.com scrambles coordinates.

This could be be the case! I've had this "moving caches" there where I had low data signal - close to high stone wall, mountains and so!

I also had this problems 2-3 times but some weeks ago and I didn't get the message about not loged in.

I changed the issue title to reflect the problem.

Well, I haven't seen this issue any earlier, but I basically request an even stricter solution in #717.

Sorry, simple question: The log says https://www.geocaching.com....
Is it correct that a secure connection is attempted ?

Yes, that's necessary for the login.

Theoretically...what would happen if the phone does not accept (or does not have installed) the corresponding certificate ?

That would be really bad, because gc.com forces ssl.

Just trying to analyse the problem of @jirikucera as it is occuring quite often for him like mentioned in #717


Add Getting Started section in the doc

How should people use the kit and get started?
Essentially we should recommend them to go to the fabric website, sign up and/or modify their gradle/pom file

Fixed


Fixed problem with infinite loading for basic setups.

Fixed problem which caused that widget presented loading screen infinitely for basic setups.
Fixed compilation issues caused by Swift 4.2 incompatibility.

Closes issue #25


Add twitter profiles

Integrate twitter profiles to survol, should display profile picture + bio + ?
@jsilvaigor

I'll take a look on this one.

I was able to get the profile picture and add it to the extension (PR #38). To get the profile description we have to call the twitter api or create some mechanism to get the profile page and render it on a virtual dom, because it's client side rendered with javascript.

Ok, I've merged #38 for now thanks

Is this closed? Or is there any work to be done here?

Hum this is not closed, #38 added twitter profile picture to tweets which is nice but it didn't integrate twitter profile pages. If you want to do it you can there is work to be done indeed!


Periodical versions clean-up

There is a use-case when you want to keep the log for last week / month / year, etc and clean-up old revisions.
But with this approach "revert" fails to work sometimes and may throw an exception: 'Cound not fully revert the document to version: X'
Does it mean that log cannot be touched or is it possible to workaround / fix it somehow?

solved it in simple way, which is: do not trigger exception when the object is not fully reverted, references #163


listenBefore broken in v3.0.2

Version
3.0.2
Steps to reproduce
Use a listenBefore and dispatch an action in that listener.
Have a componentWillReceiveProps that also dispatches an action.
Expected Behavior
In < 3.x.x the action sent by the listenBefore is emitted first, before the route transition and component update.
Actual Behavior
In 3.0.2 the action sent from listenBefore is dispatched only after the transition happened.
Using the exact same branch, just reverting back to 2.8.1 makes everything work again as expected.

What changed between 2.8.1 and 3.0.2? Any idea?
I've spent all my time recently working on v4, and I don't plan on digging back into v3 any time soon. So if you can help us figure out what changed that's breaking your app and fix it, that'd be great.

maybe it nothing in react-router but an update of the history package it uses internally?

listenbefore comes directly from history. And the 3.x version of history is a rewrite from the previous 2.x version. Sounds like a history bug to me.

Well, the history issue tracker is full of v3 bugs, most of which are being reported by people who are trying to use the router with history. If we stop using history v3 in the router, I bet nobody will care about those bugs anymore, because nobody will use history v3.
Can we update router v3 to just use history v4? All the same functionality should be there, and it seems like history v4 is fairly solid.

Also, from #3647

This code isn't going to win any beauty awards.

😞

History v3 does all that query parsing and stuff right? Would have to add that in.

I'd recommend either downgrading to history 2.x or upgrading to history 4.x. It looks like history 3.x was just a bad release, done while I wasn't looking. My apologies for that, but I don't have the time to fix 3.x right now. By all accounts things are working perfectly in 4.x.

However v4 doesn't support code splitting so elegantly anymore, does it?

@pke The story for code splitting in v4 is far more elegant than anything we've done previously. The router is just components now, so you can code split like you would with any other components.
If you need an example, see how we're doing it on the router docs website.


Add support for "editor.formatOnSaveMode"

Environment data

VS Code version: 1.49.3
Extension version: 2020.9.114305
OS and version: Windows_NT x64 10.0.19041
Python version: 2.7.9
Type of virtual environment used: Debian GNU/Linux 8 dev container from wsl2
Relevant/affected Python packages and their versions: autopep8==1.5.4, yapf==0.30.0
Value of the python.languageServer setting: jedi

Expected behaviour
When 'editor.formatOnSave' is set to true and "editor.formatOnSaveMode" is set to "modifications" (Remote setting section) , formating only the modifications section
Actual behaviour
when "editor.formatOnSave" is set to "true" in just need one of the Remote, User, or Workspace section, it always formats the whole file no matter the value of "editor.formatOnSaveMode"
I did try with a few older versions but still the same behavior

Steps to reproduce:

store code from wsl file system
reopen in a docker container
pip install  autopep8==1.5.4 yapf==0.30.0

Logs

Output for Python in the Output panel:

> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff --line-range 16 16 /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff --line-range 16 16 /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff --line-range 22 30 /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff --line-range 22 30 /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff --line-range 108 108 /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff --line-range 108 108 /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff --line-range 182 182 /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff --line-range 182 182 /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff /workspace/core/pos_warehouse_management/models/models.py.e03afc84383d353e5756914ac6d563f1.tmp
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff /workspace/core/pos_warehouse_management/models/models.py
cwd: /workspace
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2020.9.114305/pythonFiles/pyvsc-run-isolated.py autopep8 --diff /workspace/core/pos_warehouse_management/models/models.py

By somehow it doesn't include the --line-range arg in a few last call

Changing this to enhancement. This is a new setting that VSCode added and we currently don't support this.

We have marked this issue as "needs decision" to make sure we have a conversation about your idea. We plan to leave this feature request open for at least a month to see how many 👍 votes the opening comment gets to help us make our decision.

For me, the behaviour is even worse: The whole document is formatted AND most indentations are removed, actually invalidating the python code!

@rhaschke This is now supported depending on if the underlying formatter itself supports it. For example, autopep8 and yapf have this supported , but black does not support formatting a particular range.
Use these extensions:
https://marketplace.visualstudio.com/items?itemName=ms-python.autopep8
https://marketplace.visualstudio.com/items?itemName=ms-python.black-formatter
https://marketplace.visualstudio.com/items?itemName=eeyore.yapf
https://marketplace.visualstudio.com/items?itemName=charliermarsh.ruff (formatting in beta support)


None of the actions in Prompt Management tab work properly

Description
Trying to "Create New Prompt" returns this error:
Exception: Unable to retrieve data. 'message'
Traceback:

File "/usr/local/lib/python3.10/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 589, in _run_script
    exec(code, module.__dict__)
File "/app/pages/4-Prompt_Management.py", line 87, in <module>
    ApiClient.add_prompt(
File "/usr/local/lib/python3.10/site-packages/agixtsdk/__init__.py", line 690, in add_prompt
    return self.handle_error(e)
File "/usr/local/lib/python3.10/site-packages/agixtsdk/__init__.py", line 56, in handle_error
    raise Exception(f"Unable to retrieve data. {error}")

Modify or Delete action says that prompt updated/deleted withot throwing error,, but no changes occur. Adding new category also doesn't work
Operating System

[ ] Linux
[X] Windows
[ ] MacOS

Acknowledgements

[X] I am NOT trying to use localhost for providers running outside of the docker container.
[X] I am NOT trying to run AGiXT outside of docker, the only supported method to run it.
[X] Python 3.10.X is installed and the version in use on the host machine.
[X] The latest version of Docker is installed and running on the host machine.
[X] I am using the latest stable version of AGiXT.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.
[X] I have searched the existing issues to make sure this bug has not been reported yet.

Importing prompt from txt throws same error as creating new one in usual way

Hello thank you for reporting the issue,
the issue has been resolved in commit 149465284e22576ad65141a712b0149c37b17b6a, which is now merged into the main branch.
Update your docker-compose file to use the 'main' branch instead of 'latest' and restart your containers with the start.py script.

Can confirm it's working, thank you.

Thank you too!


0.2.1 -- fix build

0.2.0 did not publish correctly
this fixes its issues

thanks a lot for addressing / fixing this, @aaronmgdr and @jeanregisser  ;-)


Add support to store serialized data as a string to work around loss of type information after Couchbase failover

From<EMAIL_ADDRESS>on November 04, 2014 20:26:09
After a couchbase node failover, sessions that are pulled from a replica lose their type information.  Instead of returning a byte array, the CouchBase client returns a String, resulting in a ClassCastException:
Reproduction Steps

On a Couchbase cluster, configure a bucket for Tomcat sessions with replication enabled.
Populate the bucket with some sessions.
Bring down the node containing your target session.
Make a request to the Tomcat server with your session.

Either use a single session, and a new / empty bucket so you can tell which server is the primary for your session, or a load test with several session.
Expected output:
The Tomcat server restores the last known state of the session
What do you see instead?
The following exception and Stack Trace:
de.javakaffee.web.msm.MemcachedSessionService
loadFromMemcached
WARNING: Could not load session with id 2CAF3AD14FCB9B4F92A9E0684AB49D28
from memcached.
java.lang.ClassCastException: java.lang.String cannot be cast to [B
at de.javakaffee.web.msm.LockingStrategy.loadSessionValidityInfoForValidityKey(LockingStrategy.java:333)
at de.javakaffee.web.msm.LockingStrategy.loadSessionValidityInfo(LockingStrategy.java:328)
at de.javakaffee.web.msm.LockingStrategy.onAfterLoadFromMemcached(LockingStrategy.java:363)
at de.javakaffee.web.msm.MemcachedSessionService.loadFromMemcached(MemcachedSessionService.java:1106)
at de.javakaffee.web.msm.MemcachedSessionService.findSession(MemcachedSessionService.java:585)
at de.javakaffee.web.msm.MemcachedBackupSessionManager.findSession(MemcachedBackupSessionManager.java:216)
at org.apache.catalina.connector.Request.doGetSession(Request.java:2913)
at org.apache.catalina.connector.Request.getSession(Request.java:2310)
at org.apache.catalina.connector.RequestFacade.getSession(RequestFacade.java:897)
at javax.servlet.http.HttpServletRequestWrapper.getSession(HttpServletRequestWrapper.java:229)
at javax.servlet.http.HttpServletRequestWrapper.getSession(HttpServletRequestWrapper.java:229)
...
What version of the product are you using? On what operating system?
Amazon Linux 2014.09
Tomcat 7.0.55
memcached-session-manager-1.8.1.
Couchbase Servers Version: 3.0.0 Community Edition (build-1209)
Couchbase Client: 1.4.5

Please provide any additional information below:

On testing with a standalone Java program, it looks like values that are stored as Strings do read back correctly after a failover.  If it was possible for the transcoder to use a base64 encoded string instead of a byte array, it should work around the problem.
Original issue: http://code.google.com/p/memcached-session-manager/issues/detail?id=216

From<EMAIL_ADDRESS>on November 04, 2014 15:31:30
This was caused by Couchbase bug MB-12328 (http://www.couchbase.com/issues/browse/MB-12328)

As that issue has been fixed in version 3.0.1, working around the problem with a String is no longer necessary.  I think this issue can be closed.

From martin.grotzke on November 05, 2014 14:12:24
Ok, closing.
Status: WontFix


Calendar collection report

... and remove /data directory

this implements https://github.com/tphummel/tetris-db/issues/5


proxy: Require that HTTP bodies implement Debug

When debugging proxy issues, it's helpful to be able to log details
about a stream body. However, because we are generally generic over the
body type, and the Body trait does not require that implementors also
implement Debug, then we must make this requirement explicit for all
uses.
This change adds Debug bounds to all body types. There is no
functional change.

I'm not sure this is good, though I don't know if it's horrible either. The downside is that we might accidentally log the contents of request/response bodies which might be secret. However, that's already the case for the URls and the header fields too.
Maybe instead we should just write a DebugBodyMetadata struct that contains the interesting non-secret metadata and implements Debug?

@briansmith i'd need to audit things to be sure, but I don't think the Debug implementations actually leak any body contents.  Instead, we see output like MeasuredBody { body: Http1(Body), inner: ... } (for hyper messages).  I agree we probably want to be extra careful about logging payloads, though...

I agree we probably want to be extra careful about logging payloads, though

I think it would be too easy to implement Debug for a Body implementation in a way that does output the body contents, so if we see that as a real issue I think we should go with a safer pattern: either the one I suggested, or a better one.

I think we should table this for now and pursue a More Proper solution at some later point (maybe after we can delete our h1/h2 glue code).


add production configuration

Popiš změny
Issue link

closes #80

Zkontrolovat

[ ] funkčnost: sám jsem si zkontroloval funkčnost
[ ] název: pull requst název dává najevo co se změnilo
[ ] endpoints: zkusil jsem si s přihlášením a bez
[ ] testy: přidal jsem srozumitelné testy

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

Project coverage is 84.16%.  Consider uploading reports for the commit befb8d0 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##               main      #81   +/-   ##
=========================================
  Coverage     84.16%   84.16%
  Complexity      159      159
=========================================
  Files            56       56
  Lines           884      884
  Branches         48       48
=========================================
  Hits            744      744
  Misses          135      135
  Partials          5        5

Flag
Coverage Δ

backend
78.35% <ø> (ø)

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


Number of queries and time showing as 0

Using django 1.9.5. I upgraded to silky 1.0.0 hoping it would fix the problem. It didn't help. I'm using SQL Server

in my settings
SILKY_ENABLED = True
if SILKY_ENABLED:
    SILKY_PYTHON_PROFILER = True
    SILKY_PYTHON_PROFILER_BINARY = False
    INSTALLED_APPS.append('silk')
    MIDDLEWARE_CLASSES.insert(0, 'silk.middleware.SilkyMiddleware')

I'm getting an exception here: https://github.com/django-silk/silk/blob/master/silk/middleware.py#L136
Cannot insert the value NULL into column 'id', table 'TestDB.dbo.silk_sqlquery'; column does not allow nulls. INSERT fails. (515) (SQLParamData)

Possibly related: https://github.com/django-silk/silk/issues/161

I deleted all silk tables and migration records and re-ran migrate and Silk is working happily again.


How to manipulate Current Objects from Rust?

Forgive me if this has already been asked somewhere, but I can see how to make a Dyon function which takes Current Object parameters, but what about Rust functions? I know you can pull values from the runtime, and sometimes that will give you the current Object that you need, but not always... I'm also aware that you have access to the current_stack like this
            let test = rt.current_stack.iter().find(|(name, _)| {
                **name == "test".to_owned()
            }).unwrap();

and you can use that to get the index of a current object. Should I think get the value at that index in the main stack, to get the corresponding current object? Isn't this a fairly common use case? I can't help but think that there is, hidden somewhere, a method to do this for me, or if not that there should be. What do you think about adding something like that? For now I'll just make Rust versions which take everything they need as parameters, and then make dyon wrapper functions which take the current objects for some parameters and pass those to the more verbose Rust versions.
Thank you for your time and let me know what you think about adding some helper methods like these to the runtime, to make writing Rust libraries for Dyon easier. Perhaps we could also have some for grabbing vectors with 3 elements, when you want to represent world positions or RGB colors and not quaternions or RGBA colors?
I'm willing to help however I can, let me know if you want me to write docs for these methods or even provide implementations. Your will is my command :P

I've never manipulated current objects from Rust before. There are no helper methods for this. PRs are welcome!
You should iterate in reverse direction on the current stack.
The Vec4 type implements From and Into traits for common types, e.g. [f32; 2]. https://github.com/PistonDevelopers/dyon/blob/master/src/vec4.rs#L30

fn current_object<T: embed::PopVariable>(rt: &mut Runtime, name: &str) -> Result<T, String> {
    let current_object_index = rt.current_stack
        .iter()
        .rev()
        .find(|(name_found, _)| **name_found == name)
        .map(|x| x.1)
        .ok_or(format!("Dyon can't find a current object with the name {}!", name))
        ?;

    T::pop_var(rt, rt.resolve(&rt.stack[current_object_index]))
}
fn teleport(rt: &mut Runtime) -> Result<(), String> {
    let test = current_object::<u32>(rt, "test")?;
    println!("Beam me up, Scotty! And you too, 7 of {}", test);
    current_object::<u32>(rt, "fail")?;
    Ok(())
}

// test Dyon:
// ~ test:= 9
// teleport()

Alright, this is the implementation I have! Before I clone the repo and PR, do you have any preferences for the name of the function that grabs the current object? When I try to think of what would be consistent, I realize that most of the other methods are based on pop, but this one is more like the var method. Despite this, I can't think of a way to make this name consistent with var in a way that makes sense.
Thanks!

Closed by https://github.com/PistonDevelopers/dyon/pull/608


Logout

I've tried to logout by clear cookie(set ctx to default value) but I don't think it is a good solution. I think we can logout through http://www.facebook.com/logout.php.
It seem facebook require parameter fb_dtsg,h and ref and then redirect to https://www.facebook.com/?stype=lo&jlou=...&smuh=....&lh=...
Does anyone know where is parameter h hiding? little help plz

I agree that we should have this feature.  If you give us a couple days, we can probably find the time to look at it a little more closely.  We'll have to check exactly what information is being sent in the form before we can know how to emulate it.

Cool ! now we can logout. :D
:(
but the request after logout, facebook will ask for login to continue and next request will response html document so it cause error in parseResponse. any idea?

@bsansouci now that you made the change that all errors throw exceptions, can we simply throw an error "User Not Logged In" or something in parseResponse?

@senhly what function does the request after you've logged out?
@Schmavery though it would make sense in this case, I don't think it's a good error message. parseReponse makes the assumption that it receives JSON and it gets html, which can be caused by many things.
We should be catching any error that parseResponse throws and calling the callback with it, so if you logout and do something you'll get an error. That seems about right to me.

I think @senhly is talking about using other API functionality after logging out.  This should definitely cause an error, but it would be nice if it was a specific error and not just a parse problem.  I was assuming we would do some basic parsing of the html in parseResponse to verify that this was actually the problem.

Mmh Ok, maybe we could make another parseResponse that will throw an error {error: "Not logged in."} that we use everywhere where we are supposed to be logged in. The only place where we'd use the regular parseReponse is during the login process I'm guessing.

@Schmavery was right, it cause error when call other function after logging out. By this way, I also notice that after logging out complete the pulling request in function listen still running. We could stop listen after log out. Am I right?

Reopening so that we don't forget to fix this in listen.


Enforce 100% code coverage

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #6   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           6      6
  Lines          59     59
=====================================
  Hits           59     59

Continue to review full report at Codecov.

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


Add class transform

❤  the project! Nice and tidy.
The existing project doesn't support the use of arrow functions, which are particularly useful in react development. I don't fully understand why as I thought they were part of ES6 syntax, but at any rate, adding support for the Babel class transform seems to sort this.

Yay! Thank you so much for the PR!

You are indeed very welcome!


Tighten crd validation to have only one occurrence of a value in a list

Tighten CRD validation to have only one occurrence of a value in a list.

Thanks for the comments @askervin
I think i will try to look into this again once we have migrated to v2 generators. Closing for now.


How to get content from Dialog on button press

Hi, how do I get the content on button press (inside a closure), from let's say EditView, which in the below code example is called "cpy_from_edit_view"?
let cpy_dialog = Dialog::around(
        LinearLayout::vertical()
            .child(TextView::new("Copy from:"))
            .child(EditView::new().content(path_from).min_width(80).with_name("cpy_from_edit_view"))
            .child(TextView::new("Copy to:"))
            .child(EditView::new().content(path_to).min_width(80).with_name("cpy_to_edit_view"))
            .child(
                LinearLayout::horizontal()
                    .child(Checkbox::new_with_label("Recursive").with_name("recursive_chck_bx"))
                    .child(Checkbox::new_with_label("Overwrite").with_name("overwrite_chck_bx")),
            ),
    )

Thanks

Hi!
You can check the edit example for how to retrieve the content of an EditView. The main trick is to give a name to the EditView itself, and not to any potential wrapper around it.
In your example, .min_width(80) wraps the EditView in a ResizedView<EditView>, and that is what you assign a name to.
Instead, it may be easier to first give a name to the EditView, and then give that a minimum width:
EditView::new().content(path_from).with_name("cpy_from_edit_view").min_width(80)

Then, when you want to fetch the content:
fn button_callback(siv: &mut Cursive) {
    let content: Rc<String> = siv.call_on_name("cpy_from_edit_view", |v: &mut EditView| {
        v.get_content()
    }).unwrap();
    // Do something with the content...
}

Thank you.


Fetch for ajax call before calling onInput

Add OnFetch in order to fetch the data and update items before calling filteredItems
Add selectedItemProjection because it might be different between actual selectedItem and protectedSelectedItem

Any update on this? Have the same issue in a different context


Dspace bat file

Removes the "The entered line is too long. Syntax error" error when entering dspace script commands in Windows 10.  E.g. "dspace create-administrator"

Should we remove bin/buildpath.bat? It may be a left over from ancient java versions when the 8.3 naming convention was required.
Note: You removed dspace/config/dspace.cfg and dspace/config/modules/authentication.cfg.

I also think that bin/buildpath.bat can be removed. Shall i remove it from this PR?

@Flusspferd123 : Yes, since this PR seems to be removing the (prior) purpose of buildpath.bat, then I'd recommend you remove it in this PR.  Otherwise, it's completely unused, but still sitting in the codebase.

it could be useful to rework also the "linux" dspace script in a similar way as it will be useful to have a shorter command line also here. Moreover, it is better to keep the two scripts as close as possible
BTW: I just noted that the dspace script still mention an OAI webapp classpath that of course not longer exist

Please give this an extra carefull review. The "linux" dspace script now works on a Linux machine. The bat file works for windows. If gitBash is used on Windows the linux" dspace script does not work.
Please also notice: An empty space in the path of a dependency will break this.

I've made the tiny one-line change that was requested in this PR above (see these comments). This PR is already at +2 and was only sitting open waiting on that minor cleanup.
So, this can be merged as soon as Travis approves again.


New Jupyter Notebook Example

@bertholdreinwald Could you please check this?

Can an Admin verify this patch?

LGTM

Merged to master in commit id:
https://github.com/apache/incubator-systemml/commit/5aa6fb75b066228d723dfc88bc0ae24551812e92


fix setting batch_size attribute in batch_size finder (finishing PR #2523)

What does this PR do?

Follow up to #2523 (unfinished, stale PR)
Fixes #2484

Set model.batch_size based on model.hparams.batch_size if not defined
Changed init_val for batch_size to 0 so that it starts with user defined batch_size instead of 2 all the time

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@SkafteNicki I finished this old PR and added tests as best as I could.
Not sure why the default argument needs to be changed.


Implement blending audio feature to AnimationTree

Fixed #65750.
Follow up #71855. Allows AnimationTree to blend audio using AudioStreamPolyphonic.
This PR adds the following options to the audio track:

UseBlend

Sets whether the track will be blended with other animations. If true, the audio playback volume changes depending on the blend value.

AutoClear

Sets whether the stream cache should be deleted when the blend amount is 0 in blending. If you want to fade in the audio or continue playback after blending, this value must be false. Note: Even if this value is false, the stream will be removed from the cache when the audio is played to the end. Removed, we will send an optimized PR to eliminate the need for this AutoClear in the future.

Known issue / Todo:

The AnimationPlayer has also been changed to use AudioStreamPolyphonic, but audio blending is now only available in the AnimationTree due to #70207. It will be available after the https://github.com/godotengine/godot-proposals/issues/5972 proposal is applied to the AnimationPlayer.
Blending of the same animation with each other may result in a messed up audio. This is due to the fact that stream maps are assigned by the ObjectID of the Animation Resource, which may be resolved in the future if it becomes possible to assign arbitrary IDs to AnimationNodes.

Preview (Turn on the volume):
https://user-images.githubusercontent.com/61938263/215250708-36ec24b7-8f05-4c0c-8b27-ff980727be2c.mp4
audio_blend_test.zip

After discussions with the @reduz, it was decided to remove the AutoClear option because it is hard to use and will likely not be used.
The AutoClear option was implemented to prevent audio with 0 amount blend from playing in the background, saving memory and saving audio polyphony counts.
However, we have concluded that improvements to the AudioStreamPolyphonic API should prevent increasing the audio polyphony count in the background when the blend amount is 0 by properly pausing and playing back the audio.
This optimization will be implemented in the future as Follow up PR. Until then, we can have the workaround that increase the number of audio_max_polyphony property in AnimationTree to handle that.

Thanks!


add (0-x) fold

already tested in test_uops

prefer no new rules before UPat refactor is done, also these need explicit tests, also please read the pr diff


Sample would help to clarify when @PathSensitivity should be used

Related Samples
Frankly speaking, @PathSensitivity Javadoc, and https://guides.gradle.org/using-build-cache/#relocatability do not explain much for those who are new to @PathSensitivity.
Context
Once upon a time I've discovered @PathSensitivity is a thing, however, I didn't figure out how to use it.
The documentation describes the problem, but it does not really provide clear examples or "flow charts" to select the sensitivity.
It would probably help if a couple of use-cases was added to the documentation.
For instance:

If the task output depends on the paths of the input files, then the sensitivity should be RELATIVE_PATH
For instance, If the task produces a text report that lists all the input files and their sizes, the output depends on the paths, and the task should be re-run if the files are moved around
If the task output depends on the name of the file only, the sensitivity should be NAME_ONLY. For instance, Java Compiler adds the debugging information to the generated class file. The debugging information includes input file name, so the class should be recompiled if the file is renamed
If the task output depends on the contents of the file only, then the sensitivity should be NONE. For instance, if the task computes the total size of the input files, it does not really need file names, so it can use sensitivity=none.

I might get it wrong, but, I think path sensitivity should be chosen based on the answer to "Does task output contain path or name of the input file?"

Just in case, the current documentation (6.3) contains:

Running javac will produce the same output regardless of the name of the JavaCompile task that invoked it.

It is not quite right because the output bytecode does contain the input file name, and the source name is even visible in the stack traces.

This is still relevant


Answer not displayed for date equality question

The following query generates a response from the server, but nothing is displayed in the interface.

I was using the wrong code generation for the comparison block. Fixed.


Prevent ClassCastException in triggerSickle of TweaksImpl.java

This PR fixes an issue that me and a few others noticed on our server using VanillaTweaks. An unhandled ClassCastException would be raised when trying to harvest crops using a sickle, which would crash the game immediately. Most prominent instance of this was simply trying to break grass with a sickle.
Instead of crashing the game, here I'm suppressing the exception and writing a warning to the logs that looks something like this:
[00:28:23] [Render thread/WARN]: ClassCastException suppressed whilst triggering sickle (translation{key='block.minecraft.grass', args=[]} @ 1067, 67, -1335)

If you've got a better method of fixing this issue feel free to let me know in review, otherwise this is tested working.
It should also be noted that this fix doesn't stop the crops from being harvested. Everything still works as intended.
I'm also making this change into the 1.20 branch because that's the server version in which the bug took place. You could probably merge the changes up into the 1.20.2 branch.

Hey sorry for taking so long. Just wondering can you tell me the steps for reproducing the crash? I am not able to reproduce by breaking both grass and grass block using a hoe.

Released a new version with proper fix/


(docs) Fix typos in the facts-blacklist docs.

Corrects two misspellings, clarifies the difference between INI and
HOCON values, and wraps the examples in preformatting backticks.

Test FAILed

Build finished.
Test FAILed

CLA signed by all contributors.

@gguillotte need to close this but will add your changes to 4.4.x and then merge up from there. Thanks for fixing this up :)


Add promlint check for histogram/summary reserved names

I think I got this right, but as always, let me know if anything is incorrect.
Fixes #2618.

@brian-brazil Updated.


\u001b[31;1mCommandError: The user does not exist.", "\u001b[0m"

- name: "Get AWX/Automation Controller administrator token (2/2)"
  command: |
      kubectl exec -n awx -i -t {{ awx_pod.stdout }} --container awx-task -- /bin/bash -c 'awx-manage create_oauth2_token --user {{ admin_username }}'
  register: awx_token
  #no_log: True

produces
TASK [awx-custom-config : Get AWX/Automation Controller administrator token (2/2)] ***************************************************************
fatal: [awx.vacatejaws.xyz]: FAILED! => {"changed": true, "cmd": ["kubectl", "exec", "-n", "awx", "-i", "-t", "awx-f9c4f86cc-4gl2h", "--container", "awx-task", "--", "/bin/bash", "-c", "awx-manage create_oauth2_token --user admin"], "delta": "0:00:02.831705", "end": "2022-12-15 08:49:33.233392", "msg": "non-zero return code", "rc": 1, "start": "2022-12-15 08:49:30.401687", "stderr": "command terminated with exit code 1", "stderr_lines": ["command terminated with exit code 1"], "stdout": "\u001b[31;1mCommandError: The user does not exist.\r\n\u001b[0m", "stdout_lines": ["\u001b[31;1mCommandError: The user does not exist.", "\u001b[0m"]}

Running it again caused this error instead: https://github.com/PC-Admin/awx-ansible/issues/3

Fixed in https://github.com/PC-Admin/awx-ansible/commit/74029ce6232e16fdc11a57c95c15532ff3eb4606
By just avoiding this hack method of token collection.


HHH-15713 - UnknownTableReferenceException on @ElementCollection of @Embeddable containing a @MayToOne with a @ManyToMany

Test case to reproduce HHH-15713.

Superseded by https://github.com/hibernate/hibernate-orm/pull/5601
Thanks @markusheiden for the test!


Multi select & move of EVENTs

It should be possible to select single multiple EVENTs (just like SEGMENTs) and move them by pushing down and holding the alt key.

on the SEGMENTs we check if it is allowed to select multiple (bevore moving them) by comparing start&end times of the corresponding segments...
how should we check that on events ?
i.e. in this situation:

should it be allowed to multiple select the left H* and L ?

I think it should only be possible to select "direct neighbors" as groups of "preselected EVENTs". If L- is preselected right-clicking the left H*@18631 or the right H*@38254 should result in two preselected EVENTs. If L- is preselected right-clicking on H*@8381 should result in  H*@8381 being preselected and nothing else.

there is still a small bug that not all events get selected... gonna fix that tomorrow morning


Remove cert-manager from helm-charts

Acceptance criteria

Drop related values fields.
Bump major version.
Remove questions yaml section for Certificates.

Closing, done as part of https://github.com/kubewarden/helm-charts/pull/488.


Using SizeLimit with MemoryCache cause queries to fail

Steps to reproduce
Use MemoryCache with ASP.NET Core in Startup.cs
        services.AddMemoryCache(o => {
            o.SizeLimit = 5000;

        });

Then do a query
    public IActionResult Test([FromServices] AppDbContext db)
    {
        var x = db.PostCodes.Where(p => p.Value == "XXXX").ToList();
...
    }

The issue
You get an exception like this
                An unhandled exception has occurred while executing the request
        System.InvalidOperationException: Cache entry must specify a value for Size when SizeLimit is set.
            at Microsoft.Extensions.Caching.Memory.MemoryCache.SetEntry(CacheEntry entry)
            at Microsoft.Extensions.Caching.Memory.CacheEntry.Dispose()
            at Microsoft.Extensions.Caching.Memory.CacheExtensions.Set[TItem](IMemoryCache cache, Object
        key, TItem value)
            at
        Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryCache.GetOrAddQueryCore[TFunc]
        (Object cacheKey, Func`1 compiler)
            at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.Execute[TResult](Expression query)
            at Remotion.Linq.QueryableBase`1.GetEnumerator()
            at System.Collections.Generic.List`1.AddEnumerable(IEnumerable`1 enumerable)
            at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)

Further technical details
EF Core Version : 2.0.2
MySQL version:  5.7.22-0ubuntu<IP_ADDRESS> - (Ubuntu)
Operating system:  Ubuntu 16.04.4 LTS
Pomelo.EntityFrameworkCore.MySql version: 2.0.1

This should probably moved to aspnet/Caching repo. We use Set method directly which has no way to specify size for the entry. MemoryCache extension methods create CacheEntry itself so it should set value for size. (especially when they have check for it)

Moved to https://github.com/aspnet/Caching/issues/399

@Tratcher stopped by to talk about this.
I am reactivating to consider moving EF Core to have its own private instance of the cache service, which would prevent the issue reported by the customer. Apparently we are already doing this on other ASP.NET Core libraries and the resolution for https://github.com/aspnet/EntityFrameworkCore/issues/2052#issuecomment-106118489 no longer applies. However because memory cache is still registered as a singleton we cannot use DI to instantiate it.
@Tratcher also recommended that we include our own configurable and default size limit setting and sliding window eviction policy to mitigate the possibility of cache explosion.

Some examples provided by @Tratcher:
https://github.com/aspnet/ResponseCaching/blob/dev/src/Microsoft.AspNetCore.ResponseCaching/ResponseCachingOptions.cs#L14
https://github.com/aspnet/ResponseCaching/blob/d5430992d3bcf0333bcd9974558188d45ca8b489/src/Microsoft.AspNetCore.ResponseCaching/ResponseCachingMiddleware.cs#L40-L43
https://github.com/aspnet/ResponseCaching/blob/d5430992d3bcf0333bcd9974558188d45ca8b489/src/Microsoft.AspNetCore.ResponseCaching/Internal/MemoryResponseCache.cs#L68-L72
https://github.com/aspnet/Caching/blob/dfd3f2c5c456d4aeceae296ac2ee3d22f8d22cf7/src/Microsoft.Extensions.Caching.Abstractions/MemoryCacheEntryOptions.cs#L55-L59

Filed https://github.com/aspnet/Docs/issues/6958 for doc updates.

Notes from triage: to add to Diego's comments, we currently attempt to use the cache from the external service provider if available. We can stop doing this, meaning in all normal cases we would use our own cache. If the application is explicitly building the internal service provider and wants it to use the application service provider for this (which is not common) will there need to be common configuration for both, but that doesn't seem unreasonable.

Re-discussed in triage and decided to move this to 2.2 to at least help prevent a bug in query caching from resulting in out-of-memory. See #8223

Per discussion in triage, using this bug to track cache eviction (as indicated in last comment) is not appropriate, so filed #12905 instead.

Fixed in cbd28ff4871cc5abc626dc7b4917a0b20538e2f8


Rubocop Flagging Style/FormatStringToken on CI only

Expected behavior
Rubocop is a gem, and should be platform agnostic
Actual behavior
Rubocop on our CI environment flags the following rule. Running this locally passes this check.
features/support/helpers/screenshots.rb:12:32: C: Style/FormatStringToken: Prefer annotated tokens (like %<foo>s) over unannotated tokens (like %s).
      Time.now.strftime('%Y-%m-%d-%H-%M-%S')

Steps to reproduce the problem
Create this method. Try on CI (Jenkins) / locally
def timestamp
  Time.now.strftime('%Y-%m-%d-%H-%M-%S')
end

RuboCop version
0.52.1 (using Parser <IP_ADDRESS>, running on ruby 2.3.3 i386-mingw32)
I expect there to be some issues reproducing this. I'd rather not add this to the ignore list incase other issues crop up.

I'm closing this ticket due to no recent activity. Feel free to re-open it if you ever come back to it.


Hash example in CSP style-src documentation must use base64 encoded values

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

The example provided for hash based values is passing the direct computed hash value instead of a base64 value
Example reads:
Alternatively, you can create hashes from your inline styles. CSP supports sha256, sha384 and sha512.

Content-Security-Policy: style-src 'sha256-a330698cbe9dc4ef1fb12e2ee9fc06d5d14300262fa4dc5878103ab7347e158f'

But should instead read as:
Alternatively, you can create hashes from your inline styles. CSP supports sha256, sha384 and sha512 (in base64 form).

Content-Security-Policy: style-src 'sha256-YTMzMDY5OGNiZTlkYzRlZjFmYjEyZTJlZTlmYzA2ZDVkMTQzMDAyNjJmYTRkYzU4NzgxMDNhYjczNDdlMTU4Zg=='

(Where YTMzMDY5OGNiZTlkYzRlZjFmYjEyZTJlZTlmYzA2ZDVkMTQzMDAyNjJmYTRkYzU4NzgxMDNhYjczNDdlMTU4Zg== is the base64 encoded value of a330698cbe9dc4ef1fb12e2ee9fc06d5d14300262fa4dc5878103ab7347e158f)

Page https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy/style-src#Unsafe_style_expressions, under section Unsafe_style_expressions

The spec confirms that those hashes have to be base64 encoded.
I can take this issue.

Thanks @mikoMK ; I've assigned it to you.

While testing the example i found out that the hash also has to be in binary form before encoding it with base64. This doesn't seem obvious from a web developer's perspective. Therefore I clarified this in the article and added the example from the spec how to achieve the correct conversion.
Any feedback is welcome.

This update looks great, thanks @mikoMK .
I'm going to close this one now, but feel free to jump in @QwertyManiac if you have any other comments or concerns.


drivers i2c I2C_DW:Fix I2C scan example

Solves two similar issues listed below:
Issue 1: I2C scanner example for DesignWare hardware gets stuck indefenitely resulting in system hang up. This is because DW driver does not handle 0 byte transfer correctly which is the case for I2C scan example. Fixed it by overwriting the msg length to 1 if it is 0 and the buffer is not NULL.
Issue 2: Similarly, if the I2C pins are not pulled up (nothing connected to I2C pins), the DW hardware does not actually sending the data (assuming contention on the bus) hence not releasing the semaphore resulting in calling thread waiting forever. Fixed it by adding a timeout to k_sem_take call and return error if cannot successfully acquire it.
Fixes: #70332.
Micropython seems to tackle this issue by supporting zero length transfer through Soft I2C implementation. Since Zephyr doesn't currently support that this commit provides a temporary solution to unblock the I2C Scan example.
Link: Micropython I2C workaround

@ubieda does this happen to fix the rpi issue you ran into?


replace processor call with machine call

the current call to platform.processor is not reliable and quite frequently returns "unknown", for instance in the CI, or on arch linux. this patch replaces all occurrences with calls to platform.machine which should be way more reliable.

I wonder if this is also the cause for the unit test failure here (where the value is not mocked).
Should we fail back to x86_64? It's only arm64 and x86_64, and an equivalent to:
switch(runtime.GOARCH); {
case "arm64":
  binary = "cos-tool-arm64"
default:
  binary = "cos-tool-amd64"
}

Is probably a sane enough default

I wonder if this is also the cause for the unit test failure here (where the value is not mocked).
Should we fail back to x86_64? It's only arm64 and x86_64, and an equivalent to:
switch(runtime.GOARCH); {
case "arm64":
  binary = "cos-tool-arm64"
default:
  binary = "cos-tool-amd64"
}

Is probably a sane enough default

I think it's probably good that it breaks if the user were to use some other fringe architecture, allowing them to raise an issue about having the arch added instead of troubleshooting for ages.

In that case, it's probably worth using the Run-On-Arch action to ensure that these are actually set.
Whatever scenario in the GH runners (Google hasn't turned up much) which leads to flaky architecture detection should be squashed unless we intend to globally patch it at the top of every unit test. Or use Run-On-Arch


Am Fixbuild

@yjkimjunior since you're ahead of us in the tz (not sure if you already finished your day), I'll merge this into yours, and then merge yours

I can push that to this branch if you don't mind

No problem, as you wish. If you want to continue on your branch, you can also just close this PR


⚠ Bofrost.de has degraded performance

In 20a86ab, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1756 ms

Resolved: Bofrost.de performance has improved in 3993794.


Type declarations missing for this.$i18n and this.$t

Version
v6.5.0
Reproduction link
https://codesandbox.io/s/github/brandonwoo/nuxt-i18n-type-definition-issue
Steps to reproduce
npm install
npm run dev
What is expected ?
this.$i18n.t
this.$t
should be accessible on Vue instance without compile errors
What is actually happening?
compile error will be output in console
 ERROR  ERROR in /Users/JPZ2487/Workspace/test/pages/index.vue(41,18):                              nuxt:typescript 14:37:14
41:18 Property '$t' does not exist on type '{ components: { Logo: {}; }; data(): any; }'.
    39 |   data () {
    40 |     return {
  > 41 |       test: this.$t('test'),
       |                  ^
    42 |       test2: this.$i18n.t('test')
    43 |     }
    44 |   }

 ERROR  ERROR in /Users/JPZ2487/Workspace/test/pages/index.vue(42,19):                              nuxt:typescript 14:37:14
42:19 Property '$i18n' does not exist on type '{ components: { Logo: {}; }; data(): any; }'.
    40 |     return {
    41 |       test: this.$t('test'),
  > 42 |       test2: this.$i18n.t('test')
       |                   ^
    43 |     }
    44 |   }
    45 | }

This bug report is available on Nuxt community (#c399)

So you haven't added the vue-shim.d.ts shim, like documentation suggest to do but to be fair that appears to make no difference for Vue files with lang="ts" script blocks. It seems like for TS components you need to explicitly annotate your Vue components (pages and components) with Vue.extend(). I've added those in my fork: https://codesandbox.io/s/laughing-hertz-x4bem . Examples at https://typescript.nuxtjs.org/cookbook/components/#template also use that so I guess it's the official way of doing things.
Just don't test my fork on codesandbox as it appears that it's UI is not really working correctly in terms of types. Export to zip and test locally instead.
@kevinmarrec Can you confirm my observation that vue-shim.d.ts doesn't really work when using Options API with TS and that Vue.extend() is the way to go?
But in the end there is no issue with nuxt-i18n types. You just need to make sure that your components are export with proper types.

@rchl still getting the same issue with my project, using Vue.extend. updated the sandbox code, please take a look again

doesn't seem to display properly on codesandbox, please run locally instead
https://github.com/brandonwoo/nuxt-i18n-type-definition-issue
i think i found the problem in my project
i'm injecting another plugin into the Vue instance in my project, and defining the interface for that seems to overwrite the nuxt-i18n type definitions, resulting in an error
https://github.com/brandonwoo/nuxt-i18n-type-definition-issue/blob/master/plugins/app-api.ts#L10-L12
is there any way to preserve nuxt-i18n type definitions on the Vue instance while adding definitions of my own?

think i may be wrong, I still get the compilation error even after removing my custom plugin
https://github.com/brandonwoo/nuxt-i18n-type-definition-issue/tree/removed-plugin

In your latest case you have a typo in name of this module in tsconfig.json. nuxt-18n instead of nuxt-i18n. After fixing that (and removing your company's specific package), it works.

@rchl ah, dumb typo miss on my side... thanks for the help, much appreciated


fix the operation of the uninterruptible flag and add test checking it.

[x] I have reviewed the Contributing Guidelines
[x] I agree to license my contributions under the same license as in this repository
[x] I have verified that the tests pass

I checked this pull request with my simple example and I am not able to get the desired outcome. The case I run is a simple two federate case where each federate publishes one topic and then subscribes to the topic from the other federate.
Federate 1 JSON config:
{
  "name": "fed1",
  "log_level": 2,
  "observer": false,
  "rollback": false,
  "only_update_on_change": false,
  "only_transmit_on_change": false,
  "source_only": false,
  "uninterruptible": true,
  "coreType": "zmq",
  "coreName": "",
  "coreInit": "1",
  "maxIterations": 1,
  "period": 1.0,
  "offset": 0.0,
  "timeDelta": 1.0,
  "outputDelay": 0,
  "inputDelay": 0,
  "publications":[
  {
    "key":"pub",
    "type":"double",
    "unit":"m",
    "global":false,
    "info":"This is a random string containing some info for fed1 publication"
  }],
  "subscriptions":[
  {
    "key":"fed2/pub",
    "type":"double",
    "unit":"m",
    "required":false,
    "info":"This is a random string containing some info for fed1 subscription"
  }]
}

Federate 2 JSON config:
{
  "name": "fed2",
  "log_level": 2,
  "observer": false,
  "rollback": false,
  "only_update_on_change": false,
  "only_transmit_on_change": false,
  "source_only": false,
  "uninterruptible": false,
  "coreType": "zmq",
  "coreName": "",
  "coreInit": "1",
  "maxIterations": 1,
  "period": 1.0,
  "offset": 0.0,
  "timeDelta": 1.0,
  "outputDelay": 0,
  "inputDelay": 0,
  "publications":[
  {
    "key":"pub",
    "type":"double",
    "unit":"m",
    "global":false,
    "info":"{\"fake\": \"This is a random string containing some json for fed2 publication\"}"
  }],
  "subscriptions":[
  {
    "key":"fed1/pub",
    "type":"double",
    "unit":"m",
    "required":false,
    "info":"{\"fake\": \"This is a random string containing some json for fed2 subscription\"}"
  }]
}

Before testing with the uninterruptible flag set to true I tested my case with it set to false (just to make sure everything runs as expected). The outcome of this experiment seems good:
Federate 1:
helics@quantum:~/valueFederate$ ./valueFed valueFed1.json 5
fed name -> fed1
config name -> valueFed1.json
pubs -> [fed1/pub]
subs -> [fed2/pub]
publication keys:
	 fed1/pub
subscription keys:
	 fed2/pub
entering init mode
entered init State
entered exec mode
delta time -> 5 seconds
stop time -> 11 seconds
############## TIMING ##############
    processed time 0
    requested new time 5
    requestTime call returned 1
####################################
received updated value of 0 at 1 from fed2/pub
############## TIMING ##############
    processed time 1
    requested new time 6
    requestTime call returned 2
####################################
received updated value of 1 at 2 from fed2/pub
############## TIMING ##############
    processed time 2
    requested new time 7
    requestTime call returned 3
####################################
received updated value of 2 at 3 from fed2/pub
############## TIMING ##############
    processed time 3
    requested new time 8
    requestTime call returned 4
####################################
received updated value of 3 at 4 from fed2/pub
############## TIMING ##############
    processed time 4
    requested new time 9
    requestTime call returned 5
####################################
received updated value of 4 at 5 from fed2/pub
############## TIMING ##############
    processed time 5
    requested new time 10
    requestTime call returned 6
####################################
received updated value of 5 at 6 from fed2/pub
############## TIMING ##############
    processed time 6
    requested new time 11
    requestTime call returned 7
####################################
received updated value of 6 at 7 from fed2/pub
############## TIMING ##############
    processed time 7
    requested new time 12
    requestTime call returned 8
####################################
received updated value of 7 at 8 from fed2/pub
############## TIMING ##############
    processed time 8
    requested new time 13
    requestTime call returned 9
####################################
received updated value of 8 at 9 from fed2/pub
############## TIMING ##############
    processed time 9
    requested new time 14
    requestTime call returned 10
####################################
received updated value of 9 at 10 from fed2/pub
############## TIMING ##############
    processed time 10
    requested new time 15
    requestTime call returned 11
####################################
about to exit

Federate 2:
helics@quantum:~/valueFederate$ ./valueFed valueFed2.json 1
fed name -> fed2
config name -> valueFed2.json
pubs -> [fed2/pub]
subs -> [fed1/pub]
publication keys:
	 fed2/pub
subscription keys:
	 fed1/pub
entering init mode
entered init State
entered exec mode
delta time -> 1 seconds
stop time -> 11 seconds
############## TIMING ##############
    processed time 0
    requested new time 1
    requestTime call returned 1
####################################
received updated value of 0 at 1 from fed1/pub
############## TIMING ##############
    processed time 1
    requested new time 2
    requestTime call returned 2
####################################
received updated value of 1 at 2 from fed1/pub
############## TIMING ##############
    processed time 2
    requested new time 3
    requestTime call returned 3
####################################
received updated value of 2 at 3 from fed1/pub
############## TIMING ##############
    processed time 3
    requested new time 4
    requestTime call returned 4
####################################
received updated value of 3 at 4 from fed1/pub
############## TIMING ##############
    processed time 4
    requested new time 5
    requestTime call returned 5
####################################
received updated value of 4 at 5 from fed1/pub
############## TIMING ##############
    processed time 5
    requested new time 6
    requestTime call returned 6
####################################
received updated value of 5 at 6 from fed1/pub
############## TIMING ##############
    processed time 6
    requested new time 7
    requestTime call returned 7
####################################
received updated value of 6 at 7 from fed1/pub
############## TIMING ##############
    processed time 7
    requested new time 8
    requestTime call returned 8
####################################
received updated value of 7 at 8 from fed1/pub
############## TIMING ##############
    processed time 8
    requested new time 9
    requestTime call returned 9
####################################
received updated value of 8 at 9 from fed1/pub
############## TIMING ##############
    processed time 9
    requested new time 10
    requestTime call returned 10
####################################
received updated value of 9 at 10 from fed1/pub
############## TIMING ##############
    processed time 10
    requested new time 11
    requestTime call returned 11
####################################
about to exit

Then I tested with the uninterruptible flag to true in Federate 1 and the simulation hangs with the following output:
Federate 1:
helics@quantum:~/valueFederate$ ./valueFed valueFed1.json 5
fed name -> fed1
config name -> valueFed1.json
pubs -> [fed1/pub]
subs -> [fed2/pub]
publication keys:
	 fed1/pub
subscription keys:
	 fed2/pub
entering init mode
entered init State
entered exec mode
delta time -> 5 seconds
stop time -> 11 seconds
31152-CYSre-s0yyM-lV4gF-S3StJ<PHONE_NUMBER>):: core tick
31152-CYSre-s0yyM-lV4gF-S3StJ<PHONE_NUMBER>):: core tick

Federate 2:
helics@quantum:~/valueFederate$ ./valueFed valueFed2.json 1
fed name -> fed2
config name -> valueFed2.json
pubs -> [fed2/pub]
subs -> [fed1/pub]
publication keys:
	 fed2/pub
subscription keys:
	 fed1/pub
entering init mode
entered init State
entered exec mode
delta time -> 1 seconds
stop time -> 11 seconds
############## TIMING ##############
    processed time 0
    requested new time 1
    requestTime call returned 1
####################################
received updated value of 0 at 1 from fed1/pub
31160-JTT2k-D5tK8-oLHsA-nhyNO<PHONE_NUMBER>):: core tick
31160-JTT2k-D5tK8-oLHsA-nhyNO<PHONE_NUMBER>):: core tick

Let me know if you need more information to reproduce.

Adding valueFederate.cpp code here as well as that might give you some insight as well:
/*
Copyright © 2017-2018,
Battelle Memorial Institute; Lawrence Livermore National Security, LLC; Alliance for Sustainable Energy, LLC
All rights reserved. See LICENSE file and DISCLAIMER for more details.
*/
#include <helics/application_api/ValueFederate.hpp>
#include <helics/application_api/queryFunctions.hpp>
#include <helics/application_api/Inputs.hpp>
#include <helics/application_api/Publications.hpp>
#include <thread>
#include <iostream>

int main (int argc, const char * const *argv)
{
    if (argc != 3) {
         std::cerr << "Expecting two arguments. You have entered " << argc-1 << " arguments" << std::endl;
         return 0;
    }

    // string for the name of the federate configuration file
    std::string configString = argv[1];

    // loads configuration, registers interfaces and starts the federate
    //auto vFed = new helics::ValueFederate ("temp_name", configString);
    auto vFed = new helics::ValueFederate (configString);

    // some user messages
    std::cout << "fed name -> " << vFed->getName() << std::endl;
    std::cout << "config name -> " << configString << std::endl;

    //std::cout << "publication count -> " << vFed->getPublicationCount() << std::endl;
    //std::cout << "subscription count -> " << vFed->getSubscriptionCount() << std::endl;

    std::cout << "pubs -> " << vFed->query(vFed->getName(), "publications") << std::endl;
    std::cout << "subs -> " << vFed->query(vFed->getName(), "subscriptions") << std::endl;

    // returns a list of publication keys in the configuration file
    auto pubKeys = vectorizeAndSortQueryResult(vFed->query(vFed->getName(), "publications"));

    // returns a list of subscription keys in the configuration file
    auto subKeys = vectorizeAndSortQueryResult(vFed->query(vFed->getName(), "subscriptions"));

    // print out the publications for the user
    std::cout << "publication keys:" << std::endl;
    for(auto it = pubKeys.begin(); it != pubKeys.end(); ++it) {
        std::cout << "\t " << *it << std::endl;
    }

    // print out the subscriptions for the user
    std::cout << "subscription keys:" << std::endl;
    for(auto it = subKeys.begin(); it != subKeys.end(); ++it) {
        std::cout << "\t " <<  *it << std::endl;
    }

    // this federate is only built to handle one publication and one subscription. Will exit if that is not the case!!
    //if (vFed->getPublicationCount() > 1 || vFed->getSubscriptionCount() > 1) {
    if (pubKeys.size() > 1 || subKeys.size() > 1) {
        std::cerr << "This federate only supports one subscription and publication. You specified more and only the first one will be used!!" << std::endl;
    }

    // get the id of the publication. In the future this could support more than one!
//    auto id = vFed->getPublicationId(pubKeys[0]);
    auto id = vFed->getPublication(pubKeys[0]);
    // get the id of the subscription. In the future this could support more than one!
//    auto subid = vFed->getSubscriptionId(subKeys[0]);
    auto subid = vFed->getSubscription(subKeys[0]);

    // we are done with the setup, start the initialization
    std::cout << "entering init mode" << std::endl;
    vFed->enterInitializingMode ();
    std::cout << "entered init State" << std::endl;

    // now enter execution state
    vFed->enterExecutingMode ();
    std::cout << "entered exec mode" << std::endl;

    double currentTime = 0;
    double currentTimeTemp = 0;
    double nextTime = 0;
    double stopTime = 11;
    double timeDelta = atof(argv[2]);;
    std::cout << "delta time -> " << timeDelta << " seconds" << std::endl;
    std::cout << "stop time -> " << stopTime << " seconds" << std::endl;
    // simple loop that count from 0 through 9 to test pub/sub
    do {
        currentTimeTemp = currentTime;
        // publish our variable
        //id.publish("sent at " + std::to_string(currentTime));
        id.publish(currentTime);

          // if our subscription is updated lets get the value
        if (vFed->isUpdated(subid))
        {
            // get the latest value for the subscription
            auto val = vFed->getDouble(subid);
            std::cout << "received updated value of " << val << " at "<< currentTime << " from " << vFed->getTarget(subid) << std::endl;
        }

        nextTime = currentTime+timeDelta;
        currentTime = vFed->requestTime ((helics::Time) nextTime);

        std::cout << "############## TIMING ##############" << std::endl;
        std::cout << "    processed time " << currentTimeTemp << std::endl;
        std::cout << "    requested new time " << nextTime << std::endl;
        std::cout << "    requestTime call returned " << currentTime << std::endl;
        std::cout << "####################################" << std::endl;

    } while (currentTime < stopTime);

    std::cout << "about to exit" << std::endl;
    // end the federate
    vFed->finalize ();
    return 0;
}

any chance you can add a test case like I added but better matching your case.  there must be something with the two way transfer that is not being updated

Just added a test case to this branch that shows the hang I am getting. It does seem to be something with the two way transfer that locks the simulation.

my guess is that the time_next is not being updated properly.  so it is sending the wrong times to the other federate,. so it can't advance properly.  I may not be able to fix it until tonight since I am at the trc meeting

@Jacobhansens   try it again now.


++$+ZNEW-VIDEO Oviya Leaked.Video 2024 Tamil Actress Oviya' Leaked.Videos LINK Viral on Social Media X Today Trending TAMIL ACTRESS MMS Oviya Latest Video Leaks

+++ZNEW-VIDEO Oviya Leaked.Video 2024 Tamil Actress Oviya' Leaked.Videos LINK Viral on Social Media X Today Trending TAMIL ACTRESS MMS Oviya Latest Video Leaks..
12 sec ago, Oviya, +[Trending√ZVIDEOs!] Oviya Viral Leaked Video 2024 LINK On Social Media X Twitter [MMS70K]
TAMIL ACTRESS-MMS* Oviya Viral Leaked Full Video 2024 LINK Viral on Social Media X Today
Oviya's latest video has become a viral sensation, capturing the attention of social media users and sparking extensive discussions. This emerging digital creator, known for her engaging content, particularly her recent Oviya-themed video, has seen a surge in interest from fans eager to find where they can watch the full version. Many are turning to platforms such as Instagram, X (formerly Twitter), and Discord for access to her content.
As the excitement builds, fans are actively sharing their reactions online, showcasing the powerful influence of social media in elevating creators like Oviya into the spotlight. With her unique style and creativity, she has quickly established herself as a notable figure in the digital landscape, leaving many curious about her upcoming projects and content. The viral success of her video highlights how social media serves as a significant platform for creators to gain visibility and interact with their audience

FG

.
.
.

vf
.
.

.

.
.


Getting Piece Moves

I couldn't test this because you didn't commit your changes to the main scene.


Add definitions for angular-toasty

I created definitions for angular-toasty.
I'm far from being a TypeScript expert, so I hope they are fine. They work for me so far.
Cheers
Dominik

angular-toasty/angular-toasty.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/angular-toasty
http://bower.io/search/?q=angular-toasty
others?

[X] has a test file? (angular-toasty/angular-toasty-tests.ts or others)
[ ] pass the Travic-CI test?

@muenchdo thanks mate!


HTTP decoders should directly decode a Full object when body is empty.

Expected behavior
(original idea of @vietj)
Netty should be optimized for HTTP requests without a body, which is the most common use case.
Netty should be able to detect such empty body and let users process such request in one single event loop run.
Actual behavior
User code has to process 2 events (HttpResponse and LastHttpContent#EMPTY_LAST_CONTENT) or having to resort to HttpObjectAggregator that buffers messages.
Netty version
4.1

@slandelle I am a bit concerned that this may break users code as some may use something like:
if (msg instance of HttpRequest) {
} else if (msg instanceof HttpContent) {
}

Sure this is not 100 % right but I can see that some users use this. What we may could do is to make it configure tho.
WDYT ?

I am a bit concerned that this may break users code as some may use something like

When you know for sure you don't have an HttpObjectAggregator in the pipeline, that's currently 100% correct. Or am I missing something?
I had Netty 5 in mind for this, but Netty 4 with opt-in could be a good idea so we can experiment on performance boost.
Pretty sure @vietj would be interested in this for Vert.x and TechEmpower benchmarks :P

ping @vietj


SDK diff failing for Internal 8.0.1xx brach due to Aspire addition

Failing build: https://dev.azure.com/dnceng/internal/_build/results?buildId=2337526&view=logs&j=9ebdebc1-a11e-50fc-dfa8-efc527e54bda&t=6faae72f-bad4-57f5-43ff-88548eddc01c
Seems that the SDK added a workload manifest for Aspire that is breaking the diff test for 8.0.1xx.
Probably adding the files to the baseline is the best approach, but will verify before-hand

Already fixed in public by Matt -> https://github.com/dotnet/installer/pull/17893
Was not present in the internal branch against which we execute the tests due to failing mirroring -> https://dev.azure.com/dnceng/internal/_build/results?buildId=2335710&view=results
Mirrored the required changes manually in https://dev.azure.com/dnceng/internal/_git/dotnet-installer/pullrequest/36036, will wait for the CIs to finish and will re-ran the pipeline to verify the fix

Build passed, closing as Done


Making navbar responsive

Making navbar responsive. The alignment of the list and the button in navbar can be changed
Also I think the logo is enough the name need not to be written in the navbar

@Vanshiii203 Go ahead


Video don't restart after going back and forward

Testing zuck.js in my project I found a problem I think may be a bug, when a video starts and a go back to the past stories and click to go forward again the video start to play when stopped. I think the correct behavior is to start the video from the beginning.
I tested on https://ramon82.com/demo/zuck.js/index.html and the same problems happens, this is a know problem? There's some way to fix it?

yes sure is it possible to fix it ?
please help


Continue #127: Add NEII vocabs pid-proxy rewrites and corresponding tests

This is the continuation of PR #127.

Over to @nicholascar.

Nick not being around at the moment ... @ashleysommer , how soon do you need this committed? (I can do it now, of course, but there's a possibility of a meeting of other control body members in the next little while for which this could serve as live "training material".)

@rwalkerands
I fear this one might be a little heavy (and a bit too script-generated) to serve as training material. But if its perhaps the only material available to do training with, it can suffice.


DBZ-8034 New internal option to relax quote handling in DML parser

https://issues.redhat.com/browse/DBZ-8034
https://issues.redhat.com/browse/DBZ-8200

/packit test --labels oracle

Test failures are unrelated @jpechane

@Naros Applied, thanks! There are still cases when this can fail but I'd say it is good enough just know.


Internationalize date

#28

Hi @edg2s can you test with the current data and see if you run into the issue? I'm seeing it as of now, and think this is related to an earlier issue to do with date array length changing between NYT and JHU data. I'm thinking of rolling back this change for now until we can resolve this.

Went ahead and rolled this back. Will look into the bug. https://github.com/aatishb/covidtrends/commit/f4b966d0eb920c346b718e52b040d419ecf0d0b6
This issue as I see it is when NYT & JHU have the same latest date, switching from JHU -> NYT -> JHU breaks the graph. Let me know if you aren't able to reproduce it.

Hi @edg2s can you test with the current data and see if you run into the issue?

What is the issue? I can't see it described anywhere...

When NYT & JHU have the same latest date, switching from JHU -> NYT -> JHU breaks the graph by raising the error in the console log: TypeError: "date is undefined"
When NYT & JHU have different last dates, this is not an issue.


all: upgrade dep github.com/valyala/fasthttp

Upgrade fasthttp.

I try to review dependency changes, but these dependencies changing are huge, one has over 100k in line changes. They all appear to be frequently used libs though, so I'm going to move ahead regardless.


Data model

use cases

setup if a user share its extension with other accounts.
track the timeline that a user has its extension active.
track the pages uploaded by a user
track the earnings of a user
track the payouts of a user

Done.


Implement [skip ci] for Jenkins

GitHub Actions already respects this (link), this implements it for Jenkins as well. When [skip ci] is in the commit message of the head commit and in the PR title, the only things that will run are the sanity check + lint, so CI should be much quicker if someone manually sets this.
This is useful for PRs that need to get landed quickly and the outcome is pretty well known, such as reverts (and should only be used in these situations). Tested in #9686
Also see the related forum discussion: https://discuss.tvm.apache.org/t/rfc-ci-add-a-skip-ci-tag-to-shortcut-builds-and-tests/11589/10
@areusch

couple thoughts:

i think we should add unit tests for something like this so we can be sure it works correctly. it would be great to consider the cases of:

main has [skip ci] commit
PR branch has [skip ci] commit
no [skip ci] commit exists

we might consider submitting this for RFC considering as it's a change to the development process.

i'm also curious about @leandron's comment about flagging skipped PRs.

There are some test usages here showing how skip is triggered (i.e. the CI on the first commit in that PR takes only a few minutes): https://github.com/apache/tvm/pull/9686

Confirming sounds good, it should be ready to push now

looks like the branch failed due to a flaky test. given passing tests linked from https://github.com/apache/tvm/pull/9686, i think we can merge and be confident we aren't going to break the Jenkinsfile


[MD] func testing for data source management listing

Description
[MD] func testing for data source management listing
Issues Resolved
https://github.com/opensearch-project/OpenSearch-Dashboards/issues/2576
Screenshots
https://user-images.githubusercontent.com/109986843/195906635-449eef59-0a43-47a0-99f8-f309d502dd1b.mp4
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

related to https://github.com/opensearch-project/opensearch-build/issues/2743

The bundle test workflow will fail due to this https://github.com/opensearch-project/opensearch-build/issues/2743

Since 3.0.0 latest is not stable and your PR changed the bundle workflow, one workaround is to send a PR against 2.x branch so that it can trigger a workflow against 2.4.0 (Similar to https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3253106118/jobs/5340048924#logs)
We can check if the "Bundle Snapshot based E2E Cypress tests workflow for core Dashboards" is successful against 2.4.0.
@mpabba3003 @yibow98

Since 3.0.0 latest is not stable and your PR changed the bundle workflow, one workaround is to send a PR against 2.x branch so that it can trigger a workflow against 2.4.0 (Similar to https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3253106118/jobs/5340048924#logs)
We can check if the "Bundle Snapshot based E2E Cypress tests workflow for core Dashboards" is successful against 2.4.0. (The dummy PR to 2.x branch could be discarded after the validation.)
@mpabba3003 @yibow98

@tianleh  #336

Since 3.0.0 latest is not stable and your PR changed the bundle workflow, one workaround is to send a PR against 2.x branch so that it can trigger a workflow against 2.4.0 (Similar to https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3253106118/jobs/5340048924#logs)
We can check if the "Bundle Snapshot based E2E Cypress tests workflow for core Dashboards" is successful against 2.4.0. (The dummy PR to 2.x branch could be discarded after the validation.)
@mpabba3003 @yibow98

I raised a PR against 2.x branch and the Bundle Snapshot based E2E Cypress tests workflow for core Dashboards" is successful against 2.4.0. Reference: https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3275360924/jobs/5390106545. Please ignore the one of cypress test case failed, I will fix it in my code. All other check pass.

Since 3.0.0 latest is not stable and your PR changed the bundle workflow, one workaround is to send a PR against 2.x branch so that it can trigger a workflow against 2.4.0 (Similar to https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3253106118/jobs/5340048924#logs)
We can check if the "Bundle Snapshot based E2E Cypress tests workflow for core Dashboards" is successful against 2.4.0. (The dummy PR to 2.x branch could be discarded after the validation.)
@mpabba3003 @yibow98

Please take a look at #336  to check if all tests are passing or not

See that the tests are passing for 2.4 https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3275760558/jobs/5391007264 Great work!

Rerun https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/3275934146/jobs/5392212603 is now successful on this PR


Stopped inlining st* classes?

Describe the bug
SVGO is not inlining st[1-9] classes
To Reproduce
Steps to reproduce the behavior:

get this file: https://gist.githubusercontent.com/waltercruz/7bdf372457effd3f304b391b54b3c813/raw/b6460d64874f59b05eb2e5261aeb370a2ab1aec8/instagram.svg
compress it with svgo

Expected behavior
st1, st2 and st3 classes should be inlined
Desktop (please complete the following information):

SVGO 2.7.0
NodeJs 16.10
Mac OS Catalina

oops, my mistake. Closing!

what was it?

in the

Actually minifyStyles should remove unused classes. Will take a look later.

Hi, I've faced with the same issue with different svg image: https://gist.githubusercontent.com/mahnunchik/23066a318de2956df3c769c87e4e6cbd/raw/e0f246d0e26ce770f3c3adca748a5a454907f8bb/crypto-com-coin-cro-logo.svg
<style>
    .st1{fill:url(#SVGID_00000016760000628848746960000000291323288447655331_)}
</style>

st1 class not used and not removed.


generate_mesh.py: improve generation of history metadata

Closes #9

What if the current dir has uncommitted changes, or the changes haven't been pushed to the remote? Should the script abort and request a git commit and push so the hash will be correct?
FYI I split off https://github.com/COSIMA/make_025deg_topo/blob/master/finalise.sh as a separate script to allow experimental runs of the script, and only committing and recording provenance as a separate final step.

What if the current dir has uncommitted changes, or the changes haven't been pushed to the remote? Should the script abort and request a git commit and push so the hash will be correct?
FYI I split off https://github.com/COSIMA/make_025deg_topo/blob/master/finalise.sh as a separate script to allow experimental runs of the script, and only committing and recording provenance as a separate final step.

What about issuing a warning? Then one can still do experimental runs, but hopefully it will remind people to push before generating production versions?

Since we will want to record this sort of detailed provenance for all our generated files, could/should these parts be split off as a general-purpose tool somehow?

Yes definitely! I think a good options is to develop esmgrids to become our package for working with OM3 grids (conversion, generation etc). The mesh generation functionality could be moved there, as could the provenance metadata utils.

I didn't review in detail but maybe we should add a README, or a comment at the top of the file about the git commits. i.e.
The hash of the git commit is added to the metadata of the mesh netcdf files this script produces to uniquely identify the configuration used to generate the mesh file. To produce mesh files for sharing, ensure you are using a version of this script which is committed to git. For mesh files intended for released configuration, use the latest version checked in to the main branch of the git repository.

also needs to be pushed to github:

The hash of the git commit is added to the metadata of the mesh netcdf files this script produces to uniquely identify the configuration used to generate the mesh file. To produce mesh files for sharing, ensure you are using a version of this script which is committed to git and pushed to GitHub. For mesh files intended for a released configuration, use the latest version checked in to the main branch of the git repository.


uploading via PhpSecLib with PEM file doesn't work

Deployer version: 3.0.10
PHP version: 5.6.9
Deployment target(s) OS - Ubuntu 12.04
The upload function gives some pretty cryptic errors and fails when authenticating with PEM and using PhpSecLib for uploading. I get:
  [RuntimeException]
  NET_SFTP_STATUS_FAILURE: Failure
  NET_SFTP_STATUS_FAILURE: Failure
  NET_SFTP_STATUS_FAILURE: Failure

While the command line scp works just fine:
scp -i aws.pem file.php ubuntu@somehost.amazonaws.com:/var/www/location

Bug in PhpSecLib?

It appears that way. It seems to be messing packets up in the SFTP class

Is somebody have a work around ? that will realy help me.
Thx

The isssue still exists. I get this while using upload:
NET_SFTP_STATUS_FAILURE: Failure
...
NET_SFTP_STATUS_FAILURE: Failure
NET_SFTP_STATUS_FAILURE: Failure

Exception trace:
 () at phar://D:/bin/deployer/deployer.phar/src/Server/Remote/PhpSecLib.php:136
 Deployer\Server\Remote\PhpSecLib->upload() at phar://D:/bin/deployer/deployer.phar/src/functions.php:373
 upload() at D:\xampp\htdocs\domizeli.ru\deploy.php:37
 {closure}() at n/a:n/a
 call_user_func() at phar://D:/bin/deployer/deployer.phar/src/Task/Task.php:79
 Deployer\Task\Task->run() at phar://D:/bin/deployer/deployer.phar/src/Executor/SeriesExecutor.php:43
 Deployer\Executor\SeriesExecutor->run() at phar://D:/bin/deployer/deployer.phar/src/Console/TaskCommand.php:82
 Deployer\Console\TaskCommand->execute() at phar://D:/bin/deployer/deployer.phar/vendor/symfony/console/Command/Command.php:259
 Symfony\Component\Console\Command\Command->run() at phar://D:/bin/deployer/deployer.phar/vendor/symfony/console/Application.php:840
 Symfony\Component\Console\Application->doRunCommand() at phar://D:/bin/deployer/deployer.phar/vendor/symfony/console/Application.php:192
 Symfony\Component\Console\Application->doRun() at phar://D:/bin/deployer/deployer.phar/vendor/symfony/console/Application.php:123
 Symfony\Component\Console\Application->run() at phar://D:/bin/deployer/deployer.phar/src/Deployer.php:101
 Deployer\Deployer->run() at phar://D:/bin/deployer/deployer.phar/bin/dep:62
 require() at D:\bin\deployer\deployer.phar:4

I would love to do whatever I can to fix this issue, but that is mainly testing. I am on Windows. Is it not broken on other OSs. This bug seems to be rather important. Is there any chance it will be worked on some time soon?

I'm getting the same error when I pass only directory path and not full file path for the second parameter.
For example this throws the mentioned error:
upload('test.txt', '/some/upload/dir');
but this doesn't
upload('test.txt', '/some/upload/dir/test.txt');


Add method that can choose the signals for waiting

Hi thanks for do,
In this PR I fixed a typo, updated the dependencies and added a method to define the signals you want to wait for.
I would like to use it to wait for the CTRL+C (with os.Interrupt)
AMA

Thanks for review and merging