Request for adding Apeejay School

Url for school is https://apeejay.edu/
The various branches/districts of the school are https://apeejay.edu/all-schools/

@guneetgarg123 Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


How do  i fix it? it happens when i queue more than 10 songs

NVM i fixed trough looking at css classes

Thanks, was hoping you'd make a pull request...fixed now.


Added correct processing of links to anchors in another markdown file.

Example: my-file.md#first-headline needs to become my-file#first-headline
Also adding tests to verify correct behavior.

@thake thank you for your contribution!
Also thank you for adding some tests. I really like them! I just moved it to a separate folder from the main code and added CI step (which from my understanding right now still uses the old test command and that's why fails, but I think it's not a big deal for now).


Custom filter for contrast

Hey, could you explain to me does it is possible with this API to make a filter for contrast?
I found in filter variable name "contrast", but I don't understand which params I need to use for this filter.
https://github.com/nhn/tui.image-editor/blob/338d7b18b6b6a52962f042e31e46638d88683c4a/apps/image-editor/src/js/ui/filter.js

@bershek
Can you write it in more detail?

I mean a simple filter for contrast, does it possible to implement it with this library?

@bershek
The image-editor cannot implement filters. I'm sorry.


Regression in 2.1: Bug with opacity of a Polygon2D

Operating system or device - Godot version:
Windows 8.1 x64, Godot 2.1 RC2
Issue description (what happened, and what was expected):
I have a project, created in Godot 2.0.3 Stable. There are Polygon2D nodes, their opacity is not working in Godot 2.1 RC2. They are always 100% visible (making fog looks wierd). Tried to re-import textures, but it is not fixing problem. If I create new project and new scene in 2.1 RC2, opacity of the polygon is not working as well. Seems like something is changed in 2.1 making any polygon opcatiy working wrong.
Steps to reproduce:
In the description.
Link to minimal example project (optional but very welcome):

Fog must be barely visible and stars must blink, but opacity aint working, so the things looks wierd.

Can you create a minimal project in <IP_ADDRESS> that would trigger the bug in 2.1 RC2?

Sorry for the delay, I'll try to do it asap.

Okay, @akien-mga I was wrong about the core of the problem. I have a polygon to create fog shape, Polygon2D, it is filled with a texture. And I can't set the opacity of any polygon to anything but 100%.

Then it's likely a duplicate of #5151.

@akien-mga But in my case it working in 2.0.4 and 2.0.3, but not working in 2.1.

Yeah, I think #5151 is also about a regression, but it doesn't make it clear (and actually #5544 mentions a bug in 2.0.3). Let's reopen this one to investigate this issue as a regression, instead of as a "known bug that will be fixed in 3.0" as @reduz assumed in #5151.

If you could provide a test project that works in <IP_ADDRESS> and not in 2.1, that would be helpful to bisect the issue.

TestBug6060.zip
@akien-mga Look at this one, the minimal set of one scene and one PNG file. Open with 2.0.4 - opacity works. Opening with 2.1 - it aint working at all.

I'm bisecting....

Ok, here is the result (not unexpected in any way):
0d20ceeb61be915fea81c22c174efb9253d96974 is the first bad commit
commit 0d20ceeb61be915fea81c22c174efb9253d96974
Author: Ovnuniarchos<EMAIL_ADDRESS>Date:   Mon May 23 03:40:45 2016 +0200

    Polygon2D now exposes vertex colors.

:040000<PHONE_NUMBER>bde66dfbeba63cda2972425e91f5f24d5ff8 a22435a46c2723316dcdd5220fae4832fdaebff8 M      scene

CC @Ovnuniarchos.

The code for opacity could be implemented in the scene object, but that could be too hacky a way to solve the issue. The alternative is to drop support for vertex colors.

Is this still valid in 2.1.3 or the current 2.1 branch?

Yes, the only way to change opacity on a Polygon2D is to change vertex color opacity or use a fragment shader (I guess vertex may work too).

Fixed by #9947.


ERROR sharedbackend

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (c49e5e1 2023-04-08T00:16:13.767906000Z)
What command(s) is the bug in?
forge test --match-test
Operating System
macOS (Apple Silicon)
Describe the bug
2023-04-08T17:06:28.235952Z ERROR sharedbackend: Failed to send/recv `basic` err=GetAccount(0x55d398326f99059ff775485246999027b3197955,
(code: -32002, message: the resource eth_getBalance is not available., data: None)) address=0x55d398326f99059ff775485246999027b3197955
2023-04-08T17:06:28.280628Z ERROR sharedbackend: Failed to send/recv `basic` err=GetAccount(0x9f8ccdafcc39f3c7d6ebf637c9151673cbc36b88,
(code: -32002, message: the resource eth_getTransactionCount is not available., data: None)) address=0x9f8ccdafcc39f3c7d6ebf637c9151673cbc36b88
The application panicked (crashed).
Message:  called `Option::unwrap()` on a `None` value
Location: /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/revm-2.3.1/src/evm_impl.rs:285

Was going through the learn-evm-attacks repo and when I run the forge test --match-contract Exploit_MBCToken -vvv command, this error popped up.
In the error it said that This is a bug. Consider reporting it at https://github.com/foundry-rs/foundry so that's what I'm doing.

I need to find a way to get rid of the panic but this happens because the provider you're using failed to respond to eth_getTransactionCount

(code: -32002, message: the resource eth_getTransactionCount is not available., data: None)) address=0x9f8ccdafcc39f3c7d6ebf637c9151673cbc36b88

ok I see, sorry for that then.

I'll renaming this to improve the error message.
Curious, what provider did you use? I also saw this a few days ago on another issue

I realized, this should become a non issue with #4472

mmmm I was running forge test --match-contract Exploit_MBCToken -vvv and I don't think I set up anything else.

Will close this—we migrated to revm 3.0 and we actually have an open PR which will further improve this mesage.


Add customer 1125 to upgrade template

From New MI request

Superced by https://github.com/sourcegraph/handbook/pull/4552/files


Add a delete command

Would be handy if you could delete entries as well

I agree, a delete command would useful.
I've added a few comments to your pull request; If you'd like to address those, I'd be happy to review again and get your changes merged. Thanks for the pull request!


Unnecessary Vicemes  Bug

When I create Vicemes to the model with the correct shapekey and apply, I can see when I talk or use the viceme shape key, I can see some unnecessary vertices applied on the vicemes that I don't select. For example, I can see my mouth move with an anus spread and butt expands. Please fix this bug

This is due to vertices being modified on a vertex group or a shape key before you apply visemes. You can avoid this entirely by first separating your model by shapes before making any edits, and not doing any edits to the face.
This is probably a change you made to the mesh while a shape key or vertex group other than basis was selected.
There is a suggestion I have that could fix this, but I'm not entirely sure how feasible it will be.


KryptonIDE not being set as default editor.

I have Cesium 3.0.3 and KryptonIDE beta 16. After I updated from Cesium 3.0.2, KryptonIDE doesn't launch when I press [zoom] to edit a program; it just brings up the standard TI editor.
I tried launching KryptonIDE  to see if it needed to install itself, but it had no effect.

See my cemetech post. It has the solution there.

Press [trace] to enter the program editor.
Or re-read the freaking ReadMe unlike me. :-P
(Thanks LAX18)

If you delete the Cesium settings appvar it should automatically update.


PocketMine API Changes.

Some users have been experiencing issues updating old plugins to the lastest API in our service as mentioned in #16 and #15, this is due to plugin breaking changes in ALPHA7. We will work our best to fix this. Any more issue about this will be labeled and closed and be mentioned here.

Fixed.


[MOB-1377] video test

I don't think we can merge this fix into master. This video file will confuse users of our SDK.

Hmm. How about putting it under tests but still linking to it through GitHub like before?

Confirmed that's what we'll do for now


Jar files

What feature or improvement do you think would benefit Files?
Being able to open jar files just like zip files, as they are just zip files in disguise
Requirements

This proposal will accomplish X
This proposal will accomplish Y
This proposal will accomplish Z

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2134
Comments
No response

Thanks for the feedback, this has already been made and will be included in the next update.


govendor update leveldb

updates leveldb library to master. there were a few minor bug fixes and performance improvements. no breaking changes from looking at the new commits. there was  small change i made i wanted to get in for logsends/stats https://github.com/syndtr/goleveldb/pull/262

cc @maxtaco since this involves a deep update

For all those reviewing, https://github.com/syndtr/goleveldb/compare/8c81ea47d4c41a385645e133e15510fc6a2a74b4...4217c9f31f5816db02addc94e56061da77f288d8


Validate presence of a class in the class argument

When rendering custom class / serializer mappings:
render_jsonapi records, class: {
  :'User' => ::FrontEnd::SerializableUser,
  :'Comment' => ::FrontEnd::OnDemand::SerializableComment
}

