TC added: FullLuceneShowcasetestSearchMaitreyapraṇidhana()

@eroux, are you ok with this 'Maitreyapraṇidhana' TC ?
Is it ok, as it is now (passing) or it is supposed to fail at the present moment (i.e. have different score on some of the 8 tested docs) ?

the score computed by Lucene is not something that's really understandable by a human so I have really no clue what the scores should be (or even if it makes sense to have an opinion on the matter). What puzzles me is that the first two are equal but give different scores... I can't find a reasonable reason for that but I didn't look at the code closely... do you think there is a reason?
But in any case, I suspect the scores will vary with the version of Lucene so I'm not sure hardcoding them is a good idea... maybe you can sort by score order and check if the order is the one you expect?

I know that the scores are rather volatile and will most probably change across different versions of Lucene.
As a remedy I am checking for approximate score (with epsilon set to +/-0.01)
Your idea about checking the order is not directly applicable, as some pairs (or even triples) of docs get exactly the same score hence those may be score-sorted in any permutation and spoil the expected order.
Regarding your question
What puzzles me is that the first two are equal but give different scores... I can't find a reasonable reason for that but I didn't look at the code closely... do you think there is a reason?

Yes, there is difference - a couple of soft hyphens in the second case (but github is not displaying those). The second doc looks like 'Maitreya-praṇidhana-rāja' in a proper editor.
So, any alternative ideas?
E.g. increase the epsilon value from 0.01 to 0.1 ?

ooooh ok thanks, that's reassuring!
For the epsilon yes, let's make it a bit higher
For the soft hyphens, they should only be used by the analyzer to cut the syllable but shouldn't be indexed... can you check if the analyzer produces tokens that includes the hyphens? If so, we need to find a way to remove them, it doesn't make sense that they change the score so much...

I've isolated the bug and turned into a (failing) test in https://github.com/buda-base/lucene-sa/commit/b8a5b9d5faad6defe1b9dc77493ae8154adf15cf
I think it might be an unfortunate interaction between the lenient code and the syllable tokenizer but maybe it's more simple than that. It's not triggered when there's a soft hyphen between na and ra, which explains the difference of score.
Can you try to take a look?

I did some experiments aside from your TC and the results concur with your observation -
w/o soft hyphen between 'na' and 'ra' the tokenizer will give us 'nara'
Don't know if this was what you were asking for?

oh, I meant: can you try to fix it?

Ок :)
Where should I be looking for the problem?
SkrtSyllableTokenizer ?

that sounds like a reasonable start yes. Something to be aware of is that the r is generally a problem in lenient mode as it can be either a semi-vowel (ṛ) or a consonant (r), which the syllable tokenizer treats differently. There's no perfect solution for this, but we should just reajust (this will give other types of errors but hopefully they will be less commons)


USB2 IRQ affinity tweak

Tested on and works with both kernels: https://github.com/armbian/build/commit/599cf35bde08ae8675ded8b1d02054fe22286b0b

Wow. Great :)

Not that important but after playing around with ODROID-HC1 I realised that everything that has to share IRQs with dw-mci is somewhat performance affected. Same on ROCK64. And since USB2 ports on ROCK64 became more attractive now that we can switch connected consumers totally off (a backup HDD for example that can be operated on USB2 ports for backup operation and will be attached to USB3 port only for huge restore or desaster recovery purposes) I thought we should squeeze out everything we can. :)

Cool idea :)


Support latest as a version alias for the latest Node version

#26 discusses supporting several types of aliases. That discussion has been going on for a long time. It seems the main complications revolve around how to deal with lts.
For those of us who don't care about lts and just want latest so that we can test that our work works on latest Node automatically, could we please get support for latest first and let #26 focus on the subtleties of lts and how to implement that? It seems like these two aliases do not need to be grouped into a single feature release, in fact, it looks, like implementing them separately would simplify things and potentially make it possible to provide latest soon.

Good news!
I opened a PR to allow the current alias. :+1:
We can maybe support both current and latest aliases but I find it more convenient to use the current keyword as it is how Node.js names their versions.
PR available there: #279

Hi @zdm,
Yes, this feature is under development.


Support arm64

This PR closes #23.

Will never be realized, because GitHub virtual environment cannot support M1 Mac for now.
The built binary seems to work normally on M1 Mac, too, so currently no fix is planned on this.


[2.7] fix ZookeeperDynamicConfiguration remove cache listener mistake

fix #10059, fix ZookeeperDynamicConfiguration remove cache listener mistake

Codecov Report

Merging #10106 (fdd0d47) into master (c25284e) will decrease coverage by 0.35%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master   #10106      +/-   ##
============================================
- Coverage     60.50%   60.15%   -0.36%
- Complexity      412      444      +32
============================================
  Files          1100     1100
  Lines         44557    44510      -47
  Branches       6485     6453      -32
============================================
- Hits          26959    26774     -185
- Misses        14637    14772     +135
- Partials       2961     2964       +3

Impacted Files
Coverage Δ

...pport/zookeeper/ZookeeperDynamicConfiguration.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...ookeeper/ZookeeperDynamicConfigurationFactory.java
0.00% <0.00%> (-66.67%)
:arrow_down:

...gcenter/wrapper/CompositeDynamicConfiguration.java
0.00% <0.00%> (-63.34%)
:arrow_down:

...fig/configcenter/TreePathDynamicConfiguration.java
25.71% <0.00%> (-54.29%)
:arrow_down:

.../dubbo/metadata/report/MetadataReportInstance.java
12.50% <0.00%> (-40.63%)
:arrow_down:

.../configcenter/support/zookeeper/CacheListener.java
0.00% <0.00%> (-32.44%)
:arrow_down:

...gistry/client/migration/MigrationRuleListener.java
55.88% <0.00%> (-23.53%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-20.69%)
:arrow_down:

...mmon/config/configcenter/DynamicConfiguration.java
68.18% <0.00%> (-18.19%)
:arrow_down:

... and 74 more

Continue to review full report at Codecov.

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


Add Dockerfile

Added Dockerfile for app
Made the app a bit shorter
Improved app with string interpolation usage

:shipit: This was also a sample for corert back in the day so that's why it was...messy. :))

Sure, sure. The focus should be on the Dockerfile. Good?

If you're asking me if the Dockerfile is good, yes, it looks good.


Add OTel-Arrow exporter/receiver to contrib distributions

Adds OTel-Arrow to the contrib distribution.

We should add it to the k8s distro as well since it is an OTLP receiver/exporter


Add --proxy-bypass-list support in argv.json

Add support for --proxy-bypass-list in the argv.json file so that we can provide a default proxy bypass list to VS Code users.
Setting NO_PROXY doesn't work as a solution because it would either have to be set as a user / system environment variable thus impacting all applications, or we would have to rely on our VS Code users to explicitly pass --proxy-bypass-list as an argument to code on the command line each time they launch. Having the option to set this in argv.json would provide us with a way to package this in our default enterprise installation of VS Code so users would get this setting by default, but just for VS Code.

Could we just expose proxy-bypass-list  in argv.json. I was not aware that only a subset of our command line arguments can be configured there actually.
fyi @deepak1556

Yes we can expose it

I was not aware that only a subset of our command line arguments can be configured there actually.

Yeah the list of chromium commandline flags in quit large and always changes with version updates it would be a nightmare if we are to support all of them in argv.json. At this time we only support those on a case by case basis.

Thanks, makes sense! Should I assign this one to you?
If you are busy, I am happy to provide a PR in July if you give me a code pointer?

Here is an example of recent argv.json flag addition https://github.com/microsoft/vscode/commit/8a3f7e95d207c49a6576b387d50a43d7ece7c585, feel free to send a PR. I can review it. Thanks!

Thanks for that great example. Here's the PR https://github.com/microsoft/vscode/pull/216716
I verified that adding "proxy-bypass-list" does not produce errors in argv.json
I did not verify if this list actually gets passed to Chrome. We can ask @patrickhernon if we get this out to an insiders build.

FYI: There is a PR in progress for adding a setting for the NO_PROXY feature: https://github.com/microsoft/vscode/pull/206622

Thanks. But unlike that PR, I think this flag just needs to be passed to Chromium and it should get handled. i.e. I did not add any code in VS Code that actually handles the flag.

This bug has been fixed in the latest release of VS Code Insiders!
@patrickhernon, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version 4580ba51fe1914ca29916f829adb35930089b013 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

This flag doesn't seem to work for me unfortunately. Using mitmproxy I still see the bypassed domains:

Can confirm, preparing a fix now.

verification steps:

Setup mitmproxy https://docs.mitmproxy.org/stable/overview-installation/
Start the application with proxy server code-insiders --proxy-server="<host:port from mitmproxy>"
Check the logs to see requests from the application are intercepted, you should see some domains like *.exp-tas.com or *.visualstudio.com
Open Configure Runtime arguments and set "proxy-bypass-list": "*.exp-tas.com,*.visualstudio.com"
Restart the application with same proxy server
Confirm that requests to these hosts are not seen in the intercepted logs anymore

Patrick Hernon confirmed this works great in their enterprise. Thanks all 👏


disables page scroll when pressing spacebar

PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: #3870
What is the current behavior?
If playing a video and then navigating to a new page to cause the miniplayer to appear out of focus, pressing spacebar causes the video to pause and the page to scroll down in the background.
What is the new behavior?
Ensures that pressing spacebar only pauses the video on the miniplayer and doesn't cause the page to scroll.
Other information

@seanyesmunt I just wanted to check in when you have a sec. I updated and pushed up the view.jsx file already, but didn't touch the CHANGELOG.md yet because it was already in the Unreleased section when I checked, but maybe I'm confused there. Let me know if you need any other changes, thanks!

Nope. Sorry we've been pretty busy with paid content on lbry.tv. I'll give this another pass/merge later tonight or tomorrow morning.

Sounds good, no rush!

@cassidypignatello Can you please rebase this? It looks like it includes a lot of changes that have already been merged to master.

@seanyesmunt No problem, I just rebased this. Let me know if you need anything else!


fix(SR13): fixed exception text reinterpreted as HTML

Changeling

Fixed SR13: exception text being sent as part of the response

Nice work. Failing API test to address please.


Accept Vouch and Un-vouch on penalty #233

Added a new table for vouches
Added tests
Added a migration function

Deployment

update Accounts contract
run migration action

@IanMendozaJaimes onboarding test fails
./scripts/seeds.js test onboarding
✖  Given invited new user: should have new vouch entry
-------------------------------------------------------
    operator: deepEqual
    expected: [0, 1]
    diff: [
      1: 1 => 0
    ]
    source: at /Users/nik/work/seeds/seeds-contracts/test/onboarding.test.js:425:5
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

