[S-02-2] UOF + DDPG Implementation

to-do-list

[x] Implementation of DDPG + HER only. https://github.com/ropiens/universal-option-framwork-pytorch/pull/3
[x] Implementation of high-level policy and DIOL. https://github.com/ropiens/universal-option-framwork-pytorch/pull/4
[x] add AAES https://github.com/ropiens/universal-option-framwork-pytorch/pull/5
[ ] add Abstract Demonstration.

Results

ropiens/universal-option-framwork-pytorch

Discussion

MountainCarContinuous has some dependencies between time-series actions.

using HAC(Hierarchical Actor Critic), this gap can be handled because all states(position and velocity) could be subgoals.
on the other hand, using UOF, the gap couldn't be handled because we cannot choose the optimal subgoals(we don't know exactly about the domain information of environments)

MountainCarContinuous has just one final goal and a small range of start position.

it makes UOF harder to converge. Since UOF just uses some subgoal(even we don't know exactly this goal is good or not), we cannot get various states and achieved goals for hindsight framework.


Clean up scratch extension

Basically remove unused code and unuseful debug/test code

Done in commit 899114bb28e2ba37f6d51dcb88af68a0a5a8eda7

Change order of functions (both in code and on the scratch gui) and clean up parameters

Moved to SkelScratch


flutter update-packages --force-upgrade

Update the world to make it easier to land https://github.com/flutter/flutter/pull/83846.

Google3 is failing because g3fix cl/375847057 for an earlier flutter update-packages --force-upgrade hasn't landed in google3 yet. Setting manually to pass.


Feature: Active image for completed progress

I added images for complete steps. Its working if you set custom image or keep the default numbers.
ps: I tried not to break your linter this time :D

Hi,
I am really sorry to close the pull request. I was confused by my own pull request. I just reopened it.

Np. Im currently try to resolve the issues in the example but having trouble with the storyboard part :D

I edited the storyboard, by adding separate view controllers.
Whats the issues in the example? can you list them? We can fix it one by one if that helps you.

wait a sec till you merge. I will make some changes that the checkmark uses the correct active color from the new array :)

Now the conflicts are resolved and you could merge it ;)

Current custom image option is awesome. And if set a single property we should provide a default image like 'check.png'.

I totally agree. I think we could use the icon I chose in the example (https://iconmonstr.com/check-mark-12-svg/). Will your write an "issue" for that? Maybe I am able to build that during this week

@g00m  I'm sorry,
What you meant by "checkEverything()"

That you can check everything so you can show the user that you completed a process

Btw are updating the Cocoapod?! :)


on-run-end hook failed on Spark for concurrent dbt runs

Got this error when running multiple dbt models at the same time
Conflicting commit: {"timestamp":1727626311790,"operation":"DELETE","operationParameters":{"predicate":["true"]},"readVersion":21941,"isolationLevel":"Serializable","isBlindAppend":false,"operationMetrics":{"numRemovedFiles":"1","numRemovedBytes":"93969","numCopiedRows":"0","numDeletionVectorsAdded":"0","numDeletionVectorsRemoved":"0","numAddedChangeFiles":"0","executionTimeMs":"1736","numDeletionVectorsUpdated":"0","numDeletedRows":"2151","scanTimeMs":"1735","numAddedFiles":"0","numAddedBytes":"0","rewriteTimeMs":"0"},"engineInfo":"Apache-Spark/3.5.1 Delta-Lake/3.1.0","txnId":"4df3ddeb-e2ff-4653-86ed-a58a9d632eff"}

The error came from this query
delete from elementary.dbt_models where 1=1

I'll create the ticket in main repo


Sandbox cloudwatch logs to S3

Pull Request Objective
This piece of work is being tracked in
this
GitHub Issue.
data-prod has a cloudtrail-athena-events processor that uses a lambda to post logs to a cloudwatch logstream.
This PR replicates this for data-engineering-sandbox-a. The intention is to then extend this with another lambda that writes the logs to S3 on a schedule.
Checklist

[!NOTE]
Each items should be checked. Skipping below checks could delay your PR review!

