Having trouble gettings started with namespaces

Hi,
I want to thank you for taking the time to write such an awesome looking library. Unfortunately I am having a bit of trouble getting started. I am trying to build an ePUB viewer in javascript. ePUB documents contain a lot of xml manifest information and a nice standardized way for me to convert this to JSON would really help the project along. Below is an example of a manifest pulled from a real world ePUB
<?xml version="1.0" encoding="UTF-8"?>
<package xmlns="http://www.idpf.org/2007/opf" unique-identifier="ean" version="2.0">
  <metadata xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns="http://www.idpf.org/2007/opf">
      <dc:title>L'espagnol dans votre poche</dc:title>
      <dc:creator></dc:creator>
      <dc:publisher>Larousse</dc:publisher>
      <dc:rights>© Éditions Larousse, </dc:rights>
      <dc:identifier id="ean">9782035862464</dc:identifier>
      <dc:language>fr</dc:language>
  </metadata>
  <manifest>
    <item id="css" href="styles.css" media-type="text/css" />
    <item id="ncx" href="toc.ncx" media-type="application/x-dtbncx+xml" />
    <item id="cover" href="images/cover.jpg" media-type="image/jpeg"/>
    <item id="font2" href="Fonts/HighlanderStd-Medium.otf" media-type="application/x-font-otf"/>
    <item id="Page_1"  href="Page_1.html"  media-type="application/xhtml+xml"/>
    <item id="Page_1_jpg" href="images/Page_1.jpg"  media-type="image/jpeg"/>
    <item id="Page_2"  href="Page_2.html"  media-type="application/xhtml+xml"/>
    <item id="Page_2_jpg" href="images/Page_2.jpg"  media-type="image/jpeg"/>
  </manifest>

  <spine toc="ncx">
    <itemref idref="Page_1"/>
    <itemref idref="Page_2"/>
  </spine>
  <guide>
  </guide>
</package>

For now lets say I am only interested in the itemhref items under the spine node. What would be the right way to pull them out. I have tried multiple things but the only way I can get it work it by ripping out all the namespaces from the document. Is this a  bug in Jath or am I missing something?
BTW sorry for bugging you with such a n00b plea for help but if you can help me I would be happy to compensate by contributing a bit of documentation on how this namespace resolving thing works in your wiki.

XPath doesn't support the idea of a default namespace, so you have to map the url of your default namespace to some binding or use the local-name() function in order to refer to elements within the that namespace.
There are some examples of this if you look in samples.html. Probably the 'correct' way to approach it is to give Jath a resolver function:
Jath.resolver = function( prefix ) {
    var mappings = { def: "http://www.idpf.org/2007/opf" };
    return mappings[ prefix ];
}

Then use something like this as the template:
var template = [ "//def:spine", { idref: "@idref" } ];

Or you could alter the xpath selectors to use local-name():
var template = [ //*[local-name()='spine'], { idref: "@idref" } ];

I haven't tested these code bits. I might be able to do something a little later though.

Ok so obviously the problem is not Jath but my poor of knowledge of XPath I managed to have a bit of luck using the following template:
var template0 = {
  manifest: [ "//*[namespace-uri()='http://www.idpf.org/2007/opf' and local-name()='item']", {
    id: "@id",
    href: "@href",
    media_type: "@media-type"
  } ],
  spine: [ "//*[namespace-uri()='http://www.idpf.org/2007/opf' and local-name()='itemref']", {
    idref: "@idref"
  } ],
};

ideally I would like to tidy this up by passing Jath a resolver, I tried a few things but no luck so far. Here is one thing I thought would work:
Jath.resolver = packageDoc.createNSResolver(packageDoc)

and then use this as my template:
var template0 = {
  manifest: [ "//item", {
    id: "@id",
    href: "@href",
    media_type: "@media-type"
  } ],
  spine: [ "//itemref'", {
    idref: "@idref"
  } ],
};

By the way, thanks for your help!

No prob.
The reason your createNSResolver isn't working is because (ironically) only bound namespaces will be resolved. So probably your "dc:" namespaces will resolve this way but not the elements in the default namespace.

Ok looks like I am all sorted now. I get it, I missed the prefix part of the template you suggested. Once I added it in it is working perfectly :)

cool


refactor move scan pool from env to scanner scheduler

Proposed changes
Issue Number: close #xxx
Problem summary
The origin scan pools are in exec_env.
But after enable new_load_scan_node by default, the scan pool in exec_env is no longer used.
All scan task will be submitted to the scan pool in scanner_scheduler.
BTW, reorganize the scan pool into 3 kinds:

local scan pool
For olap scan node

remote scan pool
For file scan node

limited scan pool
For query which set cpu resource limit or with small limit clause

TODO:
Use bthread to unify all IO task.
Fix some trivial issues:

the memtable flush size printed in log is not right

Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.77 seconds
load time: 662 seconds
storage size:<PHONE_NUMBER>1 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230104105036_clickbench_pr_73433.html


Any configs for pretraining on 4 frames setting?

As titled. Thanks.

I have updated the config in https://github.com/jayleicn/singularity#pre-training, please check.


Fix linting issue with OptionsDrawer.

Fixes https://github.com/vectara/vectara-answer/issues/63

Doesn't seem to resolve the issues reported in #63 @ofermend @cjcenizal
Let me know if you want to hop on a discord call? Thanks!


Port all changes from the Pub/Sub Lite repo back to beam

Beam will be the canonical source for this IO in the future per offline discussion.
Also add the other direction of helper to CloudPubsubTransforms and add an integration test.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @aaltay
Ahmet, would you be willing to review this PR?

Build failures are unrelated.

I do not think I am the best person to review this.
@chamikaramj @pabloem - Could one of you review this or suggest a reviewer?

R: @pabloem
I'll pass to Pablo. @dpcollins-google pls add a Jira with the context.

Run Java PreCommit

Run SQL PreCommit

Please add a Jira.

 Can you see if it is related to this change? https://issues.apache.org/jira/browse/BEAM-12882


Rename master for inclusiveness

This fixes #587
I searched the repo for master and blacklist/whitelist and adjusted the code as needed. I don't think there is much to do with Travis, but if we can define a branch to pull, it might need to be switched from master to main once GitHub is updated with the new default.
My proposal for the next steps would be:

Merge in this PR to master
Create a new branch called main from the latest master and push it up
Check Travis and other third party tools to ensure they're working against main
Delete master

Thanks for this, I'll try to follow up with the next steps soon

Sure, happy to work on whatever I can help with. Thank you!

also just dropping this here for later https://github.com/github/renaming
github makes it pretty easy to rename

rename is complete :)


hidden button has interaction

Dev Home version
0.1501.522.0
Windows build number
10.0.22631.3731
Other software
OS Build Version: 10.0.22631.3731.amd64fre.ni_release.220506-1250
.NET Version: .NET 8.0.5
Steps to reproduce the bug
Go to machine config, and hover at bottom of main section (see video)
https://github.com/microsoft/devhome/assets/1462282/52a3f2ca-97ef-4eee-aaad-4e2f20191297