Hello @n13, I added the requested changes:

I fixed the sha256 function in the helper, now onboarding tests work
I removed eosjs-ecc from package.json, the eosjs v21 library doesn't use it as eosjs-ecc is deprecated and unmaintained, that's why I preferred to keep using the ecc from the eosjs v21}
Also, I added the attribute points to the punish function, it is not used in this issue but I added it because it is used in the flagging issue, so that should cause less problems when merging

Works perfectly, thank you


pushing new branch will rereference issues in previous branch

Pushing a new branch will re reference issues in the previous branch.
Steps to reproduce:

reference a issue in one branch (e.g. master) and push commits
create a new branch
commit to new branch
push new branch (via git push origin --all)

Issues with references from commits in master branch will then have another reference from the same commit but at another time.
example: https://try.gogs.io/mexthemax/TestRepoForMe/issues/3
This was tested with gogs <IP_ADDRESS>6 Beta on private instance and on try.gogs.io with same version.

Thanks your feedback!

Hi, I've pushed fixes to develop branch, you can also test on https://try.gogs.io/

I tested it on https://try.gogs.io/ with the same steps as before and the problem is solved.

@mexthemax thanks your confirmation!


Add the announcement about Grade 1 Applications

Is your feature request related to a problem? Please describe.
We have to show this announcement on the web site home page
Admission to Grade 01 2021
St. Anthony's College, Kandy.
Applications are available at the College Main Entrance. Please submit the Child's Birth Certificate (BC) to collect the Application. If you are eligible to apply in different categories, seperate applications should be submitted for each category. Deadline for the submission of completed applications is 15th July 2020.

Describe the solution you'd like
Create the announcement on landing page
Describe alternatives you've considered
N/A
Additional context
N/A
Are you working on it?
No

I'm working on it.


Add deployment scripts

@cjh1 thanks for this. If it's ready to fold in, go ahead and revoke the draft status and merge.

@cjh1 is this very old PR still valid, afayr? Thanks! :smile_cat:

I would say its still valid ...

Thanks for confirming, @cjh1. I will go ahead and merge.


simulate-dialogs with parser

This PR makes the following changes to genie simulate-dialog:

Adds the option to parse the last user utterance using a neural semantic parser, instead of reading UT from file.
For each input dialogue, output n partial dialogues where n is the number of turns in that dialogue. The i-th output has the entire conversation from the beginning till turn i.
Adds the option to only output partial dialogues where the provided semantic parser makes a mistake. i.e. if a user turn is parsed correctly, it will not be output.

Travis CI is stuck in queue for 12 hours now. @gcampax is this normal?


Automatically opening with window picker if more than one non-Neotree window open

Would it be possible to automatically open files using the window picker (w) instead of the normal open (<CR>) when more than one non-Neotree window is open?
Not even asking to be added as a function, just asking if something like this would be possible via config.
Thanks!

Hi! I'm not sure what you mean by this. Do you mean making <CR> use the open_with_window_picker mapping instead of the normal open?

Yes, although only when more than one window is currently open (excluding Neotree)

You can set window-picker's configuration to autoselect_one = true and include_current=false (similarly to how it's done in the example). This will prompt you to pick a window if there are multiple, and automatically choose when there is only one.
You can then bind the <CR> to this mapping like this
 ["<cr>"] = "open_with_window_picker",

Does that answer your question? 😁

yup! thanks very much


Steam Deck Input issue

With this mod enabled the Steam Deck controls no longer work.

After excruciating hours of researching for solution, I finally found a fix for controller issues on my Steam Deck:
Quoting a comment from "Outfit Manager" mods on Nexusmods (since this mod also inject xinput1_3.dll, i thought this fix might apply)

Steam launch option should be set to WINEDLLOVERRIDES="xinput1_3=n;xinput1_3=b" %command%
These are what I did to fix, some step might be redundant, I'm unable to test thoroughly.


Dois class

Regarding Create new Doi class and Migration Code
These changes include:

Creating a new Doi class with three new slots: doi_value, doi_category, and doi_provider
Adding a new slot to the Study class called associated_dois that have a range of the Doi class
Adding enumerations DoiProviderEnum and DoiCategoryEnum that serve as the range for the doi_provider slot and doi_category slot, respectively.
Deprecating the following slots because they will be represented by the new DOI framework as described above: dois, dataset_dois, award_dois, publication_dois, ess_dive_datasets, massive_study_identifiers, and massive_identifiers.
Including migration code to make these changes. Note: the migration code includes "changesheet" type migrations with specific studies DOI values changed because 1) we needed to capture the newly required slot doi_provider for study award DOIs 2) three new award DOIs were added to a few studies, and 3) There were a handful of DOIs that existed as award DOIs that should be dataset DOIs.
Updating the valid and invalid example data files to be in alignment with these changes and including a new invalid example specific to this change.

Per discussion with @eecavanna and @turbomam - we need to discuss how to handle the id-specific migrations in the future and how this impacts migrations that will result in invalid data.
I also added the changes to the schema as requested by @turbomam:

Changing the name of the newly created invalid data file so it is more specific
Changing the EDAM data prefix to edam.data
Adding a see_also to the DoiProviderEnum for the DoiProcessingInstitutionEnum
Adding a comment to the data_portal_subset saying that the assignment of the subset is incomplete in the schema.
Changing the doi slot name to doi_value

@marySalvi tagging you so you are award of the new schema structure for DOIs

running locally in preparation to approve and merge

Thanks for the feedback and all the typo catches @eecavanna! I fixed and added your suggestions.

@pkalita-lbl and @brynnz22 could we please add a study_category to nmdc:sty-11-hdd4bf83 before merging this PR?
https://github.com/microbiomedata/nmdc-schema/issues/1223

...could we please add a study_category to nmdc:sty-11-hdd4bf83 before merging this PR?

I would prefer the code that does that be part of the class named Migrator_from_8_0_0_to_8_1_0, specifically, since that's the version "jump" where the associated schema change was introduced. I'm OK with it being included in this same PR—I just wanted to share my opinion about which class it reside in.
https://github.com/microbiomedata/nmdc-schema/blob/9e4c58099b2fe064b6b08d3e24cfaf36458cbefb/nmdc_schema/migration_recursion.py#L271-L288
Note: Since neither that existing class, nor the one being introduced in this PR, has been included in an nmdc-schema package published to PyPI, neither migration has been run by nmdc-runtime yet.

I would like to make a 8.2.0 release of what's in main right now and then have 8.0.0 to 8.1.0 and 8.1.0 to 8.2.0 migrations  executed, including the assertion of some default study_category value for Studies that don't currently have one.
@aclum @mslarae13 can you confirm that research_study is the right study_category for nmdc:sty-11-hdd4bf83? Or tell me who else I should be asking?
Then I would like to merge this PR and make a 8.3.0 release.

@turbomam

I would like to make a 8.2.0 release of what's in main right now, including "Require study_category" and then have 8.0.0 to 8.1.0 and 8.1.0 to 8.2.0 migrations executed, including the assertion of some default study_category value for Studies that don't currently have one.
@aclum @mslarae13 can you confirm that research_study is the right study_category for nmdc:sty-11-hdd4bf83? Or tell me who else I should be asking?
Then I would like to merge this PR and make a 8.3.0 release.

