Minimal repro for decorators breaking Pass Manager drawing

Summary
This explains why https://github.com/Qiskit/qiskit-terra/pull/9868 is failing because it adds a decorator to CXDirection.__init__. That results in tox -e py -- test_pass_manager_drawer_basic failing.
This PR also fails when running tox -e py -- test_pass_manager_drawer_basic:
  21 [color=blue, fontname=helvetica, label=CXDirection, shape=rectangle];
+ 22 [color=black, fontname=helvetica, fontsize=10, label=coupling_map, shape=ellipse, style=solid];
+ 22 -> 21;
  19 -> 21;
  }

- subgraph cluster_22 {
?                   ^
+ subgraph cluster_23 {
?                   ^
  fontname=helvetica;
  label="[8] ";
  labeljust=l;
- 23 [color=blue, fontname=helvetica, label=RemoveResetInZeroState, shape=rectangle];
?  ^
+ 24 [color=blue, fontname=helvetica, label=RemoveResetInZeroState, shape=rectangle];
?  ^
- 21 -> 23;
?        ^
+ 21 -> 24;
?        ^
  }

  }

Is that expected that a decorator would change the result?
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 7 (85.71%)  changed or added relevant lines in 1 file are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.02%) to 85.371%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/transpiler/passes/utils/cx_direction.py
6
7
85.71%

Files with Coverage Reduction
New Missed Lines
%

qiskit/extensions/quantum_initializer/squ.py
2
80.0%

crates/accelerate/src/sabre_swap/layer.rs
4
97.32%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
67197

Relevant Lines:
78712

💛  - Coveralls


Feature: The description section in the Assignment box should not be left empty

Is your feature request related to a problem? Please describe
The description box in the assignment section should be either given a "*" symbol and be made compulsory, or it should display an error "the box needs to be filled" message.
Describe the solution you'd like

Describe alternatives you've considered
No response
Additional context
No response
Are you working on this?
Yes

@tanmoysrt is it ok to have this feature ?

I would like to work on this could you assign this to me


[Improvement][Spark] Support Local Spark Cluster

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
When a Spark Task executes spark-submit, the 'cluster' and 'client' deploy maps to --master yarn or --master k8s://.... I would like option to use a local Spark cluster. In other words, the equivalent spark-submit option is:--master spark://<hostname>:<port>
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I would like to have a try on this issue.

Current workaround for me is to pass --master ... --deploy-mode cluster in the extra options. Since spark-submit will use the last values, this will send task to local cluster. For example look at this log which has my own --master option which overrides Dolphin --master local:
[INFO] 2024-02-02 14:27:38.934 +0000 - Final Shell file is :
#!/bin/bash
BASEDIR=$(cd `dirname $0`; pwd)
cd $BASEDIR
export SPARK_HOME=/opt/spark-3.5.0-bin-hadoop3
export JAVA_HOME=/usr/lib/jvm/java-11-openjdk-amd64
${SPARK_HOME}/bin/spark-submit --master local
--class com.example.monitor.ScanMonitor --conf spark.driver.cores=1 --conf spark.driver.memory=512M
--conf spark.executor.instances=2 --conf spark.executor.cores=2
--conf spark.executor.memory=2G
--master spark://devel:7077 --deploy-mode cluster
file:/opt/apache-dolphinscheduler-3.2.0-bin/standalone-server/files/default/resources/monitor-0.1-jdk11.jar producer
...
24/02/02 14:27:54 INFO ClientEndpoint: Driver successfully submitted as driver-20240202142754-0003
2024-02-02 14:28:00.038 +0000 -  ->
24/02/02 14:27:59 INFO ClientEndpoint: State of driver-20240202142754-0003 is RUNNING
24/02/02 14:27:59 INFO ClientEndpoint: Driver running on <IP_ADDRESS>:35595 (worker-20240202141308-<IP_ADDRESS>-35595)
24/02/02 14:27:59 INFO ClientEndpoint: spark-submit not configured to wait for completion, exiting spark-submit JVM.

still working

still working


Add launch option for controlling external console

This checkin adds a new launch option for controlling if an external console should be
used and modifies the external console support so that it can be used with CLRDBG.
Remaining work:

I am currently sometimes seeing an access violation which leads to a hang inside of
clrdbg during shutdown. This is inside of the JIT, so I want to try upgrading my CoreCLR
before investigating more.
Support OSX

@caslan @jacdavis please review

Assuming the default is true, this looks good. @andrewcrawley Please review as well.

LGTM


update: latte-dock 0.7.5 -> 0.8.0

Motivation for this change
Bringing the new version of latte-dock to nixos
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build latte-dock

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: latte-dock
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: latte-dock
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/sg2qdapzxs6ad1avl4y0jza0xyakcz7b-latte-dock-0.8.0
shrinking /nix/store/sg2qdapzxs6ad1avl4y0jza0xyakcz7b-latte-dock-0.8.0/bin/latte-dock
shrinking /nix/store/sg2qdapzxs6ad1avl4y0jza0xyakcz7b-latte-dock-0.8.0/lib/qt-5.11/plugins/plasma_containmentactions_lattecontextmenu.so
shrinking /nix/store/sg2qdapzxs6ad1avl4y0jza0xyakcz7b-latte-dock-0.8.0/lib/qt-5.11/qml/org/kde/latte/liblattedockplugin.so
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/sg2qdapzxs6ad1avl4y0jza0xyakcz7b-latte-dock-0.8.0/lib  /nix/store/sg2qdapzxs6ad1avl4y0jza0xyakcz7b-latte-dock-0.8.0/bin
patching script interpreter paths in /nix/store/sg2qdapzxs6ad1avl4y0jza0xyakcz7b-latte-dock-0.8.0
checking for references to /tmp/nix-build-latte-dock-0.8.0.drv-0 in /nix/store/sg2qdapzxs6ad1avl4y0jza0xyakcz7b-latte-dock-0.8.0...
postPatchMkspecs
postPatchMkspecs

cc @benley

Success on aarch64-linux (full log)
Attempted: latte-dock
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/0dswkgbxqv1rd13aqax5m3d2xxsn0r33-latte-dock-0.8.0
shrinking /nix/store/0dswkgbxqv1rd13aqax5m3d2xxsn0r33-latte-dock-0.8.0/bin/latte-dock
shrinking /nix/store/0dswkgbxqv1rd13aqax5m3d2xxsn0r33-latte-dock-0.8.0/lib/qt-5.11/plugins/plasma_containmentactions_lattecontextmenu.so
shrinking /nix/store/0dswkgbxqv1rd13aqax5m3d2xxsn0r33-latte-dock-0.8.0/lib/qt-5.11/qml/org/kde/latte/liblattedockplugin.so
strip is /nix/store/zrs21zqcchgyabjf4xfimncdq16njizc-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/0dswkgbxqv1rd13aqax5m3d2xxsn0r33-latte-dock-0.8.0/lib  /nix/store/0dswkgbxqv1rd13aqax5m3d2xxsn0r33-latte-dock-0.8.0/bin
patching script interpreter paths in /nix/store/0dswkgbxqv1rd13aqax5m3d2xxsn0r33-latte-dock-0.8.0
checking for references to /build in /nix/store/0dswkgbxqv1rd13aqax5m3d2xxsn0r33-latte-dock-0.8.0...
postPatchMkspecs
postPatchMkspecs


promotepolicy crd and controller for application migrate

What type of PR is this?
/kind feature
What does this PR do?
crd and controller for application migrate
Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

@duanmengkk please help reivew it

PLZ  add some e2e tests


[SPARK-42494] Add official image Dockerfile for Spark v3.3.2

What changes were proposed in this pull request?
Add Apache Spark 3.3.2 Dockerfiles.

Add 3.3.2 GPG key
Add .github/workflows/build_3.3.2.yaml
./add-dockerfiles.sh 3.3.2

Why are the changes needed?
Apache Spark 3.3.2 released.
https://lists.apache.org/thread/k8skf16wyn6rg9n0vd0t6l3bhw7c9svq
Does this PR introduce any user-facing change?
Yes in future, new image will publised in future (after DOI reviewed)
How was this patch tested?
Add workflow and CI passed

cc @viirya @HyukjinKwon

@viirya @HyukjinKwon Merged, thanks!


Allow to train policy on actual user messages

Right now the policy is trained on intents, the labels assigned to the incoming messages from NLU.
There are models we want to try out that would benefit from the additional information the whole message provides. The training data passed into the Rasa Core policy should contain the nlu user message.

I wonder if we should have an UnfeaturizedTrainingData class that holds everything as tuples, labels, which then gets pass to the Featurizer to turn it into X, y

I wonder if you can write stories also without any intents? What about if I just want to use NER and want to have stories with actions after recognized entities and no intents at all.
Altough I have the feeling that you are ""trapped" with intents, is it possible to be more flexible? Lets say you are just interested if the intent right now is anything else than a specific intent in the sate of conversation? This I want to use for fall-back strategies.

yes we're currently experimenting with how robustly this works!

Ok that would be amazing! But I want to check if I am correct. So far, it is possible to write such custom behaviour (like check if intent is anything else than a specific intent after a specific ation or intent) in the policy's probabilities_using_best_policy function. There I can execute a specific action for my situations with this return value like:
return utils.one_hot(action, domain.num_actions)
Am I right?
Doing such things in a custom action is not possible or not a good way? Like executing a custom action inside a custom action? Still, I am asking myself how one cann call an action (like the restart action) inside a custom action?
That would be great if you can shortly reply.