[ ] I have reviewed the style guide
and ensured that my code complies with it
[ ] All checks have passed (or override label applied, if I've
used the override-static-analysis label, I've explained why)
[ ] I have self-reviewed my code
[ ] I have reviewed the checks and can attest they're as expected

Additional Comments
Do Not Merge yet - still being expanded

@jhpyke PR submitted as requested. Will expand with the S3 export module when I have some time as well. Thanks for all your help!


Unstable test results

Hi, have you ever had a situation where the result of a retest is different from the result of the last test, and what may be the reason for this?

No.. did you change some codes?

Ok, I got it. Thank you.


How to lint all commits in a PR?

I'd like to integrate conventional-changelog-lint into my CI process. I can enforce a commit convention on "client side" using Husky as per docs, but how would I enforce the convention on CI?
Assuming that a PR is a list of commits that do not exist on master, then getting those commits is simple:
git log master..$(git symbolic-ref --short HEAD)

I am not sure though whats the best way to feed them to conventional-changelog-lint. Here is what I have ended up with for now:
for sha in $(git --no-pager rev-list master..$(git symbolic-ref --short HEAD)); do git cat-file commit $sha | sed '1,/^$/d' | conventional-changelog-lint; done

The problem with the latter is that it doesn't exit if either of the validation fails.

conventional-changelog-lint --from=HEAD~$(git --no-pager rev-list master..HEAD --count)

another way to do it.

[ ]  Add CI recipe to documentation as per @gajus
[ ]  Consider adding a flag for pull/merge request scenarios

Why not just conventional-changelog-lint --from=master?

I've been fighting with getting this to work properly in GitLab CI with projects where we're just starting to adopt conventional-changelog.  I couldn't get @gajus' example above to work for a couple of reasons and had to modify it slightly, although it feels a bit "hacky" to me:
conventional-changelog-lint --from=$(git --no-pager rev-list --no-merges origin/master..HEAD | tail -n 1 )
The first issue I had was with the git config merge.ff false option being set by some developers, locally or globally, meaning the --count figure was over-inflated, leading to a failure condition.
The 2nd issue I'm currently wrestling with is although this now gives the correct starting from commit hash as I would expect from looking at git log --oneline, the lint output is actually starting at the commit after that - @marionebl is there an oddity I'm missing here?
The CI job definition is:
conventional-changelog-lint:
    stage: compliance-tests
    script:
        - git fetch
        - ./node_modules/.bin/conventional-changelog-lint --from=$( git --no-pager rev-list --no-merges origin/master..HEAD | tail -n 1 )
    only:
        - develop

Cheers

I tested
conventional-changelog-lint --from=$TRAVIS_BRANCH --to=$TRAVIS_PULL_REQUEST_BRANCH

with success so far.


refactor get_pages template tag

This is step one in a number of steps that together aim to reduce the number of queries made to the database.

With the tests passing, this is definitely much neater 🍰


Gap between sound and video

Hi, developer.
Current Behavior / Expected Behavior

Using Seekto, there is a gap between sound and video.
playerRef.current.seekTo(playerRef.current.getCurrentTime() - 5.000000, 'seconds');

<ReactPlayer
          ref={playerRef}
          url={video.url}
          controls={false}
          playing={playing}
        />

In this case, the video rewinds for 5 seconds, but there is a gap of several seconds in the audio.

Environment

React hooks
Video on Firebase Cloud Storage

Do you have any solutions?　Thank you.

when you say gap, do you mean the audio starts playing 5 seconds after video, or that they both start playing at the same time, but the audio track is 5 seconds behind the video track?

@yosh1 Does this happen with other videos? Like the sample mp4? My guess is that it’s something specific to how Firebase is delivering the data to the browser.

I'm seeing a similar issue in Chrome (just tried Firefox and it was OK). Although it happens without needing to do seekTo.
There's basically a stutter at the start of the video (always), and then the video lags the audio for the entire video. Interestingly clicking around can fix it (although not if clicking in the first few seconds, seeking back to there causes it de-sync again).
The video file is an MP4, and is coming from a URL.


python312Packages.owslib: 0.31.0 -> 0.32.0

Diff: https://github.com/geopython/OWSLib/compare/refs/tags/0.31.0...0.32.0
Changelog: https://github.com/geopython/OWSLib/releases/tag/0.32.0

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 351024

x86_64-darwin

:x: 4 packages failed to build:

python311Packages.cartopy
python311Packages.cartopy.dist
python312Packages.cartopy
python312Packages.cartopy.dist

:white_check_mark: 4 packages built:

python311Packages.owslib
python311Packages.owslib.dist
python312Packages.owslib
python312Packages.owslib.dist

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 351024

x86_64-linux

:x: 8 packages failed to build:

python311Packages.cartopy
python311Packages.cartopy.dist
python311Packages.owslib
python311Packages.owslib.dist
python312Packages.cartopy
python312Packages.cartopy.dist
python312Packages.owslib
python312Packages.owslib.dist

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 351024

x86_64-linux

:x: 4 packages failed to build:

python311Packages.cartopy
python311Packages.cartopy.dist
python312Packages.cartopy
python312Packages.cartopy.dist

:white_check_mark: 6 packages built:

python311Packages.owslib
python311Packages.owslib.dist
python312Packages.owslib
python312Packages.owslib.dist
qgis
qgis-ltr

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 351024

x86_64-linux

:white_check_mark: 10 packages built:

python311Packages.cartopy
python311Packages.cartopy.dist
python311Packages.owslib
python311Packages.owslib.dist
python312Packages.cartopy
python312Packages.cartopy.dist
python312Packages.owslib
python312Packages.owslib.dist
qgis
qgis-ltr


Brand new material instance parented to M_CesiumBaseMaterial is unusable

Back in the Unreal Engine 4 days, you could create a Cesium-compatible material really easily as a starting point for customizations:

Create a new Material Instance by right-clicking in the Content Browser.
Double-click the new material to open it.
Change the Parent field to M_CesiumBaseMaterial.

This would automatically fill in the "Layer Parameters" tab with some default layers (Background, Overlay0, Overlay1, and Clipping). You could then use this material in a tileset and it would work great.
Not anymore in UE5.
The problem is that Cesium is not able to set any of the parameters on a new material created this way. Why? Because UCesiumMaterialUserData is not attached to the material instance. Why? Because Cesium didn't attach it. Why? Because Material->GetStaticParameters()->bHasMaterialLayers returned false. Why? No idea! It clearly has material layers, they show up in the UI. Seems like a UE bug to me.
The good news is that almost any modification to the material layers will make Unreal realize that the material layers exist (the flag above suddenly goes to true) and then the material will start working properly.
Since most of the point of making a new material instance based on the Cesium base material is so that you can customize the layers, this probably isn't an urgent thing to fix.

Here's the Unreal Engine commit that broke this:
https://github.com/EpicGames/UnrealEngine/commit/773ef8ff5af512b2a0b253f135bd48b0fe5d0dcc#diff-91b70e6cb56052b79b55ab8e7330e13d1e23e39c53949971c05d7f0e12b34463L2210


pull request for e2e test at 11/27/2016 07:26:05

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Clock SSR fix

this fixes https://github.com/react-toolbox/react-toolbox/issues/328
I'm not sure it is the best solution, but it works

this is really similar to recently merged https://github.com/react-toolbox/react-toolbox/pull/323/files

Apparently it's a common issue when a component needs to check the DOM to calculate sizing... It works so let's merge. I'm going to release a new version with all these fixes during the weekend

@javivelasco thanks


VirusTotal Detection

I am pretty sure this is safe, but VirusTotal is showing 7/54 detections as Java.Trojan.GenericGB.29215. Is there a reason this is a detection?

False positive. It could be due to the inclusion of BCrypt for hashing the passwords and including an email library. Both could be strong indicators. However everyone could check the code and compile it on their own. There is no Trojan it. I also took a short peak at the dependencies and it seems safe there too.

Yeah, that's what I thought. Thanks for confirming though.


[BUG] Pii data in Splunk

Describe the bug
Ryan Thomas found the pii data from Splunk and we don't want ingest the pii.
{ [-]
address: null
birthYear: null
country: null
creationDate: 2020-08-16T23:31:07.407Z
documentType: null
email<EMAIL_ADDRESS>ep_building_block: profile_building_block
ep_method: PUT
ep_name: PII
firstname: xxxxx
healthcareProviderIDs: null
homeCounty: null
imageUrl: null
lastModifiedDate: 2020-09-15T19:55:08.521Z
lastname: xxxx
middlename: xxxx
netid: xxxx
phone: null
photoImageBase64: null
pid: 23e24f77-213f-46f2-8f5a-646bc58ce00c
state: null
testResultsConsent: null
uin: xxxxxxxxx
username: null
uuid: [ [-]
4350b7b0-b7db-4d5f-ae0d-10ceab59bf4b
c0d8f9ea-218e-4d0f-960c-2bd81be3009a
]
workCounty: null
zipCode: null
}
To Reproduce
Steps to reproduce the behavior:

Go to Rokwire Team - Illinois Splunk app
Search index=apps-rokwire_illinois sourcetype=illinois:rokwire:profile 23e24f77-213f-46f2-8f5a-646bc58ce00c

@ywkim312, as it looks like currently PII data is not needed in Splunk, let's discuss about reducing the log data for PII endpoints.

Ok.

@yoonlees, would it help if we print only the PID for future reference as a string or will it affect how Splunk parses the log data? Could you please confirm this with Ryan? Please see PR #557 for reference.

It is unclear to me what the requirements are. Yoon has a clearer understanding. My understanding is the user_uuid (or as it is sometimes in the log data, uuid or uid) is the string we need in Rokwire ... and not things like netid, email, phone, address. Whether or not we should have pid, I cannot say and defer to Yoon. I must say that it would be convenient to have a value that would allow us to relate multiple uuids to a "person" entity. (Is pid such a thing?) But I defer to Yoon's guidance.

Yes, PID links multiple UUIDs into one. @yoonlees, could you please let us know what feature change is needed here? Also, if PID is needed, would a string log work (e.g. "PID GET <PID value>") or does the PID needs to be in a JSON string like "PID GET {\"pid\": \"<PID Value>\"}".

@sandeep-ps Ryan and I had a good discussion about this and we want to keep PII and its related UUIDs and remove all other personal data. In Splunk Ryan is planning to go extra and obfuscated those IDs.
Also don't bother to change the JSON format to other format. See my example:
{
"pid": "xyz",
"uuid": [
"4350b",
"c0d8f"
]
}

Ok, I will work on it tomorrow morning.

Thanks @yoonlees and @ywkim312.

Just to clarify. Yoon wrote "keep PII and its related UUIDs".  But, I believe he meant "keep pid and its related uuids".  I'm not planning to do any further obfuscation of the pid or uuid values.  I consider them sufficiently obfuscated. So the "PII" that we want to remove from the logs are UIN, netid, phone, email, address.
Yoon, I hope I'm representing your/our intent.

@wryanthomas, this fix has been deployed to production. Please feel free to reopen or create new issue if the issue is not resolved.


Autocomplete for cities

At the moment, when you type in a city, it won't help you finish your entry with suggestions. A cool feature would be if it had a drop down menu that suggested locations similar to the name. For example, if I type in Cambridge, it could tell me Cambridge, MA, USA or Cambridge, UK to help users get the right answers!

Hey @quachtridat and @Hemansh31 , had a suggestion.
We have made PR #23, in which we store the most recent 5 searches of a user. So using that, you can show the autocomplete results even when the search bar is empty, as search engines do, ex. Google.
Me, @ShiviPro  and @khattakdev have had a discussion about same at #12.
If you like the suggestion, and need any help feel free to tag @ShiviPro or me anywhere😊 .

@Tanmay000009 I do think having recent searches pop up while the search bar is empty is a good idea. I believe it can be implemented fairly quickly because you guys solve #12 by storing data in local storage.


Issue : The product-service build is not happening

The problem is that individually the modules will not build.
Always build the project from root .

close


Add Bootstrap Brand Color Guidelines and Related Color Codes

Description
This pull request introduces guidelines for the use of Bootstrap's brand colors and includes relevant color codes to maintain consistency throughout the project. This update ensures that developers working with Bootstrap components have a clear understanding of the official branding colors and their appropriate usage.
Feature Added #40926

Note:- This is my first open-source contribution, and I welcome any feedback or suggestions for improvement. If there are any changes needed or further enhancements you would like to see in this feature, please feel free to let me know. I am more than happy to make adjustments based on your recommendations.

Thanks for your contribution @DineshPawar7, but your PR is invalid. You're not updating files but creating new ones, the PR must contain only the modified files. I'm closing this one. Feel free to open a new one.

no problem,
but can you tell me whether the solution I made is correct or not, or have I done something wrong, I am not talking about files, I am just talking about the code, is such a code acceptable.

Unfortunately, I can't see what specific changes were made (you can check the diff here: https://github.com/twbs/bootstrap/pull/40958/files), and I'm unable to run the branch locally. I suggest discussing the wording and expected content directly in the issue first. The core team hasn't yet confirmed whether this feature would be useful.

Here is a quick guide on how you can make a pull request (PR) to Bootstrap with your changes: https://github.com/twbs/bootstrap/blob/main/.github/CONTRIBUTING.md#pull-requests


Problems with Info.plist, exit code 1 and qos attributes

Hi @Doctor-Rob, I had the same issue and managed to figure it out - it's more of an Xcode problem. Just make sure you have any of the contained sample projects closed before you add the project following Brad's steps. You'll know because when you do the first step (dragging the project file across) there will be a little triangle next to it. Hope that helps.


Add new (4.0+) API for av_[de]muxer_iterate

Addresses #46.
Naming should be fine? It's closer to the FFmpeg function names than to the Rust-side Format API. Could be renamed to e.g. list_inputs/list_outputs.

Very weird. Those segfaults do not occur on my (Linux) machine. I'll have to take another look to figure this out

I narrowed it down to the Format::description function. When I remove the f.description() call in the tests, the error disappears. I would guess that the underlying code is unsound? I am not 100% sure but the iteration code is pretty simple

Found the issue: Input::description and Output::description do not check long_name.is_null(), but long_name is always null if FFmpeg is built with --enable-small. It seems like the CI docker containers use that option.

It looks like the msrv job is complaining about something relating to the lockfile.

Yeah, that's the issue addressed in #56.

Ah. Forgot to add the nullptr fix after rebasing. Will add it back later today


Update state of branches and slack link

Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

@estesp fixed

LGTM

LGTM


[BEAM-9273] Explicitly disable @RequiresTimeSortedInput on unsupported runners

Fixes [BEAM-9273]
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

 First failed post commit seems to be this one.

Thanks @kennknowles


E57SimpleReader: handle missing "/images2D" and "isAtomicClockReferen…

Hi,
The current E57SimpleReader implementation fails to read E57 files produced by las2e57 tool. Las2e57 writes E57 data in which the root node does not have "/images2D" node, and datetime nodes do not have "isAtomicClockReferenced" node.
Code changes in this pull-request should fix these incompatibilities.

Thanks Olli!


Exception when loading realm

Hi,
when I load realm I get exception in [RLMRealm defaultRealm]; that usually tells me that the schema needs to be updated.
But this is happening even with reset iOS simulator (all data deleted) and with having a migration block in place.
Code is the following:
[RLMRealm setDefaultRealmPath:[[[NSFileManager defaultManager]<EMAIL_ADDRESS><EMAIL_ADDRESS>
    [RLMRealm setSchemaVersion:2 forRealmAtPath:[RLMRealm defaultRealmPath]
            withMigrationBlock:^(RLMMigration *migration,
                                 NSUInteger oldSchemaVersion)
     {

         NSLog(@"Migration complete.");
     }];

    @try {
        [RLMRealm defaultRealm];
    }
    @catch (NSException *exception) {
        NSLog(@"Trying to open an outdated realm a migration block threw an exception. Version: %li",(long)[RLMRealm version]);
    }

What is causing this exception?
Thanks
Sascha

@souhl can you please share the exception that Realm is throwing when you call [RLMRealm defaultRealm]?

@segiddins sure:
0x10e0d43d0 <+327>: jmp    0x10e0d4440               ; <+439> at RLMRealm.mm:818
0x10e0d43d2 <+329>: leaq   0x398357(%rip), %rdi      ; @"Cannot set schema version for Realms that are already open."
0x10e0d43d9 <+336>: xorl   %esi, %esi
0x10e0d43db <+338>: callq  0x10e0dda20               ; RLMException(NSString*, NSDictionary*) at RLMUtil.mm:303
0x10e0d43e0 <+343>: movq   %rax, %rdi
0x10e0d43e3 <+346>: callq  0x10e333fee               ; symbol stub for: objc_retainAutoreleasedReturnValue
0x10e0d43e8 <+351>: movq   %rax, %rdi
0x10e0d43eb <+354>: callq  0x10e333f4c               ; symbol stub for: objc_autorelease
0x10e0d43f0 <+359>: movq   %rax, %rdi
0x10e0d43f3 <+362>: callq  0x10e333f8e               ; symbol stub for: objc_exception_throw
->  0x10e0d43f8 <+367>: leaq   0x398351(%rip), %rdi      ; @"Cannot set schema version to RLMNotVersioned."
0x10e0d43ff <+374>: xorl   %esi, %esi
0x10e0d4401 <+376>: callq  0x10e0dda20               ; RLMException(NSString*, NSDictionary*) at RLMUtil.mm:303
0x10e0d4406 <+381>: movq   %rax, %rdi

Can you please provide the backtrace at the point the exception is thrown?

@souhl I'm closing this since we need more information from you to investigate further. Please reopen if you can provide more info.

What should I do when I set migration I also get a exception "Cannot set schema version for Realms that are already open."

@wyn15811017 this happens when setSchemaVersion: is called after a realm file has already been opened. My suggestion is to debug your app to identify where you may be opening a realm file (you could set a breakpoint on -[RLMRealm init]) and make sure to call setSchemaVersion: before this point.


Final changes before 0.2.0 release

Coverage remained the same at 100.0% when pulling a43bb22ac31de29ad1dc122fc788cf29608dc231 on esben:master into bf777becea6529df4f63628daa19f75f5735aefe on XD-embedded:master.

Coverage remained the same at 100.0% when pulling a43bb22ac31de29ad1dc122fc788cf29608dc231 on esben:master into bf777becea6529df4f63628daa19f75f5735aefe on XD-embedded:master.

Coverage remained the same at 100.0% when pulling a43bb22ac31de29ad1dc122fc788cf29608dc231 on esben:master into bf777becea6529df4f63628daa19f75f5735aefe on XD-embedded:master.


Add Node.js 4 to AppVeyor

This PR adds Node.js 4 support to AppVeyor matching the Travis config (https://github.com/sass/node-sass/pull/1130)

We expect this to fail without https://github.com/sass/node-sass/pull/1128

This suggest you're adding io.js 4 because of the comment above.

@mzgol true. Fixed in 05b7c9c

node -> node.js to be consistent with the first comment?
Sorry for nitpicking. This is not terribly important. :D

@mzgol I recall there is was meant to a distinction between Node, io.js and Node.js because of trademark issues. Can't remember the specifics. I'll look into it.

I meant that the first comment says "node.js" and the third just "node". The official name is Node.js in both cases.

@mzgol https://github.com/sass/node-sass/commit/c5a16a9f10b63e88050740cb0ba863b1fe2abc69 :)


Merge 2.2 into develop

Let's prepare the next version.
It contains some cool new stuff:

the analytics now also sends the page (and is triggered accordingly)
the hitsPerPage let you select a default page
a new menu select widget \o/

The problem with build is a commit format from one that was created for the release of the beta. I'll try to fix it before the release.

Let's move this to monday. I wouldn't like to do support on a sunday :)

Merging because bad commit format...


How to change the color of the links in a network?

Hi,
I'm trying to change the color of the links, and it's not clear in the documentation how to do so.
 const methods = {
            links: this.state.nodes,
            nodes: this.state.links,
            color: (d: any) => d['color'],
            label: (d: any) => d['OccupationTitle'],
            hover: (d: any) =>  d['id'],
            zoom: true,
}

Thanks

hi @AlmahaAlmalki! apologies for the unclear docs (known bug 😅)
you're looking for the shapeConfig method, which acts as a pass-through to any Shape class that is being drawn on the screen. So if you wanted to change the stroke on every shape you would do this:
shapeConfig: {
  stroke: "red"
}

...but for visualizations with multiple shape types, like Networks which have Rect and Line shapes, you can provide a namespaced config:
shapeConfig: {
  Line: {
    stroke: "red"
  }
}

Great, thank you Dave 👍

Hi Dave,
I tried to change links color using shapeConfig, but it didn't work
 shapeConfig: {
  Line: {
    stroke: "red"
  }
}

this is the full config obj
       const methods = {
            links: this.state.links,
            nodes: this.state.nodes,
            color: (d: any) => d['color'],
            label: (d: any) => d['OccupationTitle'],
            hover: (d: any) =>  d['id'],
            zoom: true,
            tooltipConfig: {
            title: (d: any) =>  d['OccupationTitle'] + '
',
            body: (d: any) => {
                let info: string = `
${d['sector']}

 Automation Risk: ${d['AutomationRisk']}

 Education Level: ${d['EducationLevel']}

                `
                return info
           },
           shapeConfig: {
            Line: {
              stroke: "red"
              }
            }
          }
        }

but, if i changed the path stroke color in CSS it will work.


concurrent.futures._base.CancelledError when high throughput in aiohttp web application

Hi  there,
I am new in asyncio coding. Recently, I wrote a web application with aiohttp, during a request, I need do some db reading from and writing to cassandra. I use the aiocassandra as the driver, in the test environment, all stuff is ok. But in production environment, when the throughput(at the peek around 2k-3k QPS) is going high, I got these errors very frequently(50-80 per 15 minutes)
{'status': 'failed', 'content': None, 'errorCode': 'Internal Server Error', 'errorMessage': 'Traceback (most recent call last):\n  File "/app_libs/common/web/middlewares.py", line 74, in error_middleware\n    response = await handler(request)\n  File "/app_libs/common/web/middlewares.py", line 26, in request_trace_middleware\n    response = await handler(request)\n  File "/app_libs/common/web/middlewares.py", line 59, in track_time_middleware\n    response = await handler(request)\n  File "/app/routes/collecting.py", line 40, in webcollect\n    matchresult = await integration_data(request.app, request_data, config_data, cass_flag=True, kafka_flag=True)\n  File "/app/services/integration.py", line 34, in integration_data\n    result = await integration_event(app, request_data, configs, **kwargs)\n  File "/app/services/integration.py", line 62, in integration_event\n    user_data.online_data[\'page\'], app[\'cassandra\'], profile_id, user_data.device_id)\n  File "/app/services/integration.py", line 282, in check_pt_id\n    \'event\', \'device_id\', device_id, profile_id)\n  File "/app/libs/core.py", line 179, in cassandra_read\n    result.extend(await query(table, pk_str, pk_value))\n  File "/app/libs/core.py", line 168, in query\n    res = await session.execute_future(cql, (pk_v, ))\n  File "/usr/local/lib/python3.7/site-packages/aiocassandra.py", line 153, in execute_future\n    return await asyncio_fut\nconcurrent.futures._base.CancelledError\n'}

... execute_future\n    return await asyncio_fut\nconcurrent.futures._base.CancelledError\n'...

At first, I do those cassandra reading work after the request is finished, make those processes under an asynio task. Then when I got those errors, I wonder if that is caused by aiohttp framewok, as when a request is done, aiohttp may cancel tasks binded a request. Then I invoked aiojobs to make sure all those tasks would be waited even the response is returned. BUT the problem is still there, I try to put the cassandra operations into the request lifelong, take them out from the post-request background asyncio tasks, BUT, still, not solved the prolem. I search a lot on the internet, but not find anything really helpful.
I notice that this driver is wrapped with the datastax version with async query. I am not sure if I am using this driver correctlly or indeed there an issue exists. I dont know what to do with this, changing the thread/process executor or reuse the datastax non-aio verion back? Any advice from you guys?
Thank you very much. (By the way, I also use aiokafka in the same env but has not occured the same errors, and again, when the thoughput is at low level, all stuff is working.)
++++
python 3.7.4
aiohttp==3.6.0
aiocassandra==2.0.1
cassandra-driver==3.18.0

Do you have any solutions, I have the same problem here

AFAIK, aiohttp throws CancelledError into coroutine if http connection was
disconnected. So, the simplest solution is just to ignore this error (of
course if you are not in distributed transaction etc etc). Anyway, it
should not affect your request execution (because client has been
disconnected already).
On Tue, 14 Jan 2020 at 09:49, elune<EMAIL_ADDRESS>wrote:

Do you have any solutions, I have the same problem here
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aio-libs/aiocassandra/issues/108?email_source=notifications&email_token=AF7WEKAVYZBKONM4KGWDPM3Q5VOATA5CNFSM4IZGSECKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI3QANQ#issuecomment-574029878,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF7WEKCXDJDELJB7BBBNFILQ5VOATANCNFSM4IZGSECA
.

--
Best regards,
Evgeny Vorobiev

@eavorobiev Thanks for your reply, indeed this has something to do with connection disconnected
Thanks you very mush


FIRListenerRegistration will failed while connected Router disconnected from Internet and then connect back.

[READ] Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository
file a Github issue.

If this is a feature request make sure the issue title starts with "FR:".

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general Firebase discussion, use the firebase-talk
google group.
For help troubleshooting your application that does not fall under one
of the above categories, reach out to the personalized
Firebase support channel.

[REQUIRED] Step 2: Describe your environment

Xcode version: 10.1 (10B61)
Firebase SDK version: 6.0.0
Firebase Component: Firestore (Auth, Core, Database, Firestore, Messaging, Storage, etc)
Component version: 1.3.0

[REQUIRED] Step 3: Describe the problem
As Described in the title, the created Listener FIRListenerRegistration will failed if the device-connected router disconnected from Internet and then connect back. (I mean the device connected with router all the time, but the router is disconnected from Internet.)
Steps to reproduce:
Un-plug the Internet cable from Router and then connect it back....
Relevant Code:
Here is a very simple Project I used to testing... I found out Firebase Listener doesn't have this issue but Firestore's one has.
Testing Procject For Firestore.zip

Nope... You cannot fake faked "pulling the router cable" by turning off the network connection on my Mac. you have to literally pulling the router cable.
Those two are different.
If you turn off the network connection on you Mac, Max's Reachability will notice your device has disconnected with any further network connection devices.
but if you "pulling the router cable" Reachability won't notice it... just your Mac disconnected from Internet.

And Failed means if I change anything from the listening collection or document, our project won't notice that...

And also, "Pulling the router cable" is trying to simulate the unstable Internet connection from Router.

@TIEmerald Thanks for the added information.  The SDK should initiate reconnect attempts (subject to exponential backoff) once it detects the connection has been closed.  If you pull the network cable, it may very well take longer for the client to be notified that the connection has been closed, and so recovery could take longer.  But it should still reconnect and get appropriate events.  Can you try waiting longer and see if the changes eventually arrive?
Else, can you enable logging with FirebaseFirestore.setLoggingEnabled(true) and capture logs for the scenario, including ~5minutes of logs after the point that you pull the network cable?  That should make it apparent whether reconnects are happening.
THanks!

Hi @mikelehen , I have test it one more time, here is the records: (￣∇￣) ~~
I compile my project at around 9:33-34 am, and here is the Screenshot from my simulator:

Then, I un-plug and re-plugin the cable from connected router arount 9:35-36 am. After that, while my computer reconnected with Internet, I changed the value from firebase/firestore database:

The firebase value changed immediately, but not the firestore's one.... I have waited until 9:41 am... still not changing... even now it's still not changing...

And here is the logs I copied from console:
Logs For Firestore Testing.txt

@TIEmerald Thanks very much for the screenshots and logs.  This makes it very clear what you're observing and it's definitely not what I'd expect.  From the logs, the Firestore SDK hasn't been notified that the connection has failed and so we haven't initiated any retry attempts.  I still suspect that if you wait long enough, it will eventually reconnect.  So if you're willing, I'd appreciate it if you'd try waiting ~15 minutes and see if it ever recovers.  Obviously this would be unacceptable performance but it would at least verify that we're dealing with a timeout issue rather than a full bug.
In the meantime, I'll see if I can reproduce, maybe using the Network Link Conditioner in OSX.  Unfortunately this is likely something that would need to be addressed in the gRPC layer (the underlying transport that we use to communicate with the backend), so it may not be something we can fix super easily (at least in the short-term)...

Thanks, @mikelehen , I am appreciated your efforts.
Also... I am pretty sure waiting ~15 minutes won't help in here... As while I generating my last comments.. I believe I have noticed that even through the time is 9:51 am, which is around 15 mins from my testing, the Firestore Listener is still not updated..
To be honest... I am not pretty sure could we reproduce it with Network Link Conditioner.....   But my work around is whenever I detected the Internet connection is off, I will recreate the same Listener when the Internet is back.

@TIEmerald Interesting. Could you do the following and collect the logs for me?

Start your listener, unplug /re-plugin your router cable as you did before.
Make a change via the console and wait ~5min to make sure the listener doesn't see it.
Add a new listener, for a different query.

I am wondering if when you get data for the new listener, you also get the missing event for the first listener.  It really shouldn't be possible for you to "miss" events and then have a new listener work correctly.  So I'm wondering if the reason recreating the listener works for you is because it triggers us to notice the connection has died and perform reconnect logic.
Meanwhile I will try to reproduce this myself today.  Thanks!

@TIEmerald Good news.  I was able to reproduce this using Network Link Conditioner.  I will follow up with the gRPC team to see if I can get to the bottom of it. Thank you for the report. I’ll update here as I have news to share.

Cheers @mikelehen , looking forward your updates~~
I didn't get a chance to collect the logs today... let me check if I could find a time later today or not..
About the "because it triggers us to notice the connection has died and perform reconnect logic." part. Actually, there is one thing I found very interesting with our production application....
Besides "recreate the listener", I could even bring the listener back while I called a cloud function from the same device which updated the parent document of my listening collection.....
Which means... If I have a document D, which has a collection C like this:
D / C
My application A has a listener L to the collection C.
And then if I disconnect Router and connected it back, the listener L will failed, which is the issue I was describing...
After that if I called one Cloud function on Firebase server from the same application a, and this cloud function updated one property on D. I could notice that the failed listener L will back to work.....
Very strange... might be helpful thus I leave it in here... could ignore it if it's too complicated...
Thanks anyway.

Thanks for the added info.  Don’t worry about collecting the logs.  I think I have everything that I need since I can reproduce it now.

Quick Question in here, hi @mikelehen, would you mind tell me how do you reproduce it with Network Link Conditioner?  with Profile "100% Loss"? seems like I failed to reproduce it with Network Link Conditioner

@TIEmerald What I've been doing is starting the listener, enabling 100% packet loss in Network Link Conditioner, waiting 2.5 minutes, and then disabling Network Link Conditioner to restore full network connectivity.  When I do that, about 50% of the time the listener doesn't recover.  I don't know how important it is to wait 2.5 minutes.  But I tried 60, 90, 120, 150 seconds and 150 was the first time it reproduced for me. 🤷♂

en... strange things... might to do something with the base level internet communication... 🤷♂ ... I might still testing it with my re-plugin way......
Kind of annoying that my temporary solution didn't fully resolve the issue our customers are facing... could because our internet connection detection logic is not perfect, or something else, I don't know....... I think I might try to depend on firebase listener to temporarily resolve this ...
Still, thanks a lot mate~

Hi @mikelehen we encounter a similar issue today while we are testing our application with Xcode.
We have captured Firestore logs hope it will be helpful:
Scenario :
Our device connected to router all the time, and we didn't notice our device disconnected from Internet at all.  We found out Firestore Listener failed in the middle of our testing, and then we tried to reset the listener manually.
Logs :
Captured Logs for Firestore Listener failed.txt
This image will help you understand the scenario: ( I grep all lines with WatchStream)

We have updated the listened collection on 14:56 and 14:57, but nothing happened... therefore, we manually reset the listener on 14:58 and it has shown the WatchStream try to "unwatch" -> "watch" -> "Stream error" -> "backoff"  and then continued.....
Further Experience:
Then I did further experience with my simple project. And I use my re-plugin method to simulate the issue.
I found out if I reset the listener within several minute, there won't be any "Stream error".
But if I wait longer enough then I could capture the similar logs:
NewTestingLogs.txt

@TIEmerald Thanks for the added info.  I think that's probably expected.  Removing the network cable results in packet loss and TCP streams can tolerate some amount of packet loss and still recover.  So depending on how long and various timing aspects, the TCP stream may just recover (no stream error) or it may fail and close (resulting in a stream error).
I've forwarded the info on this case to the gRPC team. I'm also looking into whether we can avoid it ourselves by enabling a keepalive in gRPC...  which I believe will help detect failures much more quickly / predictably. I'll keep you updated.

I think this should be addressed by https://github.com/firebase/firebase-ios-sdk/pull/3029 which will go out in the next iOS release.  You can pick it up earlier by following the instructions at https://github.com/Firebase/firebase-ios-sdk#accessing-firebase-source-snapshots
With that change, my listener recovered within a few seconds of turning off packet loss.  It could take longer, depending on various timing of retry attempts, etc. But it should never take more than ~1 minute.
Thanks again for reporting this and all the help to track it down.

Actually, we've already branched and are stabilizing the next release. This bug fix will be on the M49 release which will be the one after that.

Thanks very much @mikelehen, you are a legend. I will give it a try today.

Hi, @mikelehen  😂 I cannot pick it up via following the instructions at https://github.com/Firebase/firebase-ios-sdk#accessing-firebase-source-snapshots, would you mind help me and tell me what's wrong in here? Thanks
target 'Testing Procject For Firestore' do
  # Uncomment the next line if you're using Swift or would like to use dynamic frameworks
  # use_frameworks!
pod 'Firebase/Core', :git => 'https://github.com/firebase/firebase-ios-sdk.git', :branch => 'master'
pod 'Firebase/Firestore', :git => 'https://github.com/firebase/firebase-ios-sdk.git', :branch => 'master'
pod 'Firebase/Analytics', :git => 'https://github.com/firebase/firebase-ios-sdk.git', :branch => 'master'
pod 'Firebase/Database', :git => 'https://github.com/firebase/firebase-ios-sdk.git', :branch => 'master'

  target 'Testing Procject For FirestoreTests' do
    inherit! :search_paths
    # Pods for testing
  end

  target 'Testing Procject For FirestoreUITests' do
    inherit! :search_paths
    # Pods for testing
  end

end

Testing Procject For Firestore > pod update
Update all pods
Updating local specs repositories

CocoaPods 1.7.0.rc.2 is available.
To update use: `sudo gem install cocoapods --pre`
[!] This is a test version we'd love you to try.

For more information, see https://blog.cocoapods.org and the CHANGELOG for this version at https://github.com/CocoaPods/CocoaPods/releases/tag/1.7.0.rc.2

Analyzing dependencies
Pre-downloading: `Firebase` from `https://github.com/firebase/firebase-ios-sdk.git`, branch `master`
[!] Unable to find a specification for 'Firebase'.

Unfortunately, there's a difference between subspecs of the Firebase pod and the actual product pods. To target a product CocoaPod from source you want the one without the slash in it.
That is, you want pod 'FirebaseCore' not pod 'Firebase/Core'. That applies to all the pods you want to get from master.


Improper handling of partial type stub packages

Pyright isn't properly handling type stub packages (i.e. a package that is named "x-stubs") if the "py.typed" file indicates that it's "partial". Import resolution is failing due to an apparent bug in the readonlyAugmentedFileSystem that was added to support the shadowing of implementation source files.
To repro, pip install types-tensorflow. Then create a new local source file with the following contents:
import tensorflow as tf

reveal_type(tf)
reveal_type(tf.keras)

The tf.keras should be evaluated as a module, but it's not. This is the first symptom of this bug.
Open the site-packages/tensorflow-stubs/__init__.pyi file in the editor. Note that there is an error reported on the line from tensorflow.keras import losses as losses. This should not generate an error and should resolve correctly. This is a second symptom of this bug.
Next, open the file site-packages/tensorflow-stubs/keras/__init__.pyi in the editor. Note that there is a similar import resolution error reported here. This is a third symptom of this bug.
Comment out the multiline from tensorflow.keras import { ... statement. Above it, type from . import layers as layers. Go to the output panel for pyright or pylance to look at the log output. Note that it contains the log: Received change text document command for closed file /<redacted>/site-packages/tensorflow/keras/__init__.pyi. This is a fourth symptom of this bug.
I don't fully understand the readonlyAugmentedFileSystem and its use of "moved entries", but it looks like the problem is related to this mechanism.
@heejaechang, I think you wrote this code, so you're probably the most qualified to diagnose and fix this issue.
This is blocking an issue in typeshed, so I'd appreciate it if you'd make this a priority.

This looks to be a problem in our moved entries not mapping directories. So the keras directory is not mapped and we don't find it when looking for imports.

it is fixed in pyright, didn't pulled into pylance yet.

This issue has been fixed in prerelease version 2023.12.100, which we've just released. You can find the changelog here: CHANGELOG.md


Makes mysqljs easier to minify and support minifier mangle - fix: Error: Received packet in the wrong sequence after minifying

Relying on the function's name of the packets makes the code not minifiable, because they will just shorten up in the minification.
Solution:
I added a new string property "_id" to all the packets and sequences. This way we can make sure the packet/sequence ids will not be minified.
All the tests are passing. I also tested this in a real application using minification and everything was working fine.
related to: https://github.com/mysqljs/mysql/issues/1655

Hey @dougwilson !
I know that this library is not that maintained anymore. I wonder how willing are you to accept this kind of contribution or even if you have the time for that :)
I will make sure I fix the failing unit test left soon (there is only one failing)
Thanks!

I know that this library is not that maintained anymore.

This is a bit offensive, especially since this is not stated anywhere, nor did you ask. It is certainly not something you should be leading with if you actually want someone who is maintaining it to accept a contribute for you :) You may want to reflect on how you approach open source projects in which folks work on during their free time. Opening with an offending statement is not a great way to start off a conversation around doing you a favor :)
I looked at your PR when you opened it and it was not passing CI. IDK what you wanted me to do besides point that you when you already pointed it out when you opened the PR... I was waiting until you fixed it, but now I'm not sure if I'm looking forward to your follow through on this after you've made such a  statement.

That's definitely not what I meant. I am very sorry if I offended you in any way! I am also doing this in my free time.
I try to contribute as much as possible to opensource and be as respectful as possible. It was definitely not my intention to start this conversation like this. I am very sorry about that.
I just wanted to make sure I was going in the right direction to solve this problem.
I will make sure all the tests are passing.

Yes, I didn't comment on it when I first saw it as I didn't have any comments to make; i.e. it seems in the right direction to me.

Thanks! The CI is now passing
Sorry once more, I never meant to be disrespectful or to criticize your work. I respect your work a lot!
Let me know if there is anything else I have to do in order to make this PR ready to merge

I squashed all the commits and rebased on top of the most recent master.
Let me know if there is anything else I can do to improve this PR.
Thanks!

Hi @lneves12 sorry I forgot to follow up on one item (your comment reminded me): it looks like this is not backwards-compatible as in some modules that integrate with this library to extend it fail completely with these changes, breaking them. An example is https://www.npmjs.com/package/zongji for instance. I haven't looked yet if there is a way to make this a backwards-compatible change or I just need to integrate it into the 3.0 alpha. You have have some thoughts, though, so just wanted to add it as a note.

Thanks for the update.
Ahh, I see that zongji library relies on the internals function.constructor.name :/
I will keep thinking of other ways of implementing this without breaking those cases :)
Off the top of my head, I can only think of somehow overwriting function.constructor.name in order to force it to have the full name even if the function is minified, but that sounds super sketchy and I am not sure the impact that could have.
Since it's something not that critical, might be better to just include this, as you suggested, in a major and assume breaking changes rather than implementing a hacky solution

@dougwilson I updated the PR to make the changes backward compatible. IIUC, it was breaking zongji because they have their own sequences/packets. This way the main protocol.js supports both. I am not sure how to test zongji integrated with mysqljs, some insight on that would be welcome
Let me know what you think

hey, any update on this PR? Just ran into this issue in production and was wondering if there's anything  could do to help get it across the finish line


Update HoC page with 2023 activity

Hi Kiki,
Can we update the Hour of Code page https://arcade.makecode.com/hour-of-code/all with a card for the 2023 activity?  I'd like to list this link as a resource in the Edu blog post.
Thanks!

Absolutely. I'm on it. I'll add it to a PR tonight.

https://github.com/microsoft/pxt-arcade/pull/6217

Looks great Kiki!  But I think the links are broken...

Ug. Looks like another place where we use smart logic to change code.  I'll fix.


ARTEMIS-4325 - Ability to failback after failover

Opening for feedback as this might not be the right solution. It does however demonstrate the functionality I'm after.
If the feature makes sense on a high level, please let me know if the implementation needs any changes or can be made better in some way.
Use case detailed in: https://issues.apache.org/jira/browse/ARTEMIS-4325

This feature makes sense to me if your are not using HA (live/backup pairs), have you already considered waiting for a topology update to know when a node is up?
https://github.com/apache/activemq-artemis/blob/main/artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientSessionFactoryImpl.java#L1567

I had the same thought about using the topology update to know when the "original" node came back. I think that would be simpler than manually checking and it would save a thread as well.

@brusdev
@jbertram
Thanks for your feedback, much appreciated!
In the case of failback between HA live/backup-pairs, that should be covered already. Since failback would only follow a failover which would only happen if the retry-logic failed first i.e both live and backup servers are unavailable.
I think failing over (and back) between different live/backup pairs in a scenario like this make some sense, as long as it doesn't happen internally within the groups. If not, would it make sense to add some additional check to inhibit failback between live-backup-pairs?
I might have missed something with the topology listener though,  it would be nice to have this happen without having to "ping" the old node... but would this work if the nodeID of the broker changes (say the broker cluster is part of a blue/green deploy or similar, such that FQDN/IP remains the same but the underlying broker is replaced by one started on a new journal)?
I can give that a try regardless, but it might take some time due to vacation.
Br,
Anton

Isn't the use-case in view here live-to-live failback after failover (implemented recently via ARTEMIS-4251)? I'd leave live/backup pairs out of this completely since there are already semantics in place for that use-case.

would this work if the nodeID of the broker changes (say the broker cluster is part of a blue/green deploy or similar, such that FQDN/IP remains the same but the underlying broker is replaced by one started on a new journal)?

The org.apache.activemq.artemis.api.core.client.TopologyMember has the node ID as well as the TransportConfiguration which you should be able to use to identify host details like name and/or IP address. There are also the isMember methods which may be useful in this use-case.

@jbertram @brusdev
Hello, back again from vacation.
I made some changes as per our discussion and rebased against main. I'm still running the failback check in a separate thread since I didn't want to potentially stall/lock the TopologyHandler but now it's not starting until the node should be available (or available soon). Let me know what you think and see if there are any other changes that should be made here.


[SC-569] Reused mock contracts

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
51

Relevant Lines:
60

💛  - Coveralls


Crash when filling in a long text about why I am giving kudos

What I did Fill in kudos with:
Etienne / Ralph Rooding / 1 / helping with uninstalling Citrix which comes with a damaged uninstaller
What I got "Oops, something went wrong"
What I expected 1 kudo

This is due to this.
I think we should just change this to a text column. @ariejan any reason specifically why this is a varchar(60) now?

The original idea was to keep the activity short so 1) it fits on the dashboard and 2) it fits in a tweet. But if more characters are needed, let's just add 'em.
varchar(255) seems good to me.