in the `migration_recursion.py I made the version number for this PR 8.1.0 to 9.0.0 since the migration this PR includes is not backwards compatible. Are you saying that I should change the migration numbers to 8.2.0 to 8.3.0? I will fix this in the code if so.

@emileyfadrosh  Would you consider Andrew's caninie portion of the TRiP project a research_study as defined as 'A detailed examination, analysis, or critical inspection of a hypothesis-driven experiment' or a consortium as defined as 'A group formed to undertake a venture that is beyond the capabilities of the individual members. Each member of the  consortium brings a high level of expertise in a specific area to ensure the successful completion of the project'. This is needed for one of the migration scripts since it is a required field now.

This would be a research_study, thanks!


Rename HttpStack to DefaultHttpStack

Overview
There was a bug that the exported HttpStack class conflicts with the HttpStack interface, preventing it from importing properly in a typescript environment.
so, I fixed this bug by renaming HttpStack to DefaultHttpStack
https://github.com/tus/tus-js-client/pull/234#issuecomment-771880925
Question
Should I also rename this HttpStack in lib/browser/index.js?

Thank you, merged in https://github.com/tus/tus-js-client/commit/bafe931887f4e46dfd2bb31d653761cc1cb4b912 with some additional changes.


[Docs] Organization repo request form is not live yet

The guide mentions the repo request form contained in the following PR:
https://github.com/salt-extensions/community/pull/15
It's not yet live.

Merged now.


issue#1447: NettyHelper in netty extension populated Logger "InternalLogger" in netty.

What is the purpose of the change
to fix issue reported by #1447: NettyHelper in netty extension populated Logger "InternalLogger" in netty.
Brief changelog
don't log context info if it's not from dubbo code base
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[x] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #1704 into master will increase coverage by 0.07%.
The diff coverage is 23.07%.

@@             Coverage Diff              @@
##             master    #1704      +/-   ##
============================================
+ Coverage     35.37%   35.45%   +0.07%
- Complexity     3765     3775      +10
============================================
  Files           628      628
  Lines         30963    30987      +24
  Branches       5438     5449      +11
============================================
+ Hits          10954    10987      +33
+ Misses        18109    18106       -3
+ Partials       1900     1894       -6

Impacted Files
Coverage Δ
Complexity Δ

...com/alibaba/dubbo/common/logger/LoggerFactory.java
25.49% <0%> (-2.78%)
3 <0> (ø)

...remoting/transport/netty4/logging/NettyHelper.java
23.58% <100%> (-0.23%)
2 <0> (ø)

...ba/dubbo/common/logger/support/FailsafeLogger.java
13.63% <28.57%> (+0.22%)
4 <1> (-1)
:arrow_down:

...g/context/annotation/DubboConfigConfiguration.java
50% <0%> (-16.67%)
0% <0%> (ø)

...baba/dubbo/remoting/transport/mina/MinaClient.java
57.81% <0%> (-1.57%)
8% <0%> (-1%)

...n/java/com/alibaba/dubbo/common/utils/JVMUtil.java
72.22% <0%> (-1.37%)
13% <0%> (ø)

...ba/dubbo/remoting/transport/netty/NettyHelper.java
31.25% <0%> (-1.01%)
2% <0%> (ø)

...java/com/alibaba/dubbo/common/utils/PojoUtils.java
49.41% <0%> (-0.15%)
63% <0%> (ø)

.../com/alibaba/dubbo/examples/rest/RestProvider.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

... and 17 more

Continue to review full report at Codecov.

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

Now, i understand DubboInternalLogger's side effect mentioned in #1447, and i've tested this PR works fine.
But i have another question, if we don't attach dubbo prefix before netty logger anymore, why don't we just remove NettyHelper from dubbo? Cause i think maybe name.startsWith("com.alibaba.dubbo.") will never matches.


DELIA-53130 : Cec_devlist return NULL when daemon not running

Reason for change:
Cec_devlist return NULL when daemon not running
Test Procedure: None
Risks: Low
Change-Id: I5d999bf8edeae18d282c33f4b33ea7622ad20b86
Signed-off-by: Anooj Cheriyan<EMAIL_ADDRESS>

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/1976/rdkcentral/rdkservices

Commit: 3cd72d433ac7943b0eaccce635dc190b1904e1e4

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/1976/rdkcentral/rdkservices

Commit: 3cd72d433ac7943b0eaccce635dc190b1904e1e4

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Spurious copyright match

Commit: 3cd72d433ac7943b0eaccce635dc190b1904e1e4


Market data query timeout.

Hi, I downloaded the code today, and when i run the market data query (run MarketDataEndpointsExample without any change), i got a read time out issue, but i can open Binance website and see the market data change, is any thing wrong?
Below is the exception:
Exception in thread "main" com.binance.api.client.exception.BinanceApiException: java.net.SocketTimeoutException: connect timed out
at com.binance.api.client.impl.BinanceApiServiceGenerator.executeSync(BinanceApiServiceGenerator.java:71)
at com.binance.api.client.impl.BinanceApiRestClientImpl.getOrderBook(BinanceApiRestClientImpl.java:72)
at com.binance.api.examples.MarketDataEndpointsExample.main(MarketDataEndpointsExample.java:26)
Thanks,
GC

Not really sure, I'm unable to reproduce the problem with the latest version. Perhaps your firewall is blocking the connection?


elastalert schedule

Hi,
i want to run the alert every morning at 9:00 AM. can you please help how can i achieve this?

Use this in your rule file:
Look for aggregation documentation here: https://elastalert.readthedocs.io/en/latest/ruletypes.html?highlight=timestamp_format_expr#rule-configuration-cheat-sheet
aggregation:
schedule: '0 9 * * * *'


How to use this for fragments containing layouts other than ListViews?

The ViewPager in my app contains layouts with ListViews, ExpandableListViews, ScrollView, RelativeLayout and LinearLayout(with a MapFragment in this).
Everything works fine for Tabs containing ListViews and ExpandableListViews! But.. not for the other layouts? How to proceed now?
Please Help me I'm trying to build an app for our college Festival.
Thanks!

@Vieuma Can u explain me , what's foodtruckHeader.setTranslationY(...)

@Adnan9011 I was working on a foodtrucks finder app, I just forgot to replace the var name : foodtruckHeader is actually just the header ViewGroup. The call to setTranslationY(...) allows to move the collapse/expand the header while the user is scrolling.


ANTs concatenate transformation matrix and multi-step reg

Hi! I would like to concatenate different transformation matrices (instead of calling antsApplyTransforms for every step) so that I may transform from source to target space.
antsApplyTransforms.....-t 23.mat -t 1.warp -t 1.mat
I was trying CompositeTransformUtil to combine/assemble 23.mat from 2.mat and 3.mat. However, antsApplyTransforms fires an error then.  When I use antsApplyTransforms and list each matrix file, then there seems to be a grid-misalignment, probably (?) because the reference for each step was not given.
antsApplyTransforms.....-t 2.mat -3.mat -t 1.warp -t 1.mat
(I ensured that input was given in the correct order)
So what is the best way have least interpolation and adequate results?

registration is all about getting the details right.     it is very common for us to do the following:

register A to B

register B to C

register C to D

map A to D via antsApplyTransforms

this sounds like what you are trying, but failing, to do.    if you share code and details, then maybe someone can help.   or rethink your process and try to debug what you've done on your own, breaking down each step until there is no longer uncertainty into the source of the problem.

Thanks :)) Problem solved. Input was a 4D volume, while the script expected a 3D volume. Would be convenient if N4BiasFieldCorrection (without -d flag) would check the dim4/pixdim4 value and decide whether the input is 4D.

would check the dim4/pixdim4 value and decide whether the input is 4D.

ITK handles more than just the nifti format.  We do an initial read of the dimension, if possible, but that's not foolproof.


could not able  see any contents

Unfortunately  , when i clicked on layout.json to open , file tries to open in visual studio code application , but the contents could  not be seen, but i am able to see in notepad

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

ID: cee82e78-02d5-bfe3-eb67-c69bdad01fa8
Version Independent ID: 72f8b130-354d-a164-cb37-3d9b0b7cff19
Content: Automate Visual Studio installation with a response file - Visual Studio
Content Source: docs/install/automated-installation-with-response-file.md
Product: visual-studio-dev15
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@RAJU2529 Please use the Report a Problem tool to report this behavior. When you do so, your report becomes a work item for the product team, which allows you to engage directly with that team so that such problems can be addressed and fixed.


Change the serial console setting for GCE

A new instance type is being introduced that requires this new console setting in order to have access to the serial console. Change the value as requested.

Looks fine but should go in to develop first.

Changing merge target


FindById() not working as expected

I may be doing something wrong; if so, please point it out. However, after closing and re-opening a database, FindById() does not find any documents.
package main

import (
	c "github.com/ostafen/clover"
	"log"
)

func main() {
        // This part is out of the README
	db, _ := c.Open("clover-db")
	db.CreateCollection("myCollection")

	doc := c.NewDocument()
	doc.Set("hello", "clover!")

	docId, _ := db.InsertOne("myCollection", doc)
	log.Printf("created document %s\n", docId)

	doc, _ = db.Query("myCollection").FindById(docId)
	log.Println(doc.Get("hello"))

        // Now, close the document, re-open it, and try to FindByID()
	db.Close()

	db, _ = c.Open("clover-db")
        // First, find by the ID created above
	doc, _ = db.Query("myCollection").FindById(docId)
	if doc == nil {
		log.Printf("didn't find the document %s\n", docId)
	}
        // That fails, so now find the document manually, get the ID from it, and try to find it
	ds, _ := db.Query("myCollection").FindAll()
	if len(ds) == 0 {
		log.Printf("didn't find any documents")
	}
	newDocId := ds[0].ObjectId()
	if docId != newDocId {
		log.Printf("the ids changed (%s != %s)\n", docId, newDocId)
	}
	doc, _ = db.Query("myCollection").FindById(newDocId)
	if doc == nil {
		log.Printf("didn't find the document (%s) I just loaded\n", newDocId)
	}
	db.Close()
}

What I do here is:

Tutorial code from README
Close the DB
Re-open it
Try to find the document previously inserted with FindById()
Find the document with FindAll(), and get the ID from it
Try to find the document using FindById() using the ID in step 5

On my end, once I close the DB no documents are able to be referenced by ID.
10126» go run .
2022/03/12 09:39:59 created document ca7e8241-52ff-4f17-84b6-cb947acbe581
2022/03/12 09:39:59 clover!
2022/03/12 09:39:59 didn't find the document ca7e8241-52ff-4f17-84b6-cb947acbe581
2022/03/12 09:39:59 didn't find the document (ca7e8241-52ff-4f17-84b6-cb947acbe581) I just loaded

Clover: github.com/ostafen/clover v0.0.0-20220302164508-28d538d46bc1
Go: go version go1.17.8 linux/amd64
Edit
I believe this is because the collection index is never updated except when documents are inserted. Specifically, the index is never refreshed when the collection is loaded -- or, at least, I can't find where this happens.

Hi, thank you for signaling this problem. As you correctly pointed out, the index was never reloaded from disk.
My last commit should fix the problem.
Let me know if it works!

Looks good; thanks for the quick fix!


bugfix: 避免openstack磁盘扩容成功后未能挂载到虚拟机上

这个 PR 实现什么功能/修复什么问题:
避免openstack磁盘扩容成功后未能挂载到虚拟机上
是否需要 backport 到之前的 release 分支:

release/2.8.0

/area region
/cc @swordqiu @yousong

Codecov Report

Merging #2451 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #2451      +/-   ##
=========================================
- Coverage    7.08%   7.07%   -0.01%
=========================================
  Files         456     456
  Lines       71749   71725      -24
=========================================
- Hits         5080    5073       -7
+ Misses      66249   66234      -15
+ Partials      420     418       -2

Flag
Coverage Δ

#aFlag
7.07% <ø> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/util/rbacutils/rbac.go
69.96% <0%> (-3.41%)
:arrow_down:

pkg/compute/models/purge.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/natstable.go
4.86% <0%> (+0.1%)
:arrow_up:

pkg/compute/models/natdtable.go
5.26% <0%> (+0.12%)
:arrow_up:

pkg/cloudcommon/cmdline/helper.go
47% <0%> (+0.42%)
:arrow_up:

pkg/appsrv/response.go
69.23% <0%> (+3.07%)
:arrow_up:

pkg/util/seclib2/seclib.go
95.74% <0%> (+4.25%)
:arrow_up:

Continue to review full report at Codecov.

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

/approve
/lgtm


Execution error

Command :
CacheName=dby ./scan4all -host test.com -csv -o 1.csv -v
Nmap done: 3 IP addresses (3 hosts up) scanned in 0.04 seconds
[INF] Port scan over,web scan starting9.17 ....
[DBG] Setting single path to "favicon.ico" and ignoring multiple paths settings
panic: interface conversion: interface is nil, not error

goroutine 3058 [running]:
github.com/hktalent/scan4all/lib/util.TestIs404({0xc0002d9aa1, 0x16})
        /home/runner/work/scan4all/scan4all/lib/util/config.go:382 +0xf8
github.com/hktalent/scan4all/projectdiscovery/nuclei_Yaml.RunNuclei(0x49cee60, 0xc000ee16e0, 0x10b45c6?, 0xc0004e5da0)
        /home/runner/work/scan4all/scan4all/projectdiscovery/nuclei_Yaml/nuclei_yaml.go:33 +0x6e5
created by github.com/hktalent/scan4all/pkg/naabu/v2/pkg/runner.(*Runner).Httpxrun.func2
        /home/runner/work/scan4all/scan4all/pkg/naabu/v2/pkg/runner/runner.go:86 +0xaa

download latest ver

@hktalent thanks,When the release version will be updated ?

V 2.7.0 wait 10 minutes


feat(example): migrate sha256 example to axiom-crypto

Due to the simplicity of the other sha256 implementation, it would be sensible to migrate this example rather than the current one.
https://github.com/axiom-crypto/halo2-lib/tree/feat/zkevm-sha256/hashes/zkevm/src/sha256

We can also consider using this instead of sha256:
https://github.com/scroll-tech/zkevm-circuits/tree/develop/keccak256

We can also consider using this instead of sha256:

We can, but we can't do it in a hurry, they're designed differently. It will take a couple of days to adapt the design when the input is already assigned at the time of the entry point to the step circuit


Fix SAWarning from sqlalchemy queries

After an fresh install on master, I had the following error when I started the webserver:
  util.ellipses_string(element.element))```

