$0 expenses / non-financial contributions

I was in an interesting thread where a community was discussing how people sometimes do work and don't want or need to be paid for it, or they want to buy or do something for the community as a gift. These kinds of contributions are often hidden in the background, meaning it's hard to get a picture of all the work actually going into a project. It occurred to me that Open Collective's system is already set up well for tracking contributions even if no funds are paid out for them. A truly transparent budget would also include volunteer/gift/in-kind contributions as a key resource.
User story
As a maintainer I want to be able to make transparent work going on in my project that is not being paid for financially. As a supporter, I want to get a fuller picture of who is contributing what to a project, whether they are asking to be paid or not.
Best solution for this problem
For a while we had "donations" in the system as a setting on an expense, but it was never fully implemented. A full implementation of that feature might meet the need here. If you could submit an expense describing what you gave an the value (of your time or what you bought), but select "donation" or tick a box for "I don't need to be paid", then people could easily transparently record more of what goes into a project behind the scenes.
MVP
If we allowed $0 expenses, people could record what they did or bought without needing to be paid.
Metrics
Number of Collectives submitting donation/$0 expenses to be more transparent to their communities.
Documentation
Add a page to the "expenses & getting paid" section, for "What if I don't need to be paid for what I did?" or "I want to donate this expense".

Isn't that what Donation payout method is for?

The Donation payment method is broken. It debits the total from the balance, when it should record it but not change the balance. I guess that doesn't matter if it $0 but I don't think $0 are allowed. Maybe we can think about a better solution for this as part of the expenses improvements, or maybe it will be addressed by the expanded contribution types in the new Collective page design.

The balance being debited sounds like a bug to me (though I don't really know that feature). Do we know why it's like this?

Another request for this in https://github.com/opencollective/opencollective/issues/6978

OCF's current process is documented here: https://docs.opencollective.foundation/how-it-works/financial-contributions/in-kind
It's currently a pretty manual process that could definitely be made more efficient.  Would want to check first how much oversight is actually required before automating everything.

While I am personal attracted to this potential and am curious about the effects it could have at a platform scale this is NOT currently a priority.

depending on what our accountants say, this may actually need to be prioritized

Just to add another +1 here, as a current and past admin of a couple of Collectives.
It is volunteer work we do, and that alone is not enough. Successful collectives always receive help along the way. Yes, some of that support is monetary, but there is also a lot that isn't!
Here is a list of some things for which the collectives I have been involved in, in the past, have received as in-kind support:

being allowed to use a meeting space for an event,
design services (logo/branding),
catering,
advertising services (e.g. announcement on newsletter / radio / meetup / etc.),
accounting and legal services,
audiovisual equipment,
etc.

It would really help if we had some way to showcase our in-kind sponsors on our page.
We are not interested in the tax side of things, though I do that is a concern in some jurisdictions.
We just need some way of showing a banner / logo, with a link to that sponsors page.


[test] Disable tests until we fix _StringProcessing

These are failing due to https://github.com/apple/swift/pull/42611. I have a fix locally that needs a bit more work to finish, so disable these tests until then.

https://github.com/apple/swift/pull/59675


bug: some commands are not working

Describe the bug

models start
models stop
chat

To Reproduce
run command:
.\cortex-cpp.exe models start llama3:tensorrt-llm-windows-ampere
Failed to read file: bad file: C:\XXX/models/llama3:tensorrt-llm-windows-ampere/model.yml

Expected behavior
All commands should work

I think the PR is done so it's closed? @vansangpfiev Please update the status of the ticket next time, thanksss

FYI @vansangpfiev moving tix into Completed in Kanban doesn't close the issue (annoyingly). You'll have to specifc in the PR itself: closes #... 🙏


Prometheus server latency metrics no longer provide labels per service and method

We recently upgraded a service from v0.14.1 to v0.18.4. In the old version there was a way to opt in to latency metrics with labels per service and method. I may be missing something, but I'm not seeing a mechanism provided that would allow for the same sort of per-endpoint latency metrics. If that is the case, is there maybe a reason those were removed that I might be missing? And if there is no reason, would anyone be opposed to adding something back in? I would be happy to submit a PR if there are no objections to the idea.

Hi @tyler-clark, you mean adding the serviceName to this?
https://github.com/higherkindness/mu/blob/master/modules/metrics/prometheus/src/main/scala/higherkindness/mu/rpc/prometheus/PrometheusMetrics.scala#L36
Probably something we missed in the migration to the new approach. I'm ok with adding back.
In the meantime, you could create your own implementation of the PrometheusMetrics and add methodInfo.serviceName here

Thanks @fedefernandez ! I'll probably take your suggestion to create my own implementation for the short term, and then work on getting the metrics added back to mu for a more long term solution.
Would the preference be for those latency metrics specifically to be another opt-in like they used to be?

Great! I think including histograms as an opt-in would be the best option, but we could add a simple param to the PrometheusMetrics builder, WDYT?

@fedefernandez That seems reasonable enough to me, but that would break binary compatibility, wouldn't it? Maybe not a huge deal as it would just mean a new minor version at this point. Thoughts?

@tyler-clark maybe we can keep the default builder and add a new one for specifying that param. I'm ok with breaking binary compatibility and release a new minor version though but I think a different builder could be enough.


A11y_AzureDataStudio_AddNewController_ScreenReader : NVDA is not announcing the error message as soon as error has thrown .

"Check out Accessibility Insights!- Identify accessibility bugs before check-in and make bug fixing faster and easier."

GitHubTags:#A11y_ADS_OctTestPass;#A11yMAS;#A11yTCS;#MAS1.3.1;MAS3.3.1;DesktopApp,#NVDA;#JAWS;#SQL AzureDataStudio;#Win32
Environment Details:
Application Name: Azure Data Studio
Version: 1.12.0-insider (user setup)
Commit: 6582debd7388b1be512fc3a9b956d3e54f6115e7
Date: 2019-10-03T08:56:27.454Z
VS Code: 1.38.0
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Windows_NT x64 10.0.18362
NVDA Version: 2019.2
JAWS Version:2019.1907.42 ILM
Additional Details:
Tool Name : NVDA , JAWS
Steps to Reproduce:

Install and Launch the application the azure data studio application.
Connect to the servers with the required credentials.
Navigate to the "SQL Server Big Data Cluster" and expand it.
Navigate to the "SQL Server Big Data Cluster controller" and click on it.
Start NVDA.
Navigate till "URL " edit box and some text and then navigate till "Add" button after that error message is thrown.
Observe the announcement by screen reader.

Actual:
Screen reader is not announcing the error message as soon as it has thrown after clicking on "Add" button.
Similar issue is also observed with JAWS .
Expected:
Screen reader should announce the error message as soon as it has thrown after clicking on "Add" button..
User Impact:
Screen reader user won't understand that particular error message has thrown  after clicking on "Add" button.
Attachment for Reference:
7949_A11y_AzureDataStudio_AddNewController_ScreenReader.pptx
Does this issue occur when all extensions are disabled?: Yes

#Azure Data Studio - New Features_win32;

Github Tags: #A11ySev2;

#Closed; #Regressed:07-14-23;


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 468 ms

Resolved: ClamAV performance has improved in 398033d.


Different outputs in different version of Ubuntu

Hello,
I'm facing a problem in the past few days. I am running the same experiment in 4 different PCs (2 with Ubuntu 16.04 and 2 with Ubuntu 18.04) and while the 2 Ubuntu 16.04 PCs manage to solve the problem perfectly, the Ubuntu 18.04 PCs behave differently.
Specifically, I'm running a multi-agent RL experiment where I have removed all the safety measures and collision are allowed. The goal is for every agent to learn its policy in order to resolve all the collisions.
The problem arises when I'm running the experiments on the Ubuntu 18.04 machines where my algorithm converges to 20+ collisions, while on the other hand the Ubuntu 16.04 PCs manage to result to 0 collisions. Of course, I've run the same experiment to all the PCs and I've tried it a lot of times. I've also checked and copy-pasted the code to be exactly the same to all PCs.
Is this a known bug between the two versions?
Thanks in advance :)

Hello, this is extremely strange! Is there any way you could give us access to the scripts you are running so that we can test it?
At the moment the only possible error I can imagine is that the SUMO version between the machines is different.

I've mad a mistake to my algorithm, but it is still weird how the one managed to solve my problem. Anyway, sorry for the false alarm. :)


ephemeral: 6.3.0 -> 6.3.1

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/ephemeral/versions.
meta.description for ephemeral is: "The always-incognito web browser"
meta.homepage for ephemeral is: "https://github.com/cassidyjames/ephemeral"
Updates performed:

Version update
Quoted meta.homepage for RFC 45

Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 6.3.1 with grep in /nix/store/w5xpswivgqff0h4fa8bhsarf1aqfgy62-ephemeral-6.3.1

directory tree listing: https://gist.github.com/3837091ac239db31952644e65eba42b1

du listing: https://gist.github.com/3bf0a248cb1b743b71bae03cca2ca58c

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
ephemeral

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w5xpswivgqff0h4fa8bhsarf1aqfgy62-ephemeral-6.3.1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ephemeral https://github.com/r-ryantm/nixpkgs/archive/372cb65467fda773d22389ff9327bd25ee8fe107.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w5xpswivgqff0h4fa8bhsarf1aqfgy62-ephemeral-6.3.1
ls -la /nix/store/w5xpswivgqff0h4fa8bhsarf1aqfgy62-ephemeral-6.3.1/bin

cc @kjuvi @worldofpeace for testing.

The fixes are in that file, but also here https://github.com/cassidyjames/ephemeral/releases/tag/6.3.2


Mouse dragging feature broken.

Recently I upgraded everything to the latest version, but mouse dragging doesn't make it scroll anymore.
Environment: Linux, nvim v0.6.0

Thanks for reporting this @aj3423!
I'm unable to reproduce this. I tried using the same version of Neovim, v0.6.0.
What is the output of :set mouse??
What is the output of :nmap <leftmouse>?
If another plugin is registering a mapping for <leftmouse>, nvim-scrollview will not clobber the other mapping. In that case, the mapping could be specified manually:
nmap <leftmouse> <plug>(ScrollViewLeftMouse)

of an alternative mouse button could be used:
nmap <rightmouse> <plug>(ScrollViewRightMouse)

