Fix monthly issue metrics workflow

Our monthly issue metrics GitHub Actions workflow is supposed to close the issue once created, but fails to do so:
https://github.com/nsidc/earthaccess/actions/runs/11623230080/job/32369942886#step:6:10
Failure was because we didn't specify the repo so it tried to base it on a local git repository, but there wasn't one because we don't need to check out the repository for the rest of the action steps.
I've updated the action to have  issue write permission (needed for the issue  PATCH API request)  as well as specifying the repo, which I confirm works in a directory with no git repository:
$ gh issue close 861 --reason "not planned" --repo nsidc/earthaccess
✓ Closed issue nsidc/earthaccess#861 (Monthly issue metrics report: 2024-10-01..2024-10-31)

I don't think there's much value to a changelog entry here, so I've not added one; happy to add one if anyone thinks otherwise.
Pull Request (PR) draft checklist - click to expand

[x] Please review our
contributing documentation
before getting started.
[x] Populate a descriptive title. For example, instead of "Updated README.md", use a
title such as "Add testing details to the contributor section of the README".
Example PRs: #763
[x] Populate the body of the pull request with:

A clear description of the change you are proposing.
Links to any issues resolved by this PR with text in the PR description, for
example closes #1. See
GitHub docs - Linking a pull request to an issue.

[x] Update CHANGELOG.md with details about your change in a section titled
## Unreleased. If such a section does not exist, please create one. Follow
Common Changelog for your additions.
Example PRs: #763
[x] Update the documentation and/or the README.md with details of changes to the
earthaccess interface, if any. Consider new environment variables, function names,
decorators, etc.

Click the "Ready for review" button at the bottom of the "Conversation" tab in GitHub
once these requirements are fulfilled. Don't worry if you see any test failures in
GitHub at this point!

Pull Request (PR) merge checklist - click to expand
Please do your best to complete these requirements! If you need help with any of these
requirements, you can ping the @nsidc/earthaccess-support team in a comment and we
will help you out!

[ ] Add unit tests for any new features.
[ ] Apply formatting and linting autofixes. You can add a GitHub comment in this Pull
Request containing "pre-commit.ci autofix" to automate this.
[ ] Ensure all automated PR checks (seen at the bottom of the "conversation" tab) pass.
[ ] Get at least one approving review.

📚 Documentation preview 📚: https://earthaccess--867.org.readthedocs.build/en/867/

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.01%. Comparing base (2f49c08) to head (ba84843).
Report is 70 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #867      +/-   ##
==========================================
- Coverage   73.88%   70.01%   -3.87%
==========================================
  Files          31       23       -8
  Lines        2003     1511     -492
==========================================
- Hits         1480     1058     -422
+ Misses        523      453      -70

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


[ISSUE-3857] Support show now() function

Description
ISSUE #3857
Implementation returns the current time of the system.
Format: "yyyy-MM-dd'T'HH:mm:ss.SSSXXX"
e.g :1970-01-01T08:00:00.001+08:00

Hi, thanks for your contribution!
Can you plz submit this PR to master branch? We actually won't add new feature to the rel/0.12 branch any more.

Hi, it is really nice to have a way to check the current server time.
The relational DBMS(s) like MySQL or PostgreSQL use built-in UDF to handle this need. But I since IoTDB doesn't support syntax like select now() (a from clause is required), I can totally understand the way you are doing it.
But since adding a new top-level statement is quite have, I suggest we make most of it and make the SQL API more extendable. Instead of simply shows a system time, maybe we could provide more information like current workload, storage usage, and so on.
In addition, in cluster mode, we should display the information for every single node in the cluster.

How is this operator defined in the cluster module? Because the time of cluster nodes may be inconsistent?


Allow newlines to be escaped in multi-line strings

Currently,
'''
foo
bar
'''

and
'''
foo\
bar
'''

