Distance between two vectors

Hello, I have a question. Is it possible to find distance between two vectors using SAT.Vector class?

You should just be able to subtract one from the other - that should give you a Vector whose magnitude is the distance between them
Example:
Given a vector A (0,1) and B (1,0), subtracting A from B gives vector C (1, -1) , which is a vector that goes from B to A if you start at B (the magnitude of that vector is the distance between them)

Yes it's right.
Anyway, I made a function to determine distance which return "some" number. But seems like this don't fit the library and make same thing like len function.


GSE161383 - SARS-CoV2ControlHostGenes

Project short name:
SARS-CoV2ControlHostGenes
Primary Wrangler:
Ida
Secondary Wrangler:

Associated files

Google Drive: folder

Published study links

Paper: Single-cell multiomic profiling of human lungs reveals cell-type-specific and age-dynamic control of SARS-CoV2 host genes

Accessioned data: super-series GSE161383

GSE161382 - scRNA-Seq
GSE161381 - scATAC-Seq

Ingest

Key Events

[x] Convert published metadata to HCA spreadsheet
[x] Manually curate dataset to meet HCA metadata standard
[x] Collect any matrix and cell-type annotation files
[ ] Are the analysis files suitable for CellxGene? If something is missing get in touch with the authors to request it
[x] Upload sheet to validate metadata
[x] Transfer raw files to ingest to validate data files
[x] Check linking using ingest graph validator
[ ] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[ ] Submit dataset to Production
[ ] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

@Wkt8 to secondary review

Hi Ida! Looks great! Just a couple things. Also we should add the ontology terms you added here to the best practices document!
donor_organism:
Use HANCESTRO ontology term for African-American for the Black/AA ethnicity donor
dissociation_protocol:
Extra space at the start of 'nuclei_extraction_RNA' description
enrichment_protocol:
I think you should add a snRNA-seq enrichment protocol for the following "75,000 nuclei were sorted
using a SH800 sorter (Sony) into 50 mL of collection buffer consisting of 1 U/mL RNase inhibitor in 5%
BSA; the FACS gating strategy sorted based on particle size and DRAQ7 fluorescence" as a FACS sorting. Particle size is unknown but it seems relevant.
analysis_protocol:
can add 'depth normalization' to the derivation_process for ATAC_umap


[CT-1204] [Feature] Add shorthand for --full-refresh flag

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
I write (and misspell!) --full-refresh a whole heck of a lot during dbt development. Would love to add a shorthand for that flag in the sameway that --select can be reduced to -s, --target can be reduced to -t etc.
Describe alternatives you've considered
Keeping on keeping on! There's a tradeoff here between clarity and brevity, so i understand we probably don't want to make every single flag shorthand!
Who will this benefit?
keyboard users
Are you interested in contributing this feature?
Yes!
Anything else?
No response

I like it...perhaps we can tackle in the click work for 1.4? @iknox-fa how easy would this be? should we wait, or do this now?

I thought that shorthand params needed to be 1 char so we could chain them together, e.g. dbt -x -q run is the same as doing dbt -xq run(and it is the same behavior for most CLI tools, like ls -la being the same as ls -l -a).
Did we check what happens now that we have a shorthand param with 2 chars?

@b-per Whoa, I didn't know about that! It isn't something I checked (or thought about) when we were deciding between -fr and a single-character version (-f).
Links I quickly found:

https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap12.html
https://stackoverflow.com/questions/67145068/does-click-support-bunching-short-options-together (click is our new in-development CLI, replacing the current one based on argparse)

Each option name should be a single alphanumeric character (the alnum character classification) from the portable character set.

If there's a lot of additional utility in doing this, we could swap to the single char version (-f)

whoa who knew! Definitely open to changing it! I have never chained flags together as i'm sure you can tell

This is another link mentioning that from the POSIX standpoint short params should be 1 char. Not sure if we are forcing ourselves to be POSIX compliant though.
I thing that -f is a pretty good candidate instead of -fr.

Despite my love for the very memeable -fr, I am convinced