The issue may have been a result of #54, which switched the mouse handling functionality to use getmousepos to fix issues and shorten the code. That should have worked on v0.6.0, but not if you were on a pre-release prior to https://github.com/neovim/neovim/pull/14937 being merged.


Introduce reset_{at|after} functions for ticker.

These functions perform a similar job to reset, by reseting the timer either after a specified duration or at an instant.
The old PR was closed, since I renamed the branch to be more descriptive.

I think the CI needs to be restarted, since three tests are still waiting for status.

bender run

@Frostie314159 Could you try to rebase this branch please?


unsupported operator ReduceL2

Hi, I have trouble converting my swin transformer v2 model from ONNX, as this model has ReduceL2 operator.
I found out that ReduceL2 operator is not supported.
In rknn-toolkit 1, there was a function named custom operator.
But I could not find its equivalent function in rknn-toolkit 2.
So, how can I run this model? My hardware is RK3568. Thanks in advance!

I have the same problem, the current rknn-toolkit2 does not support ReduceL2 operator, did you solve it?

@BigerPatax Sadly, no. We decided to use resnet-based models instead.


Causes Android System crash on Galaxy S3

Hubroid locks up as soon as I start it and then causes an Android System crash. I have a Samsung Galaxy S3.

Closing since this project seems dead and I haven't used it in years.


Guide is outdated

Hi!
I'm running HVM version 0.1.89.
I was following the guide to give HVM a spin, but ran into some issues:

hvm run "(Expression)" command doesn't work anymore, it needs a filename as an argument instead of an expression
HVM.query isn't working - it doesn't wait for user input, just assumes nil. This is regarding the following code example:

(String.concat String.nil         ys) = ys
(String.concat (String.cons x xs) ys) = (String.cons x (String.concat xs ys))

Main =
  (HVM.print "What is your name?"
             (HVM.query @name(HVM.print (String.concat "Hello, " name)
                                        (Done))))

It is actually the other way around, you're probably using the old version, since the latest wasn't published on Cargo. Type cargo install hvm again and it should work now. Current version is 1.0.0-beta. We'll release 1.0.0 soon.

Hmm.
I have 0.1.89 installed, and running cargo install hvm made no change.
Then I ran cargo install hvm --version 1.0.0-beta but that gave me the following errors:
error[E0554]: `#![feature]` may not be used on the stable release channel
 --> $HOME/.cargo/registry/src/github.com-1ecc6299db9ec823/hvm-1.0.0-beta/src/lib.rs:1:12
  |
1 | #![feature(atomic_from_mut)]
  |            ^^^^^^^^^^^^^^^

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> $HOME/.cargo/registry/src/github.com-1ecc6299db9ec823/hvm-1.0.0-beta/src/lib.rs:2:12
  |
2 | #![feature(atomic_mut_ptr)]
  |            ^^^^^^^^^^^^^^

For more information about this error, try `rustc --explain E0554`.
error: could not compile `hvm` due to 2 previous errors
error: failed to compile `hvm v1.0.0-beta`, intermediate artifacts can be found at `/var/folders/86/0m74r2xn4yz74znbcwgp85gr0000gp/T/cargo-install7kYuMp`

@divs1210 you should be able to install it like this:
rustup install nightly
cargo +nightly install --force --git https://github.com/Kindelia/HVM.git

The first command installs the nightly Rust distribution, which it looks like HVM requires for now, and then the second command installs the latest version from the git repo.

I guess my install instructions are wrong, sorry. Can someone update on the README and on guide/README and PR please?

Yeah, I'll open a PR soon.
Your instructions weren't really wrong for a first-time setup: you did say to set Rust nightly as the default, but it's easy to miss that step or set it back to stable for a different project or something like that.
Otherwise using --force and --git just makes sure it works in more scenarios, if you already have it installed or have an out-of-date git clone.

Thank you, @zicklag!
That worked. 🙂

Just a little reminder that the project README.md still tells people to install the 3 month old (broken?) stable version; and set their default rust toolchain to nightly instead of use cargo +nightly install: https://github.com/Kindelia/HVM/blob/4974a8ae7c83b775aaf627719e638d06dac4a4dc/README.md#getting-started

@racs4 this was fixed, right? Can we close this issue?

My PR #192 with the new instructions was not accepted

Maybe merge #205 as well.
It appears -c now needs to be -c true.


Powertoys settings and Quick Access windows problem

Microsoft PowerToys version
0.83.0
Installation method
GitHub
Running as admin
No
Area(s) with issue?
System tray interaction
Steps to reproduce
I was using an older version of PowerToys and this problem occurred. After uninstalling and reinstalling, or uninstalling and installing the latest version, the following problem occurred and I was unable to use PowerToys.
This problem persists whether I install the latest version, system-wide or per-user, Microsoft Store or GitHub
✔ Expected Behavior
When I double-click the task tray icon or right-click the icon and select "Settings", PowerToys settings windows appears.
When I single-click the task tray icon or right-click the icon and select "Quick Access", Quick Access window appears.
❌ Actual Behavior
When I double-click the task tray icon or right-click the icon and select "Settings", the white PowerToys settings window appears for about 2-3 seconds, then the window disappears. I can't do anything.
When I single-click the task tray icon or right-click the icon and select "Quick Access", the cursor turns into an hourglass for a moment, but then the Quick Access window never appears.
Other Software
Edition	Windows 10 Enterprise
Verion	22H2
Install date	2021/08/16
OS ビルド	19045.4651
エクスペリエンス	Windows Feature Experience Pack 1000.19060.1000.0
Bug report zip file is attached.
PowerToysReport_2024-08-08-14-58-30.zip

I have seen above two Issues is same as mine. Both of tthose.
I'm having trouble using Power Toys, so I hope they fix it soon.
II will clese this issue.


Release for Mill 0.11

Is there anything that blocks cutting a new release so that Mill 0.11 is supported?

Not that I recall. It seems master is on 0.11.x so it presumably builds fine with it already.
If you haven't already can you try it with a publish local to make sure there aren't any changes you'd want to get into the release?

I've been using master with webpack and rollup for an app on 0.11. And it works fine so far.

Great! I didn't know I even had users ;)

I tried to release it but the windows job fails. https://github.com/nafg/mill-bundler/actions/runs/7602916602/job/20704019068
Anyone able to look into it?

