Rename RAZZLE_TESTING_ADDONS environment variables

to ADDONS, make it work for released add-ons.

@stevepiercy what about now?

OK, I think this is better. What do you call the thing that checks files and the env var in order? "Resolver"? "Loader"? "Razzle"?
I'll push some commits to clean up language and formatting.

@sneridagh would you please review the entire section that I just committeed? I think I finally understand the intent of this.
The only question I have is whether the term is "profile" or "profiler". I could not find a definition for either term in the glossary or source code, so I went to React's docs: https://reactjs.org/docs/profiler.html

OK, I think this is better. What do you call the thing that checks files and the env var in order? "Resolver"? "Loader"? "Razzle"?

It's Volto who loads it in the right order :) Going deeper here is not needed, in my opinion.

The only question I have is whether the term is "profile" or "profiler". I could not find a definition for either term in the glossary or source code, so I went to React's docs: https://reactjs.org/docs/profiler.html

@stevepiercy The add-ons install "profiles" are like the Plone Generic Setup ones. That's why we chose the naming like this.

The only question I have is whether the term is "profile" or "profiler". I could not find a definition for either term in the glossary or source code, so I went to React's docs: https://reactjs.org/docs/profiler.html

@stevepiercy The add-ons install "profiles" are like the Plone Generic Setup ones. That's why we chose the naming like this.

Should we define the term "profile" in the Glossary? I think we should make it clear that a profile in Volto and a profile in GenericSetup are similar, but have some differences. I could create a new issue for that, so we can merge this one and move on.

Looks great! Thanks a lot @stevepiercy !


citra: nightly 1907 -> 1963, canary 2484 -> 2573

Description of changes
Bump versions.
citra: nightly 1907 -> 1963
citra: canary 2484 -> 2573

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Friendly ping @NickCao


Update memcache dependency.

This fixes a crash issue when an instance loses its memcache
connection.
Fixes #1098

RSLGTM


Sofia/bug 3052048

Description
In the Python SDK live tests, the test_update_phone_number_capabilities_with_invalid_phone_number is expected to fail with error 404 but is failing with 403 instead. This has been affecting only the INT environment since October 18.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Add a license to the repository

I recently came across your project Qiskit_Quantum_Computing and I noticed that the repository github.com/Tahahaha7/Qiskit_Quantum_Computing currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Hi @cbjuan!
I just added the license
I see that Qiskit has gotten even more interesting since my last time experimenting with it, can't wait to learn more about the new updates
Thank you!

Thank you!


Use fast-stable-stringify

That module claims to be faster, but is under GPL-3.
See also https://github.com/epoberezkin/ajv/issues/340

It appears that GPL cannot be used inside MIT as GPL is more restrictive, according to this: http://stackoverflow.com/questions/3902754/mit-vs-gpl-license
@nickyout would you consider changing it to MIT?
btw, the build in fast-stable-stringify fails...


Fix stack_effect computation of CALL_FUNCTION_EX on 3.6.0

Stack effect computation is broken for CALL_FUNCTION_EX on python 3.6.0. I submitted a fix to cpython (https://github.com/python/cpython/pull/202) that got merged so it should be okay in 3.6.1.

Codecov Report

Merging #22 into master will decrease coverage by -0.11%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
- Coverage   92.94%   92.83%   -0.11%
==========================================
  Files          15       15
  Lines        2382     2388       +6
==========================================
+ Hits         2214     2217       +3
- Misses        168      171       +3

Impacted Files
Coverage Δ

bytecode/instr.py
92.34% <57.14%> (-1.34%)
:x:

Continue to review full report at Codecov.

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

I will merge in a week if nobody complains.

Is it worth to add a workaround for CPython bug? It will be fixed in the next CPython bugfix release. It may be fixed even in 3.6.0 distributions that backport bug fixes.

I concur with Serhiy, I don't think that it's worth it to monkey-patch Python to fix a bug which affects a single Python version (3.6.0, will be fixed in 3.6.1 scheduled in less than 1 month).

Given the feedback, I am closing.


Update asciidoctor version and add bibtex extension

Motivation for this change
Closes #106364
Things done

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

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

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

A lot of packages use asciidoctor to build their documentation, so the nixpkgs-review is very large.


ocl: fix rgba access stride issue in blend kernel

each work item reads 2x2 rgba pixels, so both x and y access stride of read_image should be 2.

+1

merged as 684272026f5533300efc02e77ec125b945965c22


How to solve the following wrong display problem?

same problem here

Same issue. Bumping thread.

Same issue on my end.


How to add text while streaming and encoding video

Dear PedroSG94
Please let me know is there any way to add text to the live stream and encode the video?
Thanks

Hello,
You have code example in the app example.
You can add text using an Android view:
https://github.com/pedroSG94/RootEncoder/blob/master/app/src/main/java/com/pedro/streamer/utils/FilterMenu.kt#L99
Or text directly:
https://github.com/pedroSG94/RootEncoder/blob/master/app/src/main/java/com/pedro/streamer/utils/FilterMenu.kt#L308
You can test it using the app example, rotation example and use the menu button

Thanks
I did it
However when i add multi text with different positions it only shows the last text, please tell me how can i do it
Also can i add datetime to video streaming

hello,
That is because you replace the last text. You can set multiple filters using addFilter method:
https://github.com/pedroSG94/RootEncoder/blob/master/library/src/main/java/com/pedro/library/view/GlInterface.java#L83

Hi,
Please tell me how i can call the addFilter method ?
Thanks

Exactly the same way you are using setFilter method you can call addFilter.


add Terraform Test to module

provide acceptance tests using terraform test. This conflicts with #3 (proposed)
Test Scenarios:

validates that tags come back in the awscc format regardless of the given format and tags_aws come back in the aws format, regardless of the given format.
validates tags work with resources (aws_vpc & awscc_ec2_vpc)
validates account names are mapped correctly

Ive removed the internal/ but we can discuss if theres value creating a test that actually makes the same iam resources.
Note
terraform test requires the module references be:

in the same dir as the tests
source = "../.."
these are currently hard requirements. I was able to get around them by creating symlinks to our examples hcl config files

Note: https://github.com/hashicorp/terraform/issues/30071 is relevant.

Closing - pending future releases of terraform test we can re-evaluate


Add context handler

Add a handler that accepts a context.Context and extracts error context from it.

Partially fixed in #49
Extracting info from context will be added in a subsequent PR.


trouble with basset_motifs.py

Hi again,
After happily finishing the training procedure and (model testing running fine) ... I am running into troubles with basset_motifs.py:
I basically get a couple of these error:
/home/ron/anaconda2/lib/python2.7/site-packages/weblogolib/logomath.py:114: FutureWarning: comparison to None will result in an elementwise object comparison in the future.
if  self._mean ==None:
Before the script stops working throwing (aftr 25 motif steps):
Traceback (most recent call last):
File "/home/ron/tools/Basset/src/basset_motifs.py", line 659, in
main()
File "/home/ron/tools/Basset/src/basset_motifs.py", line 125, in main
filter_possum(filter_weights[f,:,:], 'filter%d'%f, '%s/filter%d_possum.txt'%(options.out_dir,f), options.trim_filters)
File "/home/ron/tools/Basset/src/basset_motifs.py", line 558, in filter_possum
while np.max(param_matrix[:,trim_start]) - np.min(param_matrix[:,trim_start]) < trim_t:
IndexError: index 19 is out of bounds for axis 1 with size 19
My learned model files are still pretty small (68 MB before 129 MB after decuda ...) Not sure if that might be a problem or hint to a faulty training run ...
Cheers,
Ron

It looks like there's a bug in the trim code. I uploaded a new version of the script that should fix it.

Looks better thanks!
Still worrying messages from the log:
Processing query 235 out of 235
Estimating pi_0 from all 1468 observed p-values.
Estimating pi_0.
Estimated pi_0=1
/home/ron/anaconda2/lib/python2.7/site-packages/sklearn/preprocessing/data.py:167: UserWarning: Numerical issues were encountered when centering the data and might not be solved. Dataset may contain too large values. You may need to prescale your features.
warnings.warn("Numerical issues were encountered "
/home/ron/anaconda2/lib/python2.7/site-packages/sklearn/preprocessing/data.py:184: UserWarning: Numerical issues were encountered when scaling the data and might not be solved. The standard deviation of the data is probably very close to 0.
warnings.warn("Numerical issues were encountered "
basset_motifs_predict.lua dnase_with_prim_erythroid_cnn_best.decuda.th motifs_out/sample.h5 motifs_out/model_out.h5
Filter 0
Filter 1
##########################################
Filter 296
Filter 297
Filter 298
Filter 299
6 segments of length 97
6 segments of length 97
Traceback (most recent call last):
File "/home/ron/tools/Basset/src/basset_motifs.py", line 668, in
main()
File "/home/ron/tools/Basset/src/basset_motifs.py", line 192, in main
plot_filter_seg_heat(filter_outs, '%s/filter_segs_raw.pdf'%options.out_dir, whiten=False)
File "/home/ron/tools/Basset/src/basset_motifs.py", line 516, in plot_filter_seg_heat
sns.clustermap(filter_seqs[:,seqs_i], metric=dist, row_cluster=True, col_cluster=True, linewidths=0, xticklabels=False, vmin=hmin, vmax=hmax)
File "/home/ron/anaconda2/lib/python2.7/site-packages/seaborn/matrix.py", line 1169, in clustermap
**kwargs)
File "/home/ron/anaconda2/lib/python2.7/site-packages/seaborn/matrix.py", line 1008, in plot
row_linkage=row_linkage, col_linkage=col_linkage)
File "/home/ron/anaconda2/lib/python2.7/site-packages/seaborn/matrix.py", line 941, in plot_dendrograms
axis=1, ax=self.ax_col_dendrogram, linkage=col_linkage)
File "/home/ron/anaconda2/lib/python2.7/site-packages/seaborn/matrix.py", line 684, in dendrogram
label=label, rotate=rotate)
File "/home/ron/anaconda2/lib/python2.7/site-packages/seaborn/matrix.py", line 502, in init
self.dendrogram = self.calculate_dendrogram()
File "/home/ron/anaconda2/lib/python2.7/site-packages/seaborn/matrix.py", line 582, in calculate_dendrogram
color_threshold=-np.inf)
File "/home/ron/anaconda2/lib/python2.7/site-packages/scipy/cluster/hierarchy.py", line 2230, in dendrogram
is_valid_linkage(Z, throw=True, name='Z')
File "/home/ron/anaconda2/lib/python2.7/site-packages/scipy/cluster/hierarchy.py", line 1412, in is_valid_linkage
name_str)
ValueError: Linkage 'Z' contains negative indices.
################