both compile to the same output:
'foo\nbar'`

Normally, a \ at the end of a line effectively means "treat this line and the next one as a single line"; I fail to see why herestrings should be an exception to that rule, particularly since it's often useful to have a long string without newline characters. (I seem to recall there being a previous issue on this, but I'm unable to locate it. #610 is related.)

That sounds reasonable.

Yep -- sounds fine to me. We just would need to make sure that it's consistent, in terms of suppressed heredoc left-hand padding, and other forms of strings ...

Care to point me in the direction of the code that handles \ at the end of lines?
I just noticed that non-heredoc strings actually provide the behavior that I want:
 'foo
 bar'

compiles to
'foo\
bar'

which is effectively equivalent to
'foobar'

So that's cool. But it'd still be nice to be able to use herestrings with the ends of lines escaped.

Here's some starting points.


old keybinding for resourceScheme "vscode-interactive" for interactive.execute

Applies To

[ ] Notebooks (.ipynb files)
[X] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
Shift+Enter doesn't execute the command in interactive window command line anymore, instead, it sends content to a new command line python interpreator.
INFO [KeybindingService]: | Resolving shift+Enter
INFO [KeybindingService]: \ From 16 keybinding entries, matched python.execSelectionInTerminal, when: editorTextFocus && !findInputFocussed && !jupyter.ownsSelection && !notebookEditorFocused && !replaceInputFocussed && editorLangId == 'python', source: user extension ms-python.python.
INFO [KeybindingService]: + Invoking command python.execSelectionInTerminal.

After debugging, it turns out that the when clause of the keybinding Shift+Enter for interactive.execute isn't matched, thus the next one for python.execSelectionInCommandLine is selected.
This is confirmed by inspecting the when clause context:

The resourceScheme is untitled, rather than vscode-interactive as expected.
I workedaround the issue by change the when clause to the following:
!notebookKernel && resourceExtname == '.interactive' || !notebookKernel && resourceScheme == 'vscode-interactive' || resourceExtname == '.interactive' && notebookKernel =~ /^ms-toolsai.jupyter\// || resourceScheme == 'vscode-interactive' && notebookKernel =~ /^ms-toolsai.jupyter\//

VS Code Version
code-server: v4.16.1 Code: 1.80.2 Commit: 2ccd690cbff1569e4a83d7c43d45101f817401dc Date: 2023-07-31T17:15:21.760Z Browser: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/117.0.2045.47
Jupyter Extension Version
2023.3.100
Jupyter logs
No response
Coding Language and Runtime Version
No response
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
None

Hi @amunger, so this will be solved by a newer jupyter extension version? Do you know which version specifically to look for?

I'm not sure which version had the right update. From the issue info, you are on 2023.3.100 and 2023.9.100 was just released last week.


Add Kinesis support

I noticed that Kinesis support was not available, and needed it for my project

Coverage decreased (-0.3%) to 50.984% when pulling fcf8f075fbc395c77e0af411e6882f18686f0e58 on carsonfarrell:master into 82cc1c3e5c1e7957ef71a7f06d92db0290581242 on MonsantoCo:master.

Coverage decreased (-0.2%) to 51.066% when pulling 68a5edba14e5be5c7daa41fac07e5a425aa58dba on carsonfarrell:master into 82cc1c3e5c1e7957ef71a7f06d92db0290581242 on MonsantoCo:master.

Coverage decreased (-0.2%) to 51.066% when pulling 43e315756d9fd4c64727ae5847e39e4398d5b370 on carsonfarrell:master into 82cc1c3e5c1e7957ef71a7f06d92db0290581242 on MonsantoCo:master.

don't worry about the decrease in test coverage, I'll review this shortly.

Cool, thanks for the fast turnaround time on replying.
Was getting questions from coworkers about whether your library was
actively supported or not, this gives me backing to continue using it.

LGTM, thanks for the contribution! I'll try and get a few of these other lingering ones in and cut a new release for you later today.

Awesome, thanks!
On Thu, Nov 3, 2016 at 9:33 AM, T.J. Corrigan<EMAIL_ADDRESS>wrote:

Merged #112
https://github.com/MonsantoCo/cloudformation-template-generator/pull/112
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MonsantoCo/cloudformation-template-generator/pull/112#event-846672104,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACG3E-sZn9Pix_Ki0Mizc8Ayu92kff8Lks5q6gzogaJpZM4Kn59g
.


Cannot pass variables from Controller to Model

I posted an topic on official forum, because I cannot post screenshot here.
http://forum.codeigniter.com/thread-65319.html
CodeIgniter 2 works good but CodeIgniter 3 doesn't.

Use the forum for support. We use github for bug tracking.

Good luck @terrylinooo


Neovim Support

I want to add support for nvim as addition editor supported by this extension. I will create a pull request and create the feature.

Hey @iwansyahp , sure, go ahead.
You will have to make sure to recognize neovim projects, so that the extension is able to open these projects with neovim. But that should work.
Thanks for the initiative! Go ahead :)


checkPassword has no type parameter

https://github.com/oasisprotocol/oasis-wallet-ext/blob/f912f132cc22bec5d70541ddf46b90d7d28c6c6d/src/background/service/APIService.js#L527
blocks #1

@lvshaoping007 do you know if there's any security issue with this parameter not being supported in the implementation?

Here is the type added in the self-test before, there is no risk here, but I will modify this question


Upgrade to Nuxt 3.15.0 led to 500 (Internal Server Error)

Immediately after upgrading to 3.15.0, and restarting dev server, I got the following error:
500
Unexpected token ';'

at createScript (node:vm:269:10)
at Object.runInThisContext (node:vm:317:10)
at ViteNodeRunner.runModule (./node_modules/.pnpm/vite-node@2.1.8_@types+node@22.10.2_terser@5.37.0/node_modules/vite-node/dist/client.mjs:398:19)
at ViteNodeRunner.directRequest (./node_modules/.pnpm/vite-node@2.1.8_@types+node@22.10.2_terser@5.37.0/node_modules/vite-node/dist/client.mjs:381:16)
at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
at async ViteNodeRunner.cachedRequest (./node_modules/.pnpm/vite-node@2.1.8_@types+node@22.10.2_terser@5.37.0/node_modules/vite-node/dist/client.mjs:206:14)
at async ViteNodeRunner.dependencyRequest (./node_modules/.pnpm/vite-node@2.1.8_@types+node@22.10.2_terser@5.37.0/node_modules/vite-node/dist/client.mjs:259:12)
at async ./node_modules/.pnpm/@headlessui+vue@1.7.23_vue@3.5.13_typescript@5.7.2_/node_modules/@headlessui/vue/dist/headlessui.esm.js:11:32
at async ViteNodeRunner.runModule (./node_modules/.pnpm/vite-node@2.1.8_@types+node@22.10.2_terser@5.37.0/node_modules/vite-node/dist/client.mjs:399:5)

IMHO a reproduction is not practical/possible for this error.
The terminal error before the Server Error 500:
ℹ Using default Tailwind CSS file                                         nuxt:tailwindcss 2:56:27 PM
  ➜ DevTools: press Shift + Option + D in the browser (v1.7.0)                              2:56:30 PM

Hello from Tailwind.config                                                                  2:56:31 PM
ℹ Running with compatibility version 4                                                     2:56:32 PM
ℹ Tailwind Viewer: http://localhost:3000/_tailwind/                       nuxt:tailwindcss 2:56:32 PM
✔ Vite client built in 29ms                                                                2:56:34 PM
✔ Vite server built in 933ms                                                               2:56:35 PM
Hello from Tailwind.config                                                                  2:56:37 PM
✔ Nuxt Nitro server built in 2951 ms                                                 nitro 2:56:42 PM
Hello from Tailwind.config                                                                  2:56:42 PM
ℹ Vite client warmed up in 272ms                                                           2:56:42 PM

 ERROR  (node:97761) [DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.
(Use node --trace-deprecation ... to show where the warning was created)

ℹ Vite server warmed up in 5639ms

I noticed the expected 'Vite client warmed up in x ms' did not appear.
Any suggestions? Should I revert back to previous version? Thank you in advance and Happy Holidays.

this seems to be a vite issue (reported https://github.com/nuxt/ui/issues/2971)
you can temporarily resolve by pinning vite to v5

Thank you @danielroe for the quick response. I tried pinning vite to 5 which didn't really work until I added to package.json
"pnpm": {
    "overrides": {
<EMAIL_ADDRESS><EMAIL_ADDRESS>    }

Then it complained that Nuxt 3.15 cannot run with<EMAIL_ADDRESS>So I followed @TheAlexLichter to downgrade Nuxt 3.14 and now the dev server is running again. I thank both of you for such a quick and effective response and wish you and your families and your team Merry Holidays and Happy New Year - salud!!


inform a project when its test suite fails while reverse dep testing

Scenario:

A project Y submits a change in source git
The change triggers testing of rpms Y, Z and W
Test suite of Z fails
source-git informs owners of Z and Y about the failure

All of this is configurable (blocking, non-blocking, via e-mail, via github issue). We should utilize infra of OSCI for reverse dep testing.

closing, this is out of scope short-term


⚠ Bofrost.be has degraded performance

In 462b6d5, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1432 ms

Resolved: Bofrost.be performance has improved in f66fe52 after 6 minutes.


Handle only "service" level changes

...where the package directory doesn't have a changed file, but the service does.
This is relevant for changes to ci.yml or the like.
I decided that because everyone will hit this, I'd put it in the common Get-PrPkgProperties. I originally was going to place this specificity in AdditionalValidationPackagesFromPackageSetFn in each language, to allow folks to select a subselection of packages in the touched service directory. (Eg only include @azure/core-lro, instead of all js packages under core)
Which would you prefer? I have a bad feeling that adding all the packages will begin overaggressive.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


[K3s] Document backupfor a single-server

Request Summary:

Currently, the backup documentation explain alternative to some evolved deployment. The case of raw single-server with sqlite seems not covered.
Details:

The thought process around sqlite is that if you’re running sqlite, you’re running a single control plane. Perhaps a better strategy than sqlite backups is to restore the cluster via a nuke and pave of k3s and reapplying workloads.
I believe we should add this approach and rationale to the backup restore page for completeness.


adding line to trigger rebuild on push to main

This would create a new container every single time we commit (push or merge) to main. I don't think this is the desired functionality

I agree that yes it should be a chain reaction logic. It is not imminent, as the urgent one was the client one, which has been successfully rebuilt.

Then I'm taking care of the propagation logic asap!

What are the EGI trusts anchors exactly used for, btw ?


Pipelines: Move GCP-specific upgrade steps to GCP section of the docs

https://www.kubeflow.org/docs/pipelines/upgrade/

@sarahmaddox could you triage this issue please?
https://github.com/kubeflow/community/blob/master/proposals/issue_triage.md

/assign


Feature/explosive enemy/#86bzebvjn

Enemy Explosive Update:
-VFX Added
-Pool logic added
-Updated gameplay

Task linked: CU-86bzebvjn Explosive-Electric Enemy


Feature: Support for optimizing for specific document types

What feature or improvement do you think would benefit Files?
A setting to automatically show/hide columns based on the folder contents would be very useful and allow us to optimize for different folder types.
Requirements

Replace "details layout columns" setting with an option for "Optimize columns in the details layout"

Add an option for "automatic" (this will switch the columns based on the content of the directory)
Add an option for documents
Add an option for photos
Add an option for git
Add an option for music files
Add an option for custom (will allow the user to choose the columns on their own)

Files Version
v2.4.83
Windows Version
Windows 11
Comments
No response

This can be easily done after columns generator implemented.


Automatic Like New ability usage

Adds Like New ability use once per farm boss round right after wormhole is on cooldown.

I would recommend that you add a random element to this (wait 1-5 seconds), we don't want everyone using Like New at the same time.
In addition, I would use a separate chance variable (maybe even 1% or 0.5%) as Like New is expensive.

Did it, I think.
... If you have any remarks on my coding, please let me know

Great, gonna merge.


[Feature request] Persist RadixSort passes or efficient cache friendly  gather implemenation

My data type is integer and each value contains a left part of a certain size plus a right part.
Right part stays constant during iteration(letter on I'll clarify it)
So an example may look like
llllllrrrrrr
where l - bit from a left part and r - bit from a right part.
My algorithm looks like
loop # iteration over data, the right part stays the same along the iterations
    Radix sort

To not repeat sorting by the right part at the each iteration I've decided to  do the following

Sort data by the right part and get gather index
loop
   apply gather kernel that places each element at appropriate position(make data sorted by the right part)
   Radix sort by the left part

Unfortunately it reduces the performance(at least where data size > 10M). I can explain it by the fact that random memory access on GPU is pretty inefficient and it's faster to do a few less random passes than one completely random pass.
But I still want to improve my implementation(by not doing the same operation multiple times).
Is it possible to save passes itself without doing the whole sort?
Or improve gather kernel in more efficient way? It's ok if it requires auxiliary memory (a will anyway allocate it for RadixSort).
What do you think?

You may have better luck getting input on algorithm design by asking on StackOverflow, but this is beyond the scope of a GitHub issue.
If there's a specific feature request, please file a new issue with a clearer description of what you're looking for.


[Audio Meter] Visualizer not working.

Tried Web and WMP.
(Incidentally, I changed setting from Local to Web while WMP was playing and Rainmeter crashed,
Log Name:      Application
Source:        Application Error
Date:          5/19/2019 07:31:30
Event ID:      1000
Task Category: (100)
Level:         Error
Keywords:      Classic
User:          N/A
Computer:      tinsori
Description:
Faulting application name: Rainmeter.exe, version: <IP_ADDRESS>98, time stamp: 0x5c8fb359
Faulting module name: Rainmeter.dll, version: <IP_ADDRESS>98, time stamp: 0x5c8fb356
Exception code: 0xc0000005
Fault offset: 0x00000000001e070e
Faulting process id: 0x418c
Faulting application start time: 0x01d50e351fcc0b18
Faulting application path: D:\Rainmeter\Rainmeter.exe
Faulting module path: D:\Rainmeter\Rainmeter.dll
Report Id: 8a2c0652-2ee2-4031-a809-8a6e708ceb81
Faulting package full name:
Faulting package-relative application ID:

Could you open the Rainmeter log, refresh Audio Meter, and tell me what it says?

I turned on logging, did "refresh skin" and Rainmeter crashed. I restarted it, and did "refresh all" from Rainmeter.
 I assume restarting your PC did nothing? Did this issue just appear with the latest beta?

Right-click on Rainmeter's tray icon and click "About". Go to the Skins tab and select Audio Meter. Play some sounds and see if the Audio Band measures are working or not.

I hadn't fiddled with Audiometer before the beta. The Audio Band measures are all zeroes.
I'll restart now...

No change. I wonder if some of the audio crapware MSI installs is causing a problem here.
I keep trying to get rid of Nahimic and it keeps coming back.

Got it.
Nahmic was uninstalled but still had two system services running.
I killed them and reloaded Rainmeter and the visualization is working now. Solved an audio problem I had with X-Plane too.

Glad you got it sorted! :)


Can't log in Facebook

It appear a Facebook error while trying to log in. FFBE sync try to log but can't pass from Facebook token as the facebook windows asks if you want to log in into the FFBE app.

@thiefdeluxe it's a known issue, unfortunately. facebook or gumi changed something, which broke sync.
there are two solutions to choose from:

switch your game binding to google. downside is that once you unbind from facebook, you cannot bind to it again.
download just the apk file from here, and install it on an android device or emulator. the apk version of sync still works with facebook. if you have any problems with the apk, you can reply here (though it may get missed), or hop onto lyrgard's discord and ask for help there (link in the footer of the builder).


How to change item view type?

Hello, great work! How i can change item view type? I want use in 3 first items different view. How i can do it?

Override getSectionItemViewType and return item type, onCreateItemViewHolder's itemType will be your custom item type.

I override this method inside my item holder, but onCreateItemViewHolder's itemType is 0 for every time (

oh, sorry, i had override getSectionItemViewType(int section, int offset) method. Thank you for help 👍


Add a hash option

This adds the ability to pass a custom hash function in as a top-level option. A user-defined hash function must have the signature hash(key: string) -> Buffer.

4.2.0 🚬


feat: add oauth 2.0.9, uuid 1.2.1 releases

update oauth with release 2.0.9
update uuid with release 1.2.1

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


$GITHUB_STEP_SUMMARY isn't documented

https://github.com/nektos/act/pull/1607 added some support for $GITHUB_STEP_SUMMARY
There is no documentation https://nektosact.com/?search=github_step_summary
Note that you can no longer use --env GITHUB_STEP_SUMMARY=/dev/stdout -- that's fine, although it'd be nice if there was a warning (this is out of scope).
The best I've found to access step summaries is:
act -r
while true; do
  act_docker_pid=$(docker ps|perl -ne 'next unless s{\s.*"tail -f /dev/null".*act-.*}{};print'|tail -1)
  if [ -z "$act_docker_pid" ]; then
    break
  fi
  echo "$(docker ps|grep ^$act_docker_pid)::"
  docker exec $act_docker_pid cat /var/run/act/workflow/SUMMARY.md
  echo
  docker rm -f $act_docker_pid > /dev/null
done

act -r means that the container is kept around
GITHUB_STEP_SUMMARY is generally defined as /var/run/act/workflow/SUMMARY.md
Note that each job is run in its own docker container so the files are independent.
(This isn't remotely ideal, but some documentation is better than nothing.)

It also should note that content written to $GITHUB_STEP_SUMMARY will be destroyed at the beginning of the next step, so anyone running a multistep workflow in ACT should not rely on it as the output from any step other than the last will not be accessible.
https://github.com/nektos/act/issues/2273

Thanks for your research on this. This workaround helped me.
It looks like the path to SUMMARY.md is hardcoded here.
So are all the other files:
docker exec <running_act_container_id> ls -al /var/run/act/workflow
...
-rw-rw-rw- 1 root root    0 Jan  1  1970 SUMMARY.md
-rw-rw-rw- 1 root root    0 Jan  1  1970 envs.txt
-rw-r--r-- 1 root root    2 Jan  1  1970 event.json
-rw-rw-rw- 1 root root    0 Jan  1  1970 outputcmd.txt
-rw-rw-rw- 1 root root    0 Jan  1  1970 pathcmd.txt
-rw-rw-rw- 1 root root    0 Jan  1  1970 statecmd.txt

Also, there seems to be some very funky stuff going on with the GITHUB_STEP_SUMMARY env vars here
coming from here
but actually stemming from another repo and codebase here
The only real solution IMHO is a dedicated feature as outlined (by you) here


Issue on Business central Extension Page.

Hello experts,
I need your attention here, as I am facing an unexpected error in BC while opening extension page.
I was tried to add bank account in BC, but in my BC instance I am not able to see a action Bank statement service on Bank account page. Like below.

And when I tried to open extension page to set up YODLEE extension( For setting up online bank account) , I got a message like below.

Can anyone get me out of this error?
Kind Regards
Sudarshan

It seems you upgraded from Invoicing to Business Central. In that case you cannot use extensions.
Please contact your CSP or Microsoft Support as instructed by the message that you pasted above.


Fix inspection for Makie 0.18

Fixes issues with DataInspector from #113 which raised Makie compat to v0.18. This PR should cover all changes made to DataInspector. It now looks a lot nicer as well. :)

NB: This PR builds on top of the changes made in #114 and currently it's necessary to ]add OSMMakie#master to test this PR. It can only be merged once OSMMakie v0.0.6 is available on the public registry (it's registered and will be auto-merged with the next batch). To indicate this, I will leave this as a draft PR and only convert it to ready once the registry is updated.

Ah nevermind, OSMMakie v0.0.6 is already published! 🎉 So this PR is good to go as well. :)
You OK with this, @Datseris ?

Did you publish a new tag of InteractiveDynamics with this in? (patch number should have been incremented here, as no v1 exists yet)

No I didn't because I merged it into the graphspace-support branch which already has an incremented version number.


Pulls JS from typekit.net

Site should be completely self-contained

Getting removed with https://github.com/ipfs/website/pull/209


Split dialects & drivers into separate projects which reference the relevant client library, MicroLite will only support MS Sql Server in the core assembly

The idea here will be that the core MicroLite library will contain the Session API and support for MS SQL Server along with all the other classes and interfaces for things like listeners, type converters etc. We will then have specific projects which reference MicroLite and the relevant .NET library for their specific database type e.g.:

MicroLite.Database.FirebirdSql
MicroLite.Database.MySql
MicroLite.Database.PostgreSql
MicroLite.Database.SQLite

Given the size of this and since it's effectively a breaking change, I'm pushing it back out to 7.0


feat(android): add string values to State enum

fixes #1688
adds string values to State enum. These values are identical to those on iOS
emits them as string from MusicEvents.PLAYBACK_STATE
returns them as string from MusicModule#getState

Actually, you need to update these in the MusicModule.it as well

@dcvz do you want to look at this before merge?


What versions of Purescript does<EMAIL_ADDRESS>support?

Since Purescript 0.9.1 doesn't explicitly specify FFI files, can<EMAIL_ADDRESS>support older versions (like 0.8)?.

Yes, 1.0.0 is for purescript 0.8. The NPM dist-tags are used to match up
the loader version with the corresponding purescript version.
On Tuesday, 21 June 2016, wegry<EMAIL_ADDRESS>wrote:

Since Purescript 0.9.1 doesn't explicitly specify FFI files, can
<EMAIL_ADDRESS>support older versions (like 0.8)?.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ethul/purs-loader/issues/62, or mute the thread
https://github.com/notifications/unsubscribe/AAVYy8QUeme_FWDLASv7-nYVuw5NfmWBks5qOG1GgaJpZM4I7Qdt
.

Looks like my team will have to find a way to just hack around purescript/purescript/issues/2097.

Is there something on the purs-loader side that can help here? Maybe remove
purescript from the peer dependencies? Just want to make sure I understand
the issue.
On Tuesday, 21 June 2016, wegry<EMAIL_ADDRESS>wrote:

Closed #62 https://github.com/ethul/purs-loader/issues/62.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ethul/purs-loader/issues/62#event-699855280, or mute
the thread
https://github.com/notifications/unsubscribe/AAVYyzNScoVr1RzTsUUVHeGGYIFNJTF9ks5qOIZMgaJpZM4I7Qdt
.

Yeah, it might be an issue. My team currently uses node 6 and has a separate install of psc in our build process. I was hoping to get away with only installing psa and then hooking onto our current install of psc 0.8 until we upgrade to a version of psc that's compatible with node 6.

I am open to removing peer dependencies in the 1.x series. It has already
been removed in the 2.x series.
On Tuesday, 21 June 2016, wegry<EMAIL_ADDRESS>wrote:

Yeah, it might be an issue. My team currently uses node 6 and has a
separate install of psc in our build process. I was hoping to get away with
only installing psa and then hooking onto our current install of psc 0.8
until we upgrade to a version of psc that's compatible with node 6.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ethul/purs-loader/issues/62#issuecomment-227616354,
or mute the thread
https://github.com/notifications/unsubscribe/AAVYyzK-kHIQx4CxUhP0dyG39lSyqftgks5qOIp_gaJpZM4I7Qdt
.

Does the 2.x series support<EMAIL_ADDRESS>

Only psc 0.9
On Tuesday, 21 June 2016, wegry<EMAIL_ADDRESS>wrote:

Does the 2.x series support<EMAIL_ADDRESS>—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ethul/purs-loader/issues/62#issuecomment-227618553,
or mute the thread
https://github.com/notifications/unsubscribe/AAVYy70LFpbM_5o29ypdz6hrH6OTQgAWks5qOI6HgaJpZM4I7Qdt
.

Bah, then yes, that would be great if you could remove the peer dependency on<EMAIL_ADDRESS>Sent from my iPhone

On Jun 21, 2016, at 9:33 PM, eric<EMAIL_ADDRESS>wrote:
Only psc 0.9
On Tuesday, 21 June 2016, wegry<EMAIL_ADDRESS>wrote:

Does the 2.x series support<EMAIL_ADDRESS>—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ethul/purs-loader/issues/62#issuecomment-227618553,
or mute the thread
https://github.com/notifications/unsubscribe/AAVYy70LFpbM_5o29ypdz6hrH6OTQgAWks5qOI6HgaJpZM4I7Qdt
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

I will do that. It might be in the next day or so. I will keep you posted.
On Tuesday, 21 June 2016, wegry<EMAIL_ADDRESS>wrote:

Bah, then yes, that would be great if you could remove the peer dependency
on<EMAIL_ADDRESS>Sent from my iPhone

On Jun 21, 2016, at 9:33 PM, eric<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
Only psc 0.9
On Tuesday, 21 June 2016, wegry<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Does the 2.x series support<EMAIL_ADDRESS>—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/ethul/purs-loader/issues/62#issuecomment-227618553
,
or mute the thread
<
https://github.com/notifications/unsubscribe/AAVYy70LFpbM_5o29ypdz6hrH6OTQgAWks5qOI6HgaJpZM4I7Qdt

.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ethul/purs-loader/issues/62#issuecomment-227620998,
or mute the thread
https://github.com/notifications/unsubscribe/AAVYywt5T6UDDJ2fhnQUsDC98msugAa_ks5qOJNFgaJpZM4I7Qdt
.

Okay, peer dependencies have been removed in version 1.1.0

Thanks!

On Jun 21, 2016, at 10:02 PM, eric<EMAIL_ADDRESS>wrote:
Okay, peer dependencies have been removed in version 1.1.0
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Welcome
On Tuesday, 21 June 2016, wegry<EMAIL_ADDRESS>wrote:

Thanks!

On Jun 21, 2016, at 10:02 PM, eric<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
Okay, peer dependencies have been removed in version 1.1.0
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ethul/purs-loader/issues/62#issuecomment-227627106,
or mute the thread
https://github.com/notifications/unsubscribe/AAVYy9MQzJhp3ZOTvgefRCxvf7Bg1VQvks5qOJ63gaJpZM4I7Qdt
.


Archival 2024

New marketing site updated with latest info and a new 3D aesthetic.

Good enough for now!


How to save gif frames as png images

I want to quickly save the gif frame as a picture (png, jpeg)

Not really an issue for this library. Use the image or png crates to reencode the buffers that are returned.


ReplyAsync is not found

Upon trying to use ReplyAsync within a command module, I'm hit with an error stating that the method is not found, using the SendMessageFunction on the channel however works as intended.

Is this using Discord.Net.Commands?

Are you using the labs version of Discord.Net.Commands?

I probably am not, Yeah I might have to do that, thanks again


Fix the capitalization of OK buttons.

Historically the Mac UI has always capitalized both letters of the word "OK", and the lower case k in many areas of this app stands out as a mistake. Amazing project, btw!

I guess I'm Ok with this change.

Your way was fine, @JohnEarnest, looked even better – 💙 from a hardcore Card-ist

Jalkut beat me to the punch on reporting this one.


Telepath - implement block deletion for StreamBlock

Also incorporates feedback from #6693.

Merged with suggested fix to the 'animate' flag in e525efb05da9aa188df747daa8b3d54f5f8660e4.

Merged with suggested fix to the 'animate' flag in e525efb05da9aa188df747daa8b3d54f5f8660e4.


[FLOC-4048] Use the blockdevice mapping.

https://clusterhq.atlassian.net/browse/FLOC-4048
Depends on #2567, #2569, #2568.

Hi Tom,
I have left numerous small comments and observations.
I've no idea if we should clean up duplicated volumes, but I don't think the branch
should land with that comment. I think the comment is hard to evaluate later,
and so won't provide value. Either we should decide to clean them up and
implement that, or we should decide not to and document why.
Please land when you are happy.

Glad this is finally going in !

Address comments.
Add news entry.

Merge when green.


Improve formatting

Docs Build status updates of commit 605b3f3:
:clock10: Full build: the average full build time is 18 min(s) 53 sec(s), this is based on the last 27 build(s) for this repository.
:warning: This is a FULL build due to a recent Docs Build service upgrade, normally no action is required on your part. In the result you find subsequent builds still performing full builds, please contact your repository admin to manually perform a full build on the 'master' branch. For more detailed info, please click here.

Docs Build status updates of commit 605b3f3:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

docset/winserver2012r2-ps/adcsdeployment/install-adcscertificationauthority.md
:x:Error

Details

docset/winserver2012r2-ps/adcsdeployment/install-adcscertificationauthority.md

[Error-InvalidMamlMarkdown]

Failed to parse this MAML markdown file due to :40:(135) '```
Import-Module ServerManager
Add-WindowsFeatu...'
 Expect Heading, please use PlatyPS to do content validation

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 4f2683a:
:clock10: Full build: the average full build time is 18 min(s) 53 sec(s), this is based on the last 27 build(s) for this repository.
:warning: This is a FULL build due to a recent Docs Build service upgrade, normally no action is required on your part. In the result you find subsequent builds still performing full builds, please contact your repository admin to manually perform a full build on the 'master' branch. For more detailed info, please click here.