The error seems to be this:
[#1] main.test.ivyJsDeps java.nio.file.AccessDeniedException: C:\Users\runneradmin\AppData\Local\Coursier\cache\v1\https\repo1.maven.org\maven2\org\scala-js\scalajs-library_2.13\1.10.1\.scalajs-library_2.13-1.10.1.jar__sha1.computed12993824255704690508.tmp -> C:\Users\runneradmin\AppData\Local\Coursier\cache\v1\https\repo1.maven.org\maven2\org\scala-js\scalajs-library_2.13\1.10.1\.scalajs-library_2.13-1.10.1.jar__sha1.computed

hmm clearing the cache seems to have fixed it. Maybe the OS should be in the coursier cache key?

Ok, 0.2.0 is on Maven Central (or it will be after whatever the delay is)!

It shows up now under https://repo1.maven.org/maven2/io/github/nafg/millbundler/

Works just fine, thanks.
Really think that should be integrated into mill at some point as without it scalajs with mill is pretty limited.

I'm definitely in that camp, but apparently a lot of people are just fine managing their package.json and bundlers by hand, like JS devs do.


Doc upload is broken

Docs uploads are currently broken, with a weird error: https://travis-ci.org/sunriseos/SunriseOS/jobs/557798186
I believe this was introduced in the update to the newer cargo-travis. Will investigate.

Fixed in cargo-travis.


Selector menu

Is your feature request related to a problem?

What solution would you like?
I'm making some changes to the code to use the app myself. I was trying to change the selector menu to a segmented style, but i wasn't able to get it working. A example of the selector is below, Is there any way you can explain how i can change this or add an branch with the changes

What alternatives have you considered?

Any additional context?
No response

Hi @TimTheBeastNL,
As your problem doesn't relate to an issue with the VanMoofKit please consider asking this kind of question on a platform like Stack Overflow.


Package runfiles along with C++ binaries in pkg_* rules

Description of the problem / feature request / question:
(https://groups.google.com/forum/#!topic/bazel-discuss/5r_Ajw_j-ZI for context)
The current pkg_* rules don't package runfiles along with C++ binaries. This behavior makes it difficult to deploy an entire C++ application to machines that don't have source access.
Can we update the packaging rules to include runfiles along with binaries?
Environment info

Operating System:
MacOS 10.13.2

Bazel version (output of bazel info release):
release 0.8.1-homebrew

Folding this into #153

Actually, it should be easy to implement pkg_runfiles in Starlark:

DefaultInfo.data_runfiles (or DefaultInfo.default_runfiles?) contains the File objects for the runfiles
FilesToRunProvider.runfiles_manifest contains File object for the runfiles manifest
It should be easy to implement a custom C++ binary or Bash script that can parse a runfiles manifest and tars/zips up the runfiles it references
Using that binary/script, our hypothetical pkg_runfiles rule could create a ctx.actions.run / ctx.actions.run_shell with the binary/script, pass all the File objects for the runfiles and the manifest as inputs, and expect just a tar or zip as the output.

WDYT?

@laszlocsomor I'm a bit late to the party, is this still the best alternative?
The #153 is still ongoing work and the proposal from @benjaminp were accepted (links for proposals are dead) but the runfiles_manifest has still absolute paths, what's the current best way to package runfiles in Docker for python

Please let's have any more discussion in #153.
It's not that it is hard to just get the runfiles into the tar file, an existing flag does this.
The hard part is a design that lets you conveniently move data and shared libraries to distinct places as appropriate for your target OS.


"Tibod", "Ulcani" and "Valea Rotundă" are listed in the drop-down menu

Description:
"Tibod", "Ulcani" and "Valea Rotundă" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Tibod" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Tibod, HARGHITA (Dealu)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Ulcani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Ulcani, HARGHITA (Dealu)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Valea Rotundă" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Valea Rotundă, HARGHITA (Dealu)".


feat: abort openai text completion when stopping responses

Pull Request Checklist

[x] Description: Briefly describe the changes in this pull request.
[ ] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation?
[ ] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?

Description
Attach an abort controller to the OpenAI streaming completion call and abort generation when pressing the stop button.
Previously, the request was not aborted or closed, so it would still be streaming in events in the background.

Changelog Entry
Added

[List any new features or additions]

Fixed

[List any fixes or corrections]

Changed

[List any changes or updates]

Removed

[List any removed features or files]

LGTM, Thanks!


react-native v.45.1

Does react-native-router-flux support react-native v.45.1?
Thanks

the latest V3 in master branch do support it.


[Bug]: Authentication failed for trigger.dev service

Error Message and Logs

Steps to Reproduce

Install the trigger.dev service in my server
Access the service and choose trigger v3
Running the initialize command in my next js project
Access trigger login url and it shown authentication failed

Example Repository URL
No response
Coolify Version
v4.0.0-beta.370
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
No response
Additional Information
I think we need to adjust some env values with this length instead of using 64 length.
https://trigger.dev/docs/open-source-self-hosting#manual
After I update the values, it works

Related: https://github.com/triggerdotdev/trigger.dev/discussions/1499#discussioncomment-11390709


Investigate if the transparency log can be used to verify the binaries (docker images) on the client side.

Suggestion details
We need a way to verify local binaries (images, jars, etc) against the Pyrsia transparency log. This task is about investigating if the Pyrsia transparency log contains enough information to confirm that a given binary can be considered "secured" in Pyrsia terms. That means if the binary was downloaded from Pyrsia or if it can be safely identified as secured when compared against the transparency log. In case the transparency log doesn't contain sufficient data then what needs to be added to the transparency log to allow this type of client side verification.
Additional details
No response

What is the significance of following items?

num_artifacts
source_hash
source_id

For example when inspecting alpine:3.17.1 from Pyrsia Network (i.e. pyrsia inspect-log docker --image alpine:3.17.1) why there are 4 entries in the Transparency Log?

artifact_hash & package_specific_artifact_id of one entry out of those 4 entries are matching to the Id of docker inspect

Is it possible to validate/verify a docker image irrespective of download of the source from Pyrsia's Transaction Log?

IMO, maybe we could start by explaining how to sign docker images and store these information in a transparent log

In my case I don't see any common keys/properties when I compare the transparency log with docker info (docker inspect) as it's shown on @dasmanas screenshot. I'm not sure why but in my case  the docker image info (docker inspect) and the transparency log are decoupled except for the docker image tags.

@karolh2000 Excuse me. Maybe "sign docker images" is causing some confusion.  I mean, we can start by explaining what information (checksums, hashes or metadata) is stored in the transparent log, and how this information is created and how a certain level of authenticity and integrity is guaranteed.

Another question, maybe this question can be divided into online and offline situations. I guess we can start with the online situation. Online means that the client is always in sync with the Pyrsia network.


electric-indent broken in YAML mode

Typing a colon : character seems to make it jump "back" an indentation level, along with general weirdness.

Looks like yaml-mode's indenter is a bit quirky. I've turned off electric-indent-mode in yaml-mode in the latest commit. That should fix it.

Thanks, working now!


feat(websocket): Websocket source!

refs #42

Source configuration supports websocket and kafka
Source.read() method is now an interable supporting the websockets API


PLAT-113933: Add a qa-a11y sample

Add a qa-a11y sample to verify A11y behavior of the Agate components.
First of all, I cloned the qa-a11y sample in Sandstone.
And then I update files and Views properly for Agate APIs and UXs.
In addition, I fixed the following things

Use "react": "^16.8.0" and "react-dom": "^16.8.0"
because Agate uses them.

All views except ReadAlert are now functional components, not React components
to make their code simple

Define useBoolArray hook
to manage open states of Popup and Drawer views.

In Sandstone: https://github.com/enyojs/sandstone/blob/develop/samples/qa-a11y/src/views/Popup.js, there are many state and handler definition.
	constructor (props) {
		super(props);
		this.state = {
			open0: false,
			open1: false,
			open2: false,
			open3: false,
			open4: false
		};
		this.handleOpen0 = this.handleOpen(0);
		this.handleOpen1 = this.handleOpen(1);
		this.handleOpen2 = this.handleOpen(2);
		this.handleOpen3 = this.handleOpen(3);
		this.handleOpen4 = this.handleOpen(4);
		this.handleClose0 = this.handleClose(0);
		this.handleClose1 = this.handleClose(1);
		this.handleClose2 = this.handleClose(2);
		this.handleClose3 = this.handleClose(3);
		this.handleClose4 = this.handleClose(4);
	}
	handleOpen = (expNum) => () => this.setState({['open' + expNum]: true});
	handleClose = (expNum) => () => this.setState({['open' + expNum]: false});

	const {open0, open1, open2, open3, open4} = this.state;

In Agate: https://github.com/enactjs/agate/blob/feature/PLAT-113933/samples/qa-a11y/src/views/Popup.js,
with useBoolArray, the code could be more concise.
	const [open, setOpen] = useBoolArray(5);

The open1, open2, open3, and open4 are replaced with open array and the handleClose0 is replaced with setOpen(0, false).

Move all controls in the Panel's Body to in the Panel's Header
In Sandstone:

In Agate

Enact-DCO-1.0-Signed-off-by: YB Sung<EMAIL_ADDRESS>

I fixed the following things based on the review.

[x] means that I fixed the issue mentioned in the review.
[ ] means that I didn't fixed it mentioned in the review because it's a component bug or not a but or something else.

Leave some comments

Let's have spaces between menu number and string

[x]  : Fixed.

Drawer: Can't close the drawer

[ ] Won't fix in the PR (A component issue). When opening a Drawer, the close button in the Drawer can't be spotted via 5-way keys. I hope that we could solve this issue in a separate issue. Until then, there is one way to spot the close button. After selecting an open button, please press 5-way up several times until the close button gets focus. If using a touch screen, then the close button can be clickable.

Dropdown: Agate Dropdown has a placeholder?

[x] No. I fixed it.

Header: Add some margin between button and horizontal line

[x] Fixed temporary. I don't think that it is the app issue, but a component issue. But I fixed it app side first. We need to revert it after fixing it component side.

ImageItem: Image doesn't look 200 X 200

[ ] Not a bug. It is the expected result because the ImageItem has a horizontal orientation prop. In this case, the ImageItem's width is 25% and the ImageItem's height is 100% of the item (not of the Image).

	&.horizontal {
		.image {
			width: 25%;
			height: 100%;
		}
	}

Panels: Panel transition direction is opposite

[ ] Not a bug. It's the expected result because if any arranger is not defined, then the default arranger is the SlideRightArranger. Panel transition direction is from left to right. So it's not a bug.

For more information, please check the https://github.com/enactjs/agate/blob/develop/Panels/Viewport.js#L80 code.

Picker: Check Vertical - Boston AIrport and Airport strings are overlapped

[ ] Fixed temporary. Marquee has not been applied to a Picker. So the text in it could be overflow. For now, I just applied the overflow: hidden. But we need to discuss our policy for this case and to fix it properly.

Popup: Buttons has focus when popup opens (check with Buttons Popup)

[ ]  Won't fix in the PR (A component issue). When showing a Popup, Spotlight doesn't move to any component in the Popup. So that's the reason why the buttons has focus. I think that we need to fix it component side in a separate issue. Until then, we could press a back key with a remote controller or a esc key in PC. If using a touch screen, then the close button can be clickable.

VritualGridList: Enable scrollbutton focused just like Scroller

[x] Fixed.

VirtualList: Enact scrollbutton focused

[x] Fixed.

I found more component bugs. I hope that we could fix them in separate issues.

If not defining the width style in the Native CheckboxItem in the VirtualList view, the text in the CheckboxItem doesn't display. So I defined it app side. But this should be fixed component side.

Expected Result:

Actual Result

When hovring the disabled Dropdown button, then the text in it doesn't display.
Actual Result
before hovering

after hovering

Depending on the text width of the year Picker in DateTimePicker, the width of the year Picker changes

@ybsung
LGTM for this sample but please make components bug tickets that you mentioned above.
I created https://jira2.lgsvl.com/browse/PLAT-118480 for image item. Looks very wired with "200X200"


Ask codegpt no response

When i ask gpt in the window, there is no response. And yet when i connect through api, it works.

@UniMars
update to version 2.0.4

@UniMars update to version 2.0.4

It is 2.0.4

Same thing is happening to me with the latest update.
I was on v2.0.4 this morning, and it was working. Updated to v2.1.1, and now nothing but empty responses.

API key is set. Attempted with gpt-3.5-turbo and gpt-4. Like UniMars, I can connect to the GPT API no problem, just not with this extension.

@dumb
We are finding cases where the OpenAI API just returns a blank response... We are going to identify the problem and display the error and if there is no error, retry until the response is delivered.
For now, I recommend you reload the editor and try the question again

@dumb
Another thing... you must have at least one editor open.
We are going to add the alert for this case as well.

I have the same problem. I don't receive any response or error message when I ask  to chat. It only show a chatbot icon with label CODEGPT. Nothing more.
Can somebody help me to solve this problem?
Thanks

I'm having the same issue. I just created my API key and used the CodeGPT: Set API Key command. After that, no response to anything I type.
I went to the OpenAI site and inspected my key, and it shows that it has never been used, so seems like my chats aren't even making it to the server?

I have the same issue. The side bar show "stop generate" seems like it is in running but will not over.

I have the same problem. It worked fine once last week. When I wanted to use it again today it doesn't do anything. I only get the empty response with the "Stop Responding" button and nothing else.
I re-entered the APU key, re-installed the extension, I even uninstalled VSCode and reinstalled everything but the problem persists.
Commit: 252e5463d60e63238250799aef7375787f68b4ee
Date: 2023-05-03T20:09:00.748Z
Electron: 22.4.8
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044
Sandboxed: Yes

Code GPT v2.1.6
Published
12/25/2022, 07:38:30
Last released
5/7/2023, 17:47:29
Last updated
5/8/2023, 13:05:39

I have the same problem. It worked fine once last week. When I wanted to use it again today it doesn't do anything. I only get the empty response with the "Stop Responding" button and nothing else.
I re-entered the API key, re-installed the extension, I even uninstalled VSCode and reinstalled everything but the problem persists.
Commit: 252e5463d60e63238250799aef7375787f68b4ee
Date: 2023-05-03T20:09:00.748Z
Electron: 22.4.8
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044
Sandboxed: Yes

Code GPT v2.1.6
Published
12/25/2022, 07:38:30
Last released
5/7/2023, 17:47:29
Last updated
5/8/2023, 13:05:39

UPDATE I found the issue here. It was a new security service on my work laptop which seems to block the API calls. When I temporarily disable the security service the extension acts normal as before.

Can you please write more information about your solution, please


To Serialize or not to Serialize

Hi,
I wish to have my Swagger UI generated by TsED and store my data using Mongoose.

In the future I would love to generate my mongoose schema from my model as well...

Currently I have a serialisation issue that I would like to discuss here? And perhaps get some idea on how to proceed?
I will create a model and a controller for storing the following Example:
export interface IExample {
    simpleProperty: string;
    complexProperty: {
        someProperty:number;
    }
}

In order to make this model show in Swagger I create the following two classes:
export class Complex {
    @JsonProperty()
    someProperty: number;
}

export class Example implements IExample {
    @JsonProperty()
    simpleProperty:string;

    @JsonProperty()
    complexProperty: Complex;
}

To store this we create a simple mongoose schema (should later be generated from POJOs)
const schemaDefinition = {
    simpleProperty: {type: String},

    complexProperty: {
        someProperty: {type: Number}
    }
}

const exampleSchema: Schema = new Schema(schemaDefinition);

export type ExampleDocument = IExample & Document;
export const ExampleStore = mongoose.model<ExampleDocument>('Example', exampleSchema);

We add a simple controller to allow saving of the examples:
@Controller("/examples")
export class ExampleCtrl {

    @Post("/")
    @Returns(200, {use: Example})
    async create(@Required() @Name("example")  @BodyParams() example: Example): Promise<Example> {
        const storable = new ExampleStore(example);
        await storable.validate();
        return  await storable.save();
    }
}

NOTE: The above will unfortunately not save the complex property
The problem is the additional class needed to decorate the nested property in order to get the wanted Swagger UI! The serialisation done by TsED is correct but not really wanted!? To save using mongoose we need the lean representation.
To make it save correctly one could do something crazy like the following:
    async create(@Required() @Name("example") @Description("The example to create") @BodyParams() example: Example): Promise<Example> {
        const storable = new ExampleStore(JSON.parse(JSON.stringify(example)));
        await storable.validate();
        return  await storable.save();
    }

The data would also have the correct form if we were to skip the converters all together adding the following in BodyParams:
 useConverter: false

but then the types would not be correct which would be undesirable.
What do you guys do in this case? Any ideas? Should we perhaps find a way to separate serialisation from swagger specification. Or, am I completely missing something here?
One could stick with an interface for the complexProperty and then make a hint for the swagger implementation - sort of:
export class Example implements IExample {
    @JsonProperty()
    simpleProperty:string;

    @JsonProperty()
    @Swagger({use: Complex})
    complexProperty: IComplex;
}

Help and suggestions are more than welcome. I would love to implement changes on both sides of the fence I just need to know where we should go on this
Kind Regards
Lasse

Hi @lwallent ,
I'm not sure if I correctly understand your problem. I need an example ;)
What is the object structure sent to the method ?
{
  ... replace the structure
}

What is the object structure when converter deserialize the object ?
{
...replace the structure
}

See you ;)
What do you want as a result?
{
...replace the structure
}

For me BodyParams() and Swagger must use the same Model, because it's the contract of service that you expose.

Hi @Romakita,
Well I guess my problem can be reduced to the presence of classes as parameters in the endpoints - normally I would just have interfaces, but generating the Swagger requires the classes.
The problem is more related to Mongoose. I should probably investigate how Mongoose reads its data from objects. The issue can be reduced to the following not working with Mongoose;
  let ex = new Example();
  ex.simpleProperty = "test";
  let cplx = new Complex();
  cplx.someProperty = 42;
  ex.complexProperty = cplx;

  const storable = new ExampleStore(ex);
  await storable.validate();
  return  await storable.save();

In this case only the following is saved (complex is missing):
{   "__v": 0,   "simpleProperty": "test",   "_id": "59d9573d229391a03c0b9369" }

Whereas the following does work:
 let ex = {
   simpleProperty: "test",
   complexProperty: {
      someProperty: 42
   }
 }

 const storable = new ExampleStore(ex);
 await storable.validate();
 return  await storable.save();

Problem is that TsED deserilisation produces the first (which is type wise correct).
This might be a mongoose thing I was just wondering if you had experienced this?
Thanks for taking your time :-)

