Upgrade to rails <IP_ADDRESS>

based off of https://github.com/ausaccessfed/saml-service/pull/266
TODO:

Address ~160 format violations
Fix ~173 tests

Dropping d73c3f1 and 4aa0db3 since they cuase an obsurd amount of tests to break ~160. leaving a reference to them though as they could be useful

Currently only failing 6 specs:
Finished in 6 minutes 13 seconds (files took 5.59 seconds to load)
2085 examples, 6 failures

Failed examples:

rspec ./spec/controllers/metadata_query_controller_spec.rb:188 # MetadataQueryController#all_entities GET valid client request MetadataInstance does allow rendering all entities initial request uncached server side cache updates server side cache
rspec ./spec/controllers/metadata_query_controller_spec.rb:659 # MetadataQueryController#tagged_entities GET valid client request MetadataInstance has entities matching tag initial request uncached server side cache updates server side cache
rspec './spec/controllers/metadata_query_controller_spec.rb[1:2:1:1:1:1:1:1:1:2:1]' # MetadataQueryController#specific_entity With URI identifier EntityDescriptor GET valid client request valid entity_descriptor initial request uncached server side cache updates server side cache
rspec './spec/controllers/metadata_query_controller_spec.rb[1:2:1:2:1:1:1:1:1:2:1]' # MetadataQueryController#specific_entity With URI identifier RawEntityDescriptor GET valid client request valid entity_descriptor initial request uncached server side cache updates server side cache
rspec './spec/controllers/metadata_query_controller_spec.rb[1:2:2:2:1:1:1:1:1:2:1]' # MetadataQueryController#specific_entity With sha1 identifier RawEntityDescriptor GET valid client request valid entity_descriptor initial request uncached server side cache updates server side cache
rspec './spec/controllers/metadata_query_controller_spec.rb[1:2:2:1:1:1:1:1:1:2:1]' # MetadataQueryController#specific_entity With sha1 identifier EntityDescriptor GET valid client request valid entity_descriptor initial request uncached server side cache updates server side cache

Not sure why, been a bit stuck trying to troubleshoot. My best guess is it is related to some change made in cc9ad75

I realised when I ran rubocop -A it was using the latest version on my system, and addressing all those errors caused major problems. The correct command was bundle exec rubocop -a

Hmm, something odd happening here, 86d71fb has passed on re-runs but not the first run. Have force pushed it but still failing
https://ap-southeast-2.console.aws.amazon.com/codesuite/codebuild/926857881019/projects/saml-service/build/saml-service%3A121125fd-f9e2-4a3b-a2ab-ede6cb6a4b15?region=ap-southeast-2

Based on @rianniello previous testing:
[1] pry(main)> FactoryBot::create :idp_sso_descriptor
=> #<IDPSSODescriptor @values={:id=>4, :entity_descriptor_id=>4, :organization_id=>nil, :error_url=>nil, :extensions=>nil, :created_at=>2022-03-09 05:33:59 UTC, :updated_at=>2022-03-09 05:33:59 UTC, :kind=>"IDPSSODescriptor", :enabled=>true, :want_authn_requests_signed=>false}>
[2] pry(main)> FactoryBot::create :sp_sso_descriptor
=> #<SPSSODescriptor @values={:id=>5, :entity_descriptor_id=>5, :organization_id=>nil, :error_url=>nil, :extensions=>nil, :created_at=>2022-03-09 05:34:16 UTC, :updated_at=>2022-03-09 05:34:16 UTC, :kind=>"SPSSODescriptor", :enabled=>true, :authn_requests_signed=>false, :want_assertions_signed=>false}>
[3] pry(main)> FactoryBot::create :sp_sso_descriptor
=> #<SPSSODescriptor @values={:id=>6, :entity_descriptor_id=>6, :organization_id=>nil, :error_url=>nil, :extensions=>nil, :created_at=>2022-03-09 05:34:32 UTC, :updated_at=>2022-03-09 05:34:32 UTC, :kind=>"SPSSODescriptor", :enabled=>true, :authn_requests_signed=>false, :want_assertions_signed=>false}>

~ curl http://localhost:8080/api/discovery/entities

{"identity_providers":[{"entity_id":"http://johns.co/lida/shibboleth","tags":[],"names":[],"logos":[],"descriptions":[],"single_sign_on_endpoints":{"soap":[]},"geolocations":[],"domains":[]}],"service_providers":[{"entity_id":"http://king.com/marylin_cartwright/shibboleth","tags":[],"names":[],"logos":[],"descriptions":[],"all_discovery_response_endpoints":[],"information_urls":[],"privacy_statement_urls":[]},{"entity_id":"http://lind.io/joey_boyer/shibboleth","tags":[],"names":[],"logos":[],"descriptions":[],"all_discovery_response_endpoints":[],"information_urls":[],"privacy_statement_urls":[]}]}%

I also attempted:
➜  ~ http http://localhost:8080/mdq/JMWhTb1X01EB_4gZ0zUfOQ/entities Accept:application/samlmetadata+xml
HTTP/1.1 404 Not Found
Cache-Control: max-age=600, private
Content-Type: text/html
Date: Wed, 09 Mar 2022 05:39:35 GMT
Referrer-Policy: strict-origin-when-cross-origin
Transfer-Encoding: chunked
X-Content-Type-Options: nosniff
X-Download-Options: noopen
X-Frame-Options: SAMEORIGIN
X-Permitted-Cross-Domain-Policies: none
X-Request-Id: ff3a2032-6b81-4a7f-b0a5-d020a77694c6
X-Runtime: 0.002850
X-XSS-Protection: 1; mode=block

➜  ~

But was unsure of how to locate the correct id to replace JMWhTb1X01EB_4gZ0zUfOQ after some digging

Ready for final review here @rianniello

@waldofouche see what other MetadataInstance you have lying around. (Perhaps made with that first call)

Ah I forgot to push this out! I'll push tomorrow morning

Hi guys,
I've just tried rolling this into our DEV environment and got an error:
Your bundle only supports platforms ["x86_64-darwin-20"] but your local platform
is x86_64-linux. Add the current platform to the lockfile with
`bundle lock --add-platform x86_64-linux` and try again.

and indeed, Gemfile.lock only has:
PLATFORMS
  x86_64-darwin-20

I think it might have surfaced only now because this PR switches the bundler version from 1.x to 2.x (1.17.2 to 2.3.9) - and our tooling deploys with the same version of bundler as what's recorded in Gemfile.lock...
Sorry, needs some cleanup....

Hi @vladimir-mencl-eresearch,
Sorry about that! I addressed it before in another app, but looks like I missed it here, I have created https://github.com/ausaccessfed/saml-service/pull/270 to add it in


What commit does the 'Resources' menu refer to?

The menu element mentions the commit 0dd39c3 on master. What does this refer to? I couldn't find the commit in both qlever and qlever UI repositories in the last year.

The element shows the current qleverUI commit. But you're right that there is no such commit 0dd39c3 in our repo. I can only guess that this is some custom commit not uploaded to version control running on http://qlever.cs.uni-freiburg.de/


Missing operand handler for MLIL SSA form corresponding to x86 cdq instruction

x86:
080480fd   cdq   {0x0}

