Louvain sources unavailable

Would it be possible to include the sources of the Louvain subdirectory? Also, what is the license of the modified Louvain sources?  Will they be released under a free software license like the rest of your code?
As it stands your software cannot be built from source, so it cannot be packaged for GNU Guix, which we use at our institute.

Ping :)
Are the sources for Louvain available?  PhenoGraph cannot be packaged as long as it bundles opaque binaries.

The project's readme mentions Louvain is taken from here.
That page in turn mentions that further development is done on sourceforge
There the commit 4 (f7ab0fd5cc87) Changing license to GPL gives it a GPLv3+ license.
I'm not in anyway related to their team, but I've discovered this while fixing an issue in Phenograph's embed Louvain.
(Feel free to grab my fix)

Though, according to the file in Phenograph's subdirectory the Louvain code was modified to accept binary input.
Probably graph.cpp: Graph::Graph was modified to work with the ouput of core.py: graph2binary

nudge
Since this is a derivative of a GPLv3 software package, it's actually required that the entire source code be made available including any code changes and additions. (Generally this library must also be licensed GPLv3 itself, not as MIT.)

@zbjornson - I guess it depends on when the files were taken from the original repository for modification. If they were taken before the license was changed to GPL (Jan 26 2012) then there is no GPL requirement, and therefore no requirement to expose the source code. This is up to @jacoblevine to say.
I agree that this has to be resolved one way or another since most institutions and businesses cannot use the package in its current form until the licensing issue is clarified.

Perhaps we can change core.py's graph2binary to generate a boring old text file, so that we no longer need any modified version of louvain.


[DOC] add maintainers guide

closes https://github.com/bids-standard/bids-website/issues/252

move maintainers from google doc to markdown

@Remi-Gau @ericearl are there any sanity checks our writer could do here, to get feet wet and hands dirty?


Not handy to produce numeric string

example : { "aFloatString" : "1.23" }
the only way to produce it for now is { aFloatString : @s(1.23) }, which does not perfectly reflect the intention of the end result.

It will be great to produce the numeric string, which can work something like this
@s(@i(max=100))

I thought the form of "@si", "@sf", "@sb" for integer string, float string, and boolean string (null string can be applied to any of these types). Any other values are presented as the form of string anyway.
The form of @s(@i) opens a window of "type mixin" for the future. That requires the improvements on the compiler of JsonTemplate.


chore(Python): added binary search tree

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] New algorithm
[ ] Optimization in previous algorithms
[ ] Code style update
[ ] Refactor
[ ] Documentation
[ ] Other, please describe:

Briefly describe the changes in this PR

added binary-search-tree algorithm

Other information:

Hi @hashfx,
Please work with this #421

Congrats on merging your first pull request!  You are invited to join the team on Github and Discord.

Thanks a lot, @MakeContributions for this opportunity!


Added MVC nuget package to Karbon.Cms.Web

The project Karbon.Cms.Web project had dependecies on local installation of MVC. Added same versions of MVC, Razor & WebPages as used in the TestHarness project.

Realized now you had an old dev branch going, maybe this was fixed in there.


errror when train finished

when I finish learning 100%, I get this error, and even if I open the mesh with the mesh lab, the file is weird

Hi, how do you extract the mesh?

Hi, how do you extract the mesh?

no this error occured

Hi, the error occurs after training finished, so you can still extract the mesh with the saved checkpoint. Please follow the instruction in readme.

Hi, the error occurs after training finished, so you can still extract the mesh with the saved checkpoint. Please follow the instruction in readme.

ah second picture is mesh error, when i open extract mesh in meshlab that error occured. Mesh is coming out really weird


Validate multiple jws entries in a VC

addresses #61
command line tree properly displays multiple jws entries
log file denotes different jws entries with an index (output is still flat)
added card tests for multi-jws (1 with and 1 without errors/warnings)

Is this on top of PR #70 or does it supersedes it?


Update to selenium 3.14.0.

Contains some changes that are covered in #402
Tests are passing.

@shankarkc Can you now check this.

@damencho  Thanks a lot for quick turnaround time. I will check tomorrow as its night here IST. I have notice that com.groupon.seleniumgridextras.grid.proxies.SetupTeardownProxy was broken in selenium grid extra due to changes in selenium grid code base. When I used SetupTeardownProxy it didnt work for me. When i removed it it worked. With the new build does nodes getting registered with hub fine? I was experiencing 500 error last time.

Yep, SetupTeardownProxy was broken in master when using latest selenium. It is fixed in 2.x branch, the one we are using.
My initiative here was that we are using some changes to run our selenium grid using selenium-extras and I just wanted to push those modifications for the community, and I just took the diffs to create patches, but apparently, I used the wrong branch :)

@damencho  please let me know the git repo/branch that I can use. I will test it and let you know the results tomorrow.

It seems to me it is not used, at least I don't see a broken icon or anything on the UI.
This is the repo and the branch: https://github.com/damencho/Selenium-Grid-Extras/tree/update-selenium-2.x

*@damencho *I found an issue. I havnt gone thought the code that is
causing it.
{
"capabilities": [
{
"seleniumProtocol": "WebDriver",
"browserName": "firefox",
"maxInstances": 3,
"version": "62",
"platform": "WIN10"
},
{
"seleniumProtocol": "WebDriver",
"browserName": "chrome",
"maxInstances": 3,
"version": "68",
"platform": "WIN10"
}
],
"loadedFromFile": "node_5555.json",
"proxy": "com.groupon.seleniumgridextras.grid.proxies.SetupTeardownProxy",
"servlets": [],
"maxSession": "3",
"port": "5555",
"register": true,
"unregisterIfStillDownAfter": "10000",
"hubPort": "4444",
"hubHost": "<IP_ADDRESS>",
"registerCycle": "5000",
"nodeStatusCheckTimeout": "10000",
"browserTimeout": "null",
"timeout": "null",
"custom": {
"grid_extras_port": "3000"
},
"downPollingLimit": "0"
}
Here is node_5555.json that got generated. It has null as timeout value.
because of it grid fails to launch
Exception in thread "main" com.beust.jcommander.ParameterException:
"-timeout": couldn't convert "null" to an integer
at
com.beust.jcommander.converters.IntegerConverter.convert(IntegerConverter.java:38)
at
com.beust.jcommander.converters.IntegerConverter.convert(IntegerConverter.java:28)
at
com.beust.jcommander.JCommander.convertValue(JCommander.java:1472)
at
com.beust.jcommander.ParameterDescription.addValue(ParameterDescription.java:238)
at
com.beust.jcommander.ParameterDescription.addValue(ParameterDescription.java:218)
at
com.beust.jcommander.JCommander.initializeDefaultValue(JCommander.java:665)
at
com.beust.jcommander.JCommander.initializeDefaultValues(JCommander.java:356)
at com.beust.jcommander.JCommander.parse(JCommander.java:339)
at com.beust.jcommander.JCommander.parse(JCommander.java:319)
at
org.openqa.grid.internal.cli.GridNodeCliOptions$Parser.parse(GridNodeCliOptions.java:48)
at
org.openqa.grid.selenium.GridLauncherV3$3.(GridLauncherV3.java:296)
at
org.openqa.grid.selenium.GridLauncherV3.lambda$buildLaunchers$2(GridLauncherV3.java:295)
at
org.openqa.grid.selenium.GridLauncherV3.buildLauncher(GridLauncherV3.java:163)
at
org.openqa.grid.selenium.GridLauncherV3.launch(GridLauncherV3.java:97)
at
org.openqa.grid.selenium.GridLauncherV3.main(GridLauncherV3.java:81)
On Thu, Aug 16, 2018 at 11:33 PM, Дамян Минков<EMAIL_ADDRESS>wrote:

It seems to me it is not used, at least I don't see a broken icon or
anything on the UI.
This is the repo and the branch: https://github.com/damencho/
Selenium-Grid-Extras/tree/update-selenium-2.x
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/groupon/Selenium-Grid-Extras/pull/420#issuecomment-413633926,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEb7V--8PThiPVOc_uV_r8LTiHpTkhHPks5uRbP6gaJpZM4WANXU
.

I see, while fixing the tests I saw that, but didn't think it will be an issue. The Gson serializer is supposed to be removing those by default ... I will check these days or next week, when I have time.

same issue is with browserTimeout too. Once its fixed I do see grid node getting registred to hub. Great work. :clap: :bow: :sparkles::sparkles:

@damencho selenium grid extra hub URL is not working ie on hub node if i run http://10.xxx.xxx.1:3000 it should open video page. But it times out

I fixed the browserTimeout and timeout.
The missing xuggle, I don't think it has anything to do with my PR, I think if you checkout just the 2.x branch you will have the same error or it is something local.
I will check our fork, cause it was running with xuggle for some time, so if there is anything around that ...

Nope, I don't see anything related to that, so branch 2.x should be running fine.
And also in the target jar-with-dependencies all the native libraries are there, are you sure it is not something local on your deployment?

Let me try running it on onther machine. Will update you Monday ist eod.
On Fri, Aug 17, 2018, 11:35 AM Дамян Минков<EMAIL_ADDRESS>wrote:

Nope, I don't see anything related to that, so branch 2.x should be
running fine.
And also in the target jar-with-dependencies all the native libraries are
there, are you sure it is not something local on your deployment?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/groupon/Selenium-Grid-Extras/pull/420#issuecomment-413766933,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEb7V1zVp1ywKHspihOEA60dlzsEKGsvks5uRl0tgaJpZM4WANXU
.