Hi @lwallent ,
I've found a solution :)
In your MongooseService you can write this:
import * as Mongoose from "mongoose";
const Document: Type<any> = (Mongoose as any).Document;

@Service()
export class MongooseService {
    static resource: Mongoose.Connection;

    /**
     *
     * @param {ConverterService} converterService
     */
    constructor(protected converterService: ConverterService) {

        MongooseService.override("set", (obj, ...args: any[]) => {
            return [
                converterService.serialize(obj),
                ...args
            ];
        });

        // optional.
        Document.prototype.toClass = function (target) {
            return converterService.deserialize(this.toObject(), target);
        };
    }
}

Example:
async save(@BodyParams() event: CalendarEvent)  {
        // CalendarEventsService.checkPrecondition(event);
        $log.debug({message: "Validate event", event});
        const eventModel = new CalendarEventModel(event);

        await eventModel.validate();
        $log.debug({message: "Save event", eventModel});
        await eventModel.update(event, {upsert: true});

        event = eventModel.toClass(CalendarEvent);

        $log.debug({message: "Event saved", event, isEventClass: event instanceof CalendarEvent});
        return event;
}

I let you try this experimental solution ;). If it work fine for you, maybe we can write a PR in the documentation ?
See you,
Romain

Hi @Romakita,
Thanks for the suggested solution! That is kind of a crazy approach :-)
I will go forward with it, however, I do not think personally that this should be added "as is" to the documentation. Rather I think the code it self should be added. I have given the whole mongoose thing some thought lately and I think we should rather make a separate folder (like Swagger) that will hold code for making TsED play nice with Mongoose . This should include code for generating the Mongoose Schema and could also hold special code like the above, so that the complexity is hidden from the day-to-day use :-)
I will have a look at it when some time comes my way! Please let me know if you are already working on some Mongoose stuff, then I will not interfere :-)
Kind Regards and thanks for taking your time
Lasse

Hi @lwallent,
Yes I've this feature in mind (socket.io too), but nothing produced code. You have typegoose as example, work fine, but it's little tricky. (also node decorators).
It's a great idea if you work on ;). I can create the story in github and write some example :).
See you,
Romain

Hi @Romakita,
I worked with typegoose for some time (their code as well) but in the end I backed away as I disliked the way my model classes had to extend the Typegoose class. I would like to have completely independent model classes and then just use the decorated classes to generate the mongooseSchema (complete separation).
You are welcome to add a separate issue/story where the details could be discussed. I can describe my take on it as well :-)
Kind Ragards
Lasse


Incompatibility with Tailscale MagicDns (Could not resolve host)

I am running tailscale on my Phone and when i try to access a server running tailscale, curl and wget are unable to resolve the hostname.

Regular urls work correctly.
When using the browser or regular termux, the hostname can be resolved:

Just noticed #2 which is probably the root cause, will investigate further.

I fixed the hostname resolution after adding the Tailscale DNS Server to resolv.conf file, which fixed it for wget,
but curl does now neither print errors nor print contents.

curl does not follow redirects by default, you need to add -L flag to it.

Ah, my inexperience with curl shows, though I am still confused as to why it seemed to partially work without -L over in Termux.
Closing this Issue as all relevant issues are resolved.


[paired_batch_size] Write score

Minor enhancement to write affinity score by default, for paired batch mode. @ysyecust

@ysyecust kindly review


Minitar v1 has been released

Minitar v1 has been released. Puppet correctly uses ~> 0.9 for its dependency specification, but it incorrectly continues to use Archive::Tar::Minitar, which has been deprecated since 2017.
https://github.com/search?q=repo%3Apuppetlabs%2Fpuppet+archive%3A%3Atar%3A%3Aminitar&type=code
There will be no further updates to v0.

Note that puppetlabs/forge-ruby does not specify the minitar dependency correctly and also uses Archive::Tar::Minitar: https://github.com/search?q=repo%3Apuppetlabs%2Fforge-ruby+(Archive%3A%3ATar%3A%3AMinitar+OR+minitar)&type=code

Thanks for the heads up @halostatue, we'll take a look


models/issue.go NewIssue() could fail due to duplicate key

The current code for models/issue.go roughly does:
func getMaxIndexOfIssue(e Engine, repoID int64) (int64, error) {
    var (
        maxIndex int64
        has      bool
        err      error
    )

    has, err = e.SQL("SELECT COALESCE((SELECT MAX(`index`) FROM issue WHERE repo_id = ?),0)", repoID).Get(&maxIndex)

   [...]

    return maxIndex, nil
}

func newIssue(e *xorm.Session, doer *User, opts NewIssueOptions) (err error) {
	opts.Issue.Title = strings.TrimSpace(opts.Issue.Title)

	maxIndex, err := getMaxIndexOfIssue(e, opts.Issue.RepoID)
	if err != nil {
		return err
	}
	opts.Issue.Index = maxIndex + 1

   [...]

	if _, err = e.Insert(opts.Issue); err != nil {
		return err
	}

   [...]
}

It's possible that under heavy loads (e.g. a small server) two sessions may attempt to use the same index number to create issues; one of the two will fail due to a race condition:

User A submits the form creating an issue for repo abc.
User B submits the form creating another issue for repo abc.
At issue.go, in the context of user A's request, getMaxIndexOfIssue() returns 99. It will be used at step 5.
At issue.go, in the context of user B's request, getMaxIndexOfIssue() also returns 99. because at the moment that's the latest issue for the repo. It will be used at step 6.
At issue.go, in the context of user A's request, newIssue() inserts A's issue with Index: 100.
At issue.go, in the context of user B's request, newIssue() attempts to insert B's issue with Index: 100 and fails with duplicate key error, because 100 was used at step 5.

This will of course depend on the database locking model.
I think the best way to handle this is to use a mutex, thus covering most scenarios.