Expected result
nothing there
Actual result
looks to have a 'on hover' interaction
Included System Information
CPU: Intel(R) Core(TM) Ultra 7 165H
Physical Memory: 31.64GB (11.27GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_0.1401.505.0_x64__8wekyb3d8bbwe (Core Widget Extension)
Microsoft.Windows.DevHome_0.1401.505.0_x64__8wekyb3d8bbwe (Hyper-V Extension)
Microsoft.Windows.DevHomeGitHubExtension_0.1400.505.0_x64__8wekyb3d8bbwe (Dev Home GitHub Extension (Preview))
Microsoft.Windows.DevHome.Canary_0.1501.522.0_x64__8wekyb3d8bbwe (Core Widget Extension)
Microsoft.Windows.DevHome.Canary_0.1501.522.0_x64__8wekyb3d8bbwe (Hyper-V Extension)
Microsoft.Windows.DevHome.Canary_0.1501.522.0_x64__8wekyb3d8bbwe (Windows Sandbox Extension)
Microsoft.Windows.DevHomeAzureExtension.Canary_0.1000.522.0_x64__8wekyb3d8bbwe (Dev Home Azure Extension (Canary))
Microsoft.Windows.DevHomeGitHubExtension.Canary_0.1500.522.0_x64__8wekyb3d8bbwe (Dev Home GitHub Extension (Canary))
Widget Service:
MicrosoftWindows.Client.WebExperience_524.13701.0.0_x64__cw5n1h2txyewy

As a data point, I can repro this on 0.1501.522.0 (Canary) as well (if Quickstart Playground's experimental feature is turned off). By comparison, I don't see this behavior in 0.1401.505.0 (Preview).

Opened bug on microsoft-ui-xaml repo


Implement Entitymanager and populate security artifacts

This PR implements some basic entitymanager functionality and also reads the keypairs and OVs from disk.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

95 of 105 (90.48%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+90.8%) to 90.826%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

server/entitymanager/entitymanager.go
95
105
90.48%

Totals

Change from base Build<PHONE_NUMBER>:
90.8%

Covered Lines:
99

Relevant Lines:
109

💛  - Coveralls


Migrate QdrantVectorStore

Migrate the following to /standard/vector_stores/concrete/QdrantVectorStore.py
https://github.com/swarmauri/swarmauri-sdk/blob/dev-0.4.1/swarmauri/community/vector_stores/QdrantVectorStore.py
Leveraging the test file /tests/vector_stores/TfidfVector_test.py as en example. Create a test file in /tests/vector_stores/concrete/QdrantVectorStore_test.py.
use the following env variable key names QDRANT_URL_KEY and QDRANT_API_KEY for url and api_key

This does not need to be migrated to community anymore, however this needs to be updated due to the new design change


chore: add ember tests for migrate init

Adding ember scenario tests for migrate init.

Looks like you have some failing linting


Action Text attachment button has no icon

Issue: Action Text attachment button has no icon
Because the old version of library is used (1.1.1):
https://github.com/sferik/rails_admin/blob/v2.0.1/lib/rails_admin/config/fields/types/action_text.rb
please update version of trix library

I fixed this by loading own css and js:
js_location { bindings[:view].asset_pack_path 'actiontext.js' }
css_location { bindings[:view].asset_path 'application.scss' }

Yes, this worked, thank you!  Maybe we could add this to the wiki? https://github.com/sferik/rails_admin/wiki/ActionText - I wasn't sure if I should just edit the wiki or no.

@exvayn @nicholalexander How did you incorporate the :actiontext attribute to work with rails_admin?

How did you incorporate action text to work with rails_admin?

@vnguyen13 there is a wiki page explaining how to get ActionText working

Since Rails 7 removed webpacker, I add the following lines but get no luck.
  config.model 'Article' do
    field :content do
      css_location { bindings[:view].asset_path 'actiontext.css' }
      js_location { bindings[:view].asset_path 'actiontext.js' }
    end
  end

I'm having the exact same issue with Rails 7 at the moment.
Now the code @xmarco posted will return the raw contents of actiontext.js and not the compiled contents.
Are there any known work-arounds to point ActionText towards your own version when you don't use webacker?


Standardizing changelog format across ManageIQ

@miq-bot add_label documentation

Not needed to be added at this time.


Change asset name to title

As discussed in #262: I would propose to change the asset field name to title. Currently it seems it is usually used for a title and not for an identifier. Name itself is ambigious and it is not really clear whether it should be a identifier or a title.

+1
Let's get this change in asap.

PR #269 implements this.

Merged - closing this issue.


fix ci and fix logo colour being off

Description
[Description of the changes you are submitting]
Checklist

[X] Have you tested the change you are submitting?
[ ] Is the commits history nice and clean?

@halal-beef Uhh

fixing the logo was a happy little accident

NOOO I DIDNT MESN TO CLOSE IT


Why setup will change cwd?

I found that setup method will change cwd. If I enter vim under fold repo/subdir/, current CWD won't be subdir but the git repo root. When I comment the setup method then it become subdir (this is vim default behaviour).
I don't think this is what setup method should do.

You can set silent_chdir to false in the setup to see which pattern or lsp is causing the cwd change.


Hello World example doesn't work

I installed the SDK, then tried
dotnet new console
which took about 5 minutes to run 'dotnet restore'
then
dotnet run
which produced error MSB3577: Two output file names resolved to the same output path  "obj\Debug\netcoreapp3.1\TestProject.Form1.resources" [C:\Users\thies\thies.csproj]
The build failed. Fix the build errors and run again.
How is it possible that your documentation is so wrong that the Hello World example fails?

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

ID: 3ad87897-30cc-e639-b91a-b3800269f7fe
Version Independent ID: 9d3015b4-cba1-0956-9ec7-4154dbb74d93
Content: .NET Core intro and overview
Content Source: docs/core/introduction.md
Product: dotnet-core
GitHub Login: @richlander
Microsoft Alias: dotnetcontent

which took about 5 minutes to run 'dotnet restore'

AFAIK, the restore process downloads some packages. So, this is dependent on your internet connection speed.

MSB3577: Two output file names resolved to the same output path "obj\Debug\netcoreapp3.1\TestProject.Form1.resources" [C:\Users\thies\thies.csproj]

The path in the error contains "Form1", which indicates a WinForms app, not a Console app. Maybe you have tried creating two apps in the same directory which caused some problem?
Can you please try creating a new folder, and run the commands again from it?
I hope this fixes your problem. If it doesn't, please reply back and wait an official answer from a Microsoft employee.

I found another example that was slightly different:
dotnet new console --output sample1
dotnet run --project sample1
This works fine, so apparently you have to give it a name.

From: Youssef Victor<EMAIL_ADDRESS>Sent: Saturday, August 1, 2020 2:28 PM
To: dotnet/docs<EMAIL_ADDRESS>Cc: John Thiesen<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet/docs] Hello World example doesn't work (#19786)
[EXTERNAL SENDER]
which took about 5 minutes to run 'dotnet restore'
AFAIK, the restore process downloads some packages. So, this is dependent on your internet connection speed.

MSB3577: Two output file names resolved to the same output path "obj\Debug\netcoreapp3.1\TestProject.Form1.resources" [C:\Users\thies\thies.csproj]
The path in the error contains "Form1", which indicates a WinForms app, not a Console app. Maybe you have tried creating two apps in the same directory which caused some problem?
Can you please try creating a new folder, and run the commands again from it?
I hope this fixes your problem. If it doesn't, please reply back and wait an official answer from a Microsoft employee.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/docs/issues/19786#issuecomment-667576797, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIDEFZU2FWJROHS5WXMOVCLR6RUGTANCNFSM4PR7IBVA.

Both examples should work I think.
Did you try a new empty folder?

That's exactly how I started. I downloaded the SDK on a machine that hasn't had .NET Core on it before and then immediately tried to run the Hello World example and got the error messages.
Now later I created a new folder and tried again and didn't get the error but got the expected output, so I don't see what made the difference.

From: Youssef Victor<EMAIL_ADDRESS>Sent: Saturday, August 1, 2020 10:36 PM
To: dotnet/docs<EMAIL_ADDRESS>Cc: John Thiesen<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet/docs] Hello World example doesn't work (#19786)
[EXTERNAL SENDER]
Both examples should work I think.
Did you try a new empty folder?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/docs/issues/19786#issuecomment-667621388, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIDEFZXZLZWJ5JPMBLBO5R3R6TNL7ANCNFSM4PR7IBVA.

If no output folder is specified it uses the current directory. There must have been other files in the current directory of the terminal you had open at the time. I'll close this issue. Cheers!


fix: correct previous SIG meeting recordings URL

Fixes #428

Solved issue #428
I changed link of Watch previous recordings from https://groups.google.com/forum/#!forum/asyncapi-users to https://www.youtube.com/playlist?list=PLbi1gRlP7pijUwZJErzyYf_Rc-PWu4lXS as mentioned by a member.

@aman-ragh thanks, I just adjusted the title of the PR to match conventional commits spec


Linux install issue

I could not install pycoral on Linux. Seems like the link is broken:
https://github.com/google-coral/pycoral/releases/download/release-frogfish/pycoral-1.0.0-py3-none-any.whl

@Cganzorig sorry, where did you get the link from?
Here is where you can get the right package for your system:
https://github.com/google-coral/pycoral/releases/

Thanks for the quick answer. It was in the official website:
https://coral.ai/software/#coral-python-api
If possible, could you update the link?

@Cganzorig thanks for reporting, we'll definitely get this fixed


[Model] LoRA support added for command-r

FIX #4742

@sergey-tinkoff can you make sure to add the new shapes to tests/lora/test_punica.py?

@sergey-tinkoff can you make sure to add the new shapes to tests/lora/test_punica.py?

Sure, done.

@Yard1  please review


Fix XProc for Saxon XQuery

This pull request fixes XProc for Saxon XQuery (src/harnesses/saxon/saxon-xquery-harness.xproc) and test it with #72 and #423.
Commits

5b1cadb66cd3e9db26dee6fea07c53220fa12e56 tests #423 with saxon-xquery-harness.xproc. The test fails (Travis, AppVeyor), because the harness isn't compatible with a recent change (#523).

fd744a30824b2002a6071132c8002fd3bbee263d fixes it by enclosing the entire output with a document node. (Travis, AppVeyor)

b47cfeebbff562ee911bfe235d251e109fcd34d2 replaces xspec-72.xspec with the original repro file (attaced to https://github.com/xspec/xspec/pull/72#issuecomment-277378508), because the file in the current master doesn't completely reproduce the problem. (No umlaut in the result HTML.)

b6d7fa8bca08de8ce6a4807bcdc6874e431a0aaf adapts xspec-72.xspec to XQuery.

e4ce93de0997f2795d095c54951af2999bffed77 wraps the Bats command line to run xspec-72.xspec. (only for readability)

499bad164dc024755023a3d489d7552afe6aa61b tests xspec-72.xspec with saxon-xquery-harness.xproc. The test fails (Travis, AppVeyor), because #72 did not include saxon-xquery-harness.xproc.

e5e3c58adf6734124df18e4b63eab2ea6e9771eb fixes it in the same way as #72. (Travis, AppVeyor)

Further work
Update Wiki to include XProc for Saxon XQuery.

Further work
Update Wiki to include XProc for Saxon XQuery.

Done (diff)


[nextcloud] Supporting full text search with Nextant

A great feature is the full text search of all your documents. To get it up and running, there are some prerequisites you would need to implement.
See installation instructions: https://github.com/nextcloud/nextant/wiki.
Would be nice to see it implemented in this great image.

no need to implement it in the docker - please keep it as clean as possible.
just set-up a second solr container and connect it - as i did :)

Gret idea, could you give me an example of your solr container setup?

Thanks,
I adapted your example to my setup and integrated it to my compose file. I linked nextcloud and solr container together. I can wget http://solr:8983/solr from nextcloud bash, but in admin console of nextcloud nextant says it cannot ping given address. What am I doing wrong?
nextcloud:
    container_name: nextcloud
    restart: always
    links:
        - mysql
        - redis
        - solr
    volumes:
        - ./data:/data
        - ./config:/config
        - ./apps:/apps2
    image: wonderfall/nextcloud

redis:
    container_name: nextcloud_redis
    restart: always
    image: redis

mysql:
    container_name: nextcloud_mariadb
    restart: always
    volumes:
        - ./db:/var/lib/mysql
    environment:
        - MYSQL_ROOT_PASSWORD=pass
        - MYSQL_USER=owncloud
        - MYSQL_PASSWORD=pass
        - MYSQL_DATABASE=owncloud
    command: --binlog-format=MIXED
    image: mariadb

solr:
    container_name: nextcloud_solr
    volumes:
      - ./solr:/opt/solr/server/solr/mycores
    image: solr:alpine

whitout the ip ?

No, when I'm trying without the port, it says wrong address format

i uses ip adress of the pc.
e.g. http://<IP_ADDRESS>:8983/solr

sorry my fault, had to copy default core to new folder and add core with bin/solr -c nextant

@jazzdd86 how did you solve the problem? because I dont understand what you did?

Right now, you don't have to fiddle around with solr config, just follow the instructions written in the README of the nextcloud image: https://github.com/Wonderfall/dockerfiles/tree/master/nextcloud#how-to-configure-nextant
and use the solr part of the docker-compose file for starting the solr container.


Auth.signedIn event triggered when not supposed to

Description
The Auth.signed event gets triggered when accessing a verify email deep link from mail
client?.auth.onAuthStateChange.listen((data) { ... })

Expected behavior
Auth.signedIn should be called only when you sign in with signInWithPassword or other signIn methods
Screenshots

Environment

Physical Device: Samsung Galaxy S7 Edge (Android 8.0.0)
OS: macOS 12.5.1
Flutter 3.3.5
Dart 2.18.2 • DevTools 2.15.0

Are you by any chance using Getx?

@chirastefan
This is actually working as expected. There are multiple occasions where AuthChangeEvent.signedIn event is triggered, one of which is when email verification is complete.
Could I ask why you thought AuthChangeEvent.signedIn would be called only when signing in with signInWithPassword? Maybe we could provide better docs around it.

hi, yes, I'm using Getx, why?

