clang: error: the clang compiler does not support '-march=native'    on poetry install

(sgk-plugin-py3.10) gkibilov@Georges-MacBook-Pro sgkplugin % poetry install
Installing dependencies from lock file
Package operations: 20 installs, 0 updates, 0 removals
• Installing hnswlib (0.7.0): Failed
ChefBuildError
Backend subprocess exited when trying to invoke build_wheel
running bdist_wheel
running build
running build_ext
creating var
creating var/folders
creating var/folders/1p
creating var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn
creating var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T
clang -Wno-unused-result -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -arch arm64 -arch x86_64 -g -I/private/var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmpf0uy_ebb/.venv/include -I/Library/Frameworks/Python.framework/Versions/3.10/include/python3.10 -c /var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmp_ylpagz7.cpp -o var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmp_ylpagz7.o -std=c++14
clang -Wno-unused-result -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -arch arm64 -arch x86_64 -g -I/private/var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmpf0uy_ebb/.venv/include -I/Library/Frameworks/Python.framework/Versions/3.10/include/python3.10 -c /var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmp0m2c3mq6.cpp -o var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmp0m2c3mq6.o -fvisibility=hidden
building 'hnswlib' extension
creating build
creating build/temp.macosx-10.9-universal2-cpython-310
creating build/temp.macosx-10.9-universal2-cpython-310/python_bindings
clang -Wno-unused-result -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -arch arm64 -arch x86_64 -g -I/private/var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmpf0uy_ebb/.venv/lib/python3.10/site-packages/pybind11/include -I/private/var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmpf0uy_ebb/.venv/lib/python3.10/site-packages/numpy/core/include -I./hnswlib/ -I/private/var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmpf0uy_ebb/.venv/include -I/Library/Frameworks/Python.framework/Versions/3.10/include/python3.10 -c ./python_bindings/bindings.cpp -o build/temp.macosx-10.9-universal2-cpython-310/./python_bindings/bindings.o -O3 -march=native -stdlib=libc++ -mmacosx-version-min=10.7 -DVERSION_INFO="0.7.0" -std=c++14 -fvisibility=hidden
clang: error: the clang compiler does not support '-march=native'
error: command '/usr/bin/clang' failed with exit code 1
at ~/Library/Python/3.9/lib/python/site-packages/poetry/installation/chef.py:147 in _prepare
143│
144│                 error = ChefBuildError("\n\n".join(message_parts))
145│
146│             if error is not None:
→ 147│                 raise error from None
148│
149│             return path
150│
151│     def _prepare_sdist(self, archive: Path, destination: Path | None = None) -> Path:
Note: This error originates from the build backend, and is likely not a problem with poetry but with hnswlib (0.7.0) not supporting PEP 517 builds. You can verify this by running 'pip wheel --use-pep517 "hnswlib (==0.7.0)"'.
(sgk-plugin-py3.10) gkibilov@Georges-MacBook-Pro sgkplugin % pip wheel --use-pep517 "hnswlib (==0.7.0)"
Collecting hnswlib==0.7.0
Downloading hnswlib-0.7.0.tar.gz (33 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Collecting numpy (from hnswlib==0.7.0)
Using cached numpy-1.25.0-cp310-cp310-macosx_10_9_x86_64.whl (20.1 MB)
Saved ./numpy-1.25.0-cp310-cp310-macosx_10_9_x86_64.whl
Building wheels for collected packages: hnswlib
Building wheel for hnswlib (pyproject.toml) ... error
error: subprocess-exited-with-error
× Building wheel for hnswlib (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [17 lines of output]
running bdist_wheel
running build
running build_ext
creating var
creating var/folders
creating var/folders/1p
creating var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn
creating var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T
clang -Wno-unused-result -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -arch arm64 -arch x86_64 -g -I/Users/gkibilov/Library/Caches/pypoetry/virtualenvs/sgk-plugin-9OQ2lEda-py3.10/include -I/Library/Frameworks/Python.framework/Versions/3.10/include/python3.10 -c /var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmpipqtqj7q.cpp -o var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmpipqtqj7q.o -std=c++14
clang -Wno-unused-result -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -arch arm64 -arch x86_64 -g -I/Users/gkibilov/Library/Caches/pypoetry/virtualenvs/sgk-plugin-9OQ2lEda-py3.10/include -I/Library/Frameworks/Python.framework/Versions/3.10/include/python3.10 -c /var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmp5qfffxak.cpp -o var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/tmp5qfffxak.o -fvisibility=hidden
building 'hnswlib' extension
creating build
creating build/temp.macosx-10.9-universal2-cpython-310
creating build/temp.macosx-10.9-universal2-cpython-310/python_bindings
clang -Wno-unused-result -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -arch arm64 -arch x86_64 -g -I/private/var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/pip-build-env-ka7xrqgo/overlay/lib/python3.10/site-packages/pybind11/include -I/private/var/folders/1p/wppj4l8x7j356pk05d_0j7f80000gn/T/pip-build-env-ka7xrqgo/overlay/lib/python3.10/site-packages/numpy/core/include -I./hnswlib/ -I/Users/gkibilov/Library/Caches/pypoetry/virtualenvs/sgk-plugin-9OQ2lEda-py3.10/include -I/Library/Frameworks/Python.framework/Versions/3.10/include/python3.10 -c ./python_bindings/bindings.cpp -o build/temp.macosx-10.9-universal2-cpython-310/./python_bindings/bindings.o -O3 -march=native -stdlib=libc++ -mmacosx-version-min=10.7 -DVERSION_INFO="0.7.0" -std=c++14 -fvisibility=hidden
clang: error: the clang compiler does not support '-march=native'
error: command '/usr/bin/clang' failed with exit code 1
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for hnswlib
Failed to build hnswlib
ERROR: Failed to build one or more wheels
(sgk-plugin-py3.10) gkibilov@Georges-MacBook-Pro sgkplugin %

(sgk-plugin-py3.10) gkibilov@Georges-MacBook-Pro sgkplugin % clang --version
Apple clang version 14.0.0 (clang-14<IP_ADDRESS>)
Target: x86_64-apple-darwin21.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin
(sgk-plugin-py3.10) gkibilov@Georges-MacBook-Pro sgkplugin %

Were you able to resolve the issue? I am facing the same

Sorry, did not get a chance to.

export HNSWLIB_NO_NATIVE=1 worked for me FYI

Thanks, I'll give it a try!


[feaLib] Put glyph alternates in the same order as makeotf

feaLib currently sorts GSUB glyph alternates by glyph order. We should figure out what exactly makeotf does, and emulate that behavior.

It should be just the same order as in the feature file, isn’t it?

I think Sascha meant for the auto generated aalt feature.
I think glyph order is good enough.

Duplicate of https://github.com/fonttools/fonttools/issues/2937


Make Frame a property and not a field within BcmCanSingleMessage and BcmCanFdSingleMessage classes

This was probably due to a copy & paste error when adapting the BcmCanMessage class. The Frames field in BcmCanMessage has to be a field so that the MarshalAsAttribute can be applied, but this is not the case for Frame within the BcmCanSingleMessage class.

Same is true for BcmCanFdSingleMessage class and the Frame field that should be a property.


[#3948] Add to function which can add customized HandshakeInterceptor

Codecov Report

Merging #3949 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3949      +/-   ##
==========================================
- Coverage    40.7%   40.69%   -0.02%
==========================================
  Files        2203     2203
  Lines       71477    71479       +2
  Branches     9846     9847       +1
==========================================
- Hits        29095    29086       -9
- Misses      39627    39636       +9
- Partials     2755     2757       +2

Impacted Files
Coverage Δ

...navercorp/pinpoint/web/config/WebSocketConfig.java
0% <0%> (ø)
:arrow_up:

...rp/pinpoint/rpc/packet/ControlHandshakePacket.java
51.51% <0%> (-24.25%)
:arrow_down:

.../profiler/sender/StandbySpanStreamDataStorage.java
72.88% <0%> (-3.39%)
:arrow_down:

...m/navercorp/pinpoint/test/OrderedSpanRecorder.java
70.19% <0%> (-1.93%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
79.51% <0%> (-1.74%)
:arrow_down:

...java/com/navercorp/pinpoint/rpc/DefaultFuture.java
74.16% <0%> (+1.66%)
:arrow_up:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
83.58% <0%> (+2.98%)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ClientClosePacket.java
66.66% <0%> (+11.11%)
:arrow_up:

...npoint/rpc/server/SimpleServerMessageListener.java
100% <0%> (+20%)
: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 b32bff1...9f8fda1. Read the comment docs.


Which tool do you used for generating the DDS image?

When I try to import my own DDS image generated from cmft, Maya crashed immediately. It seems that Maya does not support all formats of DDS. Would you please tell me that which DDS generation tool you are using?

What version of Maya are you using?

What version of Maya are you using?

I use Maya 2018 in macOS

Ah that version contains a bug in its DDS loader.
What worked for me, was to use texconv to load and save the DDS

Thanks. I will try

Did that work for you?

No. The image become all white if I convert it from 32bit float to 32bit uint or 8bit uint. Even I keep the format as 32bit float, the hue is changed.


Fix nightly compiler complaints about size_of

Because the prelude now includes size_of, the nightly compiler really doesn't like it when you fully qualify size_of.

I don't think the test failures are related BTW. It says Error: Operation not permitted which shouldn't have anything to do with this.

Yeah, I just checked by reverting the commit so the PR is blank and it's still broken. Very strange.

I believe this is something that changed with github actions but I don't have the energy to deal with it.

Yeah fair enough. They might have blocked io_uring syscalls? Anyway, I'm fine leaving this here until someone finds a fix (don't have time to look into it either).


add ability to d3 vis to live-update min-intersection requirement

if include_min_intersection_selector=True is passed to
mapper.visualize,
then a number input is included at the top of the d3 vis which permits
changing the min_intersection req for edges (links) to be drawn.
Currently does not get saved in the downloadable config file.

No update to doc examples yet, because I'm afraid of the docs.
Also, demo gif:

Closes #211


for test

Type of Change
feature or bug fix or documentation or validation or others
API changed or not
Description
detail description
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

/analyze

/analyze


fix(org-mode): Remove deprecated code snippet

Org now uses begin_src LANGUAGE, which is also provided in this package.

I renamed the code snippet to begin now.
I was not aware of these special blocks, thanks for bringing them to my attention.
The only block I don't think is necessary is the begin_latex block, see my comment at https://github.com/doomemacs/snippets/pull/86


Run projects on Apple silicon (M1)

I can't run any project from this repo on my mac mini M1. Everything works normal on my Intel machine.
I'm getting error like this:
Gradle sync failed: Sync failed: reason unknown
Please,  help me to resolve this issue.
My Android Gradle Plugin version - 4.1.2
My gradle version - 6.5
BTW if I create new project on my Intel machine and open it in my M1 - everything works

Issue resolved by installing Android SDK Command-line Tools


Fix missing exports condition

There is nothing special about this, just needs update for sveltekit 2.
Can follow instructions in https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/faq.md#missing-exports-condition
It comes out in terminal after 'yarn dev' as:
10:14:29 PM [vite-plugin-svelte] WARNING: The following packages have a svelte field in their package.json but no exports condition for svelte.

<EMAIL_ADDRESS>Please see https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/faq.md#missing-exports-condition for details.

From package.json:
"@sveltejs/vite-plugin-svelte": "^3.0.0",

Then please submit a PR?


feat: Twitter -> X

What does this PR do?

This PR changes all references to Twitter to X.

Type of change

Chore (refactoring code, technical debt, workflow improvements)

How should this be tested?

[ ] Test A
[ ] Test B

Mandatory Tasks

[x] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Checklist

Looks good but the buttons look odd. Kinda empty

I personally don't think they look that bad, but you could always upload the actual X logo and use that.

Okay let it be. Will merge it


Documentation Issue

The step 6 in the Quick start guide should be done after completing the step 7 and 8. Otherwise it throws an error.
Also the documentation does not specifically provide the directory the command should be ran in for ex: after the step 9 we will have to move to a different directory for step 10 and then follow the C++ commands step, so please take care of these issue,

@acomodi - Who is the correct person to fix this?

I can fix this, it is quite straightforward. Long term we might use tuttest to verify the readme commands are working properly

I saw that the wording was fixed to show that for Step 10 you need to be in a different directory, but I didn't see any changes to the ordering of the quick start guide steps, so I linked a pull request that puts step 6 after steps 7 and 8


Implementation of DP-InstaHide and Data Augmentation Defenses

Is your feature request related to a problem? Please describe.
Mixup, CutOut, CutMix, and Maxup are relatively simple data augmentation defenses that have been shown to provide robustness against poisoning attacks. ART currently does not have any data augmentation defenses so this implementation will provide more features and open the doors for additional data augmentation defenses. DP-InstaHide is a relatively simple training method that utilizes data augmentation defenses and has privacy guarantees. This implementation will be a good utilization of the data augmentation defenses and provide another useful tool to ART.
Describe the solution you'd like
Implementation of data augmentation defenses would involve creation of the data_augmentation submodule under art.defenses.preprocessor. Here, an abstract class for data augmentation would be created that follows the same structure as the preprocessor defenses. This abstract class would then be extended to create defenses for Mixup, CutOut, CutMix, and Maxup for the four main data augmentation defenses used with DP-InstaHide.
Implementation of DP-InstaHide would involve creating a new trainer under art.defenses.trainer which would run similarly to the adversarial trainer. This would take parameters to specify the type of data augmentation defense along with the hyperparameters.
Describe alternatives you've considered
Originally, the idea was to have data_augmentation be its own submodule under art.defenses rather than a type of preprocessor. However, after some thought and discussion, the data augmentation defenses should be a submodule under art.defenses.preprocessor.
Additional context
It might be worth splitting this into two separate issues for organization purposes. One for the implementation of data augmentation defenses and another for the implementation of DP-InstaHide.

Hi @f4str Thank you very much for proposing this new feature! I think it will of interest for many users of ART!


Initialised new multiqc version (1.14.0)

Description
New cwltool for multiqc 1.14.0
TODO

[x] Wait for #202
[x] Wait for #185

@alexiswl Do we have a before/after MultiQC report for one or more of the validation samples? Given the whole merge history it would be good to confirm that the key metrics are still present (and match between versions).

Given the whole merge history it would be good to confirm that the key metrics are still present (and match between versions).

Yep we can certainly run this on some validation samples and have added that to the to-do list in https://github.com/umccr/cwl-ica/pull/232

@alexiswl - are you happy for this to be reviewed?
Did you test the CWL definition (not the report) to confirm this new multiQC tag works?

Did you test the CWL definition (not the report) to confirm this new multiQC tag works?

Yes have run the cwltool locally and works fine.
Only discrepancy I see between the versions is that if a multiqc 'General Stat' value is NA, it will not appear in the General Stats table. This is often the case for the "Contam'n" stat and can be traced back to this commit


How to change env in 'backpack dev' or 'backpack build'

I want to change NODE_ENV in 'backpack build', this is my code, but it is not work.
cross-env NODE_ENV=test backpack build

I have the same problem, although setting up port with PORT=5003 is working correctly

Duplicate of #67


Statically initialized StrawberryIntegration leads to runtime exception: There is no current event loop in thread

How do you use Sentry?
Self-hosted/on-premise
Version
2.5.1
Steps to Reproduce
I use a mixed app with starlette and django where the starlette router resolves the route into a starlette websocket (async) or a django app (sync). The problem is, that the strawberry extensions guess the async scope here. So I get an async middleware in my sync stack if a query is executed. As far I can see from the codebase I can't opt out this behaviour, because it is added via the default integrations here.
I guess there is a general design problem with the strawberry integration. E.g. the DjangoIntegration does not require to specify the async_execution like the StrawberryIntegration does here.  The DjangoIntegration request is evaluated while runtime.
FYI & Ping @patrick91
Expected Result
depending on the request the sync or async resolver is called. Currently, this is statically injected on integration creation (or default integration injection)
Actual Result
An async middleware is added to a wsgi request

Hey @Speedy1991, you can override the behavior by adding the integration explicitly to your sentry_sdk.init with async_execution=False, see here: https://docs.sentry.io/platforms/python/integrations/strawberry/#synchronous-vs-asynchronous-execution Does that solve the issue for you?

Thanks for the follow up. Do I understand correctly that you want to use the StrawberryIntegration both in the Django app (sync) and in the Starlette app (async)? That is not possible with the current setup without initializing Sentry in both apps. You could either force the use of the sync extension everywhere, or disable the StrawberryIntegration altogether (via auto_enabling_integrations=False, see docs) and use the extension directly.
IIRC the reason why the integration needs to know if you want the sync or async version at startup is that it works via injecting an extension which is done at schema creation. So this would need a bigger revamp to support guessing the context during runtime.


Pipeline parameters are missing

Coming from: https://github.com/sharpliner/sharpliner/discussions/207
Problem
It should be possible to define parameters to pipelines as described here:
https://docs.microsoft.com/en-us/azure/devops/pipelines/process/runtime-parameters?view=azure-devops&tabs=script
It seems that the possible types are the same as with template parameters so this should be fairly easy to implement.
Goals

Create the Parameters field (no conditionals) on Pipeline
Make it possible to use shorthands from TemplateDefinition insde PipelineDefinition
Make it possible to reference parameters like in templates
Extend documentation

i think it's not that easy.
the problem is, that a a pipeline parameter has an additional property (i.e.  "displayName").
this is the real advantage of parameters on pipeline level.  they will be shown/editable/selectable in the run ui of portal.
it doesnt fit in the model so far.
we can copy the whole template parameter stuff to have a pipeline parameter with one additional property.
buts this is ugly.

I see. Is there a way we could somehow use generics and have a TemplateParameter and PipelineParameter inherit and add this?
The methods for creating these could then probably carry this type information and return whatever is needed?
It would probably be fine having 2 sets of APIs too and in the pipeline version have the displayName parameter as an argument because we frequently surface this one in the other APIs.

@brunskutscher I checked the code and I think I see a way how to implement this nicely.

Make ctors of TemplateParameter classes internal
Add parameter to the ctor for displayName, add the same property
Keep the methods for parameter creation in TemplateDefinition without display name
Copy the methods for parameter creation from TemplateDefinition into PipelineDefinition and add the displayName parameter so that you force people to set it in pipeline

This way, we don't need to copy much of the code. Only regression is that you won't be able to new the parameters but I don't think people do that anyway.
Are you still interested in this?

So it was actually super quick to implement and the change was quite small so I added this as it is a missing feature that might be desired by customers:
https://github.com/sharpliner/sharpliner/pull/210
However, I am happy to help find something else if you're interested in contributing!

Ah, thanks for the feedback!
Yeah, I agree renaming makes sense and making display name optional too. I should have left the PR up for some time to give you time to review. Sorry about that.
Can you open a PR for this? Or should I make the change?

no problem ...
coincidental i worked on this today.
but i have limited time ... so better you make the change and i will review it as soon as possible.

Fixed in #210 and #211


Exception on logout

To reproduce: go to individual history panel, then log out.
Exception:
HTTP Status 500 - javax.servlet.ServletException: org.springframework.web.util.NestedServletException: Request processing failed; nested exception is org.springframework.webflow.execution.FlowExecutionException: Exception thrown in state 'home' of flow 'individualHistory'

Should be fixed in latest commit!


fix: shutdownAsync flushes the whole queue

Problem
If the queue is bigger than flushAt after all pendingPromises are resolved, some events can still be lost since flush caps to flushAt anyway.
Discussed here with @benjackwhite https://github.com/PostHog/posthog-js-lite/pull/177#discussion_r1489359776
Changes
Instead of calling shutdownAsync twice, we call until the queue is empty.
Release info Sub-libraries affected
Bump level

[ ] Major
[ ] Minor
[X] Patch

Libraries affected

[X] All of them
[ ] posthog-web
[ ] posthog-node
[ ] posthog-react-native

Changelog notes

Added support for X

@benjackwhite there's just one test failing but its unrelated to my changes I guess.

FAIL  posthog-node/test/feature-flags.spec.ts
● match properties › with relative date operators: is_date_before, 1h, 2022-05-01 00:00:00
expect(received).toBe(expected) // Object.is equality

Expected: false
Received: true

  2040 |   ])('with relative date operators: %s, %s, %s', (operator, value, date, expectation) => {
  2041 |     jest.setSystemTime(new Date('2022-05-01'))
> 2042 |     expect(matchProperty({ key: 'key', value, operator }, { key: date })).toBe(expectation)
       |                                                                           ^
  2043 |
  2044 |     return
  2045 |   })

  at posthog-node/test/feature-flags.spec.ts:2042:75

Could it be because of https://github.com/PostHog/posthog-js-lite/pull/189#discussion_r1502437416 ?

oh indeed, much better, thank you!


Register endpoints into the Bioregistry

For smiles and inchikey prefixes to make this service more FAIR - particularly finable and accessible.
See this issue template for doing so
https://github.com/biopragmatics/bioregistry/issues/new?labels=Provider%2CNew&projects=&template=new-provider.yml&title=Add+provider+for+[X]

https://github.com/biopragmatics/bioregistry/issues/931

Can we make a list of the endpoints that would be worth it? Like descriptors, making an image, anything else? And then we put them for both smiles, inchi, and inchikey (when possible)

@cthoyt Sure, we were wondering the same. Will raise more requests soon :)