Docs Build status updates of commit 4f2683a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

docset/winserver2012r2-ps/adcsdeployment/install-adcscertificationauthority.md
:x:Error

Details

docset/winserver2012r2-ps/adcsdeployment/install-adcscertificationauthority.md

[Error-InvalidMamlMarkdown]

Failed to parse this MAML markdown file due to :39:(219) '
### --- IMPORT ---
'
 Expect Heading level 2, please use PlatyPS to do content validation

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 9cb0061:
:clock10: Full build: the average full build time is 18 min(s) 53 sec(s), this is based on the last 27 build(s) for this repository.
:warning: This is a FULL build due to a recent Docs Build service upgrade, normally no action is required on your part. In the result you find subsequent builds still performing full builds, please contact your repository admin to manually perform a full build on the 'master' branch. For more detailed info, please click here.

Docs Build status updates of commit 9cb0061:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

docset/winserver2012r2-ps/adcsdeployment/install-adcscertificationauthority.md
:x:Error

Details

docset/winserver2012r2-ps/adcsdeployment/install-adcscertificationauthority.md

[Error-InvalidMamlMarkdown]

Failed to parse this MAML markdown file due to :39:(219) '
### IMPORT
'
 Expect Heading level 2, please use PlatyPS to do content validation

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 9aa1fce:
:clock10: Full build: the average full build time is 18 min(s) 53 sec(s), this is based on the last 27 build(s) for this repository.
:warning: This is a FULL build due to a recent Docs Build service upgrade, normally no action is required on your part. In the result you find subsequent builds still performing full builds, please contact your repository admin to manually perform a full build on the 'master' branch. For more detailed info, please click here.