I have the same question :(


Add logging based on env var

Logging is enabled through export SHOPIFY_LOG=1

Let's document this in the README.


Make it compatbile with ghc-8.8.1

Fix #5

The test failure is unrelated to the code, so merging it.


Add example of what a layout file would look like

Add an example of what a layout file would look like when used with Bamboo.Phoenix. I always forget how to call render with the assigns and I'm sure others will have to look it up as well. Might be worth including an example to save some time and make it easier to use the library.

This would be super useful. Specifically, it'd be nice to see how we can use route helpers in layouts/templates since @conn isn't a thing.
(I have a decent/simple sign in email w/ layout example I can provide once it's 💯 . Just let me know if/where you want it.)

@jerodsanto It would be wonderful if you could paste an example here or open up a PR. Especially if it includes using MyApp.Endpoint in URL helpers so that people can see how to use those. Thanks for offering to help!

@paulcsmith happy to PR. Where do you want it the example to go? bamboo/lib/bamboo/phoenix.ex?
There are a few files...

Yes, Bamboo.Phoenix would be perfect!

On Jun 1, 2016, at 10:59 AM, Jerod Santo<EMAIL_ADDRESS>wrote:
@paulcsmith https://github.com/paulcsmith happy to PR. Where do you want it the example to go? bamboo/lib/bamboo/phoenix.ex?
There are a few files...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/thoughtbot/bamboo/issues/110#issuecomment-223020050, or mute the thread https://github.com/notifications/unsubscribe/AABXemwBxn7C7u-4QK8nI3mFNa_VzjhHks5qHZ4-gaJpZM4H-Du2.

Closed by #173


Separate Clinical Trial Info extractors

Summary
Separates a CSV-only and Service-CSV Clinical Trial Info extractors
New behavior
ServiceClinicalTrialInfoExtractor is the same as master's CSVClinicalTrailExtractor. When we add context into all the Epic extractors, we will add it into this extractor. We'll first try to get patient/patientId from context, and if it is not available, we'll use the BaseFHIRModule to get patient. Since I didn't start any of the context tasks, it just uses BaseFHIRModule as before.
CSVClinicalTrialExtractor uses no services at all. When/If we add context to non-Epic extractors, we will add it into this extractor. We'll first try to get patient/patientId from context, and if it is not available, we'll use the MRN as a default id. Since I didn't start any of the context tasks, I just wrote a function that does nothing right now and then defaults to MRN.
Code changes
Renamed CSVClinicalTrailInfoExtractor to ServiceClinicalTrialInfoExtractor, created new CSVClinicalTrialInfoExtractor without dependency on BaseFHIRModule. Added corresponding updates to tests.
Testing guidance
Check the code/tests. There is a related PR in icare-extraction-client if you want to try running the CSV client to see it get past the Clinical Trial Extraction successfully.
One note - I didn't start any of the context pieces in this PR because it seemed to require a number of the subtasks we have in the context task, so I tried to lay out where some of the pieces will go when we do that task.

DP1 - Talked a bit offline, but posting here to centralize the next steps: I think the ServiceClinInfoExt should live in the other EHR-specific repo, and should be renamed accordingly. The way the Extractor is designed is based on the constraints of that EHR-specific env and the information we have about what WebServices are and are not available in that env. Even though none of the modules it uses are proprietary to that env, the choices we've  made about what modules we use and in what order are informed by the EHR's constraints.

I think DP1 makes sense. I didn't really think about how the combination/steps we're doing are what make it Epic specific, and was just thinking about how the modules are not Epic specific. If somehow we discover some day this is a typical process, we can always move it back into the base repo.
I'm also going to fix DP2 now and start looking into DP3.

DP1 and DP2 are fixed here.
There's also now another PR for the Epic MEF, which has the files moved as part of DP1.
The icare-extraction-client PR is also updated to reflect changes from DP1.

3a0958a should address DP3 test-related comments.

There's one more change incoming here, but currently this looks good.


relationships not generated

The following does not fill in the relationship fields for any entities in the .json files that are generated.
entity Organization {
name String required,
description String,
createdDate ZonedDateTime,
lastModifiedDate ZonedDateTime
}
entity Region {
name String required,
description String,
createdDate ZonedDateTime,
lastModifiedDate ZonedDateTime
}
entity Location {
name String required,
description String,
latitude Float,
longitude Float,
altitude Float,
createdDate ZonedDateTime,
lastModifiedDate ZonedDateTime,
markerIcon String
}
entity FormOutline {
title String required,
description String,
createdDate ZonedDateTime,
lastModifiedDate ZonedDateTime
}
entity FormComplete {
createdDate ZonedDateTime,
lastModifiedDate ZonedDateTime
}
entity FormOutlineString {
label String required
}
entity FormOutlineFloat {
label String required
}
entity FormOutlineDatetime {
label String required
}
entity FormCompleteString {
value String required
}
entity FormCompleteFloat {
value Float required
}
entity FormCompleteDatetime {
value ZonedDateTime required
}
relationship OneToMany {
FormOutline{formoutlinestring(label)} to FormOutlineString{formoutline(title)},
FormOutline{formoutlinefloat(label)} to FormOutlineFloat{formoutline(title)},
FormOutline{formoutlinedatetime(label)} to FormOutlineDatetime{formoutline(title)},
FormOutline{formcomplete} to FormComplete{formoutline(title)},
FormComplete{formcompletestring(value)} to FormCompleteString{formcomplete},
FormComplete{formcompletefloat(value)} to FormCompleteFloat{formcomplete},
FormComplete{formcompletedatetime(value)} to FormCompleteDatetime{formcomplete},
FormOutlineDatetime{formcompletedatetime} to FormCompleteDateTime{formoutlinedatetime(label)},
FormOutlineString{formcompletestring} to FormCompleteString{formoutlinestring(label)},
FormOutlineFloat{formcompletefloat} to FormCompleteFloat{formoutlinefloat(label)}
}
relationship ManyToMany {
Region{location(name)} to Location{region(name)}
}

How did you generate this? Using the import jdl sub or using jhipster uml?
Thanks & regards,
Deepu
On 19 Jun 2016 16:21, "Brian L"<EMAIL_ADDRESS>wrote:

The following does not fill in the relationship fields for any entities in
the .json files that are generated.
entity Organization {
name String required,
description String,
createdDate ZonedDateTime,
lastModifiedDate ZonedDateTime
}
entity Region {
name String required,
description String,
createdDate ZonedDateTime,
lastModifiedDate ZonedDateTime
}
entity Location {
name String required,
description String,
latitude Float,
longitude Float,
altitude Float,
createdDate ZonedDateTime,
lastModifiedDate ZonedDateTime,
markerIcon String
}
entity FormOutline {
title String required,
description String,
createdDate ZonedDateTime,
lastModifiedDate ZonedDateTime
}
entity FormComplete {
createdDate ZonedDateTime,
lastModifiedDate ZonedDateTime
}
entity FormOutlineString {
label String required
}
entity FormOutlineFloat {
label String required
}
entity FormOutlineDatetime {
label String required
}
entity FormCompleteString {
value String required
}
entity FormCompleteFloat {
value Float required
}
entity FormCompleteDatetime {
value ZonedDateTime required
}
relationship OneToMany {
FormOutline{formoutlinestring(label)} to
FormOutlineString{formoutline(title)},
FormOutline{formoutlinefloat(label)} to
FormOutlineFloat{formoutline(title)},
FormOutline{formoutlinedatetime(label)} to
FormOutlineDatetime{formoutline(title)},
FormOutline{formcomplete} to FormComplete{formoutline(title)},
FormComplete{formcompletestring(value)} to
FormCompleteString{formcomplete},
FormComplete{formcompletefloat(value)} to FormCompleteFloat{formcomplete},
FormComplete{formcompletedatetime(value)} to
FormCompleteDatetime{formcomplete},
FormOutlineDatetime{formcompletedatetime} to
FormCompleteDateTime{formoutlinedatetime(label)},
FormOutlineString{formcompletestring} to
FormCompleteString{formoutlinestring(label)},
FormOutlineFloat{formcompletefloat} to
FormCompleteFloat{formoutlinefloat(label)}
}
relationship ManyToMany {
Region{location(name)} to Location{region(name)}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jhipster/jhipster-domain-language/issues/30, or mute
the thread
https://github.com/notifications/unsubscribe/ABDlFxlb_Pw9bs9rX27uV6_FZTKjDdejks5qNPwEgaJpZM4I5Hkr
.

I typed it into JDL studio on the jhipster site. it did not show any errors. so I put it in my project folder and typed the 'yo jhipster:import jdl ...' command to load it and it did nothing. I then tried loading each entity section on its own and it worked but the relationship section refused to load.

The import jdl sub doesnt support multiple relationship definition in the
same block yet. Please use jhipster uml for this. We are working to make
the import jdl support all options. It should land soon
Thanks & regards,
Deepu
On 20 Jun 2016 07:26, "Brian L"<EMAIL_ADDRESS>wrote:

I typed it into JDL studio on the jhipster site. it did not show any
errors. so I put it in my project folder and typed the 'yo jhipster:import
jdl ...' command to load it and it did nothing. I then tried loading each
entity section on its own and it worked but the relationship section
refused to load.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jhipster/jhipster-domain-language/issues/30#issuecomment-227026827,
or mute the thread
https://github.com/notifications/unsubscribe/ABDlF10vum4HhH4tvgZTlKhK5ya_mtteks5qNdANgaJpZM4I5Hkr
.

Im gonna take some time off for this today and see if i can migrate to the newer JDL version. @jdubois need your help to rename the jdl project to jhipster-core as we decided earlier. Cc @MathieuAA


Find in files indexing for autocomplete / instant search

One thing I love about Visual studio is the entrian source search plugin (http://entrian.com/source-search/ ). It performs autocomplete while in the search box (because I sometimes remember only the prefix of a given function), and does 'instant' (<500ms) search on our 10M+ loc solution for whole word search, file VSCode take ~5-10s on the same machine.
I understand there'd be a disk cost associated for storing this indexed data, but I for one would live happily with it if it could give me these 2 features.
Thanks

We have investigated this in the past but rejected it because it's just at odds with vscode being a simple and lightweight text editor.


uhdm: 1.80 -> 1.82

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/uhdm/versions.
meta.description for uhdm is: Universal Hardware Data Model
meta.homepage for uhdm is: https://github.com/chipsalliance/UHDM
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.82 with grep in /nix/store/nn46di06p4bl984in6vyx84085jkrrw5-UHDM-1.82
found 1.82 in filename of file in /nix/store/nn46di06p4bl984in6vyx84085jkrrw5-UHDM-1.82

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
surelog
uhdm
yosys-synlig

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nn46di06p4bl984in6vyx84085jkrrw5-UHDM-1.82 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A uhdm https://github.com/r-ryantm/nixpkgs/archive/32f2120b01d44d004c9ecbfbea25fe6367031c85.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/32f2120b01d44d004c9ecbfbea25fe6367031c85#uhdm

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nn46di06p4bl984in6vyx84085jkrrw5-UHDM-1.82
ls -la /nix/store/nn46di06p4bl984in6vyx84085jkrrw5-UHDM-1.82/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

3 packages built:

surelog
uhdm
yosys-synlig

Maintainer pings
cc @matthuszagh for testing.

https://github.com/NixOS/nixpkgs/issues/290642


inconsistent world.update

hi im using dyn4j version 3.3.0
im facing issue when updating world. when calling update with delta 0.100 the simulation speed slowing down but when calling with update with 0.016 the simulation speed goes faster.
step to reproduce
1 creating world
2 set gravity to none
3 add circle and set linear velocity to 10
run world with different delta aka elapsed time and observe the different

if we increase the scale between delta let say first test 1 second and second test 0.1 second the result still wrong

I think the missing link here is that the World has its own internal clock.  In the first case, 0.1 is larger than the default update frequency of 1/60.  The update method in this case still only performs 1 iteration of the engine.  The effect is that extra time continues to accumulate, but is never evaluated.  There’s another version of the update method that allows you to specify the maximum number of iterations to perform per call.

can you give sample code for that update usage ? i think i am doing it wrong i am sorry. there no example in documentation-andvanced page please help me to pass the test

Have you read through the java docs for the world class, specifically the update methods?
I can post an example later.

The World class has an internal clock that determines when it performs a simulation step.  The internal clock is defaulted to 1/60.  This value is configurable via the world.getSettings().setStepFrequency(double) method.
Now, lets take your example of 0.01 and examine what it will do:

Iteration 0: elapsed time = 0.01, 0.01 < 1/60 so no simulation is performed, hence (0,0) coordinates.
Iteration 1: elapsed time = 0.02, 0.02 > 1/60 so a single simulation step is performed and the remaining time = 0.02 - 1/60 = 0.00333333333, coordinates = (0.16666666666666666, 0)
Iteration 2: elapsed time = 0.01333333333, 0.01333333333 < 1/60 so no simulation is performed, coordinates = (0.16666666666666666, 0)
Iteration 3: elapsed time = 0.02333333333, 0.02333333333 > 1/60 so a single simulation step is performed, coordinates = (0.3333333333333333, 0.0)
And so on...

So as you can see, the internal clock of the World determines when it actually does simulation calculations.  If it hasn't been long enough, then nothing is done, the method immediately returns.
In the case of anything greater than 1/60, the World will perform a single simulation step, using up only 1/60 of the given elapsed time, with the rest stored in the internal time accumulator.  If the elapsed time is consistently above 1/60, the simulation will never catch up - the slowness you are seeing.
Hopefully the problem is clearer now.  The solution will depend on your use-case, but my guess is that you are looking for the update(double, int) method.  The second parameter in the update method allows you to tell the engine the maximum number of steps to perform per call.  By default its 1.
So changing the code to world.update(0.10, 10) should fix your problem:

Iteration 0: elapsed time = 0.1, 0.1 > 1/60, so we need to perform 6 steps to catch up, left over time = 0
Iteration 1: elapsed time = 0.1, 0.1 > 1/60, so we need to perform 6 steps to catch up, left over time = 0
And so on...

On each iteration we are actually performing 6 simulation steps each using 1/60 elapsed time for a total of 0.1 elapsed time.
The Javadoc has tons of information on these methods.

briliant it solve the problem 👍👍👍👍👍. i going to close this issue. what a noob me. thanks for the assistance. i saw the forum is going to retire so i ask this question here is it ok ?

Yeah, feel free to ask any questions here.


Adding Person Counting feature

Person Counter
Description:
It is a very quick implementation of counting the number of people in a room using a Mobilenet SSD detector.This can help in the Surveillance and especially in a covid time where there should not be more than a certain amount of people in the room.
Table of contents

About the Modules
Surveillence Usecase
Setup
Demo

About the Modules:
The Modules used here mainly are:

imutills
Numpy

We make use of a single-shot detection algorithm for the detection of humans that are present in the room. A single Shot detector like YOLO takes only one shot to detect multiple objects present in an image using a multi-box. It is significantly faster in speed and high-accuracy object detection algorithm. SSD also fits our requirements of high fps and lightweight python program
Surveillance Usecase:
During these times of Covid, it is a must to follow the SOP and curb the chain of transmission. In this one of the main Protocols to be followed is Social distancing and not overcrowding any rooms. The Person Counting code  will help us in the Surveillance of whether the room is overcrowded or not and thus keep an eye on whether the SOPS are being followed and then curb the spread of COIVD
Setup
To run this project locally:
Go to the directory where the project is present
Write this in your Command prompt
$pip -r rquirements.txt

Then after this in the command prompt and use this command
$ python Person_Counter.py

Demo

Hey,, I have removed rhe contribution as asked to , thanks!

Appreciate the efforts!  We notice the following:

[X] Addition of new module / improvement on existing
[X] Good flow of readme, but we notice that a few more sections can be added ( kindly look into readme of face_recognition module)
[X] Relevancy of module with surveillance system can be highlighted
[X] Please mention the output demo/gif/video/image for easy evaluation
[X] 'Contributor' section can be omitted for now

All the best!


⭐ Deploy app to App Store and Google Play Store

Is your feature request related to a problem? Please describe.
Challenge not available publicly
Describe the solution you'd like
Deploy manually the app to both stores

First task just to add screenshots

Merge on main by error..

Looks like it has been done, closing as the app will change anyways, will publish after

Closing the branch as it has been merged anyways.


Turn on I/Q Mode when SoapySDR device is created

I'm probably not doing the pull requests properly...
I'll make each commit a unique item, it looks like they all end up under the same pull request.
Please see the comments for each commit.

Thanks for the update


Support for ST-LINKV3MINIE hardware

Would it be possible to support the ST-LINKV3MINIE hardware? It also employs a STM32F723 and is readily available globally at a competitive price.

It is supported - if anything the USB VID/PID have to be added :)
While ST-Link is very cheap and nice HW, the Software makes up for it and is not open source, locked to STM32's and orthogonal to all other probe APIs.
So I always recommend other probes.

@Yatekii I don't thing the HS probe has a BSP nor support for the level translators on the ST-LinkV3MINIE.
@eflukx This could probably be added, but it would require someone with interest to get the schematics for the STLink and port the code to this hardware.
I've not checked if the schematics are open, but it should be quite doable.

Ah lol, I am stupid, I am sorry, somehow I thought this is the probe-rs repo 🤦

From the firmware point of view supporting this hardware is technically possible and should not be very difficult, however, uploading this firmware to the hardware may not be possible. Some time ago ST updated the bootloader in v3 stlinks and now some sort of a cryptographic signature is used to check the uploaded firmware. Old stlinkv3 devices use the old bootloader though, which doesn't check the uploaded firmware. For these devices it's possible to upload custom firmware.

Thanks for the insight! One can't access the debug port either I guess?

Starting from version V3J6M2 firmware enables Level2 protection which permanently disables the debug port.

Oof, well that was unfortunate... for us :sweat_smile:

Starting from version V3J6M2 (~Jan 2020) firmware enables Level2 protection which permanently disables the debug port.

Hmm.. that is kind of crappy... The STLINK's seemed a really good option given their general availability and price.
What I'm struggling with is that the STLINKV3 does not support the Nordic devices (which I typically use) and also hard-chokes (as a "feature") on non-genuine "STM32" clones.
An open firmware on these devices would really be nice.. oh well... you can't win them all...
Ps. And I guess the DFU way is also closed (by cryptographic means..) as mentioned by @Disasm
Off topic: what would be the probe-of-choice:tm: to use with probe-rs. I'm currently using JLink devices, but I'm not really impressed by the performance of these (in conjunction with probe-rs anyway)...

We have two open source probes (you can find the repos in the org) that @korken89 designed and we just started another production batch :) We sell it basically at cost on our webpage. But you can ofc assemble it yourself :)

An open firmware on these devices would really be nice.. oh well... you can't win them all...

There is still hope. IIRC new firmware can still be decrypted, so one day someone may find a code execution vulnerability which may allow bootloader dumping and it may happen that crypto used for integrity checks is so custom that it can be broken easily by looking at the bootloader code.
When someone reported that a new bootloader is now shipped with new stlinks, I ordered a couple of probes directly from ST and unfortunately they all had the old bootloader :)

