NNI.Experiment; Optimization mode gets internally set to 'maximize' when working on remote device

Describe the issue: Im fairly new to programming and NNI so maybe Im missing something. I want to launch NNI via script from python using nni.experiment. When Im working on my local machine everything seems to work well. But if i start NNI locally using a remote-desktop tool (still using training_service='local') the optimize_mode from exp.config.tuner.class_args['optimize_mode'] seems to always be set to 'maximize' internally, although it has explicitly been set to 'minimize' in my script.
This can be seen in the web-UI by looking at the top trials (trials with the highest scoring points are displayed) or the optimization curve, which highlights maximas instead of minimas. I have included the pictures at the end.
When looking at the experiment summary in the UI, 'minimize' is chosen for the tuner.
Environment:

NNI version: 2.2
Training service (local|remote|pai|aml|etc): local
Python version: 3.6.9 on remote and 3.8.7 on local
PyTorch/TensorFlow version: TensorFlow 2.4.1
Is conda/virtualenv/venv used?: Yes venv
my local device: Windows10
remote device: Ubuntu 18.04.5 LTS

Configuration:

Experiment config (remember to remove secrets!):
exp = Experiment('local')
exp.config.experiment_name = self.config['experiment_name'][self.dataset_id]
exp.config.trial_concurrency = self.config['trial_concurrency']
exp.config.max_trial_number = self.config['max_trial_number']
exp.config.search_space = self.search_space
exp.config.tuner.name = TPE
exp.config.trial_command = 'python main.py ' + str(self.dataset_id) + ' ' + self.scoring
exp.config.trial_code_directory = '../nni_regression'
exp.config.tuner.class_args['optimize_mode'] = 'minimize'
exp.config.training_service.use_active_gpu = True
exp.config.max_experiment_duration = self.max_loop_duration()
exp.start()

Search space: Pretty long, therefore not included. If it is necessary i can still post it.

This seems to be a WebUI issue. @Lijiaoa please follow.

@JanikPi Thanks for reportering this issue. It is webui bug and will be fixed on v2.5.

fixed in #3731


Using usql as a library

First of all, props on an amazing project!
I'd like to utilize some of the awesome here in my own golang app. The idea is that I could wrap around usql and utilize it to drop me into a sql prompt.
Is this possible, short of shelling out to the usql binary? Which functions/methods could I use?

There's not a whole lot of logic in usql that is specific to it. It uses chzyer/readline for doing the "readline" stuff. usql isn't much different (yet) from their multiline example.
I'm in the middle of making usql much more modular in general, and will try to make it reusable by other packages, but it's not at that state yet.

Please check out the latest version of usql, specifically the usql/handler package. It's not 100% modular yet, but I will in the next week or so refactor all the actual SQL stuff out of that package completely.

Amazing turnaround! thanks!

Well, the work on this revision had been in process for the last few days. Have been trying to make it do "some parsing" to better execute queries / behave more like psql. Not 100% there yet, but much much closer now.


Add an overview diagram

It would be nice to have an overview diagram in the readme, to get an idea of how it works.
And also to illustrate the "server app" running as another Lumino application.

Hooray diagrams! How do folk feel about the diagrams living in the drawio editable SVG format?

yeah, there's some font nonsense, but that's mostly github...

My interest is several-fold:

it's an acceptable place to draw architecture diagrams and mockups, but there aren't a lot of examples of them across the jupyter community
being able to use ipydrawio in the jupyterlite would be quite nice, as all the fancy interactive plugins would work (e.g. collapsible groups, links, etc) that are lost in the otherwise-excellent static export

How do folk feel about the diagrams living in the drawio editable SVG format?

Those are great! Even the png with embedded metadata is really useful to have a self-contained diagram.

being able to use ipydrawio in the jupyterlite would be quite nice

Absolutely. Maybe this will "just work" after adding support for ipywidgets? (also being discussed in https://github.com/jtpio/jupyterlite/issues/18)

png with embedded metadata

It's true: the image diff is nicer on github than with svg (for whatever reason). But more broadly, I'm really interested in ending up with solid SVG, as it's more accessible, prints better, looks better on 4k screens, etc. A killer feature i haven't yet figured out is making the links you can put on individual elements become "real" svg xlink attributes, as being able to click on a diagram and jump to (a line in) a file would be wonderful for these kinds of things.

support for ipywidgets

Nah, just needs the documentregistry/mime renderer machinery! The Jupyter Widget is pretty dumb, right now, as it just speaks (crazy) XML and (also crazy) configuration JSON.
To the nuts-and-bolts of including ipydrawio: I don't see quite yet how to ram some federated_extensions into pageInfo, and lemme tell you... you don't want 5000+ design assets files (70mb) in every "lite" build. To include it in a dev/environment demo, I can really only support the pip or conda installation mechanism, predictably, today... I'm not even testing it as a source extension :rofl:, as even back in the day it would occasionally crash webpack.

Closing as fixed by #278 with a follow-up issue in #279


Check type compatiable when generate cast expressions

Fix #1199

@caneGuy could you check out this fix up? https://github.com/StarRocks/starrocks/pull/1176


FINOS : up-to 66 user entry is missing under APPROVED CONTRIBUTORS list for "Goldman Sachs" organization at FINOS project

Steps :

Navigate to Signature tab : https://projectadmin.lfx.linuxfoundation.org/project/a092M00001JwBlEQAV/tools/easycla/overview/5564f18f-3252-40fc-a65b-2eef1ad45fcd/signatures
Open Signed CCLAs tab
Search Organization "Goldman Sachs"
Click on View All Approved Contributors

Actual

Expected :

@thakurveerendras Kindly review

Checked all missing data are added under approved contributor list but "GS OSS Bot" user data is still missing

One concerns :  Goldman Sachs & Co. LLC also listed while no such org mentioned in Finos sheet

Now "GS OSS Bot" user data appears on list, So making issue status QA-Done


when create multiple subscriptions,installplan status is null

Bug Report

What did you do?
 The interesting log here is
sync "operators" failed: etcdserver: request is too large

it's an unusual error. When you encounter this issue, could you also provide the api-server logs on the cluster? That should help us look into this issue more.
Also, for reproducibility, if you could provide the manifests (subscription, catalogsources) that you used to generate the installplan that would also be helpful.

Unfortunately, the log no longer exists。
But I remember that I checked the audit about apiserver. The catalog update installplan status caused the apiserver 500 error.
As you said, maybe installplan status is too large?
I found some instructions about etcd https://etcd.io/docs/v3.3/dev-guide/limit/
So，Does installplan need to limit the number of CSV installations or updates？
😄😄😄

There is very confused why single installplan need to install multiple csv and dependent resource of csv.
Maybe it need to improve here because of etcd requests limit

@aiyijing This is because OLM install / update resolution logic works at the namespace level and therefore considers all operators currently installed in the namespace via Subscription. The solution is to install these operators in other namespaces.

@dmesser
In here, Method of updating installplan.status  is PUT.
If the update method is changed to PATCH, will it be improved?
https://github.com/operator-framework/operator-lifecycle-manager/blob/2d649b0d5935ecfecfefbe56f266cc7b04d0b290/pkg/controller/operators/catalog/operator.go#L1255-L1260

@dmesser
This seems to be another solution：
Installplan be split according to  Resolving of step,  and then create and update the installplan.
Otherwise we have to modify etcd request limit,obviously it is not reasonable.

There are more restrictions:
I extend etcd  --max-request-bytes to 10Mi but it is sadly that grpc server has
restrictions of grpc send message size limit.
If I install a lot of operator in global NS and at upgrade  the index image, all operators will fail to upgrade because installplan cannot be updated successfully.
time="2022-03-29T17:16:17Z" level=warning msg="no installplan found with matching generation, creating new one" id=RpdmR namespace=operators
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing id=wdOG+ ip=install-fnm47 namespace=operators phase=
time="2022-03-29T17:16:17Z" level=info msg="skip processing installplan without status - subscription sync responsible for initial status" id=wdOG+ ip=install-fnm47 namespace=operators phase=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
time="2022-03-29T17:16:17Z" level=info msg=syncing event=update reconciling="*v1alpha1.Subscription" selflink=
E0329 17:16:18.184570       1 queueinformer_operator.go:290] sync "operators" failed: rpc error: code = ResourceExhausted desc = trying to send message larger than max (2627600 vs. 2097152)


chore(main): release 0.4.0

:robot: I have created a release beep boop
0.4.0 (2024-08-21)
Features

add zilean (2f3e877)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.4.0
:sunflower:


refactor thellier_magic

Each plotting program must work on the command line, in a notebook, and in tests.
Each command line program should have main() function that is only for the command line logic, and calls an ipmag function that does the calculation.
Each program should return a list of saved files.
Each program should be able to take a contribution/pandas dataframe argument instead of reading in a data file.

Need to figure out why more plots are generated now than previously, otherwise this is probably good to go.

Looks like that issue was a fluke -- probably had some random pngs lying around.  Everything is good here!


multiple images

How to pass multiple images into ControlNet ? imagine --control-image <multiple imgs>
Thx

unfortunately we don't support multiple control images. what were you trying to do?


🐛 [Bug] GDT has to display the correct amount

:bug: Bugreport

GDT has to display the correct amount. On our test it was not divided by 10.000. We should investigate more how to do this division transition with an component/mixin

[X] GDT has to display the correct amount divided by 10k
[ ] Have a component/mixin for this division

Testen nur möglich auf staging

GDT has only 2 places after zero, divide by 100 is enough.


Allow users to modify the 'Since Date' in their contributors cards from backend

The cards have the current date as default. We should allow for the core contributors (not backers) of a collective to edit since when they contribute to their project.

It's merged in #84


feat: Save post

Standards checklist:

[x] The PR title is descriptive.
[x] The PR doesn't replicate another PR which is already open.
[x] The PR has self-explained commits history.
[x] The code is mine or it's from somewhere with an Apache-2.0 compatible license.
[x] The code is efficient, to the best of my ability, and does not waste computer resources.
[x] The code is stable and I have tested it myself, to the best of my abilities.

Changes:
This PR brings a new feature where a user can save a post, and delete a saved post.
In the browser:

Add a new "Save post" button to save a post into localstorage
When saving a post, images and videos are not saved, only the URLs
Add a new "category" on the left to list all the saved posts.

On mobile and Electron:

Replace the "Save video" button with a "Save post" button
When saving a post, images are automatically saved on the device
When saving a post, video is downloaded only if user agrees to (a popup ask him if he wants to save the video locally)

PR #702 replaced this pull request. Closing it


feat: add gitpod for efficient user experience

