Create a new tagged template string vnode factory function: Codename: hypertag

I think we could benefit from another hyperx-like template literal. Perhaps I should look into https://github.com/hyperapp/hyperapp/issues/88 before I do this, or maybe not, programming is fun.
1
import { html, app } from "hyperapp"

const Hello = ({ name }) => html`<h1>{name}</h1>`

app({
  view: model => html`
    <Hello name="world" />
  `
})

2
import { h, app } from "hyperapp"
import { html } from "hypertag"

const Hello = ({ name }) => html`<h1>{name}</h1>`

app({
  view: model => html`
    <Hello name="world" />
  `
})

About JSX/Hyperx and some things that I'd like to explore:

JSX works great and has the best support (that's why it's the most popular option despite "breaking" JavaScript). See: https://github.com/substack/hyperx/issues/2
Currently you use babel to transform JSX into vanilla JS.
Currently you use hyperxify to transform hyperx into vanilla JS (this involves a boilerplate that some people dislike @ngryman, but there's no way around it unless hyperxify changes: see https://github.com/substack/hyperxify/pull/8)
Rolling our own tagged literal means better synergy between both projects.

Of course, if anyone wants to go ahead and work on this, just do it.
See Also:

https://github.com/hyperapp/hyperapp/issues/85
https://github.com/hyperapp/hyperapp/issues/123
https://github.com/substack/hyperxify/pull/8

Something like this would be lovely.

I have a different point of view on this. Creating and maintaining a new template function is a load of work. Already proven implementations already exist, we should leverage them instead.
The only thing that bother us for the moment is more syntax-related than feature-related. So I propose that hypertag (nice name btw) would be a wrapper to hyperx (or equivalent) and bring syntactic stuff on the table. Nothing prevents you to then implement parsing yourself later, but IMHO that's not the priority.
Example
Your example should be modified because any classic template function only allows to do this:
app({
  view: model => html`
-    <Hello name="world" />
+   ${Hello("world")}
  `
})

But it highlights the fact that template functions have function purity in mind and don't care about templates composition (i.e. components). Which is perfectly ok because it fits any use cases.
Still in a real world use case, referencing other components like jsx allows you to do is something really appreciable.
So a good first use case for hypertag would be to add this feature and be able to do like this:
import { app } from "hyperapp"
import { tag, html } from "hypertag"

tag('Hello', ({ name }) => html`<h1>{name}</h1>`)

app({
  view: model => html`
    <Hello name="world" />
  `
})

Other use cases could be treated like arguments passing cascade that purity imposes. We could automatically pass model and actions to view by default for example.
To sump up, I would propose hypertag = hyperx + syntactic sugar than yet another template function.

@ngryman Nice writeup! 👍
Depends on https://github.com/substack/hyperx/pull/50. If that gets merged, then I'd be satisfied

Your example should be modified because any classic template function only allows to do this:

I'm not sure I'm following. If it doesn't support writing JSX-style components out of the box like this:
html`<MyComponent />

I'm not sure it would be much fun to use over JSX.

@jbucaran It's because it works on any es6 environment without any tooling involved. es6 support is getting momentum and in a couple of years template literals will just work for any browser: http://kangax.github.io/compat-table/es6/#test-template_literals.

@ngryman I think we are talking about two different things.
If I ever build a template literal like hyperx, it must support:
html`<MyComponent />`

out of the box, otherwise it's a no-no.
With regards to:

works on any es6 environment without any tooling involved.

It doesn't really matter if they are natively supported or not, I don't want to ship production (or any real code) that uses a template literal function, and specially one that is a XML parser.

If I ever build a template literal like hyperx, it must support: ...`

html`<MyComponent />`

As it's not possible with hyperx out-of-the-box (or any template function I know, or I'm missing something), I thought that was a typo. In fact I don't know how you can make this work if you don't name you component somewhere. Does hyperxify allow to achieve this?

It doesn't really matter if they are natively supported or not, I don't want to ship code to production (or anywhere) that uses a template function XML parser

Well I think we diverge here. For me it's totally ok to ship this to production when you target specific environments and you don't care about the performance cost of parsing.
For example if I develop a Chrome addon or an Electron app with just some basic interactions, I can live with shipping es6 template literals and do the parsing at runtime.

To illustrate my misunderstanding on the first point: https://github.com/trueadm/t7#components.

@ngryman Well I think we diverge here. For me it's totally ok to ship this to production when you target specific environments and you don't care about the performance cost of parsing.

Nope, I'm sure we don't diverge here. I wouldn't care either in that specific environment. Here is the thing though, while I don't deny such environment exists, I think they are extremely rare. You usually do care about cost of parsing, specially in the case of JSX/Hyperx.
Run js-framework-benchmarks or DBMonster, or whatever without compiling Hyperx into h and you'll see for yourself. We're not talking about 20% slower, it's probably more like 200% to 500% slower.

To illustrate my misunderstanding on the first point: https://github.com/trueadm/t7#components.

😞

So, @jbucaran, you want to create from scratch a template library like Hyperx or t7? Those are both kinda large. I'd hope it would just be an alternative to use like everything else. t7 inspired me to try my own hand at an ES6 template solution for components, that was before I stubbled upon Hyperapp on Reddit. Here's a pen of what I came up with: http://codepen.io/rbiggs/pen/pePajZ I used a loading mechanism to notify a component about child components so that I could use them as tags inside the parent. Because I had React users in mind, I based it off of a Component class. This solution allows you to create sibling elements and append them to a container without having to wrap them in an enclosing tag. I also created a DOM patcher in 1KB that is similar to Morphdom so I could update the DOM using the result for a template literal - no virtual DOM objects necessary. But that's not in this pen. I also created a style module so I could create virtual stylesheets scoped to the component, like JSX does, but for template literals. I just never came up with a model/update solution for it.

I'd hope it would just be an alternative to use like everything else.

Yes, obviously. For real projects I'd probably use only JSX.

Awesome, thanks, I've forked it and will use it while I learn how to do this.
I'm new to this too, but I made a similar thing here.

I also created a style module so I could create virtual stylesheets scoped to the component, like JSX does, but for template literals. I just never came up with a model/update solution for it.

Can you expand on this? 🤔

yeah, you just pass it an object describing your styles, and it uses the component as the scope for the style rules and then appends the stylesheet to the document. But if a stylesheet's already there, then it just adds the new rules. Was trying to be efficient at that ;-)
I can make you a pen to see it in action if you want. No biggie.

By the way, with t7, if you cut out all the code for React and Mythril you can reduce it by half.

By the way, with t7, if you cut out all the code for React and Mythril and just leave the universal part you can reduce it by half.

Yeah, you're right. Maybe I should rather look into #88. I'm just leaving this open issue to see if I get around this someday. Totally not a priority. Support for JSX is stellar and if https://github.com/substack/hyperx/pull/50 gets merged, then I'd be happy to use it every now and then.

Actually, #88 would be great. t7 is far more advanced than Hyperx. I love how t7 handles components. Maybe you could come up with a separate plugin to make t7 work with your h function. That way i'd be a separate load from hyperapp, not affecting its overall size.

Maybe you could come up with a separate plugin to make t7 work with your h function.

Yup, yup, that's idea exactly.

t7 is far more advanced than Hyperx.

Yeah, but that's holding me back a bit.

That way i'd be a separate load from hyperapp, not affecting its overall size.

No worries, we've changed how we deal with this from the early days when hyperx was a dependency and we're not going back to that.

On a totally different note, I see you're always talking about using Codemirror for projects. I've also done some major projects with it in the past. However, I hated it, hard to customize, etc. Then I discovered Ace. Loved it. Easier to customize. By fav is Ace with Tern for code completion and Intellisense like how Visual Studio Code works.

@rbiggs I tried using Ace before CodeMirror, but couldn't get it to work (this was before HyperApp). I'd love to give it another shot.

Odd, I found it so much easier to use. I was using it with RiotJS. I started out with Codemirror, because that's what the client had. But they wanted so much customization. The code base was hard to sort out, the project lead was not friendly. Took a look at Ace, big open source group, lots of participation, easier to understand how all the parts work. Ace was easy to customize popups, etc. whereas with Coremirror you're stuck with whatever Marijn decided you needed.
Maybe I'll fork your CodeMirror example and convert it to Ace. And then add some other features. I'll put it together sometime later next week when I have time. I'll keep you informed.

Maybe I'll fork your CodeMirror example and convert it to Ace. And then add some other features. I'll put it together sometime later next week when I have time. I'll keep you informed.

Awesome! Please do :) 🙏

I got hyperx to work as desired(afaict) pretty easily:
import { h, app } from 'hyperapp';
import hyperx from 'hyperx';

const hx = hyperx((t, a, c) => h(t.match(/^[A-Z]/) ? eval(t) : t, a, c));

const state = {
  count: 0
}

const actions = {
  down: value => state => ({ count: state.count - value }),
  up: value => state => ({ count: state.count + value })
}

const Component = ({count}) => {
  return hx`<h1>${count}</h1>`;
}

const view = (state, actions) => hx`

    <Component count=${state.count}></Component>
    <button onclick=${() => actions.down(1)}>-</button>
    <button onclick=${() => actions.up(1)}>+</button>

`;

app(state, actions, view, document.body);

That's a cool & "evil" trick @dschep! 😃

That is neat and evil, sadly. I'd be afraid to use eval in production. Would be so nice to have a Hyperx type template literal solution that didn't require Hyperxify, it just converted to what you needed at the end automatically, sigh.

100% agree with @rbiggs.

Yeah. Being evil is fun sometimes :smiling_imp:  but of course only for unimportant things. Could of course do a t7 style component lib, but then you have to register, components:
import { h, app } from 'hyperapp';
import hyperx from 'hyperx';

const hax = (() => {
  const registry = new Map()
  const hx = hyperx((t, a, c) => h(registry.get(t) || t, a, c));
  hx.register = component => registry.set(component.name, component);
  return hx;
})();

const state = {
  count: 0
}

const actions = {
  down: value => state => ({ count: state.count - value }),
  up: value => state => ({ count: state.count + value })
}

const Component = ({count}) => {
  return hax`<h1>${count}</h1>`;
}
hax.register(Component);

const view = (state, actions) => hax`

    <Component count=${state.count}></Component>
    <button onclick=${() => actions.down(1)}>-</button>
    <button onclick=${() => actions.up(1)}>+</button>

`;

