EIP-1153: Transient storage opcodes

useful for testing and developing to have these opcodes optionally enabled
https://eips.ethereum.org/EIPS/eip-1153

https://direct.lc.chat/14345826/

Hm what is the usecase for using this opcodes for testing?

developing protocols that will make use of it. you can just use regular storage, but then your gas measurements are off and you're testing against a pre-transient storage version of the code

Yeah but transient storage is still going to spend the gas.
I am okay to spend time including this but only when it gets certain it is going to be included in the Ethereum mainnet.

Yeah but transient storage is still going to spend the gas.

what do you mean? it spends much less gas. hard to compare gas snapshots of contracts if you can't test with the proposed opcodes

I am okay to spend time including this but only when it gets certain it is going to be included in the Ethereum mainnet.

what's the downside of including it before it is certain, behind a flag? ethereumjs did this, which allows for testing in hardhat

Yeah but transient storage is still going to spend the gas.

what do you mean? it spends much less gas. hard to compare gas snapshots of contracts if you can't test with the proposed opcodes
My bad, I though you meant testing smart contract, i was confused bcs of that, but you meant testing this EIP?

I am okay to spend time including this but only when it gets certain it is going to be included in the Ethereum mainnet.

what's the downside of including it before it is certain, behind a flag? ethereumjs did this, which allows for testing in hardhat

I would reader spend doing something else than investing my time in this particular EIP atm, and even if the work is done by somebody else there is still a maintenance cost of this (small but still there). As it is uncertain when or even if it is going to be included and design as with every EIP is going to be changed, so for me, as I am doing revm on my own time, the best way is to wait and see where this goes.

The EIP is now CFI: https://github.com/ethereum/execution-specs/blob/master/network-upgrades/mainnet-upgrades/shanghai.md#eips-considered-for-inclusion
Would love to see this in revm and subsequently forge so users can start playing with it

Hello, I am following what is going on. There were a few major changes happening inside revm as usage of ruint so I am focusing on that.
I dont have a need to merge or do this before all clients did their implementation as it would save me a lot of time doing it later and skipping discussion/uncertainty that every concrete implementation brings.
As it stands from a few months ago, if you want to invest your own time and experiment, be free to do it in side repo, it is kinda easy to build foundry with patched revm.

Just adding here, that this feature would be desired for the arbiter project as well. Although it's not our immediate concern. Perhaps when it comes to it I could take a look at implementing this given the specification.


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1100 ms

Resolved: Bofrost.be performance has improved in ed3d079 after 8 minutes.


Add pod-density scenarios

This scenarios is useful when you only care about creating a  lot of
pods in a namespace and don't care about distribution across worker nodes.
This is similar to what used to be called pod-vertical.
Signed-off-by: Sai Sindhur Malleni<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Hey @smalleni !,
Can you make reference of this new workload in the docs?

Thanks for the reminder Raul, can you take a look now?

Hey @smalleni !,
Can you make reference of this new workload in the docs?

Thanks for the reminder Raul, can you take a look now?

lets wait for CI and MERGE!

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_kubeburner
Pass
0
00:18:58


Change in phoenix-shared-interface: Every usage of bool now aliases to int

This change has been made in phoenix-shared-interface for improved compatibility with C and other language's C-interop facilities. Relevant commit is https://github.com/GothicKit/phoenix-shared-interface/commit/60b5affc524f9ad39b38d4df0eb453ba578deb28. Adjecent commit changing usages of raw bool to PxBool is https://github.com/GothicKit/phoenix-shared-interface/commit/b2af7fc7bf1be0f7ff2e23485f8c4a061ada94be.

Thanks for the reminder. Implementation changed with: https://github.com/GothicKit/phoenix-csharp-interface/pull/27
Basically we could remove [MarshalAs(UnmanagedType.U1)] as the default of PInvoke for bool is 4byte anyways. 👍


Add rewrite to https to choice 3

Somehow it's left out.

Fixed in https://github.com/nozel-org/freebsd-apache-sparry/commit/22769405abd8883df67c6e23ffef2b85a31357ef.


Merged commonlib.aar into mobilertc.aar in the latest sdk version: v<IP_ADDRESS>33

Merged commonlib.aar into mobilertc.aar in the latest version: v<IP_ADDRESS>33,
please do compatibility with the latest version of Zoom SDK,
in the version: flutter_zoom_sdk: ^1.1.0+4, it is throwing commonlib.aar file not found error.

i am also facing this issue on flutter_zoom_sdk: ^1.1.0+4 when i run your provided example
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:checkDebugAarMetadata'.

Could not resolve all files for configuration ':app:debugRuntimeClasspath'.
Could not find :commonlib:.
Searched in the following locations:
- file:/C:/Users/DELL/AppData/Local/Pub/Cache/hosted/pub.dev/flutter_zoom_sdk-1.1.0+4/android/libs/commonlib.aar
Required by:
project :app > project :flutter_zoom_sdk
Could not find :mobilertc:.
Searched in the following locations:
- file:/C:/Users/DELL/AppData/Local/Pub/Cache/hosted/pub.dev/flutter_zoom_sdk-1.1.0+4/android/libs/mobilertc.aar
Required by:
project :app > project :flutter_zoom_sdk


Move  HAX /api/ to own project repo

We've gotten big enough in that repo that we're hitting limitations

jr devs can't run vercel dev bc it's too big for local development without a paid account
storybook build issues directly influence our ability to update apis; not a massive deal but annoying
apis should be versioned independent of front end asset builds if needed

@zdodson21 :point_up:  you should be able to call this up locally and run it now for when your doing your testing instead of pointing to prod urls if you want


Missing Microsoft.SqlServer.DTSRuntimeWrap, Version=<IP_ADDRESS>

Hello Seddryck,
While I'm trying to load a package from the filesystem:
<setup>
            <etl-run name="Generated.dtsx" path="..\SSISETLFramework.SSIS\">
            </etl-run>
</setup>

I keep getting this error:
NBi.NUnit.Runtime.TestSuite.Generated:
Exception during the setup of the test: Not possible to load file or assembly 'Microsoft.SqlServer.DTSRuntimeWrap, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=89845dcd8080cc91' or one of it's dependencys. The system was unable to locate the file.
   in Microsoft.SqlServer.Dts.Runtime.Application..ctor()
   in NBi.Core.SqlServer.IntegrationService.EtlDtsRunner.Run() em c:\Program Files (x86)\TeamCity\buildAgent\work\abf28191c66df2d3\NBi.Core.SqlServer\IntegrationService\EtlDtsRunner.cs:line 24
   em NBi.Core.SqlServer.IntegrationService.EtlRunner.Execute() em c:\Program Files (x86)\TeamCity\buildAgent\work\abf28191c66df2d3\NBi.Core.SqlServer\IntegrationService\EtlRunner.cs:line 22
   em NBi.NUnit.Runtime.TestSuite.ExecuteSetup(SetupXml setup) em c:\Program Files (x86)\TeamCity\buildAgent\work\abf28191c66df2d3\NBi.NUnit.Runtime\TestSuite.cs:line 124

In the machine where I'm developing the tests there is only SQLServer 2016. As for as I know that dll version is for SQL 2014. Do I need to install the dll on the machine?
Thanks

Ok, I've found another issue that mentions adding the version="SqlServer2016" that solves my problem.
Please consider adding this to the documentation.
Thanks

Thx for feedback ... Isn't it clear enough? http://www.nbi.io/docs/etl-define/#define-an-etl

Sorry I was looking at the samples and didn't see that.
Thanks for your time :)


Fixing name of touch function

Fixing wrong name in exported function

Aha, I should have caught that in the first merge, thanks for finding!


how to use checkpoints and log to continue training?

what's more, I'm curious how to continue training if my program was stopped by accidents! so how to use checkpoints to continue training?
thks! looking forward to your reply !

Hi, Xuanjie
For resume the training, yes you can do that.
See here: https://github.com/a1600012888/MUTR3D/blob/main/tools/train_tracker.py#L27
If you just want to load the one of the checkpoint and start a new training schdule, you can change the load_from in config file https://github.com/a1600012888/MUTR3D/blob/main/plugin/track/configs/resnet50_fpn_5frame.py#L270

Hi, Xuanjie
For resume the training, yes you can do that. See here: https://github.com/a1600012888/MUTR3D/blob/main/tools/train_tracker.py#L27
If you just want to load the one of the checkpoint and start a new training schdule, you can change the load_from in config file https://github.com/a1600012888/MUTR3D/blob/main/plugin/track/configs/resnet50_fpn_5frame.py#L270

thks！！！


Bug: new prompt not generated for each image

Dynamic prompts is only generating a single prompt for my image batches, no matter what boxes I select. Also of note, it doesn't generate new prompts even if I select "Combinatorial generation." For example, running a prompt of {red|blue}{cat|dog} with a depth of 4 will just give me 4 red cats. I am running this in automatic1111. I've tried deleting the extensions folder and reinstalling this extension, updating A1111.

Do try again as this same problem, which only surfaced yesterday, seems to have been fixed somehow, at least in my case. The prompts during batch processing are not repeating anymore; there be randomness again! I've got vladmendic's SD.NEXT automatic1111 fork, more specifically version d1ab205d.

@SteMoeller @MrUrzza can you confirm if your problem has been resolved?

No, bug still exists...Just deleted everything and reinstalled. I even tried to manally update dynamic-prompts from the installed 0.25.2 to 0.27 (by changing the pyproject.toml) but that didn't help either.
If I create one image after another the prompt is random. If I use batch size > 1 or batch count > 1 or combinatorial generation (no matter what numbers) all prompts are the same. This is also the case for the csv files written if enabled: prompt is identical for ALL images.

I can't re-create this issue. Are there any useful messages in the terminal? Are you using a colab or are you running locallly?

I am running locally... Can I add some kind of logging messages to the underlying dynamic-prompts python files to find the place where the "selection" of the prompts take place?