On 22 apr. 2016, at 08:29, Pascal Widdershoven<EMAIL_ADDRESS>wrote:
This is due to this.
I think we should just change this to a text column. @ariejan any reason specifically why this is a varchar(60) now?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Or we put a <input maxlength="60"> on it?

@CanolKabisa this might be a nice bug to look into to get acquainted with the app a bit.


Proposal for icon changes (3)

Part 3 of icon change proposals
Before:
The "Open in new tab" and "Open in new window" icons are actually outdated so the change is necessary for these two

After:
Without using the OldFluentUIGlyphs font family anymore (Only custom and Segoe)

Other extra changes:

Change the "Eject" icon to another from the mdl2 font family (visual change is minimal, no screenshot required)
Tweak the size of the new "Column view" custom icon. A little smaller because it does not match the size of the other icons of the new segoe fluent icons
Change the "Preferences" icon on the settings page (visual change is minimal, no screenshot required - Without using the OldFluentUIGlyphs font family anymore)

@XPoppyX This looks good, are these the last icons using OldFluentUIGlyphs?

@yaichenbaum I think it is still used in some parts because if I delete that font family the app crashes when clicking on certain options

i find that the Multitasking maybe shouldn't be exactly the task view symbol?

i find that the Multitasking maybe shouldn't be exactly the Windows Task View symbol as known from the taskbar?