You can use a more generic issue template and group it all together, no need to do individual issues for each, since it's clear what's going on here

https://github.com/biopragmatics/bioregistry/pull/932

Are there any endpoints that work with InChI or InChI keys? or are all endpoints working only for SMILES?

Currently, it's either SMILES or MOL block. We will extend support for InChI or InChI keys, and then submit a respective provider request later.

Great, then I would call this issue done wrt the review :)


Masks that end with a string do not automatically fill the last character

Using this mask for example:
const MASK = [/\d/, /\d/, ".", /\d/, "%"]
with an input of '221' outputs '22.1' NOT '22.1%'
The last character is only added after you input an additional character.

@mmfire118 it's not possible. See if this lib fits your case, it has suffix prop: https://github.com/CaioQuirinoMedeiros/react-native-currency-input


Add the  to script name as in the tab

Add the db name to script name as in the tab
See image

#6806

@rpalloni For now you can enable an option to create folders with connection name in script folder. And all new scripts for a connection will be saved in it's personal folder.


Display error line

Proposal
It is difficult to identify errors when the scenario becomes large and multiple includes are made.
I want to be able to display the number of lines as well as the file that had the error.
I would like to display the desc defined in step as well.
We would like to display the number of lines in the normal error output if we can achieve the following
https://github.com/k1LoW/runn/issues/121