user can now run sample and generate testcases by using  gitpod also

@Sarthak160 please help with the above issue. rest looks good!

@Sarthak160 This is awesome, Thanks!


[zh-cn] The title “接下来” has missing in 审计 article

This is a Bug Report

Problem:
The title "接下来” is missing, as shown below:

In English version of the same artical, there is a "What's Next" title which should be translate to "接下来”, as shown below:

Proposed Solution:
Add the title properly,  it should be looks like:

Page to Update:
https://kubernetes.io/zh-cn/docs/tasks/debug/debug-cluster/audit/#heading

I have fixed this in my local computer, if this issue been triage/accepted, I will link a pull request to resolve this issue.
/assign

/language zh

https://github.com/kubernetes/website/pull/41976 has been merged, the issue has been resolved.


Fix Issue #3139 (parfors reuse of reduction variable across prange blocks)

Fix for #3139

Codecov Report

Merging #3142 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3142      +/-   ##
==========================================
+ Coverage   81.14%   81.14%   +<.01%
==========================================
  Files         384      384
  Lines       75028    75039      +11
  Branches     8434     8437       +3
==========================================
+ Hits        60878    60889      +11
  Misses      12859    12859
  Partials     1291     1291

Currently waiting on https://github.com/numba/numba/pull/3127

Thanks for the patch, this looks good, will just run it through the build farm. The test fail on Travis CI is entirely unrelated to the change here.

Build farm passed. This can be merged as convenient. Thanks again.


Fix broken link about daemonized processes

Fix #269 broken link (returns HTTP code 404) about daemonized processes:
manlix@lab:~$ curl -IL http://dustin.github.com/2010/02/28/running-processes.html
HTTP/1.1 301 Moved Permanently
Server: GitHub.com
Content-Type: text/html
Location: http://dustin.github.io/2010/02/28/running-processes.html
...

HTTP/1.1 404 Not Found
Server: GitHub.com
Content-Type: text/html; charset=utf-8
...

New correct link — https://dustin.sallings.org/2010/02/28/running-processes.html:
manlix@lab:~/git/12factor$ curl -IL https://dustin.sallings.org/2010/02/28/running-processes.html
HTTP/2 200
server: GitHub.com
content-type: text/html; charset=utf-8
last-modified: Thu, 30 Apr 2020 03:17:14 GMT
...

Nice job!


New energyua design support

Changed periods selector
Changed period items selector

Result:

hi @ppokyd,
I just wanted to kindly ask if you have an estimate for when your changes might be merged?

hi @ppokyd, I just wanted to kindly ask if you have an estimate for when your changes might be merged?

hi @srolskyi, have no idea, since seems like author of the integration is not so active

Closed in favor of https://github.com/tsdaemon/ha-lviv-poweroff/pull/4


Bring out prop descriptions, for Flexbox

For Flexbox API docs would like to tease out the prop descriptions. This PR makes that feasible by exposing the description for style.
Test plan (required)

Temporarily modified the flexbox source doc: Libraries/StyleSheet/LayoutPropTypes.js to add a description.
Checked it out on local webpage: http://localhost:8079/react-native/docs/flexbox.html

By analyzing the blame information on this pull request, we identified @sahrens and @janicduplessis to be potential reviewers.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


UPM Authentication using .upmconfig.toml

Example and explanation about how to authenticate with a private UPM registry by creating an .upmconfig.toml file in the home folder.
This feature was added here: https://github.com/game-ci/unity-builder/pull/211 - but as far as I can tell it was not documented.
Changes

...

Checklist

[x] Read the contribution guide and accept the code of conduct
[ ] Readme (updated or not needed)
[ ] Tests (added, updated or not needed)

We should probably start an advanced section. I try to do it if time permits. Will merge this for now.


Changed name of sensors in the Utilities section

The sensors in the Utilities documentation have been renamed:
Changed "sensor:ehwenergy_hot_waterexternal" into "energy:ehwenergy_hot_waterexternal"
Changed "sensor:echenergy_central_heatingexternal" into "energy:echenergy_central_heatingexternal"

What is the problem here?

"sensor:ehwenergy_hot_waterexternal" and "sensor:echenergy_central_heatingexternal" do not exist, and therefore the listed queries did not work. With with the edited names "energy:..." the queries work

Are you talking about screenshot in the wiki page?

I am talking about the wiki page text and query in the utility meter section.


dumps if repo already present or wrong URL

Installed abapGit dev manually, since the docs mention that it is required for pulling additional repos.
Created include by copying all entries from my older NPL minisap.
Ran with "pull repos", it dumped. Twice. Realized what might have been the cause, commented out the abapGit line in the include and now it is happily pulling the rest in... until the second issue came up: some URLs had an extra space between the double slash and the hostname, which abapGit did not like, at all :)
All in all, a couple of rough edges (mostly with the automatic pulling) but I am quite pleased, a swiss army knife for setups :)
(alone the SSL setup and the repo definition is a huge thing for me)
EDIT: in my case there might be some other issue, since aG keeps dumping. Or maybe the 2022 trial behaves differently.

Actually it should pull abapGit now, and it has its own action option.
I have a solution to streamline that too, just need the time to do it. PoC looks OK to pull repos without having abapGit Dev installed.
But thanks for raising, I will look into avoiding dumps if abapGit isn't happy with the repo info.

Found and updated a couple of issues where abapGit errors weren't handled.

One more update - abapGit dev version is no longer needed for pulling. The option has been removed and it's just another repo for the list if you need it.

Respun A4H, installed current version, added my config include, got "Unknown signer or recipient" (#4 ?) but the SSL test utility says OK.
Fetching aG works ok, most repositories are fetched and activated ok until aG standalone throws an exception during import of abap2xlsxdemos: Activation cancelled. Turn on "Activation Popup"  in "Personal Settings" and try again
(manual activation of this repo indeed shows a popup with most objects unselected, marking them all and activating works ok though)
The repositories coming after this are not imported at all and the profile changes are not done.

Thanks, I fixed the activation popup last week (https://github.com/abapGit/abapGit/issues/6925). The issue was in abapGit, and A4H has an old aG Standalone preinstalled, so I am wondering if you're using the latest?
zdevsysinit checks what's in the system and proposes parameters accordingly, and if it finds ZABAPGIT_STANDALONE then aG Standalone is off. Check any of the boxes to perform the action anyway, so tick Standalone to pull the latest.
User profile is odd, it's the first action so shouldn't be influenced by later failures. Likewise multiple repos shouldn't influence each other, it's a simple loop and I tested failed imports quite a bit. Does the user profile update if everything else is off?
I'll try to have a look this evening, time to fire up a new system.

I may have misunderstood the comment about the standalone abapGit, because I may have used the (very old) preinstalled version. I ticked all boxes, anyway.
BTW, I copied this source
 for the main program but the include comes from a different source since the one in the readme is a broken link

OK, I've recreated a new instance and it all seems to run OK. To check things I ran everything individually, and in SapGUI.
The user profile is updated, but something is cached so it looks the same. Even /ns000 comes back with "John Doe" in the home screen. Only by starting a new session does it show my name and no picture (updated setting):

Not much I can do to address this.
Next, I did fetch aG Standalone to update without the aG popup bug, but did not fetch repos.

I might default aG Standalone fetch to yes if it's more than three months old.
 Had a hiccup with abap2xlsxdemos, but this stems from abapGit. It wasn't handles nicely so have done some updates.
I've raised the pull failure as #4, suggest closing this one.

Nice, it is working much, much better indeed! These were the abapGit failures, the definitions were saved so manually pulling them again at the end fixed the issue:
25.05.2024	18:49:18	Import repo abap2xlsxdemos failed: Activation cancelled. Turn on "Activation Popup" in "Personal Settings" and try again
25.05.2024	18:51:31	Import repo ABAP Examples failed: Activation cancelled. Turn on "Activation Popup" in "Personal Settings" and try again
25.05.2024	18:52:08	Import repo FM-Test-Data failed: APACK dependencies not met and undecided

The profile, though, is still not working: I logged in from another window without closing the first one and it was still showing the generic data; I then edited name and email, which caused the activation of the relevant objects.

OK found the profile issue. I added the current username as a parameter but wasn't in the sample template.
It did show an error in the log. Now defaulted to sy-uname if not supplied, and sample updated.

#7 for the link to the sample, which is still not working for me.

Alright, this issue is dead and buried, thank you very much for your work, really appreciate this!
Closing now...


reject :talk ++command pokes from foreign ships

closes #208
Crashing rejects the poke, right? (As opposed to producing [~ +>.$], which would accept and acknowledge?)

Yeah, I think that's right. But @cgyarvin needs to provide us with a gall design sketch.

And I initially botched that example ... for the benefit of emailers:
?|  =(our.hid src.hid)
    &(?=($earl (clan src.hid)) =(our.hid (sein src.hid)))
==

It probably should allow pokes from your moons. :dojo for example allows it, along with peering via sole. https://github.com/urbit/arvo/blob/master/app/dojo.hoon#L1102
There should probably be a helper function in zuse for this check, really, since it's what you usually want.

I added ++team to zuse - seems like an appropriate name ...
To see if the crash "rejects" the poke, I added a ++coup arm to a test app on a comet and poked :talk on a different comet. It was called, with a ++tang showing the trace:
mack
/~tipsyt-middep-nommep-simdel--dapmyl-lapdev-bonnes-falfyr/base/0/app/talk:<[2.166 3].[2.170 24]>
/~tipsyt-middep-nommep-simdel--dapmyl-lapdev-bonnes-falfyr/base/0/app/talk:<[2.165 3].[2.170 24]>

Naively, I would expect this to break ra-transmit, i.e. all message posting. Does this work with like, a channel on a fakezod that a fakenec joins and writes to?

Sorry about that -- I opened the PR and then went mostly offline on vacation ... I saw it was merged, so I assumed it was fine


There still have member under project->summary->members after delete this member!

Why there still have member after delete (remove) this member? This member can be find under "project"->Overview->Summary->Members. How to completely delete the member?
Best regards.

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

ID: 07738454-4b1a-eff5-6c16-69e963ab2663
Version Independent ID: a63f1dad-0378-5ce3-97e9-ec66afffb206
Content: Add users to a team or project
Content Source: docs/organizations/security/add-users-team-project.md
Product: devops
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@moyanming - Thank you for your inquiry.  When you remove a user from a team, you don't remove them from a team project. Please review the information provided here - https://review.docs.microsoft.com/en-us/azure/devops/organizations/security/remove-users-prohibit-access?view=vsts&branch=master-rebrand - and if you're using VSTS, https://review.docs.microsoft.com/en-us/vsts/organizations/accounts/manage-users-table-view?view=vsts&branch=master-rebrand&tabs=new-nav#manage-users.

@KathrynEE those review.docs.microsoft.com aren't available outside of Microsoft folks. The second one works when removing the review., but the first 404s

Hi @KathrynEE , it seems that the "Manage users for VSTS" should be update. I add a new issues for that #1652 .


Stop propagating parent channel args into xDS channel.

This is another step toward sharing the XdsClient between channels.

Known issues: #23933 #24242 #24243


Dockerfiles review

This branch has two intentions:

Move required environment variables to the .env file.
In the master branch, environment variables are defined in the .env, .env.prod, docker-compose.yml, docker-compose.production.yml, docker-compose.override.yml, .travis.yml and when the app is already loading, as in jarbas/settings.py. The unification comes to follow the DRY principle.
Move everything production-specific to docker-compose.production.yml and development-specific to docker-compose.override.yml. As far as I understand, this pattern was already being followed, so just did a review for enforcing it.

Since I added environment variables as ARGs in Dockerfiles, still need to check if they can be configured in Docker Hub registry for auto building the images from master. (it works locally)
Ideally, this is a work on the direction of getting the project easier to run and, by consequence, easier to configure automated deployment from the CI.

Already building in Docker Cloud, where I set values for bucket and region names. Just in case, I also defined default values in the Dockerfiles.

The test suite is expected to pass after merging https://github.com/okfn-brasil/serenata-toolbox/pull/184. cc @cuducos

I believe we're good to go here. The project is already running, as is in this branch and backed by Docker Cloud and Digital Ocean, at https://staging.jarbas.serenata.ai/.
In a following task, we can review documentation as a whole (I paid special attention only in the places mentioning what I was working on) and consider adding a Deploy to Docker Cloud button.

Just did a full setup from scratch and main services are working with just the following:
$ git clone git@github.com:okfn-brasil/serenata-de-amor.git
$ cd serenata-de-amor
$ git checkout deployment
$ cp contrib/.env.sample .env
$ docker-compose up --build

(research service fails for reasons not touched by this pull request)


HV-1812 Avoid reflection-based metadata extraction for built-in value extractors - 6.1

https://hibernate.atlassian.net/browse/HV-1812
@yrodiere not the most exciting PR but I would appreciate a second pair of eyes.
The idea is to avoid calling a method not present in the Android JDK for the built-in value extractors.

I asked the OP to test it. TBH, that's the maximum amount of work I'm willing to put into this :).


[9.1.x] ISPN-8594 Memcached not finding UTF-8 data

https://issues.jboss.org/browse/ISPN-8594

Updated

Merged, thanks


label.formatter does not work when valueAnimation: true

Version
5.0.0
label.formatter does not work when valueAnimation: true
Reproduce: test/bar-race.html

Fixed in #13792


Operator containsText matchs different results with and without fulltext index

Hello,
I've just noticed an unusual behavior with containsText operator. The result is not the same with or without fulltext index.
Without index, containsText matches with partial words, and with index, only exact words.
CREATE CLASS Car
CREATE PROPERTY Car.name STRING
INSERT INTO Car SET name = 'JAMIE'
INSERT INTO Car SET name = 'JAMES'
INSERT INTO Car SET name = 'JALES'

SELECT FROM Car WHERE name containsText 'JAM' // 2 results
CREATE INDEX Car.name FULLTEXT
SELECT FROM Car WHERE name containsText 'JAM' // 0 result

I suppose exact matching is what is expected. Partial matching is really useful too ;)
Tested on develop.

