Question about Profile:Release()

I was wondering if it is ok for a Player to call Release() function on Profile when the ProfileService function connected to BindToClose is running. Basic Usage simply suggests that you call Release function when a player leaves but what if the player left when ProfileService is already busy saving data before server shutdown will extra requests to save/release profile cause any issue?
Question 2
For games that load Profile.Data into Folder containing ValueBase instances for all the Stats and constantly update Profile.Data with all the changes made to those ValueBase instances is there an Event inside Profile that allows Dev to update Profile.Data for one last time before Profile:ListenToRelease() gets triggered? There was a similar issue with BindToClose so I had to manually add a function right below the line "2387: ProfileService.ServiceLocked = true" inside the ProfileService module which updates Profile.Data of all players one last time. I did some testing by joining the actual game server and changing some ValueBase Values then quickly pressed play in Studio to trigger the ListenToRelease event and as expected because there was nothing to update the Profile.Data in such situation the latest changes to ValueBase Values didn't save. In the next test, I add a loop to Sync Profile.Data and ValueBase every 3 seconds it worked and the player loaded with the latest Data in Studio but using loop is really not efficient and it is also not efficient to connect a function to .Changed of 200-300 ValueBase instances(per player). So can you please add an event that allows Dev to Sync Data right before ProfileService attempts to Save Profile.Data

A1:
:Release() Does nothing after a profile is already released and BindToClose will trigger releasing on its own. However you should regularly release your profiles after you're done using them as failure to do so will eventually lead to too many auto-save API calls.
A2:
ProfileService was created with some implicit security measures in mind, including steering users away from saving data before release - this would render player data completely unrecoverable after server crash as opposed to partially or even fully recoverable. New future versions of ProfileService will still include these features for full control, but for now it will stay as is.

Oh, I understand the importance of the auto-save feature and I did add one for the cache system too so it can keep Profile.Data updated with the latest Value in ValueBase instances after every 3 seconds (around 300 ValueBase per player) but since ListenToRelease can be called at any time even intentionally to dupe stuff and when it gets called the latest data is mostly inside ValueBase instances. I need to find a way to update Profile.Data with the latest Values right before it attempts to force release/save old session so it can load the Profile in another Server. Also, I wanted to further increase the yield time of the cache system because I think that 3 seconds is not healthy for the server though I did try my best to optimize the process responsible for moving the latest values of 303 * 15(No. Players) ValueBase into Profile.Data So it would be amazing if you can guide me lil bit on what kind of patch I should apply :P well if I can find the function responsible for saving data in that situation then i can add something before it which updates Profile.Data for one last time and then release/save it


Using wast in combination with wasm-encoder

I'm trying to parse a WAT Expression and emit the instructions into a wasm-encoder Function.
This is for a language compiler i'm writing that has inlines WAT code.
The problem is that wast::Instruction is not compatible with wasm_encoder::Instruction.
Also the wast::binary::Encode trait is private, so i can't use that in combination with wasm_encoder::Function::raw either.
My suggestion is to either make the wast Encode trait public, or implement From<wast::Instruction> for wasm_encoder::Instruction.

Thanks for the report! The eventual goal is to have wast depend on wasm-encoder, and at that point we could have conversion from the wast AST to types in the wasm-encoder trait, such as wast::Expression returning a Vec<wasm_encoder::Instruction> or similar. At this time though that's not yet implemented, but if you're interested to help out I think it'd be great to implement!


Consider moving to Scala 3

Scala 3 has some very nice features for beginners (enum keyword, new keywords for implicits). Moreover, most open source libraries support it by now. Perhaps we should think about moving this course over to Scala 3?

I'm aiming to do this tomorrow.


Dropped files can be visualized without backend.

dropped files are stored in localstorage, then a link is created and passed to the viewer.

This has been folded into three more recent pr merged today, so I'll suggest to close this unless you have other plans @AlbertoCasasOrtiz

I agree, I'm closing it.