The context of a chat often involves the last exchanged messages as well, not only the current message. Have you thought about doing a CNN that looks at the last X messages? Possibly one that gives greater weights to closer messages to predict next actions? Or doing a tensorflow_embedding across the last x messages (and not only the current message).
An additional idea would be to also include in this CNN the utterances of the bot (which can be highly condensed unlike unstructured user chats, so adding bot msgs to this CNN wouldn't add a lot of dimensionality)  - what a user says is often a response to what a bot said.

@andreasstuber do you speak about NLU or core? In rasa_core we use RNN for the functionality you describe


Add Scroll to last/first table row UI options to table view toolbar

see Tabulator scroll to row API: http://tabulator.info/docs/5.0/navigation#scroll-row

added scroll to first/last row UI:

closing this. will add Rows dropdown with scroll to select row options in #45.


updated ui in page title and body

updated ui in page title and body

Unwanted files added to this ,closing it


Non-issue

deleted

Next time, please don't delete the issue text. Comment that it was fixed, and if possible the solution as well. It will help others if they see it.


examples/windowsize: CursorModeCaptured doesn't work on Safari

Probably requestPointerLock should be invoked from a user-interaction event handler like 'click'.

https://whatpr.org/html/3851/interaction.html#tracking-user-activation
I guess Safari does not implement this yet.


Some interpolation leads to "Cannot read property 'range' of null" error

What version of this package are you using?
14.3.4
What operating system, Node.js, and npm version?
On macOS:
node -v
v14.1.0
npm -v
6.14.4

What happened?
Given foo.js:
function click(selector) {
  return () =>
    document.querySelector(`#app ${selector}`)?.click()
}

This error came back:
$ yarn run standard foo.js
yarn run v1.22.4
$ /Users/justin/code/searls/kamesame/node_modules/.bin/standard foo.js
standard: Unexpected linter output:

TypeError: Cannot read property 'range' of null
Occurred while linting /Users/justin/code/searls/kamesame/foo.js:3
    at SourceCode.getTokenBefore (/Users/justin/code/searls/kamesame/node_modules/eslint/lib/source-code/token-store/index.js:298:18)
    at checkSpacingBefore (/Users/justin/code/searls/kamesame/node_modules/eslint/lib/rules/template-curly-spacing.js:60:42)
    at TemplateElement (/Users/justin/code/searls/kamesame/node_modules/eslint/lib/rules/template-curly-spacing.js:119:17)
    at /Users/justin/code/searls/kamesame/node_modules/eslint/lib/linter/safe-emitter.js:45:58
    at Array.forEach (<anonymous>)
    at Object.emit (/Users/justin/code/searls/kamesame/node_modules/eslint/lib/linter/safe-emitter.js:45:38)
    at NodeEventGenerator.applySelector (/Users/justin/code/searls/kamesame/node_modules/eslint/lib/linter/node-event-generator.js:254:26)
    at NodeEventGenerator.applySelectors (/Users/justin/code/searls/kamesame/node_modules/eslint/lib/linter/node-event-generator.js:283:22)
    at NodeEventGenerator.enterNode (/Users/justin/code/searls/kamesame/node_modules/eslint/lib/linter/node-event-generator.js:297:14)
    at CodePathAnalyzer.enterNode (/Users/justin/code/searls/kamesame/node_modules/eslint/lib/linter/code-path-analysis/code-path-analyzer.js:634:23)

If you think this is a bug in `standard`, open an issue: https://github.com/standard/standard/issues
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Changing from ${selector} interpolation to just passing selector resolves the error.

Thanks for the bug report. I can't reproduce this issue with a foo.js containing just the code you posted.
@searls Can you try again using standard 15.0.0 which was released two days ago? There's a decent chance that the bug is fixed in that release?

Huzzah! 15 works. Great job as always, @feross & friends 💖


Implement search backend

Reimplement the current search:
First search catalog number :

the catalog number is composed of chunks of letters and numbers
every chunk of letters/number must be matched exactly
Letters and numbers can be either in a single string or separated by any character: PD014B is the same as PD 014B,  PD 014/B etc

If the catalog number is matched exactly, only return the result for that match and other partial matches on the cat num (if any)
If the catalog number is matched partially, return first the partial cat num matches, then return matches on title and description (have to check exactly which fields)
If the catalog number is not matched, search in other fields
Additional desirata: in the case that the cat num is not matched, use the number of citations as part of the sorting function (the higher the citations, the higher the rank)

@bandrow @masonpairish could please validate the assumptions above?

This is essentially correct.
If the catalog number is not matched, then return records if the query matches any visible or invisible field.
Please bold the match in the search result in each record.
Treat "clone ID" field breaking up the strings in the index by number/letter/special characters/combinations thereof for the index. Do not show these fields, but use them for search.

@bandrow there is one behaviour in the chunks interpretation in the catalog number which we need to validate with you.
The space seems to be treated differently from other characters. For instance, m-d1//40?3/24 does match MD-14-0324 but m d1//40?3/24 does not.
So the consequence is that chunks are either separated by a space or letter/number implicit separation, while other characters are simply ignored and cannot be used to separate chunks, right?

I don't think that space should be treated differently.
The users may use space instead of dash or another character. I have seen
many times the use (in scientific papers) of space instead of : for the
RRID.
Example RRID:AB_90755 vs RRID AB_90755.
I don't know how often this happens with catalog numbers in search, but it
would stand to reason that people searching for catalog numbers would treat
them similarly.
On Fri, Sep 16, 2022 at 3:58 AM Filippo Ledda @.***>
wrote:

@bandrow https://github.com/bandrow there is one behaviour in the
chunks interpretation in the catalog number which we need to validate with
you.
The space seems to be treated differently from other characters. For
instance, m-d1//40?3/24 does match MD-14-0324 but m d1//40?3/24 does not.
So the consequence is that chunks are either separated by a space or
letter/number implicit separation, while other characters are simply
ignored and cannot be used to separate chunks, right?
—
Reply to this email directly, view it on GitHub
https://github.com/MetaCell/scicrunch-antibody-registry/issues/52#issuecomment-1249224803,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNSWNKUK5Y7WYBTW7XGOU3V6RHEJANCNFSM6AAAAAAQIZ7NYY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
All key biological entities deserve an #RRID!
orcid.org/0000-0002-5497-0243

@bandrow I also expected the space to be just a separator like others but it definitely treated differently from other characters in the current implementation and that currently really makes a difference in search.
Now the question is whether the correct behaviour should be the current space behaviour (chunk separator) or other non alphanumeric character behaviour (ignored).

Wait all other chars are ignored and the cat num is stitched not chunked?
That will be bad. Please chunk all char differences so that cases like this
can happen. This is very common:
Search for 9101s should bring back 9101  match
Search for somestring-100 should respond when searching for somestring
If I understand the behavior you are talking about i think that
somestring-100 will search for somestring100 however that does not match
data and will not bring back results from somestring
I will try to track down a few recent tickets about this in elastic search
if useful.
On Mon, Sep 19, 2022 at 1:12 AM Filippo Ledda @.***>
wrote:

@bandrow https://github.com/bandrow I also expected the space to be
just a separator like others but it definitely treated differently from
other characters in the current implementation and that currently really
makes a difference in search.
Now the question is whether the correct behaviour should be the current
space behaviour (chunk separator) or other non alphanumeric character
behaviour (ignored).
—
Reply to this email directly, view it on GitHub
https://github.com/MetaCell/scicrunch-antibody-registry/issues/52#issuecomment-1250703157,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNSWNJW6KXWX6ZTXUSO3UDV7AN6JANCNFSM6AAAAAAQIZ7NYY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
All key biological entities deserve an #RRID!
orcid.org/0000-0002-5497-0243

somestring-100 is the same of somestring100 in my tests and it's chunked correctly as letter and numbers are implicitly chunked.
The real difference comes in place with cat numbers like 18-272-195033-0.05. If I got it right it's chunked in the database but the search works differently.  If I search for 1827/2195033005 I get the above cat num, I don't find anything for 1827<PHONE_NUMBER>

Ok gotcha those types of cases are more rare, but we should chunk at all
delimiters so it will be brought back correctly.
I probably don't need to say this, but the longer chunks should be on top
in search results.
On Mon, Sep 19, 2022 at 7:01 AM Filippo Ledda @.***>
wrote:

somestring-100 is the same of somestring100 in my tests and it's chunked
correctly as letter and numbers are implicitly chunked.
The real difference comes in place with cat numbers like
18-272-195033-0.05. If I got it right it's chunked in the database but
the search works differently. If I search for 1827/2195033005 I get the
above cat num, I don't find anything for 1827<PHONE_NUMBER>
—
Reply to this email directly, view it on GitHub
https://github.com/MetaCell/scicrunch-antibody-registry/issues/52#issuecomment-1251062301,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNSWNMRR3GXW6UATTEXTJ3V7BW3LANCNFSM6AAAAAAQIZ7NYY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
All key biological entities deserve an #RRID!
orcid.org/0000-0002-5497-0243

Not really following here: as chunks are matched exactly how can we have a longer or shorter match? Maybe when more than one possible chunk is searched (now with spaces)?
I see how that applies to the free text search in other fields when chunks are not found though.

Hi @bandrow,
I'm trying different strategies to implement effectively the first part of the match (the one over the catalog_num and the cat_num).
Yesterday with @zsinnema, we continued to experiment a little bit, and we are a little bit puzzled by exactly how the search for the exact match of the catalog_num should work exactly.
Here is some cases that I identified and where I'm unsure exactly of if some search should match or not regarding what have been said in this thread.
For entry: 18-272-195033-0.05 mg
Matching search:<PHONE_NUMBER>30.05, 18<PHONE_NUMBER>.05, 18 272-195033 0.05
Non-matching search: 18 272-195033-0.05 mg, 18 272-195033-0.05, 18<PHONE_NUMBER>.05 (and other searchs). Should they match?
Also, regarding the "ranking", why the search 18 272-195033 returns first 18-272-195033-0.05 mg then 18-272-195033 while 18-272-195033 returns first 18-272-195033 then 18-272-195033-0.05 mg? The search 18 272-195033 looks like it should return first 18-272-195033 as it's an exact match?
For entry: MD-14-0324
Matching search: MD140324, MD 14-0324, MD 14 0324, MD14 0324, MD 14 03/2/4, MD1-40324, M-D140324
Non-maching search: M D140324 (every search that cuts with a space between a chunk). Is there a real difference for the search between M D140324 and M-D140324, or should they return the same result (currently, they don't)?
For entry: MA1-81990
Matching search: MA 1-81990, MA 1 81990, MA 181990, M-A-1-8-1-9-9-0. Why all those search are matching?
Non-matching search: MA 1 8 1990, MA 1 8 1990 (cuts with a space in a number chunk)
Could it be that there exists two catalog nums like this: AB-12-34 and AB1-234?
If that's the case, should the search AB1234 match both? Same with A-B12-34?
The question is more, have the chunks really a special meaning for the search, or could we consider removing all non alphanum chars for the search and for indexing the catalog_num?
Thanks for your help and sorry for all the questions, I want to be sure I'm not missing something important.

For entry: 18-272-195033-0.05 mg
Matching search:<PHONE_NUMBER>30.05, 18<PHONE_NUMBER>.05, 18 272-195033 0.05
Non-matching search: 18 272-195033-0.05 mg, 18 272-195033-0.05, 18<PHONE_NUMBER>.05 (and other searches). Should they match?

yes all variations of the string to string should match the record. The idea with this enhancer is to bring back records when the author does not type in the correct syntax.

Also, regarding the "ranking", why the search 18 272-195033 returns first 18-272-195033-0.05 mg then 18-272-195033 while 18-272-195033 returns first 18-272-195033 then 18-272-195033-0.05 mg? The search 18 272-195033 looks like it should return first 18-272-195033 as it's an exact match?

Yes the exact match needs to be returned first.
I think I see where my "longest first" is causing some misunderstanding. Sorry about this, I should have specified longest match first (so ab search should return record ab, then record a or record b or record abc). The exact match should always be first, if exists. If other records are presented we need some additional rules.
When no exact match, or when results include many results, we should sort by number of mentions (list of PMIDs) (this will ensure that the popular antibodies will get high positions in search)
when records do not have mentions and you do not have exact match then some other rule should be implemented; what it is I am not totally sure, but I have seen approximate matches do reasonably well. People will not just be searching for catalog numbers so we can't write all rules based on that. Any match in any column should be returned.

For entry: MD-14-0324
Matching search: MD140324, MD 14-0324, MD 14 0324, MD14 0324, MD 14 03/2/4, MD1-40324, M-D140324
Non-maching search: M D140324 (every search that cuts with a space between a chunk). Is there a real difference for the search between M D140324 and M-D140324, or should they return the same result (currently, they don't)?

This is an implementation artifact for the current system, but the behavior is based on how people search. While many people will see letters and numbers as distinct, they rarely split strings between letters in mixed strings. I think to save space we chunked at the places letter to number or letter to delimiter segments.

For entry: MD-14-0324
Matching search: MD140324, MD 14-0324, MD 14 0324, MD14 0324, MD 14 03/2/4, MD1-40324, M-D140324
Non-maching search: M D140324 (every search that cuts with a space between a chunk). Is there a real difference for the search between M D140324 and M-D140324, or should they return the same result (currently, they don't)?

example same as above (we implemented the breaks at the delimiters that we found were used by users more often)

Could it be that there exists two catalog nums like this: AB-12-34 and AB1-234?
If that's the case, should the search AB1234 match both? Same with A-B12-34?

Yes that is possible, but rare. There is no problem matching both catalog number stings and marking them as exact match. Catalog numbers are created by different companies and can collide. The results, should include both sting matches.

The question is more, have the chunks really a special meaning for the search, or could we consider removing all non alphanum chars for the search and for indexing the catalog_num?

that is a fine solution just please be aware of what may happen in matching when non-alphanumeric characters are used in the query. If you imagine that I will search for something like "alpha-proteinX" with the greek letter alpha and you strip that greek letter from search by default then results will be of lower quality. I can defer to your team on this, but we may need to rethink this at some point.

Ok so sorry I thought my note might be cryptic. Apologies.
I am not sure what would happen to search results in this case:
Ok so if user types in XXX YYY ZZZ
data is:
Row1 XXX-YYY-ZZZ (0 papers)
Row2 XXX-YYY (0 papers)
then response should include Row1 before Row2.
but if they type in XXX YYY
then Row2 should be top result.
So the longest match should be used as a criterion for search result
ranking.
Hope that makes more sense.
anita
On Mon, Sep 19, 2022 at 7:47 AM Filippo Ledda @.***>
wrote:

Not really following here: as chunks are matched exactly how can we have a
longer or shorter match? Maybe when more than one possible chunk is
searched (now with spaces)?
I see how that applies to the free text search in other fields when chunks
are not found though.
—
Reply to this email directly, view it on GitHub
https://github.com/MetaCell/scicrunch-antibody-registry/issues/52#issuecomment-1251127212,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNSWNN7SZ7FQH6W47YWPSDV7B4IVANCNFSM6AAAAAAQIZ7NYY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
All key biological entities deserve an #RRID!
orcid.org/0000-0002-5497-0243


Use greenkeeper to keep dependencies up to date

https://greenkeeper.io/

Greenkeeper raised PR #110

The json clients file is only temporary until proper client definitions are implemented.  Probably easier to rename config.json.sample back config.json with a sample config, and then freeze the file from future updates.

greenkeeper broken and can't seem to fix it.  Just use yarn outdated/yarn upgrade


v0.21

Can we have an update to 0.21? I want to use the new extends feature.

Done, btw with brackets-npm-registry doing reinstall should be enough because dependencies in brackets-eslint are configured to install the latest version available.

thanks! but you hardcode the version right? https://github.com/zaggino/brackets-eslint/blob/master/package.json#L29

Ah you're right, it shouldn't be ^0.21.0 but ^0.21 or something like that. Maybe I'll just put latest in there.

http://semver.npmjs.com/ ;) I tend to use .x notation like 1.x.x, but in this case you would go with ^0, this exludes alpha and dev versions and doesn't auto switch to the soon to come v1.0.0

or 0.x.x whatever you suits you

Yes, I'll fix that ;)

hmm I'm having an issue where the plugin from the base config which a extend doesn't work. I tested it with command line and that seems to pick it up. Probably has to do with how you detect plugins.

I'll need more info on that, which plugin, config and a sample of code where it should show an error :)

Base config is from Lab https://github.com/hapijs/lab/blob/master/lib/linters/eslint/.eslintrc with eslint-plugin-hapi https://github.com/cjihrig/eslint-plugin-hapi
Code sample:
var f = function() {
    console.log('a');
};

Should give warning that an empty line is needed after function like
var f = function() {

    console.log('a');
};

I do have the plugin globally but that probably doesn't work for plugins + extends feature

I've published 0.21.1 and now it's working fine for me:

only thing I had to do to test in my project was:
npm install -g eslint-plugin-hapi
npm install lab

and added this to .eslintrc config:
extends: "./node_modules/lab/lib/linters/eslint/.eslintrc"

nice, thanks, everything works now! thanks for the efford


Add workflows to repo

Add workflows to repo
in .github/workflows like our previous repo

We already have these files


TypeError: Cannot read properties of null (reading 'value')

Fixes #154757

Checked with @jrieken on what is normal to do for these checks and he pointed me to
https://github.com/microsoft/vscode/blob/main/src/vs/workbench/api/common/extHostLanguageFeatures.ts#L1419
which uses console.log.


Update GitHub upload-artifact action

This PR updates and migrates upload-artifact github action version from v3.1.3 to v4.2.0
With version 4 artifacts are immutable so consequent uploads with the same artifact name fail.
The artifact's name changes to be unique. Also, to combine all artifacts merge-upload job is added.
This job downloads, merges, and uploads the merged artifact. All temporary artifacts are deleted.
Fixes: #30396

/test

/ci-aks

/ci-awscni

/ci-eks

/ci-external-workloads

/ci-gke

/test

Two failing tests also show the collection of cilium-sysdumps.

/test

There was a problem with some of the workflows that a failed job may not produce a sysdump
In this case, the Merge Sysdump steps fail. Just added continue-on-error: true so even there are not sysdumps steps do not fail.

/test

@brlbil just a heads up that I was unable to backport this to v1.12 because of the conflicts, so I added the backport-author label.


Heap Memory Consumption high during continues ETL updates

Our ETL process updating the DB continuously ( for more than 36 hours now) using Http REST APi . This process supposed to run 24 * 7 . TOday we observed that Orient process taking 98 % of heap size resulting in not accepting any other connections to DB ( like we are trying to connect DB using Binary protocol) , We tok the thread dump and identified that so many threads are in blocked state. This is very high priority . Unable to attach the thaed dump here so will send in mail.

Hi All,
I'm helping @luigidellaquila  on the investigation on the stack traces you sent to us, the good point is that we didn't find any deadlock, there are a lot of threads blocked but most of them are part of pool and are waiting for get some data from a queue or something like that, this is quite normal on a massive insert, all the thread pool are stretched to be able to elaborate all the data, and they may be in status blocked while getting more data. I saw other traces in status running or blocked by a running thread, so nothing wrong there.
So the fact that the server is not accepting more connection it doesn't seems related on the number of blocked thread.
For understand what is going on, i would like to know what kind of error you got on the server log when it was not accepting connection anymore, I'm thinking there is a memory related issue.
It can be very useful have a memory dump of the server process together with the sever log, that will allow us to have a complete picture.
In the meanwhile we will try to run some massive import test over http for try to reproduce the problem.

Unfortunately , We didn't collected memeory dump and error messages we got while connecting during issues we olst as we resterted process to reduce the outage time. We are also trying to reproduce the issue . Once we reproduce the issue , We will provide all the details.

hi @SatishAMP,
Thank you for your help, we will continue our investigation in parallel as well, one question that may help to delimit the case: do you use server side function in the import procedure?

Can you please explain what is Server Side function means ? We are not aware of that terminology.

Hi Satish,
a server side function is a custom Java or Javascript function defined in
the server that can be used in queries or as a REST service.
http://orientdb.com/docs/last/Functions.html
http://orientdb.com/docs/last/SQL-Functions.html#custom-functions-in-java
As far as I can remember, you are not using such a feature in the ETL, but
I don't know about the rest of the application.
Regards
Luigi
2015-05-21 16:06 GMT+02:00 SatishAMP<EMAIL_ADDRESS>
Can you please explain what is Server Side function means ? We are not
aware of that terminology.
—
Reply to this email directly or view it on GitHub
https://github.com/orientechnologies/orientdb/issues/4185#issuecomment-104287837
.

Hi , Yes your are correct , We are not using any custom java code to make RESTFul API call

@SatishAMP What's your configuration? How much total memory do you have on the machine and how much memory did you reserve to OrientDB (look in bin/server.sh file, last line)?

hi @SatishAMP,
I did some benchmark, and i produced one fix (this one: https://github.com/orientechnologies/orientdb/commit/76368218da160ba3d4fcf90588354b5c3a689b4f) that should reduce the overload of the closed connections, in this way if the database goes out of memory it will recover in a easier way.

@tglman Could you please backport this on 2.0.x (the Ameriprise version is using) ?

it's already backported.

Closed after no answer. In case please reopen this issue.


Properly link to the github issues page

Observed behavior
When you try to submit a new issue to the deprecated issue tracker ( http://code.opencv.org/projects/opencv/issues/new ), you just get a "you are not authorized" error.
Intended behavior
The link to the new github issue tracker should be shown

:+1:


Injecting Factory in $onAuth

Informations

Type
Version

Question
2.6.4

Description
In my $onAuth, I need to use a Factory. This factory contains the Firebase Admin API. With that, I check whether or not the user is authenticated.
How would I go about doing that the best way? Currently I'm just grabbing the InjectorService and getting the factory through there, but I feel like that's not how it's supposed to be done.
Is there a better way of doing this?

Normally you can do that:
@Inject()
$onAuth(@Inject(MyFactory) myFactory: MyFactory ) {

}

See you :)

I've tried that:
@Inject()
public $onAuth(@Inject(Firebase) firebase: Firebase) {}

But when I do that, I get:
TypeError: Cannot read property 'prototype' of undefined
    at Object.getClass (/Users/vincent/Code/Nodejs/x/node_modules/ts-express-decorators/src/core/utils/index.ts:23:19)
    at Function.Metadata.set (/Users/vincent/Code/Nodejs/x/node_modules/ts-express-decorators/src/core/class/Metadata.ts:424:34)
    at Function.Metadata.setParamTypes (/Users/vincent/Code/Nodejs/x/node_modules/ts-express-decorators/src/core/class/Metadata.ts:371:21)
    at /Users/vincent/Code/Nodejs/x/node_modules/ts-express-decorators/src/di/decorators/inject.ts:19:26
    at /Users/vincent/Code/Nodejs/x/dist/Server.js:12:37
    at DecorateProperty (/Users/vincent/Code/Nodejs/x/node_modules/reflect-metadata/Reflect.ts:1102:31)
    at Object.decorate (/Users/vincent/Code/Nodejs/x/node_modules/reflect-metadata/Reflect.ts:279:20)
    at __decorate (/Users/vincent/Code/Nodejs/x/dist/Server.js:4:92)
    at Object.<anonymous> (/Users/vincent/Code/Nodejs/x/dist/Server.js:43:1)
    at Module._compile (module.js:624:30)

Have declared firebase as a factory ?
import * as Firebase from "firebase";

export interface Firebase {
}
export const Firebase = Symbol("Firebase");

FirebaseAdmin.ts:
import {InjectorService} from 'ts-express-decorators';
import * as firebase from 'firebase-admin';
import * as fs from 'fs';

export interface IFirebase {
    getAdmin(): firebase.app.App;
}

export type Firebase = IFirebase;
export const Firebase = Symbol('Firebase');

const admin = firebase.initializeApp({
    credential: firebase.credential.cert(
        JSON.parse(fs.readFileSync(
            './service_account.json',
        ).toString()),
    ),
});

InjectorService.factory(Firebase, {
    getAdmin: () => admin,
});

I think in your case, @OverrideMiddleware can to be the right way:
@OverrideMiddleware(AuthenticatedMiddleware)
export class FirebaseAuthMiddleware implements IMiddleware {

    constructor(private serverSettingsService: ServerSettingsService) { }

    public use(@EndpointInfo() endpoint: EndpointMetadata,
               @Request() request: Express.Request,
               @Response() response: Express.Response,
               @Next() next: Express.NextFunction) {

        const options = endpoint.get(AuthenticatedMiddleware) || {};
        if (!Firebase.isAuth){ // something like that ;)
           next(new Forbidden("Forbidden"));
        }
        next();
    }
}

I'll try this out in a bit :)

Be aware, if you use the OverrideMiddleware decorator on AuthenticathedMiddleware, the $onAuth hook in your server will be avoided ;)


Fix logo on readme

See: https://github.com/thoughtbot/bourbon/issues/937

Ah, yes we should fix this! Thanks, Will!

Merged as b7d1619afe42104d3116bd08f0b2a0eb76f9867f.


Calculated and adjusted calculated values are mixed up when viewing a project [8]

Describe the Bug:
When viewing a project that has adjusted values, the adjusted label doesn't show up, and the adjusted value shows up for the non-adjusted label:

(these are the values for those fields in the form:
A starting point might be app/lib/theme/widgets/ReadOnlyAdjustableCalculatedValueWidget.tsx or the other adjustable calculated value widgets. Field labels are normally handled by app/lib/theme/widgets/FieldLabel.tsx, but since these widgets show two values, they need to get a second label somehow, and I think that's the problem. Could also consider refactoring these widgets because the second label is causing display and accessibility issues too.
Probability (how likely the bug is to happen, scored from 1-5): 5
(For example, probability of 5 is something like "it happens to all users every time they log in."
Probability of 1 "only happens to certain users when a really specific and unlikely path is followed.")
Effect (how bad the bug is when it does happen, scored from 1-5): 3
(For example, effect of 5 is "the entire app crashes and makes it unusable for all users" or "the bug causes the wrong data to be saved, with critical information (e.g. payment) being affected."
Effect of 1 is "It makes some styling look a little bit weird.")
Steps to reproduce the behaviour:
Screenshots:
Additional information:

OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]
Additional context
)

Is this only on the milestone report?


CORS doesn't work?

My config:
return [
    'supportsCredentials' => true,
    'allowedOrigins' => ['https://foo.example.top/'],
    'allowedOriginsPatterns' => [],
    'allowedHeaders' => ['*'],
    'allowedMethods' => ['*'],
    'exposedHeaders' => [],
    'maxAge' => 0,
];

Laravel version: 5.6.
Sever: nginx.
Anyway I got

No 'Access-Control-Allow-Origin' header is present on the requested resource.

My request is being sent from foo.example.com to bar.buz.example.com. Two same subdomains.
By the way it works without problems when sending from foo.example.com to buz.example.com.
What can be wrong?

Update.
Having HandleCors::class, inside $middleware array results 403 ("No 'Access-Control-Allow-Origin' header is present on the requested resource.").
Having \Barryvdh\Cors\HandleCors::class, inside $middlewareGroups (I use for api routes) results 500 ("Cross-Origin Read Blocking (CORB) blocked cross-origin response").

Mee too。 laravel5.7.*

I have the same problem.
@Tarasovych could you solve it?

@StudyForZX ur solution is not clear...

@StudyForZX CORS headers won't be attached for 5xx responses, just FYI.

Mee too。 laravel5.7.*
SOLVED, Because use die();

@StudyForZX how were you able to solve it?

Mee too。 laravel5.7.*
SOLVED, Because use die();

@StudyForZX how were you able to solve it?

I use the die function to return api data.This has side effects on CORS middleware.Now I'm using echo.

I'm closing this due to inactivity. If this issue is still relevant on the latest version (2.0.1 at the moment), please open a new issue with detailed information.


[MIRAI] Add annotations to accumulator crate

Motivation
This adds documentation to the code and helps drive the diagnostics produced by MIRAI to zero so that newly introduced problems are easier to find.
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
No executable code has been added. Code builds. MIRAI is happy with it.

@bors-libra r+

:pushpin: Commit e24665d has been approved by wqfish

:hourglass: Testing commit e24665d32db727baf5c68943769852f706bf4038 with merge 350c4b31cdc342f40bd22ed3fd7855f328309502...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: wqfish
Pushing 350c4b31cdc342f40bd22ed3fd7855f328309502 to master...


Placing Defect Detection Eval and Test Files from CODEXGLUE

Implemented the eval scripts and the test scripts for the task : Defect Detection

Hi @Rohan-Nandula,
Thank you for your interest in contributing to SEVAL! We only need to evaluation script, the test.jsonl, and the predictions file from the CodeXGLUE repository. The other files are for the model and we are only interested in the evaluator. I also ask that you make your readme in this format. It seems like you have used the readme from the CodeXGLUE repository and this again includes information about the model which we do not need here.
We also are requiring that you write your own test script like the one found here. This will use your test and predictions file to check that the output of the evaluator is correct.
If you have any questions feel free to drop a comment.
Thanks!

Hey Mitchell,
Thank you for pointing out the issues. I will try to rectify them as soon
as possible.
Best,
Rohan N
On Mon, Feb 5, 2024 at 1:39 PM Mitchell Huggins @.***>
wrote:

Hi @Rohan-Nandula https://github.com/Rohan-Nandula,
Thank you for your interest in contributing to SEVAL! We only need to
evaluation script, the test.jsonl, and the predictions file from the
CodeXGLUE repository. The other files are for the model and we are only
interested in the evaluator. I also ask that you make your readme in this
https://github.com/SEEDGuard/SEVAL/blob/main/src/tasks/code_generation/README.md
format. It seems like you have used the readme from the CodeXGLUE
repository and this again includes information about the model which we do
not need here.
We also are requiring that you write your own test script like the one
found here
https://github.com/SEEDGuard/SEVAL/blob/main/src/test/code_generation/test.py.
This will use your test and predictions file to check that the output of
the evaluator is correct.
If you have any questions feel free to drop a comment.
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/SEEDGuard/SEVAL/pull/17#issuecomment-1927779403, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AT5AB7KAK4RPCXUSBW4RPS3YSERPBAVCNFSM6AAAAABCXMQMPKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRXG43TSNBQGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

The defect detection algorithm has now been implemented so I am closing this pull request. Feel free to contribute to one of the other tasks!


Update Multi-Tenant support to incorporate Arch Board Feedback

Allow multi-tenant authentication by default
Remove allowMultiTenantAuthentication option from TokenCredentialOptions
Change compat environment variable AZURE_IDENTITY_ENABLE_LEGACY_TENANT_SELECTION to AZURE_IDENTITY_DISABLE_MULTITENANTAUTH
Block multi-tenant auth when credentials are configured with tenant id adfs

We never had AZURE_IDENTITY_ENABLE_LEGACY_TENANT_SELECTION, so for us is about adding support to AZURE_IDENTITY_DISABLE_MULTITENANTAUTH.

@sadasant Is this for the October release? Can you please set the appropriate milestone?

@ramya-rao-a Yup. October release. Updated! Thanks!


feat(handler): Server and environment agnostic handler

Breaking changes

Core of graphql-sse is now server agnostic and as such offers a handler that implements a generic request/response model
Handler does not await for whole operation to complete anymore. Only the processing part (parsing, validating and executing)
GraphQL context is now typed
Hook arguments have been changed, they're not providing the Node native req/res anymore - they instead provide the generic request/response
onSubscribe hook can now return an execution result too (useful for caching for example)
Throwing in onNext and onComplete hooks will bubble the error to the returned iterator

Migration
Even though the core of graphql-sse is now completely server agnostic, there are adapters to ease the integration with existing solutions. Migrating is actually not a headache!
Beware that the adapters don't handle internal errors, it's your responsibility to take care of that and behave accordingly.
http
import http from 'http';
- import { createHandler } from 'graphql-sse';
+ import { createHandler } from 'graphql-sse/lib/use/http';

// Create the GraphQL over SSE handler
const handler = createHandler({ schema });

// Create an HTTP server using the handler on `/graphql/stream`
const server = http.createServer((req, res) => {
  if (req.url.startsWith('/graphql/stream')) {
    return handler(req, res);
  }
  res.writeHead(404).end();
});

server.listen(4000);
console.log('Listening to port 4000');

http2
import fs from 'fs';
import http2 from 'http2';
- import { createHandler } from 'graphql-sse';
+ import { createHandler } from 'graphql-sse/lib/use/http2';

// Create the GraphQL over SSE handler
const handler = createHandler({ schema });

// Create an HTTP server using the handler on `/graphql/stream`
const server = http.createServer((req, res) => {
  if (req.url.startsWith('/graphql/stream')) {
    return handler(req, res);
  }
  res.writeHead(404).end();
});

server.listen(4000);
console.log('Listening to port 4000');

express
import express from 'express'; // yarn add express
- import { createHandler } from 'graphql-sse';
+ import { createHandler } from 'graphql-sse/lib/use/express';

// Create the GraphQL over SSE handler
const handler = createHandler({ schema });

// Create an express app
const app = express();

// Serve all methods on `/graphql/stream`
app.use('/graphql/stream', handler);

server.listen(4000);
console.log('Listening to port 4000');

fastify
import Fastify from 'fastify'; // yarn add fastify
- import { createHandler } from 'graphql-sse';
+ import { createHandler } from 'graphql-sse/lib/use/fastify';

// Create the GraphQL over SSE handler
const handler = createHandler({ schema });

// Create a fastify app
const fastify = Fastify();

// Serve all methods on `/graphql/stream`
fastify.all('/graphql/stream', handler);

fastify.listen({ port: 4000 });
console.log('Listening to port 4000');

TODO

[x] use/fetch adapter
[x] use/http adapter
[x] use/http2 adapter
[x] use/express adapter
[x] use/fastify adapter
[x] Update documentation
[x] Migration steps

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(Testimonial): adding new Testimonial Component

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-icons@5.7.5-alpha.fe8746.0@codecademy/gamut-illustrations@0.17.14-alpha.fe8746.0@codecademy/gamut-kit@0.5.6-alpha.fe8746.0@codecademy/gamut-labs@19.16.18-alpha.fe8746.0@codecademy/gamut-patterns@0.5.2-alpha.fe8746.0@codecademy/gamut-styles@11.6.4-alpha.fe8746.0@codecademy/gamut-tests@2.4.17-alpha.fe8746.0@codecademy/gamut@35.5.1-alpha.fe8746.0@codecademy/konamimojisplosion@0.2.48-alpha.fe8746.0@codecademy/markdown-overrides@0.5.60-alpha.fe8746.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://610c402af60a56008f360b2f--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@5.7.5-alpha.61704e.0@codecademy/gamut-illustrations@0.17.14-alpha.61704e.0@codecademy/gamut-kit@0.5.6-alpha.61704e.0@codecademy/gamut-labs@19.16.18-alpha.61704e.0@codecademy/gamut-patterns@0.5.2-alpha.61704e.0@codecademy/gamut-styles@11.6.4-alpha.61704e.0@codecademy/gamut-tests@2.4.17-alpha.61704e.0@codecademy/gamut@35.5.1-alpha.61704e.0@codecademy/konamimojisplosion@0.2.48-alpha.61704e.0@codecademy/markdown-overrides@0.5.60-alpha.61704e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://610d4ae01ec726f043b9c496--gamut-preview.netlify.appDeploy Logs


Update README.md

Thanks for your contribution. However, this behavior is intentional. If you see anything else, do feel free to open another pull request. Thanks so much :D


Add CI Instructions for Github Actions

Motivation for the change
I have started to test my dot files repository and found ZUnit to be the perfect candidate for that daunting task. I love the documentation and the philosophy behind it. I had trouble setting in up in my CI system of choice: Github Actions. You can find how I did it on my dot files repo: https://github.com/EduardoSimon/dotfiles/blob/main/.github/workflows/testCI.yml
I found interesting to share my findings.

Hi @molovo , should I close this PR?

@EduardoSimon, You can find an excellent example of github actions + zunit in the Zinit project.
Link to the Zunit code
Feel free to open a discussion or issue if you have any questions.
For transparency, I wrote 90% of the tests and am a Zinit maintainer.


Link Not Working

The left had Nav  "Azure Bot Service Documentation" > "Reference" > "Node.js SDK v3"  hyperlink has the address "https://docs.botframework.com/en-us/node/builder/chat-reference/modules/botbuilder_d.html" but when you click on it , it takes you to Azure Bot Service for v4.. What happened to the documentation for Node.js SDKv3??? please fix it!!!!

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

ID: 06eee0a7-4c80-ddae-9024-7e6dec0a789d
Version Independent ID: 768d536b-024a-3892-abf0-ab6008a376ea
Content: Bot Framework SDK for Node.js - Bot Service
Content Source: articles/nodejs/bot-builder-nodejs-overview.md
Service: bot-service
GitHub Login: @v-ducvo
Microsoft Alias: v-ducvo

Thanks for your feedback. We became aware that the V3 documentation was no longer in place and have an outstanding request to restore this information online. The team responsible for creating this build has acknowledged the issue and is working to restore the data. These links should once again work properly once the underlying documentation has been restored.


support ntile window function

Is your feature request related to a problem? Please describe.
PG definition:
ntile ( num_buckets integer ) → integer               Returns an integer ranging from 1 to the argument value, dividing the partition as equally as possible.

One example given by  @neverchanje :
SELECT MIN(n_search)
FROM (
  SELECT n_search, ntile(4) OVER (ORDER BY n_search) AS quartile
  FROM your_table
) subquery
WHERE quartile = 1;

Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

I guess we write another heap, e.g. percentile/ntile_heap, and then replace it with the top_n_heap used in batch's top_n and group_top_n executors
and then it can work?

in favor of #10038

in favor of #10038

Did they solve the same requirement? Notice that #10038 is an aggregation function while this ntile is a window function.
Btw, it's not clear whether this issue refers to streaming or batch?


Consolidate packages for actual use in projects

closes #245

This has been addressed by multiple previous PRs
#255
#254
#250
#249
#248
#247
#243


Remove the darga_migrations file from .gitignore

This file was removed in a0973007691d467c423fabe6f8274c64ae5de344 in favor of using git to retrieve the migrations from the darga branch when we run the spec.

@miq-bot add_label technical debt

👍


X trie deletes

Previous PR: https://github.com/Workiva/go-datastructures/pull/33
CODE REVIEW
Implements delete for the x-fast trie.
@beaulyddon-wf @alexandercampbell-wf @rosshendrickson-wf @ericolson-wf @stevenosborne-wf @tylertreat-wf @tannermiller-wf

+1

+1

+1


Some special words in URL paths break CSharp code generation

My API contains a path with the element method in it. See the minimum example below:
swagger: "2.0"
info:
  version: "1.0.0"
  title: "Minimum Method Example"
host: "localhost"
basePath: "/api"
schemes:
  - "http"
consumes:
  - "application/json"
produces:
  - "application/json"
paths:
  /method/my-endpoint:
    get:
      produces:
        - "application/json"
      responses:
        "200":
          schema:
            type: "string"

When I generate the code using
dotnet kiota generate --openapi minimum_example.yaml --language CSharp -o Client

This leads to a name conflict between the namespace ApiSdk.Method and the Method.Get enum that is used when building requests inside the generated RequestBuilder.
I would be happy to workaround this, but I did not yet find a way to influence the namespace names.

@fpoppinga thanks for your interest in kiota and for reporting this.
Simply adding an entry to the reserved names provider should do the trick. I didn't know that Method was a reserved keyword.
Do you happen to know whether it's only reserved for namespaces segments or is it also invalid for classes?

Just to clarify, the right thing to do would be to add method to the reserved types provider not the reserved names.
The issue is that in a given namespace (in this case ApiSdk) the compiler can't tell the difference between the Method Enum from kiota abstractions and the Method namespace.
This collection will be used to update both the namespace and model names.

Hi, thank you for the support, I appreciate it!
Is there a way to specify additional reserved types / reserved names via the CLI?

@baywet It is also a problem, when you have a model that is called Method. That is much easier to workaround though, because you can just rename the model in the OpenAPI spec.

No, the list is hard-coded.
Ok in this case you might have to add the reserved word in both providers.

So would you take a PR that adds the Method keyword to both providers? Or is this a change that users should do on demand for themselves?

I did some checks, and as far as I can see, adding the name to the CSharpReservedClassNamesProvider, which already seems to contain some similar collisions with names used by the generator.

Hi @baywet, I think there has been a regression. I added some thoughts to the PR #2951 that broke it, AFAICS.


Admin menu doesn't working

Hey, I have a problem. In config I have 4 groups and group: Operator and Moderator can't use the Report Admin Menu, can you help me please?
report.AdminGroups = { //put other non-admin groups here that should have access to the admin menu.
"operator,moderator,admin,headadmin",

You bunched the usergroups all together. You must put the usergroups in their own quotes then separated by commas. Ex: "operator", "moderator", "admin", "headadmin"

Its work! Thanks.


[10.x] fix Before/After validation rules

fixing https://github.com/laravel/framework/issues/49863
added test cases to make sure it is fixed
$v = new Validator($trans, ['x' => ['a' => ['v' => 'c']], 'y' => 'invalid'], ['x' => 'date', 'y' => 'date|after:x']);
$this->assertTrue($v->fails());

$v = new Validator($trans, ['x' => ['a' => ['v' => 'c']], 'y' => 'invalid'], ['x' => 'date', 'y' => 'date|before:x']);
$this->assertTrue($v->fails());

This breaks my date validation, I have:
        return [
            'end_date'   => ['nullable', 'date', 'after_or_equal:start_date'],
            'start_date' => ['nullable', 'date'],
        ];

Prior to this change, it is possible (and acceptable) to have a null start_date with a non-null end_date---the validator would still register that the end_date is "after" start_date. However, this PR causes that particular use case to fail.


Unused methods in ScaffoldGenerator

Discovered while fixing Rubocop complaints in #1685. Relevant sourcefile: lib/generators/rspec/scaffold/scaffold_generator.rb.
It seems that mock_file_name, should_receive, stub, stub_or_should_chain aren't used anymore, neither in code nor templates. Removing them causes no additional test failures.
Some research shows mock_file_name was useful back in 2-3-maintenance but not since.

Update: same for formatted_hash.

Removed in https://github.com/rspec/rspec-rails/pull/1685/commits/56b347ed06699bd219d1c5aca1bd9b93ed4c7014, to be closed with the linked PR.

Merged in e985c13c, closing.


Contour launches but nothing happens

Hi,
I build contour application in windows 10 2004 using vs2019 in x64 arch. When I launch contour application nothing happens.

Good morning. Many thanks for your interest in Contour. While this app is still in very active but also rather early development, I think I would need a little more information on order to reproduce.  Is this an up-to-date with indoww 10? And what branch gav you been building?
Best regards,
Christian.

Hi @christianparpart
I debugged the application and its throwing exception at return loadConfigFromFile((configHome() / "contour.yml").string(), _logger); in Config.cpp with the exception message :- Exception thrown at 0x00007FFDD4B23E49 in contour.exe: Microsoft C++ exception: YAML::ParserException at memory location 0x000000BB182FAF50.

@sonulohani
What's the contents of your contour.yml - did you modify it, or is it the one default-shipped with the binary?
What branch did you use to build contour? (NB: I'm also soon to merge wip into master branch).

Hi @christianparpart
I had just cloned your repository and followed README.md to build the source code. No modification in any source file.

Here is the yml file
contour.yml.txt
I dont understand whats the last line in yml.

That last line might be the issue, it's not valid YAML syntax. @christianparpart are you taking into account the windows CRLF?

That last line might be the issue, it's not valid YAML syntax. @christianparpart are you taking into account the windows CRLF?

Thanks for testing. Yes, the last line is indeed wrong. I am almost certain the yaml file is being generated with Unix like endings. I am more concerned about what the tail of your yaml file is, though.  I am checking ASAP.

Hi @christianparpart
Please see the default_config_yaml array in the generated contour_yaml.h.txt file. The ending part has some null values.
contour_yaml.h.txt

Update: https://github.com/christianparpart/contour/commit/5442d36734f4fd4cbc32216e3d363895fcee7fb4 should at least fix the  creating of the default contour.yml file on CRLF platforms (Windows).

Thanks for fixing the issue. I verify that the issue is fixed now.


Add guide for tools

I need to figure out where I'm going to draw the line between these guides and the concept documentation pages.

Could you add some clarity about difference between uvx --from and uvx --with?
I think they are in common in all ways except of --from is one-time usable and it declares entry points of which package should be gathered.
I can suggest to merge --with and --from into one argument (e.g. --from). By keeping it multiple-times usable it in cli, by default, main package should be inferred from first/last passed --from argument. Which also could be overridden by separate argument e.g. --main.
Examples:
# inferred first `--from` as main package, so `fastapi` executable would be available.
$ uvx --from fastapi-cli --from fastapi fastapi dev


official sqlite3 source code is C. why is node-sqlite3 written in modern C++, instead of C ?

Because yes.

can you explain why yse?

can you explain why yes?


Upgrade linkerd to 2.11.2

Fix #407

Currently use a older version of upstream so I can easily verify that this works as intended.


navigator move to new uORB::Subscription

Looks like the testing results ended up over here. https://github.com/PX4/Firmware/pull/12123#issuecomment-499263901


Synthesis produces incorrect logic (erroneously optimized to 0)

The attached file contains what's needed to reproduce the issue :
synth_bug.tar.gz
It was tested on latest oss-cad-suite build from 20210610 ( Yosys 0.9+4081 git sha1 1667ad65 ).
In the above package, the makefile will synthesize two different top level, wrapping the same module.
In one case the same signal is fed to both the submodule input and in the other case they are different.
The submodule logic is basically a AND gate, but written using a case (It's from the picorv32 sources, with as many things trimmed as possible to still show the bug).
When the two signals are separate, I expect an AND gate to be generated and that's the case. That's what in synth.good.v.
In the other case, it'd be the AND of 2 identicals signals and so this should be just a wire. And this is NOT the case ... looking in synth.bad.v you find assign pcpi_int_wr = 1'h0;  which is obviously wrong.
This  test case was narrowed down from the Hackaday Conference FPGA Badge  from 2019. The bitstream would build fine with the 2019 toolchain but would fail (it builds but the CPU misbehaves) with recent toolchain. The commit where it started breaking is the new opt dff pass merged in August 2020, however this is not what's actually causing the issue. This change just exposed it. The new DFF opt is just better at optimizing and instead of having 2 distinct FFs outputing to pcpi_wr and pcpi_ready, it realized it could use the same FF and this triggered this bug.  The final result in the SoC is that pcpi_int_wr is stuck to 0, causing latched_write to be 0 for PCPI instruction causing issues ... (this was confirmed in simulation and also on real hw with a logic analyzer looking at those signals).

Looking at intermediate steps, looks like opt_muxtree is where it goes wrong.
read_verilog mini_dut.v
hierarchy -top mini_dut_wrap_bad
flatten
proc
opt_expr
write_verilog -norename good.v
opt_muxtree
write_verilog -norename bad.v


#1708 - fixed floating visibility dropdown

What does this pull request aim to achieve? 💯
Fixes the issue with the floating dropdown menu contents when creating a post.
Please be concise and link any related issue(s):
https://github.com/FightPandemics/FightPandemics/issues/1708

[ x] I have checked that no one else is working on similar changes.
[ x] I have read the latest README and understand the development workflow.
[x ] The name of this branch is: feature/<branch_name>, or hotfix/<branch_name> if this is a hotfix, where <branch_name> briefly describes the issue(s) resolved and is prefixed with the issue number(s) (e.g. 1127-1130-update-git-branching-model). The branch is created in a cloned version of this repo, not a forked repo.
[ x] This branch is rebased/merged with the latest staging (or the latest production if it is a hotfix).
[x ] There are no merge conflicts.
[x ] There are no console warnings and errors.
[x ] On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".
[x ] The title describes the issue(s) being addressed using format: <issue_numbers> - <brief_issue_description> (e.g. #1127/#1130 - Update Git Branching Model)

Works as expected


output show current tab

i have a problem that i have 2 tab
i wanna output just show output of the current tab not all
tnx

+1, is there way to mark current tab active or smth like this?


ConfigMap not being re-apply with simple plan

What happened:
I have a plan where I want to reapply a pod and it’s configmaps, My steps are:

Delete a Pod
Delete a ConfigMap
Apply a ConfigMap
Apply the Pod
The problem is that step 3 doesn’t reapply my ConfigMap, so then step 4 never runs but yet my operator plan reports a full COMPLETED steps and plan. All phases and steps in my plan are set to serial.

What you expected to happen:
Step 3 above to reapply my ConfigMap
How to reproduce it (as minimally and precisely as possible):
Create two tasks for a Pod and ConfigMap, one to Apply and one to Delete the same Kube manifest.
Then a plan like this:
  redeploy-connectors:
    strategy: serial
    phases:
    - name: connectors-stop
      strategy: serial
      steps:
      - name: stop-connectors
        tasks:
        - connectors-stop
      - name: delete-bootsrap
        tasks:
        - connectors-bootstrap-delete
    - name: connectors-redeploy
      strategy: serial
      steps:
      - name: redeploy-connector-configmaps
        tasks:
        - connectors-bootstrap
      - name: restart-connectors
        tasks:
        - connectors-start

Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version): Client v1.17.0, Server v1.18.3
Kudo version (use kubectl kudo version): 0.15.0
Operator: Custom
Cloud provider or hardware configuration: minikube running in VirtualBox on Mac

@dboyleitrs could you also share the tasks definition, please?

Here are the task definitions:
  - name: connectors-bootstrap
    kind: Apply
    spec:
      resources:
        - connectors-configmap.yaml
  - name: connectors-bootstrap-delete
    kind: Delete
    spec:
      resources:
        - connectors-configmap.yaml
  - name: connectors-start
    kind: Apply
    spec:
      resources:
        - connectors-start-and-monitor-deployment.yaml
  - name: connectors-stop
    kind: Delete
    spec:
      resources:
        - connectors-start-and-monitor-deployment.yaml

And I'm unsure how to acquire the Controller logs.

Since posting this I realized that the way I was trying to accomplish my goal was wrong, but I still expected this KUDO plan to re-add my ConfigMap.

@dboyleitrs okay it looks like I was able to reproduce, thanks for reporting it! I'll let you know once I know more :)

Okay, I think I understand what's going on. We have a plan that basically does this:

create resource A1
delete resource A1
create resource A1

For resource deletion, we don't check for health, we assume that happened https://github.com/kudobuilder/kudo/blob/9db03a9f2be3d7bfe457ebc000881ad4433a27d4/pkg/engine/task/task_delete.go#L55
But I see that's not the case, I see that when I run in a debug it's actually doing:

create resource A1
delete resource A1
PATCH resource A1 (it thinks A1 still exists!)

By the way - if I do sleep for couple of seconds after deletion, it all works so it's really about all that happening asynchronously and us proceeding to the next step too fast.
As a solution I would propose:

add healthcheck for DeleteTask (it will query client.Get until it gets 404)
add ConfigMap to Owns here so we get reconciliation request after it got deleted https://github.com/kudobuilder/kudo/blob/f2142a9e672b40dad0daac158b4572adcf6a7308/pkg/controller/instance/instance_controller.go#L112

Question: should we do the same for create? 🤔 or is that synchronous and can you always do client.Get after create and always get your object?

Hmmm. Could we add the "foreground" flag to the delete task? That might block until children are deleted though.
The problem with "adding ConfigMap to "owns" is that we'd have to do that for all resources, right? Because this bug wouldn't only apply to ConfigMaps but all resources...
And I think that create is synchronous - When the call returns, the resource is at least created. For delete, there is the propagationPolicy, i'm not fully sure if that makes it a sychronous call...

I still wonder if it would be feasible to add watches for singular resources - i.e., add a watch for every resource that we deploy (in addition to some generic groups).
That would solve the problem as well, but I really don't know if it's ok to do that resource-wise...

@ANeumann82 isn't that basically what controller-runtime is doing though? it watches the resources and schedules reconciliation when the watch happens. What do you picture the watch triggering other than reconciliation? Just wondering 🤔

Afaik, controller runtime is watching gvk, i.e. the equivalent of k get pods -w for every type that we add to "owns".
The problem is, as we can deploy all kinds of resources with KUDO, we would have to add an "owns" for all CRDs. (Or at least the ones that we deploy with KUDO). So, the question becomes - is it better to create a global watch for all ConfigMaps, or create separate watches for each ConfigMap that we deploy with KUDO

@ANeumann82 oh yeah, that part... I was also wondering what would be a good solution to that as we should be watching everything that package creates and it can create pretty much anything 🤔

In that case we could use polling to determine that a resource has indeed been deleted. This is also what kubectl replace does when deleting and re-creating resources: https://github.com/kubernetes/kubectl/blob/master/pkg/cmd/replace/replace.go#L351.

Mmmmm... Polling would mean that we'd have to have a polling thread somewhere. Currently we completely rely on the reconcile loop, i'm not sure I would want to introduce polling now. Maybe polling in a loop while blocking the reconciliation, but that also doesn't sound good

Yeah, I was thinking of a blocking poll inside the reconcile loop. Not great, but rather low-effort.

I am no sure that's a good idea or that would be my last resort. We could reschedule reconciliation after failed delete check and even maybe leverage the RequeueAfter https://github.com/kubernetes-sigs/controller-runtime/blob/fc39662443b99ad0dcab22001571ccb9bc5b7960/pkg/reconcile/reconcile.go#L32 🤔


chore: add warning message to Tesla.Builder

related to #612
Signed-off-by: Yordis Prieto<EMAIL_ADDRESS>

@teamon, I need your approval over here. I intend to follow up by writing a proper documentation guide regarding migrating, but I would like us to send an awareness message sooner rather than later.
This is an annoying message but simple to turn off.


Enable clippy rule clone_on_ref_ptr

It is disabled by default.
https://rust-lang.github.io/rust-clippy/master/#clone_on_ref_ptr
Motivated by https://github.com/rust-lang/rfcs/issues/2588

Given that CI already runs with clippy::pedantic, isn't this covered already?  We just have memorialized it in the code, but we could do that instead.  (Adding one lint at a time seems like it might take a lot of effort.)

Given that CI already runs with clippy::pedantic, isn't this covered already? We just have memorialized it in the code, but we could do that instead. (Adding one lint at a time seems like it might take a lot of effort.)

I think the default clippy setting is allow, so clippy::pedantic does not block these. I thought we could use .clippy.toml for it but it is marked unstable so given that we don't have code that uses <arc>.clone() yet, this might be a good way to prevent it from appearing in the repository. I think the pipeline PR violates this rule already

Oh, I didn't check the rule's default enforcement.  I wonder why it never made it into the pedantic set...
If we merge this first, the pipeline code will hit it :)

I think it is considered controversial https://github.com/rust-lang/rust-clippy/issues/2048, but we can see whether it works for us :)


[FR] Documentation request for Raspbian

Is your feature request related to a problem? Please describe.
I ran the script on a brand new Raspbian installation with all updates applied.
I chose OpenVPN for legacy reasons.
I could connect to the VPN server no problem but I couldn't access the Internet from the client.
Running this command made it work perfectly :
sudo iptables -t nat -A POSTROUTING -s <IP_ADDRESS>/24 -o eth0 -j MASQUERADE
Describe the solution you'd like
Can the command be listed in the Wiki to make it easy for other people to find.
Thanks very much and thanks for a great solution, it is much apprecaited.
John

Could you by any chance have selected the wrong one? that would explain it .. as we haven't seen this happening ...

Right, If the installer showed wlan0 as an interface to pick, it means that he was connected to wifi (on my system ip address shows that if I'm connected to wifi the interface is UP, if not connected DOWN).
@johned0 do you remember if you switched from wifi to ethernet after the install?

Hi Guys, Sorry for the delayed reply, I've been out of the office.
@orazioedoardo - do you remember if you switched from wifi to ethernet after the install?
You are absolutely correct. My bad.
Yes, I originally set this up when the wifi was being used but then moved over to Ethernet.
My apologies for my oversight and thanks again for your help.
Best regards,
John


start implementation of keyboard shortcuts using tinykey for #66

This pull request lays the groundwork for handling #66.
I've only added the shortcut to play and pause the current sound when a user presses Space, since that's the only one of the listed shortcuts I was able to figure out how to add.

[x] Spacebar plays the current sound
[ ]  Pressing numbers seek the current sound (similar to youtube)
[ ] cmd + ? opens shortcuts panel

On the plus side, I did manage to add a semi-functioning shortcut that allows users to download sounds when they press the d key on their keyboard. If they are not logged in, it does absolutely nothing, as it should, but if they are logged in, pressing d opens the 'allow downloads?' dialog box and what appears to be full download functionality.

[x] d key allows users to download sounds if they are logged in

It's only semi-functioning because there's a weird sort of glitching effect that causes the download button to appear and reappear rapidly on page load. On the plus side, this effect ceases after a couple of seconds, and that glitch only appears if a user is logged in; otherwise, there's no download button, and that visual glitch doesn't seem to occur.
One last point: I did do git reset --soft HEAD~2 to undo the two commits I made for #81 as @amilajack recommended to me in #58; however, the changed files are still part of the commit in this pull request: so, even though the pre-commit check branch's commits aren't in this pull request, the changes I made for that branch are.
@akshara-sun, if you haven't run into an issue like this already, @amilajack might have a fix for it that he can explain to both of us for our benefit.

@amilajack husky objected to /cypress/plugins/index.js, so I commented out the lines in that file. That's not what caused Test / release (ubuntu-18.04) (push) to fail, is it?

I'm not sure whether we want to merge this pull request or not. It doesn't have all of the features listed in #66, but maybe merging it early will prevent conflicts later on?
If either of you, @akshara-sun or @amilajack, want to add the seek current sound and shortcuts panel shortcuts, that would completely take care of #66 and then we might be able to merge immediately. Of course, I'll take a look at this as well and see if I can do it.

Definitely. The commit in this pull request shouldn't have any references to lint-staged integration after the force push.
Also, (for GitHub), this issue closes #66.

Definitely. The commit in this pull request shouldn't have any references to lint-staged integration after the force push.

Closes #66.


How to do resizing of EKS cluster nodes?

What kind of request is this (question/bug/enhancement/feature request):
Question. If no support for this, then it is much required feature
Steps to reproduce (least amount of steps as possible):

Install EKS cluster with 2 nodes.
Try to edit cluster, UI doesn't allow to resize the cluster for worker nodes.

Result:
Expected to resize the nodes
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
v2.1.6
Installation option (single install/HA):
Single Install

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Hosted
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Cloud
Kubernetes version (use kubectl version):

(paste the output here)

Docker version (use docker version):

(paste the output here)

Folks, can anyone suggest me on this please?

AFAIK, you have to manually go into CloudFormation and edit the ASG.


deployCode/getCode no longer works with different profiles

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (14daacf 2024-04-09T00:16:23.595685880Z)
What command(s) is the bug in?
forge test
Operating System
Linux
Describe the bug
This issue is related to #7569
I have a project that requires multiple solidity versions configured as the following:
[profile.default]
src = 'src'
out = 'out'
script = 'script'
libs = ['lib']
solc_version = '0.8.16'
optimizer = true

[profile.0_6_x]
# `src` must be different fot it to work.
# We also recommend putting all Solidity test files and scripts inside `src*/`.
src = 'src-0_6_x'
out = 'out'
script = 'script-0_6_x'
# The order matters! When using `forge install` with FOUNDRY_PROFILE=0_6_x,
# it will use the first directory as the installation path.
# If the library is compatible with all versions,
# you can install it with the default profile.
libs = ['lib-0_6_x', 'lib']
solc_version = '0.6.12'
optimizer = false

Some of the tests are required to deploy contracts from 0_6_x, so I leverage the deployCode cheatcode to do it. The only gotcha was that I needed to run 2 commands when testing:
FOUNDRY_PROFILE=0_6_x forge build
# builds and runs tests with the default profile
forge test

This used to work until recently, but now I'm facing an error like this one:
Encountered 1 failing test in src/VestedRewardsDistribution.t.sol:VestedRewardsDistributionTest
[FAIL. Reason: setup failed: No matching artifact found] setUp() (gas: 0)

The specific line of code that causes that issue is:
deployCode("DssVest.sol:DssVestMintable", abi.encode(address(result.rewardsToken)))

DssVestMintable is defined in lib-0_6_x/dss-vest/DssVest.sol. To force the compiler to find it, there's a src-0_6_x/Imports.sol file that imports it.
I checked and the file out/DssVest.sol/DssVestMintable.json still exists in the directory, and it's not wiped out when I run forge build with no profile or forge test.
I'm not sure why this change in behavior was done in the first place, but I would kindly ask if we could have this capability back, otherwise it would be very painful to deal with cross-version compilation with Foundry.

cc @klkvr this also seems related to other recent deployCode issues

yeah, this is caused by the added validation of getCode cheatcode input. we've considered such usecase and added a configuration flag to disable the checks. try adding unchecked_cheatcode_artifacts = true to your foundry.toml
ref https://github.com/foundry-rs/foundry/pull/7334#issuecomment-1987154623

@amusingaxl though curious why are you using different profiles for different versions? have you had any issues with using foundry multi-version compilation? T

@amusingaxl though curious why are you using different profiles for different versions? did you have any issues with foundry multi-version compilation?

Maybe I missed when this feature was added. I've been using profiles for that for a long time and didn't mind to check back.
Although in this specific case, I do need different profiles because 0.6.x contract must have optimizations disabled, while 0.8.x can have them enabled.

Regarding that "multi-version compilation", where exactly is this documented? I can't seem to find it.

It is enabled by default if you don't have a solc version specified in the config https://book.getfoundry.sh/reference/config/solidity-compiler#auto_detect_solc

That doesn't work quite well with most workflows I know.
The standard practice is to leave a floating pragma on source files and fix the version in foundry.toml.
This avoids code hidden inside dependencies being compiled with a version that has a bug.
In fact, I only use profiles if I need to support different solidity "major" versions, but then I have exactly 2 compiler versions being applied at once, not many of them.


[Bug][Parameter Context] The problem of global parameter priority

The problem of global parameter priority，globalParam>varpool>localParam
issues-8329

@cheneychoi Please change describe from Chinese to English first

@zhongjiajie finished

@cheneychoi FYI, we have a discussion in https://github.com/apache/dolphinscheduler/issues/8329#issuecomment-1036163170

@zhongjiajie I modified this problem ,checkout from 2.0.2-release and plan to merge back to 2.0.2-release after completion. Am I doing this right?

Sorry for late review. You should fix in dev branch first, and then we will cherry-pick to next 2.0.x version. I don't understand the original design of the parameter priority, @wangxj3 @lenboo PTAL.

So, is this PR ready to go now?

Hi @cheneychoi, I change to base to dev and it seem they are lots of conflict, could you please solve it locally?

BTW, you should also change our docs later, thanks

@cheneychoi Hi,Why change the parameter priority.localParam>varpool>globalParam is right.

ping @cheneychoi again


Support brotli & ztsd compression as Content-Encoding

Introduces the ability for a registry to serve content with brotli, zstd, gzip or deflate compression using Content-Encoding mechanism.
A quick comparison on compression efficiency, applied to ubuntu top layer:
17953716 data.br
25861247 data.zstd
28382489 data.gz
71763456 raw

Not so "quite new". Supported by Chrome, Firefox, NGinx or M$ IIS since 2016

Was there a specific reason to pick andybalholm/brotli ?
the official google/cbrotli relies on native C code and I got issues with go.mod and linter using it.

Should adding brotli be done in a separate PR (giving some time to decide?)
Sure, that's an option. Just drop 2nd commit

@thaJeztah Any updates on this PR? I would like to use zstd with Registry

@ndeloof @thaJeztah Any updates on this PR? Having zstd would save a lot of network/disk i/o when using Registry


 I'll keep you posted as I make progress.

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/913 (JID-3793)

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/913 (JID-3793)


Fix typo in Audit Log Scope value

@AlexFilipin : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

Thanks, @AlexFilipin, for catching this!
#sign-off


09 - How to solve optimization problem in Pyleecan not useable

i have many questions for pyleecan and i can go to no one whether documentation or youtube, there is no good explanantion of the Methods Classes etc. and if Tutorials exist there are generally false Methods for Example in the Opimization, we vary geometrical variables without changing the winding so fill factors up to 180% are easily possible.I want set up this right to have something useable. And I do not understand how Objectives like efficiency etc. can be put in the simulation. And then there is the point of how i can add new geometries for me and maybe share it later with the community. There is no useful Documentation. Maybe anyone can help me:(

Hello,
If I understand correctly you want to run an optimization simulation that would change the geometry of a slot while keeping a constant fill factor and add an objective on efficiency. Regarding the first one, there is no function in pyleecan to do that, you will need to write yourself the setter function of the DesignVariable (a function that takes the simulation and a value in argument, the value would be the desired slot dimensions and the function would edit several parameters to adapat the slot and the winding). For the efficiency, it's still a work in progress in pyleecan there is a notebook tuto_Efficiency_map.ipynb but we will need to check if this is still relevant.
Best regards,
Pierre

Hello Bonnel,
Thank you for your support you are great.
The First issue could it look like this:
def set_slot_geometry(simulation, slot_dimensions):
"""
Setter function to adjust the slot dimensions and winding parameters
while keeping the fill factor constant.
Parameters:
- simulation: The simulation object from Pyleecan
- slot_dimensions: The new slot dimensions to be set
"""
# Extract current winding parameters
winding = simulation.machine.stator.winding

# Update slot dimensions
simulation.machine.stator.slot.H0 = slot_dimensions['H0']
simulation.machine.stator.slot.H1 = slot_dimensions['H1']
simulation.machine.stator.slot.H2 = slot_dimensions['H2']
simulation.machine.stator.slot.W0 = slot_dimensions['W0']
simulation.machine.stator.slot.W1 = slot_dimensions['W1']
simulation.machine.stator.slot.W2 = slot_dimensions['W2']

# Compute new slot area
slot_area = compute_slot_area(simulation.machine.stator.slot)

# Update winding parameters to keep fill factor constant
fill_factor = winding.fill_factor
conductor_area = fill_factor * slot_area / winding.Npcpp
winding.conductor.area = conductor_area

# Ensure other necessary parameters are adjusted accordingly
# (e.g., number of turns, layers, etc.)

def compute_slot_area(slot):
"""
Compute the area of the slot based on its dimensions.
Parameters:
- slot: Slot object with dimensions H0, H1, H2, W0, W1, W2

Returns:
- slot_area: The computed slot area
"""
# Implement the computation of the slot area based on its dimensions
# This is a simplified example; adjust as necessary for your specific slot type
slot_area = slot.H2 * (slot.W0 + slot.W1) / 2
return slot_area

Or am I on the false route🫡
Thanks again
Best regards,

Hello,
You can take inspiration from https://github.com/Eomys/pyleecan/blob/master/Tests/Validation/Multisimulation/test_slot_scale.py
Your setter is linked to a Design Variable, so I would suggest setting a float in an interval for the value rather than a dictionary
The slot area is already available with simulation.machine.stator.slot.comp_surface() and even comp_active_surface(). You can also use simulation.machine.stator. comp_fill_factor()
To make sure that the fill factor doesn’t change you can either play on the conductor dimensions or introduce a clever scaling rule on your slot that keeps the active surface constant.
Best regards,
Pierre

Hello and Thank you Bonnel, Great work you do! For me this should be
implemented in Library that the winding factor is not allowed to change and
the conductor diameter is the same, and only the Surface has changed you
got more winding so more inductivity and and Impedance in general I think
everyone who want to make a optimization for a electrical machine expects
that this feature is in it because otherwise the simulation makes no sense
because the optimizer is optimizing some thing that is not possible for
example you optimize for Tem_av so you allow to vary the geometry but you
are have not always the highest possible winding in the simulation so you
get not the best machine out of it. Lg Lukas
Am Fr., 21. Juni 2024 um 16:21 Uhr schrieb Pierre Bonneel <
@.***>:

Hello,
You can take inspiration from
https://github.com/Eomys/pyleecan/blob/master/Tests/Validation/Multisimulation/test_slot_scale.py
Your setter is linked to a Design Variable, so I would suggest setting a
float in an interval for the value rather than a dictionary
The slot area is already available with
simulation.machine.stator.slot.comp_surface() and even
comp_active_surface(). You can also use simulation.machine.stator.
comp_fill_factor()
To make sure that the fill factor doesn’t change you can either play on
the conductor dimensions or introduce a clever scaling rule on your slot
that keeps the active surface constant.
Best regards,
Pierre
—
Reply to this email directly, view it on GitHub
https://github.com/Eomys/pyleecan/issues/701#issuecomment-2182850834,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3JD5TAUQA7FTDEHNHQMDPLZIQZGNAVCNFSM6AAAAABJUV4DXOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBSHA2TAOBTGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Termux problem

Hi i have a problem for install this package
https://drive.google.com/open?id=1T7tv2GdYgslekMKu1yoQKW5BHlaTZ29Y

https://imgur.com/a/Oktaa

Hey, I'll look into this

Hey this is a duplicate of #28

How can i solve it

So I am actually working on rewriting lots so just give me sometime. I have been really busy with school recently.

I get the same error while running Python 3.6.1 Python 3 is not supported.
Python 3.6.1 (default, Dec 2015, 13:05:11)
[GCC 4.8.2] on linux

Traceback (most recent call last):
  File "python", line 705
    print """
      Reaver has been designed to be a robust and practical attack against Wi-Fi Protected Setup
      WPS registrar PINs in order to recover WPA/WPA2 passphrases. It has been tested against a
      wide variety of access points and WPS implementations
      1 to accept / 0 to decline
        """

          ^
SyntaxError: Missing parentheses in call to 'print'


ancient packing iterates serially instead of parallel

Problem
With par_iter, there is no guarantee of execution order.
If account a is dead in slot 1 and alive with zero lamports in slot 2, then if par_iter visits slot 1 first, it will unrefs account a in the index since slot 1 will be rewritten to not include the dead account a in slot 1.
Then, when slot 2 is visited, account a will report a refcount of 1. This means it could be deleted since it is a zero lamport account with only a single refcount. However, if we later fail to pack slot 1 (due to multirefs or something), then we addref account a. But, at that point, we've already treated account a in slot 2 as a single ref count zero lamport account as opposed to a multi ref alive account. We would then delete account a from slot 2, remove the slot list for account a, and decrement the refcount. This results in account a in the index (or maybe even not!) with a slot list that is empty and a refcount of 1. This means next time we run a hash of all accounts, we will find the older, 'dead' account. This produces the wrong cap and/or hash of all accounts.
Summary of Changes
Iterate slots in slot order highest to lowest. This allows us to correctly unref dead accounts and then reverse that decision without producing an indcorrect result.
Fixes #

Changes look good to me, as correctness is the highest priority. Do you have any perf numbers on how much this change actually impacts runtime for getting accounts_to_combine?

Here's a ledger tool run from a mnb snapshot that runs long enough to pack twice.
Numbers are close enough. Read is faster now. Note that the read times are measured at the per-slot level. This is a bg process. Not sure how consistent the numbers are. I imagine there is a lot of variation.
There may be less contention on the index reading now than before.
this pr:
[2024-08-27T14:08:17.325449184Z INFO  solana_metrics::metrics] datapoint: shrink_ancient_stats num_slots_shrunk=7
index_read_elapsed=185,734
total_us=1,395,205
[2024-08-27T14:15:52.031799590Z INFO  solana_metrics::metrics] datapoint: shrink_ancient_stats num_slots_shrunk=7
index_read_elapsed=199,878
total_us=2,350,142

master:
[2024-08-27T14:29:08.649332096Z INFO  solana_metrics::metrics] datapoint: shrink_ancient_stats num_slots_shrunk=7
index_read_elapsed=324,871
total_us=1,277,048
[2024-08-27T14:36:39.865026033Z INFO  solana_metrics::metrics] datapoint: shrink_ancient_stats num_slots_shrunk=7
index_read_elapsed=555,725
total_us=1,990,582


Include basic feed information in validation report

Feature request
It would be helpful if validator results included general feed information such as agency name, number of routes and trips, and service dates. When dealing with multiple files it can be helpful to have a way to identify the feed being validated in the validator results. Having the service dates visible is especially helpful when comparing validation results of older files against new ones.
Is your feature request related to a problem? Please describe.
I was unable to see feed identifying information in the validator results.
Proposed solution
I would like to have general feed information available in the validator results, such as agency name, service date range, and number of routes/trips.

The feed information was added as part of #1363


Javascript Error on Frontend

Hi - I am not sure if this is the best place to report this issue or not. If there is somewhere else more appropriate, please let me know.
I have tried running launchpad with my local minishift installation a number of times, both manually and using this template.
In both cases, after I create the project, import the template, and deploy the applications, I then need to manually create a route for launchpad-frontend, which I find odd (and may require an issue of its own). I only mention it because the problem I am seeing is on the front end, so it may be related.
After creating the route, I can follow the link to http://launchpad-frontend-launchapd.<IP_ADDRESS>.nip.io/ (for example) and am redirected to /wizard. When I click the big "LAUNCH YOUR PROJECT BUTTON", I immediately see this error.

When I open the Javascript console in Chrome, I see this stack trace.
/app.57fe4094c5fd119dc4a2.js:3 An error occurred SyntaxError: Unexpected token < in JSON at position 0
    at JSON.parse (<anonymous>)
    at e.t.json (/vendor.57fe4094c5fd119dc4a2.js:642)
    at /app.57fe4094c5fd119dc4a2.js:3
    at t.invoke (/polyfills.57fe4094c5fd119dc4a2.js:58)
    at Object.onInvoke (/vendor.57fe4094c5fd119dc4a2.js:176)
    at t.invoke (/polyfills.57fe4094c5fd119dc4a2.js:58)
    at r.run (/polyfills.57fe4094c5fd119dc4a2.js:58)
    at /polyfills.57fe4094c5fd119dc4a2.js:65
    at t.invokeTask (/polyfills.57fe4094c5fd119dc4a2.js:58)
    at Object.onInvokeTask (/vendor.57fe4094c5fd119dc4a2.js:176)
    at t.invokeTask (/polyfills.57fe4094c5fd119dc4a2.js:58)
    at r.runTask (/polyfills.57fe4094c5fd119dc4a2.js:58)
    at o (/polyfills.57fe4094c5fd119dc4a2.js:58)
    at n.invokeTask [as invoke] (/polyfills.57fe4094c5fd119dc4a2.js:58)
    at v (/polyfills.57fe4094c5fd119dc4a2.js:23)
    at XMLHttpRequest.d (/polyfills.57fe4094c5fd119dc4a2.js:23)

It looks like whatever is attempting to parse JSON is getting a string <anonymous> instead of the expected JSON, and then blowing up. But because the UI code is obfuscated, it's not clear to me where this unexpected data is coming from.
Any help getting this resolved would be much appreciated.

Do you see any errors in the backend pod log?

You don't need to create any routes. The launchpad-nginx is the main route.
I believe we have removed the extra routes, use this template instead: https://raw.githubusercontent.com/openshiftio/launchpad-templates/master/openshift/launchpad-template.yaml

@gastaldi thank you for the explanation. Creating and using the launchpad-frontend route was my problem. Closing due to user error.


logging-interceptor:2.6.0-SNAPSHOT breaks on repo. update to 2.7.0-SNAPSHOT

Previous version of the library breaks, since it is removed from the repo.
https://oss.sonatype.org/content/repositories/snapshots/com/squareup/okhttp/logging-interceptor/

Huh? What's the problem and how do we fix it?

When 2.6.0 was released, all of the 2.6.0-SNAPSHOT versions are deleted from the snapshot repo automatically. You don't need to use a snapshot anymore. Just use 2.6.0 as the version.


Issue in accessing the destination

Hi,
I followed Create and Deploy HTML5 and SAP Fiori Launchpad Site Modules tutorial and it worked fine for me.
After that i wanted to create my own app and deploy.In the process i've configured a destination in cockpit and added in my xs-app.json but while accessing i face Internal Server Error.
Could you please help me here.
Below are the necessary details.
My destination:

xs-app.json:

mta.yml:

This is the error i am facing:

Thanks.

Hi @JOSEPHSASIKANTHREDDY,
thank you for your feedback. There seems to be an issue with SAP Web IDE. We are working on this. As soon as we have an answer, we'll update this tutorial.
But regards to your Internal Server Error with your own app and deployment, it is better to open a ticket on the following component:  BC-CP-ABA.
Thank you for your understanding.
Best regards,
Merve


Design solution for referencing external databases

As a developer, I want to have a document where to record design and discussions about the integration with external resources in PDCM Finder.
Acceptance Criteria

[x] Document with the design.
[x] Meeting with Zina/Tushar to approve

Document (WIP): https://docs.google.com/document/d/1BJyG_GaMbHvWXBcCGqhU1a3My9OIvxwvbz2fbLQSmJg/edit?usp=sharing

@zperova done


fix: create a release

Description
The previous PR was a "chore", but should have been a "fix".
See #19

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