note
https://github.com/k1LoW/runn/issues/146#issuecomment-1246529165
How to display an error if the Runbook is a standard input? We may need to consider this.

Mounting method study

When parsing yaml and structuring each sentence, keep the corresponding line number in the structure.
After the error occurs, re-analyze the original source from the characteristics of the section in question to identify the relevant line

Which implementation is more efficient? And can it be more accurate in identifying the relevant rows with less impact on future enhancements?
If you have any good ideas, we'd like to discuss them in more depth.

If we are doing normal Unmarshal, it seems to be difficult to get the YAML rows.
Also, errors may not correspond to YAML lines.
Perhaps it would be better to split this problem into two parts.

Linter to check runbook format ( e.g. runn lint )

Build completely separate from the current Unmarshal structure.

Display more detailed reasons for errors that occur in steps

At least be able to see the runbook file and step key where the error is occurring.

First, implement linter so that the runn book structure can be analyzed.
At this point, it will be possible to determine which lines of the original yaml correspond to the syntax, including the steps.
After that, we will detail the errors
I thought the above process seemed like a good way to fine-tune the problem

@k2tzumi is this issue still open? I would like to work on it as part of Hacktoberfest 2022

Yes issues remain.
I have no concrete results, just a study of implementation methods on my part.

@k2tzumi can you tell me how to start or point me to a getting started guide?

I don't know if this will be helpful, but I found the following page
https://dev.to/nitinkumar30/what-how-to-do-in-hacktoberfest-2022-p36
By the way, I have never attended Hacktoberfest.
I don't understand because the conversation was so abrupt, but what would be the purpose of your participation in this Hacktoberfest?

@amoghrajesh
I was researching Hacktoberfest and realized I was mistaken.
Sorry for the odd reply.
Is the purpose of the following OK?

register runn in the maintenance repository of Hacktoberfest

As far as I can confirm, you need to have the authority to own the repository, so you cannot make the decision on your own.
Please give us a few minutes to check with owner.
Contributions are always welcome, not just for Hacktoberfest participation, so please keep in touch!

Hi @amoghrajesh !
Welcome!

can you tell me how to start or point me to a getting started guide?

Unfortunately, we are not able to provide a starting guide for this repository, as it is not fully developed.
You will need to read the code and find bugs and make suggestions, like https://github.com/k1LoW/runn/pull/71.
Please refer to past merged Issues and PRs.
Or, I am not particularly good at English, so I would be very happy to get suggestions for better updates to the README.
Happy hacktoberfest 🐙

is this issue still open?

YES! we want!!

% go run cmd/runn/main.go run testdata/book/runn_1_fail.yml
.

1) testdata/book/runn_1_fail.yml b6d90c331b04ab198ca95b13c5f656fd2522e53b
  Failure/Error: test failed on "Test for RunN (1)".steps[0]: condition is not true

  Condition:
    false
    │
    └── false => false

  Failure step (testdata/book/runn_1_fail.yml):
  3   -
  4     test: false

1 scenario, 0 skipped, 1 failure

Is it almost ready for implementation?
Would support for GithubAcrtions output format be better added in a separate Issue?


ssh-ident ignores ssh-askpass (or SSH_ASKPASS and DISPLAY vars)

I'm using ssh-add -c flag to confirm every use of my ssh keys. To get a GUI yes/no confirmation, I've installed a little script ssh-askpass that creates a simple XQuartz  window.
Now, on OSX all that is required is to place this script in the right place in /usr/X11R6/bin/ssh-askpass where ssh-agent is able to find it automatically.
However, it's also possible to set $SSH_ASKPASS environment variable pointing to the script ($DISPLAY also has to be set). This second option is something that I'm using on Windows with MSYS2 (where instead of ssh-askpass, where I'm using the default git script (git-gui--askpass) to spawn a GUI window.
It works wonderfully in both environments, however it all breaks when I use ssh-ident instead of ssh. As far as I understand it, at the moment ssh-agent is spawned, it must have access to either the default path or to the path set by  $SSH_ASKPASS (and also $DISPLAY). If it does, all subsequent calls will respect the -c flag, spawn a new confirmation window, wait for it and pass the ssh key (or not) depending on the button you click in that window.
Now, since your script spawns agents in subprocess calls, they don't have access to these default paths or environment variables and ssh-agent doesn't know it can spawn a GUI window, so it just ignores the -c flag and never passes an ssh key.
Would you please adjust your subprocess routines so that ssh-agent would have access to those two variables (and either allow user to set them separately or read them from the default environment beforehand, and if unset try to read /usr/X11R6/bin/ssh-askpass which seems to be the default path on OSX)?
OSX 10.11.3 or Win10 with MSYS2

Thanks for the report! I'm surprised I did not / nobody noticed this before - and sorry for latency.
I do not use SSH_ASKPASS or similar - so did not test it. But agent forwarding and a simple script run via BINARY_SSH shows the environment being propagated correctly now, locale and X forwarding included.
If you could confirm this now works correctly, that'd be great. In the mean time, the bug has been closed.

Unfortunately, it doesn't work. When I have "personal": "-c", option in my config files, ssh-ident silently ignores it (my guess is for the same reason — the vars aren't read) and just asks me to enter the password to the website directly even though the key is loaded properly All keys already loaded
When I launch an agent manually and manually add the same key with the -c option (with export DISPLAY=localhost:0.0 and export SSH_ASKPASS=/usr/lib/git-core/git-gui--askpass defined in my bash profile) it all works — the agent opens up the gui to ask for a confirmation.

Why does ssh-ident clear the environment in the first place?


Design a Survey for Gathering Information on Open Source Organizations and Projects

Description:
Design a comprehensive survey aimed at collecting information that will help users understand academic OSS. The goal is to gather insights into what specific data people want to know when they look at an open-source project or organization, including roles, contributions, project impact, collaboration opportunities, and key contacts.
Key Objectives for the Survey:

Identify the types of information that users find most valuable (e.g., personnel, roles, funding sources, technical platforms, community engagement).
Understand how users would prefer to access and organize this information (e.g., searchable directories, visual maps, project timelines).
Gather insights into which roles or departments in an organization users are most interested in.
Learn what collaboration or engagement opportunities users want to explore with open-source organizations and projects.
Identify additional features or challenges users face when navigating the open-source ecosystem.

Expected Outcomes:

A data-driven survey that focuses on understanding the needs and preferences of the community regarding open source projects and organizations.
Clarity on what information is most crucial for building a resource that highlights "who is doing what" within an organization or project.
Improved design and content focus for the organization section, which will be extended to other sections for further improvements.

Action Items:

Create a set of questions based on the objectives above.
Review and finalize the survey with feedback from @RichardLitt
Prepare to distribute the survey to relevant open-source communities and academic partners.

Design a comprehensive survey aimed at collecting information that will help users understand the landscape of open source organizations and projects. The goal is to gather insights into what specific data people want to know when they look at an open-source project or organization, including roles, contributions, project impact, collaboration opportunities, and key contacts.

A key difference: this isn't the landscape of open source orgs and projects, but explicitly academic OSS.
Also, may be worth looking at this: https://investinopen.org/blog/nnouncing-150-000-award-to-ioi-to-advance-understanding-of-the-research-software-ecosystem/. I think their project is slightly different - it's more project focused.

Design a comprehensive survey aimed at collecting information that will help users understand the landscape of open source organizations and projects. The goal is to gather insights into what specific data people want to know when they look at an open-source project or organization, including roles, contributions, project impact, collaboration opportunities, and key contacts.

A key difference: this isn't the landscape of open source orgs and projects, but explicitly academic OSS.
Also, may be worth looking at this: https://investinopen.org/blog/nnouncing-150-000-award-to-ioi-to-advance-understanding-of-the-research-software-ecosystem/. I think their project is slightly different - it's more project focused.

Thank you, I’ve made adjustments to the description. I reviewed the IOI project, and you're right; the two projects are slightly different. Our project adopts a broader approach by examining various entities in academia that work with open-source software—such as organizations, universities, service providers, and funders—offering a more comprehensive map of the open-source ecosystem in academia. In contrast, IOI has a narrower focus, concentrating on research software development and the infrastructure that supports it. Their work emphasizes identifying existing connections, gaps, and vulnerabilities within the research software ecosystem, aiming to improve its sustainability and resilience. Our data collection methods also vary, but we could certainly draw inspiration from IOI’s approach, particularly in identifying key gaps and opportunities. This could help us align better with our goal of creating a resource for navigating the academic open-source ecosystem.

If we want, we can also connect with them directly. I've had a few people from IOI over the years on the SustainOSS podcast, and know a few of the excellent people on the IOI team.

If we want, we can also connect with them directly. I've had a few people from IOI over the years on the SustainOSS podcast, and know a few of the excellent people on the IOI team.

That sounds like a great idea! I think connecting with them would be valuable for our work.

Cool. Let's wait until we have a clearer goal and plan for how this would look. :)

Cool. Let's wait until we have a clearer goal and plan for how this would look. :)

Okay.

Hi @RichardLitt, I have added you as a collaborator on the first draft of the survey. Please review it and drop your comments here. Are there any questions you think should be included, removed, or modified?
Thank you.

Thanks @reginankenchor. Where did you add me?

Ok, looked over it, and made some edits:

I trimmed down the intro - it had a lot of content in it, and I think it could be a bit easier to parse;
I changed "open-source" to "open source" across the document. https://opensource.org/blog/is-open-source-ever-hyphenated
I've added a few different options to some of the questions.

This survey is a bit long - and I'm not sure how useful it would be. I'm not sure how we would use a data. Do you have a plan for how to interpret the responses and to use them to add more content to the map?

Ok, looked over it, and made some edits:
* I trimmed down the intro - it had a lot of content in it, and I think it could be a bit easier to parse;

* I changed "open-source" to "open source" across the document. https://opensource.org/blog/is-open-source-ever-hyphenated

* I've added a few different options to some of the questions.

This survey is a bit long - and I'm not sure how useful it would be. I'm not sure how we would use a data. Do you have a plan for how to interpret the responses and to use them to add more content to the map?

