EventList.append() race condition fix

Moved appending to EventArray._data to end of block appending method to avoid race condition (#32), fixing issue in enDAQ Lab (https://github.com/MideTechnology/SlamStickLab/issues/329).
A couple other instance variable changes also moved to the end.

Sorry, I had thought the tests passed before I created the PR. The failing tests seem related to changes in PR #13, and since issue #31 proposes rolling those back, I'm not sure where the fix should go.

The failing tests seem related to changes in PR #13, and since issue #31 proposes rolling those back, I'm not sure where the fix should go.

I would ignore the fact that the #13 tests caught this, because the bug looks unrelated to the application changes in #13.
Specifically, the failing test cases use start=None, end=None, step=None when calling arrayValues & arrayJitterySlice (this is indicated in the test report on lines 238/242 and 271/275, respectively). And just from inspection the changes in #13 are only run when start/end/step are non-None. So the "bug" caught here I think is not a product of the application changes in #13; it just coincidentally seemed to catch some bad behavior from this PR.
I think the fixes to the behavior in this PR are fine to go here. Though this makes me want to keep the added tests from #13; I'll add that as a comment there.
...did ^this answer your question? I'm suspicious that maybe I misunderstood what you were saying...

aaaand you fixed it... cool.


StrictRedirectUriValidatorAppAuth does not seem to work for mvc.code

We can only help you if you are on the latest version. Either use the latest 2.x or 3.x
Please only use the issue tracker for bug reports and/or feature requests. For general security questions, or free or commercial support options do not use the issue tracker and instead see here for more details.
For bug reports,  include the relevant log files related to your issue. See here how to enable logging. Delete this line once you have.
Finally, please keep the issue concise and to the point. If you paste in more code than the text for the issue you are reporting then we will most likely not read it.
Issue / Steps to reproduce the problem

Clone IS4 Repo
build.ps1
Start IS4 Host
Edit mvc.code launchsettings to bind to http://<IP_ADDRESS>:whatever instead of http://localhost:44392
Start mvc.code client
Login fails.

Relevant parts of the log file
[11:35:14 Information] Microsoft.AspNetCore.Authentication.Cookies.CookieAuthenticationHandler
idsrv was not authenticated.  Failure message: Unprotect ticket failed

[11:35:15 Information] Microsoft.AspNetCore.Authentication.Cookies.CookieAuthenticationHandler
idsrv was not authenticated. Failure message: Unprotect ticket failed

[11:35:15 Information] Serilog.AspNetCore.RequestLoggingMiddleware
HTTP GET /home/error?errorId=CfDJ8Li6NyFeAl1CoW7SuCNduCN7qFJbtqCQg4FSoXtKqITBdyps30rTLsAhpERHdD_mD4AIqbpnm4N4anUsvbUEMSA7_C6piSkuaq68f0Luxz0h2tK0_LHbbL9RMmXiwzKzhsLP7Bl6yjX7tKnnAv3jnql9Jndxitkov3r5sSrbPEJunvs0s-4q8em6295mdlA3BOBEzPxfQVVYbXrB2_qjJefeIbgRkYkHDzgqpqgWKVRORHqrNjiwB2S7yJJ1M4bwdbvf2jFPLD_5KrMAWNJz2MlysLDSf8w6tn9jG7rJe-KTVU-B4bB9TsPxkPxQoIaiWjlMBaouJE7HXSYEiKfzCaY responded 200 in 368.8876 ms

After [11:35:14 Debug] IdentityServer4.Stores.ValidatingClientStore client configuration validation for client mvc.code succeeded. I would expect to see WashuIdentityServer-Local [10:32:21 IdentityServer4.Validation.StrictRedirectUriValidatorAppAuth [Debug] Checking for <IP_ADDRESS> redirect URI But I never see this.
launch ConsoleHybridWithPkce and it works flawlessly.
Additional Information
https://github.com/IdentityServer/IdentityServer4/blob/master/src/IdentityServer4/src/Validation/Default/StrictRedirectUriValidatorAppAuth.cs#L87-L90
This fails because parts[2] is 12345/oidc-signin so int.TryParse will not succeed. Is this desired? If so, this is not very explicit.

Is this a real problem - or just hypothetical?

@leastprivilege ? this is a real problem. You do not get the benefit of allowing any <IP_ADDRESS> redirect urls with a random port if the redirect url has a path like http://<IP_ADDRESS>:12345/signin-oidc. So right now this only works for the console app example.

it's for RFC8252. You can disable it.

Ideally I would keep it if it worked as I really like the idea of being able to create clients but not have to maintain a list of localhost or <IP_ADDRESS> redirect urls for developers.

it was built for a different use-case - propose a PR once you understand the above RFC.

Sounds good. I just read the whole RFC, section 7.3 shows a path is supported. I will try and get a PR together.

Why don't you create your own validator for your use case?

I saw that could be easily done but I figured this was a shortcoming off the provided StrictRedirectUriValidatorAppAuth and other people would benefit from the fix of allowing paths in the redirect URI. I still may do that, I wanted to also research and make sure I wasn't opening any security holes for my use case of not wanting to maintain developer localhost redirect URIs.

Bump.

Not forgotten. Just other priorities right now

Stalebot: Replying to keep open.

Stalebot: Replying to keep open.

Stalebot: Replying to keep open.

Stalebot: Replying to keep open.

https://github.com/IdentityServer/IdentityServer4/pull/4245

@leastprivilege I like his better since he doesn't use regular expressions.

OK - will use that then.


Fix link to source files in documentation

Look like it was overlooked while moving source files.

Ah thank you for this! Good spot 👍


[🐛] [TypeError: null is not an object (evaluating 'nativeImagePicler.launchCamera')]

Description
Explain the bug in as much detail as possible
Simply install library using npm/yarn
use import {launchCamera, launchImageLibrary} from 'react-native-image-picker'
getting error on launchCamera();
[TypeError: null is not an object (evaluating 'nativeImagePicler.launchCamera')]
getting error on launchImageLibrary();
[TypeError: null is not an object (evaluating 'nativeImagePicler.launchImageLibrary')]
Additional Information

Image Picker version: 5.1.0
React Native version: 0.64.2
Platform: Android
Development Operating System: MacOS
Dev tools:  Android Studio

as this package handles native modules, you need to rebuild
Re-build the project, it'll solve the issue

Same problem here with version 5.3.1, a rebuild did not fix it.
What else can be done?

Version 5.3.1 of react-native-image-picket uses TurboModules (TurboModules). You need to upgrade React Native to 0.68.2 or downgrade react-native-image-picke to 4.0.0

@k-angama Thank you! Downgrading to version 4.0.6 fixed it for me.

Version 5.3.1 of react-native-image-picker uses TurboModules (TurboModuleRegistry). You need to upgrade React Native to 0.68.2 or downgrade react-native-image-picker to 4.0.0

thansk

Is this just a typo? It says Picler instead of Picker.

The version 5.0.2 work for me

Thanks @AxelCM, version 5.0.2 also works for me

A versão 5.3.1 do react-native-image-picker usa TurboModules (TurboModuleRegistry). Você precisa atualizar o React Native para 0.68.2 ou fazer downgrade do react-native-image-picker para 4.0.0

Thank you, it worked for me, I used 4.0.6, and for future people who have problems, my native React is 0.63

For android api level 33 new behavior
I need to upgrade 2 dependencies

React native: 0.63.2
upgraded to react-native-image-picker : 4.10.3
upgraded to react-native-permissions: 3.7.3

Check if the api level version is 33 or grater, then request

PERMISSIONS.ANDROID.READ_MEDIA_IMAGES
PERMISSIONS.ANDROID.READ_MEDIA_VIDEO
PERMISSIONS.ANDROID.CAMERA

else (is lower than 33)

PERMISSIONS.ANDROID.READ_EXTERNAL_STORAGE
PERMISSIONS.ANDROID.CAMERA

Version 5.3.1 of react-native-image-picker uses TurboModules (TurboModuleRegistry). You need to upgrade React Native to 0.68.2 or downgrade react-native-image-picker to 4.0.0

Thanks that's work. Also I recommend to update to 4.9.0. There is no problem with building on Android
https://github.com/react-native-image-picker/react-native-image-picker/pull/1955#issuecomment-1222338760

Updating react native to 0.68.2 didn't fix the issue.
"react-native": "0.68.2",
"react-native-image-picker": "7.0.1",

Seeing this on:
"react-native": "0.74.1",
"react-native-image-picker": "7.1.2",

Thanks @AxelCM, version 5.0.2 worked for me
"react-native-image-picker": "5.0.2",
"react-native": "0.63.5",

Thanks problem solve😍😍
"react-native": "0.64.0",
"react-native-image-picker": "4.10.0",


addEventListener returns wrong info

I tried to turn on and off wifi manually several times in a short time.
then addEventListener returns previous state. not current state.
Ex. online --> offline : isConnected = true
offline--> online : isConnected = false
and calls this listener twice. and sometimes return correct info after some minutes.
addEventListener returns really unreliable info.
I can't use this function.
react-native: 0.60.5
react: 16.8.6
os: iOS 13
didn't test this function on android yet.

for me, it is called twice, once with old state and then with latest state

I don't know if it's the same thing，but it was ok before

same issue with me in android as well ios

I am also getting the same issue. addEventLisner is returning the previous state message.
If I kill the app reopen it, then it shows correct info.

👋 Are you able to provide a minimal reproducible demo project which I can run to see this behaviour?
Can you also confirm which version of the library you are using?

I'm facing similar issues. It seems like sometimes it's reporting wrong info or not reporting at all when connection status changes.

@matt-oakes , sorry, I forgot the version, but I installed latest version at that time.
and I uninstalled this library, so I can't confirm it for now.
original library worked well for me ( import {NetInfo} from 'react-native' )

I'm seeing this issue as well on Android with these specs:
react-native: 0.59.9
react: 16.8.6
os: Android 10
@react-native-community/netinfo: 4.6.1
I'll try to get it reproducible in a minimal reproducible demo project but it might take a few days.
Here's a video of the experience in our app:
https://drive.google.com/open?id=1ihFpjU892jsS27MEM__U4b73O8eDkafO

@matt-oakes I was able to create the reproducible demo project here: https://github.com/jwilliamson-qb/NetInfoDemo
The readme also contains a link to a new video of the failure:
https://drive.google.com/open?id=1FK1h8EIDsC3fU0NtzTyT6uevXB0aZc5O
^ In this video I alert the value of NetInfo state whenever the event triggers.
I've narrowed down the problem to when you transition from LTE to offline. With wifi the issue doesn't occur.


feat(gui): Add desktop notification setting

This adds a setting to disable desktop notifications, to be controlled
via configuration file
Change-Type: feat
Connects To: #2254

Seems a bit strange that this behaviour is controlled from a settings option, whereas all the other Etcher-pro related features are controlled from env-vars?


[ESMValCore pre-release v270rc2] DO NOT merge: for ESMValBOT runs only

@esmvalbot please run recipe_anav13jclim.yml

@remi-kazeroni @bouweandela do I get this right - the bot is first building an environment with the environment.yml that sits in the feature branch (here) and installs esmvaltool afterwards, inside that newly built env? If it uses a pre-built/containered env then this no bueno to test the Core rc :beer: environment then

@remi-kazeroni @bouweandela do I get this right - the bot is first building an environment with the environment.yml that sits in the feature branch (here) and installs esmvaltool afterwards, inside that newly built env? If it uses a pre-built/containered env then this no bueno to test the Core rc beer environment then

nevermind, the bot has answered my question - we're in business :money_with_wings:

agh - does the bot even upload the run logs? I can't see them :eyes:

@esmvalbot please run recipe_capacity_factor.yml

@esmvalbot please run recipe_capacity_factor.yml

agh - does the bot even upload the run logs? I can't see them 👀

Yes, you can normally see the installation and conda logs, for instance here.
I think the current issue is that we never updated the way the bot installs the Tool after we moved the R-dependencies. The Bot is still trying to do esmvaltool install R and that kills the runs. Let me have a look to fix that.

@remi-kazeroni awesome, cheers, man :beer:

@esmvalbot Please run examples/recipe_r.yml

R recipes work again now.

closing this since I am pounding both JASMIN and Levante atm, spare the bot a bit of stress

Don't delete the branch tho - am actually gonna set restrictions on it


Update ESP.cs - Changed Chams button to toggle the method on/off

Modified the Chams in ESP so that it can be toggled on/off whenever the button "Chams" is pressed, instead of staying on all the time once it's toggled on.

Merge accepted, lets just hope it doesnt crash if the original materials are not found.


Add sideEffects flag to each package.json

We need to add the following property to each package.json in the packages directory:
"sideEffects": false
This tells Webpack that the package is free from side effects and therefore can be safely tree shaken.
https://webpack.js.org/guides/tree-shaking/#mark-the-file-as-side-effect-free

https://github.com/brandingbrand/flagship/commit/5f325e147171726ec2e2bf156313f0eaa4460ed2
merged - thanks @bweissbart!


DBZ-7641 Publish snapshots to new infra

https://issues.redhat.com/browse/DBZ-7641

Tested and working - see https://s01.oss.sonatype.org/content/repositories/snapshots/io/debezium/


Fix incorrect modulesLoadedBeforeTrace check

The modulesLoadedBeforeTrace check is incorrect, should be greater than 1 as requiring this module itself counts as a "moduleLoadedBeforeTrace", so will always log warnings.
Also fixed the double spacing on the logging.
Logs:
WARN :@google/cloud-trace: Tracing might not work as the following modules  were loaded before the trace agent was initialized: ["@google"]

The regex is also borked on this for extracting package names in orgs, but that can be another PR. 👍

@googlebot CLA signed

Not sure why this fails on older node versions but fine on 5 and 6, was using v6 locally and had these issues, not sure how they could've been passing before?
Am looking into it.

@matthewloring I've updated the PR:

Regex now supports npm org modules, e.g. @google/cloud-trace instead of @google - see https://regex101.com/r/lW2bE3/4 for examples.
filesLoadedBeforeTrace now ignores @google/cloud-trace instead of @google

All tests passing.

@Salakar LGTM. Thank you for putting this together!

Landed in 2167b16941b89a33aed6d9e3bebc3f9e9cb534d0.

@matthewloring just noticed that this will break in windows, needs correcting to:
// Includes support for npm '@org/name' packages
// Regex: .*?node_modules(?!.*node_modules)\/(@[^\/]*\/[^\/]*|[^\/]*).*
// Tests: https://regex101.com/r/lW2bE3/6
var moduleRegex = new RegExp(
  '.*?node_modules(?!.*node_modules)\\' + path.sep +
  '(@[^\\' + path.sep +
  ']*\\' + path.sep +
  '[^\\' + path.sep +
  ']*|[^\\' + path.sep +
  ']*).*'
);

Forgot the \\
Want me to submit a new PR?

I've opened it as a new PR. Thanks for catching this.


[WIP] Feature/properties in boundary definitions

Add Property Definitions for Properties in Boundary Defintions
With this feature, Properties in Boundary Defintions, in addition to the original plain XML specification, can be specified using a PropertyDefintions extension element that is defined on the level of ServiceTemplate.
What's added:

Properties Definition tab on the level of ServiceTemplate (similar to Node Types, etc)

Apart from Winery's KV proeprties, specification of XML elements/types is also possible

Properties are instantiated based on the specified Properties Definition

If no Properties Definition was specified, properties can be specified as in the original implementation

If existing Properties Definition are removed or their type is changed, e.g., from XML element to KV properties, specified properties and property mappings are also removed

Changes of KV Properties Definition (adding or removing new kv pairs) is reflected in both, properties and property mappings

Property Mappings now have a select element allowing to choose defined KV properties (in case KV properties were used in Properties Definition), otherwise an original input field is kept

[ ] Ensure that you followed http://eclipse.github.io/winery/dev/ToolChain#github---prepare-pull-request. Especially, we require a single commit

[x] Ensure that the commit message is a good commit message

[ ] Ensure to use auto format in all files

[ ] Ensure that you appear in NOTICE at Copyright Holders

[ ] Tests created for changes

[ ] Documentation updated (if needed)

[ ] Screenshots added (for UI changes)

Hi @v-yussupov, what needs to be done here for the final merge?

@miwurster actually, I'm not sure if it's still needed. I think initially Lukas planned to have this feature for the paper, but eventually it was not used. @lharzenetter, is it still needed?

@lharzenetter please revise, finalize and merge this PR. Otherwise close it.


LightGBM Multiclass Not Supported?

When trying to use dtreeviz I get the error:
objective multiclass num_class:5 is not yet supported by dtreeviz's lightgbm implementation
What are my options here?

hi @mattiasthalen
indeed, there is not support yet.. I have to look what work involves to add multiclass.

@tlapusan got it, is there support in xgboost?

unfortunately not, there is also an issues about it #129 :)
during development, I focused more on binary classification. But seeing that more and more people need support for multiclass, we will try to implement it faster.
if you want to give it a try for sklearn, it should have it implemented

@tlapusan ok, can confirm it works in sklearn!

@mattiasthalen I would like to reopen this issue, because it is still valid for lightgbm :) maybe I will have time in the future to solve it, or maybe some PR from others contributors.

@tlapusan my mistake, I’m new to GitHub ☺

Two years on, any good news?

@SeanLungYu please give it a try, it should be implemented for all libraries :)