may the -f be with you both!
wanna PR it up? (and edit the description here: https://github.com/dbt-labs/docs.getdbt.com/issues/2055)?

on it!

@b-per -- out of curiousity, if i was going to chain flags together that require arguments (like -t or -s, so the args just follow the order of the flags as you combine them?
i.e.
$ dbt build -s i_miss_fr -t postgres
$ dbt build -st i_miss_fr postgres

are the same?

I believe that chaining short parameters is only supported for those that don't require any value. So build -xf works but to add a selector it will be build -xf -s my_model.


"/bs" no longer working after update to Genesis 1.0.0

I just updated from bossshoppro-2.1.0-R0.9-20240307.151018-1.jar to genesis-1.0.0-R0.4-20240601.183146-1.jar and nothing is working on my server. All /bs commands in all possible configs are no longer working. Yould you please add the command /bs as an alias again?!

The bs alias was removed and all references to BossShop and related names have been changed to Genesis. Though we're a continuation of BSP itself, we've rebranded and those commands will not work anymore.


Add a missing assert and fix some typos

Thanks for creating this! It's a nice tutorial for beginners.

Thanks!


Airflow Auth fix + Version Downgrade + Increased Docker Min Memory Limit

Describe your changes :

Worked on adding auth configs to support okta, google, auth0 and azure.
Support for Ingestion UI to create dags on Airflow using 2.1.4 version.
Increasing min limit of docker to 6 gigs (recommended)
fix #3231
referring #2162 (will close once this PR is merged)

Type of change :

[x] Bug fix
[x] Improvement

Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own.
[x] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@harshach @akash-jain-10

@ayush-shah  We should update the number while throwing memory error as well:
https://github.com/open-metadata/OpenMetadata/pull/3233/files#diff-5aecff9002d01e9aa88e724016caf56c8e331c8997a82a2fbdce13a4a984a482L148

Some UI tests are failing and might be related:
2022-03-08T19:29:13.8003269Z [INFO] Summary of all failing tests
2022-03-08T19:29:13.8009462Z [INFO] FAIL src/components/Modals/AddServiceModal/AddServiceModal.test.tsx (9.485 s)
2022-03-08T19:29:13.8015114Z [INFO]   ● Test AddServiceModal Component › form with all the Field should render for databaseServices
2022-03-08T19:29:13.8015568Z [INFO]
2022-03-08T19:29:13.8018281Z [INFO]     TypeError: ServiceUtils_1.getAirflowPipelineTypes is not a function
2022-03-08T19:29:13.8018708Z [INFO]
2022-03-08T19:29:13.8020781Z [INFO]       405 |     setSelectService(service);
2022-03-08T19:29:13.8023533Z [INFO]       406 |     if (isIngestionEnable) {
2022-03-08T19:29:13.8028631Z [INFO]     > 407 |       const ingestionTypes = getAirflowPipelineTypes(service, true) || [];
2022-03-08T19:29:13.8031110Z [INFO]           |                              ^
2022-03-08T19:29:13.8036616Z [INFO]       408 |       const ingestionScheduleList: IngestionListType[] = [];
2022-03-08T19:29:13.8037063Z [INFO]       409 |
2022-03-08T19:29:13.8039530Z [INFO]       410 |       ingestionTypes.forEach((s, i) => {
2022-03-08T19:29:13.8039897Z [INFO]
2022-03-08T19:29:13.8042927Z [INFO]       at handleServiceClick (src/components/Modals/AddServiceModal/AddServiceModal.tsx:407:30)
2022-03-08T19:29:13.8045216Z [INFO]       at onClick (src/components/Modals/AddServiceModal/AddServiceModal.tsx:1678:36)
2022-03-08T19:29:13.8049664Z [INFO]       at HTMLUnknownElement.callCallback (node_modules/react-dom/cjs/react-dom.development.js:188:14)
2022-03-08T19:29:13.8054150Z [INFO]       at innerInvokeEventListeners (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:315:27)
2022-03-08T19:29:13.8058109Z [INFO]       at invokeEventListeners (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:266:3)
2022-03-08T19:29:13.8062736Z [INFO]       at HTMLUnknownElementImpl._dispatch (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:212:11)
2022-03-08T19:29:13.8067520Z [INFO]       at HTMLUnknownElementImpl.dispatchEvent (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:87:17)
2022-03-08T19:29:13.8071911Z [INFO]       at HTMLUnknownElement.dispatchEvent (node_modules/jsdom/lib/jsdom/living/generated/EventTarget.js:144:23)
2022-03-08T19:29:13.8076238Z [INFO]       at Object.invokeGuardedCallbackDev (node_modules/react-dom/cjs/react-dom.development.js:237:16)
2022-03-08T19:29:13.8079910Z [INFO]       at invokeGuardedCallback (node_modules/react-dom/cjs/react-dom.development.js:292:31)
2022-03-08T19:29:13.8084456Z [INFO]       at invokeGuardedCallbackAndCatchFirstError (node_modules/react-dom/cjs/react-dom.development.js:306:25)
2022-03-08T19:29:13.8087854Z [INFO]       at executeDispatch (node_modules/react-dom/cjs/react-dom.development.js:389:3)
2022-03-08T19:29:13.8091622Z [INFO]       at executeDispatchesInOrder (node_modules/react-dom/cjs/react-dom.development.js:414:5)
2022-03-08T19:29:13.8099477Z [INFO]       at executeDispatchesAndRelease (node_modules/react-dom/cjs/react-dom.development.js:3278:5)
2022-03-08T19:29:13.8101279Z [INFO]       at executeDispatchesAndReleaseTopLevel (node_modules/react-dom/cjs/react-dom.development.js:3287:10)
2022-03-08T19:29:13.8102775Z [INFO]       at forEachAccumulated (node_modules/react-dom/cjs/react-dom.development.js:3259:8)
2022-03-08T19:29:13.8198945Z [INFO]       at runEventsInBatch (node_modules/react-dom/cjs/react-dom.development.js:3304:3)
2022-03-08T19:29:13.8199510Z [INFO]       at runExtractedPluginEventsInBatch (node_modules/react-dom/cjs/react-dom.development.js:3514:3)
2022-03-08T19:29:13.8200027Z [INFO]       at handleTopLevel (node_modules/react-dom/cjs/react-dom.development.js:3558:5)
2022-03-08T19:29:13.8200523Z [INFO]       at batchedEventUpdates$1 (node_modules/react-dom/cjs/react-dom.development.js:21871:12)
2022-03-08T19:29:13.8201028Z [INFO]       at batchedEventUpdates (node_modules/react-dom/cjs/react-dom.development.js:795:12)
2022-03-08T19:29:13.8201584Z [INFO]       at dispatchEventForLegacyPluginEventSystem (node_modules/react-dom/cjs/react-dom.development.js:3568:5)
2022-03-08T19:29:13.8202133Z [INFO]       at attemptToDispatchEvent (node_modules/react-dom/cjs/react-dom.development.js:4267:5)
2022-03-08T19:29:13.8202620Z [INFO]       at dispatchEvent (node_modules/react-dom/cjs/react-dom.development.js:4189:19)
2022-03-08T19:29:13.8203027Z [INFO]       at unstable_runWithPriority (node_modules/scheduler/cjs/scheduler.development.js:653:12)
2022-03-08T19:29:13.8235491Z [INFO]       at runWithPriority$1 (node_modules/react-dom/cjs/react-dom.development.js:11039:10)
2022-03-08T19:29:13.8236460Z [INFO]       at discreteUpdates$1 (node_modules/react-dom/cjs/react-dom.development.js:21887:12)
2022-03-08T19:29:13.8237710Z [INFO]       at discreteUpdates (node_modules/react-dom/cjs/react-dom.development.js:806:12)
2022-03-08T19:29:13.8238595Z [INFO]       at dispatchDiscreteEvent (node_modules/react-dom/cjs/react-dom.development.js:4168:3)
2022-03-08T19:29:13.8239674Z [INFO]       at innerInvokeEventListeners (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:315:27)
2022-03-08T19:29:13.8240654Z [INFO]       at invokeEventListeners (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:266:3)
2022-03-08T19:29:13.8241721Z [INFO]       at HTMLDivElementImpl._dispatch (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:212:11)
2022-03-08T19:29:13.8242609Z [INFO]       at HTMLDivElementImpl.dispatchEvent (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:87:17)
2022-03-08T19:29:13.8274940Z [INFO]       at HTMLDivElement.dispatchEvent (node_modules/jsdom/lib/jsdom/living/generated/EventTarget.js:144:23)
2022-03-08T19:29:13.8276136Z [INFO]       at fireEvent (node_modules/@testing-library/dom/dist/events.js:624:18)
2022-03-08T19:29:13.8277222Z [INFO]       at Function.fireEvent.<computed> [as click] (node_modules/@testing-library/dom/dist/events.js:690:36)
2022-03-08T19:29:13.8278054Z [INFO]       at node_modules/@testing-library/react/dist/pure.js:149:40
2022-03-08T19:29:13.8279106Z [INFO]       at batchedUpdates$1 (node_modules/react-dom/cjs/react-dom.development.js:21856:12)
2022-03-08T19:29:13.8315844Z [INFO]       at act (node_modules/react-dom/cjs/react-dom-test-utils.development.js:929:14)
2022-03-08T19:29:13.8316913Z [INFO]       at Function.fireEvent.<computed> [as click] (node_modules/@testing-library/react/dist/pure.js:148:28)
2022-03-08T19:29:13.8317911Z [INFO]       at src/components/Modals/AddServiceModal/AddServiceModal.test.tsx:131:17
2022-03-08T19:29:13.8318569Z [INFO]       at step (node_modules/tslib/tslib.js:143:27)
2022-03-08T19:29:13.8319383Z [INFO]       at Object.next (node_modules/tslib/tslib.js:124:57)
2022-03-08T19:29:13.8319996Z [INFO]       at fulfilled (node_modules/tslib/tslib.js:114:62)
2022-03-08T19:29:13.8320715Z [INFO]
2022-03-08T19:29:13.8321233Z [INFO]
2022-03-08T19:29:13.8355201Z [INFO] Test Suites: 1 failed, 72 passed, 73 total
2022-03-08T19:29:13.8355880Z [INFO] Tests:       1 failed, 166 passed, 167 total

@vivekratnavel the UI tests that failed are not related to this PR. I will create a separate issue to keep a track of the same

oh, the issue is already up, ref: #3282


Bump gradle to 4.0

Reason: build in https://github.com/uber/jaeger-client-java/pull/194 failed a couple of times. Locally it was passing fine, I find out that --paraller was causing the hang. Upgrade gradle to 4.0 helped too.

Codecov Report

Merging #200 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #200      +/-   ##
============================================
+ Coverage     81.29%   81.34%   +0.05%
- Complexity      469      470       +1
============================================
  Files            77       77
  Lines          1812     1812
  Branches        214      214
============================================
+ Hits           1473     1474       +1
  Misses          249      249
+ Partials         90       89       -1

Impacted Files
Coverage Δ
Complexity Δ

.../uber/jaeger/samplers/RemoteControlledSampler.java
92.95% <0%> (+1.4%)
19% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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

It still sometimes hangs on:
> :jaeger-core:test > 80 tests completed

parallel tests takes ~40s
normal: ~1min 5s

I would rather disable --parallel in travis to avoid annoying restarts.


Roll Flutter Engine from 1da070c57c87 to 998b4db1ef1e (3 revisions)

https://github.com/flutter/engine/compare/1da070c57c87...998b4db1ef1e
2023-03-24<EMAIL_ADDRESS>Roll Dart SDK from d94bd4eacd4d to 225c42bede05 (1 revision) (flutter/engine#40587)
2023-03-24<EMAIL_ADDRESS>[impeller] runtime effect doesnt accept opacity (flutter/engine#40579)
2023-03-24<EMAIL_ADDRESS>Roll Skia from 32ffab0c9432 to 86228d7534de (7 revisions) (flutter/engine#40586)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Create a single script for starting a campaign

Create a management command for creating and starting a campaign from a command-line. This should be fully automatic, including submitting batches and setting task agendas.

In the last meeting (8/7/2021) we agreed that the progress in this task could be prioritized as follows:

Addng an option to init_campaign.py to specify batches.json.
Wrapping all scripts that need to be run to create a campaign (see Examples/*/README.md) into a single one.
Extending the script to submitting multiple batches at once using a .zip file with manifest and batches following specific naming conventions.

Such on-button script would simplify creating regression tests (#4).
To further simplify creation of new campaigns, we could consider the following:

Pre-selecting newest agenda/market in the admin panel when adding a new batch.
Making it doable via the admin panel, which would be a more long-term goal.


Why don't you use the callback on socket.complete?

Hello, I'm testing out using your node module to send streams to clamav. Sometimes it looks like the "data" event doesn't get called. But since your code doesn't use the callback for the "close" event on the socket, it just hangs and never completes the request. On the requests that do not hit the "data" event, I can see the stream being successfully scanned in the clamav logs. I can provide clamav logs and configuration if that is helpful. Thanks.
I was wondering if this happens on very small files. So I increased the size of the file to over 500k and I still saw the same results.
Next I was wondering if it might happen when clamav uses its cache to determine that a file has already been scanned. So I changed the DisableCache configuration to 'yes' and still saw the same thing.
I'm wondering if you are doing this for a specific reason or if it is a bug?
To help you I have uploaded the file that seems to hit the "data" event sometimes, but other times it just goes directly to the "close" event.
Thanks for your help!
test-file.txt

After doing some packet sniffing what I found was that node js seems to be automatically sending the FIN tcp packet on the socket immediately after sending the last byte of data from the stream. As soon as Node sends that FIN packet it creating a race condition in the Clamav server. It is racing between finishing the scan / sending the response, and processing the FIN packet sent by Node.
This only happens when you pipe a stream into a socket and I have only tested on node 6.2.1.
For my purposes I am already using multipart-form/data so I already have a buffer with the binary file data. So I am just doing a socket.write(buffer) and allowing the clamav to send the FIN packet themselves after they send the response.
Definitely curious to hear if you guys have seen this issue. I am still looking for a way to tell node not to send the FIN packet after sending the entire stream into socket, allowing the other side a chance to response.

After doing some packet sniffing what I found was that node js seems to be automatically sending the FIN tcp packet on the socket immediately after sending the last byte of data from the stream.

See https://nodejs.org/api/net.html#net_event_end. I think you want allowHalfOpen: true on the socket.

@corybill were you able to find a solution for this?  I've tried several modifications, but sometimes I can't reproduce the problem at all, which makes it really hard to test if a modification fixes the problem.
I don't suppose you are able to consistently reproduce the problem in a unit test?

@gabegorelick allowHalfOpen: true does not make fix the problem for me.

https://bugzilla.clamav.net/show_bug.cgi?id=12181

It has been a while here but my solution was to use "socket.write(buffer)" instead of piping to a stream. The issue I saw that the fin packet was being sent automatically. Have you tried doing "socker.write(buffer)"?

@corybill I tried replacing with:
//    stream.pipe(channel).pipe(socket)
    socket.write("nINSTREAM\n");
    stream.on('data', function(chunk) {
        var size = new Buffer(4);
        size.writeInt32BE(chunk.length, 0);
        socket.write(size);
        socket.write(chunk);
    }).on('end', function() {
        var size = new Buffer(4);
        size.writeInt32BE(0, 0);
        socket.write(size);

        complete(stream);
    })

...it seems to be a bit more stable, but I still see the process occasionally being terminated abruptly before getting a data event on the socket.

I've logged a bug on the ClamAV system: https://bugzilla.clamav.net/show_bug.cgi?id=12181
I've also posted a question on StackOverflow: https://stackoverflow.com/questions/52072840/intermittent-issues-with-clamav-clamd-instream-on-socket

In case anybody is still interested. 😄
If feel like replacing l41 with
stream.pipe(channel).pipe(socket, { end: false }).on('end', function() {
prevents sending the FIN packet as soon as the write is finished on the socket, allowing to wait for the clamav reponse.


Implement a timer based progress reporter

Adding Victor since it's based on his reporter's implementation from task-runner.
Closes #104

Superseded by https://github.com/livepeer/catalyst-api/pull/218, which tracks all job statuses in one place and should mean we have a foundation for doing some of the load calculation / balancing work that's been discussed


community[patch]: Update doc-strings for text2vec embeddings

Add that URL of the embedding tool "text2vec".
Fix minor mistakes in the doc-string.

@hulitaitai if you can try to improve the commit titles -- it'll make it easier for other developers to understand what changes happened (most developers won't know what text2vec.py is)

I am so thankful. I learnt so much with you.

@hulitaitai if you can try to improve the commit titles -- it'll make it easier for other developers to understand what changes happened (most developers won't know what text2vec.py is)

I am so thankful. I learnt so much with you.


Prepare CI for release

This PR closes #23

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fixcheck table privilege when it's needed

check privilege on LogicalOlapScan, LogicalEsScan, LogicalFileScan and LogicalSchemaScan
Issue Number: close #xxx

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

run buildall


Explorer folder preview/thumbnails.

Description of the new feature / enhancement
Please add what was in the old versions of Windows - the ability to see on the folder image what is in it.
Scenario when this would be used?
Please add what was in the old versions of Windows - the ability to see on the folder image what is in it.
Supporting information
No response

Hi this feature was added in Windows 11.


⚠ Bofrost.fr has degraded performance

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

HTTP code: 200
Response time: 1164 ms

Resolved: Bofrost.fr performance has improved in 31bc9ec.


Support StringLiteral#split(RegexLiteral)

I would like to be StringLiteral#split(RegexLiteral) in the macro should work as String#split(Regex).
class Klass
end

DATA = <<-'EOF'
a
b
c
d
e
EOF

A = {{ DATA.split(/\n/) }}
B = {{ DATA.split(5) }}
C =  {{ DATA.split(Klass) }}
D = {{ DATA.split("\n") }}
E = {{ DATA.split('\n') }}
F = DATA.split(/\n/)
G = DATA.split(5)
H = DATA.split("\n")
I = DATA.split('\n')
# J = DATA.split(Klass)

pp! A
pp! B
pp! C
pp! D
pp! E
pp! F
pp! G
pp! H
pp! I

A # => ["a\n" + "b\n" + "c\n" + "d\n" + "e"]
B # => ["a\n" + "b\n" + "c\n" + "d\n" + "e"]
C # => ["a\n" + "b\n" + "c\n" + "d\n" + "e"]
D # => ["a", "b", "c", "d", "e"]
E # => ["a", "b", "c", "d", "e"]
F # => ["a", "b", "c", "d", "e"]
G # => ["a", "b", "c", "d", "e"]
H # => ["a", "b", "c", "d", "e"]
I # => ["a", "b", "c", "d", "e"]

https://play.crystal-lang.org/#/r/9jrt (Crystal 0.35.1)
EDIT: I was misunderstanding the doc, or I was looking at the old doc. It only has StringLiteral#split(ASTNode). But I think better if the argument type is restricted.

Workaround for now, use #gsub(RegexLiteral) to replace your regex with some unique string, then use #split(StringLiteral) with that.  E.x. {{ DATA.gsub(/\n/, "\|--\|").split("|--|") }}.


syncDrupalPreviewRoutes bundle size

I've noticed that if I add syncDrupalPreviewRoutes in my app.tsx my bundle size is increased by about 100kb however if I just include the function in my app.tsx sync still functions
function syncDrupalPreviewRoutes(path) {
  if (window && window.top !== window.self) {
    window.parent.postMessage(
      { type: 'NEXT_DRUPAL_ROUTE_SYNC', path },
      process.env.NEXT_PUBLIC_DRUPAL_BASE_URL
    );
  }
}

Router.events.on('routeChangeStart', function (path) {
  syncDrupalPreviewRoutes(path);
});

The syncDrupalPreviewRoutes is still part of the "legacy" code that was created before DrupalClient. I believe this is being bundled as part of.
Copying it to your _app.tsx is perfectly fine. (This is what I do as well.
We'll deprecate all the non-client code in 2.0 and move this code as part of docs.


Error while registering autoloader

Hello, Im trying to do a basic render but after requiring the autoload.php and doing
Xiaoler\Blade\Autoloader::register();
I'm getting the following error:
( ! ) Warning: require(./helpers.php): failed to open stream: No such file or directory in /var/www/html/some-path/vendor/xiaoler/blade/src/Autoloader.php on line 12
the line responsible inside Autoloader is:
require './helpers.php';
presumably because the  ./

fixed

fixed

I am still facing the error!


fix: simplify generics

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

#5
#4  👈
#3
#2
#1
main

This stack of pull requests is managed by Graphite.

Merge activity

Feb 26, 5:36 PM EST: @braden-w started a stack merge that includes this pull request via Graphite.


Joushua Dela Cruz - Filter

Close, need to order it country/state/city so instead
filters/united-states/middlesex-county/massachusetts/public-art-json-to-json.js
make it
filters/united-states/massachusetts/middlesex-county/public-art-json-to-json.js
Then you got it!


After upgrading Nuxt to version 3.8, modules are not functioning

Environment

Operating System: Windows_NT
Node Version:     v18.18.2
Nuxt Version:     3.8.0
CLI Version:      3.9.1
Nitro Version:    2.7.0
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      typescript, runtimeConfig, app, devtools, modules, strapi, tailwindcss
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
npm install --save-dev @bg-dev/nuxt-naiveui
npm run dev
Describe the bug
ERROR  Importing directly from module entry points is not allowed. [importing @bg-dev/nuxt-naiveui from app.vue]
console：Uncaught SyntaxError: The requested module '/_nuxt/node_modules/unenv/runtime/mock/proxy.cjs?import' does not provide an export named 'ThemeConfig' (at app.vue:2:10)
Additional context
No response
Logs
No response

Environment

Operating System: Windows_NT
Node Version:     v18.18.2
Nuxt Version:     3.8.0
CLI Version:      3.9.1
Nitro Version:    2.7.0
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      typescript, runtimeConfig, app, devtools, modules, strapi, tailwindcss
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
npm install --save-dev @bg-dev/nuxt-naiveui npm run dev
Describe the bug
ERROR Importing directly from module entry points is not allowed. [importing @bg-dev/nuxt-naiveui from app.vue]
console：Uncaught SyntaxError: The requested module '/_nuxt/node_modules/unenv/runtime/mock/proxy.cjs?import' does not provide an export named 'ThemeConfig' (at app.vue:2:10)
Additional context
No response
Logs
No response

@selvinzr how did you resolve the issue? I am seeing this too, with @vite-pwa/nuxt


[Add]: Please add <>/<> to the validation process

Organization or user
ALM-Rangers
Which repository is the extension in?
azure-boards-kanban-tools

asdas


Fixed incorrect transform

Scaling was applied first, then re-applied as concatenation of each path's transform, resulting in incorrect size / position

Thanks!


Provide RSS feed for feed readers

Description
Nowadays there are tons of information out there, that it is really hard to visit websites/blogs one by one to get read all the news around coding. Instead, providing an RSS feed every user can use its very own feed reader to keep up-to-date :)

Hey, @valentinpalkovic! Thanks for the feedback. I transferred the issue to 30-seconds-web, as it is a platform-related task, instead. This might be investigated in Q2 2020, as an enhancement, so hold on while we figure out when and how we will implement it! :wink:

Due to technical issues and the way we generate our content, ordering etc. we will not be implementing this any time soon. I'm closing this for now, but we might revisit this at a later time (maybe Q3 or Q4 2020) and reopen if we find the time to figure it out. Thanks for the suggestion anyways!


PropTypes not recognized when class not at top scope

In an example such as this, where a React class is constructed as part of a function call:
import React from 'react-native';

export function createReactClass() {
  class ReactComponent extends React.Component {
    render() {
      let {aProp} = this.props;
      return aProp;
    }
  }

  ReactComponent.propTypes = {aProp: React.PropTypes.string};

  return ReactComponent;
}

The prop-types rule incorrectly marks aProp as being missing from the props validation. If this is changed to use a static propTypes, it works as expected (I can't use static because Flow doesn't support it, unfortunately).

Thank you for resolving this so quickly!


feat: port CategoryTheory.Monoidal.Rigid.FunctorCategory

[x] depends on: #4646

This PR/issue depends on:

leanprover-community/mathlib4#4646
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#4646
By Dependent Issues (🤖). Happy coding!

bors merge

Not sure why bors rejected this one. Perhaps a commit was added while it was processing? I've merged master again to get another CI run.
bors d+

Not sure why bors rejected this one.

There was a merge conflict.

bors merge


Disable SDK Actions in the UI

This is the feature branch that disables all actions in the UI for SDK services.

All of the SDK-related PRs have been tested and merged into this feature branch. Let's :shipit:!

This works great still. Let's :ship:

Seems like everything is working fine but I can't see the menu under services. is that expected? @jfurrow @MatApple
Other than that looks good.

@weblancaster - yes that is the expected behavior! Thanks for checking :)


[core] Update renovate GitHub Action dependency updates

Having to merge #1339, #1335, #1336 as different PRs feels overkill.
The solution is based on https://github.com/renovatebot/renovate/discussions/14578.

Isn't that essentially the same group as the one above that?
Aside: If we're going to monitor PR reviews in https://github.com/mui/mui-toolpad/security/code-scanning/22, would it makes sense to always approve dependency update PRs before merging so that it doesn't distort this metric?

Isn't that essentially the same group as the one above that?

I don't know, I copied https://github.com/Automattic/vip-container-images/blob/6057834e5cc1041eeb5bf5296a55b14b1c1952ec/.github/renovate.json#L14-L21

Aside: If we're going to monitor PR reviews in https://github.com/mui/mui-toolpad/security/code-scanning/22, would it makes sense to always approve dependency update PRs before merging so that it doesn't distort this metric?

I think that the idea of having a peer review is to say that 2 people approved the changes. Renovate is a bot, does it count? I'm leaning on a not but I have personally approved the PRs for this reason before 🙃

Isn't that essentially the same group as the one above that?

@Janpot I had a look at the source, the interesting file is https://github.com/renovatebot/renovate/blob/30eb3dda394cd5e52a6c2eacf6cd93188d176e98/lib/modules/manager/github-actions/extract.ts.
From what I can gathermatchManagers: 'github-actions' matches the .workflow GitHub files. Which has 3 types of matchDepTypes:

docker containers
actions
services


Revert "feat(aws-cdk): add maven wrapper to java template"

Reverts awslabs/aws-cdk#811
As discussed in the placement PR.

Scrap this PR - it's auto generated wrongly.


New function syntax crashes Dartpad

DartPad:
https://dartpad.dartlang.org/269de5e20f399ffc9b48c4b3ea3c67a1
void main() {
  for (int i = 0; i < 5; i++) {
    print('hello ${i + 1}');
  }
}

// Works.
T runLogged1<T>(T Function<T>() run) => run();

// Works.
T runLogged2<T>({T Function<T>() run}) => run();

// Works.
void runLogged3(void Function() run) => run();

// Works.
void runLogged4({void Function() run}) => run();

// Crashes the Dart Analyzer, and I no longer get any analysis completion.
T runLogged<T>(
  T Function<T>() run, {
  String name: 'crosscheck',
  void Function(String message) onLog,
}) => run();

@lukechurch I think this should be fixed in the latest dev release. Has DartPad been updated? If not, do we have a schedule for updating it?

@devoncarew Do you know whether DartPad has been updated?

It's on 1.24.0; @lukechurch  is on vacation but back in a few.

This issue was moved to dart-lang/dart-services#316

Bumped to the dart-services repo -


Schroedinger

A class for solving the time-independent Schroedinger equation, including helper classes for meshing and background potentials. @raynol-dsouza uses a similar home-brewed class for quantum solutions to optimal volume atm.
Known TODOs:

Unit (integration?) tests for the main class
~ImportAlarm wrapping of some of the 3d plotting stuff~
~More and correct physics for thermal occupation of states~
Units

First, just actually using them correctly, not the reduced units for the electron solution currently implied
Second, ideally, leveraging @sudarsan-surendralal's work on generic pyiron unit handling, but that can possibly wait for a future PR

~Polish and upload example notebook~
?Wait for the Toolkit stuff to be available on pyiron_base and expose meshes and potentials there.

Hi @liamhuber ,
Issue with RectMesh: Suppose I want an output from RectMesh.mesh that is 1d, essentially a simple linear interpolation between 2 extrema, let's say 1 and 10, I provide
foo = RectMesh(bounds=[1, 10], divisions=10, simplify_1d=True)
what I expect the output of foo.mesh to look like (same as using numpy.linspace):
array([ 1.,  2.,  3.,  4.,  5.,  6.,  7.,  8.,  9., 10.])
but the output looks like this:
array([[[0.00000000e+00, 9.99000999e-04, 1.99800200e-03, ...,
         9.97002997e-01, 9.98001998e-01, 9.99000999e-01],
        [0.00000000e+00, 9.99000999e-04, 1.99800200e-03, ...,
         9.97002997e-01, 9.98001998e-01, 9.99000999e-01],
        [0.00000000e+00, 9.99000999e-04, 1.99800200e-03, ...,
         9.97002997e-01, 9.98001998e-01, 9.99000999e-01],
        ...,
        [0.00000000e+00, 9.99000999e-04, 1.99800200e-03, ...,
         9.97002997e-01, 9.98001998e-01, 9.99000999e-01],
        [0.00000000e+00, 9.99000999e-04, 1.99800200e-03, ...,
         9.97002997e-01, 9.98001998e-01, 9.99000999e-01],
        [0.00000000e+00, 9.99000999e-04, 1.99800200e-03, ...,
         9.97002997e-01, 9.98001998e-01, 9.99000999e-01]],

       [[0.00000000e+00, 0.00000000e+00, 0.00000000e+00, ...,
         0.00000000e+00, 0.00000000e+00, 0.00000000e+00],
        [9.99000999e-03, 9.99000999e-03, 9.99000999e-03, ...,
         9.99000999e-03, 9.99000999e-03, 9.99000999e-03],
        [1.99800200e-02, 1.99800200e-02, 1.99800200e-02, ...,
         1.99800200e-02, 1.99800200e-02, 1.99800200e-02],
        ...,
        [9.97002997e+00, 9.97002997e+00, 9.97002997e+00, ...,
         9.97002997e+00, 9.97002997e+00, 9.97002997e+00],
        [9.98001998e+00, 9.98001998e+00, 9.98001998e+00, ...,
         9.98001998e+00, 9.98001998e+00, 9.98001998e+00],
        [9.99000999e+00, 9.99000999e+00, 9.99000999e+00, ...,
         9.99000999e+00, 9.99000999e+00, 9.99000999e+00]]])

I would like to think I am doing something wrong and that this is a feature of the class. The documentation is not super clear to me. I understand that 'mesh' should indeed give me a 'mesh' (equivalent to numpy.meshgrid's output), but to use the TISE class, I need to provide it with a RectMesh object since it does not accept a numpy.ndarray or a recursive list of lists with values.

That brings me to my next comment about the TISE class: As an input, I would also like to be able to provide input.potential and input.mesh as simple numpy.ndarray objects (the dimensions of the two would obviously need to match). I believe it would be more convenient to add this also as a feature than the TISE class solely relying on objects of the RectMesh and Potential classes as respective inputs.

Issue with RectMesh: Suppose I want an output from RectMesh.mesh that is 1d, essentially a simple linear interpolation between 2 extrema, let's say 1 and 10, I provide
foo = RectMesh(bounds=[1, 10], divisions=10, simplify_1d=True)
what I expect the output of foo.mesh to look like (same as using numpy.linspace)

Yes, this is easily possible. You just need bounds=[[1, 10]] instead.
The problem is that we can't simultaneously allow [1, 10] and a shortcut to a 2d domain on x=(0..1) and y=(0..10) and also allow [1, 10] to be a 1d domain with different starting points. Any 1d array (equivalent) is interpreted as only providing the end points.

As an input, I would also like to be able to provide input.potential ~and input.mesh~ as simple numpy.ndarray objects

Done.

and input.mesh

This is possible, but I don't like it. We could support both, but this would create a terrible mess of if/else or try/except clauses throughout the code and I'm not willing to support that. We could support only numpy arrays of the right dimension, but then we'd need to implicitly back out the step size in each dimension and implement Laplacian right in this class.
My hope and expectation is that putting operators right on the RectMesh object will be generically useful for the continuum module and that this class will get lifted up out of the schroedinger module at a later date. I have a strong preference to diffuse some of the responsibility among classes. Potential is just a convenience thing, but RectMesh has real responsibilities and I don't want to dump these on TISE.

(the dimensions of the two would obviously need to match).

I no longer guarantee this now that potential accepts regular arrays. It's possible to add these (read: we probably should add this check), but since they can be set in either order it is a pain in the butt so for now any user using array potentials will just have to live dangerously.

Yes, this is easily possible. You just need bounds=[[1, 10]] instead

Aha! I feel dumb for not trying this out. But I would anyway suggest adding this in the documentation for other lost souls like mine out there who'd want to use the class.

This is possible, but I don't like it. We could support both, but this would create a terrible mess of if/else or try/except clauses throughout the code and I'm not willing to support that. We could support only numpy arrays of the right dimension, but then we'd need to implicitly back out the step size in each dimension and implement Laplacian right in this class.

My hope and expectation is that putting operators right on the RectMesh object will be generically useful for the continuum module and that this class will get lifted up out of the schroedinger module at a later date. I have a strong preference to diffuse some of the responsibility among classes. Potential is just a convenience thing, but RectMesh has real responsibilities and I don't want to dump these on TISE
Fair enough. I can live with this. However, I think it would be useful to give numpy.ndarrays directly into the bounds. For ex. for the 2d case:
foo = numpy.array([0, 1, 2, 3, 4])
bar = numpy.array([5, 6, 7, 8, 9])
mesh = RectMesh(bounds=[[foo], [bar]], divisions=None)

I ask this since if I have raw data, in my case, E(V) data from a Murnaghan type job, I would not want to fit the data to a function. But I see that brings an additional problem of having a variable step size for the Laplacian. Could we come up with a workaround for this*?

I no longer guarantee this now that potential accepts regular arrays. It's possible to add these (read: we probably should add this check), but since they can be set in either order it is a pain in the butt so for now any user using array potentials will just have to live dangerously.

Hahaa! I would then suggest that the TISE class accept only objects of the Potential class in the long run. However, I would ask (or take up) the additional job of making the Potential class more flexible, in a way that I would give it my raw potential data (as a numpy.ndarray) with a corresponding mesh (again a numpy.ndarray), and the class would then return a child of itself (I see that the child would need to have a __call__ method) AND a RectMesh wrapped around this raw data. The latter, PROVIDED my * statement is satisfied.

I believe I found the missing factor to use the Schrodinger class exclusively in pyiron units. Quoting from the documentation,

H = -(hbar^2 / 2 m)(del^2 / del x^2) + V
Pyiron units:
- m = atomic mass units
- x = Angstroms
- V = eV
Thus, to get the first term to eV we need hbar in units of sqrt(eV Angstroms^2 u).

I found by dimensional analysis that we can get the entire correction term in eV by keeping HBAR in eV s itself, and using the correction factor which looks like this. I tested it out and it works in my case with a real potential.
I would set HBAR  = scipy.constants.physical_constants['reduced Planck constant in eV s'][0] in the code, keep the mass in pyiron units (For C, that would be 12.011), and line 174 would then return:
-(HBAR**2 / (2 * self.input.mass) * 9.64853322e27) * self.mesh.laplacian(psi) + self._potential_psi(psi)
the magic number in there being the conversion factor.

I found by dimensional analysis that we can get the entire correction term in eV by keeping HBAR in eV s itself, and using the correction factor which looks like this. I tested it out and it works in my case with a real potential.
I would set HBAR = scipy.constants.physical_constants['reduced Planck constant in eV s'][0] in the code, keep the mass in pyiron units (For C, that would be 12.011), and line 174 would then return:
-(HBAR**2 / (2 * self.input.mass) * 9.64853322e27) * self.mesh.laplacian(psi) + self._potential_psi(psi)
the magic number in there being the conversion factor.

Super, that sounds good to me. I would only request that the correction factor be labeled file-level all-caps global at the top of the file, alongside HBAR and KB -- i.e. no magic numbers. Definitely include the Wolfram link next to it, or even better construct it from Scipy constants values (if such a construction is clear/even possible).

Super, that sounds good to me. I would only request that the correction factor be labeled file-level all-caps global at the top of the file, alongside HBAR and KB -- i.e. no magic numbers. Definitely include the Wolfram link next to it, or even better construct it from Scipy constants values (if such a construction is clear/even possible).

I concur! Does this also mean that you are (very politely) asking me to make these changes? :P

Super, that sounds good to me. I would only request that the correction factor be labeled file-level all-caps global at the top of the file, alongside HBAR and KB -- i.e. no magic numbers. Definitely include the Wolfram link next to it, or even better construct it from Scipy constants values (if such a construction is clear/even possible).

I concur! Does this also mean that you are (very politely) asking me to make these changes? :P

Haha, rather 'instructing' you to. You'll also want to re-execute the demo and adjust potential magnitudes and masses there to make sure the graphs still all look "pretty".
I'm not worried about the timescale you do this on though -- it's for your project anyhow, and I'd also like to hold off on merging until pyiron_continuum relies on a version of base that includes the Toolkit stuff.

I think it would be useful to give numpy.ndarrays directly into the bounds. For ex. for the 2d case:
foo = numpy.array([0, 1, 2, 3, 4])
bar = numpy.array([5, 6, 7, 8, 9])
mesh = RectMesh(bounds=[[foo], [bar]], divisions=None)

I ask this since if I have raw data, in my case, E(V) data from a Murnaghan type job, I would not want to fit the data to a function. But I see that brings an additional problem of having a variable step size for the Laplacian. Could we come up with a workaround for this*?

I see how this is useful, but it's also super dangerous. What's to stop a user from entering an array with non-uniform displacements in a given axis? We could support that in the future too, but it would involve more changes (e.g. to the math in laplacian) and I don't have time to make those rigourously right now.
I don't remember what the input to Murnaghan jobs is -- could you not generate a RectMesh before doing the atomistics jobs and use its mesh as input to both the E(V) calculation and the QM solution?

Hahaa! I would then suggest that the TISE class accept only objects of the Potential class in the long run. However, I would ask (or take up) the additional job of making the Potential class more flexible, in a way that I would give it my raw potential data (as a numpy.ndarray) with a corresponding mesh (again a numpy.ndarray), and the class would then return a child of itself (I see that the child would need to have a __call__ method) AND a RectMesh wrapped around this raw data. The latter, PROVIDED my * statement is satisfied.

Actually I do rather like the ability to provide the potential as a straight numpy array. User-defined Potential children are certainly easy enough to implement, but I worry that handling the storage/reinstantiation gets tricky if we insist on everyone making their own child class.
Much easier if they have an array (or function returning an array) in their notebook and slap that on as the potential -- then it gets serialized trivially.
A really determined user could extend the available Potentials or make their own personal pyiron submodule that includes these, e.g. if they have the same class of potential that they want to parameterize and use repeatedly. But let's not make that a necessity.

Thanks @samwaseda, excellent feedback! All implemented.

@liamhuber I was adding the conversion factor to the TISE class to solve the Schroedinger equation in pyiron units, which has mass in AMU. The examples you have in your demo notebooks give not so nice results for mass in AMU. For mass in electron mass though, they work the same as in your notebook.
Shall I keep the default mass in the class as AMU, and express the mass in the demo notebook, which I think works best for an electron, from electron mass to AMU? Or should we address this issue another way?

@raynol-dsouza yes, the demonstrations will certainly need some numeric adjustment with the new (correct) units in place. My gut is to use pyiron units overall, but keep the default mass as electron mass for Schroedinger. I haven't thought it through super deeply, but it should be possible to jimmy the demos to come out nicely by adjusting the size and depth of the potentials. My frame of reference was to think about hydrogen ionization energy (-13 eV for the ground state, -3 eV for first excited) and hydrogen 1s orbital size to give a rough length scale (this I would need to look up).
I'll take a look at your PR soon, but you could use the hydrogen attack to try to adjust the demos already if you want.
The alternative is to make them for carbon or aluminium mass or something, which should then give pretty solutions for energies and lengths that we select with just our gut intuition, and lines up well with pyiron atomistics, but feels off since quantum stuff is usually for electrons by default.

@liamhuber The notebook works fine for hydrogen, provided the bounds for RectMesh are between 0 and 1 (for the 1d case). However, I was unable to recover the ground state and 1st excited state energies.
On the 'mods_to_shcrodinger' (yeah, I mistyped) branch the following code runs well, but I get different eigenenergies.
job1d_square = pr.create.job.TISE('tise1d_square', delete_existing_job=True)
job1d_square.input.potential = SquareWell(depth=10)
job1d_square.input.mesh = RectMesh(bounds=[[0, 1]], divisions=100)
job1d_square.input.n_states = 10
job1d_square.input.mass = 1.00784
job1d_square.run()

This is exactly what I did initially! It looked a little unconvincing to me, so I followed the method I posted. If it looks like it should, then wonderful! If the default mass in electron mass, do we then scale internally to AMU? Or have a flag that specifies atom mass or something else?

This is exactly what I did initially! It looked a little unconvincing to me, so I followed the method I posted.

For me it looks like this

which is totally fine. I did find it pretty sensitive to the well width though.
The notebook should be adjusted so it all looks ok with M_e.

If it looks like it should, then wonderful! If the default mass in electron mass, do we then scale internally to AMU? Or have a flag that specifies atom mass or something else?

We should absolutely keep everything in AMU throughout. We can simply set the default mass to electron mass in AMU: physical_constants['electron mass in u'][0]. Easy peasy.

@samwaseda spurred on by your opinion that this might actually be useful, I'd like to show it at ADIS on Tuesday. I think you, Raynol, and I are all pretty happy with the overall setup, but would you mind taking a look through the documentation and user-interface (i.e. demo notebook) to double check that I'm not missing any important QoL issues?

ahhdammit, pyiron_base still needs another incremental bump before the tests will pass >.<

@liamhuber Sure! Will go through the changes and docs.

Aha, the notebook fails at storage but this is [known and already patched].
@niklassiemer I'd like to present this work at the ADIS meeting on Tuesday -- any chance we can get another base bump by then? IIRC Tuesday is the regular day, so even doing it Tuesday AM would be totally sufficient.

Sure, I will trigger a new release right away 👍 Although Tuesday is now 'regular' we are not bound to it 😄


Merge svg icons with jekyll-assets?

Hi,
I'm using grunt-svgstore for different projects. It allows me to merge all svg icons I'm using into a single sprite.
As I'm already using jekyll-assets for my sass files, I'd love to see a similar functionality here.
I could not find any issue with this topic, so I'm just asking: is this already possible or planned for the future?
Thanks

I feel like this is out of scope for us but I'm happy to let the community comment and vote for a while before the decision is made.  We mostly rely on Sprockets do carry us mostly, and we only integrate with Jekyll deeply, to do this will require us to do a lot of work that we will need to maintain between 2.3 and 3.0.

Unfortunately we have no plans to support this feature in the near future so I'm going to close out this issue since nobody has voted or requested the feature along with you.  I feel it would be a waste of development time to focus on a feature that is not widely used sadly.  We do accept donations for features though if you feel this issue is very important to you.

yeah sure no problem :) Thanks for your feedback!

+1

This ticket has been locked to reduce noise.


Don't prepend 'http://' to paths

Closes https://github.com/arp242/xdg_open.vim/issues/1
Xdg-open accepts more than HTTP urls, for example it can open local
files in a corresponding native applications or FTP uris.
This is really useful in combination with vim-dirvish, which show the
full path of every files (visually hidden).

Won't this break opening something like example.com?

It will,
xdg-open: file 'example.com' does not exist

Is this really a use case ? example.com looks like a file to me.
My use case is can accommodated with a less intrusive change though. vim-dirvish buffers contain absolute paths, which start with a /. The prefix could be added only if the input doesn't start with a /, instead of currently, if it doesn't contain ://.

Is this really a use case ? example.com looks like a file to me.

Yeah, I use it like that frequently; for example for Go imports:
import (
    "context"
    "embed"
    "fmt"
    "math"
    "strconv"

    "github.com/google/uuid"
    "github.com/mattn/go-sqlite3"
    "zgo.at/z18n"
    "zgo.at/zdb"
    "zgo.at/zstd/zcrypto"
    "zgo.at/zstd/zint"
    "zgo.at/zvalidate"
)

I can quickly go to e.g. github.com/mattn/go-sqlite3. It sometimes comes up in some Markdown files and the like too.
I think at least xdg_open#open_url() should continue to work like it does currently, but xdg_open#open() can use the literal input (and then maybe also map gX to open_url(), rather than open()).

I'll make a patch btw

I updated the master branch, which should solve this issue. See the commit and updated docs.

Thanks!


[WIP][SPARK-9686][SQL] Redirect JDBC metadata calls to Spark SQL metadata Hive client

This PR is based on the patch @navis provided in SPARK-9686.
Basically, it overrides all JDBC metadata related methods in SparkSQLCLIService, overrides session HiveConf with proper configurations, and wraps the original calls with hiveContext.metadataHive.withHiveState, so that metadata stored in the real Hive metastore can be retrieved.
Unfortunately, both @navis and I observed that this approach doesn't play well with embedded metastore, because Hive somehow tries to establish multiple connections to the local Derby metastore database while Derby doesn't support multi-connection.
I've being digging this for a while, but failed to figure out a valid fix/workaround for this issue. My impression is that, somehow the HiveMetaStoreClient belongs to metadata Hive client got a snapshot of the HiveConf of the execution Hive client, thus makes Hive try to establish a new connection instead of reusing the existing connection because this HiveConf is considered to be "incompatible" with the session HiveConf.
I'm opening this WIP one for memo and further discussion.

Test build #46539 has started for PR 9911 at commit 65aa390.

Test build #46539 has finished for PR 9911 at commit 65aa390.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46539/
Test FAILed.

Test build #47024 has started for PR 9911 at commit 054df0f.

Test build #47024 has finished for PR 9911 at commit 054df0f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47024/
Test FAILed.

I guess the fundamental issue is when we HiveSessionImpl's getMetaStoreClient is called. Right now, I see the following.

Basically, getFunction may require executionHive or metadataHive. I think other methods require metadataHive. I think to fundamentally solve the problem, we need to somehow control these metastore clients. Or, have a way to make sure they talk to the right metastore.

@liancheng I use your code and build. Then run "sbin/start-thriftserver.sh" failed. the error info is as fellow:
15/12/26 16:28:27 INFO ClientWrapper: Loaded org.apache.hadoop.hive.shims.Hadoop23Shims for Hadoop version 2.6.0
Exception in thread "main" java.lang.LinkageError: loader constraint violation: loader (instance of org/apache/spark/sql/hive/client/IsolatedClientLoader$$anon$1) previously initiated loading for a different type with name "org/apache/hadoop/hive/ql/metadata/Hive"
at java.lang.ClassLoader.defineClass1(Native Method)

Anything new on this? I'm fighting with the same issues with wrong metadata on a local derby metastore


Link beside custom component does not render

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
A link beside a custom component isn't rendering correctly. Example -
import React from 'react';

export default function TestComp(props) {
  return <span>{props.children}</span>;
}

<TestComp>This does not work!</TestComp> [This link does not work](example.com)

renders

https://stackblitz.com/edit/github-2yziua?file=docs/intro.md
Reproducible demo
https://stackblitz.com/edit/github-2yziua?file=docs/intro.md
Steps to reproduce

Add a custom component.
Render the component and write a link beside it

Expected behavior
Link should render correctly
Actual behavior
Link does not render correctly
Your environment

Public source code: https://stackblitz.com/edit/github-2yziua?file=docs%2Fintro.md,src%2Fcomponents%2FTestComp.jsx
Public site URL: https://stackblitz.com/edit/github-2yziua?file=docs%2Fintro.md,src%2Fcomponents%2FTestComp.jsx
Docusaurus version used: 2.1.0
Environment name and version (e.g. Chrome 89, Node.js 16.4): Chrome Version 104.0.5112.79
Operating system and version (e.g. Ubuntu 20.04.2 LTS): Ubuntu 20.04 LTS

Self-service

[X] I'd be willing to fix this bug myself.

Please see https://docusaurus.io/docs/markdown-features/react#markdown-and-jsx-interoperability, first example

Also note the MDX 2 changes/improves markdown and JSX interop https://mdxjs.com/blog/v2/#improvements-to-the-mdx-format.
This syntax would/will be supported with https://github.com/facebook/docusaurus/issues/4029


Remove reference to AWS for on-premises migration document.

@paulkel : Thanks for your contribution! The author, @rayne-wiselman, has been notified to review your proposed change.

Thanks for catching this. #sign-off


Allow staffing agencies to apply employees to companies

As a hiring manager, I should be able to apply employees to client companies.
Acceptance Tests

On the detail page of each staffing agency employee should be a form to apply them to a client company.
The application form should ensure that the user does not already work for the client company.
Submitting the form should create a non-approved Employee instance.

Blocked by #48

Also blocked by #51.


Send order confirmation mail via external service

[ ] Make decision for third-party provider @trangng
[ ] Create adapter for mail provider
[ ] Template: http://sphereio.github.io/commercetools-sunrise-theme/site/en/confirmation-email.html

External service independent from Sunrise.


Copyright of received images

Hello, I would like to ask about the copyright of images received using the library. I understand that the software is under the MIT license (which allows commercial use, among other things). However, does this license also apply to the generated images? Or, alternatively, where could I learn more about this topic?

This is uncharted territory legally. Wait a while and some court cases will come up. You'll have your answer then.

I am not a lawyer so consider the following opinion worthless:
In my opinion, whoever "clicked the button" owns the work.  I don't think it's really uncharted territory but it's good media story if we say it is.  We've had advanced image editing tools for years and no one has been "but you didn't manually pick each individual pixel, maybe photoshop owns the copyright for the pixels it made for you".  This is just another tool.
I would read the stable diffusion license to start with: https://huggingface.co/spaces/CompVis/stable-diffusion-license

Thank you very much for your responses.


Not sure how to run

Cant run it, installing grunt, npm install, or similar, can you tellme how are you starting your app please. Thanks!
-Asael

Take a look at the readme.

Hi! Im not sure why but this is what I receive:
$ npm install
$ npm WARN package.json<EMAIL_ADDRESS>No repository field.

This is not an error, only a warning. Follow the readme and you shouldn't have any issues.


remove manual install of exiftool in documentation and travis

exiftool is currently installed in travis and in documentation with manual command. This is no longer required as debian exiftool package are now fixed.

fixed by #129


tests: Produce a coverage file with atomic mode

We're actually using the atomic counters when running our tests because
we run them with -race (2 lines later). We should set the mode of the
resulting coverage file to match this.
Signed-off-by: Damien Lespiau<EMAIL_ADDRESS>

Coverage remained the same at 38.291% when pulling e402f0cf9b7f095e1bd6b13b8253dae26a4388ff on dlespiau:20170530-atomic-coverage into a30de71a2438d741b5bcc9e23d962dfe4d197b1b on clearcontainers:master.

lgtm

lgtm


Still no release?

Hello! I also encountered the slowness of Hourglass. Please, tell me, did you get a working OurGlass.exe?

You can't even build it, half the required deps are local to their machine


server plugin for spigot/paper

as the title states I am looking for a plugin version of this mod for paper or spigot.
the idea is to have the server send the tilt information to the client (and possibly have a command to toggle this per-player)
this would allow for those willing to have the old behaviour when joining a spigot/paper server.
while I am aware of projects that allow you load plugins with forge/fabric, having a server plugin for spigot or paper is a lot simpler.

Hey, @Scraft161. Please check out https://github.com/Charles445/DamageTilt/issues/3#issuecomment-1178517127.


muscle fatigue - change term label

Consider changing the term label for muscle fatigue MP:0003646 to "increased muscle fatigability"
SYN: increased muscle fatigue
SYN: increased susceptibility to muscle fatigue
SYN: increased muscle fatiguability
SYN: decreased resistance to muscle fatigue
SYN: reduced resistance to muscle fatigue
Also add new term:
decreased muscle fatigability (a child of abnormal muscle physiology MP:0002106)
DEF: decreased muscle exhaustion or decreased susceptibility to muscle exhaustion
SYN: decreased muscle fatiguability
SYN: reduced muscle fatigability
SYN: reduced muscle fatiguability
SYN: decreased susceptibility to muscle fatigue
SYN: reduced susceptibility to muscle fatigue
SYN: increased resistance to muscle fatigue
Annotation  form J:217522 (extensor digitorum is more fatigue resistant)

Changed term label and added all suggested synonyms for MP:0003646
Added new term decreased muscle fatigability MP:0031402 (no EQ added)


Adding more codes to Level0 folder

There are only 2 codes so far in the Level0 folder.
I wish to add few more codes to that folder.

great ! start working on it ... @vryan-06

@Vikash-8090-Yadav get me assigned for this one, I would like to add some contrib in this area :+1:

@Vikash-8090-Yadav get me assigned for this one, I would like to add some contrib in this area :+1:

Please what type of contribution u wana to do ? And which u wana to add

@Vikash-8090-Yadav
1> a contract explaining function modifiers like view vs pure when to use what.
2> Reversing an array
3> checking palindrome for strings (currently we have just for numbers)
...

@legendarykamal   great idea ... Create a new issue regarding this ..
Thank u


Command plugin can't run from CLI

Description
As some bullet points:

SSWG VSCode extension couldn't show OpenAPI generator's command plugin.
Then i noticed swift package plugin --list doesn't include the plugin either:

% swift package plugin --list
‘archive’ (plugin ‘AWSLambdaPackager’ in package ‘swift-aws-lambda-runtime’)
‘format-source-code’ (plugin ‘Format Source Code’ in package ‘swift-format’)
‘generate-manual’ (plugin ‘GenerateManual’ in package ‘swift-argument-parser’)
‘lint-source-code’ (plugin ‘Lint Source Code’ in package ‘swift-format’)

Running the command plugin fails anyway:

% swift package plugin generate-code-from-openapi --allow-writing-to-package-directory
/Users/mahdibm/Github/penny-bot/.build/checkouts/swift-argument-parser/Sources/ArgumentParser/Parsable Properties/Option.swift:570:94: error: multi-line string literal closing delimiter must begin on a new line
        message: """Optional @Options with default values should be declared as non-Optional."""
<Skipping a lot of similar errors>
error: fatalError
[49/153] Compiling Yams Constructor.swift
error: fatalError

Running from Xcode worked perfectly as usual:

...
Considering target 'GitHubAPI':
- Trying OpenAPI code generation.
Swift OpenAPI Generator is running with the following configuration:
- OpenAPI document path: /Users/mahdibm/Github/penny-bot/Lambdas/GitHubAPI/openapi.yaml
- Configuration path: /Users/mahdibm/Github/penny-bot/Lambdas/GitHubAPI/openapi-generator-config.yml
- Generator modes: types, client
- Feature flags: <none>
- Output file names: Types.swift, Client.swift
- Output directory: /Users/mahdibm/Github/penny-bot/Lambdas/GitHubAPI/GeneratedSources
- Diagnostics output path: <none - logs to stderr>
- Current directory: /Users/mahdibm/Github/penny-bot
- Plugin source: command
- Is dry run: false
- Additional imports: <none>
Writing data to file Types.swift...
Writing data to file Client.swift...
- ✅ OpenAPI code generation for target 'GitHubAPI' successfully completed.
...

Expected behavior
The list command should show the command plugin and the command plugin should run.
Actual behavior
As described in Description.
Steps to reproduce
Use https://github.com/vapor/penny-bot/releases/tag/spm-issue-6804.
Do as described in Description.
Swift Package Manager version/commit hash
Swift Package Manager - Swift 5.9.0
Swift & OS version (output of swift --version ; uname -a)
swift-driver version: 1.87.1 Apple Swift version 5.9 (swiftlang-<IP_ADDRESS>.2 clang-15<IP_ADDRESS>)
Target: arm64-apple-macosx14.0
Darwin Macbook.local 23.0.0 Darwin Kernel Version 23.0.0: Tue Aug  1 03:25:51 PDT 2023; root:xnu-10<IP_ADDRESS>.6~31/RELEASE_ARM64_T6000 arm64

The error looks like a compilation error while building argument parser, not sure why that would happen. It seems to work fine for me using
swift package plugin --allow-writing-to-package-directory generate-code-from-openapi GitHubAPI
[...]
- ✅ OpenAPI code generation for target 'GitHubAPI' successfully completed.

swift package plugin --list should be fixed, though.

Not sure why it doesn't contain the OpenAPI plugin, but this already seems wrong? Only plugin products should be available, AFAIK.
static func availableCommandPlugins(in graph: PackageGraph) -> [PluginTarget] {
        graph.allTargets.compactMap { $0.underlyingTarget as? PluginTarget }
    }

The error looks like a compilation error while building argument parser

Yeah it does look very much like a compilation error. The weird thing to me was that Xcode didn't complain about it and successfully ran the plugin.

Ooooh, took me a moment to realize, but the bug is in https://github.com/apple/swift-package-manager/blob/main/Sources/Commands/PackageTools/PluginCommand.swift#L85:
for plugin in allPlugins.sorted(by: { $0.name < $1.name }) {
    guard case .command(let intent, _) = plugin.capability else { return }

So the list stops at the first non-command plugin.

Also fixing --list so that it matches the expected behavior of only including products of direct dependencies: https://github.com/apple/swift-package-manager/pull/6806
❯ swift package plugin --list
‘archive’ (plugin ‘AWSLambdaPackager’ in package ‘swift-aws-lambda-runtime’)
‘generate-code-from-openapi’ (plugin ‘OpenAPIGeneratorCommand’ in package ‘swift-openapi-generator’)


Enhance List Behavior and Filter Functionality

Problem
The current list behavior and filter functionality result in list visibility issues when applying filters. Users have reported that when filters are applied, the list disappears, making it difficult to view subcategories within the list. This negatively impacts user experience and requires users to remove filters to see the list content again.
Description
Improve list behavior and filter functionality to maintain list visibility when applying filters.
Additional context
Attaching a gif showing the enhanced list behavior and filter functionality in action.

Implementation

[x] I would be interested in implementing this feature.

This is a duplicate of #192 and would work better as a recommendation there as a comment to keep the discussion together.


feat: (diagram) Compare EVM transaction anatomy and Starknet transaction anatomy in an excalidraw diagram.

We should think how to make the tx hash. Is it the starknet transaction hash or the hash used by the EVM user to sign. For the moment I have this

I think the tx hash should be what the user signed. Kakarot RPC should be responsible to establish a bijection between EVM tx hash (signed) and derived Starknet tx hash (for block explorer and traceability of txs)
What do you think?
Next question is how to establish a link?
Once one has a EVM tx hash, is the starknet tx hash deterministic?

If the nonce on EVM = nonce on starknet we can definitely do it: https://docs.starknet.io/documentation/architecture_and_concepts/Blocks/transactions/#calculating_the_hash_of_a_v1_invoke_transaction


ng2-simple-autocomplete is not a known element

After installing, and importing, when I try to follow the examples and paste the code in the html, I get the following error:
Angular: 'ng2-simple-autocomplete' is not a known element. 1. If 'ng2-simple-autocomplete' is an Angular component, then verify that it is part of this module. 2. If 'ng2-simple-autocomplete' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.

@rhostem any suggestion?

Sorry I couldn’t  figure out exact problem of this issue. It looks like something related to AOT compilation and angular-cli.
https://github.com/angular/angular/issues/15890
So I’m going to version up this repo for AOT compatibility with angular-library-seed guided by ‘How to create AOT/JIT compatible Angular 4 library with external SCSS/HTML templates
‘. But it will take a time…
If you’d like to use this component in your app now, copying original sources from src/ng2-simple-autocomplete folder will be temporary solution.
And this component works well with angular-starter using webpack, not using angular cli. (But build failed in AOT mode. and succeed in normal build)

@rhostem When do you expect to get an update released?

I'm not sure. Becuase I dont have focus on Angular for now.

@rhostem Alright. In the meantime, I should be able to copy index.ts and model.ts files in src/ng2-simple-autocomplete and use it as instructed?

Yes. then add the component to 'declarations' array of your module.


Gunicorn does not show template error pages

95% sure I fixed this using the Werkzeug debug view wrapper thingy.

Confirmed. This works.


Convert: mixed k-quant with legacy quant fallback

Adds a new cli argument: --fallback-type.
If tensors cannot be quantized to a k-quant because of block size issues, the fallback type will be used instead of full precision.
Very useful for SD3.5 models, because 90% of SD3.5 8B weights can't be quantized to k quants.
--type q4_k --fallback-type q4_0 has always the exact same output size as  --type q4_0, but with less degradation.
Somewhat adresses https://github.com/leejet/stable-diffusion.cpp/issues/446

I'm currently uploading quantized weights to HF, but with my cellular data, it takes very long.

sorry, I get lost in this PR. so the conclusion says a mixed (Q4_K, Q4_0) is better than Q4_1?
are we have something like (perplexity)[https://github.com/ggerganov/llama.cpp/blob/master/examples/perplexity/README.md]?
and the quantized files also confuse me, what does q4_k_4_0 mean? is it a new kind of GGML type?
can we keep following mostly file type definition?https://github.com/ggerganov/ggml/blob/d8ea053461056a5c15f071c7c5ed57d86e892750/include/ggml.h#L408-L436

and the quantized files also confuse me, what does q4_k_4_0 mean? is it a new kind of GGML type?

It's not exactly a new GGML type, it's simply a file type with different weight types mixed in it, kinda like the Q4_K_L, Q3_K_M, and such.
Q4_K is the main quantization type used, Q4_0 is the backup. The only rule to decide if a tensor is going to use the main type or the backup is if the shape of the tensors can fit a whole number of QK blocks or not.
In case of SD3.5 large, the resulting file ends up with more Q4_0 tensors than Q4_K

can we keep following mostly file type definition?https://github.com/ggerganov/ggml/blob/d8ea053461056a5c15f071c7c5ed57d86e892750/include/ggml.h#L408-L436

Even llama.cpp doesn't keep following this definition. https://github.com/ggerganov/llama.cpp/blob/master/examples/quantize/quantize.cpp#L18-#L59

the --fallback-type is difficult to understand, many options below this arg, is it possible to do fallback inside? like when --type q4_k_l, apply q4_k to most tensors, and fallback to q4_0 those mismatched.


add to fedora copr

seeing how there's already an rpm available, it probably won't be too much work + there'll be updates through dnf instead of downloading them manually

cool idea! i was planning on ironing out some small issues with the homebrew and aur packages i created last weekend first, but after that i'll look into copr ^^

some notes while I was browsing for information about copr:

it looks like this is copr and it will build packages for fedora. not sure if that means it will also work nicely for other rpm systems such as opensuse and stuff? maybe I need build.opensuse.org for that?
here is a tutorial on copr with screenshots
i need to create an srpm, which is a source rpm. currently, this is done automatically by fpm, and that doesn't really have an automatic way of extracting the srpm it creates :( so I need to create my own srpm similar to how i created a PKGBUILD for arch. i should think of ways to somehow incorporate those related repositories into this one. like, create a sub-directory build_system  in this repo, with one directory for each build system (homebrew, aur, copr, and other ones?) and then sync those with their respective upstreams on each merge with main. but that sounds like a bad idea because i cannot force-push, but im also too lazy to have to manually keep several repos synchronised, blegh. isn't there some tool like fpm that also does distribution?

hey! good news, i've managed to get mommy onto copr. you can check the project here
the repositories are currently empty, but as soon as there is a new mommy release, copr will be triggered to create new builds. (i already tested the full pipeline in a temporary project on copr that i've now deleted.)
after mommy v1.2.3 releases (probably later this weekend) you can install mommy from copr as follows:
sudo dnf copr enable fwdekker/mommy
sudo dnf install mommy

(you may also need to sudo dnf install dnf-plugins-core to enable the copr plugin for dnf.)
while installing mommy you may be asked to verify a pgp key fingerprint; it probably suffices to verify that the "From:" fields equals https://download.copr.fedorainfracloud.org/results/fwdekker/mommy/pubkey.gpg. (i will explicitly mention the key's fingerprint in the readme... once it has been generated by copr, which happens after the first build)
i'll leave another comment here once v1.2.3 has been released and the copr builds are done. (especially the first build might take a while; today i had to wait 4 hours in my test project.)
either way, let me know if you have any suggestions or comments or tips on using/installing mommy (on fedora-based systems or in general)

mommy v1.2.3 has been released, and the copr build has been completed. you should now be able to install mommy as follows:
sudo dnf copr enable fwdekker/mommy
sudo dnf install mommy

(you will also need to install the dnf-plugins-core package if you don't have it already.)
the package should be signed with a key with fingerprint E332 C8E6 ADAA 58E4 1974  7CE2 CE16 3CFF 9F79 DD8A
thank you for opening this issue! i thought it was really fun and interesting to learn about the fedora packaging ecosystem! i'll close this issue now, but feel free to leave another comment / open an issue if you need help, have questions, or have other suggestions ^^


Draft:  Support Koordinator as one batch scheduler option

Why are these changes needed?
Koordinator is a QoS-based scheduling for efficient orchestration of microservices, AI, and big data workloads on Kubernetes. It aims to improve the runtime efficiency and reliability of both latency sensitive workloads and batch jobs, simplify the complexity of resource-related configuration tuning, and increase pod deployment density to improve resource utilizations.
The integration is easy,  Koordinator support annotation way  to support gang scheduling without podgroup CR
gang.scheduling.koordinator.sh/name
gang.scheduling.koordinator.sh/min-available

Koordinator are compatible with pod-group.scheduling.sigs.k8s.io, pod-group.scheduling.sigs.k8s.io/name and pod-group.scheduling.sigs.k8s.io/min-available in community.
https://koordinator.sh/docs/designs/gang-scheduling

Related issue number
https://github.com/ray-project/kuberay/issues/2573

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

Is this PR ready for review? The PR title is still marked as "draft."

Is this PR ready for review? The PR title is still marked as "draft."

Thanks for your comment, It's still developing, and now it lacks test code.

@kingeasternsun can you please add some tests before marking this ready for review? See other implementation for batch schedulers for reference

Thanks for your advice, I'll  add the tests quickly

Is this PR ready for review? The PR title is still marked as "draft."

Hey, everything is fine now.

@kingeasternsun can you please add some tests before marking this ready for review? See other implementation for batch schedulers for reference

Hi, tests had been added .

Can you add an example YAML similar to this one https://github.com/ray-project/kuberay/blob/e9f31556c14fae6391fb27a4a96bfbe01f917d46/ray-operator/config/samples/ray-cluster.yunikorn-scheduler.yaml?

Please update main.go https://github.com/ray-project/kuberay/blob/e9f31556c14fae6391fb27a4a96bfbe01f917d46/ray-operator/main.go#L96-L97

Please update batch schedulers in helm chart https://github.com/ray-project/kuberay/blob/e9f31556c14fae6391fb27a4a96bfbe01f917d46/helm-chart/kuberay-operator/values.yaml#L60-L87

Can you add an example YAML similar to this one https://github.com/ray-project/kuberay/blob/e9f31556c14fae6391fb27a4a96bfbe01f917d46/ray-operator/config/samples/ray-cluster.yunikorn-scheduler.yaml?

Thank you for your review! I will add it as soon as possible.

Please update main.go
https://github.com/ray-project/kuberay/blob/e9f31556c14fae6391fb27a4a96bfbe01f917d46/ray-operator/main.go#L96-L97

Thank you for your review! I will fix it as soon as possible.

Please update batch schedulers in helm chart
https://github.com/ray-project/kuberay/blob/e9f31556c14fae6391fb27a4a96bfbe01f917d46/helm-chart/kuberay-operator/values.yaml#L60-L87

Thank you for your review! I will fix it as soon as possible.


Sync more to calendar then just Call Log.

In addition to having Call Log appearing on my online Google Calendar (works great for me), I would like to have SMS messages and incoming GMail logged on my GCalendar.  To take it further, you could add FaceBook posts, Tweets, navigation data, and anything else you could possibly get your hands on.  Please fill my GCal history with as much data as possible.

I came here looking for this feature too. SMS backups to google calendar would be great

+1 for SMS backup to google calendar.
+100 :) for ICS support for Google Calendar! (no pressure)


Add E2E test for "Map no-trigger elements"

### Tasks
- [ ] Add test scenario in test management tool
- [ ] Add E2E automation script

Test suite:
IBF-System test plan
Map > no-trigger
Test case:
"All Map elements are present in no-trigger mode"
Given a logged-in user
And no-trigger scenario is loaded
When navigating to {country} floods
Then chat-section shows 'no events' message
And header should show {country} floods non-triggered
And all the dashboard elements should be present:
map:

map itself showing

admin-area boundary layer
breadcrums (National View button only for no-trigger scenario)
expanded map legend
collapsed layer overview

Ready for review in: https://github.com/rodekruis/IBF-system/pull/1597


Read ReCaptcha Site Key from django.settings in sign_up_form.html

The Recaptcha site key is already in local_settings. Add it to the context for the template and use is in the template in place of the hard-coded site key that we want to keep out of GitHub.

Merged in #86.


Make the container and actorsystem protected in AutoFacDependencyResolver.cs

I need to subclass the AutoFacDependencyResolver, however I am currently unable to use the class as it is given because container and actorsystem are private. So I have to copy the class into my codebase.

What issue are you running into that you need to subclass it?
But i don't mind making it protected. Ill be happy to accept PR's.


Fix: undefined address in reverse ENS lookup

What it solves
Resolves #3507
How this PR fixes it
Just a quick fix that checks that the address isn't undefined.
I couldn't reproduce the error from Sentry. We actually don't allow empty addresses.
But this should fix it.
How to test it
Try creating a Safe with several owners, some of which are ENS owners.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 33.507%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
3229

Relevant Lines:
8614

💛  - Coveralls

Looks good to me
ENS Names loaded properly with address that have it:
0x61a0c717d18232711bC788F19C9Cd56a43cc8872
0x7724b234c9099C205F03b458944942bcEBA13408
Names in an AB also loaded properly


Formatting and grammar fixes

The only edit to the workflow is formatting

Thank you @pattmax00.
Can you please make changes without any formatting? I don't want to update the files because of whitespace changes. It also makes things hard for me to see what kind of grammar fixes you introduced.

Thank you @pattmax00.
Can you please make changes without any formatting? I don't want to update the files because of whitespace changes. It also makes things hard for me to see what kind of grammar fixes you introduced.

Yes, my mistake. I will close this PR and open a separate one.


Add support for CWL to Tooltester

Description
This PR adds support for CWL workflows to the tooltester repository. It also adds two test cases that are CWL workflows, those being,
https://github.com/dockstore/dockstore-support/blob/f2b42624bff005f7a2c11aceca4579ed4e688901/tooltester/src/main/java/io/dockstore/tooltester/runWorkflow/WorkflowList.java#L53-L54
In order to use the tooltester run-workflows command, the user is required to specify the paths to two different dockstore config files, one that references an endpoint that runs WDL (only tested with Cromwell) and one that references that an endpoint that runs CWL (only tested with TOIL).
I found it extremely challenging to find workflows on qa.dockstore.org that have CWL workflows that can be used for testing, so I copied (and slightly modifed) two workflows from here. The workflows that I copied live in this repo (this repo is owned by myself, I can move it to one controlled by dockstore before the end of my term),
https://github.com/fhembroff/wes-testing
Launching a workflow with TOIL
I found that running workflows using TOIL is simpler than I thought (originally was looking at implementing a much more complicated approach), they are launched in a similar way to how you run workflows with WDL. The only differences are,

You must use the --inline-workflow flag if you want to use any file that isn't the primary descriptor file (Like how make-array.cwl is used in this workflow). Currently, with cromwell the --inline-workflow will cause an error, and instead any file that is referenced that isn't the primary descriptor file must have an s3 address.
The test parameter file must follow the --json flag. There is no need (and if you include it, it will cause errors) for an agcWrapper.json file.
You can reference https files in a workflows that is being run by TOIL in the agc.

For example, to launch this workflow with TOIL through the dockstore CLI you would use this command,
dockstore workflow wes launch --entry github.com/fhembroff/wes-testing/manyJobs:main --json Dockstore.json

Where,
> cat Dockstore.json
{
    "script_file": {"class": "File", "path": "s3://flynn-test-bucket/make-data.js"},
    "script_argument_count": 10
}

One rather annoying thing about the TOIL endpoint is that the agc logs do not give you any information about what caused an error (all you get is EXECUTOR_ERROR), so debugging the TOIL endpoint can be very challenging. This is different to cromwell, which gives you a detailed log indicating what the errors were.
How TOIL results are displayed
Unlike when a workflow is run by Cromwell, TOIL workflows only provide timing information about the whole job (and not the individual tasks that run).
ISSUE
SEAB-5311

The workflows that I copied live in this repo (this repo is owned by myself, I can move it to one controlled by dockstore before the end of my term),
https://github.com/fhembroff/wes-testing

Also create a ticket for this so we don't forget

I have created two tickets,

SEAB-5341 this is to ensure that https://github.com/fhembroff/wes-testing is not used after the end of my co-op term.
https://github.com/dockstore/dockstore/issues/5435 to create documentation for launching CWL workflows through WES.

One rather annoying thing about the TOIL endpoint is that the agc logs do not give you any information about what caused an error (all you get is EXECUTOR_ERROR), so debugging the TOIL endpoint can be very challenging. This is different to cromwell, which gives you a detailed log indicating what the errors were.

Interestingly, we do know some of the toil developers if this becomes a larger issue

One rather annoying thing about the TOIL endpoint is that the agc logs do not give you any information about what caused an error (all you get is EXECUTOR_ERROR), so debugging the TOIL endpoint can be very challenging. This is different to cromwell, which gives you a detailed log indicating what the errors were.

@adamnovak FYI

You must use the --inline-workflow flag if you want to use any file that isn't the primary descriptor file (Like how make-array.cwl is used in this workflow). Currently, with cromwell the --inline-workflow will cause an error, and instead any file that is referenced that isn't the primary descriptor file must have an s3 address.
The test parameter file must follow the --json flag. There is no need (and if you include it, it will cause errors) for an agcWrapper.json file.
You can reference https files in a workflows that is being run by TOIL in the agc.

FYI, your links are to my-workflows, which we can't see (but I found the public versions).
By You must use the --inline-workflow flag if you want to use any file that isn't the primary descriptor file, do you mean if there is more than one descriptor in the workflow, you need to use that switch? And if there is only one descriptor, presumably it works with or without the --inline-workflow switch? If so, maybe it's easier to just say in the doc, when you get to it :), to always use the inline-workflow switch when running on WES/Toil?

For example, to launch this workflow with TOIL through the dockstore CLI you would use this command,

dockstore workflow wes launch --entry github.com/fhembroff/wes-testing/manyJobs:main --json Dockstore.json

Shouldn't this this have --inline-workflow? I know it's just a PR description, but I'm trying to understand.

FYI, your links are to my-workflows, which we can't see (but I found the public versions).

Oops, my bad. I have fixed the links.

By You must use the --inline-workflow flag if you want to use any file that isn't the primary descriptor file, do you mean if there is more than one descriptor in the workflow, you need to use that switch? And if there is only one descriptor, presumably it works with or without the --inline-workflow switch? If so, maybe it's easier to just say in the doc, when you get to it :), to always use the inline-workflow switch when running on WES/Toil?

Yes, that is correct. Will do!

Shouldn't this this have --inline-workflow? I know it's just a PR description, but I'm trying to understand.

That is correct, I have updated the description.


Removed flatbuffers and code related to loading models in Sceneform format

Closes #172

I've only tested the model-viewer and ar-model-viewer samples so I should also check if the ViewNode still works. As far as I know there aren't any dependencies on the removed code.

If you don't mind confirming the old ViewNode is still working, that would be perfect.

I also made the check on my side with a bunch of Nodes including ViewNode and confirm everything is working fine.
Thanks for making it @grassydragon

Thank you! I didn't find the time to test it yesterday 😅


the core file name is hard to access

The core file is in the name "'!opt!poseidonos!bin!poseidonos.core'"
if the ! can be fixed we dont need to escape the strings to access it
I am using version 10.6

Hello,  Thkrishn
Thanks for commenting your opinions.
Actually, we use core pattern "/etc/pos/core/%E.core" as default, to indicates full path of the core file.
%E option automatically uses "!" string to indicates the path of its executable file.
However if you want to change the core file name in your code, please fix as below.
And we also plan to configure this file name and core's detailed options through config file in the next release.(/etc/pos/pos.conf).
diff --git a/src/main/poseidonos.cpp b/src/main/poseidonos.cpp
index 8e577e6d..c2f80846 100644
--- a/src/main/poseidonos.cpp
+++ b/src/main/poseidonos.cpp
@@ -196,7 +196,7 @@ Poseidonos::_InitDebugInfo(void)
POS_TRACE_DEBUG(POS_EVENT_ID::DEBUG_CORE_DUMP_SETTING_FAILED, "Core directory will not be created");
return;
}

ret = system("echo /etc/pos/core/%E.core > /proc/sys/kernel/core_pattern");

ret = system("echo AsYouWant > /proc/sys/kernel/core_pattern");
if (ret != 0)
{
POS_TRACE_DEBUG(POS_EVENT_ID::DEBUG_CORE_DUMP_SETTING_FAILED, "Core pattern is not set properly");


Add CI Setup

Issue #, if available:
Description of changes:
Adds travis.yml
Adds ros2 monitoring messages to .rosinstall until bloom released
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

There aren't any tests for this package, but added .codecov.yml for a time when those exist.


Tooltip on vertical bar shapes: the main tip view is out of place.

The spout of "supp" tip view should be pointing to the middle point.
Alternatively "supp" tip view in this example could have no spout at all and be just floating nearby.

See error_bars.ipynb

Fixed


Config value to display or not the URL on hover at bottom of screen

This config displayUrlOnHover: false on buildConfig.js allows to show/hide the URLs hover.

@dmeza Mattermost server v4.4 hides internal URL links not supplied by the user.
Are there additional links you want to hide on hover, or just ever link alltogether?

@jasonblais since we were testing with 4.3 I still saw the issue. Testing with 4.4 this if fixed. you can close this PR.

Sounds good, thanks @dmeza for the follow-up


Single shadow sample has incorrect texture name

https://github.com/Microsoft/DirectX-Graphics-Samples/pull/441

Thanks for your pull request.  I approved it.


remove the tabler theme as dependency

WHY
BEFORE - What was wrong? What was happening before this PR?
We couldn't run the unit tests, because CRUD could not be resolved to an installable set of dependencies. That was due to backpack/crud requiring tabler-theme, which in turn required backpack/crud.
AFTER - What is happening after this PR?
tabler-theme is no longer a dependency of backpack/crud, but something that gets installed when you run php artisan backpack:install. So it becomes a requirement of the main Laravel app, not CRUD. That way:

if a developer doesn't want or like it... they can remove it entirely;
if a theme developer wants to add it as a dependency, to fall back to its views, they can do that, without risking too many version conflicts;

HOW
How did you achieve that, in technical terms?
Change in composer.json and Installation command.
Is it a breaking change?
Yes. People who install v6, even the alpha one, will now also need to run php artisan backpack:install

Is it a breaking change?
Yes. People who install v6, even the alpha one, will now also need to run php artisan backpack:install

Actually since the Demo has all themes as root dependencies, this isn't actually a breaking change for team members who use the Demo. They'll still get theme-tabler. But it is a breaking change for people who have installed it on fresh projects. Which I expect is very few people.


Get more info

I am looking at getting some more info for when I pull stock requests: specifically intraday Open, High, and Low price
In parsers.py I was able to find where you have some of the info: Line 110

xpath_map = {

    'name': '//h3[@class="companyname"]/text()',
    'symbol': '//table[contains(@class,"table3")]/tbody/tr[1]/td[1]/h3[contains(@class,"pill")]/text()',
    'exchange': '//table[contains(@class,"table3")]//div[@class="marketname"]/text()',
    'last': '//table[@id="Table2"]/tbody/tr[1]/th[contains(text(),"Last")]/following-sibling::td/text()',
    'change': '//table[@id="Table2"]/tbody/tr[2]/th[contains(text(),"Change")]/following-sibling::td/text()',
    'change_percent': '//table[@id="Table2"]/tbody/tr[3]/th[contains(text(),"% Change")]/following-sibling::td/text()',
    'volume': '//table[@id="Table2"]/tbody/tr[4]/th[contains(text(),"Volume")]/following-sibling::td/text()'
}

I was just wondering where you pull this information from so I could add my own hopefully. I also see that I will need to update api_models.py:

class StockQuote(object):
@coerce_method_params
def init(
self: object,
symbol: str,
name: str,
exchange: str,
last: Decimal,
change: Decimal,
change_percent: Decimal,
volume: int

):
    self.symbol = symbol
    self.name = name
    self.last = last
    self.exchange = exchange
    self.change = change
    self.change_percent = change_percent
    self.volume = volume

Correct me if I'm wrong? Would there be anything else I would have to update?

So yeah you're on the right track.  The stock quote data is pulled from the response of a POST request to /ajax/quotebox.aspx (see constants.py)
Here's an example response when I lookup GOOG:

GOOG

ALPHABET INC. CLASS C

(NASDAQ)

294.62

                Last
                1,827.0100

                Change
                294.62

                % Change
                19.23

                Volume
                807,349

                Day's High
                1,835.0300

                Day's Low
                1,789.4700

The data returned is limited - I'm not seeing anything for Open but it looks like you could easily scrape the Day's High and Day's Low by adding these xpath expressions to xpath_map in parsers.py:
'days_high': '//table[@id="Table2"]/tbody/tr[5]/th[contains(text(),"Day\'s High")]/following-sibling::td/text()',
'days_low': '//table[@id="Table2"]/tbody/tr[6]/th[contains(text(),"Day\'s High")]/following-sibling::td/text()'

And yes, you're correct, you'd also have to modify the StockQuote class in api_models.py to handle the additional information:
class StockQuote(object):
    @coerce_method_params
    def __init__(
        self: object,
        symbol: str,
        name: str,
        exchange: str,
        last: Decimal,
        change: Decimal,
        change_percent: Decimal,
        volume: int,
        days_high: Decimal,
        days_low: Decimal

    ):
        self.symbol = symbol
        self.name = name
        self.last = last
        self.exchange = exchange
        self.change = change
        self.change_percent = change_percent
        self.volume = volume
        self.days_high = days_high
        self.days_low = days_low

These changes are actually pretty easy so I'm going to test and commit them to master.
As for getting the open price, I'll look into seeing if I can maybe scrape an additional resource on Investopedia's end that offers a bit more information than what's available when making a request to /ajax/quotebox.aspx

actually, now that I think about it, the open price could be calculated by subtracting change from last.
self.open = self.last - self.change
I'll add that into the commit as well.

All looks great! Thanks! I wasn't sure what was available and the syntax

Merged in the changes.  Here's how to get the open, high, and low:
quote = client.get_stock_quote('GOOG')
print(quote.days_high)
print(quote.days_low)
print(quote.open)