Thank you, @RichardLitt . Yes, I’ll review the questions and see what we can trim down. My goal is to interpret the responses and identify any missing information to add to our resource.

I don't think that this survey would highlight many more resources, although it may certainly highlight some values people have around what they look for. I think if we're just looking for organizations or foundations, then this should be more targeted at projects people work on, not just what they're looking for.
I'm not sure the academic map should have just projects on it - for instance, dependencies of R - if they're not connected to a foundation, organization, university, or some other entity outside of themselves. Otherwise, we'd just be including all of the dependency tree of science. Do you follow me here?

I don't think that this survey would highlight many more resources, although it may certainly highlight some values people have around what they look for. I think if we're just looking for organizations or foundations, then this should be more targeted at projects people work on, not just what they're looking for.
I'm not sure the academic map should have just projects on it - for instance, dependencies of R - if they're not connected to a foundation, organization, university, or some other entity outside of themselves. Otherwise, we'd just be including all of the dependency tree of science. Do you follow me here?

Yes, I do. What are you proposing?

I think, if we have a survey, it should be based on two things - one, which foundations, orgs, or universities does the responder think related to open source in academia? and two, what would be useful for you to know when looking for other such types of organizations? What would you want listed?
I think separating out these into two sections would help clarify that.

I think, if we have a survey, it should be based on two things - one, which foundations, orgs, or universities does the responder think related to open source in academia? and two, what would be useful for you to know when looking for other such types of organizations? What would you want listed?
I think separating out these into two sections would help clarify that.

I agree with your suggestions. I will adjust the survey to include these two sections. The first question will focus on identifying relevant organizations, while the second question will address information needs.

Hi @RichardLitt ,
I have made adjustments to the survey questions, incorporating feedback from both you and @Nolski. Could you kindly review the updated questions and let me know if any further adjustments are needed?  Survey Link

I think this looks great. Let's post it. :)
What is your plan for sending it out?

I think this looks great. Let's post it. :)
What is your plan for sending it out?
Hi @RichardLitt,
I plan to start distributing the survey on November 20th. The rollout will begin on my social platforms, followed by sharing it with my network of friends in various open source communities and the communities I am part of. We can then reshare it every two days for the next two weeks, with the survey closing on December 3rd.
This means the survey will run from November 20th to December 3rd.


docs(operators): Fix typo

Description:
Related issue (if exists):

Pull Request Test Coverage Report for Build 8190

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 96.805%

Totals

Change from base Build 8187:
-0.2%

Covered Lines:
5242

Relevant Lines:
5415

💛  - Coveralls

Thanks @lukasz17m!


Add Jest as common dependency to workspace root

Resolves #572
What are you changing?

Removes Jest and associated dependencies from @csnx/hello-world project and adds them to the workspace root.

Why?

Currently Jest is only installed as a dependency in @csnx/hello-world, but is being used by other projects in the monorepo when running tests. We could either add Jest as a dependency to each project that requires it or—as has been done here—as a common dependency to the workspace root.

Out of curiosity: do we not need @types/jest anymore then?

@types/jest was already there in the root package.json so it's not shown in the diff as I didn't need to add it. (But it is still there 😄)


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
2e77009 -> d3a341c

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.35.3 -> v1.35.4

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.35.4
Compare Source
Bug Fixes

bug with stack arrays (#843) (4e09e9c)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

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


xorg.iceauth: 1.0.7 -> 1.0.8

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
ran /nix/store/rhhzb1qc6jyfqbhnwcaxqh21n2wjhqkq-iceauth-1.0.8/bin/iceauth help got 0 exit code
ran /nix/store/rhhzb1qc6jyfqbhnwcaxqh21n2wjhqkq-iceauth-1.0.8/bin/iceauth -V and found version 1.0.8
found 1.0.8 with grep in /nix/store/rhhzb1qc6jyfqbhnwcaxqh21n2wjhqkq-iceauth-1.0.8
directory tree listing: https://gist.github.com/271a1d9c59f88dc885e44643a3bfd630

Broken commit.


Inconsistent Number of Installs

I have recently published an extension and it currently has 13 installs.
It shows correctly on the website:

But after copying the link to my README.md file, it shows the installs incorrectly:

I have added the following code in my README.md file:

Please let me know if I did something wrong. Any help is appreciated.

I think it has something to do with the behavior of the Microsoft API that is the data source and the GitHub cache (camo.githubusercontent.com). I reproduced the problem in my environment and was able to get the correct values by super reloading the browser.

I checked now and it seems to be working correctly on a different browser. It might be a problem with Chrome's cache. I'll maybe hard refresh the browser and try. Thanks.


Add hero animations

This PR adds flourishes and animations to the hero (for large screens).
Per @akrych's comp and direction, there is now a cursor that blinks after "Hello World!" and the word "editing" in the subheader gets highlighted and the tooltip appears/disappears.
This gif loops a little faster than the site does, but gives a good idea of what it looks like:

@akrych Would love your thoughts on the animation! Thanks!

Wow, love it! <3 👍

Awesome, thanks @akrych! Going to merge.


Verify EF packages work correctly in Visual Studio 2017

We have been made aware of various compatibility issues at various stages of the development of Visual Studio 15. In particular issues around components no longer being in the GAC in low-impact installs and implicit dependencies and other assumptions in our PowerShell scripts and our treatment of AppDomains.
Many known issues have been fixed but we should keep testing our packages with recent builds.

Julie Lerman in her post "Installing EF Power Tools into VS2015" provided a fix to get the product installed in VS 2015 by adding a new node to indicate the latest version.  Is it possible to just update the manifest source.extension.vsixmanifest with the xml node code block  Pro.  This will allow people to install and use the product then move on to the verification process.  This is a hard roadblock for VS 2015.

The EntityFramework package and commands appear to be working fine on VS 2017 RC. @divega was there anything in particular that we needed to test?

@bricelam There isn't anything specific that I expect to be broken at this stage. The main concern is that changes in VS 2017 (e.g. removals of components from the GAC and the new project system) may break our commands. I just hoped we could do some smoke testing on a few combinations of supported project types, languages and EF versions (e.g. EF5 and EF6).

It looks like we work everywhere except in the new ASP.NET Core projects:

Type 'Microsoft.VisualStudio.ProjectSystem.VS.Implementation.Package.Automation.OAProject' in assembly 'Microsoft.VisualStudio.ProjectSystem.VS.Implementation, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' is not marked as serializable.

We knew we'd probably have to do some work to enable that, but I couldn't find an issue tracking it.

@bricelam ok, I didn't know :smile: I guess we need a new issue. Curious, is this ASP.NET Core with the new project system but targeting .NET Framework? Is there any other way to hit this?

You can probably also hit it if you manually edit a new .NET Standard Class Library project to target .NET Framework too/instead.

Ok, that is what I thought. I don't  understanding what the error is about but in principle I would treat this as a bug in the new project system, since it is supposed to be highly compatible. Thoughts?

For this particular error yes, we can file a bug on them.
In general however, we're probably going to need some new code to handle these types of projects (like we do in EFCore). Fixing dotnet/roslyn-project-system#755 will also help minimize the changes we need to make here.
The project file extension (.csproj) and, to some extent, the file format happen to be the same, but the actual VS project system looks and behaves differently. I think under the covers, they're both CPS now, but they end up loading a different set of extensions on top of CPS.

A goal of the "old" (existing) project types (e.g. UWP, WPF, WinForms, ASP.NET, etc.) was to preserve as much behavior as possible, but I'm not sure that was ever a goal with the new (.NET Core, ASP.NET Core, etc.) project types.

I think the testing is done. I've file #146 and dotnet/roslyn-project-system#934 to track the issues found.


Tech stack section

Can do a PR but wanted to bring it up for discussion first. Think it's worth having a column for tech stack of the company? I know tech stack is sometimes transient but may be helpful for scanning the list (which is only going to grow) if only interested in certain tech.
(Great idea btw :+1:)

@jenkoian That's already included in the example company profile and people are already contributing these in the company profiles.
Having tech stack as a column in the table would increase the table width and possible the height of the rows as well. I think this is better left for a future GitHub pages version of the list.

ah didn't even notice the company profile thing, cheers.

Appreciate it! :) Thanks for the feedback.


Is WebpackMd5Hash necessary?

Thanks so much for the awesome blog post!
WebpackMd5Hash isn't mentioned in the blog post. Why is it necessary?
https://github.com/okonet/webpack-long-term-cache-demo/blob/master/webpack.config.js#L22

After testing, I can see that WebpackMd5Hash is required. Otherwise, a different filename is generated on every build. However, based on the docs, WebpackMd5Hash simply tells webpack to use MD5 instead of its standard webpack chunkhash. Apparently its standard chunkhash isn't deterministic, which is why WebpackMd5Hash is useful. Thanks again for the great post and example!

One suggestion: WebpackMd5Hash isn't mentioned in the blog post. I assume that's an accidental omission.

Hi , this post https://medium.com/webpack/predictable-long-term-caching-with-webpack-d3eee1d3fa31 mentions using NamedModulesPlugin and NamedChunksPlugin for reasons mentioned in the blog. Are all these problems fixed if I use WebpackMd5Hash instead. ?


What is the difference between CpuRef and CpuAcc?

What is the difference between compute device options armnn::Compute::CpuRef and armnn::Compute::CpuAcc?

Hi @myth01 ,
CpuAcc or "CPU Accelerated" requires NEON support, so will only work on Cortex-A processors and a subset of Cortex-R processors.
CpuRef or "CPU Reference" is a simple implementation using portable C++. You can use this to run ArmNN on Intel-architecture computers during development of your application. It is not suitable for use in deployed applications because it has poor performance compared to the alternatives.
I hope this answers your question,
Matthew


feat/network switch

Fixes #
Changes

Adds a network switcher. By default it selects Devnet.

Checks

[x] Have you catchup your branch with the latest state in base?
[x] Have you lint your code locally prior to submission?
[x] Have you reviewed your proposed changes and removed debris?
[ ] Have you successfully ran due tests with your changes?

could it be that the top bar of the table broke a bit?


[QUESTION] Adding Analog to Existing Project

Is there a path to adding Analog to an existing Nx/Angular project? What are the required steps to add Analog support to a project?

Yes, there is a path to adding Analog to an existing project. I have a previous Analog/Nx repo here https://github.com/brandonroberts/analog-nx-15
I may create a new one with commits on each step to migrate one

Cool. I'll see if I can see the differences between my Nx repo and that. I wanted to see how much it's going to take to migrate my project over when the time comes. The blog demo you posted helps, but I need to see how to use some of the info in the front matter of the markdown, and getting a list of all posts for a listing page, etc. I just haven't played with it yet.

Yep. Need to add support for front matter in the markdown files. Getting a list of all posts could be done in different ways. Vite has a nice API for getting a list of files and their contents that you could map to a list of posts with their metadata.
https://vitejs.dev/guide/features.html#glob-import