It is not critical but annoying. This PR fixed it.

Here is some details from sqlalchemy docs: http://docs.sqlalchemy.org/en/latest/changelog/migration_10.html#order-by-and-group-by-are-special-cases

Build errors seem to be not related to this PR

here: https://github.com/airbnb/airflow/pull/1070


Enable building of pdf version of docs

Resolved by 88c2df41575325ab22278cdb770030491937e266. All *.png files had to be exported again, because latex complained about files not being png.


chore: provide OSSF security insight

Clomonitor requires a SECURITY-INSIGHTS.yml,
See https://clomonitor.io/projects/cncf/cilium#cilium_best_practices

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

We should also be able to add a security-assessments section around https://github.com/cilium/cilium.io/tree/main/Security-Reports

I found a tool in ossf https://github.com/ossf/si-tooling/tree/main/python
I don’t know if it can be of any use.

Can’t it be done at the merge time ?

The way the repo is set up means that won't happen by default, but I can also just fix it up. I note that the CODEOWNERS also needs an update, and I can fix that while we're at it.

Thanks @mmorel-35 for the contribution!


use the ROW_IDX_COLUMN constant name instead of copying the value everywhere

https://github.com/huggingface/dataset-viewer/blob/3e4f278a65c04f239dfb26823493d1fda2800d53/libs/libapi/src/libapi/response.py#L13
See the occurrences:
https://github.com/search?q=repo%3Ahuggingface%2Fdataset-viewer+__hf_index_id&type=code
Maybe  do the same for __hf_fts_score too, and move the values to constants.py?

May I take this issue?

Sure! I just assigned you the issue. Please ping me when your PR is ready!


Negative usage quotas for running VMs and running memory in vCenter

Description
In some circumstances the running usage and quotas for running VMs and running memory have negative values. Ending in a situation similar to this:
$ onegroup show 123
...
VMS USAGE & QUOTAS - RUNNING

      RUNNING VMS       RUNNING MEMORY          RUNNING CPU
     -4 /       8   -16384M /    62.5G      8.50 /    16.00
...

It is necessary to run onedb fsck to fix them:
 onedb fsck
MySQL dump stored in /var/lib/one/mysql_localhost_opennebula_2020-7-29_13:51:3.sql
Use 'onedb restore' or restore the DB using the mysql command:
mysql -u user -h server -P port db_name < backup_file

User 1 is in Group 1 users id list, but it should not
User 31 quotas: RUNNING_CPU_USED has -3 	is	1.00
User 31 quotas: RUNNING_MEMORY_USED has -2048 	is	2048
User 31 quotas: RUNNING_VMS_USED has -1 	is	1
User 47 quotas: RUNNING_CPU_USED has -2.50 	is	0.00
User 47 quotas: RUNNING_MEMORY_USED has -14336 	is	0
User 47 quotas: RUNNING_VMS_USED has -4 	is	0
User 53 quotas: RUNNING_CPU_USED has 11 	is	12.00
User 53 quotas: RUNNING_MEMORY_USED has -2048 	is	2048
User 53 quotas: RUNNING_VMS_USED has 0 	is	2
Group 1 quotas: RUNNING_CPU_USED has 2 	is	0.00
Group 1 quotas: RUNNING_MEMORY_USED has 2048 	is	0
Group 1 quotas: RUNNING_VMS_USED has 1 	is	0
Group 123 quotas: RUNNING_CPU_USED has 8.50 	is	12.00
Group 123 quotas: RUNNING_MEMORY_USED has -16384 	is	2048
Group 123 quotas: RUNNING_VMS_USED has -4 	is	2

Total errors found: 16
Total errors repaired: 16
Total errors unrepaired: 0

To Reproduce
Start VMs and import wilds from a vCenter host.
Expected behavior
Running quotas must be real.
Details

Affected Component:
Hypervisor: vCenter
Version: 5.12

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Hi, looks like this issue is still existing.

Hello @feldsam,
is there any chance that you now how to reproduce it?
If the user doesn't had too many VMs, can you share all his VM logs?
Alternatively can you share DB and all logs?
If you are willing share the info, we will contact you via email with connections details for the upload

Hi @paczerny , thanks, yes I can share, this user have only one active VM with couple of VM deployed and terminated.


Create Running_large_problems_using_QBSolv.ipynb

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

On second thought, add to the README in a new PR

OK, will add it to the readme.


upstream: reference to optional lastEjectionTime() doesn't check it exists

Hi -- I don't have a repro for this but https://github.com/envoyproxy/envoy/blob/ef9139a991fb9f7df676430f626b32db93a06932/source/common/upstream/outlier_detection_impl.cc#L336 looks like it has a bug.
It should be checking to see if lastEjectionTime() has a value before taking the value.

Could eject() race with checkHostForUneject()? E.g. in thread 1:
eject calls host_.lock()->healthFlagSet(Host::HealthFlag::FAILED_OUTLIER_CHECK);
last_ejection_time is not set yet