@mangoepic Do you have any ideas what we can use instead?


JSON Patch implementation

http://jsonpatch.com/
We will need to support JSON patch operations, which are not as simple as PUT operations. We will need to be able to distinguish between default nil values and the user explicitly setting a value to nil, as well as format the request body in accordance with the JSON PATCH schema.

https://github.com/thrive-now/JSONPatchSwift

This implementation would actually live in AzureCore. Draft PR here: https://github.com/Azure/azure-sdk-for-ios/pull/381

Since the patch operations needed for preview 1 are very simple and this has not been solved in the wider Autorest world, for now we will just handle this in autorest.swift with known limitations.


Unclear how to configure this inspection from an .editorconfig file

With Microsoft.CodeAnalysis.FxCopAnalyzers 2.9.4 installed to a project, it's not clear what naming is expected within the project's .editorconfig file to configure the severity of CA1507.
These don't seem to work:
[*.cs]
dotnet_code_quality.CA1507.severity = error
csharp_use_nameof_in_place_of_string_analyzer = true:error
CSharpUseNameofInPlaceOfStringAnalyzer = true:error

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

ID: b712c22f-88cd-12ee-395f-c082368a9c4c
Version Independent ID: 43d36c34-3194-1d80-9b68-d426ca907fb9
Content: CA1507: Use nameof instead of string - Visual Studio
Content Source: docs/code-quality/ca1507.md
Product: visual-studio-windows
Technology: vs-ide-code-analysis
GitHub Login: @gewarren
Microsoft Alias: gewarren