Will track this is a more recent issue but the plan is to create a schematic that can adjust an existing Angular project to use Analog


Cannot make it work in a fresh project (RN 0.62.2): Object.values requires that input parameter not be null or undefined

First of all thank you very much for this library you are creating. It looks really amazing and I think there is nothing better out there.
I got a problem when trying to make it work though and I am not sure if is because I am not using react-native: 0.63.x or I am doing something wrong.
The error I get is: Object.values requires that input parameter not be null or undefined.

Here is a repo: https://github.com/ferrannp/reanimated-gallery-example. It is a fresh app with react-native init using react-native: 0.62.x. The app code is a copy-paste of your example: https://github.com/ferrannp/reanimated-gallery-example/blob/master/App.js

Hi there. Sorry it took me so long to respond. The error seems to be related to the version won't f Reanimated you're using. Especially to the plugin. Also, make sure to clear metro's cache. Also it could happen if babel plugin is not working.
Could you try this same example with newer version of Reanimated2 and RN?

Hey @terrysahaidak thanks for answering! I tried in the same repo with "react-native-reanimated": "^2.0.0-alpha.4" (latest) and cleaning metro cache and I get the same error. I am trying with RN 0.62.2 because in the project I want to implement it we cannot update to 0.63.x just yet.

I see you're using your own build of the lib. Make sure to use reanimated's babel plugin for build with the same version as the version you're going to use in your app.
I will try to make sure it works on 0.62 as well.

also, RN can build TS files for you, so it's better to use the source code for now until I create a proper release.

Seems like there are some errors in the AppDelegate you've provided. I copy/pasted it from Reanimated Playground repo and now it crashes because of this error "Right side of an assignment cannot be restructured" in the js code which is executed on the UI thread (in worklet).

Hey @terrysahaidak. I updated the repo https://github.com/ferrannp/reanimated-gallery-example with:

Installing the library directly from your repo latest commit "reanimated-gallery": "https://github.com/terrysahaidak/reanimated-gallery#7588abe43f64d021d77b0bfad583b301190e0b4a"
I copied and paste your AppDelegate.mm with only this change from reanimated docs for 0.62.x:

( Additinal step for React Native 0.62.* ) Change initialization of TurboModuleManager
- (std::unique_ptr<facebook::react::JSExecutorFactory>)jsExecutorFactoryForBridge:(RCTBridge *)bridge
{
  _turboModuleManager = [[RCTTurboModuleManager alloc] initWithBridge:bridge delegate:self];
  ...

Now the app is not crashing but when I click on an image, nothing happens (I cannot click anything else because it seems there is an overlay, but you cannot see the gallery).


.extend after .refine

const timespan = z.object({
    start: z.date(),
    end: z.date()
}).refine(startIsBeforeEnd)

const annotatedTimespan = timespan.extend({
    something: z.string()
})
// ^ Fails with error TS2339: Property 'extend' does not exist on type ...

I guess this is to do with it being difficult to define methods on a ZodEffects which may or may not be there depending on the first type parameter.

I thought I've had fixed it by accessing the inner type of the refinement with .refine(...).innerType() but it just skips it. My current solution unfortunately is to rewrite all the refinements manually on each extended schema

This isn't possible given the new Zod 3 architecture I'm afraid.
Like you mentioned, refinements and transforms have been consolidated as "effects" and live in the ZodEffects class.  Zod is designed to validate a type, then pass that value through a chain of refinements/transforms (in that order). Splitting out your refinement functions is the best way to do this currently.

I'm using zod on a form that can support password changes with and without a code.

This is my validationSchema:
const validationSchema = z
  .object({
    password: z
      .string()
      .min(10, 'login:errors.passwordMinLength')
      .nonempty('common:formErrors.required'),
    passwordConfirm: z.string(),
  })
  .refine((data) => data.password === data.passwordConfirm, {
    message: 'login:errors.passwordMismatch',
    path: ['confirmPassword'],
  });

const validationSchemaWithCode = z
  .object({
    code: z.string().nonempty('common:formErrors.required'),
    password: z
      .string()
      .min(10, 'login:errors.passwordMinLength')
      .nonempty('common:formErrors.required'),
    passwordConfirm: z.string(),
  })
  .refine((data) => data.password === data.passwordConfirm, {
    message: 'login:errors.passwordMismatch',
    path: ['confirmPassword'],
  });

Ideally I'd like to extend it to add code as a required field, but because i'm trying to add a refinement for checking if the password is the same, I need to duplicate the schemas and add refine to both. Is there any easier way of representing this? Or maybe any examples of complex schemas where fields requirements changed based on selections. (I.E a type field can change the form subtly I.E on a travel form, if you're making a one way trip you just need outwards date, but on a return trip you also now need to make the return field required)
Apologies if this is off topic for this issue, but it seemed relevant.

You can model your state as a tagged union, e.g. {oneWay: true, departureDate: Date} | {oneWay: false, departureDate: Date, returnDate: Date}.
You can model your state with the password/confirm password thing in its own object, that way you can reuse its validation
Worst case scenario, I wonder if you can merge or intersect schemas which are based on objects but have seen refinements?

I've implemented this feature:
import { ZodObject } from 'zod'
import { AnyZodObject, objectInputType, objectOutputType, UnknownKeysParam, ZodEffects, ZodRawShape, ZodTypeAny } from 'zod/lib/types'
import { addEffect } from './addEffect'

export const mergeWithEffect = <Incoming extends AnyZodObject, T extends ZodRawShape, UnknownKeys extends UnknownKeysParam = UnknownKeysParam, Catchall extends ZodTypeAny = ZodTypeAny, Output = objectOutputType<T, Catchall, UnknownKeys>, Input = objectInputType<T, Catchall, UnknownKeys>>(a: ZodEffects<ZodObject<T, UnknownKeys, Catchall, Output, Input>>, b: Incoming) => {
  return addEffect(a.innerType().merge(b), a._def.effect)
}

export const mergeWithEffects = <aT extends ZodRawShape, bT extends ZodRawShape, aUnknownKeys extends UnknownKeysParam = UnknownKeysParam, aCatchall extends ZodTypeAny = ZodTypeAny, bUnknownKeys extends UnknownKeysParam = UnknownKeysParam, bCatchall extends ZodTypeAny = ZodTypeAny, aOutput = objectOutputType<aT, aCatchall, aUnknownKeys>, aInput = objectInputType<aT, aCatchall, aUnknownKeys>, bOutput extends { [x: string]: unknown } = objectOutputType<bT, bCatchall, bUnknownKeys>, bInput extends { [x: string]: unknown } = objectInputType<bT, bCatchall, bUnknownKeys>>(a: ZodEffects<ZodObject<aT, aUnknownKeys, aCatchall, aOutput, aInput>>, b: ZodEffects<ZodObject<bT, bUnknownKeys, bCatchall, bOutput, bInput>>) => {
  const base = a.innerType().merge(b.innerType())
  return addEffect(addEffect(base, a._def.effect), b._def.effect)
}

import { Effect, z, ZodType } from 'zod'
import { ZodTypeDef } from 'zod/lib/types'

export const addEffect = <Output = unknown, Def extends ZodTypeDef = ZodTypeDef, Input = Output>(base: ZodType<Output, Def, Input>, effect: Effect<unknown>) => {
  switch (effect.type) {
    case 'preprocess':
      return z.preprocess(effect.transform, base)
    case 'transform':
      return base.transform(effect.transform)
    case 'refinement':
      return base.superRefine(effect.refinement)
  }
}

import { expect, test } from '@jest/globals'
import assert from 'assert'
import { z } from 'zod'
import { mergeWithEffect, mergeWithEffects } from './mergeWithEffects'

test(mergeWithEffect.name, () => {
  const a = z.object({ name: z.string() }).refine(({ name }) => name.length > 0)
  const b = z.object({ surname: z.string() })
  const merged = mergeWithEffect(a, b)
  const result = merged.safeParse({ name: '', surname: '' })
  assert(!result.success)
  expect(result.error.errors.length).toEqual(1)
})

test(mergeWithEffects.name, () => {
  const a = z.object({ name: z.string() }).refine(({ name }) => name.length > 0)
  const b = z.object({ surname: z.string() }).refine(({ surname }) => surname.length > 0)
  const merged = mergeWithEffects(a, b)
  const result = merged.safeParse({ name: '', surname: '' })
  assert(!result.success)
  expect(result.error.errors.length).toEqual(2)
})

@colinhacks Should I open a PR?

I've worked around this by exporting my shared schema and a refiner function and then in the code sites that use it, extending the schema and wrapping the extended shema in the refiner:
/packages/shared:
export const sharedSchema = z.object({ … });
export const refineExtendedSharedSchema = (schema: typeof sharedSchema) => schema.refine(({ … }) => …);

/apps/call-site-n (this only makes sense if there are multiple code sites sharing the basic schema):
const siteSpecificSchema = refineExtendedSharedSchema(sharedSchema.extend({ … }));

This is a usable workaround, but it feels very inelegant. I was sure there was a better way before I found this issue. Would be great to get a word from @colinhacks on @DenisGorbachev's solution and whether there are any downsides which preclude it from being included in Zod - and if there aren't, if it could be included in Zod.

to get the correct types. you need to type the params as generic
export function deleteAccountRequestSchemaWithRefine<
  T extends z.infer<typeof deleteAccountRequestSchema>,
>(schema: z.ZodSchema<T>): z.ZodSchema<T> {
  return schema.refine((data) => !(data.reason === 'Other' && !data.feedback), {
    message: 'Feedback must be filled',
    path: ['feedback'],
  });
}

@bryanltobing Would you mind expending how you are using your solution?
 T extends z.infer<typeof schema>, this part is not working for me

function refineSchema<T extends z.Schema, R extends ReturnType<T["refine"]>>(
  schema: T,
): R {
  return schema.refine((values) => ....),
  ) as R;
}

This will resolve to correct types even in cases where the schema uses discriminatedUnion
@gilles-yvetot  eg.
const customerSchema = refineSchema(z.Object({name: z.string().trim().min(3)}))

The solution where we add the refinement via a function works pretty well but it feels awkward when using it with the rest of the Zod API, and once you get around to adding the correct types to the function, it can start to feel somewhat complicated. My solution was to just put the arguments for refine() inside of an array and then just call refine(...args) on any shared schema that needs to be extended. Still not the greatest solution because anyone using the code needs to understand that they should always call refine() with the args array but the readability is a bit more clear IMO and the amount of boilerplate is close to the same. This is what that solution looks like:
const sharedSchema = z.object({
  start: z.date(),
  end: z.date()
})

export const startIsBeforeEnd: Parameters<
  (typeof sharedSchema)['refine']
> = [
  ({ start, end }) => start < end,
  {
    message: 'Start must be before end'
  }
]

const timespan = sharedSchema.refine(...startIsBeforeEnd)

const annotatedTimespan = sharedSchema.extend({ something: z.string()}).refine(...startIsBeforeEnd)

I've implemented this feature:
import { ZodObject } from 'zod'
import { AnyZodObject, objectInputType, objectOutputType, UnknownKeysParam, ZodEffects, ZodRawShape, ZodTypeAny } from 'zod/lib/types'
import { addEffect } from './addEffect'

export const mergeWithEffect = <Incoming extends AnyZodObject, T extends ZodRawShape, UnknownKeys extends UnknownKeysParam = UnknownKeysParam, Catchall extends ZodTypeAny = ZodTypeAny, Output = objectOutputType<T, Catchall, UnknownKeys>, Input = objectInputType<T, Catchall, UnknownKeys>>(a: ZodEffects<ZodObject<T, UnknownKeys, Catchall, Output, Input>>, b: Incoming) => {
  return addEffect(a.innerType().merge(b), a._def.effect)
}

export const mergeWithEffects = <aT extends ZodRawShape, bT extends ZodRawShape, aUnknownKeys extends UnknownKeysParam = UnknownKeysParam, aCatchall extends ZodTypeAny = ZodTypeAny, bUnknownKeys extends UnknownKeysParam = UnknownKeysParam, bCatchall extends ZodTypeAny = ZodTypeAny, aOutput = objectOutputType<aT, aCatchall, aUnknownKeys>, aInput = objectInputType<aT, aCatchall, aUnknownKeys>, bOutput extends { [x: string]: unknown } = objectOutputType<bT, bCatchall, bUnknownKeys>, bInput extends { [x: string]: unknown } = objectInputType<bT, bCatchall, bUnknownKeys>>(a: ZodEffects<ZodObject<aT, aUnknownKeys, aCatchall, aOutput, aInput>>, b: ZodEffects<ZodObject<bT, bUnknownKeys, bCatchall, bOutput, bInput>>) => {
  const base = a.innerType().merge(b.innerType())
  return addEffect(addEffect(base, a._def.effect), b._def.effect)
}

import { Effect, z, ZodType } from 'zod'
import { ZodTypeDef } from 'zod/lib/types'

export const addEffect = <Output = unknown, Def extends ZodTypeDef = ZodTypeDef, Input = Output, Base extends ZodType<Output, Def, Input> = ZodType<Output, Def, Input>>(base: Base, effect: Effect<unknown>) => {
  switch (effect.type) {
    case 'preprocess':
      return z.preprocess(effect.transform, base)
    case 'transform':
      return base.transform(effect.transform)
    case 'refinement':
      return base.superRefine(effect.refinement)
  }
}

import { expect, test } from '@jest/globals'
import assert from 'assert'
import { z } from 'zod'
import { mergeWithEffect, mergeWithEffects } from './mergeWithEffects'

test(mergeWithEffect.name, () => {
  const a = z.object({ name: z.string() }).refine(({ name }) => name.length > 0)
  const b = z.object({ surname: z.string() })
  const merged = mergeWithEffect(a, b)
  const result = merged.safeParse({ name: '', surname: '' })
  assert(!result.success)
  expect(result.error.errors.length).toEqual(1)
})

test(mergeWithEffects.name, () => {
  const a = z.object({ name: z.string() }).refine(({ name }) => name.length > 0)
  const b = z.object({ surname: z.string() }).refine(({ surname }) => surname.length > 0)
  const merged = mergeWithEffects(a, b)
  const result = merged.safeParse({ name: '', surname: '' })
  assert(!result.success)
  expect(result.error.errors.length).toEqual(2)
})

@colinhacks Should I open a PR?

Trying out your approach right now and TypeScript seems to be unable to resolve the return type of mergeWithEffect :/

I've implemented this feature:
import { ZodObject } from 'zod'
import { AnyZodObject, objectInputType, objectOutputType, UnknownKeysParam, ZodEffects, ZodRawShape, ZodTypeAny } from 'zod/lib/types'
import { addEffect } from './addEffect'

export const mergeWithEffect = <Incoming extends AnyZodObject, T extends ZodRawShape, UnknownKeys extends UnknownKeysParam = UnknownKeysParam, Catchall extends ZodTypeAny = ZodTypeAny, Output = objectOutputType<T, Catchall, UnknownKeys>, Input = objectInputType<T, Catchall, UnknownKeys>>(a: ZodEffects<ZodObject<T, UnknownKeys, Catchall, Output, Input>>, b: Incoming) => {
  return addEffect(a.innerType().merge(b), a._def.effect)
}

export const mergeWithEffects = <aT extends ZodRawShape, bT extends ZodRawShape, aUnknownKeys extends UnknownKeysParam = UnknownKeysParam, aCatchall extends ZodTypeAny = ZodTypeAny, bUnknownKeys extends UnknownKeysParam = UnknownKeysParam, bCatchall extends ZodTypeAny = ZodTypeAny, aOutput = objectOutputType<aT, aCatchall, aUnknownKeys>, aInput = objectInputType<aT, aCatchall, aUnknownKeys>, bOutput extends { [x: string]: unknown } = objectOutputType<bT, bCatchall, bUnknownKeys>, bInput extends { [x: string]: unknown } = objectInputType<bT, bCatchall, bUnknownKeys>>(a: ZodEffects<ZodObject<aT, aUnknownKeys, aCatchall, aOutput, aInput>>, b: ZodEffects<ZodObject<bT, bUnknownKeys, bCatchall, bOutput, bInput>>) => {
  const base = a.innerType().merge(b.innerType())
  return addEffect(addEffect(base, a._def.effect), b._def.effect)
}

import { Effect, z, ZodType } from 'zod'
import { ZodTypeDef } from 'zod/lib/types'

export const addEffect = <Output = unknown, Def extends ZodTypeDef = ZodTypeDef, Input = Output, Base extends ZodType<Output, Def, Input> = ZodType<Output, Def, Input>>(base: Base, effect: Effect<unknown>) => {
  switch (effect.type) {
    case 'preprocess':
      return z.preprocess(effect.transform, base)
    case 'transform':
      return base.transform(effect.transform)
    case 'refinement':
      return base.superRefine(effect.refinement)
  }
}

import { expect, test } from '@jest/globals'
import assert from 'assert'
import { z } from 'zod'
import { mergeWithEffect, mergeWithEffects } from './mergeWithEffects'

test(mergeWithEffect.name, () => {
  const a = z.object({ name: z.string() }).refine(({ name }) => name.length > 0)
  const b = z.object({ surname: z.string() })
  const merged = mergeWithEffect(a, b)
  const result = merged.safeParse({ name: '', surname: '' })
  assert(!result.success)
  expect(result.error.errors.length).toEqual(1)
})

test(mergeWithEffects.name, () => {
  const a = z.object({ name: z.string() }).refine(({ name }) => name.length > 0)
  const b = z.object({ surname: z.string() }).refine(({ surname }) => surname.length > 0)
  const merged = mergeWithEffects(a, b)
  const result = merged.safeParse({ name: '', surname: '' })
  assert(!result.success)
  expect(result.error.errors.length).toEqual(2)
})

@colinhacks Should I open a PR?

should we PR this solution? @colinhacks

I am pretty new to Zod and do not know if this is a very elegant solution and I feel like there are some semantic issues with what I am doing, but I had a goal of extending a UserSchema to allow for different roles to have different kinds of validation. I have refinement steps that I would like performed for every schema extending the UserSchema and refinement steps for one of the schemas extending the UserSchema, StudentSchema.
To have my StudentSchema include all of the properties from my UserSchema and retain the appropriate typing I use .and instead of .extend:
import { z } from 'zod';
export const userSchema = z.object({
firstName: z
.string()
.min(1, 'First name is required'),
lastName: z
.string()
.min(1, 'Last name is required'),
email: z
.string()
.min(1, 'Email is required') // Ensures "required_error" triggers first
.email('Not a valid email'),
emailConfirmation: z
.string()
.min(1, 'Email confirmation is required')
.email('Not a valid email'),
}).refine((data) => data.email === data.emailConfirmation, {
message: 'Emails do not match',
path: ['emailConfirmation'],
});
const studentSchema = z.object({
dateOfBirth: z.coerce.date({
required_error: 'Date of birth is required',
}),
}).refine(
(data) =>
!userAgeUnder18(data),
{
message: 'Student too young to create an account.',
path: ['dateOfBirth'],
}
).and(userSchema);
I don't feel like this is the ideal way to solve this problem, but this seems to have done the trick for me after a couple hours of banging my head against the wall without finding any simple solutions. I'm not sure if there are any drawbacks to this approach, but it seems like one potential solution to a problem that seems pretty tricky.

You can export a function that takes a callback:
const sharedSchema = z.object({ … });
export const refineExtendedSharedSchema = (callback: (typeof sharedSchema => typeof sharedSchema)= ((s) => s)) => {
    return callback(sharedSchema).refine(...)
}


TOCO failed

Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: ADD, AVERAGE_POOL_2D, CONCATENATION, CONV_2D, FLOOR, FULLY_CONNECTED, L2_NORMALIZATION, MAX_POOL_2D, MUL, PACK, RELU, RESHAPE, RSQRT, SHAPE, STRIDED_SLICE. Here is a list of operators for which you will need custom implementations: BatchNormalization, Merge, RandomUniform, Switch.

Please provide the information
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (or github SHA if from source):

Provide the text output from tflite_convert
# Copy and paste here

Also, please include a link to a GraphDef or the model if possible.
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@widiakha, Provide the information asked in the template. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Revert "Shrink uptermd Docker image by 5 MB, 20% (#225)"

This reverts commit bb763d6cdd9e5b1da00ffe794c71b3dfe0422828.
@bcspragu I have to revert https://github.com/owenthereal/upterm/pull/225 because the production community server shares the same image and needs sh to compute fly.io node address at boot time: https://github.com/owenthereal/upterm/blob/2e48f7341cf51a291bb626ddfc70ae56da0b49d0/fly.toml#L11-L12. I think we might have to stick with the Alpine image or if you have a suggestion that a smaller image that has a shell in it.

Ah, apologies for breaking things! As for adding a shell to SCRATCH, I think that defeats most of the purpose (and some of the size benefits) of it. Some alternatives, from what I think are best to worst:

Add FLY_* "auto-detection" directly to uptermd. If FLY_ALLOC_ID and FLY_APP_NAME are set, use those variables

Can optionally hide this behind a --autodetect-fly flag to make it opt-in
I'm suggesting this because it looks like the only usage of sh here is variable substitution (and some string munging)
Am also happy to do this work

Make a separate Docker image for Fly that includes a shell

Makes sense if there's other sh usage in the Fly setup

Copy the shell over from the builder step

This probably won't actually work, with dynamic libs and whatnot, and probably isn't a good approach.

Not having a shell is also a useful security invariant for some modalities of compromise, gives an attacker less of a toolkit for introspecting things.

I think the FLY_* shouldn't be built into uptermd because it's designed to be cloud agnostic. Making a separate Docker image for Fly is the way to go if we want to have a minimum image for self-hosting while keeping a shell for the fly.io deployment.

One more idea because I think even Fly users would benefit from the lack of shell: A small, auxiliary flycfg binary that does the "env lookup to uptermd flag" conversion. That way, Fly still has a separate, Fly-specific image, uptermd has no Fly-specific flags, and both images benefit from lack of shell for size + security

A small, auxiliary flycfg binary that does the "env lookup to uptermd flag" conversion. That way, Fly still has a separate, Fly-specific image, uptermd has no Fly-specific flags, and both images benefit from lack of shell for size + security

Thanks for the feedback. I created a binary uptermd-fly for this purpose and use Docker's build target for Fly's deployment: https://github.com/owenthereal/upterm/pull/261


Wrong injector when handover parameter from Sightly to SlingModel

Required Information

[ x] AEM Version, including Service Packs, Cumulative Fix Packs, etc: 6.4.4
[ x] ACS AEM Commons Version: > 4.0.0
[ x] Reproducible on Latest? yes

Expected Behavior
When handing in parameters from htl to SlingModel a String Object is handed to JavaClass
e.g.
@Inject
@org.apache.sling.models.annotations.Optional
Object linkClasses;

corresponding html
data-sly-use.hello="${'de.eggs.core.models.HelloWorldModel' @ linkClasses=['custom-link']}"
Actual Behavior
After installing ACS-Commons > 4.0.0 the handed Object is not a String or String Array anymore, instead a I18N Object is injected
Some more wrong Injection Mapping occure after installing ACS-Commons > 4.0.0 I will provide more later
Steps to Reproduce
Install Project from
https://github.com/eggsunimediaGmbH/acs-commons
Insert HelloWorld Component
Install ACS-Commons 4.4 Debugging class de.eggs.core.models.HelloWorldModel See that htlParameter property is now I18N class
Links
https://github.com/eggsunimediaGmbH/acs-commons

Is there some way we can convert your debugging class into a proper unit test?  That would help a lot!

Is there some way we can convert your debugging class into a proper unit test? That would help a lot!

I try

Added a jUnit Test which reproduce the issue in the attached git repo
https://github.com/eggsunimediaGmbH/acs-commons/blob/master/core/src/test/java/de/eggs/core/models/HelloWorldModelTest.java

Thanks for the testcase.
It seems that the problematic part is the I18NInjector. If you don't register this service in your unittest, everything is fine. This injector has been added in ACS AEM Commons, so it matches with your observation.
For me it seems that this I18NInjector has a higher priority than the other injectors and can also handle any injections of objects, which are assignable from I18N (and of course Object can be assigned from any class).
I would think in 2 different ways now:

Can you improve your injection by not using Object htlParameter, but a something more specific class?
In parallel we should improve the I18NInjector in a way, that it does not handle this Object case. Can you raise a dedicated ticket for it (referencing this testcase)? I don't know if reducing the priority is enough; also this would be a major change which is potentially affecting everyone already using this Injector.

@joerghoh
reg 1.) With Object class i can handle String and String[] hand overs, thats why it is there.
reg 2.) https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2158