Docs Build status updates of commit 9aa1fce:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

docset/winserver2012r2-ps/adcsdeployment/install-adcscertificationauthority.md
:x:Error

Details

docset/winserver2012r2-ps/adcsdeployment/install-adcscertificationauthority.md

[Error-InvalidMamlMarkdown]

Failed to parse this MAML markdown file due to :39:(219) '

### IMPORT
'
 Expect Heading level 2, please use PlatyPS to do content validation

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Implement docopt

Something like: https://github.com/themattrix/bashup/blob/master/TODO.rst#docopt

Closed in favor of #9.


ModuleNotFoundError: No module named 'fvcore.common.registry'

Thanks for your nice work!
I have installed all of libraries following your install.md. But when i run "python tools/run_net.py --cfg configs/Kinetics/C2D_8x8_R50.yaml NUM_GPUS 1 TRAIN.BATCH_SIZE 8 SOLVER.BASE_LR 0.0125 DATA.PATH_TO_DATA_DIR path_to_your_data_folder", i got the following error:

I find the "fvcore" in my installed packages:

I'm sorry but this is out of the scope of the repo. As such I'm closing the issue. Check out StackOverflow for help.


Feature/rwa 654 limit search result

parametrise the limit camunda search result

superseded by https://github.com/hmcts/wa-task-monitor/pull/29/files


Testing PR import workflow

DO NOT SUBMIT
Thanks for your contribution! Unfortunately, tensorflow/runtime is currently not
accepting contributions. Please see the
Contribution Guidelines for
more information.

Test completed, closing pull request.


edwards: optimize point negation

This PR adds benchmarks for Neg(...) of points in Edwards variants, and an optimization to avoid redundant assignment in point coordinates.
If we run the new benchmarks available in the second commit of this branch against the same ones after the optimization here's what we get:
name               old time/op  new time/op  delta
pkg:github.com/consensys/gnark-crypto/ecc/bls12-377/twistededwards goos:linux goarch:amd64
Neg/Affine-16      4.60ns ± 2%  2.23ns ± 2%  -51.59%  (p=0.008 n=5+5)
Neg/Projective-16  5.28ns ± 3%  2.24ns ± 1%  -57.60%  (p=0.008 n=5+5)
Neg/Extended-16    7.73ns ± 1%  5.02ns ± 6%  -35.04%  (p=0.008 n=5+5)
pkg:github.com/consensys/gnark-crypto/ecc/bls12-378/twistededwards goos:linux goarch:amd64
Neg/Affine-16      4.77ns ± 5%  2.23ns ± 2%  -53.33%  (p=0.008 n=5+5)
Neg/Projective-16  5.29ns ± 4%  2.25ns ± 2%  -57.55%  (p=0.008 n=5+5)
Neg/Extended-16    7.68ns ± 2%  4.88ns ± 1%  -36.51%  (p=0.008 n=5+5)
pkg:github.com/consensys/gnark-crypto/ecc/bls12-381/bandersnatch goos:linux goarch:amd64
Neg/Affine-16      4.58ns ± 1%  2.22ns ± 1%  -51.60%  (p=0.008 n=5+5)
Neg/Projective-16  5.19ns ± 2%  2.30ns ± 4%  -55.64%  (p=0.016 n=4+5)
Neg/Extended-16    8.02ns ± 7%  5.01ns ± 2%  -37.48%  (p=0.016 n=5+4)
pkg:github.com/consensys/gnark-crypto/ecc/bls12-381/twistededwards goos:linux goarch:amd64
Neg/Affine-16      4.92ns ±13%  2.24ns ± 1%  -54.54%  (p=0.008 n=5+5)
Neg/Projective-16  5.61ns ± 9%  2.25ns ± 2%  -59.90%  (p=0.008 n=5+5)
Neg/Extended-16    8.04ns ± 5%  5.08ns ± 2%  -36.78%  (p=0.008 n=5+5)
pkg:github.com/consensys/gnark-crypto/ecc/bls24-315/twistededwards goos:linux goarch:amd64
Neg/Affine-16      4.90ns ± 9%  2.25ns ± 1%  -54.10%  (p=0.008 n=5+5)
Neg/Projective-16  5.22ns ± 2%  2.27ns ± 2%  -56.61%  (p=0.008 n=5+5)
Neg/Extended-16    7.80ns ± 5%  4.90ns ± 3%  -37.12%  (p=0.008 n=5+5)
pkg:github.com/consensys/gnark-crypto/ecc/bls24-317/twistededwards goos:linux goarch:amd64
Neg/Affine-16      4.71ns ± 3%  2.29ns ± 4%  -51.37%  (p=0.008 n=5+5)
Neg/Projective-16  5.32ns ± 3%  2.27ns ± 2%  -57.32%  (p=0.008 n=5+5)
Neg/Extended-16    7.75ns ± 4%  5.01ns ± 1%  -35.39%  (p=0.008 n=5+5)
pkg:github.com/consensys/gnark-crypto/ecc/bn254/twistededwards goos:linux goarch:amd64
Neg/Affine-16      4.61ns ± 1%  2.30ns ± 9%  -50.13%  (p=0.008 n=5+5)
Neg/Projective-16  5.22ns ± 1%  2.25ns ± 2%  -56.86%  (p=0.016 n=4+5)
Neg/Extended-16    7.64ns ± 4%  4.87ns ± 3%  -36.27%  (p=0.008 n=5+5)
pkg:github.com/consensys/gnark-crypto/ecc/bw6-633/twistededwards goos:linux goarch:amd64
Neg/Affine-16      5.22ns ± 5%  2.56ns ± 5%  -50.99%  (p=0.008 n=5+5)
Neg/Projective-16  6.48ns ± 7%  2.59ns ± 4%  -60.04%  (p=0.008 n=5+5)
Neg/Extended-16    8.96ns ± 1%  6.60ns ± 3%  -26.32%  (p=0.008 n=5+5)
pkg:github.com/consensys/gnark-crypto/ecc/bw6-756/twistededwards goos:linux goarch:amd64
Neg/Affine-16      6.85ns ± 4%  2.73ns ± 1%  -60.17%  (p=0.008 n=5+5)
Neg/Projective-16  7.25ns ± 9%  2.77ns ± 6%  -61.82%  (p=0.008 n=5+5)
Neg/Extended-16    10.4ns ± 1%   7.2ns ± 1%  -31.14%  (p=0.008 n=5+5)
pkg:github.com/consensys/gnark-crypto/ecc/bw6-761/twistededwards goos:linux goarch:amd64
Neg/Affine-16      6.04ns ± 2%  2.82ns ± 7%  -53.36%  (p=0.016 n=4+5)
Neg/Projective-16  7.11ns ± 4%  2.86ns ± 3%  -59.79%  (p=0.008 n=5+5)
Neg/Extended-16    10.3ns ± 2%   7.4ns ± 6%  -28.31%  (p=0.008 n=5+5)