So a mutex would actually be at the wrong level - it needs to be at the DB level (Consider clusters). There's only really three options to fix it:

Just retry and hope you can get it right once - Adding a mutex to the web server level will help prevent hitting the problem if you're not in a cluster but should also reduce the likelihood of hitting the problem if you are.
Use a DB stored procedure to generate the issue number. (Not guaranteed to completely solve this issue on all DBs IIRC.)
Change the transactional isolation mode on your DB. (Also not guaranteed to completely solve the issue on all DBs IIRC.)

I think option 1 is probably a reasonable thing to do, but someone with more specific DB knowledge could look into how much work it would be to add 2 or 3 - but I think you still need retries when I last looked into this.

If we want finish that on DB level, it's a complex SQL and specific for different databases. So I think the simple and dirty method is just retry it 3 times if it failed because of exist issue number.

Ultimately, I would like to consider dropping the index column from the table. It should be something deducible from the other columns.

This should be resolved  by #8309


Node crashed

Hello
I stepped away from the computer and when I returned (computer was continuously on) I found that one of my nodes on Google Cloud had crashed with the following output:
numerous lines of:
"Called from file "list.ml", line 117, characters 24-34","
followed by
"Called from file "src/lib/transition_frontier/transition_frontier.ml", line 199, characters 4-157",
"Called from file "src/lib/transition_router/transition_router.ml", line 192, characters 4-136",
"Called from file "src/lib/transition_router/transition_router.ml", line 261, characters 6-129",
"Called from file "src/deferred0.ml", line 56, characters 64-69",
"Called from file "src/job_queue.ml" (inlined), line 131, characters 2-5",
"Called from file "src/job_queue.ml", line 171, characters 6-47"
]
}

coda_crash_report_2020-12-18_11-56-53.921016.tar.gz

I've opened PR #7115 to detect and handle this error.
I suspect this is a result of a 'dirty' shutdown, which corrupted one of the databases; you should be able to work around this error by manually clearing ~/.coda-config until that fix is merged and released.


organize demo version of regression pipeline job

combine all demo tests into the single regression suite to be able to check core tests using single run

Carina-Demo-Regression-Pipeline is created


build images for py3.11

What does this PR do?
adding some more images

Before submitting

[x] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?
Make sure you had fun coding 🙃

:books: Documentation preview :books:: https://torchmetrics--2138.org.readthedocs.build/en/2138/

I fixed some mypy issues but not sure why it was raised/flocked in this "unrelated" change/PR


[WIP] Add option to use Pytorch

Model runner currently only supporst keras. We would like to use
either keras or pytorch here.
Current behavior was essentially checking the name of the framework
and importing a module with a correctly formatted name. This change
essentially does the same thing, adding a second conditional
statment in import_pkg().

Just opening this PR as I am testing these changes. Thought it would be good for us to have a reference point in case we need to loop others in.

Do we need to do anything with line 146?: runner_utils.keras_clear_session(framework)
Either add a conditional here or add a pytorch version if necessary?

Ah, gross. You are right. Why would we make a variable called framework if they just meant to hard code keras in there? haha


docs: corrects minor typos and grammar in README

Hey team, I went through the README this morning and came across a few minor errors. I did some proofreading for you and have submitted these changes here. Hope that helps!

Awesome! Thanks a lot!!!

Thanks guys!


Added data dumps from system running FreeNAS-11.2-U8

Added some data pulled from a system running FreeNAS 11.2. Useful for documenting data to expect from the API, and maybe also for data to serve when testing with fakeserver.

Closing this stale PR


Dependency Review

Does this action work together with GitHub's Dependency Review?
See:

https://docs.github.com/en/code-security/supply-chain-security/understanding-your-software-supply-chain/about-dependency-review
https://github.com/actions/dependency-review-action

There is a note in Dependency review enforcement stating that it should work:

Note: The dependency review API and the dependency submission API work together. This means that the dependency review API will include dependencies submitted via the dependency submission API. This feature is currently in public beta and subject to change.

If anyone knows how to set them up, we could add it as an example in the README.md.

A basic example (in my mind) should be) — would you agree?
name: Dependency Review

on:
  push:
    branches:
      - master
  pull_request:

permissions:
  contents: write # for submitting dependency graph

jobs:
  dependency-review:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - uses: scalacenter/sbt-dependency-submission@v2
      - uses: actions/dependency-review-action@v3
        if: github.event_name == 'pull_request'

Here is my example live:

https://github.com/MPV/github-dependency-review-for-scala/pull/1

However I haven't gotten it to work yet, as actions/dependency-review-action prints things as:

No snapshots were found for the head SHA d4763439a5e5d2b08ddc3cf0d01ab3debb988654.
Dependency Changes being empty.

...as seen here:

https://github.com/MPV/github-dependency-review-for-scala/actions/runs/6767526427/job/18390299078?pr=1

I have a theory what makes it not work in PRs, as mentioned in upstream issue (to be confirmed):

Looks to me it’s using the GITHUB_SHA:

https://github.com/search?q=repo%3Ascalacenter%2Fsbt-dependency-submission sha&type=code

Should it be using something like github.event.pull_request.head.sha instead (if called from a PR)?
As per:

https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request

Gradle actions have/had the same issue, as described here:

https://github.com/actions/dependency-review-action/issues/545#issuecomment-1714344904
https://github.com/hfhbd/ComposeTodo/pull/797

If I understand correctly we should use github.event.pull_request.head.sha, when it is available,  instead of GITHUB_SHA. @MPV would you like to contribute?

Seems workaround doesn't work:

https://github.com/MPV/github-dependency-review-for-scala/pull/5

...so some kind of code change here will be needed here, not sure which exact change.

I am not sure you can override GITHUB_SHA to some custom value. We should better fix sbt-dependency-submission to read the value from a different env var.

Yeah they describe it in the first sentence here too:

https://docs.github.com/en/code-security/supply-chain-security/understanding-your-software-supply-chain/about-dependency-review#best-practices-for-using-the-dependency-review-api-and-the-dependency-submission-api-together

If I understand correctly we should use github.event.pull_request.head.sha, when it is available,  instead of GITHUB_SHA. @MPV would you like to contribute?

I don't have any idea how to change this repo to get it to work. Ideas?

I made a PR for seeing if the event is a pull request and then using the correct sha. See if that works for you @MPV.


Absolute date at HTTP API query parameter "from" and "until" doesn't work properly when today is February 29th

FOUND IN: graphite v0.9.10
USE CASE:
Suppose that today is February 29th 2016.
Try to call HTTP API to retrieve last year (2015) statistics by using query parameter "from" and "until".
Following error will be thrown:
Traceback (most recent call last):
  File "/opt/python2.7/site-packages/django/core/handlers/base.py", line 111, in get_response
    response = callback(request, *callback_args, **callback_kwargs)
  File "/opt/graphite/webapp/graphite/render/views.py", line 45, in renderView
    (graphOptions, requestOptions) = parseOptions(request)
  File "/opt/graphite/webapp/graphite/render/views.py", line 239, in parseOptions
    fromTime = parseATTime( queryParams['from'] )
  File "/opt/graphite/webapp/graphite/render/attime.py", line 45, in parseATTime
    return tzinfo.localize(parseTimeReference(ref), daylight) + parseTimeOffset(offset)
  File "/opt/graphite/webapp/graphite/render/attime.py", line 94, in parseTimeReference
    refDate = refDate.replace(year= int(ref[:4]))
ValueError: day is out of range for month

This issue can be fixed by modifying
/opt/graphite/webapp/graphite/render/attime.py, line 94: at following part:
  elif len(ref) == 8 and ref.isdigit(): #YYYYMMDD
    refDate = refDate.replace(year= int(ref[:4]))

    try: # Fix for Bug #551771
      refDate = refDate.replace(month= int(ref[4:6]))
      refDate = refDate.replace(day= int(ref[6:8]))
    except:
      refDate = refDate.replace(day= int(ref[6:8]))
      refDate = refDate.replace(month= int(ref[4:6]))

become:
  elif len(ref) == 8 and ref.isdigit(): #YYYYMMDD
    refDate = refDate.replace(day=1)
    refDate = refDate.replace(year= int(ref[:4]))

    try: # Fix for Bug #551771
      refDate = refDate.replace(month= int(ref[4:6]))
      refDate = refDate.replace(day= int(ref[6:8]))
    except:
      refDate = refDate.replace(day= int(ref[6:8]))
      refDate = refDate.replace(month= int(ref[4:6]))

Yep, looks like need to be fixed in 0.9.x and master too

I refactored a few things in 0.9.14/15 w/r/t attime - this may already be fixed.


Prevent sharing actions from creating new instances of the App

On Android, when any other application uses the system share menu to open and send data to the Application, a new instance is created.
A common way to prevent it would be just forcing singleInstance or singleTask in AndroidManifest.xml:
<activity
...
android:launchMode="singleInstance"
...>
</activity>

But when this is set the following error is thrown:
Attempt to invoke virtual method 'android.content.Intent android.app.Activity.getIntent()' on a null object reference
getSharedText
    ShareMenuModule.java:35
invoke
    Method.java
invoke
    JavaMethodWrapper.java:374
invoke
    JavaModuleWrapper.java:162
run
    NativeRunnable.java
handleCallback
    Handler.java:751
dispatchMessage
    Handler.java:95
dispatchMessage
    MessageQueueThreadHandler.java:31
loop
    Looper.java:154
run
    MessageQueueThreadImpl.java:194
run
    Thread.java:761

Add to your app android manifest:
android:documentLaunchMode="never"

I didn't look at it deeply again yet, but when I do, I will take a look at those two commits: https://github.com/pedroordep/react-native-share-menu/commit/156ab35e09f6e8e8ffd7a1f444341b2e537ed4aa and https://github.com/Minds/react-native-share-menu/commit/64bb761db4b74f184aac1f9c8ddb37202d8922fb... you can look at them too :)


Add support for Google Vertex AI models (Bison and Gecko)

Google has recently introduced a new machine learning platform called Vertex AI, which includes several interesting models such as Bison and Gecko (learn more here). These models have potential for further expanding the capabilities of the 'outlines' project.
I propose updating the 'outlines' project to incorporate Google's Vertex AI models (Bison and Gecko) into the toolset, allowing users to leverage the power of these cutting-edge generative AI models.
I would be happy to contribute by submitting a pull request to implement this feature. Do let me know if this has community and maintainer support, and any additional requirements or suggestions for implementing this update.
Looking forward to the community's feedback and collaborating on this exciting enhancement!