LLIL:
14 @ 080480fd   edx:eax = sx.q(eax)

MLIL (SSA Form):
6 @ 080480fd   edx#1:eax#2 = 0

Attempting to access the MLIL instruction in question results in the following traceback:
>>> current_function.medium_level_il.ssa_form[6]
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "/home/ehennenfent/binaryninja/plugins/../python/binaryninja/mediumlevelil.py", line 631, in __getitem__
    return MediumLevelILInstruction(self, core.BNGetMediumLevelILIndexForInstruction(self.handle, i), i)
  File "/home/ehennenfent/binaryninja/plugins/../python/binaryninja/mediumlevelil.py", line 239, in __init__
    self.operands.append(value)
UnboundLocalError: local variable 'value' referenced before assignment

This issue does not occur when using the non-ssa form of the MLIL instruction. It likely occurs because MediumLevelILInstruction.ILOperations[core.BNGetMediumLevelILByIndex(current_function.medium_level_il.ssa_form.handle, 6).operation] returns an empty list, which results in the block of conditionals starting at line 184 in mediumlevelil.py getting skipped, so the value variable is never set.

Fixed in dev >850 per #749


Created basic landing page

Created just a landing page with content and no proper css
@CodiumAI-Agent /review

PR Review

 ⏱ Estimated effort to review [1-5]
3, because the PR includes changes across multiple files involving both frontend and backend adjustments. The changes include CSS modifications, content updates, and API endpoint changes, requiring a careful review of styling consistency, functionality, and potential impact on other components.

 🧪 Relevant tests
No

 🔍 Possible issues

Possible Bug: The removal of certain CSS classes and their replacement with new ones without a corresponding update in the HTML structure might lead to styling issues or unstyled content.

 This will make your CSS more maintainable and make it easier to implement a theme switcher or adjust styles globally. [important]

relevant linecolor: white;
relevant fileclient/frontEnd/src/Styles/home.css
suggestion

For background images, especially for sections like .page1 and .section2, consider adding background-position: center; to ensure the background covers the desired area properly on all screen sizes. [medium]

relevant linebackground-size: cover;
relevant fileForge Savant/SMPS.bru
suggestion

Ensure the new endpoint /smps is properly secured and validated to prevent unauthorized access or potential data leakage. If applicable, implement rate limiting and logging for monitoring. [important]

relevant lineurl: http://localhost:5000/smps
relevant fileclient/frontEnd/src/Components/Home.jsx
suggestion

For images like processor, graphicsCard, etc., always include meaningful alt text for accessibility reasons. This helps users who rely on screen readers to understand the content of the images. [important]

relevant line

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review which includes several types of feedbacks, such as possible PR issues, security threats and relevant test in the PR. More feedbacks can be added by configuring the tool.
The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.

When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:

/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:

[pr_reviewer]
some_config1=...
some_config2=...

See the review usage page for a comprehensive guide on using this tool.


upgrade k8s.io deps

Signed-off-by: Matias Charriere<EMAIL_ADDRESS>What does this PR do?

Upgrade giantswarm/k8sclient to 7.1.0.
Upgrade sigs.k8s.io/cluster-api to v1.5.2.
Migrate sigs.k8s.io/cluster-api-provider-aws to v2 (2.2.4).
Upgrade sigs.k8s.io/cluster-api-provider-azure to v1.11.4.
Upgrade sigs.k8s.io/controller-runtime to v0.16.3.
Upgrade other dependencies.
Replace capi.ClusterLabelName with capi.ClusterNameLabel.

What is the effect of this change to users?
Changes doesn't change UX.
What does it look like?
Changes doesn't change UX.
Any background context you can provide?
General upgrade.
What is needed from the reviewers?
Do the docs need to be updated?
No
Should this change be mentioned in the release notes?

[x] CHANGELOG.md has been updated

Is this a breaking change?
No

Since the UX is not changing, there is nothing I can comment here. Just saying thanks for the effort and diligence!


doc: Update postman and readme policy payload

Description

Update the postman collection and the curl commands to reflect the changes in policy creation payload.
#158

Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[x] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

@kss682 could you please still add the latest suggestion from @CDiezRodriguez ?


Dark mode support?

Description
A very recent update from the OSM website team itself: https://en.osm.town/@osm_tech/113482687301656785

🚀 https://www.openstreetmap.org/ is now also available in #DarkMode 🕶 🔦 🤓  #Black #OpenStreetMap #UI

Then, obviously, the next question to ask is whether the iD editor would get a dark mode update in the future.
Screenshots
As of now, there seems to have no dark mode options for the iD editor, which results in UI inconsistencies, like this example (ignore the green line):

Duplicate of #8214


AttributeError: 'module' object has no attribute 'CERT_CERT_REQUIRED'

I'm not sure why, but this suddenly started happening:
Traceback (most recent call last):
  File "zappa/cli.py", line 1010, in handle
    sys.exit(cli.handle())
  File "zappa/cli.py", line 166, in handle
    self.deploy()
  File "zappa/cli.py", line 230, in deploy
    self.zappa.create_iam_roles()
  File "/Users/rjones/Projects/Zappa/lib/Zappa/zappa/zappa.py", line 924, in create_iam_roles
    self.credentials_arn = role.arn
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/boto3/resources/factory.py", line 339, in property_loader
    self.load()
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/boto3/resources/factory.py", line 505, in do_action
    response = action(self, *args, **kwargs)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/boto3/resources/action.py", line 83, in __call__
    response = getattr(parent.meta.client, operation_name)(**params)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/client.py", line 159, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/client.py", line 483, in _make_api_call
    operation_model, request_dict)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/endpoint.py", line 117, in make_request
    return self._send_request(request_dict, operation_model)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/endpoint.py", line 146, in _send_request
    success_response, exception):
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/endpoint.py", line 219, in _needs_retry
    caught_exception=caught_exception, request_dict=request_dict)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/hooks.py", line 227, in emit
    return self._emit(event_name, kwargs)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/hooks.py", line 210, in _emit
    response = handler(**kwargs)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/retryhandler.py", line 183, in __call__
    if self._checker(attempts, response, caught_exception):
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/retryhandler.py", line 251, in __call__
    caught_exception)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/retryhandler.py", line 266, in _should_retry
    return self._checker(attempt_number, response, caught_exception)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/retryhandler.py", line 314, in __call__
    caught_exception)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/retryhandler.py", line 223, in __call__
    attempt_number, caught_exception)
  File "/Users/rjones/Projects/Zappa/baby_flask/env/lib/python2.7/site-packages/botocore/retryhandler.py", line 356, in _check_caught_exception
    raise caught_exception
AttributeError: 'module' object has no attribute 'CERT_CERT_REQUIRED'

Latest botocore, latest boto3, latest urllib3, python 2.7.10, OSX. Code hasn't touched, seems to still work on Travis but not on my local system. I'm not sure what has changed to cause this.

This was my fault. It's a weird one though: I had a file called 'ssl.py' in my project. This caused a conflict with something in boto3 or botocore. I don't really see how that namespace conflict can happen - any chance you can look into that?

From what I can tell, the call stack is:

botocore.endpoint.Endpoint._get_response
requests.session.send
requests.adapters.<*>.send()
urllib3.connectionpool.urlopen
urllib3.connectionpool._make_request
urllib3.connectionpool._validate_conn
urllib3.connection.<*>.connect
urllib3.util.ssl_.resolve_cert_reqs