The TL;DR is quite simple: a .Set(...) call will copy coordinates that need to be negated afterward, so that's unnecessary work.
I double-checked, and this optimization looks to be already present for the base curves, so it was only missing for edwards variants.

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


VIDEO RUN

Hey,
I literally just started to learn how to code and honestly I would love a video run through on how to run this.
Please help.

I no longer maintain this repo @Edwin-Kiarie I do not no if this works anymore.


[silx view] Add a dataset selection table (or something like that)

The goal is to be able to select datasets to be used as axes and as signals

Selection mechanism
Interpretation of the selection

For the time being, interpretation of 1D and scatter plots should be enough.

Done by #1817. It provides a way to manually compound and display nd-arrays with custom axes, or scatters.


Added authors and year of publication for books in the Java section.

Deleted one duplicate entry and added httpS where available.
This is for (part of) issue #2383.

btw, I just saw issue #1926, so I'm happy to pull some of the dates from the past 5 years. To be honest, for Java, some of the older material isn't very useful, so anything prior to 2010 would be useful to note.

@tedyoung , this is awesome. I agree with the syntax you initially used to add the year. I'll see about revising the linter, and adding some documentation so we can close the add-year issue. I may ask you to review that when it's done, if you'd be willing.

Happy to review, I was going to update the docs anyway, with respect to where the "remarks" go (the stuff in parentheses/italics), but I'll wait for your update first.

I see now. The linter was complaining about

Java: The Legend (Just fill the fields with any values) - Ben Evans, 2015

but is happy with

Java: The Legend - Ben Evans, 2015 (Just fill the fields with any values)

and there are previous entries like

Java: The Legend (2015) - Ben Evans

so I think we'll stick with the latter


failed against deno v1.44.3

Run denoland/setup-deno@041b854f97b325bd60e53e9dc2de9cb9f9ac0cba
  with:
    deno-version: 1.44.3
Error: Could not resolve a version for the given range.

Looks like related to https://github.com/denoland/website_feedback/issues/76

Looks like fixed now https://github.com/denoland/website_feedback/issues/76#issuecomment-2216496917


[Feature] make sticky scroll follows the indentation level from inside out

Problem
atm the sticky scroll show the data based on indentation which is good but only from outside in ex.

however, this falls short if u try to move deeper in the indentations levels more than the editor.stickyScroll.maxLineCount which is 4 ex.

notice the indentations guide border, the data i want to actually view is hidden, because the sticky information is counted from outside in, not from inside out which is how it should be.
Solution
the sticky part should follow the active indentations level/guide position and update accordingly instead of relying only on editor.stickyScroll.maxLineCount.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

/duplicate of https://github.com/microsoft/vscode/issues/168428

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


WIP: Add docs for unique and custom subdomains

📚 Context
Streamlit Community Cloud is launching the unique subdomain feature for apps on July 21, 2022 and will soon give users the ability to pick a custom subdomain for apps.
🧠 Description of Changes

Replace the Your app URL section of Deploy an app with docs for unique and custom subdomains.

Size:

[x] Small
[ ] Not small

🌐 References

[x] Notion

Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

This PR should be merged after the Cloud release on 7/21.


Remove Unnecessary Blueprint Plugin Material Numbers Logic now that we have new Conduit

Description
Resolves #18962

Removes unnecessary mat id logic that fixed a bug in older versions of conduit. That bug has been fixed, so now this logic can be removed.

Type of change

[x] Bug fix~~
[ ] New feature
[ ] Documentation update
[ ] Other

How Has This Been Tested?

Waiting on new conduit to test.
Reminders:

Please follow the style guidelines of this project.
Please perform a self-review of your code before submitting a PR and asking others to review it.
Please assign reviewers (see VisIt's PR procedures for more information).

Checklist:

[x] I have commented my code where applicable.~~
[ ] I have updated the release notes.
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have confirmed new and existing unit tests pass locally with my changes.~~
[ ] I have added new baselines for any new tests to the repo.
[x] I have NOT made any changes to protocol or public interfaces in an RC branch.~~

@JustinPrivitera  I merged the 3.4RC, which should pick up the new CI setup that is using conduit 0.9.1.
We can check the CI results to make sure the material id changes look good.

@cyrush Will CI results run the blueprint tests? I can go ahead and do a visit build with the new conduit and see what happens.

@cyrush Will CI results run the blueprint tests? I can go ahead and do a visit build with the new conduit and see what happens.

yes, it should run the blueprint tests and we can download the artifact that includes the html summary

Looks like we have something we need to explore related to the volume fractions

Hmm, it is still using conduit 0.8.8 -- need to look deeper.

All good now, thanks for the patience w/ the container update.

Merge w/ develop in #19305


Support SPDX 3.0 as input to the Evaluator

Supporting SPDX 3.0 as input file format to the Evaluator will enable the use case below:

Review SPDX files provided by third parties (suppliers/vendors) to your organization
Enables using ORT Evaluator and Reporter for code that is build using tools not supported by ORT such as Yocto

Note that this new feature only targets SPDX 3.0 and not any of the prior versions this is due to quirks in the SPDX spec that make it difficult to convert SPDX < 2.x to an ORT result file.

I'm confused why we should support SPDX 3.0 as input to the Evaluator instead of the Analyzer (which we already do support for SPDX 2.x in the form of SpdxDocumentFile). Because essentially, all we need to do with an SPDX BOM is to convert it to the ORT Analyzer result format. And from there we can pass it to any further ORT steps. Can you elaborate @tsteenbe?

Some ORT users only want to verify provided SPDX against their policy and don't want to run the analyzer or scanner. This makes sense for example in the case of:

a supplier providing a SPDX file for a proprietary-licensed binary that includes private OSS forks/snippets for which the source code is not available and its package can not be found in public package repositories.
build system like Yocto who have a lot of flags and my require custom scanning which is not supported by ORT and the user has their own analyzer/scanner tooling for this.

That said I can also see an use case where an organization wants to use the analyzer on provided SPDX file by a supplier to verify SPDX data correctness and/or use scanner/downloader to get a copy of the OSS code to create source code bundles/verify license conclusions in the SPDX file.

An additional use case we have suggested and need:
We would like to enrich a SPDX generated through other tooling with correct download descriptors based on analyzer + downloader logic.
Assumption is that the source code links generated on the supplier side on the source code itself is the most accurate source.
Thus: given a generated SPDX

analyzer can map components to findings and annotate missing components (both in SPDX and in its own output)
downloader takes the information from (1) and includes the most accurate download coordinate for the source code into the SPDX


made contrib flatten layer accept tensors with dynamic shapes

Can one of the admins verify this patch?

@ppries sorry we're taking a long time to review. Could you merge and re-upload?

@drpngx Done!

I've added a test for the case of unknown dimensions – the known dimensions case is already handled in testFlattenBatchSize, and it seems silly to add another identical test only for semantics. Let me know if you disagree.
It is currently failing testInvalidRank, which I can fix by checking the length of get_shape and reverting to the previous way of asserting rank > 2 (it is currently done using TF control_flow ops). This then assumes that the rank is static – I don't know enough about TF to know if that is a problem.
In any case, I am already using some information from get_shape to decide whether I can propagate shape information to outputs or not, so it might not be an issue. Let me know what you think, then I'll make whatever changes are needed to pass all tests.

Do I need to do anything?
@sguada

No you should be fine. Jenkins, test this please.


Add method to submit "source verifications" / ask for corrections

Users should be able to link a github account. Need to attach camera picture (open camera to take one) or existing image.
The line that the user is trying to "correct" / "verify" would follow the same template as db repo corrections do today.

https://developer.github.com/apps/building-oauth-apps/


Consolidation operation reference tests on mainnet invalid for the wrong reasons

While testing our implementation of process_consolidation (EIP-7251) I notice something weird in the reference tests for consolidation.
We have multiple invalid_<something> scenarios in the tests, however, in our implementation they are all failing on the same condition:
assert get_consolidation_churn_limit(state) > MIN_ACTIVATION_BALANCE

The only exception is the test invalid_exceed_pending_consolidations_limit, because it fails in a condition before.
I have reviewed our current implementation and could not find any other reason to explain this issue, and now I believe that our test vectors might be wrong, setting up the wrong scenarios. It can be hard to miss because the test report will succeed, given it is expecting a failure in all tests.
I'd appreciate it if other teams could confirm this behavior. If it is only happening to us then I'll need to dig deeper to figure out what is going on :)