I know I've seen that before, but I can't replicate it at the moment. I would guess that one of the filters isn't activating at all and causing the statistical code to choke. I never got much value out of those heat map plots, so I'm going to turn them off by default and keep my eye out for an opportunity to fix the bug next time that I see it.

Sorry for the delay and thanks for the workaround, the same example now runs without complaints.


use of reportes in karma

why we use reporters in karma

Closing for now. Please file a new issue with more details if this is still relevant.


kindergarten-garden: add canonical data

Closes #565
Notes:

The default students indicated in the README are assumed, and alternate students are indicated as input when appropriate.
While some tracks tested all 12 students for the full garden, it only seemed necessary to include a few tests here.

Thanks for the feedback! I'll get right on those. Should I just push the new commits, and squash them when it's all approved?

@holdenout yes, please push the new commits, then squash away 😄

It appears that there now is a commit in this PR that modified the rail-fence-cipher canonical data. Is that correct?

Please don't include my commit.

Oops, I'm relatively inexperienced with git and did not mean to do that. I'll fix it up.

Don't worry, we've all been there I'm sure. Feel free to yell if you need help (what I would personally do is to use an interactive rebase to remove it, but there could be various different ways to accomplish the goal so don't limit yourself to only what I say)

Reminder to myself to look at this again (will delete this comment)

I have no objections to this being merged. :+1:
(but have not reviewed it closely enough to hit the approve button.)

"implementations": [
  {
    "track_id": "csharp",
    "url": "https://github.com/exercism/xcsharp/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "clojure",
    "url": "https://github.com/exercism/xclojure/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "ecmascript",
    "url": "https://github.com/exercism/xecmascript/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "elixir",
    "url": "https://github.com/exercism/xelixir/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "fsharp",
    "url": "https://github.com/exercism/xfsharp/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "go",
    "url": "https://github.com/exercism/xgo/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "haskell",
    "url": "https://github.com/exercism/xhaskell/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "javascript",
    "url": "https://github.com/exercism/xjavascript/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "lua",
    "url": "https://github.com/exercism/xlua/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "perl5",
    "url": "https://github.com/exercism/xperl5/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "python",
    "url": "https://github.com/exercism/xpython/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "ruby",
    "url": "https://github.com/exercism/xruby/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "scala",
    "url": "https://github.com/exercism/xscala/tree/master/exercises/kindergarten-garden"
  },
  {
    "track_id": "swift",
    "url": "https://github.com/exercism/xswift/tree/master/exercises/kindergarten-garden"
  }
]