Ps. And I guess the DFU way is also closed (by cryptographic means..) as mentioned by @Disasm

Original DFU located in system memory is disabled due to Level2 protection and does not support USB HS peripheral which is connected to the USB connector. For the update purpose ST flashed their own DFU-like bootloader which decrypts the firmware to update and (depending on the version) performs integrity checks.


Replace Puppet.initialize_settings with Puppet::Test::TestHelper

TestHelper offers a public API for (re)initialising Puppet reliably,
which is preferable to using initialize_settings. It also avoids
conflicts when multiple libraries call the same method.

Thanks for the PR one of us will hopefully look at this soon.

I think this is good to merge. :+1:
In my testing it adds 20% to our runtime. I think this is acceptable cost given it's the right way to do it.
@DavidS do you have any opinions about this change?

@rjw1 it does clean out all the caches, so the impact on runtime is not surprising. Given that @domcleal has found this, I do expect he's run into one of those nasty edge cases where one test had poisoned the well for another one, in which case there is no good way around it.

Luckily it wasn't so much a test, but I was using puppet-syntax together with the kafo_parsers gem - both used initialize_settings, and Puppet threw an error because it had already been initialised by the other gem.
Though puppet-syntax has a guard to check if Puppet's already initialised, kafo_parsers didn't, so using the proper setup/cleanup mechanism in both gems seemed the best way to fix compatibility issues permanently.

The code is good and the explanation makes sense so I'm going to merge it. @rjw1 we should add a ticket to get a new release issued.


[Bug] Crashes when Viewing profile through the User Logo

Describe the Bug
My Discord Will give an error 9000 when I press someone's User logo When using Better Discord
To Reproduce
Click on Someone Profile Picture in a server.
Expected Behavior
Your Discord will enter a screen saying "YOU DIED" under saying "Looks Like you got slaughtered by Error Level 9000"
Screenshots

Discord Version
My Better Discord version is 0.3.5
Additional Context

That screen is either added by a theme or plugin. Sounds like you have the plugin StatusEverywhere. Can you confirm?

That screen is either added by a theme or plugin. Sounds like you have the plugin StatusEverywhere. Can you confirm?

Yeah I have status everywhere. How would I fix it?

Disable StatusEverywhere and reload (ctrl+r). Then see if clicking the "User logo" still crashes your Discord.

Welp that fixed the issue for me. Ty rauenzi.

yeah, it's fixed. Would there be a way to report the issue to that dev? I really like that plugin

There's an open issue for it already https://github.com/rauenzi/BetterDiscordAddons/issues/268

Hey ! How i can disable StatusEverywhere ?

I don't know where it is in my coumputer

i have the same problem i dont know how to fix it its not doing it to my other pcs

Same issue, it occurred even tho i dont have statusEverywhere  plugin. It happened after i updated ImageUtils, and now i removed it but no luck.

how can i turn the  StatusEverywhere off
@ZaneDragonBorn

