New procedure registration process

Previously a procedure was added to the kernel by passing the bytecode to the kernel and letting it validate and instantiate it. This has a few problems, the most pressing of which is that the constructor of any contract must be run before we can be sure what we are validating, therefore it must always be deployed first. Rather than relying on the kernel to do this, initial deployment of the contract is simply done by the user/developer and the contract address (along with a name and capabilities) is passed to the kernel. The kernel will then validate the procedure and add it to its procedure table (or reject it accordingly).
This is still done via a direct registerProcedure ABI call to the kernel.

Looks good so far. Please update the test descriptions in /withentryproc/*.js so that we could differentiate which tests are which in output.

Looks like i'm getting massive delays in tests due to the whitelisting:

What;s the difference from the previous times?

Here's the speedup:

I replaced the verification with this:
if (ins == 0x54) {return 1;} // SLOAD
if (ins == 0x55) {return 2;} // SSTORE

if (ins == 0xa0) {return 3;} // LOG0
if (ins == 0xa1) {return 4;} // LOG1
if (ins == 0xa2) {return 5;} // LOG2
if (ins == 0xa3) {return 6;} // LOG3
if (ins == 0xa4) {return 7;} // LOG4

if (ins == 0xf0) {return 8;} // CREATE
if (ins == 0xf1) {return 9;} // CALL
if (ins == 0xf2) {return 10;} // CALLCODE
if (ins == 0xf4) {return 11;} // DELEGATECALL
if (ins == 0xf5) {return 12;} // CREATE2
if (ins == 0xff) {return 13;} // SELFDESTRUCT

if (ins >= 0x60 && ins <= 0x7f) {
    i += ins - 95;
}

Here's the speedup:

I replaced the verification with this:
if (ins == 0x54) {return 1;} // SLOAD
if (ins == 0x55) {return 2;} // SSTORE

if (ins == 0xa0) {return 3;} // LOG0
if (ins == 0xa1) {return 4;} // LOG1
if (ins == 0xa2) {return 5;} // LOG2
if (ins == 0xa3) {return 6;} // LOG3
if (ins == 0xa4) {return 7;} // LOG4

if (ins == 0xf0) {return 8;} // CREATE
if (ins == 0xf1) {return 9;} // CALL
if (ins == 0xf2) {return 10;} // CALLCODE
if (ins == 0xf4) {return 11;} // DELEGATECALL
if (ins == 0xf5) {return 12;} // CREATE2
if (ins == 0xff) {return 13;} // SELFDESTRUCT

if (ins >= 0x60 && ins <= 0x7f) {
    i += ins - 95;
}

So it's about 3-5x slowdown

Let's wait until I fixed the previous bug then benchmark it again.

@JakeOShannessy: Here's a blacklist that checks for unknown opcodes:
if (ins == 0x54) {return 1;} // SLOAD
if (ins == 0x55) {return 2;} // SSTORE

if (ins == 0xa0) {return 3;} // LOG0
if (ins == 0xa1) {return 4;} // LOG1
if (ins == 0xa2) {return 5;} // LOG2
if (ins == 0xa3) {return 6;} // LOG3
if (ins == 0xa4) {return 7;} // LOG4

if (ins == 0xf0) {return 8;} // CREATE
if (ins == 0xf1) {return 9;} // CALL
if (ins == 0xf2) {return 10;} // CALLCODE
if (ins == 0xf4) {return 11;} // DELEGATECALL
if (ins == 0xf5) {return 12;} // CREATE2
if (ins == 0xff) {return 13;} // SELFDESTRUCT

if(
    (ins > 0x0b && ins < 0x10) ||
    (ins > 0x1d && ins < 0x20) ||
    (ins > 0x20 && ins < 0x30) ||
    (ins > 0x3f && ins < 0x40) ||
    (ins > 0x40 && ins < 0x50) ||
    (ins > 0x5b && ins < 0x60) ||
    (ins > 0xa4 && ins < 0xf0)
) {
    return 100;
} // UNKOWN OPCODE

if (ins >= 0x60 && ins <= 0x7f) {
    i += ins - 95;
}

Yeah that's what I meant by using ranges. But let's get everything working first before we worry about efficiency.

Not against that, but it slows tests by a large margin. I don't see why it's an issue to simply use the blacklist at this time.

Propose to delay whitelist with #71

Because the blacklist doesn't work either.

Ok, the tests are now passing properly, and validation is being properly tested, with the exception of SLOAD calls, which are still used in a couple of places for testing (we don't currently have a read capability).

Is this ready for review?

yes

Please change test names to differentiate tests with entry proc and without. Otherwise good to merge.


Allow reporting annotations without creating a new check

Right now this action adds a new check to report all annotations. Ideally I'd want to disable this behavior and just report annotations "normally" like other actions do.
For example, one of the linters I use is markdownlint and I enabled a problem matcher for that to create annotations. Another one is yutailang0119/action-android-lint that uses the GitHub actions toolkit to create the annotations from a Android XML file.

Currently the Github API only allows pushing annotations by creating a new check run. I have asked Github to provide alternative APIs but as of now this cannot be done. More or less related to #2.

I took a better look at the android linter I mentioned and it simply said it prints the annotations to stdout via the command helper from the toolkit: https://github.com/actions/toolkit/blob/main/packages/core/src/command.ts
I'll see if I can make a PR for it later


support for relationship links when no data exists

It appears that I may have been mis-understanding/underestimating the value of links provided by the JSON API-spec. I'm using Ember on the front-end can use the related link of a relationship to asynchronously load relationship data when needed. For example, if I have a customer model with a pets relationship and that pets relationship contains ONLY a links attribute, Ember Data will use the related attribute of links to automatically load that data. This allows for much more efficient loading of data, on-demand.
However, with the mapper, we currently require relationship data in order to build said relationships.
My proposal is a method for passing a list of relationships to the mapper. These relationships can than be cross-referenced against the relationships passed along with the Bookshelf model. If there is no relationship in the model, the mapper would simply build the appropriate link. Otherwise, if there is data, the mapper would build out the relationship as normal.
Thoughts on this?

As a follow-up, it seems that this is something the jsonapi-serializer can handle - the ignoreRelationshipData option needs to be passed as an option so that the data element is not included.

@chamini2 any thoughts on this one? Something I need to get implemented at some point.

I've been thinking about it and I'm not sure how we would go about implementing it, since we build the links with information from the relations. The proposal of passing a list of relationships would work like an option to say build the links for these relations if you don't find it?

The proposal of passing a list of relationships would work like an option to say build the links for these relations if you don't find it?

Exactly. This becomes a true advantage when lazy-loading data on the frontend. Ember Data will do this by default, which prevents having to include everything or hit the database just to return a list of relationship ids.

I had to implement a workaround for this. I'm running the equivalent of this code:
options.relations.fields.forEach(function (relation) {
  if (model.relations && !model.relations[relation]) {
    // Simulating an empty collection in json-mapper
    model.relations[relation] = {toJSON: function toJSON() { return []}};
  }
}
mapper.map(model, type, options)

Hope this makes sense - I'm processing the relations.fields array passed as an option to the mapper, and for each relation if it's not already in the model.relations object, I create a fake empty collection. The toJSON function is all that's needed for the map function to work.
However, this results in an empty data: [] which I'd like to also get rid of. As far as I can tell, getting rid of this definitely requires changes to jsonapi-mapper.


【Entitas1.14.1】jenny auto import，not contexts

I followed the steps on the document and after importing, the automatic import did not appear contexts。

Normal should be like this？

Normal should be like this？

Got the same with unity 2022.2 :(

For some reason I got it working by importing twice. Just run dotnet Jenny.Generator.Cli.dll again.

For some reason I got it working by importing twice. Just run dotnet Jenny.Generator.Cli.dll again.

How to run this, plese help me. I am new to UnityDev

Normal should be like this？
Hope this reply is not too late. You should download 2 file named Jenny and Entitas from github: https://github.com/sschmid/Entitas/releases/tag/1.14.1
Create your project, and create an arbitrary C# script to generate an Assembly-CSharp file. Comback to Unity, open Tool/Jenny/Preferences, click auto import and override. Click Edit Jenny.Properties, it will open a path file and copy things from this link (Match-One game): https://github.com/sschmid/Match-One/blob/main/Jenny.properties. Save this file and now you can create components, generate code using the command "dotnet Jenny.Generator.Cli.dll gen".
Hope you find it well.

Thanks for your help！


Make the CLI interactive

Currently, sherlodoc's cli has two mode, single request and "toplevel".
The single request mode works as follow :
$ sherlodoc search "list map"

and return the results of the query "list map".
This is not very practical for a real user, because you need to type "sherlodoc search" over and over to make multiple request (its very good for cram tests though).
We have a "toplevel" mode that is slightly better. With the following command :
$ sherlodoc search

sherlodoc will read queries on stdin each time you press enter, and print the results.
This is not very ergonomic, and we would like something more similar to the website were the results are given on the go when you type. This requires to use some TUI framework like ncurses, because when the user adds a letter you want to remove already print result to print the new ones.

(In the mean time, I personally use the command rlwrap sherlodoc search which relies on the well-known(?) rlwrap utility to add line-edition and history to any program... so we could also document this trick in the readme :-°)

I think this could be done with minttea


feat: fix monorepo issues with sherif

This PR adds Sherif, a monorepo linter, and fixes the current issues that were reported by running bunx sherif@latest:

Move the root dependencies to devDependencies
Order alphabetically all dependencies and devDependencies
Use the same version across the monorepo for typescript (^5.5.4), react (^18.3.1), and react-dom (^18.3.1)
Add two new lint:repo and lint:repo:fix scripts to run Sherif and fix issues with Sherif

Each step is in a separate commit, so feel to just apply the fixes without adding the two new scripts by dropping 4e6928ba7066363a2844c20adb1be8628f47435b if you don't want to add Sherif.

Love it, thank you so much!


Example App bugfix

The example app doesn't work. I have fixed this.
And I have also added the example app to run on GH actions to ensure we don't break it.

So the formatter is what broke the Docker logging?

So the formatter is what broke the Docker logging?

Nothing to do with docker logging for this PR. But this will help me debug this better.


Deactivate the invalidation servlet

Since 0.8.5 we don't use the invalidation servlet anymore. With this change it will be deactivated by default but can be activated by using the dolphin.properties
If everything works well we can remove the servlet completely in a later version

This change is

Coverage decreased (-0.05%) to 65.055% when pulling 2a59dbea2cc1750ba76712d469a2d7cc17748220 on deactivate-invalidation-servlet into 7da209bb37fcd460c8b631c1d86ada315dc90ed2 on master.

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


chore: generate outputs concurrently

Description

generate outputs concurrently，I guess the outputs is isolate from each other
Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

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

[ ] Read the Contributing Guidelines.
[ ] Read the Pull Request Guidelines and follow the Commit Convention.
[ ] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] 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.

@HomyeeKing from a cursory look I don't think we can do this, Outputs should be built sequentially because Plugins are shared in each build and they may create resources on buildStart that may not be suitable for parallel builds. If you would like to propose this change, you need to check how rollup works here and all the implications.

https://rollupjs.org/guide/en/#generatebundle
ok, so the procedure is the outputs with same plugin, like  generatebundle this kind of hook will process them sequential and finally write right?

em.. I still think the outputs is isolated because the plugin just attach to the current output, it's nothing with the others,
as the generatebundle, the outputOption has already generated, so this is the last hook of generate

Also it seems that there is a failing test :thinking:
Please fix it, if you think it's worth it :slightly_smiling_face:

what will make the legacy-polyfill test failed,  this project has already used Promise.all

It's done intentionally: https://github.com/vitejs/vite/commit/ab805228e55bd79275b6ac67b0408419a8ab5c70

As we discussed yesterday with Evan, Node is single-threaded anyways. So we wouldn't gain a huge performance boost out of this.
The headaches with this change are to high as that we should care.
So thanks for your interest in improving Vite @HomyeeKing, but I will close this PR for now.
But you are welcome to work every time on other improvements as you like :slightly_smiling_face:


Custom Blocksets :)

First off I wanted to say amazing job on the tool it has been very helpful for some of my friends who build. One thing that was mentioned to me though (as one of my friends build more in modded minecraft) was the functionality for adding custom resources to the tool so you can get palettes for blocks that are not in the game.
After some fooling around I was able to introduce a "Custom Blockset" option.
Example:
https://user-images.githubusercontent.com/61068742/143781864-881f7155-957c-482b-ae2c-4da2f2670355.mp4
I also fixed a css issue with popup windows. (Firefox does not support backdrop-filter's)

Cheers 👍

I will admit this is a little bit monkey patched in due to the fact I usually use react. It has been tested and I have yet to find issues with it however 😆

Wow that's an amazing job, thanks for contribution!!


Update _helpers.py

socks5h & socks4a

It doesn't make any sense, just use socks4 and socks5

socks4 and socks4 resolve hostname locally
for some reason (eg. china),  hostname has been blocked. proxy failed .
socks5h and socks4a resolve hostname by socks server.

...resolve hostname...

Use  rdns (which means "remote resolve") argument

...resolve hostname...

Use rdns (which means "remote resolve") argument

Can we make 'rdns' automatically enabled by using prefix socks4a:// ?
And is it possible to have a compatible scheme socks5h:// for some scripts?

Can we make 'rdns' automatically enabled by using prefix socks4a:// ?

No. You should pass rdns argument explicitly (for socks4 proxy it is False by default)

And is it possible to have a compatible scheme socks5h:// for some scripts?

Use socks5 scheme, for socks5 proxies rdns=True by default

Can we make 'rdns' automatically enabled by using prefix socks4a:// ?

No. You should pass rdns argument explicitly (for socks4 proxy it is False by default)

So could it possible to have a easy way that using socks4:// for local resolve and socks4a:// for remote resolve?

And is it possible to have a compatible scheme socks5h:// for some scripts?

Use socks5 scheme, for socks5 proxies rdns=True by default

Sorry for edit.
Some commands like curl use local resolve with socks5:// which socks5h:// for remote. It's great to have a compatible scheme to guarantee remote resolving for the entire environment.

So could it possible to have a easy way that using socks4:// for local resolve and socks4a:// for remote resolve?

Sorry for edit.
Some commands like curl use local resolve with socks5:// which socks5h:// for remote. It's great to have a compatible scheme to guarantee remote resolving for the entire environment.

Everything is described above. Use socks4/socks5 scheme and rdns parameter for remote resolve.


AWS: Fix undefined variable

Previously, det_region variable was not defined if instance_id was not provided by Falcon platform.

ack


BIGTOP-3656. Fix Mpack's build and deployment problems on Ubuntu.

Description of PR
Currently, building Mpack fails on Ubuntu due to an unwanted binary file, and deployment with it also fails due to a wrong repository definition. This PR fixes them.
How was this patch tested?
Built Mpack with this PR and deploy a cluster via Ambari locally.
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'BIGTOP-3638. Your PR title ...')?
[x] Make sure that newly added files do not have any licensing issues. When in doubt refer to https://www.apache.org/licenses/

Oh, I've just noticed the same fix had already been submitted as #852. I'll close this PR and review that instead.


Implement Adam as a LearningRule

Citation: Adam: A Method for Stochastic Optimization. Diederik Kingma, Jimmy Ba. http://arxiv.org/abs/1412.6980
Implementation in Theano by Alec Radford (@newmu): https://gist.github.com/Newmu/acb738767acb4788bac3

Is someone already working on this? If not, I might try it out next week.

I'd like to request this please :+1:

Starting working on this in #1425.


Added conversion method for reqwest::Response to http::Response<_>

As mentioned in #1258

[x] Created an Into method to convert request::Response -> http::Response
Now what you have to do is implementing From for what you want to convert.

Feedback appreciated as this is my first contribution to the project.
Thanks!

Any issues with this PR apart from its out of date now?
I'm using the reqwest client in the hyper http proxy example and need to return http::Response.
I'm currently building a new http::Response, but this Into seems much better.

Oh, lemme see. Two things stick out at me:

Is Decoder public? I don't think it should be.
It's generally preferred to implement From, instead of Into, which you then get for free.


[MSG] Gallery Info

Likely not a release requirement for first version of messaging
Requirements

[ ] Gallery name
[ ] Gallery locations
[ ] Gallery description
[ ] Gallery response time
[ ] Call gallery button

1 day

Issue moved to artsy/collector-experience #296 via ZenHub


<EMAIL_ADDRESS>install: node-pre-gyp install --fallback-to-build fails on node 8.2.1

OS Platform: Node version 8.2.1 with npm 5.3.0 on Ubuntu 17.10
There seems to be an issue with Node 8 in this file: https://github.com/node-inspector/v8-debug/blob/master/src/debug.cc
Relating to a depreciated API call (new api source: https://github.com/v8/node/commit/293fc84613293b350d35a04a84801661b8c29bd9 )

8.1.4 works fine.

Attempted with<EMAIL_ADDRESS>and it still fails with the same error

Seems like the issue is the reliance on this api call: https://github.com/v8/node/commit/293fc84613293b350d35a04a84801661b8c29bd9

If you look at the beginning of your errors, do you see anything related to Python? For me I had the wrong version installed. It needs to be v2.7. See: https://github.com/nodejs/node-gyp

Sadly no, but on my system I have python 2.7 installed as python and python 3.6 under python3

Additionally, I downgraded my NodeJS from the latest (8.2.1 ) to the LTS 6.11.2. I think after this I was able to continue. Based on all my findings NodeJS 8.* won't work with this package.

Same here except that I found version 1.0.1 does work with node 7.
I raised this issue because I think I found why it does not work in node 8. When my life returns to sanity I'll take a crack at making the change

See also https://github.com/nodejs/node/issues/14411.  It sounds like not only does it not work now, things will only get worse with node 9.

I keep running into this issue and there seems to be no solution...

<EMAIL_ADDRESS>install /Users/xxx/.nvm/versions/node/v10.15.3/lib/node_modules/node-inspector/node_modules/v8-debug
node-pre-gyp install --fallback-to-build

macos install error

<EMAIL_ADDRESS>install /Users/lihaibo/.nvm/versions/node/v10.15.3/lib/node_modules/node-inspector/node_modules/v8-debug
node-pre-gyp install --fallback-to-build


#24: Renamed labels to keys in TimeSeries

https://github.com/cloudera/spark-timeseries/issues/24
Another find/replace PR.
Running on low sleep so my observational skills may not be in top form. Let me know if I missed anything..

Looks good, merging this in


Accessibility issue when resolution is set

Hi I am currently having an issue with Accessibility when the app resolution is set to the devicePixelRatio with a full screen canvas.
I have created a Pen to show the issue: https://codepen.io/c-andrews/pen/abbWJZd?editors=0010
Im currently working in Chrome. I don't seem to be able to tab into anything on Safari on a Mac but thats another issue.
When tabbing in Chrome the highlighted div is not over the button. If i remove the resolution from the App constructor then things are fine but I don't get the quality I need.
I am using the latest version of Pixi: 5.1.5.
I've also attached a screen shot:

Any help would be really appreciated, please let me know if i am missing something.

Try autoDensity: true, near resolution. Its not magic, it sets view.style.width and view.style.height.


Docker images for Tensorflow 2.6.2 are missing

TensorFlow 2.6.2 was released, but I cannot find docker images for TF 2.6.2 in docker hub.
https://hub.docker.com/r/tensorflow/tensorflow/tags?page=1&name=2.6.2

Since it is similar to Tensorflow version 2.6.1, the docker Images for version 2.6.2 are not included in docker hub.
You can use the equivalent Images for 2.6.2 with 2.6.1.

@sachinprasadhs Thank you for the response!


Added University Controllers and Tests

In this PR, we add a controller for the university service.
Closes #15

LGTM, link works and controller seems to be workiing


Text Dectection can return text in arbitrary order

Under certain circumstances, 	SparseArray textBlocks = detector.detect(frame) is either extracting text in some artibrary order and/or storing it arbitrarily in such a way that the text cannot be output in correct sequential order.
Look at the attached image.  Then look at the output from MobileVision and from Tesseract.  MobileVision's output is not displayed top to bottom.  Tesseract's output is displayed top to bottom.
I can't always duplicate the issue (ie, by photographing similarly presented text snippets), but mostly I can.  In every case, Tesseract will read top to bottom (even though the error rate might be a little higher).
Whether or not MobileVision's detector is trained to read top to bottom I can't say for sure, but the use of the SparseArray seems to be involved.
Below please find some log entries:
This is what the SparseArray holds.  Note the keys.
HASH: {3=com.google.android.gms.vision.text.TextBlock@2fcceb9,
4=com.google.android.gms.vision.text.TextBlock@28b692fe,
5=com.google.android.gms.vision.text.TextBlock@24fa2a5f,
7=com.google.android.gms.vision.text.TextBlock@1be72cac,
9=com.google.android.gms.vision.text.TextBlock@a129d75,
12=com.google.android.gms.vision.text.TextBlock@3364e10a}
There are 2 ways to step through the hash: either by using an index into the hash or by accessing the value of a hash entry by calling up its key.  These methods are effectively the same.
Here is the value (ie, text) held in each element of the hash.  We see the index used to step through the hash, the corresponding key of that element and the corresponding value (text) in that element:
block 0: key: 3, value: hate, but individual
block 1: key: 4, value: convinced, thanks to a sense of historical
block 2: key: 5, value: DEBORAH FELDMAN AUTHOR
block 3: key: 7, value: In Germany, I've certainly encountered
block 4: key: 9, value: courage, as well, from people who are
block 5: key: 12, value: responsibility, of the need to take a stand against hatred.
Note that the element held by block 3 has key 7 and value "In Germany, I've certainly encountered".
These are the first words of the text in the image.  There is no way to put them at the start of the text.
If you simply step through the hash with an index (0 to size - 1), you won't see the first words of the text until you reach the 4th block.  and although you can ask which key is held in the 4th block (block 3), there is no way to know that the element with key 7 is supposed to be output first.
Normally, a hash is not maintained in any discernible order and access is meant to be by key, but in this case knowing that key 7 is in the 4th block is of no help, since we don't know that key 7 is supposed to be the first one we should look at.  sorting the keys serves no purpose, and sorting the values is a waste of time.
If the image were multi-columnar and/or with heavy noise to confuse the extractor, I would expect jumbled output.  But that Tesseract - supremely susceptible to noise or stray characters implying another column somewhere - can figure out a simple top to bottom read, leads me to guess that the bug (if bug there is) is in how the detector assigns the hash keys.  It doesn't matter where the element with the first line of text is stored in the array, it just has to have the lowest key (otherwise, you don't know it's supposed to come first).  It doesn't have to be the 0th key, it just has to be the lowest key.  Non-text portions of the image have a right to be identified by key (even though they contain no text), but assigning key 7 to the first text line and key 3 to some line that comes after it makes no sense.  In fairness, the detector can assign keys however it sees fit, but there has to be some way for the developer to know the actual text order.
MOBILEVISION OUTPUT AS A WHOLE BLOCK:
hate, but individualconvinced, thanks to a
sense of historicalDEBORAH FELDMAN
AUTHORIn Germany, I've
certainly encounteredcourage, as well, from
people who areresponsibility, of the
need to take a stand
against hatred.

TESSERACT'S OUTPUT:
‘ln Germany, I’ve
certainly encountered
hate, but individual
courage, as well, from
people who are
convinwd, thanks to a
sense of historical
responsibility, of the
need to take a stand
against hatred.’
DEBORAH mom
AUTHOR
You should find the original image here:

Thanks for taking the time to read this.
-go

i must have posted this to the wrong group, sorry.  i figured out a workaround.  no harm, no foul

What is Textholder

just a class to hold the upper left coordinate of a text block and the text in the box.  sorry, i failed to include it in the code.
as mentioned in my previous explanation, the only (mostly) reliable way to know where each line of text appears in the ocr'd document is to use its topmost left coordinate within the document.  google vision's implementation doesn't do that.  as a result, what might be line 3 appears as the 0th element in google's list of blocks.  if you simply step through google's list of blocks, you would end up showing line 3 as the first line of text.  i worked with it enough to know that it was the case when i was testing google's text detection.  it was easy to demonstrate.  you couldn't step through the list by index, and you couldn't step through it by using whatever object it was that the index pointed to.
the closest i could come to overriding that behavior was to create my own list of blocks and store a text line's coordinates along with the text.  i then used the coordinates as the sort field.  the assumption being that if you sorted the lines based on their coordinates within the document, your output would be top to bottom.  for whatever reason, google bases its ordering on some other factor.  in most cases, google's output is fine, but not always.  i couldn't figure out a pattern in order to replicate the behavior, but when the behavior occurred, it was easy to see what was happening and to make it happen over and over with the same document and to figure out some kind of solution to override the behavior.
side note: i'm an old c programmer.  i apologize for failing to disguise that mindset in any java code i had to write to make the thing work.
class Textholder implements Comparable{ //public class Textholder {     private float Y;      private String text;        Textholder(float Y, String text) {           this.Y=Y;           this.text = text;       }      String getText() {          return text;     }      float getY() { return Y;      }  public int compareTo(Textholder th) {     if (Y == th.getY())     return 0;     else if (Y > th.getY())      return 1;     else       return -1; } }

go
On Tuesday, May 8, 2018, 11:33:25 AM EDT, Chidera<EMAIL_ADDRESS>wrote:

What is Textholder
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks it worked.

more like worked(around), i'm sorry to say.  good old plain tesseract - for all its problems - never failed to read top to bottom.  its child is blindingly fast, if slightly dyslexic.
On Wednesday, May 9, 2018, 4:16:22 PM EDT, Chidera<EMAIL_ADDRESS>wrote:
Thanks it worked.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


install-rocm.md: Problem with ROCm 5.0 and SSCP

Clang bundled with ROCm 5.0.2 is incompatible with SSCP, causing a compilation error.

Ugh, thanks. Not surprising that this doesn't work, but still sad. Given the role of SSCP as the upcoming new "default" compilation flow once it is more stable, we may have to rephrase this eventually to discourage ROCm clang more strongly.

It works fine with newer ROCm's, at least 5.4 that I have. It's that 14-but-not-quite version that's shipped with ROCm 5.0 specifically that is the problem. And unfortunately, some HPC installations of interest still use ROCm 5.0.2.

So, hipSYCL does not even need to be built against ROCm clang to take advantage of the ROCm LLVM backend :-)

But we lose things like #ifdef __gfx90a__ and other architecture-specific optimizations, don't we?

It works fine with newer ROCm's, at least 5.4 that I have. It's that 14-but-not-quite version that's shipped with ROCm 5.0 specifically that is the problem. And unfortunately, some HPC installations of interest still use ROCm 5.0.2.

Yeah, so the usual problem with in-between clang versions :(

But we lose things like #ifdef gfx90a and other architecture-specific optimizations, don't we?

Not necessarily. While macros will be unsupported for this purpose, the SSCP flow is designed to provide similar mechanism as e.g. nvc++ if target() to support target specialization. There is a lot of infrastructure already in place for this, and it is supported already for __hipsycl_if_target_host()/__hipsycl_if_target_device. APIs for device-specific specialization are not yet publicly available but will come too :)

There is a lot of infrastructure already in place for this, and it is supported already for __hipsycl_if_target_host()/__hipsycl_if_target_device

It is still somewhat limited. E.g., constructs like __hipsycl_if_target_device(atomicAdd(ptr, 1.0F);); don't work with generic, while working fine in explicit- and implicit multipass modes. And I don't think just declaring (but not defining) these functions for host pass will work, will it?

It is still somewhat limited. E.g., constructs like __hipsycl_if_target_device(atomicAdd(ptr, 1.0F);); don't work with generic, while working fine in explicit- and implicit multipass modes. And I don't think just declaring (but not defining) these functions for host pass will work, will it?

Well, you cannot use HIP source code in kernels. But you will likely be able to use inline assembly, and calling builtins from AMD device bitcode libraries like OCML or OCKL (https://github.com/RadeonOpenCompute/ROCm-Device-Libs) already works today by just declaring them (as long as they are only used on the AMD path).
It might also be possible link against bitcode libraries generated the clang HIP toolchain but that is still more unclear.
In other words, in your example you could not call atomicAdd because that is defined only inside HIP header files, and we cannot recreate HIP headers on the fly. You should however be able to just invoke the builtins that atomicAdd invokes under the hood. (assuming of course that you don't just want to call the SYCL atomics in the first place :) )

And I don't think just declaring (but not defining) these functions for host pass will work, will it?

There is no dedicated host pass, it's a unified host-device pass :) But I get what you are asking: What will happen when linking the host binary and there are undefined function declarations, which only get resolved on the device side.
The short answer is: This is supported, because device-only code is removed from the host binary by the SSCP compiler. So it will detect that these declarations are only used inside kernels, are therefore device code, and then remove them from the host code.
Because of this, we can actually even support SYCL_EXTERNAL device functions that are resolved dynamically in device code from a library that is opened via dlopen(). So you could even have a plugin architecture for kernels :D

Well, you cannot use HIP source code in kernels. But you will likely be able to use inline assembly, and calling builtins from AMD device bitcode libraries like OCML or OCKL (https://github.com/RadeonOpenCompute/ROCm-Device-Libs) already works today by just declaring them (as long as they are only used on the AMD path).

Yeah, unless those are arch-specific. Or, well, if one wants to target both AMD and NVIDIA devices. But that, as I understood, will also be doable soon :+1:

assuming of course that you don't just want to call the SYCL atomics in the first place :)

The last time I checked (late fall 2022), clang/llvm could not use "unsafe" AMD atomics when they were hidden under all the layers of SYCL, while plain atomicAdd were optimizable, with the right compiler flags.

The short answer is: This is supported, because device-only code is removed from the host binary by the SSCP compiler. So it will detect that these declarations are only used inside kernels, are therefore device code, and then remove them from the host code.

Ok. I thought the code for the host was always generated, my bad.
Re-declaring all the functions is an annoyance, but yeah, that is pretty cool, as long as JIT can optimize the code well enough. Unfortunately, GROMACS refuses to build for generic target as-is, and I don't have time to fix all the problems now and check out its performance :(

Yeah, unless those are arch-specific. Or, well, if one wants to target both AMD and NVIDIA devices. But that, as I understood, will also be doable soon

Not sure what you are referring to exactly. AMD OCKL/OCML builtins work across all AMD GPUs, but need to be configured for the appropriate ISA by linking with the right ROCm bitcode libraries. JIT takes care of that automatically.
Of course they won't work on NVIDIA, but that's expected if you start to write backend-specific code paths :)

The last time I checked (late fall 2022), clang/llvm could not use "unsafe" AMD atomics when they were hidden under all the layers of SYCL, while plain atomicAdd were optimizable, with the right compiler flags.

Well in that case it might be a better idea to first try to invoke our __hipsycl_sscp_* builtins for atomics directly once they are there before trying to reimplement things ;)

Re-declaring all the functions is an annoyance, but yeah, that is pretty cool, as long as JIT can optimize the code well enough. Unfortunately, GROMACS refuses to build for generic target as-is, and I don't have time to fix all the problems now and check out its performance :(

We already have headers for AMD ocmkl/ockl and NVIDIA libdevice - so you could just include those, I think, and not redeclare yourself :)
I don't expect it to work with Gromacs just yet. Atomics and group algorithms are not yet implemented which I think you need.
As for performance - I'm working on an IWOCL paper on SSCP at the moment, and so far it looks promising. Especially on AMD I am seeing cases where it outperforms the old compilation flow by 20% without substantial regressions so far. And I have not yet even started to look at optimizing SSCP perf in particular :)
JIT goes through LLVM -O3 pipeline after linking (so you always effectively get lto), then it goes to hipRTC which also optimizes.

Not sure what you are referring to exactly. AMD OCKL/OCML builtins work across all AMD GPUs, but need to be configured for the appropriate ISA by linking with the right ROCm bitcode libraries. JIT takes care of that automatically.

I was referring to things like __ockl_atomic_add_noret_f32, which are valid on all architectures, but only make sense on certain ones.

Of course they won't work on NVIDIA, but that's expected if you start to write backend-specific code paths :)

Well, in multipass mode, one can already both have the cake (portability) and eat it too (backend-specific code paths) :)

We already have headers for AMD ocmkl/ockl and NVIDIA libdevice - so you could just include those, I think, and not redeclare yourself :)

Currently, not so much. We cannot include NVIDIA headers if there's no CUDA installed on the machine, so we cannot compile on HPC systems that only have ROCm stack available. More PP macros can solve the problem, of course, but that is ugly.

Atomics and group algorithms are not yet implemented which I think you need.

That can be danced around with intrinsics/builtins. A bigger problem would be that we rely on knowing the sub-group size at compile time. E.g., we do reduction manually, and we need to know both the number of steps to do and the size of the local memory buffer (and whether the buffer is needed at all). That is what can be hard to optimize, I think.

As for performance - I'm working on an IWOCL paper on SSCP at the moment, and so far it looks promising. Especially on AMD I am seeing cases where it outperforms the old compilation flow by 20% without substantial regressions so far. And I have not yet even started to look at optimizing SSCP perf in particular :)

:+1:. Do you have any idea why JIT works better than AOT compilation?

I was referring to things like __ockl_atomic_add_noret_f32, which are valid on all architectures, but only make sense on certain ones.

Ok. Sure, but we are talking about creating optimized code paths for particular HW already, so having to specialize to some extent already is expected, no?

Well, in multipass mode, one can already both have the cake (portability) and eat it too (backend-specific code paths) :)

Only if you ignore Intel hardware ;) With SSCP Intel is just as well supported as AMD and NV. In multipass.. it's complicated ;)

Currently, not so much. We cannot include NVIDIA headers if there's no CUDA installed on the machine, so we cannot compile on HPC systems that only have ROCm stack available. More PP macros can solve the problem, of course, but that is ugly.

Yes, that's true for multipass, but I was talking about SSCP where we e.g. already ship ockl headers as part of hipSYCL:
https://github.com/illuhad/hipSYCL/blob/develop/include/hipSYCL/sycl/libkernel/sscp/builtins/amdgpu/ockl.hpp
So I think you could just include that file and invoke those builtins directly without having to declare them yourself.

That can be danced around with intrinsics/builtins. A bigger problem would be that we rely on knowing the sub-group size at compile time. E.g., we do reduction manually, and we need to know both the number of steps to do and the size of the local memory buffer (and whether the buffer is needed at all). That is what can be hard to optimize, I think.

You would have to do something like:
if(__hipsycl_sscp_get_subgroup_max_size() == 32){
  run<32>();
else if(__hipsycl_sscp_get_subgroup_max_size() == 64) {
  run<64>();
}

Alternatively, you could query the backend and specialize based on that. But I don't think it would make a difference, because those queries return constants which should allow SSCP runtime kernel outlining to throw away to unneeded paths.

Sure, but we are talking about creating optimized code paths for particular HW already, so having to specialize to some extent already is expected, no?

Yes, fair enough. Does not make sense to complain about a known limitation which is soon to be fixed anyway :)

Yes, that's true for multipass, but I was talking about SSCP where we e.g. already ship ockl headers as part of hipSYCL:
https://github.com/illuhad/hipSYCL/blob/develop/include/hipSYCL/sycl/libkernel/sscp/builtins/amdgpu/ockl.hpp

Missed them. That's convenient.

But I don't think it would make a difference, because those queries return constants which should allow SSCP runtime kernel outlining to throw away to unneeded paths.

We'll see. I'm a bit skeptical given how often we have to fight with the compiler over similarly "minor" things, but maybe these conditionals will be optimized just fine.

Missed them. That's convenient.

Our __hipsycl_sscp_* builtin implementations need to have declarations too so they can call backend builtins, that's why we have those headers :-)

We'll see. I'm a bit skeptical given how often we have to fight with the compiler over similarly "minor" things, but maybe these conditionals will be optimized just fine.

Well, the host-device code separation relies on a similar mechanism using if(__hipsycl_sscp_is_device){}, so that better be reliable ;)
The only thing that could be an issue in this case is that in the AMD path, there's an additional if-statement based on __oclc_wavefrontsize64 to support both wave32 and wave64 modes on AMD.  __oclc_wavefrontsize64 only gets resolved by hipRTC, after our kernel reoutlining steps already have already run. However, the AMD bitcode libraries like ockl themselves rely heavily on being able to optimize precisely these if-statements, so hipRTC will have to ensure this too.
If hipRTC is not thorough enough here, we could however always insert the correct value ourselves during JIT. Bottom line is that it should be possible to support this either way.


⚠ Bofrost.fr has degraded performance

In ac64ccd, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2407 ms

Resolved: Bofrost.fr performance has improved in e8ba08e.


Gate Chalice crash with Gesture of the Drowned

Gate chalice appears to hardlock the game and throw NREs in other mods when activated. When we found this initially, we had Gestures of the Drowned, which may have been the issue. I'm looking to test this with DebuggingToolKit to try and narrow down the issue.
Twitch clip where we first found this happening: https://clips.twitch.tv/ExquisiteAlluringRhinocerosRedCoat-Ty0gba6pXsmBwxph

it seems like this is actually a mod compatibility issue with Spikestrip or similar, not an issue with Gate Chalice- sorry! <3


How do I know  that which clientId is online

Try dashboard.
https://github.com/emqtt/emqttd_dashboard


Redirect docs subdomain

The config in the previous PR did not work, this PR will

Remove the config applied with #736
Add redirect config to _redirects file in order to remain consistent with existing redirects

Hm I think this has happened after I deleted the old readthedocs site - that site was serving redirects from old config to the new pages.
Given those aren't 1:1 either (some content was re-organised) we may want to consider porting across the old rules... I'll see if I can dig them out!

Here is the full list: https://github.com/jetstack/cert-manager/blob/release-0.12/docs/external_redirects

Which may also want integrating with this: https://github.com/jetstack/cert-manager/blob/release-0.12/docs/redirects for completeness..


Login attempts, no limit...

I get my login password wrong 5 times, I get the "ip blocked" message, but if I try to log in immediately after with the correct password, I log in easily.
Basically it's like there's no blocking on incorrect login attempts...

The server is a local or remote?
Are you using a proxy?
IPs stored in ip_lock and user_session are the same?

Remote server, Plesk with nginx (inverse proxy to apache).

IPs stored in ip_lock and user_session are the same?
Where can I see it?

The first step in login process is check if the IP is locked.
https://github.com/eusonlito/Password-Manager/blob/929313d801ba2bae567c22103aaeb335fbe9cc45/app/Domains/User/Action/AuthCredentials.php#L33
If the IP is blocked then it will throw an exception and the process will not continue with the credential verification.
Please check the IPs stored in ip_lock and user_session database tables.
Do you have the default lock and check values in .env:
# How many auth fails are allowed
AUTH_CHECK_ALLOWED=4
# Check fails in last X seconds
AUTH_CHECK_TIME=60
# Lock IP X seconds
AUTH_LOCK_TIME=60

Okay.
In "user_session" there are my login attempts, and the IP is that of the client from which I try to login (so so far so good).
Instead "ip_lock" is empty, it doesn't log the IP to lock out when failing attempts.

And no errors inside storage/logs/laravel folder?
The ip lock is only available using the ip_lock table.

And no errors inside storage/logs/laravel folder?
No. I deleted "storage/logs/laravel-2023-01-14.log", simulated the problem again and no file is created, so no, it leaves no errors...

My env:
`[APP.....]
APP_ENV=production
APP_DEBUG=false
DEBUGBAR_ENABLED=false
DB_CONNECTION=mysql
[DB.....]
DB_LOG=false
SESSION_DRIVER=cookie
SESSION_LIFETIME=45000
SESSION_COOKIE=password-manager
SESSION_ENCRYPT=false
AUTH_CHECK_ALLOWED=5
AUTH_CHECK_TIME=60
AUTH_LOCK_TIME=60
AUTH_TFA_ENABLED=false
AUTH_CERTIFICATE_ENABLED=false
AUTH_API_SECRET_ENABLED=false
AUTH_COUNTRY_ENABLED=false
AUTH_COUNTRY_ALLOWED=
AUTH_COUNTRY_IP_WHITELIST=<IP_ADDRESS>
LOG_CHANNEL=stack
LOG_REQUEST=false
LOG_DATABASE=false
LOG_MAIL=true
LOG_CURL=false
LOG_DAYS=30
[MAIL.....]
CACHE_ENABLED=true
CACHE_DRIVER=file
CACHE_PREFIX=password-manager-cache-
CACHE_VERSION=1
CACHE_TTL=3600
SENTRY_DSN=
SENTRY_TRACES_SAMPLE_RATE=0.0`

Ok, I have tested my environment and works fine.
I will test creating a new installation using your settings.

If I add a line with my ip to "ip_lock", the login is blocked (great).
The problem is then that, when it reaches the bad attempts, it doesn't register the ip to exclude in the "ip_lock" table.

Ok, thanks, I understand the problem :)
Thanks!

I was looking here:
https://github.com/eusonlito/Password-Manager/blob/master/app/Domains/IpLock/Action/Create.php#L24
I'll wait for your directions. :)
Thanks for the support.

Solved. Was a problem with a Transaction.
I'm adding tests with this case :)


add ability to publish multiple versions per package

useful when build script produces cjs/dual/esm versions
sample config from main "package.json":
{
  ...
  "scripts": {
    "changeset:publish": "npm run build && changeset publish"
  },
  ...
}

and from individual packages:
{
  ...
  "publishConfig": {
    "tag": "alpha",
    "versions": {
      "npm": {
        "directory": "npmDist"
      },
      "esm-only": {
        "suffix": "-esm",
        "directory": "npmEsmDist"
      }
    }
  }
  ...
}

maybe there is a better approach?

Goal: Allow changesets to kick off multiple new releases with different versions/tags whenever a release must be published.
Options:

write custom publish script instead of changeset publish

pro: complete control
con: we have to re-implement all functionality within changeset publish

call changeset publish more than once with different inputs

pro: nothing need change within changeset publish
cons:
(a) have to carefully update changeset publish in terms of detecting "unpublished packages" -- which version?
(b) seems confusing in a monorepo where the publishing behavior may differ per package

allow individual packages to declare that they should be published as separate versions

the approach of this PR

Hmm, I think this is rather a niche use case - usually, people are publishing different flavors of a package within the package itself. Any particular reason why do you want to publish multiple package names?

Yes we want an esm only version for purists and a dual cjs/esm version for broader compatibility.
Esm only reduces package size and avoids the dual package hazard when all dependents use it.
See https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c?permalink_comment_id=3850849#gistcomment-3850849

I started this issue for a project of mine, but it looks to me that this workflow and https://github.com/changesets/action/issues/262 are the technical impediments to introducing changesets at https://github.com/graphql/graphql-js so I wouldn’t call it niche per se, although there are perhaps better solutions.
another perhaps better option is for the esm only package to track releases of the dual version and extract the esm only version…

I realize that it would be easier for you if we'd support this within Changesets. However, this is the first time that somebody has asked for this and I still consider this to be a rather niche use case. I'd like to avoid needless complexity, configuration options etc.
That being said - this doesn't have to stop you from using Changesets in your project. All of this depends on how your current setup looks like... but I would do smth like this:

write a "prepare" script that builds all packages into multiple publish dirs etc
in that script adjust the root's package.json#workspaces to point to your publish dirs
call changeset publish
revert step 2

On top of that, you might want to actually auto-generate some changeset files for your extra ESM variants. I'm not sure if you want to simply duplicate changeset content or if you want to point to the dual package content or what - that's up to you.
Note that the content of this PR doesn't cover everything that would have to be done to land this as a feature here. All dependent packages should also get published, specified dependencies should likely be adjusted to their ESM-only variants etc. That's why I think this is, at the moment, a complexity that I'd like to avoid in the core. All of that can be scripted around Changesets.

That seems like a reasonable approach. I am actually using changesets in "non monorepo mode" so I haven't encountered all the extra complexity that dependent packages might entail. I'll give it a go -- thanks!
Any thoughts on https://github.com/changesets/action/issues/262 ?

It worked! Thanks again @Andarist !!!


Add tolerations and local storage to the deployment

The deployment has affinity in the form of nodeSelector but not tolerations for taints. This PR adds optional tolerations to the deployment resource.
Also, an optional persistent volume claim is created if enabled in the values; this allows the deployment to have a persistent storage to keep the emails, in case it is required.
The environment variables for the storage have to be set manually in the values like:
extraEnv:
  INBUCKET_STORAGE_TYPE: "file"
  INBUCKET_STORAGE_PARAMS: "path:/storage"

Error: UPGRADE FAILED: template: inbucket/templates/pvc.yaml:1:14: executing "inbucket/templates/pvc.yaml" at <.Values.persistence.enabled>: nil pointer evaluating interface {}.enabled


Compilation Error on macOS

For a build with Metal according to the README, see:
(base) ➜  bert.cpp-future git:(master) make -C build
[ 50%] Built target ggml
[ 58%] Building CXX object src/CMakeFiles/bert.dir/bert.cpp.o
In file included from /Users/turbo/dev/bert.cpp-future/src/bert.cpp:9:
/Users/turbo/dev/bert.cpp-future/src/bert.h:186:22: warning: 'bert_tokenize' has C-linkage specified, but returns incomplete type 'bert_tokens' (aka 'vector<int>') which could be incompatible with C [-Wreturn-type-c-linkage]
BERT_API bert_tokens bert_tokenize(
                     ^
/Users/turbo/dev/bert.cpp-future/src/bert.h:192:22: warning: 'bert_detokenize' has C-linkage specified, but returns user-defined type 'bert_string' (aka 'basic_string<char>') which is incompatible with C [-Wreturn-type-c-linkage]
BERT_API bert_string bert_detokenize(
                     ^
/Users/turbo/dev/bert.cpp-future/src/bert.cpp:358:33: error: no matching function for call to 'min'
    memcpy(output, str.c_str(), std::min(n_output, str.size()));
                                ^~~~~~~~

Yeah, I think that probably needs an <algorithm> include. More importantly though, we got merged into llama.cpp, so this repo is mostly defunct. Check it out over there!


How can I store in wide dataset looks like input_image/output_image/metainfo ?

I have complicated dataset looks like:
input_image.jpg, output_image.jpg, metainfo.json ...
I can't do this:
pil_dataset = wds.WebDataset(url).to_tuple("jpg", "jpg", "json")
Is there any ideas how to do this?

I'm not sure what you mean. The WebDataset filename conventions exist so that grouping is possible. If you have arbitrary names like that, how do you know where one sample ends and the next one starts?
In any case, if your file names do not conform to standard WebDataset conventions, you have several options:

you can use WebDataset(..., rename_files=some_function)
you can explicitly construct a DataPipeline analogous to WebDataset and use a custom grouper

The rename_files variety is preferred because it will ensure that subsequent processing steps get the kind of file names they expect.


Bump to 2022 UNWPP Estimates

Changes

Total and 18+ denoms shifted to UNWPP 2022 Estimates (still 2021)
Updated UNWPP links (some dead, different file types)
Jersey, Guernsey, and Kosovo now come from UNWPP directly (were CIA WFB before)
Docs updated

Closes #74


Unhold not working in Chrome Version 61.0.3163.79 (Official Build) (64-bit)

I'm using freeswitch as server and using sipjs for calling, answering, holding and unholding.
Although holding work without any error, unholding is not working most of the time. Yes it works sometime if I unhold within 3 or 4 secs may be. I will paste the log in the following.

Working situation
Fri Sep 08 2017 12:56:23 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | RTCIceChecking Timeout Triggered after 5000 milliseconds
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:23 GMT+0900 (Japan Standard Time) | sip.transaction.ict | Timer D expired for INVITE client transaction z9hG4bK7641051
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:24 GMT+0900 (Japan Standard Time) | sip.dialog | new UAC dialog created with status EARLY
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:37 GMT+0900 (Japan Standard Time) | sip.dialog | dialog 29amhb6dnf2hp2ta40vuegbn7tj4qjZHUt05t4rNc7N  changed to CONFIRMED state
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:37 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | stream added: VgKZOIOzWKA5zi8dFJ1Did6pZGNHqW7c
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:40 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | RTCIceGatheringState changed: complete
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:42 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | RTCIceChecking Timeout Triggered after 5000 milliseconds
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:51 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | already have local media
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:52 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | stream removed: VgKZOIOzWKA5zi8dFJ1Did6pZGNHqW7c
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:55 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | already have local media
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:55 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | stream added: VgKZOIOzWKA5zi8dFJ1Did6pZGNHqW7c
sip-0.7.8.js:2900 Fri Sep 08 2017 12:56:55 GMT+0900 (Japan Standard Time) | sip.transaction.ict | Timer B expired for INVITE client transaction z9hG4bK5668212

Not working situation
Fri Sep 08 2017 12:57:22 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | already have local media
sip-0.7.8.js:2900 Fri Sep 08 2017 12:57:23 GMT+0900 (Japan Standard Time) | sip.transaction.ict | Timer B expired for INVITE client transaction z9hG4bK2815773
sip-0.7.8.js:2900 Fri Sep 08 2017 12:57:24 GMT+0900 (Japan Standard Time) | sip.transaction.ict | Timer M expired for INVITE client transaction z9hG4bK2815773
sip-0.7.8.js:2900 Fri Sep 08 2017 12:57:25 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | stream removed: VgKZOIOzWKA5zi8dFJ1Did6pZGNHqW7c
sip-0.7.8.js:2900 Fri Sep 08 2017 12:57:27 GMT+0900 (Japan Standard Time) | sip.transaction.ict | Timer B expired for INVITE client transaction z9hG4bK1308245
sip-0.7.8.js:2900 Fri Sep 08 2017 12:57:27 GMT+0900 (Japan Standard Time) | sip.transaction.ict | Timer M expired for INVITE client transaction z9hG4bK1308245
sip-0.7.8.js:2900 Fri Sep 08 2017 12:57:30 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | already have local media
sip-0.7.8.js:2900 Fri Sep 08 2017 12:57:36 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | already have local media
sip-0.7.8.js:2900 Fri Sep 08 2017 12:57:38 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | already have local media
sip-0.7.8.js:2900 Fri Sep 08 2017 12:57:39 GMT+0900 (Japan Standard Time) | sip.invitecontext.mediahandler | already have local media

For holding and unholding, both of the log first saying already have local media and then stream removed for holding and stream added for unholding.
In not working situation, already have local media only show and no stream removed log is outputting.

Please try out the PR branch which not only up-ends how we deal with media, but also adds official support for hold/unhold. The PR is close to complete, and will also mark a new version.

We believe that this issue may be resolved in the latest tagged version 0.8.0. Can you please retest with that version and let us know if this is still an issue. We will close this issue on 11/1/17 if we do not hear back. Thanks.

I retested it and unfortunately, I could not regenerate the unholding error again using SIP version 0.7.8. I could not think what cause that problem. Since, unholding works perfectly now, it is okay to close the issue. Thanks for the reply @james-criscuolo  and @egreenmachine.


ARM container numcodecs fix

Modulus Pull Request
Description
Workaround fix for #175
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The CHANGELOG.md is up to date with these changes.
[x] An issue is linked to this pull request.

Dependencies

/blossom-ci

This PR looks good overall. But maybe we should think about how we can make Dockerfile changes like this and test container builds it in an environment that mimics an end user's and doesn't have all these custom wheels available.

Thanks @akshaysubr , you are right. I have done such testing for this and the previous PR. i.e. these builds work, even when these pre-built wheels are not available as it just reverts to source build in that case. The motivation to add these checks for pre-built wheels was to speed up internal builds. We need to do them nightly and having things build from source every time slows things down.
That being said, you brought up an interesting point. I have been discussing this with @NickGeneva recently. We need to have a robust source build instructions that mimic the docker build behavior. A lot of projects have a third_party folder to track such dependencies and we should consider adding such to Modulus. Prepared this issue to track this further: https://github.com/NVIDIA/modulus/issues/176

/blossom-ci


1.2.dev0

Prepare for next release

Codecov Report

Merging #79 (ffd906d) into main (333a1cd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage   94.64%   94.64%
=======================================
  Files           8        8
  Lines         392      392
=======================================
  Hits          371      371
  Misses         21       21

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


cyfast-extension-plugin

Repository URL
https://github.com/bhupathiraju1998/cyfast-extension
New Repository Name
cyfast-extension-plugin
Description
CyFAST Extension
How to configure
Free style Project :-
select the cyfast extension plugin in the build step and give necessary input values like project id and orchestration id , username , password and the VM url.
Pipeline project:-
select the pipeline syntax while genrating the pipeline and go to snippet generator and select the general build step and select cyfast extension and give necessary input values like project id and orchestration id , username , password and the VM url.
GitHub users to have commit permission
<EMAIL_ADDRESS>Jenkins project users to have release permission
<EMAIL_ADDRESS>Issue tracker
Jira

Security audit, information and commands
The security team is auditing all the hosting requests, to ensure a better security by default.
This message informs you that a Jenkins Security Scan was triggered on your repository.
It takes ~10 minutes to complete.
Commands
The bot will parse all comments, and it will check if any line start with a command.
Security team only:

/audit-ok => the audit is complete, the hosting can continue :tada:.
/audit-skip => the audit is not necessary, the hosting can continue :tada:.
/audit-findings => the audit reveals some issues that require corrections :pencil2:.

Anyone:

/request-security-scan => the findings from the Jenkins Security Scan were corrected, this command will re-scan your repository :mag:.
/audit-review => the findings from the audit were corrected, this command will ping the security team to review the findings :eyes:.
	It's only applicable when the previous audit required changes.

Only one command can be requested per comment.

(automatically generated message, version: 1.19.14)

:x: Jenkins Security Scan failed.
The Security team was notified about this.

/hosting re-check

After the parent pom correction here is the result of the security scan. Next step for you is to follow the instructions to make it pass :)

Stapler: Missing POST/RequirePOST annotation
You can find detailed information about this finding here.
FastBuilder.java#192
Potential CSRF vulnerability: If DescriptorImpl#doCheckName connects to user-specified URLs, modifies state, or is expensive to run, it should be annotated with @POST or @RequirePOST

Stapler: Missing permission check
You can find detailed information about this finding here.
FastBuilder.java#192
Potential missing permission check in DescriptorImpl#doCheckName

Jenkins: Plaintext password storage
You can find detailed information about this finding here.
FastAction.java#10
Field should be reviewed whether it stores a password and is serialized to disk: accessToken

FastBuilder.java#43
Field should be reviewed whether it stores a password and is serialized to disk: password

https://github.com/bhupathiraju1998/cyfast-extension/blob/88fe01a9893dfa7e2a0dc71f7be852fc3256a288/src/main/java/io/jenkins/plugins/sample/FastBuilder.java#L199 does not look like something that should remain in production code.

/hosting re-check


Sentry Flutter Web

Sentry flutter web currently uploads the source maps e.g. below, and you can see every filename in our repo. Is there any way to prevent this from happening while ensuring errors can be traced in Sentry issues viewer?
Chrome sources tab:

Existing configuration in pubspec.yaml
sentry:
  upload_native_symbols: true
  upload_source_maps: true
  include_native_sources: true
  project: xxx
  org: xxx
  auth_token:
  wait_for_processing: false
  log_level: error
  commits: false
  web_build_path: build/web
  url: ~

Closed duplicate of https://github.com/getsentry/sentry-dart-plugin/issues/71


New package: FileCloud.FileCloudDrive version <IP_ADDRESS>53

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Resolve #177371

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978334957
2024-10-03 09:45:35.824 [CLI ] ShellExecute installer failed: 1633
MSI (s) (98:10) [09:45:35:650]: Product: FileCloud Drive -- Installation failed.
This installation package is not supported by this processor type. Contact your product vendor.
2024-10-03 09:45:34.477 [CLI ] ShellExecute installer failed: 1633
MSI (s) (58:08) [09:45:34:320]: Product: FileCloud Drive -- Installation failed.

Automated error analysis suggests -1978334957 may mean Package not supported by the system.
Automated error analysis suggests 1633 may mean ERROR_INSTALL_PLATFORM_UNSUPPORTED - This installation package is not supported by this processor type. Does this manifest have Architecture: x86? It might need to be x64.
(Automated response - build 896.)

Verify:


Small typo or overlap

In "The Critical Damping Coefficient" part the second figures "Momentum β" axis starts from .0 and goes to 0.0. I assume that it should start with 1.0.  My browser: Google Chrome, System: Windows 10.

The issue is overlap, with it getting cut off. Proposed fix in #71.

Thanks chris and sam, closing this issue now


Not correct raw select witch IF - convert field type

Laravel Version: 5.6.25
PHP Version: 7.2
Database Driver & Version: MySQL 5.6

Description:
Not correct raw select witch IF
i 'am find bug in DB fasade - selectRaw() and select(DB::raw()) sql expression include's directive IF - change output type field
Steps To Reproduce:
example: selectRaw("IF(1=1,1,'string')") - result  = '1' - as string, but not integer, How??
if two parametr - integer,
selectRaw("IF(1=1,1,2)") - result = 1 - is integer - correct

Heya, unfortunately we don't support this version anymore. Can you please try to upgrade to the latest version and see if your problem persists? If so feel free to reply and we'll try to have a look.


Strange comparison in RTC code

Hello M5Stack.
I'm using your code with ESP-IDF and the compiler gave me the warning : "comparison is always true" at the following location :
https://github.com/m5stack/M5Core2/blob/f9589994bfc432119a9121f3f42f8a1ade7105f9/src/RTC.cpp#L221
Indeed, the types of the fields Minutes, Hours, ... of RTC_TimeTypeDef are all uint8_t and an unsigned integer is always positive. So the first 4 tests in the SetAlarmIRQ method seem to be useless, and the variable irq_enable is thus also not needed. Or did I miss something?
Please tell me if you want me to submit a pull request with the simplified version (without the tests).

Thanks for catching the problem, it might be caused by some oversight, you can try to post a PR to help us optimize it.

well since all the attributes are aways >= 0 the optimiser removes the test so it's just as if the code was
int RTC::SetAlarmIRQ(const RTC_TimeTypeDef &RTC_TimeStruct) {
  uint8_t out_buf[4] = {0x80, 0x80, 0x80, 0x80};
  out_buf[0] = ByteToBcd2(RTC_TimeStruct.Minutes) & 0x7f;
  out_buf[1] = ByteToBcd2(RTC_TimeStruct.Hours) & 0x3f;
  uint8_t reg_value = ReadReg(0x01);
  reg_value |= (1 << 1);
  for (int i = 0; i < sizeof out_buf; i++) WriteReg(0x09 + i, out_buf[i]);
  WriteReg(0x01, reg_value);
  return 1;
}

int RTC::SetAlarmIRQ(const RTC_DateTypeDef &RTC_DateStruct,
                     const RTC_TimeTypeDef &RTC_TimeStruct) {
  uint8_t out_buf[4] = {0x80, 0x80, 0x80, 0x80};
  out_buf[0] = ByteToBcd2(RTC_TimeStruct.Minutes) & 0x7f;
  out_buf[1] = ByteToBcd2(RTC_TimeStruct.Hours) & 0x3f;
  out_buf[2] = ByteToBcd2(RTC_DateStruct.Date) & 0x3f;
  out_buf[3] = ByteToBcd2(RTC_DateStruct.WeekDay) & 0x07;
  uint8_t reg_value = ReadReg(0x01);
  reg_value |= (1 << 1);
  for (int i = 0; i < sizeof out_buf; i++) WriteReg(0x09 + i, out_buf[i]);
  WriteReg(0x01, reg_value);
  return 1;
}

using that code would remove the warnings

Fixed. https://github.com/m5stack/M5Core2/commit/7938c7ee42277fede8c0dee39e0b906bb7ee87e7


Error when trying to open public folder in Finder on MacOS

my log id: f064258319bcd6214a84571c

Unfortunately this is either a macOS or an osxfuse bug, we're not sure:

osxfuse/osxfuse#478
osxfuse/osxfuse#421

It seems to be pretty rare and goes away after a restart I think.

Well, unfortunately it does not go away after restart on my machine.
Log id from just after restart: 8824a2001cd4ac0f1de1211c

Hmm ok, thanks.  Unfortunately Keybase logs don't help much with this, since the error is coming from another layer before the request even gets to us.  Are you able to navigate directly to the folder in Finder, and/or using cd on the terminal?

Yes, I can - everything is fully functional from commandline - navigating to the folder, copying files, etc. So this bug is not critical.


Failing to deploy at dev

It looks like the current master version, when run with the dev SDK, fails to grind deploy.
build-project-templates
  running dart pub get (/Users/brettmorgan/Documents/GitHub/dart-services/project_templates/null-safe/dart_project)
  /Users/brettmorgan/Documents/GitHub/dart-services/flutter-sdk/bin/cache/dart-sdk/bin/dart pub get
  Resolving dependencies...
  Waiting for another flutter command to release the startup lock...
  Running "flutter pub get" in pubspec-scratchqADF9Y...
  Running "flutter pub get" in pubspec-scratch7JVTcT...
  + async 2.8.2
  + bloc 7.2.1
  + characters 1.1.0 (1.2.0 available)
  + charcode 1.3.1
  + clock 1.1.0
  + collection 1.15.0
  + crypto 3.0.1
  + ffi 1.1.2
  + file 6.1.2
  + flutter 0.0.0 from sdk flutter
  + freezed_annotation 0.14.3 (0.15.0 available)
  + google_fonts 2.1.0
  + http 0.13.4
  + http_parser 4.0.0
  + intl 0.17.0
  + js 0.6.3
  + json_annotation 4.1.0
  + lints 1.0.1
  + meta 1.7.0
  + nested 1.0.0
  + path 1.8.0
  + path_provider 2.0.5
  + path_provider_linux 2.1.0
  + path_provider_macos 2.0.2
  + path_provider_platform_interface 2.0.1
  + path_provider_windows 2.0.3
  + pedantic 1.11.1 (discontinued replaced by lints)
  + platform 3.0.2
  + plugin_platform_interface 2.0.2
  + process 4.2.3
  + provider 6.0.1
  + riverpod 0.14.0+3
  + sky_engine 0.0.99 from sdk flutter
  + source_span 1.8.1
  + state_notifier 0.7.1
  + string_scanner 1.1.0
  + term_glyph 1.2.0
  + typed_data 1.3.0
  + vector_math 2.1.0 (2.1.1 available)
  + win32 2.2.9
  + xdg_directories 0.2.0
  Changed 41 dependencies!
  running flutter packages get (/Users/brettmorgan/Documents/GitHub/dart-services/project_templates/null-safe/flutter_project)
  /Users/brettmorgan/Documents/GitHub/dart-services/flutter-sdk/bin/flutter packages get
  Running "flutter pub get" in flutter_project...
      6.1s

  Because dartpad_sample depends on flutter_test any from sdk which depends on characters 1.2.0, characters 1.2.0 is required.
  So, because dartpad_sample depends on characters 1.1.0, version solving failed.
  pub get failed (1; So, because dartpad_sample depends on characters 1.1.0, version solving failed.)
  failed: Unable to exec /Users/brettmorgan/Documents/GitHub/dart-services/flutter-sdk/bin/flutter, failed with code 1

Closing out, no longer required


Update UC Browser in keyboardevent-key.json

Tested via BrowserStack on Samsung Galaxy S7, UC Browser 12.10 with https://jsbin.com/jiyazimoha/1

Invalid PR since it adds new browser version

Just tested it myself with the latest version of UC Browser and it fails in my test:
https://tests.caniuse.com/?feat=keyboardevent-key
And yes, can't add a version, though I'm about to do an update.


ceph-handler: group listen topics and condition

We are using multiple listen topics with the handlers. That means that
we are notifying 4 tasks for each handler.
Instead we can group the listen on an include_tasks and based on the
group condition.
Before:
NOTIFIED HANDLER ceph-handler : set _mon_handler_called before restart for mon0
NOTIFIED HANDLER ceph-handler : copy mon restart script for mon0
NOTIFIED HANDLER ceph-handler : restart ceph mon daemon(s) for mon0
NOTIFIED HANDLER ceph-handler : set _mon_handler_called after restart for mon0
NOTIFIED HANDLER ceph-handler : set _osd_handler_called before restart for mon0
NOTIFIED HANDLER ceph-handler : copy osd restart script for mon0
NOTIFIED HANDLER ceph-handler : restart ceph osds daemon(s) for mon0
NOTIFIED HANDLER ceph-handler : set _osd_handler_called after restart for mon0
NOTIFIED HANDLER ceph-handler : set _mds_handler_called before restart for mon0
NOTIFIED HANDLER ceph-handler : copy mds restart script for mon0
NOTIFIED HANDLER ceph-handler : restart ceph mds daemon(s) for mon0
NOTIFIED HANDLER ceph-handler : set _mds_handler_called after restart for mon0
NOTIFIED HANDLER ceph-handler : set _rgw_handler_called before restart for mon0
NOTIFIED HANDLER ceph-handler : copy rgw restart script for mon0
NOTIFIED HANDLER ceph-handler : restart ceph rgw daemon(s) for mon0
NOTIFIED HANDLER ceph-handler : set _rgw_handler_called after restart for mon0
NOTIFIED HANDLER ceph-handler : set _mgr_handler_called before restart for mon0
NOTIFIED HANDLER ceph-handler : copy mgr restart script for mon0
NOTIFIED HANDLER ceph-handler : restart ceph mgr daemon(s) for mon0
NOTIFIED HANDLER ceph-handler : set _mgr_handler_called after restart for mon0
NOTIFIED HANDLER ceph-handler : set _rbdmirror_handler_called before restart for mon0
NOTIFIED HANDLER ceph-handler : copy rbd mirror restart script for mon0
NOTIFIED HANDLER ceph-handler : restart ceph rbd mirror daemon(s) for mon0
NOTIFIED HANDLER ceph-handler : set _rbdmirror_handler_called after restart for mon0

After:
NOTIFIED HANDLER ceph-handler : mons handler for mon0
NOTIFIED HANDLER ceph-handler : osds handler for mon0
NOTIFIED HANDLER ceph-handler : mdss handler for mon0
NOTIFIED HANDLER ceph-handler : rgws handler for mon0
NOTIFIED HANDLER ceph-handler : mgrs handler for mon0
NOTIFIED HANDLER ceph-handler : rbdmirrors handler for mon0

Signed-off-by: Dimitri Savineau<EMAIL_ADDRESS>

jenkins test centos-non_container-all_daemons

jenkins test centos-non_container-update

jenkins test centos-non_container-update


Cannot create user using admin panel

In the current NewCantus, users can be added via command line (createsuperuser), but if you try to create them via the admin page, it doesn’t work (the password is stored as plain text, rather than being hashed and then stored, and entering the specified password in the password field on the login page doesn’t grant the user access, since the hash of the password doesn’t match the plaintext stored in the user’s .password attribute).
>>> User = get_user_model()
>>> user = User.objects.filter(username="user1")[0]
>>> admin = User.objects.filter(username="admin")[0]
>>> user
<User: User 1>
>>> admin
<User: admin>
>>> user.password # user1 was created via the admin panel, and their password is stored in plaintext
'user1'
>>> admin.password # admin was created via `python manage.py createsuperuser`, and only a hash of their password is stored
'pbkdf2_sha256$320000$7oowaZ0ygK9rXmmcCmeT4P$NoEK1NZCL20LKoLG5OcwrKh9om1VWvGaYluqLwLadO```

This has been fixed somewhere along the way. Closing.


Remove initial silence?

Hello 👋
Thanks a lot for this awesome library! 🎉
I have a question: is it possible to remove, somehow, the initial silence created by the anti-aliasing filters?
Thanks in advance!

The silence is there to make sure the resampled data starts "nicely", without any spikes or such. You could cut off the silence afterwards, just trim it off the start of the first chunk. The issue is to know how many frames to remove. I could add a method to get the initial delay in frames, would that help?
The alternative is to add an option to not include the delay on the first chunk at all. This might be useful, but it would make the already complicated initialisation of the resampler even worse..

Yes, I've tried to trim of the first chunk, but I faced that problem about how much to cut. Exposing the initial delay in samples would definitely help.
Thanks a lot!

See https://github.com/HEnquist/rubato/issues/54

EDIT: BTW, I've found the silence to be proportional to chunk size. I guess that's how it works. If so, maybe we should mention that on the documentation.

That's how it works for the FFT resampler. For the others you have more direct control via the since length.

I'm not sure if I'm currently hitting this too, but i'm currently trying to write some tests against my dj audio engine which uses rubato and was suprised to see a ramping on the samples even if my own ramping was disabled on the first frame. Is this really needed if i take care of ramping up the first chunk myself? Is this ramping happening also when changing the tempo?  Or more general asked, when is this ramping happening and how?

What ramping do you mean? Amplitude? That's only on the first chunk and you can trim it off. Use the new methods in https://github.com/HEnquist/rubato/issues/54 to know how much to trim then.

It's a delay to make space for the pre-ringing of the sinc AA filter. In most cases you can safely trim it off.

Would there be latency added from the FFT? (e.g.: 1/4 * BLOCK_SIZE)

You mean for the FFT resampler? Yes it also has an anti aliasing filter with a similar pre-ringing. The length is half the FFT size. The new methods for getting the delay size works here too. I will publish the next version soon, just busy with other stuff.

The new version 0.13 was just released, should help here.

Thanks a lot @HEnquist  👍


[vcpkg.cmake] fix X_VCPKG_APPLOCAL_DEPS_INSTALL

This also does a script audit of this code

/azp run

/azp run


Add troubleshooting section to Studio docs to delete the ~/.cache/prisma folder

I had continuous issues with Prisma Studio on my machine, it would basically never work because it seemed for some reason it was using an old version of a query engine. The reason apparently was that there were caching issues with the ~/.cache/prisma folder. Once I entirely deleted the folder, everything was working properly.

To be more specific, the folder to delete would be ~/.cache/prisma/studio (on macOS & Linux), and %appdata%\ Prisma Studio on Windows

That Windows path does not exist on my Windows machine, although I used Studio before.
%AppData%/Prisma/Studio does exist though.

Oh whoops, that 's what I meant!


Get expected size of video asset during SSG to create placeholder

Just a thought at this point and haven't really thought about where to do it, but figured since we're more often than not generating our sites, we could get the expected aspect ratio of a video during generation to create a placeholder at the correct size to prevent sites jumping when the asset loads...
Further more, if we're really feeling it, I bet we could grab the first frame of the video for a placeholder image... but that may be more effort than is worth.

Ooooh, caching the aspects to disk during SSG is s cool idea.  I think we could even use Nuxt's fetch() hook for that ... and this package makes it look pretty easy: https://www.npmjs.com/package/get-video-dimensions.  Cool idea!


return requires an argument

In Javascript, return; in a function returns undefined
In eslisp, the built-in return macro requires an argument, so returning without a value requires writing (return undefined)

Thanks for catching that!
I've published 0.7.6 which should fix that to allow zero arguments.

Thanks for fixing!


calico-kube-controllers fix concurrent map writes issue

Description
Add mutex lock to the Set function to prevent a race condition that causes the process to panic with fatal error: concurrent map read and map write

Related issues/PRs
fixes 8705

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

 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.10 out of 11 committers have signed the CLA.:white_check_mark: mgleung:white_check_mark: Behnam-Shobiri:white_check_mark: tomastigera:white_check_mark: fasaxc:white_check_mark: danudey:white_check_mark: hjiawei:white_check_mark: mazdakn:white_check_mark: coutinhop:white_check_mark: tmjd:white_check_mark: caseydavenport:x: zamogYou have signed the CLA already but the status is still pending? Let us recheck it.

/sem-approve

Thanks for the PR.
Based on your issue, I'm not 100% convinced this will fix the race. If the concurrent write is also occurring in the same Set method, then maybe, but, It seems from your logs of the issue that:

reflect.DeepEqual() being called on the cache item is happening concurrently to a write on that item.

The item being mutated is a WorkloadEndpointData, particularly, it seems like a mutation on it's labels map may be the culprit. Therefor I'm guessing the panic arises from a write to this struct's labels while the reflect pkg reads them.

Do we know what component is writing to that labels map while reflect is reading it? And can we confidently say that synchronising calls of the Set method removes that concurrency?
On a side-note, our process would be to commit this fix to master branch, rather than directly to the release branch, to ensure we don't regress in future releases. We can then backport the master patch to 3.26.

@zamog I think we'd be open to adding any necessary locking here, but we need to understand how the concurrent writes happen: adding locks without understanding why is not generally safe.  If you can come up with a UT or description of how the concurrent writes happen then we can make a fix.


Make interaction models pluggable

This PR opens up the interaction model pattern to having alternate RDF::LDP::Resource descendants do the work of instantiating and populating resources.

:+1:

If a release with this will help you, let me know. Else I'll hold off a bit. I have a local app I want to try this out on.


Upload videos

Is there any plan to support video uploads? This would be great so my friends can upload and share videos on Discord.
Thanks

Yep :slightly_smiling_face:
https://github.com/Hubro/clipface/issues/2


Support AR sampling

Ability to draw autoregressive (AR) NP samples using neuralprocesses
https://arxiv.org/abs/2303.14468

AR sampling now supported (https://github.com/tom-andersson/deepsensor/commit/8d2ae2fcb74e0e0a66b03aa5e46f1547695f5eab, https://github.com/tom-andersson/deepsensor/commit/14340f97c5a62bcd2970cfaff570c6842fb4f0b8, https://github.com/tom-andersson/deepsensor/commit/03acaa0d2067031eb26c1b672df2732af1b0a302)


Fail with invalid arg when attempt to use call creds on insecure channel

In creating the test for failing call creds in https://github.com/grpc/grpc/pull/8102, found that call creds on an insecure channel seemed to be silently ignored.
This makes a check for an attempt to do so at the client stub.

the ruby wrapper is probably the wrong place to do this, issue moved to https://github.com/grpc/grpc/issues/8142


Naming output of MSGFplus paired spectra/db better

The old output schema resulted in output named "forward.mzML" since that is the unchangeable name of the paired element that was the spectra dataset.
I changed to use the name attribute of the element containing the dataset pair, which results in proper spectra file names. I use name and not element_identifier since the element itself is a collection and will raise a KeyError if element_identifier is used.
A bit annoying this didnt end up in yesterdays PR.

I don't understand the Travis output, should we retry the test?

Can you close and reopen this should trigger it. Sorry on my mobile.

Test was canceled? Reopening again.

Is anyone but @bgruening able to merge stuff? (Also Björn are you alright?)


Make sure fail on formatting also happens in pull requests

See title and related issue #18

Great, thanks for the quick merge!


fix(nav): add new enterprise link

fix newrelic script not being loaded
fix enterprise link not going to enterprise page
remove gelato and galileo links from nav

Branch name is a misnomer.

@coopr Good to merge?

LGTM, but really, I'm the wrong guy to review PRs.
Yes, I'm totally happy with this being deployed to prod - if you are happy too.

The review is because this changes the header on the website to remove the links to Galileo and Gelato, hence why :)

(Which is in alignment with our objectives, but just making sure it isn't too soon to deploy).

Oh, yeah, that, thanks @thibaultcha - sure, :shipit:


blinking screen while close the popup screen.

Environment

Platforms

Versions

Android:
iOS:
react-native-modal:
react-native:
react:

Description

Reproducible Demo

麻烦问下，我也出现了这个问题

麻烦问下，我也出现了这个问题

Check this, it fixed for me https://github.com/react-native-modal/react-native-modal/issues/268#issuecomment-493464419


Realm crashes when Phone has low storage

Goals
To be able to run the app or view even when phone has a low storage
Expected Results
When going to the view, don't crash
Actual Results
It crashes with this error:
Thread 10: Fatal error: 'try!' expression unexpectedly raised an error: Error Domain=io.realm Code=1 "fcntl() inside prealloc() failed allocating 1184 bytes, new_size=1184, cur_size=0, allocated_size=0, event: No space left on device" UserInfo={NSLocalizedDescription=fcntl() inside prealloc() failed allocating 1184 bytes, new_size=1184, cur_size=0, allocated_size=0, event: No space left on device, Error Code=1}
Steps to Reproduce

Fill phone with data (may be useless) to a point where phone prompts low storage. prefereably lower than 10 mb
Open app
Go to a view using realm

Code Sample
self.token = realm.objects(Data.self).sorted(byKeyPath: "date", ascending: false).observe({ change in ...
Version of Realm and Tooling
Realm framework version: 3.14.1
Xcode version: 10.2
iOS/OSX version: 10.14.4

If you don't want to crash when an i/o error occurs then handle the error with do/catch rather than using try!.


Update monitor.py

fix cpu percentage sign

Hi kjames2001,
The figure shown is the Load Average on the CPU, it is not the CPU usage as a percentage. You'll notice that this number never goes much above 4 (meaning all four cores are fully utilised).

Sorry for not reading the command properly before commiting. Thank you for the amazing scripts!


'NoneType' object has no attribute 'api_versions'

Hi @emmett9001 ,
I've got pykafka 2.7.0 installed and what I see is that when I try to connect to a Kafka cluster, which is disconnected at that time, the following error occurs:
'NoneType' object has no attribute 'api_versions'
Normally I should get a SocketDisconnectedError, but since a change in cluster.py which is incorporated in 2.7.0 the above error appears. The fetch_api_versions function is new and is being called once from the Cluster class constructor. The fetch_api_versions function is getting the api version info from the response object, but when a connection isn't possible, the following line in fetch_api_versions results in above error:
if response.api_versions:
And that's because reponse is None.
Would be nice if this could be fixed.
I'm referring to commit 80a74ca on URL:
https://github.com/Parsely/pykafka/commit/80a74cacc8f6d48d749a7a08432208f4cdc9c00d#diff-27e6827c14ac03e8b996238c6e8813be
Kind regards,
Ruben

Thanks for pointing this out, @rubenbouman. See the fix here https://github.com/Parsely/pykafka/pull/794


Add Support for Selecting the channels a user follows instead of it being all

#2 has been added so streamers can be set to ignore


Add my Tech School

@papitorcks Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@papitorcks You might miss my previous comment. Please react.

@papitorcks I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


HikariCP returns also SQLTransientConnectionException when connection can be retried

HikariCP after version 2.4.1 (issue https://github.com/brettwooldridge/HikariCP/issues/349) can also return SQLTransientConnectionException. Seems it could also be handled same way as SQLTimeoutException.

Thanks for the tip. I'm going to add support for this one as well, most probably next week because I'm currently stuffed with tasks. If you feel like submitting a Pull Request, feel free. It's an easy fix.

Thanks for the Pull Request. I'm going to release a new version with this fix.


How to return id after insert

i have a query like below:
INSERT INTO table (field1,field2)  VALUES('val1','val2')  RETURNING id
how can i create this query by pypika

PostgreSQLQuery.into(self.table_abc).insert('val1', 'val2').returning('id')
more examples here: https://github.com/kayak/pypika/blob/master/pypika/tests/test_inserts.py#L468

@gleb-kov
thanks for your support. That works well 💯


Non adm users who don't have access to commands api are unable to use the SDK

Current behaviour on successful login NewSession will go and run the system info command. But it does't not take into account that some people might not have that access and would only like to use the SDK to read certain objects. Hence I suggest adding a flag to control the current behaviour.

@scottdware whats your thoughts on this ?

Hi @amitsaini-au. Apologies for the delay. Let me review this, so far it looks like a great idea. Thank you!


Stat override message

Part of https://github.com/lambdaclass/champions_of_mirra/issues/280
Motivation
We should have a specific message for the case in which we want to override a stat's value, despite its current value.
In the immediate future this will be used to set the energy to 0 after casting the ultimate skill, in the near future it will be used to set other stats such as the health.
Summary of changes

Create a new StatOverride message.
Send this message when casting a unit's ultimate skill, setting its energy to 0.

How to test it?
You can use the SocketTester (remember to move it to the gateway directory) to simulate a battle and check that this new message is sent.
If you feel more comfortable using the client, start the client using this branch, go to BattleManager.cs and add a Debug.Log() in the PlayOutSteps() method, in the StatOverride case of the switch statement. Run a battle and there you should see how the frontend receives this new message when a unit is about to use the ulti.
Checklist

[X] Tested the changes locally.
[X] Reviewed the changes on GitHub, line by line.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Let's implement statoverrides for heal caps

To test with SocketTester:
IEx.configure(inspect: [limit: :infinity])

{:ok, user} = Champions.Users.register("Lotu")

campaign_progress = user.super_campaign_progresses |> hd()

{:ok, pid} = Gateway.SocketTester.start_link

Gateway.SocketTester.fight_level(pid, user.id, campaign_progress.level_id)

Nice! I'll add it to the description


Can this library be used in Kotlin?

I am using kotlin to make a video player with a scrollable recyclerView which can show more than one video. However, I just don't know how to set up the adapter in Kotlin. I noticed that @PierfrancescoSoffritti  had turned the core into Kotlin, but not the example codes. I am pretty new for android, so I don't know how to turn the java code into kotlin well( I tried the "convert java into kotlin ", but got codes with a lot of error...)
So my question is, can this library be used in Kotlin? Or it's just for Java?
And if it can be used in Kotlin, what should I do in the adapter class...
p.s. Sorry my English is poor...

So my question is, can this library be used in Kotlin?

yes it can be used with Kotlin.

yes it can be used with Kotlin. Then update the example codes too


"Error adding card to checkout retrying..." in locales that are not USA

When i try to test Nvidia Clerk with 2060 and a locale different from USA i get this message: "Error adding card to checkout retrying..." and the notification is not sent.

Notifications working here,(twilio,& telegram) however no links to cart, or add to cart is not

Currently just an NVIDIA problem. Will leave open until resolved.

Closing in favor of #158