Obsolescent features

The standard has formally obsolescent and deleted features:

[ ] Alternate return — see B.3.2.
[ ] Computed GO TO — see B.3.3.
[x] Statement functions — see B.3.4.
[ ] DATA statements amongst executable statements — see B.3.5.
[ ] Assumed length character functions — see B.3.6.
[ ] Fixed form source — see B.3.7.
[x] CHARACTER* form of CHARACTER declaration — see B.3.8.
[x] ENTRY statements — see B.3.9.
[ ] Label form of DO statement – see B.3.10.
[x] COMMON and...
[ ] ...EQUIVALENCE statements, and...
[ ] ...the block data program unit – see B.3.11.
[ ] Specific names for intrinsic functions – see B.3.12.
[ ] FORALL construct and statement – see B.3.13

This implies we should move the character* to Obsolescent

Specific names for intrinsic functions check proposed in https://github.com/PlasmaFAIR/fortitude/pull/254.


Can anyone help explain what qps means? I haven't quite understood, thank you.

usually means queries per second
like in a search engine
On Tue, May 7, 2019 at 2:37 AM huihuima<EMAIL_ADDRESS>wrote:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mlperf/inference/issues/107, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACCS4T7SZZ7HINXFYPSB3G3PUFEUNANCNFSM4HLHC5ZA
.

@David-Levinthal You are right. Thank you very much.


App Design Definition - Create Sketch of Profile Screen

This task involves designing the profile screen for a billiards lesson scheduling app. It includes creating a detailed sketch that outlines the layout, user interface elements, and user experience flow. The profile screen will feature:

User Photo: A section at the top for displaying the user's profile picture.
User Information: Fields displaying the user's name, email, phone number, and other relevant details.
Edit Button: An option to edit personal information, such as changing the profile picture, updating contact details, etc.
Upcoming Lessons: A list or section showing the user's upcoming scheduled lessons.
Past Lessons: A section displaying the user's past lessons history.
Logout Button: An option to log out from the app.

The design aims to provide a comprehensive and organized view of the user's profile, ensuring easy access to personal information and lesson history. The layout will prioritize usability and a clean, intuitive interface, enhancing the overall user experience.

Design draft created.


Add transcript haplotypes endpoint

This endpoint returns a serialised JSON version of a
TranscriptHaplotypeContainer; this container contains details
of all observed haplotype sequences for a transcript, both the CDS
and translated protein. The haplotypes are based on phased genotype
data, at the moment the only source for this is 1000 genomes.

Looks ok, I've not spun up a test server to look at the docs.

does the example config file also need updating?
https://github.com/Ensembl/ensembl-rest/blob/release/83/ensembl_rest.conf.default
And if you have access to the ensembl-rest_private repo, can you update the config files there where required?

It works off the existing VCF config, so as long as that's set up OK (the one that's there for genotypes), it should be good.
There is one update to the default config, but that's just adding an example transcript ID.

ok, thanks
we should be able to test in the next couple of weeks once we set up the REST server, so we might get back to you then ;-)


examples/run_dsin.py make me confused

In example of DSIN, you define a new variable in line 24:
sess_number = np.array([2, 1, 0])
and add it to the input array:
x += [sess_number]
but feature_dim_dict did not define this variable's name.
I try to run without it,but get error:
"Error when checking model input: the list of Numpy arrays that you are passing to your model is not the size the model expected. Expected to see 10 array(s), but instead got the following list of 9 arrays"
why?
what mean of this variable and how to use it?

sess_numberis an additional parameter indicating the number of sessions,so it does not need to be included in feature_dim_dict

so, sess_number is the session index? but you just add two sessions:sess1_item,sess2_item?

I found sess_number or behavior_length in examples equals behavior_feature_list item length which value are not zero. so I guess this parameter looks redundantly. if zero has different meaning.why not use -1 or other interger represent the end of one session?

@showkeyjar
hi,I am sorry for didn't reply in time due to recent busy work!

sess_number is the number of valid sessions of each sample, it is not the session index
sess_number can be unequal to behavior_length,behavior_length indicates the max numbers of behavior in each session.
0 is for padding.Because I use Input() as placeholder and the dtype is not string,so the data in a batch must be equal shape

ok,thanks.

其实还有些细节：

sess_number 有可能被理解成这个人总共的sess总数（DSIN定义默认sess_max_count=5），虽然我是理解成当前记录行时间往前的session数，<=5；
sess_len_max在定义DSIN的时候有注释到，但却没有实际用到；考虑到example里面定义session物品特征时候maxlen=4，意味着最大值应该是在数据处理开始就限定了序列长度maxlen=4；


Migrate to new relations

Probably a bug -
class LocatedIn(Relation, NomanslandRelationMixin, VersionMixin):
    pk_old = [28, 95]  # pk of Property in apis_relations
    subj_model = ["apis_ontology.institution", "apis_ontology.place"]
    obj_model = ["apis_ontology.place"]

    @classmethod
    def reverse_name(cls) -> str:
        return "located in [REVERSE]"

Institution Located in Place appears both in Institution Located in Place AND Location located in place

Could you try with https://github.com/acdh-oeaw/apis-core-rdf/commit/e9a31116fa6278da038637b89cd6cb1acaa000d5 ?


[doc]: Document version-tagging specification

Is your feature request related to a problem? Please describe.
Problem:
As a taproot-asset protocol developer I would like to have a document specifying the git tag standard in order to ensure that tags are conformant to various downstream needs such as the version-checking script and regex matches.  This specification would be in addition to https://semver.org/
[edit] @ffranr found this helpful site, as referenced in #720
https://jubianchi.github.io/semver-check/#/version/0.3.2-alpha.2
It would be helpful to avoid mistakes or reloading all the context and requirements each time a release is performed and instead document the requirements.
Describe the solution you'd like
A document, added to the repo, specifying what the required form of git tags are to use for:

Releases
Release candidates
Alternative releases

Describe alternatives you've considered
Not documenting the standard and spending time each release recalling them.

Closed in favour of https://github.com/lightninglabs/taproot-assets/issues/729


Rename the -p flag?

Do we want to rename the --package-dir flag? Right now, it expects a directory that contain ArgoCD application files but this behaviour will likely change soonish. If that happens, we should consider changing this flag's name.
We should ensure that --package-dir flag works going forward for backward compatibility purposes though.

Do we want to rename the --package-dir flag?

I'm in favor to rename it in order to been able to create an IDP cluster using packages coming from different locations. Of course, that will imply that we prefix each package with a URI even if we will upload the package resources using a folder/dir
--package files:///a/b/c
--package https://
--package ssh://
--package git://
-- package ftp:///

yeah should be an easy lift to add the new flag as an alternative and plan to deprecate this in a v0.6 release.


New version: ValueAtRisk v1.2.0

Registering package: ValueAtRisk
Repository: https://github.com/chm-von-tla/ValueAtRisk.jl
Created by: @chm-von-tla
Version: v1.2.0
Commit: 7f6b23b61f8d2a093cbd17b0ab90f2a3ba4d1d4c
Reviewed by: @chm-von-tla
Reference: https://github.com/chm-von-tla/ValueAtRisk.jl/commit/7f6b23b61f8d2a093cbd17b0ab90f2a3ba4d1d4c#commitcomment-79641758
Description: Value-at-Risk for Julia