to turn off plugins, you have to restart discord( if you have seen the error. Go to settings, plugins and toggle.
Or using RUN. Press Win+R( idk for mac, type "%appdata%/BetterDiscord" without the "" marks. And delete the desired plugin.


Add country summary endpoint and implement browse_rds

relates to BAK-1148

Will supersede document_browse (which will need removing)
Browse functionality not implemented
Country browsing implemented on the summary endpoint
Adds tests

We're going to have some fun merging this with what I've been up to... let's catch up in the morning (sorry :disappointed:)


golist => godecls

Codecov Report

Merging #102 (5c63111) into main (0d279c0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #102   +/-   ##
=======================================
  Coverage   93.89%   93.89%
=======================================
  Files          14       14
  Lines        5572     5572
=======================================
  Hits         5232     5232
  Misses        261      261
  Partials       79       79

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 0d279c0...5c63111. Read the comment docs.


Data: project addition by coderhersh

Add a new project to the list
I want to add a weather app in this repository which I made in React JS.
Kindly, assign this issue to me  under GSSOC'23.
https://github.com/priyankarpal/ProjectsHut/assets/56130865/ba744a46-e907-42f7-9270-5b65ca0ded5a
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Completed. Thanks for your contributions 🎉 ❤


Unable to save Configuration

Description:
Unable to save a configuration change for all entities, as well as Workload, with the Catalog Cloud Optimize.
Steps to Reproduce
Change any element in the Configuration page (universal setting, or for a specific workload). Then select save. The save button then just spins and spins, never completing a save. Upon refresh, no values change.
Expected Behaviour
Upon save, configuration would be retained and new executions of Optimize would reflect the new config change.
Relevant Logs / Console output
Your Environment
Additional context
Please feel free to reach out to me asap with additional questions.

I was unable to reproduce this and could indeed save a change on the Optimization Configs page. @dralph3344 can you check this again and let us know if you still see an issue?

Hey @jpvajda, Thanks for looking at this. There are some capabilities around adjusting the configuration that we should investigate to make it a little easier.
That being said, Im actually good on this issue and was able to adjust the configuration. Thanks!


[go_router] redirect to shell router

hello
it is for long time i try to fix this problem but i couldn't find the solution from the official videos and documentation.
i wanna use redirect while the user logs in. It goes to the Shell route, not the homepage. What is the solution  for that case as you know most of the developers use Shell route for landing page while user logging in .
please as soon as possible answer me
import 'package:buy_online_shopping/_shared/services/user_preferences.dart';
import 'package:buy_online_shopping/screens/auth/presentation/pages/sign_in.dart';
import 'package:buy_online_shopping/screens/auth/presentation/pages/sign_up.dart';
import 'package:buy_online_shopping/screens/intro/presentation/pages/onboardingpage.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';
import 'package:go_router/go_router.dart';
import 'package:buy_online_shopping/screens/intro/presentation/pages/splash.page.dart';
import 'package:buy_online_shopping/screens/landing/presentation/pages/home.page.dart';
import 'package:buy_online_shopping/screens/landing/presentation/pages/landing.page.dart';
import 'package:buy_online_shopping/screens/website/presentation/pages/web_links_page.dart';
import 'package:buy_online_shopping/_helpers/utils.dart';
`final goRouteProvider = Provider((ref) {
return GoRouter(
routerNeglect: true,
initialLocation: SplashPage.route,
navigatorKey: Utils.mainNav,
redirect: (context, state) async {
var userLoggiedIn = await RememberUserPrefs.readUserInfo();
    if (userLoggiedIn != null) {
      //i wanna return Shell route not landing page
      //it should be  GoRouter.of(Utils.mainNav.currentContext!).go('/homepage');
      //but it will return null

      return LandingPage.route;
    } else {
      return SplashPage.route;
    }
  },
  routes: [
    GoRoute(
      parentNavigatorKey: Utils.mainNav,
      path: SplashPage.route,
      builder: (context, state) {
        return const SplashPage();
      },
    ),

    GoRoute(
      parentNavigatorKey: Utils.mainNav,
      path: OnboardingPage.route,
      builder: (context, state) {
        return const OnboardingPage();
      },
    ),
    GoRoute(
      path: SignIn.route,
      builder: (context, state) => const SignIn(),
    ),
    GoRoute(
        path: SignUp.route, builder: (context, state) => const SignUp()),

//Bottom Appbar
ShellRoute(
navigatorKey: Utils.tabNav,
builder: (context, state, child) {
return LandingPage(child: child);
},
routes: [
GoRoute(
parentNavigatorKey: Utils.tabNav,
path: HomePage.route,
pageBuilder: (context, state) {
return const NoTransitionPage(
child: HomePage(),
);
},
),
GoRoute(
parentNavigatorKey: Utils.tabNav,
path: WebLinksPage.route,
pageBuilder: (context, state) {
return const NoTransitionPage(
child: WebLinksPage(),
);
},
),
]),
]);
});
`

Hi @rebazjabar
The goal of this Github repo is not to assist in personal code or the whole package/plugin project but to improve Flutter framework reporting bugs or feature requests.
It seems your question is best suited for support channels like StackOverflow, so please ask the same there for solutions if any. If you believe there is a bug in go_router package and propose a change, please use the appropriate new issue template and fill out the requested details so that we can address this issue properly.
Closing from here as this is more of a how-to question rather than a bug or feature request.


Setting a BIT field to 1 with TVPs results in a 0

SQLServerDataTable table = new SQLServerDataTable();
table.addColumnMetadata("Wat", Types.BIT);
table.addRow(1);

This table will store a 0 into your database, because the 1 gets converted to "1" with toString, then parsed to false with Boolean.parseBoolean.
I personally disagree with SQLServerDataTable::addRow's general approach of "call toString on the argument then parse it in a lax manner". I think it would be better to:

Handle known types specifically, e.g. if you attempt to set a BIT field with a Byte, cast it to a Byte and handle the value
Handle known values specifically, e.g. allow 0, 1, true, false to set a BIT field
Reject unknown types and nonsense values with an exception

Related: #538 (via #531 ).

Thanks @gordthompson for relating PR to the issue.

Handle known types specifically, e.g. if you attempt to set a BIT field with a Byte, cast it to a Byte and handle the value

I guess that's implemented in PR.

Handle known values specifically, e.g. allow 0, 1, true, false to set a BIT field

We will update PR to parse inputs (1, 0) for Boolean type (True, False) respectively.

Reject unknown types and nonsense values with an exception

SQL Server does that for us, so additional logic in Java is not needed. We rely on SQLException and throw it directly. Is there any specific case where it will be helpful [other than integers 0,1 for Boolean that will now be handled]?

Thanks for the update. I disagree with your point that we can rely on SQL Server to reject unknown types because the code frequently calls toString on the values passed in then parsing them. So all the downstream code sees is whatever type the string is subsequently parsed as.
To clarify I've come up with some examples by looking at the code (though I haven't run these so correct me if I'm wrong). I believe the PR does fix the BIT case that I reported but there are still several cases where I think the code continues to be overly permissive and others where I think the PR has introduced regressions:
Object unknownTypeInstance = new Object() { @Override public String toString() { return "1"; } };

{
    SQLServerDataTable table = new SQLServerDataTable();
    table.addColumnMetadata("c1", Types.BIT);

    //     Before PR: insert 0
    //      After PR: insert 1
    // What I expect: REJECT because the type is unknown
    table.addRow(unknownTypeInstance);

    //     Before PR: insert 0
    //      After PR: insert 1
    // What I expect: REJECT because of truncation
    table.addRow(0.6f);

    //     Before PR: insert 0
    //      After PR: insert 1
    // What I expect: REJECT because of truncation (debatable given 1.0f is representable exactly, but I think a rule of
    //                "you can't set integral fields with floating point types" is easy to grok).
    table.addRow(1.0f);

    //     Before PR: insert 0
    //      After PR: insert 0
    // What I expect: insert 0
    table.addRow(0);

    //     Before PR: insert 1
    //      After PR: insert 1
    // What I expect: insert 1
    table.addRow(1);

    //     Before PR: insert 0
    //      After PR: insert 0
    // What I expect: REJECT because of truncation
    table.addRow(3);
}

{
    SQLServerDataTable table = new SQLServerDataTable();
    table.addColumnMetadata("c1", Types.SMALLINT);

    //     Before PR: REJECT with NumberFormatException
    //      After PR: insert (1234567 % 2^16)
    // What I expect: REJECT because of truncation
    table.addRow(1234567);

    //     Before PR: insert 1
    //      After PR: insert 1
    // What I expect: REJECT because the type is unknown
    table.addRow(unknownTypeInstance);

    //     Before PR: REJECT with NumberFormatException
    //      After PR: insert 1
    // What I expect: REJECT because of truncation
    table.addRow(0.6f);

    //     Before PR: REJECT with NumberFormatException
    //      After PR: insert 1
    // What I expect: REJECT because of truncation
    table.addRow(1.0f);

    //     Before PR: REJECT with NumberFormatException
    //      After PR: insert 3
    // What I expect: REJECT because of truncation
    table.addRow(3.0f);
}

@alexnixon Thanks for your feedback!

I disagree with your point that we can rely on SQL Server to reject unknown types because the code frequently calls toString on the values passed in then parsing them. So all the downstream code sees is whatever type the string is subsequently parsed as.

Sorry for the confusion, I meant to say about Datatype Parser and not SQL Server - thanks for clearing out.
We are currently reviewing regressions and will get back to you soon on this. Welcoming any more suggestions/improvement areas for #538 .

Thanks. I've added my suggestions onto #538 directly.

Closing issue as PR #990 is now merged.


fix(core): deprecation warning with Python 3.7

DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working

Thanks @linkdd .  Please note that I no longer maintain this library.  If it is important to you then I encourage you to fork.

Thank you for the quick reply :)
I am indeed using it in a few projects, would you mind adding me as a collaborator on PyPI so I can push updates ?

I'll probably pass on handing out PyPI credentials until there is some
obvious established alternative.
Passing out PyPI credentials poses a security concern generally.  If you
end up maintaining a fork for a while and it becomes a standard though then
yes, I'd be happy to consider transferring ownership.
On Fri, May 24, 2019 at 8:54 AM David Delassus<EMAIL_ADDRESS>wrote:

Thank you for the quick reply :)
I am indeed using it in a few projects, would you mind adding me as a
collaborator on PyPI so I can push updates ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/mrocklin/unification/pull/3?email_source=notifications&email_token=AACKZTAGRBWJBVLCDAYS3VTPW7XQJA5CNFSM4HPPSOV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWFNU6I#issuecomment-495639161,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACKZTE2EEIBXGRMVBE62BDPW7XQJANCNFSM4HPPSOVQ
.

On the new PyPI you can add collaborators that can only push new releases, I would be using my own credentials.
See this page (only you has access) : https://pypi.org/manage/project/unification/collaboration/

Understood.   Sorry, I should be more clear.  While you seem nice I don't
actually know or trust you yet.  Giving unknown individuals push access to
software projects is a security concern.  (this situation is similar to one
of the major security breaches in the JS space earlier this year).
On Fri, May 24, 2019 at 9:01 AM David Delassus<EMAIL_ADDRESS>wrote:

On the new PyPI you can add collaborators that can only push new releases,
I would be using my own credentials.
See this page (only you has access) :
https://pypi.org/manage/project/unification/collaboration/
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/mrocklin/unification/pull/3?email_source=notifications&email_token=AACKZTFJE5LGNHO25N6RUUDPW7YNHA5CNFSM4HPPSOV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWFOPTI#issuecomment-495642573,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACKZTGHTFF6MUY7R7DJI6DPW7YNHANCNFSM4HPPSOVQ
.

Of course, I understand, I'll reference my fork in my Pipfile then :)
The library as it is is perfect for me, so it would be only bug fixes that I would push.


allow C++ as community name

Requested by @pawbyte on Twitter.
We currently disallow + in community names. They're using cplusplus. I tried UPDATEing under the hood to C++ but we actually 404 if the slug doesn't match the constraint.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/1363809-allow-c-as-community-name?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

Closed until communities (or tags) return.


incomplete pre-built Docker image deployment documentation?

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/828
Originally created by @dmor on 2019-07-24 16:58:09

When using https://www.meltano.com/docs/tutorial.html#using-docker the instructions say I should run docker pull meltano/meltano however when I do I get:
(Meltano) Danielles-MBP:Projects daniellemorrill$ docker pull meltano/meltano
Using default tag: latest
Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?

(Meltano) Danielles-MBP:Projects daniellemorrill$ which docker
/usr/local/bin/docker

I thought this might be because I ran the command from inside my virtual environment, so I gave it a try outside the vm but got the same error

View 4 previous comments from the original issue on GitLab


Rename function/method get_price()

In actor.py on branch main-workflow there is one function and one method get_price(). I would suggest to rename one of them.

As the method is doing nothing but calling the non-class function with an added option-parameter, I think this is fine, they have the same/common purpose and be named the same.


Access to the server object instance in callback methods

It would be really handy if I could access the instance of Teleport::Config::Server so I could do something like this...
before_recipes do
    # If we don't have the cron option on the server, just set it to an empty array
	if !@server.options.key?(:cron)
		@server.options[:cron] = []
	end
end

after_recipes do
	# Do something with @server.options[:cron] - perhaps install them into crontab
end

I'd like to have my recipes create cron entries, e.g.: mysql database backups but currently I can't see a nice integrated way to get shared data between the original server definition, all the recipes and then out at the end.
Hopefully that makes sense :)

Good idea. I'll try to add that.

Awesome :)


Remove Mapsui.Maui.sln

I fixed the Maui/Windows Project and it works now with Visual Studio 2022 (without Preview)

Pro Tip for running in Visual Studio 2022 and in Android Emulator, Hot Reload has to be disabled or else it won't start.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: inforithmics:x: Thomas StockerThomas Stocker seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I fixed the Maui/Windows Project and it works now with Visual Studio 2022 (without Preview)

Does this mean you fixed the problem you mentioned in your earlier comment? This one:

The mapsui.ui.Maui project is now in the Mapsui.sln This can be opened and compiled in Visual Studio 2022 But for execution and Testing there is still Visual Studio 2022 Preview needed.

Yes it works now with Visual Studio 2022 17.1 with disabled Hot Reload.


React as Global?

Why is React listed as a global here. I assume that it should be imported / required in each file?

This is simply to stop jshint complaining about React in the global namespace.

I understand what it does buy why are you expecting React to be in the global namespace?

Good point :P must have crept in at some point for some unknown reason. Happy to accept a pull request if you wanna contribute to the repos based on your questions raised.


A generic method cannot define an instance of its containing interface

[@jvasileff] The problem occurs when a method attempts to define an implementation of its interface, and the method has a parameter that involves a type parameter of the interface.
This:
interface Foo<T> {
    shared Foo<U> map<U>(T f)
        =>  object satisfies Foo<U> {};
}

fails with:
anonymous_0_ is not abstract and does not override abstract method <U>map(com.redhat.ceylon.compiler.java.runtime.model.TypeDescriptor,U) in Foo - Ceylon Backend Error

This does work:
interface Foo<T> given T satisfies Object {
    shared Foo<U> map<U>(Callable<U, [T]> f)
            given U satisfies Object
        =>  object satisfies Foo<U> {};
}

which is fairly common, but this slight variation doesn't:
interface Callable2<out Return, in Arguments>
        given Arguments satisfies Anything[] {}

interface Foo<T> given T satisfies Object {
    shared Foo<U> map<U>(Callable2<U, [T]> f)
            given U satisfies Object
        =>  object satisfies Foo<U> {};
}

[Migrated from ceylon/ceylon-compiler#2072]

Is there a workaround for this idiom?

@JohnWillow taking care not to re-use type parameter names, the code below compiles and is similar to the original example:
interface Foo<T> {
    shared Foo<U> map<U>(T f) => newMap<U, T>(this, f);
}

Foo<V> newMap<V, T>(Foo<T> othermap, T f)
    =>  object satisfies Foo<V> {};

@jvasileff I made it work. Thanks!

Pretty alike situation:
shared interface Foo satisfies Bar{}
shared interface Bar<Constraint> {
    shared default Foo<Type> constrained<Type, Args>()
            // given Args satisfies Constraint
            => object satisfies Foo<Type>{};
}

Does compile, just warns about Constraint and Args not being used.
But if you uncomment the satisfies line, the following error raises:
error: Ceylon backend error: anonymous_0_ is not abstract and does not override abstract method <Type,Args>constrained(TypeDescriptor,TypeDescriptor) in Bar
  where Type,Args,Constraint are type-variables:
    Type extends Object declared in method <Type,Args>constrained(TypeDescriptor,TypeDescriptor)
    Args extends Constraint declared in method <Type,Args>constrained(TypeDescriptor,TypeDescriptor)
    Constraint extends Object declared in interface Bar


ARC-136 remove csrfProtection middleware from jira/sync to resolve retry issue in Safari

This change stems from https://github.com/integrations/jira/issues/151.
Context
Currently, if a customer clicks ‘Submit’ from the GitHub configuration page (jira-configuration.hbs) in Safari to retry a normal or full sync, nothing appears to happen. There is no feedback in the UI and the customer has no idea what’s happened. The error logged to the console in production is Failed to load resource: the server responded with a status of 400 (Bad Request). In development it’s Failed to load resource: the server responded with a status of 403 (Forbidden) with an additional error thrown in the terminal by our web worker: ForbiddenError: invalid csrf token.
In March of 2020 apple updated Safari to block all 3rd party cookies. This means, when we make a request from a Jira site to a GitHub app, the request tries to send cookies from session storage that aren’t there.
Browser Comparison - Network Req and Stored Cookies
I’ve only included Chrome and Safari in this comparison but Firefox behaves the same way as Chrome.
Req Headers in Chrome vs Safari

Stored cookies for https://jira-integration-production.herokuapp.com  (this would be your ngrok/localtunnel APP_URL in dev - Chrome vs Safari

If I uncheck Safari’s website tracking for ‘Prevent cross-site traffic’, I see a cookie header in the req, as well as cookies in storage for http://jira-integration-production.herokuapp.com.

Where is this happening in our app?
This appears to be the only place in our app where we try to make a request from a Jira site, in my case https://rachellerathbone.atlassian.net/, with a cookie for http://jira-integration-production.herokuapp.com. Apart from the POST request to jira/sync all other requests made to the production heroku url are made from https://jira.github.com/: new tab opens, request is made from new tab, tab closes and returns user to the Jira site. The POST request to retry a sync is the only request that doesn’t follow the same pattern implemented everywhere else in the application.
If I make the same request from one of our other pages, I see a cookie header.

This request ultimately still fails but it fails in verifyJiraMiddleware not on an invalid csrf token.

It appears this issue happens when we pass csrfProtection, make an AJAX request with _csrf as a param, and the request is made from the customer’s Jira site, not from a page off  https://jira.github.com/
After much deliberation about the path forward to resolve this issue that was first raised 2.5 years ago we have decided to remove csrfProtection. It is not critical and we plan to follow up with a more thorough reimplementation once we have completed the migration of the app.

Closing in favour of https://github.com/integrations/jira/pull/440


unable to export data from file 800M size from one dbase to another...it is hanging up

Description
dbeaver failing to export data from one dbase to another.  Dbeaver ver 23.0.4.
out of memory error.
DBeaver Version
23.0.4
Operating System
Windows
Database and driver
postgresql 14.6
Steps to reproduce
No response
Additional context
No response

Hello @jgrover21,
Could you please provide the source table's DDL?
You can try increasing the maximum allowed memory DBeaver can use as a possible workaround. Please see https://github.com/dbeaver/dbeaver/wiki/Command-Line

It's been a while since no update here, so I'm closing this issue. If it's still actual, please let me know and provide additional information.


APIKEY_INVALID

I've tried everything possible:

node.bittrex.api
normal request pinging a url
Postman
creating a new account
Creating new keys

but I always end up getting this message:
results {"success":false,"message":"APIKEY_INVALID","result":null}
What am I doing wrong?

It turned out that I had an extra space in my apikey.
I found out by looking at the url that was being sent out.

This happens from time to time even with valid api keys. i believe this is an issue on Trex side and i have already called a ticket on them. hoping they sort this soon, it's surely breaking loads of apps.


preference parameter isn't read when asking for isochrones with Valhalla engine

Here's what I did and what I got
Asking for isochrones with default values
isochrones = api.isochrones(
    locations=(5.86659,45.26936),
    profile='bicycle',
    interval_type='time',
    intervals=[i * 60 for i in (20,50,80,110)],
)

and asking for isochrones using the preference attribute (but no options attribute)
isochrones_shortest = api.isochrones(
    locations=(5.86659,45.26936),
    profile='bicycle',
    interval_type='time',
    intervals=[i * 60 for i in (20,50,80,110)],
    preference="shortest",
)

both return the same result.
By adding dry_run=True we can see that the preference is not used :
url:
http://localhost:8002/isochrone
Parameters:
{
  "headers": {
    "User-Agent": "routingpy/v1.3.0",
    "Content-Type": "application/json"
  },
  "timeout": 60,
  "json": {
    "locations": [
      {
        "lon": 5.86659,
        "lat": 45.26936
      }
    ],
    "costing": "bicycle",
    "contours": [
      {
        "time": 20.0
      },
      {
        "time": 50.0
      },
      {
        "time": 80.0
      },
      {
        "time": 110.0
      }
    ]
  }
}

However, if you add any values in options, the preference is taken into account :
isochrones_shortest = api.isochrones(
    locations=(5.86659,45.26936),
    profile='bicycle',
    interval_type='time',
    intervals=[i * 60 for i in (20,50,80,110)],
    preference="shortest",
    options={"foo": 12},
    dry_run=True,
)

Output :
url:
http://localhost:8002/isochrone
Parameters:
{
  "headers": {
    "User-Agent": "routingpy/v1.3.0",
    "Content-Type": "application/json"
  },
  "timeout": 60,
  "json": {
    "locations": [
      {
        "lon": 5.86659,
        "lat": 45.26936
      }
    ],
    "costing": "bicycle",
    "contours": [
      {
        "time": 20.0
      },
      {
        "time": 50.0
      },
      {
        "time": 80.0
      },
      {
        "time": 110.0
      }
    ],
    "costing_options": {
      "bicycle": {
        "foo": 12,
        "shortest": true
      }
    }
  }
}

Here's what I was expecting
I was expecting the  preference parameter to be taken into account, as per the documentation, even if options is None.

Here's what I think could be improved
After looking at the code and according to my understanding, in isochrones with Valhalla engine, the preference parameter is only read if the options parameter is not None. See https://github.com/gis-ops/routingpy/blob/e5eb103bac330cefdb54aae23deb30f408815911/routingpy/routers/valhalla.py#L441
A possible fix could be to do as with directions, and replace if options by if options or preference on line 441 ; cf. https://github.com/gis-ops/routingpy/blob/e5eb103bac330cefdb54aae23deb30f408815911/routingpy/routers/valhalla.py#L232

Oops, yeah you're absolutely right. Do you time for a PR?

probably worth to look at the isochrones as well.

Sure, will do !


Does xlink:href work on 'a tag'?

I can add an 'a tag' to the rectangle element and I can add a link to plain text using r2d3 but for some reason I cannot get it to add a link to the 'a tag'.
I created a html file so that someone could download it and test it in ie8. https://dl.dropboxusercontent.com/u/23726217/test.html

Hi there,
This project is no longer being maintained so this issue is being closed. Thank you for using r2d3 and taking the time to report this issue.


fix: allow user defined password in pooler url

What kind of change does this PR introduce?
Bug fix, feature, docs update, ...
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-3 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.04%) to 58.759%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/utils/connect.go
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

