Fix status 404 for LIST records

Changes
Removed check from ListRecord method which searches for result name when the record lister returns empty array. This was incorrect because a "-" can be used as result name to list records across multiple results.
Fixes #560
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:

[ ] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[x] Tested your changes locally (if this is a code change)
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user-facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contain the string "action required" if the change requires additional action from users switching to the new release

Release Notes
ACTION REQUIRED: List records will return 200 - OK status and empty array in response when the filter criteria doesn't match any records

Is there a test case for -?

@khrm There are tests for "-" result name, but I have added a test specifically for this case where result name is "-" and the filter doesn't match anything.

This PR should have a release note - we are changing the behavior of an API response. Some might consider this a "breaking change" that warrants an ACTION REQUIRED prefix.

@adambkaplan Added a release note.
This behaviour was particular to ListRecords. Now it will align with ListResults and ListLogs


DHCP log entries not showed in reverse oder

Hi,
regardless of the selection "Show log entries in reverse order (newest entries on the top) This section of the log file shows still the oldest entries on the top.
Jakob

style updated as well, thanks!


Eating/ Holding Interact Issue

When I hold left trigger to eat or pull back bow it acts as if I am taping interact over and over. This a known issue?

I found the same with mod version 0.19.0 on xbox 360/one wired controllers, but it works fine in 0.19.3 so try a different version if possible.

This was a bug in 0.19.0 and has since been fixed in newer versions


Add test for local quorum queries while a node is bootstrapping

Add test for local quorum queries while a node is bootstrapping

@aboudreault What's the status on this?

I tried to finish that last week... but now getting issues related the FlushWriter in 2.0... should be good to work on this this week.

No rush, just wanted to make sure we still wanted to keep this PR open. We have a lot built up

ready for review

LGTM, +1


export langium-cli intenals?

We currently have dev script (similar to watch and serve), that watches .ts and .js files and rebuild on change. We want to also make that script that also watches .langium files and regenerate on change without the need of running to watch mode, but when trying to import langium-cli it throws error since it doesn't export anything except bin, that's why I'm requesting to export internals so we could use them in that script.
I'm aware that there's a workaround to do that by running the generate command on the shell, but I feel doing so in typescript is better :)

Will be addressed in https://github.com/eclipse-langium/langium/pull/1125 with 24a9eba.

Sorry to disturb you, but how should we actually use the generate function? I tried using all parameters, but it always exited with one exit code with --help prompt not showing what's the error:
// inside .esbuild/build.ts
await generate({ file: '../packages/parser/langium-config.json' })

Usage: build [options] [command]

Options:
  -V, --version                   output the version number
  -h, --help                      display help for command

Commands:
  generate [options]              Generate code for a Langium grammar
  extract-types [options] <file>
  help [command]                  display help for command
 ELIFECYCLE  Command failed with exit code 1.

I tried generate by execute it in shell but it appears it can't read files from previous dir:
$ pnpm langium generate --file=../packages/parser/langium-config.json
Reading config from ../packages/parser/langium-config.json
Failed to read config file. [Error: ENOENT: no such file or directory, open '../packages/parser/langium-config.json'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: '../packages/parser/langium-config.json'
}

A side from that, I'm not sure why, when I try to import langium-cli in the root dir to use it in .esbuild/build.ts it throws errors that it can't be found:

But it can be easily imported in packages/*:

Not really sure what is the problem.

Although I was able to generate by executing command in shell but still can't generate using generate command.

Yep, I made a mistake in https://github.com/eclipse-langium/langium/commit/24a9ebaf442b22a0cfaba2a0fccad2d94c65a2b7. Reopening.


Support CLP(B)

Hi,
Is supporting CLP(Z) just a matter of vendoring in Markus Triska's clpz.pl file?
What would be required at the lower level C code in order to support it.
Any orientative explanation of what would be required would be much appreciated so I can then dig deeper into understanding the requirements.
all best,
jgart

Hi, I'm not sure what you mean, as library/clpz.pl is already included in
the Trealla release.
On Sat, 6 July 2024, 09:00 Jorge Gomez, @.***> wrote:

Hi,
Is supporting CLP(Z) just a matter of vendoring in Triska's clpz.pl
https://github.com/triska/clpz/blob/master/clpz.pl file?
What would be required at the lower level C code in order to support it.
I'm trying to understand the requirements.
Any orientative explanation of what would be required would be much
appreciated.
all best,
jgart
—
Reply to this email directly, view it on GitHub
https://github.com/trealla-prolog/trealla/issues/564, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNKSER3W7EKANHRB76XQJTZK4QRTAVCNFSM6AAAAABKN3COEOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM4TGMRRG44DCMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Oops, for some reason I thought it was not included yet. I forgot to check that.
Is there any further work to do on clpz.pl in trealla or is it at feature parity with scryer?

As far as I can tell it works as expected. CLP(B) however is still missing
as it doesn't work properly yet.
On Sat, 6 July 2024, 10:20 Jorge Gomez, @.***> wrote:

Oops, for some reason I thought it was not included yet. I forgot to check
that.
Is there any further work to do on clpz.pl in trealla or is it at feature
parity with scryer?
—
Reply to this email directly, view it on GitHub
https://github.com/trealla-prolog/trealla/issues/564#issuecomment-2211532365,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNKSEWCBLUB5RVEJIV7RULZK4Z35AVCNFSM6AAAAABKN3COEOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJRGUZTEMZWGU
.
You are receiving this because you commented.Message ID:
@.***>

What's left to do on CLP(B) or what is the currently difficulty with it?

Unknown problem that I can't seem to get to the bottom of. I'll return to
it soon. CLP(B) & Tabling are the outstanding problems.
On Sat, 6 July 2024, 10:25 Jorge Gomez, @.***> wrote:

What's left to do on CLP(B) or what is the currently difficulty with it?
—
Reply to this email directly, view it on GitHub
https://github.com/trealla-prolog/trealla/issues/564#issuecomment-2211535579,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNKSETEZ3OICSADNO3BJULZK42P7AVCNFSM6AAAAABKN3COEOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJRGUZTKNJXHE
.
You are receiving this because you commented.Message ID:
@.***>

I've changed this issue name to "Support CLP(B)" since I saw that there is no ticket open for it currently.

What is Tabling? Can you point me to a reference on tabling so that I can orient myself to it. Thanks!

Yes, the Scryer code for tabling should be portable but doesn't work.
On Sat, 6 July 2024, 10:31 Jorge Gomez, @.***> wrote:

You mean this?
https://en.wikipedia.org/wiki/Tabled_logic_programming
—
Reply to this email directly, view it on GitHub
https://github.com/trealla-prolog/trealla/issues/564#issuecomment-2211539249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNKSESAN372TYP2AUVPVZDZK43G5AVCNFSM6AAAAABKN3COEOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJRGUZTSMRUHE
.
You are receiving this because you commented.Message ID:
@.***>

Oh, ok
https://www.scryer.pl/tabling


[N7 Account Settings] Convert LOGIN_LIST from Array to Object

Tracking issue: https://github.com/Expensify/Expensify/issues/182122
Problem
Currently we store logins in Onyx as an Array of logins.
As outlined in the N7 Account Settings doc, in the near future we'll need to be able to associate errors / loading states with individual logins (in the future we'll call them "contact methods"). If we keep this data as an array, we aren't able to update individual logins so we'll have to re-set the entire login list any time we add error messages, remove them, set loading states, etc.
Solution
Update the logins in LOGIN_LIST to be an object / dictionary of logins. The keys will be the login (partnerUserID) and the values will be the full login details as they exist today. This will make it very easy to use Onyx to add / modify / remove keys to individual logins.
~We'll need to write a migration for this so that any existing user with an array loginList will be converted to an object.~

Note: We agreed no migration is needed for this

Additionally, we'll have to look into our backend to update the format of any API commands returning loginList data. Examples:

https://github.com/Expensify/Web-Expensify/blob/d5ff4d26afeeb8a91eb985ccf2bd7485eeb689b9/lib/AppInit.php#L202-L207
https://github.com/Expensify/Web-Expensify/blob/d5ff4d26afeeb8a91eb985ccf2bd7485eeb689b9/lib/UserAPI.php#L1517-L1521

Possibly don't need migration, can just tell everyone to log out / log in 🤷

In doc comments we agreed this isn't needed, so closing

Reopening b/c this is needed, just no migration is necessary

Taking off hold b/c we want to start working on N7 Account Settings again, at least the profile pages

PRs in review:

https://github.com/Expensify/App/pull/10959
https://github.com/Expensify/Web-Expensify/pull/35282
One more to clean up App (since we shouldn't have to handle arrays coming back from server after ^

2 PRs to go!

The Web-E PR just got merged, so now only 1 left - cleaning up App!

Last PR is up! https://github.com/Expensify/App/pull/12919

Adding Bug to get a bug-zero assigned to help pay out @thesahindia who tested the PR!

Yeah agree this is good to go 👍

Accepted, thanks @alexpensify

Payment has been sent via Upwork


Setting uninstalled extension...

I noticed that I don't have vscode-github-issue-notebooks installed anymore and the log shows me that it got uninstalled this morning by settings sync. I am quite certain that I didn't do that and now it is time understand what happened

I am sorry that I got to it late because of too many things and did not notice it.
Above logs say that extension is not there in the remote and so it has removed it locally. Reason why it is not there on remote is that another instance would have uninstalled the extension locally and pushed the state to remote.
You can find which instance has pushed it by checking the activity view:

F1 > Settings Sync: Show Synced Data
Focus on Sync Activity (Remote) view
Type Extensions to filter the other resources

Please check the first entry that does not has your extension to find the instance that removed the extension.

If you think you have not removed the extension manually, please see if we can get the settings sync logs and shared process logs from that machine?

You mean something like this? Does that mean the web client uninstalled the extension?

Safari seems to be #8 and I see the same change but nothing in the log (btw I do see a bunch of other extensions that have been uninstalled without me recalling doing that)

Yes, it seems one of the web clients has removed the extension from remote. I do not know why - may be extension does not exist and wrong state got pushed - just speculating.
Since, settings sync runs on every browser tab, it could be one of those tabs. You will find all settings sync logs from all tabs here:

Is it possible to get the logs from the day when it happened and may be previous ones. It would be helpful to find the log that pushed the state without the extension to remote. You will see in the logs that remote is updated and removed the extension. For eg: [2021-09-17 10:39:44.666] [settingssync] [info] Extensions: Updated remote extensions. Removed: ["ms-vscode.vscode-github-issue-notebooks"]. Hopefully I can understand what would have happened by seeing that log.

this is what I am seeing in the log from the day
[2021-09-08 11:13:58.544] [userDataSync.log] [info] Snippets: No changes found during synchronizing snippets.
[2021-09-08 11:13:58.661] [userDataSync.log] [info] GlobalState: No changes found during synchronizing ui state.
[2021-09-08 11:13:59.524] [userDataSync.log] [info] Extensions: Uninstalled local extension. ms-vscode.vscode-github-issue-notebooks
[2021-09-08 11:13:59.556] [userDataSync.log] [info] Extensions: Updated last synchronized extensions. Skipped: ["dbaeumer.vscode-eslint","ms-vscode.vscode-selfhost-test-provider","chrisdias.vscode-opennewinstance","kisstkondoros.vscode-codemetrics","jrieken.references-plusplus","ipatalas.vscode-postfix-ts","jrieken.regex-notebook","jakearl.handydandy-notebook","jrieken.browser-links","eamodio.gitlens","ms-vscode-remote.remote-containers","ms-vscode-remote.remote-ssh-nightly","ms-vscode-remote.remote-ssh-edit-nightly","wayou.vscode-todo-highlight","kisstkondoros.vscode-gutter-preview"].
[2021-09-08 11:13:59.559] [userDataSync.log] [info] Sync done. Took 1034ms
[2021-09-08 11:18:59.561] [userDataSync.log] [info] Auto Sync: Triggered by Interval

This log also says that the remote does not have this extension. May I know if you have found any log with following entry?
[userDataSync.log] [info] Extensions: Updated remote extensions. Removed: ["ms-vscode.vscode-github-issue-notebooks"].

YEah, also found that in a later log
[2021-09-08 11:11:25.146] [userDataSync.log] [info] GlobalState: No changes found during synchronizing ui state.
[2021-09-08 11:11:44.548] [userDataSync.log] [info] Extensions: Updated remote extensions. Removed: ["ms-vscode.vscode-github-issue-notebooks"].
[2021-09-08 11:11:44.573] [userDataSync.log] [info] Extensions: Updated last synchronized extensions. Skipped: ["2gua.rainbow-brackets","jrieken.references-plusplus","jrieken.browser-links","chrisdias.vscode-opennewinstance","dbaeumer.vscode-eslint","eamodio.gitlens","jrieken.regex-notebook","ms-dotnettools.vscode-dotnet-sdk","ms-dotnettools.vscode-dotnet-runtime","wayou.vscode-todo-highlight","ipatalas.vscode-postfix-ts","ms-vscode.vscode-markdown-notebook","ms-vscode.vscode-selfhost-test-provider","kisstkondoros.vscode-codemetrics","kisstkondoros.vscode-gutter-preview","jakearl.handydandy-notebook","ms-vscode-remote.remote-ssh-nightly","ms-vscode-remote.remote-ssh-edit-nightly","ms-vscode-remote.remote-containers"].
[2021-09-08 11:11:44.576] [userDataSync.log] [info] Sync done. Took 19560ms

Thats it - this is exactly the one and it matches the timing too. Two minutes after this in the another tab, settings sync removed the extension.
Wondering why this tab has removed the extension. This happens when remote sync state and local last sync state are same and locally extension was removed. I believe that you would not have removed it. So I am wondering why the locally installed extensions do not has this extension.
In web, settings sync runs on every tab and therefore there can be chance that they run in parallel. So, I am suspicious that there can be a race condition causing this. I have to say I can't prove it though.

Moving this to under discussion as it is not clear how this would have happened other than some suspicions.

After investigating together, we found that it is because installed extensions on the web is excluding extensions if they cannot be resolved and Settings Sync thinks that as removal and removing the extension which is confirmed from above logs

Made a tactical fix:

Settings Sync skips syncing extensions if there are errors while parsing installed extensions. Extensions get synchronized as soon as the errors are resolved.

I will do a better long term fix in next milestone that involves more changes - https://github.com/microsoft/vscode/issues/134224
To verify:

Set up settings sync between desktop and web
In web, open installed extensions resource using command Developer: Open Installed Web Extensions Resource
Change one of the extension's location to point to invalid URL. Eg: Change the path property in location of extension to a non existing extension
Refresh the page and make sure that extension is not removed in desktop. Check settings sync log that an error message is displayed while syncing.
Fix the extension location and make sure sync works again.

/verified
Invalid paths don't cause a sync.


Fix example notebook to use pymc 4

Updates this notebook to use pymc4 correctly. Also removes the negation in the density function as blackjax does not expect a potential function but the actual density function.

LGTM, thanks for the fix!


Cut new version

Can we cut a new version of this package? I use webpack externals heavily and this commit makes the import/no-extraneous-dependencies no longer warn on missing externals. Thanks!

I too am looking forward to the next release, specifically for the ignorePackages enhancement to the extensions rule – many thanks to @collinsauve and @millerized for your work on this in PR #827 :tada:
@benmosher @ljharb: Do you have an estimated date for the next release? Many thanks in advance :slightly_smiling_face:

No estimated date; it will be released as soon as I have time to do so.

Well...we can add import/no-default-export to this mix.

@benmosher so hyped for this release!

no worries @ljharb, you've been doing so much already! much appreciated

shipped it! v2.9.0 is published. had to follow my own RELEASE document so hopefully I didn't botch it 😄

🎉

Thanks so much. You all rock.

new stuff to add to airbnb-base! https://github.com/airbnb/javascript/pull/1743

No need to wait any longer...Thanks!


Vector2#angle_to returns zero when the left side is zero

Godot version
4.0 beta 17
System information
Windows 10
Issue description
Vector2#angle_to always returns zero when the vector in which the method is being called from is zero.
The angle from vector a to vector b can be found with atan2(b.y - a.y, b.x - a.x), and naturally, if vector a is the origin (0, 0) the angle is just the angle of vector b, this is not the case with Godot's angle_to implementation:
real_t Vector2::angle_to(const Vector2 &p_vector2) const {
	return Math::atan2(cross(p_vector2), dot(p_vector2));
}

The following piece of code should print 0.78539816339745 (45 in degrees), but the output is zero.
print(Vector2.ZERO.angle_to(Vector2(1, 1)))

Steps to reproduce
Just run print(Vector2.ZERO.angle_to(Vector2(1, 1)))
Minimal reproduction project

Is this just the difference between angle_to and angle_to_point()?
https://docs.godotengine.org/en/3.6/classes/class_vector2.html#class-vector2-method-angle-to
angle_to_point() is indeed as you describe:
real_t Vector2::angle_to_point(const Vector2 &p_vector2) const {
	return Math::atan2(y - p_vector2.y, x - p_vector2.x);
}

Yes that's a misunderstanding, Vector2.ZERO is the origin, it doesn't have an angle to any other vector. An angle requires two directions and the origin is not a direction, so the result is as expected.
What you seem to want is Vector2(1, 0).angle_to(Vector2(1, 1)) (which is indeed equivalent to angle_to_point()). It's the vector (1, 0) which marks the angle 0 on the trigonometric circle, not the origin point.


[nz-dropdown-list] behave weirdly inside virtual scroll list

Reproduction link
https://stackblitz.com/edit/angular-n4ef8f
Steps to reproduce

open link
click on hover me to show the dropdown list
scroll down or up
observe the results

What is expected?

the dropdown list should not overflow the virtual scroll.
the dropdown list should not be repeated under different clicked elements/buttons.

What is actually happening?

the dropdown list overflows the virtual scroll.
the dropdown list shows under different elements/buttons than the clicked one.

Environment
Info

ng-zorro-antd
12.0.2

Browser
all

tried to fix this using nzvisable but it started to behave even more weirdly

anyone found a solution for this? thanks!


Removes Reddit Integration on the Mac

Removes Reddit FeedProvider
Removes Reddit related UI, Assets, and Extension Preferences pane
Adds deprecation alert

😞 that Reddit has put us in this position, but makes sense.


Implement Yaydoc to generate documentation

Yaydoc is a documentation generation project maintained by FOSSASIA. Implement yaydoc in open-event-orga-app to generate documentation and also register open-event-orga-app to Yaydoc CI for continuous documentation deployment.

I would like to work on this.

@sch00lb0y all open event projects need to come together.
Parent Project: https://github.com/fossasia/open-event
Sub projects:

https://github.com/fossasia/open-event-orga-server
https://github.com/fossasia/open-event-frontend
https://github.com/fossasia/open-event-android
https://github.com/fossasia/open-event-webapp
https://github.com/fossasia/open-event-orga-app

The final documentation is to goto the gh-pages branch of  fossasia/open-event.

@sch00lb0y what is the progress on this?

@mejariamol I have fixed this issue. I have sent PR to the respective repositories. You can close this issue.

I have sent PR to the respective repositories

Is that mean, doc page is viewable now? So that README.md can be modified accordingly.


Files are corrupted on deduplicated volumes that were created as NTFS-compressed

https://support.microsoft.com/en-us/help/3066174/files-are-corrupted-on-deduplicated-volumes-that-were-created-as-ntfs

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

ID: 0cd25a70-630b-f699-521c-656e552edcec
Version Independent ID: 12dcd918-26ec-c651-5605-ce98cf57077f
Content: Data Deduplication interoperability
Content Source: WindowsServerDocs/storage/data-deduplication/interop.md
Product: windows-server
Technology: storage-deduplication
GitHub Login: @wmgries
Microsoft Alias: wgries

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


app.group is not thread-safe

The way app.group is implemented adds state to the application. If the following code is run on many threads many times, eventually the routes will be malformed.
app.group("a") {
    app.get("b") { _ in ... }
    app.get("c") { _ in ... }
    app.group("d") {
        app.get("e") { _ in ... }
        app.get("f")  { _ in ... }
    }
}

I think the easiest solution here is to add locks.

👍


Added cypress fixture for submiting article question

closes #284
It is pretty easy when you have a single post request...
I will try the same thing on login, we will see how that goesLiveURL: https://smartfaq-cypress-comment-fixture.surge.sh

Yep, makes sense, will add it 👍
Regarding mocking login functionality, what do you think about it, should we go in that direction or not? Please check #280

I believe your approach works best in the case of login, but I'm dubious we need to mock login?
Shouldn't we integrate-test everything as close to the user as possible? Just not having side-effects such as flooding the backend with new feedback messages. Imho we can afford to actually login, otherwise we will need to mock bookings, etc as well.
What do you think?

I think the same, but the only thing that concerns me is this cy.wait(1000); timeout.
But yea.. maybe im over engineering it at this point of time, lets see how it goes and then adapt 😁

One option to "clean-up" code of cy.waits would probalby be overwrite some timeouts in the cypress.json ? https://docs.cypress.io/guides/references/configuration.html#Timeouts

Hi! @git-toni, @dinodsaurus
I'm not sure about what it's the better approach, but probably, not has sense mock all the responses in the e2e test, anyway, I made a new command for mock all the requests, either HTTP as login or a GraphQL. You can see it in the last PR #280.
I did a mock of all the requests in that test, I guess isn't necessary, I could change it. We can discuss in that other PR.
Thanks!


Setup Sysdig Dashboard

ca7f8f - Digital Trust Monitoring Services

@rajpalc7, Why is this ticket on this project?  I believe this ticket was tracking the work; https://github.com/bcgov/DITP-DevOps/issues/47


Improve code formatting and language in README.md

I made some QOL improvements to the README.md, by adding code formatting and proper formatting for key functions and attributes.

Create a pull request to the dev branch, not the main branch.
On Wed, Dec 13, 2023 at 11:54 AM David Uzondu @.***>
wrote:

I make some QOL improvements to the README.md, by adding code formatting
and proper formatting for key functions and attributes.
You can view, comment on, or merge this pull request online at:
https://github.com/Scriptkidd98/aria-ease/pull/26
Commit Summary

5f2b3ae
https://github.com/Scriptkidd98/aria-ease/pull/26/commits/5f2b3ae479025ae93dff75cf839bea4b8141a729
docs: Improve code formatting and language in README.md
95db18f
https://github.com/Scriptkidd98/aria-ease/pull/26/commits/95db18f1b6cc6b5550f749bc89bcf24b1681ffd1
docs: Improve code formatting and language in README.md

File Changes
(1 file https://github.com/Scriptkidd98/aria-ease/pull/26/files)

M README.md
https://github.com/Scriptkidd98/aria-ease/pull/26/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5
(14)

Patch Links:

https://github.com/Scriptkidd98/aria-ease/pull/26.patch
https://github.com/Scriptkidd98/aria-ease/pull/26.diff

—
Reply to this email directly, view it on GitHub
https://github.com/Scriptkidd98/aria-ease/pull/26, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKI66BEWBSFWDSYC45PS7RTYJGCNFAVCNFSM6AAAAABAS6KVOSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGAZTSNBVGA3DQNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Update to use tailwind for building css

Tasks

[ ] remove all custom styling to make use of https://tailwindcss.com/ and generate styles.

Hello! I can give it a try. Could you assign it to me?

Yep sure @redpop - Give me a shout if you have any questions.


[Feature Request] Keep a copy of sent e-mails and let admins browse through them

https://github.com/boxbilling/boxbilling/issues/819

There's already the ability to log emails sent


Add support for HTML5 autofocus

Adding support for the HTML5 attribute. @ajimix Let me know if you have any questions.

Hey @gastonfig thanks for your contribution :)


Prevent time outs because no output was received

Lately we see a lot of timed-out builds (for instance):
No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.

Adding travis_wait (see) to the before_script should fix the problem.

@ceeram What can I do to speed-up the merge process?


Webjob Webhooktrigger issue

I'm attempting to POST to a webjob function with webhooktrigger but I'm getting the error
An attempt was made to access a socket in a way forbidden by its access permissions
Do you have any idea why and how I could solve that? Thank you!

What kind of language/code are you using for your WebJob? The sandbox does not allow every kind of socket listening.

I'm using C# .NET and the webjob is already running on Azure.

I just discussed with others. Webhooktrigger is an experimental thing in the WebJobs SDK (not part of Kudu), and it will likely probably be removed as we are taking a different direction. So I would suggest not using it at this time.


[XPU][TritonGEN] Replace subgroup reduce and scan

Replace usage of the TritonGEN dialect subgroup reduce and scan operations with equivalent operations from the SPIR-V dialect.
This closes #2892.

Please fix test_reduce[...xor_sum...] failures.

Yeah, I already have a hunch what's causing it, will fix 👍


fix wrong quotes around inline codes

related PR:

#6522

The root cause is fixing on remark-smartypants: here
But It's hard to tell when it gets merged so in the meanwhile this typo fix can be helpful.

I think no need to merge this PR, let's wait to fix it in the upstream package https://github.com/silvenon/remark-smartypants/pull/80

@dimaMachina Yes, It's merging in no time. I will just change this PR to update dependency after new version releases 😋

@dimaMachina I just found it they don't rely on silvenon/remark-smartypants.
the remark-smartypants here is implemented itself. Sorry for confusing 🥲. I can refer to your work to fix this.

@2wheeh thank you of course you can refer 👍

@2wheeh thanks for fixing this! I'm still seeing it broken in the preview deployment, could you take a look?

@2wheeh thanks for fixing this! I'm still seeing it broken in the preview deployment, could you take a look?

@rickhanlonii Have you checked they are cached ?

Ah, can you increment the cache key here: https://github.com/reactjs/react.dev/blob/main/src/utils/prepareMDX.js#L10

Ah, can you increment the cache key here: https://github.com/reactjs/react.dev/blob/main/src/utils/prepareMDX.js#L10

@rickhanlonii done it: 1c6d531
and it looks fine in preview now.
except in search bar :

Awesome, thanks! The search should update the index after deploy.


The links in the profile menu are not working as intended

Describe the bug
The links in the profile menu are not working as intended
Expected behavior
When clicking the link, it is expected to redirect to the respective social media page, but currently, it is not functioning as intended.
Screenshots

Additional context
Add any other context about the problem here.

I would like to work on this issue.

#21 solves this isssue

Pull request is made


Respect .ignore files

Without this, depending on your project structure, you're going to mostly measure dependencies and generated code. Which can be useful sometimes but should not be the default. Se tokei's --no-ignore-* flags for inspiration. I know mezura has --exclude, but it isn't as convenient and featureful. There are crates to parse various ignore files.

The "problem" with adding such functionality is that mezura doesn't support pattern matching for paths, extensions, etc. that can be found in .ignore files, like tokei does. This would be a nice addition at some point.
Right now, mentioning specific folders with --dirs and adding --exclude if necessary is your best bet. It has to be done once, and after that, it can be saved and loaded as a configuration.


dataTable pdf page size

how to set pdf fit the page

The package uses Laravel Excel to generate the csv, xls & pdf. Better check their documentation how to configure it.
On the other hand, I suggest you use [Laravel Snappy] as the pdf generator. The pdf output would be the same as the print view.
See this issue for ref: https://github.com/yajra/laravel-datatables/issues/718


Create ppc64le release

Minor patch that will release ppc64le artifacts upon new tags.

@mgiessing can you please verify the binaries from this release test run? https://github.com/dmcgowan/containerd/releases/tag/v1.7.0-alpha.ppctest

Signed-off-by: mgiessing<EMAIL_ADDRESS>
Also, please sign the commit using your real name rather than username (https://github.com/containerd/project/blob/main/CONTRIBUTING.md#sign-your-work). Ideally configure it in git so we don't have to add a record in mailmap.

@dmcgowan Thanks! I followed https://github.com/containerd/containerd/blob/main/docs/getting-started.md#option-1-from-the-official-binaries to install & verify the binaries from the release test run.
Furthermore I ran the integration test (make integration) which also succeeded
[...]
time="2022-04-26T17:19:54.315837088-04:00" level=info msg="Event monitor stopped"
time="2022-04-26T17:19:54.315885987-04:00" level=info msg="Stream server stopped"

ok  	github.com/containerd/containerd/integration/client	334.186s

Signature is also updated to my real name, but I guess that is not applied retroactively to previous commits.

Signature should also be updated by now - if you need anything else let me know.
Thanks!

Any idea when this will produce the package in the release and can this be cherrypicked into release/1.6 branch as well?

Yeah, I would appreciate that as well!


CLI for M365 version for script "Copy a library view across multiple libraries in destination site"

🎯Aim
The aim of this issue would be to add a CLI for M365 version of script Copy a library view across multiple libraries in destination site.
🤔How
I think one of the best approach would be to use the following commands:

list-view-get
list-view-list
list-view-add
list-view-set

@reshmee011 thanks for noticing 👍. Closed then 😅


Call them 2i2c managed JupyterHubs rather than hubs

https://github.com/2i2c-org/pilot-hubs/issues/314 talks
about the need to be very explicit in what we call the
service we provide. The word 'hub' is very overloaded.
2i2c managed infrastructure has an important property that our naming
should reflect.  It isn't a new thing we are developing, but a
deployment of an upstream, purely open source product. This means
our users can always:

Take their infrastructure from 2i2c, and run it themselves
with help from the community.
Work with us to train their own staff to run their JupyterHub,
so they can participate fully in running the infrastructure. This
gives them organizational control over what is built, and they
can walk away easier if needed.
Any improvements we make in terms of features built or
operations smoothed will be made in upstream repositories, and

I think directly using the word JupyterHub is a good way to
communicate this.
Alternative suggestions:

just 'JupyterHub'
2i2c JupyterHub
JupyterHub managed by 2i2c

I prefer (2) if needed.
Question is, how does this related to the Jupyter Trademark
policy (https://jupyter.org/governance/trademarks.html)?
Particularly:

Any commercial use of the Jupyter trademarks in product or company
names must be approved first by Project Jupyter. Some uses, like calling
a company “The Jupyter Company,” or a product “Jupyter Hosting” or
“Jupyter Cloud” will be refused. This is because they are overly broad,
or confusing as to whether Jupyter is open source or commercial, or
whether your product or organization is affiliated with or sponsored by
Project Jupyter.

(from
https://jupyter.org/governance/trademarks.html#uses-that-always-require-approval)
So no matter what, I think this needs to go to the Jupyter steering
council. I personally believe that '2i2c managed JupyterHub' is fine,
but that's not for me to decide.

I think this is a great addition - totally agree that our naming should reflect the value that we think the hubs bring. We don't want it to seem like 2i2c has created some new thing 👍
Gonna do a bit of thinking out loud...
I think there are a few things we need to balance:

Reflecting the values of 2i2c and the service, as noted above
Having names that are memorable and easy to remember (perhaps with long-form and short-form versions)
Having names for the various ways that people refer to the service. E.g., "the service in general", vs "each individual hub"
Having names that distinguish between the flavor of hubs we offer (e.g., educational, daskhub, etc)

So given the above, I think that this is where this PR lands:

For the service in general: the 2i2c Managed JupyterHub Service
For an individual hub: your managed JupyterHub

A few thoughts:

These are more than just managed JupyterHubs, they're more like managed JupyterHub distributions. Should that go into our naming at all? Can we pull inspiration from similar markets in other tech sectors, like linux distributions?
To that extent, we should similarly come up with a name that is not "2i2c-specific" for the flavors of JupyterHub we run, no? Or do we just generically call them "research hubs" or "educational hubs"?
I'm a bit worried about #2 above - "2i2c Managed JupyterHub Service" is punchier but much more verbose. Is there a way we can come up with a more succinct/memorable short-hand version of the same thing, without causing problems for #1 above?

I think we are definitely headed in the right direction here though, we don't want people to think that a 2i2c hub is a different kind of thing from what's already out there.

So given the above, I think that this is where this PR lands:
* For the service in general: `the 2i2c Managed JupyterHub Service`

* For an individual hub: `your managed JupyterHub`

Generally in agreement. What does the addition of the word 'Service' in the end mean to you?
I like your managed JupyterHub, without any 2i2c references.
What do you think of the trademark issues?

Re: service, I just think it’s a different thing than the Jupyter hubs which is why I called it service. There is a single Jupyter hub, the collection of Jupyter hubs that 2i2c manages, and then there is the “act of managing Jupyter hubs for others, that 2i2c carries out”, which I’d call the Managed jupyter hub service. Does that make sense? We could call it something else, like project? How does the “managed Jupyter hub project” sound?
sorry for all the weird spaces above, those are added by my ipad haha
for trademarks, I think there’s it’s likely fine to use but value in asking regardless. But we should try to nail down the terminology we want to use first before we ask them. Maybe @fperez has an intuition there?

/cc @colliand @rabernat  as well, who might have opinions on this terminology.

A few comments - looking at the page again, I feel like the Kinds of Hubs section is not really accurate. We differentiate "self serve" and "bespoke" hubs, which doesn't seem correct given what we're doing. Perhaps we should either remove this section entirely, and instead direct people to the pilot.2i2c.org site? Or update that language to reflect our current structure of base hub, dask hub?

@choldgraf yeah, I opened #44 to restructure the 'kinds of hubs' doc

@choldgraf I've moved the features page rewording to https://github.com/2i2c-org/2i2c-org.github.io/pull/49 (will rebase this on that once that's merged). So we can keep the conversation here focused on naming only

I'm gonna merge this in as I think that it is an improvement on what we have there, and we may be rearranging things in a bit which will create a ton of merge conflicts if we don't get it in


Resizable unpacked area

Hello,
could it be possible to make the expanding area resizable? When i put there more complex layout i just dont respect its height.
In my case multile lines of icons.

It's true that it is a bit too rigid with respect to flexible heights. I'll give it a look this weekend!


Refactor Time Period Component

Definition of Ready

[ ] Does this meet the INVEST criteria?
[ ] Does this have a clearly defined user?
[ ] Does this have an estimate?
[ ] Does this have acceptance criteria?
[ ] Does this have sufficient context (wireframes, business rules, etc.)?
[ ] Is this traceable to a feature, epic, or PI?

User Story
As a
I want
So that
Acceptance Criteria
Given
When
Then
Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[ ] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[ ] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context
(User journeys, design mockups, architectural diagrams, business rules, data element definitions)
Refactor time period component during I+P sprint
Testing
Test Case:
Tester:
Definition of Done

[ ] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[ ] Has the code been unit tested and peer reviewed?
[ ] Have the functional tests been executed?
[ ] Have all defects been reviewed and dispositioned, resolved, or deferred?
[ ] Has the user story been accepted by the Product Owner?
[ ] Is the user story ready to be deployed to the test/staging environments?

During backlog refinement on 9/8, the team decided to put this ticket on the product backlog to revisit the ticket at a later time. More discussions needs to happen about the value of this task

Ticket no longer valid


Unhandled exception. System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\runneradmin\AppData\Local\Temp\wingetcreate'.

Brief description of your issue
The executable exits with an error after successfully submitting a pull request when running in GitHub actions.
Steps to reproduce
See the error here and the way it's installed/used here.
Expected behavior
It doesn't exit with an error
Actual behavior
Telemetry Settings
------------------
The Windows Package Manager Manifest Creator collects usage data in order to improve your experience.
The data is anonymous and collected only by Microsoft.
By default, telemetry is enabled but can be disabled by running `wingetcreate settings` and editing your settings file.

Manifest validation succeeded: True

Submitting pull request for manifest...

Pull request can be found here: https://github.com/microsoft/winget-pkgs/pull/120206

Unhandled exception. System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\runneradmin\AppData\Local\Temp\wingetcreate'.
   at System.IO.Enumeration.FileSystemEnumerator`1.CreateDirectoryHandle(String path, Boolean ignoreNotFound)
   at System.IO.Enumeration.FileSystemEnumerator`1.Init()
   at System.IO.Enumeration.FileSystemEnumerator`1..ctor(String directory, Boolean isNormalized, EnumerationOptions options)
   at System.IO.Enumeration.FileSystemEnumerable`1..ctor(String directory, FindTransform transform, EnumerationOptions options, Boolean isNormalized)
   at System.IO.Enumeration.FileSystemEnumerableFactory.FileInfos(String directory, String expression, EnumerationOptions options, Boolean isNormalized)
   at System.IO.DirectoryInfo.InternalEnumerateInfos(String path, String searchPattern, SearchTarget searchTarget, EnumerationOptions options)
   at System.IO.DirectoryInfo.GetFiles(String searchPattern, EnumerationOptions enumerationOptions)
   at System.IO.DirectoryInfo.GetFiles()
   at Microsoft.WingetCreateCLI.Common.CleanUpFilesOlderThan(String cleanUpDirectory, Int32 cleanUpDays)
   at Microsoft.WingetCreateCLI.Program.Main(String[] args)
   at Microsoft.WingetCreateCLI.Program.<Main>(String[] args)

Environment
https://github.com/actions/runner-images/blob/main/images/win/Windows2022-Readme.md

Will be fixed by #452


Article for BFS

📚 Documentation
(A clear and concise description of what the issue is.)
Have you read the Contributing Guidelines on Pull Requests?
Yes
How BFS works and Implementation of it in c#

hey, @SalmaZakaria will you be working on this issue? If not please let me know I will be doing it.

@nagasrihith608 I am working on it


Train with 16k data

Hi, thanks for the awesome work.
I have a question here, if i train the model using 16k data , will the trained model guarantee the performance?

Thanks for the quick reply, i will do some experiments on the 16k data

@ws-choi Interesting. I am thinking of 16KHz sampling rate as well for realtime conference. It would be appreciating if you will get a good result.


If you use an invalid book id when downloading, no error is shown

shibby.sh -d --lib 3034690 -b 19028248

No download path provided -- will save the audiobook to ~/audiobooks

That is where the output ends. There should be an error or something that says the book ID doesn't exist or returned nothing.

In general we need better error handling throughout. Another place this doesn't show an error is if you are placing a hold and it doesn't work.


CI: Investigate setup bundler script failures

Investigate the intermittent failures in our setup bundler script on ruby versions 2.6 and below.
Error examples:
DEBUG: running 'gem update --system --force'
ERROR:  Error installing rubygems-update:
	Unable to resolve dependency: user requested 'rubygems-update (= 3.0.6)'
ERROR:  While executing gem ... (NoMethodError)
    undefined method `version' for nil:NilClass
Updating rubygems-update
DEBUG: obtaining pre-installed Bundler version
DEBUG: pre-installed Bundler version is 2.3.22
DEBUG: running 'gem install bundler'
Successfully installed bundler-1.17.3 as a default gem
1 gem installed
DEBUG: running 'gem list bundler'
bundler (2.3.22, default: 1.17.3, default: 1.17.2)
DEBUG: RUBY_VERSION = 2.6.10
DEBUG: running 'gem uninstall bundler'
Successfully uninstalled bundler-2.3.22
DEBUG: running 'gem list bundler' again
bundler (default: 1.17.3, default: 1.17.2)
DEBUG: running 'bundle config'
/opt/hostedtoolcache/Ruby/2.6.10/x64/bin/bundle:23:in `load': cannot load such file -- /opt/hostedtoolcache/Ruby/2.6.10/x64/lib/ruby/gems/2.6.0/gems/bundler-1.17.3/exe/bundle (LoadError)
	from /opt/hostedtoolcache/Ruby/2.6.10/x64/bin/bundle:23:in `<main>'
DEBUG: running 'bundle install'
/opt/hostedtoolcache/Ruby/2.6.10/x64/bin/bundle:23:in `load': cannot load such file -- /opt/hostedtoolcache/Ruby/2.6.10/x64/lib/ruby/gems/2.6.0/gems/bundler-1.17.3/exe/bundle (LoadError)
	from /opt/hostedtoolcache/Ruby/2.6.10/x64/bin/bundle:23:in `<main>'
Error: Process completed with exit code 1.

Retrying download gem from https://rubygems.org/ due to error (2/4): Gem::RemoteFetcher::UnknownHostError timed out (https://rubygems.org/gems/rb-fsevent-0.11.2.gem)Installing rb-fsevent 0.11.2
Fetching rb-inotify 0.9.10
Installing rb-inotify 0.9.10
Fetching listen 3.0.8
Installing listen 3.0.8
Fetching method_source 0.9.2
Installing method_source 0.9.2
Fetching minitest 4.7.5
Installing minitest 4.7.5
Fetching minitest-stub-const 0.6
Installing minitest-stub-const 0.6
Fetching mocha 1.14.0

Retrying download gem from https://rubygems.org/ due to error (2/4): Gem::RemoteFetcher::UnknownHostError timed out (https://rubygems.org/gems/mocha-1.14.0.gem)
Retrying download gem from https://rubygems.org/ due to error (3/4): Gem::RemoteFetcher::UnknownHostError timed out (https://rubygems.org/gems/mocha-1.14.0.gem)Installing mocha 1.14.0
Using newrelic_rpm 8.10.1 from source at `.`
Fetching parallel 1.22.1
Installing parallel 1.22.1
Gem::RuntimeRequirementNotMetError: parallel requires Ruby version >= 2.5. The
current ruby version is <IP_ADDRESS>4.
An error occurred while installing parallel (1.22.1), and Bundler cannot
continue.

Installing coderay 1.1.3
Fetching ffi 1.15.5
Installing ffi 1.15.5 with native extensions
Gem::RuntimeRequirementNotMetError: ffi requires Ruby version >= 2.3. The
current ruby version is 2.2.0.
An error occurred while installing ffi (1.15.5), and Bundler cannot continue.
Make sure that `gem install ffi -v '1.15.5' --source 'https://rubygems.org/'`
succeeds before bundling.

In Gemfile:
  listen was resolved to 3.0.8, which depends on
    rb-inotify was resolved to 0.9.10, which depends on
      ffi
Make sure that `gem install parallel -v '1.22.1' --source
'https://rubygems.org/'` succeeds before bundling.

In Gemfile:
  rubocop-minitest was resolved to 0.22.0, which depends on
    rubocop was resolved to 1.36.0, which depends on
      parallel

Gem::RuntimeRequirementNotMetError: rubocop-ast requires Ruby version >= 2.6.0.
The current ruby version is <IP_ADDRESS>.
An error occurred while installing rubocop-ast (1.21.0), and Bundler cannot
continue.
Make sure that `gem install rubocop-ast -v '1.21.0' --source
'https://rubygems.org/'` succeeds before bundling.

In Gemfile:
  rubocop-minitest was resolved to 0.22.0, which depends on
    rubocop was resolved to 1.36.0, which depends on
      rubocop-ast

This issue appears to have magically resolved itself just as suddenly as it began. I suspect rubygems may have had some issues, that have now been resolved, that resulted in these failures. If we see these again for some reason, we will look back into it.


Security: XXE in initDocumentParser

The method initDocumentParser() in the XMLSchedulingDataProcessor.java does not forbid DTDs, which allows a context-dependend attacker to perfom an XXE. The vulnerability is confirmed working in version 2.2.3.
I reported this issue to the MITRE cooperation already which assigned the identifier CVE-2019-13990 to this vulnerability. Please confirm the existence of this vulnerability so that the CVE-entry can be completed and published.
The OWASP Foundation provides an XXE Prevention CheatSheet that explains in detail the steps to prevent this security issue.
Credits to Andreas Schoedl for working out the relevant code section affected by this issue.

@krabbenpuler I've been taking a look at this, and I think I have a patch I can send as a PR for master. Do you have a specific test case I can verify it with?

Sure, the following attack vector worked in my case:
`

]>

xxe

native

&xxe;

org.quartz.jobs.NativeJob

true

false

`

That's great - thanks for that.

PR for master: https://github.com/quartz-scheduler/quartz/pull/472

I'll send an email, but I'm happy to file a CLA if needed.

Same patch for the quartz-2.2.x branch. https://github.com/quartz-scheduler/quartz/pull/473

@krabbenpuler You said you got the CVE, why are the versions in the CVE up to and including 2.3.0 instead of 2.3.1?

@ddillard Good point, I'm actually not sure why only the 2.3.0 made it into the CVE. I'll inform MITRE about that as soon as possible.

@jgallimore  How can we avoid this issue in current 2.3.0 version without your patch? Any workaround or setting in the qutarz configuration file  can avoid this issue?

@julyhurt I think it depends on how you're using and driving Quartz. For context, I'm coming from Apache TomEE, where we have an old version of TomEE that uses Quartz for EJB timers, so we're using it as a library, rather than standalone. Its possible that our use of Quartz isn't actually vulnerable, but this issue flagged up on a scan, so I'm more than happy to help fix it.
In terms of mitigating this without a patch - if you're passing XML to Quartz, and that XML is untrusted, I would do something to check it before handing it over to Quartz - so that might mean parsing it with a well configured parser first, and then passing it to Quartz.
There's no 2.3.x branch, but I'm happy to patch 2.3.0 and create a branch for that. I'm not a committer on Quartz though, so I can't actually get anything into the repository, or get a release made. I have tried posting to the user and dev forums, both of which seem to have disappeared (for me at least) and I've tried emailing SoftwareAG to see if they need a signed CLA, per the guidelines here: https://github.com/quartz-scheduler/quartz/blob/master/docs/contribute.adoc.
If yes, do you have committed a patch for it?

@vineetpandey It does affect 2.3.1. There's no 2.3.x branch for me to create a PR against. There doesn't seem to be much movement here, but it seems like there are folks interested in patches, so I'm trying to push some new binaries so people can try them out. I would create a 2.3.x as part of that.

As there hasn't been any activity on this for over a month now, I have forked this repository, and created builds for 2.2.x and 2.3.x, and have staging repositories for both on oss.sonatype.org (these would go on to Maven central if promoted):
https://oss.sonatype.org/content/repositories/orgtomitribe-1071/ -> 2.2.4 (source: https://github.com/tomitribe/quartz/tree/quartz-2.2.4)
https://oss.sonatype.org/content/repositories/orgtomitribe-1072 -> 2.3.2 (source: https://github.com/tomitribe/quartz/tree/quartz-2.3.2)
I'm doing some testing on these here, but I'd be grateful if anyone else can try too (@krabbenpuler, maybe you could try your tests against these to see if my patch holds up)
Please note that the groupId for these artifacts is org.tomitribe.quartz as opposed to org.quartz-scheduler, so your dependency would need to be
    <dependency>
      <groupId>org.tomitribe.quartz</groupId>
      <artifactId>quartz</artifactId>
      <version>${quartz.version}</version>
    </dependency>

I don't know how long the staging repositories will be kept on oss.sonatype.org for, but I suspect its 14 days. We'd need to promote or drop them in that time.

@jgallimore a colleague of mine will test your build soon since I'm currently out of office. Thanks for your effort though, glad to see that some people are willing to work on that issue!

I am actually surprised Terracotta has not jumped on this since CVE's are a high priority across the industry now.  @jgallimore much appreciated your work hopefully your PR will be accepted and an official release will be done by the Terracotta team.

So the mechanism for the vulnerability to be exploited is someone allowing their quartz to be configured by arbitrary submissions from a client? Seems like a bizarre use-case but happy for the vulnerability to be fixed. Doesn't strike me as urgent to fix but I'd also comment that I like to see projects releasing often with minor fixes as well.

@davidmoten I believe you are correct.  We do in fact have a Quartz UI we built for our clients that lets them change all aspect of their scheduled jobs without a code change.  However they are all admin secured pages and not customer facing pages so it is OK in our case.
I think the real problem is the Whitelist services (we use Sonatype IQ Server) list it as a critical vulnerability of 9 out of 10.  And my client's management team freaks out when they see this and they have corporate policies not to release software with critical vulnerabilities.

@melloware I see, thanks for the detail.

So from what I've gathered from reading this thread, my application should be fine as I use the Quartz library just to set jobs to run at specific times of the day (the times are set in an S3 bucket by a user, and then on start up of the service I request the contents of the bucket)?

What's put in the S3 bucket to set the times?
How is the bucket secured?
Do you trust the user and their input?

If they're providing XML that is parsed with a parser configured in initDocumentParser() then it is potentially vulnerable.

The times are simply Strings e.g. "times": [ "01:00:00" ]
The user should be a trusted one (assuming no one has gained access to their AWS account).
In my application I've created a DateUtils method to extract the hour and minutes of the times (from the list of strings), and then use the CronScheduleBuilder.dailyAtHourAndMinute method to set the trigger time.

@jgallimore Thanks for the help!

Your use of CronScheduleBuilder sounds ok, but obviously everyone needs to make their own assessments, as there are different ways of using Quartz. I can't guarantee that a specific use-case is "fine" or not.
Checking whether this method is called in your system is a start: https://github.com/quartz-scheduler/quartz/blob/master/quartz-core/src/main/java/org/quartz/xml/XMLSchedulingDataProcessor.java#L166

@chrisdennis I know you mostly work on Ehcache but since you are a member of Terracotta can you get this critical CVE merged and Quartz published?  or know who to contact to get it published?

@jgallimore Thanks for the resolution. Much appreciated.
Any chance if this quartz fix can get Terracotta published ?

You're very welcome, and I appreciate the reviews, feedback and comments from everyone here.
I also would like to see a Terracotta version published, but I haven't seen any activity on this project from its committers for something like 7 months, so I'm not sure what will happen.
I did publish my binaries (under a org.tomitribe.quartz groupId) on Maven central in case anyone wishes to use them: https://mvnrepository.com/search?q=org.tomitribe.quartz. Corresponding source code is here: https://github.com/tomitribe/quartz/tree/quartz-2.2.4 and here: https://github.com/tomitribe/quartz/tree/quartz-2.3.2.

@zemian - you are one of the significant contributor of quartz.
Can you get this Terracotta published or redirect to correct person?

@vineetpandey I have been trying to contact @zemian and @chrisdennis they are both active TerraCotta commiters for Quartz and Ehcache and it has not gotten any response.

When you lose interest in a program, your last duty to it is to hand it off to a competent successor.
-The Cathedral and the Bazaar

@melloware I'm working on trying to get a future plan for Quartz figured out, but this is something that will have to happen on its own timeframe - it's likely going to involve a lot of moving parts, many of which I have little to no influence over. Once I have the beginning of a plan figured out I'll likely file an issue for it here. In the meantime I'll look in to whether I can independently get this specific issue released on a shorter timeframe.

Much appreciated @chrisdennis

Hi @chrisdennis - any chance of this fix getting an incremental release in Maven?

@mtarnawa He is working on it.  They are planning on getting a 2.3.2 in Maven with these security fixes.

Just want to get #475 merged in and then I'll move forward with releasing. Looking like that will happen next week now.

Thanks @chrisdennis, @melloware, @jgallimore!
Much appreciated your efforts.
Looking forward to the official release.

@chrisdennis thanks for the contribution, this security vulnerability has been there for a while. May I know when you plan for the release? :)

@flowergoo It was released yesterday as 2.3.2 and is available on Maven Central.


Missing options/arguments?

The README.md has several options outlined, for example, --exclusion and --hr-relevant.
But there are no examples of how to use them and when I simply try to include them, there is an error message saying that those arguments are unrecognized. Were they dropped?
python3 -m ppp demo.xlsx -l English -f doc -t old -p standard -r -e > demo.doc
usage: __main__.py [-h] [-d] [-H] [-o OUTPATH] [-l LANGUAGE [LANGUAGE ...]]
                   [-f {html,doc} [{html,doc} ...]]
                   [-t {default,old} [{default,old} ...]]
                   [-p {standard,detailed} [{standard,detailed} ...]]
                   xlsxfiles [xlsxfiles ...]
__main__.py: error: unrecognized arguments: -r -e

Hmm, that is a good question. When I left PMA, I believe these options were still available. I'd be surprised that they were dropped, but that seems to be the case based on what you are linking there.


New version: GNNGraphs v1.3.0

Registering package: GNNGraphs
Repository: https://github.com/CarloLucibello/GraphNeuralNetworks.jl
Created by: @CarloLucibello
Version: v1.3.0
Commit: e6f4d8027202d54fb033aac73c48881fdabeacb7
Reviewed by: @CarloLucibello
Reference: https://github.com/CarloLucibello/GraphNeuralNetworks.jl/commit/e6f4d8027202d54fb033aac73c48881fdabeacb7#commitcomment-148667030
Description: Graph Neural Networks in Julia

can be closed in favor of #120448


IssueListOptions State::All seems not to work

I'm not sure about this, but it seems that State::All does only list open issues.
When importing the imag repository with https://github.com/matthiasbeyer/git-dit-github-import/, I get ~20 issues imported, but it should be ~400.

Many collections apis have a list method that returns a vec of the first page of results and a iter method that returns a stream over all results. Issues is missing iter http://lessis.me/hubcaps/hubcaps/issues/struct.Issues.html. I'll try and add that in the next release


Update centos

CentOS 8  specific changes for successful build

MariaDB is still broken when using Galera. It fails to start the cluster. I didn't have time to fix that part and had to move on. I will open an issue with the details and try to set aside some time to work on that next week. Everything else builds fine with these changes.


Bus Error

When I run the tool f5ls on a fast5 file I get the following error.
...
eventdetection/events/size=12393
(mean=92.6346, stdv=2.68832, start=28195744, length=7)
basecall(0)/group_list=1D_000
basecall(0)/seq_size=6997
Bus error
Similar behaviour is observed for f5-full or any other tool (such as nanopolish) that uses fast5 library when accessing events in fast5 files.  The information about the system which I get the error is as follows.
Processor: ARMv7 Processor rev 3
Operating system: Ubuntu 16.04.3 LTS
The output from gdb and backtrace is attached herewith.
gdb_out.txt
The bus error seems to have originated from inside the HDF functions. However, I do not think that this is a bug in the HDF library as the h5dump tool provided by HDF output the following without any issues.
h5dump.txt
Can you shed some light on this to fix the issue?

After 2 days of hectic debugging, I think I found the issue. The data type for the Events group as given by h5dump is as follows,
         GROUP "BaseCalled_template" {
            DATASET "Events" {
               DATATYPE  H5T_COMPOUND {
                  H5T_IEEE_F64LE "mean";
                  H5T_IEEE_F64LE "start";
                  H5T_IEEE_F64LE "stdv";
                  H5T_IEEE_F64LE "length";
                  H5T_STRING {
                     STRSIZE 5;
                     STRPAD H5T_STR_NULLPAD;
                     CSET H5T_CSET_ASCII;
                     CTYPE H5T_C_S1;
                  } "model_state";
                  H5T_STD_I64LE "move";
                  H5T_IEEE_F32LE "weights";
                  **H5T_IEEE_F32LE "p_model_state**";

For p_model_state  the data type is H5T_IEEE_F32LE.  However, in fast5.hpp:393 it is declared as double instead of float as below.
struct Basecall_Event
{
    double mean;
    double stdv;
    double start;
    double length;
    **double p_model_state**;
    long long move;

The Bus error dissappeared after I did  this change.

Hi @hasindu2008,
Thanks for looking into this. I don't have access to an ARM development machine so I have limited ability to help.
This issue may have been caused by changes to ONT's fast5 file format (the internal data types have changed a few times in the past). Can you let me know which dataset caused the problem?
Thanks,
Jared

Hi @jts
The dataset that I used was downloaded from (https://github.com/nanopore-wgs-consortium/NA12878/blob/master/Genome.md) under the FAST5 (Signal Level files) section. I am attaching one of those fast5 files here for your convenience. After I changed the data type for p_model_state from double to float, now things are working perfectly.
test.zip


Standardize on cross-spawn

There a known issues with spawn on Windows -  joyent/node#2318 as an example.
As a work around we sometimes use cross-spawn. We will move all usage of spawn to cross-spawn.

https://github.com/joyent/node/issues/2318 seems to me like a correct behaviour by design.
Would t child_process.exec for our tests and running git? That should solve all problems related to directly launching an executable.

Would child_process.exec work for our tests and running git

Good question. I have not tested. I see no issue sticking with cross-spawn it's well maintained, and very small.


'tng_loss' vs 'loss' in in training_step

Before asking:

[x] 1. search the issues.
[x] 2. search the docs.

If you still can't find what you need:
What is your question?
What is the difference between tng_loss and loss in in training_step?
It seems both are displayed in my progress bar although only tng_loss is part of prog (unless loss comes from somewhere else but I don't think I specified it somewhere). But tng_loss jumps around alot more. My guess is tng_loss (or whatever I specify in prog is per batch). If this is correct the main question is, what is loss in the output, is it the same as specified in training_step and if yes, why is it different from tng_loss in the displayed output and why is it displayed although it is not part of prog?
Code
My training_step based on the the documentation example:
def training_step(model, data_batch, batch_nb):
    input, label = data_batch

    out = model.forward(input)
    loss = model.loss(out, label)

    output = {
        'loss': loss, # required
        'prog': {'tng_loss': loss, 'batch_nb': batch_nb} # optional
    }
    # return a dict
    return output

loss and tng_loss are the same here. anything in the ‘prog’ key of the dict is for progress bar display only.


Compatibility issue due to wrong item tag?

(All mods running on latest 1.20.1 version, Forge 47.2.0)
When using Every Compat, Create, Garnished and MrCrayfish Furniture, the game crashes upon clicking/rendering hedges for all the nuts. This is easily avoidable though through Every Compats config where they can just be disabled.
The main issue I'm having is that the Sepia Stem and the Stripped Sepia Stem (Hyphae probably has the same issue?) not being registered correctly by the game, resulting in a lot of the furniture missing or using oak as fallback texture. Is it because their item tag is "#garnished:sepia_logs" instead of "#garnished:sepia_stems"?
Also, when throwing in Supplementaries into the mix of mods, the log even spits out a warning:
[Render thread/WARN] [Supplementaries/]: Failed to generate hanging sign extension texture for garnished:sepia. Could be that the target mod isnt registering their wood type properly java.lang.NullPointerException: Cannot invoke "net.minecraft.client.resources.model.Material.m_119203_()" because the return value of "net.minecraft.client.renderer.Sheets.m_246984_(net.minecraft.world.level.block.state.properties.WoodType)" is null
I hope this helps in someway or another to hopefully fix this. If logs or anything else is needed, I'll provide it!

Apologies, but which mod adds the hedges that you are referring to? It doesn't seem like those are automatically generated with the mods listed.

No, Supplementaries and Garnished do nothing unusual on their own. The crash only occurs with the non-existent hedges when they are loaded. Altough the item Shelf and Sign Post from Supplementaries generate Sepia Variants normally unlike Every Compat and MrCrayfish Furniture, interestingly enough.

It is certainly weird. I'm currently talking with the dev to try and figure it out so that built-in support can hopefully be implemented. Thank you for your patience and understanding regarding this!

No problem! I'm happy to somewhat help. Appreciate your work and looking forward to updates!

Let me know if it's still broken even after updating!

I tried the same setup as originally, just with downgrading Create to .d. Hedges now "work" without disabling them and them crashing the game. Stripped Sepia is still not being recognized, sadly. Uses Stripped Oak as fallback. All Stripped variants for Furniture are still missing.
Supplementaries is also still looking for it, like in the original report.

[Render thread/WARN] [Supplementaries/]: Failed to generate hanging sign extension texture for garnished:sepia. Could be that the target mod isnt registering their wood type properly java.lang.NullPointerException: Cannot invoke "net.minecraft.client.resources.model.Material.m_119203_()" because the return value of "net.minecraft.client.renderer.Sheets.m_246984_(net.minecraft.world.level.block.state.properties.WoodType)" is null

It would've used stripped Sepia to make hanging signs automatically if a mods wood set does not have it. Just how with Every Compat the other Furniture sets would "generate" if you can call it that.

Not sure why, a fix was pushed to Moonlight Lib. Have you updated that?

Yes, I try to keep everything on the latest version. Exception being Create for now, obviously. Using Moonlight Lib 2.8.49. Even tried with a fresh instance for Forge. Don't know if there is something I can do or have to do on my end?

There's not really much else I can do without having to refactor the entire wood set, which would delete any pre-existing instance of the Sepia stems/hyphae in anyone's world. I think you may have to, unfortunately, just disable sepia inside of the config. As for the leaves, the same goes for that as well, unfortunately.

I know this is closed, but I just wanted to add, that there seems to be an issue with some mods that add some sort of stem, be it from the nether or not. So this issue might not be entirely on your side!

I remember talking with the developer, and they said that they had added "stem" and "hyphae" to the word detection list. If it is truly an issue non-exclusive to Garnished, then it is out of my control. Apologies for this, but I would recommend creating a report with the "Every Compat" developer/GitHub page.

Oh yeah, no worries, just wanted to let you know! I want to dig a bit more into this before creating the report to have it as much information as possible. Thank you for looking into it as much as you, were able to!


Preferred way to set changelog header section

Hi there,
I have a short question: What would be the preferred way to set a changelog header section like:
# Changelog

All notable changes to this project will be documented in this file. See [Conventional Commits](https://conventionalcommits.org/) for commit guidelines.

e.g. https://github.com/conventional-changelog/conventional-changelog/blob/master/packages/conventional-changelog-angular/CHANGELOG.md
Should this be defined in a config file or is there a better way to achieve this? Maybe generate an empty "templated" Changelog.md to which the action "appends" changes?
Any help is highly appreciated.

I can't seem to find a option to do this inside the preset/config but we can add it our selfs when we are writing the changelog the file here

I see, thank you for explaining.
I found this yesterday: https://github.com/conventional-changelog/conventional-changelog-config-spec/blob/master/versions/2.1.0/README.md#header-string
I am a new user of the conventional-changelog-action and not quite sure how to configure it properly yet.
Feel free to close this issue, if this is out of the scope of the action.


JoyStream extension protocol specification missing

The README mention a specification reference which seems to be missing, it would be a lot easier to read than deep-dive in the code.

We have not had time to keep our protocol in synch with our old specification, but we will get to it, thank you, we were not aware of the rotten link.


fix posture integration test IDs for production

PR summary

PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

:tada: This PR is included in version 1.2.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update library for Prettier 2

Description
Fixes #1
When used with Prettier ^2.0.0, the library won't work because of the following error:
Error: Cannot find module 'prettier/parser-babylon'

As @elyesbenabdelkader pointed out in #1 , Prettier now uses parser-babel from version 2. These changes:

update prettier dependency version to ^2.0.0
fix parser import from parser-babylon to parser-babel

Checklist

[x] My code follows the code style of this project and yarn lint --fix runs successfully.
[x] I have updated the documentation where necessary.
[x] New code is unit tested and all current tests pass when running yarn test .

@ifiokjr  Do you want me to make a new branch to keep compatibility with Prettier@v1 ?

Looks good 👍


Add Llama-2 models

Addressing in https://github.com/neelnanda-io/TransformerLens/pull/352
[x] Implemented Llama-2-7B and Llama-2-13B
[ ] Implement Llama-2-70B architecture (add Grouped-Query Attention)

Out of curiosity, does TransformerLens currently support models that need multiple devices to run inference (aka. model too big to fit in one device's RAM)? If not, is this the main bottleneck to implementing the Llama-2-70B architecture.
Sorry if this has feature has been added, I've been skimming the docs/issues and I havn't been able to find it yet.

It already supports multiple devices, you need to pass the n_devices
parameter to from_pretrained (or n_devices maybe?). The bottleneck on llama
2 70b was grouped query attention which is currently being added to support
mistral, so it should be easy to add llama 2 70b soon
On Fri, 3 Nov 2023, 10:52 pm Mansi Sakarvadia, @.***>
wrote:

Out of curiosity, does TransformerLens currently support models that need
multiple devices to run inference (aka. model too big to fit in one
device's RAM)? If not, is this the main bottleneck to implementing the
Llama-2-70B architecture.
Sorry if this has feature has been added, I've been skimming the
docs/issues and I havn't been able to find it yet.
—
Reply to this email directly, view it on GitHub
https://github.com/neelnanda-io/TransformerLens/issues/353#issuecomment-1793212370,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASRPNKJDLJJXUJQVKBDOD5TYCVYSPAVCNFSM6AAAAAA2T7T25GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJTGIYTEMZXGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


HDDS-1571. Refactor ContainerPlacementPolicy.

#HDDS-1571
Refactor ContainerPlacementPolicy to Placement Policy for future exploit.

The path looks overall good. Just one small document issue.

+1. Committed to branch HDDS-1564.


Handle multi-arch container images

Closes #263. See also #309 for other fixes to the pipeline that relies on these helpers.

Is there a workflow where the changes are tested? Looks like we need to manually trigger one?

I think the changes are only used in the triage jobs, which need more fixes to work properly. Some of those fixes are included in https://github.com/NVIDIA/JAX-Toolbox/pull/315, but I think we also need a more stable (i.e. green) baseline for running the triage jobs to make sense.
Perhaps we can trigger the triage job manually in the meantime if you want to test this PR?

Tried launching https://github.com/NVIDIA/JAX-Toolbox/actions/runs/7016290505 with

What this PR is doing? I didn't understand it from the description.

What this PR is doing? I didn't understand it from the description.

It's fixing some of the helpers for multi-architecture OCI images (when we have both amd64 and arm64 behind the same container registry tag). We weren't seeing the problem this fixed because the auto-triage is not working right now.


[EPIC] .NET 8 Data for Beginners video series

Adapt EF Core for Beginners (.NET 6) content to new .NET Data for Beginners series (.NET 8)

[x] Submit outlines
[ ] Intro video - https://github.com/dotnet/docs/issues/41616
[ ] SQL/ADO.NET - https://github.com/dotnet/docs/issues/41617
[ ] Intro to Object Mappers: Dapper and Entity Framework Core - https://github.com/dotnet/docs/issues/41619
[ ] Entity Framework Core Migrations
[ ] Entity Framework Core Migrations
[ ] Web apps with EF Core
[ ] NoSQL
[ ] Publishing

Back burnered


Confusing "Inbox" which is not the Inbox, when tapping on notification of multiple new mails

I have several folders. Most are set to be polled (second class for push), just one, the Inbox, is set to be pushed (1st class for push). In the "Fetching mail" settings I have "Poll folders" set to "1st &2nd" (only 2nd isn't possible) and Push folders to "only 1st". At least I hope I set this correctly. No mailbox is set to be shown in the "unified inbox".
When K9 puts up a notification that there are several new mails, and I tap on it, then...
Expected behavior
I expect that the Inbox is opened, or possibly, a mailbox corresponding to one of the messages from the notification. (There are also notifications of single emails, which open the particular email directly. The issue does not pertain to this case.)
Actual behavior
However, I seem to get a sort of unified inbox of ALL unread messages (not just the new ones since last notification) from all polled and pushed mailboxes alike (or maybe from just the mailboxes that contain new messages; this is difficult to see). The heading however is Inbox. This is very confusing, because many of these messages are not actually in the Inbox.
When I leave this "Inbox" by tapping on the < top-left, and then tap on Inbox, I do get the Inbox as I expected it.
Steps to reproduce

See above, basically

Environment
K-9 Mail version: 5.403 (ok, my own variant with the old icons, from https://github.com/Rhialto/k-9/releases )
Android version: 5.1
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

Now tracked under #5565


cursor.eachAsync() terminates early in v2.3.13+

@neumino the commit that fixed #263, 4efa6e6b8393acb7f98ed00d3340e6abd65e123b, appears to have broken cursor.eachAsync(). It only iterates over the first ~6k rows before mysteriously resolving. I've confirmed my code still works if I roll back to v2.3.12.

Thanks for the report @marshall007 - Fixed in 2.3.17


[docs] Fix docs for CLI options

When running markmap-cli locally (version 0.5.3) I see these CLI options in markmap -h.
Potentially what is documented in this README was outdated?
Especially the --no-toolbar option seemed to be missing.
Not sure what happened to --enable-mathjax and --enable-prism.
Are these maybe always enabled now (and hence the CLI options were removed)?

Sorry for the late response. Thanks for the update.

Not sure what happened to --enable-mathjax and --enable-prism.
Are these maybe always enabled now (and hence the CLI options were removed)?

The features are detected by plugins and included on demand now, so the options are not needed.


Update dependencies

Just to check: have you seen https://github.com/CQCL/pytket-qiskit/pull/349 ?

Just to check: have you seen #349 ?

Ah, no I hadn't.


style(Mathlib/Tactic/*): rename wrongly capitalized filenames

Mathlib.Tactic.Congrm
→ Mathlib.Tactic.CongrM
Mathlib.Tactic.Existsi
→ Mathlib.Tactic.ExistsI
Mathlib.Tactic.Widget.Congrm
→ Mathlib.Tactic.Widget.CongrM
Mathlib.Tactic.Widget.Gcongr
→ Mathlib.Tactic.Widget.GCongr
Also, we rename the namespaces and subroutines in these files.

I don't think we want to make this change without renaming the namespaces and subroutines in the code as well?
And I'd argue sudo is either SUDo for substitute user do or Sudo as a single word.

@erdOne
OK, I'll rename the namespaces and subroutines.
However, sudo means "superuser do" so SuDo is correct.

@erdOne
I'm sorry to misunderstood the naming convention: Acronyms are written upper-/lowercase as a group, not all abbreviation.
So these names are capitalized correctly:
Mathlib.Tactic.Linarith.*
Mathlib.Tactic.SudoSetOption
I revert these changes.

@adomani
The main purpose of thgs PR is fixing filenames in Mathlib.Tactic, so renaming test/Existsi.lean is extra.

bors merge


Fix compilation error against musl

Musl seems to use an int rather than an unsigned int for the args to the ioctl function as glibc does. This results in a compilation error when building with for example, target = armv7-unknown-linux-musleabihf
This patch changes the type alias to the correct type on musl crates. I imagine this might also be a problem on other libc implementations, but I'll leave those as they are since I can't test them.

Hi, thanks for your contribution.
Could you please fix the CI error by running cargo fmt and force pushing the branch? While you're at it, I'd prefer a "sys: Fix compilation ..." prefix on the commit message
.
I'll merge the commit and release a new version to crates.io afterwards :)

Of course, should be fixed now


Port Console's LoadingBox component

Port Console's LoadingBox component
Addresses following problems:

flickering due to different spinners used by the Console and
by the plugin. Solved by using the same component.
loading state (which results in flickering) when the user enters
for the second time an extension page (although data is cached).
Solved by using a delay.

Note that the component uses the CSS directly  from the Console.
Reference-Url: https://github.com/openshift/console/blob/52cf627de2e8e4164176fc49edbea9c8a5ed4c92/frontend/public/components/utils/status-box.tsx#L70
Signed-off-by: Radoslaw Szwajkowski<EMAIL_ADDRESS>

@sjd78 @yaacov

I like a spinner taking over the isLoading duties from the StandardPage already!

This is not that straightforward: we are adding not replacing the load-in-progress and empty state handlers in StandardPage. To get full user experience we should have 2 full paths:

no items to display - empty state in "success" state, LoadingDots for in-progress and probably some error state (makes no sense to display a table then)
more then one item to display -  rows in the table in "success" state, spinner inside a table (LoadingDots will cause flickering) and the existing inline error

We should also handle the transitions between both paths. A follow-up is needed here.
Note however in current state of the code it's quite hard to trigger loading state for the second time: useK8sWatch API seems not to toggle the flag once loaded (the updates are received via websocket).  The loading state might be triggered  by blocking access to inventory i.e. on the Provider screen.


Ajax with Html page

Why this plugin doesn’t work with a full html page, even though it is set as Ajax?
I’m trying to use a magnific popup to show this page https://www.thepcwow.com/ajax/gur.html
But notne of my attempts worked, and on iFrame it gets the black bkg, and  I can’t change that either... what should I do?

I guess Ajax wouldn't work because of CORS.
iframe is a right choice but you need to custom markup and style them by yourself
http://jsfiddle.net/boatkung/eg8xw9d1/
(default markup made for iframe of video. It will keep 16/9 ratio and too much wide for your content)


"Poiana Largului", "Roșeni", "Ruseni", "Săvinești" and "Topoliceni" are listed in the drop-down menu

Description:
"Poiana Largului", "Roșeni", "Ruseni", "Săvinești" and "Topoliceni" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Poiana Largului" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Poiana Largului, NEAMȚ (Poiana Teiului)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Roșeni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Roșeni, NEAMȚ (Poiana Teiului)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Ruseni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Ruseni, NEAMȚ (Poiana Teiului)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Săvinești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Săvinești, NEAMȚ (Poiana Teiului)".

Step 8
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 9
Type "Topoliceni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Topoliceni, NEAMȚ (Poiana Teiului)".


New version: RecurrenceAnalysis v1.0.4

Registering package: RecurrenceAnalysis
Repository: https://github.com/JuliaDynamics/RecurrenceAnalysis.jl
Created by: @Datseris
Version: v1.0.4
Commit: becc782147e93b3f99359f1f8729762c5900a737
Reviewed by: @Datseris
Reference: https://github.com/JuliaDynamics/RecurrenceAnalysis.jl/commit/becc782147e93b3f99359f1f8729762c5900a737#commitcomment-36629719

Please close this in favor of https://github.com/JuliaRegistries/General/pull/7405


Safari 16.1 shipped today

Safari 16.1 shipped today for macOS, iOS and iPadOS.

https://webkit.org/blog/13399/webkit-features-in-safari-16-1/
https://developer.apple.com/documentation/safari-release-notes/safari-16_1-release-notes

Also, now that Safari 16.1 has shipped, it'd be a good time to create the ability for Can I Use to receive JSON data about Safari 16.2.

Thanks, has now been updated. Safari 16.2 has also been added.


Not found error

I've set these lines to my settings:
$api = new MySQL_CRUD_API(array(
'hostname'=>'localhost',
'username'=>'xxx',
'password'=>'xxx',
'database'=>'xxx'
));
However I get a 'not found (entity)' error.
What am I doing wrong?

Did you add the name of a table to your URL?

I use this URL: http://localhost/~username/api.php
(as you've noticed probably: I'm on a mac)
Where should I put the table name?

Like this:
http://localhost/~username/api.php/tablename

Great! That works. It shows me all the records found in that table. Thanks for the answer.
Now i want to use this API in an angular application. Therefor I need the result as a JSON output. Am I right to append your '?transform=1' method to get a json output? However when I use $http.get("api.php/bml?transform=1").success(function(data){
$scope.records = data;
});
I get no result. Note: 'bml' is the table name.
I'm running this application on localhost.

Instead of
api.php/bml?transform=1
Try the full URL.
Also use the debugger to see what's going on.
Good luck.


refactor(GroupTheory/PGroup,Sylow): Replace Fintype.card with Nat.card

A lot of the group theory library still needs to be switched over from Fintype.card to Nat.card. This PR switches over GroupTheory/PGroup.lean and GroupTheory/Sylow (there was enough dependence between the two that it was simplest to do them together).

Thanks :tada:
bors merge


How to solve problem with scale/size differences when registering images

Hi, I'm trying to register two segmentations of proximal femurs, but due to image size differences (and maybe due to a different cut off point), elastix has problems with registering the images.
I am using a similarity transform to also take into account the scaling. Since the images aren't the same size, I have resampled the movingImage to fit the fixedImage, but the movingImage segmentation is much smaller than the fixedImage segmentation.
Does anyone know a (simple?) solution for this? Thanks in advance!
Image:

Code:
parameterMap = sitk.GetDefaultParameterMap('rigid')
parameterMap['Transform'] = ['SimilarityTransform'] # Scaling is also applied
parameterMap['AutomaticScalesEstimation'] = ['true']

elastixImageFilter = sitk.ElastixImageFilter()
elastixImageFilter.SetFixedImage(fixedImage_box)

movingImage_box = sitk.Resample(projection_box, fixedImage_box)
elastixImageFilter.SetMovingImage(movingImage_box)

elastixImageFilter.SetParameterMap(parameterMap)
elastixImageFilter.LogToConsoleOn()
elastixImageFilter.Execute()

resultImage = elastixImageFilter.GetResultImage()
transformParameterMap = elastixImageFilter.GetTransformParameterMap()

Your images hopefully have a pixel resolution (i.e., 1 unit/px) associated with it. ITK uses the physical spacing of the image to do most calculations. elastix uses this information, so essentially a pixel at index [2,2] in an image with resolution of 1.5x1.5mm would be at 3x3mm. elastix will compare values to develop registration based on their position in physical space rather than pixel index space. So if you know this information your images shouldn't need to be scaled by the transformation beyond minor changes.
You can set this manually:
moving_im.SetSpacing((1.0,1.0))
fixed_im.SetSpacing((2.0,2.0))

Thank you, that works! Do you also have a solution for the fact that the moving femur is much longer (in shape) than the fixed femur? Due to this, the registration is sometimes not that good. I now solved this by just increasing the size of the fixed image by adding padded 0's at the bottom, but I don't know if a better solution is available.

You could try adding a binary mask. Just make sure it has the same origin, spacing and size as your image. Here is a nice primer on SimpleITK images and how to get and set this info:
https://simpleitk.org/SimpleITK-Notebooks/01_Image_Basics.html
The other thing to look into is AutomaticTransformInitialization in your registration parameters. Check out the elastix manual for a bit on this:
https://usermanual.wiki/Document/elastix490manual.1389615963/html

Thank you for answering!
I am (now) already using the AutomaticTransformInitialization, that seems to help a bit.
The addition of a mask didn't really do anything, probably since the input images are already binary images and sort of boxed to the region of interest.
However, I guess I'm quite content with the results of my solution so I will just keep this approach. There probably isn't a solution that solves everything.


Fixes spelling issues and import alphabetical issues

Using goreportcard.com I noticed a few spelling errors. Also a few of the import were out of order.
I really love codegangsta/cli and wanted to help improve it.

Nice, thank you for the cleanups!


This is really similar to Control.Monad.Free.Church

As /u/yakrar points out on reddit, this seems to be really similar to Control.Monad.Free.Church from free, judging from the instances. Just FYI.
Maybe you can inspire @ekmett to work out the connection to his blog post series? :) http://comonad.com/reader/2011/free-monads-for-less-2/

Gotcha
{-# LANGUAGE RankNTypes #-}

module Foo where

import Control.Monad

newtype Freer f a = Freer { runFreer :: forall m.

I really think you want to compare to the fused-effects approach from https://people.cs.kuleuven.be/~tom.schrijvers/Research/papers/mpc2015.pdf, who claim performance on par with mtl.
Basically, it also uses a specialisation of Codensity/Cont to have the same cheap bind instance (ignore the types, just look at the implementation, it's all Cont).

Hi! Thanks for the interest and the links! I'm traveling right now, but can't wait to take a look when I get a chance.

fwiw I think this is the inverse:
freer2F :: Functor f => Freer f a -> F f a
freer2F (Freer m) = m $ \f -> F $ \kp kf -> kf $ fmap kp f

I went down this road for a bit; turns out the F monad doesn't optimize as nicely as just doing all of your binds in the eventual monad.

@isovector Is that also the case if you use the church encoded FreerT instead of Freer?  (FreerInput f a (m r) -> m r) -> m r

The "eventual monad" is visible here as m in FreerT. The overall type of a computation in FreerT is just some function that accepts a monadic effect producing callback which can be fed effect requests, and somehow invokes it with a bunch of effect requests, sequencing them together into a final m r it spits out.
In Control.Monad.Trans.Free.Church they CPS away even that FreerInput datatype, but that version's really hard for me to wrap my head around.

The monad instance for FreerT of course doesn't require anything of anything; you don't even need a functor instance on f. So you should be able to use it with a functor that's an open union of functors or whatever else you like.


Remove unneccessary AppsandFeaturesEntries * manifests/g/Guyutongxue/Vscch/4.0.9/Guyutongxue.Vscch.installer.yaml

Microsoft Reviewers: Open in CodeFlow

URL:
Invalid URI: The hostname could not be parsed.
(Automated message - build 899)


Compilation-Errors on Linux

When compiling OmniView on Linux, I've encountered several issues during the cmake process. Given the less than ideal developer experience, I'm considering whether it would be beneficial to include the necessary prerequisites in the README.md file. These prerequisites could be listed as follows:
apt install build-essential autoconf gtk-doc-tools libtool intltool curl zip unzip tar

This would help streamline the setup process for other developers and potentially reduce the number of issues encountered during the compilation process.

Sounds good to me, but keep in mind that apt only works for some distros.


Connection error! ER_NOT_SUPPORTED_AUTH_MODE: Client does not support authentication protocol requested by server; consider upgrading MySQL client

I CANT CONNECT MY DATABASE WITH VISUAL STUDIO CODE

me too


Set up the Kanban board

Kanban task assigned
Kanban Board: https://github.com/AndreaM2429/JavaScriptCapstone/projects/1
We are 2 group members.

Hello @AndreaM2429  & @ClaudiaRojasSoto
Your project is complete!  There is nothing else to say other than... Congratulations! 🎉

Highlights

You implemented all requirements :heavy_check_mark:
You used the proper Project template :heavy_check_mark:

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

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


Error "unknown shorthand flag: 'n' in -network

Getting the following error:
unknown shorthand flag: 'n' in -network

Might be due to me running it in ubuntu 20?

Double-dash (--) is required when supplying command line args in extended format.
https://docs.docker.com/engine/reference/run/#network-settings

i.e. --network not -network


Integrated FileSaver.js

SaveAs doesn't really do saving as on Firefox/Linux and Chromium/Linux.

This doesn't work on my Mac on any of Safari, Firefox, or Chrome.

This library hasn't been updated in two years. There's a new fs API, and basically the call we want is https://developer.mozilla.org/en-US/docs/Web/API/Window/showSaveFilePicker. Unfortunately, it's experimental and isn't available on all browsers. Fortunately, it can be polyfilled quite easily: https://github.com/jimmywarting/native-file-system-adapter/

This doesn't work on my Mac on any of Safari, Firefox, or Chrome.

Yeah it was mostly a demo of "saveAs" actually just being a polyfiller for "downloadWithThisGivenName".

Fortunately, it can be polyfilled quite easily: https://github.com/jimmywarting/native-file-system-adapter/

I didn't know about this, this is very cool. It seems that it should also be able to handle drag and drops, which we could use for imports.

Fortunately, it can be polyfilled quite easily: https://github.com/jimmywarting/native-file-system-adapter/

I didn't know about this, this is very cool. It seems that it should also be able to handle drag and drops, which we could use for imports.

Oh that would be really cool. We could also let people drag and drop a .hom file into a signature (#523).

I'll close this PR and have a crack at integrating that in the coming days.


Compiler does not enforce read only properties

Our team hit an issue when turning on strict in our code. Consider the following code:
class bug {
public get id() { return 5; }
}
var b = new bug();
b.id = 10;
In this example, the id property only declares a getter, however, it is still possible to assign a value to it in TS. This is neither a syntax or compile time error in TS (1.4.0). Is there a workaround? Or Is this something that is already being fixed?
Diego

see #12


Project still Active ? :)

Is this Project still Active ? :)

I’m happy to accept pull requests for bug fixes, but otherwise I wouldn’t say this project is particularly active. It’s been fairly feature-complete for a while, and Nightmare itself hasn’t had a single new commit to its master branch in more than a year.
Were you after anything in particular?

(FWIW, I don’t actually have any use-cases for this or Nightmare in my own work these days, since most of what I used to use it for is better done with Puppeteer or Spectron now.)

Sir, is there anyway to Hire you for 1 - 3 hours of consultation ? are you available on upwork or freelancer  ? :)


New version: syvaidya.openstego version 0.8.6

Result: Installation Failed

Manifest
Add/Remove Programs

Name
OpenStego

Version
0.8.6

Publisher
Samir Vaidya

ProductCode

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 5647
Microsoft Reviewers: Open in CodeFlow

@wingetbot run

@wingetbot run


Changing backend docker images to registry.access.redhat.com/ubi8/ubi-minimal:8.4

Changing backend docker images to registry.access.redhat.com/ubi8/ubi-minimal:8.4
Signed-off-by: Raj Das<EMAIL_ADDRESS>
Proposed changes
Summarize your changes here to communicate with the maintainers and make sure to put the link of that issue
Types of changes
What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices applies)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] I have signed the commit for DCO to be passed.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)

Dependency

Please add the links to the dependent PR need to be merged before this (if any).

Special notes for your reviewer:

Closing this PR. Not required now


The links to "10-100 VC Features per image" and "10-100 Updown Features per image" are invalid

Hi, the links to "10-100 VC Features per image" and "10-100 Updown Features per image" are invalid. Can you update new links?
Thanks!

have you owned it yet?