The problem seems to be a general one... even when I use the example "A diamond ring set on a {gold|platinum} band." I get the following identical prompts in the csv file:
positive_prompt,negative_prompt
A diamond ring set on a {gold|platinum} band.,
A diamond ring set on a gold band.,
A diamond ring set on a gold band.,
)

@adieyal It was working in Vlad 1111, even when it was not in Automatic 1111. Looks like its working in automatic now.

Now it appears to be broken in vlad, with the variables not being assigned. Meta data in the images for the prompt {red|green}{dog|cat} comes back with  {red|green}{dog|cat}, not something like red dog. The following error appears in cmd on startup. This just start to happen when I updated sd dynamic prompts
┌───────────────────────────────────────── Traceback (most recent call last) ─────────────────────────────────────────┐
│ C:\Users\Urzza\automatic\extensions\sd-dynamic-prompts\install.py:21 in get_requirements                              │
│                                                                                │
└─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
FileNotFoundError: requirements.txt not found.

The above exception was the direct cause of the following exception:

┌───────────────────────────────────────── Traceback (most recent call last) ─────────────────────────────────────────┐
│ C:\Users\Urzza\automatic\modules\scripts.py:246 in wrap_call                                                          │
│                                                                                │
│   245 │   try:                                                                                │
│ > 246 │   │   res = func(*args, **kwargs)                                                                           │
│   247 │   │   return res                                                                                │
│                                                                                │
│ C:\Users\Urzza\automatic\extensions\sd-dynamic-prompts\sd_dynamic_prompts\dynamic_prompting.py:106 in ui              │
│                                                                                │
│   105 │   │   callbacks.register_pnginfo_saver(self._pnginfo_saver)                                                 │
│ > 106 │   │   callbacks.register_prompt_writer(self._prompt_writer)                                                 │
│   107 │   │   callbacks.register_on_infotext_pasted(self._pnginfo_saver)                                            │
│                                                                                │
│                                               ... 1 frames hidden ...                                               │
│                                                                                │
│ C:\Users\Urzza\automatic\extensions\sd-dynamic-prompts\install.py:73 in get_dynamic_prompts_version                   │
│                                                                                │
│                                                                                │
│ C:\Users\Urzza\automatic\extensions\sd-dynamic-prompts\install.py:31 in get_requirements                              │
│                                                                                │
└─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
Exception: Failed to parse requirements file.
Create LRU cache (max_size=16) for preprocessor results.
16:56:02-430461 ERROR    Calling script:
                         C:\Users\Urzza\automatic\extensions\sd-dynamic-prompts\scripts\dynamic_prompting.py/ui:
                         Exception
┌───────────────────────────────────────── Traceback (most recent call last) ─────────────────────────────────────────┐
│ C:\Users\Urzza\automatic\extensions\sd-dynamic-prompts\install.py:21 in get_requirements                              │
│                                                                                │
└─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
FileNotFoundError: requirements.txt not found.

The above exception was the direct cause of the following exception:

┌───────────────────────────────────────── Traceback (most recent call last) ─────────────────────────────────────────┐
│ C:\Users\Urzza\automatic\modules\scripts.py:246 in wrap_call                                                          │
│                                                                                │
│   245 │   try:                                                                                │
│ > 246 │   │   res = func(*args, **kwargs)                                                                           │
│   247 │   │   return res                                                                                │
│                                                                                │
│ C:\Users\Urzza\automatic\extensions\sd-dynamic-prompts\sd_dynamic_prompts\dynamic_prompting.py:106 in ui              │
│                                                                                │
│   105 │   │   callbacks.register_pnginfo_saver(self._pnginfo_saver)                                                 │
│ > 106 │   │   callbacks.register_prompt_writer(self._prompt_writer)                                                 │
│   107 │   │   callbacks.register_on_infotext_pasted(self._pnginfo_saver)                                            │
│                                                                                │
│                                               ... 1 frames hidden ...                                               │
│                                                                                │
│ C:\Users\Urzza\automatic\extensions\sd-dynamic-prompts\install.py:73 in get_dynamic_prompts_version                   │
│                                                                                │
│                                                                                │
│ C:\Users\Urzza\automatic\extensions\sd-dynamic-prompts\install.py:31 in get_requirements                              │
│                                                                                │
└─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
Exception: Failed to parse requirements file.

Found a copy of requirements.txt and put it into sd-dynamic-prompts. Fixed the error, but not sure why it was missing. Died after updating via the webui. @adieyal


Updated point-like IRFs specifications

Now allowing a keyword within point-like IRF HDUs and an additional HDU with the description.
Still to do:

proper links to the different headers
add an example script
?

@TarekHC - Thanks!

Yes, having one example file in the right format is very important. Please add a Python script and FITS file and link to it, like we do in other places.

I see you're defining formats for 2D and 3D arrrays, like we have for background. I would suggest to just define the 2D format for now, and remove the 3D format. That would be equivalent to what we have for AEFF. Having RAD_MAX cuts really only makes sense when you have AEFF. So if that particular 3D format is needed in the future (I doubt it's a good set of axes and will be used), it could be added later, in a PR adding it for AEFF and RAD_MAX, and maybe EDISP as well.

If you can put a rendered HTML version up on the web somewhere, it helps with review and to get good feedback on the new version.

Yes, having one example file in the right format is very important. Please add a Python script and FITS file and link to it, like we do in other places.

Ok, I'll do it by today probably.

I see you're defining formats for 2D and 3D arrrays, like we have for background. I would suggest to just define the 2D format for now, and remove the 3D format. That would be equivalent to what we have for AEFF. Having RAD_MAX cuts really only makes sense when you have AEFF. So if that particular 3D format is needed in the future (I doubt it's a good set of axes and will be used), it could be added later, in a PR adding it for AEFF and RAD_MAX, and maybe EDISP as well.

You are absolutely right. We should separate those pull requests. Although I do not agree with the fact that it will not be used... With CTA FoV size, radial symmetry will very likely not be good enough to reach the systematic uncertainty required. If I have time, I might do the pull request after we merge this one.

If you can put a rendered HTML version up on the web somewhere, it helps with review and to get good feedback on the new version.

Yes, I think I can also do that.

With CTA FoV size, radial symmetry will very likely not be good enough to reach the systematic uncertainty required.

Agreed. But we don't have MC simulation with sufficient statistics to map out the field of view now for any existing IACT or CTA, and when someone starts doing it, I think it's likely that they will do it brute force and just for a grid of DETX, DETY. It's very inefficent to produce or also to store it that way.
So I would really suggest we wait to see what people do, and then only define formats that are motivated by a use case of at least one person producing such data and having a little bit of experience with it and to show that it's really useful. Trying to first write the spec without prototyping isn't a good idea in general, and we already saw this here with the point-like format, which we added to the spec first, but not in a good way, because we didn't have the experience from writing and reading such files.

Hi again,
You may find the rendered version of the specs here:
http://www.gae.ucm.es/~thassan/gamma/irfs/point_like/index.html
I removed the rad_max_3d format and  added an example fits file + script to generate it. Please review or merge if you agree with everything.

@cdeil I think it makes more sense to merge now. The point-like part was not usable as it was, so better to make it usable asap, even if it is not used yet. @jjlk will see if this is good enough or it needs to be improved.

I see, the incorrect statement on THETA binning was copy & paste from
http://gamma-astro-data-formats.readthedocs.io/en/latest/irfs/full_enclosure/bkg/index.html
@TarekHC - OK to remove it from there also? If yes, can you do it or should I?

Sure, no problem.


3.1.3 RE3, room 218, Hitting dogs with some weapons crashes game

Version
3.1.3
Seed
R730-C4ZH-ZXHH4HMJ9Y000QJZZZZZZZ1ZZZZZZZZZZZZZZZZZZZZZ
Scenario
RE 3 (Jill)
Player Character
Alfred
What happened?
In room 218, when shooting a dog with a magnum, a rocket launcher, or with a shotgun E at point blank, game crashes. Probably crash happens when the dog's head is supposed to explode. Other room (402) containing dogs in this seed works fine without crashes.

I found out that even hitting dogs with an Assault Rifle in room 218 crashes game in my seed. When using Assault Rifle, first shot knocks off a dog, second shot crashes the game. Maybe, problem is not in the head explosion.


Dynamic semver from git describe

@AnnaShaleva, what do you think? We can make it the first commit after v0.99.3 (hopefully it works from Windows) and then leverage it in many ways. It looks bad, I know. But maybe worth doing. Then we could even drop *-pre tags completely after some time.

Needs to be rebased to include abbrev changes.

I'll commit it manually, the old way. And we'll have better ways after that

Couldn't pass by without trying it myself
git describe --tags --match "v*" 2>/dev/null | `IFS=. read -a x; p=${x[2]/[0-9]*-/-pre-}; p=${p##[0-9]}; x[2]=$((${x[2]%%-*}+1))$p; echo "printf "%s.%s.%s" ${x[@]}"`

NVM, it doesn't work on the tag itself.

@fyrchik, bashisms detected!

In fact the best tool to do this is awk, but I won't even try pretending like I know something about it except that it exists.

Committed and pushed, let's see how it goes. Can be optimized in any way if need be.


After "stopRecording" camera still busy.

[system info]
window10
nw.js v0.27.3
Node v9.3.0
Chromium 63.0.3239.108
Hi,
startRecording and stopRecording are working very well, but my webcam is still in working after stopRecording function called! *webcam LED does not turn off.
So, I tested that start & stop recording in my nw application and open RecordRTC demo page which is "https://www.webrtc-experiment.com/RecordRTC/", click the "Start Recording" button then console said "(index):391 onMediaCapturingFailed: {name: "NotReadableError", message: "", constraint: "", toString: ƒ}"
any idea?

I think the AudioContext needs to be stopped/cleared as it's still listening to the device, off the top of my head now I can't remember how it's done. But I'm pretty sure it's something to do with that, if that's what you mean?