Hello @lvca is there any update with this enhancement? Thanks :)

I'm forwarding this to Enrico that worked on Lucene plugin.

@studiodev
which version of OrientDB?
i tried with the latest snapshot and works even with index

I'm closing this issue. In case you're able to reproduce it again against last 2.1, please comment/reopen this issue. Thanks.

@lvca,
We experienced this same issue on orientdb 2.1.
Reading through this thread we tried a string with minimum 3 characters and it worked. However, the moment this string gets a space it doesnt work again. Is there a way to handle this?
e.g.
This worked
SELECT FROM Employee WHERE fullName containstext 'Tony'
This didn't
SELECT FROM Employee WHERE fullName containstext 'Tony Star'
I have a FULLTEXT HASHED INDEX for this fullName field.
Appreciate your help.


Feature/brunch1

We discussed this in the dev catchup and the feeling is that brunch should be a dev dependency since it would be preferable if it wasn't deployed to production. The reason being that it is an unnecessary surface area in terms of security risk. This is pertinent given the snyk warning. Do you think this be resolved given that you run brunch specifically when the env is production?

@st3v3nhunt wrote:

Should the assets referenced within the css be hashed too? What are the implications of the externally hosted assets e.g. the fonts using this approach?
Are the images cached? They change so rarely, do we need to bother?