I would be interested in support for Google Vertex AI models!

We can support these models in the same way we support OpenAI models. However, from what I see in the documentation unlike OpenAI it is not possible to send a token mask over via the API, which means that the constrained generation features won't be available. They would therefore only be usable for simple completion.


✨ feat: supported updateEmail (#860)

Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for new/updated/fixed functionality.
[ ] I have updated/added dartdoc comments with ///.

Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

Related Issues

Codecov Report

Merging #866 (656bd01) into main (32f59cf) will increase coverage by 10.12%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@             Coverage Diff             @@
##             main     #866       +/-   ##
===========================================
+ Coverage   81.54%   91.66%   +10.12%
===========================================
  Files         247       18      -229
  Lines        3105      420     -2685
===========================================
- Hits         2532      385     -2147
+ Misses        573       35      -538

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


Allow custom header for new blobs

We're using the latest feature that allows us to change file/blob extensions to write log CSV files/blobs (don't ask why) and it would be really nice if we were able to write a custom header, containing the column names, to each new blob that gets created.
Detecting that a new blob gets created is pretty easy, you just need to check that the result of appendBlobClient.createIfNotExists() has the field succeeded set to true (see docs).
Simply being able to pass a header, eg. header: "My custom header", in the options would be enough and would really help.
Edit: changed "file" to "blob"

What do you think?

I think that would be perfect 👍

I will include this functionality in release 1.4.3

Great! Will try this first thing tomorrow. Thanks.

Forgot to give an update: works perfectly, thanks!


Custom TFS check-in notes are not exported to commit message

TFS allows check-in notes to be added to changesets. Git-tfs (v0.25.1) supports only the default TFS check-in notes 'Code Reviewer', 'Security Reviewer' and 'Performance Reviewer'. Any other (custom) check-in notes are not exported and thus missing in the git commit message.
Git-tfs should export all (custom) TFS check-in notes.

Please assign me. I'm gonna fix this issue.

Thanks. We are waiting your pull request ;-)

The implementation for TFS to git is ready. What should we do with git to TFS? I don't expect it to work already. It might also not be a straightforward implementation. It's also not part of this issue. It's fine to me to only support TFS to git.

Feel free to create the pull request,  now. That way, we would be able to
discuss about that on the PR.
I'm OK for just supporting Tfsvc to git for the moment. The one who need
the other way will have to contribute ;-)

I think that's merged so we could close....


Finding MAC Address

@suraj7086  contribute here https://github.com/Ishaan28malik/Get-Dev-Resources


Thank you

This is not an issue, just want to say thanks. Thank you for compiling this information. I will have this repo in my favorites and share it with anyone asking something about go. Thank you so very much.

I also want to thank you. This is a really beautiful repo. I am reading epub format. keep rocking!

I also want to thank you. This is a really beautiful repo. I am reading epub format. keep rocking!

Thank you for this. One question where do i get the epub format?

friend compiled into an ebook and shared with me.

Thanks for all the support. I really appreciate it!

Thank you for making this! It's really helpful -- go Pokemon go!

Thank you for making this! It's really helpful -- go go go!

friend compiled into an ebook and shared with me.

Can you share with us compiling method ?

friend compiled into an ebook and shared with me.

Plllls, share with us :)
And thanks y'all for this repo.

awesome repo! thank you for your efforts!

Thank you, and yes, I also would like to have an e-book

I've put together an ebook here! Happy reading!

Thank you.
It's easy to read but doesn't miss any important information.
I am thinking to have study group with your PDF book, and also translate to Korean with my study mate, though I am not guarantee the quality.
Of course not for the monetary reason, but for the Korean how would get helped with Korean translated version of this book.
If you think it is okay, could you guide me how to do that?
Thanks in advance

Thanks for reaching out @nicewook. I like your idea. One thing you could do is to come up with a rough plan on how do to so and send it to me. We can work together then and validate it.

Thanks for reaching out @nicewook. I like your idea. One thing you could do is to come up with a rough plan on how do to so and send it to me. We can work together then and validate it.

Let me share you a brief plan.
Pre-Preparation

Personal reading
Make agreement with you for the rough rule
Set the tool(GitHub, Google Docs, etc.) accordingly original text format (PDF, Markdown, or Google Docs?)

Preparation

Make rule and schedule for the translation
Gather study member and share the rules

Study and Translation concurrently
Study

8 to 10 weeks, every Tuesday night 07:30PM-09:30PM in Korean Time (Online with Zoom)

Translation

Pre-translation: English to Google translation
Every twe weeks, three times (means 6 or more weeks)

the first week. one person in charge translate each part who is in charge, and PR(pull request)
the second week. At least 2 reviewer check the PR and merge

If possible, I'd like to have a channel to communicate with you. anything you like (ex. email, messenger, etc.)

Thank you for your efforts! This repo is awesome!

Feel free to reach out via the email I have in my profile, @nicewook.


avoid calling head on empty list

The following description is based on faker-<IP_ADDRESS>. Version <IP_ADDRESS> on github doesn't work for me: *** Exception: ../data/en.giml: openFile: does not exist (No such file or directory).
The following is the relevant part of the current implementation of Faker.Internet.safeEmail:
-- | Returns random username, i.e. "k.johnson"
userName :: Faker String
userName = do
    fName <- N.firstName
    lName <- N.lastName
    let loweredFName = loweredLetters fName
        loweredLName = loweredLetters lName
    ind <- randomInt (0,2)
    return $ case ind of
               0 -> loweredFName ++ "." ++ loweredLName
               1 -> head loweredFName : '.' :loweredLName
               _ -> head loweredFName : loweredLName

loweredLetters :: String -> String
loweredLetters str = map toLower $ filter isLetter str

If ind > 0 and loweredLetters fName == "" then error is triggered in the implementation
of Prelude.head. The following ghci session demonstrates that this can happen with faker-<IP_ADDRESS> and gimlh-<IP_ADDRESS>.
λ import Data.Char
λ import Faker.Name -- version <IP_ADDRESS>
λ import Control.Monad
λ let loweredLetters str = map toLower $ filter isLetter str
λ replicateM_ 10000 $ firstName >>= \x -> when (null $ loweredLetters x) (error "empty first name")
*** Exception: empty first name

Here is another GHCI session that uses safeEmail directly, but takes longer to trigger the issue:
λ import Control.DeepSeq
λ import Control.Monad
λ import Faker.Internet -- version <IP_ADDRESS>
λ replicateM_ 50000 $ safeEmail >>= \x -> x `deepseq` return ()
*** Exception: Prelude.head: empty list

The proposed fix simply retries calling userName until loweredFName is not null. In practice at most a single retry should be needed and even that case is rare.

Great! Thank you! Version <IP_ADDRESS> is not working because of this local testing environment as I'm very beginner in haskell and didn't used to other ways of running my program in development mode: https://github.com/gazay/faker/blob/master/src/Faker/Utils.hs#L66. In production mode I using https://github.com/gazay/faker/blob/master/src/Faker/Utils.hs#L36 to get right paths to data files. But as I test my functionality just in ghci in src/Faker folder it will not work anywhere else.


Plugin support

(Suggestion, im not sure how to add the label)
Im not experienced with go at all, but I suggest getting a good plugin system from the start.
After a little googling I came across this , which might be good.

The Plugin Stuff in go is not really good atm.
I tried many ways, and allways ended up with pipes to other processes or rest calls.
I suggest you check out io.Pipe() and exec.Command() and start the plugin as exec and pipe input data into the executable and receive answers from the executable in a predefined format.

The best bet would be the go plugin definition by hashicorp, it's also used in famous tools like terraform and packer.


Add another challenge in dotproduct proof?

In paper Halo: Recursive Proof Composition without a Trusted Setup , section 3.1:

And in paper Bulletproofs: Short Proofs for Confidential Transactions and More, protocol 1:

Do we need to add another challenge x in DotProductProofLog and DotProductProofLog, as shown above, to avoid the Prover cheat?

@3for Thanks for bringing this to my attention! I checked with the authors of [37] and indeed, we need to add the additional challenge.

@3for Thanks for bringing this to my attention! I checked with the authors of [37] and indeed, we need to add the additional challenge.

Glad to help~

Glad to help~


expected behaviour deleted files

hi,
i run the latest npm dat version:
npm uninstall -g dat
npm install -g dat

mkdir send
echo 'hi' > send/foo.bar
dat share send/

# another screen
mkdir receive
dat clone key receive/
cat receive/foo.bar # hi

but now i delete send/foo.bar and dat pull receive/, but the receive directory still contains the file with "hi".
now i do echo 'hi hi hi' > send/foo.bar and pull the changes from the receive folder, and foo.bar now contains 'hi hi hi'.
is this the expected behaviour?
i guess what i am asking is:

how does dat deal with deleted files?
how does dat deal replaced files? like, can i cp foo.bar send/foo.bar? currently, the replaced file in send/ changes, while receive/ does not after pull.

thanks a lot!

how does dat deal with deleted files?

It doesn't 😄 . The current version does not do anything about deleted files on either end.

how does dat deal replaced files? like, can i cp foo.bar send/foo.bar? currently, the replaced file in send/ changes, while receive/ does not after pull.

Right now once a file is added there is no way to remove that metadata entry. So you could add it in another place send/foo.bar but it'd still be in the original location too.
However, we should be supporting the full range of file system actions better in the next few weeks.

ah, that explains a lot. my use case is the following: i put a database dump in a dat shared directory, then changes to the database are made. ideally, i would like to dump the updated database json in the same dat folder, so that people pulling it can then import tge database changes.
how would you recommend to do that with dat without creating many different database dumps?
thanks a lot!

how would you recommend to do that with dat without creating many different database dumps?

Updating the same file repeatedly will work fine. You can dump to db-dump.json regularly and all the clients will automatically download the latest file.
Right now the CLI does not store the history of this file, so it'll be impossible to roll back db dumps. But it'll be easier to store the full history or selectively sync specific files in the future.
Your Zoo projects looks cool! Excited to see where that goes.