cc @hwwhww


Template custom_filter.html missing in pip install

The installation by pip install (v0.1) doesn't provide the template custom_filter.html, that is present with the 0.1 zip release instead.
Plaease, also note that the form tag in the templatefilter_query.html is not closed.

@pyMan Thanks a lot for pointing me to these bugs. I've fixed the form tag in filter_query and fixed the paths in MANIFEST.in. The next release uploaded to pypi will include the missing templates.

@pyMan Thanks a lot for pointing me to these bugs. I've fixed the form tag in filter_query and fixed the paths in MANIFEST.in. The next release uploaded to pypi will include the missing templates.

This is fixed in version 0.2

This is fixed in version 0.2


fix invalid null handling in filter

Which issue does this PR close?
This fixes #295.
This is a very simple solution to the problem and doesn't change any filtering behavior (though we may simplify the filtering if this PR is ok). In case of null values in a boolean mask I do a mask & null_bitmask operation and create a new boolean predicate that has no null values (all nulls are false), due to the AND operation.

This looks cool! Also if we can simplify / speed up filters this way, that would be very interesting, there is still quite some optimization potential in the filter kernel I believe.

Codecov Report

Merging #296 (b51d484) into master (3665296) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #296   +/-   ##
=======================================
  Coverage   82.49%   82.50%
=======================================
  Files         162      162
  Lines       43980    44002   +22
=======================================
+ Hits        36283    36305   +22
  Misses       7697     7697

Impacted Files
Coverage Δ

arrow/src/compute/kernels/filter.rs
92.98% <100.00%> (+0.58%)
:arrow_up:

Continue to review full report at Codecov.

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

@jhorstmann  / @Dandandan  do you think this one is ready to go?

I checked out the branch to have a look with a bit more context. The logic looks good and makes this kernel a lot easier to use. The test looks good and should cover exactly this problem.
One minor thing: There is a doc comment with a warning about null values above the filter function, which is now out of data.

One small future improvement might be not creating a new array for filters, but computing / passing the changed buffer instead.
But the removal of undefined behavior for this kernel is really good I would say :+1:

Rerunning the CI as part of it showed red for some reason.


It looks a little strange

Hoo.. oO
Did you take the project from the repo or from cocoapods ?

from repo

how to fix this problem?

I have the same issue


Render existing map instance to vector tile

Essentially I am looking for a python equivalent to this flow in node:
https://github.com/mapnik/node-mapnik/blob/master/test/vector-tile.test.js#L42-L46
I have an existing map instance and would like to just set the map bbox and render to a vector tile protobuf. The current flow for raster tiles looks something like this:
m = Map(tilesize, tilesize)
load_map(m, stylesheet_path)

m.zoom_to_box(bbox)
img = Image(m.width, m.height)
render(m, img, scale)