internal/utils/connect.go
1
84.4%

internal/gen/keys/keys.go
5
11.76%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
6135

Relevant Lines:
10441

💛  - Coveralls

not secure. workaround is to pass in via stdin or env var.


fix the command output

fix the command output

This change is

LGTM thanks! (Travis build is broken but that's unrelated to this PR)

@chenhuan12 can you rebase this PR?
Context: The Jenkins build is working again (mungedocs is a dependency that was moved into this docs repo), but you need to have the latest master for it to be able to run properly


Data Loading Error

When I try to run the code provided here, I end up hitting a RuntimeError here: https://github.com/tkipf/c-swm/blob/e944b24bcaa42d9ee847f30163437a50f0237aa0/train.py#L104 when running your Shapes-2D build/train functions: python train.py --dataset data/shapes_train.h5 --encoder small --name shapes
Specifically, the error says: RuntimeError: DataLoader worker is killed by signal: Killed. It seems to be coming from the fact that the dataloader is having trouble multi-processing the training set loading. But when I look through your utils files, I am not seeing why this error would exist.
This same error has occurred on both a windows machine as well as a linux machine.
Here's the full trace:
Traceback (most recent call last):
  File "/home/aadharna/miniconda3/envs/cswm/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 724, in _try_get_data
    data = self.data_queue.get(timeout=timeout)
  File "/home/aadharna/miniconda3/envs/cswm/lib/python3.7/multiprocessing/queues.py", line 104, in get
    if not self._poll(timeout):
  File "/home/aadharna/miniconda3/envs/cswm/lib/python3.7/multiprocessing/connection.py", line 257, in poll
    return self._poll(timeout)
  File "/home/aadharna/miniconda3/envs/cswm/lib/python3.7/multiprocessing/connection.py", line 414, in _poll
    r = wait([self], timeout)
  File "/home/aadharna/miniconda3/envs/cswm/lib/python3.7/multiprocessing/connection.py", line 921, in wait
    ready = selector.select(timeout)
  File "/home/aadharna/miniconda3/envs/cswm/lib/python3.7/selectors.py", line 415, in select
    fd_event_list = self._selector.poll(timeout)
  File "/home/aadharna/miniconda3/envs/cswm/lib/python3.7/site-packages/torch/utils/data/_utils/signal_handling.py", line 66, in handler
    _error_if_any_worker_fails()
RuntimeError: DataLoader worker (pid 19014) is killed by signal: Killed.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "train.py", line 104, in <module>
    obs = train_loader.__iter__().next()[0]
  File "/home/aadharna/miniconda3/envs/cswm/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 804, in __next__
    idx, data = self._get_data()
  File "/home/aadharna/miniconda3/envs/cswm/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 771, in _get_data
    success, data = self._try_get_data()
  File "/home/aadharna/miniconda3/envs/cswm/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 737, in _try_get_data
    raise RuntimeError('DataLoader worker (pid(s) {}) exited unexpectedly'.format(pids_str))
RuntimeError: DataLoader worker (pid(s) 19012, 19014) exited unexpectedly
(cswm) aadharna@penguin:~/PycharmProjects/c-swm$

I'll continue poking around and update when I find the root cause.

Same situation, maybe the RAM is too small to run the train?


Scroll improvements

Add scrolling tests. Unfortunately I couldn't figure out a way to make these work in PhantomJS 2.0 (it seems scrolling iframes worked in 1.9, but our test suite requires 2.0)

Don't scroll to top on partial replacement (https://github.com/rails/turbolinks/pull/291#issuecomment-98488300 cc @drewhamlett)

Don't restore scroll position on transition cache hit (which IMO is the expected behavior on a Turbolinks.visit() / navigation click)

@reed: works for you?

@Thibaut i tried it and i noticed:
If i use link_to with remote: true,  it restores scroll position but progress bar is not showing.
If i use normal link_to, progress bar is working, but scroll position is not restored.
In controller i have:
render :some_view, change: :some_element_id
Did i miss something?

Please somebody merge this.

Refactored and added Turbolinks.visit(url, scroll: false) to opt out of the default "scroll to top" behavior. In a future PR this could be enabled on a link-per-link basis using <a href="url" data-turbolinks-scroll="false">.
@kaspth Let me know what you think.
PS: no doc for now. I need to revamp the README first (tomorrow or next week-end).

@kaspth @rafaelfranca good to :ship:?

@Thibaut lovely, :smile:.

:shipit:


Grouping event properties by event rather than by property name.

Describe the solution you'd like
We are currently tracking data sync times using AppCenter Analytics. Each sync event contains the following properties in the dictionary. Table being synced, time it took that table to sync and the user's id who initiated the sync.
When we go to view the data in App Center all of the sync table names are grouped together and all of the sync times are grouped together and there is no way to correlate the data.
For instance if we see a long sync time there is no way to tell which user and for which table that long sync time belongs to.
Describe alternatives you've considered
Not sure if this is able to be correlated in Analytic Insights
In the image ... which TimeSpan belongs to which Sub Process?

Thanks for pointing this out. We have made a note of this request added this in our backlog.


  When I entered the amount in the UI, the transaction that was created was for 0.XXXX0999.  I noticed it before broadcasting, so recreated it, this time I entered 0.XXXX1, but I realised the same thing happened again when I received a notification from the payments portal (shown below):

We have received only 0.XXXX0999 of the 0.XXXX1000 BTC ... leaving a remaining balance of 0.00000001 BTC. This can happen if you forgot to account for the coin TX fee when sending to us or if you are using a coin that only allows 6 digits after the decimal and didn't round up to the 6th digit.

I had to make an additional 600 sats transaction (because of dust limit) to pay my invoice completely.
I can see the transaction fee was added separately, so I'm wondering what went wrong?  I don't do a lot of transactions, so I'm not sure if I did everything correctly.
PS: If needed, I can share the Tx privately.

Hi @craigraw, sorry about the delay, busy couple of weeks.
I tried to compile from master, but ran into some java version issues I think (I used the latest from the Fedora repos, openjdk 17).  But then I noticed there was a new release, so I tried the 1.5.2 release, and the issue is fixed!  Thanks for the prompt resolution :)

Great, thanks for reporting back!


Revamp 2

Revamp -2 for Android SDK
@aaustin @EvangelosG @TheMarshal99

PR review ready
@aaustin @EvangelosG @Sarkar

so awesome 👍 Can you update the change log with migration suggestions as well?


Why is "remark-validate-links" repeated in the output?

Is this intentional?

Oh, yeah, that’s intentional. Most projects emit different kinds of stuff, so it’s useful to note both the plugin (called source) and category (ruleId). This project only has one “category”, so ruleId and source are the same.
So, this is kind-off intentional. Do you have a suggestion for an alternative?
P.S. thought that just sprung to mind: we can also differentiate between non-existing file warnings and non-existing headings with the ruleId?

Oh, this makes sense. I don't think it's clear to the user, though, when they are the same — certainly makes sense when they differ.
What do you think of having only one of them print if they are the same? I understand if not, and we can close this issue.

P.S. thought that just sprung to mind: we can also differentiate between non-existing file warnings and non-existing headings with the ruleId?

This is a fine idea 👍

Yeah you’re right. Do you feel it’s very important to do this soon? I think it should be in a major release, and I’m fine with that, but I can also bundle it with the next major release (although that could be in a while).

Bundling it with a major release seems fine to me. No hurry!

Should be published in a few minutes!


Tray icon on top of borderless windowed game

Anyway I can solve this? When I launch the game I have the whole bar on top of it, if I switch desktop and go back to the game it's gone but the tray area stays. I'm using openbox
edit: it happens in fullscreen mode as well
polybar 3.1.0-git
[bar/obox]
monitor = HDMI-0
bottom = true
fixed-center = true
width = 100%
height = 40
offset-x = 0
offset-y = 0
background = ${var.background}
foreground = ${var.foreground}
radius = 0.0
line-size = 0
border-size = 0
border-color = ${var.border}
padding = 2
module-margin-right = 2
font-0 = DejavuSansMono:size=10;4
font-1 = NotoSansCJKJP:size=10;4
font-2 = FontAwesome:size=10;4
modules-left = ewmh nyaa mpd
modules-center =
modules-right = keyboard volume date
dim-value = 1.0
override-redirect = false
tray-position = right
tray-maxsize = 16
tray-offset-x = 0
tray-offset-y = 0
tray-padding = 1
tray-scale = 1.0
tray-background = ${var.background-light}

Seems like a lot of people have that issue. It has already been reported in #425. Can you try out the PR #1203 and check if polybar now prints any errors when you go to fullscreen.
Please report your findings over at #425 (also post the full polybar output).
There is no fix for this right now (except for switching window managers)
I am going to close this, the issue is going to be handled over at #425.


Workflow + Engine - Add ability to change TaskRun Timeout as user/team

Is your request related to a problem? Please describe.
There is a setting for Flow administrator to change the Timeout of TaskRun, which is now by default 180 minutes. If user's flow needs longer than 180 minutes to complete, he has to ask Administrator to change it.
Describe the solution you'd like
Allow users to customize the Timeout of TaskRun.
Describe the benefits or justification for this request
Users can help themselves adjust the timeout, without asking the administrator.