PR #256 created for some tweaks to this PR.
The variable qs (in https://github.com/nhsuk/profiles/pull/250/files#diff-624f0d73fb727e28599e9c0cafef6db5L17) is used for cache busting the JS too.
As part of this change it would be good to get the JS done in the same way.

Superseded by #250


MediaType routing not observed

Consider the following case:
    @Path("{id}")
    @PUT
    @Consumes("application/vnd.clearwater.v2+json")
    @Operation(summary = "Updates the quote...")
    public void updateQuoteV2(QuoteV2 quote, @PathParam("id") Integer id, @Context SecurityContext context) {   }

    @Path("{id}")
    @PUT
    @Consumes(MediaType.APPLICATION_JSON, "application/vnd.clearwater.v1+json")
    @Operation(summary = "Updates the quote...")
    public void updateQuote(Quote quote, @PathParam("id") Integer id, @Context SecurityContext context)  { }\

What I want for output is a single entry on the swagger-ui page with the RequestContent merged.  Unfortunately, only the last Operation read is displayed.
Based on my review of the code, the problem is caused in the swagger-jaxrs2 module when the io.swagger.v3.jaxrs.Reader#setPathItemOperation is called.  In this method, or within PathItem (since there may be multiple implementations of OpenAPIReader), I think we need to merge the Operations if it has previously been set.
Because I do not know this code base well, I'm issuing this ticket in the event that merging at this location is a bad idea (and with the hope that this issue can quickly be resolved).  If you agree with my assessment, but can not address this issue right away, please let me know and I will create a pull request.
The following are possible fixes:

Update PathItem to perform the merging on each method type (fix only needed in one place if there are multiple OpenAPIReader implementations).
Update just the io.swagger.v3.jaxrs.Reader#setPathItemOperation to perform the merge.
Create my own OpenAPIReader and register it with the OpenAPIConfiguration#readerClass

Or.. anything else you might come up with.

As I've looked further, the issue stems from the fact that the OpenApi 3.0 spec is path-centric.  Consequently, the use of media-type versioning is implicitly discouraged.  To use media-type versioning, the spec encourages that all media-type related changes be configured under a single path.  As a result, any header-type info annotated on each dispatch method (e.g. summary, description, deprecation, etc.), must be handled differently.  We can't simply deprecate a path; when only a specific version is being deprecated...
There may be a possibility to use media-type extensions to address this issue, but a complete solution will require updates to the SWAGGER-UI project as well.
** These are just notes as I consider possible solutions and pull request. **

See associated OAS Specification case: (https://github.com/OAI/OpenAPI-Specification/issues/2142).  In the meantime, we/I could add a custom extension to deal with some of these issues.  Deprecation, documentation, etc. will need to wait for an updated OAS specification.


azd down not purging keyvault?

Steps:

I ran azd down and purged keyvault.
I ran azd up and ran into this error

Is this a "I have to wait some amount of time" problem? I haven't changed anything about the template code since the initial provision/up.

@rajeshkamal5050 I think this has been resolved so I will close.


Add CLI

Building on https://github.com/TheThingsNetwork/node-app-lib/blob/master/bin/monitor, but then properly using Commander.
Downlink example:
./bin/ttn -r eu -a 'hello-world' -k 'my-key' -d 'my-uno' -m '{"led":true}` -p 1

Uplink example:
./bin/ttn -r eu -a 'hello-world' -k 'my-key' -t `messages`

Topic could also be a full topic and can be used multiple times.
There should also be an -o json option to get the response outputted in json

What is the problem that we solve as compared to ttnctl?
This is another thing to maintain and seems confusing.

Nothing really. I actually added https://github.com/TheThingsNetwork/node-app-lib/blob/master/bin/monitor at some point to quickly test but I agree it's better to stick with one CLI.


subcommands should permit dashes

def cmd := `[Cli|
  foo-bar VIA doNothing; ["0.0.1"]
  "installCmd provides an example for a subcommand without flags or arguments."
]

gives
10:2:
unknown identifier 'foo'
10:6:
unknown identifier 'bar'
10:2:
failed to synthesize instance
  Sub String

But I'd like to be able to call subcommands on the command-line with dashes in the middle of them

One alternative for now until I get to this is to use Cmd.mk directly: https://github.com/mhuisi/lean4-cli/blob/58c23fca676bb7121be4eda6eae936fb98aa2507/Cli/Basic.lean#L718

Oh, I think you can use the DSL actually. Just put the name in quotes and it should work?

Ah, indeed that works, thanks!  It would be nice to get a better error message, but I'm happy to put things in quotes.


Allow to set up the feed rate

Hi there!
OctoPod is definitely one of the best OctoPrint clients out there, but I can't seem to find any way to set the feed rate from it (short of sending an M220 command via the terminal).
Am I missing something, or is it actually not available? Will it be added?
Thanks!
(Slightly OT: if you want, I can translate the app's strings into Italian. I'm a professional freelance translator and I'd be more than happy to give back to the project!)

Ciao Daniele,
Thanks for the kind offer. I will contact you when I get to issue #17 for multi language support. In the panel tab, you can control the flow rate for the extruder. Is that the feed rate you are looking for or is it something else?
BTW, there seems to be no way to read the current flow rate value but to only set it. I had to start with some default value just like OctoPrint does.
Grazie,
Gaston

Hi Gaston,
I was referring to the actual feed rate, which is the multiplier the printer applies to movements. In OctoPrint, you can find it on the Control pane, at the bottom left:

It's essentially the same as rotating the controller wheel or using the up/down buttons on the printer when using Marlin and Marlin-derived firmwares, but it can be handy to set remotely; I sometimes need to slow down the first layer with some nasty filaments.
It looks like OctoPrint just sends an M220 Sxxx command, where S is a parameter generally going from 50 to 150 (with 100 as the default): http://marlinfw.org/docs/gcode/M220.html
There seems to be no way to read the current value (in fact, changing it directly on the printer doesn't affect OctoPrint) but most of the time it's just left at 100%, so that would be a safe default in my humble opinion.
Do let me know about the translation!

Excellent info. I had no idea of this other setting. Will file it for
upcoming release.
Thanks
Gaston
On Mon, Sep 3, 2018 at 9:30 AM Daniele Nicolucci<EMAIL_ADDRESS>wrote:

Hi Gaston,
I was referring to the actual feed rate, which is the multiplier the
printer applies to movements. In OctoPrint, you can find it on the Control
pane, at the bottom left:
[image: image]
https://user-images.githubusercontent.com/3470104/44996004-e0c0da80-afa5-11e8-8b87-cf364e185583.png
It's essentially the same as rotating the controller wheel or using the
up/down buttons on the printer when using Marlin and Marlin-derived
firmwares, but it can be handy to set remotely; I sometimes need to slow
down the first layer with some nasty filaments.
It looks like OctoPrint just sends an M220 Sxxx command, where S is a
parameter generally going from 50 to 150 (with 100 as the default):
http://marlinfw.org/docs/gcode/M220.html
There seems to be no way to read the current value (in fact, changing it
directly on the printer doesn't affect OctoPrint) but most of the time it's
just left at 100%, so that would be a safe default in my humble opinion.
Do let me know about the translation!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gdombiak/OctoPod/issues/65#issuecomment-418157446,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtVI2bgWLK9k5JayID8t3zD4Lnd2vVaks5uXVkwgaJpZM4WXnrp
.

--
*You can't control the wind, but you can adjust your sails --  Yiddish
proverb *

Ciao Daniele, I just pushed changes so we can add new translations for OctoPod. Let me know if you are still interested in helping add the Italian translation. I added the Spanish (Latin America) as an example.
Grazie,
Gaston

Absolutely! Since I've never done pull requests and I'd have to read up on that, if you want I can "seed" the translation by just sending you the file, if that's okay.

Sure. Can you send me your email to gdombiak AT gmail.com? Thanks! :D


Notification: I have created an AppImages article on The Hornery

Hi,
As I noticed your README only lists about one-fifth or so, of your AppImages I have decided to provide a more comprehensive list on my website, The Hornery.
Thanks for your time,
Brenton

Thanks Brenton. You are more than welcome to update the wiki page here. Keep in mind, though, that my AppImages are examples only, and the real objective is for upstream application authors to provide their own, official AppImages.


Update exceptions.json

Added automerge exception for Unciv - averages to 2 releases a week
Unciv itself is build in a Github Actions pipeline - if you would rather I use the Github action for flatpak releases I could do that, not sure if there are problems in "transferring" from the bot to the github action

What's blocking you from manually merging the two PRs?

I have 8 different distribution paths for my app, I don't have time or attention for manual work per release
If your new policy is that everything requires manual approval, that's also a valid answer, and I'll remove this app from Flathub entirely
(Not in like a bad threatening way, just practical considerations, don't mean to come off as rude, but I'd rather not have outdated versions lying around where people will use them and complain about ancient irrelevant problems)

That was always the policy - the status checks, not pushing directly to main etc was for that. It just wasn't enforced and automerge allowed a way out. Automerge was introduced only for extra-data sources, but started being used as an all purpose automation.
Without a proper reason I can't really allow this exception, then pretty much everyone will say the same thing to get it.

OK, no problem, got it 👍🏿
If I use the Github action, can that lead to a touchless deploy? Or would that require manual intervention as well?

You can't really upload to Flathub with github-actions, direct-upload is only available to a few trusted publishers like Mozilla etc.
You can however host your own Flatpak repo, I have a bunch of examples here https://docs.flatpak.org/en/latest/hosting-a-repository.html#hosting-a-repository-on-gitlab-github-pages

Thanks for your help!


Link to GitHub list of good first issues

Added a hyperlink https://github.com/kubernetes/website/contribute at line 19 (Improve existing content ) which points to "good first issue," available.
This will help new contributors to make their first contribution.
Closes #20160

@Heena-PyCloud would it be OK if I updated the PR title to explain the motivation for this change?

@Heena-PyCloud would it be OK if I updated the PR title to explain the motivation for this change?
Sure @sftim !
I will also make a note of it for the next time :)

/retitle Link to GitHub list of good first issues

/label tide/merge-method-squash
/lgtm
/approve


Crash on M5

Problem
There is a stack overflow on M5. Even after https://github.com/project-chip/connectedhomeip/pull/9263 went in, we're still seeing a stack overflow in our testing. This is related to the generation of the rotating device id in https://github.com/project-chip/connectedhomeip/pull/8888.
Proposed Solution
short term: Can we disable rotating device ID on ESP32 as a short term fix to unblock testing?
Long term: Is this an exceptionally stack heavy process? Is there a more appropriate place to do this work? Related - is there a proper way to determine the max stack size for espressif? Related - see https://github.com/project-chip/connectedhomeip/issues/6825

@cecille , do you have any stack trace or logs? i tried it on my esp32 and i don't see any stack overflow

Taking this back for now - I thought it was related to the previous crash, but this is a fun new crash. After some bisecting, it looks like it was introduced in
cb5d39e70 support different dhcp client in Linux (#9490)

Guru Meditation Error: Core  1 panic'ed (InstrFetchProhibited). Exception was unhandled.
Core  1 register dump:
PC      : 0x800e0001  PS      : 0x00060c30  A0      : 0x800f5dec  A1      : 0x3ffe6ee0
A2      : 0x800e0001  A3      : 0x3ffd0d64  A4      : 0x00000001  A5      : 0x00000000
A6      : 0x3ffbc710  A7      : 0x000000c0  A8      : 0x800f1fb8  A9      : 0x00000001
A10     : 0x3ffd0d64  A11     : 0x00000001  A12     : 0x00000000  A13     : 0x3ffe6f00
A14     : 0x3ffe6ee0  A15     : 0x3ffe6e50  SAR     : 0x00000007  EXCCAUSE: 0x00000014
EXCVADDR: 0x800e0000  LBEG    : 0x4000c2e0  LEND    : 0x4000c2f6  LCOUNT  : 0x00000000
Backtrace:0x400dfffe:0x3ffe6ee0 0x400f5de9:0x3ffe6f00 0x400f5e3a:0x3ffe6f20 0x400f5ffe:0x3ffe6f40 0x400f6025:0x3ffe6fa0 0x40095235:0x3ffe6fc0
0x400dfffe: moveHue(unsigned char, unsigned short, unsigned char, unsigned char, bool) at /home/cecille/chip/connectedhomeip/examples/all-clusters-app/esp32/build/../third_party/connectedhomeip/src/app/clusters/color-control-server/color-control-server.cpp:458
0x400f5de9: chip::DeviceLayer::Internal::GenericPlatformManagerImplchip::DeviceLayer::PlatformManagerImpl::DispatchEventToSystemLayer(chip::DeviceLayer::ChipDeviceEvent const*) at /home/cecille/chip/connectedhomeip/examples/all-clusters-app/esp32/build/esp-idf/chip/../../../../../../config/esp32/third_party/connectedhomeip/src/include/platform/internal/GenericPlatformManagerImpl.cpp:262
0x400f5e3a: chip::DeviceLayer::Internal::GenericPlatformManagerImplchip::DeviceLayer::PlatformManagerImpl::_DispatchEvent(chip::DeviceLayer::ChipDeviceEvent const*) at /home/cecille/chip/connectedhomeip/examples/all-clusters-app/esp32/build/esp-idf/chip/../../../../../../config/esp32/third_party/connectedhomeip/src/include/platform/internal/GenericPlatformManagerImpl.cpp:222
0x400f5ffe: chip::DeviceLayer::PlatformManager::DispatchEvent(chip::DeviceLayer::ChipDeviceEvent const*) at /home/cecille/chip/connectedhomeip/examples/all-clusters-app/esp32/build/esp-idf/chip/../../../../../../config/esp32/third_party/connectedhomeip/src/include/platform/PlatformManager.h:364
(inlined by) chip::DeviceLayer::Internal::GenericPlatformManagerImpl_FreeRTOSchip::DeviceLayer::PlatformManagerImpl::_RunEventLoop() at /home/cecille/chip/connectedhomeip/examples/all-clusters-app/esp32/build/esp-idf/chip/../../../../../../config/esp32/third_party/connectedhomeip/src/include/platform/internal/GenericPlatformManagerImpl_FreeRTOS.cpp:179
0x400f6025: chip::DeviceLayer::PlatformManager::RunEventLoop() at /home/cecille/chip/connectedhomeip/examples/all-clusters-app/esp32/build/esp-idf/chip/../../../../../../config/esp32/third_party/connectedhomeip/src/include/platform/PlatformManager.h:296
(inlined by) chip::DeviceLayer::Internal::GenericPlatformManagerImpl_FreeRTOSchip::DeviceLayer::PlatformManagerImpl::EventLoopTaskMain(void*) at /home/cecille/chip/connectedhomeip/examples/all-clusters-app/esp32/build/esp-idf/chip/../../../../../../config/esp32/third_party/connectedhomeip/src/include/platform/internal/GenericPlatformManagerImpl_FreeRTOS.cpp:204
0x40095235: vPortTaskWrapper at /home/cecille/chip/esp-idf/components/freertos/port/xtensa/port.c:168

After some bisecting, it looks like it was introduced with 3a505010973 Virtualize System::Layer interfaces (#9366)

From the DispatchEventToSystemLayer in the trace, this is a duplicate of #9563 fixed by #9569.


fix unstable feature test

Fixes #12, a race against the system clock, which lead to the WaitUntilAnnotationFeatureTest occasionally failing.

This change is

[Current coverage][cc-pull] is 86.94%

Merging [#13][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #13   diff @@
==========================================
  Files           237        237
  Lines          2933       2933
  Methods           0          0
  Messages          0          0
  Branches        157        157
==========================================
  Hits           2550       2550
  Misses          344        344
  Partials         39         39

Powered by Codecov. Last updated by [4ba4d9b...1589275][cc-compare]
[cc-base-branch]: https://codecov.io/gh/testIT-WebTester/webtester2-core/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/testIT-WebTester/webtester2-core/compare/4ba4d9b68d26e520cf79c4e51bc12e5c0eee5ace...1589275a2d156a64c86b94554ded5a3bde280d14
[cc-pull]: https://codecov.io/gh/testIT-WebTester/webtester2-core/pull/13?src=pr

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Ft about

What does this PR do?

The About Page and its components( Annual Reports and Members) have been set up.
Emphasis put on data fetching so no CSS design at this point

Is this change ready to hit production in its current state?

[ ] Yes
[ ] No

Status of maturity (all need to be checked before merging):

[ ] I consider this code done
[ ] I've tested this locally
[ ] This branch has been LGTM'ed by a reviewer

How should this be manually tested?

git checkout main
git pull origin
git checkout origin/ft-about
yarn start

What are the relevant tickets?

None

Screenshots (optional)

@chrismclarke I've made the changes regarding downloading the annual reports and also added some export csv data

Many thanks @KingMike100
I noticed that we currently don't have a way to share uploaded media (backend upload folder is in gitignore) so I get some small errors when running locally. I've added a better catch for these errors and I'll remove from gitignore, upload some correct media files. But otherwise looks good to me!


Change View::run_dirbrowser() to return nonstd::optional<std::string>

https://github.com/newsboat/newsboat/pull/2561#discussion_r1357348873

coverage: 60.093%. remained the same when pulling ce1e0739baa9d5cb98c1c45cfe4c9aa899da84f5 on bogdasar1985:empty_path into ca977d51a0ea9bc08d302d9ce58e424090823a87 on newsboat:master.


pipeline.Exec returns redis: nil when some key does not exist

For example, a pipeline with a HGET command:
pipeline := client.Pipeline()
pipeline.HGet("not_exist_key", "foo")
cmds, err := pipeline.Exec()
if err != nil {
    log.Fatal(err)
}

It may be reasonable to return an error when some commands execute failed, but it make me confused.
The commands execute normally, and I got the result in the cmds,  but the returned err indicates me that the pipeline executes failed.
If I just check the err, then I will miss the normal result. So I must check the cmds regardless of the pipeline err.

Just check for redis.Nil
if err != nil && err != redis.Nil {
    log.Fatal(err)
}


chore(docs): fix invalid repo links

Rationale
The old links is invalid.
Detailed Changes
Influxdb_iox repo change into influxdb main branch. So I modified the links.
Test Plan
no need test.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

thanks, you can ignore the CLAassistant.
https://github.com/apache/incubator-horaedb/issues/1449


Fix typo

Just a very quick change. From "pragmatically" to "programmatically" (which is what I believe was originally intended).

Thanks!


chore: fix a typo in document and execute cargo schema in contracts

Description
Fix a typo in a document in code and execute cargo schema in contracts to reflect it to schemas.
Types of changes

[ ] Bug fix (changes which fixes an issue)
[ ] New feature (changes which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] ETC (build, ci, docs, perf, refactor, style, test)

Checklist

[x] I followed the contributing guidelines.
[ ] I have updated the documentation accordingly. (Not Needed)
[ ] I have added tests to cover my changes. (Not Needed)
[x] The PR title and commits are followed conventional commit form.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Periscope will only run correctly in the aks-periscope namespace

Describe the bug
Periscope cannot generate blob name for upload to storage if it is in a namespace other than aks-periscope. I'm not sure if this is a known and accepted limitation, but I couldn't find it documented and it tripped me up.
The reason for the hard-coded namespace dependency is:

We need to generate a blob with a timestamp that's unique to the deployment/run, but shared between all pods.
The method for getting this timestamp is to list all pods in the aks-periscope namespace, and take the creation timestamp of the last one.

Aside from tying us to a particular namespace, this is vulnerable to timing inconsistencies if an additional pod is created after another pod starts executing this code path. It also prevents us from addressing an issue raised in #157 (being unable to re-run Periscope for an existing DaemonSet). So we should probably be looking for a different approach anyway.
A bit of careful planning might enable us to address both this and #157 together. If the timestamp can be passed to periscope from something external to the pod (like the contents of a mounted file that can be watched), it could solve all of the above as well as giving us a potential means to trigger further 'runs' of an existing DaemonSet.
To Reproduce

Deploy using the usual yaml resource specification, but change the namespace to (e.g.) aks-periscope-test.
Wait for pods to be running
Check storage in the Azure Portal. Assuming everything else was set up correctly, the logs should have uploaded successfully, but the blob name will be <no name>.

Expected behavior
The blob name should be something like 2022-03-28T21-26-02Z
Screenshots

Desktop (please complete the following information):
N/A
Additional context
N/A

This is related to an intermittent bug (which occurs more frequently on newer Windows clusters), in which the node folders in the output file structure are divided between more than one timestamped root container.
By using the DIAGNOSTIC_RUN_ID variable now being supplied to Periscope by consuming tools, we can fix both of these bugs.


Optimize the max score calculation in the Query Phase of Hybrid Search

Description
Optimize the max score calculation in the QueryPhase of Hybrid Search.
Latency reduced
(Number of Shards) x (Number of topDocs found on each shard)

Issues Resolved
764
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@vibrantvarun Do you have comparison in latency before and after this change?

@vibrantvarun Do you have comparison in latency before and after this change?

Sure, I will add the latency graph. I am having some issues in the setup. Probably, once I resolve the issues I can add that flame graph in the issue of this PR. But, to unblock me on to raise my Sorting PR this PR needs to be merged.

Final max score is calculated at the coordinator level as part of normalization processor workflow. I think we can skip it calculation at shard level, maybe put 0.0.
@vibrantvarun can you check this theory, my concern is that 0.0 score doesn't trigger any unwanted code path in core logic, plus check edge cases?


Add Gas Overflow Check

Describe your changes and provide context

Add int64 Gas Overflow Check for gas wanted

Testing performed to validate your change

Verifying on local chain
Added unit tests

NOTE: We probably want to revisit why the consensus params MaxGas is a int64 but each tx's gas is a uint64


Actually test the cluster view

I'm not sure whether it was done on purpose but the current test for clusters doesn't set show_markers, so effectively the cluster view returns an empty list.

No sure I understand what you mean but,
the tests were not updated for a long time, so it maybe a bug in the tests.
I checked anyway.co.il and unstable and clusters seem to appear fine.

It is now, but I broke it in b407f67d8d5b1fbfbaf0a5af01444d2e19a5711c and the unit test didn't fail. This pull request should make sure it doesn't happen again.


fix: load import map from eszip

What kind of change does this PR introduce?
This PR contains a fix to correctly load import map from eszips.

changes were incorporated into #217


Coordinator stops responding to http requests

Hi,
A couple of days ago my coordinator server stopped responding to http requests. Instead it hangs, never returning anything, until client raises a timeout error. There are no tracebacks in coordinator's log, its overall activity seems pretty normal (it still continues logging, never freezing completely). I tried to debug it with jstack -l, but couldn't find anything useful, unfortunately (coordinator.txt). Restarting coordinator/other nodes/cleaning up deep storage and tmp directories didn't help either.
Ubuntu 14.04, Druid 0.9.0, openjdk-7-jdk:amd64 7u121-2.6.8-1ubuntu<IP_ADDRESS>.
Thanks.

Ah, figured it out. Turns out it was a network interface issue: I used to run coordinator with _JAVA_OPTIONS="-Djava.net.preferIPv6Addresses=true -Djava.net.preferIPv6Stack=true", and disabling it solved the problem. Also I should've posted that on Google's Druid support group rather then here. Apologies.


deploykey_rsa: No such file or directory

Fixes #179

Ooh Didn't see this PR, made my own fix :\

@aheuermann would you mind opening a separate PR for docs fix

No problem, I knew the env variables was the real issue. Thanks for the fix!


Regression: Darktable segfaults with rocm 1.7. It was working on rocm 1.6.

Steps to reproduce on Ubuntu 17.10, with rocm 1.7:

Make sure you have the rocm ppa in your repository.
Uninstall rocm and restore the firmware for amdgpu (work-around for another bug)
apt remove rocm-dkms rocm-opencl
apt autoremove
apt install --reinstall linux-firmware
reboot
Install darktable and verify that it works:
apt install darktable
darktable
Install rocm afresh
apt install rocm-dkms rocm-opencl
reboot
Start darktable and watch it segfault.
darktable
If it does not segfault: Go to settings and ensure that OpenCl is available and enabled, then restart darktable.

Is there any way to downgrade to rocm 1.6?
This bug is a showstopper for me!

You know you can run Older OpenCL on 1.7 you do not have to swap out the whole foundation.    I ask our SQE group to see if Darktable is having an issue with the new 1.71 release we are working on  I let you know if we see anything
PS you can downgade via the archive by up packing the tar for 1.6.4 and installing from local repo.

One thing we need to check if Darktable is has a kernel that is doing out of bound memory reference, since,  we use guard pages that will fault the app upon an out of bounds memory reference.

Thank you gstoner! My Darktable is back!
For those who come across this thread. Here is how you get Darktable back to work:
Install rocm-dkms:
apt install rocm-dkms
From this site: http://repo.radeon.com/rocm/archive/
download and extract apt_1.6.4.tar.bz2
Now got to the directory apt_1.6.4/pool/main/r/rocm-opencl and install rocm-opencl_1.2.0-1464666_amd64.deb.
dpkg -i rocm-opencl_1.2.0-1464666_amd64.deb
Start Darktable and enjoy the speed of OpenCl!

This issue is solved since rocm opencl 1.2.0-2018030103


Refactor Conda Release Pipeline

[x] Eliminate conda artifacts from each individual service
[x] Centralize all conda generation/packages under conda/conda-releases
[x] Add new build definition conda-sdk-client.yml which will function as the primary integration point for beginning of release
[ ] Need to hash out dynamic meta.yml utilizing single common AZURESDK_CONDA_VERSION from a meta.yml vs doing a mass update of all the recipes.
[ ] Need to get msal/msrest/identity building in this pipeline

When queuing for release, we will :

Closing as this is FAR too spammy on the build system.


typo

Codecov Report

Merging #820 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #820      +/-   ##
==========================================
+ Coverage   80.99%   80.99%   +<.01%
==========================================
  Files          51       51
  Lines        4825     4826       +1
==========================================
+ Hits         3908     3909       +1
  Misses        917      917

Impacted Files
Coverage Δ

vent/helpers/meta.py
72.69% <0%> (-0.25%)
:arrow_down:

tests/test_api_plugins.py
96.51% <0%> (-3.49%)
:arrow_down:

vent/api/plugins.py
89.97% <0%> (-1.72%)
:arrow_down:

vent/api/actions.py
81.65% <0%> (+1.65%)
: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 9e618a3...6ebac09. Read the comment docs.


"Run utPLSQL test" is not showing up in sql developer tool menu option

OS: MacOS Mojave 10.14.6
Database Version: "Oracle Database 19c Enterprise Edition Release <IP_ADDRESS>.0 - Production
Version <IP_ADDRESS>.0"
SQL Developer Version: <IP_ADDRESS>
utPLSQL Version: v<IP_ADDRESS>49
utPLSQL-SQL Developer extension Version : v1.1.1
Executed Headless installation to install utPLSQL on oracle database.
Installed sql developer tool
Installed utplsql-sql developer extension without any issues but it's not showing up "Run utPLSQL test" in the menu option
This is how I installed utPLQL-SQL developer extension;
In SQL Developer - Help - Check for Updates
I install from local files --> utplsql_for_SQLDev_1.1.1.zip
New Installs: Database - utPLSQL for SQL Developer
press Finish

To finish installing updates, SQL Developer needs to restart.
Do you want to restart now? --> Yes
How to install utPLSQL for SQL Developer?
Can I provide more information?

When you select Tools -> Feature from the main menu, you should see something like this in the Installed Updates tab:

If utPLSQL for SQL Developer is missing then something went wrong during the installation of the extension. In this case I suggest to try it again and post every message you get during the installation process.
If the extension is installed, then please select Help -> About from the main menu and query for utPLSQL in the extensions tab. You should see the following:

The status should be fully loaded.
Please check also the extensions log via View -> Extensions Diagnostics. It's expected to be empty

If not, please post the content of this window.
Thank you.

Hi Philip, Thanks for your response.
utPLSQL for SQL Developer is missing under Tools -> Feature -> Installed Updates tab

utPLSQL is missing under Help -> About
Here's how I installed attaching screenshots;
Step#1:
In SQL Developer - Help - Check for Updates -->install from local files --> utplsql_for_SQLDev_1.1.1.zip

Step#2:
Click "Next" --> Click "Finish"

Step#3:
Do you want to restart now? Yes

Did I miss any step? Please provide me more information

This sounds similar to an issue I blogged about here. But you did not mention the "Some updates have already been installed since SQL Developer was started" message.
However, It looks like something went wrong during the internal JDeveloper/SQL Developer installation procedure. I assume that you find the following file $TMPDIR/$USER/jdeveloper-deferred-updates.txt and a directory named
/usr/$USER/.sqldeveloper/tmp containing files created during the installation procedure. This is an indication that something went wrong.
It looks like that your environment (user rights, Java version, ...) leads to errors during Oracle SQL Developer extensions installations. I assume you cannot install any SQL Developer extension. You may try Show Me Password. Can Please confirm that?
I created an additional, standard user (uid 502, no adminstrator rights) on my macBook and I installed a dedicated SQL Developer app, owned by this user. And in this setup I cannot install any SQL Developer extension as well. Does not matter if I use JDK8 oder JDK11. Strange.
Do you use a user with a uid different to 501 (open a terminal and type id to find out)?

I found a workaround for installing SQL Developer extensions when running as an (additional) standard user on macOS.
I've tested this successfully with an additional user named sqldev without administrator rights on my machine. Without this workaround I could not install any extension in SQL Developer.
1. Make sure you own the sqldeveloper.app
The sqldeveloper.app must be owned by the current user. If this is not the case or if you are unsure then download SQL Developer, extract the archive file and move it to a folder of your choice.
2. Start SQL Developer from command line
Open a terminal window and run the commands described in the following stpes.
2.1 Determine your JAVA_HOME
/usr/libexec/java_home

in my case the result is
/Library/Java/JavaVirtualMachines/jdk-11.0.5.jdk/Contents/Home

2.2 Go to the bin directory of SQL Developer
Execute the next command in the terminal window, adjust the path before SQLDeveloper.app accordingly:
cd /Users/sqldev/Downloads/SQLDeveloper.app/Contents/Resources/sqldeveloper/sqldeveloper/bin

2.3 Start SQL Developer
Run this command in the terminal window to start SQL Developer:
./sqldeveloper

2.4 Enter the Path of your JAVA_HOME
SQL Developer will prompt you for the path of your JAVA_HOME.
 Oracle SQL Developer
 Copyright (c) 2005, 2018, Oracle and/or its affiliates. All rights reserved.

/Users/sqldev/Downloads/SQLDeveloper.app/Contents/Resources/sqldeveloper/sqldeveloper/bin/../../ide/bin/launcher.sh: line 885: /usr/lib/exec/java_home: No such file or directory
Type the full pathname of a JDK installation (or Ctrl-C to quit), the path will be stored in /Users/sqldev/.sqldeveloper/19.2.1/product.conf

Enter result of step 1 here and press enter.
In my case this was /Library/Java/JavaVirtualMachines/jdk-11.0.5.jdk/Contents/Home.
3. Install Extensions
Now you can install utPLSQL for SQL Developer via Help -> Check for Updates... as you've done before. SQL Developer needs to be restarted to complete the process.
4. Done
From now on you can start SQL Developer by double clicking on sqldeveloper.app. No need to do it via the terminal window. However, if you plan to update, install or uninstall extensions then you have to do that again as described above.
I hope you can now install utPLSQL for SQL Developer.
Please note that this issue is not directly related with utPLSQL. It's a flaw of the Oracle SQL Developer extension installation process and should be addressed by Oracle. Maybe you can post your findings in the SQL Developer Forum. Thank you.

@itguyforautomation could you please give a feedback if the solution is working for you? Thank you.

Thanks very much for your assistance @PhilippSalvisberg , SQL developer tool works perfectly and able to run unit tests using utplsql.


Randstad - The number of jobs posted on Peviitor (15) and the one posted on the company website (17) isn't the same.

ENVIROMENT: production
URL: https://peviitor.ro/
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open "https://peviitor.ro/" in the browser.
Type "Randstad" in the search bar.
Check the number of jobs available for Randstad.
Open "https://www.randstad.ro/locuri-de-munca/lucreaza-la-randstad/".
Count the number of jobs displayed.

EXPECTED RESULTS:
The number of jobs posted on "https://peviitor.ro/" and the number of jobs posted on the company website should be the same.
ACTUAL RESULT:
There are 15 jobs posted on Peviitor website while on the company website there are posted 17 available jobs.

Before start testing go to https://scrapers.peviitor.ro
in search bar type "randstad"
click on randstand folder
click on "run scraper"
wait to scraper to finish
and compare the results
I tested it and the scraper returns the same result


Add option to increase/decrease level of redundancy

Hello, thank you for your package!
QR codes have several levels of redundancy which increases/decreases their size. It would be great if cades allowed to choose the redundancy in order to increase the robustness or reduce the number of pixels.

Thanks for your suggestion.
With version 0.3.0 I added support for qr code error correction levels.
They should be available using the typst package manager as soon as this PR lands.

Thank you!


Can't see that locked up stake accounts aren't part of circulating supply

Problem
When viewing an account, there's no indication that it is locked up and therefore not part of the circulating supply.
Proposed Solution
Add an indicator, perhaps in an "Account Details" dialog. Something that mimics the information currently available in the output of solana stake-account <ACCOUNT_ADDRESS>.

@garious this info in the "Account Details" page now, is that sufficient?

Yep, thanks so much!


Create styles (CSS) and implement

Work together with @AmeliaNestler to decide what looks best and fits Northwest Green Chemistry's theme

Create local css for modal: text to black from white (global css)

Referencing neutral color chart propose some off-white colors for tags body, h2 and a, and make the a tag slightly larger, in file app/app.global.css

body color: #F6F9ED;
h2 color: #FDF2EE;
a color: #FEF6E4;

font-size: 110%;

Reference pen


Do you mean snake_case instead of snack_case?

Description of change
Fix typo in README

Thanks a lot. Seems like I thought of having a snack while writing that : laughing:


Create Enormous Input Test.cpp

Related Issue
Solved issue number: #206
added solution to the problem "Enormous Input Test" listed on codechef
Language used for coding
C++
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] New and existing unit tests pass locally with my changes.

Follow pull request template.

Follow pull request template.

Please recheck @Nivedita967

Is it now okay?

Yes, all do follow for all pull request.

You have documented your code. So check all the checkboxes and it is possible to commit "Enormous Input Test.c" in this pull request itself.

You have documented your code. So check all the checkboxes and it is possible to commit "Enormous Input Test.c" in this pull request itself which belongs to same issue.

It will not happen again @Nivedita967


using provider metadata (tags, ...)

Multiple users reported a use-case to sync tags or arbitrary metadata from the secret provider. Yaml spec is defined in comment below.
Implementation status:

[x] Azure KV
[ ] AWS Secrets Manager
[ ] AWS Parameter Store
[ ] GCP Secrets Manager
[ ] Hashicorp Vault
[x] Kubernetes

The CRD changes were implemented in #1062 and look like this:
apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  name: example-external-secret
spec:
  # [...]
  data:
  # fetch all the tags in JSON format
  - secretKey: dev-secret-test
    remoteRef:
      key: dev-secret-test
      metadataPolicy: Fetch

  # fetch a specific tag which name must be in property
  - secretKey: dev-secret-test
    remoteRef:
      key: dev-secret-test
      metadataPolicy: Fetch
      property: tagname

  dataFrom:
  # similar like above, but returns an map[string]string.
  - extract:
      key: test
      metadataPolicy: Fetch

I'll track the implementation in the first comment :point_up_2:.

duplicate of #1872


No Entity to report Average Salt level

Hey there, looking through my app and I'm seeing that there's a diagnostic for instant salt, that is showing in the integration as pool_chlorinator_salt_level, but there's no corresponding entity for Average Salt. I'd rather alert off of average salt because the value is continuously available, while Instant Salt is only available while the system is actively chlorinating.
PS: Any updates on local connectivity?

I believe this was added already. Local integration is available through another developer.


With lazyload

Hello, thanks for this sliders, I tried a lot of them and this is the only only that uses a nice html.
I tried to use with lazyload but I don't know why the pictures do not show up :( Or is not needed to use lazyload at all? I have a lot of large images and wory that site loading would slow down.
Thanks a lot!

Anyone?
Thanks.


Add systemd unit files for automatic backups

For this to work you need to set the PGPASSWORD environment variable which can be done with systemctl edit silentselene-backup.service and typing
[Service]
Environment="PGPASSWORD=Password123"

Looks good, just add a comment (in the file or in a .md file somewhere) specifying that you need to set PGPASSWORD like that, since otherwise I'll forget if we ever have to move


Main dev

1)normalize objectstore as a normal rse
2)optimize s3objectsotre sitemover to have less HEAD operations: Dan reported that we had 4~5 times of 'HEAD' operations than 'GET' and 'PUT' operations. It caused load problem on objectstore.
3)update rucio sitemover to support os upload
4)set objectstore keypair in environment, rucio site mover will use it.

Thanks a lot Wen.


Merge development to master branch

I would like to merge the development branch to master. As date I would suggest next Tuesday, as  I'll be out of office from today evening. Would this be ok for you? @eagle00789

I'll do some more tests in the meantime, but next tuesday is fine with me

Op 3 jun. 2015 om 11:07 heeft Konstantin Wolff<EMAIL_ADDRESS>het volgende geschreven:
I would like to merge the development branch to master. As date I would suggest next Tuesday, as I'll be out of office from today evening. Would this be ok for you? @eagle00789
—
Reply to this email directly or view it on GitHub.

Then I will close this issue for now. I'll let you know before merging. If you find any show stoppers, just let me know and I will delay the merge.


Dependency Analysis

Would be nice if we could identify dependencies within a repo's code base
This could be more applicable once we get to the phase of viewing actual code, but doesn't necessarily have to depend on it
Ultimately the functionality would parse common dependency manager files and list out all the dependencies with links to original sources (at least when they're on GitHub 😄)
CocoaPods
Carthage
NPM
Rake
etc

GitHub already does something like this, but that doesn’t seem to work w/ CP/Carthage and friends: https://github.com/rnystrom/GitHawk/network/dependents

I think it’s actually a completely broken feature maybe? 😂 https://github.com/jquery/jquery/network/dependents

So I think that's the other way around, who uses this repo rather than taking a repo and listing what is uses. Obviously have it both ways would be nice!
I think "what does this repo use" is pretty simple as the information is all there
We could obviously use CocoaPod's API to see who uses the IGLK pod, or the NPM lookup for the same thing but it's a lot more involved


[Registry Preview] Support tabs in the UI to open multiple files

Description of the new feature / enhancement
Having multiple tabs with different files at the same time can be helpful. Can we add a tab feature.
Scenario when this would be used?
Working with multiple files.
  I hated MDI windows years ago.  Rejoiced for years, when Office sent each document to its own window instead of having a single app window with multiple files in it.
For me, it's as big (or spiritual) an issue for me as some people see tabs versus spaces in source.

Related and more widely-scoped issue: #26029
Whitespace around other parts of the line, not just the beginning of the line, also causes issues.
regedit accepts whitespace around these tokens, so Registry Preview should, too. 🙂

@qJake
You misunderstood something here. What I mean with this issue is having ui tabs for multiple files. 😅

@qjake, tabs and this are different.
 What I mean with this issue is having ui tabs for multiple files. 😅

My bad! I searched for a whitespace issue before posting mine, and this came up. Thanks for clarifying the title! 👍


my_decompile fails on IDA 7.0

I realize this repo doesn't strive to support Python 2 / IDA < 7.4, but I think a lot of it works out of the box except for this code right here: https://github.com/fireeye/FIDL/blob/e6ceb000cda43b450717eb171309c02dee06dd4f/FIDL/decompiler_utils.py#L1070-L1073
The version of this function present in 7.0 only accepts an address and a failure pointer, no flags. Would it be possible to somehow detect which one is available and call that? Or to add a new Python 2.7 and IDA < 7.4 release to call decompile() without the flags?

Hi,
the IDAPython API for the decompiler changes quite frequently, many times breaking backwards compatibility. This means that it would be a large effort for us to keep up / support all versions. We have decided to try to support the latest version only.
That said, if you are really interested, tell me a way to DM you and I'll try to give you some pointers :)

Hi Carlos,
Sure! You can DM me on Twitter or email me<EMAIL_ADDRESS>

@carlosgprado I have not heard from you yet so I would like to reiterate that the my_decompile function is the only broken component on 7.0 in many areas of this library, but the fact that all of the entrypoints expect your custom wrapper to be passed in—only to use the cfunc_t it carries—prevents me from working around this.
If your functions accepted cfunc_t instead, I could work around it in many cases without modifying the source. I realize that is not a very organized way to present an API, so it would be ideal if my_decompile could simply support the two different versions of ida_hexrays.decompile() :)
If that is not acceptable either, then I would like to publish a version for <7.4 on pip that I could maintain for others in my position. What do you think about this?


vimPlugins.typescript-tools-nvim: init at unstable-2023-10-04

Description of changes
Resolves #251855

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

could you add plenary as a dependency in pkgs/applications/editors/vim/plugins/overrides.nix (follow the examples), maybe lspconfig too. Also would be nice if you could run a nixpkgs-review on the result as these updates tend to break rust-based plugins

oh right, thanks!
does the commit message for updating the overrides.nix have a specific format?

vimPlugins.typescript-tools-nvim: add overrides for instance

Also would be nice if you could run a nixpkgs-review on the result as these updates tend to break rust-based plugins

I tried that earlier with the --post-result flag but my machine was struggling and I only have a few GBs left from my 256GB hard drive 😢.

Also would be nice if you could run a nixpkgs-review on the result as these updates tend to break rust-based plugins

If the update breaks some plugins, i could drop the commit

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2767


Extensive loading time for REXX parser

Hi,
I am using the VSCode extension IBM Z Open Editor to edit REXX file - extgension works fine. However I have REXX files with more than 2000 LoC and 2-3 dozens of Entrypoints. After opening a file like this it takes around 1min before outline is shown. Can this be improved?

Yes, the REXX performance when editing large or complex files is a known issue. We are working towards fixing it, although the underlying cause is a tricky one to resolve.

Many years ago - I was even using "vi" for C programming. Already at this time there was the possibility to include a "symbols" file  into the editor - and jump straight to the module in which method is implemented. This was not live parsing - it all comes from a file that was created (I believe) with a tool called "ctags".
All I wanna say - I understand that starting a Java VM and do live parsing will take a while. I am also okay, when this time is spent the FIRST time a file is loaded (only) - after parsing has been done, the results should be saved to some (hidden) file - and next time I start the IDE and load the module it will hopefully show the links faster.

Fixed in Z Open Editor 2.2.0


Question: can one plot hidden properties?

Hello, thanks for the great work!
I was wondering, can I plot hidden page properties like updated-at and created-at?
The concrete use-case I'm thinking about is getting a sense of the most active projects/pages, with a heatmap for instance.
Cheers,

At the moment the properties charts only work for properties of blocks, not of pages.  Sorry.


chore(main): release docs-site 1.8.0

:robot: I have created a release beep boop
1.8.0 (2024-06-01)
Features

docs-site: phrasing (#17461) (1279732)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/taikoxyz/taiko-mono/releases/tag/docs-site-v1.8.0 :sunflower:


Fix more low-hanging Coverity warnings

This PR fixes ~10 warnings out of 22.  Can you fix DCO?

DCO fixed.
Is there some way to get that signed-off-by magic text added automatically, or skip the check for committers with a CLA on file? I don't understand why it's different from the CLA we spent ~weeks getting the lawyers to sign.

cc @caniszczyk re: DCO workflow pain. We are working with GH to hopefully eventually make the flow better. The reason to use DCO is so that other companies don't have to spend weeks getting the CLA signed. 😉

All checks now pass.


Support automatically determining build tools

Add support for automatically determining what are the build-tools and extra-libraries needed to install any Cabal package.
In 1.0, the required build-tools are expected to be declared manually in .halcyon/sandbox-extra-apps.
Most of the work towards automation is already done, but is too slow to use on every push:
$ time (
>     lib_constraints=$( cabal_list_planned_lib_constraints /tmp/haskell-on-heroku-examples/hello-yesod-platform ) &&
>     echo "${lib_constraints}" | cabal_get_all_package_files 1 /tmp/lib_package_files &&
>     echo "${lib_constraints}" | cabal_dump_all_package_build_tools /tmp/lib_package_files | sed 's/^/build-tools: /' &&
>     echo "${lib_constraints}" | cabal_dump_all_package_extra_libs /tmp/lib_package_files | sed 's/^/extra-libs: /'
> )
build-tools: hsc2hs
build-tools: alex
build-tools: happy
extra-libs: z

real  0m11.015s
user  0m15.550s
sys   0m2.530s

This could be sped up by cabal get being able to output just the package description. (https://github.com/haskell/cabal/issues/1954 and haskell/cabalPR#1977).

Moved to https://github.com/mietek/halcyon/issues/9.


Find function by ordinal/hash + Dinvoke

I added two overloads to DynamicInvoke->Generic. GetLibraryAddress & GetExportAddress now support searching by ordinal.
You can grab PEBear (https://github.com/hasherezade/pe-bear-releases) to do some testing if you need to look up ordinals.
In code you can now do something like this:
MessageBox.Show("SamRidToSid -> " + String.Format("{0:X}", (
    SharpSploit.Execution.DynamicInvoke.Generic.GetLibraryAddress("samlib.dll", "SamRidToSid", true)
    ).ToInt64()));

MessageBox.Show("SamRidToSid -> " + String.Format("{0:X}", (
    SharpSploit.Execution.DynamicInvoke.Generic.GetLibraryAddress("samlib.dll", 0x32)
    ).ToInt64()));

I added another two overloads to GetLibraryAddress & GetExportAddress to support hash based export lookups. To facilitate the lookup I also added a small function, GenerateUniqueAPIHash, to compute keyed HMAC-MD5.
In code you can now do something like this:
Console.WriteLine(SharpSploit.Execution.DynamicInvoke.Generic.GenerateUniqueAPIHash("TpReleaseWait", 0xaabb1122).ToLower());

MessageBox.Show("TpReleaseWait -> " + String.Format("{0:X}", (
    SharpSploit.Execution.DynamicInvoke.Generic.GetLibraryAddress("ntdll.dll", "TpReleaseWait")
    ).ToInt64()));

MessageBox.Show("TpReleaseWait -> " + String.Format("{0:X}", (
    SharpSploit.Execution.DynamicInvoke.Generic.GetLibraryAddress("ntdll.dll", "9b0af3c44cecb470b24d56b7ccafcf5b", 0xaabb1122)
    ).ToInt64()));

I added 3 new functions to Dinvoke -> NtAllocateVirtualMemory, NtFreeVirtualMemory, NtQueryVirtualMemory.
Both NtAllocateVirtualMemory, NtFreeVirtualMemory now have generic wrappers that can be called. NtQueryVirtualMemory is used as part of a wrapper, GetFilenameFromMemoryPointer. This wrapper allows you to provide a process handle and an arbitrary memory pointer and if the pointer is backed by a file on disk you will get the full path to that file, if not it returns String.Empty. I also added some extra missing constants and status codes.

Fantastic work @FuzzySecurity !


Project on Modrinth

Hey, thanks for your work on the project. I would love to see the mod available on Modrinth as well. Maybe you could consider uploading the builds there as well. Thanks ^^

Hello! We actually planned to do that, I just forgot to upload it there. Should be on Modrinth now as intended.


Streaming Api now available with both POST and GET request methods

A post request can avoid the common 2000 characters url length limit.
It is useful if you need to request a very long list of hashtags or twitter ids.

@gabfusi perhaps the better solution would be to just default to POST always?
Is there a link documenting this change in the Twitter API?

I found the link - https://dev.twitter.com/streaming/reference/post/statuses/filter.  Looks like its only documented for the public API (not site and user).  If this is the case, we would not be able to default to POST as I suggested above.

@desmondmorris yes, that's the case.
My approach was quick and rough, but that script is on production right now and works well.
How about adding an additional method to perform post requests?

Closing as this pr would rename the npm package.  These changes have landed in @gabfusi's fork.


Add capability to render to cpu buffer or double buffer

Objective
The goal of the PR is to provide a straightforward way to:

Render to a cpu buffer that can be used to output images files.
Render to a double buffer, allowing the image to be easily used as a texture in the same render layer.
Simplify rendering to a texture. Doing so without having to directly manipulate the render graph.

There are three examples showing each of these respective methods:
examples/3d/frame_capture_cpu.rs
examples/3d/frame_capture_double_buffer.rs
examples/3d/frame_capture_gpu.rs
The FrameCapturePlugin is also setup to allow use of multiple, simultaneous, renders using a variety of targets.
The location of the FrameCapturePlugin was lightly discussed on discord. There isn't currently a great place for utility plugins like this one.

Super cool! A few comments on the examples:

Please add quick module level doc comments at the top of each explaining what their trying to do.
It feels very awkward that these are in the 3D folder. There's fundamentally nothing in them that wouldn't work for 2D. @mockersf, are you on board with creating a rendering examples folder? I know we've discussed this before, but this is a clear case of the need.
Can we save the rendered textures to file? If so, that would be incredibly valuable to demonstrate in an example in the root level tests folder

@alice-i-cecile

Please add quick module level doc comments at the top of each explaining what their trying to do.

Just added some doc comments. Is what I added the sort of thing you are looking for?

It feels very awkward that these are in the 3D folder.

Agreed. I'll have to do some testing to make sure these can work in 2D as well.

Can we save the rendered textures to file?

Yep! That's what the frame_capture_cpu example currently does, and is part of the motivation for adding this feature.

WRT the comments, see https://github.com/bevyengine/bevy/pull/4438 :)

@DGriffin91 does this fix #22? If so, please add it to the PR description.

@alice-i-cecile It should at least help with the "copy texture data from the gpu to an image file" portion. I just tried making a headless example and haven't been able to.
If I manually add plugins:
.add_plugin(CorePlugin)
.add_plugin(TransformPlugin)
.add_plugin(HierarchyPlugin)
.add_plugin(AssetPlugin)
.add_plugin(ScenePlugin)
.add_plugin(RenderPlugin)
.add_plugin(CorePipelinePlugin)
.add_plugin(GltfPlugin)
.add_plugin(PbrPlugin)

I get an error on this line:
https://github.com/bevyengine/bevy/blob/328c26d02c50de0bc77f0d24a376f43ba89517b1/crates/bevy_render/src/lib.rs#L128
thread 'main' panicked at 'Requested resource bevy_window::windows::Windows does not exist in the `World`.
                Did you forget to add it using `app.add_resource` / `app.init_resource`?
                Resources are also implicitly added via `app.add_event`,
                and can be added by plugins.', crates\bevy_render\src\lib.rs:128:41

And if I try the wgpu backend: None from the headless_defaults.rs with:
.insert_resource(WgpuSettings {
    backends: None,
    ..default()
})
.add_plugins(DefaultPlugins)

I get the error:
thread 'main' panicked at 'Sub-App with label 'RenderApp' does not exist',
[...]\bevy_app\src\app.rs:884:27

So I'm not sure if headless rendering in generally possible in bevy yet. And I also don't know if making headless rendering possible is in the scope of this PR.

Sounds good. Headless rendering has been a bit of a headache for a long time, so that doesn't surprise me. Fixing that is definitely out of scope.

waiting on the "high level camera driven render targets" PR

@alice-i-cecile It should at least help with the "copy texture data from the gpu to an image file" portion. I just tried making a headless example and haven't been able to.
If I manually add plugins:
.add_plugin(CorePlugin)
.add_plugin(TransformPlugin)
.add_plugin(HierarchyPlugin)
.add_plugin(AssetPlugin)
.add_plugin(ScenePlugin)
.add_plugin(RenderPlugin)
.add_plugin(CorePipelinePlugin)
.add_plugin(GltfPlugin)
.add_plugin(PbrPlugin)

I get an error on this line:
https://github.com/bevyengine/bevy/blob/328c26d02c50de0bc77f0d24a376f43ba89517b1/crates/bevy_render/src/lib.rs#L128
thread 'main' panicked at 'Requested resource bevy_window::windows::Windows does not exist in the `World`.
                Did you forget to add it using `app.add_resource` / `app.init_resource`?
                Resources are also implicitly added via `app.add_event`,
                and can be added by plugins.', crates\bevy_render\src\lib.rs:128:41

And if I try the wgpu backend: None from the headless_defaults.rs example:
.insert_resource(WgpuSettings {
    backends: None,
    ..default()
})
.add_plugins(DefaultPlugins)

I get the error:
thread 'main' panicked at 'Sub-App with label 'RenderApp' does not exist',
[...]\bevy_app\src\app.rs:884:27

So I'm not sure if headless rendering in generally possible in bevy yet. And I also don't know if making headless rendering possible is in the scope of this PR.

Only the WinitPlugin should be disable to prevent the windowing and associated rendering. This scripts works with frame_capture_cpu example:
App::new()
...
        .add_plugins_with(DefaultPlugins, |group| group.disable::<WinitPlugin>() )
...

However, only one frame is rendered and then the app exits, since WinitPlugin provides built-in eventloop to trigger app to run forever.

@DGriffin91 https://github.com/bevyengine/bevy/pull/4898 has been merged, could you update this PR? I would love to see it move forward.

I tested this PR on 2d camera and it just works. But I think the whole extract_camera_phases thing is creating redundant code, as it is identical to what is done for 2d/3d camera

I tested this PR on 2d camera and it just works (by changing 3d subgraph rendering to 2d, etc). I think maybe the whole extract_camera_phases thing is creating redundant code, as it is identical to what is done for 2d/3d camera

If I remove this (below) from extract_camera_phases with the frame_capture_cpu example, it doesn't output the 3D model and I just get a blank grey png.
.insert_bundle((
    RenderPhase::<Opaque3d>::default(),
    RenderPhase::<AlphaMask3d>::default(),
    RenderPhase::<Transparent3d>::default(),
))

@DGriffin91 Yes, I didn't mean it's unnecessary, it's just a duplication of  what is done in camera3d pipeline

@zimond Gotcha, that makes sense. And thank you! We are discussing how to get this setup with the new camera driven stuff on discord here in case you are interested: https://discord.com/channels/691052431525675048/986788850535915540/

I'm looking forward to using this feature. I hope it gets merged soon!

Is it possible to save a PNG image with the wgpu::Texture struct? I only found a way to do this with the wgpu::Buffer which requires the use of wgpu::CommandEncoder::copy_texture_to_buffer method.

@paulkre Check out this newer PR. It outputs PNG images: https://github.com/bevyengine/bevy/pull/5550

@dgriffin91 shall I close this out in favor of #5550?

@alice-i-cecile Yep, I think that makes sense.


Adding a shared database

Is there a function in superlogin-client similar to to the addUserDB() function in the original superlogin module? I need to add new CouchDB databases, which can be shared by multiple users.

You should be able to find the URLs of your custom DB using #getDbUrl().
Basically all the database a user can access are visible under the property userDBs of the session.

Yes, I know about userDBs. But is there a way to create a shared database on the CouchDB server from client site?

No there is no api for it.
If your feel It's really necessary you should talk with @colinskow to add a route for it.
But from my perspective it goes beyond the current scope.
Such an api would mostly be needed for cms with admin permission.

Ah, I misunderstood superlogin's addUserDB(). It is only adding existing databases to a user, but cannot create a new one. Thanks for your answers!


Fix/NarrativeQA-test dataset

Checklist:

[ ] I've added Google style docstrings to my code.
[ ] I've used pydantic for typing when/where necessary.
[ ] I have linted my code
[ ] I have added tests to cover my changes.

NarrativeQA-test.jsonl   dataset have some  problem  .
in some of the records it contain  double quotes two time

because of this it will generate error


Error importing VTooltip during production webpack build with UglifyJS

I recently ran into an error that seems like it's coming from this library. When running my production webpack build, I get the following error:
ERROR in vendors-c487e72a0905278b058b.min.js from UglifyJs
Unexpected token: keyword «const» [vendors-c487e72a0905278b058b.min.js:134515,0]

When I investigated this error and line, it led me to the code imported from dist/v-tooltip.umd.js
// dist/v-tooltip.umd.js

// ... other code

const __vue_script__ = script;

// ... other code

/* style */
const __vue_inject_styles__ = undefined;
/* scoped */
const __vue_scope_id__ = undefined;
/* module identifier */
const __vue_module_identifier__ = undefined;
/* functional template */
const __vue_is_functional_template__ = false;

In my Vue code, I'm simply importing the library with these lines:
import VTooltip from "v-tooltip"
Vue.use(VTooltip)

It looks like these consts were added only a day ago in https://github.com/Akryum/v-tooltip/commit/19cd67b43b9736d435dce28ddaad074e29b00a25 to dist/v-tooltip.umd.js
Since it's common practice to ignore transpiling code imported from node_modules (because they should be transpiled already), I think these consts need to be removed from the dist/v-tooltip.umd.js file.
Let me know if you have any questions or if I can provide any more information.
Thanks!

this broke my app entirely...

reverting back to "v-tooltip": "2.0.0-rc.33" "fixed" it

Same issue here.

I'm having yet the same issue with v2.0.3.
Reverted back to 2.0.0-rc.33 by now.

Still happening to me on v2.0.3.

Exact same error as @overallduka on v2.0.3

@overallduka set dependency in package.json to "2.0.0-rc.33", NOT "^2.0.0-rc.33" or "~2.0.0-rc.33". There is a "let" command on 2.0.3 version, which makes it ES6, hence the error. You can use anything up to 2.0.2.


refactor(linter): read exported_bindings_from_star_export lazily

Merge activity

Dec 22, 3:38 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.