Given that the readme discusses how to do the map->tile projections (https://github.com/mapzen/mapbox-vector-tile#coordinate-transformations-for-encoding), I am assuming this is not particularly straightforward. Any suggestions on the easiest way to get a mapnik Map into the format accepted by the encoder here?

Hi! Unfortunately, there's no easy way to do that with this mapbox-vector-tile code at the moment, and it looks like official bindings are still in the works.
However, you could try using avecado, which builds some Python bindings for generating vector tiles. The documentation isn't extensive, but the docs on the make_vector_tile function seem to suggest that it does something close to what you want.
Hope that helps!

Cool, just wanted to make sure I was not missing something super obvious.
I use python datasources, so I can just grab the features from there and do the tile projections myself. Thanks for the quick response.


AOS is affecting my CSS hover animation

This is:

Bug

Specifications

AOS version: 2.3.1
OS: ChromeOS
Browser: Chrome

Expected Behavior
I'm using AOS to fade 4 boxes up, with a 100ms delay on the first, 200ms on the second, 300ms on the third and 400ms on the fourth. Works great. I also have a hover animation on each block with a transition of .3s. I expect each box to fade up 100ms after the previous one, (which is working) and I expect the hover animations to take .3s
Actual Behavior
What's happening is that for some reason, data-aos-delay="400" is affecting my hover animation. So the hover animation is being delayed by the same amount as specified by the data-aos-delay value
Steps to Reproduce the Problem

Visit my github page: https://welbinator.github.io/bizland/
The first section after the hero section, you'll see 4 boxes.
Hover over each one, and notice how the animations have a delay

Detailed Description
data-aos-delay="[some-number]" is affecting my hover animations. Here's the repo: https://github.com/welbinator/bizland
Possible Solution

nevermind problem was my own.


Changing API version of Clients

Issue summary
Write a short description of the issue here ↓
Changing API version of Clients (Admin GraphQL, Storefront) without relying on context.
Expected behavior
What do you think should happen?
Should be able to override/change api version used on clients using additional parameter
Actual behavior
What actually happens?
Cannot change api version without initializing shopify context
Tip: include an error message (in a <details></details> tag) if your issue is related to an error
Steps to reproduce the problem

Use api client without context, apiversion defaults to unstable

Reduced test case
The best way to get your bug fixed is to provide a reduced test case.

Checklist

[x] I have described this issue in a way that is actionable (if possible)

Yes would really love to see this functionality to set a specific version on the client (or as a param when initialising it).

Is it possible to get this looked at/added soon? Crucial feature for us, would love to see it implemented.

A workaround I found that seems to be working so far is just manually specifying the version after the import, i.e.
Shopify.Context.API_VERSION = '2022-04';

This is without using Context.initialize anywhere too. An official solution would be preferred, but hopefully this helps for now.

A workaround I found that seems to be working so far is just manually specifying the version after the import, i.e.
Shopify.Context.API_VERSION = '2022-04';

This is without using Context.initialize anywhere too. An official solution would be preferred, but hopefully this helps for now.

@shopsideau
Great! It is a simple solution and works. Thank you very much.


add version annotations to integration_test pipeline

Description
This PR adds a buildkite "info" annotation containing the versions being tested in each integration_test pipeline run.
A demonstration of the annotations can be found here:
https://buildkite.com/chef/chef-chef-server-master-integration-test/builds/310
Signed-off-by: Christopher A. Snapp<EMAIL_ADDRESS>Issues Resolved
#2087
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[X] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco
[ ] PR title is a worthy inclusion in the CHANGELOG

The install and upgrade versions for Chef Server and the install version of Chef Backend are all required for the pipeline to function.  As a result the environment variables are populated by the pipeline's .expeditor/integration_test.pipeline.sh execution script before Terraform is initialized.
The Elasticsearch version isn't required and currently has its default set by Terraform which doesn't surface its value at a level the pipeline script can see.  I've added the print out of the variable value if you are just looking to see if the default version was overridden.
If you need to see the default version number printed out then I'll have to relocate where that default is set or read the default value from the Terraform code.
As for the external PostgreSQL version, we currently don't support overriding that so I'd have to implement that variable override before we can print out a meaningful value.


Chunkwrap is misbehaving!

Oh dear, oh dear:

There's clearly a problem with the way chunkwrap is behaving.
Via #issuefication-zone / #hsmusic-chat.

Fixed in 4f91ee5b! 🥳


Update svg-filters.json

The domain for the specific resource has been taken over by a hosting company, and as such, I am removing the link.
In case you'd like to preserve the resource there is an archive link is available here:
http://wayback.vefsafn.is/wayback/20140721113025/http://electricbeach.org/?p=950
(At the time of writing this Archive.org is having ElasticSearch problems so that's why the alternative Wayback link)

Thanks!


how can fix the problem "Memory cannot be recycled" in  my web service  build by tensorflow + flask

### This is my code:
import numpy as np
from flask import Flask, request
import tensorflow as tf

from tensorflow.python.platform import gfile
from app.utils.log_util import LogUtils
log = LogUtils.get_stream_logger(__name__)

app = Flask(__name__)

with gfile.FastGFile('../20190517-152605.pb', 'rb') as f:
    graph_def = tf.GraphDef()
    graph_def.ParseFromString(f.read())

graph = tf.Graph()
with graph.as_default():
    tf.import_graph_def(graph_def, input_map=None, name='')
    # print(tf.get_default_graph().as_graph_def().node)
    images_placeholder = tf.get_default_graph().get_tensor_by_name("input:0")
    embeddings = tf.get_default_graph().get_tensor_by_name("embeddings:0")
    phase_train_placeholder = tf.get_default_graph().get_tensor_by_name("phase_train:0")
    model = {
        'images_placeholder': images_placeholder,
        'embeddings': embeddings,
        'phase_train_placeholder': phase_train_placeholder
    }

sess = tf.Session(graph=graph)

@app.route('/', methods=['POST'])
def classify():
    log.info('ss')
    # data = request.files.get('data').read()
    face = np.load("test.npy")
    images_placeholder = model['images_placeholder']
    embeddings = model['embeddings']
    phase_train_placeholder = model['phase_train_placeholder']
    result = sess.run(embeddings, feed_dict={images_placeholder: face, phase_train_placeholder: False})
    print(result)
    return str(200)
app.run(host='<IP_ADDRESS>',port=12480)

memory grow situation :
request times: |1 | 2 |4 ... |100 ... n
memory: |400+MB |600+MB | 700MB ... |2GB ... 2GB

@johnsonyep Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

@johnsonyep Could you provide more details about the issue and context?

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


feat: PenaltyKickState

@SoroushMazloum
Do not forget to update idls (proto, thirft) version
Do not forget to update the change log

@naderzare
Hi,
I updated .thrift and .proto versions and penalty_kick_state is only filled when state is penalty mode.
But I will update the change log in another commit or pull request.


devise_token_auth initializer breaks omniauth paths

I am using the gem for a bare api token authentication and I have noticed that the initializer breaks my omniauth paths.
routes.rb
mount_devise_token_auth_for 'User',
  controllers: {
    sessions: 'api/sessions',
    registrations: 'api/registration',
  }

devise_token_auth.rb
DeviseTokenAuth.setup do |config|
  config.change_headers_on_each_request = false
  config.token_lifespan = 3.months
end

Without the initializer auth/google_oauth2 redirect to the google authetication page, but when i add the initializer auth/google_oauth2 raises No route matches [GET] /auth/google_oauth2
Is there something i can do? Could the initializer override the previous declared omniauth paths?

Have you checked what the routes are with rake after the change?

These are my devise_token_auth routes
               api_validate_token GET      /api/validate_token(.:format)                    devise_token_auth/token_validations#validate_token
                      api_failure GET      /api/failure(.:format)                           devise_token_auth/omniauth_callbacks#omniauth_failure
                                  GET      /api/:provider/callback(.:format)                devise_token_auth/omniauth_callbacks#omniauth_success
                                  GET|POST /omniauth/:provider/callback(.:format)           devise_token_auth/omniauth_callbacks#redirect_callbacks
                 omniauth_failure GET|POST /omniauth/failure(.:format)                      devise_token_auth/omniauth_callbacks#omniauth_failure
                                  GET      /api/:provider(.:format)                         redirect(301)

omniauth.rb
Rails.application.config.middleware.use OmniAuth::Builder do
  provider :google_oauth2, ENV['GOOGLE_KEY'],   ENV['GOOGLE_SECRET'], :client_options => { :ssl
     => { :verify => !Rails.env.development? } }
end

What's interesting is that without the devise_token_auth initializer both auth/google and api/google path work. With the initializer auth/google is undefined while api/google still works.

Seems like it's working as expected then, devise_token_auth moves the oauth paths to /api (Thats where you mounted it in routes.rb) - it might not fully take effect until the gem is initialized.

But i do not want to use omniauth with devise_token_auth and even so, I should be able to declare multiple oauth paths without the gem moving them to a different path.

How are you mounting the omniauth routes themselves? Try mounting devise_token_auth somewhere other than api/auth, or having the base omniauth routes mounted elsewhere so there isn't conflict and see if that helps.

Seems it's working as expected, @alexandrapersea feel free to reopen if you think isn't working.


angularjs.TypeScript.DefinitelyTyped includes an ancient version of jquery.d.ts

See this stack overflow post:
http://stackoverflow.com/questions/32088619/angularjs-definitely-typed-visual-studio-2015/32111765#32111765
If you add this package to a TypeScript project in Visual Studio and then reference it, you'll see hundreds of compiler errors. The reason why is because the d.ts file for jQuery appears to be ancient. For example, it uses "bool" instead of "boolean".
I was able to fix it by upgrading the jQuery package, or adding jQuery.TypeScript.DefinitelyTyped to the project, which replaced the old version with a new one that was correct.

@staxmanade ?

This is the first I've heard about the issue.
In the .nuspec definition we place package dependencies like angularjs depends on jquery and it uses the following format to define it's dependencies
"<dependency id="$dependentPackageName" />"
Note there is no version specified and on this page https://docs.nuget.org/create/versioning#Specifying-Version-Ranges-in-.nuspec-Files it says empty = latest version. so it should be pulling the most recent jquery version...
What happens if you remove jquery.Typings.DefinitelyTyped and install from scratch?
Hmmm

I just tested it and the issue is when you installed angularjs.typescript.definitelytyped 4.1.4, it apperantly says it needs jquery.Typings.DefinitelyTyped 0.1 version and install that instead of the latest version.
I uninstall the angularjs.typescript.definitelytyped, then it showed the latest jquery.Typings.DefinitelyTyped version and so I installed it. Afterwards I installed angularjs.typescript.definitelytyped

I've run into this a few times as well. It's due to the NuGet package manager UI in Visual Studio 2015 changing the dependency behavior from prior versions. It used to grab the most recent version (stable or prerelease depending on your settings) but now it lets you pick from

Ignore Dependencies
Lowest (default, at least on my machine)
Highest Patch
Highest Minor
Highest

Since versions aren't specified for the dependencies this is why the first jquery release is being used instead of the most recent.

You might want to comb through the other Nuget packages which have dependencies on other libraries and make similar changes to whatever you do here.

I just ran into this issue on vs2015.  jquery.Typings.DefinitelyTyped was version 001.  I upgraded it to latest stable, errors disappeared.

does this issue still occur?

:white_check_mark: :white_check_mark: :white_check_mark: Answer :white_check_mark: :white_check_mark: :white_check_mark:
For anyone running into this issue: check the following setting in Visual Studio:

The default behavior may not have been “Lowest” before which may explain why this issue is new to Visual Studio 2015.  To get things to work with the current angular TS definition package, you need to  choose "highest" when you install or upgrade your package.


ReactNative0.51 iOS reload Flash back

Environment
I upgrade react-native from 0.44.6 to 0.51, and keep the index.ios. js. After the successful upgrade,  when I modify the ReactNative code and reload,the iOS APP always crash.Sometimes the error code in XCode means my host down,but it's  normal;sometimes it's no error code.
Steps to Reproduce
run app in Xcode
reload
Expected Behavior

It's normal to reload.
Actual Behavior

I find something


Shaylen Naidoo: Conditional Send Should Support Logical Operations

Email
<EMAIL_ADDRESS>Description
Allow sending of mail merges to use several logical conditions such that we can use operations like AND, OR and NOT to join the multiple logical conditions.

This is now possible in Rich Text Mailman, because all of the fields are rendered using the Handlebars templating engine. (to, cc, bcc, subject, body, and condition)
For example, If you have two columns "My Condition 1" and "My Condition 2":
To only send emails when both are TRUE use {{#and [My Condition 1] [My Condition 2]}}
To send emails if either are TRUE use {{#or [My Condition 1] [My Condition 2]}}
More powerful logical operators exist, documentation will be forthcoming


[History] 'Back' button directs to Users page

Environment: Windows 10 Pro 22H2 Google Chrome Version 120.0.6099.225 (Official Build) (64-bit)
Reproducible: always
Preconditions

Site https://zero-waste-staging.onrender.com/en is opened.
Admin is logged in

Steps to reproduce

Go to 'History' page
Scroll down the page
Click 'Back' button

Actual result
'Back' button directs to Users page
Postconditions

Instead of 'Back' button 'Up' button should be created
'Up' button should turn the page up

User story and test case links
E.g.: "User story #100
Test case"
https://github.com/ita-social-projects/ZeroWaste/assets/145603083/a39389ab-4335-4e59-8948-cbe38ace2228

Issue is solved by creating pagination on the page. 'Back' button is taken away


"Run Python in Terminal" button in the Python extension should run cells in a #%% Python script.

Testing microsoft/vscode#129288
With the python extension installed you have the run icon in the top right. With an interactive window opened I expected the interactive cells to get executed there. I'm not sure what the best UI would be though as I understand it can be a mixed file

The button now says run python file in terminal to disambiguate


fix(docs): broken link for community contribution lifecycle and processes

Which problem is this PR solving?

The current link is broken https://github.com/open-telemetry/community/blob/main/CONTRIBUTING.md

Short description of the changes

Fixes the broken link for contribution processes which were moved in https://github.com/open-telemetry/community/pull/2051

Thank you @pichlermarc for reviewing and merging this PR.
Can you approve workflows on https://github.com/open-telemetry/opentelemetry-js-contrib/pull/2545?
I want to confirm that it fails the CI, and then push formatting fixes.


Yarn upgrade ccxt

Let us know if you need any assistance on behalf of CCXT. Thx!

This looks great, thanks a lot @blair!


Support type as field name on schema

Required Functionality
Currently for schema generator, synth use key: value. key for field / column name and value contain schema description. Therefore using string literal type will conflict with schema definition container object.
Proposed Solution
Use case
Example conflicted schema:
{
  "type": "array",
  "length": {
    "type": "number",
    "range": {
      "high": 1,
      "low": 0,
      "step": 1
    },
    "subtype": "u64"
  },
  "content": {
    "type": "object",
    "ip_address": {
      "optional": true,
      "type": "string",
      "pattern": "[a-zA-Z0-9]{0, 45}"
    },
    "user_agent": {
      "optional": true,
      "type": "string",
      "pattern": "[a-zA-Z0-9]{0, 1000}"
    },
    "hash": {
      "optional": true,
      "type": "string",
      "pattern": "[a-zA-Z0-9]{0, 64}"
    },
    "user_id": {
      "optional": true,
      "type": "string",
      "pattern": "[a-zA-Z0-9]{0, 50}"
    },
    "type": { <-- Duplicate Object key
      "optional": true,
      "type": "string",
      "pattern": "[a-zA-Z0-9]{0, 5}"
    },
    "created_at": {
      "type": "string",
      "date_time": {
        "format": "%Y-%m-%d %H:%M:%S%Z",
        "subtype": "date_time",
        "begin": null,
        "end": null
      }
    },
    "id": {
      "type": "number",
      "id": {},
      "subtype": "u64"
    }
  }
}

Thanks for this! I'm surprised it hasn't popped up already. Will fix and add to an upcoming release.

I'm going to write this down here to both collect my thoughts and show my openness for discussion regarding the proposed solution:

As a workaround, we'll allow the "type_" key, which will automatically be associated with the attribute named type without clashing with our internal "type" key. Note that we will only allow this for this one key.
I think the problem stems from mixing predefined keys and attribute names. Introducing a clear split between those would be the better solution from a design stand point. If we introduce such a change, we'll allow both variants for a while and then introduce warnings for the old variant that show the required change. After a while, we remove the old variant.


Prepare Search Beta Release August 2023

Prepare Search Beta Release August 2023

API change check
API changes are not detected in this pull request.

/azp run java - search - ci


loader: fix wrong progress in query-status for loader

What problem does this PR solve?

Issue Number: close #3252
What is changed and how it works?
fix metrrics load logic
Check List
Tests

Manual test (add detailed scripts or steps below)
revert old logic

Code changes
Side effects
Related changes
Release note
`fix a bug that wrong progress in query-status for loader`.

Codecov Report

Merging #4093 (ae875f4) into master (3873d39) will decrease coverage by 1.8599%.
The diff coverage is 66.5841%.

Flag
Coverage Δ

cdc
58.7203% <66.5841%> (+0.4837%)
:arrow_up:

dm
52.2086% <ø> (-3.8259%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4093        +/-   ##
================================================
- Coverage   57.0741%   55.2141%   -1.8600%
================================================
  Files           478        489        +11
  Lines         56551      60374      +3823
================================================
+ Hits          32276      33335      +1059
- Misses        20978      23664      +2686
- Partials       3297       3375        +78

this is just revert old logic please do a quick review,
/cc @GMHDBJD  @lance6716

/merge

/run-verify
/run-kafka-integration-test
/run-dm-integration-test

/run-dm-integration-tests

/run-kafka-integration-test
/run-dm-integration-test

/run-dm-integration-test


Does not produce valid X(HTML)

Both HTML and XHTML need to have at least html, head and title to be valid. And a DOCTYPE declaration is very useful to specify the encoding of the document.

Markdown produces an (X)HTML fragment. It was never intended to do any more than that. If you want a complete HTML document, then it is your responsibility to wrap it. This has been the behavior of the original Markdown implementation from the beginning, which we mirror with not intention of changing.
As an aside, we only output Unicode text, so an encoding would not be applicable anyway. True, the CLI outputs an encoded file, but this is a library first (to be called by other Python code). The CLI is only provided as a convenience. The assumption is that the output of the CLI would be piped to something else which wraps the (X)HTML fragment.

To add another voice:
For the library, I can understand that producing the HTML wrapping is out of scope - after all, a library is by definition not intended to work on its own. However, the command line script can run on its own, and would be much more useful if it offered an option to produce a complete HTML document.
Are there objections to adding a command line option to do HTML wrapping?
Something like:
-c, --complete-html
Output a complete HTML document, instead of an HTML fragment.
The document will contain an encoding declaration, so web browsers
can display it correctly even as a local file.

The option would just prepend a standard HTML document start, including <meta charset="utf-8">. I'd be happy to provide a patch/pull request if there is interest in including this.
As far as I can see, it should be possible to do this in main.py, so there is no need to change the library itself.

Are there objections to adding a command line option to do HTML wrapping?

Yes. The implementations which include something of the sort need to support some sort of templating system which is something other than Markdown. If you want that, then use a Static Site Generator (which is its intended purpose). If you don't need that sort of sophistication, then use the existing tool with pipes (etc) to build your complete document (you are using the command line after all; might as well use it to the full). There's no need for us to add anything as it would either be out-of-scope or is already possible with the existing tools.

If you don't need that sort of sophistication, then use the existing tool with pipes (etc) to build your complete document (you are using the command line after all; might as well use it to the full). There's no need for us to add anything as it would either be out-of-scope or is already possible with the existing tools.

Yes, creating a complete HTML document is possible with existing tools - but if all you want is an HTML document you can read in a browser, it is a bit tedious.
The use-case I have in mind is: You have a markdown document, and want to convert it to HTML to read it conveniently. Maybe to quickly preview markdown you write yourself, or some documentation that is provided as markdown... Having a simple command line utility to do this is useful.
This situation led me to the markdown_py script, and I was a bit annoyed that it omits the HTML header/ending. Thus my question about adding an option. I could not find any command line utility that does this.
If you feel this use case is not significant enough, then I won't insist further (and hack up a wrapper script). Anyway, thank you for the feedback.

No matter how simple of a solution was provided, there would inevitably be someone who would argue that it shouldn't be to much to add just a little bit more. I have to draw the line somewhere and I have drawn the line where I have because the original reference implementation (markdown.pl) didn't include such a feature either. The creator of Markdown saw it as being out-of-scope, so I do as well.

Yes, I see your point about scope creep.
Also, I just noticed that pandoc ( http://pandoc.org/ , or on GitHub at https://github.com/jgm/pandoc ) has exactly what I wanted. The pandoc command line tool has the option --standalone to produce complete HTML documents. So I'll just use that and not bother you anymore :-).
Maybe the links will help others who are looking for a simple command-line converter Markdown -> HTML.
Thank you for your detailed feedback about the project scope, and happy hacking!


Failed Pull From RedHat Registry

Img is unable to pull from the RedHat registry, for example, the registry.access.redhat.com/ubi7-minimal:latest image.
Expected
Img should be able to pull registry.access.redhat.com/ubi7-minimal:latest.
Actual
Pull command fails with error: Error: could not resolve digest for registry.access.redhat.com/ubi7-minimal:latest
Steps to Reproduce
$ docker run -it --rm --privileged r.j3ss.co/img pull registry.access.redhat.com/ubi7-minimal:latest
Pulling registry.access.redhat.com/ubi7-minimal:latest...
Error: could not resolve digest for registry.access.redhat.com/ubi7-minimal:latest

Details
This error comes from containerd, but I am not sure why it is failing in this way for this registry, but works for others.
https://github.com/containerd/containerd/blob/b8631c750c512ee9a927d1076b76c452e8a54ba1/remotes/docker/resolver.go#L237
This needs some more investigation to determine the cause.

This looks like it's because Red Hat's container image manifests are served from a CDN and don't include the Docker-Content-Digest header.
OCI spec says this header is optional, so I would expect containerd to calculate the digest itself as a fallback rather than erroring.

#285


Error while using CheckM

When running pcalf-annotate-workflow a strange error occured :
pcalf-annotate-workflow -i pcalf_datasets_results/genomes.yaml -o pcalf_annotate_results --gtdb /run/media/guest-biom/data4/GTDB/release214 --checkm /run/media/guest-biom/data4/CheckM
Error in rule cm_bins_into_batches:
jobid: 0
output: /run/media/guest-biom/dat2_encrypt/Projet_Bioinfos/pcalf/231110_Scan_cyanobacteria_wt_checkm/pcalf_annotate_results/checkm-res/tmp/bins/batch_2/batch.tsv
RuleException:
KeyError in file /home/guest-biom/opt/pcalf/pcalf/workflow/annotate/rules/checkm.smk, line 205:
'batch_2'
File "/home/guest-biom/opt/pcalf/pcalf/workflow/annotate/rules/checkm.smk", line 205, in __rule_cm_bins_into_batches
File "/home/guest-biom/miniconda3/envs/pcalf/lib/python3.9/concurrent/futures/thread.py", line 58, in run
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
What is strange is that the processes still progressed after that.

can you send me the genomes.yaml please ? What is the batch size ?

genomes.txt
I add to change the .yaml into .txt but it is the correct file.
What is a batch :3 ?

I also add that the directory Scan_cyanobacteria_wt_checkm/pcalf_annotate_results/checkm-res/tmp/bins/batch_2 have been created, but not batch.tsv

Final point on the subject. Sometimes this error kill the process. Sometimes the process is able to proceed. I have no clue as why this behaviour change

The problem is clearly nested in thoses lines :
params:
batch = lambda wildcards:GENOMESBATCH[wildcards.batch],
run:
batch_genomes = params.batch[str(wildcards.batch)]
I am positively terrified on how you can call a variable (wildcards) that has not been defined in the code (this rule takes no inputs)

Okay. There is something fishy in genomes_into_batches() in Snakefile.
When called by cm_bins_into_batches, the function return not one dictionary but five identical dictionaries, which probably messes up the rest of the code. This behaviour is linked to INPUT.items().

My current understanding of the issue is :
_ rule cm_bins_into_batches calls GENOMESBATCH
_GENOMESBATCH seems to return not one dictionnary, but 5 identical dictionnaries at once.
_The issue seems to be linked to INPUT.items() which return 5 times the content of genomes.yaml
However INPUT.items() is not bugged. Apparently the workflow calls the GENOMESBATCH twice, and the first time everything is working fine. Only the second time the fonction returns 5 dictionaries.

I also don't understand how wildcards can be defined in the run section of this rules. And I think it is somewhat important.
params:
batch = lambda wildcards:GENOMESBATCH[wildcards.batch],
run:
batch_genomes = params.batch[str(wildcards.batch)]

I fix the broken part of the cm_bins_into_batches  rule.