I tried dumping to the same file and dat works like a charm, thanks, that was easy :) I am looking forward to those dat changes in the future.
And zoo wouldn't be nearly as convenient without dat, so a huge thanks for all the effort you guys put into such a tremendeously useful project.


wasmtime --env drops stdout

given an example that displays output on stdout - like a "hello world" application - wasmtime --env appears to drop stdout entirely.
><> cat src/main.rs
fn main() {
    println!("hello world!");
}
><> cargo build --target wasm32-wasi
    Finished dev [unoptimized + debuginfo] target(s) in 0.00s
><> wasmtime ./target/wasm32-wasi/debug/wasm32-wasi-demo.wasm
hello world!
><> wasmtime --env 'FOO=bar' ./target/wasm32-wasi/debug/wasm32-wasi-demo.wasm
><>

Tested with both 0.8.0 and the development release available on the releases page. Both demonstrate the same issue.

My coworkers cannot appear to reproduce the issue. I will assume that this happens to be something on my end, and will continue investigating as I find time.

I'm also unable to reproduce this with current out of master.
What's the environment you're reproducing this in?

Hi @peterhuene!
Here's what I know so far:
><> wasmtime --version
wasmtime 0.7.0
><> uname -a
Linux ultralisk 5.0.0-37-generic #40~18.04.1-Ubuntu SMP Thu Nov 14 12:06:39 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

This was with a fresh linux .tar.xz build from the dev release channel.
My terminal is plain ol' /bin/bash, using the Terminal app on Ubuntu.
Running with RUST_LOG=debug wasmtime ... and reviewing the thread dumps shows no difference in output between wasmtime and wasmtime --env.
I'm going to assume there's something on my end interfering with the terminal output. Trying to isolate the issue in a linux container to reproduce the issue.

Yep. Couldn't reproduce the issue from within a linux container. Closing as there's nothing actionable here on wasmtime's end. I'll update the ticket if I find anything more concrete for others. Thanks for reaching out!

Is it actually running anything? You can put in a infinite loop to see if it completes.
You can also check if stderr has same issue - and print in infinite loop to see if there is a flush issue.

Good call @rene-fonseca. I just checked with another sample that runs in an infinite loop. The program exits immediately, so it appears that nothing is being executed.

What's the exit status?  Perhaps it's signaling?  Can you run it under gdb/lldb?

exit code 71.
><> wasmtime --env 'FOO=bar' target/wasm32-wasi/debug/helloworld-demo.wasm
><> echo $?
71

a document on linux system errors indicates this would be a protocol error: EPROTO.

From /usr/include/sysexits.h, this actually resolves as an OSERR. I'll see what I can find under gdb/lldb but it does look like a localized issue.
#define EX_OSERR        71      /* system error (e.g., can't fork) */

In this case it's a process exit status, so it's not an errno value. 71 is sometimes EX_OSERR.
It might be wasm's startup code, which uses EX_OSERR in the init code to report unlikely failures like __wasi_environ_get etc. failing, or malloc failing before main. It's possible that if your environment doesn't have much memory, the first call to memory.grow prompted by the first malloc call could fail. If so, updating to a new wasi-libc may help, as we avoid calling memory.grow until we actually need it.
Beyond that, we should consider giving wasmtime verbose messages, which could be used to indicate things like "I just failed a memory.grow call", so make it easier to debug these kinds of problems.

I happened to hit this as well, and it turned out to be a bug in wasi-libc -- see https://github.com/CraneStation/wasi-libc/pull/159. Keeping this issue open because we still should add better ways to debug these kinds of errors.


Get type fe: issue #145

References issue #145
What I did in this PR:
src/components/GetDetails/index.js

stopped importing itemType from data.json.
cleaned up selected_brands from dummy data.
returned Clarifai's items as objects with a name, itemType and an id.
removed unnecessary else statements.
setState with airtable's itemTypes and the current state (Clarifai's items).
cleaned up code from console.logs.

client/src/components/Shared/Form/index.js

modified items' values in the dropdown to be objects with name, itemType, and id.

client/src/components/Shared/Popup/index.js

modified items' values in the popUp to be objects with name, itemType, and id.

server/controllers/getTypes.js

added a name's property in each itemType object.

So you get all item types from Airtable, but when you add the item to Airtable you do not use any relationship between items and types tables "it's still working as it was when using mock data (in JSON)"


Touchevents in Chrome Beta & Dev for Android do not fire

Take a look at http://jsbin.com/dehupokofu/2/edit and click on the triangle.
var map = L.map('map').setView([51.5, -0.09], 13);

L.tileLayer('http://{s}.tile.osm.org/{z}/{x}/{y}.png', {
    attribution: '© OpenStreetMap contributors'
}).addTo(map);

var polygon = L.polygon([
    [51.509, -0.08],
    [51.503, -0.06],
    [51.51, -0.047]
]).addTo(map).on("mousedown touchstart", function (e) {
  window.alert(e.type)
})

On my android device (running Nougat 7.0):
Firefox (Stable, Beta & Aurora) fires a mousedown event, no touchstart event though.
Chrome (Stable) fires a mousedown event, no touchstart event though.
Chrome (Beta & Dev) doesn't fire any events.
I guess chrome broke backward compatibility by not firing mouse events anymore and touchevents don't seem to be supported natively by LL?
P.S. Example uses LL 1.0.1 and I also tested LL-master with the same results

@Ceremony64 I am running Nougat 7.0 and Chrome Beta 55.0.2883.63 and I have tested your link. Tried to click on the triangle and got a mousedown event. Is there anything I am missing? Can you test again?

Seems to have been fixed with the new versions of Chrome! Thanks!


Subsystem - Slides

Log of Slides Programming Progress

subsystem code is done, need to implement in tele -- will do once we're sure which controller input we want to map it to


Pathfinder Community Sheet v1.81 - darkmode changes

Changes / Comments

added some changes to help with the dark mode option.

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Incorrect information for Azure Devops Docker Compose azureContainerRegistry

In the docs is clearly says Name of the Azure Container Registry while it really requires the Azure Container Registry Login server.
In the docs:
azureContainerRegistry(Azure Container Registry) | (Required) Name of the Azure Container Registry.
General feedback:
The amount of time my team wastes on misleading, ambiguous, or incorrect documentation is adding up. Azure's poor documentation is a hidden cost that Microsoft passes to its customers and I hope that Azure starts to appreciate and corrects this massive deficiency.
Thank you

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

ID: d767aebb-620e-9ad3-e906-3e2bafa02ee9
Version Independent ID: c8cb5a48-ccdf-778c-1547-a10db74cb56c
Content: Docker Compose task - Azure Pipelines
Content Source: docs/pipelines/tasks/build/docker-compose.md
Product: devops
Technology: devops-cicd
GitHub Login: @azooinmyluggage
Microsoft Alias: atulmal

@RothNRK Thanks for the feedback. Seems like you are referring the container registry types. Based on the types, there are arguments detailed in the document.
If you are using ACR,

then you have to use the azureContainerRegistry as your parameter.

If you are using any other container registry types,

then you have to use the containerregistrytype as your parameter.

The document already has a sample example snippets which covers both the scenarios. Hope this helps.

@RothNRK Thanks for the feedback. Seems like you are referring the container registry types. Based on the types, there are arguments detailed in the document.
If you are using ACR,

then you have to use the azureContainerRegistry as your parameter.

If you are using any other container registry types,

then you have to use the containerregistrytype as your parameter.

The document already has a sample example snippets which covers both the scenarios. Hope this helps.

The document has been updated and I'll be closing the issue. Thank you for the feedback and feel free to open a new issue if you have any more feedback.


Search - Submit button is not disabled in Search when "Prevent self-approvals" is enabled

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.75-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: Exp
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Device used: Mac 15.0 / Chrome
App Component: Search
Action Performed:
Precondition:

Delayed submission is enabled.
Admin is the approver.
"Prevent self-approvals" is enabled.

Submit an expense.
Note that Submit button is disabled.
Go to Search.
Click Submit button on the same expense in Step 3.

Expected Result:
Submit button should also be disabled in Search when "Prevent self-approvals" is enabled.
Actual Result:
Submit button is not disabled in Search when "Prevent self-approvals" is enabled.
User can still submit expenses in Search.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[x] iOS: Standalone
[x] iOS: HybridApp
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/f8089a6d-ab47-4abd-861f-16800e06a0ff
View all open jobs on GitHub

Not repro on production
https://github.com/user-attachments/assets/3be6cb39-da04-48b8-b145-192a42c78a3e

Looking into it

Fixed in staging

Sorry @allgandalf for the delay, I was OOO the past few days due to holiday travel.
Here's the Upwork contract: https://www.upwork.com/nx/wm/offer/105455673. Please let me know once you've accepted the contract and I will complete payment. Thanks!

All paid, thanks again!


Image Alt progress stuck at 0% does not progress

Win 10 x64

You'd need to open the dev tools in Koa11y to see if there are any errors being thrown. Are you using Koa11y 2.0.0 or 3.0.0.
If it is version 2, you can just right-click the Logo in the app. If it is version 3, we've stopped shipping the dev tools with the app to reduce the file size of the distribution copy. So you'd need to do download and install Node.js, then download the source code of Koa11y 3, unzip it, navigate to that folder in command prompt and run npm install (will take a minute or two) and then npm start.

Or you can paste in the array of image URL's you were using and I can try on my end.


cargo fmt + fix UB

Fixes #9. I didn't care to repair the unused functions replace_slice and insert_slice, which are not exposed anyway.
cargo fmt wasn't intentional, hx formatted automatically and I also didn't care enough to reapply the fix from scratch.

@Geal is this package still maintained? A merge and a version bump shouldn't eat too much of your time I presume


Fixed EINVAL error with fs.utimesSync() due to incorrect Unix timestamp

This should address issue #17

@mseminatore: Thanks for the PR. Two things:

Since I don't work at LinkedIn anymore, we'll have to track down someone who can merge into this repositor. Or, you can open up a PR on my fork of this project.

I'm not sure I understand why this problem just became apparent now. Can you provide a test case that demonstrates the problem by failing before your change and passing afterwards?

@avik-das I can reproduce this using the following code:
// utimes-test.js
var fs = require('fs');
var now = Date.now();