app(state, actions, view, document.body);

BTW, I'm mostly interested in this so I can use hyperapp for simple 100% client-side projects without a build step. For example I have the above as a codepen with no build step, including hyperapp & hyperx from unpkg.com & wzrd.in respectively.

@dschep Also check https://github.com/hyperapp/html.


Create additional HTTP endpoints, improve templating

Endpoints include:

[x] Delete a scene
[x] ~Create a template scene instance~ Quick clone a scene

Template refactor:

[x] Remove @templateScene naming convention, allow any scene to be used as a template
[ ] Deal with TTL exception for @templateScene, whereby expireAt is not set in the mongo document (what actually enforces expiry behavior in mongodb). Perhaps via flags where persist: true, ttl: n are set, but some new flag (retainTTL: true ?) does same bypass of expireAt so every time someone loads the scene from persist (or instantiates a sceneTemplate), the object is kept with a "fresh" TTL

Splitting this one one off
Deal with TTL exception for templateScene, whereby expireAt is not set in the mongo document (what actually enforces expiry behavior in mongodb). Perhaps via flags where persist: true, ttl: n are set, but some new flag (retainTTL: true ?) does same bypass of expireAt so every time someone loads the scene from persist (or instantiates a sceneTemplate), the object is kept with a "fresh" TTL

Otherwise resolved as of 9643946


Bug reload first

Hello it is necessary reload the plugins brackets css color preview for reload the color.
Bye.

Closing/opening the file works too, but it could be improved.


[dashboard] adding an option to duplicate slices when "Saving AS"

refactored Checkbox out of CheckboxControl
added new checkbox on the dashboard save modal
allowed backend logic to clone the slices
proper new owner assignement

Coverage decreased (-0.1%) to 69.214% when pulling 2a5d85ff0ddfb79919e1bbbe7022316733c91434 on mistercrunch:save_dash_silces into 42dd64e4138c1d411d45e457acaf9e35c74f1135 on apache:master.

Coverage decreased (-0.1%) to 69.214% when pulling 2a5d85ff0ddfb79919e1bbbe7022316733c91434 on mistercrunch:save_dash_silces into 42dd64e4138c1d411d45e457acaf9e35c74f1135 on apache:master.

LGTM


Base.py does not allow anymore manual update to the plot and duplicate figures

"__manual = True" is no longer an argument in widget to specify a manual update to the plot
instead there is a full function "interact_manual" similar to former "interact(__manual=True)"
manual updates are useful for slow function (DC 2.5D for example)
widgetify create an independent figure that is not overwritten by the widget. The result is that we end up with 2 figures: the updated one and the very first one.

#74 implements an update to Base.py to deal with these 2 things: include a manual option and avoid duplicated figures.
The main drawback that I was not able to fix is that now, when you load the widget for the first time, there is no figure at all. You have to run it at least one to see the figure appears.
Another drawback (inconsistency due to the difference between interact and interactive functions):
app with manual = False are created and displayed through:"app = auto_app(); display(app)"
app with manual = True  are created and displayed through:"app = manual_app()" only (display would duplicate it)
doc:    ###http://ipywidgets.readthedocs.io/en/latest/examples/Using Interact.html

Seems the function interactive_manual is at least in preparation:
https://github.com/jupyter-widgets/ipywidgets/issues/1261

#74 now fix the figure issue. Only thing that I have left is the inconsistency for calling the app.


Information about team

On this team, we have two persons @ragangithub and @tjay1760
https://github.com/ragangithub/JavaScript-Capstone/projects/1

Hello @Team  , :wave:
I'm Levy, your PR reviewer
Good job so far :+1:
There are some issues that you still need to work on to go to the next project but you are almost there!

Status: REQUIRED CHANGES ♻
Check the comments under the review.

[ ] Kindly remove the cards for student C since you have a group of two students.

[ ] KIndly make sure the group task cards are assigned to both students instead of one. We should be able to see both students that are assigned on the task card.
YOUR GROUP TASK CARD

EXAMPLE OF EXPECTED GROUP TASK CARD

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @levy002 in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi @ragangithub & @tjay1760  👋,
Your project is complete!  There is nothing else to say other than... it's time to merge it.
Congratulations! 🎉 🥳🥇
Highlights

All project cards are present ✔
Kanban board is almost complete ✔

Optional suggestions

N/A

Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Fix broken html_root_url link

All html_root_url redirect to 404 github page. I fixed this with add path /libc.

cc @alexcrichton

Thanks for the PR! I think these are mostly just intended for rustdoc itself, though, which I believe already appends this information?

I'm going to close this because I think the URLs are correct as-is


Implement Temperature Service

Blocked by #23.

Closed, we'll do a monolithing key-value updating thing in the spaceapi_server crate.


publicUpload parameter in share updates

Describe the bug
Sending publicUpload parameter in PUT request to update a share (only upload permission) returns an invalid permission value, reflected in Android mobile client.
Steps to reproduce
Steps to reproduce the behaviour:

Create a public link, for example in that way:

curl -k -X POST -u <username>:<pwd> https://<url>/ocs/v2.php/apps/files_sharing/api/v1/shares -d 'path=/Documents&shareType=3&name=myLink&permissions=1'

A link over Documents with basic permission 1 (Download/View) created.

