A syntax error

In a sequnce file "https://github.com/google/riscv-dv/blob/master/src/riscv_privileged_common_seq.sv", there are two syntax error should be corrected.
(1)  line 199:
instrs.push_back($sformatf("csrw 0x%0x, x%0d // satp", SATP, cfg.gpr[0]));
should be:
instrs.push_back($sformatf("csrw 0x%0x, x%0d # satp", SATP, cfg.gpr[0]));
(2) line 208:
instrs.push_back($sformatf("csrs 0x%0x, x%0d // satp", SATP, cfg.gpr[0]));
should be:
instrs.push_back($sformatf("csrs 0x%0x, x%0d # satp", SATP, cfg.gpr[0]));
Some guys of this project should do some double check, and modify them.

Thanks for reporting, this is fixed now

Thanks for reporting, this is fixed now


BugFix: When loading non-strict, make sure to read in the tensor bytes

Added a bug fix where when you load in a model in non-strict mode, if it skips a parameter load it still reads it in so that the BinaryReader can seek forwards to the next tensor.
NOTE: I also modified the behavior so that if you specify a parameter in the "skip" list, then it no longer requires the dimensions to match.
I haven't updated the release notes either, what version should I list this bugfix under?
Also, as part of this feature I wrote a unit test which covers the unit test from the previous PR, so I removed the old one.

@shaltielshmid : thx you for helping us and your valuable contribution ☺

I haven't updated the release notes either, what version should I list this bugfix under?

It'll be 0.101.2 -- please add a header for that, as well as updating https://github.com/dotnet/TorchSharp/blob/main/build/BranchInfo.props with the new patch release number.

Sounds good, I'll update the version.
One note: My other PR for loading/saving in python format is ready, just waiting for a different package to merge my PR there. Should we release these two items together, and make it a minor version?

That one will require a more thorough review, and we usually don't do a release for just one of two PRs.
The 0.101.1 release was an exception, because it addressed a significant enough bug that had blocked several users for quite a while to make it worthwhile.
Sent from Outlookhttp://aka.ms/weboutlook