then context-swtich to thread 2:
checkHostForUneject() is called
if (!host->healthFlagGet(Host::HealthFlag::FAILED_OUTLIER_CHECK)) { passes through
last_ejection_time is accessed

Is that plausible? If so we could mutex-protect a few member-vars...it's a little suspicious that there are some atomics here but no mutexes ensuring synchronization. It might be also possible to just re-order the statements in eject() to eliminate this, but it feels like then we should hold the ejection time in an atomic too.
....
Thinking about how this might be working a little more, this could also happen if there were two DetectorHostMonitorImpl which point to the same host, which is held in the structure as a weak_ptr<Host>. I think the use of weak_ptr here conveys a possibility that two different DetectorHostMonitorImpl instances point to the same Host, and thus the guard-statement that @junr03 referenced at the top of checkHostForUneject() may not be consistent with the DetectorHostMonitorImpl.
So...that seems to indicate there can only be one DetectorHostMonitorImpl per Host, but then I don't know why the weak_ptr.
And I'm still not sure whether in production two different threads may access checkHostForUneject() and eject().
@junr03 @mattklein123

Are we relying on exception propagation across libevent callbacks?
Or catching exceptions somewhere that might be thrown in the lastEjectionTime accessor?
It's possible the way we compile stuff locally might make that behave differently.

Here's the stack-trace from where we saw this crash:
std::__u::__throw_bad_optional_access
std::__u::optional::value
Envoy::Upstream::Outlier::DetectorImpl::checkHostForUneject
Envoy::Upstream::Outlier::DetectorImpl::onIntervalTimer
Envoy::Upstream::Outlier::DetectorImpl::DetectorImpl::$_0::operator()
std::__u::__invoke
std::__u::__invoke_void_return_wrapper::__call
std::__u::__function::__default_alloc_func::operator()
std::__u::__function::__policy_invoker::__call_impl
std::__u::__function::__policy_func::operator()
std::__u::function::operator()
Envoy::Event::TimerImpl::TimerImpl::$_0::operator()
Envoy::Event::TimerImpl::TimerImpl::$_0::__invoke
event_process_active_single_queue
event_process_active
event_base_loop
Envoy::Server::InstanceImpl::run
Envoy::MainCommonBase::run
...

I don't know much about this code path but I was guessing (without data) this might have been during shutdown.

Are we relying on exception propagation across libevent callbacks? Or catching exceptions somewhere that might be thrown in the lastEjectionTime accessor?

No, definitely not. I will take a look at the code later today or tomorrow. I would like to see if I can see the bug before we merge the other PR.


oozie coordinator input-event does not work

hi,
I have three coordinators A, B and C.
The coordinator of B and C depends on the output of A. That is, if the output of A is ready, coordinator of B and C will run.
So, I use an input-event to control such dependency.
The structure of coordinator B and C look like
<coordinator-app name="B" frequency="1440" start=${start} end=${end} timezone="UTC" xmlns="uri:oozie:coordinator:0.1">
   <datasets>
      <dataset name="input1" frequency="1440" initial-instance=${start} timezone="UTC">
         <uri-template>hdfs://localhost:9000/tmp/revenue_feed/${YEAR}/${MONTH}/${DAY}</uri-template>
      </dataset>
   </datasets>
   <input-events>
      <data-in name="coordInput1" dataset="input1">
          <instance>${coord:current(0)}</instance>
      </data-in>
   </input-events>
   <action>
      <workflow>
         <app-path>hdfs://localhost:9000/B/workflows</app-path>
      </workflow>
   </action>
</coordinator-app>

So, if hdfs://localhost:9000/tmp/revenue_feed/${YEAR}/${MONTH}/${DAY}/_SUCCESS is created, the coordinator B and C will be triggered to run their workflow.
the coordinator of A looks like:
<coordinator-app name="B" frequency="1440" start=${start} end=${end} timezone="UTC" xmlns="uri:oozie:coordinator:0.1">
   <action>
      <workflow>
         <app-path>hdfs://localhost:9000/A/workflows</app-path>
      </workflow>
   </action>
</coordinator-app>

its ${start} and ${end} are same as B and C.
The workflow of A will create hdfs://localhost:9000/tmp/revenue_feed/${YEAR}/${MONTH}/${DAY}/_SUCCESS
However, the coordinator of B and C are still waiting for the hdfs://localhost:9000/tmp/revenue_feed/${YEAR}/${MONTH}/${DAY}/_SUCCESS
Even though I use output event of coordinator of A, the workflow of B and C are still wanting for the created input dataset.
<coordinator-app name="B" frequency="1440" start=${start} end=${end} timezone="UTC" xmlns="uri:oozie:coordinator:0.1">
    <datasets>
        <dataset name="input1" frequency="1440" initial-instance=${start} timezone="UTC">
        <uri-template>hdfs://localhost:9000/tmp/revenue_feed/${YEAR}/${MONTH}/${DAY}</uri-template>
        </dataset>
    </datasets>

    <output-events>
        <data-out name="coordOutput1" dataset="output1">
            <instance>${coord:current(0)}</instance>
        </data-out>
    </output-events>
   <action>
      <workflow>
         <app-path>hdfs://localhost:9000/A/workflows</app-path>
      </workflow>
   </action>
</coordinator-app>

If I submit the workflow of A without its coordinator, then the workflow of B and C will be triggered.
I am not sure if something missing in my coordinator of A.
Thank you!

Can you please send it to user@oozie.apache.org?Bigger community.Oozie - Project Mailing Lists
|   |
|   |   |   |   |   |
| Oozie - Project Mailing ListsProject Mailing Lists These are the mailing lists that have been established for this project. For each list, there is a subscribe, unsubscribe, and an archive link. Name Subscribe Unsubscribe Post  |
|  |
| View on oozie.apache.org | Preview by Yahoo |
|  |
|   |
--Mohammad
 On Friday, September 4, 2015 7:49 AM, Ming-Hsuan-Tu<EMAIL_ADDRESS>wrote:

hi,I have three coordinator A, B and C.The coordinator of B and C depends on the output of A. That is, if output of A is ready, coordinator of B and C will run.So, I use an input-event to control such dependency.The structure of coordinator B and C look like

hdfs://localhost:9000/tmp/revenue_feed/${YEAR}/${MONTH}/${DAY}

${coord:current(0)}

hdfs://localhost:9000/B/workflows

So, if hdfs://localhost:9000/tmp/revenue_feed/${YEAR}/${MONTH}/${DAY}/_SUCCESS is created, the coordinator B and C will be triggered to run their workflow.the coordinator of A looks like:

hdfs://localhost:9000/A/workflows

its ${start} and ${end} are same as B and C.The workflow of A will create hdfs://localhost:9000/tmp/revenue_feed/${YEAR}/${MONTH}/${DAY}/_SUCCESSHowever, the coordinator of B and C are still waiting for the hdfs://localhost:9000/tmp/revenue_feed/${YEAR}/${MONTH}/${DAY}/_SUCCESSIf I submit workflow of A without coordinator, then the workflow of B and C will be triggered.I am not sure if something missing in my coordinator of A.Thank you!—
Reply to this email directly or view it on GitHub.


Broken after new AUTO1111 update.

Hello,
Since the new update from AUTO the extension stops working. Issue state:
File "I:\stable-diffusion-webui\modules[scripts.py](https://scripts.py/)", line 256, in load_scripts
script_module = script_loading.load_module(scriptfile.path)
File "I:\stable-diffusion-webui\modules\script_loading.py", line 11, in load_module
module_spec.loader.exec_module(module)
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "I:\stable-diffusion-webui\extensions\sd_webui_SAG\scripts[SAG.py](https://sag.py/)", line 18, in
from modules.script_callbacks import on_cfg_denoiser,CFGDenoiserParams, CFGDenoisedParams, on_cfg_denoised, AfterCFGCallbackParams, on_cfg_after_cfg
ImportError: cannot import name 'AfterCFGCallbackParams' from 'modules.script_callbacks' (I:\stable-diffusion-webui\modules\script_callbacks.py)
Hope you can fix it soon. Thank you.

FYI. Your stealth_pnginfo also broken due to new update.

Can confirm it stops working because script_callback.py and sd_samplers_kdiffusion.py need to be patched again.
After manually patching it because the patcher didn't work I managed to get it working.

That link didn't work for me so I've created my own patch here on GitHub's gists - https://gist.github.com/CamWiseOwl/eb13ba59e8750717f20db3af7f8364db
Looks like it's only sd_samplers_kdiffusion.py that needed updating for the latest update.

That link didn't work for me so I've created my own patch here on GitHub's gists - https://gist.github.com/CamWiseOwl/eb13ba59e8750717f20db3af7f8364db
Looks like it's only sd_samplers_kdiffusion.py that needed updating for the latest update.

Would you mind explaining briefly how I can apply this script? I don't mind looking up the finer details, I just need to understand where this script is applied. Thanks.

Strange, it works for me...
Here goes mediafire then - https://www.mediafire.com/file/ozddr7248gmecr0/patched.zip/file

That link didn't work for me so I've created my own patch here on GitHub's gists - https://gist.github.com/CamWiseOwl/eb13ba59e8750717f20db3af7f8364db
Looks like it's only sd_samplers_kdiffusion.py that needed updating for the latest update.

Would you mind explaining briefly how I can apply this script? I don't mind looking up the finer details, I just need to understand where this script is applied. Thanks.

Either download @Squibeel's zip and directly replace the files or save my gist somewhere then run git apply in the root A1111 folder like the readme describes how to do -
git apply --ignore-whitespace path/to/updated.patch

Strange, it works for me... Here goes mediafire then - https://www.mediafire.com/file/ozddr7248gmecr0/patched.zip/file

Thank you but still did not work.
Traceback (most recent call last):
File "I:\stable-diffusion-webui\modules\scripts.py", line 256, in load_scripts
script_module = script_loading.load_module(scriptfile.path)
File "I:\stable-diffusion-webui\modules\script_loading.py", line 11, in load_module
module_spec.loader.exec_module(module)
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "I:\stable-diffusion-webui\extensions\04.sd_webui_SAG\scripts\SAG.py", line 18, in
from modules.script_callbacks import on_cfg_denoiser,CFGDenoiserParams, CFGDenoisedParams, on_cfg_denoised, AfterCFGCallbackParams, on_cfg_after_cfg
ImportError: cannot import name 'AfterCFGCallbackParams' from 'modules.script_callbacks' (I:\stable-diffusion-webui\modules\script_callbacks.py)


(GH-531) Improve branch detection on github actions

During a normal branch build on GitHub actions, the branch reference
is prefixed with 'refs/heads' and should be removed from reference
to get the correct branch name. This pull request makes the change
necessary by checking the tempName if it starts with that string
and remove it if it exist.
This is also considered an enhancement of the code I previously provided
to detect build providers for GitHub Actions (which is currently unreleased code).

@AdmiringWorm thanks for doing this!


Add changes needed to onboard ART pipelines

These changes are missing in release-4.18 branch

/retest

/label cherry-pick-approved


json to UML doesn't seem to work

I use PlantUML among other things to visualize json data, as described here. The preview in VS Code works just fine.
However, to get this into my project documentation, I want to turn the .puml into .svg and include it in markdown files.
For this I use node-plantuml, and it worked just fine with my class diagrams, calling puml generate -s api-example.puml -o api-example.svg.
However, with my json data, inside the .uml I have:
@startjson
{
  ...
}
@endjson

This should be valid, but in the output .svg I get no visualization, but rather the text No @startuml found.

@vowstar I noticed that node-plantuml-back supports this feature, but the result is still bugged. I think bumping the plantumlVersion to something a bit more recent would fix it.
(manually replacing the .jar with the latest from sourceforge fixed it for me, but that's a manual step I would like to avoid in my setup; especially if I want to generate all UMLs as SVGs in CI, that would obviously not work)

@vowstar I noticed that node-plantuml-back supports this feature, but the result is still bugged. I think bumping the plantumlVersion to something a bit more recent would fix it.
(manually replacing the .jar with the latest from sourceforge fixed it for me, but that's a manual step I would like to avoid in my setup; especially if I want to generate all UMLs as SVGs in CI, that would obviously not work)

Sure. After testing, the old version does not work, the latest version is no problem.


[FeatReq] Add support for apps and libs that require two console to develop (like Textual)

Issue
Certain python apps and libraries require multiple terminal consoles open at the same time.
This is because the TUI interface is displayed in one console, while the debug messages are displayed in another console.
Textual is an example. Textual is the most popular TUI framework for Python, used to develop apps like Elia, Memray, Parllama and Posting.
The dev mode of Textual requires some special setup on two consoles. You can read about it here:
https://textual.textualize.io/guide/devtools/
https://github.com/Textualize/textual-dev
Currently Aider cannot monitor or display two consoles at the same time, and consequently developing Textual apps with it is impossible. Because the AI needs both the console outputs (the debug logs messages and the TUI snapshots) added to the chat to understand what is going on. This is also somewhat related to this issue: https://github.com/paul-gauthier/aider/issues/912 .
Please add support for displaying and adding to chat multiple consoles output.
P.S.
Writing tests for Textual is also difficult for the same reason. But you can use a tool like pytest-textual-snapshot to handle those UI tests in Aider once you implement the dual console output. Please add support for it. You can read about it here:
https://textual.textualize.io/guide/testing/
https://github.com/Textualize/pytest-textual-snapshot
Thanks! 😊 🙏
Version and model info
No response

Thanks for trying aider and filing this issue.
I don't think there is a natural way for aider to support this. Aider's /run command simply runs a program captures its stdout/stderr. That won't work with a TUI app at all that uses complex terminal control sequences.
You need a CLI program you can run that will simply dump the textual console logs and then exit. It doesn't look like textual console supports that, as it runs forever printing logs. You may be able to write such a CLI program without too much trouble (maybe even using aider to write it).

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.

Can you implement a plug-in system for Aider? I would be happy to write a plugin for letting Aider receive the log output from Textual (all I need to do is to adapt the code from here:  https://github.com/Textualize/textual-dev/blob/main/src/textual_dev/client.py ).


[Question] Database divergence after multiple isolated failures.

I'm not sure where to ask this. Since I'm using MariaDB operator + MaxScale so I decided to ask here. Please let me know if I need to ask this question somewhere else.
I am using MariaDB operator + MaxScale to set up a database cluster with 2 instances. I've set it up and currently am testing it. My testing steps are as follows:

Bring the primary database instance down. MaxScale automatically detects this and promote the other instance to master.
Write some data to this new master.
Bring this master down, now the cluster has no databases active.
Bring the original master up, MaxScale detects this and promotes it to master. However, since the other instance is down, it is not aware of the new data written.
Bring the other instance up, MaxScale detects this and adds this instance as slave to the database cluster.

Now there's new data written in the slave, but they are not present in the master. I think this outcome makes sense, but I would like to avoid it at all costs.
For example, at step 5, the primary instance should detect there's some data written to the slave and not present in the primary. It then should either copies this data over, or MaxScale should instead promote the slave to be the new master.
How can I avoid this database divergence?

Would it make sense to add logic to properly select a new master after all nodes are down except master, followed by that master is also down? For example, in my scenario, let's call my instances database A (master) and database B (slave).

A is down -> B becomes master (Only 1 database left and that is also master).
Applications connect to B and write some new data.
B is down -> no active databases.
A is up - add logic here so that since the last master is B and it was the only database left in the cluster, there's a high chance that there's new data written to B that only B knows, so A shouldn't be promoted to master
B is up -> B becomes master and A replicates data from B.

Also, for what it's worth, I've also tried setting auto_rejoin = false and expected that nodes after failure shouldn't be rejoined to the cluster automatically. However, they still join the cluster and my problem still persists.

Hey there @HuyVo2112 !
Thanks a lot for your detailed explanation. This question should be directed to the MaxScale team, as it has to do with its internals, you can reach them out in our Slack channel:
https://r.mariadb.com/join-community-slack


atuin search -i can't enter result into terminal

As the title says, when I press enter in the search ui, I can't paste the content to the terminal
atuin 0.9.1
GNU bash, version 5.1.16
https://user-images.githubusercontent.com/25676220/165783956-d565cd28-88c6-4535-9cb5-780c447d0e69.mp4

The search TUI isn't really intended to be called manually. Because of how the rendering works, the command you select isn't written to stdout as you might expect
If you want to call it yourself, you might find this useful: https://github.com/ellie/atuin/blob/7cdd00b56fbc937316e6669960792d807ad7e308/src/shell/atuin.zsh#L37

The search TUI isn't really intended to be called manually. Because of how the rendering works, the command you select isn't written to stdout as you might expect
If you want to call it yourself, you might find this useful:
https://github.com/ellie/atuin/blob/7cdd00b56fbc937316e6669960792d807ad7e308/src/shell/atuin.zsh#L37

Thanks, I know why


Ceplas session01

add slides from last week's ceplas demo
not added to sidebar yet, check out in watch-mode:

http://<IP_ADDRESS>:8080/docs/teaching-materials/disseminations/2023-05-09_CEPLAS-StartYourARC-S01/2023-05-09_CEPLAS-StartYourARC-S01.html

S.31: I belong to Team Freiburg and already changed it

Sorry and thanks!


Installation problem - 404 error '<EMAIL_ADDRESS>is not in the npm registry

Describe the bug
npm install gives 404 error for media-engine node_module
Expected Behaviour
npm install succeeds without any error
Steps to Reproduce the Problem

Download the repo as zip file
Run the command - npm i @voxeet/voxeet-web-sdk @voxeet/react-components<EMAIL_ADDRESS>--save
Below error is shown

npm WARN ERESOLVE overriding peer dependency
npm WARN Found: @voxeet/media-engine@undefined
npm WARN node_modules/@voxeet/media-engine
npm WARN
npm WARN Could not resolve dependency:
npm WARN peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN node_modules/@voxeet/voxeet-web-sdk
npm WARN   peer<EMAIL_ADDRESS>from the root project
npm WARN   1 more (@voxeet/react-components)
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/@voxeet%2Fmedia-engine - Not found
npm ERR! 404
npm ERR! 404  '<EMAIL_ADDRESS>is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.
npm ERR!      C:\Users\USER\AppData\Local\npm-cache_logs\2021-02-26T06_54_04_852Z-debug.log

Please upgrade to the latest version of the sdk:
<EMAIL_ADDRESS>

@FabienLavocat Perhaps it's best to re-open this. @voxeet/react-components itself is still expecting v2.3.0 to be installed.
Force install gets the job done, but it would be better to update the expected dependency.

@wg-daniel I created an internal ticket to have the team fixing the dependency version. Thanks!

Will be fixed with #107


[Feature] Add option to auto-generate drive parameters when using manual QEMU arguments setting

Describe the bug
If I enable Advanced: bypass configuration and manually specify arguments in QEMU settings, my VM doesn't boot because the Drives configuration is not passed to QEMU anymore.
Additionally on iOS, if I manually specify the drive path as an argument, the VM doesn't boot (maybe because the drive file isn't opened for read/write by the app; and the path can't be outside the sandbox).
Proposed Fix
It could be fixed by showing another toggle if the Advanced: … is set: Automatically add Drive and Sharing arguments according to configuration.
Work already in progress
I am already in the process of implementing the proposed fix. Here's a screenshot showing the added toggle:

There’s been a couple of proposals to improve the custom args page and my feeling is that the advanced args is only for debugging (testing qemu basically). If there’s anything you want to do that isn’t possible, there should be a direct UI way of doing it.

I was trying to get a macOS 10.11 VM set up but it needed custom arguments. On second thought, maybe it would have been sufficent to put these arguments at the end as Custom Arguments. More generally I do think it's a bummer that (AFAIK) no VM can be booted with entirely custom arguments on iOS.

Looking far ahead I want UTM to not be completely married to QEMU. There could be options for other backends so I don’t want all these .utm floating around that depend on qemu arguments.

That's fair. I'll go ahead and close this but leave a hint for future reference. Here's how to include the drive configuration when using the custom QEMU arguments setting:
In UTM, the QEMU arguments are built on the fly when the VM starts. So to force UTM to include a set of configuration arguments even with the custom toggle enabled, go to file /Managers/UTMQemuSystem.m and in startWithCompletion add the necessary calls in an else block:
- (void)startWithCompletion(void (^)(BOOL, NSString * _Nonnull))completion {
    [self argsRequired];
     if (!self.configuration.ignoreAllConfiguration) {
         [self argsFromConfiguration];
+    } else {
+        // add args for drives and sharing because those are impossible to type
+        [self argsForDrives];
+        [self argsForSharing];
     }
     [self argsFromUser];
     [self startQemu: …];
…


Aimed Shot not working

Multi Shot is working but not aimed shot

Full error for me:
Date: 2019-09-09 01:02:45
ID: 1
Error occured in: Global
Count: 1
Message: ..\AddOns\YaHT\YaHT.lua line 280:
   attempt to perform arithmetic on local 'castTime' (a nil value)
Debug:
   YaHT\YaHT.lua:280: ?()
   YaHT\YaHT.lua:399:
      YaHT\YaHT.lua:394

May have been introduced with #25 (cc @Pat-). In my testing, spellID is returning 0 from CombatLogGetCurrentEventInfo (not sure why), so castTime on the next line is assigned as nil.
Possible workaround / fix: At line 228 in YaHT.lua, change spellID to name.


NIOConcurrencyHelpers: make LLP64 friendly

On LLP64 platforms, Int is mapped to intptr_t and UInt is mapped
to uintptr_t.  Use the newly minted CNIOAtomics operations to
support these semantics.
[One line description of your change]
Motivation:
[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]
Modifications:
[Describe the modifications you've done.]
Result:
[After your change, what will change.]

Presumably based on #1401.


Update CMake readme for changed dependencies

Previously, oeenclave depended on oelibc, so oelibc never needed to be specified. However, as this dependency was removed, users must specify oelibc too. This is especially important for C enclaves, where the user won't include oelibcxx, which would otherwise pull it in (and is the reason this wasn't caught by the samples).

Thanks for catching this with your C sample @paulcallen.

@paulcallen and I are adding some more changes to this PR to update the readme further.

bors r+


Added js_vars "style" function

Added a new js_vars function type style to control ex. a:hover and
pseudo-elements etc. on transport="postMessage" that cannot be changed with JS directly. Adds a style tag to the head of the customizer preview and adds the code from js_vars element and property. Updates the code on option change.

Excellent!
I Merged it just now, thanks!

Thanks :+1:

This is brilliant thank you!


Update ffp plugin

Description
Updates FFP plugin as per patches from Olivier Cessenat. The plugin uses a non-BSD licensed third party package. It does so by dynamically loading it if it is installed in VisIt's lib dir or if the user specifies an env. variable, VISIT_FFP_STRIPACK_PATH.
How Has This Been Tested?
Olivier provided new data files, .py code and test baselines. I confirmed my modifications to Olivier's patch produce the same images and added baselines.
The only remaining question is how to host the stripack library and document to users how to obtain it (as well as its license terms).

The only remaining question is how to host the stripack library and document to users how to obtain it (as well as its license terms).

Might be a good idea to add this as an extra ticket in order to track this.

Some of the lines run a bit long. It might be nice to split this into multiple lines where we can to follow style guidelines.

I neglected to mention...most of this code is Olivier's. I just added the dynamic loading of stripack. I don't want to change the code too much from what Olivier sent and so should have indicated this to reviewers.
I see @rusu24edward already approved. But, @biagas if you review, don't worry about reviewring more than up to line 145 of avtffpFileFormat.C


Incompatibility with OrganizablePlayScreens

Hey!
There is an incompatibility with OrganizablePlayScreens's folders, when i enter in one of it, Multiplayer-Button make me quit the current server instead of display the list of associated servers.

I just tested and I could indeed reproduce it. However by seeing what it does it may have to be fixed by the other developer. I will look into it more and let you know.

Do you know how to build java projects? I think I have fixed it but I would like you to test before I release it. I have committed it to the repository, let me know, thanks

I've just compiled the mods and this version has fixed the bug, thank you very much!

No worries, enjoy


Snapshots didn't save certificates

When I restore the backup, not found certificates.

This should be working on newer releases. Feel free to reopen if needed.


Make use of Grafana HTTP client

Refactor backend to make use of the HTTP client from grafana-plugin-sdk-go rather than the client from the http package. This allows us to use middlewares that enable forwarding of auth headers.

Testing with TLS. @bossinc will help


New release?

Hi, would it be possible to cut a new PyPI release to incorporate some recent changes? I'm mainly interested in a release that ships with https://github.com/asher/chronos-python/pull/39

done. 1.1.0 should be in pypi now


'CAGradientLayerType' is only available in macOS 10.14 or newer - XCode 16 beta

:rocket:
Describe the bug
SwifterSwift cannot compile
To Reproduce
Use Xcode 16 beta and compile the package
Expected behavior
SwifterSwift compiles without errors
Screenshots

Desktop (please complete the following information):

OS:  macOS 14.6 (23G80)

Additional context
Add any other context about the problem here.

[x] I've read, understood, and done my best to follow the Contributing guidelines before opening this issue.

Fixed in #1197

Hello,
Any indication on when this will be released please ?

@so-creations we were waiting for macOS 15 and Xcode 16 to be available. I'll prepare a release soon.

@so-creations releasing version 7.0.0 now.

Thanks a lot @guykogus! Keep up the good work ;)


Update JSON-API documentation

fixes #3742
draft mode for now while I look for typos and test the links and menus
I'm not confident I have all of the endpoints captured here yet.  Please comment if you see any missing.

I think this may be ready for a first review now.

This looks great so far!
I made some changes so that the examples can be expanded underneath. It's based on this. If you could transform the other things into that format, that would be great!

Yes, this looks great!
I've made some tweaks to the first two endpoints, if you could apply those to the rest as well.
Also could you apply the same section to the Recursion section of the docs and update the examples as well? I think we're missing some. Thanks!

Yes, this looks great!
I've made some tweaks to the first two endpoints, if you could apply those to the rest as well.

sure, I've applied those changes throughout.

Also could you apply the same template to the Recursion section of the docs and update the examples as well? I think we're missing some. Thanks!

ok - should I make a new PR or would you like it in this one?

close/reopen to restart faulty tests

ok - should I make a new PR or would you like it in this one?

You can put it in this one

thanks Raph! I think it looks good, but when I do the rich diff display, the include doesn't look like it is rendering correctly.  Perhaps the viewer isn't capable?  Not sure...

Also, I saw one other thing that needed to be removed so I took it out.

The include statement is for the mdbook builder. You can run it and display the docs with just serve-docs


feat(app): update historical protocol run table and drawer

Closes AUTH-457
Overview
Update HistoricalProtocolRun component and expanded HistoricalProtocolRunOffsetDrawer to accommodate protocol files and new designs.
Test Plan

navigate to a device details page and scroll down to Recent Protocol Runs
verify that HistoricalProtocolRun component matches new designs
verify that clicking anywhere along the run item expands the files + offset drawer
verify that Histori <img width="668" alt="Screenshot 2024-06-20 at 9 41 01 AM" src="https://github.com/Opentrons/opentrons/assets/47604184/b64c8923-cbba-411e-9573-d45c22fadff9"> calProtocolRunOffsetDrawer component matches new designs, including for no file/offset data

Changelog

add new translation keys
update HistoricProtocolRun to match designs, including addition of "Files" column
expand HistoricProtocolRunDrawer to display files associated with run

Review requests
auth js
Risk assessment
low

@ncdiehl11
let's use useFeatureFlag


functools.partial on cython's cdef functions ?

==> Ask Cython team for guidance / ideas

Functionality via a cython cdef class?

DONE! The anwser is to use scipy.LowLevelCallable.


[REFACTOR] Shrink Longhorn container image

Is your improvement request related to a feature? Please describe

Some Longhorn container images are fairly big.
harv41:~ # crictl image ls | grep "longhorn"

docker.io/longhornio/backing-image-manager                                  v1.4.3                                     db916c3638c1c       353MB
docker.io/longhornio/csi-attacher                                           v3.4.0                                     03e115718d258       56.2MB
docker.io/longhornio/csi-node-driver-registrar                              v2.5.0                                     cb03930a2bd42       20.9MB
docker.io/longhornio/csi-provisioner                                        v2.1.2                                     0f0a0f7990768       52.9MB
docker.io/longhornio/csi-resizer                                            v1.3.0                                     1df30f0e25552       55.4MB
docker.io/longhornio/csi-snapshotter                                        v5.0.1                                     53ae5b88a3380       56.5MB
docker.io/longhornio/livenessprobe                                          v2.8.0                                     e899260153aed       19.6MB
docker.io/longhornio/longhorn-engine                                        v1.4.3                                     97b06aa073605       749MB  // note
docker.io/longhornio/longhorn-instance-manager                              v1.4.3                                     58e18670be17a       751MB  // note
docker.io/longhornio/longhorn-manager                                       v1.4.3                                     f70f099c6183f       282MB
docker.io/longhornio/longhorn-share-manager                                 v1.4.3                                     4e4f3ddc7424c       202MB
docker.io/longhornio/longhorn-ui                                            v1.4.3                                     16f0d1cc414ba       209MB
docker.io/longhornio/support-bundle-kit                                     v0.0.25                                    53f2d6ac20fa1       415MB

For example, the instance manager container image has make, gcc
harv41:~ # kk exec -i -t -n longhorn-system instance-manager-e-ee7f9a38f72f904d35a052dde1f403e4 -- /bin/sh

sh-4.4# make --help
Usage: make [options] [target] ...
Options:
  -b, -m                      Ignored for compatibility.
  -B, --always-make           Unconditionally make all targets.

sh-4.4# gcc  --help
Usage: gcc [options] file...
Options:

sh-4.4# which gcc
/usr/bin/gcc

sh-4.4# which make
/usr/bin/make

From the instance manager docker file, it has such blocks:
https://github.com/longhorn/longhorn-engine/blob/8d6d46b7a70cd882213ca2f4debec70f300e66e7/package/Dockerfile#L22C1-L36C17
# Build liblonghorn
...
# Build TGT

The final docker image is a mixture of building environment and running process.
Describe the solution you'd like

Decouple the building and running, shrink the published container image size.
Describe alternatives you've considered

Additional context

It will be handled in https://github.com/longhorn/longhorn/issues/5911 and optimized by the multi-statged build like https://github.com/longhorn/longhorn-instance-manager/commit/1158c45467d94dde9d29cff5a887c2704d88e3f9.
cc @ChanYiLin

@derekbit Thanks, good to know this is on track. Your commit https://github.com/longhorn/longhorn-instance-manager/commit/1158c45467d94dde9d29cff5a887c2704d88e3f9. has been standing for some days. Hopefully it can be merged soon.
Harvester is more sensitive to Container image size.
cc @bk201 @markhillgit

@w13915984028
https://github.com/longhorn/longhorn-instance-manager/commit/1158c45467d94dde9d29cff5a887c2704d88e3f9 is merged. I mean all components can be optimized like it.  :)

are they also backported?

It only stays in Longhorn v1.5+

Closing as a duplicate of https://github.com/longhorn/longhorn/issues/5911.


Add a config to make all fields Optional

Discussed in https://github.com/samuelcolvin/pydantic/discussions/3089

Originally posted by christophelec August 11, 2021
Proposal : adding a Config key to set all fields of a model as Optional
Context :
In FastAPI, the OpenAPI spec is defined via Pydantic models.
To create a partial update endpoint in FastAPI, I'd like to be able to create a Model derived from another, but with all fields Optional, without having to manually duplicate all fields.
For example, let's say I have an API dealing with user, and creating a user takes as input :
class User(BaseModel):
  first_name: str
  last_name: str
  age: int
  address: str

If I want to have an endpoint to be able to partially update the user (e.g. change only the first_name), I'd like an endpoint where I can send :
{"first_name": "NEW"}

To have this behavior, currently I have to either :

Create a specific model for patching, with all fields set as Optional :

class PatchedUser(BaseModel):
  first_name: Optional[str]
  last_name: Optional[str]
  age: Optional[int]
  address: Optional[str]

Set all fields as Optional in the first model, and then add back manual validation down the road in the context of a user creation (where the fields are mandatory) (https://fastapi.tiangolo.com/tutorial/body-updates/#partial-updates-with-patch)

The first solution results in a lot of duplication, and therefore a lot of potential bugs.
The second one makes the schema less clear, and validation less easy.
Proposed solution
I would like to be able to derive a model from this base model, but switching all the fields to Optional. A possible usage could be :
class PartialUser(User):
  class Config:
    all_optional = True

This way, the PartialUser would keep all the fields from User, but setting them as Optional.
The result in FastAPI would look like this :
@router.post("/users/")
async def create_user(
    user: User,
):
    # All fields are mandatory
    return create_user(user)

@router.patch("/users/{user_id}/")
async def update_user(
    user_id: int,
    user: PartialUser,
):
  # Only fields to modify will be kept
  return update_user(user.dict(exclude_unset=True))

I'm willing to open a PR if you think this is a good idea.

Hello @christophelec
In the meantime you can see https://github.com/samuelcolvin/pydantic/issues/2272#issuecomment-771205557.
For feature requests, better use the discussions section btw

Hello @PrettyWood, I opened the discussion last week but it did not trigger much discussion except a :+1:, so I went ahead and opened the issue, as well as started a PR as it would be really useful on my side.
The PR is here : https://github.com/samuelcolvin/pydantic/pull/3121, and I just saw your comment about class kwargs, which indeed would be nice : I'll update accordingly.
I'm ok to close the issue

Per discussion in #3179, the current advice is to wait for v2

I don't think we'll have a special solution for this in v2. This is just not possible to express to Python's type system. The best I could come up with is the following, but I think it would work in v1 as well so it's nothing new:
from copy import deepcopy
from typing import Any, Optional, Tuple, Type, TypeVar

from pydantic import BaseModel, create_model
from pydantic.fields import FieldInfo

class User(BaseModel):
  first_name: str
  last_name: str

def make_field_optional(field: FieldInfo, default: Any = None) -> Tuple[Any, FieldInfo]:
  new = deepcopy(field)
  new.default = default
  new.annotation = Optional[field.annotation]  # type: ignore
  return (new.annotation, new)

BaseModelT = TypeVar('BaseModelT', bound=BaseModel)

def make_partial_model(model: Type[BaseModelT]) -> Type[BaseModelT]:
  return create_model(  # type: ignore
    f'Partial{model.__name__}',
    __base__=User,
    __module__=User.__module__,
    **{
        field_name: make_field_optional(field_info)
        for field_name, field_info in User.model_fields.items()
    }
    )

PartialUser = make_partial_model(User)

print(PartialUser(first_name='Adrian'))
#> first_name='Adrian' last_name=None

Can we reopen this for v2? In typescript, it's just Partial<User> 🤷🏻♂

We'll support it as soon as python has a type that allows us to express this in a way that type checkers can understand.
If you really want this, I could post on discuss.python.org advocating for this feature in the language, post the link here and I'll reply and support it.
You're best option might be to use a typeddict where you can set total=False, though I still don't think there's a way to "partialise" an existing model.

Thanks @samuelcolvin for responding.

If you really want this

Yes, I honor the work you are doing (as I'm also the OSS dev) and if you can raise the query upstream (python) it will be a great addition to pydantic because people (mostly FastAPI devs) always wanted this. I'll do my best to support this query.
I tried typedDict with my knowledge but I failed because I wanted to "partialise" the class inherited from BaseModel just like you mentioned.
Best Regards,
JD

I don't think we'll have a special solution for this in v2. This is just not possible to express to Python's type system. The best I could come up with is the following, but I think it would work in v1 as well so it's nothing new:
from copy import deepcopy
from typing import Any, Optional, Tuple, Type, TypeVar

from pydantic import BaseModel, create_model
from pydantic.fields import FieldInfo

class User(BaseModel):
  first_name: str
  last_name: str

def make_field_optional(field: FieldInfo, default: Any = None) -> Tuple[Any, FieldInfo]:
  new = deepcopy(field)
  new.default = default
  new.annotation = Optional[field.annotation]  # type: ignore
  return (new.annotation, new)

BaseModelT = TypeVar('BaseModelT', bound=BaseModel)

def make_partial_model(model: Type[BaseModelT]) -> Type[BaseModelT]:
  return create_model(  # type: ignore
    f'Partial{model.__name__}',
    __base__=User,
    __module__=User.__module__,
    **{
        field_name: make_field_optional(field_info)
        for field_name, field_info in User.model_fields.items()
    }
    )

PartialUser = make_partial_model(User)

print(PartialUser(first_name='Adrian'))
#> first_name='Adrian' last_name=None

Since we can't really do much better than that without the Python ecosystem reaching a consensus and adding typing support for a partial operator, I'm closing this as a wontfix.

@adriangb Thank you so much for the solution. Seems like Pydantic v1.10 throws TypeError: Object of type 'ModelField' is not JSON serializable when I try to callschema_json() on the PartialUser object.
However, when testing with Pydantic v2.0.3, the error is no longer thrown. I'm not sure why the older version of Pydantic was throwing the error. In any case, I plan to update to the latest version, so it's not a major concern for me. I just wanted to post the issue here so that anyone else facing the same error with older Pydantic versions will know that updating might resolve it.

Has anyone actually tried the code? I had to replace
    __base__=User,
    __module__=User.__module__,

with
    __base__=model,
    __module__=model.__module__,

This works just fine. One potential improvement: The @adriangb solution will override all default values of the original model and set them to None. If you want to preserve defaults you need to modify accordingly:
import pydantic_core

...

def make_field_optional(field: FieldInfo, default: Any = None) -> Tuple[Any, FieldInfo]:
  new = deepcopy(field)
    if not isinstance(field.default, pydantic_core.PydanticUndefinedType):
        new.default = field.default
    else:
        new.default = default
  new.annotation = Optional[field.annotation]  # type: ignore
  return (new.annotation, new)


Before create event event to reject or accept creating events

Hi,
Is it possible to reject an event after the user has selected the time slot he needs?
i.e. I want to validate the event and see if there is an overlapping with a maximum of 2 events, if so, I want to reject creating a new event.
To summarize, do we have such events:

before dragging or creating to reject the creation e.g. I want to allow a maximum of 1 event at a time i.e. the user can create one and only one event at a time.
before creating but after dragging to reject or accept an event i.e. if an event meets some condition, I want to accept it, otherwise, I want to reject it.
on create an event successfully so I can know which event is created (only new events, not all events)

Hi again,
I also need to reject some resizing event. i.e. if resizing and meet some condition, I want to revert the event to its original state.

Hi again,
I also need to reject some resizing event. i.e. if resizing and meet some condition, I want to revert the event to its original state.

Hi!
Try the onEventCreate prop. If you return false, then the creation is cancelled.
For the resizing: event-duration-change event
For the dragging: event-drop
You can find all the informations in the documentation.

Thanks for your answer.

Try the onEventCreate prop. If you return false, then the creation is cancelled.

I meant at the time of clicking. e.g. if a user clicks at a specific time, I don't want to accept his click and show him some feedback that he can't select this time.
this must happen before dragging.

For the resizing: event-duration-change event

Can I reject a resizing and revert the event back to its original state?
I'm sorry but I couldn't find such things in the documentation.

You can find everything here:
https://jsfiddle.net/y3qu9sm5/2/
I hope it helps :)

Duplicate of #398, please continue thread in there.


vgg16.pkl without aws cli

I understand it's available for aws cli but i dont get a available aws cli account (no credit card as a student ),so i'm wondering may you provide a dropbox link for vgg16.pkl ? Appreciate that.

Hi,
I am trying to get access to the files using S3 browser as suggested in the readme.
I am stuck after creating an account there. How to finally get the data (using requester pays buckets!).
Please help.


Add-Website Template plus Home Page Build

 Generate an astro template for the creation of the company website

 Build the Company Website Home Page, Nav Bar and Footer


Deep put

Currently put [immutably] only sets the value to the object path. I was wondering if it would be possible to merge the new object to the current.
Sample Data
var user = {
  name: 'Brian Mae S. Bodollo',
  mother: {
    name: 'Marian Rivera',

    mother: {
      name: 'Claire Borromeo',

      daughters: [{
        name: 'Marian Rivera'
      }]
    }
  },
};

Current
var updated = put(user, {
  'mother.mother.daughters.0.age': 5,
  'mother.mother.daughters.0.height': 6,
  'mother.mother.daughters.0.width': 6
});

Suggestion:
var updated = put(user, 'mother.mother.daughters.0', {
  age: 5,
  height: 6,
  width: 6
});

Maybe we could have a new method (named update) instead of updating the functionality of put?
Let me know if I missed out anything. Thanks!

Quick reply for now. I recently fixed a bug with put. I think the example you provided should work as expected with the latest version of 101.
Try it out and see if the problem remains:
'npm i --save 101@latest'
Lmk,
Typed using my thumbs..

On Aug 25, 2016, at 2:24 PM, Kier Borromeo<EMAIL_ADDRESS>wrote:
Currently put [immutably] only sets the value to the object path. I was wondering if it would be possible to merge the new object to the current.
Sample Data
var user = {
name: 'Brian Mae S. Bodollo',
mother: {
name: 'Marian Rivera',
mother: {
  name: 'Claire Borromeo',

  daughters: [{
    name: 'Marian Rivera'
  }]
}

},
};
Current
var updated = put(user, {
'mother.mother.daughters.0.age': 5,
'mother.mother.daughters.0.height': 6,
'mother.mother.daughters.0.width': 6
});
Suggestion:
var updated = put(user, 'mother.mother.daughters.0', {
age: 5,
height: 6,
width: 6
});
Maybe we could have a new method (named update) instead of updating the functionality of put?
Let me know if I missed out anything. Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yea, I'm using the latest version. It doesn't seem to be fixed 😕  (Demo).
For clarification, the end result (to the json mentioned previously) should be:
console.log(user.mother.mother.daughters[0], updated.mother.mother.daughters[0]);
// { name: 'Immaculate Borromeo' }
// { name: 'Immaculate Borromeo', age: 5, height: 6, width: 6 }

I checked the tests, and didn't find any test about this scenario.

I see what you are saying now. Sorry, looked at that too quick on my phone.
Let me think about this a bit. Would keypather.flatten be sufficient for your needs:
var keypather = require('keypather')()
var put = require('101/put')

var update = keypather.flatten({
  age: 5,
  height: 6,
  width: 6
}, '.', 'mother.mother.daughters.0')
put(user, update)

Yea. But to simplify even more, I was wondering if we could achieve the ff:
var update = require('101/update');

var updated = update(user, 'mother.mother.daughters.0', {
  age: 5,
  height: 6,
  width: 6
});

Similar to `put but merges the object.

Could we extend '101/assign'?

Assign current only supports a depth of 1

And doesn't support keypaths

I think we could do that, but I think it could complicate assign in both functionality and API. IMO, I would prefer a new function like update :D.

The plan for assign is to possibly deprecate it in v2.0.0.
Actually to get this in before then how about '101/extend'. I am traveling this week, but happy to take a PR.
Typed using my thumbs..

On Aug 26, 2016, at 10:26 PM, Kier Borromeo<EMAIL_ADDRESS>wrote:
I think we could do that, but I think it could complicate assign in both functionality and API. IMO, I would prefer a new function like update :D.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Oh, how about we name it merge? _.merge is like _.assign but recursively merges the object.

👍
Typed using my thumbs..

On Aug 27, 2016, at 1:25 AM, Kier Borromeo<EMAIL_ADDRESS>wrote:
Oh, how about we name it merge? _.merge is like _.assign but recursively merges the object.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Query Frontend splitting metrics check is flaky

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
Fixes #6951, by using receiver instead of prometheus instance.

Verification

@yeya24 , for the store tests, what is the directory string that I should specify, there is one instance, where they use e.ShareDir()/tmp, but when used this, I don't think receiver can access data from there, is there some additional config for this?
https://github.com/thanos-io/thanos/blob/d388b7403ddc87eda29a51ad8577a33880bc6b87/test/e2e/query_frontend_test.go#L601

@pawarpranav83 You can just go ahead and change this code to use remote write to ingest data instead. It should be the same as creating the block directly

ok, so only one query is failing -
{
        name:           "binary expression",
        qryFunc:        func() string { return `http_requests_total{pod="1"} / http_requests_total` },
        expectedSeries: 2,
},

https://github.com/pawarpranav83/thanos/blob/8889222e610f032889bcb555385c7a60c1b38577/test/e2e/query_frontend_test.go#L872-L876
How do we solve this?

ok, so only one query is failing -
{
        name:           "binary expression",
        qryFunc:        func() string { return `http_requests_total{pod="1"} / http_requests_total` },
        expectedSeries: 2,
},

https://github.com/pawarpranav83/thanos/blob/8889222e610f032889bcb555385c7a60c1b38577/test/e2e/query_frontend_test.go#L872-L876
How do we solve this?

just a guess, but can you try sorting the results before comparing? Can you print the results too please, im curious what is different!

It was the sample value since it was 0, it was giving NaN, when changed to 1, it worked :)
So changing it in all tests now.

And on a completely unrelated topic, do u know any talks that explain the working of the store gateway (mainly the caching system), I couldn't find any so...

It was the sample value since it was 0, it was giving NaN, when changed to 1, it worked :)

Good catch!

re-run test?
And are there any more corrections / changes to be done?


Not an issue, a possible contribution...

Hi there
I didn't know how to contact you directly.
Your geonames code is great, but we also need to be able to search by postcode. So I hope you don't mind but I've modified your code to search https://secure.geonames.org/postalCodeSearchJSON as well.
Would you like a copy?
Regards
Gemma

@yemling We're glad this is useful for you.
A Pull Request with your changes would be the best way for us to consider how to add your changes into the plugin.  Would you be willing to submit one?
This is a great way to get in touch with us to discuss this issue.  Do you need direct contact info?

Thanks for getting back to me.
I'm sorry, I'm pretty useless when it comes to github. If you can walk me through a pull request, then I'm more than happy to help.

Resolved via PR some time ago


htop: 2.2.0 -> 3.0.0

This is strictly speaking a fork of htop (https://htop.dev/) - see https://github.com/hishamhm/htop/issues/992 for more details, however considering Debian, Ubuntu, Arch and Gentoo have replaced their htop package with the forked version of it, this should be fine too for nixpkgs. If it should be a separate package anyway, feel free to tell me.
Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-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)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval

@ofborg build htop

The homepage should be changed to https://htop.dev/.


Update pyiron_base and pyiron_atomistics

Results in a conflict with pyiron_continuum: @muh-hassani Can you take a look at this?


Updated UVC camera component to support SSL connections

Description:
Added support for SSL in UVC component (added to latest version 0.11.0)
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#7690
Example entry for configuration.yaml (if applicable):
 # Example configuration.yaml entry
 camera:
  - platform: uvc
    nvr: IP_ADDRESS
    key: API_KEY
    ssl: USE_SSL # new field

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @carlos-sarmiento,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


Bump rexml from 3.2.4 to 3.2.5

Bumps rexml from 3.2.4 to 3.2.5.

Release notes
Changelog
Commits

Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>Co-authored-by: dependabot[bot]<EMAIL_ADDRESS>

Going to merge this automatic PR from dependabot that happened on my fork.


Update Doxygen redirect to use /latest on new site

Even if its a trivial change please tag me before merging to keep me in the loop. We are on this together

Ah, sorry, since this wasn't stable yet, I wanted to iterate to get it fixed.
Now that this is stable, I'll certainly wait for your approval before any further updates.