Hi @digglife its a great question. We have another enhancement that talks about advanced configuration for tasks where you could specify time outs or no network access or secrets and more.
But w.r.t. timeout would be a great addition but does pose security concerns in that you wouldn't want a user to set a task with 24hour timeouts as that could mess up the cluster if too many were run with that sort of timeout.
What would probably need to happen is a quota introduced for timeout, with a default specified timeout. And then a user could adjust within the quota. OR maybe the workflow timeout overall will kill any tasks that exceed the whole workflow timeout.

@BenjaminRuby to check with @gchickma on the importance for IBM in this

This is available in v4 both at a WorkflowRun and TaskRun level.
Needs to be added to the UI to make available. Tagged for Flow 4.0.0

@timrbula we need to add the timeout field to the Workflow / Tasks. Considering right now in the UI its hard to edit each task's configuration... for now we should add to the Workflows config screen
Future state might be to add a tab to the Tasks modal or move that to a right hand panel

This has been implemented for the Workflow and is linked to Quotas.
Need to implement specific settings per task somehow

Implemented via the API
Could be implemented a couple of different ways in the UI

Part of the tasks edit modal with tabs
New modal with a configure / settings button added to the draggable task
Only on the code editor view for the Workflow


Add shorthand method for creating a Color from RGB components

Add the static method decided on in #41.

Thanks!


update github actions

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

Project coverage is 65.53%. Comparing base (f9320c0) to head (e6c8995).
Report is 3 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #159      +/-   ##
==========================================
- Coverage   70.82%   65.53%   -5.30%
==========================================
  Files          39       38       -1
  Lines        1714     1703      -11
  Branches      379      276     -103
==========================================
- Hits         1214     1116      -98
- Misses        426      506      +80
- Partials       74       81       +7

Flag
Coverage Δ

unittests
65.53% <ø> (-5.07%)
:arrow_down:

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

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


How to use  the "MSAL Angular 2.X sample" with AD B2C?

Library