yes, I noticed that the AuthChangeEvent.signedIn event is triggered even for other events, for example, auth.resetPasswordForEmail()
Well what's the purpose of having enum AuthChangeEvent {   passwordRecovery,   signedIn,   signedOut,   tokenRefreshed,   userUpdated,   userDeleted, } if only signedIn and signedOut are called?
I would expect AuthChangeEvent.passwordRecovery to be called when I call auth.resetPasswordForEmail(), AuthChangeEvent.userUpdated when the email is verified, but instead only AuthChangeEvent.signedIn is called every time for the above events/actions.

I'm using that access token and that listener to check if the user is logged in or not in my app. So in this case, after calling auth.resetPasswordForEmail() I automatically log in. Is this the intended flow or am I doing something wrong?
The same case is for verifying the email.

I would expect AuthChangeEvent.passwordRecovery to be called when I call auth.resetPasswordForEmail(), AuthChangeEvent.userUpdated when the email is verified, but instead only AuthChangeEvent.signedIn is called every time for the above events/actions.

AuthChangeEvent.passwordRecovery is called when you call auth.resetPasswordForEmail(), click on the recovery link sent to the inbox and come back to the app via deep link so that you can then redirect the user to a password reset page to reset the password.

I understand that flow and I did that(as mentioned in the documentation), but as I've mentioned above, that event does not seem to trigger an AuthChangeEvent. passwordRecovery event, only a AuthChangeEvent. signedIn

@chirastefan
I cannot seem to reproduce your issue. I just tested it, and I can confirm that upon opening the password recovery link, I get AuthChangeEvent.signedIn followed by AuthChangeEvent.passwordRecovery. This is working as expected, as upon clicking email confirmation, you are signed into the app, hence the initial signedIn event, and then followed by the passwordRecovery event because the deeplink was for password recovery.
You can see the code here where the events are being sent.
In your code, you can handle password recovery event like this in your code, and you should see it working.
supabase.auth.onAuthStateChange.listen((data) {
    final event = data.event;
    if(event== AuthChangeEvent.passwordRecovery) {
      // redirect user to update password page
    }
  });
}

On what should I base the logic for logged in or not?

If session is not null in your onAuthStateChange, the user is signed in.
If you are still not seeing AuthChangeEvent.passwordRecovery event in your app, could you provide a detailed steps to reproduce, or perhaps create a loom video or something to share how you are testing it?

Hi, I managed to reproduce the same flow, signedIn event and then a passwordRecovery event.
My point is that I cannot use that event listener or the auth.currentSession to check if the user is logged in, because that will be non-null even for an email confirmation, password recovery or other events.
So I cannot use that event listener to set for example a loggedIn bool flag on the client and I also cannot use the currentSession to check if the user is logged in or not.
Shouldn't the signedIn event be trigged only when the user signs in with auth.signInWithPassword() or other sign in methods? and shouldn't auth.currentSession be non-null only when the user logs in with auth.signInWithPassword() or other sign in methods?

"4. opens the email and clicks on the link, which takes the user back to the app <- At this point the user is logged in." Why? it's not necessary to be logged in in order to change your password.
"If a user is changing their password, they better be a user who has verified their identity, hence should be logged in". So the listener auth.onAuthStateChange and/or the auth.currentSession are not useable when it comes to checking if the user is logged in or not, because it's triggered(the event) and not null(currentSession) for other events(password reset, confirm email, etc)

Why? it's not necessary to be logged in in order to change your password.

That is just how Supabase works right now. Once you click on the password recovery link, you are logged in.

I guess this happens not only for the reset password feature but for the email confirmation also.

@chirastefan Yup, both email confirmation and password reset case, once the user clicks on the email link and are brought back to the app, they are considered signed in. This is how Supabase works.

@chirastefan I cannot seem to reproduce your issue. I just tested it, and I can confirm that upon opening the password recovery link, I get AuthChangeEvent.signedIn followed by AuthChangeEvent.passwordRecovery. This is working as expected, as upon clicking email confirmation, you are signed into the app, hence the initial signedIn event, and then followed by the passwordRecovery event because the deeplink was for password recovery.
You can see the code here where the events are being sent.
In your code, you can handle password recovery event like this in your code, and you should see it working.
supabase.auth.onAuthStateChange.listen((data) {
    final event = data.event;
    if(event== AuthChangeEvent.passwordRecovery) {
      // redirect user to update password page
    }
  });
}

On what should I base the logic for logged in or not?

If session is not null in your onAuthStateChange, the user is signed in.
If you are still not seeing AuthChangeEvent.passwordRecovery event in your app, could you provide a detailed steps to reproduce, or perhaps create a loom video or something to share how you are testing it?

I know when this recovery event is not fired, when we chose AuthFlowType.pkce. I don't know if this expected. But it would be nice if we had the recovery password flow implement in all authFlowTypes.

@dshukertjr I would love to hear some feedback about my comment. I know you might have a bunch of other stuff to do, but this is a critical issue,

@talamaska
Would you open a separate issue as it seems like a separate issue from the original issue of this thread. I'm going to close this issue as the original issue is the intended behavior of Supabase.


Translating woocommerce buttons.

Can you reproduce this issue on default Wordpress theme (eg Storefront)?
Can you reproduce this issue when all other plugins are disabled except WooCommerce, Polylang and Hyyan WooCommerce Polylang Integration?
What product versions and settings are you using when this issue occurs?

PHP:
WordPress:
WooCommerce:
Polylang: [state if using Polylang PRO]
Hyyan WooCommerce Polylang Integration:
Browser:

Steps to Reproduce

What I Expected
What Happened Instead
WordPress Environment

```
Copy and paste the system status report from **WooCommerce > System Status** in WordPress admin here.
```

Hello All,
What do I need to do to translate all woocommerce buttons? Like add to cart, checkout, etc.
Before Polylang or  Lingotek was doing this but now all my buttons are in english to matter what language I select on my website.
Looking forward to your reply.
Regards,
Ray

In Woopoly I selected

Translation Downloader
Download Woocommerce translations when a new polylang language is added

So I thought this would handle the translation of all Woocommerces basic functions.

Nobody?

?

Hello?


Probot not closing issues

Heya!
First of all, thanks for probot - it's amazing!
I've had it installed and running on ts-loader for a couple of weeks now. It started out all guns blazing, marking things as wontfix and then closing them a week later. However it appears to have stopped closing older issues. I wonder if you might have any idea why? This is my stale.yml
https://github.com/TypeStrong/ts-loader/blob/master/.github/stale.yml
And here's my issues: https://github.com/TypeStrong/ts-loader/issues
As you can see lots of older issues are being left as is...
For example this one: https://github.com/TypeStrong/ts-loader/issues/54
Is marked as stale but hasn't been closed. I can't work out why...

@johnnyreilly Is this working now? We were having some OOM issues on Stale so it would fail to work, but it should be resolved now.

@tcbyrd I have the same problem. markComment and closeIssue don't work.
See
https://github.com/Semibold/Weibo-Picture-Store/blob/master/.github/stale.yml
https://github.com/Semibold/Weibo-Picture-Store/issues/87

@Aqours Your example is exactly why I want to make some updates to Stale. I tested your stale.yml and noticed the following errors, but there's currently nothing that reports this to you:
Error: ValidationError: child "closeComment" fails because ["closeComment" must be a string or false]

Since you have true, that's getting interpreted as a boolean, when what it wants is a string (a comment to use when it closes the issue). If you update that and/or just remove the closeComment field, it should start working.

@tcbyrd Thanks. It works after setting closeComment to false

Awesome! I'm going to close this since I think we've resolved all the OOM issues that were causing it to crash. Anyone else please feel free to re-open this is you're still having problems.

Sorry for the delay in responding - I'm not sure the issue is resolved since I still have examples of stale but not closed issues: https://github.com/TypeStrong/ts-loader/issues/54

@johnnyreilly That issue isn't labeled wontfix. The call to add the label probably failed while we were dealing with OOM issues, and since the bot's comment was added < 60 days ago it hasn't come up in the search for stale issues. Tomorrow is 60 days since Jan 19th, so it should get labeled again on tomorrow's sweep, or you can mark it wontfix manually today to see if it gets auto-closed on the next sweep. Providing a report of what Stale is doing on your repo is something I'm hoping to add as well. Right now if anything goes wrong it just fails silently with no way to re-try.


update infra

use .net sdk to generate assembly info from msbuild props
remove defaults
refactor extract shared props in a Directory.Build.props file
append compiler defines, instead of override defaults (like DEBUG)
pin .net sdk to 2.1.401
default of dotnet pack is Debug configuration, use Release
test Release conf, otherwise is built two times
use build.bat to create the packages in appveyor

Sweet!


Support multiple route pairs in docker labels

re: https://github.com/umputun/reproxy/discussions/76
In some cases, one container may expose multiple endpoints, for example, public API and some admin API. They can be on different ports and the goal is to allow such functionality to be defined with docker provider (labels?)
I can think of two ways to implement it:

add numeric suffix to reproxy.*.N, i.e. reproxy.server.1=example.com, reproxy.dest.1=blah and so on. This may increase the number of labels and could be ugly in extreme cases
similar to the first approach but combine the rule in this style of static provider, i.e. reproxy.1=m.example.com,/api/(.*)./someplace/$1,/ping. This is harder to read but will minimize the number of labels

both approaches can be combined too, and the user can choose which one he likes better. The current labels should be supported too as we don't want to break compatibility for no good reason
if anyone has better ideas pls share

cc @avowkind

Thanks for looking at this.
Here's an example that might illustrate.

split the labels into multiple groups.
indicate which port each group should use. or assume from the order of the port defs

version: '3.5'
services:
  computer-vision:
    image: pfr-computer-vision:latest
    ports:
      - 14000:14000 <-- port 1
      - 14001:14001  <-- port 2
    labels:
      reproxy.1.ping: '/api/v1/ping'
      reproxy.1.route: "(/api/v1)"
      reproxy.1.dest: "$scheme://$host:$port.1/$path"
      reproxy.1.enabled: true
      reproxy.2.ping: '/api/v2/ping'
      reproxy.2.route: "(/api/v2)"
      reproxy.2.port: 14001
      reproxy.2.dest: $$1
      reproxy.2.enabled: true

yeah, this is the format I've implementing except for one thing - I don't see the need in reproxy.N.enabled because this label enables the container for discovery, not the particular route. So, the current reproxy.enabled should be sufficient.
The N should be in [0:9] range, i.e. reproxy.7.server, reproxy.0.port, and so on. Current labels back-compatible and treated internally as reproxy.0.*
Check out the :master image as you have a chance. I have not tested it on the real setups yet but covered with enough unit test, so hopefully, it will work as designed.
pls note - this is not limited to different ports only and reporxy.1.* and reproxy.2.* labels may have the same port or even the default one (the first exposed if nothing defined in reproxy.N.port). I have recently witnessed a complex config with a lot of custom routes mapped for a single container and the same port (service)