@beo4 Maybe you can use the @Source annotation in the meantime to fix the issue you face.


RFC enable check

prerequsites: this repo is installed in satellite systems

low priority, it would be a mess


⚠ ClamAV has degraded performance

In dd24ab6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 285 ms

Resolved: ClamAV performance has improved in 6532833.


Update to 2.11.11 and add windows build

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Closes #16, closes #15, closes #3, closes #1

I guess I wasn't recommending to remove BLAS on Linux and MacOS. This would indeed be a regression and since everything is and has been working fine with BLAS there, I don't see a reason to change it. It is only the linking to BLAS on Windows that is a separate issue, not the linking to BLAS on all platforms. We could consider linking to libcblas on non-windows, though, which would hopefully mean we don't need Fortran at all on any platform. So maybe add back in libcblas and see if it gets picked up on non-Windows. It doesn't hurt to have it available on all platforms, it just won't be picked up on Windows.

Works now. Including cblas on Win and all other Fortran stuff on unix.

Great! I couldn't see what you did to get cblas.lib and lapack.lib to be installed on Windows. When I installed the cblas conda package locally, I didn't get the static libraries, as far as I could see. Anyway, great that it's all working now, but LAPACK was not being linked on Windows. I know you had asked about it yesterday. Yes, it is also good to link LAPACK. I committed a change to add that for Windows as well.

Hmm, I don't know why the CI jobs on Azure are not being launched.

Will lapack require a fortran compiler?

Apparently not.

Everything looks great now, finally!

@wolfv @h-vetinari Any comments? This should be much faster to review than the previous PR on which you both had a look already.

@jpfeuffer Should we just merge? I'm still working on releases for the rest of the stack, but Osi is done and I will push on the rest if we can get this done.

On the one hand, I am pretty confident that I addressed the comments from last time. With the new PR also the commit history looks much better.
On the other hand, they might not have been fully done with their reviews.
But I think there is nothing that could not be fixed with a smaller follow-up PR.

Sorry, haven't had time for this recently. From a quick look this looks good to me, so feel free to go ahead!

Yay! Yes, I will give it a try!


Add support for attachment's title and title link

This adds support for title and title_link attachment's attributes: https://api.slack.com/docs/attachments (see "title and title_link" section)

Looks really good. Could you modify ClientFunctionalTest and AttachmentUnitTest to account for the new field, and squash your commits once the suite is passing? Then this should be good to merge :+1:

Updated and squashed. Build passing.

Fantastic, thanks!


Prevent sending empty message

Fixes #118
Port https://github.com/jupyterlab/jupyter-ai/pull/946

The condition to disable the button was also applied to the 'cancel' button (visible when editing a message).
I removed this condition, since an edition should always be cancellable, AFAIK.


Lf table upgrade

This PR updates the cancer biomarker widget to the new table component, greatly simplifying from the previous custom table.
As before, all the data is batch loaded (see note below), but now using the useBatchDownloader hook for consistency.
The cancer biomarkers API endpoint does allow for pagination, so a server side approach on the table would leverage that. However the endpoint doesn't support search, so to my understanding table filtering (specifically requested in the ticket) would not be possible with server side approach. This could be changed/improved in the future.
The whole custom/filterTable.js component is redundant and could be removed: it contains examples of the old summary charts, but I imagine that these would be implemented differently now if so required.
Closes https://github.com/opentargets/platform/issues/1122

Converting the PR to draft as there seem to be some bugs - testing with ERBB2 doesn't work as I was expecting, so I need to revise the loading of data.

Ready for review, however the latest commit dcfeb01 is not showing in the PR.
Github was a little flakey this morning, so perhaps that might be the cause.


Fixed prompt or specific prompt

1.I have run the demo successfully, but the generated voices sound from different speakers when I ran the code twice. It might be load the different speaker presets, how can I lock the speaker prompt when I ran the code multiple times and the output sounds from same speaker?
2.How can I customize my own prompt?

Hi @Reeshark  you can use a history prompt like:
python -m bark --text "Hi suno, this is a test.[laughs] I am very happy today." --history_prompt "bark/assets/prompts/en_speaker_9.npz"

The sound will come out a bit more consistent.


Created UI for adding a hospital

What
Added the UI for adding a hospital
Why
So admin users can add hospitals
Screenshots

Notes
N/A

There is an SQL query to sort that out @StephenThomson . Me and @jivdhaliwal have ran it on stage and prod so the hospitals table should be populated.


how to call class method with string function name?

example:
class A {
B(){
return true;
}
C(){
return false;
}
}

var str = "B";
how to call function B with str;

Dup of https://github.com/dart-lang/sdk/issues/34909


Sorry something went wrong while deploying assets from spfx

Hello Experts ,
As per the guide line give at
https://docs.microsoft.com/en-us/sharepoint/dev/spfx/web-parts/get-started/provision-sp-assets-from-package
I am following same steps but when ever i am trying to do "Add an app" then i can not add the app and it says "Sorry some thing went wrong with adding the app "

Below is steps to reproduce,
Followed the guideline and created a new solution
below is elements.xml file
`

<Field ID="{060E50AC-E9C1-4D3C-B1F9-DE0BCAC300F6}"
        Name="SPFxAmount"
        DisplayName="Amount"
        Type="Currency"
        Decimals="2"
        Min="0"
        Required="FALSE"
        Group="SPFx Columns" />

<Field ID="{943E7530-5E2B-4C02-8259-CCD93A9ECB18}"
        Name="SPFxCostCenter"
        DisplayName="Cost Center"
        Type="Choice"
        Required="FALSE"
        Group="SPFx Columns">
    <CHOICES>
    <CHOICE>Administration</CHOICE>
    <CHOICE>Information</CHOICE>
    <CHOICE>Facilities</CHOICE>
    <CHOICE>Operations</CHOICE>
    <CHOICE>Sales</CHOICE>
    <CHOICE>Marketing</CHOICE>
    </CHOICES>
</Field>

<ContentType ID="0x010042D0C1C200A14B6887742B6344675C8B"
        Name="Cost Center"
        Group="SPFx Content Types"
        Description="Sample content types from web part solution">
    <FieldRefs>
        <FieldRef ID="{060E50AC-E9C1-4D3C-B1F9-DE0BCAC300F6}" />
        <FieldRef ID="{943E7530-5E2B-4C02-8259-CCD93A9ECB18}" />
    </FieldRefs>
</ContentType>

<ListInstance
        CustomSchema="schema.xml"
        FeatureId="8bab261f-ed6f-4663-9fd5-51D042f6633d"
        Title="SPFx List"
        Description="SPFx List"
        TemplateType="100"
        Url="Lists/SPFxList">
</ListInstance>

`
below is schema.xml file
can not add here. i have attached the file instead

and finally below is package-solution.json
{
"$schema": "https://developer.microsoft.com/json-schemas/spfx-build/package-solution.schema.json",
"solution": {
"name": "assetdep-1-client-side-solution",
"id": "c55f6765-d7c7-42c0-a557-78b3162dc1a5",
"version": "<IP_ADDRESS>",
"includeClientSideAssets": true,
"features": [{
"title": "asset-deployment-webpart-client-side-solution",
"description": "asset-deployment-webpart-client-side-solution",
"id": "8bab261f-ed6f-4663-9fd5-51d042f6633d",
"version": "<IP_ADDRESS>",
"assets": {
"elementManifests": [
"elements.xml"
],
"elementFiles":[
"schema.xml"
]
}
}]
},
"paths": {
"zippedPackage": "solution/assetdep-1.sppkg"
}
}
depedencies in package.json :
"dependencies": {
"@microsoft/sp-core-library": "1.7.1",
"@microsoft/sp-webpart-base": "1.7.1",
"@microsoft/sp-lodash-subset": "1.7.1",
"@microsoft/sp-office-ui-fabric-core": "1.7.1",
"@types/webpack-env": "1.13.1",
"@types/es6-promise": "0.0.33"
},
"devDependencies": {
"@microsoft/sp-build-web": "1.7.1",
"@microsoft/sp-tslint-rules": "1.7.1",
"@microsoft/sp-module-interfaces": "1.7.1",
"@microsoft/sp-webpart-workbench": "1.7.1",
"gulp": "~3.9.1",
"@types/chai": "3.4.34",
"@types/mocha": "2.2.38",
"ajv": "~5.2.2"
}
I am not sure what i am doing wrong. it is frustrating.
Please suggest if someone have some solution.
If possible please suggest if there any other smooth way for deploying assets.
Thank you in advance for your response