[x] @azure/msal-angular^2.0.0-alpha.0
[x]<EMAIL_ADDRESS>
Description
I just downloaded this sample.
It runs fine "as is".  However, I am having trouble getting the sample to work with my own AD B2C tenant.
The login/logout functionality works fine, but when I try to get to the profile page, I get the following error:
ERROR ServerError: invalid_request: AADB2C90117: The scope 'user.read' provided in the request is not supported.
I see the following in the console:
app.module.ts:39 MSAL logging:  [Fri, 13 Nov 2020 21:05:02 GMT] :  : 1.7.2 : Info - Emitting event: msal:acquireTokenFailure
app.module.ts:39 MSAL logging:  [Fri, 13 Nov 2020 21:05:02 GMT] :  : 1.7.2 : Info - Emitting event: msal:acquireTokenStart
app.module.ts:39 MSAL logging:  [Fri, 13 Nov 2020 21:05:03 GMT] :  : 1.7.2 : Info - Emitting event: msal:acquireTokenFailure
core.js:4197 ERROR ServerError: invalid_request: AADB2C90117: The scope 'user.read' provided in the request is not supported.
Correlation ID: 4c9aeeef-7fe5-4f5c-a12b-abf69b5bb2af
Timestamp: 2020-11-13 21:05:03Z

    at ServerError.AuthError [as constructor] (http://localhost:4200/vendor.js:92152:24)
    at new ServerError (http://localhost:4200/vendor.js:95255:28)
    at ResponseHandler.push.u8tD.ResponseHandler.validateServerAuthorizationCodeResponse (http://localhost:4200/vendor.js:96427:19)
    at AuthorizationCodeClient.push.u8tD.AuthorizationCodeClient.handleFragmentResponse (http://localhost:4200/vendor.js:96715:25)
    at PopupHandler.<anonymous> (http://localhost:4200/vendor.js:100448:52)
    at step (http://localhost:4200/vendor.js:92016:23)
    at Object.next (http://localhost:4200/vendor.js:91997:53)
    at http://localhost:4200/vendor.js:91990:71
    at new ZoneAwarePromise (http://localhost:4200/polyfills.js:10142:33)
    at __awaiter (http://localhost:4200/vendor.js:91986:12)

Any suggestions on how to fix this?

@hhubik user.read is a MS Graph scope, and will not work with B2C. You will need to create a custom scope in your app registration, and then use it against your own API. You can see this done in our MSAL Angular v1 B2C sample: https://github.com/Azure-Samples/active-directory-b2c-javascript-angular-spa
We'll be sure to add an MSAL Angular v2 B2C sample. cc: @jo-arroyo @derisen

@jasonnutter thanks for the suggestion.  If I use my own API, what is the best way to get the user's name (or display name)?  Do I need to call MS graph from my API?  I use Azure Functions for my APIs.  Does what you said above about the custom scope apply to Azure functions as well?

Upon  a closer look, it looks like I can get the basic info (including the oid and the name) of the authenticated user from MSAL using the getAccounts() method.  I would still appreciate a sample that shows how to protect an API implemented as an Azure Function.


Fix PCRE stack limit for context parsing, omitting preg

#369
Fixes issue with silent preg_match_all failure on parsing of long JSON contexts from log text, avoiding JIT Stack Limit due to recursion.

Composer test passes, testing in Laravel project using log-viewer successful.

@essell looks great! could not find a way to break it. Shipping it 🚢

@essell tagged v3.9.0


Warning when get_batch can be very slow

When entering loop 97, if len(observations) is not much larger than batch, it is extremely slow (minutes), due to the way the algorithm works. This was very confusing to me so I thought it might be valuable to add a warning, although I'm not sure if it's needed and I'm not sure if this is the best way. Happy to go with what you think.
This can wait until the optimiser update, cheers.;

Looks like I was wrong about this, it's just a search algorithm that scales poorly

It is indeed not a particularly fast implementation, but a naive linear search version where one really could use binary search, a priority queue, or so. We might optimize that at some point, but it definitely won't happen soon. If you're keen to look into it, a PR would be very welcome... :+1:

Fair enougth, it's new and not many people have implemented it. Optimisation is a bit much to expect!
I have a version where I used numpy linear search, it's 10x faster and scales a little better. I'll make it a PR and let you guys decide if that small improvement is worth merging.

This would be welcome, although we would have to re-implement this in TensorFlow soon


Feature/okn 1781

Replace to single quote where twig embeds are to avoid warnings
Remove header organism which only contained an svg logo

Maybe change the includes to use single quotes as well?

Maybe change the includes to use single quotes as well?

Good point, the application doesn't warning when there is an include with double quotes, but will change that too :)

Supa kewl!


Update appfilter.xml

Description
Adding icons
Type of change

adding new icons: wechat, alipay, vimusic

Icons added

Wechat (com.tencent.mm)
ViMusic (it.vfsfitvnm.vimusic)
Alipay (com.eg.android.AlipayGphone)

I'll be closing this PR as it just adds an already present link.


[BUG] Text Elements are not displayed - Canary Version

Describe the bug

Text Elements are not displayed in frontend,

Steps to reproduce:

open shopping experiences
choose any block with text element
check the frontend

Admin Area

Frontend Area
Text Elements are not visible, just the image elements

Hello @jonaletil ! Thanks for your issue. Problably we fxed t via FIX-1065. Can You checked this issue again?

We are closing the issue because it was fixed and no further issue related to this were reported :)


Final modifications to make netlify dev work with workshops

@BrianMMcClain this looks all good to me but I was hoping for one more set of eyes. Could you give this one last look and test in the deploy preview then review and merge?

@BrianMMcClain this looks all good to me but I was hoping for one more set of eyes. Could you give this one last look and test in the deploy preview then review and merge?

Just to be clear, did you mean test locally? The deploy preview isn't working, but that's normal for workshops in deploy previews, I get the following trying to launch a workshop:
{"error":"Cannot read property 'search' of undefined"}

However, locally it is working correctly.


add error trace in log service

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

...

New functionality

...

Test plan
How are these changes tested?

[ ] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs repository?

#2548  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sanketkedia and the rest of your teammates on  Graphite


Refresh token in logs is disabled for Korifi

Does this PR modify CLI v6, CLI v7, or CLI v8?
CLI v8. A corresponding PR is open for the v8 branch with #2294.
Description of the Change
Disables refresh token call for the cf logs command when targeting Korifi (CF on K8s). We are not using UAA for AuthN/Z, so this call was resulting in errors.
Why Is This PR Valuable?
This will allow users of Korifi to use the cf logs command without causing an error on the refresh token call.
Why Should This Be In Core?
This change extends previous core functionality to allow targeting of Korifi (CF on K8s) clusters.
Applicable Issues
[Feature]: CF CLI user can skip oauth/token requests on Korifi
How Urgent Is The Change?
This PR to update master is lower priority than #2294.
Other Relevant Parties
Korifi users.

Requested review from cli team as i no longer have cli context

Closing and moving conversation to https://github.com/cloudfoundry/cli/pull/2294.
If this is mento to go to v8 then we first review the pr to v8 and then cherry-pick


Columnstore Index Maintenance Not Working

Description of the issue
We built new Non-Clustered Columnstore indexes on the database and ran the following to check if they are being pulled into the index maintenance and no not see them. I sanitized the database name in the code below.
SQL Server version and edition
SQL 2019 - Version 15.0.4033.1 - Developer
Version of the script
2020-01-26 14:06:53
What command are you executing?
EXECUTE dbo.IndexOptimize @Databases = 'Database'
, @Indexes = 'Database.dbo.Orders.NCCI_Orders'
, @FragmentationLow = 'INDEX_REORGANIZE,INDEX_REBUILD_ONLINE,INDEX_REBUILD_OFFLINE'
, @FragmentationMedium = 'INDEX_REORGANIZE,INDEX_REBUILD_ONLINE,INDEX_REBUILD_OFFLINE'
, @FragmentationHigh = 'INDEX_REORGANIZE,INDEX_REBUILD_ONLINE,INDEX_REBUILD_OFFLINE'
, @FragmentationLevel1 = 10
, @FragmentationLevel2 = 30
, @MinNumberOfPages = 1
, @LogToTable = 'Y'
, @Execute = 'N'
What output are you getting?
Date and time: 2020-06-19 16:05:10
Server: Server
Version: 15.0.4033.1
Edition: Developer Edition (64-bit)
Platform: Windows
Procedure: [Database].[dbo].[IndexOptimize]
Parameters: @Databases = 'Database', @FragmentationLow = 'INDEX_REORGANIZE,INDEX_REBUILD_ONLINE,INDEX_REBUILD_OFFLINE', @FragmentationMedium = 'INDEX_REORGANIZE,INDEX_REBUILD_ONLINE,INDEX_REBUILD_OFFLINE', @FragmentationHigh = 'INDEX_REORGANIZE,INDEX_REBUILD_ONLINE,INDEX_REBUILD_OFFLINE', @FragmentationLevel1 = 10, @FragmentationLevel2 = 30, @MinNumberOfPages = 1, @MaxNumberOfPages = NULL, @SortInTempdb = 'N', @MaxDOP = NULL, @FillFactor = NULL, @PadIndex = NULL, @LOBCompaction = 'Y', @UpdateStatistics = NULL, @OnlyModifiedStatistics = 'N', @StatisticsModificationLevel = NULL, @StatisticsSample = NULL, @StatisticsResample = 'N', @PartitionLevel = 'Y', @MSShippedObjects = 'N', @Indexes = 'Database.dbo.Orders.NCCI_Orders', @TimeLimit = NULL, @Delay = NULL, @WaitAtLowPriorityMaxDuration = NULL, @WaitAtLowPriorityAbortAfterWait = NULL, @Resumable = 'N', @AvailabilityGroups = NULL, @LockTimeout = NULL, @LockMessageSeverity = 16, @StringDelimiter = ',', @DatabaseOrder = NULL, @DatabasesInParallel = 'N', @LogToTable = 'Y', @Execute = 'N'
Version: 2020-01-26 14:06:53
Source: https://ola.hallengren.com
Date and time: 2020-06-19 16:05:10
Database: [Database]
State: ONLINE
Standby: No
Updateability: READ_WRITE
User access: MULTI_USER
Recovery model: SIMPLE
Is accessible: Yes
Date and time: 2020-06-19 16:05:15
Completion time: 2020-06-19T16:05:15.0511110-05:00

OMG, I dug deep into the code and found that on line 1971 there is a reference to @MinNumberOfPages = 0 and it turns out that Columnstores show zero pages used!  So everything is working as it should, but we don't have any documentation anywhere that says to use @MinNumberOfPages = 0 in order to maintain columnstores in a separate job (still need to use INDEX_REORGANIZE in order for them to be maintained too).

don't suppose there is any plan to update the code to address this scenario. We just came across the same problem. Luckly we're using clustered so we can try this workaround but ideally we'd not need to do this for making that work.

The premise is wrong: https://docs.microsoft.com/en-us/sql/relational-databases/system-dynamic-management-views/sys-dm-db-column-store-row-group-physical-stats-transact-sql?view=sql-server-ver15

I have the same problem, I have dozens of indexes showing 50-98% fragmentation, and all are columnstore indexes. I have tried tweaking parameters down to 1% fragmentation level and 1 minimum number of pages. I cannot get the script to touch these indexes. Our application performance is awful.

The detail @christillman references is correct. It's a little confusing as this SQL leads to different percentages than what SSMS shows users in the UI.
A while back I built out a new proc using the fragmentation calculation from that documentation link. We now have 2 jobs scheduled, the original indexoptimize and my new proc which I tied into the same logging and execution process this solution uses. For us it's enough but you have to handle columnstore outside of this. At least for now. I'm not sure they are even planning to address this given the previous comments.

This was also updated recently... https://docs.microsoft.com/en-us/sql/relational-databases/indexes/reorganize-and-rebuild-indexes?view=sql-server-ver15
"Starting with SQL Server 2019 (15.x), Azure SQL Database, and Azure SQL Managed Instance, the tuple-mover is helped by a background merge task that automatically compresses smaller open delta rowgroups that have existed for some time as determined by an internal threshold, or merges compressed rowgroups from where a large number of rows has been deleted. This improves the columnstore index quality over time. For most cases this dismisses the need for issuing ALTER INDEX ... REORGANIZE commands."


Extensions don't stay enabled

Describe the bug
If you enable an extension then restart Lens, the extension is disabled.
To Reproduce
Steps to reproduce the behavior:

Enable an extension (tried @jakolehm/lens-extension-gke-sync and @nevalla/kube-resource-map)
Verify it works
Restart Lens
Observe that the extension is disabled

Environment (please complete the following information):

Lens Version: 2023.5.251645-alpha
OS: Linux (AppImage), MacOS

If the lens-extensions.json file looks like this before starting Lens:
{
	"__internal__": {
		"migrations": {
			"version": "1.0.0"
		}
	},
	"extensions": [
		[
<EMAIL_ADDRESS>			{
				"enabled": true,
				"name": "@andrea-falco/lens-multi-pod-logs"
			}
		]
	]
}

It looks like this right after Lens has started:
{
	"__internal__": {
		"migrations": {
			"version": "1.0.0"
		}
	},
	"extensions": [
		[
			"0",
			[
<EMAIL_ADDRESS>				{
					"enabled": true,
					"name": "@andrea-falco/lens-multi-pod-logs"
				}
			]
		],
		[
<EMAIL_ADDRESS>			{
				"enabled": false,
				"name": "@andrea-falco/lens-multi-pod-logs"
			}
		]
	]
}

This would explain why the extension is not enabled after restarting Lens. I have seen this before, do you remember what the problem is @Nokel81 ?

Looks likes a call to .entries() are being serialized instead or something.
Maybe making a unit test with that initial state and expecting that after the application starts up that the state is the same could help here.

@Nokel81  I think this is the culprit. Is this migration just to catch when extensions is not an object? If it is an array we probably can do nothing(?)

Also, does the migration key really not change the version before and after?

It does change. That was copy-paste chicanery on my part, I edited my lens-extensions.json file to have just one extension in it, missed the version change.

Okay good

I believe that the intended behaviour was to convert between an object based storage and an entry based one since the backing type changed from observable.object() to observable.array().

Ok. I'm going to double check this because I see the migration version changing for a dev run but it appears the alphas are not changing it(?) On my windows machine, latest alpha, after testing with it and previous alpha:
{
	"__internal__": {
		"migrations": {
			"version": "1.0.0"
		}
	},
	"extensions": [
		[
			"0",
			[
				"0",
				[
					"0",
					[
						"0",
						[
							"0",
							[
								"0",
								[
									"0",
									[
										"0",
										[
											"0",
											[
												"0",
												[
													"0",
													[
														"C:\\Users\\jiim\\AppData\\Roaming\\Lens\\node_modules\\cloud-provider-sync-extension\\package.json",
														{
															"enabled": false,
															"name": "cloud-provider-sync-extension"
														}
													]
												]
											]
										]
									]
								]
							]
						]
					]
				]
			]
		],
		[
			"1",
			[
				"1",
				[
					"1",
					[
						"1",
						[
							"1",
							[
								"1",
								[
									"1",
									[
										"1",
										[
											"1",
											[
												"1",
												[
													"1",
													[
														"C:\\Users\\jiim\\AppData\\Roaming\\Lens\\node_modules\\lens-certificate-info\\package.json",
														{
															"enabled": false,
															"name": "lens-certificate-info"
														}
													]
												]
											]
										]
									]
								]
							]
						]
					]
				]
			]
		],
.
.
.

Maybe this is caused by me going from latest to alpha to latest...

Hmmm
https://github.com/lensapp/lens/assets/40840436/a6c9057e-d3b6-4f1e-b0dd-2f74fae605b7

Oh I see, that is the single store the now uses a new method of calculating the store version number, which is based on the declared migrations.
But migrations are only declared on main so I guess it defaults to the version of 1.0.0 on renderer. I think the proper fix would be to not read from the file system at all on renderer and instead retrieve the initial state from main.


MusicPlayerGui

Added a Music Player Gui with the help of python

Issue #71
Plz consider the PR

Kindly edit the source code properly.


feat(object store): introduce object prefix for opendal object store

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
This PR introduce a new system params use_new_object_prefix_strategy to control whether split object prefix. In general,

if it's a new cluster, the use_new_object_prefix_strategy is set to true and all opendal object store will split prefix;
if it's an old cluster, the use_new_object_prefix_strategy is set to false and everything keep the same with before.

Thus, we can guarantee backward compatibility.
Once an origin customer find some issue with the performance and want to split prefix for his old cluster, it's required to execute a script to completely rename the data in the object store and modify the system params in some way.
So in a word, this pr will not affect the behavior of an already exist cluster.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

PTAL❤cc @hzxa21 @Li0k

Backward compatibility tests pass.
**

**


 Latest beta.
import {Entity, IEntity, PrimaryKey, Property} from "mikro-orm";

@Entity()
export class Person {
	@PrimaryKey()
	id: string;

	@Property({columnType: 'varchar', nullable: true, default: null})
	displayName: string;
	@Property({columnType: 'varchar', nullable: false})
	email: string;
	@Property({columnType: 'varchar', nullable: true, default: null})
	photoURL: string;

	constructor(person: Partial<Person>) {
		this.id = person.id!;
		this.displayName = person.displayName!;
		this.email = person.email!;
		this.photoURL = person.photoURL!;
	}
}

export interface Person extends IEntity<string> {
}

	beforeEach(async () => {
		createdBy = new Person({id: PERSON_ID, email: 'email address'});
		await orm.em.persistAndFlush(createdBy);
	});

ValidationError: Trying to set Person.displayName of type 'string' to 'undefined' of type 'undefined'
    at Function.fromWrongPropertyType (node_modules/mikro-orm/dist/utils/ValidationError.js:18:16)
    at EntityValidator.validateProperty (node_modules/mikro-orm/dist/entity/EntityValidator.js:36:43)
    at /node_modules/mikro-orm/dist/entity/EntityValidator.js:21:49
    at Array.forEach (<anonymous>)
    at EntityValidator.validate (node_modules/mikro-orm/dist/entity/EntityValidator.js:16:30)
    at ChangeSetComputer.computeChangeSet (node_modules/mikro-orm/dist/unit-of-work/ChangeSetComputer.js:20:24)
    at UnitOfWork.findNewEntities (node_modules/mikro-orm/dist/unit-of-work/UnitOfWork.js:149:50)
    at UnitOfWork.computeChangeSets (node_modules/mikro-orm/dist/unit-of-work/UnitOfWork.js:117:18)
    at UnitOfWork.commit (node_modules/mikro-orm/dist/unit-of-work/UnitOfWork.js:75:14)
    at EntityManager.flush (node_modules/mikro-orm/dist/EntityManager.js:210:36)
    at EntityManager.persistAndFlush (node_modules/mikro-orm/dist/EntityManager.js:177:20)
    at Context.<anonymous> (test/db/index.spec.ts:23:16)
    at processImmediate (internal/timers.js:439:21)
    at process.topLevelDomainCallback (domain.js:131:23)

This is actually the same as #132 I think, I just fixed it in the master and rebased to dev, but didn't publish new version.
Please try v3.0.0-alpha.8, that includes the fix.

Fixed! Thanks!