support Ed448 signing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 59 (23.73%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 56.662%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

jwa/eddsa/eddsa.go
14
59
23.73%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
4359

Relevant Lines:
7693

💛  - Coveralls


[Help Wanted] How to support generate type int(not type int64)?

Problem statement
Actually I need to use swagger to generate my modes in go code. But I find I can hardly generate type int, always int64.
Since I need to keep back compatibility the project, I can hardly change the type. So in this case, do go-swagger meet my demand? If the answer is yes, how to make it.
Please remove the sections that don't apply
Swagger specification
Steps to reproduce
Environment
~ $ uname -a
Darwin AllenSundeMacBook-Pro.local 16.7.0 Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT 2017; root:xnu-3789.70.16~2/RELEASE_X86_64 x86_64
~ $ swagger version
version: 0.12.0
commit: 8135eb6728e43b73489e80f94426e6d387809502
~ $ go version
go version go1.8.3 darwin/amd64

Thanks in advance

Discussed briefly in #675

Yeah, it is brief in #675 . While I think we have not get the solution. So I suggest that it is better for us to keep this issue open, since maybe other users would have the same issue. @gregmarr

IMO int is not a good option to support when it comes to contracts.
Consider the following: you have an arm32 client on which int is int32, however your server is amd64. At this stage it's perfectly valid for the server to return int32 max value + 1, this will cause the client to overflow.
So while go allows int as type I think for API contracts int is too ambiguous as definition leading to subtle but hard to debug failures.
Similarly other languages may choose to default to int32 type instead of int64 type regardless of platform.

This is also causing an issue for me. Although I do agree that int and uint are not good for enforcing contracts, it shouldn't be dictated by the framework if the language inherently supports it. If strict typing needs to be enforced, that can be done in the swagger spec file.


How to get path of cropped Image?

How can get path of cropped Bitmap file?

Hi,
SimpleCropView does not save the cropped image to local file, getCroppedBitmap() just returns Bitmap. Please try sample.


influxdb sink not work with option retention

--sink=influxdb:http://monitoring-influxdb.kube-system.svc:8086?retention=1h0m0s
the option retention dose not work.
using curl get the default retention policy of dababase k8s, the retention is set to 0s
env:
heapster 1.4.0
influxdb v1.1.1
in the function createDatabase, when command 'CREATE DATABASE %s WITH NAME "default"`, sink.c.DbName' execute, the default retention policy was created, and retention is set to 0s, and the funciton createRetentionPolicy will not be execute.
func (sink *influxdbSink) createDatabase() error {
	if err := sink.ensureClient(); err != nil {
		return err
	}

	if sink.dbExists {
		return nil
	}
	q := influxdb.Query{
		Command: fmt.Sprintf(`CREATE DATABASE %s WITH NAME "default"`, sink.c.DbName),
	}

	if resp, err := sink.client.Query(q); err != nil {
		if !(resp != nil && resp.Err != nil && strings.Contains(resp.Err.Error(), "already exists")) {
			err := sink.createRetentionPolicy()
			if err != nil {
				return err
			}
		}
	}

	sink.dbExists = true
	glog.Infof("Created database %q on influxDB server at %q", sink.c.DbName, sink.c.Host)
	return nil
}

This is a duplicate of: https://github.com/kubernetes/heapster/issues/1579

closed as duplicate


add array length validation

Adds length validations for array objects for examples tags.

This looks legit to me at first glance. Would love some coverage in the JavaScript spec before merging.
:+1:

@joecorcoran I've added Javacript specs. Jasmine tap reporter doesn't support nested describe, so I had to name the nested descriptions manually to be consistent with the reporter output.

thx @musaffa!


feat: LC-179 external video

Jira issue(s)

Overview
Screenshots
Testing prerequisites
Testing scenarios

[ ] A
[ ] B
[ ] C

Notes

closed due to https://github.com/Selleo/lms-core/pull/195


Update Makefile for CI and remove PR check script

Updated to the Makefile to work with Origin CI and removed PR check script as we're running the two make targets in prow in parallel.

Running make now changes the contents of assets/bindata.go - any chance we can commit the new version, or lock in to a specific version go-bindata so that it doesn't stage changes to assets on every go-bindata release?

@vkareh I was looking at locking the version of go-bindata specifying @3.1.3 didn't seem to work. Let me look at it you are right its changing (had to remove my older version)

@jharrington22 - what's the status of this? Can we merge as-is or does it need more work?

@vkareh I need to check the PR for openshift/release then we can merge this.

This one: https://github.com/openshift/release/pull/9633 ?

@vkareh this is updated, once merged I'll remove the hold on openshift/release#9633

/lgtm

@jharrington22 - can we merge this now or are we still waiting for more CI work?

@vkareh this is good to go.


Feat/k8s use versioned images

Description
Makes Orchest use versioned images instead of :latest as a default.  Starting from this PR the version with which orchest-ctl runs with depends on the latest tag of Orchest. This branch adds a k8s-beta tag to Orchest, which reflects our temporary k8s images that have been pushed to dockerhub.
Given the use of the k8s-beta tag, and the use of the IfNotPresent image pull policy, if you want to test local changes you will now have to build with the current tag and version. Example:
eval $(minikube -p minikube docker-env)
bash scripts/build_container.sh -i orchest-ctl -o k8s-beta -t k8s-beta

@yannickperrenet base images need to point to the correct version as well, see https://github.com/orchest/orchest/pull/799/commits/7b3037996d3531a34fd0fbf12a7f0383c1069f46, let me know if your question was answered


BindableProxyFactory not inject in FunctionConfiguration#afterPropertiesSet  after adding spring-boot-starter-web-services

Hi, I'm using spring cloud version 2020.0.4 and spring-boot-starter-parent 2.5.7.
in my sample located in github , i'm trying to use spring-boot-starter-web-services after adding basic spring cloud dependencies. but after adding that, no queue was created.
i try to debug and only thing that i found is when program reach to FunctionConfiguration#afterPropertiesSet , BindableProxyFactory  beans of type array is empty after i add spring-boot-starter-web-services.
Where is the mistake?
and my stackoverflow post about this issue

I found this spring.cloud.function.definition property for define a function for creating queue. but i need know
how can automatically create queues in modules more than one function with this type of definition?

This does not seem related to spring cloud config

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Support local installations of beautifiers

From @cdfa on December 11, 2018 13:49
Is your feature request related to a problem? Please describe.
Yes, I was trying to get this working for a project where in which the dev-dependencies state everything I'm using to develop the project except for editor specific stuff, but unibeautify seems to be unable to find beautifiers (like beautifier-js-beautify) when they are installed locally.
Describe the solution you'd like
Support locally installed beautifiers.
Describe alternatives you've considered

Having the beautifiers globally installed, but the other stuff locally, with a comment in the package.json saying the beautifiers need to be installed globally.

Additional context
Copied from original issue: Unibeautify/unibeautify#213

OK, yes, the CLI only looks for globally installed beautifiers via g-search, then loads them via requireg.  It would be an enhancement to add support for local installations.

Applicable code:
https://github.com/Unibeautify/unibeautify-cli/blob/master/src/utils.ts#L3-L20
https://github.com/Unibeautify/unibeautify/blob/master/src/DependencyManager/NodeDependency.ts#L40-L52
It may require a change to Unibeautify core at https://github.com/Unibeautify/unibeautify/blob/master/src/DependencyManager/NodeDependency.ts#L44-L46 as well as CLI to search for global in addition to local.
Pull Requests very welcome!

@Glavin001 is there a package out there that searches or lists locally installed packages?  I can't seem to find one.

Not really.  Looking for something like this, only it searches in local:  https://github.com/TremayneChrist/npm-g-search.  Guess we could borrow that code and modify it to our needs...


Need help:Why my code made an error?

type b struct{
	Result string   `json:result`
	Error_info string   `json:error_info`
	Level string   `json:level`
	Score string   `json:score`
}

here is my struct
{"_idip_msg_id_":["7884_1560700537_<IP_ADDRESS>_6693_1046452841"],"result":["0"],"error_info":[""],"level":["1"],"score":["100"],"_idip_req_id_":["2"]}
And here is my json
But error says
main.b.Level: ReadString: expects " or n, but found [, error found in #10 byte of ...|{"Level":["1"],"Scor|..., bigger context ...|{"Level":["1"],"Score":["100"],"_idip_req_id_":["2"],"_idip_|...
What's wrong with my code

en..... could you put your code here?

It looks like your struct does not match your json string. I think the following struct may work.
type b struct{
	Result []string   `json:result`
	Error_info []string   `json:error_info`
	Level []string   `json:level`
	Score []string   `json:score`
}

@jxww   You are right, of course, the following json string also works.
{"idip_msg_id":"7884_1560700537_<IP_ADDRESS>_6693_1046452841","result":"0","error_info":"","level":"1","score":"100","idip_req_id":"2"}


fix: cache whole response (#125)

Adding helper functions to cache, so useQuery will not create new object on every render

Thanks @standy. Published as v0.4.5.


Debug visualizer

We should have a debug visualizer application that allows you to compute arbitrary isochrones and see their constituent paths—this is a good way to find bugs in the algorithm.

See RouterVisualizer.jl


Fix incorrect issuel filter menu style

Before:

After:

Yup, in old releases the "active" items are set to "primary / green / blue" colors, and at that time many elements have that eye-catching colors.
Then most unnecessary eye-catching colors are removed (set to grey, etc), so I also agree this "active" color fits better.

Sorry, might have merged a bit too soon here, but the active merge button was just too tempting.


Development/full version of qwest should work with full versions of dependencies

IMHO, if I have amd config,
require.config({
  baseUrl: '/script',
  ...
  paths: {
      ...
      'qwest': ['lib/qwest'],
      'pinkyswear': ['lib/pinkyswear'],
      'jquery-param': ['lib/jquery-param']
  },
  ...
  shim: {
    ...
    'pinkyswear': {
      exports: 'pinkySwear'
    },
    'qwest': {
      deps: ['jquery-param', 'pinkyswear']
    }
  }
});

and the paths point to the full versions of the files, the library should work without requiring a build.
Actual:
Uncaught ReferenceError: PINKYSWEAR is not defined

UPDATE
Even switching for the built version I cannot get qwest to work with requirejs.  I'm using the above config.
define(['qwest'), function(ajax) { console.log(ajax.post); /*undefined*/ })

Is there any documented setup for qwest and amd/requirejs?

Thanks fort reporting, I'll investigate that ASAP

I don't use RequireJS so I need some explanations on how you're using it. How are you using qwest? Do you pick the source file on the repo? Or installing it with NPM?
The minified version, at the root, is for the browser directly, all dependencies are included and qwest is referenced in window.
The NPM version (available with require('qwest') and existing in the build/ directory), imports pinkyswear and jquery-param by itself.
The qwest source file should never be used.
Can you enumerate me your needs for RequireJS then I can see if I can create a specific build for it? Are you importing directly qwest via a package manager? Or are you composing your build by picking the files you're needing?

@pyrsmk You can use the following browserify command to work with require.js
browserify -s qwest index.js > qwest.js

I am unable to submit a pull request at this moment, but that should give you what you need to fix this problem.

Sorry for the late! I'm now working on better building system for qwest, based on @jtenner 's comment. I just don't really know how to do this without breaking things in user updates. I'd created an UMD branch, but it won't be available on NPM... So it seems I must push the changes on the master branch... But before doing anything crazy, is anyone have a better solution to test this?

I've just pushed the new build process on the UMD branch.
There's a qwest.min.js that is the minified source file, and a qwest.standalone.min.js that is bundled with its dependencies and should work in the browser.
I'm not quite sure, but the qwest.min.js can only work on Node.js, isn't it?
@jtenner : if you have time for this, can you verify if all seems ok to you?

Ok... I'm making the tests, there will be only one file : qwest.min.js. It should work with any environment. I'm also working on an implementation for web workers. Stay in touch.

@pyrsmk making a standalone build should work perfectly in every environment.  Common.js, UMD, and browser.
Cheers,
-Josh

Should be fixed now, reopen if needed

what's about support for bower too?

I don't really understand your question, because qwest is already supported in bower.

I meant, installing the library using bower and then using requirejs to fetch the code, it does not register. It;s undefined.
I had to use the npm version in the end and it works. Will be good to have it work in bower too

Hum... It is weird... The main file in package.json and bower.json is the same. I assume that the issue comes from the dependencies, which are defined in package.json but not in bower.json because PinkySwear is not available on bower.
If my assumption is right, we'll need to wait for the future v5 version (PinkySwear will be dropped in favor of better/stronger implementation).

@tm57 : I'm not used to RequireJS but did you take the good qwest version? Because the one located in src/ is not meant to be used as is, you should take the qwest.js version at the root of the project.


[WIP] Add markdown components

Take a look and see if this is what we want to achieve in #37 , I also included some screenshots

Web version, in which I think only quotes are broken,

Native version, links are broken (we discussed that), and something is broken with the lists. Also, text in code blocks should have different colors with different themes background, now it's the same as all texts, so with dark color we have light text on light background and it's hard to read

Native version without hyperlink, links work then

Based on screenshots -

Web version looks good.
First native version breaks the "Preserving simple messages" directive in #37 by adding a padding above and below all messages.
It's not clear to me why links are broken in the native version, given that there is an "autolink" rule specified in react-native-markdown-package.
Where is the background color in "Native version without hyperlink"?

Ad 2 - yeah, I saw that after I compared more carefully, its easy to fix with styling
Ad 3 - for some reason, when Markdown component is inside Hyperlink it returns object and is not displayed, don't know why
Ad 4 - style for message background is currently in Hyperlink, if we remove hyperlink it should be in TouchableOpacity

Okay awesome! Based on that it seems like you're pretty close!!

Okay awesome! Based on that it seems like you're pretty close!!

yeah, I guess I just need to reuse this to create our own markdown component and hopefully figure out what's wrong with lists?

Yeah! Once you've verified that the approach will work (I guess links are all that's left?), you can start on the actual PRs.
I would start by just replacing react-native-hyperlink (on native) and react-linkify (on web) with a single-rule autolink markdown parser, as described by my comment last night in #37.

For now, we have rules for autolink and to preserve simple messages, but since simple-markdown uses rule for link in autolink, I've also included this one.

One more thing - for now, I had to remove styling emoji-only text, need to figure out how to put this in styles for markdown.

Let me know when this is ready for a review! (I'm assuming you're still working on it since it's listed as WIP)

I think I've checked all the cases for emoji only message and it looks like before

 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.

In #104 I noticed yet another issue with newlines. If I sent a message like Test\n\nA it looked like TestA. I also tested it against the version of this PR before https://github.com/Ashoat/squadcal/pull/67/commits/04d79735e1ffc0f2dba9a8cc0fea520dae4b64ef, and found it was broken in a different way: it looked like Test\nA.
I think I've got it fixed with https://github.com/Ashoat/squadcal/pull/67/commits/04d79735e1ffc0f2dba9a8cc0fea520dae4b64ef. Things I had to change:

I wasn't able to figure out a way to set the text style in InnerTextMessage. Whatever style I set would mess up the children, and collapse newlines together. So instead, I am passing a prop textStyle to the Markdown component, which then passes it to the logic in rules.react.js through the state variable.
The prop darkStyle seemed confusing next to textStyle, since one is a style but the other is a boolean. So I renamed darkStyle to useDarkStyle.
Because the text style is now set in the rules again, I had to bring linkPresent back.
I had to edit the paragraph RegEx again, this time to make sure it didn't match any trailing newlines. We want every single newline to be converted to its own paragraph. This time I simplified the RegEx greatly so it just matches single lines. I also had to switch from anyScopeRegex to blockRegex so we wouldn't match paragraphs within paragraphs.
Because the RegEx change made the \n\n that gets appended turn into two paragraph elements, I had to disable those with disableAutoBlockNewlines: true.
I made the emoji-only style just 2x the fontSize of the textStyle we get passed in.
I had to set the paragraph style to nowrap so every paragraph would start a new line.

Requests for you:

Can you rebase #104 against the updated version of this branch?
When you were working on this PR, I know you tested a bunch of edge cases. Can you test them again on this updated version to make sure things look good?
Can you make sure to test the web version of the Markdown component to make sure it can handle newlines well? I'm wondering if we'll need to port any of the above changes over to web. The important thing to test is that it should never merge newlines together. This should hold true even if there is a newline at the start or end, since we need to calculate an Entry's height while somebody is editing it and adds a newline.


PolyBase head node information is out of date

In the Process Overview section, bullet 2, it says "A head node can only be designated on an instance running SQL Server Enterprise." That appears to have changed in SQL Server 2019, and head nodes can now be created on Enterprise or Standard according to the chart here: https://docs.microsoft.com/en-us/sql/sql-server/editions-and-components-of-sql-server-version-15?view=sql-server-ver15#Programmability
Specifically note 4, which reads "4) Prior to SQL Server 2019 PolyBase head node requires Enterprise Edition."
[Enter feedback here]

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

ID: acaaf25c-0a4e-05af-dc8f-307ff92e3b17
Version Independent ID: 14446de6-0c23-fae4-135c-4cc34aff96a3
Content: Configure PolyBase scale-out groups on Windows - SQL Server
Content Source: docs/relational-databases/polybase/configure-scale-out-groups-windows.md
Product: sql
Technology: polybase
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@awilbert -- Adam, thank you for your feedback.  I am looking into this article.


add an option to define the locale for the BigDecimal Parser annotation

This allows to parse values that are created in a non standard locale format like german which uses a colon as the decimal separator.

I'm sorry for the reformatted header. I just discovered that now, seems like my IDE did some adjustments.

Codecov Report

Merging #19 into master will decrease coverage by -0.45%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master      #19      +/-   ##
============================================
- Coverage     69.28%   68.83%   -0.45%
- Complexity        0      213     +213
============================================
  Files            68       68
  Lines           625      629       +4
  Branches         71       56      -15
============================================
  Hits            433      433
- Misses          167      171       +4
  Partials         25       25

Impacted Files
Coverage Δ
Complexity Δ

...provider/ParseBigDecimalCellProcessorProvider.java
22.22% <ø> (-17.78%)
2 <ø> (+2)

Continue to review full report at Codecov.

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


Detecting canonical updates

Hi!
Parsing the push response for uniqush-push versions < 2.x I could detect GCM Canonical IDs (https://developers.google.com/cloud-messaging/registration#keeping-the-registration-state-in-sync) and remove the old registrations within my own implementation.
As now in case of "push.DeliveryPointUpdate" only ode "UNIQUSH_SUCCESS" is received there is no longer the possibility to detect this scenario, right?.
Is it possible to introduce a new additional result code, add a new property to ApiResponseDetails and/or add a error message?
Thanks & kind regards
// Clemens

@FischC It's definitely reasonable to do. Can you help me understand your use case a bit more, like how you have it set up? Are you keeping track of things in another database for other purposes? If we can understand others' use cases better, we can better enhance the APIs to meet more of these use cases.

Dear mishan!
Thanks for your reply. Your assumption (https://github.com/uniqush/uniqush-push/issues/114) is correct, I'm using a custom CRM light system in combination with uniqush push: APP <-> CRM API <-> CRM light system <-> Messaging module <-> uniqush-push!
And the use-case is that I need to flag the contact in case the e.g. App is e.g. no longer installed (dropped in uniqush-push terms), etc.! Thus I trigger some callback URL within the messaging module  to notify the CRM light system in case of UNIQUSH_REMOVE_INVALID_REG,UNIQUSH_UPDATE_UNSUBSCRIBE and errors UNIQUSH_ERROR_NO_DEVICE, UNIQUSH_ERROR_NO_DELIVERY_POINT and UNIQUSH_ERROR_BAD_DELIVERY_POINT. This is working as a charm and maybe my issue could help others to interpret the push responses ... :-)
Unfortunalty some versions of our App still produce canonical IDs (via our CRM API) and in case a canonical ID update in uiqush-push < 2.x occurred, the messaging module informed the CRM system, which then starts some recovery procedures in order to detect duplicated contacts and to resolve this duplicates.
Eventually a new status UNIQUSH_SUCCESS_UPDATED could be introduced indicating that the push was successful and uniqush-push updated the delivery point?
Thanks & kind regards
// Clemens

Hi all!
push#113 solved this issue for me ;-)
Thanks
// Clemens


oui: add CC8CBF (Tuya Smart Inc.) to nonesp

Found on NOUS Smart Socket A1 (MFG 01/2022).
New versions of the A1 are using a BK7231N instead of an esp8266.

I have them supported:
https://github.com/openshwprojects/OpenBK7231T_App
There is also tuya-cloudcutter for Beken chips.

Thanks!


Jenkins on OpenShift and mult-branch

Pelorus depends on jenkinsPipelineStrategy and is designed to work with Jenkins builds in OpenShift. We have some engagements that use Jenkins on top of OpenShift and uses multi branch strategy.
This issue is to discuss and align on a solution that will enable Pelorus to work with Jenkins using multi branch.

As noted in the https://bugzilla.redhat.com/show_bug.cgi?id=1785291 the JenkinsPipelineStrategy is deprecated from OpenShift version 4.3.z.
With that I am closing this issue. Note that we currently have a way of configuring Pelorus with images using image exporter documented at:
https://pelorus.readthedocs.io/en/latest/GettingStarted/configuration/ExporterCommittime/#using-commit-time-with-images


Append blocks information to exceptions

For errors thrown during compose/activate/etc., they should be annotated with which blocks threw the error.
Without having to specify the block name from the block itself
For example a block named Math.Add with this line:
throw ActivationError("Invalid input")
should be able to produce the following error message
Math.Add(0x00f82300): Invalid input
where 0x00f82300 is indicating this specific block instance (could later be replaced with source line info)

RA-57


Cross-Validation-from-Scratch #114

I have code for 4 methods involved for cross validation.
Leave One Out (Linear Reg)
Leave P Out (Linear Reg)
K fold (Log Reg)
K Stratifiedfold (Log Reg)
Please review it.
(Sorry it's issue #114 )

Not a class type implementation, @Dishita2602 Please add a class type implementation

Not a class type implementation, @Dishita2602 Please add a class type implementation

Can you please be more clearer ?

Hey @Dishita2602,
please go through the Contributing Guidelines.
We need a class-type implementation of the algorithm/technique from scratch using nothing but NymPy.
No external modules/libraries are allowed.
To know more about classes in Python please refer here.

Hey @Dishita2602,
please go through the Contributing Guidelines.
We need a class-type implementation of the algorithm/technique from scratch using nothing but NymPy.
No external modules/libraries are allowed.
To know more about classes in Python please refer here.

Issue was not created by me so I was not knowing that you guys are only looking for cross-validation implementation without any libraries. Also I had commented in issue that I am going to use these 4 methods but nobody replied to it. That's why I did PR as per my understanding.

Hey @Dishita2602,
please go through the Contributing Guidelines.
We need a class-type implementation of the algorithm/technique from scratch using nothing but NymPy.
No external modules/libraries are allowed.
To know more about classes in Python please refer here.

Issue was not created by me so I was not knowing that you guys are only looking for cross-validation implementation without any libraries. Also I had commented in issue that I am going to use these 4 methods but nobody replied to it. That's why I did PR as per my understanding.

Hey @Dishita2602,
we might have missed that, apologies. But we have mentioned everything in the Contributing Guidelines.
Please make the amendments in the current code-base. Thank you.

Hey @Dishita2602, you willing to complete it?

This doesn't follow the contributing guidelines mentioned and PR creatrr is inactive. Hence closing this PR


use less granular progress

Use a few larger progress displays rather than many smaller ones (also, don't nest progress displays).
From the cli:
Syncing files to device...                           0.2s
Performing hot reload...                             1.1s
Reloaded 230 of 499 libraries.
Hot reload performed in 2018 ms.

More importantly, this works better in IDEs. It's nicer to be able to show one progress message for a few hundred milliseconds, instead of flashing 4-5 smaller ones.

@johnmccutchan, for the change to how we report the hot reload stats.

Fix https://github.com/flutter/flutter/issues/5399


Added missing translate for Hungarian language

Already merged #811 which is similar with more

@bl4ckdu5t this actually has more 😉

@gergo85 or @bl4ckdu5t can you please resolve the conflicts?

The [ 'V', 'H', 'K', 'Sze', 'CS', 'P', 'Szo' ] is a correct typography instead of [ 'V', 'H', 'K', 'SZe', 'CS', 'P', 'SZo' ].


Request: Please add TSX support

At first it seemed simple to just do the PR myself, but there were parts of the code I didn't understand, so I'm requesting the change instead...
Intel's Transactional Synchronization Extensions (TSX) support is somewhat unpredictable, given that there was a bug that led it to be disabled in Haswell and many Broadwell CPUs, and that it's not included in every Skylake and Kaby Lake model. So having it in CPUID would be especially helpful.

Detection of TSX is split into HLE (Hardware Lock Elision) and RTM (Restricted Transactional Memory) detection. It would seem like TSX = HLE & RTM in general terms. Both of which are available.
I will add a separate function for this.I have not had the chance to verify this on hardware, so please so that if possible.

Added to master.


Fix json2pb compile error of examples

What problem does this PR solve?
Issue Number: fix #2481
Problem Summary:
What is changed and the side effects?
Changed:
Side effects:

Performance effects(性能影响):

Breaking backward compatibility(向后兼容性):

Check List:

Please make sure your changes are compilable(请确保你的更改可以通过编译).
When providing us with a new feature, it is best to add related tests(如果你向我们增加一个新的功能, 请添加相关测试).
Please follow Contributor Covenant Code of Conduct.(请遵循贡献者准则).

LGTM


Fix unused return value

The crash happens when the function returns a value which is ignored on Redscript side.
For example, having defined this method:
public native class AudioSystemExt {
	public final native func RegisterEmitter(...) -> Bool;
}

Would work fine when used this way:
let registered = system.RegisterEmitter(...);

But would cause a crash when used this way:
system.RegisterEmitter(...);

lgtm!


Access to the AST for macro parameters

Thought I'd separate this out from https://github.com/jakemac53/macro_prototype/issues/6#issuecomment-884332068 since it is a different request.
The basic idea is to be able to manipulate and consume the AST of expressions as macro arguments, similarly to the proposed statement macros in https://github.com/jakemac53/macro_prototype/issues/24#issuecomment-884310221, but in the declaration context.
(Copied from  https://github.com/jakemac53/macro_prototype/issues/6#issuecomment-884332068)

The advantage of AST inspection means that you can have fewer magic strings and utilize more of what the programmer has already written to ensure valid programs with raw unresolved identifiers. If you could inspect the AST of the macro arguments as well it could solve the issues with raw identifiers instead of strings in #23 (comment) as well . The nim programming language has a concept of untyped / typed / static arguments for their macros. https://nim-lang.org/docs/tut3.html. TLDR; untyped arguments just have to be parseable and are not semantically checked or resolved or expanded, and are passed to the macro as a syntax tree. Typed are similar but are resolved. Static are passed as their value.
In Dart that might look like:
class MyUntypedMacro implements ClassDeclarationMacro {
  MyUntypedMacro(UntypedExpression ast, TypedExpression typedAst, int value);
  void visitClassDeclaration(
      ClassDeclaration declaration, ClassDeclarationBuilder builder){
      // `ast` is the actual unresolved AST for (1 + 2)
      // `typedAst` is the actual resolved AST for SomeClass.someMethod
      // `value` is 10
   }
}
@MyUntypedMacro((1 + 2), SomeClass.someMethod, 10);
class MyClass {}

Where the macro system encounters an UntypedExpression or TypedExpression in the macro constructor it would hand the unresolved or resolved ast when actually constructing the macro instance, whereas the exact value 10 gets passed into the constructor since it is neither an Untyped or Typed expression. This would be limited to expressions, possibly blocks? (though in Nim that is not the case). From the user's point of view, a TypedExpression or UntypedExpression argument of a macro is essentially typed as dynamic.

As a more concrete example, I implemented basic support for such macros in my fork:
Here is a functional widget macro using this approach:
https://github.com/TimWhiting/macro_prototype/blob/main/flutter_example/lib/macros/functional_widget2.dart

Macro Implementation
extension on analyzer.ParameterElement {
  TypeReference get typeRef =>
      AnalyzerTypeReference(type.element! as analyzer.TypeDefiningElement,
          originalReference: type);
}

class FunctionalWidget2 implements ClassDeclarationMacro {
  final ResolvedAST buildMethod;

  const FunctionalWidget2(this.buildMethod);

  @override
  void visitClassDeclaration(
      ClassDeclaration declaration, ClassDeclarationBuilder builder) {
    final build = buildMethod.ast;

    if (build is! ast.FunctionExpression) {
      throw ArgumentError(
          'Build Method is not a Function Expression ${build.runtimeType}');
    }
    final positionalParams =
        build.parameters!.parameterElements.where((p) => p!.isPositional);
    if (positionalParams.isEmpty ||
        positionalParams.first!.type.getDisplayString(withNullability: true) !=
            'BuildContext') {
      throw ArgumentError(
          'FunctionalWidget functions must have a BuildContext argument as the '
          'first positional argument');
    }
    final namedParams = {
      for (var p = 0; p < build.parameters!.parameters.length; p++)
        if (!build.parameters!.parameterElements[p]!.isPositional)
          build.parameters!.parameters[p]:
              build.parameters!.parameterElements[p]
    };

    var positionalFieldParams = positionalParams.skip(1);
    var fields = <Code>[
      for (var param in positionalFieldParams)
        Declaration('final ${param!.typeRef.toCode()} ${param.name};'),
      for (var param in namedParams.values)
        Declaration('final ${param!.typeRef.toCode()} ${param.name};'),
    ];

    var constructorArgs = <Code>[
      for (var param in positionalFieldParams)
        Fragment('this.${param!.name}, '),
      Fragment('{'),
      for (var param in namedParams.keys)
        Fragment(
            '${param.isRequired ? 'required ' : ''}this.${param.identifier!.name}, '),
      Fragment('Key? key, }'),
    ];

    var widgetName = declaration.name;
    var constructor = Declaration.fromParts(
        ['const $widgetName(', ...constructorArgs, ') : super(key: key);']);

    var method = Declaration.fromParts([
      '''
@override
Widget build(BuildContext context) ${build.body.toSource()}
'''
    ]);

    builder.addToClass(Declaration.fromParts([
      ...fields,
      constructor,
      method,
    ]));
  }
}

and the application:
@FunctionalWidget2((BuildContext context,
    {String? appTitle, String? homePageTitle}) {
  return MaterialApp(
      title: appTitle ?? 'Flutter Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: MyHomePage(title: homePageTitle ?? 'Flutter Demo Home Page'));
})
class MyApp2 extends StatelessWidget {}

Ideally I'd like to get rid of the class declaration and generate that as well using an additional UntypedAst parameter, but this is currently impossible since annotations have to be attached to a declaration
@FunctionalWidget2(MyApp2, (BuildContext context,
    {String? appTitle, String? homePageTitle}) {
  return MaterialApp(
      title: appTitle ?? 'Flutter Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: MyHomePage(title: homePageTitle ?? 'Flutter Demo Home Page'));
})
// Problem no element to attach the annotation to

In order to make this work the annotation would have to be on the library or be like the proposed statement macro (omitting the @ - which would be actually really nice).
FunctionalWidget2(MyApp2, (BuildContext context,
    {String? appTitle, String? homePageTitle}) {
  return MaterialApp(
      title: appTitle ?? 'Flutter Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: MyHomePage(title: homePageTitle ?? 'Flutter Demo Home Page'));
})

In general with this you can use bare identifiers / references instead of Strings in the macro instantiation, which makes it look a ton better. It looks more like a complete program, rather than a partial hacky program that magically turns Strings into real identifiers and code.

I had some similar thoughts, and I do generally like this idea 👍 . There are some things to work out with the approach, a couple initial thoughts:

We really are trying to avoid actually exposing any AST classes. This makes macros very brittle - for example just look at how many versions the analyzer has had. We don't want the language itself to be hampered by exposing this api. Is it enough to just expose these arguments as Code classes? We have considering adding some sort of "stringly typed" api to Code to allow you to traverse it without actually getting access to the ast classes also - no details at the moment though.
I think the Resolved versions of code are likely problematic since we can't resolve implementation code until all macros are done, or at least the first two phases. Do we really need this "resolved" version, or is an unresolved ast sufficient? Possibly we only expose resolved asts in the "definition" phase?

cc @munificent

Note that also with this approach, if we give you bare "Code" objects, then you can just use those directly in your macro when building up the actual code, and we should be able to map that back to the original source location. It is a bit weird to set breakpoints in annotations to be sure, but we could make that work I think.

@goderbauer @Hixie do you think this feature would assist at all with the other cases we have discussed where we really want to generate multiple constructs from a single one (stateful widget + state class for instance)?
I would be less concerned about for instance performing some more significant mutations on a class, (splitting it into two classes), if that class was passed more clearly as a construct to a macro, as opposed to just annotated with a macro?

I have many (not all coherent or consistent) opinions here. I'm not sure they are helpful, sorry.

Not quite sure what you mean by a stringly typed api, essentially I imagine it being very similar to the AST, just making sure you can't access / modify the compiler's AST yourself.

What I mean is not a static api, think of accessing the AST through more of a map-like structure (with string keys) rather than an actual api. The usability would be poor, but we could possibly avoid breaking people as much. We haven't delved deeply into this, its a very early idea :).

Yes, the analyzer goes through lots of versions, but they reflect the changes in the language. Ideally the AST api would be separate from the analyzer.

The api would be different from the analyzer AST - my point is just that it would be every bit as brittle. Given this would be a core API shipped with the language, that is a very risky business. It ends up hampering us from doing any language feature that would be breaking for the AST api, because that would break existing macros, and that isn't a place we want to be in.

It ends up hampering us from doing any language feature that would be breaking for the AST api, because that would break existing macros, and that isn't a place we want to be in.

Isn't it already breaking for code generators? Or I guess code generators can specify a version of the analyzer which works on multiple compiler versions, whereas this would be tied to a compiler version since that is what would pass the macro the argument? Even if this is the case, it seems like users would just end up creating a package with a strongly typed API to transform the map-like structure into something they can easily navigate. Good use case for the views proposal. I guess a stringly typed api could change independently, and the package could support different ways of navigating the stringly typed api with some sort of version checking / error recovery to handle multiple versions of the compiler. So I guess I answered my question there.
It still seems like it would be just as breaking to change keys of the map-like structure as it would to change a class structure. Either way, macros might stop working, with a class structure it would be an macro compile time error, and with a map (accessing a key that changed) it would be a macro runtime error. Seems like the AST or macros need to follow language versioning unless macros only add and don't introspect. The analyzer already changes the language version when it updates the API for a newer compiler doesn't it?

Isn't it already breaking for code generators? Or I guess code generators can specify a version of the analyzer which works on multiple compiler versions, whereas this would be tied to a compiler version since that is what would pass the macro the argument?

Yes, but code generators exist outside of the sdk. In effect, it isn't the languages problem. And having analyzer constraints also does help. Macro authors could do very tight sdk constraints but it would be a large burden on those authors, while the vast majority of the time they actually would not be broken.

Even if this is the case, it seems like users would just end up creating a package with a strongly typed API to transform the map-like structure into something they can easily navigate.

Exactly, a package is a whole different ballgame. It can be separately versioned and controlled - and it isn't specified as part of the language/core libs.

It still seems like it would be just as breaking to change keys of the map-like structure as it would to change a class structure.

It won't fix all situations, to be sure. But it would eliminate the cases where something is statically breaking but not dynamically breaking. Again this isn't a fully fleshed out thing though, just something we have been thinking about.
I really don't think exposing full AST classes is on the table at all at this time. It is simply too damaging to our long term ability to modify the language. We are trying to figure out if we can give similar power (at least the important bits), without the same problems. Even if the api kind of sucks to work with.

do you think this feature would assist at all with the other cases we have discussed

Being able to pass a method to a macro would be useful for some of the use cases I looked at, see also https://github.com/jakemac53/macro_prototype/issues/23#issuecomment-883609780.
With this proposal, do the methods have to be static? Or could you also pass in an instance method of a class? There are some use cases where writing something like this would be nice and concise:
class Foo {
  @CallWhenChanged(Foo._handleBarChange)
  int bar = 10;

  // Gets called whenever a new value is assigned to bar.
  void _handleBarChange() {
   // Do whatever...
  }
}

That is an interesting thought and it would help you to be able to not use Strings to represent expressions like that. I think that probably what we would actually want is to just allow you to write any expression, and maybe allow you to define what the resulting type of that expression should be.
So for instance maybe the parameter type for that example would be Expression<void Function()>. The expressions would be probably be resolved in the scope of the class, so you would actually write @CallWhenChanged(_handleBarChange) (without the Foo). We would need to make sure you aren't then allowed to use that expression outside of the class (in a library declaration), but that is likely doable.

Note that my current prototype can't implement this - that would require a more real prototype as it would involve new language features.


Avoid spaghetti when reading multiple sheets

As @Intuos5 mentioned here.

Due to the complexity of GH's tree processing, I would propose utilizing Pancake's Assoc feature.

read multiple sheets.zip
In previous releases of PancakeSpreadsheet, the plugin failed to display Assoc-related components if PancakeSpreadsheet was loaded before Pancake. This is fixed in the latest commit.

Didn't know this function yet, this will come in handy with the attributes I set!
Do you have a solution for the situation where I label the street type in the first cell (B3 in this case)?

I think I could go about renaming them to be Type so it doesn't overpopulate the deconstruct assoc. But then I lose out on the street type name in the grasshopper list that I am also using. 🤔

Is there a way for you to sequentially read each spreadsheet in a Read XLS component and construct a list like this:
{Spreadsheet number; Column/ row number}?

I do like the Assoc, but in this particular case, I prefer to have the headers of the tables as first item of each branch. That way, I can easily verify the data by looking at the first item and for operating on the data, I cull it in advance.

Another option for me would be to leave the headers in the Assoc, perhaps as a component option. Is that something you might be able to add?
I would then have to read the spreadsheet name and add replace the first item in the branch {*;0} to get the result I am looking for. That would also be a good solution for me.

@Intuos5 I created a new component. Plz try it and see if it satisfies.

PancakeSpreadsheet.zip

Awsome! I recently had to bugfix my script because I went over the 25 sheets I had wired up, so I added some more. But this works exactly the way I hoped it would. I have replaced the spaghetti components with the new component and the result of the most complex script is still the same. Thank you very much! This is gonna be insanely helpful for my work.
Will you add this to the package manager version of Pancake Spreadsheet too in time?

@karakasa Awsome! I recently had to bugfix my old script because I went over the 25 sheets I had wired up, but I no longer have to check that now. This works exactly the way I hoped it would. I have replaced the spaghetti components with the new component and the result of the most complex script is still the same. Thank you very much! This is gonna be insanely helpful for my work. Will you add this to the package manager version of Pancake Spreadsheet too in time?

yes. after I make a new icon for that component and determine its name.

Done.


WIP: Add Nuget build targets

I've added two new build targets to publish the server and client packages on Nuget (I've also added the .Server suffix to the server project). At the moment, the targets only work if run specifically (e.g. build PublishServer) on a machine where Bazinga's Nuget API key has been set.
Things to consider:

[ ] Make the publish automatic on the release server?
[ ] Add the Shared project as an independent package?
[ ] Remove the old nuget targets? (Nuget, PublishNuget)

Note: A beta version of both packages has already been released to Nuget.

I added a new target to publish the Type Provider version for Fable through npm.


add Google Cloud setup instructions

Awesome, thank you! If you'd like me to send you a couple Juice Shop stickers and postcards, please feel free to send me an email or Gitter-PM with your address!


[-] Received unknown error: (constraintViolation) 0000200B: AtrErr: DSID-033E0F1

Hello my brother, when using your tool to create a machine user remotely, an unknown error is reported
Certipy v3.0.0 - by Oliver Lyak (ly4k)
[+] Trying to resolve 'DC01.test.local' at '<IP_ADDRESS>'
[+] Authenticating to LDAP server
[+] Bound to ldaps://<IP_ADDRESS>:636 - ssl
[+] Default path: DC=test,DC=local
[+] Configuration path: CN=Configuration,DC=test,DC=local
[*] Creating new account:
sAMAccountName                      : JACK-PC1$
unicodePwd                          : LUCFG2bR2ARmhtkE
userAccountControl                  : 4096
servicePrincipalName                : HOST/JACK-PC1
RestrictedKrbHost/JACK-PC1
dnsHostName                         : 'DC01.test.local'
[-] Received unknown error: (constraintViolation) 0000200B: AtrErr: DSID-033E0F1
B, #1:
0: 0000200B: DSID-033E0F1B, problem 1005 (CONSTRAINT_ATT_TYPE), data 0,
Att 9026b (dNSHostName)

i also have thins problem, test on windows server 2012, 2016, 2019

windows下去掉单引号，或者改成双引号。

Hello
This means the domain controller has been patched and you can therefore not create a machine account with the same dns host name as the domain controller


Capitalised versions of true and false values

Please can I have this change? It's very convenient to have these extra values.
I can refactor, as the values are just repeating and being transformed in some way, but I would need guidance. The codebase is a bit beyond me.

Yes you can, thank you <3

Thank you!


MacOS published releases are broken

Bug type
Other
Component name
No response
What happened?
Published applications using dotnet publish on MacOS (or, specifically M1 systems) are broken.

Expected behavior
The page should render as normal.
Reproduction link
This is reproducable on the default template
Reproduction steps

Run dotnet new mudblazor --host server --name MudblazorTest
Run dotnet publish -c Release on MacOS (tested on M1 Pro)
Open the created executable
View the logs in the console, view broken render, etc.

Relevant log output
/Users/stage/Documents/github/MudblazorTest/bin/Release/net6.0/publish/MudblazorTest; exit
info: Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager[62]
      User profile is available. Using '/Users/stage/.aspnet/DataProtection-Keys' as key repository; keys will not be encrypted at rest.
info: Microsoft.Hosting.Lifetime[14]
      Now listening on: http://localhost:5000
info: Microsoft.Hosting.Lifetime[14]
      Now listening on: https://localhost:5001
info: Microsoft.Hosting.Lifetime[0]
      Application started.   Request path: '/_framework/blazor.server.js'.
      Microsoft.JSInterop.JSException: Could not find 'mudElementRef.getBoundingClientRect' ('mudElementRef' was undefined).
      findFunction/<@https://localhost:5001/_framework/blazor.server.js:1:497
      findFunction@https://localhost:5001/_framework/blazor.server.js:1:465
      E@https://localhost:5001/_framework/blazor.server.js:1:2606
      beginInvokeJSFromDotNet/s<@https://localhost:5001/_framework/blazor.server.js:1:3494
      beginInvokeJSFromDotNet@https://localhost:5001/_framework/blazor.server.js:1:3475
      _invokeClientMethod/<@https://localhost:5001/_framework/blazor.server.js:1:72061
      _invokeClientMethod@https://localhost:5001/_framework/blazor.server.js:1:72047
      _processIncomingData@https://localhost:5001/_framework/blazor.server.js:1:70089
      kt/this.connection.onreceive@https://localhost:5001/_framework/blazor.server.js:1:64492
      connect/</o.onmessage@https://localhost:5001/_framework/blazor.server.js:1:48826

         at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
         at MudBlazor.MudDrawer.UpdateHeight()
         at MudBlazor.MudDrawer.OnAfterRenderAsync(Boolean firstRender)
         at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle, ComponentState owningComponentState)
info: Microsoft.AspNetCore.Routing.EndpointMiddleware[1]
      Executed endpoint '/_blazor'
info: Microsoft.AspNetCore.Hosting.Diagnostics[2]
      Request finished HTTP/1.1 GET https://localhost:5001/_blazor?id=UEjXONNqroieXA1SOAyj0Q - - - 101 - - 84.6108ms

Version (bug)
6.2.2
Version (working)
No response
What browsers are you seeing the problem on?
Firefox, Safari
On what operating system are you experiencing the issue?
Mac OSX
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Hello.
From your asp.net core log I can see
GET https://localhost:5001/_content/MudBlazor/MudBlazor.min.js - - - 404
GET https://localhost:5001/_content/MudBlazor/MudBlazor.min.css - - - 404

and below
Could not find 'mudElementRef.getBoundingClientRect' ('mudElementRef' was undefined)

Which indicates that the JS and CSS were no loaded which is not related with MudBlazor for now. Therefore it looks like an problem with aspnetcore.

Hi, thank you for the response. I was wondering about this too - but I really am not sure why this would happen. This is a brand new project after all, I have not configured anything outside of the default template. I would be happy to try any ideas on my personal system.

Have you checked if the files can be accessed in browser via this urls https://localhost:5001/_content/MudBlazor/MudBlazor.min.js and https://localhost:5001/_content/MudBlazor/MudBlazor.min.css?
have you also checked this issues:
https://github.com/dotnet/aspnetcore/issues/45249
https://github.com/dotnet/aspnetcore/issues/42081
https://github.com/dotnet/aspnetcore/issues/38212

Thank you for these suggestions. I found that this specific comment fixed my issue. Going to test on Windows now, but this fixed the MacOS issue I was having.
https://github.com/dotnet/aspnetcore/issues/38212#issuecomment-1094357982


[release-4.15] HOSTEDCP-1513: Support annotation scoping for hostedcluster resources

This is a manual cherry-pick of https://github.com/openshift/hypershift/pull/3702 to the release-4.15 branch.

/retest

/retest

/retest

/retest

/retest


[Feature] [Docs] add OIDC documentation for v 24.0

Use case
updated to 24.0beta1 all fine.
should we change in config for OIDC ?
scope: ["openid", "profile", "email"]
to add preferred_username,name,picture ? using Authentik.
We should update the documentation for this.
Description
Update documentation for OIDC in v 24.0
Contribution

[ ] I can write the design doc for this feature
[ ] I can contribute this feature

How can it be implemented?
Update documentation I can give my example of current using Authentik

I plan to update the current OIDC documentation and provide more structure. It should probably be structured like this:

Short introduction, possibly with a list of known limitations
Generic setup instructions: the OIDC configuration of Headscale should be described once with a description of required parameters (issuer, client_id, client_secret or client_secret_path) and some useful optional parameters such as scope.
IdP specific instructions where we could rehash parts of the existing configuration.

Anything missing that should be covered?
@masterwishx it'd be great if you could share your Authentik configuration, thx.


[BUG] FigureWidgetResampler, pandas time-index series.

There is a bug with the x-axis view when passing a non pd.(datetime)Index (i.e, a pd.Series) as hf_x (and likely x -> to test). An int-view of the x is shown as x-axis.
Related: #208
import pandas as pd; import plotly.graph_objs as go; import numpy as np
from plotly_resampler import FigureWidgetResampler

# Generate data
count = 1_000_000
df = pd.DataFrame({"dt": pd.date_range(start="2021-01-01", periods=count, freq="S")})
df["value"] = np.random.randn(df.shape[0])

# Creat the figure
fig = FigureWidgetResampler(go.Figure())
# Setting the `hf_x` data via a `pd.Index` works
# fig.add_trace(go.Scatter(mode="lines"), hf_x=pd.Index(df["dt"]), hf_y=df["value"])
# Setting the `hf_x` data via a `pd.Series` creates an x-axis int0view
fig.add_trace(go.Scatter(mode="lines"), hf_x=df["dt"], hf_y=df["value"])
fig

Adding the trace its hf_x data via a pd.Series seems to create an int view (see 📷 1) whereas adding the hf_x via a pd.Index produces a data-x-axis (see 📷  2)

📷 1:

📷  2:

TODO:

[ ] verify this behavior for the FigureResampler
[ ] Create (failing) tests for this
[ ] Fix the bug

It appears adjusting this line in figure_resampler_interface class fixes this bug.
Noteworthy, this bug only seams to appear for non tz-localized series.


Clean up serialization

I have added some notes about our existing serialization protocols at https://github.com/ioam/param/wiki/Serialization.
We still need to note shortcomings (e.g. things that are problematic to serialize), and I think we should clean up the existing serialization a bit. E.g. while there could be stuff for serializing an object that's common between pickle and script_repr we don't provide any standard way to do that.

Note: see https://github.com/ioam/param/issues/166#issuecomment-325225105

There's been progress on serialization, and remaining work is covered in other open issues and PRs.


Added a config option to reduce the number of full_node worker processes to reduce ram usage

There is a bug on github that prevents me from creating an actual pull request to your fork: https://stackoverflow.com/questions/6675994/is-it-possible-to-fork-a-fork-in-github
Can you please merge this code change?
https://github.com/pinksheetscrypto/covid-blockchain/pull/21

@grayfallstown hey this has been merged to the main branch. Again, thanks for the suggestion!


how can i know track objects is who

when i use traking model  to track objects, how can i know who is who，thanks

ImageAI currently doesn't provide objects or person tracking. You might want to use methods like Kalman filter with the data generated by ImageAI for this purpose. See the link below on examples for Kalman filter.
https://towardsdatascience.com/kalman-filters-a-step-by-step-implementation-guide-in-python-91e7e123b968

thanks


Popup for Alert Name Input on Logs Page

Description
On the logs page by add a popup that appears when the user clicks on the bell icon. The popup should include an input box allowing the user to enter the alert name.

Popup for Alert Name Input on Logs Page


Fix unintended aborts in dialer

Fix a bug where DialRequest aborts wrong dial attempts.

Thanks for opening this - can you please add a test to catch any regressions that may occur in the future?

Thanks for opening this - can you please add a test to catch any regressions that may occur in the future?

Currently busy with a different issue, but will create or adapt a unit test tackling this bug soon.

I put a simple test together, you can add it to test/dialing/dial-request.spec.ts:
it('should abort other dials when one succeeds', async () => {
  const connection = mockConnection(mockMultiaddrConnection(mockDuplex(), await createEd25519PeerId()))
  const actions: Record<string, () => Promise<any>> = {
    '/ip4/<IP_ADDRESS>/tcp/1231': async () => {
      await delay(10000)
    },
    '/ip4/<IP_ADDRESS>/tcp/1232': async () => {
      await delay(10)

      return connection
    },
    '/ip4/<IP_ADDRESS>/tcp/1233': async () => {
      await delay(10000)
    }
  }

  const signals: Record<string, AbortSignal | undefined> = {}
  const dialRequest = new DialRequest({
    addrs: Object.keys(actions).map(str => new Multiaddr(str)),
    dialer: new DefaultDialer(new Components(), {
      maxParallelDials: 3
    }),
    dialAction: async (ma, opts) => {
      signals[ma.toString()] = opts.signal
      return await actions[ma.toString()]()
    }
  })

  await expect(dialRequest.run()).to.eventually.equal(connection)

  expect(signals['/ip4/<IP_ADDRESS>/tcp/1231']).to.have.property('aborted', true)
  expect(signals['/ip4/<IP_ADDRESS>/tcp/1232']).to.have.property('aborted', false)
  expect(signals['/ip4/<IP_ADDRESS>/tcp/1233']).to.have.property('aborted', true)
})

The thing is it passes against this PR and against master so it might not be exposing the issue you are fixing here.
Could you adapt it so it fails against master and passes against this PR?

@robertkiel: can you add a test that shows what you're fixing?  It's currently not clear what is being fixed.  If there is a problem, we do want to get this merged.

@robertkiel: can you add a test that shows what you're fixing? It's currently not clear what is being fixed. If there is a problem, we do want to get this merged.

Thanks a lot for coming back to it. Currently working on the unit test and will give an update once I'm done.

Thanks!  You may need to merge master again to get the build passing.

@robertkiel Could you please fix CI errors?

Omw to fix them. Looks like there are some issues with the @libp2p/interfaces package, so I expected them to be fixed independently.


Build a Product Landing Page -- YouTube Video Playback Error

Challenge Name
Applied Responsive Web Design Projects: Build a Product Landing Page
Challenge Link
http://beta.freecodecamp.com/en/challenges/applied-responsive-web-design-projects/build-a-product-landing-page
Issue Description
Embedded YouTube Video fails to load and instead displays error when clicked on, even when the browser is refreshed. See screenshot below.
Browser Information

Browser Name, Version:  Google Chrome, Version 56.0.2924.87 (64-bit)
Operating System: Windows 10 Pro 64Bit
Mobile, Desktop, or Tablet: Desktop

Screenshot

Closing as duplicate https://github.com/freeCodeCamp/freeCodeCamp/issues/12753. Feel free to continue the discussion over there. Happy coding!


that's not help that's just saying what it is!

Just give a step by step guide how to resolve this!

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

ID: 082295f3-9acb-6b3f-cdb4-425d9a249df0
Version Independent ID: ce3c3863-2426-420e-4387-9d73cc949437
Content: MSB3270: There was a mismatch between the processor architecture of the project being built 'architecture' and the processor architecture of the reference 'reference', 'architecture'. - MSBuild
Content Source: docs/msbuild/errors/msb3270.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@Elantonio
I wonder how this could be more helpful. The solution would be different for different projects, so I'm not sure there is a step-by-step way to resolve this. It depends on the intended output architecture, and the availability of architecture-specific builds of the referenced assemblies. It could be that there's no available version of a referenced assembly for the target architecture (provided for by a third-party vendor, for example), thus making it impossible to build the intended target for that architecture.


BDC dashboard shows spinning circles when connecting to non-existant controller...

Issue Type: Bug

Setup bdc
Connect to controller endpoint
Delete bdc
Select the same endpoint from #2 and click Manage

You will get error below. However the dashboard is left with spinning circles & ADS keeps spinning in the task manager. Dashboard should quit gracefully and not consume CPU.
Error retrieving BDC status from https://pl-ubuntu-m-0:30180 - connect ECONNREFUSED <IP_ADDRESS>:30180
Azure Data Studio version: azuredatastudio-insiders 1.13.0-insider (6851b2091fbc209250edda48087ea2a32a2c7e43, 2019-10-14T21:04:50.772Z)
OS version: Windows_NT x64 10.0.18999

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz (8 x 2904)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.85GB (11.67GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

ssmskeymap
kev
1.1.0

admin-tool-ext-win
Mic
0.0.2

agent
Mic
0.43.0

azuredatastudio-postgresql
Mic
0.2.2

dacpac
Mic
0.7.0

import
Mic
0.11.0

profiler
Mic
0.10.0

sql-vnext
Mic
0.16.0

![ads](https://user-images.githubusercontent.com/10999577/66798880-2ef01f00-eec4-11e9-8b54-b07b53250ac7.png)

I'll take a look at the behavior when we can't connect to a cluster - that's definitely not ideal behavior.
One question though - what do you mean ADS is "spinning in the background"? It shouldn't be doing anything while it's waiting for the status request to come back so if it's consuming extra CPU cycles that'd be very unexpected behavior.

https://github.com/microsoft/azuredatastudio/pull/8103


Method not found

I'm trying to implement Embedded Pages, but the application show to me this error.

I have followed the guide and i don't know how to fix this issue

Fixed. Your library version isn't updated compared to Discord.Net.Labs.
To fix this issue, downgrade Discord.Net.Labs to 3.4.2


Can't import the named export 'AnimationMode' from non EcmaScript module

Hi!
I'm trying to run React demo and faced an issue:
<EMAIL_ADDRESS>Can't import the named export 'AnimationMode' from non EcmaScript module (only default export is available)

Would appreciate any help.

This means that your bundler does not handle CJS-ESM interop well. The ESM file window-manager/dist/index.mjs imports a name from white-web-sdk's CJS code and your bundler does not know how to deal with it.
This often happens in webpack's default behavior that it uses a stricter rule (the Node.js 14 behavior) for .mjs files where they can only import the default name from CJS sources. One possible way to configure that is adding something like this:
rules: [
  {
    test: /\.mjs$/,
    type: 'javascript/auto',
  },
],

..but that's not always the case e.g. if you're using the babel loader to transform all ESM to CJS before webpack could parse it then the rule above would have no chance to be applied.
In general, I would suggest you to use a more modern bundler that could handle these things like esbuild or vite, etc.

Thanks, it makes sense.

@hyrious as a workaround, do you have fastboard bundled as a single file like commonjs module?

You can create one easily with esbuild:
mkdir test && cd test
npm add -D esbuild @netess/fastboard jspdf
echo 'module.exports = require("@netless/fastboard")' > main.js
npx esbuild main.js --bundle --format=cjs --outfile=bundle.js

@hyrious looks like commonjs doesn't fit my needs, could you help me with building the UMD module?

You can use rollup/webpack to create the UMD wrapper on top of the output of esbuild. On the other hand, if you only want a file that can be put in the <script> bundle, you can let esbuild output the iife format with a global name:
npx esbuild main.js --bundle --format=iife --outfile=bundle.js --global-name=fastboard


Auto-set DataLoader.worker_init_fn with seed_everything

What does this PR do?
Proposal:
Make Lightning add a default worker init function for seeding numpy when the user also calls seed_everyting.
This prevents nasty bugs where data augmentations are the same across workers (when using e.g. numpy and not torch as random number generator).
As suggested by Ethan https://github.com/PyTorchLightning/pytorch-lightning/discussions/6957#discussioncomment-596478 , this feature can be turned on or off with seed_everything(workers=True|False)
Default is currently on (False). It doesn't affect parity, and is therefore a safe feature to introduce.
However, the user has to remember to turn this on, something that likely gets forgotten even if we offer this convenient feature. We may want to consider turning it on in the future.
Vote:

🚀 Default seed_everything(workers=True)
🎉 Default seed_everything(workers=False)

Addresses: #6957
Explanation
To understand the seeds and randomness in the workers, consider the following example:
import os
import torch
from torch.utils.data import Dataset

from pytorch_lightning import LightningModule, Trainer, seed_everything

import numpy as np
from torch.utils.data import Dataset

class RandomDataset(Dataset):
    def __getitem__(self, index):
        # numpy random state is the same in all workers!
        return np.random.randint(0, 10, 3)

    def __len__(self):
        return 16

class BoringModel(LightningModule):

    def __init__(self):
        super().__init__()
        self.layer = torch.nn.Linear(32, 2)

    def training_step(self, batch, batch_idx):
        print("rank:", self.global_rank, "batch:", batch.view(-1).tolist())
        return None

    def configure_optimizers(self):
        return torch.optim.SGD(self.layer.parameters(), lr=0.1)

def run():
    train_data = torch.utils.data.DataLoader(RandomDataset(), batch_size=2, num_workers=2)
    model = BoringModel()
    trainer = Trainer(
        default_root_dir=os.getcwd(),
        gpus=2,
        accelerator="ddp",
        limit_train_batches=4,
        max_epochs=2,
        weights_summary=None,
        progress_bar_refresh_rate=0,
    )
    trainer.fit(model, train_data)

if __name__ == '__main__':
    # ---------------------------------------------------
    # comment the line below to get duplicated batches!
    # ---------------------------------------------------
    seed_everything(1)
    run()

On the master branch, running the above code we get an output like this:
rank: 0 batch: [5, 4, 5, 4, 9, 9]
rank: 0 batch: [5, 4, 5, 4, 9, 9] <---- repeated
rank: 0 batch: [7, 5, 2, 2, 6, 5]
rank: 0 batch: [7, 5, 2, 2, 6, 5] <---- repeated

rank: 1 batch: [8, 9, 4, 6, 8, 8]
rank: 1 batch: [8, 9, 4, 6, 8, 8] <---- repeated
rank: 1 batch: [0, 0, 8, 6, 9, 7]
rank: 1 batch: [0, 0, 8, 6, 9, 7] <---- repeated

As you can see, the batches within the rank are repeated, meaning both workers produce the same batch/augmentation!
As suggested in the PyTorch docs and this blog post, we can fix this by providing custom worker function:
def worker_init_fn(worker_id):
    worker_seed = (torch.initial_seed() + worker_id) % (2**32)
    torch.manual_seed(worker_seed)
    numpy.random.seed(worker_seed)
    random.seed(worker_seed)

dataloader.worker_init_fn = worker_init_fn

This fixes the problem for training on a single gpu. See below the output, in each rank we get different samples. Good!
rank: 0 batch: [2, 2, 3, 0, 8, 9]
rank: 0 batch: [7, 2, 0, 0, 2, 2]
rank: 0 batch: [9, 6, 5, 0, 3, 1]

rank: 1 batch: [2, 2, 3, 0, 8, 9] <---- repeated from rank 0
rank: 1 batch: [7, 2, 0, 0, 2, 2] <---- repeated from rank 0
rank: 1 batch: [9, 6, 5, 0, 3, 1] <---- repeated from rank 0

However we still have problem with multi-gpu!! We need to add the global rank to the seed as well! The (hopefully) correct init is:
def worker_init_fn(worker_id):
    global_rank = rank_zero_only.rank
    worker_seed = (torch.initial_seed() + worker_id + global_rank) % (2**32)
    torch.manual_seed(worker_seed)
    numpy.random.seed(worker_seed)
    random.seed(worker_seed)

dataloader.worker_init_fn = worker_init_fn

And our script produces random samples without duplicates across all ranks and workers:
rank: 0 batch: [2, 2, 3, 0, 8, 9]
rank: 0 batch: [7, 2, 0, 0, 2, 2]
rank: 0 batch: [9, 6, 5, 0, 3, 1]
rank: 0 batch: [0, 8, 2, 6, 3, 2]

rank: 1 batch: [1, 6, 2, 5, 5, 7]
rank: 1 batch: [9, 0, 4, 8, 1, 4]
rank: 1 batch: [7, 2, 2, 1, 6, 6]
rank: 1 batch: [5, 3, 6, 5, 6, 2]

Note: To reproduce this you need to run on linux, where you will see the duplicated samples. On the other platfors (darwin, win) you won't get duplicates, but the seeds in the worker processes are not deterministically chosen.
References:
[1] https://tanelp.github.io/posts/a-bug-that-plagues-thousands-of-open-source-ml-projects/
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Hello @awaelchli! Thanks for opening this PR.

In the file pytorch_lightning/utilities/seed.py:

Line 83:29: W292 no newline at end of file

Do see the Hitchhiker's guide to code style

Hi everyone!
Please consider the suggestions I have made on the related PyTorch issue concerning the construction of these worker init functions. Collapsing these seed values down to just 32-bits is risky. PyTorch's PRNG uses 64-bit seeds, so I highly recommend keeping at least that size.
Reusing the same seed for both random and np.random is problematic because they both use the same algorithm.
[~]
|47> random.seed(0xdeadbeef_bad0cafe)

[~]
|48> random.getstate()[1][0:10]
(2147483648,
 151973328,
<PHONE_NUMBER>,
<PHONE_NUMBER>,
<PHONE_NUMBER>,
<PHONE_NUMBER>,
<PHONE_NUMBER>,
 880682663,
<PHONE_NUMBER>,
<PHONE_NUMBER>)

[~]
|49> np.random.seed([0xbad0cafe, 0xdeadbeef])

[~]
|50> np.random.get_state()[1][0:10]
array([2147483648,  151973328,<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>,  880682663,<PHONE_NUMBER>,<PHONE_NUMBER>], dtype=uint32)

DataLoader already reseeds the torch RNG and the stdlib random with a base_seed + worker_id scheme. I'm not familiar enough with Lightning to know if pl_worker_init_function is being run after that happens or if you are replacing that logic entirely. If the former, then once inside the worker process, torch.initial_seed() will already return base_seed + worker_id. There's no need to add it again. It seems to me that adding in the rank as well might risk collisions. Using SeedSequence and concatenating the base_seed, worker_id, and rank would help avoid that, as well as avoid collisions with the stdlib random state.
Thank you everyone for your attention.

@rkern Thanks a lot for bringing this to my attention. I tried to incorporate your suggestions for the bumpy state sequence.
Regarding torch.manual_seed: The worker_id is only unique to the current process, but in distribute training we have several of these processes (could be across multiple servers) and we need to make the derived seed unique.
First I had torch.manual_seed(base_seed + worker_id + rank) but as you pointed out, there could be a collision. For example, when worker_id=0, rank=1 would give the same derived seed as worker_id=1, rank=0. I mitigate this issue by multiplying the worker_id with the base seed. Here is the current implementation:
def pl_worker_init_function(worker_id: int) -> None:
    global_rank = rank_zero_only.rank
    process_seed = torch.initial_seed()
    # back out the base seed so we can use all the bits
    base_seed = process_seed - worker_id
    ss = np.random.SeedSequence([base_seed, worker_id, global_rank])
    # use 128 bits (4 x 32-bit words)
    np.random.seed(ss.generate_state(4))
    torch.manual_seed((process_seed * global_rank) % (2**64))

You can spawn derived SeedSequences from the one you're making for numpy in order to get high-quality, deterministic, but distinct seeds for torch and the stdlib random too. Ad hoc methods for deriving seeds can have bad interactions with some PRNG algorithms. For example, consider when global_rank is a power of 2. Your seeds will just be bit-shifted versions of others in the workflow. For some PRNGs, that's not going to matter (e.g. Philox), but for others, it might. I'm getting lost in all of the C++ and CUDA, but I think the default PyTorch PRNG is one of the 64-bit MTGP Mersenne Twisters. Looking at the relevant part of its seeding function, I think that bit-shifted seeds will result in merely-bit-shifted states, which isn't great.
def pl_worker_init_function(worker_id: int) -> None:
    global_rank = rank_zero_only.rank
    process_seed = torch.initial_seed()
    # back out the base seed so we can use all the bits
    base_seed = process_seed - worker_id
    ss = np.random.SeedSequence([base_seed, worker_id, global_rank])
    # use 128 bits (4 x 32-bit words)
    np.random.seed(ss.generate_state(4))
    # Spawn distinct SeedSequences for the PyTorch PRNG and the stdlib random module
    torch_ss, stdlib_ss = ss.spawn(2)
    # PyTorch takes a 64-bit seed
    torch.manual_seed(torch_ss.generate_state(1, dtype=np.uint64)[0])
    # use 128 bits expressed as an integer
    stdlib_seed = (stdlib_ss.generate_state(2, dtype=np.uint64).astype(object) * [1<<64, 1]).sum()
    random.seed(stdlib_seed)

Let's not miss this before merging. We don't want this to go unnoticed

Can you try and give this functionality more visibility throughout docs? Maybe also setting it where seed_everything is used

@rkern thanks again for your feedback. For torch < 1.6 manual_seed only accepts 32-bit integers so I had to do a slight modification. I also noticed that on macos and win our custom worker init function for numpy is not necessary. There seems to be no duplication of random state there. I couldn't find a clear answer why but I think the reason is that on these platforms the worker processes get created with a different method.

multiprocessing uses spawn instead of fork on Windows (because there is no fork()) and on macOS in recent Pythons (I think because fork() is a little wonky when interacting with other macOS facilities). This means that instead of copying the state of the process, a new Python process is started up from scratch. That means that the global PRNG underneath np.random gets initialized with fresh OS entropy in each child process. Thus, you will get distinct states, but not deterministic states. You still need the worker_init_fn to give them appropriate seeds if you want determinism.

@rkern One light bulb after the other turns on. You are of course right, I should have seen this in my testing. I was too much focused on avoiding duplicate states that I was completely blind to see the non-deterministic state. Thanks a lot for your comment. Appreciate your contribution!

Thanks everyone for your constructive feedback. It was fun to implement this :)

It was decided that we have to keep it False to keep parity between versions (it could influence reproducibility from one version to the next).


Files still cached on a local environment

Hi everyone,
All of my assets files continue to be cached on a local environment with a good configuration.
'cache' => array('production'),

My environment is correctly set to 'local', so the detection on Laravel is not the problem.

+1


docs: update nuxt.

Add @wechatferry/nuxt package install.

Oh, it's not necessary. Under normal circumstances, pnpx automatically handles the installation of dependencies, so we don't need to install packages separately. It's my problem.


Fix for node 7

This doesn't pass install correctly on node 7 on Travis.
@nexdrew has pointed out that it is likely due to https://github.com/abbr/deasync/issues/73
We should put in a PR for node-etcd to update with a new deasync (and then get tests passing)

PR in https://github.com/stianeikeland/node-etcd/pull/67

We don't use sync etcd queries. I'll move us over to etcdjs (npm/etcetera#153).

This has been long fixed \o/


Package for PyPI

This would be a fun thing to have in PyPI.  Any interest in a patch to get it setup there?

Yes, why not ;-)
What is needed from my side, any account?

Hey. Indeed PyPI package would be very cool. @nitram509 You need a setup.py and Pypi account to create a package. Additionally you need to decide what would you like to call this project.
I can manage the rest. :-)


Add Universidad Nacional Hermilio Valdizán

@nothyG Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


Rule for adjusting the power of Strength, Speed & Weakness statmodifiers.

Strength, Speed, MarkOfAvalon(weakness), ReplenishArmor and ReplenishBarkArmor all have an AdditiveBonus which can be used to adjust the effect of the respective potions/speels. This can be used to make strength potions give you +3, Weakness be less crippling etc.
It would be nice to have a rule to adjust these settings.

Resolved by #198


PERL-937 Gossip cluster time during SDAM

This commit also moves cluster_time into _Topology and leaves
MongoClient as a facade.  This keeps cluster state in _Topology and
avoids a circular reference.

Coverage decreased (-0.007%) to 76.991% when pulling 83935571cb4e603069fed3d53043196d62626b6b on xdg:PERL-937-sdam-gossip-clustertime into 553b69c07bc09060f248715dec7584d7c32b7fc2 on mongodb:master.


Add GPU memory check for CUDA code

This PR adds compute-sanitizer to check the GPU memory leak for all the CUDA tests.
Note that the compute-sanitizer looks for CUDA APIs in a code and will issue an error when applied to a CPU test. This error could not be suppressed unfortunately (https://docs.nvidia.com/compute-sanitizer/ComputeSanitizer/index.html#error-suppression). Therefore, we add new internal options to apply the compute-sanitizer check to the CUDA tests only.
All the 50 tests (including GPU memory check) are passed on Derecho's A100 GPU, using the nvhpc/23.7.
fix #415

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

Project coverage is 92.25%. Comparing base (555be4e) to head (e13eeb0).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #544   +/-   ##
=======================================
  Coverage   92.25%   92.25%
=======================================
  Files          45       45
  Lines        3407     3407
=======================================
  Hits         3143     3143
  Misses        264      264

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Porting to KituraNIO [WIP]

Description

Motivation and Context

How Has This Been Tested?

Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

#83 Does the switch to Kitura-NIO when the environmental variable KITURA_NIO is set.
Closing this PR.


When retrying the download - reset the file pointer

We need to seek back to 0,0 on the file that we open here:
https://github.com/pivotal-cf/pivnet-resource/blob/master/downloader/downloader.go#L58
when we retry a download. Not seeking back will lead to you continuing to write to the open file descriptor at the point you left off at.
You are going to need this:
https://golang.org/pkg/os/#File.Seek
before the continue here:
https://github.com/pivotal-cf/pivnet-resource/blob/master/downloader/downloader.go#L107

fixed by https://github.com/pivotal-cf/pivnet-resource/commit/060707061630bca3610efef3f05c7c0a2b8bde88


Fix azure may hide some errors

Q
A

🐛 Bug fix?
yes

🚀 New feature?
no

⚠ Deprecations?
no

❌ BC Break
no

🔗 Related issues
none

❓ Documentation
no

Description
When an error occur while retrieving result, we were hiding it ...

You did find this based on the failed Acc tests ?

Yeah, I'm working on another pr to fix ACC test : the azure authentication was broken ...


Fix issue with overriding configuration parameters

Let's assume that we have the following code:
const rabbot = require('rabbot')
rabbot.addConnection({
  uri: 'amqp://localhost',
  user: 'not-guest',
  pass: 'super-secret'
}

Without this fix the following happens:

uri is parsed
uriOpts contains a null value for user
a null value for user from uriOpts is applied over valid user value from configuration via Object.assign
getOption is called to assign value for this.user
and this.user will always be guest, because parameters.user is null at this time

Proposed PR filter out all nullable properties from uriOpts before applying it over parameters, which prevents overriding valid properties from configuration.

Coverage decreased (-0.08%) to 88.873% when pulling 827fbf0b79b67ef7f4d2d0d06ea3cd9662ece3c0 on rpieciorak:patch-2 into 80b63c08bdb32d04fd36502f24180e0c8e8f69ee on arobson:master.

Coverage decreased (-0.08%) to 88.873% when pulling 827fbf0b79b67ef7f4d2d0d06ea3cd9662ece3c0 on rpieciorak:patch-2 into 80b63c08bdb32d04fd36502f24180e0c8e8f69ee on arobson:master.


Should min_hash and min_hash_spark return the same result?

Hi, thank you for your great work!
As title, I wonder if these two codes will return the same result or not.
I tested these two codes with default configuration on the same dataset (Oscar 2201 - gl) but the output datasets from them are different. The output datasets have the same number of rows after removing duplicates but the content is different.
And the number of rows that I get after deduplicated is different from your example. But maybe it shouldn't be a problem because I tried the code on several different machines and it all outputs the same number of rows.

Great question!
Unfortunately, there is some randomness I cannot control, especially for spark:
df = df.withColumn("__id__", F.monotonically_increasing_id()).cache()

The above line in spark version assigns ids randomly (e.g. [1, 2, 50, 51, 53, 54]), although they are still monotonically increasing. This means duplicates within a cluster will be removed randomly as a result. But the number of removed duplicate documents should be the same from the same algorithm.
If your dataset has an index column, you can modify the code to use that index, in which case, it should give you the same results.

Thank you so much for the suggestion.
Actually I realized that I was wrong. When I use your original code, both minhash andminhash_spark give the same result. Previously I didn't sort the results of minhash_spark by id so I thought that these two codes would give different results.
Maybe with other larger datasets or a different spark cluster, the results will be different because as you mentioned, the monotonically_increasing_id  function is non-deterministic. I just want to correct my mistake so others may not be misunderstood.
I also tried changing to use the id column of the dataset as you suggested, and the results of the two codes are the same.
Once again, thank you so much for your great work!


Fails to run in Ubuntu 22.04 LTS

Hello @paulormart !!
Due to Ubuntu 22.04 LTS migrating from OpenSSL 1.1 to 3.0, when trying to run the current release in those systems, it returns:

crunch: error while loading shared libraries: libssl.so.1.1: cannot open shared object file: No such file or directory

Greetings!

Hello Paulo!
Building from source in an Ubuntu 22.04 solves this issue.. so it is not a real problem of your code or repository, but of the releases targeting specific environments.
If you agree, I will close this issue now.. Thanks a lot! :-D