fs.utimesSync('a.txt', now, now);
console.log("Success!");

c:\sepia-test>echo hello > a.txt
c:\sepia-test>node utimes-test.js
I am not certain why this wasn't an issue either.  Maybe the prior code works on other OS's or different Node versions.  In any case the current NodeJS documentation  is clear that Date.now() can't be used as is for atime and mtime.
Snipped:
Note: the arguments atime and mtime of the following related functions follow these rules:

- The value should be a Unix timestamp in seconds. For example, Date.now() returns milliseconds, so it should be divided by 1000 before passing it in.
- If the value is a numeric string like '123456789', the value will get converted to the corresponding number.
- If the value is NaN or Infinity, the value will get converted to Date.now() / 1000.

@avik-das OK, PR to your fork sent as well.

@avik-das Do you have any contact with anyone who can accept this PR?  It would be nice to have this fix integrated.

@mseminatore - This repo appears to be abandoned.  A couple of us created a fork called "replayer" to start accepting commits.   You'll need to rename sepia=>replayer.
Could you submit your PR there?

@aneilbaboo I've not done this before.  Can you share the specific steps that I would need to take?  I assume it is more than simply renaming my forked repository.  That won't connect my fork to your repo though will it?
@avik-das Have you had any luck contacting the LinkedIn owners?

@mseminatore - That won't work.
I think this is right, but haven't checked it:

fork replayer & clone replayer

add a remote pointing at your sepia repo
git remote add mysepia git@github.com:msemiatore/sepia.git

checkout your sepia master branch to a new local branch in your local replayer repo, giving it a new name, to avoid confusing it with the replayer master branch.  E.g., call it the einval branch.
git fetch mysepia && git checkout -b einval --track mysepia/master

checkout the replayer master branch
git checkout master

merge your branch into master and enjoy fixing the merge conflicts
git merge einval
This should mostly be renamings of sepia => replayer in your code.

issue a pull request

If you're having trouble or would just prefer me to do it, that's fine too.
I took a look at the changes.  They seem fine.


Latex updates to Googlenet PR

Have added some  latex  updates that are required for PR doc creation by merging https://github.com/johko/computer-vision-course/pull/223/

Closing. these changes are already created as a PR on johko.  https://github.com/johko/computer-vision-course/pull/223 and https://github.com/johko/computer-vision-course/pull/224 . We should wait till these PRs are merged and sync from there  instead.


Update Careers page copy and functionality changes

@bstein Tagging you because you recently worked on this but feel free to ignore if you're too busy

@nring I added a comment about the magic number but otherwise this looks good to me!


Rename files

Describe your changes
fixes: #1732
Related issue number or link (ex: resolves #issue-number)
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@shivam-sharma7 If you feel hard to achieve this, is it okay to assign the issue #1732 to someone else?


Test discovery fails with pytest-libfaketime

Environment data
VS Code Version: 1.54.2
Commit: fd6f3bce6709b121a895d042d343d71f317d74e7
Date: 2021-03-11T00:56:09.783Z (4 days ago)
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0

Extension version (available under the Extensions sidebar): v2021.2.636928669

Python version (& distribution if applicable, e.g. Anaconda): 3.9.2

Type of virtual environment used: pipenv

Value of the python.languageServer setting: Jedi

Expected behaviour
Tests are correctly discoverd
Actual behaviour
Test discovery fails with when pytest-libfaketime is installed
Steps to reproduce:
I created a small repo to easily reproduce the issue:
https://github.com/giallu/vscodetest

clone the repo and create the virtual env with pipenv install
Discover tests using pytest and root set to . or animals
open the project in vscode: tests discovery and test running works
install pytest-libfaketime with pipenv install pytest-libfaketime
test discovery fails like this:

Test Discovery failed:
SyntaxError: Unexpected end of JSON input

Logs
I am attaching here the Python Test Log window content; I tried in sequence: discover tests, run tests (install pytest-libfaketime), discover tests, run tests.
Interestingly, tests are a success  before and after the library installation, it's only the discovery phase that is failing.
python /Users/giallu/.vscode/extensions/ms-python.python-2021.2.636928669/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir /Users/giallu/vscodetest -s --cache-clear .
============================= test session starts ==============================
platform darwin -- Python 3.9.2, pytest-6.2.2, py-1.10.0, pluggy-0.13.1
django: settings: vscodetest.settings (from ini)
rootdir: /Users/giallu/vscodetest, configfile: pytest.ini
plugins: django-4.1.0
collected 1 item

animal/tests.py .                                                        [100%]

- generated xml file: /var/folders/j8/_m5rss0s475_92j2207sdhxh0000gn/T/tmp-23058OVmOvk3C0G3U.xml -
============================== 1 passed in 0.52s ===============================
python /Users/giallu/.vscode/extensions/ms-python.python-2021.2.636928669/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir /Users/giallu/vscodetest -s --cache-clear .
Test Discovery failed:
SyntaxError: Unexpected end of JSON input
python /Users/giallu/.vscode/extensions/ms-python.python-2021.2.636928669/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir /Users/giallu/vscodetest -s .
Test Discovery failed:
SyntaxError: Unexpected end of JSON input
re-exec with libfaketime dependencies
============================= test session starts ==============================
platform darwin -- Python 3.9.2, pytest-6.2.2, py-1.10.0, pluggy-0.13.1
django: settings: vscodetest.settings (from env)
rootdir: /Users/giallu/vscodetest, configfile: pytest.ini
plugins: libfaketime-0.1.2, django-4.1.0
collected 1 item

animal/tests.py .                                                        [100%]

- generated xml file: /var/folders/j8/_m5rss0s475_92j2207sdhxh0000gn/T/tmp-230583iuiT9nNMlnD.xml -
============================== 1 passed in 0.55s ===============================

Hi @giallu, thank you for reaching out!
What is being printed when you run the discovery command directly in your terminal? python /Users/giallu/.vscode/extensions/ms-python.python-2021.2.636928669/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir /Users/giallu/vscodetest -s --cache-clear .

The output is empty when the library is installed
(vscodetest) ➜  vscodetest git:(master) ✗ python /Users/giallu/.vscode/extensions/ms-python.python-2021.2.636928669/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir /Users/giallu/vscodetest -s --cache-clear .
(vscodetest) ➜  vscodetest git:(master) ✗ pipenv uninstall pytest-libfaketime
Uninstalling pytest-libfaketime...
Found existing installation: pytest-libfaketime 0.1.2
Uninstalling pytest-libfaketime-0.1.2:
  Successfully uninstalled pytest-libfaketime-0.1.2

Removing pytest-libfaketime from Pipfile...
Locking [dev-packages] dependencies...
Locking [packages] dependencies...
Building requirements...
Resolving dependencies...
✔ Success!
Updated Pipfile.lock (536b9f)!
(vscodetest) ➜  vscodetest git:(master) ✗ python /Users/giallu/.vscode/extensions/ms-python.python-2021.2.636928669/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir /Users/giallu/vscodetest -s --cache-clear .
[{"rootid": ".", "root": "/Users/giallu/vscodetest", "parents": [{"id": "./animal", "kind": "folder", "name": "animal", "parentid": ".", "relpath": "./animal"}, {"id": "./animal/tests.py", "kind": "file", "name": "tests.py", "parentid": "./animal", "relpath": "./animal/tests.py"}, {"id": "./animal/tests.py::AnimalTestCase", "kind": "suite", "name": "AnimalTestCase", "parentid": "./animal/tests.py"}], "tests": [{"id": "./animal/tests.py::AnimalTestCase::test_animals_can_speak", "name": "test_animals_can_speak", "source": "./animal/tests.py:10", "markers": [], "parentid": "./animal/tests.py::AnimalTestCase"}]}]

Is there anything I need to do to drop the "needinfo" label? If @kimadeline is waiting for that before looking at this again, it could be a long wait 😅

No there is noting for you to do, don't worry I haven't forgotten about you 🙂

Hi @giallu, thank you for your patience!
pytest-libfaketime writes in stdout, you can see it if you execute the discovery command without the adapter script from the extension:
(15669-pytest-libfaketime) python -m pytest --rootdir /path/to/my/folder/15669-pytest-libfaketime -s --cache-clear animal
re-exec with libfaketime dependencies
====================================================== test session starts =======================================================
platform darwin -- Python 3.9.1, pytest-6.2.3, py-1.10.0, pluggy-0.13.1
django: settings: vscodetest.settings (from env)
rootdir: /path/to/my/folder/15669-pytest-libfaketime, configfile: pytest.ini
plugins: libfaketime-0.1.2, django-4.1.0
collected 1 item

animal/tests.py .

======================================================= 1 passed in 0.39s ========================================================

The adapter relies on the output of this command to discover the tests, as such the re-exec with libfaketime dependencies line is causing the issue.
We have plans to address that with https://github.com/microsoft/vscode-python/issues/11729, feel free to upvote it to help us bump up its priority! In the meantime, the only workaround I can suggest is to find a plugin that doesn't write to stdout.
Closing in favor of #11729.

Thank you. In the meanwhile I did exactly what you suggested and changed to https://github.com/adamchainz/time-machine that does not seem to suffer for the same issue.


quantized models

Does this support quantized models by any chance?

@marsupialtail  yes, I tried , you can quantize the onnx models like this, then use quantized model for inference
from onnxruntime.quantization import quantize_dynamic, QuantType

#quantize encoder
model_input = "temp1/t5-own--encoder.onnx"
model_output = "temp1_compressed_onnxruntime/t5-own--encoder.onnx"

quantize_dynamic(model_input, model_output, weight_type=QuantType.QInt8)

#quantize decoder
model_input = "temp1/t5-own--decoder-with-lm-head.onnx"
model_output = "temp1_compressed_onnxruntime/t5-own--decoder-with-lm-head.onnx"

quantize_dynamic(model_input, model_output, weight_type=QuantType.QInt8)


[test] Update IOR checks

Codecov Report

Merging #2076 (c77fb22) into master (e98078a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2076   +/-   ##
=======================================
  Coverage   86.74%   86.74%
=======================================
  Files          52       52
  Lines        9270     9270
=======================================
  Hits         8041     8041
  Misses       1229     1229

Continue to review full report at Codecov.

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

@jenkins-cscs retry all

I am fixing this PR