Please ignore/close. The automerge issues have been fixed in #65177


[SavageWorlds-Tabbed] Adding italian translation

Just a first batch, if it works I will keep translating. Main focus on UTF-8 with words like Agilità or Età, to be checked if they are rendered correctly
Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Greetings,
Translation should be submitted on Crowdin rather than through the json like this. Send me an email at<EMAIL_ADDRESS>and I will get you an invite for Crowdin for future translations. :)
Thanks,
Cassie


Remove numpy<1.24 restriction from requirements.txt

Fixes https://github.com/quantumlib/Cirq/issues/5967

@maffoo proper_repr was updated as part of https://github.com/quantumlib/Cirq/pull/5991 to produce valid reprs for numpy 1.24


image number zero

The image numbers in the similarity report are all zero. This was working fairly recently so there must've been  breaking change recently.

@dchud I think I caused this. If you have any spare cycles could you help me identify what the problem is?

The way it's written now media-graph.json can arrive prior to count-media.csv, which would trigger the image rendering before the counts are available.  It's hard to replicate reliably because these two responses could return in any order.


Check for binary setting existence instead of its length

This was crashing the neovim extension. The binary path we receive from setting can be null if that doesn't exists. This PR introduces a null check for that.
Thanks!

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


Issue about Cover image path

The cover image's path is hard coded in  js/SimpleCore.js (changeReadModel).
When root value is NOT "/" in site's _config.yml, the cover image path will be broken.
For instance,
in site's config.yml,
url : http://site/sub
root:/sub/

the cover image path would ignore the sub, and be http://site/images/cover-night.jpg

Let me know if you can fix the bug:)

@longqzh Sorry and thanks for this report, I will check it as soon as possible

@longqzh Bug has been fixed in version 0.0.9.


Update asmdiffer

This pulls the fix for calll *0x123456 instructions that currently break a number of x86 scratch.
I also update the lockfile to poetry 1.6.0.

NVM this is still broken.

This got done in #849


Code uses ofVec3f (old) instead of glm::vec3 (new)

Here a large commit changing the core's examples to glm, as an example:
https://github.com/openframeworks/openFrameworks/commit/b8473052bdaf49f8cb362ae1fcfdc887ab988105
:)

Solved on 40deade7c8dedb23abc62f7024fb035506bee665


Spectator Mode

Story: As a player I want to be able to spectate my friend’s games by joining them.
Acceptance Criteria:

[ ] I can join a friend by visiting his profile and click "spectate".
[ ] I join via friend list by clicking "spectate".
[ ] Upon joining, I can see my friend's game.

Priority: low
Author: SoPra Group 9
Estimate: 4h

Couldn't be completed since we lost a group member in the beginning.
Please find further explanation in the group report.


Design error handling UI

Hoping these few examples are enough to get us going. Though there aren't too many forms yet. Basic idea is:

#acacac for input helper positioned 30px to the right of input
$linode-green checkmark or $linode-red cross appears 5px to the left of the input helper without moving it
Bad input is highlighted using $linode-red and the background is set to $linode-light-red (#FFE7E7) for now


Update doc wording for IMPLICIT_COMMAND_DEPEDENCIES

Update doc wording for IMPLICIT_COMMAND_DEPEDENCIES  [ci skip]
Reword, remove a bit of redundancy by describing actions containing && in one place instead of repeatedly. Actually mention the case of setting the cvar to True.
Doc-only change.
Contributor Checklist:

[ ] I have created a new test or updated the unit tests to cover the new/changed functionality.
[ ] I have updated src/CHANGES.txt (and read the README.txt in that directory)
[X] I have updated the appropriate documentation

My local git work tree must be completely broken. Getting rid of this one and starting over.


Bug Fix Branch 0.10.9 "...exposed beyond app through Intent.getData() " at Android N

This PR is addressed to #358. It solves the exposed security exception on Android devices with API version >= 24. In older API's the code hasn't changed, still uses file://....

I'm curious what is necessary to complete this pull request. Is there anything I could do to help with it?

Thank you for this PR!


Rename "launch" -> "deploy" and avoid the term "remote"

Following the docs changes that are coming on the next chart release, this PR is updating the terms we use in the docker extension. "Launch" is renamed to "Deploy".
This is how the main page looks like now:

I updated the layout of the view:


P frames for H.264 encoder

Hi @jcodec !
For some time I observe your Future Development paragraph (https://github.com/jcodec/jcodec#future-development). You mentioned in it about adding P-Frames to H.264 encoder, but i still see in H264Encoder class in your code that all frames are marked as I-Frames.
Are you planning to add this frames division in the near future? If yes, do you know more or less time of this release?

+1

This is happening now. Added support for P16x16 macroblocks and pure P-frames (no mixing with I-mbs yet).
At this point P-frames are 3x smaller then I-frames on average.
Will keep you posted.


relative to $(workspaceRoot)

Favorites are stored with absolute paths. I wish it is possible to add favorite relative to workspaceRoot. In that case moving/renaming project and/or working on the same project on different computers (with different paths) would work.
Solution would be to use paths relative to vscode variable ${workspaceRoot}

I have the same need, when I sync a project with others and the workspace path is different all favorites are not working, inside the json i noticed there are two fields that remain null even if i edit manually: workspaceRoot and workspacePath
    {         "type": "File",         "name": "c:\\Wamp\\www\\testsite\\routes\\web.php",         "parent_id": null,         "fsPath": "c:\\Wamp\\www\\testsite\\routes\\web.php",         "workspaceRoot": null,         "workspacePath": null,         "id": "SswrZzXBEHy92d5q"     }
However, as you can see here  ${workspaceRoot} is deprecated
In my case the .code-workspace file is inside the www folder, I tried a lot of possibilities setting paths relatives to ${workspaceFolder} but none of them worked

I was just looking to do the same. After taking a look into the current plugin code to see how it sets up its paths, I managed to make it work for me:

"name": "path/to/folder/or/file/relative/to/workspace/root/without/leading/or/ending/slash"
"fsPath": null
"workspaceRoot": null
"workspacePath": null

@AcidGreg I tried this, not working though. Here are my settings
settings.json
 "favorites.storageRegistry": [
    "/Users/username/favorites.json",
  ],

favorites.json
  {
    "type": "File",
    "name": "/Users/username/Documents/notes.md",
    "parent_id": null,
    "fsPath": null,
    "workspaceRoot": null,
    "workspacePath": null,
    "id": "5EM2LpqXfIiBybvm"
  }

    "name": "/Users/username/Documents/notes.md",

That path does not seem to be relative to the workspace. ;-)
I only tried it with a relative path, as I wanted the whole thing to be portable.


How to test the dataset of Set5(Y channel) using the trained model (RGB)

hello,I want to ask you how to test the dataset of Set5(Y channel) using the trained model (RGB). In other words, the model input channel is 3 but the test dataset's channel is 1. How do you solve it?

Hello.
We just use RGB input/output and convert the output image to YCbCr space.
Then we use a Y channel only when evaluating traditional benchmark sets such as Set5/Set14/B100/Urban100.
Thank you!

Thank you for your answer！And I have another question, are the codes that convert the images to the YCbCr space and calculate the PSNR of the Y channel images provided，or  these should be written by myself ?

For the above datasets I mentioned, my code calculate PSNR of the Y channel automatically.
Please use an argument such as --data_test Set5.
If you have any troubles, set benchmark=True when calling calc_psnr to evaluate the PSNR of Y channel

Ok, I understand it. Thanks a lot!


Add tern:SamplingCollection, an implementation of sosa:SamplingCollection

Added tern:SamplingCollection an implementation of sosa:SamplingCollection from the recent SOSA ontology PR.
Added SHACL shapes for tern:SamplingCollection
I followed the same patterns in this one that tern:ObservationCollection uses.
The purpose of SamplingCollection is to group one or more Samplings with common properties or attributes, with the ability to hoist the shared properties to the SamplingCollection.
This is a supporting PR, showing evidence of implementation in support for the pending "SamplingCollection" PR in SOSA https://github.com/w3c/sdw-sosa-ssn/pull/227
Note, I don't have topbraid composer, so I edited the tern.ttl and tern.shapes.ttl by hand for this, I couldn't export a new version of tern.ttl.txt or tern.ttl.svg or any of the html files.

Where is sosa:SamplingCollection? I see sosa:ObservationCollection in the SSN Ext but not that class...
Since a Sampling is some kind of Observation - SSN doesn't link them with upper classes but also doesn't show disjointness - perhaps we can just use Observation Collections to house Samplings.
There is already a TERN Observation Collection class defined, so if my suggestions here are followed, we don't need to do anything.

Sampling collection is still in PR: https://github.com/w3c/sdw-sosa-ssn/pull/227
They may need to see one more profile or implementation to get it over the line.


Developed commands to change focus between preview editor and references within Peek View

This PR fixes part of #23001 because now one can, from within Peek View, change focus between preview editor and references tree with KeyCode.F2.

I said 'part of' because there is still no setting available yet, to select which component will receive focus by default when Peek View is opened (preview editor or references).
I chose KeyCode.F2 to by default fire this command because:

It's ergonomically confortable since it's next to F4 which is used to go to next reference within Peek View references;
Personally, I think people wouldn't do major changes like renaming elements from inside Peek View. (F2 is also the default KeyCode for renaming elements on Language Server, so this behavior would be overwritten by the feature provided by this PR). I think one could easily close Peek View to rename element with F2 if needed;
F2 KeyCode is just a default and can be changed from VS Code Settings, so one can go back to previous behavior and rename elements from inside Peek View.

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

@jrieken Hi! The link for this PR is missing on 1.42 release notes :sweat_smile: could you add it please? I've also updated my profile with my name (Gustavo Cassel) today. Big thank you!

Sorry, issue had the wrong milestone... I will update

No problem! Thank you 😅


Add a description field may be shown only if user wishes to see, but I think it should be there.

@tejasg Can you eleborate more on what you would like to see in the description?


Command completion after a flag is broken

SpongeVanilla version: 1.16.4-8.0.0 (commit 3753b5e3630bb22f25402d1f8452f7eb8afb7ce2
Java version: 8
Operating System: Windows 10
Plugins: Cosmos

Hello, today I'm gonna write a serie of issues that are in my mind or just as todos in my plugin for api-8.
Sorry for the spam.
So, for this first issue, I encoutered a problem using flags in command.
Move command : /cosmos move [--safe-only] [<targets>] [<world>] [<x> <y> <z>] [<pitch> <yaw> <roll>]
So when I complete my move command like that:
/cosmos move --safe-only
I got the following ui completion:

And it's infinite :

Thanks for reading this issue

And that's correct. Flags are intended for use multiple times - that's how they worked in 7, that's how they work in 8. If you don't want that behaviour, just create a literal and put it in your command itself.

And that's correct. Flags are intended for use multiple times - that's how they worked in 7, that's how they work in 8. If you don't want that behaviour, just create a literal and put it in your command itself.


Use symfony controllers for the endpoint

🔖 Changes

Install Symfony to use the basic controllers logic
Create a mapping between the user and its configuration

Note: This PR would be better than this other one https://github.com/php-lightning/lnaddress/pull/17

🎥  Demo
https://user-images.githubusercontent.com/5256287/230677048-75f6e564-cde2-4c01-9044-1fb9b975307b.mov

This approach requires

a shell access and uses a unnecessary (in my opinion) backend that will be permanently listening (more resources, CPU, RAM, TCP port to listen on)
more manipulation from the user before to be ready-to-use (more configuration to change is less easy to use)

in order to be used versus the other PR that could allow to be used on a simple web hosting plan and where the code is just executed when solicited.
It's a bit like a nuclear plant to light a bulb.
In my opinion user experience and resources optimization is slightly better with the previously closed PR #17 .

Oh! Actually, I am flattered somehow by this, because usually people like the idea of using symfony. So, you are the first person that likes my framework/Library(Gacela) more than the standard most popular framework symfony :)
However, there is a trick to this point: I didn't removed Gacela, and I just introduced some little dependencies from Symfony to use the controller, which is very easy and powerful.
To your three points (written in the PR):

there is no need to a shell access. It's based on the other approach.
there is no need different BE, it would be an identical solution as the previous one.
Similar DX, simpler in my point of view, because you don't need to override an entire file (copy-pasting another file) but just add a new backend in the global configuration in your app.

I reviewed again the last changes, and I found there are a few things that I could try to remove to make it even simpler. But in theory, whatever we used inside (with Symfony or without), I thought this is the best DX we wanted: a global config mapping instead of copy-pasting entire files.
Let me know if this is the case or you disagree for a particular reason that I might have missed.
I will focus on reduce the number of dependencies and the code I added. Worst scenario, we can rollback to what we had yesterday and think about another approach if you are still not convinced after trying it.


Update to support ESP-IDF v5 in develop branch

Shifting PR https://github.com/lvgl/lvgl_esp32_drivers/pull/184 to develop branch instead. This has all the past changes. Tested against ESP-IDF prerelease-v5.0 and release-v4.4 on the M5Stack Core2 for AWS using a modified version of this example: https://github.com/lvgl/lv_port_esp32/tree/feat/new_driver_test
ESP-IDF v5.0.0 incorporates a major update to the real-time kernel and number of other changes. This is my quick stab at adding support for the new toolchain version before the official release. Tested on the M5Stack Core2 for AWS IoT EduKit and works great.
There are memory and performance advantages to using pdMS_TO_TICKS instead of portTICK_PERIOD_MS or the deprecated portTICK_RATE_MS macros. That's why you'll see mostly universal changes that switches to that.

Sorry for the delay in getting time for this. I've gone ahead and removed all the #include "rom/gpio.h" but maintained compatibility with ESP-IDF v5.0 as it sits currently.


Subtask 2 : Change Start button functionalities.

This is a subtask of the issue : https://github.com/rilling/OpenTracksConcordia/issues/192
The pull request for this task is linked here : https://github.com/rilling/OpenTracksConcordia/pull/198
Commit : Link


Move or fork mtsamplesFR under medkit-lib

It would be desirable to move or keep a fork of mtsamplesFR for posterity.
medkit.tools.mtsamples currently relies on hardcoded paths which are out-of-control of the organization.
Plan for action:

[x] Move or fork mtsamplesFR to the medkit-lib org
[x] Adjust the paths to point to the medkit-lib repo

Closed by eb1a3cd


fix: refactor server package out of internal directory

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 46.443%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
4243

Relevant Lines:
9136

💛  - Coveralls

@kushsharma any reason to move server to pkg directory? The idea so far has been to keep pkg as something which are generic packages and which at some point can even be moved out of repo.

Yeah, I wanted to allow importing a few consts and methods from outside the shield as well. If it's put in an internal package, nobody can import them outside shield.


Latitude and Longitude are not loaded in for GFS Grib2 datasets

I am using https://nbviewer.org/gist/peterm790/92eb1df3d58ba41d3411f8a840be2452 as a basis to kerchunk over NOAAs GFS Wave data on S3 here: https://noaa-gfs-bdp-pds.s3.amazonaws.com/index.html#gfs.20221028/06/wave/gridded/
Using that I created the following script to generate the reference jsons which works well:
import xarray as xr
import datetime as dt
import fsspec
import ujson
import hvplot.xarray
from kerchunk.grib2 import scan_grib
from kerchunk.combine import MultiZarrToZarr

fs_read = fsspec.filesystem('s3', anon=True, skip_instance_cache=True, use_ssl=False) # For now SSL false is solving my cert issues **shrug**
fs_write = fsspec.filesystem('')

hourly_files = fs_read.glob('s3://noaa-gfs-bdp-pds/gfs.20221028/06/wave/gridded/gfswave.t06z.atlocn.0p16.*.grib2')
files = sorted(['s3://'+f for f in hourly_files])
files

so = {"anon": True, "use_ssl": False}
json_dir = '../datasets/gfswave_atlantic/'

def make_json_name(file_url, message_number): #create a unique name for each reference file
    date = file_url.split('/')[3].split('.')[1]
    name = file_url.split('/')[7].split('.')[0:5]
    return f'{json_dir}{date}_{name[0]}_{name[1]}_{name[2]}_{name[4]}_message{message_number}.json'

def gen_json(file_url):
    print(f'Processing {file_url}...')
    out = scan_grib(file_url, storage_options=so)   # create the reference using scan_grib
    for i, message in enumerate(out): # scan_grib outputs a list containing one reference per grib message
        out_file_name = make_json_name(file_url, i)  # get name
        print(message)
        with fs_write.open(out_file_name, "w") as f:
            f.write(ujson.dumps(message)) # write to file

for f in files[0:2]:
    gen_json(f)

However, when I load in one of the reference json files with xarray the coordinate arrays are just flat:
fs = fsspec.filesystem("reference", fo=reference_jsons[0], remote_protocol='s3', target_options={'anon': True}, remote_options={'anon': True, 'use_ssl': False})
mapper = fs.get_mapper('')

ds = xr.open_dataset(mapper, engine='zarr', backend_kwargs={'consolidated': False}, chunks={})
ds

When I load in the same dataset locally using xarray, you can see how the coordinates are supposed to look:
import matplotlib.pyplot as plt

ds_r = xr.open_dataset('../datasets/gfswave.t06z.atlocn.0p16.f003.grib2')
ds_r

I saw https://github.com/fsspec/kerchunk/issues/238 and the associated PR so i tested before and after that merge with no luck. I am not sure if this is a bug or incorrect usage by myself. Thanks!

It seems that we have the latitude and longitude directions the wrong way around at location https://github.com/fsspec/kerchunk/blob/main/kerchunk/grib2.py#L204 , https://github.com/fsspec/kerchunk/blob/main/kerchunk/grib2.py#L207 (ie., swapping [0]<->[:, 0]). Would you mind changing that locally to see if it then does the right thing? If yes, then we'll need to update those couple of lines to cope with the two coords either way around.

Yeah ill give that a shot, thanks!

That gets the data written it looks like:
"latitude\/.zarray":"{\n    \"chunks\": [\n        331\n    ],\n    \"compressor\": null,\n    \"dtype\": \"<f8\",\n    \"fill_value\": null,\n    \"filters\": null,\n    \"order\": \"C\",\n    \"shape\": [\n        331\n    ],\n    \"zarr_format\": 2\n}","latitude\/0":"base64:AAAAAACAS0C71t6nqmpLQHatvU9VVUtAMYSc9\/8\/S0DsWnufqipLQKcxWkdVFUtAYgg57\/\/\/SkAd3xeXqupKQNi19j5V1UpAk4zV5v+\/SkBOY7SOqqpKQAk6kzZVlUpAxBBy3v9\/SkB\/51CGqmpKQDq+Ly5VVUpA9ZQO1v8\/SkCwa+19qipKQGtCzCVVFUpAJhmrzf\/\/SUDh74l1qupJQJzGaB1V1UlAV51Hxf+\/SUASdCZtqqpJQM1KBRVVlUlAiCHkvP9\/SUBD+MJkqmpJQP7OoQxVVUlAuaWAtP8\/SUB0fF9cqipJQC9TPgRVFUlA6ikdrP\/\/SEClAPxTqupIQGDX2vtU1UhAG665o\/+\/SEDWhJhLqqpIQJFbd\/NUlUhATDJWm\/9\/SEAHCTVDqmpIQMLfE+tUVUhAfbbykv8\/SEA4jdE6qipIQPNjsOJUFUhArjqPiv\/\/R0BpEW4yqupHQCToTNpU1UdA374rgv+\/R0CalQoqqqpHQFVs6dFUlUdAEEPIef9\/R0DLGachqmpHQIbwhclUVUdAQcdkcf8\/R0D8nUMZqipHQLd0IsFUFUdAcksBaf\/\/RkAtIuAQqupGQOj4vrhU1UZAo8+dYP+\/RkBepnwIqqpGQBl9W7BUlUZA1FM6WP9\/RkCPKhkAqmpGQEoB+KdUVUZABdjWT\/8\/RkDArrX3qSpGQHuFlJ9UFUZANlxzR\/\/\/RUDxMlLvqepFQKwJMZdU1UVAZ+APP\/+\/RUAit+7mqapFQN2NzY5UlUVAmGSsNv9\/RUBTO4veqWpFQA4SaoZUVUVAyehILv8\/RUCEvyfWqSpFQD+WBn5UFUVA+mzlJf\/\/REC1Q8TNqepEQHAao3VU1URAK\/GBHf+\/REDmx2DFqapEQKGeP21UlURAXHUeFf9\/REAXTP28qWpEQNIi3GRUVURAjfm6DP8\/REBI0Jm0qSpEQAOneFxUFURAvn1XBP\/\/Q0B5VDasqepDQDQrFVRU1UNA7wH0+\/6\/Q0Cq2NKjqapDQGWvsUtUlUNAIIaQ8\/5\/Q0DbXG+bqWpDQJYzTkNUVUNAUQot6\/4\/Q0AM4QuTqSpDQMe36jpUFUNAgo7J4v7\/QkA9ZaiKqepCQPg7hzJU1UJAsxJm2v6\/QkBu6USCqapCQCnAIypUlUJA5JYC0v5\/QkCfbeF5qWpCQFpEwCFUVUJAFRufyf4\/QkDQ8X1xqSpCQIvIXBlUFUJARp87wf7\/QUABdhppqepBQLxM+RBU1UFAdyPYuP6\/QUAy+rZgqapBQO3QlQhUlUFAqKd0sP5\/QUBjflNYqWpBQB5VMgBUVUFA2SsRqP4\/QUCUAvBPqSpBQE\/ZzvdTFUFACrCtn\/7\/QEDFhoxHqepAQIBda+9T1UBAOzRKl\/6\/QED2Cik\/qapAQLHhB+dTlUBAbLjmjv5\/QEAnj8U2qWpAQOJlpN5TVUBAnTyDhv4\/QEBYE2IuqSpAQBPqQNZTFUBAm4E\/\/Pz\/P0AQL\/1LUtU\/QIXcupunqj9A+ol46\/x\/P0BvNzY7UlU\/QOTk84qnKj9AWZKx2vz\/PkDOP28qUtU+QEPtLHqnqj5AuJrqyfx\/PkAtSKgZUlU+QKL1ZWmnKj5AF6Mjufz\/PUCMUOEIUtU9QAH+nlinqj1AdqtcqPx\/PUDrWBr4UVU9QGAG2EenKj1A1bOVl\/z\/PEBKYVPnUdU8QL8OETenqjxANLzOhvx\/PECpaYzWUVU8QB4XSianKjxAk8QHdvz\/O0AIcsXFUdU7QH0fgxWnqjtA8sxAZfx\/O0Bnev60UVU7QNwnvASnKjtAUdV5VPz\/OkDGgjekUdU6QDsw9fOmqjpAsN2yQ\/x\/OkAli3CTUVU6QJo4LuOmKjpAD+brMvz\/OUCEk6mCUdU5QPlAZ9KmqjlAbu4kIvx\/OUDjm+JxUVU5QFhJoMGmKjlAzfZdEfz\/OEBCpBthUdU4QLdR2bCmqjhALP+WAPx\/OEChrFRQUVU4QBZaEqCmKjhAiwfQ7\/v\/N0AAtY0\/UdU3QHViS4+mqjdA6g8J3\/t\/N0BfvcYuUVU3QNRqhH6mKjdASRhCzvv\/NkC+xf8dUdU2QDNzvW2mqjZAqCB7vft\/NkAdzjgNUVU2QJJ79lymKjZABym0rPv\/NUB81nH8UNU1QPGDL0ymqjVAZjHtm\/t\/NUDb3qrrUFU1QFCMaDumKjVAxTkmi\/v\/NEA65+PaUNU0QK+UoSqmqjRAJEJfevt\/NECZ7xzKUFU0QA6d2hmmKjRAg0qYafv\/M0D491W5UNUzQG2lEwmmqjNA4lLRWPt\/M0BXAI+oUFUzQMytTPilKjNAQVsKSPv\/MkC2CMiXUNUyQCu2heelqjJAoGNDN\/t\/MkAVEQGHUFUyQIq+vtalKjJA\/2t8Jvv\/MUB0GTp2UNUxQOnG98WlqjFAXnS1Fft\/MUDTIXNlUFUxQEjPMLWlKjFAvXzuBPv\/MEAyKqxUUNUwQKfXaaSlqjBAHIUn9Pp\/MECRMuVDUFUwQAbgopOlKjBA9hrBxvX\/L0DgdTxmoKovQMrQtwVLVS9AtCszpfX\/LkCehq5EoKouQIjhKeRKVS5Acjylg\/X\/LUBclyAjoKotQEbym8JKVS1AME0XYvX\/LEAaqJIBoKosQAQDDqFKVSxA7l2JQPX\/K0DYuATgn6orQMITgH9KVStArG77HvX\/KkCWyXa+n6oqQIAk8l1KVSpAan9t\/fT\/KUBU2uicn6opQD41ZDxKVSlAKJDf2\/T\/KEAS61p7n6ooQPxF1hpKVShA5qBRuvT\/J0DQ+8xZn6onQLpWSPlJVSdApLHDmPT\/JkCODD84n6omQHhnutdJVSZAYsI1d\/T\/JUBMHbEWn6olQDZ4LLZJVSVAINOnVfT\/JEAKLiP1nqokQPSInpRJVSRA3uMZNPT\/I0DIPpXTnqojQLKZEHNJVSNAnPSLEvT\/IkCGTweynqoiQHCqglFJVSJAWgX+8PP\/IUBEYHmQnqohQC679C9JVSFAGBZwz\/P\/IEACcetunqogQOzLZg5JVSBArE3EW+f\/H0CAA7uaPFUfQFS5sdmRqh5AKG+oGOf\/HUD8JJ9XPFUdQNDalZaRqhxApJCM1eb\/G0B4RoMUPFUbQEz8eVORqhpAILJwkub\/GUD0Z2fRO1UZQMgdXhCRqhhAnNNUT+b\/F0BwiUuOO1UXQEQ\/Qs2QqhZAGPU4DOb\/FUDsqi9LO1UVQMBgJoqQqhRAlBYdyeX\/E0BozBMIO1UTQDyCCkeQqhJAEDgBhuX\/EUDk7ffEOlURQLij7gOQqhBAGbPKhcr\/D0DCHrgDdaoOQGuKpYEfVQ1AFPaS\/8n\/C0C9YYB9dKoKQGbNbfseVQlADzlbecn\/B0C4pEj3c6oGQGEQNnUeVQVACnwj88j\/A0Cz5xBxc6oCQFxT\/u4dVQFACn7X2ZD\/\/z9cVbLV5VT9P64sjdE6qvo\/AARozY\/\/9z9S20LJ5FT1P6SyHcU5qvI\/6xPxgR3\/7z+OwqZ5x6nqPzFxXHFxVOU\/qD8k0jb+3z\/unI\/BilPVP2j09WG9UcU\/AAAAAAAAAAA=","latitude\/.zattrs":"{\n    \"_ARRAY_DIMENSIONS\": [\n        \"latitude\"\n    ],\n    \"long_name\": \"latitude\",\n    \"standard_name\": \"latitude\",\n    \"units\": \"degrees_north\"\n}","longitude\/.zarray":"{\n    \"chunks\": [\n        301\n    ],\n    \"compressor\": null,\n    \"dtype\": \"<f8\",\n    \"fill_value\": null,\n    \"filters\": null,\n    \"order\": \"C\",\n    \"shape\": [\n        301\n    ],\n    \"zarr_format\": 2\n}","longitude\/0":"base64:AAAAAABAcECrqqqqqkJwQFZVVVVVRXBAAQAAAABIcECsqqqqqkpwQFdVVVVVTXBAAgAAAABQcECtqqqqqlJwQFhVVVVVVXBAAwAAAABYcECuqqqqqlpwQFlVVVVVXXBABAAAAABgcECvqqqqqmJwQFpVVVVVZXBABQAAAABocECwqqqqqmpwQFtVVVVVbXBABgAAAABwcECxqqqqqnJwQFxVVVVVdXBABwAAAAB4cECyqqqqqnpwQF1VVVVVfXBACAAAAACAcECzqqqqqoJwQF5VVVVVhXBACQAAAACIcEC0qqqqqopwQF9VVVVVjXBACgAAAACQcEC1qqqqqpJwQGBVVVVVlXBACwAAAACYcEC2qqqqqppwQGFVVVVVnXBADAAAAACgcEC3qqqqqqJwQGJVVVVVpXBADQAAAACocEC4qqqqqqpwQGNVVVVVrXBADgAAAACwcEC5qqqqqrJwQGRVVVVVtXBADwAAAAC4cEC6qqqqqrpwQGVVVVVVvXBAEAAAAADAcEC7qqqqqsJwQGZVVVVVxXBAEQAAAADIcEC8qqqqqspwQGdVVVVVzXBAEgAAAADQcEC9qqqqqtJwQGhVVVVV1XBAEwAAAADYcEC+qqqqqtpwQGlVVVVV3XBAFAAAAADgcEC\/qqqqquJwQGpVVVVV5XBAFQAAAADocEDAqqqqqupwQGtVVVVV7XBAFgAAAADwcEDBqqqqqvJwQGxVVVVV9XBAFwAAAAD4cEDCqqqqqvpwQG1VVVVV\/XBAGAAAAAAAcUDDqqqqqgJxQG5VVVVVBXFAGQAAAAAIcUDEqqqqqgpxQG9VVVVVDXFAGgAAAAAQcUDFqqqqqhJxQHBVVVVVFXFAGwAAAAAYcUDGqqqqqhpxQHFVVVVVHXFAHAAAAAAgcUDHqqqqqiJxQHJVVVVVJXFAHQAAAAAocUDIqqqqqipxQHNVVVVVLXFAHgAAAAAwcUDJqqqqqjJxQHRVVVVVNXFAHwAAAAA4cUDKqqqqqjpxQHVVVVVVPXFAIAAAAABAcUDLqqqqqkJxQHZVVVVVRXFAIQAAAABIcUDMqqqqqkpxQHdVVVVVTXFAIgAAAABQcUDNqqqqqlJxQHhVVVVVVXFAIwAAAABYcUDOqqqqqlpxQHlVVVVVXXFAJAAAAABgcUDPqqqqqmJxQHpVVVVVZXFAJQAAAABocUDQqqqqqmpxQHtVVVVVbXFAJgAAAABwcUDRqqqqqnJxQHxVVVVVdXFAJwAAAAB4cUDSqqqqqnpxQH1VVVVVfXFAKAAAAACAcUDTqqqqqoJxQH5VVVVVhXFAKQAAAACIcUDUqqqqqopxQH9VVVVVjXFAKgAAAACQcUDVqqqqqpJxQIBVVVVVlXFAKwAAAACYcUDWqqqqqppxQIFVVVVVnXFALAAAAACgcUDXqqqqqqJxQIJVVVVVpXFALQAAAACocUDYqqqqqqpxQINVVVVVrXFALgAAAACwcUDZqqqqqrJxQIRVVVVVtXFALwAAAAC4cUDaqqqqqrpxQIVVVVVVvXFAMAAAAADAcUDbqqqqqsJxQIZVVVVVxXFAMQAAAADIcUDcqqqqqspxQIdVVVVVzXFAMgAAAADQcUDdqqqqqtJxQIhVVVVV1XFAMwAAAADYcUDeqqqqqtpxQIlVVVVV3XFANAAAAADgcUDfqqqqquJxQIpVVVVV5XFANQAAAADocUDgqqqqqupxQItVVVVV7XFANgAAAADwcUDhqqqqqvJxQIxVVVVV9XFANwAAAAD4cUDiqqqqqvpxQI1VVVVV\/XFAOAAAAAAAckDjqqqqqgJyQI5VVVVVBXJAOQAAAAAIckDkqqqqqgpyQI9VVVVVDXJAOgAAAAAQckDlqqqqqhJyQJBVVVVVFXJAOwAAAAAYckDmqqqqqhpyQJFVVVVVHXJAPAAAAAAgckDnqqqqqiJyQJJVVVVVJXJAPQAAAAAockDoqqqqqipyQJNVVVVVLXJAPgAAAAAwckDpqqqqqjJyQJRVVVVVNXJAPwAAAAA4ckDqqqqqqjpyQJVVVVVVPXJAQAAAAABAckDrqqqqqkJyQJZVVVVVRXJAQQAAAABIckDsqqqqqkpyQJdVVVVVTXJAQgAAAABQckDtqqqqqlJyQJhVVVVVVXJAQwAAAABYckDuqqqqqlpyQJlVVVVVXXJARAAAAABgckDvqqqqqmJyQJpVVVVVZXJARQAAAABockDwqqqqqmpyQJtVVVVVbXJARgAAAABwckDxqqqqqnJyQJxVVVVVdXJARwAAAAB4ckDyqqqqqnpyQJ1VVVVVfXJASAAAAACAckDzqqqqqoJyQJ5VVVVVhXJASQAAAACIckD0qqqqqopyQJ9VVVVVjXJASgAAAACQckD1qqqqqpJyQKBVVVVVlXJASwAAAACYckD2qqqqqppyQKFVVVVVnXJATAAAAACgckD3qqqqqqJyQKJVVVVVpXJATQAAAACockD4qqqqqqpyQKNVVVVVrXJATgAAAACwckD5qqqqqrJyQKRVVVVVtXJATwAAAAC4ckD6qqqqqrpyQKVVVVVVvXJAUAAAAADAckD7qqqqqsJyQKZVVVVVxXJAUQAAAADIckD8qqqqqspyQKdVVVVVzXJAUgAAAADQckD9qqqqqtJyQKhVVVVV1XJAUwAAAADYckD+qqqqqtpyQKlVVVVV3XJAVAAAAADgckD\/qqqqquJyQKpVVVVV5XJAVQAAAADockAAq6qqqupyQKtVVVVV7XJAVgAAAADwckABq6qqqvJyQKxVVVVV9XJAVwAAAAD4ckACq6qqqvpyQK1VVVVV\/XJAWAAAAAAAc0ADq6qqqgJzQK5VVVVVBXNAWQAAAAAIc0AEq6qqqgpzQK9VVVVVDXNAWgAAAAAQc0AFq6qqqhJzQLBVVVVVFXNAWwAAAAAYc0AGq6qqqhpzQLFVVVVVHXNAXAAAAAAgc0AHq6qqqiJzQLJVVVVVJXNAXQAAAAAoc0AIq6qqqipzQLNVVVVVLXNAXgAAAAAwc0AJq6qqqjJzQLRVVVVVNXNAXwAAAAA4c0AKq6qqqjpzQLVVVVVVPXNAYAAAAABAc0ALq6qqqkJzQLZVVVVVRXNAYQAAAABIc0AMq6qqqkpzQLdVVVVVTXNAYgAAAABQc0ANq6qqqlJzQLhVVVVVVXNAYwAAAABYc0AOq6qqqlpzQLlVVVVVXXNAAAAAAABgc0A="
...

But then the dimensions dont line up:

Ill keep debugging, looks like the dims dont line up now. thanks for the help

Perhaps like this?
--- a/kerchunk/grib2.py
+++ b/kerchunk/grib2.py
@@ -200,11 +200,8 @@ def scan_grib(
                         x = x.reshape(vals.shape)
                     else:
                         dims = [coord]
-                        if coord == "latitude":
-                            x = x.reshape(vals.shape)[0].copy()
-                            inline_extra = x.nbytes + 1
-                        elif coord == "longitude":
-                            x = x.reshape(vals.shape)[:, 0].copy()
+                        if coord in ["latitude", "longitude"]:
+                            x = np.unique(vals.shape)
                             inline_extra = x.nbytes + 1
                 else:

That gave me incorrect dimensions of 331 and 2.
Doing this gives me the correct dimensions for latitude and longitude:
                        if coord == "latitude":
                            x = x.reshape(vals.shape)[0].copy()
                            inline_extra = x.nbytes + 1
                        elif coord == "longitude":
                            x = x.reshape(vals.shape)[1].copy()
                            inline_extra = x.nbytes + 1

But then reading it back in, the dimensions dont line up:
Output exceeds the [size limit](command:workbench.action.openSettings?[). Open the full output data [in a text editor](command:workbench.action.openLargeOutput?7547f96d-a284-42af-9577-27a5e17a3418)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In [7], line 4
      1 fs = fsspec.filesystem("reference", fo=reference_jsons[0], remote_protocol='s3', target_options={'anon': True}, remote_options={'anon': True, 'use_ssl': False})
      2 mapper = fs.get_mapper('')
----> 4 ds = xr.open_dataset(mapper, engine='zarr', backend_kwargs={'consolidated': False}, chunks={})
      5 ds

File ~/Developer/zms/env/lib/python3.10/site-packages/xarray/backends/api.py:539, in open_dataset(filename_or_obj, engine, chunks, cache, decode_cf, mask_and_scale, decode_times, decode_timedelta, use_cftime, concat_characters, decode_coords, drop_variables, inline_array, backend_kwargs, **kwargs)
    527 decoders = _resolve_decoders_kwargs(
    528     decode_cf,
    529     open_backend_dataset_parameters=backend.open_dataset_parameters,
   (...)
    535     decode_coords=decode_coords,
    536 )
    538 overwrite_encoded_chunks = kwargs.pop("overwrite_encoded_chunks", None)
--> 539 backend_ds = backend.open_dataset(
    540     filename_or_obj,
    541     drop_variables=drop_variables,
    542     **decoders,
    543     **kwargs,
    544 )
    545 ds = _dataset_from_backend_dataset(
    546     backend_ds,
    547     filename_or_obj,
...
   3162                 f"length {size} on {k!r} and length {dims[dim]} on {last_used!r}"
   3163             )
   3164 return dims

ValueError: conflicting sizes for dimension 'longitude': length 331 on 'swell' and length 301 on {'latitude': 'latitude', 'longitude': 'longitude', 'orderedSequenceData': 'orderedSequenceData', 'step': 'step'}

I edited my diff, sorry about that. The other place we have absolutes is L181
                else ["longitude", "latitude"]

Something about the data must tell us the correct order for these!

OK got it working now :) np.unique wasnt working, it was the your original solution combined with updating `['longitude', 'latitude'] on line 181 that got it working. I will try and find if there is metadata that can help specify this and send a PR if youd like

Maybe the order of list(m) is the thing to check? m has a lot or attributes.
Thanks for working with me!

So gridType reports regular_ll which in grib speak means (lat,lng). Checking against the HRR example, HRR reports a lambert projection so i will for now add a check for regular_ll so this wont break existing working grib files. Ill put a PR up shortly


Feature/raunaks0001

Added HelloWorld Program and a code to check if the given number is odd or even

correct implementation


Unable to donwload source file from nuget source server (https://nuget.smbsrc.net)

The following is an example. The server returned (404) Not Found.
https://nuget.smbsrc.net/src/LinkExtensions.cs/r3vl8vqKvXGxUOF8HRnjRZCjSzXQ3PURGx8--MFtcZo=/LinkExtensions.cs
The source location was retrieved from symbol pdb of System.Web.Mvc.
Tried a few other source files and got the same result. It used to work. Did the source file be moved?

This service is not actually hosted by the NuGet team. I would recommend that you reach out to https://twitter.com/TripleEmcoder who owns the nuget.smbsrc.net service.


[TIC-293] Add /org/remove_user endpoint + major folder structure refactoring

Restructured and organized the apis that all sat in the api.py file -- it is now in a folder, where each endpoint path is in its own respective file.
Added the /org/remove_user request
Added tests to make sure that any added request is added to the necessary places

This will be moved to feat/TIC-299 (the name was incorrect), rebased on top of a new feature branch feature/TIC-293 (the epic name)


Remove unused identity property in HiLoSequence

Remove unused Identity property in HiLowSequence, see #438
and removed some unused variables (thanks to coderush)

Pulled in via a rebase.


Bug report about application cJSON parse  (IDFGH-13276)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.4-dev-78-gd4cd437ede-dirty
Espressif SoC revision.
ESP-WROVER-KIT V4.1
Operating System used.
Linux
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
None
Development Kit.
ESP32-Wrover-Kit v2 | qemu
Power Supply used.
USB
What is the expected behavior?
The program should execute without any memory errors or crashes.
What is the actual behavior?
The program crashes with an unhandled exception indicating an illegal memory access error.
Steps to reproduce.

Provide a JSON string input to the syz_cjson_parse function.
Parse the JSON string using cJSON_Parse.
The program encounters an illegal memory access during the parsing process.
While fuzz testing FreeRTOS using Syzkaller, I encountered an illegal memory access error in the cJSON module.
this is my SPEC which can trigger the bug

static long syz_cjson_parse(volatile long a0)
    {
        const char *json_string = (const char *)a0;

        // Parse the JSON string
        cJSON *json = cJSON_Parse(json_string);
        if (json == NULL) {
            return -1; // Return error code
        }

        // Delete the cJSON object
        cJSON_Delete(json);

        return 0; // Success
    }

Debug Logs.
 Exception was unhandled.

Backtrace: 0x4008a57a:0x3ffc0f70 0x400e2cb6:0x3ffc0f80 0x400e2cde:0x3ffc0fa0 0x400e5755:0x3ffc0fc0 0x400e3511:0x3ffc0ff0 0x400e6b08:0x3ffc1040 0x400e6bd9:0x3ffc11f0 0x400e6da7:0x3ffc1210 0x40152aab:0x3ffc12d0 0x4008d35d:0x3ffc1300

It appears there is an illegal access at components/json/cJSON/cJSON.c in the cJSON_ParseWithOpts function. The specific error occurs on this line buffer_length = strlen(value) + sizeof(""); If the value string is pointing to an illegal memory address, then an illegal memory access occurs when strlen() is called. So in this function you should first determine if the pointer is pointing to the correct memory address, and then call the strlen() function.
CJSON_PUBLIC(cJSON *) cJSON_ParseWithOpts(const char *value, const char **return_parse_end, cJSON_bool require_null_terminated)
{
    size_t buffer_length;

    if (NULL == value)
    {
        return NULL;
    }

    /* Adding null character size due to require_null_terminated. */
    buffer_length = strlen(value) + sizeof("");

    return cJSON_ParseWithLengthOpts(value, buffer_length, return_parse_end, require_null_terminated);
}

More Information.
It appears there is an illegal access at components/json/cJSON/cJSON.c in the cJSON_ParseWithOpts function. The specific error occurs on this line buffer_length = strlen(value) + sizeof(""); If the value string is pointing to an illegal memory address, then an illegal memory access occurs when strlen() is called. So in this function you should first determine if the pointer is pointing to the correct memory address, and then call the strlen() function.

Most likely this should be forwarded to https://github.com/DaveGamble/cJSON which maintains the cJSON library that is picked up as-is here.

Most likely this should be forwarded to https://github.com/DaveGamble/cJSON which maintains the cJSON library that is picked up as-is here.

Thanks for the advice.

Closing in favour of https://github.com/DaveGamble/cJSON/issues/875


NaNs and infinitys are displaying as null

Context
NaNs and ±Infinitys (possible float values in PostgreSQL) are displayed as null values, leading to problems with summaries (e.g., max(column) will strangely say null is the largest value of a dataset). Even further, they cause the widgets to break.
Steps to Reproduce

Import a dataset (which is a CSV but GitHub won't let me do CSVs) with NaNs and Infinitys: inf-nan-values.txt
Change the column type of column val to 'number' in the Builder
Observe that the val column has values 1.2, null, and null.
Run this query to see if those values are actually null-valued:
SELECT val is null FROM inf_nan_values
Observe that the results are all false, indicating that the values are not null valued.
Casting val to text produces infinity and NaN.
Calculate the max of the dataset produces null, but casting to text gives NaN again.
Create a widget on the column val and observe that the widget does not work.

Current Result
Infinity and NaN are shown as Null when they are actually not and produces different information when used in calculations. These values break the widgets.
Expected result
The values of infinity and NaN are displayed in the builder what they really are to avoid confusion. These values should also not break the widgets, but the user should be able to remove these values easily so that the widgets can work as expected. For example, a broken widget could say 'Cannot make widget because column has values of Infinity'.
Browser and version
Chrome 58.0.3029.110 (64-bit), macOS 10.12.4
.carto file
None, but this 'csv': inf-nan-values.txt
Additional info
Information can be displayed to communicate that there are NaNs and Infinitys in the dataset, and that they are removed when constructing the widgets. NumPy has very good behavior for dealing with these two values.

Well @andy-esch, that's the result SQL provides to the UI. I feel that issue doesn't belong to this repository. In fact, this is the SQL, and this is the result:
{
rows: [
{
cartodb_id: 1,
the_geom: "{"type":"Point","coordinates":[-3.7038,40.4168]}",
latitude: 40.4168,
longitude: -3.7038,
val: 1.2
},
{
cartodb_id: 2,
the_geom: "{"type":"Point","coordinates":[-3.7048,40.4178]}",
latitude: 40.4178,
longitude: -3.7048,
val: null
},
{
cartodb_id: 3,
the_geom: "{"type":"Point","coordinates":[-3.7058,40.4188]}",
latitude: 40.4188,
longitude: -3.7058,
val: null
}
],
time: 0.002,
fields: {
cartodb_id: {
type: "number"
},
the_geom: {
type: "string"
},
latitude: {
type: "number"
},
longitude: {
type: "number"
},
val: {
type: "number"
}
},
total_rows: 3
}

Thanks @xavijam, do you recommend SQL API repo then?

I'm not sure, but looks like, @rafatower could you help us here?

@rafatower, want me to open separate tickets about this elsewhere?

@andy-esch there are several places that could be the source of this issue and at the moment I don't really know where the ticket for the fix should belong. So no need for you to open separate tickets until we further investigate the issue.
How often do you reckon this issue is happening to our users?

Actually I see two different issues here:

Queries to API SQL should not cast Infinity  and NaN values to null (SQL-API)
Widgets should support Infinity and NaN values or, at least, does not break (Windshaft-cartodb)

Sorry for the slow response @rafatower, I was going to leapfrog to see how many of my tables had NaN and Infinity values, but haven't gotten around to it yet. I actually don't have a good sense of this, but I wouldn't be surprised if these values cause unexpected behavior that's not easy to debug in the widgets.
I think I was seeing this clearly because in cartoframes, pandas DataFrames are exported to CSVs and then sent to CARTO via the Import API, and the numpy.nan were changed to nan in the CSV in numeric columns.
I also agree on both points that @dgaubert makes. On Infinity in the widgets, we maybe should have a warning to filter them out if they exist and flag that in the filter that there are n Infinities. Otherwise, the scales/y-axis we use won't work.

PR in CartoDB-SQL-API: https://github.com/CartoDB/CartoDB-SQL-API/pull/420

PR in Windshaft-cartodb: https://github.com/CartoDB/Windshaft-cartodb/pull/700

Acceptance plan:

Install the branches of SQL-API & Windshaft Cartodb in a staging dedicated server
Prepare two datasets containing Infinities & Nans, one of them with overviews
Create a map with Builder and try all widget types on both tables; check widgets run without errors and results are correct (infinities ignored for min/max ranges, etc.)

Hey @jgoizueta! This acceptance is kind of tedious, any help you need.. you know! ;)

Preliminary Acceptance Results (to be completed next monday)
@dgaubert has kindly join me checking this and what seemed to be just a routine and tedious test became a bug-hunting session through multiple CARTO components 😅
Regarding the changes done here in the sql-api & windshaft-cartodb PRs we found a couple of issues that are already solved:

Initially count behaviour was different without overviews (NaNs, Infinites where excluded), but dani has promptly changed tha, now the behaviour with or without overviews is including all records (Nulls, NANs, Inifinities) in the count.
The dataview specializations for overview failed to forward the new queries parameter to the base class (generic dataviews) (and formula wasn't using it)

But we also found some problems affecting other components. We'll open tickets in the corresponding repos for them:

Pop ups show NaNs and Infinities as null (because of having been converted to JSON).
Overview tables are created averaging numerical columns for all the rows (including NaNs, Infinities), so in columns with NaN values the lower level overviews quickly became full of NaNs.
That problem compounds with a bug in the UI: when a histogram is empty (0 bins), this error occurs: histogram-data-model.js:80 Uncaught TypeError: Cannot read property 'start' of undefined. (for tables with NaNs lower-level overview tables are full of NaNs, yielding empty historgrams and triggering this problem)
When a formula widget is added to the map, the fourmula is avg initially, but it is show as COUNT in the UI. (STR: create a map, add a widget, leave default parameters; if you edit it, it shows COUNT, but the values displayed are averages).

Hey @jgoizueta
Please, let me know when this PR is ready to deploy.
💌

I'm moving this to deploy, since the acceptance tests are successful regarding the fixes done here to SQL-API and WIndshaft-Cartodb and having this in production is a big improvement over the current situation (where widgets break in the presence of NaNs, etc).
The issues revealed during acceptace affecting various CARTO components can be handled independently in separate tickets.


Broken VSync on Windows 11 with NVIDIA

OS: Windows 11 21H2
GPU: NVIDIA GeForce GTX 1070, 466.63 driver

Hi! Seems like commit 4005f70 breaks VSync again on my system
glfwSwapBuffers utilizes all available CPU time (~90% one core load) with a single window and swap interval set to 1
But with more windows, at least one of which has non-zero swap interval, things get more interesting: the frame rate is multiplied by the number of windows (in this example, I have 75 Hz monitor and two windows are created) and frame time looks very jittery and strange

With DwmFlush, multi-window swap also performs strangely. CPU usage gets dropped almost to zero and frame time is like without DwmFlush at first look, but if all the windows has swap interval set to 1, frame rate and frame time looks just fine
upd: swap interval 1 on both windows gives normal frame rate and frame time even without DwmFlush, but CPU usage is huge
P.S. sorry for the bad English, it's not a my first language

Thank you for reporting this! Unfortunately I don't know yet what to do about it.

I'm curious if this patch solves these issues: https://github.com/comp500/glfw/commit/08b3a9ee50f6752824826ef240f1d01db2a882ac; I've been testing some changes of window flags and this seems to work quite well for resolving tearing/VSync issues in borderless fullscreen on NVIDIA drivers. I'm likely going to upstream the patch (likely without removing auto-iconify, though I think it should be disabled by default) sometime soon, see https://github.com/comp500/BorderlessMining/issues/19 for more information.

Oh, interesting! I'd like to know the results of that too.

I am also seeing what I think is a VSYNC issue with Cygwin X11 under Windows 10. The frame rate is in the thousands of FPS and the scaling of a full window textured quad is all wrong.


http: unifying client types

It has been a pet peeve of mine for years that the test upstream and test client have different enums for the same codecs.
Unifying them, but leaving legacy typedefs for folks who don't want to update all their filters/tests.
Risk Level: Low (enum change, theoretical legacy support)
Testing: n/a
Docs Changes: n/a
Release Notes: n/a

I think this PR is just too large for clang tidy to finish.  @snowp would you be Ok if I just force-merge over that?

Fine with me as long as we don't break main as a result :)

yeah, I think it's just timing out on the diffs.  will revert if I'm wrong :-)


Gitstream fails with fatal: bad revision but the check never completes

Describe the bug
Gitstream fails with fatal: bad revision but the check never completes and stays in the running/in progress state.
To Reproduce
Create a fatal: bad revision error
Expected behavior
If a fatal: bad revision error occurs, the gitstream check fails
Screenshots

Additional context
Error: {"message":" \n Git command failed. reason: Command failed: cd repo && git diff sc-52094/25-message-details-total-revenue-card'...'marketing-overview-file-refactor' -- .cm/*.cm\nfatal: bad revision 'sc-52094/25-message-details-total-revenue-card...marketing-overview-file-refactor'\n"}

Error: Process completed with exit code 67.

Add any other context about the problem here.
Just some questions about this bad revision error, and it might be silly but

is there a best way to avoid this when it seems fine locally and in github but then gitstream is throwing the error?
it seems to mostly (maybe only) occur when a feature branch is pointing at another feature branch, rather than develop

it still seems to occur on these PRs even after the base automatically changes to develop

Feel free to close this if it will be covered by #62 - seems like it might be but wanted to make sure and also ask my questions about the bad revision error in case I can get some clarity on it

A fail-safe mechanism is implemented and should prevent this. Under the hood what happens here is:


Hello Aex12

I am trying to used your stuff for 0.10.48 nvm node.. can you help me? I can't get it working, maybe you can make support for it?
Peace!

0.10 is seriously old, there is no reason to have backwards compatibility that far. Use NVM to upgrade!


Inherit structuredidentifier from relaton-iso

Like relaton-iec, relaton-bsi should be inheriting its encoding of structuredidentifier from relaton-iso, instead of inheriting the base class from relaton-bib. In particular, in https://github.com/metanorma/metanorma-bsi/issues/300 , relaton-bib is failing to parse structuredidentifier/project-number, which I am generating because I expect relaton-bsi to be the same as relaton-iso on that point. See the grammar for relaton-bsi, which also uses structuredidentifier/project-number.

Is holding up PDF generation...


Add micro syntax and link to awesome cooklang recipes

Add links to cooklang-micro syntax highlighting and awesome cooklang recipes.

Thanks @nicholaswilde. Sorry, somehow I missed this.

Thanks @nicholaswilde. Sorry, somehow I missed this.

No worries!


Add options force_date and free_structure to enable more generic use for this action.

Regarding #35 and #34
But this PR only works for the notebooks now.

@hamelsmu please feel just to drop the whole idea (i.e to delete the PR and the two issues)

Ok please discuss feature requests in the fastai forums thank you

Didn’t mean to close the PR just wanted to say let’s discuss in forums and then come back here

I vote for adding this feature into project like an option. Personaly I interested in free structure.
I'd like to create such blog:

Linear regression:

...;
...;
...;

KNN:

...;
...;
...;

...


Not working HS - ticket nr. 258202

Hello guys,
because your support via mail is too slow i am asking you for help here.
Please check ticket 258202
First launch of HS was OK but after few hours it went offline and since that time it is not working, maybe it could be online for some time because i have tried to restart HS many times, but now it seems dead at all. Also green LED turn off.
you asked for many pics of components inside and also for video how LEDs inside working, but no response since that time.
here is link for video:
https://drive.google.com/file/d/1bVVRS_oLp1cDvDRlMD-P8M6MHNxxLa3l/view?usp=sharing
we are solving this issue almost month and i am tired with that...so if this has no solution, please give me the contact person for claim.
thank you

If you have a ticket open this will be dealt with directly via the ticket.


Multi lib support build in Angular 6

Bug Report or Feature Request (mark with an x)
- [X] bug report -> please search issues before submitting
- [X] feature request

Area
- [X ] devkit
- [x] schematics

Versions
Angular CLI: 6.0.8
Node: 8.9.1
OS: darwin x64
Angular: 6.0.7
Repro steps
I am not even sure if this is a bug or more it should be a feature or I am missing something.
I have multi lib project created with libraries called: A, B. B library imports some modules from A.
When I run ng build B  it fails as it is missing dependencies that exists inside A. So I need to run manually A and then B to make all works.
And I am not sure how to configure existing project to trigger build for  project A first and then B. Are there a way how to specify this dependency?
Desired functionality
Option to specify library depedency so when I build project B it will automatically build project A or some kind of option to specify ng build A,B
Mention any other details that might be useful

There isn't a way currently to recursively build projects, but this is something we want to add.

Maybe possible using TS 3.0 Project references? https://blogs.msdn.microsoft.com/typescript/2018/07/30/announcing-typescript-3-0/#project-references

@intellix it helps, but is not enough by itself. Building a library is not just a matter of running typescript. Other things, like css preprocessors and ngc also need to run.

Relates to #11023

Duplicate of https://github.com/angular/angular-cli/issues/11002


issue-1682 Make gv-omas ffdc compliant

Exposing to requester incoming known checked exceptions from OMRS. The rest are collected into the GlossaryViewOmasException with request data
Signed-off-by: Liviu Popa<EMAIL_ADDRESS>

#1682


feat: Add Marubozu and CandleStick wrappers

Description

Add CandleResult and CandleProperties.
Add Marubozu.

Checklist

[x] My code follows the existing style, code structure, and naming taxonomy
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have performed a self-review of my own code and included any verifying manual calculations
[x] I have added or updated unit tests that prove my fix is effective or that my feature works, and achieves sufficient code coverage.  New and existing unit tests pass locally and in the build (below) with my changes
[x] My changes generate no new warnings and running code analysis does not produce any issues
[x] I have added or run the performance tests that depict optimal execution times
[x] I have made corresponding changes to the documentation

Need to rebase.

Also, the tests for Marubozu does not check the values on CandleProperties. Maybe need to update the tests in the future..?

Is it available to use ConvertToCandleResults() outside the class in C#? It's applied internal keyword so that it's not able to access from Python. If you have plan to support it as a public API, I will implement it on Python-side.

Is it available to use ConvertToCandleResults() outside the class in C#? It's applied internal keyword so it's not able to access from Python. If you have plan to support it as a public API, I will implement it on Python-side.

I did not consider making it public, yet.  I’ll be taking a look at certain internal tools when I provide some examples on how to use the library to make custom user indicators.  It might be helpful in that use case.

Oh, I meant that ConvertToCandleResults() is using at tests for CandleProperties, so I need it for testing in Python.

Oh, I meant that ConvertToCandleResults() is being used at tests for CandleProperties, so I need it for testing in Python.

Oh, I see.  Now that I think about it, since it’s internal, there’s probably no need to re-test the CandleProperties in the Python wrapper?

Ah, now it makes sense, since CandleProperties just supports a few more values derived from OHLCV. Okey

Then, this pr is ready to be merged, review please.


Legend (and hint and error message) for the group of radios / checkboxes announced when focusing a conditionally revealed field whilst using VoiceOver

Conditionally revealed controls are inside the <fieldset> that wraps the radios / checkboxes.
This could potentially cause confusion, as in some screen readers the label (the contents of the legend) and any other text associated with the fieldset using aria-describedby (such as a hint or error message) would be read out when the revealed.

For example, when focusing the conditionally revealed 'email address' field in the example from the Design System (above), Voiceover with Safari (14) on macOS (10.15.7) announces:

"Email address, edit text, How would you like to be contacted? Select all options that are relevant to you. You are currently on a text field. To enter text in this field, type."

The hint text "Select all options that are relevant to you" applies to the set of checkboxes, and does not make sense in the context of the email address field.

Testing with the list of assistive technology / browser combinations in the service manual (plus NVDA / Chrome and JAWS / IE as the next most popular desktop screen reader combinations according to the most recent WebAIM screen reader survey) this seems to mostly affect Voiceover, at least in terms of the way the legend is read out.
No other screen reader read out the legend from the parent fieldset when focusing the text field (at least as tested with the default settings).

Show full testing output

Assistive technology / browser combination
Result

JAWS 2020 / Chrome
✅Email address, edit, type of text

NVDA 2020.3 / Firefox 81
✅Email address edit has autocomplete blank

NVDA 2020.3 / Chrome 86
✅Email address edit blank

JAWS 2020 / IE 11
✅Email address, edit, type of text

Voiceover / Safari (macOS Catalina)
❌"Email address, edit text, How would you like to be contacted? Select all options that are relevant to you. You are currently on a text field. To enter text in this field, type."

Voiceover / Safari (iOS 14)
❌Email address, text field, Select all options that are relevant to you. Double tap to edit.

Talkback / Chrome 71 (Android 7)
✅Email address edit box. Double tap to enter text.

I've updated the title of the issue to focus on the behaviour rather than the implementation, and the fact that it mostly seems to affect VoiceOver.


 Needs a complete refactor before moving forward. Need to re-evaluate all data structures.

Redux implementation for all major data structures in the state, including jobs, aois, tiles, session via abb4f49fb52a23a28ea4f322c4a770e9695b0416


Better Model Spefication.

I feel like the model specification is missing some information that makes the implementation of each method possible.
Right now it seems that each method signature (the parameters it recieves) and the expected output is missing. The examples provided seem a little too vague.
For instance the saveToken method states "Must return an object with the request user, client and scope" but no info is provided regarding what parameters it takes in and the example output is { accessToken: '123dsd21eydgeweg2e' } which does not seem to match what is stated on the description.
Is there any info I'm missing? Thanks.
P.S: Also this might be related to #287, since it would be much more easy to implement based off the examples.

+1

+1 - it's really annoying. I am trying to implement it and have no idea, what is wrong

SQL:
model: https://github.com/manjeshpv/node-oauth2-server-implementation/blob/master/components/oauth/models.js
ORM: https://github.com/manjeshpv/node-oauth2-server-implementation/tree/master/components/oauth/sqldb
Mongo:
Dump: https://github.com/manjeshpv/node-oauth2-server-implementation/tree/master/components/oauth/mongodb
MongooseODM: https://github.com/manjeshpv/node-oauth2-server-implementation/blob/master/components/oauth/mongo-models.js

+1 thanks @manjeshpv for those links they are very helpful. I still feel it would be good to update the model specification to explicitly show what is needed for each function and when (and how) to fail the challenges.

I wrote a full model specification for myself, detailing how the signature of each function is supposed to look and which function is required by which grant flow. Also, there's a minimal, annotated example for every function.
This currently includes functionality added by pending PRs, so I'll have to see what made it into the master so far and what hasn't. Note to myself: Might be a good idea to submit several PRs; one for the current master and one for every PR that adds to/changes the model.
I'll put that in an appropriate format and submit PR(s) sometime this week.

Hi maxtruxa,
Any progress on those PR's with better docs? could really use some
Paul

Documentation now includes a model overview giving a brief overview over the various supported grants and indicating which model functions are used by which grant.
The model specification describes what is expected from each model function in detail.
Please direct any suggestions for improvements to the PR (#369). 😄


Scalar is not exported as part of arrow (it is only exported in arrow_array)

Describe the bug
the newly introduced Scalar is not exported as part of arrow (it is only exported in arrow_array)
https://docs.rs/arrow-array/46.0.0/arrow_array/struct.Scalar.html
While I appreciate the desire to have more granular crates in arrow, forcing end users to explicitly use subcrates (and thus also patch them / keep them up to date) is inconvenient
To Reproduce
Look for scalar in Arrow
https://docs.rs/arrow/latest/arrow/index.html?search=scalar
Expected behavior
Perhaps a good way to fix this would be to explicitly re-export all of arrow_array rather than just a subset https://github.com/apache/arrow-rs/blob/master/arrow/src/array/mod.rs 🤔
Additional context
I found this while working through https://github.com/influxdata/influxdb_iox/pull/8577 (the upgrade to arrow 46 in IOx) and updating to use the new kernels

I definitely think it would be good to revisit the layout of the top-level arrow crate to better reflect the crate topology. Ideally as you say it would just re-export the underlying crates as is

label_issue.py automatically added labels {'arrow'} from #4791


Certified Associate Java® SE 8 Programmer I Study Guide Exam 1Z0-808

TABLE B.1 План по изучению материалов для сдачи экзамена по Java
по книге Jeanne Boyarsky, Scott Selikoff
Date Task

[x] February 1 - February 11 Read Introduction, Appendix B, and Chapter 1
[x] February 12 – February 14 Answer Chapter 1 Review Questions
[x] February 15 – February 25 Read Chapter 2
[x] February 26 – February 28 Answer Chapter 2 Review Questions
[x] March 1 – March 11 Read Chapter 3
[x] March 12 – March 14 Answer Chapter 3 Review Questions
[x] March 15 – March 25 Read Chapter 4
[x] March 26 – March 28 Answer Chapter 4 Review Questions
[x] March 29 – April 9 Read Chapter 5
[x] April 10 – April 12 Answer Chapter 5 Review Questions
[x] April 13 – April 23 Read Chapter 6
[x] April 24 – April 26 Answer Chapter 6 Review Questions
[x] April 27 – May 5 Take practice exams and practice with flashcards
[ ] May 6 Take exam

[ ]

[ ]

[ ]

http://enthuware.com/forum/viewtopic.php?f=2&t=1468&start=10

нужно купить за $10 1Z0-808 JA+ V8
http://enthuware.com/

TABLE B.2 План по изучению материалов для сдачи экзамена по Java JCP
по книге Jeanne Boyarsky, Scott Selikoff
Date Task

[ ] May 22 - May 28 Read Introduction, Appendix B, and Chapter 1
[ ] May 29 - May 31 Answer Chapter 1 Review Questions
[ ] June 1 – June 11 Read Chapter 2
[ ] June 12 – June 14 Answer Chapter 2 Review Questions
[ ] June 15 – June 25 Read Chapter 3
[ ] June 26 – June 28 Answer Chapter 3 Review Questions
[ ] June 29 – July 8 Read Chapter 4
[ ] July 9 – July 11 Answer Chapter 4 Review Questions
[ ] July 12 – July 22 Read Chapter 5
[ ] July 23 – July 25 Answer Chapter 5 Review Questions
[ ] July 26 – August 4 Read Chapter 6
[ ] August 5 – August 7 Answer Chapter 6 Review Questions
[ ] August 8 – August 18 Read Chapter 7
[ ] August 19 – August 21 Answer Chapter 7 Review Questions
[ ] August 22 – August 31 Read Chapter 8
[ ] Septemper 1 – Septemper 3 Answer Chapter 8 Review Questions
[ ] Septemper 4 – Septemper 14 Read Chapter 9
[ ] Septemper 15 – Septemper 17 Answer Chapter 9 Review Questions
[ ] Septemper 18 – Septemper 28 Read Chapter 10
[ ] Septemper 29 – October 1 Answer Chapter 10 Review Questions
[ ] October 2 – October 9 Take practice exams and practice with flashcards
[ ] October 10 Take exam

https://mincong-h.github.io/2017/04/06/oca-review-8-notes/

http://www.javaranch.com/game/game2.jsp

купил треенажер экзамена!

прочитать
Asynchronous Messaging Made Easy With Spring JMS


Edit CRD Causes Errors

If you add a CRD like the following
apiVersion: helm.cattle.io/v1
kind: HelmChart
metadata:
  name: traefik-test
  namespace: kube-system
spec:
  chart: stable/traefik
  version: 1.64.0
  set:
    rbac.enabled: "true"
    ssl.enabled: "true"

let the helm jobs finish and you will have a new traefik pod. now edit the CR and change the version to 1.65.0. The onchange comes through and it tries to spin up a job but since the job already exists from the previous run it will try to rewrite the existing one and some job fields are immutable and you will get something like the following:
E0716 12:58:30.843044       1 controller.go:118] error syncing 'helm-controller/grafana': handler helm-controller: failed to update helm-controller/helm-install-grafana batch/v1, Kind=Job for helm-controller helm-controller/grafana: Job.batch "helm-install-grafana" is invalid: spec.template: Invalid value: <<<large blob of job data>>>  field is immutable, requeuing

need to handle edits a bit nicer and figure out how to do official upgrades to charts.

We'd like to migrate from using Terraform Helm provider to using this controller but unfortunately can't move forward until chart version upgrades work.
Happy to contribute a pull request to fix the immutable field issue. However, after a few hours spent reading through the code and trying various changes it seems the fix needs to be made in the wrangler codebase. Initial thoughts were that Wrangler needs to support Kubernetes API patch operations.

I have done little bit more digging and found that somewhere in the library adding NO_PROXY env variable, not the one defined in here https://github.com/rancher/helm-controller/blob/master/pkg/helm/controller.go#L334.
This env var is added even before helm controller properly started. So the next time controller trying to update, if the env doesn't have NO_PROXY setup, patch operation is trying to remove that from env.


x/tools/gopls: consider to turn off buildvcs inside gopls

gopls version
golang.org/x/tools/gopls v0.8.1
compiled with go1.18 (google-golang)
What did you do?
We have a project that has a third-party go project as its git submodule.
my_project
  +--- go.work
  +--- their_project
               +--- go.mod
               +--- main.go

go.work has only ./their_project in it.
I  opened vscode from my_project , so gopls sees my_project as the workspace root.
What did you expect to see?
Working normally.
What did you see instead?
All go files in the their_project directory get errors with source go list.
gopls log:
[Trace - 23:38:54.242 PM] Received notification 'textDocument/publishDiagnostics'.
Params: {"uri":"file:///home/hakim/projects/my_project/their_project/cmd/tester/main.go",
"diagnostics":[{"range":{"start":{"line":0,"character":0},"end":{"line":0,"character":0}},
"severity":1,
"source":"go list",
"message":"error obtaining VCS status: main package is in repository
\"/home/hakim/projects/my_project/their_project\" but current directory is in repository
\"/home/hakim/projects/my_project\"\n\tUse -buildvcs=false to disable VCS stamping."}]}

I understand this is an error message from go, and I can work around by disabling the buildvcs flag.
However, what's puzzling is, if I run go list from their_project, it runs without an error.
$ go list ./triage-party/...
error obtaining VCS status: main package is in repository "/home/hakim/projects/my_project/their_project" but current directory is in repository "/home/hakim/projects/my_project"
        Use -buildvcs=false to disable VCS stamping.
error obtaining VCS status: main package is in repository "/home/hakim/projects/my_project/their_project" but current directory is in repository "/home/hakim/projects/my_project"
        Use -buildvcs=false to disable VCS stamping.

$ cd ./triage-party; go list ./...
go list ./...
github.com/they/their_project/cmd/server
...

I don't know what's right default behavior for the go command (this is probably a niche use case) cc @bcmills
but these confusing error messages make me think gopls disable vcs build stamping by default when running go commands.
(why does gopls have to care about build stamping?)

(why does gopls have to care about build stamping?)

This is somewhat related to #29666, in that go list is doing (a lot of!) extra work to compute the .Stale field for executable targets, which gopls probably doesn't really care about in this context.

If gopls doesn't care about staleness (as I suspect), then -buildvcs=false seems like a fine workaround.

This may also be another situation in which -buildvcs=auto (which will be the default after #51748 is addressed) should omit the VCS metadata rather than emitting an error.


the backprop and adaptive learning rate code

I wonder know how you realize your new back propagation and the adaptive learning rate in each layer?
will you open source the relevant code in future?

The provided code uses cdist to compute the L1 distance https://github.com/huawei-noah/AdderNet/blob/c49d06dbf5da44b9b754e30da14138097837707a/adder.py#L28
To realize the new back propagation, you can follow https://pytorch.org/docs/master/notes/extending.html to write a function to replace cdist, where the forward is the L1 norm while the backward use the new back propagation and the adaptive learning rate.


Versioned Formula is blocking re-install or fresh install

Describe the bug
The line https://github.com/shivammathur/homebrew-php/blob/master/Formula/php%407.4.rb#L26 casues brew install<EMAIL_ADDRESS>to fail with: Error<EMAIL_ADDRESS>has been disabled because it is a versioned formula!
PHP versions
7.4 (did not test others)
To Reproduce
brew install<EMAIL_ADDRESS>Work around*
brew install<EMAIL_ADDRESS>Will show: Warning<EMAIL_ADDRESS>has been deprecated because it is a versioned formula!
Expected behavior
For installation to complete
Screenshots/Logs
Error<EMAIL_ADDRESS>has been disabled because it is a versioned formula!
Additional context
N/A
Are you willing to submit a PR?
Possibly if I can get it to work

@sschueller
<EMAIL_ADDRESS>still exists in homebrew-core tap. So brew tries installing it from there when you run brew install<EMAIL_ADDRESS>As it is disabled there, you get the error.
<EMAIL_ADDRESS>For using the formula from this tap, It is advised to always use the full qualified formula name i.e<EMAIL_ADDRESS>https://github.com/shivammathur/homebrew-php#install-php
For this issue to be fixed the disabled formula should be removed from the core tap. Please raise this issue there.


Add tuple-remove-fst, DoWhile-output , DoWhile-output-ith.

tuple-remove-fst take a tuple and return a tuple that the first Non-Tuple(TNil) typed term is removed.
DoWhile-output give the output tuple of a loop, which is the body with pred removed.
DoWhile-output-ith give the ith-output of a loop.

Actually I think I don't need tuple-remove-fst for now. Shall I close it?

sure


Feature request: add & distribute py.typed marker file

Hello!
I'm using this library in my codebase and I noticed that this repo already has type annotations and is running mypy in CI. Could a py.typed file be added to the repo and included in package_data (per https://peps.python.org/pep-0561/) so mypy is able to transitively type check my codebase against the type annotations in this repo?
I also kicked off the process internally at $WORK to get approval to contribute to this repository. If you are open to this feature request, I am happy to send a PR myself if you don't have time to implement this.

there is no public python api, depend on the internals at your own risk

woops wrong repo