@sctaix please check this page:
chrome://settings/content/camera?search=camera

Make sure that webrtc-experiment.com isn't added in the blocked list.

I have a same problem with multiStreamRecorder, then i try to recreate multiStreamRecorder befor every record attempt and after some iterations i got error : Failed to construct 'AudioContext': The number of hardware contexts provided..
I search for AudioContext.close()  and find that it was fixed a month ago but last release at dec 2017. I think it`s a time to create a new release.

Did you ever figure this out? When  I start a record and stop it the camera is still showing busy like its recording. How can I explicitly say to turn the camera back off?

@shogan94 you can try this:
recorder.stopRecording(function() {
    var blob = recorder.getBlob();
    // now manually stop the camera
    cameraStream.getTracks().forEach(function(track) {
        track.stop();
    });
});

cameraStream

@muaz-khan what cameraStream refers here ? is it same stream we get from navigator.getUserMedia


pylance adding problems to my project

Environment data

Language Server version: 2020.12.2
OS and version: Macos Big Sur version 11.1
Python version: Python 3.9.0 64-bit (venv)

Expected behaviour
I should not see any problems about files from the extension
Actual behaviour
I am getting 50 or more problems from /Users/*******/.vscode-insiders/extensions/ms-python.vscode-pylance-2020.12.2/dist/bundled/stubs/matplotlib/init.pyi
(the asterisks are my username which i do not want to share
Logs
No logs
XXX

Code Snippet / Additional information

XXX

Are you getting problems for these even when the file is unopened? What are your settings? Are you using a type checking mode other than the default? What are the problems themselves?

Are you getting problems for these even when the file is unopened? What are your settings? Are you using a type checking mode other than the default? What are the problems themselves?

I tried changing the location of the project and it seemed to fix the issue.
I actually don't get this problem. should I close the issue?

I tried changing the location of the project and it seemed to fix the issue.
I actually don't get this problem. should I close the issue?

If you can't reproduce it anymore, then close it, yes. I'm wondering if maybe you opened up your home directory such that we were treating the bundled files as "in the workspace" or something.

If you can't reproduce it anymore, then close it, yes. I'm wondering if maybe you opened up your home directory such that we were treating the bundled files as "in the workspace" or something.

This issue is closed because I cannot reproduce it any more. if you have the same issue reopen thisissue

This issue is closed because I cannot reproduce it any more. if you have the same issue reopen thisissue


⚠ API - Lens has degraded performance

In 6c5dfdf, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 503
Response time: 360 ms

Resolved: API - Lens performance has improved in 87275b3 after 8 minutes.


Allow to send payload from search to execute

I started to write the hain-plugin-alias, an alias plugin for hain.
Basic idea is to define alias-names that start commands, like "ssh" starts "putty.exe -ssh". Now I need to be able to send commandline parameters from the search to the execution.
What I would like to do is search: "ssh my.hostname.tld"
Which runs "putty.exe -ssh my.hostname.tld".
Any ideas how to achive that?
Plugin-URL: https://github.com/schmic/hain-plugin-alias

You can pass whole query to execute function by assigning whole query as payload in search function.
and you can use Regular Expression to parse query.
or you just find the first whitespace character and split text into two parts using the whitespace, then the first part will be command and the rest will be parameters.

@appetizermonster I already did excactly like you said. Cheers.


An option to disable auto update feature

Autoupdate feature seems great for rapidly growing extensions like this, but sometimes it makes problems for people with low-speed internet connections.
I am looking for a solution to disable the auto-update feature of python language server. Is there any option to disable it? I need to update it whenever I want, not in the middle of my work or when I'm in a rush!
Thank you

I would like to contribute to this feature. How I can help to implement this feature?

We have recently released a new and more performant language server called Pylance which likely solves these issues you are reporting.  You can download Pylance from the marketplace and install it alongside the core Python extension. Because our team's development efforts have shifted to support the new language server, we'd encourage you to try it out.

Closing in favor of https://github.com/microsoft/vscode-python/issues/7084#issuecomment-740926004


RemoveSound stuck in an infinite loop

Hello,
removeSound is getting stuck in an infinite loop.  it does remove the item from the cache, but then there's an infinite loop logging: WARN "Phaser.Cache.isSoundDecoded: Key "sfx_xxx" not found in Cache."
cheers

I believe this was caused because you didn't correctly destroy any Phaser.Sound instances that relied on the cached audio file before calling removeSound. I've added in a small guard to protect against this in the dev branch, but it should still be dealt with correctly in the game code imho.


I recently started being shown Community Notes in Spanish when connecting from a Mexican IP with no obvious way to see the English version, though my phone's language is English and I prefer English notes.

It would be a better UX to default to showing notes in the language of the device and even better to allow the user to select their preference order for notes and best to expose notes in all available languages to the user.

The title was meant to be part of the comment.  I tried to fix it and change the title to "Notes Language Selection", b it don't have the bits to even edit my comment, let alone the title.
So restrictive the settings here!

Hi @beej, thanks for reporting this case.
Could you share with us what languages you have selected on this screen?
https://twitter.com/i/flow/language_selector

Thank you!
If you uncheck Spanish on that screen, it should (after a delay) stop showing Community Notes in Spanish for you.
But that setting controls your entire experience on the platform, so you would see less Spanish content in general.
Is that ok, or are you looking for a way to filter Spanish content only from Community Notes?

Thanks.  That's helpful, but I think the system should default to showing Community Notes in the user's preferred language when available.
Don't you?
The post I noticed this on has Community Notes in at least English and Spanish as I'd previously been shown the note in English, presumably before the Spanish note had been published.

To be clear, I'm not looking to filter anything.  I'm looking for users to have control over which language note is shown when multiple notes are available.

Thanks beej.
That setting is the app's main way to know which languages you prefer. And it looks like you selected Spanish at some point while using it.
We use that setting to determine which notes we show you in the Community Notes timelines, as well as which notes are sent to your notifications asking for ratings.
There is one case we don't yet solve for: If a specific Tweet has multiple notes rated as Helpful, and they are in different languages, we show them randomly. This is an edge case we didn't anticipate and will be fixing soon. We'll show the note based on the settings selected above.
Let me know if there's anything else we're missing.

Great!
Nope, you're not missing anything.
Will this issue be used to track the code changes?

We're not using this repo to track projects and changes to the product.
But we've discussed the edge case I mentioned and have added it to our (long!) list of things we need to get done.
I will close this for now, as it seems like we know what is happening and have a path to resolution.
Thank you and let us know if anything else comes up, we can always re-open this issue.


Color differences

There are high color differences in Edge and FireFox. Edge works surprisingly better then FireFox with the graphics and the colors. The main difference is the subdivision of the single windows with input, output and plotting. The view of the windows is highly different as the view in Edge.

We have to see whether this can be corrected.

We've corrected the appearance for all browsers. The FireFox style is not as modern as the other ones but now more modern than before. The style of the tool in FireFox can't be that modern as in Edge for example, because FireFox has not standardised their HTML styles for input forms like range-sliders yet.


What you need?

It's not clear from the instructions where I am supposed to install all the software and do the cloning - on my local environment or google cloud?

@Srikanth421 all of the requirement should be installed in your local environment.


Make port more random for test_socket_with_bind_to_interface test

Issue #, if available:
#665
Description of changes:
The error that we are getting is that the address is already in use. Try randomizing the port to something other than 1024. We don't encounter issues with other tests that use ports starting with 8, so this should work.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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

Project coverage is 80.32%. Comparing base (e5fe40e) to head (a6c2a0e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #666      +/-   ##
==========================================
- Coverage   80.39%   80.32%   -0.07%
==========================================
  Files          28       28
  Lines        5992     5992
==========================================
- Hits         4817     4813       -4
- Misses       1175     1179       +4

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


add remote predict thread pool

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

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

This PR is not running integration test workflow
Can we add corresponding unit test?

Re-open to trigger the corresponding github actions.


Og convolute divisor

What's Changing and Why
This addresses FR #763 for convolution divisors. Apologies if it's not 100% how you'd have done it but I've tested two scenarios and it works as expected.
You could change the order of the arguments so that this is near the end. Honestly, I don't love these anonymous incoming variables which might be this or that depending upon the order. :P
What else might be affected
Testing should be performed when the user includes the other optional arguments.
Tasks

[ ] Add tests (?)
[x] Update Documentation
[ ] Update jimp.d.ts (?)
[ ] Add SemVer Label (?)

Test code I used
# Within the context of a Node Express app's router in this case...
router.get('/', function(req, res, next) {
  var kernel =  [
                  [1,  4,  6,  4, 1],
                  [4, 16, 24, 16, 4],
                  [6, 24, 36, 24, 6],
                  [4, 16, 24, 16, 4],
                  [1,  4,  6,  4, 1]
                ];
  var divisor = 256;
  Jimp.read('test.png', function(errRead, fileIn) {
    if (errRead) throw errRead;

    // Also tested this by omitting the divisor argument below
    // which results in a very white output, as expected
    fileIn.convolute(kernel, divisor, function(errConvolute, fileAdjusted) {
      if (errConvolute) throw errConvolute;

      fileAdjusted.write('blurred.png', function(errWrite) {
        if (errWrite) throw errWrite;

        res.render('index', { title: 'Done' });
      });
    });
  });
});

@OutsourcedGuru This is currently a breaking change since the new prop must come first. If you put divisor as the last argument I am pretty sure you can achieve a non breaking change. For this to get merged you must at least get the old tests passing


Unable to load the Pretrained Models

When I am trying the to load the pretrained models , it gives an error stating that there are missing keys in the state dictionary.

Sorry, I uploaded the file old. I modified it. If you have any problems, please let me know again.

Thank you . It works now . If you dont mind could you also add the pretrained models for the other architectures i.e tiny , base and the large version (x2, x3 , x4) . I suppose you have added for tiny_x2 alone .

Sorry. I uploaded the wrong file. Thank you for letting me know. I uploaded a new file. If you have any problems, please let me know again. Thank you.

I am able to load the rest of the models as well. Thank you .

@abreorussel  Brother, can you run the test set through its pre training model? I have been unable to run. I don't know what the structure of his test set is and where the test set is located. If you can run, can you show me. Thank you, brother.


Not able to install with setuptools

It looks like the BUILD file conflicts with the build/ directory setuptools attempts to create on a case insensitive filesystem. Not sure how we can get around this? The same applies to pesos, too.

no longer necessary as pants support has been removed.

:+1:


Hello, dnt work on colab

Traceback (most recent call last):
File "/content/refacer/app.py", line 17, in
refacer = Refacer(force_cpu=args.force_cpu,colab_performance=args.colab_performance)
File "/content/refacer/refacer.py", line 35, in init
self.__init_apps()
File "/content/refacer/refacer.py", line 94, in __init_apps
sess_swap = rt.InferenceSession(model_path, self.sess_options, providers=self.providers)
File "/usr/local/lib/python3.10/dist-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 383, in init
self._create_inference_session(providers, provider_options, disabled_optimizers)
File "/usr/local/lib/python3.10/dist-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 424, in _create_inference_session
sess = C.InferenceSession(session_options, self._model_path, True, self._read_config_from_model)
onnxruntime.capi.onnxruntime_pybind11_state.NoSuchFile: [ONNXRuntimeError] : 3 : NO_SUCHFILE : Load model from inswapper_128.onnx failed:Load model inswapper_128.onnx failed. File doesn't exist

I know a colab that is using this Civitai link to the .onnx file and it's currently working
!wget https://civitai.com/api/download/models/85159 -O inswapper_128.onnx
Hope it helps


[Feature Request] Customisable scheduling algorithm

Use Case
We are building an I/O bound REST API that needs to fetch and aggregate data from other 3rd party APIs in real time. During development we found out that for some of these services just establishing a TCP connection and performing the TLS handshake could take to the tune of 500ms-1s, and with a traditional nginx+fpm setup we have to pay this price for every single request coming through our API. Switching to roadrunner makes it possible to leverage persistent HTTP/1.1 connections, so that when a worker manages to reuse pre-established HTTP connections to 3rd parties we are seeing total time savings of up to -70%.
Problem
Despite this any upstream HTTP server may see fit to close the connections themselves after some period of inactivity, usually at about 30-60 seconds. And since roadrunner allocates the requests to each worker in a round robin fashion, when we have a high number of workers coupled with low traffic any given worker tends to stay enough time inactive for its persistent HTTP connections to time out and lose the time savings.
I believe that for our use case a different scheduling algorithm would work better. One that would always try to allocate requests to the same workers unless they are busy with another one, something like "give the task to the worker with the smallest PID that is not busy right now"1.
Proposal
To that end it would be really helpful to be able to customize roadrunner's scheduling algorithm in its configuration file, something like this:
http:
  workers:
    pool:
      scheduling: "round robin" (default) | "other"

Our team doesn't have a lot of Golang expertise, but if you like this proposal I guess that with a bit of guidance we could have a stab at contributing this feature ourselves.

I reckon this scheduling algorithm must have a well-known name, but I can't seem to find it.

Hi, that's an interesting use-case and I will try to think it through in details this weekend.
I have a couple of questions:

How unified are API calls you make?
Do you have to use PHP curl queries or you change them (a.k.a. do you have any abstraction mechanism which can be easily swapped).

What you are looking for is to alter the allocateWorker method in StaticPool implementation. This is hard to do since the current implementation is performance optimized stack, but possible.
However, another option you might want to check is to move HTTP calls at the golang end and use RPC server functionality, in this case, PHP would only be responsible for the mapping part of the flow. Since you are going to be relying on the server to make calls keep-alive connections will be shared between workers. The RPC penalty will be irrelevant considering the duration of the calls you make. Plus, the implementation of "CURL" like service in RR is much more straight forward and predictable by behavior (but you have to remember to set the max timeout on HTTP client in golang!).

FYI, the current mechanism is not round robin. It's stack in a FIFO form.

Each request to our API usually leads to 5-20  external HTTP requests to 1-3 different services (but these numbers are expected to grow), and are currently serving about 1.5 million daily requests. At the idlest times of the day we may still handle less that 1rps.

Yes, we are using an internally developed library that is an abstraction over curl_multi, and the actual implementation can be swapped because it implements an interface. It can send as much requests as we need concurrently, but their parsing is sequential. Nevertheless we managed to optimize the app to a point where over 99% of the time is spent inside curl_multi_select waiting for the remaining requests to finish.

OK. Have you considered the approach I suggest? In this case, you might achieve even better performance by reusing shared connection pool. It won't be complex to implement functionality similar to curl_multi_select on golang, but it depends on what type of calls you make.

If this approach won't work I can think of how to achieve allocation approach you would like to use.


Clarification for composite indexes

When using a query with a filter (name = "foo") and multiple sort orders (order by c.created asc, c.color), there must be two compound indexes  added. One for the order and one for the filter with order (and in this order), otherwise querying the DB will fail with an error. Please clarify this in the docs. Also it is not really clear, that those property paths must be case sensitive.

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

ID: f1c7d0e0-b932-085e-6816-0b2219a1f37b
Version Independent ID: b463dc0b-313c-d90a-b8aa-c2b64004f812
Content: Manage indexing policies in Azure Cosmos DB
Content Source: articles/cosmos-db/nosql/how-to-manage-indexing-policy.md
Service: cosmos-db
Sub-service: nosql
GitHub Login: @seesharprun
Microsoft Alias: sidandrews

Hi @bexxx ,
Thanks for sharing this feedback. We will look into this and update as appropriate.
Hi @seesharprun,
Could you please look into this request, thanks.


Spec import cleanup

Clean up spec imports.. based off of the kind work done here
There's one tricky thing in there.. I had to add a comment around partition, because it doesn't really work as a pipeable operator, and we need to fix that.

Pull Request Test Coverage Report for Build 8225

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

Totals

Change from base Build 8207:
0.0%

Covered Lines:
5777

Relevant Lines:
5958

💛  - Coveralls


rcl_executor_spin_some() returns error code 1 on microros service server during CLI invocation

I am running a custom ROS2 service on an ESP32 using the micro_ros_arduino library. The service shows up in the shell via ros2 service list as well. On the micro-ros side, I am initializing the service and message data objects using a memory buffer via the micro_ros_utilities_create_static_message_memory() API. This might be a side-effect of issue #1090 but I am not sure, hence this bug report.

Hardware description: esp32
RTOS: arduino
Installation type: git clone + docker build
Version or commit hash: humble

Actual behavior
When I invoke this service from the CLI, I see this initial output on the shell:
requester: making request: my_custom_svc.srv.MyRead_Request(queries=> [my_custom_svc.msg.MyReadQuery(name='something', aspects=['something-else'])])

.. and then nothing. It hangs waiting for a response from the server, which never returns.
On the server, I get an error return code (value 1) from the rcl_executor_spin_some() invocation.
At the exact same time, here's what I see as the output from the micro-ros agent (running with verbosity = 6):
[1657929970.659653] debug    | Replier.cpp        | read_fn           | [==>> DDS <<==]        | client_key: 0x00000000, len: 64, data:  0000: 01 0F 2E 32 1A A6 5D 76 01 00 00 00 00 00 12 03 00 00 00 00 01 00 00 00 01 00 00 00 0C 00 00 00 0020: 75 6C 6E 61 72 5F 6A 6F 69 6E 74 00 01 00 00 00 09 00 00 00 70 6F 73 69 74 69 6F 6E 00 00 00 00

[1657929970.659886] debug    | UDPv4AgentLinux.cpp | send_message     | [** <<UDP>> **]        | client_key: 0x196442D4, len: 76, data:  0000: 81 80 04 00 09 01 44 00 00 0C 00 08 01 0F 2E 32 1A A6 5D 76 01 00 00 00 00 00 12 03 00 00 00 00 0020: 01 00 00 00 01 00 00 00 0C 00 00 00 75 6C 6E 61 72 5F 6A 6F 69 6E 74 00 01 00 00 00 09 00 00 00 0040: 70 6F 73 69 74 69 6F 6E 00 00 00 00

[1657929970.686920] debug    | UDPv4AgentLinux.cpp | recv_message     | [==>> UDP <<==]        | client_key: 0x196442D4, len: 13, data:  0000: 81 00 00 00 0A 01 05 00 05 00 00 00 80

Additional information

What are some general causes for error code # 1 from this method?
Is there any tool I could use to interpret the hex data from the agent output above?
On looking at the rcl_executor_spin_some() code, there are a lot of PRINT_RCLC_ERROR statements, but I don't see any error logs via the serial port. How should one go about enabling and viewing such microros error outputs via the serial port?

Provide code for replicating

We should probably fix https://github.com/micro-ROS/micro_ros_arduino/issues/1090 first if its related to the problem.
Then, we will take a look if the problem persist.

Closing for the time being since I haven't been able to reproduce this since #1090 was fixed. Will re-open if that changes.


Added Missing ResponseCode.

Hi.Can you merge this in your base branch please.
Also maybe it would be usefull to set a Travis  CI job

Yeah, a Travis job would be good. I'll look into setting something up. Thanks for the PR


Call to a member function getAddress1() on a non-object (1)

My environment is concrete5 v8.0.3 with Community Store v1.0.1. The store is configured with a single product.  After checkout, using invoice payment, attempting to view the customer via Members, by clicking on the username, results in the following exception:
Exception Occurred: /[...]/concrete/attributes/address/controller.php:330 Call to a member function getAddress1() on a non-object (1)
Data is entered into both address fields during checkout.
Other users that were added manually via Members can be viewed/edited.

I'm not ruling out community store's fault here, but from the sounds of it the error is showing on a concrete5 core dashboard page, with a core attribute type - perhaps it's a core bug with the attribute  (I know the address attribute type had a few issues).
Or have I misunderstood this and this is on a community store dashboard page?

You are correct, the exception is thrown on the core dashboard page.
I tested with with v<IP_ADDRESS> and the exception does not occur with that version. I've also verified that there is in fact an address1 record in the atAddress table, with appropriate foreign keys to the attribute tables.
I would tend to agree that it's something with the core. I'll continue to investigate.

Did a bit more debugging to try to determine the core issue. I haven't found it, but have determined that the getValue() method in the following segment is returning null:
$value = $this->getAttributeValue()->getValue();
$this->set('address1', $value->getAddress1());
...
The exception occurs on the second line. getAttributeValue() is returning a valid object, but getValue() returns null. Since I can see a valid address in the database, my sense is that this is a bug on the "getter" side of this. The attribute code has seen quite a bit of change since v5.7.
I did a fresh install, but the same exception occurred. Is anyone else running v8.0.3 with Community Store v1.0.6 and NOT having this problem?

I think this pull request may also fix this particular issue:
https://github.com/concrete5-community-store/community_store/pull/176

I just tested with concrete5 v8.1.0 and Community Store v1.0.8 with pull request #176. I completed another purchase and hit the same exception when trying to view the (new) user via Dashboard -> Members. I haven't had time yet to look more closely, but have attached the debug dump and call stack.
debug-out.txt
call-stack.txt

I just tested it again with that combo and I'm able to view the members page with the details I'd expect, both the billing and shipping address are complete with the values I used in the order.
Previously broken accounts are still there though, pull request #176 won't fix though, only the way new user attributes are applied.  Are you certain that you're viewing the recently created user, after you've marked the order as paid?  I've tested it both with direct payments and ones where they are marked paid afterwards.
You can tweak concrete5's /concrete5/attributes/address/controller.php file to not die on this error quite easily, you can change the function to look like this:
public function form()
    {
        $this->load();
        if (is_object($this->attributeValue) && $value = $this->getAttributeValue()->getValue()) {
            $this->set('address1', $value->getAddress1());
            $this->set('address2', $value->getAddress2());
            $this->set('city', $value->getCity());
            $this->set('state_province', $value->getStateProvince());
            $this->set('country', $value->getCountry());
            $this->set('postal_code', $value->getPostalCode());
        } else {
            $this->set('address1', '');
            $this->set('address2', '');
            $this->set('city', '');
            $this->set('state_province', '');
            $this->set('country', '');
            $this->set('postal_code', '');
        }
        $this->set('search', false);
        $this->addFooterItem(Core::make('helper/html')->javascript($this->getView()->action('load_provinces_js')));
        $this->addFooterItem(
            Core::make('helper/html')->javascript($this->getAttributeTypeFileURL('country_state.js'))
        );
        $this->set('key', $this->attributeKey);
    }

(but I was testing with the default code)


Problems with output name for wavelength calibrated files

The wavelength calibration has a system for naming the wavelength-calibrated files that is somewhat obscure.
The class WavelengthCalibration has two methods to save files
_save_science_data and _save_wavelength_calibrated the first call the second but the second is called from the same method that calls the first. So it is confusing.
Needs a full review but basically the problem is that it relies on the previous file name because in the non-live version the name is constantly updated after any of the methods that perform any process in the reduction and the inconvenience is that when called from an external tool the file name which is stored in GSP_FNAM has no differentiation factor which results in a file being overwritten.

Fixed in #310


Save-DbaUpdate

it would be awesome to have a Save-DbaUpdate that saves the updates required for a server. So specify the server, it gets the version, builds the chain until the most recent patch, searches (maybe this? https://www.catalog.update.microsoft.com/Search.aspx?q=sql+server) and saves them all to a path.
It would also be helpful to be able to just specify the build.
This could help with both Update-DbaInstance and Install-DbaInstance
/cc @nvarscar what do you think? i believe you have a lot of this code in your stuff.

I'm not big at parsing web content, but having an option to download the patch is definitely a great idea and can be easily integrated into Update-DbaInstance. But how do you envision it being used in the Install command?

oh, that's true :/ i was thinkign it'd be awesome to have something that would go from nothing to ISOs and Updates but ISOs arent really available. So scratch that.
I'm not a huge fan of parsing s well, cuz it breaks. Maybe catalog has an RSS Feed. Ugh, it's supposed to but appears to throw an error

mind also that not every released CU stays downloadable forever....


IDVA6-1384 - Bug Fix - Navigating to previous page when changing languages

Link to Jira:
https://companieshouse.atlassian.net/browse/IDVA6-1384
Changes:

Updated accept/decline invitation url to function with apostrophes in company name
Added referrer url functionality to detect if user has changed the language on the page to prevent redirection to previous page

CI: No security warnings found


Have on*pressed() functions for classes like Entity, Component

Currently I'm doing if (Luxe.input.*pressed('...')) inside on*down(), having on*pressed() would save lots of function calls. Maybe have interfaces for the common event slot functions to ensure they're present where they can be used.
(Guess this is probably on the to-do list? But putting it here anyways.)
Edit: And maybe rename the on*up() functions to on*released(), to make them consistent with Luxe.input. Adjectives (down, up) refer to the current state of the input medium, while past tense (pressed, released) refer to the prior state.

Actually, you're using two different usage patterns and conflating them as one.
Don't call Luxe.input calls inside of the on handlers of the same type - that's a bit of an anti pattern.
There are two types: event based, and immediate query.
Event based is the callbacks, i.e on___ where ___ is the event.
Immediate query is "anywhere", where you ask for the state.
Secondly, up and released are not the same concept. Immediate queries include the concept of "released this frame" or "pressed this frame". It's possible that because input is dispatched when they happen, they are past tense because they're explicitly asking about a specific type of state.
"Is the key down" (or not down) doesn't care about the frame, only the state, so a key that is already held down would return true every frame. i.e, you'd get multiple esc releases per second (60 usually), not just the one. The released/pressed are set for one frame, rather than a state flag.
Generally, you use the event handlers, which exist on every class that usually needs them (State, Scene, Entity, Component, Game, etc). The immediate queries are just a different pattern, and have different uses.
Personally I avoid the immediate queries except for quick things, because it often leads to hamming input code where it shouldn't be and creating more rigid input mappings that can be harder to make configurable. Because the input events happen as soon as they are triggered, it also means event handlers have less inherent latency, your state will always be set as soon as it happens, rather than potentially in the next visible frame after.

Actually, you're using two different usage patterns and conflating them as one.
I'm aware of the distinction. I need to check for pressed and released. But I can't solely use event-based handling*, because Component doesn't support onpressed. If I use solely immediate query, I'd have to query every frame via update(), querying only when a key is down is still an improvement over that.

(*technically, I could, if I keep track of which frames the keys are down and up, and infer pressed from that. But that's just being silly.)

"Is the key down" (or not down) doesn't care about the frame, only the state, so a key that is already held down would return true every frame

Right. But Component's onkeyup() isn't called every frame when a key is up, it's called whenever a key is released. (I haven't tried the other classes, so I'm assuming the behaviour is the same for them) Granted, it's silly to fire an event whenever any of the keyboard's 100+ keys are up, so if you sit and think about it, you can tell what the function's supposed to do. But it's more natural to keep the naming consistent with Luxe.inputs use of '*release', and so not have to sit and think about it, or already be used to the naming inconsistency.

Reread your reply and did some testing. Turns out on*down() isn't called every frame a key is held down. It's called on the initial press (lower latency here), then after that only when Luxe decides it's time to 'repeat' the 'keypress' (so higher latency afterwards). Like when you hold down a character key in most text editors. In particular, when holding down a key, there's a significant pause between the first and second on*down() calls.
I think I understand what you were trying to explain. Luxe.input stores information about the state of the input during each frame (as I expected), whereas the *Event dispatches occur at Luxe's discretion.
Sorry for the confusion!
But:

it's still useful to have on*pressed(), that's only called when a key is pressed and not triggered again by that same key until after it's released and pressed again.
I assumed from the name on*down() that they were called every frame the input is held down, partly because of the similarity with Luxe.input's *down names.

Some additional notes,

All behaviour of key input and friends is de facto, these are OS level events, consistent across every OS, browser, etc. I haven't deviated from conventions in naming nor behaviour. I also won't be, as they are existing “standard” (and they aren't inconsistent in their layout, as they don't overlap in this way).

only when Luxe decides
at Luxe's discretion

Key repeats (and all other events) come from your operating system keyboard driver. The events are directly propagated into usable form, as they happen. There is no luxe code doing anything to the input, except for the immediate api's, which collect state and expose it for convenience.

it's still useful to have on*pressed()

A key down event is identical in behaviour here, the same key cannot send another key down until the key is released (logically/physically) with one exception - key repeat events. In the event of this specific "soft" case, the repeat flag on the key event is set to true. If you don't care for the repeat specific events, ignore those :)
I'd rather not add redundancy over an existing pattern. If anything, a config for rejecting repeated keys internally could be added but I generally prefer explicit vs implicit control, so it won't be added just yet. What you are after seems pretty clear, and right there. Since you mentioned you're abstracting this, you could always emulate the behaviour on your level as well.

Maybe the code comments should be more detailed

I can always add more details, sure.

All behaviour of key input and friends is de facto, these are OS level events, consistent across every OS, browser, etc. I haven't deviated from conventions in naming nor behaviour. I also won't be, as they are existing “standard” (and they aren't inconsistent in their layout, as they don't overlap in this way).

Key repeats (and all other events) come from your operating system keyboard driver. The events are directly propagated into usable form, as they happen.

Thanks for the info, I hadn't realized all this. I'm a hobbyist who's only recently started really looking into what industry does, often it feels like people respond assuming I have the same experiences they do (even when they know I'm new to things).
Btw I really appreciate all the detailed explanations! In the guides as well, e.g. back when I was trying to figure out Unity, an explicit description of the component pattern would have saved me a lot of time and frustration.

If anything, a config for rejecting repeated keys internally could be added but I generally prefer explicit vs implicit control, so it won't be added just yet. What you are after seems pretty clear, and right there. Since you mentioned you're abstracting this, you could always emulate the behaviour on your level as well.

Yeah. It's not a major problem, just a shame that currently my code will have to run lots of unnecessary function calls.

No worries, I try to be aware of that constantly, but the edge goes both ways, if you explain "obvious" things it's also demotivating (as is not explaining when needed).
In general all these types of detail and reasoning will be documented out in guide form, and will cover the whole base line of why too (the "design" links on the home page of the docs alludes to this) but as with much of the alpha status, these things are a work in progress and don't exist yet as a side effect of time.
There was a guide here which was outdated - but i've updated it now and will add more information in future.


go.mod: update cilium/ebpf

This includes the changes that have been merged upstream:
https://github.com/cilium/ebpf/pull/920
This should improve the performances of loading bpf modules.
This PR removes the temporary performance workaround introduced in the SocketEnricher (https://github.com/inspektor-gadget/inspektor-gadget/pull/1201) since it is now done by cilium/ebpf directly.
Note: deprecated tags in generated files are removed, see: https://github.com/cilium/ebpf/pull/888

This PR introduces the following CI failure:
=== CONT  TestEventEnrichmentRaceCondition
time="2023-02-15T17:19:47Z" level=warning msg="WithTracerCollection: failed to open mntns reference for container 16eb1f83-7f7c-47fb-86e0-b9ae9bc0106c: too many open files"
    gadgets_test.go:247: failed generating events: failed to run command: running command: open /dev/null: too many open files
--- FAIL: TestEventEnrichmentRaceCondition (2.50s)
=== CONT  TestContainerRemovalRaceCondition
    gadgets_test.go:176: failed generating events: failed to run command: running command: open /dev/null: too many open files
--- FAIL: TestContainerRemovalRaceCondition (2.60s)

But I don't know how it is related.
The tests pass fine on my computer.

Forget my previous message. The tests are not flaky and are always failing. I can reproduce it on my computer with $ go test -exec sudo -test.v ./....  I'll investigate further.

@alban would you mind rebasing on top of https://github.com/inspektor-gadget/inspektor-gadget/pull/1330? I think that will fix the issue.

@alban would you mind rebasing on top of https://github.com/inspektor-gadget/inspektor-gadget/pull/1330? I think that will fix the issue.

Done

It seems to bring other issues:

Gadget tcpconnect: falling back to standard tracer. CO-RE tracer failed: failed to load ebpf program: field IgTcpcV4CoE: program ig_tcpc_v4_co_e: map .rodata: load BTF: load btf: invalid argument: [1] DATASEC .rodata size=280 vlen=7: type_id=5 offset=272 size=8 Invalid type (45 line(s) omitted)
$ go run -exec sudo -tags withebpf ./cmd/local-gadget/... traceloop -r docker
Error: error creating tracer: loading ebpf program: field IgTraceloopE: program ig_traceloop_e: map .rodata: load BTF: load btf: invalid argument: [1] DATASEC .rodata size=264 vlen=9: type_id=5 offset=96 size=168 Invalid type (62 line(s) omitted)

The diff in cilium/ebpf:

git log ea299d354cf6..a478cd839367

Could be related to https://github.com/cilium/ebpf/issues/921?

I updated this PR so that it cherry-picks this PR:
collection: work around excessive copying of kernel BTF
https://github.com/cilium/ebpf/pull/920
But without taking all upstream changes due to:
btf: vmlinux .data..percu Datasec is refused by the kernel
https://github.com/cilium/ebpf/issues/921

Great! Still LGTM, feel free to merge.


Topic /dock publishing invalid type robot_create_msgs::msg::Dock instead of DockStatus

Bug report
Required Info:

Operating System:

OS: Ubuntu 20.04

Installation type:

Binaries and from Source

Version or commit hash:

release

DDS implementation:

Fast, and Cyclone using ROS2 Humble/Galactic

Client library (if applicable):

Steps to reproduce issue

On ROS2 Humble while connected to Create3 (eth/wifi)
ros2 topic echo /dock

Expected behavior
/dock topic echos as normal showing the information about the docking status
Actual behavior
undefined message type error
Additional information
I see that the dock message has changed from galactic to humble so I just wanted to put in an issue so that this change can be documented in the Create3 ROS2 setup documentation. In the installation instructions for ROS2 humble on a client, it could be noted that some messages will be incompatible. Thanks!

To verify -- is your Create 3 running Galactic (G.x.x) or Humble (H.x.x) firmware?
How have you installed ROS 2 Humble on your Ubuntu 20.04 machine?

Hey there, thanks for the quick reply. To be clear, my machine is running Humble on Ubuntu 22.04 while the create3 is running Galactic.
Yes I think it may be useful to add a note on the Humble and Galactic setup pages about making sure your ROS2 versions match otherwise there may be instability/message mismatches. That would be great but maybe people read all of the docs completely (which I clearly did not do, I apologize) before they get setup. I will say closed for now but feel free to add that note in the ROS2 setup! Thanks and enjoying the platform!


Vehicles Gain Speed From Floor Ice

Description

Driving vehicles over ice permanently makes them go faster/takes out their friction.

More specifically it seems to remove their acceleration, so they dont have to speed up to top speed. I’m guessing when it goes on the ice, the game makes it have that same slow acceleration that players walking on ice would get, and then when it comes off the ice it removes that slowed acceleration, but in the process would remove the slow acceleration inherent to the vehicle.
Also fast wheelchair go brrrrrr

Vehicles removed awaiting better implementation.


Updating the status of the data collection and the image of the host structures

@davidlmobley sampl_host_structures_coded.jpg has the name of each host listed underneath each structure and then one sampl_host_structures_1.jpg is just an image of the structures, no naming included.
I just deleted the old host cdx file and added sampl_host_structures_coded.cdx (has names) and sampl_host_structures_1.cdx (just the host images, no names)

And I added two PDFs, sampl_host_structures_coded.pdf (has host names) and sampl_host_structures_1.pdf (no host names, just structures)

Structures only, no names

Structures plus names

OK, @bergazin I'll approve and merge this one. But ultimately we want to progress towards a layout where there is a README.md explaining what each file is and linking to it. Adding files without explaining what they are will make it harder for people who want to find what they want.
(Check out the SAMPL6 repo for something laid out well and very functional, e.g. see this page https://github.com/MobleyLab/SAMPL6/tree/master/host_guest which has a README.md (which gets automatically displayed when you visit it) which says what's in each file. As a result of the good organization on this repo, the better part of 100 different people have used the repo for things and we almost never field questions on it... So we want to progress towards that here.)


[HOTFIX] [848] Listings dont appear in the published tab on their last day before expiry

Trello card URL:
https://trello.com/c/LMjeNx09/848-listings-dont-appear-in-the-published-tab-on-their-last-day-before-expiry
Changes in this PR:
Make jobs that expire today count as published.

 Listings don't appear in the published tab on their last day before expiry


Failed to get handle for right Joy-Con or Pro Controller, exiting

I've compiled it successfully but when I plug in the Pro Controller via USB and run uinputdriver, I get:

Failed to get handle for right Joy-Con or Pro Controller, exiting...

I'm on Ubuntu 18.04.1.

Hey I'm on Ubuntu 16.04. I fixed this by running uinputdriver with 'sudo'

I don't have Ubuntu installed anymore but that was probably my issue, closing


[exiv2] Update Exiv2 to 0.28.0

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags.

@CCInc, when I tested the features by command ./vcpkg install exiv2[*]:x86-windows I get this error.
Please take a look:
-- Could NOT find Python3 (missing: Python3_EXECUTABLE Interpreter)
    Reason given by package:
        Interpreter: Cannot use the interpreter "C:/Program Files/WindowsApps/PythonSoftwareFoundation.Python.3.10_3.10.3056.0_x64__qbz5n2kfra8p0/python.exe"

CMake Warning at cmake/findDependencies.cmake:35 (message):
  Python3 was not found.  Python tests under the 'tests' folder will not be
  executed
Call Stack (most recent call fi

Python3 was not found.  Python tests under the 'tests' folder will not be
executed

If possible, disable the tests, don't enable python.

Hi @JonLiu1993 , I made the changes you requested. The python issue is just a warning, they use python as a test suite runner, and we do not run the tests anyways, it should be safe to ignore.

@CCInc, I disabled to find Python3 but when I re-tested, I encountered a new error, could you please take a look?
https://github.com/Exiv2/exiv2/blob/22b65ccfea31921ef9f96d5ec627433d932364bb/cmake/findDependencies.cmake#L33-L36
#find_package (Python3 COMPONENTS Interpreter)
#if (NOT Python3_Interpreter_FOUND)
#    message(WARNING "Python3 was not found. Python tests under the 'tests' folder will not be executed")
#endif()

error:
CMake Error at xmpsdk/CMakeLists.txt:29 (target_link_libraries):
  Target "exiv2-xmp" links to:

    EXPAT::EXPAT

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

CMake Error at src/CMakeLists.txt:186 (target_link_libraries):
  Target "exiv2lib" links to:

    EXPAT::EXPAT

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

-- Generating done (0.1s)

@CCInc, could you please fix the conflicting files?

@JonLiu1993 I fixed building all the features available for Ubuntu, including the expat issue you mentioned.

Pinging @CCInc Can you address the review suggestions? Thanks!

Ping @CCInc for response.

Closing this PR since it seems that no progress is being made. Please ping us to reopen if work is still being done.


Fix format specifiers for %s and %q

I imagine this will need some more testing considering I still have barely any knowledge what I'm doing, I made it work in my own tests though so /shrug?
Fixes #812

well it works but the tests hate it, ok back to headscratching trying to make this work

Received feedback that these were designed to be explicit, gonna close it again lol


CPU Memory Visualization is Crowded

Currently, the CPU Memory visualization shows the address of EACH BYTE next to that byte's value.  Additionally, the addresses are not aligned to an e.g. 8- or 16-byte boundary.

This makes it very busy visually, and hard to read versus other formats.

You might even want to add column headers for readability by new users, e.g.:
     +0 +1 +2 +3 +4 +5 +6 +7 +8 +9 +a +b +c +d +e +f
210: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
220: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
230: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
240: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................

(with obvious style hints to show that the row / column headers are separate from data)

Thanks for the feedback!  I've added some thoughts below, per your request.

For those experienced with reading hex data ... better one for beginners

Someone unfamiliar with base-16 will not derive much use from any layout of hex-dumped data.  I
Anyone who has ever read hex-dumped data has likely seen it aligned to some boundary.
Your first blog post must necessarily assume a reader that Intel assembly, I expect they're comfortable with standard grids that start at 0, instead of starting at e.g. 0x24e.

The addresses are not supposed to align to 8- or 16-byte boundaries, it starts at the highest address of stack and grows with the stack pointer as the stack grows

Stack addresses are only unaligned by chance (I expect you just masked of the top 20 bits of a real program state).  It is entirely possible to fixup the (displayed) addresses to be aligned -- I'd actually recommend this.
Being aligned will likely aid comprehension and learning, since most people are familiar with grids, whether base-10 or base-16.

For this it's more important to me that the user understands exactly what byte is in a given address in the stack

I think this could be done very informatively with a mouseover and row/column headers.

I really do appreciate all your insights and feedback on this, I just don't happen to agree.
There is a reason I visualized it as I did initially and I stand by that decision.
However, there's definitely a place for the format you suggest, I'm very grateful for your feedback and it will be invaluable when I get to that.


skipCompile test

This is a draft PR to satisfy a request to allow people to allow files to skip compilation attempt while still being present for use as includes.  Specifically, this change would target includes like util_i_library that could compile on their own, but consistently throw errors because they don't have their own function definitions.  Not ready for merge.

Did some rework to make it more palatable.  Think it's ready for review.  Did testing on multiple files on windows and linux builds.  Unable to test macos locally.


Use Availability Zone IDs

Description
Use Availability IDs instead of names
Closes #234

@antonbabenko, sorry for the delay. i've got to testing this and seems ok to me.
I tested with the new added example and with the regular simple vpc example

Thanks @DrFaust92 for this PR!
I have simplified it a bit by removing the extra variable and updated an existing simple-vpc example.
v2.25.0 has been just released.


Eliminate the irritating "findDOMNode is deprecated in StrictMode" error.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

This is a well known bug with React Bootstrap, that is actually a bug in React Transitions. There are no plans of fixing it. There are two workaround. One - disable all the cool animations we use (sliding panes, sliding modals, etc...), which is not nice.
The other solution is not to run in Strict Mode. This will disable the warning, but also disable all the other very useful strict mode warnings. Strict mode warns about updating the state in the wrong place, updating state of unmounted components, etc... We use it quite a lot, disabling it will make our debugging lives a lot harder.
So I'm afraid we're going to have to live with that.


Not compatible with android !! Only errors !

I just wanted to us this package . But I failed to use this , using local file from phone's storage like file://..... .mp4 got error !
I already use  android:requestLegacyExternalStorage="true" , I knew about this . But Nothing works . So I used for remote file ,that is not also working !
Errors :
Exception in native call from JS
setDataSource failed : status = ................................
So , I want to know just how should the video source url be ????? How can I use this package ? Let me know please

Make sure your application have files and media permission (check app info also)


Add support for Bootstrap cards

I'd like to add a page for projects using the bootstrap card deck format. Is this feasible?

 Issue #16

Hi!
What you're asking for is not possible at the moment. Mainly because Terrassa does not have any type of support for Bootstrap.
But you can add your own CSS styles (you can see how here if you want) if that's enough to solve your problem.
Sorry for the inconvenience.


fixed the bad request url at banner

This PR fixes the bad resource link mentioned in issue https://github.com/facebook/create-react-app/issues/13649. The issue was that the app was making a request to the URL camo.githubusercontent.com, which resulted in a 404 page not found error. I downloaded the original image and added it locally to avoid any external requests, ensuring the image will load permanently. The link has been updated accordingly.
Issue https://github.com/facebook/create-react-app/issues/13649 is now resolved.

@akashprasher  can u please provide me more detail please

@akashprasher  can u check now i have removed the  {' '}

Please Merge this pull request ...!!

Can u please close this PR as soon as possible please


Testing mail client

Write scenarios and  automated test cases for mail client.
Deliverables

Write test cases in TestQuality and link them to the testing sheet.
Automate the test cases.
Integrate your work with ZeroCI.

Estimation Time

Once you explore the client, please set the estimation time and update the ticket

Estimate Time:
10 hours.

Trying to run SMTP server, then started in mail client manual test, and test scenarios.

Done with test case and test quality for (Email & sendgrid clients).


feat: open webview from image action

What does this PR do?
Removes the quick pick command, and instead triggers opening the webview, switching to
the build page, and preselecting the image.
Screenshot / video of UI
N/A
What issues does this PR fix or reference?
How to test this PR?
Tests still to-do.

Is this needed?

This is removing the quick pick and making all the paths go to the webview - isn't that the intention?

Added tests, and rebased on main. Ready for full review.


Change default config please

Can't build release APK because of old build version in build.gradle
Change:
`android {
compileSdkVersion 22
buildToolsVersion 22.0.1
defaultConfig {
    minSdkVersion 16
    targetSdkVersion 22
    versionCode 1
    versionName 1.0
}
lintOptions {
    abortOnError false
}

}To:android {
compileSdkVersion 28
buildToolsVersion 28.0.3
defaultConfig {
    minSdkVersion 16
    targetSdkVersion 28
    versionCode 1
    versionName 1.0
}
lintOptions {
    abortOnError false
}

}`

hi @Glatoo which  build.gradle file are you referring to?

Closing this issue, if you are still facing problems feel free to open


Camera flyto destination wrong with no default ellipsoid

The camera flyTo is not properly working when using a generic ellipsoid, for example a lunar one (ellipsoid radius: 1737400).
The final cartographic camera position is the one in WGS84 ellipsoid.
The problem is visible in the following sandcastle example.
The flyTo destination position is set to
(1737500, 0, 0)
The final  camera position when the flyTo finishes is
(6378237, 0, 0)
Sandcastle example:
sandcastle
Browser:
any
Operating System:
any

Just commited a trivial fix for this 9498


CreateZone for InMemory DNS provider

Allows to create a zone in inmemory dns provider. Needed for testing

👍


how to

hi, please can make a short how-to video or screenshots to install and use this script correctly? this is hard to understand beginners.

pls  do check my gui demo
first line in readme

https://youtu.be/Sqj0uO9VCy4 @susantha1995

https://youtu.be/Sqj0uO9VCy4 @susantha1995

thank you very much for your video.  I tried this one https://github.com/wanghaisheng/autovideo  but there is no any video uploaded. i followed all steps. please check the attached screenshot. I created a new profile on firefox and also added a cookie.json file. all added in GUI. but it didn't upload any video in my folder.
please can you check the screenshot to check the problem here?

thank you!

for video files you need prepare a video and thumbnail file
their name should be the same
and you should press the middle button to check whether they are in pair then press third button

for video files you need prepare a video and thumbnail file their name should be the same and you should press the middle button to check whether they are in pair then press third button

I prepare videos in a folder and added them to gui. but I didn't add thumbnails because I'm planning to upload 20-30 videos once so I can't create thumbnails. so without add thumbnails can't I upload videos??

yes  video image should in pair
in this demo gui  you should prepare thumbnail   what i decribe is the logic behind this demo
you can edit for sure

can I hire you to build a custom bot?

mail me at<EMAIL_ADDRESS>


FR - cp437 output?

Hello. I was wondering if it would be possible/interesting for you to implement a cp437 code page so I can host this on a telnet BBS? :)

It's not something that I've looked into before, but it sounds interesting.
I've got a couple of things on my to-do list, then I'll take a look at if it's possible / how much refactoring it would take.
Thanks for the suggestion!

Free Pascal default Video is cp437. It was ported to unicode recently (see https://www.mail-archive.com/fpc-devel@lists.freepascal.org/msg14512.html). To run in in cp437 the terminal should be set to cp437 and it should work, according to the source code: https://github.com/slibre/freepascal/blob/master/rtl/unix/video.pp


A11y tabbing issues

Version 2.2 master (11/26/18)
What kind of request is this (question/bug/enhancement/feature request):
Bug, Usability / Accessibility

[x] When you TAB and open a menu your next TAB does not go down the menu.
Screenshot example shows that we tabbed and opened the Tools menu, then when we hit tab again we went to the user menu, not the items in the menu itself.

[x] You can open multiple menus at a time when using TAB.

Version: Master (2.3) (5/3/19)
I tested this again following the steps to reproduce. Both of these areas are fixed now and I find the tabbing navigation to be much improved. I also like that I can tab past menus -- Only if I press the down arrow or space to open a menu do we tab thru the options before moving on to other navbar items and areas of the UI. This is perfect.


Update pokemon.json

This is better than saving locally

I saw some red portion @HoeenCoder can u fix them ?


Missing variable braces on $pagination-margin-start calc

Nope, it was just the PhpStorm that triggered the error and wanted to let you know...

I don't see any issue in our compiled code, so I'm inclined to leave this as-is. Will monitor if others run into issues though.

After compiling the scss (version 5.2 beta), the generated css will display '$pagination-margin-start' and '$table-border-width'.
to solve the issues:
in _variables.scss
$pagination-margin-start:           calc($pagination-border-width * -1) !default;
should be
$pagination-margin-start:           calc(#{$pagination-border-width} * -1) !default;
in _tables.scss
border-top: calc(2 * $table-border-width) solid $table-group-separator-color;
should be
border-top: calc(2 * #{$table-border-width}) solid $table-group-separator-color;


Problem with export of ImageCollection

Hi there,
I have some issues with downloading an ImageCollection and I would appreciate any help! I'm following an earlier tutorial from this community.
Here's my code:
import ee
import random
from geetools import batch

ee.Authenticate(auth_mode='localhost')
ee.Initialize(project='ee-user')

seed = random.randint(0, 10000)

collection = (ee.ImageCollection('COPERNICUS/S2_SR_HARMONIZED')
              .filterDate('2023-01-01', '2023-01-31')
              .filter(ee.Filter.stringContains('MGRS_TILE', '20M'))
              .filter(ee.Filter.lt('CLOUDY_PIXEL_PERCENTAGE', 10))
              .randomColumn('randomKey', seed)
              .sort('randomKey')
              .limit(3)
              .select(['B2', 'B3', 'B4', 'B8']))

print('Number of images', collection.size().getInfo())

size = collection.size().getInfo()
clist = collection.toList(size)
for i in range(size):
    image = ee.Image(clist.get(i))
    iid = image.id().getInfo()
    name = 'jan_'+iid
    print('downloading '+ name)
    batch.image.toLocal(image, name)

And here's the error:
(user) [user"user]$ /user/export.py
Number of images 3
downloading jan_20230126T144729_20230126T145426_T20MKU
Traceback (most recent call last):
  File "/user/data.py", line 402, in _execute_cloud_call
    return call.execute(num_retries=num_retries)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/user/lib/python3.12/site-packages/googleapiclient/_helpers.py", line 130, in positional_wrapper
    return wrapped(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/user/lib/python3.12/site-packages/googleapiclient/http.py", line 938, in execute
    raise HttpError(resp, content, uri=self.uri)
googleapiclient.errors.HttpError: <HttpError 400 when requesting https://earthengine.googleapis.com/v1/projects/ee-user/value:compute?prettyPrint=false&alt=json returned "Image.bandNames, argument 'image': Invalid type.
Expected type: Image<unknown bands>.
Actual type: Feature.". Details: "Image.bandNames, argument 'image': Invalid type.
Expected type: Image<unknown bands>.
Actual type: Feature.">

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/user/export.py", line 28, in <module>
    batch.image.toLocal(image, name)
  File "/user/lib/python3.12/site-packages/geetools/batch/image.py", line 37, in toLocal
    scale = scale if scale else int(tools.image.minscale(image).getInfo())
                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/user/lib/python3.12/site-packages/ee/computedobject.py", line 107, in getInfo
    return data.computeValue(self)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/user/lib/python3.12/site-packages/ee/data.py", line 1107, in computeValue
    return _execute_cloud_call(
           ^^^^^^^^^^^^^^^^^^^^
  File "/user/lib/python3.12/site-packages/ee/data.py", line 404, in _execute_cloud_call
    raise _translate_cloud_exception(e)  # pylint: disable=raise-missing-from
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ee.ee_exception.EEException: Image.bandNames, argument 'image': Invalid type.
Expected type: Image<unknown bands>.
Actual type: Feature.

This script is not compatible with version 1.0 of the lib as exportation has not been refactored yet. Could you try using 0.6.14 ?
Alternatively can you look at my answer in https://github.com/gee-community/geetools/issues/256 ?

Hi, thanks for your reply! I already downgraded to 0.6.14. Is it really a problem with batch?

I think yes,
maybe the latest changes to the tools module have conflict in the batch context, or the collection you have is a featureCollection nwhich sounds strange.
The problem is that there was no tests before so I'm building them on the spot every time I refactor an existing method. There are many cases where I need multiple iteration to cover all the edge cases. The v1 of the lib will eventually be more stable but i still need to work on batch

As downloading to local now requires a dedicated Google Cloud Storage bucket, I didn't recoded this mechanism in the v1 of geetools. If you need to download imageCollection in GEE compatible formats, you can still use the following methods: https://geetools.readthedocs.io/en/latest/usage/export.html


Remove dev.md

Remove dev.md docs, since it should reside in Mirantis/hmc repo only.

Consider adding a link to where find the latest devdoc


Use SPDX license identifier

Right now, we haven't define how should the license identifier should be specified. I saw Zenodo uses SPDX identifier and I think we should basically enforce the same.
Here is a list of them: https://spdx.org/licenses/

I'd be in favor to limit valid licenses (to the SPDX list), too 👍

looking at the python package @oeway proposed in https://github.com/bioimage-io/python-bioimage-io/issues/68 I have some follow up questions, that should be be discussed here instead:
This packages provides additional information on each license, so should we:

isDeprecatedLicenseId - bool value telling you whether this license definition is oudated or not.

warn for a deprecated licence (or fail?)?

isFsfLibre - bool value telling you whether this license is "free software" by the definition of FSF.
isOsiApproved - bool value telling you whether this license is "open source" by the definition of OSI.

require it to be free/open source?? (I think we agreed on not enforcing any specific license groups, but I want to make sure I didn't just make that up...)

This packages provides additional information on each license, so should we:

isDeprecatedLicenseId - bool value telling you whether this license definition is oudated or not.

warn for a deprecated licence (or fail?)?

isFsfLibre - bool value telling you whether this license is "free software" by the definition of FSF.
isOsiApproved - bool value telling you whether this license is "open source" by the definition of OSI.

We can perhaps allow anything in the list, but produce deprecation note in the output?

require it to be free/open source?? (I think we agreed on not enforcing any specific license groups, but I want to make sure I didn't just make that up...)

I think we don't need to enforce any further rule as a start, the current spec only defines a model, which does not necessary need the source code (e.g. one can make a pytorch model in torch script format without providing the source code). I mean open source licence are typically apply to software, but here, for model, notebook, and dataset, those "content-oriented" license such as CC-BY is perhaps more relevant.
One way to avoid super long discussion is to perhaps leave it open-ended for now ;)

let's not merge this, but incorporate it into #80

implemented in 5599987


Update nextclade & pangolin modules

PR checklist
Nextclade & pangolin modules are getting outdated and were requested to be updated.

[X] This comment contains a description of changes (with reason).
[X] If you've fixed a bug or added code that should be tested, add tests!
[X] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/viralrecon branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[X] Usage Documentation in docs/usage.md is updated.
[X] Output Documentation in docs/output.md is updated.
[x] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

I'm trying to find when the --reference  option for nextclade dataset get was removed. However, I've not been able to find it (yet). I therefore decided to remove the param --nextclade_dataset_reference which was the argument that got passed to --reference.


Suggestion: Graph animations options on appearing

It would be cool if there's an option to add animation on appearing. :)

I just added it to the last prerelease version(that can be deactivated with IsAnimated chart's property)!
Looking for feedbacks before release!


Getting started and understanding Materialize CSS

We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner. Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
Brief Summary:
Materialize CSS is a UI component library that is created with CSS, JavaScript, and HTML. It is created and designed by Google. Materialize CSS is also known as Material Design. It is a design language that combines the classic principles of successful design along with innovation and technology. Google's goal is to develop a system of design that allows for a unified user experience across all their products on any platform.
Key Takeaways:
You will learn how to write fewer CSS codes, achieve responsive design, and turn your UI design into a world-class functional code.
References:
materializecss.com

link with a closed issue #418 due to inactivity


Scanning DATE type into time.Time

When I try to scan a field which is a column of type DATE (written in Postgres as 2024-09-27) I get the following error:
parsing time "2024-09-27" as "2006-01-02T15:04:05Z07:00": cannot parse "" as "T"

Because ksql is trying to scan it as a DATETIME. Is there any way to make it scan as a PostgreSQL DATE (which has YYYY-MM-DD format)?

Hello @giovanni-orciuolo, I am actually traveling so I only have my cellphone right now, but I might still be able to help.
So by default KSQL only forwards it's query arguments to the underlying database client, I am assuming you are using kpgx or kpgx5 as the backend adapter.
The only exception to this is if you are using a modifier, if you are using one of the timeNowUtc* modifiers they do expect the database column to be a DATETIME which would explain the error.
If this is the case there are several ways of working around this:

If you don't really need to use a modifier you could just stop using it, maybe a default value of now() for this column would be enough.
If you do need it you could either change the database column type or create your own modifier (you can copy the original definitions of the timeNow modifier and change it)
It might be possible to submit a PR to make the existing modifier work with both types, but this one is more complex as it needs to work on postgres, SQLite, MySQL and sqlserver, and it also needs to be reasonably efficient

Tell me if this makes sense, I might be wrong in some of my assumptions

Hey @giovanni-orciuolo, I am back from my vacation, I just checked the code for the time modifiers and actually I don't think I am doing any custom parsing of date strings there:
https://github.com/VinGarcia/ksql/blob/master/internal/modifiers/time_modifiers.go
Which implies this is not handled by KSQL at all, it might be a different problem.
Can you get me some info on what database/driver you are working with and also an example of the table definition + the Go code with the query you are trying to run using KSQL?

Hmm, so this problem is probably not related to KSQL as we don't do anything with the query params, so I will close it for now.
If you think this is indeed caused by KSQL feel free to create a new issue (or reopen this one, I am not sure if you would have the permissions for that though =P)


Use ALF for the embed consent modal

Self-explanatory. Nothing fancy going on here, although if you don't like the web layout please say 😅 I can't decide if it's good or not

If we used Prompt instead of Dialog for this would it look any better? (And then put the buttons ontop of each other instead of side by side) No idea, just a thought.

@haileyok how's this?

I slightly modified the spacing here. As long as you're good with this, everything else LGTM.