The code that's triggering the attr error is here:
    if isinstance(candidate, str):
        res = getattr(ssl, candidate, None)
        if res is None:
            res = getattr(ssl, 'CERT_' + candidate)
        return res

urllib3 has a couple of checks that catch the ImportError you would normally get if you try to pull in something from a module that's missing expected attributes.  This unfortunately masks the original error.  You can see that here:
try:  # Test for SSL features
    import ssl
    from ssl import wrap_socket, CERT_NONE, PROTOCOL_SSLv23
    from ssl import HAS_SNI  # Has SNI?
except ImportError:
    pass

So as a result of executing that block above the ssl attribute exists because of your local ssl.py file in your project so the first import works), but the getattr(ssl, ...) from urllib3.util.ssl_.resolve_cert_reqs raises an AttributeError.
Let me know if there's anything else I can do to help.

Wow, thanks for diving into that one!
I think I can close this because it's not a botocore related issue. I'll notify upstream.


Changing Text of Background Message

First things first: thank you for this awesome plugin!
Is there a way to change the text of a Background Notification? We are using push notifications to send Alerts for certain Machines, and the user has the possibility to set an alias name clientside for the id of the machine (so even for the same machine two users can have different aliases in their apps). Receiving a message i would like to replace the id of the machine with the aliasname the user assigned in his app.
Is there any possibilty to intercept and modify the message before it is displayed? No problem to do this in foreground as i get the callback and can handle the payload myself. But what to do in case the app is in background?
Thanks and Cheers

Please forget the question. As the App can not only be in the background but also killed and still the notifications should work a callback makes no sense (becasue the app is not even running)


chore: update Jest

Without this update #7217 & #7219 will keep failing

CC/ @SimenB to make sure we're doing what we're supposed to do

Nothing here jumps out to me. A bit odd removing require.resolve fixes things, but shouldn't hurt.

Any particular reason why you don't go to v29? I'd be happy to help upgrading (I actually sent a PR at work 6 hours ago introducing react-router into the code base 😅)

@SimenB My plan was to first upgrade to v28 and make that work, but I can go straight to v29 as well if you think that would make it easier for you

I'm somehow surprised there's a difference. 😅 Breaking changes 28->29 (beyond snapshot format changes, which are configurable) were chosen as they were negligible.

@SimenB I just didn't look into upgrading to v29 yet 🤷♂
So if you say there's no real difference, I'll just go ahead and upgrade to v29 👍

Ah! Fair enough enough 🙂 happy to take a look if it proves troublesome - feel free to tag me 👍

@SimenB I guess the same issues with read-only globals will still be present in v29 so we still need to find a solution for that?

@SimenB we're now getting the following warning

● Validation Warning:
Unknown option "watchPlugins" with value ["jest-watch-select-projects", "jest-watch-typeahead/filename", "jest-watch-typeahead/testname"] was found.
This is probably a typing mistake. Fixing it will remove this message.
Configuration Documentation:
https://jestjs.io/docs/configuration

Looking at the docs, I don't see what we're doing wrong tbh as watchPlugins is just there 🤔
https://jestjs.io/docs/configuration#watchplugins-arraystring--string-object
Next to that we're still getting the same read-only errors as we already had in v28

TypeError: Cannot assign to read only property 'atob' of object '[object global]'

Together with the non-constructor error (which is probably because of the globals not being set)

TypeError: Response is not a constructor

Unknown option "watchPlugins"

sounds like https://github.com/jestjs/jest/issues/13576 🙁 should probably dig deeper into it and figure out the root cause.

Next to that we're still getting the same read-only errors as we already had in v28

probably https://github.com/jestjs/jest/pull/12642 (currently https://github.com/jestjs/jest/blob/642267f6848869a64861c3176eb5a6d5980639c6/packages/jest-environment-node/src/index.ts#L79-L128)
both atob and btoa are already globals in Node since v16, so I don't think you need to polyfill them at all? it should at least be done conditionally

both atob and btoa are already globals in Node since v16, so I don't think you need to polyfill them at all? it should at least be done conditionally

@SimenB We'll remove the atob & btoa polyfills in #7206, but if people decide to override a global it should just be possible, no?

I'm unable to reproduce the override issue... Works fine for mw with both JSDOM and Node envs.
I'm also not seeing that issue running yarn test:primary on my own machine.

Is there any other command I must run to reproduce it?

Is there any other command I must run to reproduce it?

@SimenB No, that's the exact command I run locally as well

I'm also not seeing that issue running yarn test:primary on my own machine.