Update the link, by changing permissions to 4 (Upload only) including publicUpload (available as https://doc.owncloud.com/server/next/developer_manual/core/apis/ocs-share-api.html#update-share)

curl -k -X PUT -u <username>:<pwd> https://<url>/ocs/v2.php/apps/files_sharing/api/v1/shares/<share_id> -d 'path=/Documents/&shareType=3&name=MyLink&permissions=4&publicUpload=true'

Actual behavior
Link updated with permissions 15, not 4. Check with
curl -k -u <username>:<pwd> https://<url>/ocs/v2.php/apps/files_sharing/api/v1/shares

that returns
<permissions>15</permissions>
Expected behavior
In the final GET after updating permissions:
<permissions>4</permissions>
Setup
Version: 2.0.0-beta.4+27b71bf52
Compiled: 2022-07-06 00:00:00 +0000 UTC

created via docker compose file:
version: "3.7"

services:
  ocis:
    image: owncloud/ocis:latest
    ports:
      - 9200:9200
      - 9215:9215
    environment:
      OCIS_INSECURE: "true"
      OCIS_URL: "https://myServerIP:9200"
      IDM_CREATE_DEMO_USERS: "true"
      IDM_ADMIN_PASSWORD: "admin"
    restart: "no"
    entrypoint:
      - /bin/sh
    # run ocis init to initialize a configuration file with random secrets
    # it will fail on subsequent runs, because the config file already exists
    # therefore we ignore the error and then start the ocis server
    command: ["-c", "ocis init || true; ocis server"]

Additional context
publicUpload has been used in clients and still supported in oC10 with correct results. Documentation related (not sure if outdated, i hope it does not):
https://doc.owncloud.com/server/next/developer_manual/core/apis/ocs-share-api.html#update-share
By removing the publicUpload parameter from the PUT request, everything works fine and sets permissions 4 as expected. But, parameter in request payload is correct and should be handled.

@micbar Can you please let me know how we want to handle this? By overwriting the permissions parameter the publicUpload parameter behaves the same way as in oc10 but it's definitly confusing.

@micbar ping

@jesmrec publicUpload always overwrites the permissions. This is compatible with oc10.


ABAP Unit for ZCL_ABAPGIT_BRANCH_OVERVIEW

I would try to add ABAP Unit tests for https://github.com/larshp/abapGit/blob/master/src/zcl_abapgit_branch_overview.clas.abap like described here: http://docs.abapgit.org/development/tests.html
so that this issues are covered:
https://github.com/larshp/abapGit/issues/1381
https://github.com/larshp/abapGit/issues/1379

simple unit test for method PARSE_COMMITS added in https://github.com/larshp/abapGit/pull/1698

its an ongoing task to do unit tests, don't think we should keep this open, closing


Forms and back button error

Error occurs when i submit form and then press back button.
TypeError: page.cache[page.href] is undefined
There are condition that prevents to store page cache for forms. I think it cause this error.
I fix it by simple check 0491fe455d25d752388bfdd053ec0206fd967903
But maybe here is better solution?

I had the same problem: After submitting a form, the back button was broken.
This fix absolutely solved the problem and should make it into the core!
Did you already Add a pull request for your code?

Yes, this one worked for me to! Can you request a pull?

Any data why this PR hasn't been accepted?


Publish - must specify a draft path

When I run:
bundle exec jekyll publish

it returns:
jekyll 3.7.3 | Error:  You must specify a draft path.
However, running:
bundle exec jekyll draft "Test Post"

works find and creates the file in _drafts

You need to specify the draft you want to want to the _posts folder to publish it.
This command basically moves a file from the _drafts folder to the _posts folder.


Cannot set the records per page in a record list block

Describe the bug
I cannot change the number of records per page in a record list. It's always 20.
To Reproduce

Open a page
Add a record list block
Select a module with records
Set the "records per page value" to 10

Hit save and visit the saved page. You can see it's not 10, but the default 20.
If you go back to edit the record list you can also see that the value is not the 10 inserted, but that it's 20 again.

Fixed in https://github.com/cortezaproject/corteza-js/commit/c66c928280c2b01de6a7da9487ae02dc595c8287


Find out what is wrong with the contract

WHY
When implementing call_function I was using a status message contract from the https://examples.near.org/rust-status-message example (this one https://github.com/near-examples/rust-status-message/blob/master/res/status_message.wasm)
It was deployed successfully on my test account yellowpie.testnet.
But calling methods of this contract returned errors like:
on set_status:
wasm execution failed with error: FunctionCallError(HostError(ProhibitedInView { method_name: \"attached_deposit\" }))

on get_status
"wasm execution failed with error: FunctionCallError(HostError(GuestPanic { panic_msg: \"panicked at 'Cannot deserialize the contract state.: Custom { kind: InvalidData, error: \\\"Not all bytes read\\\

The error was the same when I did calls with Elixir API and with the cURL requests.
WHAT
Find out what I did wrong while deploying the contract, maybe lack of permissions.

issue was in the contract corrupted state.
well, not really corrupted - I forgot to reset account when deployed new contract over the old one.

issue was in the contract corrupted state.
well, not really corrupted - I forgot to reset account when deployed new contract over the old one.


runtime: flexible layering configuration.

To support TDS and to provide operator flexibility in how it relates to
existing runtime layering, this PR introduces support for explicit
control over layering. This eliminates the distinguished role of
subdirectory and override_subdirectory, for example, in favor of
arbitrary disk overlay layers.
This PR deprecates the existing runtime configuration in favor of the
new layered variant.
As a bonus, the PR fixes some of the runtime stats and related docs.
Risk level: Low
Testing: various unit tests (including server_test) additions.
Part of #6708.
Signed-off-by: Harvey Tuch<EMAIL_ADDRESS>

Will merge given two LGTMs, please leave further feedback and I'll do a followup PR.


updateSite API method not working

Describe the bug
The updateSite API method doesn't seem to work. It works perfectly when calling the API directly, but doesn't seem to make the required changes when making the change through CLI.
Tested in cases:
https://netlify.zendesk.com/agent/tickets/64498 (INTERNAL-ONLY LINK)
https://answers.netlify.com/t/updating-a-sites-custom-domain-from-the-cli-does-not-work/43578
To Reproduce
Steps to reproduce the behavior:

Create a Netlify website and get its API ID.
Run the following command:

netlify api updateSite --data '{ "site_id": "id-from-above", "name": "footer" }'
Configuration
  System:
    OS: macOS 11.5.2
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 166.34 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.8.0 - /usr/local/bin/node
    npm: 7.21.0 - /usr/local/bin/npm

Expected behavior
The command should actually update the site's settings.
CLI Output
It logs the output from the API returned data.

Hi @Hrishikesh-K,
The correct syntax is netlify api updateSite --data '{ "site_id": "id-from-above", "body:" { "name": "footer" } }'. Does this work for you?

Thank you for suggesting that. That's it, it works!


Added table/options.WithCallOptions options for append custom grp…

…c call options into session.{BulkUpsert,Execute,StreamExecuteScanQuery}
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Patch coverage: 11.86% and project coverage change: -18.39 :warning:

Comparison is base (52ea1bb) 64.73% compared to head (5d14280) 46.34%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #691       +/-   ##
===========================================
- Coverage   64.73%   46.34%   -18.39%
===========================================
  Files         228      154       -74
  Lines       22246    16077     -6169
===========================================
- Hits        14401     7451     -6950
- Misses       7075     8296     +1221
+ Partials      770      330      -440

Flag
Coverage Δ

?

1.17.x
46.16% <11.86%> (-18.30%)
:arrow_down:

1.19.x
46.36% <11.86%> (-18.10%)
:arrow_down:

1.20.x
46.36% <11.86%> (-18.14%)
:arrow_down:

integration
?

macOS
46.28% <11.86%> (-0.06%)
:arrow_down:

secure
?

ubuntu
46.28% <11.86%> (-0.06%)
:arrow_down:

unit
46.34% <11.86%> (-0.10%)
:arrow_down:

windows
46.10% <11.86%> (-0.32%)
:arrow_down:

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

Impacted Files
Coverage Δ

internal/table/statement.go
0.00% <0.00%> (-75.56%)
:arrow_down:

internal/table/transaction.go
50.36% <0.00%> (-13.53%)
:arrow_down:

table/options/options.go
37.06% <0.00%> (-22.77%)
:arrow_down:

table/table.go
24.59% <ø> (-52.41%)
:arrow_down:

internal/table/session.go
44.82% <31.81%> (-36.11%)
:arrow_down:

... and 168 files with indirect coverage changes
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 in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix compilation error causes when thrust is enabled

Build fails during cythonizing.
cupy/core/include/cupy/complex/cpow.h(22): error: identifier "y" is undefined

Oops, sorry for my mistake...😔

Jenkins, test this please

Related: #2579

Jenkins CI test (for commit d1645304724de24da767636d7a17bdd733ecdd3a, target branch master) succeeded!


Can't open storage accounts

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.21.3
Regression From
No response
Architecture
x64
Storage Explorer Build Number
No response
Platform
Windows
OS Version
Windows10
Bug Description
Error message whenever I am trying to open storage Accounts
Unable to retrieve child resources.
Details:
{
"name": "FetchError",
"message": "request to https://management.azure.com/subscriptions/136ecfc6-bf51-4274-b9bb-f56d5bd2f99e/resourcegroups?api-version=2021-01-01 failed, reason: write EPROTO<PHONE_NUMBER>:error:100000f7:SSL routines:OPENSSL_internal:WRONG_VERSION_NUMBER:../../third_party/boringssl/src/ssl/tls_record.cc:242:\n",
"stack": "FetchError: request to https://management.azure.com/subscriptions/136ecfc6-bf51-4274-b9bb-f56d5bd2f99e/resourcegroups?api-version=2021-01-01 failed, reason: write EPROTO<PHONE_NUMBER>:error:100000f7:SSL routines:OPENSSL_internal:WRONG_VERSION_NUMBER:../../third_party/boringssl/src/ssl/tls_record.cc:242:\n\n    at ClientRequest. (C:\Users\jose.felipe\AppData\Local\Programs\Microsoft Azure Storage Explorer\resources\app\node_modules\node-fetch\lib\index.js:1483:11)\n    at ClientRequest.emit (events.js:315:20)\n    at TLSSocket.socketErrorListener (_http_client.js:469:9)\n    at TLSSocket.emit (events.js:315:20)\n    at emitErrorNT (internal/streams/destroy.js:106:8)\n    at emitErrorCloseNT (internal/streams/destroy.js:74:3)\n    at processTicksAndRejections (internal/process/task_queues.js:80:21)",
"type": "system",
"errno": "EPROTO",
"code": "EPROTO"
}
Steps to Reproduce

Launch Azure Storage Explorer
Expand Storage Accounts

Actual Experience
An error message is being displayed and no storage accounts are listed.
Expected Experience
Show my 2 storage accounts
Additional Context
No response

@felipearaujokantar can you give system proxy a try? Read more about it here: https://docs.microsoft.com/en-us/azure/storage/common/storage-explorer-network#use-system-proxy-preview Please let us know if it does not solve this problem. Thanks!

Closing due to lack of response. Please give system proxy a try. If you continue having problems after trying to use it, please open a new issue. Thanks.


System clock not synchronized

Description
Hello, in the prod environment all flatcar cars do not have the clock synchronized via NTP server, instead in staging machines with flatcar have active synchronization with NTP servers
Machines re-emerge on Azure on two different resource groups but configured in the same way, the only thing that changes is the amount of machines.
In prod not having the timing synchronized I cause problems with JWT tokens since I’m ahead even a few minutes sometimes
Prod not sync clock

Staging sync clock

Hi, can you compare and paste the output of sudo timedatectl show-timesync --all and sudo systemctl status systemd-timesyncd on both machines. Is networking configured in the same way for both environments? Are you sure both VMs can access the NTP servers?

and also sudo timedatectl timesync-status.

Hello,
the network configuration is configured in the same way on both Azure resource groups but on the prod environment when trying to contact an NTP server goes in timeout
here the screen of the 3 commands indicated
Prod screen

Staging screen

That's weird - can you try pinging that ntp server (<IP_ADDRESS>) from prod to see if its a connectivity issue?
You can try a different ntp config for the prod env. Replace the contents of /etc/systemd/timesyncd.conf with:
[Time]
NTP=pool.ntp.org

and then reboot or systemctl restart systemd-timesync

Hello,
ping in both prod and staging environments does not work even if it is all configured correctly
Also even after editing the file and restarting systemd-timesyncd continues to timeout

Should I try to contact Microsoft for information about the network configuration?

Yes please - that would probably be the best course of action.

I'll close this, since it seems like an Azure networking issue.


Image load event not firing when cache is warm

It seems when the image exists in cache, meaning an image view has already loaded it at some point prior in the app, the load event does not fire on subsequent ImageViews.

@dested What platform are you seeing this behavior on?

Android
#849  It seems to be this issue.

The issue is indeed the same as described in #849.
The problem is that when the image comes from the cache it is applied instantly when the ImageView is created. The listener to notify of the load event is not registered at this point.
We will make an Android client side adjustment to make sure that the image is only set after the initial listener is registered.

Fixed in the 2.0 stream.


Problems when I parsing Chineses PDF documents

Hi, When I use partition_pdf(file=io.BytesIO(file.file.read()), languages=["chi_sim"]) to parse Chinese pdf documents, I found that the result was  to split the paragraph text into a line text as a element. And anothre problem is element.type isn't accurate, should be "UncategorizedText" but actually is "Title".

ok,let me give an example.I will give the two documents,One is the raw pdf file,another is the json which i use the code below to get.The bug is some elements should be Title,But it is UncategorizedText in the result. Also, the result shows that some paragraphs can't recognized ,you can see the json,a line in paragraphs is recognized as a element.So a paragraphs is split into many elements.I think it is not a good result.
Hope your reply,Thanks!!!
import json
from typing import Iterable, Optional

from unstructured.documents.elements import Element
from unstructured.partition.pdf import partition_pdf
from unstructured.staging.base import elements_to_json, _fix_metadata_field_precision, elements_to_dicts

elements = partition_pdf(filename=r"C:\Users\A\Desktop\test.pdf",
                         languages=["chi_sim"])  # bytes -> BinaryIO

def elements_to_json_chi(
        elements: Iterable[Element],
        filename: Optional[str] = None,
        indent: int = 4,
        encoding: str = "utf-8",
) -> Optional[str]:
    """Saves a list of elements to a JSON file if filename is specified.

    Otherwise, return the list of elements as a string.
    """
    # -- serialize `elements` as a JSON array (str) --
    precision_adjusted_elements = _fix_metadata_field_precision(elements)
    element_dicts = elements_to_dicts(precision_adjusted_elements)
    json_str = json.dumps(element_dicts, ensure_ascii=False, indent=indent, sort_keys=True)

    if filename is not None:
        with open(filename, "w", encoding=encoding) as f:
            f.write(json_str)
        return None

    return json_str

elements_to_json_chi(elements, filename="./test_json.json")

also,it shows that in package unstructured.staging.base the func elements_to_json has some encoding bugs in chinese.The parameter ensure_ascii in json.dump,I think shoule be false.
test.pdf
test_json.json

Thank you for the example! We're tracking this and will investigate as soon as we can.

ok，thanks for your help!!!


CSS: Animated Resizing of .panelContents when Window Size is less than 768 pixels.

https://github.com/webgpu/webgpu-samples/assets/62450112/fe4619d4-deec-481b-8475-136e2a32c56e
In order for this animation to be performant, I replaced the React state update with direct updates to the DOM based on window resizing and clicks, which prevents the state-triggered re-render that would stall the animation/transition. Since I'm unsure whether updating the DOM in this manner would be considered a "brittle" or "unidiomatic" practice, and since it modifies a longstanding component of this repository's code, I've tried to do my best to make the change as simple and unobtrusive as possible, as well as adding comments to the new window resize listener and stylePanel functions.

indeed it is risky to muck directly with the DOM because it could interfere with React. Would a pure CSS solution be possible?
haven't tested it, but perhaps something like:
.panel .panelContents {
   display: block;
   transition: max-height 0.25s ease-in;
 }

.panel[data-expanded=false] .panelContents {
    max-height: 0vh;
}

.panel[data-expanded=true] .panelContents {
  max-height: 100vh;
}

indeed it is risky to muck directly with the DOM because it could interfere with React. Would a pure CSS solution be possible?
haven't tested it, but perhaps something like:
.panel .panelContents {
   display: block;
   transition: max-height 0.25s ease-in;
 }

.panel[data-expanded=false] .panelContents {
    max-height: 0vh;
}

.panel[data-expanded=true] .panelContents {
  max-height: 100vh;
}

This was similar to a solution I tried earlier, using the current data-expanded state object to trigger the transition. However, the issue with that approach was that the state change seemed to trigger a re-render that made the animation imperceptible. I do agree with you that most of this can probably be changed to css, perhaps with the vanilla Javascript only containing a call to setAttribute for data-expanded rather than relying on React state. I'll work on that.

Should be good now, I cut down the Javascript to just setting the attribute on the panel ref.

Just to reiterate, setAttribute is called rather than tying data-expanded to a React state variable because setting a state variable causes the entire component to re-render, including the canvas that's displaying the WebGPU code. You can actually see this problem on the current version of the website, where clicking the expand button on the cornell sample will stall the webpage for several seconds.

Sorry for taking a while to look at this again.
I don't feel comfortable mixing the DOM attribute manipulations while also using React here. Not an expert on this at all, but I think React provides facilities to say that things should not rerender. I believe React.memo helps with this.

Sorry for taking a while to look at this again.
I don't feel comfortable mixing the DOM attribute manipulations while also using React here. Not an expert on this at all, but I think React provides facilities to say that things should not rerender. I believe React.memo helps with this.

Not sure if this is good practice, but I figured memoizing the component was the easiest way to fix the issue.

Nice! I think it's great. I verified with the React devtools profiler that we're not seeing any rerenders of that Component now

Glad to hear, think I just overcomplicated it earlier😅, but I think it's a good visual/usability fix.


XCode 12.5 Carthage build fails apparently due missing Info.plist

Goals
Update to current Realm via Carthage
carthage update --use-xcframeworks --verbose
Expected Results
Realm frameworks updated and built, able to build my app and iMessage extension using them.
Actual Results
*** Fetching realm-cocoa
*** Checking out realm-cocoa at "39930fe2a8c4a207cb832ce8cf156f2e25db4f6a"
*** Building scheme "Realm" in Realm.xcworkspace
/usr/bin/xcrun xcodebuild -workspace /Volumes/EXSANT1A/justme/dev/myapp/Carthage/Checkouts/realm-cocoa/Carthage/Realm.xcworkspace -scheme Realm -configuration Release -derivedDataPath /Volumes/EXSANT1A/justme/Library/Caches/org.carthage.CarthageKit/DerivedData/12.5_12E262/realm-cocoa/39930fe2a8c4a207cb832ce8cf156f2e25db4f6a -sdk iphoneos ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES archive VALIDATE_WORKSPACE=NO -archivePath /var/folders/jz/qqrj75nd1sv76c_wc9rzr15h0000gs/T/realm-cocoa SKIP_INSTALL=YES GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO CLANG_ENABLE_CODE_COVERAGE=NO STRIP_INSTALLED_PRODUCT=NO (launched in /Volumes/EXSANT1A/justme/dev/myapp/Carthage/Checkouts/realm-cocoa)Command line invocation:
    /Applications/Xcode.app/Contents/Developer/usr/bin/xcodebuild -workspace /Volumes/EXSANT1A/justme/dev/myapp/Carthage/Checkouts/realm-cocoa/Carthage/Realm.xcworkspace -scheme Realm -configuration Release -derivedDataPath /Volumes/EXSANT1A/justme/Library/Caches/org.carthage.CarthageKit/DerivedData/12.5_12E262/realm-cocoa/39930fe2a8c4a207cb832ce8cf156f2e25db4f6a -sdk iphoneos ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES archive VALIDATE_WORKSPACE=NO -archivePath /var/folders/jz/qqrj75nd1sv76c_wc9rzr15h0000gs/T/realm-cocoa SKIP_INSTALL=YES GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO CLANG_ENABLE_CODE_COVERAGE=NO STRIP_INSTALLED_PRODUCT=NO

User defaults from command line:
    IDEArchivePathOverride = /var/folders/jz/qqrj75nd1sv76c_wc9rzr15h0000gs/T/realm-cocoa
    IDEDerivedDataPathOverride = /Volumes/EXSANT1A/justme/Library/Caches/org.carthage.CarthageKit/DerivedData/12.5_12E262/realm-cocoa/39930fe2a8c4a207cb832ce8cf156f2e25db4f6a
    IDEPackageSupportUseBuiltinSCM = YES

Build settings from command line:
    CARTHAGE = YES
    CLANG_ENABLE_CODE_COVERAGE = NO
    CODE_SIGN_IDENTITY =
    CODE_SIGNING_REQUIRED = NO
    GCC_INSTRUMENT_PROGRAM_FLOW_ARCS = NO
    ONLY_ACTIVE_ARCH = NO
    SDKROOT = iphoneos14.5
    SKIP_INSTALL = YES
    STRIP_INSTALLED_PRODUCT = NO
    VALIDATE_WORKSPACE = NO

Version 10.7.2 already present
note: Using new build system
note: Using codesigning identity override:
note: Planning build
note: Analyzing workspace
note: Constructing build description
note: Build preparation complete
error: There is no Info.plist found at '/Volumes/EXSANT1A/justme/dev/myapp/Carthage/Checkouts/realm-cocoa/core/realm-monorepo.xcframework/Info.plist'. (in target 'Realm' from project 'Realm')

** ARCHIVE FAILED **

Build Failed
	Task failed with exit code 65:
	/usr/bin/xcrun xcodebuild -workspace /Volumes/EXSANT1A/justme/dev/myapp/Carthage/Checkouts/realm-cocoa/Carthage/Realm.xcworkspace -scheme Realm -configuration Release -derivedDataPath /Volumes/EXSANT1A/justme/Library/Caches/org.carthage.CarthageKit/DerivedData/12.5_12E262/realm-cocoa/39930fe2a8c4a207cb832ce8cf156f2e25db4f6a -sdk iphoneos ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES archive VALIDATE_WORKSPACE=NO -archivePath /var/folders/jz/qqrj75nd1sv76c_wc9rzr15h0000gs/T/realm-cocoa SKIP_INSTALL=YES GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO CLANG_ENABLE_CODE_COVERAGE=NO STRIP_INSTALLED_PRODUCT=NO (launched in /Volumes/EXSANT1A/justme/dev/myapp/Carthage/Checkouts/realm-cocoa)

This usually indicates that project itself failed to compile.

Steps for others to Reproduce
Cartfile contains
github "https://github.com/realm/realm-cocoa" "master"
carthage update --use-xcframeworks --verbose
Note that my home directory is on an external drive /Volumes/EXSANT1A
Code Sample
n/a
Version of Realm and Tooling
ProductName:	macOS
ProductVersion:	11.3
BuildVersion:	20E232

/Applications/Xcode.app/Contents/Developer
Xcode 12.5
Build version 12E262

/usr/local/bin/pod
1.9.3
(not in use here)

/bin/bash
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin20)

/usr/local/bin/carthage
0.38.0
(not in use here)

/usr/local/bin/git
git version 2.23.0

Nuking the entire Carthage folder resulted in a successful build.


can not run on Mageia Linux

Hello,
I have Mageia Linux x86_64 with RTX 3060 6GM V-RAM.
First I must install tk via pip3, because there is no generic packages in Mageia's repo.
OK, but when I install the requirements can not run the app:
$ python3 UVR.py
2023-01-21 00:20:05,154 - UVR BEGIN
No splash screen.
Traceback (most recent call last):
  File "UVR.py", line 650, in <module>
    class MainWindow(TkinterDnD.Tk if is_dnd_compatible else tk.Tk):
  File "UVR.py", line 1623, in MainWindow
    def cached_source_model_list_check(self, model_list: list[ModelData]):
TypeError: 'type' object is not subscriptable

Looks like there is missing the tkinterdnd package. Installing via pip3 fails:
$ pip3 install tkinterDnD
/usr/lib/python3.8/site-packages/secretstorage/dhcrypto.py:16: CryptographyDeprecationWarning: int_from_bytes is deprecated, use int.from_bytes instead
  from cryptography.utils import int_from_bytes
/usr/lib/python3.8/site-packages/secretstorage/util.py:25: CryptographyDeprecationWarning: int_from_bytes is deprecated, use int.from_bytes instead
  from cryptography.utils import int_from_bytes
Defaulting to user installation because normal site-packages is not writeable
Collecting tkinterDnD
  Downloading tkinterDnD-0.0.0.tar.gz (1.9 kB)
  Preparing metadata (setup.py) ... done
ERROR: Could not find a version that satisfies the requirement tkinter (from tkinterdnd) (from versions: none)
ERROR: No matching distribution found for tkinter

I have already installed tkinter3-3.8.14-1.1.mga8.x86_64
Any suggestions?
Best regards,
Rossen

I found that tkinterDnD2 work fine with python2 and 3, installed, but I see the same error message

I will not migrate to Ubuntu :/

Thank you @Heersin
This fix works in Mageia


FIxing AOT

Codecov Report

Merging #368 into master will decrease coverage by <.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #368      +/-   ##
==========================================
- Coverage   64.38%   64.37%   -0.01%
==========================================
  Files          14       13       -1
  Lines       29496    29491       -5
  Branches      139      139
==========================================
- Hits        18990    18985       -5
  Misses      10431    10431
  Partials       75       75

Impacted Files
Coverage Δ

modules/core/constants.ts
100% <ø> (ø)

modules/core/accessor.ts
81.81% <ø> (ø)

modules/components/tag-input.ts
73.63% <50%> (ø)
:arrow_up:

spec-bundle.js
64.18% <0%> (-0.01%)
:arrow_down:

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 9140e1d...b3787e2. Read the comment docs.


Be able to make the distinction between original case and rectified case

Description

As a user, I want to see which cases were rectified.

Scope of Work

[x] Be able to process text files
[x] Be able to detect whether a case had multiple hearings
[x] Be able to make the distinction between the original case and a rectified case with their different "No dossier"
[x] Be able to parse a user-defined amount of randomly selected cases
[x] Save extracted data in JSON format

Acceptance Criteria

The user is able to see which cases had multiple hearings

Demo requirement

The user chooses a set amount of cases to randomly verify if actually rectified.
The system provides the set amount of cases and clearly identifies which ones have been rectified with the case's title (textfile title) and its rectification status, "No dossier" (both rectified and original) as well as "No demande"

Story Points

20

Priority

High

Risk

High

Documentation, files, diagrams, etc. relating to this feature:
Url, books, etc.
Changes:

[Optional]

The demo requirement will show that the functionality is working. The implementation is different: the system should, as a general rule, not rely on data from cases that have been appealed, since a higher court could be saying the literal opposite of the lower court ruling.
The basic user doesn't need to know these distinctions; just what's valid caselaw at the end of the day.
Unverified, priority high.

Our hypothesis was that, since a higher court's decision overrules a lower one, that the information found in the appeal is a more accurate representation of the case. As a prerequisite to using the appropriate data (i.e. the fact as it is described within the appeal, which contradicts the fact as it is described within the original case), we must be able to distinguish between the various "phases" of a claim.
This ability to distinguish is described in the current user story.

In light of @arekmano comment, verified.

Great, this is done. Create new user story for appeals, etc. once data is available.
List of all situations where a decision could be overthrown: https://www.rdl.gouv.qc.ca/fr/decision/contestation-d-une-decision


question

i checked up o n this and i found no aswser to my question where is the code wich connects the client to the server i just need the code that conects the client now is this client sided im not sure so please dont
......................................................................................................................................................................................................................................................................................................................................................................................................................................................

that is client sided, that put on client no on Server

Duplicate and client sided.
Please stop asking these type of issues..


Lists - unreadable black elements in Dark mode

Hello,
It seems the Dark mode is not fully readable with some HTML elements. As an example, various kind of lists (Definition list, Option list, Field list, etc.) are rendered with a black main item in Dark mode. See an example here: https://sphinx-themes.org/sample-sites/piccolo-theme/kitchen-sink/lists/#field-list

@alorence Thanks - good catch!
I'll add an extra style for dt tags.

I've pushed the fixes to PyPI - hopefully should fix the problem.
I think sphinx-themes.org rebuilds daily, so hopefully will appear on there soon.
Thanks again 👍

Thanks ! I will try as soon as possible ;)
As far as I know, sphinx-theme rebuilds weekly

This sample documentation was generated on Mar 07, 2022, and is rebuilt weekly.

From https://sphinx-themes.org/sample-sites/piccolo-theme/
I think the whangs will be visible next monday


Expire the session cookie if there previously is one (with valid token), but the data container is empty

The session cookie should be forcefully removed if:

the client sent it previously
the data in it (at the end of the request) is empty

Closed by #11


Fixed gulp once and for all

This fixes the final issue in the gulpfile, a missing return

This Pull Request has been merged by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

4
0
0
4

Vote Breakdown

User
Yes
No
Abstain

ags131
✔

lurr
✔

daboross
✔

tedivm
✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
True


fix(docs): kustomize build folder of uat & prod

Currently, when make a promotion from test to uat or uat to prod, the kustomize still build the manifest of test stage, it makes the application cannot start because of the same nodePort, and the html content is not correct

Can you check it? @krancour


Table of contents not shown

Informations
Web related issue, so qiskit and python versions or operating system don't give more info
What is the current behavior?
Although "Table of Contents" appears on the right sidebar, it doesn't appear after "About Qiskit Experiments" section

Steps to reproduce the problem
Go to: https://qiskit.org/documentation/experiments/index.html and compare the right sidebar panel sections with the sections on the page
What is the expected behavior?
Both sections should appear, not only the first one

Hey @coruscating !! could you start working on this? do you need help? :)

This has been fixed in #983 and is live now: https://qiskit.org/documentation/experiments/dev


Delete all records option

So far I've been loving using pocketbase and all the features it offers. However, it would be nice for there to be deleteAll() option for removing all records in a DB.

See https://github.com/pocketbase/pocketbase/issues/2238. It exists in the roadmap but remains a very low priority.


Provide documentation and examples for how to install and use with non-npm normal html and vanilla systems.

When i tried to do what you did in your tutorial it did not work. The code did not highlight, there was no way to make a new line. I therefore want to switch to using this, but I am unsure of 2 things:

Can i use hljs (highlight js) and if so how can i use a custom theme and use a downloaded version.
How do i use this in a normal webpage?

Any tutorials and/or answers are much appreciated.

@WhyFenceCode Thank you for reaching out here! I have put together a demo HTML file on CodePen here. I should probably do the same for Prism.js and put them in the README, so please leave this issue open, but do let me know if this helps.

Thank you so much, that helps a lot. You have just solved so many problems with a code editor I am writing. Do you want to be credited when I release @WebCoder49 ?

Also, I am unable to make it scale to 100% width without overflow, and I can not remove margins/padding

Thank you so much, that helps a lot. You have just solved so many problems with a code editor I am writing. Do you want to be credited when I release @WebCoder49 ?

Yes please, that would be nice! 😊 If it is OK, please could you credit me as the maintainer of code-input but also provide a link to https://github.com/webCoder49/code-input/graphs/contributors?

Thank you so much, just an idea to fix this, make sure that your style sheet runs first, so that the others will override it. I might pr this when I have time

Also, how do i download the files, and are all of them needed as I dont have npm or any package manager for my editor.

Also, how do i download the files, and are all of them needed as I dont have npm or any package manager for my editor.

You can download a ZIP file under the green Code button on the main repository page, and extract its contents. I'd suggest keeping the README, JSON and d.ts files there as they will make it easier to remember where you got the code-input package from, work out its version number if you report later bugs, and give you a better editing experience. The code-input LICENSE file must be left and distributed with your software, as a legal requirement.
Out of the other files, having both .min.js and .js, .min.css and .css versions are useful as .min should be used in production and non-minified in testing. However, you can delete the tests and .github folders, and delete any plugins in the plugins folder you are not using. This will help make your repository smaller in size, but is not necessary.
This won't activate NPM or anything, as you are downloading the files from GitHub, not NPM (which is an alternative). I hope this clarifies things!

Thank You For ALL YOUR HELP

One last thing, when I use this it scales in an odd way (the entire page moves left) any ideas why? Using both width and height mess it up. Ill post my code:
https://github.com/WhyFenceCode/verbatim_code_editor
The styles applied are in editor.css

fixed everything, thank you so much for all your help, im gonna close this and open one to remind you about docs


How do you handle the ignore region in the dataset? I have not found relevant code in this repo.

I trained centerNet on UA-detrac, without considering the ignore region. The trained model can not find small vehicle.
So I think the ignore region is important.
Thx!

Hi! Personally, I don't consider the ignore region during training. I know that it probably penalizes the network for some incorrect "false positives", real detected vehicles that are not in the ground-truth, but I think the impact should be minimal. If you try it, let me know if it improves the performance!

Thx !
On UA-detrac training+ testing set, my classmate trains centerNet, which trained by the author on coco,  and it performs worse than before on small object obviously.  I think the impact is important.
I' ll consider the ignore region and share my results with you.

The code of handling the ignore region can be reffered  from  (an ECCV paper' s work):
https://github.com/shijieS/DMMN/blob/d9ef345f1cb9367aea36fe46d3299ee1f8677e3b/dataset/ua/ua_reader_backup.py#L111


(maint) Rewrite AppVeyor test_script as PowerShell

Previously test_script was a combination of individual batch
commands and PowerShell code.  Unify the code as PowerShell given
PowerShell is already being used for Get-WinSystemLocale.
Be more explicit about exiting the script to prevent spurious
AppVeyor failures that look like:
Command exited with code 1

CLA signed by all contributors.

Ok @adrienthebo you shouldn't see those AppVeyor failures anymore now -- but in the event that you do for some reason, please let me know!

@Iristyle you're my hero, thank you!


Add more precise tag for function/class declaration names

Similar to https://github.com/codemirror/lang-python/commit/162be8f40aebb19334cf229d52e918e626a060ec.
I referenced the tests from lezer-parser/javascript, which has tests for FunctionDeclaration/VariableDefinition and ClassDeclaration/VariableDefinition.
For the pull request, I am assuming that for purposes of syntax highlighting that ClassDefinition/VariableName and FunctionDefinition/VariableName in python should give the same output as ClassDeclaration/VariableDefinition and FunctionDeclaration/VariableDefinition in javascript.
Even though in javascript there is both VariableDefinition and VariableName, while in python there's only VariableName. This is a slight deviation from codemirror 5 since both function definition names and function parameter names are given the same cm-def span.
The alternative for FunctionDeclaration/VariableDefinition, instead of
"FunctionDeclaration/VariableDefinition": t.function(t.definition(t.variableName))

can be
VariableDefinition: t.definition(t.variableName), // already exists
"FunctionDeclaration/VariableDefinition": t.function(t.definition(t.definition(t.variableName)))

The drawback for this is if you wanted to highlight function definition names in both python and javascript, you would have to have an additional tag:
  {tag: [tags.function(tags.definition(tags.variableName)), tags.function(tags.definition(tags.definition(tags.variableName)))],
   color: "#6f42c1"},

Applying a tag modifier twice is a no-op, so t.defintion(t.defintion(x)) equals t.definition(x). The patch looks good as it is.

Applying a tag modifier twice is a no-op, so t.defintion(t.defintion(x)) equals t.definition(x). The patch looks good as it is.


Backup phrase

[x] Text changes "Enter the word exactly like it was displayed"

[x] Please change from all 3 places (2 on Backup phrase confirmation screen and Health check screen )

[x] Typo error

Completed 2.2.10 (615)


Use different Optimizers for two branches of a network. Branches use each other to output results when finally calculating Loss

Click to expand!
Issue Type
Support
Source
binary
Tensorflow Version
tf1.11
Custom Code
No
OS Platform and Distribution
No response
Mobile device
No response
Python version
3.5
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
I want to calculate the KLD loss using the output of the two branches, and there will always be an error where one of the results is empty.

Standalone code to reproduce the issue
self.teacher_loss = self.get_teacher_loss()   #  branch 1 loss
 self.stu_loss = self.get_stu_loss()            # branch 2 loss

 with tf.variable_scope('teacher_optimizer'):   #optimizer 1
    ....
 with tf.variable_scope('stu_optimizer'):       #optimizer 2
    ...

Relevant log output
No response

Thank you! I try

@NiuYingchun Could you please let us know if you have tried to upgrade to the latest TF version?
Thank you!

@NiuYingchun Could you please let us know if you have tried to upgrade to the latest TF version? Thank you!

Thank you, I'm trying. I haven't finished it yet

@NiuYingchun Older versions of TF(1.x) are not actively supported. Please upgrade to latest TF version and let us know if the issue still persists in newer versions? Thank you!

Thank you! It worked in TF2

@NiuYingchun Glad it worked for you!
Thank you for the update. Could you please move this issue to closed status if it is resolved for you?
Thank you!


Add telemetry datapoint for cell execution elapsed time.

We need this for a couple reasons:

We are considering using this information to decide whether to users may benefit from Azure usage.
To get a better understanding of the impact debugging has on cell execution once cell debugging is shipped.

Dupe of #6283
Going to add this in that item anyway.


fix(deps): update dependency cookie to v0.4.1

This PR contains the following updates:

Package
Type
Update
Change

cookie
dependencies
patch
0.4.0 -> 0.4.1

Release Notes

jshttp/cookie
v0.4.1
Compare Source
==================

Fix maxAge option to reject invalid values

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

:tada: This PR is included in version 5.0.36 :tada:
The release is available on:

v5.0.36
GitHub release

Your semantic-release bot :package::rocket:


Add suppression of elements in disabled operation

If an operation is disabled you can opt to suppress the display of contained elements in the scene:

This option is also available in the operation property:

This option is only available if the operation has been disabled, active operations will always be displayed.

Is that icon a ghost? Rather than an eyeball?


UML StateChart error when 2 states have the same name in different regions

The mane of the state is used as the ID so it does not go well when we have state with the same name in two different regions (or probably composite state). We should probably use a qualified name to avoid this.

Well, it seems name == ID in plantuml, so that we will have to qualified name as the name for the state (which will show up in the diagrams). We can try to generate the shortest possible qualified that is enough to make sure names are unique. In most cases, it should not change anything (the shortest qname being the name if name is unique).

Note that I just use indexes (e.g., MyState_2) instead of qualified names that can rapidly become long


Delegate removal should not require proper contiguous sublist

The spec says the following about delegate removal:

If the first operand is null, the result of the operation is null. Otherwise, if the second operand is null, then the result of the operation is the value of the first operand. Otherwise, both operands represent invocation lists (§20.2) having one or more entries, and the result is a new invocation list consisting of the first operand's list with the second operand's entries removed from it, provided the second operand’s list is a proper contiguous sublist of the first's. (To determine sublist equality, corresponding entries are compared as for the delegate equality operator (§12.11.9).) Otherwise, the result is the value of the left operand. Neither of the operands' lists is changed in the process. If the second operand's list matches multiple sublists of contiguous entries in the first operand's list, the right-most matching sublist of contiguous entries is removed. If removal results in an empty list, the result is null.

My problem is with the phrase "proper contiguous sublist", specifically the "proper" part. As I understand it (and correct me if I'm wrong, since I'm not a native English speaker), this would imply that if the two lists are the same, this condition is not satisfied and the result should be the left operand. This is clearly nonsense (especially since it would mean that the last sentence about null would never apply) and it's not what the compiler does.
So, should the word "proper" be removed from the paragraph above?

We'll try to discuss this next week.

Thanks @svick.
In the very early days of TG2, I worked on the words in this subclause and provided the example. On reading them now, here are my observations:

According to Wiktionary, the term proper when used in the context of proper subset excludes the possibility if one list being contained within another, identical list, so the use of proper seems to be inappropriate, or at least, misleading.
In the case of having two invocation lists, it is misleading to say that “the result is a new invocation list consisting of …” when, in fact, the result is null when the lists are the same. While we do say that near the end, that’s too far away from this initial claim to effectively negate it.
Rewriting the text as a series of (nested) bullets makes it easier to read.
Extending and rearranging the example to show all the options, in order of the corresponding bullets, reinforces the text.

Here then is my proposed replacement:

The semantics are as follows:

If the first operand is null, the result of the operation is null.
Otherwise, if the second operand is null, then the result of the operation is the value of the first operand.
Otherwise, both operands represent invocation lists (§20.2).

If the lists compare equal, as determined by the delegate equality operator (§12.11.9), the result of the operation is null.
Otherwise, the result of the operation is a new invocation list consisting of the first operand's list with the second operand's entries removed from it, provided the second operand’s list is a sublist of the first's. (To determine sublist equality, corresponding entries are compared as for the delegate equality operator.) If the second operand's list matches multiple sublists of contiguous entries in the first operand's list, the right-most matching sublist of contiguous entries is removed.
Otherwise, the result of the operation is the value of the left operand.

Neither of the operands' lists (if any) is changed in the process.

Example:
delegate void D(int x);
class C
{
	public static void M1(int i) { /* ... */ }
	public static void M2(int i) { /* ... */ }
}
class Test
{
   static void Main() {
      D cd1 = new D(C.M1);
      D cd2 = new D(C.M2);
      D cd3 = null;                   // null
      cd3 -= cd1;                     // => null
      cd3 = cd1 + cd2 + cd2 + cd1;    // M1 + M2 + M2 + M1
      cd3 -= null;                    // => M1 + M2 + M2 + M1
      cd3 = cd1 + cd2 + cd2 + cd1;    // M1 + M2 + M2 + M1
      cd3 -= cd1 + cd2 + cd2 + cd1;   // => null
      cd3 = cd1 + cd2 + cd2 + cd1;    // M1 + M2 + M2 + M1
      cd3 -= cd1 + cd2;               // => M2 + M1
      cd3 = cd1 + cd2 + cd2 + cd1;    // M1 + M2 + M2 + M1
      cd3 -= cd2 + cd2;               // => M1 + M1
      cd3 = cd1 + cd2 + cd2 + cd1;    // M1 + M2 + M2 + M1
      cd3 -= cd2 + cd1;               // => M1 + M2
      cd3 = cd1 + cd2 + cd2 + cd1;    // M1 + M2 + M2 + M1
      cd3 -= cd1;                     // => M1 + M2 + M2
      cd3 = cd1 + cd2 + cd2 + cd1;    // M1 + M2 + M2 + M1
      cd3 -= cd1 + cd1;               // => M1 + M2 + M2 + M1
   }
}

end example

@RexJaeschke Thanks for looking into this. I agree that your replacement is better, though I have one small nit:

the right-most matching sublist of contiguous entries is removed

I think it would be clearer to say "last matching sublist" instead of "right-most", since a list is naturally ordered first-to-last, but I would say ordering left-to-right is technically not specified. (Also, it may be confusing to people who use a right-to-left language, I'm not sure.)

Thanks, @svick. Putting on my eccentric implementer's hat, what if I search the list back-to-front, instead of front-to-back? In that case, the last-matching sublist could be at the front rather than at the back!
How about we say "positionally last matching sublist"?

@RexJaeschke Personally, I think that's unnecessary. You can search the list in whatever order you want, but that doesn't change the inherent order of a list, and I think it's clear that an unqualified "last" is talking about that order.
But I'm fine with the wording you suggest too.

Rex to create a PR.

Handled by PR #407.


feat[pages]: Warn users when _routes.json is ignored

A custom _routes.json file can be specified only by projects in Advanced
Mode. This commit ensures that if a user tries to publish a project,
not in Advanced Mode and with a custom _routes.json file, they will be
warned about the custom routes file being ignored.

Closing due to design changes. Thanks!


Feat: Adding the GSAP(JS LIBRARY) animations to the website

Describe the feature
So for this good website if animations are added then there will be an improvement in the UI of the website and also helps the user feel good
Add ScreenShots

all of them will be coming by sliding and the ui will be enhanced
Record

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

This will simply make the website much slower , I dont want that Sorry


Alphanumeric TEID field

Hello,
Tried to inlcude alphanumeric in TEID field of GTP as per standard but it only accepts 4 bytes integer.
Thanks

@pstav I don't think you can use alphanumeric chars in the TEID field. TEID is always a 4 byte integer, su basically you could translate it to a 4 char ASCII string, however, in the end it's still a 32 bit number.
https://en.wikipedia.org/wiki/GPRS_Tunnelling_Protocol#GTP_version_2


⚠ Accela has degraded performance

In 42bde20, Accela (https://aca-prod.accela.com/santaclara) experienced degraded performance:

HTTP code: 200
Response time: 22876 ms

Resolved: Accela performance has improved in 82c7da9.


correct index

todos.splice(index, 1) in destroy makes the elements shift; therefor loops index has to be aligned has to be corrected

@indus It worked as expected previously. Have you actually tested it?

I´ve checked it again and I still have to say it doesn´t work for me.
Maybe I should have left a note on the steps to reproduce:
You have to add two or more todos and check two or more successive before you try to clear them to see the problem.
if you start with the array [x, x, o] where x is flagged for clearing, you will jump over the second x
Loop Start
i = 0; [x, x, o]; => splice
i = 1; [x, o]; => no splice
Loop End

reversing the loop (as seen here) should also work
for (var i = this.list.length - 1; i >= 0; i--) {
		if (todos[i].completed) destroy(todos[i])
}

Ok. I'll merge the existing change. It's the simplest fix.


ARC for arbitrary verification logic instead of accounts

Based on https://eips.ethereum.org/EIPS/eip-4337 and using https://developer.algorand.org/docs/get-details/dapps/smart-contracts/#smart-signatures

Closing as https://loginid.io/ have a solution for this
Exemple https://testnet.algoexplorer.io/tx/DXM5HEMM232AYRBTVEHYQRTQ5AU65NOAQORXQ64MXM7QIGDK4N2A


Array not has findLastIndex function in Ts.

I know Ts is Js's super, but I find when i set a variable and set Array type, but it  not includes findLastIndex ,   findLast function, and i get a  error(Attribute "findLastIndex" not found on type "any[]" ).
Finally, I  set it's type is any only.

You should follow template which asks you to provide search terms you tried
Dulicate of #48829 can be find by searching for findLastIndex

@IllusionMH  I found a temporally solution for this problem, just create a global.d.ts file in anywhere of your project and type this code:
export {}

declare global {
  interface Array<T> {
    findLastIndex(
      predicate: (value: T, index: number, obj: T[]) => unknown,
      thisArg?: any
    ): number
  }
}

Ready! You can use this approach without wrongs from your typescript linter.


Gateway not loading

Hello looking to see how i can get past this error i am receiving. I left the IP address blank to use the defualt
Device:  xxxx Update home data error: Error: self-signed certificate. Reconnect in 15s.

What happened if you add correct IP adress? Please also enable debug mode in plugin config advanced section and get the log.

i still get the following if i put the IP address of the gateway. These are the logs i see after enabling degug mode:
7/12/2023, 1:55:54 PM] [enphaseEnvoy] Device:  Solar Gateway, Update info data error: Error: getaddrinfo ENOTFOUND envoy.local. Reconnect in 15s.
[7/12/2023, 1:56:09 PM] [enphaseEnvoy] Device:  Solar Gateway, Update home data error: Error: self-signed certificate. Reconnect in 15s.

Here is my config {
"devices": [
{
"name": "Solar Gateway",
"envoyFirmware7xx": false,
"envoyPasswd": "xxxx",
"disableLogInfo": false,
"disableLogDeviceInfo": false,
"enableDebugMode": true,
"powerProductionMax": false,
"powerProductionMaxAutoReset": 0,
"powerProductionMaxDetected": 5400,
"energyProductionLifetimeOffset": 0,
"powerConsumptionTotalMax": false,
"powerConsumptionTotalMaxAutoReset": 0,
"powerConsumptionTotalMaxDetected": 10000,
"energyConsumptionTotalLifetimeOffset": 0,
"powerConsumptionNetMax": false,
"powerConsumptionNetMaxAutoReset": 0,
"powerConsumptionNetMaxDetected": 10000,
"energyConsumptionNetLifetimeOffset": 0,
"supportProductionPowerMode": false,
"supportPlcLevel": false,
"supportEnsembleStatus": false,
"supportLiveData": false,
"liveDataRefreshTime": 1000,
"metersDataRefreshTime": 3000,
"productionDataRefreshTime": 5000,
"enableRestFul": false,
"restFulPort": 3000,
"enableMqtt": false,
"mqttDebug": false,
"mqttHost": "xxx",
"mqttPort": 1883,
"mqttPrefix": "home/envoy",
"mqttAuth": false,
"mqttUser": "xxxx"
}
],
"platform": "enphaseEnvoy"
}

Please put correct IP address in config, restart plugin and get the whole debug log. Looks like your envoy is not available in network.

i was able to get it configured, i was using firmware 7.xxx and had to get a token. However after adding the bridge into Homekit i am not finding any of the sensors? I see the sensors in Homebridge but not Homekit

Are you read this section:
https://github.com/grzegorz914/homebridge-enphase-envoy#about-the-plugin

Yes I used the eve app and can see the bridge but the accessories didn't come over with the bridge..

Are you ad the accessory itself to the HomeKit.ap?

they didnt show up in Home Kit either

Are you read this:
https://github.com/grzegorz914/homebridge-enphase-envoy/wiki#adding-to-homekit
and this:
https://github.com/grzegorz914/homebridge-enphase-envoy/wiki#troubleshooting
Can you also post whole debug log after restart plugin?

ok i got the accessories to show up in Eve now. Currently working on the top bar and i'm getting an Error on Meter Response: Moved Permanently.
Any idea what this error means?

I can no help you one debug log, I need to know what happened in your system. Go to plugin settings Advanced Section and enable Debug Mode. Restart plugin and get the whole log.

TopBar is not working with firmware 7.x.x


Storage layout doesn't match

Describe the bug
When comparing the storage layout with Optimism's L1StandardBridge.sol, it doesn't match.
Optimism slot 50
Mainnet L1StandardBridge (https://etherscan.io/address/0x99C9fc46f92E8a1c0deC1b1747d010903E884bE1#code)
    {
      "astId": 4171,
      "contract": "src/L1/L1StandardBridge.sol:L1StandardBridge",
      "label": "superchainConfig",
      "offset": 0,
      "slot": "50",
      "type": "t_contract(SuperchainConfig)6776"
    },

Our implementation slot 50 and 51
    {
      "astId": 16167,
      "contract": "src/L1/L1StandardBridge.sol:L1StandardBridge",
      "label": "systemConfig",
      "offset": 0,
      "slot": "50",
      "type": "t_contract(SystemConfig)22983"
    },
    {
      "astId": 16249,
      "contract": "src/L1/L1StandardBridge.sol:L1StandardBridge",
      "label": "superchainConfig",
      "offset": 0,
      "slot": "51",
      "type": "t_contract(SuperchainConfig)22125"
    }

Configuration

Severity: Medium High
based on this resource https://github.com/crytic/slither/wiki/Upgradeability-Checks.

Impact
Thanos and OP stack do not have the same storage layout. A future update of Optimism is likely to corrupt the proxy when we are not aware of the storage layout difference.
Recommendation
Reverse the order of slots, systemConfig and superchainConfig.
Exploit Scenario
Demo

Thank you @usgeeus
I think there is no issue here. There are 2 reasons

Thanos is a new chain. If we upgrade smart contracts in future, the upgrades will not based on Optimism, it based on Thanos itself. So we don't need to keep the order of storage slots.
When upgrading smart contracts, what we can do it keep all previous variable, and everything will be safe.
When talking about issues, issues should be something unexpected, out-of-control and incorrect. Because upgrading is on our control, so it should not be an issue

Ok thanks, I'm closing this issue


Stack_overflow_2019

Investigated the participation rates based on ethnicity and differences in income between men and women on survey_results_sample_2019.csv

@Yodha-Sudarsi Could you refer to this issue raised.

@sanjay-kv , can you assign this issue to someone who is interested? I thought I could work on the project, but I have to learn more.


No info on Storage

[Enter feedback here]
I am really surprised that there is no documentation on storage here. How would one provision and plan for storage. The documentation should be improved, the pricing page does not talk about any storage at all. For any such offerings, you should also provide details on storage, network, and compute.

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

ID: 98de8b0f-6cc5-8d9f-6329-f1661e3dedb0
Version Independent ID: a6eff3cd-3859-0cc1-c29b-c7ece21f6f6c
Content: Overview of Azure Dedicated Hosts for virtual machines - Azure Virtual Machines
Content Source: articles/virtual-machines/dedicated-hosts.md
Service: virtual-machines
Sub-service: dedicated-hosts
GitHub Login: @cynthn
Microsoft Alias: cynthn

@knssub Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thanks for the feedback! We are going to be updating the Dedicated Hosts docs this quarter, so I will add this into the plan.
@brittanyrowe FYI - I'll add this into the work items.
#please-close


🔥 Clean all dart code

Pull Request Checklist

[x] I have read the UME contribution document and understand how to contribute, commit the code according to the rules. 我已阅读过 UME 贡献文档，并了解如何进行贡献，按照规则提交了代码
[ ] I have added the necessary comments in code to ensure that other contributors can understand the reason for the change. 我在修改中已经添加了必要的注释，以确保便于其他贡献者理解修改原因
[ ] The code has been formatted by dartfmt before push. 代码在提交前已经经过 dartfmt 进行了格式化
[x] Change does not involve the adjustment of test cases. Or all existing and new tests are passing. 改动不涉及测试用例调整，或 example 工程与单元测试已经完全跑通

@github-actions Flutter drive / drive_ios (iPhone 8) (pull_request) Failing after 1m
26s
Run subosito/flutter-action@v2.7.[1](https://github.com/bytedance/flutter_ume/actions/runs/3096437324/jobs/5011989193#step:6:1)
Run $GITHUB_ACTION_PATH/setup.sh -p -c '/Users/runner/hostedtoolcache/flutter/:channel:-:version:-:arch:' -k 'flutter-:os:-:channel:-:version:-:arch:-:hash:' -n 'any' -a 'x64' stable
Unable to determine Flutter version for channel: stable version: any architecture: x64
Error: Process completed with exit code 1.

@AlexV525 The pipeline looks failed again :|

It passed after retrying 😮💨

https://github.com/rxlabz/cyclop/pull/11 Here's the PR of cyclop.


H1 elements not being indexed

I'm trying to implement Pagefind on a site to improve the search, and it does not seem to be indexing the content of H1 elements.
The site is browsable here:
https://esphome-docs.web.app/index.html
The screenshot below shows searching for "Automations and Templates" and it is clearly ignoring the H1 header on the current page.
According to the docs, H1 headers should rank well above body text.
I suspect, though have not confirmed, that other Hx elements are also not being indexed.
I'm using Pagefind 1.1.0.

👋 hey @clydebarrow
The ranking here does seem worse than I would expect by default, though it is indexing the headings, they're just being lost in the soup of results a bit.
This is where tweaking the ranking parameters would help. I can see in the site source you have configuration for this, but it's currently not being applied since it isn't inside a ranking object. Currently you have:
    window.addEventListener('DOMContentLoaded', (event) => {
        new PagefindUI({
            element: "#search",
            showSubResults: true,
            pageLength: 0.0,
            termSaturation: 0.8,
            termFrequency: 0.4,
            termSimilarity: 1.0
        });
    });

But that should be:
    window.addEventListener('DOMContentLoaded', (event) => {
        new PagefindUI({
            element: "#search",
            showSubResults: true,
            ranking: {
              pageLength: 0.0,
              termSaturation: 0.8,
              termFrequency: 0.4,
              termSimilarity: 1.0
            }
        });
    });

Additionally, from a quick test, these parameters seem to do a little better:
pageLength: 0.0,
termSaturation: 1.6, // raised this value to favor the high-density pages
termFrequency: 0.4,
termSimilarity: 6.0 // raised this value to trim out some shorter word stems from muddying results

but it's currently not being applied since it isn't inside a ranking object

Reminds me again why I dislike Javascript :-(
So with that fixed and your suggestions applied, it now seems to be working well. Thanks!


snap builds fail to release

The login file location is wrong; the login file is copied into the git root at $GIT_ROOT/edgex-snap-store-login, and then in the dockerfile we execute this: https://github.com/edgexfoundry/edgex-go/blob/13a2f8ee5ede957d2082c06417adfd63e4c50c48/snap/Dockerfile.amd64.build#L5
which means that the login file is located in the container at /build/edgex-snap-store-login, however the entrypoint.sh script uses /edgex-snap-store-login: https://github.com/edgexfoundry/edgex-go/blob/13a2f8ee5ede957d2082c06417adfd63e4c50c48/snap/entrypoint.sh#L12-L20
See for example: https://jenkins.edgexfoundry.org/view/Snap/job/edgex-go-snap-master-stage-snap-arm/111/consoleFull
17:15:20 Snapped edgexfoundry_0.7.0-20181009+13a2f8e_arm64.snap
17:15:22 Usage: snapcraft login [OPTIONS]
17:15:22
17:15:22 Error: Invalid value for "--with": Could not open file: /edgex-snap-store-login: No such file or directory

We should also add the --force option to the docker image prune line as this is what happens in the build:
17:15:57 WARNING! This will remove all dangling images.
17:15:57 Are you sure you want to continue? [y/N] Total reclaimed space: 0B


Fix: [Clear Button is active when no input is selected in Dropdown]

Proposed Changes

Fixes #9263
Added a state const [hasTyped, setHasTyped] = useState(false);.
Modified handleInputChange to set hasTyped(true) when the user types
In handleSelectionChange, set setQuery(optionLabel(selectedOption)); to update query after selection.
Set hasTyped(true) after selection to ensure the "Clear Selection" button is shown.
In the ComboboxButton rendering logic, updated the condition for showing the "Clear Selection" button:
Added setHasTyped(false) in the onClick handler for clearing selection to reset the hasTyped state.

Behaviour after the changes

The website allows users to search and select options from a dropdown, with the input dynamically filtering results as the user types.
The "Clear Selection" button only appears after the user types or selects an option, making it easy to clear the selected value.
The input field reflects the current query, and the dropdown updates based on the user's input, offering a smooth and interactive search experience.

Screen Records
Screencast from 2024-12-02 00-14-08.webm

Summary by CodeRabbit

New Features

Enhanced interactivity in the AutoComplete component with improved input tracking and selection handling.
Introduced a clear button that dynamically appears based on user input or selection.

Bug Fixes

Refined control flow for input handling to ensure accurate state updates.

@Jacobjeevan I had not touched the other codes but its giving errors in Cypress Tests that are being shown in different components. And the clear button is working smoothly after making the code changes

@ANIR1604 Any updates?

@ANIR1604 Any updates?

Now It's working Perfect Sir @Jacobjeevan
Screencast from 2024-12-06 03-55-34.webm

@Jacobjeevan Solved all the errors related to clear selection at all instances.

LGTM

@rithviknishad Ok I will be working on this tomorrow. Can't I change it to null at Time 0 ? In my point of view it will be a great approach to give it a null value.

@rithviknishad Sir plz review my PR ... Are we ok to go with these changes ?


Update androidx.test.uiautomator to 2.3.0-beta01

Context: cl/589671047
Updates UiObject2#setGestureMarginPercentage which was renamed for consistency.

This change is

@eric100lin

@ko1in1u for review


Fixed VS 2022 Mac compatibility

Visual Studio 2022 on Mac marks the project as invalid if the project Guid is set. The easiest way to fix it is to add an additional check and disable 'EnableGodotProjectTypeGuid' for the SDK when the Visual Studio version is less than 17.0.
Bugsquad edit:

Fixes https://github.com/godotengine/godot/issues/67029.

See also the issue about this: https://github.com/godotengine/godot/issues/67029
How does this PR behave in regard to the SDK issues and Godot plugin mentioned there?
Is there a better way to detect VS for mac? If not what are the consequences of removing that GUID for VS 2019 and earlier on windows?

See also the issue about this: #67029 How does this PR behave in regard to the SDK issues and Godot plugin mentioned there? Is there a better way to detect VS for mac? If not what are the consequences of removing that GUID for VS 2019 and earlier on windows?

It fixes the abovementioned issue for VS 2022 Mac. Restoring the Nuget projects makes VS 2022 fully working, even without the Godot Addin (Addin could not be fixed ATM since there is no recent version of MonoDevelop.AddinMaker).
The condition allows it to work with VS 2019 Mac but I can't check the earlier versions.
Will re-check it on Windows tomorrow and maybe add one more condition to avoid issues there.

Test results.
Mac
[-] VS 2017: broken, might work if addin is installed

addins are not working anymore - repository is down
.NET SDK 6.0 and 7.0 detected in corresponding folders
project is not opening when ProjectTypeGuid is set, removing it allows the project to open
Godot libraries are not resolved
can't compile, unknown error

[*] VS 2019: broken on M1 might work on Intel

addin is installed and working
.NET SDK 6.0-7.0 cannot be found on M1
project is opening, regardless of ProjectTypeGuid
Godot libraries are not resolved on M1
can't compile on M1

[+] VS 2022: works

addin could not be installed, major compatibility issues
SDK detected
project is opening only if ProjectTypeGuid is disabled
Godot libraries are resolved after Nuget Restore
compiles without any issues

[+] VS Code: works

addin works
SDK detected
project always opens (you can't break it, it works on folder level not with the .csproj)
Godot libraries are resolved
compiles with MSBuild, can fail if no .NET SDK is installed

Windows
[-] VS 2019: doesn't support .Net 6.0 even with Preview Features hack

no addin available for Godot 4.x and VS 2019
.NET SDK installed and MSBuild sees it, but not the Visual Studio
project is opening regardless of ProjectTypeGuid
Godot libraries are downloaded with Nuget but not loaded
doesn't compile

[+] VS 2022:  works

addin not in the repository, haven't checked the one from the github
SDK detected
project is opening every time
Godot libraries are resolved
compiles without any issues

[+] VS Code: works

addin works
SDK detected
project always opens (you can't break it, it works on folder level not with the .csproj)
Godot libraries are resolved
compiles with MSBuild, can fail if no .NET SDK is installed

** Conclusion **
Since migrating from Mono to .Net 6+ we are restricted to MSBuild project types and the latest Visual Studio versions that don't rely on ProjectTypeGuid but sometimes fail if it's invalid. There is literally zero need for it anymore for the 4.x branch. Visual Studio Code is the main future target for Godot C# development and is not affected by the .csproj files at all.
I vote to remove the EnableGodotProjectTypeGuid or force it to false without any extra condition.

The ProjectTypeGuid has already been removed in 3.x (see https://github.com/godotengine/godot/pull/44135/commits/cd339ff002bc8db141965a04db16d1e279ca3ce8), the reasoning was:

The property has no effect as the older VS project system doesn't
work with Sdk style projects.
The presence of the property was preventing Visual Studio for Mac
from opening the project if the Godot extension is not installed.

So I think it makes sense to remove it altogether.

The ProjectTypeGuid has already been removed in 3.x (see cd339ff), the reasoning was:

The property has no effect as the older VS project system doesn't
work with Sdk style projects.
The presence of the property was preventing Visual Studio for Mac
from opening the project if the Godot extension is not installed.

So I think it makes sense to remove it altogether.

Removed and committed new code.

You'll have to remove <GodotProjectTypeGuid> from Sdk.props as well (see the linked commit cd339ff002bc8db141965a04db16d1e279ca3ce8).
Also, you'll need to squash the commits before this PR can be merged. The contributing documentation contains information about squashing in case you need it.
Feel free to reach out in the development chat if you need help.

Thanks! And congrats for your first merged Godot contribution :tada:


Group Education - Dashboard

Issue
Section 1
Once there have been a group education session it will be recorded on the dashboard as follows:
Number of Group session | number of group encounters on health education in this specified month.
Total attendees | total number of all the participants in those Group session that took place in that specific month
Unique Patients |  the number of patients in the month that showed up for any group education.
Section 2
The Group Education | topics that were covered in the group education this  specified month.
Attendance | Number of people who attended the session on that topic
On the topics table, once the user click on the blue arrow, they will see the list of participants on that specific education topic session. The list will appear only at Community level. At health center level on the list of topics we will not have the arrows.

Landing page
Group education
Once you click on a topic arrow

Todo

Create the screen per design
Implement the Logic

@adamhstewart
Deployed to Nutrition env. Ready for QA.

https://nutrition-ihangane.pantheonsite.io/app/#dashboard/group-education
QA Review
Found issues and questions

The list on the group education page contains the different educations, not the topics as shown in the issue description.

The consequence of point #1 is that when clicking on the arrow, the participants of the specific session are listed, not the participants in a specific topic.

Checked and approved:

The Dashboard page contains a Group education tab.

@balagan73
I believe that current behaviour is correct. Each entry presents a session with all it's topics.
Had it been the way you describe it, there would be no meaning to the date.


Add tests from WhichBrowser

I've created a separate Composer package the tests I run for WhichBrowser. The package is called whichbrowser/testrunner and includes the runner.php script and all the yaml files. It is automatically updated whenever there is a new release of WhichBrowser.
If you want I can probably create a pull request for the actual extracting of the UA strings.
But it is also possible to extract it by using the runner.php script from the command line:
php vendor/whichbrowser/testrunner/runner.php list
or
php vendor/whichbrowser/testrunner/runner.php list television
or
php vendor/whichbrowser/testrunner/runner.php list mobile tablet television gaming

@NielsLeenheer thank you. I will look into it today and check now far i get in the new packages
https://github.com/ThaDafinser/UserAgentParserMatrix

@NielsLeenheer they are included now here: https://github.com/ThaDafinser/UserAgentParserMatrix/tree/master/data/datasets
The results are also available here: https://github.com/ThaDafinser/UserAgentParserMatrix/tree/master/data/results


N16: Tetra and ogmo3

Part of https://github.com/rust-gamedev/rust-gamedev.github.io/issues/365.

Merged, thanks for the PR!


Add dgof package

Add:
install.packages("dgof", ref="1.4")
Then you can remove the install command in nkc's plot_density_loess_fit rscript.

Added