actually, I just deployed it with multi-routes to one of my boxes and it worked fine. This is the relevant part of the compose:
feed-master:
    image: umputun/feed-master:master
    container_name: feed-master
    hostname: feed-master
    restart: always
    logging: *default_logging
    labels:
      reproxy.1.server: feedmaster.umputun.com
      reproxy.1.route: '^/(.*)'
      reproxy.2.server: feedmaster.umputun.com
      reproxy.2.route: '^/echo/image.png$$'
      reproxy.2.dest: '/image/echo-msk'
      reproxy.3.server: feedmaster.umputun.com
      reproxy.3.route: '^/all-umputun/imgage.png$$'
      reproxy.3.dest: '/image/all-umputun'
      reproxy.4.server: feedmaster.umputun.com
      reproxy.4.route: '^/echo$$'
      reproxy.4.dest: '/rss/echo-msk'
      reproxy.5.server: feedmaster.umputun.com
      reproxy.5.route: '^/all-umputun$$'
      reproxy.5.dest: '/rss/all-umputun'

    environment:
      - MHOST
      - TELEGRAM_TOKEN=${TELEGRAM_TOKEN}
      - DEBUG=false
      - FM_DB=/srv/var/feed-master.bdb
      - FM_CONF=/srv/etc/fm.yml

I'm still waiting for labels port and I'll delete Traefik :-)

I'm still waiting for labels port (I'm use docker swarm) and I'll delete Traefik :-)

Not sure what is missing with the current reproxy.port, can you provide details?

I'm still waiting for labels port (I'm use docker swarm) and I'll delete Traefik :-)

Not sure what is missing with the current reproxy.port, can you provide details?

Sorry. I'm didn't notice this before. I'll have to check this.
Reproxy could not found containers for one service on two docker swarm servers. I'm thought this may fix with the port.

hi @umputun , please add port/enabled to File provider description, it may be helplfull


Separate directories for map files

Hi!
I find it rather odd that the option mapRoot exists without a corresponding option which outputs the map files to said directory. I have to manually move them after transpiling, which is annoying (AND keep the same directory structure).
Here is a SO question which is valid for my case aswell: https://stackoverflow.com/questions/43052200/can-i-place-webpack-source-maps-and-source-code-files-in-seprate-folders
I guess it's kinda related to: https://github.com/microsoft/TypeScript/issues/6723

Why did you ignore the issue template?
TypeScript design non-goal:

Provide an end-to-end build pipeline. Instead, make the system extensible so that external tools can use the compiler for more complex build workflows.

@kitsonk I don't really think this falls under Provide an end-to-end build pipeline.. The compiler produces 3 types of file as output, .js files, .js.map files, and d.ts files. You can currently control the output directory for two of those (--outDir and --declarationDir), why not the third ? Especially since we do have the option to set the mapRoot directory.

Not much different from declarationDir but we waited a pretty long time on that one


⚠ Bofrost.be has degraded performance

In ceea929, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1324 ms

Resolved: Bofrost.be performance has improved in 17b9fc5 after 20 minutes.


useQueries hook causing infinite re-renders when using result as a dependency in useEffect

Other Similar Issues: #2991
Infinite re-renders happen when trying to update local state with other than reference type of queries response  in useEffect where we added queries result in dependencies array.
And also when any query response is not consistent(Meaning for first call of "posts" search gives [posts] and when no posts, return response as {message: "No data found."}), You will infinite re-renders.
Here are some examples to get better idea:
function App() {
  const [fetchedReferenceTypeData, setFetchedReferenceTypeData] = useState([]);
  const [localReferenceTypeState, setLocalReferenceTypeState] = useState([]);
  const [primitiveState, setPrimitiveState] = useState(1);
  const queries=useQueries([
    {
      queryKey: 'posts',
      queryFn: ()=>fetchPosts()
    },
    {
      queryKey: 'bookmarks',
      queryFn: ()=>fetchBookmarks()
    },
  ])

  // ✅ - No Issues
  useEffect(() => {
    setPrimitiveState(2);
  }, []);

  // ✅
  useEffect(() => {
    setPrimitiveState(3);
  }, [queries]);

  // ✅
  useEffect(() => {
    setLocalReferenceTypeState(["1"]);
  }, [primitiveState]);

  // 🔴 - Cause Infinite re-renders since we get new array from useQueries everytime in this case.
  useEffect(()=>{
    // Updating the local reference type with data other than the response from queries
    setLocalReferenceTypeState(["1"]);
  }, [queries]);

  // 🔴
  useEffect(()=>{
    // Updating the local reference type with data other than the response from queries
    setFetchedReferenceTypeData(["1"]);
  }, [queries]);

  // ✅
  useEffect(()=>{
    // Assume here posts response type id [] of posts
    setFetchedReferenceTypeData(queries.data.data);
  }, [queries]);

  // 🔴
  useEffect(()=>{
    // Can still cause infinite re-renders when there is in-consistent response from any of the queries
    // Assume here posts response type is {} when there are no posts, In this we can directly add some cond in fetchPosts func itself to return consistent data type.
    setFetchedReferenceTypeData(queries.data.data);
  }, [queries]);

  return
This is the weird hook from react-query.
;
}

Your minimal, reproducible example
Please look at code snippet provided
Steps to reproduce
Hope you got better Idea from above snippet.
Expected behavior
useQueries should memoize the response and should not force the consumed component to update same response when adding as dependency in useEffect.
How often does this bug happen?
Every time
Platform
OS: MacOS
Browser:Chrome
TanStack Query version
v3.39.3

same as useQuery, where the top level object is also a new object in every render cycle, the useQueries array is "new", but the data property in each one is not.
The code snippets don't make much sense to me, as they don't show any real life use-case. What would you do in a useEffect that runs whenever any data changes? Writing to local state is a bit of an anti-pattern in itself. So can you please show an example, preferably a codesandbox reproduction, where this feature would be useful?