Just tried it out myself as wel locally and indeed not seeing the read-only message anymore when using Node v18
I tried using Node v20 (we're using latest in our workflow) and got the errors showing up again
However, that makes the TypeError: Response is not a constructor error in Node v18 even stranger to me as the only changes in this PR are about Jest 🤔

note that fetch is also available globally in newer versions of node - including Response etc

The team has decided to still polyfill them (for now) as Fetch API is still marked as Experimental in the Node docs

Another datapoint in favor of going straight to jest 29... I upgraded react-router to use it a few months back and didn't run into any issues 🤞

@jenseng This PR is already using v29, but still has issues for some reason 🤔

Ok I may have tracked it down... Request and friends are coming through as undefined, thus the error when you do new ThingThatIsDeclaredButUndefined()... here's what's happening:

jest.setup.js tries to install Request and friends via installGlobals
installGlobals gets them from @remix-run/web-fetch
@remix-run/web-fetch is getting imported its the browser conditional export{
  "name": "@remix-run/web-fetch",
  // ...
  "exports": {
    ".": {
      "types": "./dist/src/lib.node.d.ts",
      "browser": "./src/lib.js", // <- it's picking this up 🤷
      "require": "./dist/lib.node.cjs",
      "import": "./src/lib.node.js"
    },
    // ...

./src/lib.js re-exports the global fetch/Request/Response, since they should be available in a browser. But they are undefined in this case.

As for why jest is picking up the browser conditional export, perhaps @SimenB would have a better idea? Maybe the require one should come first?

@jenseng Jest is only choosing the browser export when testEnvironment is set to 'jsdom', which makes sense imo
Although https://github.com/remix-run/web-std-io/pull/43 will have some changes to the browser field, I would still suspect that even if we're overriding the globals with the globals that they would be present as we're only testing with Node v20 in our Jest tests, which is a version where these globals are already present by default
So this would mean that we're just overriding these globals with themselves, which should be fine, no?

So this would mean that we're just overriding these globals with themselves, which should be fine, no?

I've confirmed that global.Request and friends are undefined at this point in ./src/lib.js. So perhaps jsdom isn't actually loaded yet at this point?

I've confirmed that global.Request and friends are undefined at this point in ./src/lib.js. So perhaps jsdom isn't actually loaded yet at this point?

Then I have no idea what's happening tbh 🤷♂
Hopefully @SimenB can give us some more insights on what's happening 🤔

JSDOM does not implement fetch (https://github.com/jsdom/jsdom/issues/1724), so it'll only be available globally in the Node environment.

I'm working on the atob override in v20 - that's a bug in Jest.

@MichaelDeBoey please give https://github.com/jestjs/jest/releases/tag/v29.6.4 a try

JSDOM does not implement fetch ([jsdom/jsdom#1724](https://github.com/jsdom/jsdom/issues/1724)), so it'll only be available globally in the Node environment.

How would you suggest we go from here?
It's not unreasonable to use built-in functionality in the browser export, so nothing we can/should do on the @remix-run/web-* side imo

please give https://github.com/jestjs/jest/releases/tag/v29.6.4 a try

Updated to latest version 👍

It's not unreasonable to use built-in functionality in the browser export, so nothing we can/should do on the @remix-run/web-* side imo

Agreed - I'd manually polyfill fetch in the jest setup file

If you want the node polyfilling and not the browser ones, you can set up a module mapper using require.resolve

@SimenB I'm personally not a fan of overriding the exportConditions I think, as that would mean all dependencies will use the node export instead of the browser one, right?
If so, this could hide some other bugs I suppose 🤔
So I guess moduleNameMapper is the only solution here?

least hacky is probably just
if (typeof window.fetch === 'undefined') {
  require('whatwg-fetch');
}

in your jest setup or something like that. I'd say that's the least likely to hide actual bugs

🎉


Ref/bug/input trigger behaviour

Must be merged after #149 Ref/feature/gizmo cancel
Remove the press interactions on all hotkey settings in the input system. They have changed the trigger from press only to release only.
#861me9jzt

Task linked: CU-861me9jzt Fix some button interactins triggered on release only


env.g.dart not generated Flutter 3.19

I followed the exact steps as described here: https://pub.dev/packages/envied. Also followed some SO threads for the same issue in 2023.
env.g.dart is not generated, whatever I try. The file is also not in the .dart_tools/build folder.
Thanks for any suggestions.

Hi,
I just tried re-generating one of my projects using Dart 3.3.0 (ships with Flutter 3.19.0) and it all works fine, so it seems to me as if you have an error in your setup. That being said, I did not try using it on a Flutter 3.19.0 project.
Can you confirm it works for you in Flutter 3.16.9?
Also, can you clone the example in this repo and try to rebuild it?

Duplicate of https://github.com/petercinibulk/envied/issues/89

Thanks @techouse, I followed the example and now it works. I had to move the files to the lib folder for it to work


fix: get the size of the file

getRawFileDocumentForStream does not return the length of file if the stream is open.

I'm not sure why this is necessary. Are you calling Bucket::getFileDocumentForStream() on an active GridFS upload stream before you close it? If so, WritableStream::close() and WritableStream::fileCollectionInsert() will not yet have been called. The latter method is where we set the md5 and length fields before inserting the fs.files document.
The current length of a writable upload stream is obtained via WritableStream::getSize(). You can access that value through the stream_stat() handler:
$size = fstat($stream)['size'];

Thank you for your help. Indeed "fstat" does the job. But I wanted to map the result of getFileDocumentForStream to an entity that implements \MongoDB\BSON\Unserializable without having any other manipulation to do.
But I understand that retrieving the size of a writable file is not necessarily logical...


No notification when item is dropped to initial position

Hi,
I am using your drag and drop feature in a GridView. To prevent the screen from rotating while an item is dragged I lock the screen to the current rotation in DraggableItemAdapter.onCheckCanStartDrag and unlock it in DraggableItemAdapter.onMoveItem in my implementation of the DraggableItemAdapter. However I don't get a call to onMoveItem if the from-position and the to-position are equal, so I cannot unlock the screen when the dragged item is dropped back to its initial position. So my question is: Do you have a workaround for that or could you provide information when the dragged item is dropped at the same position?

Hi. You can use RecyclerViewDragDropManager.OnItemDragEventListener to observe drag & drop event. It provides onItemDragStarted() and onItemDragFinished() callbacks that you are looking for :wink:
mDragDropMgr.setOnItemDragEventListener(
    new RecyclerViewDragDropManager.OnItemDragEventListener{
        @Override
        void onItemDragStarted(int position) {
            // lock screen rotation
        }

        @Override
        void onItemDragPositionChanged(int fromPosition, int toPosition) {}

        @Override
        void onItemDragFinished(int fromPosition, int toPosition, boolean result) {
            // unlock screen rotation
        }
    });

Thank you very much! That solved it.


docs: update LICENSE

Updates LICENSE with contents from license
Partial fix to #78 🦕

Yep, will need to add this file to the ignore list to prevent OwlBot updates. Closing. Will send a larger PR shortly.


Fix exchange tests

When upgrading from solc version 4.21 to 4.25 in the
trustlines-contracts-bin package the exchange tests stopped working.
This is no wonder, because the tests were just wrong. The question
remains, why they were running before even though they were wrong.

Codecov Report

Merging #232 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #232   +/-   ##
========================================
  Coverage    63.56%   63.56%
========================================
  Files           47       47
  Lines         2981     2981
========================================
  Hits          1895     1895
  Misses        1086     1086

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 3c92ff8...a9a2990. Read the comment docs.


fix: avoid type imports of the instrumented package in the built assets

Which problem is this PR solving?
https://github.com/open-telemetry/opentelemetry-js-contrib/pull/992 explains the issue very well in the context of NestJS instrumentation.
It turns out we have the same issue with importing following packages.

[x] aws-sdk
[x] cassandra-driver
[x] knex
[x] mysql2
[x] router

... and more that are not addressed in this PR.
To repro the issue one has to delete hoisted node_modules and do npm i in the metapackage.
Short description of the changes

InstrumentationBase and InstrumentationNodeModuleDefinition generics now use any,
Imported some types for aws-sdk and mysql2 implementation, how should I address any licensing topics?
made startSpan on cassandra instrumentation private,

See the issue referenced above.
I'm removing references to any types imported from the instrumented package from exported interfaces of the instrumentation.

Although I'm not sure we want to go through with this because of https://github.com/open-telemetry/opentelemetry-js/pull/2256

I suspect we will need to do this with all or most of our instrumentations, making the generic sort of pointless. Maybe we can vendor the types ourselves? That would be a lot of work though.

The generic is not used at all by @opentelemetry/instrumenation therefore I'm quite sure it would be possible to refactor the instrumentation setup in a way to hide this from public interface.
But there are quite some instrumentations which offer hooks and these hooks use types as they pass data from the instrumented library via these hooks. We could use unknown or any there and let the typescript user cast.

The generic is not used at all by @opentelemetry/instrumenation therefore I'm quite sure it would be possible to refactor the instrumentation setup in a way to hide this from public interface.
But there are quite some instrumentations which offer hooks and these hooks use types as they pass data from the instrumented library via these hooks. We could use unknown or any there and let the typescript user cast.

I agree unknown is probably the best option

The generic is not used at all by @opentelemetry/instrumenation therefore I'm quite sure it would be possible to refactor the instrumentation setup in a way to hide this from public interface.

I support doing this

But there are quite some instrumentations which offer hooks and these hooks use types as they pass data from the instrumented library via these hooks. We could use unknown or any there and let the typescript user cast.

I think for types in hooks the best option is to defer responsibility to the library author:

Use unknown or any with comments near the hook interface explaining on how to type if used.
add a dependency on @types/foo if it only implements types and is a thin package.
Copy relevant interfaces into the types.ts file to be exported from instrumentation.

Any other comments, @Flarna, @dyladan? Any feedback on this and whether this is the way you intended it to go towards will be highly appreciated.

The implementation looks good to me, but the licensing I think we need to be careful of for vendoring outside code (especially code from a company like AWS).
As an example, I looked into the aws-sdk-js types license. It is listed in https://github.com/aws/aws-sdk-js/blob/master/package.json#L87 as Apache-2.0
Full license available here: http://www.apache.org/licenses/LICENSE-2.0
The Apache-2.0 license allows redistribution of copyrighted works as long as the following conditions are met:
   4. Redistribution. You may reproduce and distribute copies of the
      Work or Derivative Works thereof in any medium, with or without
      modifications, and in Source or Object form, provided that You
      meet the following conditions:

      (a) You must give any other recipients of the Work or
          Derivative Works a copy of this License; and

      (b) You must cause any modified files to carry prominent notices
          stating that You changed the files; and

      (c) You must retain, in the Source form of any Derivative Works
          that You distribute, all copyright, patent, trademark, and
          attribution notices from the Source form of the Work,
          excluding those notices that do not pertain to any part of
          the Derivative Works; and

      (d) If the Work includes a "NOTICE" text file as part of its
          distribution, then any Derivative Works that You distribute must
          include a readable copy of the attribution notices contained
          within such NOTICE file, excluding those notices that do not
          pertain to any part of the Derivative Works, in at least one
          of the following places: within a NOTICE text file distributed
          as part of the Derivative Works; within the Source form or
          documentation, if provided along with the Derivative Works; or,
          within a display generated by the Derivative Works, if and
          wherever such third-party notices normally appear. The contents
          of the NOTICE file are for informational purposes only and
          do not modify the License. You may add Your own attribution
          notices within Derivative Works that You distribute, alongside
          or as an addendum to the NOTICE text from the Work, provided
          that such additional attribution notices cannot be construed
          as modifying the License.

      You may add Your own copyright statement to Your modifications and
      may provide additional or different license terms and conditions
      for use, reproduction, or distribution of Your modifications, or
      for any such Derivative Works as a whole, provided Your use,
      reproduction, and distribution of the Work otherwise complies with
      the conditions stated in this License.

It seems to me that we at least have to add a copyright header at the top of the file stating it is apache-2.0 licenced and has not (or has it?) been modified. We must also retain the NOTICE.txt file https://github.com/aws/aws-sdk-js/blob/master/NOTICE.txt although we are allowed to append our own wording to it.
Personally I would like to have someone with more legal knowledge look over it. Maybe we can get legal resources from the CNCF

I reached out to the CNCF to see if we can get legal resources to ensure we're doing this correctly

@dyladan, separating changes to aws code out from this PR for now then.
Let me know when you hear back from CNCF here: https://github.com/open-telemetry/opentelemetry-js-contrib/pull/1066
Sorry for the force-push... Removed the commits touching aws-sdk instrumentation from the history.


No device shows up

Hi,
After installing it, nothing shows up, I found this in the registry:
_Logger: homeassistant.config_entries
Source: config_entries.py:537
First occurred: 14:28:16 (1 occurrences)
Last logged: 14:28:16
Error occurred loading configuration flow for integration daikin_altherma: cannot import name 'DeviceInfo' from 'homeassistant.helpers.entity' (/usr/src/homeassistant/homeassistant/helpers/entity.py)_
Any help is appreciated,
Iñaki

Something is not right with HA installation. Which version of Home assistant you are using?

System Health

version
2021.1.5

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

virtualenv
false

python_version
3.8.7

os_name
Linux

os_version
5.4.86

arch
x86_64

timezone
Europe/Madrid

Home Assistant Community Store

GitHub API
ok

Github API Calls Remaining
5000

Installed Version
1.10.1

Stage
running

Available Repositories
1112

Installed Repositories
4

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Hass.io

host_os
Home Assistant OS 5.10

update_channel
stable

supervisor_version
2022.05.3

docker_version
19.03.13

disk_total
30.8 GB

disk_used
5.9 GB

healthy
true

supported
failed to load: Unsupported

board
ova

supervisor_api
ok

version_api
ok

installed_addons
File editor (5.2.0), Samba share (9.3.0), Check Home

Assistant configuration (3.10.2), Terminal & SSH (8.10.0)

Lovelace

dashboards
2

mode
auto-gen

resources
1

El dl., 13 de juny 2022, 17:38, Tadas Danielius @.***>
va escriure:

Something is not right with HA installation. Which version of Home
assistant you are using?
—
Reply to this email directly, view it on GitHub
https://github.com/tadasdanielius/daikin_altherma/issues/26#issuecomment-1154078189,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR34J6SNKIYZEGTVJPWKNDVO5IXJANCNFSM5YUCLEIA
.
You are receiving this because you authored the thread.Message ID:
@.***>

That is very old. You should upgrade you HA to the latest version.

You we're right, It was a old HA version issue...
Thank you!!!


tmm/use select pause

adds pause option for useSelect #14
adds stale value to result for useSelect #10
adds initial state tests for data hooks

Ah, amazing, I'll test it out.


add example config for mock-oidc-server

fixes #104
Important:

both the deliver application and the mock-oidc-server need to be available to the end user
the deliver application needs to connect to the mock-oidc-server for token validation
within a docker network, this means two kinds of access:

from host to container (via port mapping)
from container (deliver) to container (mock-oidc-server)

port mapping provides access for the end user as localhost:<mapped-port>
automatic network bridged mode provides access from container to container via <container-name>: <port>. So if the oidc server is named oidc, it is available as http://oidc:4000 to the deliver application. This is problematic, as it cannot provide this as the authentication endpoint to the end user ("go to http://oidc:4000/auth" -> whoops this only works for me). I tried using the network mode "host", which uses the host network, but that does not work on mac (docker desktop runs within a VM!). So I ended up, proxying "localhost:4000" to "oidc:4000" with "caddy", so the deliver application could access it the same way the end user can. Better ideas?

replaced by devcontainer pr


wrong results during training and testing Overlook module

Hi, I was trying to reproduce your work on city->foggy setting, after training source model, everything seems to be ok:
INFO:root:AP for person = 0.2404
INFO:root:AP for rider = 0.3151
INFO:root:AP for car = 0.3397
INFO:root:AP for truck = 0.0629
INFO:root:AP for bus = 0.2125
INFO:root:AP for train = 0.0568
INFO:root:AP for motorcycle = 0.1718
INFO:root:AP for bicycle = 0.2643
INFO:root:Mean AP = 0.2079
INFO:root:tested model: source/faster_rcnn_source_1_20_740.pth
INFO:root:test time: 82.1747s

but as I try to train and test the overlooking module, I got very low losses like:
INFO:root:[epoch  1][iter  170/5930] time cost: 2.384635, lr_faster_rcnn: 1.00e-04
INFO:root:                      fastRCNN loss:0.0010
INFO:root:                      cons_loss: 0.0000,gw_loss:0.0003,gw_global_loss:0.0001,label_loss:0.0003
INFO:root:                      cons_weight: 0.00,gw_ins_weight:0.10,gw_global_weight:0.10,label_weight:1.00
INFO:root:[epoch  1][iter  180/5930] time cost: 2.385559, lr_faster_rcnn: 1.00e-04
INFO:root:                      fastRCNN loss:0.0009
INFO:root:                      cons_loss: 0.0000,gw_loss:0.0003,gw_global_loss:0.0001,label_loss:0.0003
INFO:root:                      cons_weight: 0.00,gw_ins_weight:0.10,gw_global_weight:0.10,label_weight:1.00
INFO:root:im_detect: 101/500, detect time:0.036s, nms time:0.003s
INFO:root:[epoch  1][iter  190/5930] time cost: 2.432270, lr_faster_rcnn: 1.00e-04
INFO:root:                      fastRCNN loss:0.0009
INFO:root:                      cons_loss: 0.0000,gw_loss:0.0003,gw_global_loss:0.0002,label_loss:0.0004
INFO:root:                      cons_weight: 0.00,gw_ins_weight:0.10,gw_global_weight:0.10,label_weight:1.00
INFO:root:[epoch  1][iter  200/5930] time cost: 2.446995, lr_faster_rcnn: 1.00e-04
INFO:root:                      fastRCNN loss:0.0009
INFO:root:                      cons_loss: 0.0000,gw_loss:0.0004,gw_global_loss:0.0001,label_loss:0.0003
INFO:root:                      cons_weight: 0.00,gw_ins_weight:0.10,gw_global_weight:0.10,label_weight:1.00

after all iters, the results was weird:
INFO:root:VOC07 metric? Yes
INFO:root:AP for person = 0.0004
INFO:root:AP for rider = 0.0000
INFO:root:AP for car = 0.0003
INFO:root:AP for truck = 0.0000
INFO:root:AP for bus = 0.0000
INFO:root:AP for train = 0.0000
INFO:root:AP for motorcycle = 0.0000
INFO:root:AP for bicycle = 0.0001
INFO:root:Mean AP = 0.0001

Do you have similar experience about this or have you got some methods to fix this?
I only have 3090 recourses so I changed pytorch and cuda together with cuda>=11:
>>> torch.__version__
'1.8.0+cu111'

my python version is 3.9.15
and due to the error below:
ERROR:root:Error(s) in loading state_dict for vgg16:
        Missing key(s) in state_dict: "RCNN_base1.0.weight", "RCNN_base1.0.bias", "RCNN_base2.0.weight", "RCNN_base2.0.bias", "RCNN_base2.3.weight", "RCNN_base2.3.bias", "RCNN_base3.0.weight", "RCNN_base3.0.bias", "RCNN_base3.3.weight", "RCNN_base3.3.bias", "RCNN_base4.0.weight", "RCNN_base4.0.bias", "RCNN_base4.2.weight", "RCNN_base4.2.bias", "RCNN_base4.5.weight", "RCNN_base4.5.bias", "RCNN_base4.7.weight", "RCNN_base4.7.bias", "RCNN_base4.9.weight", "RCNN_base4.9.bias", "RCNN_base4.12.weight", "RCNN_base4.12.bias", "RCNN_base4.14.weight", "RCNN_base4.14.bias", "RCNN_base4.16.weight", "RCNN_base4.16.bias".
        Unexpected key(s) in state_dict: "RCNN_base.0.weight", "RCNN_base.0.bias", "RCNN_base.2.weight", "RCNN_base.2.bias", "RCNN_base.5.weight", "RCNN_base.5.bias", "RCNN_base.7.weight", "RCNN_base.7.bias", "RCNN_base.10.weight", "RCNN_base.10.bias", "RCNN_base.12.weight", "RCNN_base.12.bias", "RCNN_base.14.weight", "RCNN_base.14.bias", "RCNN_base.17.weight", "RCNN_base.17.bias", "RCNN_base.19.weight", "RCNN_base.19.bias", "RCNN_base.21.weight", "RCNN_base.21.bias", "RCNN_base.24.weight", "RCNN_base.24.bias", "RCNN_base.26.weight", "RCNN_base.26.bias", "RCNN_base.28.weight", "RCNN_base.28.bias".

I modified line 20 of Overlook/utils.py like below:
# fasterRCNN.load_state_dict(state_dict)
fasterRCNN.load_state_dict(state_dict,False)

thanks for your reply in advance :)

However the result is still difficult from the paper as https://github.com/Flashkong/Source-Free-Object-Detection-by-Learning-to-Overlook-Domain-Style/issues/7 pointed out since I used python3.
I do only have 3090 resources. It seems that 3090 doesn't match the version of python2.7 and python2.7's highest supported version of cuda and pytorch.

However the result is still difficult from the paper as #7 pointed out since I used python3. I do only have 3090 resources. It seems that 3090 doesn't match the version of python2.7 and python2.7's highest supported version of cuda and pytorch.

I'm only achieving a 24.24 mAP, which falls far short of the 35.8 mAP advertised in the paper. It's strange that even though I'm running Python 3 instead of 2, the difference is so noticeable. This seems to be more pronounced in the Cityscape dataset compared to others, but I'm unsure how to explain or correct this discrepancy. When evaluating the Pascal to Clipart dataset, the results are relatively close to the advertised mAP.

However the result is still difficult from the paper as #7 pointed out since I used python3. I do only have 3090 resources. It seems that 3090 doesn't match the version of python2.7 and python2.7's highest supported version of cuda and pytorch.

I'm only achieving a 24.24 mAP, which falls far short of the 35.8 mAP advertised in the paper. It's strange that even though I'm running Python 3 instead of 2, the difference is so noticeable. This seems to be more pronounced in the Cityscape dataset compared to others, but I'm unsure how to explain or correct this discrepancy. When evaluating the Pascal to Clipart dataset, the results are relatively close to the advertised mAP.

I've given up working on this repo now. I'm considering renting GPUs to build the same environment as the author claimed if I find it really necessary for my future work. By the way, I can reproduce normal mAPs in my environment using this repo,  on which this LODS repo is based.

Thank you for your reply. I appreciate it. I'll definitely give it a try, although I must admit that my primary interest lies in source-free domain adaptation for object detection. By the way, if you happen to know of any other methods that have achieved promising results and come with working code, I would greatly appreciate it if you could share them.

However the result is still difficult from the paper as #7 pointed out since I used python3. I do only have 3090 resources. It seems that 3090 doesn't match the version of python2.7 and python2.7's highest supported version of cuda and pytorch.

I'm only achieving a 24.24 mAP, which falls far short of the 35.8 mAP advertised in the paper. It's strange that even though I'm running Python 3 instead of 2, the difference is so noticeable. This seems to be more pronounced in the Cityscape dataset compared to others, but I'm unsure how to explain or correct this discrepancy. When evaluating the Pascal to Clipart dataset, the results are relatively close to the advertised mAP.

I've given up working on this repo now. I'm considering renting GPUs to build the same environment as the author claimed if I find it really necessary for my future work. By the way, I can reproduce normal mAPs in my environment using this repo, on which this LODS repo is based.

Thank you for your reply. I appreciate it. I'll definitely give it a try, although I must admit that my primary interest lies in source-free domain adaptation for object detection. By the way, if you happen to know of any other methods that have achieved promising results and come with working code, I would greatly appreciate it if you could share them.


Clean up syntax examples

Purpose of this pull request
This pull request (PR) cleans up syntax examples in these articles:

https://help.sumologic.com/docs/search/behavior-insights/logexplain/
https://help.sumologic.com/docs/search/behavior-insights/logreduce-keys/
https://help.sumologic.com/docs/search/behavior-insights/logreduce-values/

Issue number: GitHub issue #2558

Select the type of change:

[x] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

Thanks, @kimsauce.


Make sure lead assignee set before creating quote

This adds a validation check so that a quote cannot be created if no lead assignee has been set.

Codecov Report

Merging #613 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #613      +/-   ##
===========================================
+ Coverage    96.49%   96.49%   +<.01%
===========================================
  Files          126      126
  Lines         3819     3822       +3
  Branches       328      329       +1
===========================================
+ Hits          3685     3688       +3
  Misses          95       95
  Partials        39       39

Impacted Files
Coverage Δ

datahub/omis/order/validators.py
97.91% <100%> (+0.03%)
:arrow_up:

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 bb5135b...860f261. Read the comment docs.


Prevent space between icon and text from being part of hyperlink

I notice in the social sidebar that the image and text are part of the same hyperlink, causing the space between the image and text to be underlined as part of the hyperlink. This commit ensures that only the text is part of the hyperlink.

Hi!
First of all, thanks for your pull request! I appreciate the contribution, but I cannot accept this PR at the moment. It's with heavy heart that I have decided that I can no longer maintain this theme due to lack of time and changing interests. To makes sure the theme lives on however and not goes to waste, I've donated it to the Pelican community. It can now be found in the official pelican-themes repository. I'd suggest you re-submit your pull request there.
Sorry for the inconvenience!


My Alexa node-red nodes keep reconnecting

I've 8 Alexa devices defined on https://alexa-node-red.bm.hardill.me.uk (user diedvdyk)
I've defined them in node-red too.
In node-red I see them disconnect and reconnect every 2 seconds or so.
This results in Alexa failing to interact with the devices from time to time.
I dont know what's wrong and how to debug this ....
So, please help ?

Found the issues by reading some closed issues.
I had multiple alexa-home-conf accounts.


Add S3 session-token configuration

You may disregard these recommendations if you used the Edit on GitHub button from dvc.org to improve a doc in place.

❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.
🐛 Please make sure to mention Fix #issue (if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.
Please choose to allow us to edit your branch when creating the PR.
Thank you for the contribution - we'll try to review it as soon as possible. 🙏

Hi @isidentical welcome to the docs repo 🙂
Couple general notes:

If possible please add explicitly the core PR in the description of these (I see #5240 is linked from there to here but it's a bit easier to find it in the desc vs browsing through the timeline)
Keep in mind you can open branches in this repo directly, no need to fork! In fact we prefer that (a review app is deployed + makes it easier for us to take over docs PRs when/if needed (not the case here).

Thanks 🙏

Hi @isidentical welcome to the docs repo 🙂
Couple general notes:

If possible please add explicitly the core PR in the description of these (I see #5240 is linked from there to here but it's a bit easier to find it in the desc vs browsing through the timeline)
Keep in mind you can open branches in this repo directly, no need to fork! In fact we prefer that (a review app is deployed + makes it easier for us to take over docs PRs when/if needed (not the case here).

Thanks 🙏

Hi @isidentical welcome to the docs repo slightly_smiling_face

Hey! Thanks for the tips @jorgeorpinel

Hi @isidentical welcome to the docs repo slightly_smiling_face

Hey! Thanks for the tips @jorgeorpinel

@isidentical this change went into DVC 1 right? IF so I'll backport (copy over the changes here) to the v1 branch.

@isidentical this change went into DVC 1 right? IF so I'll backport (copy over the changes here) to the v1 branch.

@isidentical this change went into DVC 1.11.11 right? If so I'll backport (copy over the changes here) to the v1 branch.

I don't think so (at least I couldn't find it here, https://github.com/iterative/dvc/commits/1.11)

@isidentical this change went into DVC 1.11.11 right? If so I'll backport (copy over the changes here) to the v1 branch.

I don't think so (at least I couldn't find it here, https://github.com/iterative/dvc/commits/1.11)

Found it here: https://github.com/iterative/dvc/releases/tag/1.11.11 🙂
Backported to v1 in 0b703e09.


How to build the project?

Thank your for your cross-platform C++ alternative to reText. 😃
But I do not understand how to compile it 😞
I have customized the file rstpad.local.pro on my Linux Fedora 26:
# Python
PYTHON_LINK = -lpython2.7 -lpthread -ldl -lutil -lm -Xlinker -export-dynamic
PYTHON_INCLUDE_PATH = /usr/include/python2.7
PYTHON_SYS_PATH = /usr/lib/python2.7/site-packages

# Web engine
DEBUG_WEBENGINE_REMOTE_PORT = 0

But how to open the project using Qt Creator?
Is there a way to build the project from command line?

Opening the rstpad.pro file in Qt Creator should do it.
I didn't try to build it from the command line (as I've used Qt Creator) but this could help:
https://stackoverflow.com/questions/19206462/compile-a-qt-project-from-command-line

Oops, it's my fault, I had disabled Qmake support last year! 😕
This is what I had in menu Help > About Plugins...
[x] AutotoolsProjectManager
[x] CMakeProjectManager
[x] GenericProjectManager
[ ] QbsProjectManager
[ ] QmakeAndroidSupport
[ ] QmakeProjectManager
[x] QtSupport

Therefore, I have enable all Qt plugins and many more in other sections.
My rstpad.local.pro content is:
# Python
PYTHON_LINK = -lpython2.7 -lpthread -ldl -lutil -lm -Xlinker -export-dynamic
PYTHON_INCLUDE_PATH = /usr/include/python2.7
PYTHON_SYS_PATH = /usr/lib/python2.7/site-packages

# Web engine
DEBUG_WEBENGINE_REMOTE_PORT = 0

I proceed as the following:

Start Qt Creator 4.2.2 on Fedora 26 (Linux)
Enter menu File > Open File or Project...
Select file rstpad.pro

Now I can open the project 😃
To fix the following error Unknown module(s) in QT: webenginewidgets, I have installed qtwebengine using sudo dnf install qt5-qtwebengine-devel.
23:12:38: Running steps for project rstpad...
23:12:38: Configuration unchanged, skipping qmake step.
23:12:38: Starting: "/usr/bin/make"
/usr/bin/qmake-qt5 -spec linux-g++ CONFIG+=debug CONFIG+=qml_debug -o Makefile ../rstpad/rstpad.pro
Project WARNING: Not deplying Python libraries - platform not supported
Project ERROR: Unknown module(s) in QT: webenginewidgets
make: *** [Makefile:186: Makefile] Error 3
23:12:38: The process "/usr/bin/make" exited with code 2.
Error while building/deploying project rstpad (kit: Desktop)
The kit Desktop has configuration issues which might be the root cause for this problem.
When executing step "Make"
23:12:38: Elapsed time: 00:00.

Success the project build without any error  👍
I run the application using command line:
../build-rstpad-Desktop-Debug/app/RSTPad

I open a file *.rst but I have on the right pane window an error message
ImportError: No module named docutils.core

The package python2-docutils was already installed on my computer:
$ ls -gG /usr/lib/python2.7/site-packages/docutils
total 744
-rw-r--r--. 1  7564 Dec  9  2016 __init__.py
-rw-r--r--. 2  4444 Feb 11 20:04 __init__.pyc
-rw-r--r--. 2  4444 Feb 11 20:04 __init__.pyo
-rw-r--r--. 1  1589 Jul 11  2012 _compat.py
-rw-r--r--. 2  1654 Feb 11 20:04 _compat.pyc
-rw-r--r--. 2  1654 Feb 11 20:04 _compat.pyo
-rw-r--r--. 1 29426 Jun 25  2012 core.py
-rw-r--r--. 1 23765 Feb 11 20:04 core.pyc
-rw-r--r--. 1 23507 Feb 11 20:04 core.pyo
-rw-r--r--. 1  3959 Jan 19  2012 examples.py
-rw-r--r--. 2  3767 Feb 11 20:04 examples.pyc
-rw-r--r--. 2  3767 Feb 11 20:04 examples.pyo
-rw-r--r--. 1 35434 Jan  1  2013 frontend.py
-rw-r--r--. 2 30315 Feb 11 20:04 frontend.pyc
-rw-r--r--. 2 30315 Feb 11 20:04 frontend.pyo
-rw-r--r--. 1 17054 Apr 12  2015 io.py
-rw-r--r--. 1 15754 Feb 11 20:04 io.pyc
-rw-r--r--. 1 15529 Feb 11 20:04 io.pyo
drwxr-xr-x. 2  4096 Jul  9 01:18 languages
-rw-r--r--. 1 77359 Feb 16  2015 nodes.py
-rw-r--r--. 1 94940 Feb 11 20:04 nodes.pyc
-rw-r--r--. 1 94480 Feb 11 20:04 nodes.pyo
drwxr-xr-x. 3  4096 Jul  9 01:18 parsers
drwxr-xr-x. 2  4096 Jul  9 01:18 readers
-rw-r--r--. 1 57572 Jun 25  2012 statemachine.py
-rw-r--r--. 1 56711 Feb 11 20:04 statemachine.pyc
-rw-r--r--. 1 56294 Feb 11 20:04 statemachine.pyo
drwxr-xr-x. 2  4096 Jul  9 01:18 transforms
drwxr-xr-x. 3  4096 Jul  9 01:18 utils
drwxr-xr-x. 9  4096 Jul  9 01:18 writers

Do you have any idea to fix this python module import error?

You'll need to copy the contents of the PYTHON_SYS_PATH directory to a directory called "libs" (in the application build directory, next to the executable).
The filesystem structure should look something like this:

rstpad <-- the executable
libs/

docutils/
pygments/
other python lib directories and files..

Normally, this happens automatically during the build process, but I've only defined it for Windows and OS X builds so far. You can either copy it manually or add another else block to rstpad.pro :slightly_smiling_face:

Thanks for your help, I have created a directory and created links:
cd build-rstpad-Desktop-Debug/app
mkdir libs
ln -s /usr/lib/python2.7/site-packages/* .
ln -s /usr/lib/python2.7/* .

However I have the error:
ImportError: No module named unicodedata
  File "/home/olibre/rst/pad/build-rstpad-Desktop-Debug/app/libs/docutils/core.py", line 20, in <module>
    from docutils import frontend, io, utils, readers, writers
  File "/home/olibre/rst/pad/build-rstpad-Desktop-Debug/app/libs/docutils/frontend.py", line 41, in <module>
    import docutils.utils
  File "/home/olibre/rst/pad/build-rstpad-Desktop-Debug/app/libs/docutils/utils/__init__.py", line 17, in <module>
    import unicodedata

There are similar errors:

https://github.com/revolunet/sublimetext-markdown-preview/issues/28
https://github.com/d0ugal-archive/RstPreview/issues/1
https://stackoverflow.com/q/26583944/938111
https://stackoverflow.com/q/44473282/938111

However I have finally  fixed the issue 😄 and I can play with a beautiful and working RSTPad 😉
The fix:
cd build-rstpad-Desktop-Debug/app/libs
ln -s $HOME/.local/lib/python2.7/site-packages/* .
ln -s /usr/lib64/python2.7/lib-dynload/* .

I am to tired tonight to provide more testing in order to improve rstpad.pro.
Thank you
-- olibre


Update package.json for zod to 3.22.3 or greater

https://github.com/colinhacks/zod/releases/tag/v3.22.3
Fixes #2609
The current regex used for email validation contains "catastrophic backtracking", specifically ([A-Z0-9_+-]+.?)*. This gets evaluated inefficiently by JS, resulting in an exponential increase in execution time for failed matches.
This can be replicated easily - here's execution time against ^([A-Z0-9_+-]+.?)[A-Z0-9_+-]@([A-Z0-9][A-Z0-9-].)+[A-Z]{2,}$

Hi! Thanks for taking the time to raise this issue and submit a PR! I was actually just working on a fix for this so I pushed those changes into this PR.
Unfortunately, we can't limit people from using zod versions less than 3.22.2, but we can update the peer dep to support it for those who are patching their zod dependency.
I also updated our zod dependency internally to the most recent version that doesn't suffer from #320.

It also looks like withZod will have to be updated to support some change in zod's types.


Clearing response before redirects & bit of ReSharper refactor code

Clearing Response

I'm not a fan of those reshaper refactoring changes, can you please remove those and only keep the response clearing? Thanks!

I just left the Response.Clear before the redirect.
Thanks for this project, is very useful for our company.

Thank you... :)

@braathen is it possible to upgrade the nuget package with this small change?

I don't have any possibility to do that at the moment as I currently don't have a Windows development environment. I can make you a package co-owner if you want to do it?

That sounds good I can do that.

Great! So create an account, send me the userid and I'll give you access, you can email me at firstname dot lastname at gmail dot com.


migration tool reanalyzes all parts in a library every time it analyzes one

In the debugger, I have seen AnalysisDriver._computeAnalysisResult calls with withUnit = true for every file in a multi-part library, which appears to force re-analysis of the full library.  Exactly why we are doing this and what we might do instead is TBD.  This is a big reason migration of web_ui is so slow.

If you're using an AnalysisSession to perform analysis, then you can use getSourceKind to determine whether a file is a part. If you then use getResolvedLibrary for libraries and iterate over the already analyzed parts, you can keep track of which part's you didn't find a library for and analyze only those at the end (or signal an error if that's more appropriate).

The pipeline migration uses is somewhat different.  The dartfix (shared with dartdev) pipeline uses AnalysisServer.getResolvedUnit recursively over a file tree built with Resource.getChildren().    The way the resolved units are processed is going to be challenging to build a similar fix for as a lot of context information isn't in the right place for us to dedupe the getResolvedUnit calls.... at least, so far.  Still looking.

https://dart-review.googlesource.com/c/sdk/+/138940 now has a proposed fix.


Update NuGetScratch related doc

This is fixing part of the issues in https://github.com/NuGet/docs.microsoft.com-nuget/issues/2996

I think we also need to update https://learn.microsoft.com/en-us/nuget/reference/cli-reference/cli-ref-environment-variables doc