@r-k-b Which version of Visual Studio are you using? Configuring analyzer rules using EditorConfig isn't supported in Visual Studio version 16.2.

@gewarren 16.1.3, but I'm not expecting it to show in VS yet - I was checking the build output from msbuild (which is 16.1.76+g14b0a930a7 ... are msbuild & VS versions tied together?)

@mavasani Is it possible to configure the severity of FxCop rules from .editorconfig for command line builds?

Is it possible to configure the severity of FxCop rules from .editorconfig for command line builds?

This should be possible very soon once Visual Studio VS2019 16.3 is released. You should be able to use the following syntax in your editorconfig file:
[*.cs]
dotnet_diagnostic.CA1507.severity = error


[CPP] Fix broken on case-sensitive FS, remove warnings (reorder/unused)

This is for the shared model (affects Android, etc).
Fixed at https://github.com/Microsoft/AdaptiveCards/pull/1681 kindly review and merge.

Thank you guys; this was very informative!


Add support for image pull per runtime class - K8s KEP 4126

What is the problem you're trying to solve
Add support to containerd to identify images as a tuple of (imageName, runtimeclass).
In k8s v0.29.0-alpha.3 tag, CRI support needed to add runtimeHandler field to Image struct was added. It also has the kubelet changes needed to support image pull per runtime class under a feature gate 'RuntimeClassInImageCriApi' (turned off by default).
Describe the solution you'd like
Tasks:

[ ] Update containerd/main cri-api to v0.29.0-alpha.3
[ ] Add runtimeclass label to containerd images
[ ] Update containerd GC to remove images only if it is not referenced by any other runtime class
[ ] Extend containerd CRI image store to reference image as a tuple of (imageName, runtimeclass)
[ ] Extend PullImage() CRI API to honor the runtime handler passed
[ ] Add functional and e2e tests
[ ] Extend crictl.exe and ctr.exe

Additional context
No response

cc @mikebrow @dmcgowan @jsturtevant @marosset @aravindhp

This feature is warmly welcomed, which definiteky  helps to solve the issue we are facing when enabling Nydus snapshotter for confidential containers:)

Extend containerd CRI image store to reference image as a tuple of (imageName, runtimeclass)
Extend PullImage() CRI API to honor the runtime handler passed
Extend crictl

When we were first looking into this I made a POC of these changes so let me see if my branch is still sitting somewhere 😅

Extend containerd CRI image store to reference image as a tuple of (imageName, runtimeclass)
Extend PullImage() CRI API to honor the runtime handler passed
Extend crictl

When we were first looking into this I made a POC of these changes, so let me see if my branch is still sitting somewhere 😅

Sure! I have a draft PR in containerd/main but going to be breaking the changes into smaller separate PRs to make review easier.

This is a nice one to have. Wonder if it's likely to be backported to 1.7?

As this is a new feature, we generally wouldn't backport it.

/cc @zvonkok

PR is pending review: https://github.com/containerd/containerd/pull/10533 . This work is intended for containerd/2.1 and containerd is currently busy with releasing containerd/2.0


#652 - iOS: Fixed podspec to support use_frameworks