If the left-hand key isn't correct, you will get a cryptic error
undefined method `new' for nil:NilClass

I think a solution would be to update _build in renderer to something like:
def _build(object, exposures, klass)
  raise "#{ object.class.name.to_sym } not found in the class option passed to jsonapi_render" if klass[object.class.name.to_sym].blank?

  klass[object.class.name.to_sym].new(exposures.merge(object: object))
end

This way, if you've forgotten to map something there will be a helpful error message.

I agree that the error message is not very helpful. However, checking for blank? is not enough, as it would still allow non-existing classes. Probably something along the lines of rais ... unless klass[...].is_a? JSONAPI::Serializable::Resource. @dawidof What do you think?

I think its a good idea. I' also spent some time in past to figure out from where this error is coming.

Looks like this was resolved by https://github.com/jsonapi-rb/jsonapi-serializable/commit/914ccdecef9ec8b6f7ebca96bada5825fe99408f


fix: ignore exceptions when comparing imported to existing minutes

Fixes #3749

Oops - will replace this with a new PR in a bit.


Modified challenge list

Changes proposed in this pull request:

Challenge list html file modified to make its UI similar to the current EvalAI application.
Modified logic to display hosted challenges UI.
Challenege list styles are changed by using the styles ported from current EvalAI project.
Publiclist component and challenge card component are also modified correspondingly.
Some utility component like side-bar and dash-footer component have been added.

Link to live demo: http://pr-143-evalai.surge.sh

Codecov Report

Merging #143 into master will increase coverage by 1.07%.
The diff coverage is 71.94%.

@@            Coverage Diff             @@
##           master     #143      +/-   ##
==========================================
+ Coverage   53.35%   54.42%   +1.07%
==========================================
  Files          55       58       +3
  Lines        2356     2473     +117
  Branches      253      259       +6
==========================================
+ Hits         1257     1346      +89
- Misses       1026     1051      +25
- Partials       73       76       +3

Impacted Files
Coverage Δ

...onents/home/twitter-feed/twitter-feed.component.ts
100% <ø> (ø)
:arrow_up:

src/app/components/nav/footer/footer.component.ts
62.5% <100%> (+1.2%)
:arrow_up:

...omponents/nav/dash-footer/dash-footer.component.ts
100% <100%> (ø)

.../components/utility/side-bar/side-bar.component.ts
100% <100%> (ø)

...bliclists/challengelist/challengelist.component.ts
41.41% <54.54%> (+0.96%)
:arrow_up:

src/app/services/auth.service.ts
36.75% <57.14%> (+13.46%)
:arrow_up:

...pp/components/publiclists/publiclists.component.ts
84.21% <76.92%> (+22.3%)
:arrow_up:

...nents/nav/simple-header/simple-header.component.ts
80% <80%> (ø)

... and 1 more

Impacted Files
Coverage Δ

...onents/home/twitter-feed/twitter-feed.component.ts
100% <ø> (ø)
:arrow_up:

src/app/components/nav/footer/footer.component.ts
62.5% <100%> (+1.2%)
:arrow_up:

...omponents/nav/dash-footer/dash-footer.component.ts
100% <100%> (ø)

.../components/utility/side-bar/side-bar.component.ts
100% <100%> (ø)

...bliclists/challengelist/challengelist.component.ts
41.41% <54.54%> (+0.96%)
:arrow_up:

src/app/services/auth.service.ts
36.75% <57.14%> (+13.46%)
:arrow_up:

...pp/components/publiclists/publiclists.component.ts
84.21% <76.92%> (+22.3%)
:arrow_up:

...nents/nav/simple-header/simple-header.component.ts
80% <80%> (ø)

... and 1 more

Continue to review full report at Codecov.

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

@lunayach @Shekharrajak please review the PR.

Hi @RishabhJain2018, did we get the Surge to work properly? At this stage, both @sanketbansal and @Sanji515 are pushing PR's that are having UI changes (reproducing from EvalAI). They do attach some screenshots with a PR, but sometimes that may not be enough to see the working of the code entirely.

It is bad that you messed all the commits in all the PRs. But still you can improve your PRs.

One PR should be opened for one particular feature or fix.
If there are lot of things to do in one task, divide it into pieces and open PR
Making small PR will get more chances of early review, comments and merge.

Use git features to separate all irrelevant commits in all your PRs.

I want to help you to improve your collaboration and Open Source Software development workflow skills but it looks like you don't want to follow the comments https://github.com/Cloud-CV/EvalAI-ngx/pull/143#issuecomment-503592390, https://github.com/Cloud-CV/EvalAI-ngx/pull/149#issuecomment-513453402

I want to help you to improve your collaboration and Open Source Software development workflow skills but it looks like you don't want to follow the comments #143 (comment), #149 (comment)

I am working hard to improve these skills. The file changes in this PR will get reduced after merging #141
As it has the dependencies on some changes present in that PR.

Thanks for letting me know.


feat(media): limit max framerate for encoders

This may help with some devices having trouble when the encoder could be configured above 30 fps for 1080p.

Just a question. If my devices supports e.g. 4k 60FPS, would it not be logical to expose an override in the API for max frame rate? Or is this a limitation on Infinity?

Just a question. If my devices supports e.g. 4k 60FPS, would it not be logical to expose an override in the API for max frame rate? Or is this a limitation on Infinity?

Infinity itself is limited to 1080p30 (see docs), but you could technically go higher than that in a direct media call provided your device supports it.
We can and definitely should make it configurable, maybe with a soft limit of 30fps for transcoded calls.
This particular PR mostly addresses an issue with one specific device we have where it's unable to handle 1080p60, but the framework attempts to configure the encoder to just that, which in the end results in failures when more than one video stream is active.


Slowness in Rendering with Event Markers

Expected behavior and actual behavior:
Expected: Chart can render with ~1000 events markers in 2 seconds or less.
Actual: Chart rendering with ~1000 events markers incurs additional 40-45 seconds in Internet Explorer 11, additional 5-8 seconds in Chrome.
Steps to reproduce the problem:
See examples feeding in ~1000 event markers at these codepens:
https://codepen.io/jkishkunasGE/pen/BRYyeN?editors=1001
https://codepen.io/jkishkunasGE/pen/zwRGae?editors=1001
Overhead time is more apparent/measurable when running locally with chart time series and event data set directly in the properties:

Setup and run the px-vis-timeseries codebase locally.
Modify the demo/index.html code to have the event config and event data used in the above codepens.
Measure the time taken with and without the event data/config to determine the overhead time for the event markers.
Vary the event markers between font awesome, custom images, and unicode, and observe that timings do not improve.
Vary the rendering to canvas and progressive rendering settings from true/false and observe that timings do not improve.

Environment (component version number, Browser (including version), operating system, hardware, etc):
Component: px-vis-timeseries v1.1.1, px-vis v1.1.9
Google Chrome v58 as well as Internet Explorer v11 (did not test other browsers)
OS: Windows 7, 64-bit
CPU: Intel Xeon CPU E5-1650v3 @3.5Ghz, 3.5Ghz
RAM: 32 GB
Screenshots (optional, but very helpful):
Performance issue...see the codepens above.
Code examples
See examples feeding in ~1000 event markers at these codepens:
https://codepen.io/jkishkunasGE/pen/BRYyeN?editors=1001
https://codepen.io/jkishkunasGE/pen/zwRGae?editors=1001

I've been able to reproduce. I don't think we expected that many events being created, in this example we are paying the price of stamping 1000 px-vis-events (which themselves stamp one tooltip each, ouch).
One solution to investigate would be to have 1 px-vis-event handle all the events, having only 1 tooltip moving around and drawing everything itself.
How critical is this improvement to you? The change would be non-trivial and might need to be prioritized against other features.
We could temporarily limit the problem by adding a property to not create a tooltip, but that might defeat the purpose of the event... Also note for us to  remove the debounce in drawing which hurt perf

This is a major issue for us, so your help would be greatly appreciated. My TPM is in touch via email on the priority. Internet Explorer is particularly problematic, taking minutes of time to render a few charts on screen. But even Chrome incurs some impact, as noted, and it is causing us to miss our target loading time for charts.
We would be good with having an option to eliminate tooltips from the event markers, we use different markers to denote different types of events, so the marker itself is sufficient for our use case. Would it be quick to try eliminating the tooltips and see what difference it makes, to gauge the impact?
I'm not certain what you meant on the "debounce" option, could you explain?
I'm open to other ideas, too.
Thanks!

I've added an enableTooltip option in eventConfig and made a minor tweak (the debounce thing, , this was just meant as a note for us when looking at this). If you set that to false the events should draw a bit more than twice as fast (which is still pretty slow for a lot of them...).
Anything more will require a rework of the events and will take some time, I understand that our TPMs are in contact regarding that.
available in px-vis 1.1.10

ps: disabling the tooltip will solve the massive memory hog that the events are currently causing in the charts.

Thanks for the quick turnaround. I see results like you mention in Chrome (twice as fast rendering), but not in Internet Explorer 11. Did you see any noticeable improvement in IE?

Question - I no longer see any event marker tooltip in px-vis 1.1.10, with or without the added "enableTooltip" setting, and even with it set to true. I'll leave it set to false - but can you confirm whether event marker tooltips still show if it is set to true?

I noticed the same thing yesterday, I have a fix ready, I'm going to bundle it with another change to events to allow formatting the timezone/timestamp in the tooltip of events. Release sometime today.
Regarding the perfs I ran a few tests yesterday on the new version with the upcoming fix and I did see a similar gain on IE:
drawing a timeseries with 4k points, 1k markers (in ms):
              No markers         Markers with tooltip       Markers without tooltip

Chrome       289                             2245                                   1130
IE                 993                            32170                                14665
I don't expect the fix to have changed the performance, but you can pick up the new version once I've released it and try again

The fix is available in px-vis 1.1.11

Trying px-vis 1.1.11 and the tooltips are now always there. Can you confirm that the fix works in px-vis 1.1.11? If so, can you give a code snippet or codepen example so that I can ensure I am setting the setting correctly?

How are you setting the eventConfig? The enableTooltip part has to be within a specific event type, e.g:

chart.eventConfig = {
          "fa":{
            "color": "blue",
            "icon": "fa-camera",
            "type": "fa",
            "offset":[0,0],
            "lineColor": "red",
            "lineWeight": this._chartOptions.eventsNoLine ? 0 : 1,
            'enableTooltip': this._chartOptions.eventsNoTooltip ? false : true
          },
          "unicode":{
            "color": "green",
            "icon": "\uf015",
            "type": "unicode",
            "offset":[1,0],
            "lineWeight": this._chartOptions.eventsNoLine ? 0 : 1,
            'enableTooltip': this._chartOptions.eventsNoTooltip ? false : true
          },
          "default":{
            "lineWeight": this._chartOptions.eventsNoLine ? 0 : 1,
            'enableTooltip': this._chartOptions.eventsNoTooltip ? false : true
          }
        };

I'm currently rebuilding demos but it's working locally

Thanks! I was adding directly under eventConfig. It works when I add it per-event-type.

v2.1.3 now includes a new markerConfig and markerData feature, allowing to draw a lot of markers above and below the chart. This feature uses canvas and can handle thousands of markers.


Python REPL (serial input) support

This PR enables use of the Python REPL with a CircuitPython device just like in the Mu Editor.

Hey @jos-b, any interest in this PR?

Hey @dunkmann00! Huge apologies, I completely missed this PR.
I'll give it a test now, thank you!

This works great! Thank you for the contribution!

I'll push this out now in v0.7.0, I think it warrants a release alone!

Published

Great! I appreciate it!


Replaced Mapbox with osmdroid (Nearby activity)

Fixes #5372
What changes did you make and why?
✔ Replaced MapBox with OSMDroid
✔ Tested most of the pre-existing features
Tests performed (required)
Tested betaDebug on Samsung S21 FE with API level 33.
Screenshots (for UI changes only)
https://github.com/commons-app/apps-android-commons/assets/114223204/501e79a8-bf8e-4a3a-8499-f0490e46af3c
🔧 Need to work on tests

Feel free to add @Ignore to the tests that are failing, they can be solved later by anyone as another GitHub issue.
And pull from main to resolve the small conflict in app/build.gradle.
Then this pull request can be merged into the main branch. :-)

Hi @nicolas-raoul , I have made the changes as advised by you and I have also fixed the marker click issue.
https://github.com/commons-app/apps-android-commons/assets/114223204/f69676a2-d648-4961-bf28-5cf387ac6c42

I created https://github.com/commons-app/apps-android-commons/issues/5408


Cannot import name 'DRESModel' from 'mteb' .

When I run the eval code from repo of e5-mistral, the error occur:
Traceback (most recent call last):
File "/share/minghao/Projects/TestCIL4Embedding/unilm/e5/mteb_beir_eval.py", line 12, in
from mteb import MTEB, AbsTaskRetrieval, DRESModel
ImportError: cannot import name 'DRESModel' from 'mteb' (/share/kunluo/Envs/cms/lib/python3.9/site-packages/mteb/init.py)

This issue seems to be related the repo of e5-mistral using an outdated version of MTEB. Should be solved either by downgrading the MTEB version (to ~1.2.X is my guess) or updating their scripts to match the most recent version of MTEB. I believe you can import DRESModel from:
from mteb.evaluation.evaluators.RetrievalEvaluator import DRESModel

Feel free to reopen if the issue persists after changing the version.


AID vs HEALING

This issue is around the fact that (known issue) healing shouldn't be allowed to raise characteristics above starting value (i.e. shouldn't adjust max).

At the same time we should probably consider implementing a similar limitation for aid as is seemingly present for healing.


module federation with angular 13

Hi
Do you have an example of a working demo of module federation with angular 13?  I would really appreciate that.

Please find it here:
https://github.com/manfredsteyer/mf-angular-showcase


Rname identifiers that match rust reserved keyword list

Append '' to the identifiers. e.g. 'type' is renamed to 'type'.

That would make sense, but as the same file could be used as source for different languages with varying constraints, I figured it would be better to circumvent the issue in a nice way

That would make sense, but as the same file could be used as source for different languages with varying constraints, I figured it would be better to circumvent the issue in a nice way

Right, but I don't feel it circumvents the problem: as a Rust user, I would dislike the type_ field since it looks odd. So I would petition the person who maintains the PDL file to change the field name. I would do the same for r#type, so both solutions are equally annoying from that perspective.
Also: will this not have a bad side effect of marking the field as unused?


toJSON inconsistency between find() and create() for associations

Am using Waterline ORM with SailsJS. For the purposes of this explanation, my domain model contains two models - Admedia and AdmediaField. An Admedia can have many admediaFields. An admediaField can belong to only one Admedia. So one to many.
Admedia.js
fields: {
collection: 'admediafield',
via: 'admedia'
},
AdmediaField.js
admedia: {
required: true,
notNull: true,
columnName: 'admedia_id',
model: 'admedia'
},
Now:
when I create a new AdmediaField, toJSON returns with just the id of the Admedia (to which it is associated)
when I find a new AdmediaField, toJSOn returns with a populated Admedia object (to which it is associated).
It is really helpful in SailsJS, for REST responses etc to be able to override the toJSON() method and customise it according to requirements.
Because of the differences above, all the model objects require hacks like below to cater for either a Create / Get op type:
toJSON: function () {
var obj = this.toObject();
obj.admedia = obj.admedia.id ? obj.admedia.id : obj.admedia;
delete obj.mappings;
delete obj.createdAt;
delete obj.updatedAt;
return obj;
}
This is a really simple example, it gets more pronounced in cases where there are greater number of complex associations.
Issue: Can we make toJSON consistent so that regardless of Action it consistently returns the same JSON output?
Benefit: That way, conditional logic can be removed / haviing to explicitly override toJSON all the time can be avoided.

Indeed there is an issue for associations, and its in the toObject methods of waterline. You might want to have a look at some workarounds towards the end of the thread here: https://github.com/balderdashy/waterline/issues/317#issuecomment-87274671

Hi @arcseldon, regarding:

Be nice if Create() could create and populate. Perhaps I am missing knowledge of such a method.. ?!

There is already another request for that in #354.
As @anasyb mentioned, the issue with .toJSON() seems to be the same in #317.
Please continue this discussion there, closing this as duplicate. Reopen if you feel otherwise.


feat: add an anyhow-like Result type for easier error handling

The Result<T> type provided by a library like anyhow is extremely useful for converting errors of different types into a single type using the ? operator:
async fn fetch_cats(count: CatCount) -> Result<Vec<String>> {
    if count > 0 {
        // make the request
        let res = reqwasm::http::Request::get(&format!(
            "https://api.thecatapi.com/v1/images/search?limit={count}",
        ))
        .send()
        .await?
        // convert it to JSON
        .json::<Vec<Cat>>()
        .await?
        // extract the URL field for each cat
        .into_iter()
        .take(count)
        .map(|cat| cat.url)
        .collect::<Vec<_>>();
        Ok(res)
    } else {
        Err(CatError::NonZeroCats.into())
    }
}

However, anyhow::Result can't be directly rendered into the view of a Leptos component, because anyhow::Error doesn't implement std::error::Error. (In fact, it can't, because it impl<T> From<T> for anyhow::Error where T: std::errror::Error // etc. would mean that an Error implementation for Error conflicts with the default From<T> for T...)
I initially tried using anyhow::Result internally, but this caused issues because anyhow::Error is not Clone, causing lifetime troubles where trying to simply render something like a Resource<(), Result<T>> into the DOM. (Because this forces Resource::with rather than Resource::read, which means a reference, which means a lifetime that's not 'static...)
However, we can get around this by doing something very similar to what anyhow does: providing a leptos::error::Error type that simply wraps an Arc<dyn Error + Send + Sync>. Now, leptos_dom can render anything that it could render before (any Result<T, E> where E: Error + Send + Sync), because those are all Into<Error>), but it can also do the "convert any kind of error into a single type" trick which makes anyhow::Result so convenient.

This can be extended with some additional implications for server function error handling.
Before PR

Rendering Result<T, E> requires E: Error
Therefore ServerFnError must implement Error
Therefore can't implement From<E: Error> for ServerFnError

After PR

Render Result<T, E> requires E: Into<leptos::Error>
Therefore ServerFnError doesn't need to implement Error, only Into<leptos::Error>
Therefore we can implement From<E: Error> for ServerFnError if we remove the Error implementation from ServerFnError itself
We can then implement Into<leptos::Error> for ServerFnError with a newtype that actually implements Error
This will allow use of ? inside server functions, while also allowing server function results to be rendered directly in the view

To do: adapt additional server function examples to use the try operator.

This looks really nice and will simplify user code a lot :+1:
Another feature that would improve things would be a custom serialization/deserialization implementation for the Error that's returned from server functions. If you iterate over the causees and collect the descriptions, then you could actually reproduce the whole error stack on the client side.
Of course, that doesn't give all the full original Display an Debug behavior, but at least you could see the full error stack on the client side.

Another feature that would improve things would be a custom serialization/deserialization implementation for the Error that's returned from server functions. If you iterate over the causees and collect the descriptions, then you could actually reproduce the whole error stack on the client side.
Of course, that doesn't give all the full original Display an Debug behavior, but at least you could see the full error stack on the client side.

So, this is an interesting idea and would certainly be good for developer experience when debugging, but I worry that it might raise some significant security concerns: remember that all of this information would be serialized to JSON and sent from the server to the browser, meaning a malicious actor could have access to a significant amount of information about how your app is structured.


Update environment.yml

The original environment.yml needs some additional arguments

Added "--extra-index-url https://download.pytorch.org/whl/cu113" for torch
Added "--find-links https://data.pyg.org/whl/torch-1.11.0+cu113.html" for pyg
Added "@ git+https://github.com/jvkersch/pyconcorde" for pyconcorde
Change numpy version to "numpy>=1.21.0" for pyconcorde

I didn't test this, but looks good to me 👍


ethminer+opencl+nvidia = 100% CPU on two threads

I just started playing with the opencl ethminer with a gtx1070. I'm getting about 25Mh/s but two CPU threads are running at 100% doing nothing. One thread is spinning calling clock_gettime() [I think from ethash_cl_miner::search()] and the other thread seems like it doing something with nanosleep(). I didn't build with enough debug symbols to figure out more.
It seems this has been an ongoing problem dating back to 2015 (according to google).
I would have tried the cuda miner, but it was failing to build: (cuda SDK 8.0 on Fedora 25)
[ 42%] Building NVCC (Device) object libethash-cuda/CMakeFiles/ethash-cuda.dir/ethash-cuda_generated_ethash_cuda_miner_kernel.cu.o
/usr/local/cuda/include/math_functions.h(8897): error: cannot overload functions distinguished by return type alone
/usr/local/cuda/include/math_functions.h(8901): error: cannot overload functions distinguished by return type alone

Before I put any effort into it, is anyone else seeing this problem?

I see one core being fully occupied with ethminer in opencl mode. In cuda mode CPU is free.
I built cuda-miner from master branch with cuda-7.5, 8.0.47, 8.0.61 on nixos-17.03. Saw no such errors.

I am aware of the issue with OpenCL on Nvidia, but since you can have CUDA, you can avoid it. Getting things built on various *nix flavors is not my specialty.

I worked around the compile time problem, it is related to having a version of gcc that is not support by cuda.
Searching for those two line numbers pointed to a workaround, but I just commented them out long enough to get the code to compile.
The nvidia+opencl problem was only actually consuming 100% cpu on one thread (htop was showing thread cpu and a composite CPU usage for the main process).
Except for the 100% cpu issue, there doesn't seem to be much of a difference in performance between the opencl and cuda versions.

I am also having this issue. 100% of one core is being used by the Xorg process after starting ethminer. I have tried both -G (opencl) and -U (cuda) modes.
I am using Arch Linux with:
cuda 8.0.61
ethereum-genoil-git - https://github.com/Genoil/cpp-ethereum/commit/2d692d263f3ba74a3612ad3bbec9e7cfac413a1f
linux 4.11.5

I can't get GTX970 to run on CUDA mode, it just says all CUDA devices are busy. After some grinding I think it is related to NVIDIA OpenCL busy wait issues.
https://github.com/pandegroup/openmm/issues/1541


Resizing of TKinter application not working

When trying to resize the tkinter window does the app black out and become unresponsive

Made the window none resizable  as a fix in 8ae56bfe1882d0adf6dc82d0b838767bbb8c1eb9


Node security fixes and circleci cleanup

To follow up on the updates in #418 I cleaned up the build a little bit, updated to the latest released versions of Node (as of yesterday), and store the built css in CircleCI for easy viewing/troubleshooting.

👋 @trezy / @BcRikko, or someone else in nostalgic-css/nes-css who is able to approve; when you get a chance are you able to review this?
It's been open since June and I'd like to get it merged soon.

Thanks so much for these updates, @virtuoushub! Neither @BcRikko or myself are nearly as active on this project as we once were, so I apologize for the delay and appreciate your patience. ❤

No worries, appreciate your time. Thanks for approving the changes. 👾


Improve insert performance for ES storage

Change properties and n to not indexed field
Set refresh to false (_version does not need to be refreshed)

and modified pio version in tests to pass travis tests.

LGTM. By the way, are you working with ES 5.2/5.3? @mars bumped into issues with ES 5.1.

Thank you for merging it.
Our production environment runs on ES 5.1, and my local dev is on ES 5.3.


Passing values from a php variable to the Chart.

Hey guys,
I'm trying to pass some variables contained in a MySql but without success.
For testing, I did the following:
On test.php
<?php
ar1 = $ array ('1', '2', '3', '4', '5', '6', '7');
ar2 = $ array ('20', '21', '22', '23', '24', '25', '26');
?>

On index.php

<canvas id="canvas" height="100%"></canvas>

But now I have another error.
When inserting the result of php on the chart, it separates each number in a row.
That is, if there is 120, It separates 1, 2 and 0. Each in a row!
What will?

Which by the way the son_encode is really not needed since you are to pass in a comma-separated list of values. Not like you are encoding an array or something.
data: [65, 59, 80, 81, 56, 55, 40]


Update and fix WeightNorm Wrapper.

Move the weights v and g to the wrapper itself, since they are
more tie to the concept of weight norm, not the wrapped layer.
Update the build() method to only create weights, and not do
weights manipulation. Setting the init value based on the inputs
are done in call().
Remove the tf.function annotation on call, for debugability.
Renamed few function and add some comments for readability.
Added get_config() for serialization.

https://github.com/tensorflow/addons/issues/203

My only other thought would be, is it documented anywhere that build method shouldn't contain tensor computations? The only reason I was able to pin a specific issue was because I had a nightly break so it limited it to 2 or 3 commits. Otherwise this would have been really challenging to figure out why serialization wasn't properly working (no error was thrown)

Merging so we can get working CI back and I've slated the questions about keras objects for the monthly meeting. This PR is not gated by an answer.

For the original failure, I don't think it is related to from_config/get_config(). Having said that, the get_config/from_config() is a public contract that custom layer should implement.
For the question about build() method contains actual tensor computation, I don't think there is an enforcement for that. https://keras.io/layers/writing-your-own-keras-layers only state that its recommended to create weights in build() and do computation in call(). I think the change from Tom was trying to make that more explicit and expose the issue if user does so.

You are owner of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.


feat(shielded): Store Sapling & Orchard note commitment trees in finalized and non-finalized state

Motivation
In order to implement #3156, we need to be able to query Sapling & Orchard note commitment trees by the height of the corresponding blocks.
Solution
Zebra was already storing the trees by height in the finalized state, but it used to delete the previous tree when inserting a new one. This PR changes that and makes all the previously stored trees available.
This PR also adds the trees to the non-finalized state and stores them in a BTreeMap.
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work
The implementation of #3757.

@mergifyio update

I updated this branch to pick up the fixes to the full sync test.

~~Optional: can you change the Updates column for sapling and orchard note commitments to Never in the state RFC?
https://github.com/ZcashFoundation/zebra/blob/main/book/src/dev/rfcs/0005-state-updates.md#rocksdb-data-structures~~

I did this in another PR, to avoid merge conflicts.

They weren't confusing at all; they were pretty helpful, actually. I shouldn't have pushed unfinished work!
I don't think we have test vectors for note commitment trees, so I just compared the tree at the tip to the one queried by height.
I was trying to figure out why, for example, the function hash https://github.com/ZcashFoundation/zebra/blob/03d123b42826d15b6c60808e4abdef0776c0ba36/zebra-state/src/service/finalized_state/zebra_db/block/tests/snapshot.rs#L209-L210 can be called on state, the type of which is FinalizedState. The function is implemented for ZebraDb, so why can we call it on FinalizedState?
I would expect the function to be called like this:
state.db.hash(foo);

instead of
state.hash(foo);

I found this https://doc.rust-lang.org/stable/reference/expressions/method-call-expr.html, but that seems unrelated to the case above.

That's what I was missing. Thanks, Conrado. The link I posted isn't unrelated, it makes sense together with the Deref trait.
Should I regenerate the state now?

Should I regenerate the state now?

I think so, be aware that it should be easier now with the CI changes, but I don't exactly how :rofl: I think we automatically regenerate the state now so you may just need to change the disk reference

Oh, I don't actually know how we do that. @gustavovalverde would you be able to give me some hints regarding the state regeneration? I used to follow the steps in the Onboarding doc, but they seem defunct now. When I run
docker build --build-arg SHORT_SHA=$SHORT_SHA -f docker/Dockerfile.test -t zebrad-test .

I get unable to prepare context: unable to evaluate symlinks in Dockerfile path: lstat /home/m/docker: no such file or directory. If you can't help, don't worry, I'll try to figure it out.

That's what I was missing. Thanks, Conrado. The link I posted isn't unrelated, it makes sense together with the Deref trait.

Yep, I did the impl Deref to avoid making a huge search-and-replace change across all the tests. But I added db. in the production code, because we want the wrappers to enforce read-only state access from the ReadStateService.

Oh, I don't actually know how we do that. @gustavovalverde would you be able to give me some hints regarding the state regeneration? I used to follow the steps in the Onboarding doc, but they seem defunct now. When I run
docker build --build-arg SHORT_SHA=$SHORT_SHA -f docker/Dockerfile.test -t zebrad-test .

I get unable to prepare context: unable to evaluate symlinks in Dockerfile path: lstat /home/m/docker: no such file or directory. If you can't help, don't worry, I'll try to figure it out.

I think state updates should just all work automatically now, but this is the first time we've tested a database version change.
I'll approve this PR, then we can find out if it works!

I think state updates should just all work automatically now, but this is the first time we've tested a database version change.

Great, then I'll remove the corresponding part from the Onboarding doc.

@mergifyio update

I just pushed some commits to add Rust struct snapshots for the note commitment trees.


Add Bureau of Internet Accessibility

Added the Digital Accessibility blog from the Bureau of Internet Accessibility website - https://www.boia.org/blog

Thanks for the addition. Could you add it in the correct place alphabetically please.

Sure, I'll do that.


Parity with qtcoin on plotting example.

I think we should push for partiy with qtcoin on as many of the openrave examples as we can. Right now,  the plotting example fails due to a segmentation fault:
openravepy._openravepy_0_9.openravepy_ext.openrave_exception: openrave (Assert): [/homes/mklingen/prdev_catkin/src/or_rviz/src/OpenRaveRviz.cpp:1699] -> void or_rviz::OpenRaveRviz::ConvertMesh(const float*, int, const int*, int, std::vector<geometry_msgs::Point_<std::allocator<void> > >*) const, expr: indices

I am having trouble running the examples, even in QtCoin, in my VM. I am not sure if it is an issue with my VM, my OpenRAVE installation, or the examples themselves. Which examples are are you able to run successfully? How exactly are you running them?

openrave.py --example tutorial_plotting --viewer="or_rviz"

I will be fixing this on or_interactivemarker now


⚠ API - Relation Service has degraded performance

In 36da40c, API - Relation Service ($RELATION_SERVICE_API_URL) experienced degraded performance:

HTTP code: 200
Response time: 769 ms

Resolved: API - Relation Service performance has improved in dc2bd5c after 206 days, 18 hours, 5 minutes.


Increased test coverage for models.Images package to 100%

Refactored existing tests to call BeforeCreate methods implicitly for images and imagesets
Added tests to check for presence of orgId for images and imageSets
Added tests for package list without a distribution

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[x] Tests update
[ ] Refactor

Checklist:

[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] I run make pre-commit to check fmt/vet/lint/test-no-fdo

From 95.2% to 100%

need to fix lint failure  level=error msg="Running error: context loading failed: no go files to analyze

Thing is I can't tell what the issue is that lint is reporting. Any ideas?

Codecov Report
Base: 35.23% // Head: 35.35% // Increases project coverage by +0.12% :tada:

Coverage data is based on head (3a22d6d) compared to base (53da4c1).
Patch coverage: 44.44% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1533      +/-   ##
==========================================
+ Coverage   35.23%   35.35%   +0.12%
==========================================
  Files          63       63
  Lines        8739     8739
==========================================
+ Hits         3079     3090      +11
+ Misses       5257     5246      -11
  Partials      403      403

Impacted Files
Coverage Δ

cmd/spec/main.go
0.00% <0.00%> (ø)

pkg/clients/imagebuilder/client.go
34.37% <50.00%> (ø)

pkg/models/images.go
100.00% <100.00%> (+7.21%)
:arrow_up:

pkg/models/commits.go
33.33% <0.00%> (+33.33%)
:arrow_up:

pkg/models/installers.go
33.33% <0.00%> (+33.33%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


remove ' to array create

Lisp has several ways to do the same thing, the idea of rum is to be a simple lingaugem with a (clear) Form of doing operations (that is the philosophy of rum).
I find redundant to have two forms of array declaration

array(10 20 30)
'10 20 30

Let's remove the '?

to agree,
two ways of doing the same thing is bad

I agree
Em dom, 26 de nov de 2017 às 12:46, Cesar Gimenes<EMAIL_ADDRESS>escreveu:

to agree,
two ways of doing the same thing is bad
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rumlang/rum/issues/55#issuecomment-347013402, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEpfGVu6CDTNcWnYjTlSHLgkzV749dM7ks5s6XmpgaJpZM4Qqy3d
.


[epic] Publisher Page

US: As a Consumer I want to see a publisher’s profile so that I can discover their packages and get a sense of how active and good they are
As a Publisher I want to have a profile with a list of my data packages so that:

Others can find my data packages quickly and easily
Can see how many data packages i have
I can find a data package i want to look at quickly [they can discover their own data]
I can find the link for a data package to send to someone else

Tasks

[x] Design page #79
[x] Implement page in frontend using mocks - covered in items in this epic https://github.com/frictionlessdata/dpr-js/issues/53
[x] Implement Publisher info API #202
[x] Implement API to get publisher packages - part of search epic #200
[x] Use this in frontend - https://github.com/frictionlessdata/dpr-js/issues/58

Analysis
Key features

Publisher details

Title and username
Picture
When they joined
[Short description text (how is this set? We don't have a profile editor atm)]

List of packages
Members - list users who are members (all public by default atm)

Questions:

Do we implement UI in Flask or React
Decide re showing private packages. [if got jwt we can determine it is member/owner of the publisher]. Then we do not  filter the packages result set, show all packages with tag private.

Publisher API {1}
API
GET /publisher/{name}

//Response

{
    "name": "",
    "description":"",
    "packages": [
        {
            "name": "package1",
            "description": "",
            "private": true
        },
        {
            "name": "package2",
            "description": "",
            "private": false
        }
    ],
    "members": [
        {
            "name": "mem1",
            "role": "member"
        },
        {
            "name": "mem2",
            "role": "owner"
        }
    ]
}

Page Architecture (if we use ReactJS)
Design in React:
Publisher profile will be live in:
<Route path="/:name" component={PublisherProfile}/>

PublisherProfile component will hold the whole page
PublisherProfile:
Publisher profile is mostly data presentation component.
Thie Component will have two children component

PackageList

PackagePreview

Description [first 200 letter]
Name
Build passing or not [right now mock]
Private/ Public [mock right now]

UserList

UserPreview

Full Name
Gravatar
Role

API
/api/profile/publisher/
Response:
{
  "title": "", //like display name
  "joined":"",
  "writeup:"some details writeup about the publisher",
  "contact": {
<EMAIL_ADDRESS>   "phone":"1234-1111"
  },
  "country": "somewhere on earth"
}

FIXED.
The url is https://staging.datapackaged.com


proposal: strings/bytes: Graphemes and GraphemesReversed

Proposal Details
I propose to add the functions Graphemes and GraphemesReversed to the packages strings and bytes.
#14820 proposes to add such functionality to /x/text, but iterators have made this function easier to write and use, so I think these function belong in std.
First, a few tables are needed, but these are worth their space.
These mostly consist of other tables, so it can be optimized. https://www.unicode.org/reports/tr29/#Grapheme_Cluster_Break_Property_Values
Also these can be used in regexp, for example this regex can used to match a grapheme: \p{gcb=CR}\p{gcb=LF}|\p{gcb=Control}|\p{gcb=Prepend}*(((\p{gcb=L}*(\p{gcb=V}+|\p{gcb=LV}\p{gcb=V}*|\p{gcb=LVT})\p{gcb=T}*)|\p{gcb=L}+|\p{gcb=T}+)|\p{gcb=RI}\p{gcb=RI}|\p{Extended_Pictographic}(\p{gcb=Extend}*\p{gcb=ZWJ}\p{Extended_Pictographic})*|[^\p{gcb=Control}\p{gcb=CR}\p{gcb=LF}])[\p{gcb=Extend}\p{gcb=ZWJ}\p{gcb=SpacingMark}]*|\p{any}.
// https://www.unicode.org/Public/UCD/latest/ucd/auxiliary/GraphemeBreakProperty.txt
var PREPEND = &unicode.RangeTable{}
var CONTROL = &unicode.RangeTable{}
var EXTEND = &unicode.RangeTable{}
var SPACING_MARK = &unicode.RangeTable{}
var REGIONAL_INDICATOR = &unicode.RangeTable{}
var L = &unicode.RangeTable{}
var V = &unicode.RangeTable{}
var T = &unicode.RangeTable{}
var LV = &unicode.RangeTable{}
var LVT = &unicode.RangeTable{}
// https://www.unicode.org/Public/16.0.0/ucd/emoji/emoji-data.txt
var EXTENDED_PICTOGRAPHIC = &unicode.RangeTable{}
// https://www.unicode.org/Public/UCD/latest/ucd/DerivedCoreProperties.txt
var INCB_LINKER = &unicode.RangeTable{}
var INCB_CONSONANT = &unicode.RangeTable{}
var INCB_EXTEND = &unicode.RangeTable{}

These are some constants and a helper function.
func RunesReversed(s string) iter.Seq2[int, rune] {
	return func(yield func(int, rune) bool) {
		for i := len(s); i > 0; {
			r, size := utf8.DecodeLastRuneInString(s[0:i])
			i -= size
			if !yield(i, r) {
				return
			}
		}
	}
}

const LF = '\n'
const CR = '\r'
const ZWJ = '\u200d'

And then you can concatenate the files and generate these tables:
var re = regexp.MustCompile(`(?m)^(?<startRange>[[:xdigit:]]+)\.{0,2}(?<endRange>[[:xdigit:]]+)?\s*;\s*(?<property>\w+)(?:\s*;s*(?<subProperty>\w+))?`)

func gen() {
	matches := re.FindAllStringSubmatch(GBP_TXT, -1)
	for _, match := range matches {
		startRange, err1 := strconv.ParseUint(match[1], 16, 32)
		endRange, err2 := strconv.ParseUint(match[2], 16, 32)
		if err1 != nil {
			panic("should not be")
		}
		if err2 != nil {
			endRange = startRange
		}
		newRange := make([]rune, 0)
		for r := startRange; r <= endRange; r++ {
			newRange = append(newRange, rune(r))
		}
		var rangeTable *unicode.RangeTable

		switch match[3] {
		case "InCb":
			switch match[4] {
			case "Consonant", "Extend", "Linker":
			default:
				continue
			}
		case "Prepend":
			rangeTable = PREPEND
		case "Control":
			rangeTable = CONTROL
		case "Extend":
			rangeTable = EXTEND
		case "Regional_Indicator":
			rangeTable = REGIONAL_INDICATOR
		case "SpacingMark":
			rangeTable = SPACING_MARK
		case "L":
			rangeTable = L
		case "V":
			rangeTable = V
		case "T":
			rangeTable = T
		case "LV":
			rangeTable = LV
		case "LVT":
			rangeTable = LVT
		case "Extended_Pictographic":
			rangeTable = EXTENDED_PICTOGRAPHIC
		default:
			continue
		}
		newRangeTable := rangetable.New(newRange...)
		*rangeTable = *rangetable.Merge(rangeTable, newRangeTable)
	}
}

I created a sample implementation that focuses on easy readability and understandability, so I left out caching and ASCII optimizations.
func Graphemes(s string) iter.Seq2[int, string] {
	return func(yield func(int, string) bool) {
		if s == "" {
			return
		}
		currIdx := 0
		lowIdx := 0
		currSize := 0
	loop:
		for ; ; currIdx += currSize {
			currRune, size := utf8.DecodeRuneInString(s[currIdx:])
			currSize = size
			nextIdx := currIdx + currSize
			if nextIdx >= len(s) {
				goto ret
			}
			nextRune, _ := utf8.DecodeRuneInString(s[nextIdx:])
			if currRune == CR && nextRune == LF {
				continue
			}
			if unicode.Is(CONTROL, currRune) || currRune == CR || currRune == LF {
				goto isBreak
			}
			if unicode.Is(CONTROL, nextRune) || nextRune == CR || nextRune == LF {
				goto isBreak
			}
			switch {
			case unicode.In(currRune, L) && unicode.In(nextRune, L, V, LV, LVT):
				continue
			case unicode.In(currRune, L, V) && unicode.In(nextRune, V, T):
				continue
			case unicode.In(currRune, LVT, T) && unicode.In(nextRune, T):
				continue
			}
			if unicode.Is(EXTEND, nextRune) || nextRune == ZWJ || unicode.Is(SPACING_MARK, nextRune) {
				continue
			}
			if unicode.Is(PREPEND, currRune) {
				continue
			}
			if unicode.Is(INCB_CONSONANT, nextRune) {
				inCbLinkerCount := 0
				for _, prevRune := range RunesReversed(s[:nextIdx]) {
					if unicode.Is(INCB_LINKER, prevRune) {
						inCbLinkerCount += 1
						continue
					}
					if unicode.Is(INCB_EXTEND, prevRune) {
						continue
					}
					if unicode.Is(INCB_CONSONANT, prevRune) && inCbLinkerCount > 0 {
						continue loop
					}
					break
				}
			}

			if currRune == ZWJ && unicode.Is(EXTENDED_PICTOGRAPHIC, nextRune) {
				for _, prevRune := range RunesReversed(s[:currIdx]) {
					if unicode.Is(EXTENDED_PICTOGRAPHIC, prevRune) {
						continue loop
					}
					if unicode.Is(EXTEND, prevRune) {
						continue
					}
					break
				}
			}
			if unicode.Is(REGIONAL_INDICATOR, currRune) && unicode.Is(REGIONAL_INDICATOR, nextRune) {
				riCount := 1
				for _, prevRune := range RunesReversed(s[:currIdx]) {
					if unicode.Is(REGIONAL_INDICATOR, prevRune) {
						riCount += 1
						continue
					}
					break
				}
				if riCount%2 != 0 {
					continue
				}
			}
			goto isBreak
		isBreak:
			if !yield(lowIdx, s[lowIdx:nextIdx]) {
				return
			}
			lowIdx = nextIdx
		}
	ret:
		yield(lowIdx, s[lowIdx:])

	}
}
func GraphemesReversed(s string) iter.Seq2[int, string] { /* The same, but backwards  */}

Related Issues and Documentation

proposal: unicode: add emoji properties #45264 (closed)
proposal: x/text/rangetable: lazy construction of RangeTables #45282 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

This belongs in x/text. It's too wrapped up in Unicode details to belong in the low-level strings package.

The rules are actually not that difficult. It is just this table I've implemented.
https://www.unicode.org/reports/tr29/#Grapheme_Cluster_Boundary_Rules
currRune (×| ÷) nextRune
Unicode standards like case also have caveats and are in std. I think graphemes are useful enough to be unicode or strings

Can you point to some code that would use these new functions?  Bear in mind https://go.dev/doc/faq#x_in_std.  Thanks.

The rules are actually not that difficult. It is just this table I've implemented. https://www.unicode.org/reports/tr29/#Grapheme_Cluster_Boundary_Rules currRune (×| ÷) nextRune Unicode standards like case also have caveats and are in std. I think graphemes are useful enough to be unicode or strings

That table alone exceeds the conceptual complexity of the rest of the strings package, which must not depend on the Unicode character class tables. The right place for this change (if anywhere) is the unicode package. I'll retitle the issue to redirect the most obvious criticism.

I wrote github.com/apparentlymart/go-textseg/textseg to fill this gap for some of my own projects. (Unfortunately because my repository contains some Unicode-licensed content pkg.go.dev won't render the docs. :man_shrugging: )
A big chunk of the early work there was dealing with the fact that the relevant character tables were not already exported from anywhere as unicode.RangeTable. I did include unicode.RangeTable values generated from the source data, but since I was generating things anyway I ended up implementing the actual tokenizer in terms of a Ragel-language transform of the raw data, rather than using the range tables, since that allowed dealing with the UTF-8 recognition and grapheme boundary recognition all at once in a single state machine.
I also have a suite of tests that were mechanically generated from the test data provided by Unicode, in case that's useful for cross-checking a new implementation.
If this were in the standard library or in x/text then I would likely cease development of my module. However, developing that module has not been a major workload since usually it's just a matter of obtaining the latest version of the tables from Unicode and re-running the generators. There was one major version of Unicode that significantly changed the algorithm, but since then only the tables have changed.


fix: A set of qdata/qwatch related fixes

Issue being fixed or feature implemented
Fix/tidy up a few qdata/qwatch related parts, improve performance for regular non-watching nodes
~based on #5744 atm~
What was done?
pls see individual commits
How Has This Been Tested?
run tests
Breaking Changes
n/a
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

dropped c5a66ff and 16f5e00, adjusted d0a4894

utACK for squash merge; LGTM if you think c5a66ff is valuable, let's open a new PR targeted for v21 with release notes?

https://github.com/dashpay/dash/pull/5772

Approval contingent on CI

Gitlab had some unrelated issues https://gitlab.com/dashpay/dash/-/jobs/5770189961, restarted it


Show workspace username in modal profile list

Describe your changes

Resolves MOD-2220

Changelog
In modal profile list, the user's GitHub username is now shown as the name for the "Personal" workspace.

@prbot approve


ADR: JavaScript Modules exports convention

Please share your thoughts, concerns, feedbacks ... constructive criticism is welcome
closes #12

In case you want the ADR to address the strongest arguments for "default exports", let me take a stab at laying out what that is.
I was a big believer in "default exports".  Even though I abandoned them, I still believe "default exports" is an expression of the "single responsibility principle".
Under this line of thinking, the resolution would've been:

"a module MUST represent a single, exportable idea."

In other words, if you feel the need to export multiple things, you have NOT done the work of discovering what that modules single responsibility is.

In other words, if you feel the need to export multiple things, you have NOT done the work of discovering what that modules single responsibility is.

You had the answer almost, the module itself is the single responsibility principle 🙂
Exporting multiple things, although it may feel that is doing multiple thing, it is not. Especially if your programming language itself is not pure object oriented like Java or C#.
Now, it your module is all over the place, then I resonate with your concern but that is another conversation outside of this ADR scope.
Your concerns are there regardless of the strategy of how you export things, for example, I can export a single object (as it is done in CJS) wiht many things in it and have the same concern. Or even export a class (an object for that matter) and have a lot of static functions in it just because.
So I totally agree with your concern but your are feeling weird about such thing for the wrong reasons I think.
does that makes sense?

Moved the ADR to reviewing

👍


Stax porting

Hello,
I have finished ported the Solana application for our latest device Ledger Stax
My work is taking place on top of the swap modifications I made
https://github.com/LedgerHQ/app-solana/pull/44
I have separated the new feature in three independant PRs:

Add more Ragger tests for automated non regression testing and Stax screen reproducability
https://github.com/LedgerHQ/app-solana/pull/61

Extract the UI code from the logic handling code for the current graphical library BAGL
https://github.com/LedgerHQ/app-solana/pull/62

And finally the Stax porting itself for all screens
https://github.com/LedgerHQ/app-solana/pull/59

conclusion here https://github.com/LedgerHQ/app-solana/pull/69


IRC resources. irc2go.com -> irc.openstreetmap.org

We have some links to irc2go.com. D'you think we should link irc.openstreetmap.org instead?
There's a URL trick for doing this. The chan parameter sets that form value. So for example instead of http://en.irc2go.com/webchat/?net=OFTC&room=osm-gb we'd link https://irc.openstreetmap.org/?chan=osm-gb
I think it would be better for a couple of reasons. 1) I suppose being on a "openstreetmap.org" domain looks more professional/reassuring.  2) irc2go (links like this http://en.irc2go.com/webchat/?net=OFTC&room=osm-gb ) tries to launch a java applet, and who uses java applets these days? (I seem to have managed to browse the web for four months on a new laptop without ever needing to, so this link basically doesn't work for me unless I do some horrible set-up process)

Sure, switching to irc.openstreetmap.org seems ok.

I was noticing today that the chan parameter doesn't actually work. It looks like it's working (so much so that we didn't notice the problem) bit it goes on to connect you to the default #osm channel.
Seems like a bug in cgiirc which I've reported here: https://github.com/dgl/cgiirc/issues/12

Can't you just use https://webchat.oftc.net/?channels=osm-gb instead? Works fine

Lack of response upstream from the cgiirc developers, so that's annoying. I think we should fork the landing page form and make it more OpenStreetMap-styled anyway, so that could be the ultimate solution.
In the meantime @FredrikLindseth 's suggestion seems good. There's something nice and "native" about linking to the people that actually host the IRC channels, and they have a webchat interface with working channel links! That's cool. Didn't know about it.

Tagged this is a good first issue for anyone who wants to contribute.  Task here would be to replace any current irc2go.com or irc.openstreetmap.org links with webchat.oftc.net links.

can I work on this issue. As I am new to this project can you please specifically tell exactly what part of codebase should I work on?
Thanks..

Welcome! Look at the JSON data files under 'resources'.  It's going to look quite similar to this commit: https://github.com/osmlab/osm-community-index/commit/366f9a63a4d892747e22b2a33655cf4cece5c40e  but changing the URLs again.


Display Download speed and latency on a i2c display like SSD1306

Hello there..I tried this project and it works great...but I wanted to display one or two parameters like the download speed and latency on a i2c display like the SSD1306 using the SDA and SCL pins on the pi ..
I've used this repo to display different items on the display with nice animations and other features :

I can use this repo with the internet-pi one..to display the parameters..but I need some help..where is the file and code in which the speedtest-docker functions take place..and can I write some bash scripts or python along with it..any help is appreciated..Thank You

This is the display library repo

I've improved the code a little bit for clarity. Feel free to use those functions to feed data to your i2c screen.
def getDownloadSpeed():
	try:
		response = requests.get('http://localhost:9090/api/v1/query?query=speedtest_download_bits_per_second[1h]')
		if (response.status_code == requests.codes.ok):
			values = response.json()['data']['result'][0]['values']
			return float(values[0][1]) # bps
		else:
			return 0
	except:
		return -1

def getUploadSpeed():
		try:
			response = requests.get('http://localhost:9090/api/v1/query?query=speedtest_upload_bits_per_second[1h]')
			if (response.status_code == requests.codes.ok):
				values = response.json()['data']['result'][0]['values']
				return float(values[0][1]) # bps
			else:
				return 0
		except:
			return -1

def getPing():
		try:
			response = requests.get('http://localhost:9090/api/v1/query?query=speedtest_ping_latency_milliseconds[1h]')
			if (response.status_code == requests.codes.ok):
				values = response.json()['data']['result'][0]['values']
				return float(values[0][1]) # ms
			else:
				return 0
		except:
			return -1

Hi @IM-TechieScientist it's homer from discord


Extra ' mark added automatically when bolding text at the end of the bolded text

When using T3LT in VSCode, if I go to bold the word 'text' this is what happens:
When I put '' in front of text and then '' at the end, an extra ' is automatically inserted, so I get:
''text'''

https://user-images.githubusercontent.com/16529/161400634-ffdb3ea4-8b59-46f9-bacd-500aace876ce.mp4
Is this what you are experiencing?

No - it looks like this

:

Try changing the vscode setting named "Auto Surround" to either 'brackets' or 'never'.

@MinusGix  -  thanks

(it is the same thing) 👍


Config

This pull request updates the delete() function in the motorcycle controller.

⚠⚠⚠Hey guys please don't merge this PR until we figure out how to resolve the complication we are having with the commits 👍.  https://github.com/TamaraDimov/vespa-rentals-backend/pulse


Created new Command - Watch2Gether

Where you can create a room over the Watch2Gether API and optional with a preload video.
Also, some minor appearance changes regarding Mentioning and the Footer for the Response Messages.

I let the Branch because there are some crypto things in @maxsteinwand Discord Bot that we need to have a look if we want to convert that


[Feature] Highlight Changed Files for the Assessment of Programming Exercises

Checklist

[x] I tested all changes and all related features locally.
[x] Server: I followed the coding and design guidelines.
[x] Server: I added multiple integration tests (Spring) related to the features (with a high test coverage)
[x] Server: I added @PreAuthorize and check the course groups for all new REST Calls (security)
[x] Server: I implemented the changes with a good performance and prevented too many database calls
[x] Server: I documented the Java code using JavaDoc style.
[x] Client: I followed the coding and design guidelines.
[x] Client: I documented the TypeScript code using JSDoc style.
[x] Client: I added multiple screenshots/screencasts of my UI changes

Motivation and Context

Currently, the tutor or instructors who assess students submissions can not directly see which files were changed or newly added during the students participation on a programming exercise. This PR introduces a highlighting, so that the tutor/instructor directly know which files they need to assess and which not (as those have not been changed).
Description

Added new endpoint to return if a file was changed/is new or not
Calling the new endpoint when tree structure for file browser is loaded
Made sure this is only for manual assessment of programming exercises

Steps for Testing

Log in to Artemis
You can create a programming exercise or use a existing one
Open the student code editor or the instructor code editor ("Edit in editor")
Check that the files are not highlighted and check in network tab that files-change REST call is not called
Go to Assessment Dashboard
Open/Start Assessment
Check that the files that were changed/are new are highlighted and check in network tab that files-change was called (See screenshots below)

Test Coverage

RepositoryService.java:  86%
Coverage of RepositoryProgrammingExerciseParticipationResource is ~70%. Did not decrease through this PR as introduced method has coverage of 100%. Will be increased in a follow up.

Screenshots

When assessing programming exercises, see that the call files-change is called:

When accessing as student, see that the call files-change is not called

Not sure if you noticed but there are a couple Codacy issues in your integration tests.

@ivanchimeno Those are just because of the e.g. testGetId_forbidden() structure. We are ignoring those :)


Partial server side HTML rendering

Disclaimer I haven't dig too much into jte nor tried it yet, so excuse me if I'm saying something stupid here

First of all, it's very nice to see a fresh kid in town as it seems that the other templating engines for Java are not evolving anymore.
I would like to ask if there are plans/possibility to implement partial HTML rendering. There is probably a better term for that, but what I mean here is:

click on a button
the browser goes to server
the server processes the request
renders a new part of the HTML and sends just a part of it to the browser
browser renders a small part of the screen

So that this:
.
.
.

    <button>Click me</button>

.
.
.

becomes, for example, this:
.
.
.

    <button>Click me</button>

Message from server

.
.
.

Without a full browser refresh.
I could be wrong here again, but I believe that's one of the features of vaadin, but the last time I tried it, I got sick of it pretty quickly.
That being said, if this is not possible today, I bet it's not trivial to implement. Though could be a very nice feature to have IMHO.
I'm not here only to ask for the feature, but if there's a possibility/interest in doing it, I would like to help to implement such a feature with some guidance, if I/maintainer have the time.

Hey @renannprado,
thanks for the heads up! I've played a bit with partial rendering, bit like Amazon does it in their checkout flow, but there's nothing serious to see yet.
I'd love to try an HTMX integration one day, but haven't found the time yet:
https://htmx.org/

@casid I've never heard of htmx, but it looks pretty good at first sight. At the moment I cannot foresee that jte would need some kind of support in order to use htmx, right? it should be possible to compile & render partial jte files, right?

@renannprado Yes.

@casid looks like this lib is also interesting, but nowadays it's only easy to find information about it + Ruby on Rails: https://hotwire.dev/

I'm not sure how this is related to JTE or any template rendering engine at all. You can do this:
Let's say an https://example.com/page.html is opened in the browser.

User clicks on a button on this page.
JavaScript in browser intercepts button click event.
JavaScript sends a request to the server to a special endpoint: https://example.com/partial.html
Server processes this request, generates an HTML code and sends it to browser.
JavaScript in browser receives this HTML code from the server and inserts it into page, without reloading page at all.

Just cleaning up issues and closed this, since this would be something built on top of jte.

@izogfif Take a look at htmx.org . It perfectly integrates with JTE. It allows you todo exactly what you want. I am currently using it on a project with Spring Boot + JTE + HTMX.


Spell the name of this module correctly in the README.

Coverage remained the same when pulling 36dcd8a03464bd55db6e191602e986ef3467982a on markstos:patch-1 into e8341cae3471dae2783406fd94e820d6d75e769a on regexps:master.


Atom vertical line between curly brace

Hello, sorry for the bothering, but i installed atom 1.0.19 and i can't get the vertical line between two curly brackets when i'm inside a function, the line in the red ellipse in the image(the photo is not of my version), is absent from my installed version and i can't get, i searched for a package but i didn't find, please if some one help me with a name of a package to added it or a configuration, and sorry for my bad english

Go to Settings and make sure Show Indent Guide is enabled. If it is, and you still don't see an indent guide, then try using a different syntax theme (like One Dark) - it could be that the syntax theme you're using is hiding them. Does that help?

thanks it works.

thanks


Good

Good
Originally posted by @Savioure in https://github.com/ton-blockchain/intellij-ton/issues/103#issuecomment-2188800954

Да

Good


[LOC] Possibly incorrect source text in an article

Hello,
During translations of one of the articles it was brough to our attention that there might be an error in the sentence below:
Failure to sign in will result in the users being able to use the app
Which does not make sence
The sentence can be found within the article below:
https://docs.microsoft.com/en-us/deployoffice/privacy/essential-services
GH article:
https://github.com/MicrosoftDocs/OfficeDocs-DeployOffice/blob/live/DeployOffice/privacy/essential-services.md

@OfficeGlobal  -- thanks for bringing this to our attention. We're checking into it, but I suspect there's a "not" missing from that sentence.

@OfficeGlobal -- we have received confirmation that the sentence is missing the word "not."  We've updated the article and re-published. Thanks for bringing this to our attention.


License missing

There is currently no license text in the repository. That makes it very hard to use. Would be great, if you would add a license.

Done.


9anime: HTTP error 500 when fetching anime details

Source information and language
9anime 13.35 (English)
Steps to reproduce
Go to 9anime latest series.
Tap on an anime
Expected behavior
It should populate a series with episodes
Actual behavior
Pops up HttpException: HTTP error 500
Aniyomi version
Beta r6135
Android version
Android 11
Other details
I suspect it's because consumet API is unstable and it will give server errors
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Yes, I'm having this exact issue. Emphasizing this!

@jmir1
We don't like to be an app extension, so please stop to not waste your time.

@jmir1
We don't like to be an app extension, so please stop to not waste your time.

Yo what's up, is this the real 9anime?

Your app too bro, it seems hard to run without our site.

Of course, if he didn't say that. We just want to talk peacefully.
If you feel that's funny.... Take your time.

@nine-anime
Your website does not have any terms and conditions or anything that says we cannot fetch data from your website.
It does state however, that you are only linking to content from third parties, so why shouldn't we be allowed to do the same?

The point is your app use our server resources, we don’t want to be flooded without any benefits, no new user, no visitor, just more resource usage.
We do not want to waste our time on cat and mice game, so we’re here. Of course, if you don’t agree, we can play until one stopped like Saiko.
On 14 Mar 2023 at 18:04 +0700, jmir1/aniyomi-extensions @.***>, wrote:

view it on GitHub

Locking this because the discussion has nothing to do with this issue


CodeGen error when URL is empty

Describe the bug/problem
When the URL field is empty and you click on view code button, some codegen throws an error.
Steps to Reproduce the bug/problem

Type something in URL field,
press view code button.
select any language

CURL
Node.js (fetch)
Javascript (fetch)
Node.js (axios)
Javascript (axios)

remove the URL text.

https://github.com/foss42/apidash/assets/72187898/a24b69af-d3da-49c3-ae1a-4cfeb9b423a7
Expected behavior
This should not show an error, instead  The entire code should be display with URL as ''
Device Info (The device where you encountered this issue):

OS: [e.g. Windows, MacOS]
Version: [e.g. Catalina 10.15.7, Monterey 12.3.1, Windows 11 22H2]
Browser (only if you encountered the issue while running the web app): [e.g. chrome, safari]

I'll raise a PR soon, stay tuned!

@opxdelwin I think i got the solution

Duplicate of #157


[QUERY] trace-id in Cosmos DB refresh every 5 minutes is reused

Library name and version
Microsoft.Azure.Cosmos v3.41.0
Query/Question
My WebAPI code uses Azure.Monitor.OpenTelemetry.AspNetCore v1.2.0 instrumentation and the Microsoft.Azure.Cosmos v3.41.0 SDK.
I am seeing periodic GET request to Cosmos DB every 5 minutes. I am suspecting this is the SDK refreshing some Cosmos region information.

All operations seem to be reusing a correlation id (trace-id) of an initial API request to my API.
Is it expected that the trace-id is reused for these periodic background calls? Is there a way to influence (reset?) the trace context used for these operations?
Environment
.net 8.0 code deployed to a Azure Website
Azure.Monitor.OpenTelemetry.AspNetCore v1.2.0
Microsoft.Azure.Cosmos v3.41.0

Gi @andrasg.  Thanks for reaching out and we regret that you're experiencing difficulties.   I've transferred this to the repository where the Microsoft.Azure.Cosmos package is maintained as #4581.  This will ensure that the folks best able to assist have visibility.


Fixed to pass with verilator --lint-only.

When I ran flow.tcl on openLane 2023.05, an error occurred in STEP 1. In my opinion, there are a few things that should be fixed, which I propose in this pull request.

Hi @ryos36 do you have the same issue with the latest update to the repo?

Hi, I used 9e1a4933eb6a88822595fabb93587cfdcec28cb2, it seems same issue.

Can you please post the error? Also, it is intentional that the example is a 16 bit counter not a 32 bit counter.

sorry late reply.
I used "openlane/2023.05/flow.tcl".
This version uses verilator for lint check.
typescript.txt

you can check use verilator like as followings.
$ verilator --lint-only -Wall --Wno-DECLFILENAME --top-module user_proj_example -I /foss/pdks/sky130B/libs.ref/sky130_fd_sc_hd/verilog/primitives.v -I /foss/pdks/sky130B/libs.ref/sky130_fd_sc_hd/verilog/sky130_fd_sc_hd.v /foss/designs/caravel_user_project/openlane/user_proj_example/../../verilog/rtl/defines.v /foss/designs/caravel_user_project/openlane/user_proj_example/../../verilog/rtl/user_proj_example.v -Wno-fatal --relative-includes


extstore: fix data bugs on high overwrite key

Scenario required:

likely item_age=2 (or some low value), so cache memory is otherwise not used to hold data very long.
one, or a few keys, are frequently updated and re-written to disk.
at some point extstore compaction runs

There are two faults to this bug:

We may accidentally "rescue" an older version of the frequently
overwritten key. More likely the data will go missing.
More likely, extstore will log too many "deletes" against an active
disk page. It will then think the page is empty when it isn't and
invalidate it.
This results in other random items disappearing when they shouldn't.

The fix is simple in retrospect: when validating an in-memory header against data on disk, we check that the page and page version are the same, but it was not checking that the offset on disk is correct. Now it checks all three identifiers which fixes the bug.
Holding this for a bit until I can run more tests against it, but I can't think of any reason why it would be incorrect.

uh oh. blew up here but tests are passing locally... race condition?
ah got chunked-extstore to fail. I wonder if the tests were accounting for this breakage already :(

still unsure if the test or code is wrong. it's only failing for large items.
Can see it finding headers for keys but the offsets are all wrong. will resume tomorrow..

oooooooooooooooooooooooo right this is readback chunk offset, not total page offset. need to add the read buffer's page offset.

All clean. Gonna let it sit and merge after I check a few other tasks in case I think of something.

Released in 1.6.21


Execution failed for task ':react-native-webview:compileDebugJavaWithJavac'.

I'm using React Native 0.71.8. and web view version 12.2.0. I'm getting the following error,
FAILURE: Build completed with 2 failures.

1: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':react-native-webview:compileDebugJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.
-----------
* What went wrong:
java.lang.StackOverflowError (no error message)

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
==============================================================================

* Get more help at https://help.gradle.org

BUILD FAILED in 55s

    at makeError (D:\Hexxion\PlatinumAccess\PlatinumAccessApp\node_modules\@react-native-community\cli-platform-android\node_modules\execa\index.js:174:9)
    at D:\Hexxion\PlatinumAccess\PlatinumAccessApp\node_modules\@react-native-community\cli-platform-android\node_modules\execa\index.js:278:16
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async runOnAllDevices (D:\Hexxion\PlatinumAccess\PlatinumAccessApp\node_modules\@react-native-community\cli-platform-android\build\commands\runAndroid\runOnAllDevices.js:82:7)
    at async Command.handleAction (D:\Hexxion\PlatinumAccess\PlatinumAccessApp\node_modules\@react-native-community\cli\build\index.js:108:9)
info Run CLI with --verbose flag for more details.

By downgrading to 12.1.0 will solve this problem. Just remove ^ , and keep it like this,
"react-native-webview": "12.1.0"

By downgrading to 12.1.0 will solve this problem. Just remove ^ , and keep it like this,
"react-native-webview": "12.1.0"

Downgrading to 12.1.0 worked for me. It looks like 12.2.0 update maked this build error.

This is likely a duplicate of https://github.com/react-native-webview/react-native-webview/issues/2978

Hello everyone .. This is caused by https://github.com/react-native-webview/react-native-webview/commit/75e7801a6255b37f6887d29d10b9ba688e051416 which I merged a couple of days ago to mitigate large number of crashes reported in some OEM devices.
The motivation behind the fix was to leverage the change https://android-review.googlesource.com/c/platform/frameworks/support/+/1978527 which added a workaround to the androidx.core module to avoid stripping the File provider metadata.
The fix to androidx.core is available from https://developer.android.com/jetpack/androidx/releases/core#core-1.8.0-alpha06 (which was released more than an year ago) onwards. Could you folks upgrade to this version to avoid the crash ? Transitive dependency can be explicitly resolved as @mlazari suggested above https://github.com/react-native-webview/react-native-webview/issues/2978#issuecomment-1573469986
Thanks !

By downgrading to 12.1.0 will solve this problem. Just remove ^ , and keep it like this,
"react-native-webview": "12.1.0"

This worked quite well

I'm looking into making the usage of the new construction conditional on the availability of the new version of android modules .. Please hold on.

I had to downgrade to 11.26.1 because of this issue, but I've just tried 13.3.1 and it built and ran without problem on Android (and on iOS).

I had to downgrade to 11.26.1 because of this issue, but I've just tried 13.3.1 and it built and ran without problem on Android (and on iOS).

Yes on 13.3.1 it worked quite well

@gavrichards Yes, the change that was causing it was reverted in 12.2.1

@gavrichards Yes, the change that was causing it was reverted in 12.2.1

Thanks, I'm mentioning it here because this issue is still open.


Edit Collection Page

Overview of the Feature Request
Create the edit collection page in the SPA. Reuse the form to create a collection.
What kind of user is the feature intended for?
SPA Users
What inspired the request?
Q4 Roadmap
Any open or closed issues related to this feature request?
This issue will integrate with:

https://github.com/IQSS/dataverse-client-javascript/pull/204
https://github.com/IQSS/dataverse/pull/10925

Waiting for :

https://github.com/IQSS/dataverse-client-javascript/issues/214
https://github.com/IQSS/dataverse/issues/11012


chore: add gnu.org to LINKINATOR_IGNORE

What type of PR is this?

What this PR does / why we need it:
Fix the flaky link detetct:
[200] site/public/latest/tasks/quickstart
[0] https://www.gnu.org/software/make
[200] site/public/latest/tasks/traffic/grpc-routing

Which issue(s) this PR fixes:

Fixes #

dup of #3879


Supply seems to incorrectly deactivate lower tracks during rollout

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Supply appears to deactivate lower tracks when promoting to a new track. From what I can tell, this seems to result in APKs being unavailable to users who are whitelisted for the lower tracks.
Let's take an example here, where user A is whitelisted for the alpha track, but not the beta track. When an APK is promoted via supply from alpha to beta, we'd expect user A to be able to continue to receive the build. This doesn't appear to be the case - when looking at my Play Store console, it appears that the APK was removed from the alpha track during the promotion process.
I'm reasonably confident that this line (https://github.com/fastlane/fastlane/blob/master/supply/lib/supply/uploader.rb#L67) is the source of this bug. It appears that Supply is deliberately clearing the original track when promoting, which appears to be causing unintentional behavior. Note that it doesn't matter if the user passes the check_superseded_tracks option or not - this happens outside of that block of logic.
It's possible that the behavior of the Google APIs have changed since this was authored two years ago - maybe it was necessary at one point? But it certainly seems like this is no longer needed.
I'd be happy to roll a PR adding a flag to bypass this line if we believe that's a reasonable fix. But I figured I would check with the experts here to see if my understanding is at all reasonable before attempting to fix.

Is there any reason why one would actually want the current behavior?

Personally, I'm not sure the current behavior would ever really be desired. But maybe there's a specific reason out there with some setup of channels that would be helpful to have the deactivation. (Hard pressed to imagine a real use case, but I'm probably just being unimaginative)

I of course ask this because I saw you PR that adds a new option to get to the "probably correct" default state. If we really don't think anyone out there really could expect this behavior with the current documentation, maybe we can just change the default behavior and add the option as a way to get the other behavior back if really wanted @joshdholtz ?

Sounds reasonable to me - I'm happy to make that change, but will leave that up to the core team's discretion here! Let me know if that's the direction we want to go, and what docs I should update.

Any updates here - I'd really like to get this submitted soon, since this is beginning to cause problems with our release process.

@tmtrademarked I thiiinnnkkkkk you are right. This is possibly started behaving differently/wrong after Google Play added more tracks and custom tracks 🧐 I would have to do some playing around with this but I think it would probably be safe to see if removing that line fixes things.
IMO, that line seems like it has a little bit of a hack anyway so maybe its not needed anymore? If you want to try removing it and creating a PR and seeing if it works that would be 💯 Otherwise we may need to add an option for a track (or tracks) to demote 🤔

I've been running this locally for promoting my builds so far with the new option applied. (Effectively the same as just removing that line) So far, it seems to work just fine, and it matches my understanding of how this should work from the Play perspective.
My only concern here would be that removing that line technically results in a slight change in behavior. It's possible this breaks some workflow out there, although I'm hard pressed to imagine what could possibly be relying on this.
So yeah - TLDR, I'm good with removing the line entirely and not adding the option if we think that's more correct. Just give me a thumbs up here if that's the approach we think makes the most sense, and I can update the PR.

For some evidence of this working, here's my Play release screen after using these changes:

Before this change, the Alpha and Beta tracks would have had "fake" releases in them with a default name of the date - and notably, no binaries associated with them.

@tmtrademarked If you wanna create a PR with this change that would be 💯 I would love to test it out 😊

Alright - #13351 has been updated. There's no more option, it just doesn't deactivate tracks when promoting.

@joshdholtz @janpio - any further thoughts here? (Sorry to keep nagging!)


2010: Spaces in Names and/or States Not Captured

We still have an issue of not capturing as a new speaker or speaking a member with a space in their name. Examples, EDDIE BERNICE JOHNSON. Also those with "of New York" or any other state with multiple words. So the regex for newspeaker and (I think) speaking needs to be changed. I see that the speaking regex does not match the newspeaker regex. Maybe you can explain the logic for why those two lines of regex do not need to match. I know we are fundamentally re-working this, but I think we still need to address this within the current context unless the new logic is going to be ready to roll fairly soon.
So this line:
re_newspeaker = r'^(<bullet> |  )(?P<name>(%s|(((Mr)|(Ms)|(Mrs))\. [-A-Z\'a-z\']+( of [A-Z][a-z]+)?|((Miss) [-A-Z\'a-z\']+)( of [A-Z][a-z]+)?))|((The ((VICE|ACTING|Acting) )?(PRESIDENT|SPEAKER|CHAIR(MAN)?)( pro tempore)?)|(The PRESIDING OFFICER)|(The CLERK)|(The CHIEF JUSTICE)|(The VICE PRESIDENT)|(Mr\. Counsel [A-Z]+))( \([A-Za-z.\- ]+\))?)\.'
is still not capturing spaces in names. I think wherever we have [-A-Z\'a-z\'] we need to allow a space character as well. Actually, it may be replace all occurrences of a-z\' in the line of regex with a-z\'\s. I think this would allow a space to be part of the name or state. I would just edit it, but I am afraid of breaking it.
I propose:
re_newspeaker = r'^(<bullet> |  )(?P<name>(%s|(((Mr)|(Ms)|(Mrs))\. [-A-Z\'a-z\'\s]+( of [A-Z][a-z\s]+)?|((Miss) [-A-Z\'a-z\'\s]+)( of [A-Z][a-z\s]+)?))|((The ((VICE|ACTING|Acting) )?(PRESIDENT|SPEAKER|CHAIR(MAN)?)( pro tempore)?)|(The PRESIDING OFFICER)|(The CLERK)|(The CHIEF JUSTICE)|(The VICE PRESIDENT)|(Mr\. Counsel [A-Z]+))( \([A-Za-z.\- ]+\))?)\.'
Let me know what you think.

FWIW, I think this issue will also fix some of the issues of not capturing extensions properly as well. We get screwed up quite a bit there because they seemingly always list the member with their state in extensions, so anytime someone from new york, jersey, north dakata, carolina, whatever is giving and extension of remarks, we miss tagging their entry as a new speaker.

I am going to hold up on other things until we nail this down.


Pin EMSDK version for build where useful

Also fix test that fails at the moment

This small PR just helps to make all checks happy

Thanks for fixing this! Looks like the WASM builds are fixed now.
It looks like CI is failing right now because black would make changes to a couple of test files (test_helper.py and test_cli.py) that you have not changed. I don't understand why that is; I thought they were working before. Maybe black was updated recently?
IMHO this is worth merging as is, but I would not mind if you wanted to run black on the test/ directory.


SE part wording revision.

Comments and updates are welcome!! I think there is really room for further improvements.

@johnament Thanks a lot for revision. I'll update.

LGTM


Http Adapter Fixes for Stargate

Includes:

Formatting fixes
Token externalized
Minimal cycles introduced for defaults

this is a new pattern being applied to built-in scenarios to avoid confusion with initial usage.

Standardize for nb5

@jshook this one is good for second review.


HttpSM: fix no_dns_forward_to_parent for cache stale

Fixes the case where a stale asset will result in a DNS lookup on the remap.config destination with this option enabled.

[approve ci cmake]

[approve ci cmake]

Cherry-picked to v9.2.x


Proposals: improve handling of dataset_uuid and curator_tag in API interactions

I propose a few pieces of "tidying up" work with regard to the way that we wrangle the curator_tag and/or dataset_uuid in the local dataset upload case as well as other API interactions. These are discussion items and do not represent approved, mandated units of work yet.

DELETE /v1/collections/{collection_uuid}/datasets should instead use a path variable identifier at the end like /v1/collections/{collection_uuid}/datasets/{identifier}, rather than asking for a dataset_uuid or curator_tag passed in as a query parameter. It is more semantically logical. The identifier identifies a singular resource, and since we do not allow curator_tags to take the form of a uuid, we can use the logic we already have in the dataset submission lambda to distinguish between the two. Plug-and-play! This should be extended to other applicable endpoints as well.

All endpoints should be configured to handle 01234567-89ab-cdef-0123-456789abcdef.h5ad as the communication of a dataset uuid. There could easily end up being confusion given that we are, at present, requiring uploads to include a .h5ad suffix, whether for a dataset_uuid or a curator_tag, but then when the curator uses one of those as an identifier for a different API endpoint (such as the DELETE mentioned above), then suddenly the .h5ad is dropped from the uuid but not from the tag. This is a bizarre behavior that we should try to avoid from the client standpoint.

In general I'd like to see us move toward "identifier" as a term that could mean either a uuid or a tag. We've begun using this terminology in variable naming where appropriate.

Per triage with @jahilton and @Bento007 closing as not required.


NavigationView is missing acrylic and shadow

Describe the bug
When you open a NavigationView in LeftMinimal or LeftCompact modes, it will expand as a flyout but it doesn't have acrylic or a shadow.
Steps to reproduce the bug
Create a NavigationView with PaneDisplayMode as LeftMinimal or LeftCompact.
Screenshots

NuGet package version
2.7.1, 2.8.0-prerelease
Windows version
Windows 11 (21H2): Build 22000
Additional context
The Figma design does have both of them, and it respects the design guidelines since it's an overlay.

@karenbtlai this looks similar to an issue we recently encountered

Oh yes, I remember investigating this. Turns out something happened with some dependencies, since DropShadows now require set elevation and depth for shadows to show. This is why all flyout shadows still work, since they have set elevation in codebehind already.
Should this be a prioritized fix? @ranjeshj

Isn't Acrylic replaced by Mica for this particular use?


[FEATURE] Support AIDA-X reamps

Just like #364, but for AIDA-X's input.wav from their training notebook.
It seems that it's identical to v1_1_1.wav, plus an additional ~2 minutes of audio appended. If that's the case, then it can be supported by concatenating the new audio into the training split and grabbing the same validation split for consistency with NAM's other training files.
The soft-match is a little different since we can't assume that the validation signal is at the end. The flip side is that users could append more data onto the end easily instead of needing to inject it into the middle. (For that matter, I could implement a second soft-match for the NAM files in the same way...hmm!)

Do this as an Extension