@TkDodo for useQuery this is fine since you'll very likely be destructuring the object and depend on the specifics you need, but for useQueries this is a bit different because you need to access each value to be able to do so. One example I can think of, and that I'm facing right now is if you need to do memoization depending on the queries:
function Component() {
  const queries = useQueries();
  const someExpensiveCalculation = useMemo(() => {
    queries.map(/* ... */);
  }, [queries, /* [...] */); // memoization is broken because of `queries`
}

The only way I could fix it for that case for now was to pick the attributes I needed (data, isError, isLoading, etc) and to force a shallow comparison to make the reference stable.

I'll think about this problem. I guess what would be good is to have a combine of some sort that could iterate over all queries, something like:
const result = useQueries({
  queries: [query1, query2, ..., queryN],
  combine: (queries) => ...
})

whatever you return from combine would be the result, so per default, it would just be: combine: (queries) => queries to be backwards compatible.
I'm not sure if we could do this and have it work well with TypeScript, so I'm looping in @artysidorenko

Thanks!
The way I solved it for now was by doing something like:
function shallowCompareObjects<T extends {}>(a: T, b: T) {
  return Object.keys(a).every((key) => a[key] === b[key]);
}

function shallowCompareCollections<T extends {}>(a: T[], b: T[]) {
  return a.length === b.length && a.every((item, index) => shallowCompareObjects(item, b[index]));
}

function useMemoizedCollecionPick<T extends {}, P extends keyof T>(array: T[], pickValue: readonly P[]): Pick<T, P>[] {
  // eslint-disable-next-line react-hooks/exhaustive-deps
  const stablePickValue = useMemo(() => pickValue, [JSON.stringify(pickValue)]);
  const value = useMemo(() => array.map((item) => pick(item, stablePickValue)), [array, stablePickValue]);
  const prevValue = useRef(value);
  return useMemo(() => {
    if (shallowCompareCollections(value, prevValue.current)) {
      return prevValue.current;
    }
    prevValue.current = value;
    return value;
  }, [value]);
}

const result = useQueries();
const pickedResult = useMemoizedCollecionPick(result, ['data', 'error', 'isLoading']);

which gets me a little bit closer to useQuery destructuring inside whatever hook I need to use.
I'm guessing combine could work too assuming you update it only when the values change. TS for it should be fine with something like this

Heya, yeah I just had a play around locally as well, I think something like the below will work well in getting the types for this*.
export function useQueries<T extends any[], CombinedResult = QueriesResults<T>>({
  queries,
  context,
}: {
  queries: readonly [...QueriesOptions<T>]
  context?: UseQueryOptions['context']
  combine?: (result: QueriesResults<T>) => CombinedResult
}): CombinedResult {

*the result parameter doesn't suffer from those other callback param-typing issues; all the inference happens inside QueriesOptions so you get the correct type in the combine function

I started working on this, but it's not trivial. I wanted to add it on observer level to have this feature available in core, but we determine suspense and error boundary features on framework level. If combine takes away the information we need, we can't do that anymore.
So I think it needs to happen on framework level, after suspense / errorBoundary handling. But there, we'd have a harder time memoizing the previous value to also guarantee referential stability. It might be that I'll add combine first, but it won't have any referential stability improvements, and it would also run on every render.
Oh also, if it comes, it will come in v5 because we have so many changes already that I don't want to deal with the conflicts 😅 .

shipped in v5 alpha.34

For those who can't update yet, my current fix for memoizing from the queries is to increment a counter on success and then memoize from that counter:
// Doesn't work
const queries = useQueries(/* */);
const expensiveValue = useMemo(() => {/* */}, [...queries]);

// Does work
const successfulQueriesCounter = useRef(0);
const queries = useQueries([
  { onSuccess: () => successfulQueriesCounter.current = successfulQueriesCounter.current + 1 }
]);
const expensiveValue = useMemo(() => {/* */}, [successfulQueriesCounter.current]);


cert file link is incorrect on http, file does not exist

After running databox the non SSl site's "download certificate" link doesnt resolve, points to a file that deosn't exist. Should probably point to rootCert.pem but points to something else.

It was fixed in this https://github.com/me-box/core-container-manager/commit/aa6af0e3a1df7b9f36d40bce14bee506127c881a
Its version 0.3.2 still has this bug, you can test the fix by running ./start-databox latest
This fix will be included in the next release.

I found this is still an issue. I have installed the certificate, but still databox UI doesn't find it when running as non-ssl site.


Set arbitrary pool

In some cases (scenes) I want just to operate on some pools not all.
The idea is to auto generated method like:
 public void SetPools (bool setCore, bool setInput, ... ) {
     if (setCore)
           core = CreateCorePool ();
     if (setInput)
           input = CreateInputPool();
     ...
 }

Instead of setting every pool like:
Pools.sharedInstanced.core = Pools.CreateCorePool();
Pools.sharedInstanced.input = Pools.CreateInputPool();
...

Pools is a partial class, so you can basically implement as many methods that make sense for you as you like.
The code generator is very flexible. You can duplicate the existing PoolsGenerator and modify it to your liking and select it in the Entitas Preferences.

I'd suggest you create your own version of PoolsGenerator and make it generate your suggested code


cargo-nextest: fix tests

Description of changes
Another try at https://github.com/NixOS/nixpkgs/pull/199810.
Fixes https://github.com/NixOS/nixpkgs/issues/199803
Upstream pull request: https://github.com/nextest-rs/nextest/pull/626
Things done

Built on platform(s)

[X] x86_64-linux
[X] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

it is failing on linux now
i realized it wasn't fix on my pr because i had a typo, it should work now


Migrate to dep

Let's use dep to manage our dependencies
https://golang.github.io/dep/docs/introduction.html

We're going with modules instead.


Is it possible to declare a relationship as eager loaded?

I know you can use .Include() on a query to eager-load a related object.
What I want to do, though, is declare a relationship so that it always eager-loads, e.g. If my Thing object must always have a related OtherThing that I want to have access to:
modelBuilder.Entity<Thing>(entity =>
    entity.HasOne(t => t.OtherThing)
        .WithMany(o => o.Things)
        .HasForeignKey(t => t.OtherThingId)
        .EagerLoad(true); // or something like that

Is it possible to achieve this somehow?

See #1985, #2953


[storage] Fixing typos

I know this PR is kind of obnoxious, but I thought I could help.
I was able to detect these by using Code Spell Checker on VSCode.

@HarshaNalluru hi! some help here would be appreciated <3

Though typos, we shouldn't be shipping breaking changes, please make sure of that!!
/cc - @jeremymeng


[Starter Bug] [SR-1608] NSStream

What's in this pull request?
NSStream "is-a" implementation with tests
Input/Output streams "has-a" implementation with tests
Resolved bug number: 1608 (https://bugs.swift.org/browse/SR-1608)
I have some questions i have outlined in the PR as well

i accidentally submitted it un rebased and squashed to apple/master instead of my forks's master. will close and resubmit shortly


Allow @import to import global variables

There is a small error in importing global variables from other http files. If variables are imported with import and ref, it is working fine. If only import is used nothing gets imported, except a request was already executed from other file.

fixed in httpac 3.0


Raffstore control

Hi,
I have 15 Raffstores on my house and want to use it for shading....
But with your blind control its only possible to control the high of the blinds not the angle of the lamellae.
Do you have plans to include this in your building block?
Thanks

Hallo,
There is already some implemented in the latest version of the node. Maybe it is not released.
Regards


Bug fix for streaming records

Fix the bug where the pos/index in record stream is not moving forward correctly.
Fix the bug where the peek method starts from record 1 rather than 0.
Before the fix, some Linq methods will not consume the record when the record has been visited.
This is very wrong as it enables the user to buffer the pos in the stream and revisit it latter.
The bug is caused by not forwarding the current pos/index in the stream before yielding the record back.
The peek method now would peek the record that will be returned by calling Enumerator.MoveNext()
which means it will return record in pos 0 before Enumerator.MoveNext() is called the first time.

:+1:


Add tool to print label statistics

See IRTK's labelStats. (CC @soundray)

As noted (#696) I can emulate labelStats with the shell function below, just that it is comparatively slow (15 s versus 0.2 s).
mirtklabelstats () {
    ref=$1 ; tst=$2 ;
    evaluate-overlap $ref $tst -labels -table stdout -precision 6 | tail -n +2
    | tr , ' '
    | while read label tp fn fp tn tpr tnr dsc jsc ; do
        echo $label,$[$tp + $fn],$[$tp + $fp],$tp,$fn,$fp,$jsc,$dsc
    done
}


Add seller registration

Resolves #29. Adds a new role seller, who can create and update products

Thanks for the review 🚀


Support Stable Diffusion inpainting

I would like to use the SD 2 inpainting model from https://huggingface.co/stabilityai/stable-diffusion-2-inpainting with Bumblebee. From reviewing the code it doesn't seem like there is support for this as of yet and I would like to add it if possible.

PRs certainly welcome!

Dupe of #120.


flow-parser: #8776 isn't fixed?

Flow version: 0.165.1
https://github.com/facebook/flow/issues/8776 is closed, but maybe it isn't fixed.
Reproduce
const { parse } = require("flow-parser");

const code = `const { /* comment */ a = 0 } = b;`;

const options = {
  comments: false,
};
const result = parse(code, options);
console.log(JSON.stringify(result.body[0]));

Expected behavior
{
  "type": "VariableDeclaration",
  "declarations": [
    {
      "type": "VariableDeclarator",
      "id": {
        "type": "ObjectPattern",
        "properties": [
          {
            "type": "Property",
            "key": {
              "type": "Identifier",
              "name": "a",
              "typeAnnotation": null,
              "optional": false
            },
            "value": {
              "type": "AssignmentPattern",
              "left": {
                "type": "Identifier",
                "name": "a",
                "typeAnnotation": null,
                "optional": false
              },
              "right": {
                "type": "Literal",
                "value": 0,
                "raw": "0"
              }
            },
            "kind": "init",
            "method": false,
            "shorthand": true,
            "computed": false
          }
        ],
        "typeAnnotation": null
      },
      "init": {
        "type": "Identifier",
        "name": "b",
        "typeAnnotation": null,
        "optional": false
      }
    }
  ],
  "kind": "const"
}

Actual behavior
{
  "type": "VariableDeclaration",
  "declarations": [
    {
      "type": "VariableDeclarator",
      "id": {
        "type": "ObjectPattern",
        "properties": [
          {
            "type": "Property",
            "key": {
              "type": "Identifier",
              "name": "a",
              "typeAnnotation": null,
              "optional": false
            },
            "value": {
              "type": "AssignmentPattern",
              "left": {
                "type": "Identifier",
                "name": "a",
                "typeAnnotation": null,
                "optional": false
              },
              "right": {
                "type": "Literal",
                "value": 0,
                "raw": "0"
              }
            },
            "kind": "init",
            "method": false,
-          "shorthand": true,
+          "shorthand": false,
            "computed": false
          }
        ],
        "typeAnnotation": null
      },
      "init": {
        "type": "Identifier",
        "name": "b",
        "typeAnnotation": null,
        "optional": false
      }
    }
  ],
  "kind": "const"
}

Note: When use comments: true option, shorthand property is true as I expected.
Additional Context
This is also blocker of prettier (https://github.com/prettier/prettier/pull/11845)

Thanks for raising this!
It looks like the fix didn't actually apply to cases where the shorthand object property is on the LHS of an assignment. It should be straightforward to fix for the LHS as well.
I am mystified why this just started being a problem in a recent version, as reported in #8776, since the change we believed was the culprit there doesn't affect LHS properties...


Visible URL used as a link target is not transforme into API URL (nor UID)

Something we have seen in production Volto:

User creates link in text block by selecting text and clicking linking action
Because of huge amount of similarly named pages, user is unable to choose the right page by searching it
As a workaround, user pastes visible URL from browser as a link target (e.g. "https://volto.kitconcept.com/a-new-page")
User saves the page. Link seems to somewhat work. User is happy.

Unfortunately, the target URL is saved as such and never transformed into the backend API url nor UID at the backend.
A better link target dialog might help to fix this somewhat, but I wonder if there is realiable way to detect published URL and transform it into API URL during save.

We saw this happening as well. We should convert those links into UUIDs internally.

I’d guess that either Volto frontend or middleware should detect this and convert the visible URL into API url. As long as the API URL really exists on the backend, REST API (block transforms) ould convert it into UUID.
Timo Stollenwerk<EMAIL_ADDRESS>kirjoitti 6.6.2020 kello 16.22:

We saw this happening as well. We should convert those links into UUIDs internally.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Currently, internal links are converted to UID-based URLs in general. If this still doesn't work, please open a new issue.


Styled Scroll Bar

Acceptance

[x] As a user I should see a styled scrollbar designed according to the figma mockup instead of the default scollbar.

Worked on this on pr #369

I created a styled scroll bar for the page


add go-mod support and turkish chars + golangci-lint

well, i did not changed a lot. just gofumpted code. added 3-4 lines (chars for mapping Turkish letters). if you check you'll see nothing big has changed 8)

You are missing the point of my comment ^^
Either way, I am not the maintainer and that was just a friendly tip.


Backport identical tab names fix

Fixes Atom hanging indefinitely upon encountering two identical filenames located in different directories in 1.3.x.
(Is cherry-picking the correct way to do this?  It feels really weird creating a PR that uses someone else's commits)

Thanks for this @50Wliu!

And yeah, once we've already merged it into master, we have no choice but to :cherries: pick. It creates some duplicate entries in the history, but it's the approach we're going with for now.


Installation with pipsi works but pew does not.

Hi, I have fresh OsX and I wanted to install pew the recommended way using pipsi however it looks like there is some problem.
@berdario could you please look at this pipsi issue 144 might be related problem?

You should use PipX instead of Pipsi.
Pipsi project is stopped since August 2015…

The installation instructions should probably be updated. I'll look into that.

Uh, I didn't even notice that pipsi releases had been lagging far behind. I was still happily using it.
Since pipsi still seems to work in some use cases, I think we should add a mention for PipX, rather than substituting it (or is PipX so much better?)
I haven't yet got around to reproduce the issue that you reported, sorry about that


Can't drag the scrollbar outside of the vscode window

On Arch Linux - gnome 3.30. Seems like in the latest version (1.27.2) the scrollbar can't be dragged while the cursor is outside of the vscode window. Not sure when this happened but it used to be possible. Without releasing the mouse button:

No code change in there since >1yr. If this is a regression, then it must be caused by the Electron update.

Not sure if this is related or helps but the scrollbar works fine when the "release notes" are displayed.

It is fixed on 1.31.1


Not a issue, but a question about your project (comparating it to another solution) #76

Hi,
I would like to ask/understand, if there is a major difference in features offered by your lib, comparated to this one:
https://firelordjs.com/
Thanks you,

This project is easy to drop into a Svelte app and makes it effortless to use Firestone server and client side. It also includes the ability to use types for type safety and intellisense.
At a quick glance, FireLordJS is framework agnostic, doesn't offer convenient Svelte initialization nor components but is more comprehensive in its type offerings.


Streaming directly into file on storage downloads.

This won't work until google-resumable-media==0.2.2 is actually out, but I did install it locally and the unit tests are passing.
Fixes #3703.

OK great the release it out now, will restart the build!


refine metabolomics/sample processing schemasheet

I obtained TermsUpdated_organicmatterextraction.xlsx from @anastasiyaprymolenna who obtained it from @mslarae13. It's in a format that could be a nice schemasheet with a little tweaking.

[x] Added second schemasheets header row with terms from the LinkML/scemasheets voculary
[x] added is_a column with two high level classes

Activity
MaterialEntity

[ ] New is_a parents will need definitions and/or slot URIs
[ ] Links/slot_uris should be the URI of a term, not a URL that points to a web page on which the term appears
[ ] Is Aliquot a MaterialEntity or an Activity?
[ ] Include Activity suffix on Activity term labels? should be consistent
[ ] Strong preference should be given to terms from MIxS, the OBO foundry, NCIT

[ ] Avoid  IUPAC Gold Book and National Cancer Institute Proteomic Data  Commons if possible. They don't provide logical definitions.
[ ] Example challenge: there are OBO and NCIT Acidification terms, but they have overly specific roots like "food processing" or "physiological process"

[ ] In schemasheets, slots should be defined outside of classes, and then assigned inside of classes, with slot_uage overrides if necessary
[ ] Double check agreement between slot_uri and textual and logical definitions
[ ] It's almost always a bad practice to change the label of a term defined elsewhere. The definer's label should be the LinkML name and or title, or minimally should appear in the aliases.
[ ] We should retain has_input and has_ouput slots to show how the classes modeled in this sheet relate to one another.

I would prefer to use PATO's highly reused concentration term over EDAM's. Its URI is http://purl.obolibrary.org/obo/PATO_0000033 and it can be found on this page: https://www.ebi.ac.uk/ols/ontologies/pato/terms?iri=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FPATO_0000033
The CURIE form preferred in LinkML schemas would be PATO:0000033

Isn't http://purl.obolibrary.org/obo/CHMO_0001640 a better URI for filtration, compared to http://purl.obolibrary.org/obo/CHMO_0000999 ?

we're not using schemasheets any more


Jest tests aren't working

Compilation of the project works fine but as soon as try to run tests it fails with an odd syntax error that is not present when the craco-esbuild is disabled in craco.config.js.
Perhaps there is a way to add this plugin for build & start leaving tests as is?
Any help would be appreciated.
yarn test
yarn run v1.22.10
$ craco test --watchAll=false
 FAIL  src/utils/__test__/utils.spec.ts
  ● Test suite failed to run

    Transform failed with 1 error:
    <stdin>:59:4: error: Expected "}" but found ":"

      at failureErrorWithLog (node_modules/esbuild/lib/main.js:1443:15)
      at node_modules/esbuild/lib/main.js:1254:29
      at node_modules/esbuild/lib/main.js:606:9
      at handleIncomingPacket (node_modules/esbuild/lib/main.js:703:9)
      at readFromStdout (node_modules/esbuild/lib/main.js:573:7)
      at runServiceSync (node_modules/esbuild/lib/main.js:1808:3)
      at Object.transformSync (node_modules/esbuild/lib/main.js:1650:3)
      at Object.process (node_modules/esbuild-jest/esbuild-jest.js:54:36)
      at ScriptTransformer.transformSource (node_modules/@jest/transform/build/ScriptTransformer.js:464:35)
      at ScriptTransformer._transformAndBuildScript (node_modules/@jest/transform/build/ScriptTransformer.js:569:40)
      at ScriptTransformer.transform (node_modules/@jest/transform/build/ScriptTransformer.js:607:25)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        1.565 s
Ran all test suites.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

I am using react-scripts 4.0.3

Happy to accept a pr to add an option for this :)

Published as 0.3.3 thanks for the pr @grigory-bogush

hey @grigory-bogush
did you have any lead on why transformation fails? 🤔


I'm still getting this issue['RNGADBannerView was not found in UI']  after trying all the instructions

Same here


[Merged by Bors] - fix(lint_style): only print style errors once overall

The previous code meant that once an error was found, all errors so far would be printed after checking every module, leading to hundreds of repetitions of the same warning.

Thanks!
bors r+

Thanks for the really fast review!
For posterity: this regressed in #14676.


Add TA-88047

This change is

@rmloveland Can you amend your commit and force-push? Retrying the build in Netlify did not properly sync the status to GitHub.


[WIP] Fixed dwm-driver to compile on linux, as a temporary half-way solution until FAST.Farm is available.

DO NOT MERGE THIS YET!
Dear OpenFAST team,
This  is a dirty fix for the DWM driver. This should compile ONLY on Linux (tested only on CentOS 7.4). You will notice there are debug messages (I am not experienced with fortran and I didn't have the time to learn a debugger).  I have left the debug messages so a potential other developer will not have to put them there again.
Due to the fact that FAST.Farm is still not publicly released, I needed to fix this. This is shared so it may be useful for others. The results were briefly validated with what DWM gives on Windows.
The DFLIB was changed to IFQWIN where needed.
The hardcoded paths were changed to linux hardcoded paths so this runs on linux.
I didn't have the patience to find out how to get platform independent paths in fortran.
This is not ready for merging - the paths should be platform independent.
We are sharing in hope that this is going to facilitate other researchers.
On behalf of the ETH-WINDMIL team,
Charilaos Mylonas

@mylonasc see https://github.com/OpenFAST/openfast/issues/2 and https://github.com/rafmudaf/openfast/tree/bugfix/dwm_driver.
If that does not solve your current problem, I'm happy to help find a full solution. As stated in issue #2, we don't intend to fix DWM driver since fast farm will be released soon(ish) so I'd like to close this pull request and move any discussion to issue #2.
PS the Sys*.f90 configuration files set a PathSep variable that should be available in modules using NWTC Lib.


Initial Go Serverless Application Example

Initial Go serverless application example which demonstrates how to use
Amazon API Gateway WebSockets and Go. This example contains all the source,
IaC, configuration files, and build scripts to build and deploy the example
to AWS.
Issue #, if available:
Description of changes:
Initial Go Serverless Application Example
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Will somebody review this please?


reverse #96

PR 96 introduced some unintended changes. Now reversing them back.

Codecov Report

Merging #106 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   52.64%   52.64%
=======================================
  Files         100      100
  Lines        1474     1474
=======================================
  Hits          776      776
  Misses        698      698

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 ca371fe...0dd7cd7. Read the comment docs.

Coverage remained the same at 49.249% when pulling 0dd7cd729a8da229868925e857bab8944e58c70d on yingzhao-1:yizha/Reverse_96 into ca371fe5fc282216a0207867a3a77a2f7b6c21ec on Azure:master.


Revert "Add test for variable interpolation in HTML"

Fix broken travis build.

Reverted addition of test case that tried to add an interpolated string inside of an html comment, but it's evaluated as plaintext like this says:

Jade does not have any special syntax for conditional comments. If your line begins with < then it is > treated as plain text.

A test case this there for a reason. The code should be fixed if the test fails.

I don't understand... sounds like you are proposing a feature be added? In my mind this is currently working as designed.

We allow interpolation in plain text. We already have // for true comments.  HTML comments are intended for things like knockout conditionals, where you would want interpolation to work.

That makes sense. Thanks for clarifying.


Complete I32x4 swizzles

This PR both completes all swizzles for I32x4 and adds tests for them.
Test coverage is now 20.90% because all new lines are covered.
I will add more tests for existing code next.

Thanks!


Fix typos

Found few misspellings. No code change.

Hello @szepeviktor thanks a lot.
I will ask @pxpm to merge.
Thanks.

Thanks @szepeviktor
Much appreciated 🙏

Glad to contribute.


⚠ Portainer has degraded performance

In d09052d, Portainer (https://portainer.wpang.net) experienced degraded performance:

HTTP code: 200
Response time: 5784 ms

Resolved: Portainer performance has improved in 13b0946 after 19 minutes.


firstNotification.launchImageName not working

Hi,
I would like to add an image on the localnotification but i've got this error:
'launchImageName' is inaccessible due to 'internal' protection level
Can you help me to fix it please ?
Thanks in advance

I have not been able to reproduce your issue but I created a test to prove it works
https://github.com/d7laungani/DLLocalNotifications/blob/4b2c3a20c8d38a889a920ad0e1f6e9cf57d80a9e/DLLocalNotificationsTests/DLLocalNotificationsTests.swift#L151

I have the same issue, not sure if it's related to me using Swift 5.0 but reading the code the error seems to make sense, default visibility in Swift is internal and the member launchImageName is declared without a visibility modifier.
Are you sure that launchImageName shouldn't be declared as public var launchImageName: String? ?
Not sure how this isn't encountered by more people.


In some instances, the animation procedure breaks.

Current fix: play the animation, then wait 1 tick and play it again.


eligibility beacon should use a confidence param bigger than hdist

Overview / Motivation
hdist basically defines the the amount of layers that pbase can get stuck before self healing
we should make sure the confidence param is bigger than hdist to avoid fatal errors
The Task
maybe its better to use hdist+1 as the confidence

confidence parameter should be about the size of the tortoise window, as written in https://github.com/spacemeshos/go-spacemesh/issues/1056

Will handle as part of work on self-healing


Project Manager fully integrated

I know that there is an extension for this, but I think a project Manager would be really useful for everyone and better if it starts on startup with VSCode. I have to wait ~7 sec until I can open the project manager extension.
When you have a look at Web/PHPStorm, they have a nice overview on the start (can be a pop-over box) where all projects are listed. This would be very useful for people that are working on many different projects. So they could easily and faster switch between them.
Here is the extension many people are using daily (like me): https://github.com/alefragnani/vscode-project-manager

Average VS Code start up time is around 2s. Please check with the project manager extension to know why it is slow.
There are no plans to integrate project manager in VS Code.

What about an API to give this extension the ability to great a popover layout that displays recent and saved projects on startup plus sort options for categories or tags?
Sent with GitHawk

There is a view API where you can build the views you want


Unneeded runtime dependencies

The following runtime dependencies are not, as far as I can tell, actually needed at runtime:

bower
ember-changeset
ember-changeset-validations

I think bower can just be removed entirely.
The addon is built to use ember-changeset and ember-changeset-validations, but is usable without them, so putting them in the runtime dependencies makes the addon less lightweight, and requires maintenance to keep up with the latest version (e.g. #69).
I propose that we move them from dependencies to devDependencies and then either:

Add info to the documentation about installing them
Install them into apps via a blueprint

I guess this would require a major version bump as it would be a breaking change to anybody using ember-changeset or ember-changeset-validations that hasn't put them in their own package.json.
Thoughts? I'm happy to open a PR removing bower and implementing either of the options above.

Hi @bendemboski, thanks for bringing this up! I just discussed it with my colleague, and we'd agree to splitting out ember-changeset and ember-changeset-validations and installing them via a blueprint.
I guess that bower is still needed for running tests, so instead of dropping it completely we could move it to devDependencies.
We're looking forward to your PR!

Closed in v1.0.0


Backstage Working Group - DevOps Automation - Mar 30, 2023

Date
Thursday, Mar 30 2023 - 11am ET; 3pm UK
Untracked attendees

Name
Firm
Comment

Meeting notices

FINOS Project leads are responsible for observing the FINOS guidelines for running project meetings. Project maintainers can find additional resources in the FINOS Maintainers Cheatsheet.

All participants in FINOS project meetings are subject to the LF Antitrust Policy, the FINOS Community Code of Conduct and all other FINOS policies.

FINOS meetings involve participation by industry competitors, and it is the intention of FINOS and the Linux Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws. Please contact<EMAIL_ADDRESS>with any questions.

FINOS project meetings may be recorded for use solely by the FINOS team for administration purposes. In very limited instances, and with explicit approval, recordings may be made more widely available.

Agenda

[ ] Convene, roll call, welcome new people
[ ] Display FINOS Antitrust Policy summary slide
[ ] Approve previous meeting minutes
[ ] Review Backstage WG project board
[ ] AOB, Q&A & Adjourn (5mins)

Decisions Made

[ ] Decision 1
[ ] Decision 2
[ ] ...

Action Items

[ ] Action 1
[ ] Action 2
[ ] ...

Zoom info
Join Zoom Meeting

https://zoom.us/j/95521041942
Meeting ID:<PHONE_NUMBER>2

Github Repo: https://github.com/finos/devops-automation/
Project Board: https://github.com/orgs/finos/projects/33
Mailing List: Email<EMAIL_ADDRESS>to subscribe to our mailing list

Hi all 👋🏻

Hi @mjorgen

Invalid meeting ID ?

The Zoom details are now the following : https://zoom.us/j/95521041942

Hi!

Hello

hello!

Hey!

Hello


mixing of async and callback styles

@Eomm @mcollina,
I have been trying to follow the book as I am learning many things I wish I had before diving into my own Fastify backend. I ran into a Fastify warning FSTWRN002 in Chapter 8. It was sourced to the loader.js. If I remove the "next" callback, the warning does not display.
The warning I am getting:
(node:12133) [FSTWRN002] FastifyWarning: The application-schemas plugin being registered mixes async and callback styles, which will result in an error in fastify@5
File in question:
Chapter 8/schemas/loader.js

Hi,
thanks for reading the book.
The FSTWRN002 warning spots wrong plugin interface declaration such as:
WRONG declaration
// async function with next
async function plugin(fastify, opts, next) {}

// sync function and missing `next`
function plugin(fastify, opts) {}

If I remove the "next" callback, the warning does not display.

I assume the plugin function is an async function, for this reason removing the next callback argument is correct.

In the book I checked that the spelling is correct and the warning is not there:
https://github.com/PacktPublishing/Accelerating-Server-Side-Development-with-Fastify/blob/3335e1297a717a0ff8d1170436d09b9f8693fefe/Chapter 8/schemas/loader.js#L5-L11


[Feature Request] File Encryption

Since the PIN lock is really just for show, it would be neat to have actual encrypted files. There's a great app NoteCipher by @n8fr8 which uses SQLCipher (source), a library for strong encryption. Since NoteCipher is more of a demo app (and thus not the greatest on features and usability), it would be cool to see it implemented in Writeily which is beautifully designed.

We're happy to answer questions and help anyone take this on.  You can contact us via IRC, email, and our mailing list https://guardianproject.info/contact
There is also https://github.com/guardianproject/IOCipher which makes it really easy to store files in an per-app encrypted store.  So if writeily is file-based or SQL-based, either way should be easy.

I'm interested in looking at adding this - I've been meaning to find an excuse to learn how to use SQLCipher and IOCipher (I'm looking at integrating them into https://github.com/i2p/i2p.i2p-bote.android to improve its encrypted filesystem).
I've pulled down the writely-pro repo and built it. How does it interact with regular writely? The Google Play page indicates that Writely Pro is an extension of Writely, but I can't find any reference to it in writely-pro other than anddown/anddown.iml, which is only relevant for Android Studio.

Wow, wasn't expecting this much response! Glad to see the interest.
@str4d I believe Writeily Pro is a wholly separate app than Writeily, which it may be based on but doesn't receive many updates.

yep, @tedthqme is correct.
https://github.com/jffrymrtn/writeily/issues/18#issuecomment-66896034
also, this is awesome, guys


NotFound from 'POST https://api.github.com/actions/runner-registration'

This is the log when using a docker-compose.yml and docker-compose up -d:
runner_1  |
runner_1  | --------------------------------------------------------------------------------
runner_1  | |        ____ _ _   _   _       _          _        _   _                      |
runner_1  | |       / ___(_) |_| | | |_   _| |__      / \   ___| |_(_) ___  _ __  ___      |
runner_1  | |      | |  _| | __| |_| | | | | '_ \    / _ \ / __| __| |/ _ \| '_ \/ __|     |
runner_1  | |      | |_| | | |_|  _  | |_| | |_) |  / ___ \ (__| |_| | (_) | | | \__ \     |
runner_1  | |       \____|_|\__|_| |_|\__,_|_.__/  /_/   \_\___|\__|_|\___/|_| |_|___/     |
runner_1  | |                                                                              |
runner_1  | |                       Self-hosted runner registration                        |
runner_1  | |                                                                              |
runner_1  | --------------------------------------------------------------------------------
runner_1  |
runner_1  | # Authentication
runner_1  |
runner_1  | Http response code: NotFound from 'POST https://api.github.com/actions/runner-registration'
runner_1  | {"message":"Not Found","documentation_url":"https://docs.github.com/rest"}
runner_1  | Response status code does not indicate success: 404 (Not Found).
runner_1  | .path=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
runner_1  | Starting Runner listener with startup type: service
runner_1  | Started listener process
runner_1  | Started running service
runner_1  | An error occurred: Not configured
runner_1  | Runner listener exited with error code 2
runner_1  | Runner listener exit with retryable error, re-launch runner in 5 seconds.

Needless to say that it doesn't even start up. Any idea whats going on?

This is my docker-compose.yml:
version: "3.7"

services:
    runner:
      image: tcardonne/github-runner:latest
      environment:
        RUNNER_NAME: "runner-x"
        RUNNER_REPOSITORY_URL: ${RUNNER_REPOSITORY_URL}
        RUNNER_ORGANIZATION_URL: ${RUNNER_ORGANIZATION_URL}
        GITHUB_ACCESS_TOKEN: ${GITHUB_ACCESS_TOKEN}
        RUNNER_TOKEN: ${RUNNER_TOKEN}
        RUNNER_LABELS: ${RUNNER_LABELS}
      volumes:
        - /var/run/docker.sock:/var/run/docker.sock

According to the github doku 404 is usually an authentication error.
https://docs.github.com/en/rest/overview/troubleshooting
Try using only RUNNER_REPOSITORY_URL or RUNNER_ORGANIZATION_URL
And only RUNNER_TOKEN or GITHUB_ACCESS_TOKEN
it worked for me perfectly. But i never tried to provide both tokens.

I go this error when the token expired after a few hours.


NOBUG: Add validation and error message for slugs

Jira Ticket:
Relates to CM-314 / CM-318
Jira Ticket URL:
Relates to https://bcparksdigital.atlassian.net/browse/CM-314 / https://bcparksdigital.atlassian.net/browse/CM-318
Description:

Added a slug validation ^[a-z0-9\-\/]+(?:-[a-z0-9\-\/]+)*$|^$ to allow editors to enter lowercase letters, numbers, hyphens, or slashes, but no spaces, based on this doc
Added this validation to Pages, ParkSubPages, ProtectedAreas, and Sites
Added a custom error message to let editors know

maybe we could a bunch of different slug validators with different regex expressions and stack them for the particular field based on it's unuque circumstances?

slugCharacterValidator - checks for valid characters and consecutive forward slashes (the current validator plus my comment above)
slugLeadingSlashValidator - a leading slash is required
slugNoLeadingSlashValidator - a leading slash is not allowed
slugTrailingSlashValidator - a trailing slash is required
slugNoTrailSlashValidator - a trailing slash is not allowed
slugNoSlashValidator - foward slashes are not allowed in the slug

you would stack them like this.....
    beforeUpdate: async (params, data) => {
      slugCharacterValidator(data.Slug)
      slugNoSlashValidator(data.Slug)
    },

This is looking good, but I noticed a couple of small bugs while testing.

If I leave the slug blank on a Site, I get the error about consecutive forward slashes.  I believe sites should still allow blank slugs?

We probably need to add better handling for nulls.  I was getting this error in my console
[2023-01-18T17:58:58.111Z] error TypeError: Cannot read property 'slice' of undefined

Changed regex2 to ^(?!.*\/\/)[a-z0-9\-\/]+(?:-[a-z0-9\-\/]+)*$|^$, which allows editors to keep slug empty on Sites page

Can you make one more small change please, and set the slug to be required for pages and protected-areas?

My only note is that the error message disappears quickly. I'd triple the time before it goes away, or just leave it if it has a working 'x' and it's not covering other things.
Let's be careful about re-using this, as it's pretty strict. Sometimes we'll need '.' or '#' in the slug.
In this case, it looks good to me.

@foley-lynn Yes, that's a shame that the error message fades quickly. Unfortunately, I couldn't find an easy solution for that since it's a part of the Strapi lifecycle hooks. This validation will need to be changed after updating v4 anyway, so I think it's OK not to take much time for this at this moment.
FYI 🙂

Strapi notification - It fades after 3 sec 😅
Throw error message in lifecycle - There is a difference between v3 and v4 lifecycle hooks, so we need to update this again...

Sounds good, thanks for looking.


Buildslaves on separate machines?

This role seems primarily focused on configuring the master. What would be my best path forward for setting up slaves on other boxes?

It really depends what you're trying to build. We've used the Swarm plugin to connect up workers successfully (because it doesn't require any manual steps) and then create a role for each workload we want to run, and configure the swarm agents environment to broadcast appropriate labels.

I ended up making a bespoke role specific to my workload, which boiled down to:

Install packages (debhelper, devscripts, git, cowbuilder, python-virtualenv, default-jre-headless)
create jenkins-slave user (with homedir, shell, etc).
inject SSH key from master
add known sources for our SCM providers (bitbucket, etc)
miscellaneous config files for the above.

And then I had to go in manually on the master and add each host from the web UI, including specifying the work dir.
So most of that is pretty customized to my scenario, but it does feel like there'd be room for a generic "Jenkins SSH Slave" role that could handle setting up the master's key, installing Java, and then wiring things up on the master end to talk to it.


User authentication failing for some users

Library

<EMAIL_ADDRESS>
Framework
Reactv16
Description
msal.js is used to authenticate the users against the active directory. For most of the users it works fine but for some users, the authentication fails.
Error Message
The authentication fails with the following message : A silent sign-in request was sent but none of the currently signed in user(s) match the requested login hint.
Security

[ ] Is this issue security related? NO

Regression

[ ] Did this behavior work before?  No

Reproduction steps
var authContext = new Msal.UserAgentApplication({
auth: {
clientId: clientId,
authority: authorityUrl,
postLogoutRedirectUri: window.location.origin,
redirectUri: window.location.origin
},
cache: {
cacheLocation: "sessionStorage",
storeAuthStateInCookie: true
}
});
Expected behavior
The user should be authenticated.
Browsers/Environment
This is not a browser specific issue.

@divya2603 Are the users that this is failing for, guest users and is it consistent for those users? There is a known issue where this may fail if a guest user's username is too long. Try sending sid to the extraQueryParameters you include in the acquireTokenSilent call instead of login_hint and see if this resolves the issue. sid is an optional claim you will need to enable in your app registration if you have not done so already. More information here

Closing due to inactivity. Let us know if you are still facing this issue after attempting the above suggestion. Thanks!


Support for other providers?

Other models/providers like Claude, Gemini are getting competitive.
Any plans to add API support for them as well?

Hi,
If you are still interested in multiple providers, take a look at my project, GPTMobile


Replicating complexity in builder.py in ibek yaml

There are some cases where the builder.py is doing things that we probably should replicate in  ibek support YAML.
e.g. in the asyn module we have a tidy way of adding configuration items to the asyn object as follows:
    def Initialise(self):
        print('%sConfigure("%s", "%s", %d, %d, %d)' % (
            self.DbdFileList[0], self.asyn_name, self.port_name, self.priority,
            self.noAutoConnect, self.noProcessEos))
        for key, value in self.options.items():
            print('asynSetOption("%s", 0, "%s", "%s")' % (
                self.asyn_name, key, value))
        if self.input_eos is not None:
            print('asynOctetSetInputEos("%s", 0, %s)' % (
                self.asyn_name, quote_c_string(self.input_eos)))
        if self.output_eos is not None:
            print('asynOctetSetOutputEos("%s", 0, %s)' % (
                self.asyn_name, quote_c_string(self.output_eos)))

In particular the options dictionary is really useful here. Also inheritance allows for this pattern to be reused in a few Asyn classes.
At present I have this in YAML which is adequate for the simple streamdevice I have made so far but is pretty limited and the approach does not scale.
https://github.com/epics-containers/ibek-support/blob/14384f341a8000d31c3013ec3b5bd618aa0d17f2/asyn/asyn.ibek.support.yaml#L99C1-L112C93
@coretl @GDYendell do you think we should support dictionary arguments for this purpose?

Don't you have to explicitly add the 8 lines to add disconnectOnReadTimeout as an arg in the support yaml anyway? Or do you want to be able to provide any arbitrary key-value to be called as asynSetOption in the ioc yaml, without having to add it to the support yaml?

I think the key value pair is the best way to achieve this without lots of lines of YAML. but then there is no build time checking.
I think I'm OK with needing an arg for each option but I don't want the pre_init to get really long and unreadable because of too many jinga ifs.

I remember the conversation but not the use of enums...
I was thinking more:
{% for attr in ["baud", "parity",... ] %}
asynSetOption( {{ name }}, 0, {{ attr }}, {{ locals()[attr] }} )
{% endfor %}

yep - this is why IRL conversations are a bit rubbish.

I'm pretty sure we have this covered in 3.0.0b6.

we have sub-entities
we have pre_defines and post-defines which can be of type list and referenced from other objects that have your id

Leaving open to look at cleaning up the asynSetOption feature in Asyn module


Factor out json serialization code from qps report.cc

Factor out json serialization code from report.cc and move it in with ParseJson(), so as to localize the code that deals with proto APIs and thus gets affected by namespace differences between internal and external versions of proto.

LGTM.

Marking ready_to_merge since the test directly affected (performance smoke) has passed.

Code formatting is wrong
https://grpc-testing.appspot.com/job/gRPC_pull_requests/9109/config=opt,language=sanity,platform=linux/testReport/junit/(root)/tests/tools_distrib_clang_format_code_sh/

@@ -103,7 +103,7 @@

 void JsonReporter::ReportQPS(const ScenarioResult& result) {
   grpc::string json_string =
-    SerializeJson(result, "type.googleapis.com/grpc.testing.ScenarioResult");
+      SerializeJson(result, "type.googleapis.com/grpc.testing.ScenarioResult");
   std::ofstream output_file(report_file_);
   output_file << json_string;
   output_file.close();
--- /local-code/test/cpp/qps/parse_json.cc	2016-05-19 21:48:11.572275097 +0000
+++ /tmp/tmp.XsYCHaFySd	2016-05-19 21:51:40.761845935 +0000
@@ -72,8 +72,8 @@
   grpc::string binary;
   grpc::string json_string;
   msg.SerializeToString(&binary);
-  auto status = BinaryToJsonString(type_resolver.get(), type, binary,
-                                   &json_string);
+  auto status =
+      BinaryToJsonString(type_resolver.get(), type, binary, &json_string);
   GPR_ASSERT(status.ok());
   return json_string;
 }
Command exited with non-zero status 1

I was a little surprised it was right, given that I hadn't actually run clang-format yet -- apparently it wasn't.
Fixed.

@dklempner , you need to run the dockerized clang-format to make sure that you get the right version. Your clang-format only took care of one of the needed fixes, not both.

It's not a version issue. The issue is that I didn't run clang-format against both affected files.
(I also don't have docker working at the moment.)

Needs resolving.

I have resolved.

I should be able to handle this from this point.


[googleapis] update to v0.6.0 release

Update the googleapis package to v0.6.0.

What does your PR fix? Fixes issue #

N/A

Which triplets are supported/not supported? Have you updated the CI baseline?

No change.

Does your PR follow the maintainer guide?

Yes, I think so.

@coryan
Thanks for this PR.
It looks good to now.
The regression on Linux seems to be not related to this PR.
I will re run it later.

/azp run

Thanks for the PR!


allow public access to some views

Allow public access to source_set#index, source_sets#show, source#show, and guide#show.  Only show admin menus and information on these views if the admin is logged in.
This addresses task #8085.

rebased to develop

Looks good when I run it.


Scala Kinesis Enrich: add ability to retrieve resolver and enrichments from DynamoDB

This is business-level data - it would be great to have it sitting in DynamoDB as it makes it super-easy to update without re-deploying servers...

This would actually be really useful for the Hadoop flow too, because there are issues in the Hadoop flow with trying to pass too much configuration data as Hadoop command-line arguments.
So we should probably be thinking of this as a Scala Common Enrich thing with hooks into Scala Kinesis Enrich and Scala Hadoop Enrich.

Bringing forwads. The Scala DynamoDB library is ready to go now: https://github.com/piotrga/async-dynamo

@fblundun - I have updated the desc as per our chat - over to you!

According to this documentation, there is no way to query every items whose hash key begins with a particular string, so instead I am using a scan to get all items from the table and filter them locally. This involves examining every item in the table so it's best if the table doesn't contain too many extraneous items. What do you think of this approach @alexanderdean ?

Hey Fred - when you say all items, do you mean all items or all keys?

I mean all items.

Okay - I can't see a way around this without restricting all configurations to total less than 400kb. Let's go ahead with your approach. The table will be called something like snowplow-configuration and thus won't contain a lot of extraneous items.


Add State Validation

[x] Create state validation class and inject repository into class
[x] Write logic to validate against database
[x] Add validation and error message to DTO

added src/pipes/state-validation.pipes.ts
added validation to DTO parameter
updated hour-unit-data.repository to allow state inputs to be lowercase (previously if a state was lowercase, it would return an empty list)

Currently testing before submitting a pull request.

Submitted pull request easey-emissions-api/13


Default to --capture=no when debugging pytest

From the discussion on StackOverflow, this is due to https://github.com/microsoft/debugpy/issues/899#issuecomment-1095384631. So it seems that VSCode should default to --capture=no when debugging pytest.
Originally posted by @int19h in https://github.com/microsoft/debugpy/issues/947#issuecomment-1159013741

@karthiknadig Can I work on this issue?

@harupy go ahead.

I don't know why we need default to --capture=no, because if we set it in settings.json:
{
    "python.testing.pytestArgs": [
        "tests",
        "--no-cov",
        "--capture=no"
    ],
    "python.testing.unittestEnabled": false,
    "python.testing.pytestEnabled": true,
    "testing.defaultGutterClickAction": "debug"
}

even you can see it as a command option in sys.argv --capture=no is not used.
We could say the same for --no-cov because coverage steals tracing api (sys.settrace), so if we are debugging turn of the coverage.

How about --no-cov?

Thanks


RDF data input

Hi,
what are the supported graph data formats? Can RDF graphs be used as input?

Or all of our library modules operate on our data-structure GraphsTuple. In utils_np and utils_tf we provide util functions to convert from other formats, of which the most well known is networkx. Specifically we support networkx.OrderedMultiDiGraph, where the keys to the nodes are integer numbers from 0 to num_nodes - 1.
Based on a quick search it seems that there are simple ways to convert from RDF graphs to networkx graphs.
You would need to be careful to re-index the nodes to satisfy the key constraint specified above if required, and also store the node, edge and global features in a key named "features" (as in our examples.
But beyond small data conversion it should be quite straight forward.
Hope this helps!


refactor : Removed all V1 parts.

Proposed changes
fixes #12  , removed all V1 parts in the code, comments and tests.
Types of changes

[x] Refactoring (improve a current implementation without adding a new feature or fixing a bug)

Checklist

[x] I have read the CONTRIBUTING documentation
[x] Lint and unit tests pass locally with my changes

Codecov Report

Merging #18 into master will decrease coverage by 3.24%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
- Coverage   94.09%   90.85%   -3.25%
==========================================
  Files          40       24      -16
  Lines         983      667     -316
==========================================
- Hits          925      606     -319
- Misses         58       61       +3

Impacted Files
Coverage Δ

src/connection/connection.rs
100% <ø> (ø)
:arrow_up:

src/api/api.rs
33.33% <ø> (-66.67%)
:arrow_down:

tests/lib.rs
100% <ø> (ø)
:arrow_up:

tests/connection_test.rs
100% <ø> (ø)
:arrow_up:

tests/connection_manager_test.rs
100% <100%> (ø)
:arrow_up:

src/api/two/two.rs
78.57% <0%> (-7.15%)
:arrow_down:

src/utils.rs

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 b66c574...447f53a. Read the comment docs.

This is only half done. All files under the Two namespace need to be moved into the root Api namespace and tests adjusted.