When using the Firebase library we are required to add use_frameworks! :linkage => :static to the Pod file of the consuming app. This has until now caused some issues with building Skia on iOS (a workaround exists, but it shouldn't be necessary).
The reason for the problem is that RN Skia uses subspecs with header mappings (we are trying to preserve some of the file structure to make our external Skia includes build correctly) which isn't used when use_frameworks are set. This causes XCode to not find the include files.
This commit fixes this by removing the subspec and adding header search paths for the Skia pod. Only the pod file is changed, so this will not affect Android build.
Testplan:

Run pod-install in the example and verify that Xcode builds the example correctly.
Add use_frameworks! :linkage => :static to Example/iOS/Podfile (line 5-6) and disable Flipper (comment out line 42)
Run pod-install in the example and verify that Xcode builds the example correctly.

fixes #652
fixes #1395

I've now added a PR (https://github.com/Shopify/react-native-skia/pull/1400) that should fix this once and for all - you can test it by updating the RN Skia podspec file in your node_modules folder if you want. Hope this solves the problem! :)


US Federal Holidays corrections

Describe the bug

Although some states observe Confederate Memorial Day, it is not a US Federal holiday.  Please refer to the following links:
https://www.opm.gov/policy-data-oversight/pay-leave/federal-holidays/#url=2022
https://en.wikipedia.org/wiki/Confederate_Memorial_Day
In addition, Federal Holidays are always observed on a weekday.  I can't recall any exceptions to this.  There may be presidential inaugurations or other days declared by executive order as a Federal Holiday, but these would only affect Federal Government employees, not municipal garbage collection.
Configuration
Do you configure the integration in YAML or Config Flow (GUI)? :
Config Flow for Holidays.

Error log/debug information

N/A

Screenshots

This is a snippet from June 22.  Note that both entries are in error.  Juneteenth should be on Monday, and Confederate Day is not a Federal holiday.

Version
Integration version:
1.4
Home Assistant version:
core-2021.12.10
supervisor-2021.12.2

I only see Juneteenth, not the Confederate day, so I cannot fully replicate that.
This is what the holidays library gives as an output:

I've found an issue that might be related to that: https://github.com/dr-prodigy/python-holidays/issues/594
Is that perhaps the issue? If yes, then this will be sorted once they make a release with the bugfix, and I will make a release referring to the new version. Ok?

Yes, you can close this.  In the meantime, I need to figure out how to edit the calendar..


ADS: Dialogs

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #2587

PR #2642   👈
PR #2636
PR #2594
PR #2610
PR #2609
PR #2606
PR #2590
PR #2629
PR #2588

This comment was auto-generated by Graphite.


BUG - Wonky position and animation of the search icon

The search icon is misplaced a bit in the implementation. It should be a bit lower, and a bit less padding left/right around it. If this can eb fixed on the ui-components level, that would be even better.
I am also not sure that the animations play out correctly. You can perhaps see in the slow motion capture that the zoom lens jumps to the left and then disappears. It should just simply disappear and reappear, without any jumping.

@oliversberzs

@gpeuc done & merged

Validated with https://github.com/SAP/guided-answers-extension/releases/tag/sap-guided-answers-extension%401.23.6
@gpeuc Please feel free to reopen this issue if it needs more work
@oliversberzs thanks for the fix


Efficient time variables.

Sub tasks:

[x] Implement newEVar
[x] Implement readEVar
[x] Implement writeEVar
[x] Implement modifyEVar
[x] Implement tweenEVar
[x] Check efficiencyTesterEVar
[x] Check that EVar is identical to Var:

[x] Read/write static values.
[x] Read/write values that change over time.
[x] Read/write static values with time-travel (waiting with negative values)
[x] Change a value over 1 second. Go back 0.5 seconds and modify the value at that timestamp.
[x] Change a value over 1 second. Go back 0.5 seconds and change the value over 1 second from that timestamp.
[x] Write a static value, go back 0.5 seconds, change the value over 1 second from that timestamp.

Lovely code!
I think you were right about TweenValue, btw. It is cleaner to implement as TweenValue Duration (Time -> a). Try changing the implementation and see if the testsuite still passes.


mono-addins: fix build

This build fails I assume for the same reason #46060 does.
We'd also prefer to use msbuild as it would fix this but it's not packaged
Motivation for this change
This build failure
Failure adding assembly ../bin/Mono.Addins.dll to the cache: Strong name cannot be verified for delay-signed assembly

ZHF: #45960
Things done

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

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

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

@GrahamcOfBorg build mono-addins

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: mono-addins
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: mono-addins
Partial log (click to expand)
make[2]: Leaving directory '/build/source'
make[1]: Leaving directory '/build/source'
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/m4a5ydiwdihgx0yjzq5riix4ybh2g465-mono-addins-1.2
gzipping man pages under /nix/store/m4a5ydiwdihgx0yjzq5riix4ybh2g465-mono-addins-1.2/share/man/
patching script interpreter paths in /nix/store/m4a5ydiwdihgx0yjzq5riix4ybh2g465-mono-addins-1.2
/nix/store/m4a5ydiwdihgx0yjzq5riix4ybh2g465-mono-addins-1.2/bin/mautil: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/m4a5ydiwdihgx0yjzq5riix4ybh2g465-mono-addins-1.2...
/nix/store/m4a5ydiwdihgx0yjzq5riix4ybh2g465-mono-addins-1.2

This pins the direct mono dependency to mono48, but mono-addins -> gtk-sharp_2_0 -> mono still brings in mono5. (I found out when investigating why this doesn't eval on aarch64).

Thanks for doing something that I should have been doing :smile:

Can mono48 and mono5 be mixed at runtime? Did you test that these addins actually work now?

Excellent question, I would think that you could not mix them but I'm not familiar with mono
(nudge to the respective maintainers in nixpkgs).
As for testing I can say that pinta builds and runs though I'm not sure what it uses the addins for.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: mono-addins
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

tomboy uses mono46 and depends on mono-addins and gtk-sharp_2_0. So there was a mix of mono versions before this PR...

Anyway, I will merge this as it improves the current situation, and tomboy is the only other package depending on it (banshee is marked as broken).

backported: 26d65faa626


Keithmaibach/sc 67548/home page super admin

Description
add home pages for admins and review admin
What type of PR is this? (check all applicable)

[ x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📦 Chore (Release)
[ ] ✅ Test

Related Tickets & Documents
https://app.shortcut.com/codechallengell/story/67548/home-page-super-admin
https://app.shortcut.com/codechallengell/story/67545/home-page-reviewer-manager
Steps to QA
navigate to root route under both roles
UI Accessibility Checklist

Note the following checklist is not an exhaustive list. Check
the WebAIM checklist for
a user-friendly checklist with more details. Aim to meet at least AA conformance level where applicable. Check
the WCAG guidelines for the full guidelines.

[ x] Markup uses semantic HTML?
[ ] Additional context added through aria-roles and aria-labels?
[ ] Checked
with Wave Browser Extension
for accessibility errors?
[ ] Usable with a keyboard and navigable in a logical order?
[ ] Usable using a screen reader (e.g. Voiceover for Mac) with enough context for the user?

General Checklist

[x ] ✅ Provide tests for your changes where relevant
[ x] 📝 Use descriptive commit messages
[ x] 📗 Update any related documentation and include any relevant screenshots
[ x] 📱 For UI-related tasks, ensure responsive design is implemented across multiple screen size
[x ] 🗂 Move your relevant story/task to the Ready for Code Review state

Hi @bkmaibach - added some additional guarding on routes.  We will need somethign more sophisticated later.

See jurisdicitons index PR


Update docs on valid scoreThreshold range for QnAMakerOptions

Fixes #1808
Description
JS repo doesn't explicitly let users know that the valid scoreThreshold range is from 0.0 to 1.0, unlike the C# repo.
Specific Changes
Updated the QnAMakerOptions.scoreThreshold doc to specify range explicitly, in line with the docs on the dotnet side
Testing
n/a -- it's a comment change

Pull Request Test Coverage Report for Build 109416

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.767%

Totals

Change from base Build 109384:
0.0%

Covered Lines:
7935

Relevant Lines:
9431

💛  - Coveralls


Update .NET SOEs to support ESDK 11

Thanks @aashislamsal. Will merge this PR on July 21.

I am working on updating screenshots and texts I will assign you after I am done.

I guess I am done with updates. Please let me know if you see any issues.

FYI, PR merged, @davi3690


Add tests for RemoveByPath

It seems that there are cases when RemoveByPath does not work correctly. Add tests, fix problems.
Also, add it to Array and CompositeTypeInterface.

Also, add it to Array and CompositeTypeInterface.

That part wasn't done


TelegramClient Example not found

Sorry but I can't find any example regarding how to use a TelegramClient to send message.
If you can help please with a simple example.

The library is not implemented yet.

From<EMAIL_ADDRESS>Sent: 12.01.2015 12:50
To<EMAIL_ADDRESS>Subject: [SharpTelegram] TelegramClient Example not found (#1)
Sorry but I can't find any example regarding how to use a TelegramClient to send message.
If you can help please with a simple example.

Reply to this email directly or view it on GitHub:
https://github.com/Taggersoft/SharpTelegram/issues/1

Ok is there any other way that can help to send telegram messages using c#, because the telegram api focus on web based application.
The QT is too complicated but if you know it can help me to send messages I will try,
Thanks for the fast reply.

Hi,
How much time to implement the API? How much contribution do you need to speed up development?
Thanks

Excause me bot Do you thinkof finishing it !?

Try Notificatio, it's allow to send Telegram messages from c# http://landing.notificatio.me


Does not work in engines

bundle exec rails generate scenic:view

/home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/scenic-1.2.0/lib/generators/scenic/view/view_generator.rb:69:in `views_directory_path': undefined method `join' for nil:NilClass (NoMethodError)
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/scenic-1.2.0/lib/generators/scenic/view/view_generator.rb:14:in `create_views_directory'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/thor-0.19.1/lib/thor/invocation.rb:133:in `block in invoke_all'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/thor-0.19.1/lib/thor/invocation.rb:133:in `each'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/thor-0.19.1/lib/thor/invocation.rb:133:in `map'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/thor-0.19.1/lib/thor/invocation.rb:133:in `invoke_all'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/thor-0.19.1/lib/thor/group.rb:232:in `dispatch'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/railties-<IP_ADDRESS>/lib/rails/generators.rb:157:in `invoke'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/railties-<IP_ADDRESS>/lib/rails/commands/generate.rb:13:in `<top (required)>'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/railties-<IP_ADDRESS>/lib/rails/engine/commands.rb:19:in `require'
	from /home/alex/.rvm/gems/ruby-2.2.3@sexi/gems/railties-<IP_ADDRESS>/lib/rails/engine/commands.rb:19:in `<top (required)>'
	from bin/rails:12:in `require'
	from bin/rails:12:in `<main>'

Hopefully I'm doing something wrong :)

No, that makes sense. You'd need to copy the views directory into the host app along with the migrations you copy over.

I'm getting the same error running the generator in an engine. I created a db/views directory in the engine.

@calebthompson I can rails g migration in engine, but can't rails generate scenic:view, because views_directory_path': undefined.

Any workaround for this? I'm trying to build a solidus extension so I can't copy the views in the main app 👎


Environment
$ pip show asr-evaluation
Name: asr-evaluation
Version: 2.0.2
Summary: Evaluating ASR (automatic speech recognition) hypotheses, i.e. computing word error rate.
Home-page: UNKNOWN
Author: Ben Lambert
Author-email<EMAIL_ADDRESS>License: LICENSE.txt
Location: /home/sfalk/miniconda3/envs/t2t/lib/python3.5/site-packages
Requires: termcolor, edit-distance
Required-by:

The 4 on the SER (sentence error rate) line means that 4 of the 5 sentences were "incorrect".  That appears to be true because the order matters.  So only the first one in the two files is fully correct.  Does that make sense?

@belambert Oh, of course.

Sorry for the delay getting back to you... something does look wrong, I'm taking a look.

Oops, I see.  There was a bug which got fixed, but the latest version wasn't in PyPi.  Can you try updating to the latest version 2.0.3?  I think the command would be something like pip3 install -U asr-evaluation.

@stefan-falk ^^^

@stefan-falk I'm very sorry for the delay getting back to you.  You encountered a bug that was fixed a while ago, but wasn't pushed out to pypi.  I updated it in pypi, so you should be able to just update.  I believe the command is something like pip3 install -U asr-evaluation.

@belambert Alright, good to know - thank you :)


Replace np.random by random

To be able to fix random state with a single command:
random.seed(12345)

instead of
np.random.seed(12345)
random.seed(12345)

this would now cause breaking changes in users code if they were previously setting np.random.seed to fix the seed in the transforms...
Perhaps a better solution would be to pass in a random state to the functions, and if it is None it uses the current random state (got by np.random.get_state).
Thoughts @fmassa ?

What about linking seed to actually torch.manual_seed ?

@ducha-aiki in this case all torchvision random stuff is needed to rewrite with torch.random what is much more breaking change than just drop np.random

I think we need to have some uniformity in here, so we might need to go through a breaking change (which is very small though).
I think we might want to pass a seed argument to the random transforms. But just passing the seed is not enough because we don't want to always have the same transforms, nor to always have to manually change the seed.
Maybe we will need to pass a initial random state, which will be updated by the random transforms.
Something like
def random_flip(img, random_state=None):
    if random_state is not None:
        torch.set_rng_state(random_state)
    # perform the random operations using torch
    ...
    # now update the random state
    if random_state is not None:
        random_state.copy_(torch.get_rng_state)
    # and maybe set the torch rng state back to its original value

But that's a bit annoying.
What do you think?

I think this should be unified. Thanks!

I use np.random in my dataset class, and noticed the issue on multithreading behaving the same for different threads.
My fix is adding worker_init_fn=lambda x: np.random.seed((torch.initial_seed() + x) % (2 ** 32)) to the DataLoader initialization.
If I also use your transform functions that uses the standard random library, should I add a line to the worker_init_fn also, and actually I do not understand what's the problem behind the multithreading, can you give some explanation?

random and np.random behave differently if multithreading type is "fork".
Take a look at this snippet:
import multiprocessing as mp

import random
import numpy as np

def task():
    print("-- Task --")
    print(random.random(), np.random.rand())
    print(random.random(), np.random.rand())
    print("-- End Task --")

random.seed(12345)
np.random.seed(12345)

workers = [mp.Process(target=task, args=()) for i in range(4)]

print("Run")
for w in workers:
    w.start()

and the output
Run
-- Task --
0.6062518296570525 0.9296160928171479
0.7142844140096419 0.3163755545817859
-- End Task --
-- Task --
0.6959326394013083 0.9296160928171479
0.31228532647566887 0.3163755545817859
-- End Task --
-- Task --
0.4936887984487045 0.9296160928171479
0.7509582156538067 0.3163755545817859
-- Task --
-- End Task --
0.6168844031764428 0.9296160928171479
0.5782864572237567 0.3163755545817859
-- End Task --

So, you do not need to add new seed for the workers if use random.
HTH

Thanks!


Check for .tweet-text children before using them

Fix for https://github.com/kennethreitz/twitter-scraper/issues/32
Idk if a try / except IndexError would be more readable

LGTM


[FEATURE] Scrap multiple webs at the same time.

It would be cool to have the possibility to scrap multiple webs in parallel. Maybe using async having an option to limit the number of sites to scrap.

https://github.com/J-Gallo/scrapper-module/commit/3b15297546694b725428739c54c270ab5d39022e


Coong Wallet Application

Project Abstract
Coong wallet is a website-based multichain crypto wallet for Polkadot & Kusama ecosystem that aims to mitigate the inconsistent wallet experience on desktop & mobile and bring a seamless onboarding new users experience to the ecosystem.
Grant level

[ ] Level 1:  Up to $10,000, 2 approvals
[x] Level 2:  Up to $30,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied and aptly renamed (coong_wallet.md).
[x] I have read the application guidelines.
[x] A BTC, Ethereum (USDC/DAI) or Polkadot/Kusama (aUSD/USDT) address for the payment of the milestones is provided in the application.
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
[ ] I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @sinzii:matrix.org

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

@Noc2 Thank you for the support & questions

What are the potential security issues of using the localStorage?

If the site is under an XSS attack then the localStorage could be exposed, but generally, I think the app should have defensive mechanisms for such vulnerability.
Also, private keys & seed phrases stored in localStorage are encrypted using user's wallet password, so even if an adversary had the data, the risk of losing accounts & funds is relatively low.

It might make sense to combine such a wallet with the social recovery pallet: https://github.com/paritytech/substrate/tree/master/frame/recovery. For lower security, the recovery could be set up with a single trusted entity.

Thanks for the suggestions, this would be a helpful feature for users. We'll include this in the roadmap for future development.

Do you have plans to integrate a simple faucet into the solution?

I don't think we have that plan for now, but things might change in the future.


Add css variable to customize tab padding

Tab padding is set at 28px left and right for now, but the spec allows for variable size padding, and the base MDC styling has a Sass variable to control padding size.

Duplicate of #550


 Built using TensorFlow and Keras.
Details instructions on how to train and test the model locally are contained in the README.md file. This file details the model implementation and training process, and gives an evaluation of its performance. It also contains sample image segmentation outputs on the test dataset.

train.py contains the code required to train the U-Net model
modules.py contains the blocks of the U-Net model and loss functions
dataset.py contains the code required to load the NIfTI files
predict.py contains the code required predict the segmentation of an image in the test dataset
util.py contains additional helper functions used throughout the code
paths.py contains the default paths to the data files in the root directory
the assets folder contains all images used in the readme file

s4644427

This is an initial inspection, no action is required at this point

Solves problem: The problem was solved very well, all labels reached over 0.75 DICE score. All experiments were well recored with DICE and loss plots. Excellent dataset and label distribution analysis.
Implementation: All functions are well-implemented. It would be better to implement UNet in Class. Hyperparameters are hard-coded rather than using ArgParse.
Design: Nice
Comment: insufficient comments for functions. Many functions do not have docstring.
Difficuclty: Easy

Marking

Good/OK/Fair Practice (Design/Commenting, TF/Torch Usage)
Good design and implementation.
Spacing and comments.
No Header blocks. -1
Recognition Problem
OK solution to problem. -1
Driver Script present.
File structure present.
Good Usage & Demo & Visualisation & Data usage.
Module present.
Commenting missing. -1
No Data leakage found.
Difficulty : Easy. UNet2D (Easy Difficulty)-10
Commit Log
Good Meaningful commit messages.
Some/Adequate Progressive commits. Somewhat less.-1
Documentation
Readme :Good.
Model/technical explanation :Good.
Description and Comments :Good.
Markdown used and PDF submitted. PDF checked.
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch). Submitted on time.
No Feedback required.
Request Description is good.
TOTAL-14

Marked as per the due date and changes after which aren't necessarily allowed to contribute to grade for fairness.
Subject to approval from Shakes

Segmentations not one hot encoded -2 additionally


[QUESTION] Any tips on how I can reconcile parts "produced" by other parts?

Hi,
Today I came across a "philosophical" question about how to reconcile when a part is produced by other parts.
The reality is that we order one tube of 6 meters. I want to specify to InvenTree that, in order to produce a tube of 1 meter, it will consume one 6m tube to create 6 1m tubes.
I tried using build orders, but they can only do the inverse. 6 6m tubes would make 6 1m tubes.
Is this possible to reconcile, or do you have any tips on how I can handle this effectively?
Kind regards,

I would suggest that the simplest approach would be to set the "units" of the Tube part to "metres" - you will then have a single stock item of 6m which you can "use" to make smaller lengths?

Also we now have opened up a page for discussions which are separate from issues - where this would be a more appropriate query :)

Thank you so much Schrodinger! I'll go to the discussions right away.

@SchrodingersGat Hey I've been playing all days with units in InvenTree but it seems as they are strings, they don't really do anything meaningful in terms of dividing bigger items into smaller ones. Care to throw me a bone?

@Guusggg as far as I know units are just what is shown; You can use fractions of a part (0,25 cm of a part that is per 1m for example)  in a BOM (it can not be tracked then) -> if you need it tracked make a part with the metered part as the only BOM item.

As @matmair suggested, the "units" are only an aid to the user, they do not provide any functional logic. My suggestion was simply to specify that stock levels for the "Tube" are recorded in units of "metres" and then plan your bills of materials accordingly


Global Secure Access and Microsoft Defender

Hi
Are there any plans to incorporate this into https://security.microsoft.com? It would be really useful to have at least the reporting components of this technology stack on the Defender Portal.
Also there was no mention of iOS/Mac in the document. I'm assuming there will be support for it at some point.
I will be testing this technology shortly.

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

ID: a38ffcf6-31e4-abc7-d13f-1535236e6401
Version Independent ID: 4dc4d4fa-b377-be53-fe99-f23c3dfdebba
Content: Get started with Global Secure Access (preview) - Global Secure Access
Content Source: docs/global-secure-access/how-to-get-started-with-global-secure-access.md
Service: global-secure-access
GitHub Login: @shlipsey3
Microsoft Alias: sarahlipsey

@peterjoinobits
Thanks for your feedback! We will investigate and update as appropriate.

@peterjoinobits
Microsoft is constantly working to improve its security offerings and may consider incorporating Microsoft Entra Global Secure Access into the portal in the future. The document you referenced does not mention particular support for iOS/Mac. However, Microsoft Entra Global Secure Access supports a wide range of platforms and devices, including Windows, macOS, iOS, and Android.
For any technical queries or clarifications,  I'd recommend working closer with our team, we encourage to utilise Microsoft Q & A platform. Kindly raise your query on Microsoft Q&A Platform

@peterjoinobits
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close


Use splash hash rather than index for caching

This allows to add/ remove splash texts while not "corrupting" the cache.
This has been tested on Arch which is the distro I use, BTW!

This allows to add/ remove splash texts while not "corrupting" the cache.
This has been tested on Arch which is the distro I use, BTW!

Hi!
Nice idea with hashing the texts!
Sadly, the hash function seems to not really hash consistently, which kinda makes the cache not a cache anymore x>

Is this only a problem on my machine?

the problem is the use of the 'hash()' function. It returns the hash of objects, so every time you execute the program, a new object will be created (even for an identical string) and therefore the hash of the different object will also be different

Fixed with #30


chore(master): release 3.0.0

:robot: I have created a release beep boop
3.0.0 (2023-11-29)
⚠ BREAKING CHANGES

geth blockchain and hardhat (#105)

Features

bee 1.17.2 (#97) (1fea708)
geth blockchain and hardhat (#105) (6d6da0b)
phase4 redis (#102) (c6a6f7f)

Bug Fixes

byte updates (a9a01aa)
contract migration for the new bytecode (0a6f877)
update contract addresses (548bc22)
update contracts bytecode (1aa9ee2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/fairDataSociety/fdp-play/releases/tag/v3.0.0 :sunflower:

Release v3.0.0 addresses this.