C#/F# have search for a kid not in the class ("Potter" for four plants).
JS/ES6/Ruby create multiple gardens with the same kids then check that the right plants are assigned to the right kids after both are created, ensuring that the multiple gardens don't improperly overwrite each other's data. (Perl has cases with the same data but doesn't create two gardens simultaneously)
Lua checks that the kids and plants are case-insensitive.
Go has a bunch of errors cases (unequal number of rows, missing newlines, odd number of columns, duplicate kid name, something other than RGCV in the diagram, kid not in the class like C#/F#)
If anyone really wants these cases in the JSON, speak up soon. I'm not considering them blockers but they are useful to note down.

I hate to block my own PR, but I'd be willing to either add some of that in (either as tests or just in the comments). I suppose another option could be merging and then raising an issue for those little bits. Not sure what would work best for the exercism workflow. Or it could be left as is.

Well these are good questions. Depending on what the exercise is trying to teach (fun fact! it seems from https://github.com/exercism/discussions/issues/78 that we don't even know!) those tests may or may not be appropriate.
The error tests and case insensitivity test make sense and could be encoded in JSON, but don't appear to be strictly necessary.
The test with two simultaneous gardens would at least need a comment to go along with it.
I'm happy to stick with these as a first pass. Though note that I only did this exercise in one track so I am not the most knowledgeable about all the approaches!
If in the future some track creates a test generator and finds that it deletes a test that track thought was essential, at that time we may see a proposal to add those tests to x-common.

Thanks for your work!

🎉 thanks @holdenout!


Clarifying BlocksByRange

When discussing with @cemozerr, we realized that it was not abundantly clear that BlocksByRange should or should not guarantee to return blocks in the same chain (assuming newer than latest finalized). For example take the following:

Time 1: canonical view is A, B, C, D
Time 2: start serving request — send A, B
Time 3: local canonical switches to A, B, C’, D’
Time 4: might be any of the following
a. continue serving same request with original chain -- send C, D
b. continue serving same request with new chain -- send C’, D’
c. stop serving request

"Clients MUST respond with blocks from their view of the current fork choice" is not clear as to whether you fix your view before sending the response or if you should change your view halfway through the request. And thus it is not clear if sending an "inconsistent" chain is okay or malicious.
Chatted with @AgeManning, Lighthouse current does (a). If a peer responds with (a) or (c), all good. If a peer of lighthouse responds (b), it is considered malicious.

Chatted with @prestonvanloon and realized that the spec was ambiguous in another way. (Assuming C' and D' exist when the request begins) Prysm currenty sends A, B, C, C', D, D' (i.e. the entire sorted-by-slot block tree on the range requested (assuming after finalized)). Although this was not the intended behaviour of the authors of the spec, it is not immediately obvious that it is not a desirable behaviour.
On the one hand, more information can be better (send whole tree, and you decide), but on the other hand, this doesn't provide us with strict bounds on the amount of messages an honest node might respond with.   This was the original intention of the spec, but might be worth just quickly discussing the tree approach
If the above is to send just a single chain, MUST the chain be consistent? If so, should clarify that a non-consistent set of blocks is malicious

cc: @arnetheduck @nisdas

Another thing: we don't actually specify that blocks MUST be valid in BeaconBlocksByRange or BeaconBlocksByRoot. That said, I expect most clients would disconnect/de-score a peer that was responding with invalid blocks. Likely worth noting this explicitly


feat: add roles and groups to userProfileDTO and set the message

fixes: https://github.com/appsmithorg/appsmith/issues/24606

/ok-to-test

/build-deploy-preview skip-tests=true

/ok-to-test


Will it be safe to install the deployment agent to on premise production environment

Hi,
We are trying to figure out a secure way to deploy web app from Azure DevOps to our on-premise environments (Dev, QA, Staging, and Prod).
My security team has asked, how installing the agent is safe for our on-premise environments (Dev, QA, Staging, and Prod)? I have googled but didn't find any clue. I wonder if Azure DevOps product team use deployment groups to deploy apps to their on-premise "production" server? Any best practice?
Thanks in advance.

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

ID: 09fa0c14-9937-41bc-cc34-179edb379083
Version Independent ID: 10fe3600-21a5-12db-5e21-5ade3a8fa484
Content: Provision agents for deployment groups - Azure Pipelines
Content Source: docs/pipelines/release/deployment-groups/howto-provision-deployment-group-agents.md
Product: devops
Technology: devops-cicd-release
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

@gautamksr  Thank you for the question.
This issue is more support-related, for support options, please see:

Azure DevOps support
Azure DevOps on StackOverflow
Azure DevOps community

Otherwise, for issues relating to the documentation please include your references in the comments below.

Closing this issue now.


Allow options for file open/create

One can now open and create files in the different netCDF formats (CDF-1,2,5) and classic mode.
Some options are not exposed, as they may be deprecated in future editions.
@samoylovfp Would this be a good approach to your problem? The api would be
let mut file = netcdf::create_with("file.nc", Options::CLASSIC | Options::NETCDF4);

That is perfect!
Many thanks for this.

@samoylovfp I've uploaded the crate with this PR as 0.5.2. Please let me know if something is amiss


Give maintainers permissions to maintain this repo

Is your proposal related to a problem?
Can't currently approve of PR's

Gave access to Team, Admins, Maintainers and Triagers.


Convert new array[0] to Array.Empty

Overview
Instead of allocating 0 length arrays, it's recommended to use the statically allocated empty array instance from Array.Empty.
https://docs.microsoft.com/en-us/visualstudio/code-quality/ca1825?view=vs-2019
Not the biggest deal; I'm not sure if this is worth enforcing a coding guideline over, but something I felt like cleaning up.

/azp run

feel free to merge once the conflict is resolved :)

/azp run


hide keyboard on swipe

Hi! Thanks for great library
How can I hide keyboard on left/right swipe

hey man, did you figure this out?  i was about to log the same issue

@Nathan187 no(

If  remember correctly, the library already had a gestureCompletionBlock.   Just duplicate that for a "gestureStartBlock' and you can "endEditing" in this block.
I used this to put a semi-transparent overlay on top of the center screen while the sidemenu is visible on the left/right.

Closing due to inactivity.


[DO NOT MERGE] Remove AdvancedSearch

Note: Merging blocked until we have added the blue banner to finders and added redirect links for advanced search.
This removes the code required to render AdvancedSearch.
AdvancedSearch is/was a generic finder for rendering
search results filtered by topic and group.
For example, search/advanced?group=guidance_and_regulation&topic=%2Feducation.
Group was a supergroup as defined in the GovukDocumentTypes
and topic was a taxon content item.
These were deprecated because we can provide the same
content via the new supergroup finders.
This was done because it'll make the codebase easier
to work with, and provide a more consistent experience
for end users when they use finders.
We are taking extra steps to ensure that the deprecation
happens gracefully. Advanced search links should continue
to work: they will redirect to the appropriate supergroup
finder. The blue header at the top of topic pages will
be added to the top finders when users come from a topic
page, and the topic facet will be hidden.
https://trello.com/c/bzD1tqoa/764-remove-code-for-topic-page-finders-from-finder-frontend-m

Search page examples to sanity check:

http://finder-frontend-pr-1214.herokuapp.com/search/all
http://finder-frontend-pr-1214.herokuapp.com/search/research-and-statistics
http://finder-frontend-pr-1214.herokuapp.com/search/advanced?group=guidance_and_regulation&topic=%2Feducation
http://finder-frontend-pr-1214.herokuapp.com/drug-device-alerts
http://finder-frontend-pr-1214.herokuapp.com/find-eu-exit-guidance-business

Other finders

I like this a lot!!1!


Add meeting notes from contribution discussion

@zachlatta Comments addressed.

@zachlatta I think the tests broke. Could  I get a  second set of eyes on this?

@zachlatta ping

Sorry about the delay. Merged!


Run in the browser error

Excuse me,  I according to the use of the README.md, run in the browser error.
error: Uncaught Error: Minified React error #37; visit http://facebook.github.io/react/docs/error-decoder.html?invariant=37 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
Do you know what's going on?

hi! The project still in development, so I wouldn't recommend you to use it in your projects, I make changes all the time and it's so unstable.
If you want to play with it just clone repo and use npm run dev to run server in development environment. This will run dev server and webpack builds the scripts with all the source maps and not minified/uglyfied
Running npm start runs server with prebuilded minified scripts, so you can not see full error messages,  console.log's and other


[Geolocation] navigator.geolocation.getCurrentPosition doesn't get executed first time it is called

I've upgraded from RN 0.16.0 to 0.17.0-rc and navigator.geolocation.getCurrentPosition() doesn't get executed any more.
However, if I insert "navigator.geolocation.getCurrentPosition(()=>null, ()=>null)" before the actual call, it does get executed. Maybe this is related to new lazy loading feature?

This was recently fixed with the following commit. https://github.com/facebook/react-native/commit/f48dbbecd658123e6fdaacf1b5762692d4d1d7f5

Great. Thanks!


Add some notes on the discussions and fixed some typos

I added some of the notes that I took that were not yet included in the summary. You can complement / modify as you see fit.
Also, the link https://teachbooks.io/manual/credits.html does not seem to work, is this the wrong website or is it a placeholder?

thanks alex, these are great!
Good catch on the credits page - it was part of a PR in the manual that only recently got merged. Works now!


0.20 fix to master

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


Annotations don't align with youtube downloaded tracks

Hi,
I'd like to use this dataset to train one of my networks. I managed to download all the audio using download_youtube_mp3s.py but the annotations in dataset/beats_and_downbeats seem to be off. I'm using Sonic Visualiser to overlay the annotations on top of the audio, and I've randomly chosen tracks from the dataset but none of them appears to be correctly annotated (not even the ones that are marked with 1.0 alignment score). I can't make use of the Harmonix_melspecs.tgz because my pre processing chain is different.
I understand this problem might be caused by use of different CODECS or compression formats, and the Audio Alignment notebook is here to compensate for that, but it looks to me that the alignment code depends on some local files not available to the public.
I'd like to be able to experiment with this dataset, and for that I would either need some new beat/downbeat annotations that match the audio I've just downloaded from youtube, or a dropbox to the files for which the annotations in dataset/beats_and_downbeats are correct. Having both of them in the same place would be great.
Thanks

Unfortunately, there might be some differences with some of the YouTube versions with the ones used to do the actual annotations. Nevertheless, I am happy to provide additional spectral features computed from the original audio files.. Please, let me know what type of features and their specific parameters and I will create a new link if that works for you.

I guess this works but it doesn't give me much room for experimentation. Currently, I'm using the preprocessing chain in madmom's RNNDownBeatProcessor, namely a multi-resolution (1024, 2048, 4096) filtered log spectrogram & their spectral fluxes laid on top.
downbeats.py
define pre-processing chain
    sig = SignalProcessor(num_channels=1, sample_rate=44100)
    # process the multi-resolution spec & diff in parallel
    multi = ParallelProcessor([])
    frame_sizes = [1024, 2048, 4096]
    num_bands = [3, 6, 12]
    for frame_size, num_bands in zip(frame_sizes, num_bands):
        frames = FramedSignalProcessor(frame_size=frame_size, fps=100)
        stft = ShortTimeFourierTransformProcessor()  # caching FFT window
        filt = FilteredSpectrogramProcessor(
            num_bands=num_bands, fmin=30, fmax=17000, norm_filters=True)
        spec = LogarithmicSpectrogramProcessor(mul=1, add=1)
        diff = SpectrogramDifferenceProcessor(
            diff_ratio=0.5, positive_diffs=True, stack_diffs=np.hstack)
        # process each frame size with spec and diff sequentially
        multi.append(SequentialProcessor((frames, stft, filt, spec, diff)))
    # stack the features and processes everything sequentially
    pre_processor = SequentialProcessor((sig, multi, np.hstack))

It should yield a 1D vector of 314 values for each time frame :D
Thanks a lot for your time and for replying so quickly

Great, thanks for sharing. I created a little script to compute these features, I'm running it right now:
https://github.com/urinieto/harmonixset/blob/master/src/compute_madmom_audio_features.py
Once I have all the features computed and uploaded, I'll post the link here.

Thanks Oriol! Looking forwards to get my hands on that data 😄

Few, this took some time to upload. Here you have them: https://www.dropbox.com/s/ayd15svnibqvcjw/Harmonix_set_madmom_features_20210715.tgz?dl=0 :)

It took some time to download too! Safari just couldn't take it :)) I'll do some alignment checks and let you know if everything looks good. Thanks again for your time!

Hi Oriol,
I'm trying to verify the validity of the annotations & features, and I've noticed (randomly) that some of them seem to be a bit ahead.
Here I'm printing the feature matrix for 0348_blackout-seq.npy. The pictures zoom on the bottom part (Y-axis), the 1024 log-filtered spectrogram which should be the most "time accurate". Is this kind of "error" expected within your dataset?

Here's the code I'm using to print:
import numpy as np
import matplotlib.pyplot as plt
file = open(PATH_TO_ANNOTATION_FILE)
lines = file.readlines()
file.close()
beat_indexes = []
for l in lines:
parts = l.split('\t')
beatpos = float(parts[0]) # taking just the time instants
beat_indexes.append(np.round(beatpos / 0.01)) # divide beatpos by 10ms to find out the beat index
beat_indexes = np.array(beat_indexes, dtype='int')
print(beat_indexes)
features = np.load(PATH_TO_FEATURES_FILE)
max_value = np.max(features)
features[beat_indexes, :] = max_value
plt.imshow(features.T, aspect='auto', origin='lower')
plt.show()
*notice that I'm rounding in order to decide to which frame the beat position given in seconds corresponds, and that may introduce an error of maximum 5ms

Hey Alex, that's an interesting find! That is not expected (i.e., the annotations shouldn't have any type of (negative) offset). If you have the time, I'd encourage you to create a PR with the corrections for a future Harmonix Set version.

I will need to take a bit more time to look at more files and conclude, but I'm happy to open a PR when ready. Btw, before computing the madmom features, did you check that the annotations in dataset/beats_and_downbeats were indeed properly aligned with your waveforms?
For instance, I would be curious to see where do the beat markers land on this section of your "0348_blackout.mp3". In case you are not aware of this, a very easy way to test this is to use Sonic Visualiser, drag the audio file in the window, and then drag the annotations text file as it is, and it will parse the rows and columns. You can then choose to ignore the segments, or beat numbering columns, or use them as labels.

Hi Alex,
This could be due to decoder offset. Depending on the decoder you are using
to load the samples from the compressed audio files, there can be a
different amount of zero padding inserted or truncated from the beginning
of files. This is evident when certain versions of librosa on say linux vs.
macOS. On MacOS it will use the CoreAudio decoder, where on Linux it will
use the FFMpeg decoder. You actually see the same difference in offsets
when loading the same audio file via WebAudio in different browsers (e.g.,
Chrome vs. Safari vs. Firefox), depending on the codec and bitrate used.
All results in the dataset were produced using FFmpeg on Linux IIRC.
Perhaps the features were produced on a different platform?
On Wed, Jul 21, 2021 at 11:27 PM Alex Voina @.***>
wrote:

I will need to take a bit more time to look at more files and conclude,
but I'm happy to open a PR when ready. Btw, before computing the madmom
features, did you check that the annotations in dataset/beats_and_downbeats
were indeed properly aligned with your waveforms?
For instance, I would be curious to see where do the beat markers land on
this section of your "0348_blackout.mp3". In case you are not aware of
this, a very easy way to test this is to use Sonic Visualiser, drag the
audio file in the window, and then drag the annotations text file as it is,
and it will parse the rows and columns. You can then choose to ignore the
segments, or beat numbering columns, or use them as labels.
[image: Screenshot 2021-07-22 at 09 14 30]
https://user-images.githubusercontent.com/32097135/126596840-8cf9bff9-17b7-435c-9e2d-232e7069e6b3.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urinieto/harmonixset/issues/9#issuecomment-884684035,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AELA437F64F6NCTOUNMTLWTTY6235ANCNFSM5AGMMDXQ
.

Yeah, I produced these features using macOS (with the madmom library specified in the info.json file in the tgz file above). It would be ok if all the offsets are consistent (that's an easy fix, likely due to the different decoders used as Matt mentioned), but it looks like this offset only happens in some of the annotations if I understand correctly?

Hi Matt, Oriol
Sorry for the late reply.
Nope, the offset might be present in all the files actually and it is probably because of the decoder (thanks @MCMcCallum), I've encountered this issue in the past. I randomly chose some songs that had a clean beat line such that the test is more obvious.
If the annotations in dataset/beats_and_downbeats were made using FFmpeg on Linux IIRC, then @urinieto should see the offset too, when loading the mp3 and laying the annotations on top.
From my experience with the decoding padding/truncation issue, the decoding offset did not have a constant value. Sometimes I'd see 1058 samples, sometimes 48, so I'm afraid applying a correction algorithm would be error-prone. Perhaps @MCMcCallum can help with a link, with the features computed on a linux machine?
Thanks!


Add a prop to remove settings table button in the header

What?
There are two issues in this PR

If we set two props disableEdit and disableDelete is true. It will have a small bug UI

I add props to disable the settings table. And remove the last column when set three props disableEdit, disableDelete, disableSettingsTable are true.

@nguyenquocdaile hey good catch, one small thing. Instead of adding another prop I suggest that we hide the context menu button if both disableEdit and disableDelete are set to true trough a computed property.

@nguyenquocdaile hey good catch, one small thing. Instead of adding another prop I suggest that we hide the context menu button if both disableEdit and disableDelete are set to true trough a computed property.

Hi @Haberkamp, Another prop ( disableSettingsTable ) is to disable settings table here :D. it's not related to the context menu button.

If a set three props disableEdit, disableDelete, disableSettingsTable is true. The last column will remove also.


Deploy package into Cluster UI

User Story
User Persona: Ashton (Package Deployer)
When I click Deploy new package button on deployment page
Then the deployment wizard opens
Starting with the screen that shows me all of the local Zarf packages on my system
I see the name of the package with a button next to it to deploy
When I click deploy package
Then the wizard proceeds to step 2 (Step 1 of init wizard)
Technical

[x] Zarf needs to search for Zarf packages on the local system
[ ] When user selects package to deploy > $ Zarf package deploy [package name]

@YrrepNoj @jeff-mccoy Can you add your thoughts about how to do this?

Good news, we already did the backend for that, just need to wire a ui to it 🤓

yay!!

An api endpoint for this exists.


Add License section in readme.md

[x] I checked to make sure that this issue has not already been filed.

[x] I'm reporting the issue to the correct repository (for multi-repository projects) (optional)

@shriya030 pls assign this issue to me.

@ishitakapoor26 Which license will you add?

Please assign this issue to me, I can add an MIT license

I can add an MIT license.
Please assign me this issue under LGMSOC'21

@varshitha1707 Okay go ahead!


getHomeworkDescription() not working

When used returns the error:
 Possible Unhandled Promise Rejection (id: 688):
Error: HomeworkDetails: Invalid API request: params must contain ['homework_id'] properties

I made a spelling mistake in the request.


🔣 The TTY backend should eat up or replace all control characters and LS U+2028 and PS U+2029.

I have two proposals for the replacement:

□
�

One exception is CR. To handle CRLF smoothly, CR should probably just be removed. Another is (horizontal) tabs, which is discussed in #66.

Closed by 9ef07f6071075a2b6097abd35f74c2f0d29b2b3c.


[BUG] Team assignment broken on VSH maps

Describe the bug

While playing on VSH maps (tested on official Valve servers), TF2BD has trouble properly recognizing player teams - instead of showing everyone except current Hale as "Friendly", it shows half of the players as "Friendly" and "Enemy", as if I was playing on a normal server. This prevents features such as auto-kick from working reliably.

To Reproduce

Join a Valve server hosting a VSH map (Nucleus/Distillery/Skirmish/Tiny Rock)
Look at the team assignment in TF2BD.

Logs
2023-08-04_15-40-14.log
Desktop:

OS: Windows 11
Version of bot detector: v1.5.7

I'll look into it.

valve, please
I can make it so that it tries to vote everyone i guess but this is.. lol

Can the tool detect the map name?
In that case, an option to do that on vsh maps would be an idea.
Or simply a toggle next to the chat spam toggles would be also working.

not really tested code, you can test for yourself in the latest build artifacts.
enabling this option will make auto kick cycle through every cheater in the entire server (hopefully)
reopen if issues arise.
I'll make it a release in.... when I feel like it lmfao


strings.replace

Replaces matches for pattern in string with replacement.
Checklist

[ ] implementation
[ ] tests
[ ] jsdoc

Details
console.log(strings.replace('Some say potayo', 'potayo', 'potahto'));
>'potahto'

References

_.replace

Closed as 'invalid' b/c this method is already built-in to JS


Warn user about problematic locale settings

Some tools might silently fail or hang otherwise. See #10988

Just noticed: The bundle exec fastlane test phase of the Circle build is full of this output now:
^D^D[07:32:19]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
[07:32:19]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
[07:32:19]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
[07:32:19]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables

(and all through the test)
Intentional?

Seems like we need to reconfigure circle? 😅

@janpio See #11003, it's fixed now

After updating from Fastlane 2.64.1 to 2.68.0 I get several WARNINGs
Before
Martijns-MacBook-Pro:Frameit Martijn$ fastlane frameit silver
[23:23:14]: Seems like launching fastlane takes a while - please run
[23:23:14]:
[23:23:14]: $ [sudo] gem cleanup
[23:23:14]:
[23:23:14]: to uninstall outdated gems and make fastlane launch faster
[23:23:14]: Alternatively it's recommended to start using a Gemfile to lock your dependencies
[23:23:14]: To get started with a Gemfile, run
[23:23:14]:
[23:23:14]: $ bundle init
[23:23:14]: $ echo 'gem "fastlane"' >> Gemfile
[23:23:14]: $ bundle install
[23:23:14]:
[23:23:14]: After creating the Gemfile and Gemfile.lock, commit those files into version control
[23:23:14]: Get started using a Gemfile for fastlane https://docs.fastlane.tools/getting-started/ios/setup/#use-a-gemfile
[23:23:17]: Using device frames version 'latest'
[23:23:17]: Framing screenshot './screenshots/fr-FR/iPhone6-01_RemoteTab-589996837d813aace56d9a0cf7b764a0.png'
[23:23:25]: Added frame: '/Users/Martijn/iMKapps/debug/Frameit/screenshots/fr-FR/iPhone6-01_RemoteTab-589996837d813aace56d9a0cf7b764a0_framed.png'
[23:23:25]: Framing screenshot './screenshots/background.png'
[23:23:26]: Could not get title for screenshot ./screenshots/background.png. Please provide one in your Framefile.json
After
Martijns-MacBook-Pro:Frameit Martijn$ fastlane frameit silver
[23:26:02]: Seems like launching fastlane takes a while - please run
[23:26:02]:
[23:26:02]: $ [sudo] gem cleanup
[23:26:02]:
[23:26:02]: to uninstall outdated gems and make fastlane launch faster
[23:26:02]: Alternatively it's recommended to start using a Gemfile to lock your dependencies
[23:26:02]: To get started with a Gemfile, run
[23:26:02]:
[23:26:02]: $ bundle init
[23:26:02]: $ echo 'gem "fastlane"' >> Gemfile
[23:26:02]: $ bundle install
[23:26:02]:
[23:26:02]: After creating the Gemfile and Gemfile.lock, commit those files into version control
[23:26:02]: Get started using a Gemfile for fastlane https://docs.fastlane.tools/getting-started/ios/setup/#use-a-gemfile
[23:26:04]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
[23:26:05]: Using device frames version 'latest'
[23:26:05]: Framing screenshot './screenshots/fr-FR/iPhone6-01_RemoteTab-589996837d813aace56d9a0cf7b764a0.png'
[23:26:13]: Added frame: '/Users/Martijn/iMKapps/debug/Frameit/screenshots/fr-FR/iPhone6-01_RemoteTab-589996837d813aace56d9a0cf7b764a0_framed.png'
[23:26:13]: Framing screenshot './screenshots/background.png'
[23:26:14]: Could not get title for screenshot ./screenshots/background.png. Please provide one in your Framefile.json
[23:26:14]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
Martijns-MacBook-Pro:Frameit Martijn$ [23:26:14]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
[23:26:14]: WARNING: fastlane requires your locale to be set to UTF-8. To learn more go to https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables
At the last it hang and I have to press CTRL-C to abort.

@funnel20 Have you tried going to the link in the warning (https://docs.fastlane.tools/getting-started/ios/setup/#set-up-environment-variables) and setting the correct variables? Does it still show this error?

@milch I'm sorry, I didn't. I did now, but I have no clue how to get these 2 EXPORT line in what.
Forgive me, but I really liked the concept of fastlane snapshot, fastlane frameit silver and fastlane deliver. That's all I know of your fantastic tool chain. And of Terminal basically...
I'm on a MacBook Pro with macOS 10.13.1, so if you can guide me (or extend the referred documentation) that would be much appreciated.

Open your terminal, and run the following code (copy exactly as shown here):
echo -e "export LC_ALL=en_US.UTF-8\nexport LANG=en_US.UTF-8" >> ~/.bashrc

This will make sure all the necessary variables are set. Afterwards, restart your terminal, and the warnings should not be shown anymore

@milch Thanks a lot, you're the man!
Problem solved.
I think it can help other users to add this to the docu. Or I might be dumbest...

@milch Hmm, unfortunately the warning came back after several runs of frameit.
Any idea? I even restarted the MacBook.

@funnel20 Do you still have this problem? Can you run echo $LANG from the terminal and see if the output is en_US.UTF-8?
I would add the script I posted to the docs, but it does not work generally for all setups. I made some guesses as to what your setup might be and it seems to have worked, at least at first.

@milch Thanks for getting back to me. It's still there.
The output of that command is nothing:
Martijns-MacBook-Pro:Frameit Martijn$ echo $LANG

Martijns-MacBook-Pro:Frameit Martijn$

That is weird, can you also run ls ~/.bashrc ~/.bash_profile ~/.zshrc ~/.config/fish/config.fish and echo $SHELL and post the output please?

Here you go:
Martijns-MacBook-Pro:Frameit Martijn$ ls ~/.bashrc ~/.bash_profile ~/.zshrc ~/.config/fish/config.fish
ls: /Users/Martijn/.config/fish/config.fish: No such file or directory
/Users/Martijn/.bash_profile	/Users/Martijn/.bashrc		/Users/Martijn/.zshrc
Martijns-MacBook-Pro:Frameit Martijn$ echo $SHELL
/bin/bash
Martijns-MacBook-Pro:Frameit Martijn$

Hmm, that's interesting... Can you try running the following two scripts and then restarting your Terminal, to see if that fixes the locale settings for you?
echo -e "export LC_ALL=en_US.UTF-8\nexport LANG=en_US.UTF-8" >> ~/.bash_profile
echo -e "export LC_ALL=en_US.UTF-8\nexport LANG=en_US.UTF-8" >> ~/.zshrc

Martijns-MacBook-Pro:Frameit Martijn$ echo -e "export LC_ALL=en_US.UTF-8\nexport LANG=en_US.UTF-8" >> ~/.bash_profile
Martijns-MacBook-Pro:Frameit Martijn$ echo -e "export LC_ALL=en_US.UTF-8\nexport LANG=en_US.UTF-8" >> ~/.zshrc
Martijns-MacBook-Pro:Frameit Martijn$ echo $LANG

Martijns-MacBook-Pro:Frameit Martijn$

After restarting Terminal:
Martijns-MacBook-Pro:Frameit Martijn$ echo $LANG
en_US.UTF-8
Martijns-MacBook-Pro:Frameit Martijn$

And no more errors when running Frameit:
Martijns-MacBook-Pro:Frameit Martijn$ bundle exec fastlane frameit silver
[14:59:03]: Using device frames version 'latest'
[14:59:03]: Framing screenshot './screenshots/fr-FR/iPhone6-01_RemoteTab.png'
[14:59:10]: Added frame: '/Users/Martijn/iMKapps/debug/Frameit/screenshots/fr-FR/iPhone6-01_RemoteTab_framed.png'
[14:59:10]: Framing screenshot './screenshots/background.png'
[14:59:11]: Could not get title for screenshot ./screenshots/background.png. Please provide one in your Framefile.json
Martijns-MacBook-Pro:Frameit Martijn$

👍

I hope this will now last longer then several days ago.

Perfect! I don't know why, but it seems like your configuration file switched from ~/.bashrc to ~/.bash_profile within the last couple days. Maybe you installed an update, or some other tool that changed this? Honestly I've never seen a Mac using the ~/.bash_profile file before, only linux, so I don't know what could've caused this

The only change that I can think of is the High Sierra root bug that auto-installed on last Friday:
https://support.apple.com/en-us/HT208315


Plots: Getting new plot points

Residents of a settlement should regularly get new plot points at some rate.
Should the amount depend on the size or number of residents in the settlement?

Settlements will get a prosperity that determines how many plot points residents will gain.
It has yet to be determined when these plot points are acquired.

The concept of prosperity has been added. Next step is to use it for something.


[14.0.x] ISPN-14189 Wrong numbers are shown after clear/rebuild index action

https://issues.redhat.com/browse/ISPN-14189
https://issues.redhat.com/browse/JDG-6235
backport of https://github.com/infinispan/infinispan/pull/10823

Integrated into 14.0.x, thanks @fax4ever !


Duplicate messages for AMQP receiver node when re-deploy flow

Use following simple flow :
AMQP receiver node --> Debug node
Deploy flow. When node receive a message, only one message received is showed in the debug toolbox.
Change something on the flow (i.e. only the node position) and re-deploy the flow.
When node receive a message, two messages are showed in the debug toolbox.

Closed on behalf of 09c68a0678a109ad569b18a23534f0e3208f703b


Add filter examples to metrics

Description & motivation
This metrics documentation is largely a copy-paste from the parent GH spec, and said "see below" for filter examples but there was no below. Now there is.
@drewbanin a couple of questions where the implementation diverged from the spec:

Is operator always required, or is it assumed to be "=" if not specified?
for is_paying is true, is "is" required as an operator, and would it need to be " is "? (The latter in particular would be icky)
1.0.0-rc1 rejected true without quotes because True is not of type 'string'. Likewise for ltv >= 100. Is that overly aggressive type expectations? (cc @jtcohen6 - I can open an issue on the core repo if so)

Pre-release docs
Is this change related to an unreleased version of dbt?

[x] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[ ] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

@joellabes Thanks for playing around with metrics, and for the great feedback!

This metrics documentation is largely a copy-paste from the parent GH spec, and said "see below" for filter examples but there was no below.

You caught me :)
Quick answers from my perspective:

Is operator always required, or is it assumed to be "=" if not specified?

There's no default defined within dbt, and this is a required property, so it will raise an error if not provided. A downstream tool or macro which leverages this metric definition could assume a default operator when not provided, but I could also imagine a filter like this, given that (on many databases) where is_bool_col is valid shorthand for where is_bool_col is true:
      filters:
        - field: is_bool_col
          operator: ""
          value: ""

for is_paying is true, is "is" required as an operator, and would it need to be " is "? (The latter in particular would be icky)

I think the spacing would be the responsibility of the downstream tool / macro. Hopefully they'd be kind enough to template the query like:
where
{% for filter in filters %}
    {{ filter.field }} {{ filter.operator }} {{ filter.value }}
    {{ "and" if not loop.last }}
{% endfor %}

1.0.0-rc1 rejected true without quotes because True is not of type 'string'. Likewise for ltv >= 100. Is that overly aggressive type expectations?

Yes, that's an open TODO right now. I think we encountered validation issues with making this accept absolutely Anything, but I think we could change it to accept any reasonable data type:
    value: Union[str, int, float, bool]

It's definitely workable to just use strings ('true', '100') in the meantime, but I think this would be an uncontroversial improvement. Mind opening an issue for it? (Note to self: this would constitute an artifact schema change, and require us to update manifest v4 at /schemas.getdbt.com.)

There's no default defined within dbt, and this is a required property, so it will raise an error if not provided

Lovely! I'll update these docs accordingly.

It's definitely workable to just use strings ('true', '100') in the meantime, but I think this would be an uncontroversial improvement

Yeah it'd be good to open it up ASAP, because otherwise we'll need to document it as "put everything in quotes, and your strings in "'double quotes'" so that one set of quotes makes it through to Jinja", right?

Mind opening an issue for it?

https://github.com/dbt-labs/dbt-core/issues/4294

@runleonarun could you have a peek at this too please?


Update README.md

https://www.facebook.com/profile.php?id=100040677634079
It's hacked. Can you save me? I do not have an email address. Can you help me?

It's hacked. Can you save me? I do not have an email address. Can you help me?

Holle!!!

pip install Pygments

Hey

https://www.facebook.com/profile.php?id=100040677634079 It's hacked. Can you save me? I do not have an email address. Can you help me?

http://CONTRIBUTING.md

هی

تو میتونی

On Fri, 15 Apr 2022 at 1:10 PM Arastoo @.***> wrote:

هی
تو میتونی
—
Reply to this email directly, view it on GitHub
https://github.com/github/feedback/pull/9297#issuecomment-1099968058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWWGRIDLYOBGZGCNDWN5Y63VFETPXANCNFSM5KXR5HQA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
null


paypal button debit or credit card shows create paypal account, how can hide it?

Hi, I have been worked on this integration but I have a problem, how can render the paypal button for debit or credit card without show the create paypal account seccion. I just want like the example on your storybook.

But instead of that I get a form with a seccion for create a new paypal account and is required, how can disable that seccion or making not required.

I hope you can help me, thanks.

Hi @JELugoM it is not possible to disable that feature in the guest checkout flow in the popup. The inline guest feature is still new and only supported in certain countries. When it's not supported, it falls back to the guest checkout flow in the popup.


Replace references of static externs to addr_of!. We need to do this …

…because of https://github.com/rust-lang/rust/issues/114447 .
I need this, because I plan to send PR to crate libc, which will add environ static extern for UNIX. Unfortunately, environ generates annoying warning during testing due to https://github.com/rust-lang/rust/issues/114447 , so I'm sending this PR to ctest2 first.
I tested this ctest2 PR with my fork of crate libc, which adds environ, and all works great.
Also, even if you opposed to proposed crate libc change, this ctest2 PR is of general usefulness anyway.
Also, this ctest2 PR always unconditionally generates addr_of! invocation. This is okay, because ctest2's MSRV is 1.56.0, and addr_of! was added in 1.51.0.
Please, make ctest2 release after this PR is applied

CI is failing. But I think this is not because of this PR. If you want, I can try to fix CI anyway

@JohnTitor , I fixed CI!!!
I added two commits (second and third), and now CI works. First commit (i. e. introducing addr_of!) is needed for CI, too. So, all 3 commits are needed for CI. Please, review and apply this PR


cli demo code？

yeah i would like to see cli code if you could, this is fantastic

https://github.com/raven38/rf_inversion

https://github.com/DarkMnDragon/rf-inversion-diffuser
Yeah, I also reimplemented a notebook, you can try that

I'm currently working on SD3.  If you aim to use RF-inversion SD3 or SD3.5, you can take my implementation.
https://github.com/pureexe/rf-inversion-sd3
Also, thank for DarkMnDragon give an example of how to implement on Flux


Mention Lombok in README

The README files for all chapters (and/or the general README file for the repository) should mention Lombok and link to instructions for installing it in the reader's IDE.

I've added a general message about Lombok in the root README.adoc file. I may or may not add it to the individual chapter/project README files. I'll leave this issue open until I decide.

For now I'm going to close this issue. I still have plans to refine some of the README files. When I do that, I'll reconsider adding more info regarding Lombok.

I have installed Lombok in your IDE. Still I am getting the error when i am hitting the /design url in chapter 2.


Static IP should be created in nodeResourceGroup

The instructions are to create the static ip in myNetworkResourceGroup , but the static ip is expected in the nodeResourceGroup in subsequent steps

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

ID: 7179d4ae-db6e-7f7c-a855-db0e0475dae9
Version Independent ID: 1a6fa0e8-e175-ae4d-a9d6-d170255a598e
Content: Use a static IP with a load balancer in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/static-ip.md
Service: azure-kubernetes-service
Sub-service: aks-networking
GitHub Login: @asudbring
Microsoft Alias: allensu

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

@bkokster
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

#reassign:schaffererin

Hi, @bkokster. Thank you for your feedback! I reviewed the article and tested the deployment steps to verify, and you're right, the doc is incorrect. I made proper updates to the doc, which you can see in this PR. The changes to the doc will be live soon. I will now close this GitHub issue. Thank you again! #please-close


Docker image - Youtube not working

Youtube feed get added as untitled and show no articles.  Tried several links and get the same result for all.  Tested the one below in another app and it displays as expected.
https://www.youtube.com/feeds/videos.xml?channel_id=UCZNhwA1B5YqiY1nLzmM0ZRg

To make YouTube feeds work on your local installation, you'll need to register for a YouTube API key, which you can override in docker_local_settings.py under YOUTUBE_API_KEY.


sentryEsbuildPlugin is not a function

Environment
Sentry esbuild plugin version 0.3.0 run on esbuild 0.16.12
Steps to Reproduce
Followed the usage guide in https://github.com/getsentry/sentry-javascript-bundler-plugins/tree/main/packages/esbuild-plugin
// esbuild.config.js
const sentryEsbuildPlugin = require("@sentry/esbuild-plugin");

require("esbuild").build({
  plugins: [
    sentryEsbuildPlugin({
      include: ".",
      ignore: ["node_modules", "esbuild.config.js"],
      org: process.env.SENTRY_ORG,
      project: process.env.SENTRY_PROJECT,
      authToken: process.env.SENTRY_AUTH_TOKEN,
    }),
  ],
});

Expected Result
Sentry plugin would run
Actual Result
It broke the build with error TypeError: sentryEsbuildPlugin is not a function

Hi @senordelaflor can you console.log sentryEsbuildPlugin and share what it prints?

@lforst This is what prints for sentryEsbuildPlugin:
{ default: [Getter] }

I got the same error, I worked around by sentryEsbuildPlugin.default({...})

Yeah this is something we gotta fix. Thanks for letting us know!  We'll likely change the default export to a named export before we promote the package to a stable version.


Advertise dotting by contributing to `yorkie-js-sdk' examples

Dotting is meant to be used by many people who are interested in pixel canvas. To gather users, we should show dotting to as many developers as we can.
yorkie-js-sdk is an opensource library that I have contributed last year. To briefly give you some information on it, Yorkie is an opensource library that allows you to create a multiplayer editor (ex. Figma). It basically uses CRDT to manage concurrent operations.
If anyone is interested in contributing to yorkie, please give me a notice beforehand since this contribution will need some guidance.

@lerrybe is working on this in collab-dotting! Hooray🤩!

Closing this issue!


Consolidate error reporting

This code moves the formatting logic of a template error into the exception class itself. The reasons are:

Reduce the code needed to raise an error
Ensure that exception formatting is consistent
Store the exception message as its own attribute. For example, the application might want to output the exception location not in text form (like "foo/bar/template.html, line 57, in include"), but might actually load the template and point to the line with the error.

I like the idea, although I've implemented it a bit differently here: 880f90e2c4a04314cd1ee9397da2e6b1de8654a3. Thanks for the suggestion.

Looks good, thank you!


Feature/unified error handling

Fixes #466
The API service reduces server errors to a single string, and all toasts that display errors just access the single error string that is thrown.

@ikethecoder the toasts have been unified to just be start cased, and I went with making them all closeable.


[ARCTIC-1277] Support writing partial and out-of-order field data into mixed format tables

Why are the changes needed?

fix: #1277
Due to initiating a task writer with an incorrect schema when writing partial field data into a mixed format table, task writers will have a field type cast exception.
For example:
Arctic Table Schema:
id int,
name string,
age,
opt timestamp

Flink Table Schema:
age,
id,
opt timestamp

Flink task only writes three field data into an arctic table and also the order of fields is different from the arctic table's fields.
Brief change log

Modify the correct schema in the FlinkTaskWriterBuilder class
The flink 1.12, 1.14, and 1.15 versions are affected.
Add the mixed-hive and mixed-iceberg table writing and reading tests

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before making a pull request

Documentation

Does this pull request introduces a new feature? (yes / no) no
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

LGTM

This patch contains a partial-update feature which needs a proposal, so close it.


added some german translation strings

Added some missing german translations, there might be still some missing

Hi, please read https://github.com/gogits/gogs/wiki/Contributing-Code#before-you-start.

Oh, sorry for the noise


[Question]: Controller & Service file naming convention

Migrating from Swagger 2.* to OpenAPI, and I am wondering... is there not a way to keep my file names for Controllers & Services? The naming convention was much better in Swagger IMO...
For example, for a path /account/{accountId}/client/{clientId}/contacts, I could priorly have it just route to a AccountsContoller.js file... When working with oas-tools, it generated something hideous like
/generated-server/controllers/accountaccountIdclientclientIdcontactsController.js for the path.
Ideally I'd like to have multiple paths pertaining to "accounts" just live in one controller, rather than individual controllers for each path. Is this possible? Seems like it should be.
If I have to use that convention, I will have to redo all my pathing in addition to copy+pasting in the old controller code.
For illustration's sake, I've attached what I want my folder to look like, vs. what oas-tools generated.

Any suggestions?

Thank you for this @alesancor1! My old spec still had x-swagger-router-controller in it, thus oasTools was not operating as I hoped / expected.


Azure landing zone accelerator missing config page to Integrate your Azure environment with GitHub

Describe the bug
Following the guidance from here:
https://github.com/Azure/Enterprise-Scale/blob/main/docs/reference/wingtip/README.md
Specifically:

What will be deployed?
(Optionally) Integrate your Azure environment with GitHub (Azure DevOps will come later), where you provide the PA Token to create a new repository and automatically discover and merge your deployment into Git.

Not sure if I'm missing something, but I don't see a way to configure the GitHub PA Token.
Steps to reproduce

Start the Azure landing zone accelerator - I'm using the url from the button at the top of the page noted above: https://portal.azure.com/#view/Microsoft_Azure_CreateUIDef/CustomDeploymentBlade/uri/https%3A%2F%2Fraw.githubusercontent.com%2FAzure%2FEnterprise-Scale%2Fmain%2FeslzArm%2FeslzArm.json/uiFormDefinitionUri/https%3A%2F%2Fraw.githubusercontent.com%2FAzure%2FEnterprise-Scale%2Fmain%2FeslzArm%2Feslz-portal.json
Step through the tabs/pages:
2.1 Deployment Settings
2.2 Azure Core Setup
2.3 Platform management, security, and governance
2.4 Network topology and connectivity
2.5 Identity
2.6 Landing zones configuration
2.7 Decommissioned/Sandbox
2.8 Review + create

Expected: Tab/page to configure the GitHub connection
Screenshots

@pkorolo can you update the docs please for this one?
@W0GER we have removed the "Platform DevOps and automation" section last week as per the whats new page: https://aka.ms/alz/whatsnew and https://github.com/Azure/Enterprise-Scale/wiki/Deploying-ALZ-Platform-DevOps#operating-the-azure-platform-using-azops-infrastructure-as-code-with-github-actions
You can still use AzOps if desired, by following the setup steps from the AzOps repos as linked to on the page: https://github.com/Azure/Enterprise-Scale/wiki/Deploying-ALZ-Platform-DevOps#operating-the-azure-platform-using-azops-infrastructure-as-code-with-github-actions
Thanks

@W0GER thank you for bringing this to our attention.
@jtracey93 will deal with this asap.

@W0GER I have submitted PR to iron-out the documentation deployment guidance redundancies, to so avoid future confusion.
As @jtracey93 pointed out, AzOps GH integration has been removed from ALZ portal experience.

PR tbr


Takes long time to open the application

The application is very slow to load for the first time.  http://showcase.ripple.foundation/
Can you please check this?
Thanks.

thanks for flagging @linto17
@kbeloborodko Kirill, can you please check out and benchmark please?
thanks
T

@linto17 hi! Thank you for the inquiry! Could you please provide more information about the problem you have experienced? How much time did it take to load the page? (it would be wonderful if you had an opportunity to make a video using some tool like jing, for example, so we can see what the issue is)

Hi, It took 2.38 minute to load. I have used incognito(chrome) to open the application. Application URL http://showcase.ripple.foundation/ there is no console logs in chrome developer console while loading.

@kbeloborodko Do you need more info to reply to this ?

@linto17 @tony-shannon
I am sorry for the late reply.
We have tested it several times using incognito(chrome), hard reload and the result is usually about 17 seconds to load the site for the first time.
http://joxi.ru/12M7nQ7sMYZNJr
http://joxi.ru/p276jL6FoRG47m
Could you check it one more time, please?
In case you still have the same issue could you record console logs?
Also, it can be useful to check traceroute command to showcase.ripple.foundation and internet connection as well, just to exclude these reasons.

Thanks.


suspend cstoku and MasayaAoyama in sig-docs-ja

They are busy for their main business, do not have time to review PRs.
They will come back in the future, when time permits.

/assign @xiangpengzhao

I leave a comment to say this is a valid request as another SIG Docs JA owner :)

/retest

Would it be possible to remove the lines entirely for cleanliness? If / when adding them back, this PR can be referenced or an approver can review the git history.

@jimangel Sure thing.

@jimangel But what about this though? https://github.com/kubernetes/website/blob/master/OWNERS_ALIASES#L80
https://github.com/kubernetes/website/pull/17515

We discussed in sig-docs-maintainer channel on Slack and decided to go it as-is for now.

/lgtm
/approve


feat: better configure ReactMarkdown

Enable remark-gfm so that all GitHub-Flag Markdown is supported,
including tables (Fixes cdklabs/construct-hub#197).
Properly sanitize all generated markdown using rehype-sanitize,
ensuring we do not render anything that GitHub would not.
Transform image and link URLs that are repository-relative, so that such
links are valid. These resolve to GitHub distribution points in the same
way that npmjs.com does, so this change is neutral for packages that are
not configured with a GitHub repository (Fixes cdklabs/construct-hub#198,
fixes cdklabs/construct-hub#199).

YASSS! So much goodness!


Sort Array to fix #2188 from radare

I close the PR yesterday since I didn't test properly and also I've added a sort method to num in case if necessary in the future.

I think @karel- 's implementation should perform better. I'm waiting for an updated pullreq, but i think its better to serialize the string into a native array with sdb_fmt apis, and then sort it by just swapping pointers

The swapping pointers sounds better. I spoke with @karel- yesterday about the issue 2188 and I hope he continue working on the implementation


test-branch/execute-job/quoted-authorization/925bc54e-37f5-4df8-b3fc-cf36a3229a58

PR for testing 98f5163d159572da7badcd7a54a97768bc03a379

/canonical/self-hosted-runners/run-workflows 98f5163d159572da7badcd7a54a97768bc03a379


Cursor appears busy while clustering

Cursor changes after the merge selected and re-cluster button is clicked and remains in the busy state till clustering is done.
resolves #2033

@darecoder1999 testing on Firefox.  I see the same results of regular cursor or cross cursor, but not a busy cursor after sliding the # Rows in Cluster.  Here there was a computing clusters in console for almost 1 minute after I clicked to WAIT on the browser yellow banner warning.

Oh! okay @thadguidry. I'll check it again and make the required changes.

@thadguidry sir, can you please share your dataset?

@darecoder1999 It is on the original issue.


Launch config results in "SyntaxError: Cannot use import statement outside a module"

The steps described in "Update your hosted app" instruct users to create a launch.json file to set appropriate environment variables in order to enable the the bot and the messaging extension.
I created it following the steps and when I attempt to run it, I get the following error:
Debugger listening on ws://<IP_ADDRESS>:56459/357500ce-0d72-4f89-8ce8-88ca37e31451
For help, see: https://nodejs.org/en/docs/inspector
Debugger attached.
Waiting for the debugger to disconnect...
<repopath>\msteams-samples-hello-world-nodejs\src\app.js:4
import path from 'path';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at wrapSafe (internal/modules/cjs/loader.js:1047:16)
    at Module._compile (internal/modules/cjs/loader.js:1097:27)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1153:10)
    at Module.load (internal/modules/cjs/loader.js:977:32)
    at Function.Module._load (internal/modules/cjs/loader.js:877:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:74:12)
    at internal/main/run_main_module.js:18:47
Process exited with code 1

Version info:
git: 2.24.1.windows.2
node: v12.16.3
npm: v6.14.4
gulp local: v4.0.2
gulp CLI: v2.3.0

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

ID: c4fa27cb-544f-f8a2-97a7-4b31528c1b80
Version Independent ID: fa69cd6f-1ef4-706d-2816-aa709714b0aa
Content: Get started with App Studio and Node.js - Teams
Content Source: msteams-platform/tutorials/get-started-nodejs-app-studio.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

@erinha , Could you please check this working Javascript_nodejs sample code?


T1008 change cache test key

Description

Additional context

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines
[ ] Read the Contributing Guidelines and follow the Commit Convention
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

PR 제목 Commit Message Convention에 맞게 수정해주세요!

Closes도 붙여주세요~


Neo4j OGM Bolt driver exception in native mode (GraalVM Native image)

Expected Behavior
I am building an api connecting to a neo4j database using OGM bolt driver. I need to be able to build and run the application in native mode exactly like we run in regular JVM.
Current Behavior
The application is perfectly functional in regular JVM mode. When I build it using GraalVM native mode the driver class is not found/ recongnized. Also, please note that I have the required dependencies in my pom.xml. (Provided stacktrace excerpt below).
Possible Solution
Add support for OGM in native mode?
Stacktrace
Caused by: org.neo4j.ogm.exception.core.ConfigurationException: Could not load driver class org.neo4j.ogm.drivers.bolt.driver.BoltDriver
at org.neo4j.ogm.session.SessionFactory.newConfiguredDriverInstance(SessionFactory.java:271)
at org.neo4j.ogm.session.SessionFactory.(SessionFactory.java:89)
at com.config.SessionFactoryProducer.produceSessionFactory(SessionFactoryProducer.java:33)
at com.config.SessionFactoryProducer_ProducerMethod_produceSessionFactory_90de702675c65462441375a80e04e7de183112f0_Bean.create(SessionFactoryProducer_ProducerMethod_produceSessionFactory_90de702675c65462441375a80e04e7de183112f0_Bean.zig:192)
at com.config.SessionFactoryProducer_ProducerMethod_produceSessionFactory_90de702675c65462441375a80e04e7de183112f0_Bean.get(SessionFactoryProducer_ProducerMethod_produceSessionFactory_90de702675c65462441375a80e04e7de183112f0_Bean.zig:222)
at com.config.SessionFactoryProducer_ProducerMethod_produceSessionFactory_90de702675c65462441375a80e04e7de183112f0_Bean.get(SessionFactoryProducer_ProducerMethod_produceSessionFactory_90de702675c65462441375a80e04e7de183112f0_Bean.zig:245)
at com.service.AccountServiceImpl_Bean.create(AccountServiceImpl_Bean.zig:179)
... 55 more
Caused by: java.lang.ClassNotFoundException: org.neo4j.ogm.drivers.bolt.driver.BoltDriver
at com.oracle.svm.core.hub.ClassForNameSupport.forName(ClassForNameSupport.java:60)
at java.lang.Class.forName(DynamicHub.java:1194)
at org.neo4j.ogm.session.SessionFactory.newConfiguredDriverInstance(SessionFactory.java:266)
... 61 more
Context
We are trying to use the ogm bolt driver and connect to neo4j backend and then trying to build a native image using Quarkus+GraalVM
Your Environment

OGM Version used: 3.2.14
Java Version used: GraalVM CE 20.10 (Java 1.8.0_251)
Neo4J Version used: Neo4j-enterprise-3.5.8
Quarkus Version: 1.6.1.Final
Bolt Driver Version used (if applicable): 3.2.14
Operating System and Version: Mac OS Mojave 10.14.6

Thanks for your time reaching out @sachinsudhi. As I stated in April here https://github.com/quarkusio/quarkus/issues/8530#issuecomment-612129980 Neo4j-OGM is not happening from our side in native GraalVM mode anytime soon.
The Java-Driver for which we provide the quarkus extension is tested on our side and ensured to work on GraalVM native.
We might gonna explore options with Spring Data Neo4j 6, that is to be released in autumn. The Spring team is working on native compilation and from our side, only the CypherDSL needs to be changed not to use method handles.
Thanks again.


Relate to IIIF 57 CMYK

Create tickets for audit tool.
Have this ticket own the audit process for existing web material.
Write a separate ticket for volume-manifest-builder

@jimk-bdrc See the discussion between Elie and I here.  For now I think we'll only want to worry about audit tool.

@eroux @TBRC-Travis it would be simple enough to mark it in dimensions.json with the file mode [PIL documentation](https://pillow.readthedocs.io/en/3.0.x/handbook/concepts.html?highlight=color space)

@eroux should the CMYK color space be a warning, or an error in audit tool?

hmm good question for the warn vs error... I don't have a strong opinion on that, I think both are good.
Yes, I think indicating the mode when it's CMYK would be good. I think it's best not to indicate the mode on each image when it's the expected one, meaning:

for .jpg when it's L or  RGB
for .tif when it's 1

I sort of remember that at some point the code of the tool was writing this PIL mode but I'm not 100% sure...

Yes, this change shows you relocated pilmode, but it was still very conditional.

Discuss with @TBRC-TimB in 2021

buda-base/buda-iiif-server#57 is closed, so there's no need for this.


Will this library be updated to be WebGPU based soon?

I assume that moving to WebGPU would dramatically speed up processing times currently limited by WASM but would love to hear the communities thoughts.

Very interested in this. Would love to support with contributions as well.

Same here, would love to help out where possible if the maintainers are open to adding webgpu support


x/net/http2: no public method to send PING frames

There is, as far as I can tell, no method for sending a PING frame to an open HTTP/2 connection right now. While in most cases there is no direct need to send PING frames, I've encountered multiple cases where it's important, lest you discard the connection and recreate it:

Apple's APNs HTTP/2 API is picky about what it receives and only supports PING frames as a way to ensure a healthy connection. Sending dummy requests may get your connection closed.
Go's HTTP/2 lib chokes on idle sockets on Google Compute Engine VMs (see #14607 for some info). If the socket happens to be an HTTP/2 connection (which it often is in my case), being able to send PING frames would be a viable workaround for this issue.
Amazon's AVS HTTP/2 API expects a PING frame at least once per 10 minutes and it should be possible to send them. (Luckily, there's a workaround to call /ping in this case.)

So my suggestion is to somehow expose a way to send PING frames.

I suggest that x/net/http2 should periodically send PING frames to the peer, and kick muted connections from the connection pool automatically. This feature should be off by default, and could be turned on by setting some kind of options.

Like @blixt, I'm implementing APNS client and lack of access to PING frame is a bit of a show stopper for pushing to prod environment. It makes sense to have periodical ping as proposed by @zjx20, but it would be great to have also some kind of read-only access to connection stats for both PING and GOAWAY frames (to have connection behaviour in metrics).
@bradfitz: As I'm on a bit tight schedule and not sure can wait for the official fix, could You recommend some temporary solution which will allow access to ping on client side? I'm still quite green in GO and can't figure out how to extend the connection without actually forking x/net/http2 and making exported getter for Framer in http2ClientConn.

PING frames are also useful for measuring the true end-to-end RTT, which you can't reliably measure at the transport level due to split TCP in cellular networks. I don't have a need for this feature, but if it's implemented, I would vote for a blocking API so the caller can time the RTT. Something like:
func (s *h2connection) Ping(opaque uint64) (uint64, error)

If you need to PING periodically, you could always do so from a timer loop.

@tombergan, what is the return uint16 in your example? If it's the data, it must match the ping opaque value in the sent frame, so it's redundant, no?

That's right, I missed that the ACK must echo the opaque value from the original PING. The uint64 return value is not necessary.

Any update on this?
I think this ping method should get a context as first argument to let the caller manage deadline and cancellation. Something like:
func (cc *ClientConn) Ping(ctx context.Context) error

Here is a proposal implementation: https://go-review.googlesource.com/#/c/29965/


finetuning failing

i m getting following error when follwoing the examples finetune steps.
RuntimeError: Expected to mark a variable ready only once. This error is caused by one of the following reasons: 1) Use of a module parameter outside the forward function. Please make sure model parameters are not shared across multiple concurrent forward-backward passes. or try to use _set_static_graph() as a workaround if this module graph does not change during training loop.2) Reused parameters in multiple reentrant backward passes. For example, if you use multiple checkpoint functions to wrap the same part of your model, it would result in the same set of parameters been used by different reentrant backward passes multiple times, and hence marking a variable ready multiple times. DDP does not support such use cases in default. You can try to use _set_static_graph() as a workaround if your module graph does not change over iterations.
Parameter at index 195 with name model.encoder.layer.11.output.LayerNorm.weight has been marked as ready twice. This means that multiple autograd engine  hooks have fired for this particular parameter during this iteration.

You should show your command and the environment so that I can analyze the possible cause.

In my case, In my case, it happened when I didn't apply the "deepspeed" setting.
If I didn't apply the --deepspeed setting, I shouldn't have applied the --gradient_checkpointing option either.


Daily reset

Now the daily scheduled tasks will reset, if the current date doesn't match the date when an item was checked.
I needed to downgrade nanoid to version 3.3.4, due to dropped support of CommonJS by the nanoid project.
Using version 4 leads to Errors in my tests
SyntaxError: Cannot use import statement outside a module
See Change Log nanoid

looks nice. keep up good work


update styleguide

Coverage remained the same at 100.0% when pulling e827faacb84a8d8937bf911df313e766214251e1 on update-styleguide into 53869b3a3473aaf8dcfb12cbe72c976ef98e56a5 on master.

Coverage remained the same at 100.0% when pulling f89b15190cfa505f3a7c49ed6335b98ef42ddb3d on update-styleguide into 53869b3a3473aaf8dcfb12cbe72c976ef98e56a5 on master.

Coverage remained the same at 100.0% when pulling fb898133edbe89bd050e4e2b561c807c12d9573c on update-styleguide into 53869b3a3473aaf8dcfb12cbe72c976ef98e56a5 on master.

Coverage remained the same at 100.0% when pulling fb898133edbe89bd050e4e2b561c807c12d9573c on update-styleguide into 53869b3a3473aaf8dcfb12cbe72c976ef98e56a5 on master.

Coverage remained the same at 100.0% when pulling bb9e421517c916c4142b31de12fcac0cc62e37f5 on update-styleguide into 53869b3a3473aaf8dcfb12cbe72c976ef98e56a5 on master.


Update sites.yml add beuke.org

Website (if applicable): https://beuke.org
This PR is:

[x] Adding a new domain
[ ] Updating existing domain
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (darktheme.club site)
[ ] Other not listed

[x] The domain is in the correct alphabetical order
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ page, particularly the red section about inappropriate content, and I attest that this site is appropriate based on those criteria.

[x] Check to confirm

- domain: beuke.org
  url: http://beuke.org/
  method: darkonly
  last_checked: 2023-10-17

Thanks!


Create a convenience script to run lint quickly locally

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Run npm run lint-changed
Expected Result:
This script does not exist in our package.json
Actual Result:
Instead we should run ESLint on the JS files that changed between the current branch and the main branch:
eslint --fix "$(git diff --name-only main -- "**/*.js")"

Workaround:
Just run lint on all files, which can take about 5 minutes.
Version Number: 1.3.9-12
Reproducible in staging?: n/a
Reproducible in production?: n/a
Issue reported by: no reporting bonus
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1683126144968009
View all open jobs on GitHub

Triggered auto assignment to @conorpendergrast (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero process steps for feature requests for more details.

Well Rory you just let me know how I can help here

Nothing really needed here, I just put up a PR for this minor QOL improvement. No payments will be due.

This is done


Add wave2 Wi-Fi settings

Wi-Fi settings are configured differently with MikroTik's wave2 product line. For 802.11 AC Wave2 devices there is an optional extra package to configure Wave2. For 802.11 AX devices Wave2 will be default.
We have a device on the demo server with Wave2:
MikroTik hAP AC^3
RouterOS v7.5 with Wi-Fi Wave2 extra package

@mthoodlum Which host?
When I search for wave there are no results.

lab-ROSv7-hAP-AC3

Here is the device link, it updates but does not send chart data https://demo.ispapp.co/#host/635462f4bbb1d166b6f1f01e

Response from Mikrotik.
Hello.
Thank you for the script example. I see the problem now.
It arises because of how '[find]' is interpreted differently, depending on whether the menu for the preceding command exists or not.
If the wifiwave2 package is not installed , the [find] block in /interface wireless security profiles get [find name=test] value-name=wpa2-pre-shared-key invokes /interface/wireless/security-profiles/find
When the wifiwave2 package is installed and there is no /interface/wireless/security-profiles/find to invoke, the [find] block invokes /find
'/find' does not take 'name' for an argument. Invoking it leads to a syntax error and syntax errors are not handled by 'do {} on-error={}'.
So, as a solution, I can suggest rewriting the supplied script so to avoid ambiguity
:do {
   :local wIfKey ([/interface/wireless/security-profiles/get [/interface/wireless/security-profiles/find name=test] value-name=wpa2-pre-shared-key]);
} on-error={
}

Or just using the profile name with the 'get' command
:do {
   :local wIfKey [/interface/wireless/security-profiles/get number=test value-name=wpa2-pre-shared-key];
} on-error={
}

I'll consult with my colleagues what could be improved in the scripting behaviour. Perhaps it would be more consistent if calling an existing command with a non-existent parameter resulted in the same type of error as calling a non-existent command.
Best regards,
Toms Filatovs

Fixed in 2.20


Deterministic Encryption and Random Encryption (CSFLE MongoDB)

It will be possible to support Deterministic Encryption and Random Encryption?
https://docs.mongodb.com/manual/core/security-client-side-encryption/#deterministic-encryption
String DETERMINISTIC_ENCRYPTION_TYPE = "AEAD_AES_256_CBC_HMAC_SHA_512-Deterministic";
String RANDOM_ENCRYPTION_TYPE = "AEAD_AES_256_CBC_HMAC_SHA_512-Random";

Advantages:

With Deterministic fields, we can apply search queries and also use them as indexed fields.
And we can also continue to use Random encryption for the fields that we want.

That would undermine crypto strength by allowing rainbow table attacks. Of course that could be an accepted leverage between crypto and ease of index, but in the README I've outlined another solution for this:
https://github.com/bolcom/spring-data-mongodb-encrypt#encrypting-an-indexed-field
The upside of having a hash is that you could merge multiple fields into that hash, making it much harder to brute force the original values. Furthermore, if you carefully design your hash key space to be small enough, there will be multiple hits for each hash key, making if effectively impossible to determine the exact original values.
That aside, if you wish to make an MR that makes this pluggable (like the CachedEncryptionEventListener vs. ReflectionEncryptionEventListener), I'm happy to merge!

That would undermine crypto strength by allowing rainbow table attacks. Of course that could be an accepted leverage between crypto and ease of index, but in the README I've outlined another solution for this:
https://github.com/bolcom/spring-data-mongodb-encrypt#encrypting-an-indexed-field
The upside of having a hash is that you could merge multiple fields into that hash, making it much harder to brute force the original values. Furthermore, if you carefully design your hash key space to be small enough, there will be multiple hits for each hash key, making if effectively impossible to determine the exact original values.
That aside, if you wish to make an MR that makes this pluggable (like the CachedEncryptionEventListener vs. ReflectionEncryptionEventListener), I'm happy to merge!

Thanks for the reply. I understand your point and it is a balance between a crypto strength and the possibility to have deterministic encryption and random together.
In fact, to use the native mongo CSFLE features it needs the libmongocrypt and mongocryptd (Enterprise Edition Only) and on this project, you use your own manual encryption mechanism so to add this would be a totally different approach.
I will take a look at this and if I came with a solution using this project I will propose a change. Until there I think you can close this. Thanks

Thanks for the reply. I understand your point and it is a balance between a crypto strength and the possibility to have deterministic encryption and random together.
In fact, to use the native mongo CSFLE features it needs the libmongocrypt and mongocryptd (Enterprise Edition Only) and on this project, you use your own manual encryption mechanism so to add this would be a totally different approach.
I will take a look at this and if I came with a solution using this project I will propose a change. Until there I think you can close this. Thanks

The enterprise edition of mongo does a very similar thing indeed, but not compatible with this lib, nor is that the goal. The upside of this lib is that it's free, easily & seamlessly integrates with spring-data, and encryption happens strictly on client-side only.

The enterprise edition of mongo does a very similar thing indeed, but not compatible with this lib, nor is that the goal. The upside of this lib is that it's free, easily & seamlessly integrates with spring-data, and encryption happens strictly on client-side only.


Auto height for dodgeY plots

Two possibilities:

let dodgeY set a flag that says “there is a y dimension” (although there is no y scale).
measure the actual range of data before setting the height.

The first approach would not adapt the height to the actual height of the stack, but the second approach might be too complex to implement, since autoHeight is currently called long before the initializers.

oops, duplicate of #905


Get actual size of data

Hello, where can I download the actual size of data, for the same comparison and results presented in the paper? In this repository, I see only a sample of data being used.
Thank you,
Vasiliki

Due to the copyright requirement, I can't share the data in public.


Nev version fails to create new user properly.

Most recent version fails to create new users properly. This can be reproduced.
Benutzerverwaltung
Simple user configuration for antiX linux
Version: 2024.06.15

antix-user:
  Installiert:           2024.06.15
  Installationskandidat: 2024.06.15
  Versionstabelle:
 *** 2024.06.15 500
        500 http://ftp.halifax.rwth-aachen.de/mxlinux/packages/antix/bookworm bookworm/main amd64 Packages
        100 /var/lib/dpkg/status

After creating the new user it is not possible to login with this very user from login screen.
Precisely this works fine as long not upgrading from the former version:
Benutzerverwaltung
Einfache Benutzerkonfigurierung für antiX Linux
Version:2023.07.08

$ apt-cache policy antix-user
antix-user:
  Installiert:           2023.07.08
  Installationskandidat: 2024.06.15
  Versionstabelle:
     2024.06.15 500
        500 http://ftp.halifax.rwth-aachen.de/mxlinux/packages/antix/bookworm bookworm/main amd64 Packages
 *** 2023.07.08 100
        100 /var/lib/dpkg/status

Immediately after installing the new package version on a system the login fails for new users, while it still works for all users created with the former version of antix-user on the very system. No other packages have been upgraded the same time to make sure no other possible origin of the issue is drawn into here.
Additional remark: There are several translations missing in the new version. Untranslated strings show up e.g. in popup windows.
Tested on antiX 23.1 and 23.2, both full runit, and 64 and 32 bit both, Live.

Additional pieces of information and steps to reproduce:
Create a testuser1 with the executable from version 2024.06.15 on an antiX 23.2 testing system or antiX 23.1 fully apt-upgraded system, Live or frugal install. (Not sure whether the issue is also present in default installed systems.)
$ md5sum  '/usr/bin/antix-user'
a5b4ca22f824d2460bd7531a15697abf  /usr/bin/antix-user
replace the executable file in /usr/bin/ (and nothing besides this) by the file version found in package version 2023.07.08
$ md5sum  '/usr/bin/antix-user'
11e25afd534f370b486c3e8af03ef8cc  /usr/bin/antix-user
Create a testuser2 with this old executable on the very same system.
Logoff and try logon in gui.
You can not login with testuser1
You can perfectly login with testuser2
After replacing the executables again against each other, so the new version is in use again, new users created by this tool can't login any longer. All users created by the former version executable have no issues.


[WIP] bazel: Shift dependency validate tool to bazel

Commit Message:
this is a reopening of #18085 partly for the purposes of testing
im not sure whether it would be best to shift this tool to bazel first and then to envoy.dependency.check or just to the check tool directly
either way i need to figure out separating the data
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

This one is difficult
The recursive bazel queries that it does //source/... etc pull in huge amounts of transitive dependencies and cant be shifted to a genquery
I think we want to separate out the data from the processing further (ultimately i want to shift this to envoy.dependency.check so we kinda want to boil the data down to json somehow) - but i think this is a good first step at least just getting the tool into bazel
I wanted to be confident that it was checking the same thing so i added some len() s around query results in the logging, and compared this to current #19568
im pretty confident it does the right thing

re optimization - i think its probably not faster (maybe slightly slower) but it uses a lot less memory by using exclude clauses in the bazel queries, rather than getting both inclusive and exclusive sets, and then filtering in python


Vulnerability roundup 85: glib-networking-2.62.3: 1 advisory [6.5]

search, files

[ ] CVE-2020-13645 CVSSv3=6.5 (nixos-20.03)

Scanned versions: nixos-20.03: a84b797b28e. May contain false positives.

20.03 is Frozen and EOL.
As such, issue is closed.