Hello Mukesh,
Could try each part seperately.
First try to provision only the columns. If they appear:
provision columns and contenttype.
If they appear provision listinstance.
If you are using example code, it might be better to generate your own guids using a generator.
Kind regards
Simon

this is fixed in 1.14 beta


Cannot install fisherman

Here is the shell session where I present all the versions of the necessary software and then try to do an installation. Since I use vim-mode, you can identify the new commands by [I] ~ $.
[I] ~ $ fish --version
fish, version 2.2.0
[I] ~ $ git --version
git version 2.7.4
[I] ~ $ uname -a
Linux lt15713 3.13.0-83-generic #127-Ubuntu SMP Fri Mar 11 00:25:37 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
[I] ~ $ tree ~/.config/fish/
/home/alexandre/.config/fish/
├── config.fish
├── config.fish~
├── fishd.f01faf4ed963
├── fish_history
├── fish_read_history
├── functions
│   └── fish_prompt.fish
└── nvm-wrapper
    ├── CODE_OF_CONDUCT.md
    ├── nvm.fish
    └── README.md

2 directories, 9 files
[I] ~ $ tree ~/.local/share/fish/
/home/alexandre/.local/share/fish/
└── generated_completions

1 directory, 0 files
[I] ~ $ vimcat ~/.config/fish/config.fish

#alex# sbt path
set -x ANDROID_HOME $HOME/adt-bundle-linux-x86_64/sdk
#set -x ANDROID_SDK_ROOT $HOME/adt-bundle-linux-x86_64/sdk
#set -x ANDROID_SDK_HOME $HOME/adt-bundle-linux-x86_64/sd

set -x PATH $ANDROID_HOME/tools:~/node_modules/.bin:~/sbt/bin:~/android-studio/bin:~/bin:$PATH

set -x PATH ~/nodejs/bin:$PATH

set -x PATH ~/bin $PATH

set -x WINDOWS_USER alex.martins

#Linuxbrew
set -x PATH $HOME/.linuxbrew/bin $PATH
set -x MANPATH $HOME/.linuxbrew/share/man (manpath)
set -x INFOPATH $HOME/.linuxbrew/share/info $INFOPATH

set -x PATH ~/.local/lib/aws/bin:$PATH

#AWS CLI autocomplete
#complete -C aws_completer aws

set -x NVM_DIR ~/.nvm
source ~/.config/fish/nvm-wrapper/nvm.fish
nvm use default 1> /dev/null
#[ -s "$NVM_DIR/nvm.sh" ] and . "$NVM_DIR/nvm.sh"  # This loads nvm
# [[ -r $NVM_DIR/bash_completion ]] and . $NVM_DIR/bash_completion

#complete -C '~/.local/lib/aws/bin/aws_completer' aws
set -x PATH $HOME/.local/lib/aws/bin:$PATH

set -x  NEW_RELIC_LICENSE_KEY dddddddddddddddddddddddddddddddddddddd

#UkLive Scripts
set -x PATH $HOME/workspace/UkLive/scripts/helpers:$PATH

set -x VISUAL vim

fish_vi_mode
set -x EDITOR "$VISUAL"

[I] ~ $ curl -sL get.fisherman.sh | fish
manpath: warning: $MANPATH set, ignoring /etc/manpath.config
 × Downloading Fisherman to ~/.local/share/fisherman

Sorry, but I couldn't install Fisherman. Some things you can try:

 → Upgrade your Git version (^2.0)
 → Upgrade your Fish version (^2.2.0)
 → Make sure you can write to
 → Make sure you are connected to the Internet

I could have simplified this but maybe leaving it complete will give someone extra hints about what extra information you need from me.

@alexmsmartins Ok, so it seems you couldn't even download Fisherman. Do you have write permissions in ~/.local/share?

Hi @bucaran . I double checked permissions and I do have write permissions to ~/.local/share. This is inside a regular $HOME directory and I have right permissions everywhere.

@alexmsmartins
In that case, Fisherman must be downloaded to ~/.local/share.
But here is one fun thing you can try:
curl -sL get.fisherman.sh | env TRY_ME=true fish

Can we troubleshoot this together? join.fisherman.sh

@bucaran
I do not see anything strange.
Here is the output
[I] ~ $ curl -sL get.fisherman.sh | env TRY_ME=true fish
manpath: warning: $MANPATH set, ignoring /etc/manpath.config
# Clone Fisherman repository to XDG_DATA_HOME/fisherman
git clone --quiet --depth 1 https://github.com/fisherman/fisherman /home/alexandre/.local/share/fisherman

# Create Fisherman configuration directories in XDG_CONFIG_HOME/fisherman
pushd '/home/alexandre/.local/share/fisherman'

if test (uname) = 'FreeBSD'
    gmake > /dev/null
else
    make > /dev/null
end

popd

# Download Fisherman index file without a timeout
__fisher_index_update 0

# Install CLI utilities. See github.com/fishery/{spin,debug}
fisher_install 'spin' 'debug' --quiet

# Reload your shell session to make Fisherman immediately available
exec fish < /dev/tty

@alexmsmartins OK, we are making progress. Fisherman actually runs that exactly. If you try running each of those instructions one by one we could find exactly the place where you are getting stuck.
Start with
git clone --quiet --depth 1 https://github.com/fisherman/fisherman /home/alexandre/.local/share/fisherman

and let me know what happened.

I was having this problem today the requirements do not let you know that you are missing make. You should test if make ran and report that as the problem rather than the git and fish versions.

The curl installer has no dependency on make, just fish and some basic unix
tools like sed.

On Mar 22 2016, at 10:44 pm, xMrWhite<EMAIL_ADDRESS>wrote:

I was having this problem today the requirements do not let you know that
you are missing make. You should test if make ran and report that as the
problem rather than the git and fish versions.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or [view it on GitHub](https://github.com/fisherm
an/fisherman/issues/166#issuecomment-199846099)![](https://github.com/notifica
tions/beacon/ABlxkay-ovRcl_5FZT-w9SubVcLnXVubks5pwAA-gaJpZM4H0kcG.gif)

Are you sure because your commands up there
    if test (uname) = 'FreeBSD'
    gmake > /dev/null
else
    make > /dev/null
end

run make

oh wow how'd I miss that. Thanks! :smile:

@xMrWhite Oh, so that was the problem! Thanks. This should be easy to check and explain in the troubleshoot report.
https://github.com/fisherman/install/issues/3

I mean i'm not @alexmsmartins but I was installing fish and fisher only a new ubuntu 14.04 cloud today and got the exact same output  sudo apt-get install build-essential made it work. I did not try just installing make.

@xMrWhite Yes, and thanks for the solution snippet.

@bucaran
Running the commands one by one works with a little tweak. Basically, I need to reload the shell before running __fisher_index_update.
Here is the shell session.
Welcome to fish, the friendly interactive shell
Type help for instructions on how to use fish
[I] ~ $ fisher
fisher: command not found
[I] ~ $ git clone --quiet --depth 1 https://github.com/fisherman/fisherman /home/alexandre/.local/share/fisherman
[I] ~ $ pushd '/home/alexandre/.local/share/fisherman'
[I] ~/.l/s/fisherman (master) $ make # I want to see the output
mkdir -p $HOME/.config/fish/ && touch $HOME/.config/fish/config.fish
echo "set fisher_home /home/alexandre/.local/share/fisherman/" | sed "s|/$||;s|$HOME|~|" > $HOME/.config/fish/config.fish.fisher
echo "set fisher_config $HOME/.config/fisherman" | sed "s|$HOME|~|" >> $HOME/.config/fish/config.fish.fisher
echo "source \$fisher_home/config.fish" >> $HOME/.config/fish/config.fish.fisher
awk 'FNR==NR{ print; a[$0]; next } !($0 in a) || /^$/' $HOME/.config/fish/config.fish $HOME/.config/fish/config.fish.fisher > $HOME/.config/fish/config.fish.tmp
mv $HOME/.config/fish/config.fish.tmp $HOME/.config/fish/config.fish && rm $HOME/.config/fish/config.fish.fisher
mkdir -p $HOME/.config/fisherman/cache
touch $HOME/.config/fisherman/fishfile
** Reload your shell and type 'fisher' to get started **
[I] ~/.l/s/fisherman (master) $ popd
[I] ~ $ __fisher_index_update 0
__fisher_index_update: command not found
[I] ~ $ exec fish < /dev/tty
manpath: warning: $MANPATH set, ignoring /etc/manpath.config
Welcome to fish, the friendly interactive shell
Type help for instructions on how to use fish
[I] ~ $ __fisher_index_update 0
[I] ~ $ fisher_install 'spin' 'debug' --quiet
[I] ~ $ exec fish < /dev/tty
manpath: warning: $MANPATH set, ignoring /etc/manpath.config
Welcome to fish, the friendly interactive shell
Type help for instructions on how to use fish

I have fisherman installed now but if you want me to run any more experiments, let me know.

Just wanted to add that I ran into the same issue as @alexmsmartins. To fix it I ran the steps from TRY_ME but reloaded the fish config before running any of the fisher commands. Seems to work now.

@jimfleming Hey, thanks, what was the OS in your case?

I edited my original post shortly after hitting enter. Here's the full OS version: OS X (El Capitan) 10.11 (15A284).

@jimfleming Do you have make? Try make -v.

Yup, here's the output:
GNU Make 3.81
# license omitted
This program built for i386-apple-darwin11.3.0

In that case the problem was not make. I understand that @alexmsmartins's problem (or maybe that was @xMrWhite or someone else) was they didn't have make.
Well, I am going to close this as resolved, since all of you were able to install Fisherman correctly, but do please ping me if you try to re-install Fisherman and something does not go well as I was not able to reproduce this problem in two different macs.


[MM-50268] Add additional fields to HTTP logger

Summary
This PR adds the HTTP method, the URL and the user agent to the HTTP request logger.
The logger now logs the message before handling the request. Take makes debugging easier as the flow is more natural.

Ticket Link
https://mattermost.atlassian.net/browse/MM-50268

I thought about removing the duplicate path field, but kept it as it useful to have that information in the other log messages down the stack.

Codecov Report
Base: 20.74% // Head: 20.74% // No change to project coverage :thumbsup:

Coverage data is based on head (29fa5db) compared to base (76a4501).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #451   +/-   ##
=======================================
  Coverage   20.74%   20.74%
=======================================
  Files          80       80
  Lines        5366     5366
=======================================
  Hits         1113     1113
  Misses       4114     4114
  Partials      139      139

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

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