From: shaltielshmid @.>
Sent: Monday, November 6, 2023 8:52 AM
To: dotnet/TorchSharp @.>
Cc: Niklas Gustafsson @.>; Comment @.>
Subject: Re: [dotnet/TorchSharp] BugFix: When loading non-strict, make sure to read in the tensor bytes (PR #1137)
Sounds good, I'll update the version.
One note: My other PR for loading/saving in python format is ready, just waiting for a different package to merge my PR there. Should we release these two items together, and make it a minor version?
—
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/TorchSharp/pull/1137#issuecomment-1795479493, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AANVMB6L5DWYDB4STSHSCWDYDEIUTAVCNFSM6AAAAAA656LPCGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJVGQ3TSNBZGM.
You are receiving this because you commented.Message ID: @.***>

Ok! I'll update with the changes you mentioned and the version.

Done. Undid breaking change + increased version.


Create the parties table

Title:
Create the parties table
Body:
User Story

As a user, I can update a candidate's party affiliation.

As a user, I can request a single candidate's information, including party affiliation.

As a user, I can request a list of all the parties.

As a user, I can request a single party's information.

As a user, I can delete a party.

As a user, I can request a single candidate's information.

As a user, I want to delete a candidate.

As a user, I want to create a candidate.

All parties related query routes have been completed.


Connecting to server task should be cancelled if user presses Esc to go back to main menu or attempts to connect to another server during this time

ManagerNetwork.StartClient(ip, port) should make use of TokenManager
https://github.com/GodotModules/GodotModulesCSharp/blob/7f2e7604bdc17251a7837e5e5b9b3c4b05a488b2/Scripts/Scenes/SceneGameServers.cs#L73-L74
Please have a look at
https://github.com/GodotModules/GodotModulesCSharp/blob/7f2e7604bdc17251a7837e5e5b9b3c4b05a488b2/Scripts/Managers/TokenManager.cs#L3-L27
and see how it is used with Web POST / GET tasks. (WebManager.cs)

Latest commits cancel the task for waiting for client to connect but not the actual client task.
Thread safety needs to be considered here.


[master] Bump Mesos to nightly master a5b9f6e

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/50167a73bd53884b11330b2598d220f71aef7c17...a5b9f6e1cdb2ec26baf6e49706e1a0d59f3ce4d1
https://github.com/dcos/dcos-mesos-modules/compare/d566b1416c6833b5ea03e681bc1bdcb14aee5277...92e932c524fac7349d3a1e31b780bacefdfd21c5

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review


load_dotenv() is called no matter what

Describe the bug
It appears that load_dotenv() is called by this library no matter what, which can have impact on the entire operating environment if running in a situation where the user does not expect this behavior.
To Reproduce
Steps to reproduce the behavior:
Use/load the module, without calling load_env() in the calling program, and see that any environment variables you set in .env are now visible in os.environ.
Expected behavior
I would not ever expect a third party module to automatically call load_dotenv() for me, because of the impact across the entire operating environment for the running process.  Instead, I'd expect the library to simply document what environment variables I can set for configuration, and if I choose to use .env to set them I can load_dotenv() myself before using the module.
Additional context
Causes issues outside of the module itself if load_dotenv() calling is not expected.

Ideally python-dotenv would be a dev-dependency at best for the library.

@entropi Thanks for opening the issue.  We will look into this and provide an update next week.

@entropi this is addressed in version 4.0.0 as we considered it a positive but breaking change

Thank you!


Full Elasticity

Full elasticity should be implemented in short order. That is, automatic rebalancing of processing when nodes drop from a cluster or are added to a cluster.
Since Dempsy was built from the ground up with elasticity in mind, this should be straightforward.

Elasticity has been working since 0.8. The 0.9 refactor it still works. What's missing from both is the ability to transfer the state during cluster topology changes.


WIP: support aws new regions without sdk upgrades

infra issue managed-clonerefs: manifest unknown
retest should be fine

@flavianmissi  Does the failure of  e2e-hypershift job is related to the pr? if no I will add qe-approved label

The hypershift failures do seem related to the image registry, here's what I see in the logs of one of the runs:
2024-12-05T08:28:35.209665387Z time="2024-12-05T08:28:35.209596755Z" level=error msg="s3aws: WebIdentityErr: failed to retrieve credentials\ncaused by: SerializationError: failed to unmarshal error message\n\tstatus code: 405, request id: \ncaused by: UnmarshalError: failed to unmarshal error message\n\t00000000  3c 3f 78 6d 6c 20 76 65  72 73 69 6f 6e 3d 22 31  |<?xml version=\"1|\n00000010  2e 30 22 20 65 6e 63 6f  64 69 6e 67 3d 22 55 54  |.0\" encoding=\"UT|\n00000020  46 2d 38 22 3f 3e 0a 3c  45 72 72 6f 72 3e 3c 43  |F-8\"?>.<Error><C|\n00000030  6f 64 65 3e 4d 65 74 68  6f 64 4e 6f 74 41 6c 6c  |ode>MethodNotAll|\n00000040  6f 77 65 64 3c 2f 43 6f  64 65 3e 3c 4d 65 73 73  |owed<Mess|\n00000050  61 67 65 3e 54 68 65 20  73 70 65 63 69 66 69 65  |age>The specifie|\n00000060  64 20 6d 65 74 68 6f 64  20 69 73 20 6e 6f 74 20  |d method is not |\n00000070  61 6c 6c 6f 77 65 64 20  61 67 61 69 6e 73 74 20  |allowed against |\n00000080  74 68 69 73 20 72 65 73  6f 75 72 63 65 2e 3c 2f  |this resource.</|\n00000090  4d 65 73 73 61 67 65 3e  3c 4d 65 74 68 6f 64 3e  |Message><Method>|\n000000a0  50 4f 53 54 3c 2f 4d 65  74 68 6f 64 3e 3c 52 65  |POST</Method><Re|\n000000b0  73 6f 75 72 63 65 54 79  70 65 3e 53 45 52 56 49  |sourceType>SERVI|\n000000c0  43 45 3c 2f 52 65 73 6f  75 72 63 65 54 79 70 65  |CE</ResourceType|\n000000d0  3e 3c 52 65 71 75 65 73  74 49 64 3e 53 37 44 5a  |><RequestId>S7DZ|\n000000e0  37 4d 41 46 42 51 44 32  33 58 35 4b 3c 2f 52 65  |7MAFBQD23X5K</Re|\n000000f0  71 75 65 73 74 49 64 3e  3c 48 6f 73 74 49 64 3e  |questId><HostId>|\n00000100  6e 6c 4a 2f 73 61 4a 6a  36 5a 59 4f 53 71 59 44  |nlJ/saJj6ZYOSqYD|\n00000110  34 77 78 71 6c 53 78 64  70 4b 65 73 77 6f 59 63  |4wxqlSxdpKeswoYc|\n00000120  38 56 77 65 36 62 72 39  4c 54 51 4f 6e 31 42 30  |8Vwe6br9LTQOn1B0|\n00000130  45 4d 59 6d 4d 75 47 76  45 37 79 33 67 64 74 45  |EMYmMuGvE7y3gdtE|\n00000140  73 4d 54 43 35 71 6c 34  31 46 63 3d 3c 2f 48 6f  |sMTC5ql41Fc=</Ho|\n00000150  73 74 49 64 3e 3c 2f 45  72 72 6f 72 3e           |stId></Error>|\n\ncaused by: unknown error response tag, {{ Error} []}" go.version="go1.22.7 (Red Hat 1.22.7-1.el9_5) X:strictfipsruntime"

So it seems that setting REGISTRY_STORAGE_S3_REGIONENDPOINT to https://s3.amazonaws.com in hypershift is causing the 405. I must find the time to look into this.

@flavianmissi Do you find any clue for the issue? or i can do something thing for it?

I haven't made progress here yet @wewang58. I'm planning to dig a little more today, thanks for the reminder!
I'll post an update here later today.

Here's the clean the output of the operator logs from my previous comment:
s3aws: WebIdentityErr: failed to retrieve credentials
caused by: SerializationError: failed to unmarshal error message
	status code: 405, request id:
caused by: UnmarshalError: failed to unmarshal error message
	00000000  3c 3f 78 6d 6c 20 76 65  72 73 69 6f 6e 3d 22 31  |
	<?xml version=\"1.0\" encoding=\"UTF-8\"?>.<Error>MethodNotAllowed<Message>The specified method is not allowed against this resource.</Message><Method>POST</Method><ResourceType>SERVICE</ResourceType><RequestId>S7DZ7MAFBQD23X5K</RequestId><HostId>nlJ/saJj6ZYOSqYD4wxqlSxdpKeswoYc8Vwe6br9LTQOn1B0EMYmMuGvE7y3gdtEsMTC5ql41Fc=</HostId></Error>
caused by: unknown error response tag, {{ Error} []}" go.version="go1.22.7 (Red Hat 1.22.7-1.el9_5) X:strictfipsruntime"

This didn't help much. I added a commit to this PR enabling debug logs in the s3 sdk in the hopes that we can get more details about the failure on the next hypershift job run. 🤞🏼


feat(scanner): Initialize Scanner v4 health checks

Description
This initializes the Scanner v4 gRPC health checks as defined in https://github.com/stackrox/stackrox/pull/7630.
Checklist

[x] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
Nah

Whats the plan for authorization?
Assuming the health check service will not be requiring mTLS so that k8s is able to call it, but the IndexerService will require it.

Correct, TLS is not currently supported by the probe (see here). Note it is possible to set different auth rules for different endpoints. For example, take a look at the image service. So, we can set some auth rules for the other endpoints, and not for this one

superseded by https://github.com/stackrox/stackrox/pull/9531


[WebAPI] Sanitizer API

Acceptance criteria

[x] The listed features are documented sufficiently on MDN

[ ] BCD is updated

[x] Interactive example and data repos are updated if appropriate

[ ] The content has been reviewed as needed

Features to document

Sanitizer API

Related Gecko bugs

[ ] Sanitizer API incorrectly hidden behind a pref — this is actually a bug to correctly hide the Sanitizer interface behind a pref as it is being developed, but it might be nice to get this spec documented, depending on what its status is in Chromium.

For folks helping with Firefox-related documentation features — make sure above AC have been done, but also:

Set bugs to dev-doc-complete
Add entry to Firefox release notes if feature is enabled in release
Add entry to Firefox experimental features page if feature is not yet enabled in release

Update - I've created draft pages which I need to go through, edit and add examples too 👍 This will remain in progress for the time being

How is this one going @Rumyra ?

Funny you should ask - I just finished the pages. Next to do is get them up and make sure all BCD, kumascript & everything is inline. This might be in 24hrs as I'm on Goog tomorrow 👍

That is perfectly OK — we've still got plenty of time until the launch.

I'll be adding all the info for this tomorrow... Just one thing: I've created a demo, however it explicitely uses the config object, which has not been implemented yet. see https://wicg.github.io/sanitizer-api/#dictdef-sanitizerconfig (it's optional), so am thinking to hold off adding to demo repo for the moment (basically it doesn't work 😆)
I have documented it however and am happy to add a note for now.

Yeah, just stick to code snippets for now, and add a note explaining that it doesn't work yet.

Temp checklist

[ ] Kumascript
[ ] BCD
[ ] Docs
[ ] Bug
[ ] Release notes
[ ] Experimental

Vinyl is on BCD for this one - I'll update as per 👍

Thanks @Rumyra , looking good so far.

Did this one get finished, @Rumyra ?

Did this one get finished, @Rumyra ?

dev-doc-needed flag needs to be removed from the bug (sorry I'm unable to do this)
Added BCD data myself 🤷🏻♀ apologies I should have followed up on this one...
https://github.com/mdn/browser-compat-data/pull/8794
Last little thing:

[ ] Check pages once BCD data has been merged
https://developer.mozilla.org/en-US/docs/Web/API/Sanitizer

dev-doc-needed flag needs to be removed from the bug (sorry I'm unable to do this)
Added BCD data myself 🤷🏻♀ apologies I should have followed up on this one...
https://github.com/mdn/browser-compat-data/pull/8794
Last little thing:

[ ] Check pages once BCD data has been merged
https://developer.mozilla.org/en-US/docs/Web/API/Sanitizer

dev-doc-needed flag needs to be removed from the bug (sorry I'm unable to do this)

No worries, I can do that. Do you mean removed, or set to dev-doc-complete? The work has bene done, right? We are not cancelling the work?

dev-doc-needed flag needs to be removed from the bug (sorry I'm unable to do this)

No worries, I can do that. Do you mean removed, or set to dev-doc-complete? The work has bene done, right? We are not cancelling the work?

😆 yes that's what I meant - I'm going to take a break now before my brain mis fires everything

😆 yes that's what I meant - I'm going to take a break now before my brain mis fires everything

😆 yes that's what I meant - I'm going to take a break now before my brain mis fires everything

No worries, and done!
https://bugzilla.mozilla.org/show_bug.cgi?id=1673309#c11

😆 yes that's what I meant - I'm going to take a break now before my brain mis fires everything

No worries, and done!
https://bugzilla.mozilla.org/show_bug.cgi?id=1673309#c11


linting issue

Fixing a file with linting turned off, then turning linting on, but not flushing the cache causes issues because php-cs-fixer is now regarding that file with invalid syntax as fine, because it skipped linting it, even though the previous run of php-cs-fixer broke the file.

The issue is we consider the null linter to have linted the file. We need to change that concept to introduce the concept of skipping linting instead.

I prefer to have NullLinter instead of skipping linting. It simplify code a lot.
Why not add information which linter was used to cache file?

I guess that will do the equivalent thing.

@keradus I've implemented your idea here: https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/1845.

Closing due to merge of #1845


Strange "ignoring extra certs" warning message when in sst start mode, upon 1st function invocation

When running sst start, the first time a function is invoked, this warning message is displayed:
Warning: Ignoring extra certs from /etc/pki/tls/certs/ca-bundle.crt, load failed: error:02001002:system library:fopen:No such file or directory
This appears to have cropped up without any updates to the @serverless-stack/* packages.
https://serverless-stack.slack.com/archives/C01JG3B20RY/p1636072195234900

Yeah I can confirm that I ran into this as well.


Add reference to the fastcore tests in the Extract tests tab

Fixes issue #517
As somebody who is just learning nbdev, I was trying to figure out where nbdev testing functions come from, specifically test_eq(). So I went to the nbdev docs and clicked the first tab that has test  on it and to my surprise did not find the test_eq function definition. After some exploration I discovered the tests are actually defined in fastcore, but this was not obvious. This PR adds a link inside an important box on the nbdev testing page to the fastcore testing page to make this relationship clearer.
Before:

After:

Ok nevermind then.


Style: Fix typo in sat/vB

Just want to correct the capitalization in vB of the recent style guide update.

ACK, thanks!


Added support to textScaleFactor

Added a new property, the textScaleFactor, that map the corresponding textScaleFactor of the standard Text widget.
This property is fundamental in our production app, to correctly manage the accessibility options for scaled text.

Published as 1.7.0


Switch back to MSI Mystic Light possible?

Hi there,
not sure if this is normal.
Got a "MSI B450 Tomahawk Max" and now, MSI Mystic Light won't open up again after using MSIRGB.
PC is creating a instance of MSIML, as i see in the Task-Manager.
Also, "Pumpkin" is blue for me, so the colors are inverted here, while other B450TomahawkMax-Users say it's working flawlessly.
Am i just too dumb?
Greetings,
Frechbass

Seems unlikely that MSIRGB would cause Mystic Light not to work again, but I won't rule it out.
Mystic Light shouldn't be running at the same time as MSIRGB. I'm not sure if that's what you are doing.
Other than that, I'm not sure what I can do. Have you tried reinstalling Mystic Light?

Did you fix this?

Sorry for the silence...
Yeah, a reinstall fixed it.
After your update im hooked to your version again, everythings fine now.
No inverted colors, or something like that.
Thought about to do some custom-scripts in the future.
We will see.
Thanks for fixin' & have a nice day. :)

Great! You too :)


Flagging the same word twice

Description

If an unwelcome word appears in a user's message more than once, it should preferably tagged only once. This happens in a strange and inconsistent fashion as the screenshots say.
Screenshots

Additional Context

If the word is appended with a question mark (and I assume with any other special character, it appears as a new flag).
Join Eddie's discord community here

Checkout- #357 #393

Ahh, found the issue-
https://github.com/EddieHubCommunity/EddieBot/blob/848352364668d4ae5a226bb364223fd71c4f54f1/src/words.ts#L44-L49
The regex replaces symbols with ' ' but doesn't have something for ? and <word>? was seen as a new word, when we were putting that in a set to get unique list of words, single occurrence.


Figure out how to prevent errors with custom elements when transpiling to ES5

This code:
import steal from "@loader";

class El extends HTMLElement {
  connectedCallback() {
    this.innerHTML = "The e-l";
  }
}
customElements.define("e-l", El);
const el = document.createElement("e-l");

...will throw when loaded with stealjs. This is the error:
Uncaught TypeError: Failed to construct 'HTMLElement': Please use the 'new' operator, this DOM object constructor cannot be called as a function.
    at new El (index.js!eval:67)
    at HTMLDocument.createElement (<anonymous>:1:1554)
    at execute (index.js:9)
    at doExecute (steal.js:935)
    at ensureEvaluated (steal.js:1000)
    at doEnsureEvaluated (steal.js:963)
    at evaluateLoadedModule (steal.js:920)
    at steal.js:1140

Without the import statement this code works fine, but with it, it throws because it is being transpiled to ES5.
We either need to figure out how to prevent this when using Steal, or we need to fix this in Steal and make sure it works with other module loaders as well. Otherwise people will run into this issue all the time when using can-stache-define-element.

You can set the forceES5: false option in the steal config to prevent this.


Surah Info not showing complete text in mobile browser

I'm submitting a ...

[ x] bug report
[ ] feature request
[ ] other

Current behavior:

surah Info is not showing completely in mobile browser
Expected / new behavior:

The surah info section should be scroll-able to view full text
For issue, please provide steps to reproduce it:

Steps
to
reproduce
In Android
Open Chrome
Go to quran.com/2
settings
surah info

Anything else:

There is no scrollbar in surah info section

Only partial surah info text is shown


contrast stretching

Hello,
I am interested in implementing the contrast stretching [1] feature on top of cornerstone.
Would you be able to give some high-level guidance on how I can go about doing the same.
Best, Anupam
http://what-when-how.com/embedded-image-processing-on-the-tms320c6000-dsp/contrast-stretching-image-processing/

Windowing, also known as grey-level mapping, contrast stretching, histogram modification or contrast enhancement

Reference Source

It sounds like "Contrast Stretching" is what is commonly referred to as Window Leveling. This functionality is already supported. You can see a basic example of it in this repository, and as the WW/WC tool in cornerstone-tools.
If I've misunderstood your question, please feel free to continue the discussion.

Contrast stretching is not the same thing as window leveling. There are different ways to accomplish it. One, way as in the link, to determine the histogram and knock out the statistically bad pixel. In their case they used the data between the 5th and 95th percentiles. Other names I've seen are histogram stretching, histogram equalization, histogram normalization, and histogram flattening. See

https://slidetodoc.com/image-enhancement-in-the-spatial-domain-human-vision/
https://stackoverflow.com/questions/69595948/how-do-you-normalize-an-image-histogram-stretching-using-image-js-and-javascri

I appreciate the correction!


Update Project.toml

Codecov Report

Merging #330 (77be4f7) into main (cb7ed06) will increase coverage by 1.62%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #330      +/-   ##
==========================================
+ Coverage   77.99%   79.62%   +1.62%
==========================================
  Files          40       40
  Lines        1477     1477
==========================================
+ Hits         1152     1176      +24
+ Misses        325      301      -24

see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Type2] Allow Incremental Refresh type schedules to be added via server.schedules.add_to_schedule

Summary
For content with incremental refresh set up, allow for functionality to assign schedules as Incremental Refresh.

Description
Currently, there is no option to add incremental refresh type schedules via TSC. This functionality exists on Tableau Server when content is set up for incremental refreshes. server.schedules.add_to_schedule has an option for task_type (code), but no string option is acceptable for incremental type.

This is being worked on internally and should be available in 2023.

Any progress ?


Fix typo when referencing new tag for Docker labels

Change-type: patch
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>

@balena-ci I self-certify!


Remove unused json/simplejson packages

Since the move to use request package instead of urllib package,
There is no need for an external JSON loads function.
In addition, The import sys statement has been removed from the same reason.
Relevant commits:
Fix url forming issue
Change urllib for requests

In general,
Default value is evaluated only once. This makes a difference when
the default is a mutable object such as a dictionary:
>>> from alpha_vantage.timeseries import TimeSeries
>>>
>>> ts1 = TimeSeries(key='demo')
>>> ts2 = TimeSeries(key='demo')
# ts1 and ts2 are not identical objects.
>>> ts1 is ts2
False
# but the proxy attribute of both refer to the same default dict which is a mutable object.
>>> ts1.proxy is ts2.proxy
True
>>> id(ts1.proxy), id(ts2.proxy)
(2181378373456,<PHONE_NUMBER>456)
>>> ts1.proxy['http'] = 'http://<IP_ADDRESS>:3128'
>>> ts1.proxy
{'http': 'http://<IP_ADDRESS>:3128'}
>>> ts2.proxy
{'http': 'http://<IP_ADDRESS>:3128'}

For more information:
why-are-default-values-shared-between-objects
There are few possible fixes, but I choose to be consistent with  requests.sessions.Session implementation.
https://github.com/requests/requests/blob/master/requests/sessions.py#L501
Note: This should be a different PR.

I will be creating the release soon ;) Thanks for the good work @eitanp82


[API] get_account_history unnecessarily restricts result size

get_account_histroy contains a FC_ASSERT(limit <= 100) which prevents retrieving more than 100 results at any one time. As the owner of my own witness, I should be able to decide how many results this function returns, regardless of performance degradation to my own hardware.

Duplicate of #326


Tag v1.0.1 is outdated and contains a bug, fixed in main branch

Tag v1.0.1 is outdated and contains a bug, fixed in main branch
main branch
https://github.com/enhance-dev/enhance-element/blob/5e3a95881441eeeec0caa5750474708b0b718945/index.mjs#L51
v1.0.1
https://github.com/enhance-dev/enhance-element/blob/1acd139f586aa849629130fc27ddc35edba72910/index.mjs#L51

We released v1.0.2 to fix this issue.


Remove version: Tencent.COSBrowser version 2.10.1

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://cos5.cloud.tencent.com/cosbrowser/cosbrowser-setup-2.10.1.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 896)

URL: https://cos5.cloud.tencent.com/cosbrowser/cosbrowser-setup-2.10.1.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 896)


InAppPurchaseAndroidPlatformAddition does not have the launchPriceChangeConfirmationFlow method as specified in the README of the package

Is there an existing issue for this?

[X] I have searched the existing package issues
[X] I have read the guide to filing a bug

What package does this bug report belong to?
in_app_purchase
What target platforms are you seeing this bug on?
Android
Have you already upgraded your packages?
Yes
Dependency versions
pubspec.lock
[Paste file content here]

Steps to reproduce
I would like to implement the flow to handle subscription price changes on Android.
As specified in the README of the package we should use the launchPriceChangeConfirmationFlow method, but it's not defined in the InAppPurchaseAndroidPlatformAddition class
Expected results
The documentation should probably be updated to describe how to handle the subscription price changes for Android
Actual results
``
Code sample
Code sample
[Paste your code here]

Screenshots or Videos

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Hi @OlegNezv. Thanks for filing this issue. Can you explain further on what you mean by it's not defined in the InAppPurchaseAndroidPlatformAddition class. From the InAppPurchaseAndroidPlatformAddition class documentation, we have:

/// Initiates a flow to confirm the change of price for an item subscribed by the user.
///
/// When the price of a user subscribed item has changed, launch this flow to take users to
/// a screen with price change information. User can confirm the new price or cancel the flow.
///
/// The skuDetails needs to have already been fetched in a
/// [InAppPurchaseAndroidPlatform.queryProductDetails] call.
Future launchPriceChangeConfirmationFlow(
{required String sku}) {
return _billingClient.launchPriceChangeConfirmationFlow(sku: sku);
}

Can also confirm you are on the latest version of the in_app_purchase package: in_app_purchase: ^3.1.7?

Hi @OlegNezv. Thanks for your patience and for attaching this. I can see the issue you described now on my end. I'll label this as a proposal to change the documentation since a change has been made. Thank you!

@mvanbeusekom This looks like missed docs updates from the recent API migration.

Fixed in https://github.com/flutter/packages/pull/4643

Please update in changelog on flutter.dev on next release

Maybe please update in changelog with more info regarding this on flutter.dev on next release

I'm not sure what page on flutter.dev you are referring to; please file a new issue with details if there are outdated pages on the Flutter website.

I don't understand why such minimal information is always passed

I'm not sure what you are referring to when you say "always passed". The breaking change was described in the CHANGELOG when it was made, and the README update has details about the flow. If there's something that still hasn't been updated we'll need specific details.


[Snyk] Upgrade html-snapshots from 2.16.0 to 2.17.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade html-snapshots from 2.16.0 to 2.17.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 4 days ago, on 2024-02-05.

Release notes

Package name: html-snapshots

2.17.0 -<EMAIL_ADDRESS>

2.16.0 -<EMAIL_ADDRESS>

    from html-snapshots GitHub release notes

Commit messages

Package name: html-snapshots

2fa0371 Merge pull request #628 from localnerve/rc-2.17.0
33c917f @ 2.17.0 -<EMAIL_ADDRESS>9871dac Merge pull request #627 from localnerve/snyk-upgrade-69e1862e8fdf7b29a5cc0359632008f6
1b97ba0 fix: upgrade html-snapshots from 2.15.0 to 2.16.0
4a3f379 Merge pull request #626 from localnerve/snyk-upgrade-42b6aab7a009e4fe7e629da545ca6fd2
b54f597 fix: upgrade html-snapshots from 2.15.0 to 2.16.0
b98c226 Merge pull request #625 from localnerve/snyk-upgrade-32c147e4cde16b926fe5eb7f1bac67ce
5247c69 fix: upgrade html-snapshots from 2.15.0 to 2.16.0
2668630 Merge pull request #623 from localnerve/snyk-upgrade-dfe038349b8dd34fefaf01358e43be8b
e9c5731 fix: upgrade html-snapshots from 2.15.0 to 2.16.0
74a5b27 Merge pull request #622 from localnerve/snyk-upgrade-46075fdaea563c9699729d8c40ad7916
f7fd0c1 fix: upgrade html-snapshots from 2.15.0 to 2.16.0
79a1616 Merge pull request #621 from localnerve/snyk-upgrade-8387b982e6df0844b37cbea138029a09
06cd91c fix: upgrade html-snapshots from 2.14.0 to 2.15.0

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

coverage: 99.245%. remained the same
when pulling 03b9e812995cdbf6c487564269a14572411b88e1 on snyk-upgrade-a5b8382c677c347380c6e89ab2e8b4b5
into 66fb3fb3b694812b6416277e2dd8b65ed8d58d83 on master.


[Windows] Allow minimize or close to tray

I did a search for similar issues and it seems something like this used to exist, but I guess it went away?
I want YakYak running 24/7 but not cluttering up my taskbar and instead minimized to my tray, notifying me when I receive new messages.
It seems possible to do this currently but the flow is pretty clunky. I'm forced to right-click the tray icon and select "Toggle minimize to tray". Or am I missing something?
In an ideal world (at least mine), minimize would minimize to the taskbar, close (X) would minimize to the tray, and you would exit the application by right-clicking the tray icon. If the tray icon feature was disabled, then close (X) would exit.
v1.3.0 on Windows 10

I concur that this should be how it works. In addition, double-clicking the tray icon should open the Yakyak window,
Whatsie, the Electron-based Whatapp client, works exactly like that, so it should be possible.

Yep, totally possible. The company I work for has an Electron-based app that works similarly as well: https://github.com/Automattic/wp-desktop

+1  - should be this way for Linux (LXDE) as well

On 1.3.3 (Win10), Even when "close to tray" is checked in the tray options, clicking X still exits the application.

the close to tray issue is solved in #397 :)

I am on Windows 10 and can confirm that it neither minimizes or closes to tray.

This should be solved in the next version, #397 pull request has not been merged yet and has this fix.
If you want to test it before hand try to find a file called resources/app/main.js and replace close with unload:
return mainWindow.on('close', function(ev) {

with
return mainWindow.on('unload', function(ev) {

run yakyak and it should work (credit to @nerijus )

see #451 for the beta release. It should solve the problem


Do not overwrite default warning filter or formatter

Overview
Closes #3287

Removes modifications of the default warning formatter
Removes the global supression of numba deprecation warnings

Discussion
As per the issue above, the unexpected import-time side-effects can be surprising and cause nefarious bugs. As a general rule, importable libraries should avoid modifying or monkeypatching other libraries.
One consequence of this change is that the warnings produced by SHAP will be the default verbose formatting, rather than the prettier "message-only" format:

However, whilst I admittedly do prefer the concise formatting, I don't think this preference for formatting has anything to do with shap specifically, so it should be out-of-scope for the shap package to fiddle with generic python behaviour like this. Users are still able to modify the handling of warnings as they see fit.

As an alternative, we could preserve the current warning formatting with a function like this that tracks and restores the original global state. This would involve replacing all instances of warnings.warn.
def warn(msg: str) -> None:
    """Raise a UserWarning with a custom formatter that just shows the message"""
    # Temporarily override the warning formatter, then restore previous state
    previous_formatter = warnings.formatwarning
    warnings.formatwarning = lambda msg, *args, **kwargs: str(msg) + '\n'
    warnings.warn(msg)
    warnings.formatwarning = previous_formatter

I'm actually inclined to just leave the default warning formatter in place for simplicity. Open to suggestions. Whatever we do, it seems like we should not permanently override the global state.

Just to clarify, @CloseChoice do you have any preferences regarding removing the formatting tweak entirely vs implementing the "pretty_warn" func?

I think removing it entirely as you did is totally fine. I wouldn't spend time to find a solution that keeps the old formatting AND doesn't overwrite the global warnings. If you have a quick solution please go ahead, but I would not spend even 15 minutes on that, tbh. If the formatting disturbs us in any way we can still open another issue and find a solution for that.


Script for Compleo Solo Advance not available

I would like to use evcc, but the script for my wall box Compleo Solo Advanced, which I need to use as it is paid by employer, is not available. Would be nice if it could be added.
Thanks a lot in advance.

Is there any api documentation, like for modbus?

I would like to use evcc, but the script for my wall box Compleo Solo Advanced

Did you try standard OCPP ?


Update - Event add frontend done

Completed the frontend for adding events.
User can pick a date and add event title, location, and timetable. Event description section is ignored as usually calendar is not used to announce event details.

Apparently, client has requested to change color scheme. I would update the scheme in the next sprint.


Dim-only bulbs

What is happening?
Hi, i have a few Hue white ambiance lights and a zigbee dimmer switch (Ecodim) that controls regular lights.
With the Hue app/bridge, setting a scene would also adjust the brightness for the dimmer, the scene presets plug-in doesn’t do anything when these bulbs are selected when setting a scene. Is it possible to change this?
What should be happening?
When I set a scene to “rest” or “relax”, the contrast of the selected dimmer switch should be updated to match te scene. This would allow me to set a scene for the whole room.
Reflection 1
Balanced View
Is this issue tracker the right place for this topic?
I can manually change the brightness of the dimmer with home assistant.
Please enter the scene_preset version you're using
1.2.2
Please enter the Home Assistant version you're using
2024.2.2
Details about your setup
ECO-DIM.05 connected to Zigbee controller SLZB-06, Zigbee2MQTT.
Reflection 2
HA and Zigbee2MQTT both running in Docker on a Synology NAS
Time and Effort

[X] Yes, I did my homework.
[X] I understand that my issue will be deleted with no further comment if I did not do my homework.

I initially decided against supporting that, because I did not think that it would be of much use, however since it is useful to you, it is now possible with version 1.2.3

That’s a lightning fast fix :-)
Thanks a lot!


Install failed

I am trying to install this but when I am in Xcode -> Build Phases I can't find libStreamingKit.a.
I followed the instructions for "Mostly Automatic" install. Can you explain what you mean by "add Pods.xcodeproj"? Am I supposed to just make a folder called Pods.xcodeproj?

I went through the manual install process on iOS and was able to get it working. I may submit a PR to update the directions since Pods.xcodeproj exists under the Pods folder not the ios folder.

Yes I'm sorry this is not really the best install process. I wanted to keep the pod dependency to avoid versionning it but I didn't find a nice way to do this, if you have any idea I'll take it !

I'm unable to successfully build this for android.  I've followed the steps in the read.me to the letter.  Getting the following error.
/Users/stockley/websites/straightline/mobile/Straightline/android/app/src/main/java/com/straightline/MainApplication.java:27: error: cannot find symbol
          new ReactNativeAudioStreamingPackage(MainActivity.class)
              ^
  symbol: class ReactNativeAudioStreamingPackage
1 error
:app:compileDebugJavaWithJavac FAILED

@magicspon Did you also add the line import com.reactlibrary.ReactNativeAudioStreamingPackage; at the top of MainApplication.java ?

@tlenclos I shall check and let you know shortly... thanks


[ISSUE #708] Fix the issue that PushConsumer reentrant pop doesn't work as expected

Which Issue(s) This PR Fixes

Fixes #708
Brief Description

How Did You Test This Change?

Codecov Report
Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 47.18%. Comparing base (3036b04) to head (b13e272).

Files
Patch %
Lines

...mq/client/java/impl/consumer/ProcessQueueImpl.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #709      +/-   ##
============================================
- Coverage     55.82%   47.18%   -8.64%
  Complexity      651      651
============================================
  Files           259      207      -52
  Lines         15856    12385    -3471
  Branches       4089     3737     -352
============================================
- Hits           8851     5844    -3007
+ Misses         6717     6253     -464
  Partials        288      288

Flag
Coverage Δ

java
61.68% <0.00%> (ø)

nodejs
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

May I know when this fix will be released? This issue often occurs during network fluctuations.


Documentation on ReadTheDocs returns 404

Is it only on my side?

All of the documentation has been merged into the README here along with comprehensively commented examples in the res://tests/examples folder.

Thanks for the precision, I was confused since the links menu of the WAT GUI was still taking me to the old ReadTheDocs.
I must have missed an update. :)
Is there a specific reason why the ReadTheDocs was dropped in place of a big README, other than it being easier to maintain?

Oh, also the link from the github wiki  is still redirecting to the removed ReadTheDocs.

Thanks for the precision, I was confused since the links menu of the WAT GUI was still taking me to the old ReadTheDocs.
I must have missed an update. :)
Is there a specific reason why the ReadTheDocs was dropped in place of a big README, other than it being easier to maintain?

The links were removed in the new livewire release (or at least should have been).
Beyond being just easier to maintain, I felt that I kept writing a lot for what is really a simple code example and they were just too big for the amount of info I was trying to convey.


Support filtering on the arguments of a marker

Thanks @mshriver for reporting this.
Some teams have a need for filtering on not only the name of a marker but also the argument. E.g.
        "markers": [
          {
            "args": [],
            "kwargs": {},
            "name": "tier2"
          },
          {
            "args": [
              "Critical"
            ],
            "kwargs": {},
            "name": "importance"
          },
          {
            "args": [
              "ContentViews"
            ],
            "kwargs": {},
            "name": "component"
          },
          {
            "args": [
              "High"
            ],
            "kwargs": {},
            "name": "importance"
          }
        ]

It would be nice if Ibutsu could handle these more complicated markers.

Though this functionality may still be advantageous, its no longer necessary for my team's use of Ibutsu.
We've started recording test case metadata in junit's user_properties field, which is stored as metadata by ibutsu under metadata.user_properties. This gives my team the filtering we were looking for.


Introduce collection.batch for uploading to a single collection in batches

This PR introduces collection.batch for uploading data to a single collection using the client's background batching algorithm. It splits out the ThreadPoolExecutor dependency from the original batch class so that it can be shared between all instances of the batching algorithm in the interest of minimising thread-count for users that spin up multiple concurrent batching algorithms

Closing in favour of https://github.com/weaviate/weaviate-python-client/pull/655


Project status

Hi all.
Tell me, please, what is the status of project after shutting down Tourbine Labs and the transition of the team to Slack. Is it possible to use rotor in production or is it better to look for an analog?

Per the README, it is unsupported.
Depending on how recent a build of envoy you’re using, it may or may not work.
I can’t recommend using it in production for a new deployment.


no internet when connected

when i get connected, there is no internet connection. i have tried restarting the server and it doesnt work

make sure you use the Preshared key.
for me it is works fine

same problem, have you installed wireguard on a device in your local network to which you're trying to connect?

I guess it's an issue with the ipv6 address. I deleted the ipv6 configured for the [interface] address on the client and gave it a try。
I successfully switched to another installation script, https://github.com/hwdsl2/wireguard-install。

Same issue.

I'll try with the https://github.com/hwdsl2/wireguard-install and if I find a solution I'll tell you.

I've tried the other script and the problem persists.

I've tried the other script and the problem persists.(I tested all script)

https://www.tecmint.com/set-permanent-dns-nameservers-in-ubuntu-debian/
Set a Permanent DNS


docs: expand developer documentation

Add more documentation for local development, testing, linting and formatting

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.905%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2792

Relevant Lines:
4034

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 67.871%

Files with Coverage Reduction
New Missed Lines
%

conda_smithy/configure_feedstock.py
2
82.98%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
2791

Relevant Lines:
4034

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 67.871%

Files with Coverage Reduction
New Missed Lines
%

conda_smithy/configure_feedstock.py
2
82.98%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
2791

Relevant Lines:
4034

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.871%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2791

Relevant Lines:
4034

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
277 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.3%) to 68.162%

Files with Coverage Reduction
New Missed Lines
%

conda_smithy/utils.py
12
81.31%

conda_smithy/cli.py
118
53.62%

conda_smithy/configure_feedstock.py
147
83.14%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
2835

Relevant Lines:
4081

💛  - Coveralls


Imporve single bundle test

Apply missed suggestion from review comments in #34

Apply missed suggestion from review comments in #34


Microsoft Python Language Server consumes ~100% of CPU

Microsoft.Python.Language.Server consumes around 100% of my CPU running VSCode on MacOS. I can already hear the sound of CPU fans running at full speed just like when I'm running Android Emulator. I only have three python files opened in one workspace. Before this happened, there was a status in the status bar telling that Python Language Server is being updated. Please tell me how can I provide more details to resolve this issue. Thanks.
Attached below is the screenshot of Activity Monitor.

Please file this against the Python extension: https://github.com/Microsoft/vscode-python


refactor grid so action buttons collapse correctly

@discodavey I think we should still try to remove the right margin/padding on the alternate buttons (i.e. Cite) so that we can make the aside column narrower still (down to 133px). That would mean it matches the designs and then the text at the top of the aside will fit a bit better. Happy to pair on it if needed.

@chris-huggins I feel by removing margins on the alternate buttons that if we introduce another button that was slightly shorter or longer it could cause the buttons to butt up to each other without the margin.
All depends if we are looking at future proofing this. If like other components that we have built where future proofing is not being bothered with then I am happy to do this,

@chris-huggins I have reduced the max-width and alternated the margin-right on the action buttons. Will let you know when it's ready to review

@discodavey Thanks. I was going to say we should be fine with buttons. I only anticipate us adding a fourth after the "Share" button for "Follow". Any other changes will need some consideration so we can address any problems caused by margins then. I think it's more important to ensure the text fits correctly than worrying about changes to buttons.


Separate Demand Flexibility Data Input

:rocket:

[x] Is your feature request essential for your project?

Describe the workflow you want to enable
I wish to split the input file for demand flexibility into two files that assign different allowance for shifting the demand upward or downward. This will enable a more flexible behaviour of flexible loads by allowing them to have different extend of flexibility in increasing or curtailing.
Describe your proposed implementation
The script read.jl should take two csv files: 'demand_flexibility_up.csv' and 'demand_flexibility_dn.csv' instead of a single file 'demand_flexibility.csv'
The class "demand_flexibility" should have two dataframes to hold the data in the input csvs
The upper bound of the decision variables in the opt model 'load_shift_up' and 'load_shift_dn' should be different according to the two input files.

I think this functionality sounds good and the proposed implementation seems straightforward. There are some other things that would also need to be implemented:

We would need to update loop.jl as well so that the upper bounds on load_shift_up and load_shift_dn are updated for intervals i > 2.
Within both model.jl and loop.jl, we would need to call two instances of _make_bus_demand_flexibility_amount (right now we only have one) for bus_flex_amt_up and bus_flex_amt_dn.

Based on Tuesday's discussion, part of the interest in splitting up the demand flexibility parameter (other than for just providing more user flexibility) seemed to be the possibility of having demand that could only be curtailed (i.e., "sheddable demand"). If we want this to truly be explored, we should revisit issue #125, which discusses including a representation of sheddable demand. With the constraints that are established for the current implementation (i.e., "shiftable demand"), the functionality of sheddable demand would not be able to be realized.

I think this functionality sounds good and the proposed implementation seems straightforward. There are some other things that would also need to be implemented:

We would need to update loop.jl as well so that the upper bounds on load_shift_up and load_shift_dn are updated for intervals i > 2.
Within both model.jl and loop.jl, we would need to call two instances of _make_bus_demand_flexibility_amount (right now we only have one) for bus_flex_amt_up and bus_flex_amt_dn.

Based on Tuesday's discussion, part of the interest in splitting up the demand flexibility parameter (other than for just providing more user flexibility) seemed to be the possibility of having demand that could only be curtailed (i.e., "sheddable demand"). If we want this to truly be explored, we should revisit issue #125, which discusses including a representation of sheddable demand. With the constraints that are established for the current implementation (i.e., "shiftable demand"), the functionality of sheddable demand would not be able to be realized.

Agree. It is tricky to handle load balance during a given time window in the mean time dealing with sheddable demand.


Tensor Parallelism Support for AffineQuantizedTensor

Recently we landed https://github.com/pytorch/ao/pull/939 to support tensor parallelism for int8 weight only quantization, another example: https://github.com/pytorch/ao/pull/785
now we can support tensor parallelism for other types of quantization as well.

[x] float8 weight only @jainapurva - #1003
[x] float8 dynamic activation @jainapurva - #1078
[ ] uintx weight only @melvinebenezer
[ ] int4 weight only quant - @jerryzh168
[ ] int8 dynamic act + int8 weight - not tested, add your name here
[ ] fpx - add your name here

Steps
1. Create test
Since we don't have many tests today, we can optimize for readability for now, so we can copy paste the test cases to a https://github.com/pytorch/ao/blob/main/test/dtypes/test_affine_quantized_tensor_parallel.py instead of inheriting from these test cases
For new tests you can follow https://github.com/pytorch/ao/blob/c87cc9b7286a46e9dfc076fa2417eb9b64ccc807/test/dtypes/test_affine_quantized_tensor_parallel.py#L133-L153 to create your own test case
2. Run the test
python test/dtypes/test_affine_quantized_tensor_parallel.py
3. Add support for missing ops until test passes
We'd expect people to add some slicing ops etc. to the corresponding TensorImpl tensor subclass

@jerryzh168 should'nt this Issue be kept open until other points in the description are fixed ?
I have a WIP #1026

Hi @jerryzh168, I had two questions

What does fpx mean here exactly?
For int8 dynamic act + int8 weight  only writing tests is left?

@p4arth

fpx is referring to this API: https://github.com/pytorch/ao/blob/7aaf0ff979b8e254b0c55cb8353ce5a73c7c257f/torchao/quantization/quant_api.py#L97, docs: https://github.com/pytorch/ao/tree/main/torchao/dtypes/floatx

For int8 dynamic act + int8 weight  only writing tests is left? - I think so, but will need to make sure the test passes, tests will be similar to https://github.com/pytorch/ao/blob/7aaf0ff979b8e254b0c55cb8353ce5a73c7c257f/test/dtypes/test_affine_quantized_tensor_parallel.py#L133


Show links from tokens to holders

It can be fetched from the Blockscout /tokens/{address_hash}/holders endpoint.

Does it really need to be backend indexed or can it be done from the frontend only ?


evaluate this lib

it's a fork of agl/ed25519, which was later merged into golangs x/crypto https://godoc.org/golang.org/x/crypto/ed25519
the repos have diverged somewhat
do we even need the fork ?

Development happens here: https://github.com/golang/crypto/tree/master/ed25519

do we even need the fork ?

As long as we don't use any functionality from the internal package (if, I don't see where we do), we should rather use the maintained golang/crypto version instead. There were some changes recently that adapt to a published RFC to prevent signature malleability.
(Also this might be related https://github.com/dedis/kyber/issues/117)

I had a slightly closer look at go-crypto this weekend. There, we do in fact use two methods (extra25519.PublicKeyToCurve25519) which are not part of x/crypto:
https://github.com/tendermint/go-crypto/blob/915416979bf70efa4bcbf1c6cd5d64c5fff9fc19/pub_key.go#L69-L76
and
https://github.com/tendermint/go-crypto/blob/7fb3f704b38492fb24cc6f807baee524ef7f4f8d/priv_key.go#L63-L68
@ebuchman noticed this here already.


[Crash] "indexed pool nonces inconsistent when adding from backtrack"

This one happened during or shortly after a Golden Hour.
(monitor.ml.Error
  (Failure
     "indexed pool nonces inconsistent when adding from backtrack. Trying to add ((payload\
    \n  ((common\
    \n    ((fee 10) (nonce 64)\
    \n     (memo\
    \n      \"\\001\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\000\\0
...

The full stack trace is a whopping 10,000+ (!) Lines
https://gist.github.com/yourbuddyconner/71d9c19aad39ca2f1a953632e5525547

likely same or related to #3576 or #3644

Received these Fatal errors a couple more times since.
(monitor.ml.Error
  (Failure
    "Invalid user command! Error was: Nonce in account 162 different from nonce in transaction 161, command was: $user_command")
  ("Raised at file \"stdlib.ml\", line 33, characters 17-33"
    "Called from file \"src/sequence.ml\", line 128, characters 27-34"
...

CC @deepthiskumar

Is this still happening? Closing until we see this again.


Won't work with https://www.youtube.com/watch?v=D8t8A8E_Tqc

Throws 403 forbidden error.

Duplicate of #6451. Remove cache with youtube-dl --rm-cache-dir.


highlighting for empty positions in visual block mode & field/templating

Is your feature request about something that is currently impossible or hard to do? Please describe the problem.
I am a visual person, and shapes and outlines help me format documents. Visual block mode does not highlight empty positions. It would be helpful for me to see the boundaries of the text as an object better, especially when I try to be neat and keep things in multiple rows on the same line.
Describe the solution you'd like
Implement some mechanism to highlight empty positions in visual block mode, maybe through a new highlight type for empty positions.
Describe alternatives you've considered
I have just thought of this and have not looked deeply into solutions.
Additional context
You could maybe block out a document with fields or something even, using a new empty position type. Maybe that would be completely unrelated. Kind of like making the entire document a field.
Hey in fact what about implementing some type of templating through drawing or erasing empty/non-empty positions? you could just copy paste the templates containing fields using visual block mode too.

can you please share a screenshot of what exactly you mean? What exactly you mean with empty positions?
I think it may work with either some custom colorscheme or with some listchars values and :set list set.

I'm guessing they mean this: https://github.com/0xAdk/full_visual_line.nvim

I use that plugin and it works well but sometimes can lag on larger files. A built-in option would be nice

yeah initially what i was thinking is what full visual line plugin does, but just around the boundary of the item, and which I could move with key presses in directions like a game sort of. sort of like 'the world's hardest game' flash game with the movement but it could move within empty positions in the document (leftward empty positions would need to be converted to spaces when placed, rightward empty spaces contained in the visual block don't necessarily need to), but then I also started to think, what if the visual block I selected coud be remembered as a field in some way to where the edges of that visual selection would act as a end-of-line so that I could continue just writing on multiple lines contained in that visual block selection as if it were a field? if this were done, you could use visual block selection to basically also create a bounding box (maybe as an option) for text input confined to that area. Now that I think about it this probably has a plugin of some type, but then if you wanted to type in the text area left of the multi line visual block you allocated in line with your existing document, you still face the issue with breaking the in line allocated block of text to the right when you return. so some type of bounding box behavior would fix this, a bounding box that prevents new lines from being added, but still formats correct indentation as if the document just shrank to the size and position of the bounding box.
but then i started thinking what if there could be field creation mechanism for vim where you could quickly define a layout for a document consisting of fields of this nature sort of like laying out a newspaper, or if you've ever cut words out of a magazine to write a fake ransome note in school, or for some poetry or english creative writing project. the words sort of lign up as like a left to right game of tetris. if empty positions in a document could impede visual block boundaries maybe using some new position type (not necessarily visible) as a way to emulate a behavior of separate fields and some logical restriction or virtual shrinking of the document to bounding boxes, the bounding box could be a "drawable" field type position where one could easily block out a visual representation of where you want to place your code or info and vim would follow that.
maybe you couldn't do multiple fields and save them, but you could have one active drawn field defining the boundaries of a 'virtual' document (bounding box custom shape) that would respect the new, impeding, empty position type.
theres various things that would have to be worked out how it would handle text that was inserted beyond the confines of the bounding box, but it would definitely extend the visual block mode a bit more even with 1 bounding box field like this. hope I didn't write all this and this actually already exists.

It sounds like my visual line mode suggestion differs from what @LaptopDev wants. I still think https://github.com/0xAdk/full_visual_line.nvim would be a great addition to core Vim so in the interest of keeping these two requests separate, I'll make another GitHub issue for that.

It sounds like my visual line mode suggestion differs from what @LaptopDev wants. I still think https://github.com/0xAdk/full_visual_line.nvim would be a great addition to core Vim so in the interest of keeping these two requests separate, I'll make another GitHub issue for that.

hi ColinKennedy
Can I use this full_visual_line.nvim in my vim directly? I don't want to switch nvim now.

full_visual_line.nvim uses Neovim's extmarks feature. I don't think Vim has an equivalent that would work in this case but I could be wrong.

full_visual_line.nvim uses Neovim's extmarks feature. I don't think Vim has an equivalent that would work in this case but I could be wrong.

Ok, thanks for your replying.


Excited about grav

just used grav today for the first time.
& i am very excited about Grav & i want to translate it into Arabic if you allow me to.
can i ?

The core can be translated here: https://crowdin.com/project/grav-core
The admin plugin can be translated here: https://crowdin.com/project/grav-admin
Just join crowdin and start translating!
Thanks!
On March 23, 2016 at 5:15:53 AM, Ahmed Badawy<EMAIL_ADDRESS>wrote:
just used grav today for the first time.
& i am very excited about Grav & i want to translate it into Arabic if you allow me to.
can i ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub


Full URL HTTP transport support

Please tell us about your environment:

[ ] winston@3

What is the problem?
Could not set up the default HTTP transport to use only a full url.
For example, if I want to send a request to https://test.io/test_endpoint, I could not find any option to do so, because the default transport expects to receive port, host and more parameters which are not relevant in this case.
Maybe I have missed the option somewhere, if so, I'll be glad to be pointed to the right place.
If the feature really does not exist, I'd love to work on it

@Hyaxia I have a similar issue, I'm doing it like this :
new winston.transports.Http({
        host: "https://test.com/abc",
        headers: {
          'Authorization': <Auth_token>,
          'api_key': <Api_key>,
          'Content-type': "application/json",
        },
        method: "POST",
        ssl: true,
})

Did you find a solution? If so, how are you doing this?

@Hyaxia I have a similar issue, I'm doing it like this :
new winston.transports.Http({
        host: "https://test.com/abc",
        headers: {
          'Authorization': <Auth_token>,
          'api_key': <Api_key>,
          'Content-type': "application/json",
        },
        method: "POST",
        ssl: true,
})

Did you find a solution? If so, how are you doing this?

And does it work?
I have written a transport myself since I could not find any other option.
I guess you can also change the default http transport that comes with winston but its a little harder i think.
If you would like I can post the code for the transport here, but it will take some time, its in a private network.

It did not. And I'm not sure if this is the right way to go about. Would be very grateful if you could show me how you have worked this out.
Thanks

It did not. And I'm not sure if this is the right way to go about. Would be very grateful if you could show me how you have worked this out.
Thanks

I'll try to remember to do it in the next week, ping me if by the end of next week I haven't responded.

For the time being, I have creating this repo that has a custom http transport to solve the problems I had...


[WIP] Sentence similarity for Spacy

This PR closes #241 by adding sentence similarity for spacy

Hi @devpramod! Thanks for this! Excited to see this out there!
A couple of quick things

Could you make sure quality passes please? (make quality)
Could you also add a GA workflow for spaCy by doing a copy of https://github.com/huggingface/huggingface_hub/blob/main/.github/workflows/python-api-espnet.yaml.

Hi @osanseviero,
I did do make quality, I didn't see any changes though, as you can see below.
black --check api_inference_community tests All done! ✨ 🍰 ✨ 11 files would be left unchanged. isort --check-only api_inference_community tests flake8 api_inference_community tests

@devpramod by any chance did you add the wrong file? I see you added a espnet GA workflow, but not for spacy (the new file should be named python-api-spacy.yaml, and all uses of espnet should be renamed to spacy in the yaml file).

@osanseviero I changed the GA Workflow for SpaCy. I see that make style and make quality are failing in the tests. When I run the commands It doesn't seem to be reformatting. I'm looking into the issue.

Related failing style/quality, you might have a different version.
Please make sure you have installed the correct versions by doing pip install -e .[quality] and retrying make quality

Hi @osanseviero, I have fixed the code quality issue and the test is now passing.


V0.1.5 dev

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4a1d2d5) 100.00% compared to head (8bbeca5) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##            master       #13   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          397       453   +56
=========================================
+ Hits           397       453   +56

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


im getting a make error with persistence.hpp

Hi,
I get a compile error with persistence.hpp using opencv3 with cmake
cmake -D WITH_EIGEN=OFF -D WITH_QT=ON -D CMAKE_BUILD_TYPE=RELEASE -D CMAKE_INSTALL_PREFIX=/usr/local .
I noticed someone had reported the problem before:
https://github.com/Itseez/opencv/issues/5664
In my case I get :
usr/local/include/opencv2/core/persistence.hpp:1107:27:   required from 'cv::FileNodeIterator& cv::operator>>(cv::FileNodeIterator&, std::vector<_RealType>&) [with _Tp = std::basic_string<char>]'
/usr/local/include/opencv2/core/persistence.hpp:1130:8:   required from 'void cv::operator>>(const cv::FileNode&, std::vector<_RealType>&) [with _Tp = std::basic_string<char>]'
../Control/InputGenerator.cpp:62:27:   required from here
/usr/local/include/opencv2/core/persistence.hpp:793:41: error: no matching function for call to 'read(cv::FileNode, __gnu_cxx::__alloc_traits<std::allocator<std::basic_string<char> > >::value_type&, std::basic_string<char>)'
                 read(**it, vec[i], _Tp());

through an operator >> call you end up calling
read( const FileNode& node, std::vector<_Tp>& vec, const std::vector<_Tp>& default_value = std::vector<_Tp>() )

where if  _Tp is std:string,  then a required version of a read function such like the one existing for uchar in persistence.hpp:
static inline
void read(const FileNode& node, uchar& value, uchar default_value)

does not exists for  function with parameters of std:string type like it exists for a few types like uchar above.
Yet if you look in opencv 2.4.12
this function existed in core/operations.hpp and it had an std:string overload.
Perhaps this is  a bug, where an overload of read() function for std:string has been missed?
Thanks,
Kostas

Please add some sample code to reproduce this problem

yes, indeed.  I have reverted to using the old library instead of 3.0.
Below I extracted the lines below, which used to give the compilation error:
 const std::string path = 'Some Path to a yml file containing key-value pairs';

 std::vector<std::string> imgPaths;

  cv::FileStorage fs = cv::FileStorage(path, cv::FileStorage::READ, "UTF-8");

//The line below hit the missing read() function
  fs["imgNames"] >> imgPaths;

'''

Use cv::String instead of std::string (there are some issues with std::string, so public interfaces migrated to cv::String):
std::vector<cv::String> imgPaths;

OK, that worked thank you.
Had to convert the code to vectorcv::String and solved the problem - additionally had to link against the opencv_imgcodecs for imread to work now.
Cheers,
K.


[New game]: Coloron Game

🎮 Game Request
Coloron is a fun game where you must pay attention to the color of the ball since the color of the ball changes after every jump!
Point down the features
Pay attention to the color of the ball since the color of the ball changes after every jump.
1 Tap = Red
2 Tap = Yellow
3 Tap = Purple
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @sabhisharma-ise !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @sabhisharma-ise!
We are already having a similar game request in #1817 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.

Hey @sabhisharma-ise ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Bullrich/delete drip endpoint

Removed the endpoint /drip in replacement of the new endpoints /drip/bot and /drip/web
Closes #217

Ink docs already deployed with a new endpoint?

Yes! I tested it this morning
https://github.com/paritytech/ink-docs/commit/130a0d4d125623f1c60c0bb41d78d0c6c4b3b9b0


Different lengths of labels on X axis

Describe the bug
The graph uses non-uniform long captions, so the size may be smaller due to wrong titles.
To Reproduce
Steps to reproduce the behavior:

Do many scans in EMBArk for different FWs.
Check the OS distribution's graph
See error

Expected behavior
Uniform long labels.
Screenshots

Desktop (please complete the following information):

OS: Ubuntu Server 22.04.1

Not so nice:

As the correct info is shown we need to think about the way to handle this issue.
Probably we could just strip the long version details (if the short one is part of the long one) for statistics?

I think the default label like "Linux / vX.YY.ZZZ" would be OK, the 2nd slash and the remaining parts are unnecessary.

Not that easy, as there could be also some other kernel versions e.g. if the firmware update ships multiple kernel versions we can't drop one of them.
On EMBA side I think we need to split the kernel versions and report for every kernel version a separate entry in the f50.csv. On EMBArk side we can then check for the main kernel version and use only this for the dashboard. What do you think @BenediktMKuehne ?

On EMBA side I think we need to split the kernel versions and report for every kernel version a separate entry in the f50.csv

I think that's unnecessary.
The easiest/best would be to just change the rendering filters in EMBArk.

Sure, I will, thank you.
Will come back the results end of this day.

I went to ~/embark folder and used your commands and I started the service. The scan was OK, I followed the logs in putty:

The detailed view is also OK:

but the main dashboard is emtpy now:

This was a clean system, I installed it a few days ago. I didn't execute the emba_updater before the git commands.
Do you think it is worth to make a brand new installation with testPR/95-different-lengths-of-labels-on-x-axis? I mean
$ git clone https://github.com/BenediktMKuehne/embark.git -b 95-different-lengths-of-labels-on-x-axis
?

No, It shouldn't make a difference
I probably made a mistake 😅

@torabi12
Now it should work

I tested with the same kkeps.bin file, which has a long kernel version but sorry I have the same empty dashboard at the end. Another issue is that the vendor and label creation does not work for me:

the same error for the label creation also.
I followed the scan from terminal and it was OK:

@torabi12 now everything should work as expected

It works, but the linux label is long on the dashboard:

and on the details page:

Adding vendor and label were OK also.

I may not need to bother with this, but since I've run quite a few tests with partial module selection where S24 was not running, each of these cases has a separate anonymous graph displayed on the dashboard. I think that when running a partial scan you should not create a separate column, which might make the statistics worse, but might give a more accurate result.

I'll have to test that particular firmware to find out where that graph comes from.
And adjust the truncation.

I did a new installation of EMBArk today with a basic scan and the details page is OK:

but the dashboard left empty after the scan:

The FW counter and the CPU/MEM graph are working:

but the others are empty for me.


update docs on customizing the mock link

:tada: This issue has been resolved in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


shift issue

Describe your bug here.
the space bar doesnt work when shift is held in dialogue editor
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

Haxe issue.


Update platform db search path when --lib is passed to afu_synth_setup

Now that platform_db libraries may be in release hw/lib trees, afu_platform_config
must be able to find the library.  Normally it uses $OPAE_PLATFORM_ROOT.  When
the root is not defined but --lib is passed to afu_synth_setup the db search
path must be updated.

Does that mean that the 'platform' folder could be installed in a folder of OPAE such as /usr/lib/platform or /var/lib/platform ?

The main one already is installed with OPAE in share/opae/platform. These commits add automatic searches for databases stored with board releases, in addition to the search of the OPAE platform tree.

The board release doesn't have OPAE in it, right?

I'm not sure how to answer that. The board release ships with OPAE RPMs, but OPAE can be installed from source too. The board release depends on some installation of OPAE. The two are separate.


log in form validation

https://code.sololearn.com/WjMn6A8mB0bR/?ref=app

@Danmirz18 Can you be more specific about your issue? This repository's issue forum deals with azure login action.

@Danmirz18 Closing this issue as the problem statement seems unclear. Feel free to reopen if the problem is occurring while using azure/login action.


Fix typo

*Issue #, if available: N/A
*Description of changes: Fix typo in the configuration command.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Wow! Thanks!


Updating AuTest to use Proxy Verifier v2.4.2

The v2.4.2 version of Proxy Verifier includes logging updates for SETTINGS and WINDOW_UPDATE frames. This will be useful for HTTP/2 session and stream window testing.

Cherry-picked to v9.2.x


intake alliance color selection utilizing screen_selector

Added to screen/selector:
get_auton_type - returns the auton type as a short (0 = Blue, 1 = Red, -1 = Skills)
Added to subsystems/intake:
set_alliance - sets private field alliance_color based on auton_type input. (Blue = false, Red = true, Skills = no change)
Added to main:
passes auton_type from screen selector into the intake subsystem inside of initialize function

Considering that this code is relevant to the code from the intake, would it make more sense to have the logic in the intake files?
This would remove somewhat irrelevant code from the selector, and put it with the intake, where the color sensor is managed, making more sense in terms of state.

Pushed changes:
screen/selector.cpp and selector.h are no longer involved.
get_auton_type was added to auton.h and auton.cpp.
Changes to main: auton_type is now passed into the intake subsystem inside of autonomous function.

Removed get_auton_type from auton.h and auton.cpp.
Removed parameter passing in for intake.set_alliance() in main.cpp
Gave intake header file access to auton header file.


give roundstart borgs names

About the PR
roundstart borgs get a name from the modified autoborg dataset
borgs made during the round are unaffected
Why / Balance
funny + they were unused + borgs should have names that arent just "cyborg" roundstart that sucks
Technical details
no
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
no
Changelog
:cl:

tweak: Roundstart borg jobs get funny names.

Honestly they should really use the character’s name but this works as an in-between solution

people that leave borg on low with a human char shouldnt use chars name, which is something i do

ok then don't do that, roundstart Borgs should have some way to get the name they want instead of being named "beepface IV" or "Cyborg"

come to think of it how does the game even replace urist mchands with the characters name...

We should do it like ss13 where theres a secondary naming option for whenever you become a borg

and mime and clown...

i would rather a different dataset is used, as this one has quite a few joke names that could be kinda weird as defaults.

yeah this dataset is for the malf AI's autoborger which has some funny names in it, i dont think this would be good as a roundstart name dataset

ok time to make malf :trollface:

@EmoGarbage404 using a new dataset with the malfier named removed

merg

the names in this are still pretty bad i think i'd rather just write a bunch from the ground up lol

too much effort when most of them are still funy

Closing because some of these names are pretty bad and the submitter doesn't want to improve them. Please re-open if you plan on improving these names.

then point out which ones are bad...

yes i will compile a list on discord for you you just need to join the discord joinjoin the discord join the discord join the discord

I think the only appropriate round-start borg names from this list are:

Bob
Joe
Beep
Beep II
Tour Guide-otron
Boombox
Taffy
Boop
Boop II
Buzz
Buzz II
Toaster
Head of Robots
EVA 1

yes i will compile a list on discord for you you just need to join the discord joinjoin the discord join the discord join the discord

no killer

@Partmedia reopen

Suggestions for borg names: (feel free to take any of these)

Megabyte
Scrap
Fake-Drone
Firewall
Guybot
Processing Device
Waste Disposal
Trashbin
Microwave
Annihilator
Tinker
Epsilon
Omega
Neon
AI-bot

I am unoriginal so most of these are taken from random generators or lists online.

Probably beyond the scope of this PR but I think it would be best if there two lists which are combined to create a randomized borg name. The first half consists of nouns and adjectives ("Boop", "Beep", "Toaster", etc), with the latter half consisting of "bot", "otron", "drone", etc. And then maybe slap a number on the end of it. So you'd end up with names like this:

Beepbot 200
Junkotron 500
Robustinator 9000

Specialized borgs would have their own list of nouns and adjectives, so a roundstart janiborg might have names like:

Trashbot 3000
Cleaninator 8000
Slipotron 5000

Mr. Robist
Mini-Ripley
Robotron
C-3-D2
R2-PO
Robby
Robobot
Sonny
Autobot
Makeshifter
Beebee
Wheetley
Twin
Keller

Sure for now customisation later.

Skeeboop
Mr. Roboto
Powernet
Beatbot
Deceptiborg
Buddy Bot
Empty Chassis
Le Borgue
Toaster 2.0
Borganic Lifeform
Stevie
Automaton
Beepers
Cute-Bot
Be-Nice the Borg


Shamindra macreset 0

too many changes. Guessing this wasn't meant to be a PR :)

Sorry @atomantic - I'd forked and made personal changes and pushed up accidentally. Thanks for being understanding :)


Resource Move Support not reflecting correctly any longer

[Enter feedback here]

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

ID: 23df5354-6833-a5da-0eb3-d860150c8b78
Version Independent ID: 682ec206-0fda-3e0b-5841-543860a87e34
Content: Move operation support by resource type - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/move-support-resources.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

Seems like the recent update of this article are displaying incorrect information for resource move Support. A number of resources are now listed as "No" for cross subscription move. An example is Microsoft.SQL/Servers and microsoft.alertsManagement/smartDetectorAlertRules. These were supported previously according to the article.

@mwgroenewald Thanks for the feedback.
Assigning @tfitzmac for visibility, review and document enhancement consideration as appropriate.

@mwgroenewald - thanks for this feedback. I am rolling back to the previous version of this article while I figure out why the data is wrong.

The fix should get published this morning.
#please-close


move hellopromise from openwhisk/catalog to openwhisk-catalog

hellopromise is recently added, it depends on greeting.

I think this is close with this https://github.com/openwhisk/openwhisk-catalog/blob/master/packages/samples/helloPromises/javascript/helloPromises.js


Tensorflow Required Dependancy For Saving Checkpoint?

Hi,
I tried following the example in https://github.com/google/flax/discussions/1876#discussioncomment-2143866 and ran into an ModuleNotFoundError. I was able to fix it by install tensorflow, but was wondering if tensorflow should be a required dependancy.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Flax, jax, jaxlib versions (obtain with pip show flax jax jaxlib:

Name: flax Version: 0.6.0
Name: jax Version: 0.3.17
Name: jaxlib Version: 0.3.15

Python version: python=3.8
GPU/TPU model and memory:  CPU

Problem you have encountered:
ModuleNotFoundError: No module named tensorflow
What you expected to happen:
checkpoints should be able to imported without throwing any errors
Logs, error messages, etc:
File "test.py", line 5, in <module>
    from flax.training import train_state, checkpoints
  File "/home/user/anaconda3/envs/jax-dep/lib/python3.8/site-packages/flax/training/checkpoints.py", line 36, in <module>
    from tensorflow.io import gfile  # pytype: disable=import-error
ModuleNotFoundError: No module named 'tensorflow'

Steps to reproduce:
import jax.numpy as jnp
import jax
import flax
import flax.linen as nn
from flax.training import train_state, checkpoints
import optax
import numpy as np

class Net(nn.Module):
        features: int

        @nn.compact
        def __call__(self, x):
            x = nn.Dense(self.features)(x)
            x = nn.Dense(self.features)(x)
            return x

model = Net(features=2)
params = model.init(jax.random.PRNGKey(0), jnp.ones((1, 2)))

tx = optax.adam(learning_rate=0.0001)
state = train_state.TrainState.create(apply_fn=model.apply, params=params, tx=tx)

CKPT_DIR = 'ckpts'
checkpoints.save_checkpoint(ckpt_dir=CKPT_DIR, target=state, step=1)
restored_state = checkpoints.restore_checkpoint(ckpt_dir=CKPT_DIR, target=state)

assert jax.tree_util.tree_all(jax.tree_util.tree_map(lambda x, y: (x == y).all(), state.params, restored_state.params))

There is work in progress to remove TensorFlow as a dependency in #2073 and #2444.
This should be solved in the future.

There is work in progress to remove TensorFlow as a dependency in #2073 and #2444.
This should be solved in the future.

On Sep 16, d9e6e44b545b7805ab827be499b2afa689f7e673 (v0.6.1) has the tensorflow dependency. Just commenting that neither #2073 and #2444 (although not complete yet) is aware of this change, so we look forward to have the tensorflow dependency removed.
I can also provide a help by writing some PRs; it would be great to systemically prevent such accidental use of tensorflo (in setup.py).

@sudo-michael @wookayin Check out the new Checkpointing guide by @IvyZX
https://github.com/google/flax/blob/main/docs/guides/use_checkpointing.ipynb

UPDATES: #2586 has been merged.


flatbuffer::Table - vTable format

Hello there,
I am currently working on a solution to get a tflite-micro model working on memory that is split onto different (physical) devices (I am working with C++).
It would be very helpful if I had some information how the vTable (flatbuffers.h - class Table) is stored in the .tflite format. Is there any documentation on the format I couldnt find, or would you mind sending me some examples?
Thanks for your help!
PS: My main problem is to find the head of the vTable

Hi @ClaasMeints! Sorry you're hitting issues. Unfortunately we don't have any additional documentation on the flatbuffer format, the closest thing is this Colab which shows how to use the Python API to manipulate files:
https://colab.research.google.com/drive/11R9hd_0zvW0y3mesitNtF8dCjf27xP9T?usp=sharing
Closing this bug on here, since I think the question might be more suited for the flatbuffers project.


Stepping off end of collection when desired key is not in the payload but other ignored keys are present

Version: 0.5.2
All platforms

When decoding a nested structure if the desired key is not present in the structure and ignoreUnknownKeys is set to true, MsgPackDecoder will step outside the bounds of the nested structure and continue searching for the key in the parent structure until it hits the end of the payload. Causing Kotlin Serialization to report required keys are missing in the parent structure.
As an example, take the given json encoded payload:
{
    "a": 1,
    "b": {
        "d": 2,
        "e": 3
     },
     "f":4
}

And the following Kotlin serializable class
@Serializable
data class Example(val a: Int, val b: Example.Child, val f: Int) {

     @Serializable
     data class Child(val c: String? = null)
}

ClassMsgPackDecoder when trying to find "c" will start a recursive call to BasicMsgPackDecoder.decodeElementIndex which if ignoreUnknownKeys is true will continue recursively calling BasicMsgPackDecoder.decodeElementIndex until it either finds "c" or in this case hits the end of the payload. Kotlin serialization will report that it could not find the required key "f"
ClassMsgPackDecoder needs to get the result of BasicMsgPackDecoder.decodeElementIndex so that it can properly increment decodedElements

Thank you for reporting this and thank you for taking your time to investigate the issue.
Feel free to open a PR if you wish to fix this yourself. If not, I will take it on as soon as I have the time.


Unsupported Java 8?

https://github.com/korlibs/korge-next/runs/6829604895?check_suite_focus=true#step:7:7449
I'm getting this stacktrace when running it with Java8:
e: java.lang.NoSuchMethodError: java.lang.Integer.parseUnsignedInt(Ljava/lang/CharSequence;III)I
	at androidx.compose.compiler.plugins.kotlin.inference.SchemeStringSerializationReader.number(Scheme.kt:342)
file or directory '/Users/runner/work/korge-next/korge-next/korge-compose/src/posixMain/kotlin', not found
	at androidx.compose.compiler.plugins.kotlin.inference.SchemeKt.deserializeScheme$item(Scheme.kt:209)
file or directory '/Users/runner/work/korge-next/korge-next/korge-compose/src/nativeMain/kotlin', not found

Is this intended?

Compose needs 11 at least. This is documented.

https://github.com/JetBrains/compose-jb/tree/master/tutorials/Getting_Started#new-project-wizard

Thanks for the info! Maybe it could check the runtime before and providing a meaningful error when configuring the plugin?
Also, is it final the JVM 11 version requirement? Any chance to allow Java8? Since this is Kotlin targeting 1.8 should be easy right?

Supporting Java 8 will add some burden. For example we won't be able to used the new API, which was added in JDK 9-11, and we have to live with the bugs, which were fixed in the newer versions of JDK. JDK 11 is a balance between the need for a lower version and a burden which this support adds.

Maybe it could check the runtime before and providing a meaningful error when configuring the plugin

This check was in the past, probably something is broken, or accidentally removed. We will fix this.

Oh, that's a pity. The Kotlin compiler still supports Java8, as well as proguard and all the other plugins I was using. So including compose would force users of my libraries to migrate to Java11.
I used to believe that by using Kotlin you could target Java8 and only get APIs available there so avoiding using APIs not available by mistake, and that Kotlin already provide a lot of goodies to make development enjoyable even in Java8. I also thought that Java8 had many patch releases fixing bugs, while still allowing people to using Java 11 or greater if they want.

Do you have users who are stuck on Java 8 in their environment and can't upgrade to Java 11 or later?

I believe that some people might be stuck with Java8 because old software and incompatibilities introduced in Java9 (like module visibility and things).
We also have very low level libraries like one for time management or data structures that could be used by potentially any Kotlin developer and some even for other JVM languages.
Also it is more likely that a normal user is able to run a jar built for Java8 than one for Java11 even nowadays.
Unfortunately I don't have specific numbers about how many users would be affected, but sometime ago we published a Java11 version by mistake and some users were affected and reported it not working by incompatible signatures of APIs existing in both versions (ie. some ByteBuffer API).

@soywiz please note, that you can target JDK 8 (bytecode & api, not runtime) in your library or application code, JDK 11 is needed:

At runtime;
For running the compiler itself (this is your exception);
Also JDK 15+ is required for creating installers/packages (but using it is optional).
It is possible to setup Gradle scripts, so that one JDK is used as a compilation target, another for running the compiler, and yet another for using at runtime/packaging.

Also it is more likely that a normal user is able to run a jar built for Java8 than one for Java11 even nowadays.

Is that really a "normal" desktop user? We provide a way to package an app to deb/msi/dmg package, so that end users would not think about JDK at all. They would just use a compose application like any other desktop software. For example, Intellij 2022.2 & all related IDEs runs on JDK 17, while previous versions ran on JDK 11 for a while. I don't think, that end users cared about that, and Intellij has a lot of users.
Moreover, using the latest JDKs would be beneficial for end users, as newer JDKs contain new low pause GCs, performance optimisations, bug fixes & security fixes. Some of bug fixes are really important. For example, there is an important accessibility related fix, which only landed in JDK 18 https://github.com/openjdk/jdk18/pull/72 Without it accessibility on Windows with HiDPI display is unusable.

This check was in the past, probably something is broken, or accidentally removed. We will fix this

@igordmn we only have a check for creating a distribution. The exception in the issue came from the compiler plugin. The JDK, which runs the compiler, can be configured in multiple ways, so I don't think we can reliably check Kotlin compiler's JDK in Gradle plugin. This can be done in the compiler plugin, but then it should be done by the upstream.

BTW despite the requirement in the readme, a compose desktop app might actually run on JDK 8 (if you setup a separate JDK for running the compiler itself).  In Compose 1.1 Skiko code, that previously used JDK 9 API for cleaning native objects was replaced with another implementation, which can work on JDK 8. However, some other parts might not work, and I don't know if we can or want to actually support JDK 8 as a runtime target

you can target JDK 8 (bytecode & api, not runtime) in your library or application code

Another clarification. Right now 1.2.* Compose Desktop Gradle plugin sets the target bytecode level to 11 by default, however this is not intended, the change was made during Compose 1.1 development, then reverted in 1.1 release, but I forgot to cherry-pick a few commits into the master. In particular this one https://github.com/JetBrains/compose-jb/commit/91fdcb8f9da001a747bc3736dd71543aac6ee53d

@soywiz I think, it is fixed with the latest alphas (745+), which uses 1.8 by default if not set.

I don't think there's anything to report. They said it may or may not run, and not supported as a target.

If you mean that Im using 1.7.10 instead of the supported 1.7.0: That error also happened on 1.6.20 and 1.7.0 with other versions. Strangely it runs fine on macos/linux/windows. Only related to ios, but not related to version mismatch

Kotlin 1.7.10 is not supported. Java 8 is not supported.

Should be reported as a separate bug then so that it doesn't get lost in here

Supporting Java 8 will add some burden. For example we won't be able to use the new API, which was added in JDK 9-11, and we have to live with the bugs, which were fixed in the newer versions of JDK. JDK 11 is a balance between the need for a lower version and a burden which this support adds.

Maybe it could check the runtime before and providing a meaningful error when configuring the plugin

This check was in the past, probably something is broken, or accidentally removed. We will fix this.

but the only new API that i found used compiler's bytecode is the auto-generated StringConcatFactory and can be replaced with StringBuilder

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


globalSetup.ts script ignores tsconfig.json

🐛 Bug Report
When tsconfig.json has defined compilerOptions.baseUrl, absolute paths can be used within test files for importing items. However, same TS code placed in globalSetup script handles only relative paths. Looks like tsconfig is simply ignored while running globalSetup.
To Reproduce
Steps to reproduce the behavior:

checkout https://github.com/4ekki/jest-globalSetup-import-bug. Note that first line of scripts/globalSetup.ts file is :
import { sleep } from "../tests/src"
run yarn test smoke
change first line of scripts/globalSetup.ts file to:
import { sleep } from "src"
run yarn test smoke

As a result:
On 2, globalSetup script is run successfully with relative path. Also, import with absolute path is successfully handled in test file.

On 4, globalSetup script fails to import sleep from src, using absolute path:

Expected behavior
On 4, absolute paths in globalSetup.ts are resolved using compilerOptions.baseUrl property from tsconfig.json
Link to repl or repo (highly encouraged)
https://github.com/4ekki/jest-globalSetup-import-bug
Run npx envinfo --preset jest
$ npx envinfo --preset jest
npx: installed 1 in 1.962s

  System:
    OS: Windows 10
    CPU: (8) x64 Intel(R) Core(TM) i7-4790 CPU @ 3.60GHz
  Binaries:
    Node: 10.5.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.7.0 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 6.1.0 - C:\Program Files\nodejs\npm.CMD

Can you please confirm that the issue exists and is related to jest, not to something else?

Your reproduction doesn't seem complete:

There's no script directory

Sorry for that, my bad. I've recreated the package from scratch, should be fine now.

Did you have a chance to look into this?

Not yet. Note that this is very unlikely to be a bug in jest as we do not support tsconfig at all out of the box (we use babel), most likely your issue is with ts-jest. However, I'd like to verify (although I haven't found the time yet)

I've duplicated this to ts-jest tracker as well - https://github.com/kulshekhar/ts-jest/issues/1107