Add identity multihash

This is part of https://github.com/ipfs/go-ipfs/issues/3896

cc @Kubuxu @diasdavid @lgierth
We've talked about this for a while. 0x00 is the right number to use, yeah?

@whyrusleeping could you point to that discussion. I remember the proposal but I'm not recalling exactly what were the main pros/cons

@diasdavid https://github.com/multiformats/multihash/issues/13

I'm not sure we should use 0x00 codes. The general norm with string representation of bytes is that high zero bytes can be stripped. One prominent example of this is IPv6 adresses, where a 00ab segment can and will be shortened to just ab. While it's technically fine because we can define the semantics of multicodecs ourselves, it goes against intuition and invites confusion and issues down the road.
We should treat the identity codec just as any other codec.

Previous discussions here: https://github.com/multiformats/multihash/issues/13 && https://github.com/multiformats/multicodec/pull/30

As a note to self, if we don't use the 0x00 code then this merged PR will have to be amended. It's a shame the 0x1d code is taken, as a play on id.

@lgierth brought up a good point. I see that @jbenet proposed 0x50 too? Would that work for everyone?
@JustinDrake yes, we would have to amend that part of the code.

I've amended this PR to use 0x50 instead of 0x00.

So it turns out we're already using 0x00 for identity encoding in multibase :/
Let's stick with 0x00 here then, unless someone has a better idea. Sorry about that @JustinDrake :)

And let's also finally settle on ONE central codec table to rule them all (the one in the multicodec repo). These annoying inconsistency issues keep coming up and by now I really don't care about the arguments for one-table-in-every-repo.

Wait, before we carry on here: how is 0x00 gonna be represented in a string? Null-bytes in strings are a bad idea.

@lgierth it will be converted to whatever encoding gets used.
Though i don't know if the argument holds that we must use 0x00 here just because we use 0x00 for multibase. Then the binary multibase encoding of something that is using an 'identity' multicodec would start with two zero bytes.

Though I don't know if the argument holds that we must use 0x00 here just because we use 0x00 for multibase.

Fair. And seems that here we always get saved most of the times by base encoding on top.
Let's go with 0x00 then

@lgierth No worries. I've reverted back to using 0x00 👍

@lgierth ping.

ping

Let's do it :+1: I'm still slightly concerned with starting with two zero-bytes in base2, but we don't avoid that by using something else here, it'll start with 0x00 one way or the other because of multibase.

Though i don't know if the argument holds that we must use 0x00 here just because we use 0x00 for multibase.

Yeah it's not a very strong argument.

Then the binary multibase encoding of something that is using an 'identity' multicodec would start with two zero bytes.

That's neat though!

(long since fixed)


Tests are not passing with Zend EventManager 3

When writing some evolutions, I encountered a problem with the test suite. Details can be found on travisci.
There was 1 error:
1) AcMailerTest\Service\MailServiceAbstractFactoryTest::testListeners
ReflectionException: Method getListenersByEventName does not exist
/home/travis/build/acelaya/ZF-AcMailer/test/Service/MailServiceAbstractFactoryTest.php:359

According to the migration guide, there may be no new ways to do this feature: Zend EventManager v3 - Migration - Removed
Solution might involve a reflection object, and making the events attribute public so we can count the number of listeners under the key we want. I would advise against it as it means testing against the actual EventManager class and not the EventManagerInterface.
$em = $mailService->getEventManager();
$refObject = new \ReflectionObject($em);
$method = $refObject->getMethod('getListenersByEventName');
$method->setAccessible(true);

$this->assertCount(3, $method->invoke($em, MailEvent::EVENT_MAIL_PRE_SEND));

$em = $mailService->getEventManager();
$refObject = new \ReflectionObject($em);
$attr = $refObject->getProperty('events');
$attr->setAccessible(true);
$this->assertCount(3, $attr->getValue($em)[MailEvent::EVENT_MAIL_PRE_SEND][1][0]);

Another solution I can think of is to trigger the event and see the result. I am definitely not sure what kind of result we can expect and how to count how many listeners we went through, I don't know the event manager enough here...

I think I had to deal with this some time ago in another project.
I'll try to fix it.

This is duplicated #142, so I'm closing this.


Cancel all jobs / all jobs under program

List programs
List jobs under a program
List all active or running jobs (not sure if api supports this)
Cancel all jobs
Cancel all jobs under a program

get_processor

get processor and list calibrations are done.  But all in first batch are not currently supported.

List programs, List jobs under a program, list all active or running jobs are done.
--- these are the only ones missing now:
Cancel all jobs under a program - we could implement this by the cancel_job() one by one
Cancel all jobs - in a project? we could also implement this by running cancel_job() one by one
@wcourtney is there a plan for "cancel call for multiple job_ids" in the service?

Pinging @wcourtney - is this still necessary? Or close if it's done.

We don't have any plans for bulk cancellation in the API. We could provide a "kill switch" helper for this if we think it's common - list jobs filtered by state (RUNNING or READY) and cancel each. I expect that the issue with doing this today is that we use a blocking API interface, but we're looking into migrating to an asyncio format that should make this more bearable.
The semantics of "cancelling a program" are strange and I think it would be a low priority until at least one user other than Dave thinks about "programs" :-).

Closing as Won't Fix - let's rely on asyncio support. Feel free to re-open if you this is a higher priority than I suspect.


enhancement request:Save State in all views, navigating up the bread crumb should go back to where you are in the view

Is there an existing issue for this?

[X] I have searched the existing issues

Enhancement description
Save State in all views, navigating up the bread crumb should go back to where you are in the view
🧑💻 Implementation
No response
Anything else?
No response

I'll make it


How can I download data files, CommonVoice etc..

"We also provide data preparation and loading functions for the LibriSpeech, CommonVoice and SpeechCommands datasets, based on SpeechBrain recipes." in your git.
But, When I run the code according to the Recipe you shared, the following error appears.

How can I download data files??

Hi,
The recipes can load the local dataset, but not download it (except Speech command). You have to do that manually, and unpack them in your data folder.

Thank you for answer.
I would like to create an adversarial example through an attack model. Can you provide some samples on how?

Applying the recipes - like you were doing - is the right way! All you need is the dataset first.

Thank you..
I looked at several python codes and it seems to be explained in detail very well.

Hello~
I want to test using transformer model with cw algorithm.
When I used "python fit_attacker.py attack_configs/LibriSpeech/universal/ctc_5000bpe_fit.yaml", I met this error

If possible, can you tell me how to use the CW algorithm attack method using the transformer model in this git,robust_speech?

Discussion in #9 for the last question


Consider using a Dictionary instead of a KVP array in ViewElement

This might be a little better for randomly accessing the attributes by using their unique identifier (AttributeKey.KeyValue).

ViewElement are so ephemeral that I think the added construction time would outweigh the benefit
But it depends on usage scenario and adding either a on-demand-populated table (initially null) may be useful


⚠ Minecraft has degraded performance

In 679326f, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1270 ms

Resolved: Minecraft performance has improved in 0b90f0b.


MB-14703: Fail if the number of spectral lint errors increases

Jira ticket for this change
Summary
Fail the build if the number of spectral lint errors increases

Messages

:book:
:link: MB-14703

  Generated by :no_entry_sign: dangerJS against d373a5ca83007baaa99d724f08aeb54a3d280aad


Platform-specific overscroll behavior, etc (NOT FOR REVIEW)

Added support for ClampOverscrolls to ScrollableViewport
Added PlatformScrollBehavior: adds an OverscrollIndicator on Android, leaves bounce overscrolling (the default) enabled on IOS.
Most scrolling demos wrap their contents with PlatformScrollBehavior.
The flexible toolbar examples now just clamp overscrolling.

Hans, I'm just cleaning out our PR queue, so I'm going to close this for now. Feel free to reopen once you're ready for review. Thanks!

No problem. I think this is a dead-end anway.


Crashing on iOS when scrolling

Found a problem today in my app. I use ui-image in a grid scroll view and when I scroll down the list the app will sometimes crashes violently. There is no error shown in the console, and the only way I can get an error is by going into the logs (see attached image).
Demo project is attached.
ui-image-crash.zip

Oops, it looks like this could be the same issue as https://github.com/nativescript-community/ui-image/issues/32 ? Was never solved though?

Using the v8 runtime? Can you try without it?

That seems to be the problem. Was using v8 when it was crashing. Switched to "tns-ios": "6.5.3" and the problem is gone. Any idea how to fix it w/ v8? Should I report it over there?

Same problem here with ns7 / v8

@pboulch @keerl did not really have the time to look at it. But it would be good to report on the v8 runtime repo. THey might have an idea

I did report it there https://github.com/NativeScript/ns-v8ios-runtime/issues/81 it may helpful for @pboulch to post on that issue as well.

same issue here when scrolling table of images. ns7 / v8
Seems like the crash in iOS pod sdwebimage @ this function:

(void)sd_setImageWithURL:(nullable NSURL *)url
placeholderImage:(nullable UIImage *)placeholder
options:(SDWebImageOptions)options
context:(nullable SDWebImageContext *)context
progress:(nullable SDImageLoaderProgressBlock)progressBlock
completed:(nullable SDExternalCompletionBlock)completedBlock

@Jaroslav84 @keerl @pboulch please can you try 4.0.14?

@farfromrefug same story.
I'm loading 50 images into a tableList (Angular) with small thumbnail png's. After scrolling (on device) it crashes the same way.

@farfromrefug, still crashing for me as well with the 4.0.14.

Works fine with<EMAIL_ADDRESS>

still crash
"@nativescript/ios": "^7.0.6
"@nativescript-community/ui-image": "^4.0.14"

@georgemark0v try to remove platforms/node_modules/hooks and build again

Also confirming this fixed the issue. Good to close.

@georgemark0v try to remove platforms/node_modules/hooks and build again

Solved. Thanks.


Dangerous effect of using --dry-run: makes unannounced changes to EKS cluster DaemonSets

Bug report
using --dry-run or --dry-run-helm-values does changes to eks aws-node. This should not be done, and while it is outputted to logs in code the logs are discarded when using --dry-run.
General Information
cilium-cli: v0.15.17 compiled with go1.21.4 on darwin/arm64
cilium image (default): v1.14.4
cilium image (stable): v1.14.4

Problematic code:
https://github.com/cilium/cilium-cli/blob/036d768db9768be6292ea8eb2cefd38e6485d277/install/install.go#L681C1-L694C4
How to reproduce the issue

Use EKS cluster
Run something similar to cilium install --version 1.14.4 --values values-migration.yaml --dry-run-helm-values
Watch how the node selector is changed

Thanks for the report @krokofant. I've opened #2166 with a fix to no longer make any changes to the cluster in dry-run mode.

and while it is outputted to logs in code the logs are discarded when using --dry-run.

This is intentional so the helm values can be written to stdout and easily be piped to other commands.


Cannot connect to dropbox or google drive

Problem :
Cannot connect to dropbox or google drive. Connection page of dropbox says that  the browser is unsupported.
Steps to reproduce :

Open Tools/KeeAnywhere Setting
Select "add" and "dropbox"

Screenshots

Environment
Windows 10
KeePass v2.45
KeePassLibC (1 .x File Support) v1.38(0x01B3)
KeeAnywhere v1.6

Thank you for quick reply i will see that


[Speed Dial] fab size messes up alignment

[x ] The issue is present in the latest release.
[x ] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

When I pass FabProps to the SpeedDial component, the fab size changes, but then all of the speed dial actions are misaligned.
    `<SpeedDial FabProps={{ size: "small" }} ...> `

Expected Behavior 🤔
When I pass FabProps to the SpeedDial component, the fab size change, and all of the actions should properly align to the FAB just like it does with the default medium-sized FAB

Steps to Reproduce 🕹

https://codesandbox.io/s/material-demo-0ixhi
Steps:

Add SpeedDial Component
Pass FabProps={{size:'small'}} to the SpeedDial component

Context 🔦

I'd like to provide multiple actions in a constrained space on mobile.  All of the buttons for each list item won't fit without using something like a speed dial.

Your Environment 🌎

Tech
Version

Material-UI
v4.4.3

Material-UI (lab)
v4.0.0-alpha.30

React
16.8.6

Browser
Version 78.0.3904.70 (Official Build) (64-bit)

I was able to work around this by assigning a custom class to the SpeedDialAction with a modified margin:
const useStyles = makeStyles(theme => ({
...
  fabAction: {
    margin: "0 8px 0 0"
  }
}));
...
<SpeedDialAction
key={action.
icon={action.icon}
tooltipTitle={action.name}
onClick={handleClose}
className={classes.fabAction}
/>

https://codesandbox.io/s/material-demo-0ixhi

@goleary Thank you for the report. What do you think of this fix?
diff --git a/packages/material-ui-lab/src/SpeedDial/SpeedDial.js b/packages/material-ui-lab/src/SpeedDial/SpeedDial.js
index 2b349707f..104ddf0ec 100644
--- a/packages/material-ui-lab/src/SpeedDial/SpeedDial.js
+++ b/packages/material-ui-lab/src/SpeedDial/SpeedDial.js
@@ -34,6 +34,7 @@ export const styles = theme => ({
   root: {
     zIndex: theme.zIndex.speedDial,
     display: 'flex',
+    alignItems: 'center',
     pointerEvents: 'none',
   },
   /* Styles applied to the Fab component. */

Do you want  to submit a pull request? :)

An even better solution! I would, but it looks like @itayyehezkel beat me to it.


Add terminateDebuggee capability

If my guess is right we can set it and code for it, even with keeping the old version compatibility. If this is true (old versions will just have the "stop" as plain terminate, new versions will have the additional option to "Disconnect" (in this case we'd get terminateDebuggee; false in the disconnect request and would issue a -target-detach).

After some tests: for a attach configuration we'll get Disconnect + Disconnect and Suspend, as soon as we support both (but not if only one of those is added):
		response.body.supportTerminateDebuggee = true;
		response.body.supportSuspendDebuggee = true;

but I see no option to actually do "Disconnect and Suspend" in a portable way.
What possibly works on Linux is to send SIGSTOP, then detach - but only there. Tried with a mingw32 process and mingw32 process outside of this extension - sending SIGSTOP killed the process.
Retesting with a launch configuration resulted in the same two buttons.
If this is not done then we have the existing two buttons: detach for attach, stop for launch.
It looks to me that we therefore should (at least currently) not support this (I haven't checked if updating a dependency would change anything, but I guess it wouldn't).
Should we close this issue?
Should I commit the following changes nonetheless?
--- a/src/gdb.ts
+++ b/src/gdb.ts
@@ -49,6 +49,22 @@ class GDBDebugSession extends MI2DebugSession {
                response.body.supportsEvaluateForHovers = true;
                response.body.supportsSetVariable = true;
                response.body.supportsStepBack = true;
+
+               // CHECKME - things to possibly implement:
+
+               // seems to be only working together (at least in vscode),
+               // but looks to be not applicable to a system process
+               // see https://github.com/WebFreak001/code-debug/issues/357
+               //response.body.supportTerminateDebuggee = true;
+               //response.body.supportSuspendDebuggee = true;
+
+               // This should have the benefit of faster restart. Do we care?
+               // If supported we'll receive a RestartRequest, if not the DAP will receive a stop,
+               // then will be restarted itself (so we go through the complete initialization).
+               // For "launch" RestartRequest should send a stop with terminate,
+               // for "attach" it should either be a detach + attach or send pause and request GDB info to be
+               // reloaded (is "show directories" + "directories" enough?)
+               //response.body.supportsRestartRequest = true;
                this.sendResponse(response);
        }

diff --git a/src/mibase.ts b/src/mibase.ts
index 5cf2abf..7dc938c 100644
--- a/src/mibase.ts
+++ b/src/mibase.ts
@@ -170,11 +170,16 @@ export class MI2DebugSession extends DebugSession {
                this.quitEvent();
        }

        protected disconnectRequest(response: DebugProtocol.DisconnectResponse, args: DebugProtocol.DisconnectArguments): void {
-               if (this.attached)
-                       this.miDebugger.detach();
-               else
-                       this.miDebugger.stop();
+               if (args.terminateDebuggee == undefined)
+                       if (this.attached)
+                               args.terminateDebuggee = false;
+                       else
+                               args.terminateDebuggee = true;
+               if (args.terminateDebuggee)
+                       this.miDebugger.stop();
+               else
+                       this.miDebugger.detach();
                this.commandServer.close();
                this.commandServer = undefined;
                this.sendResponse(response);


Merge with ember-cli version?

Do we want to convert this to ember-cli? Haven't looked at how many commits back I did that, but ember-cli version is here: https://github.com/brittanystoroz/its-five-o-clock-somewhere. There might be some annoying manual updates depending on how much has gone on with this app since then, and it's probably in dire need of dependency updates.
Just something to think about before we try adding in service workers. Might be good for people to see a SW example in an app using a popular framework.

I'd love for it to be an actual Ember CLI app, so even if there are annoying manual things to do I say it's worth it. Maybe even just manually moving over the app/ files into a new Ember CLI app would work if there's too much manual mucking about?

tofumatt

On Tue, Jun 30, 2015 at 2:35 PM, .DS_Storoz<EMAIL_ADDRESS>wrote:

Do we want to convert this to ember-cli? Haven't looked at how many commits back I did that, but ember-cli version is here: https://github.com/brittanystoroz/its-five-o-clock-somewhere. There might be some annoying manual updates depending on how much has gone on with this app since then, and it's probably in dire need of dependency updates.
Just something to think about before we try adding in service workers. Might be good for people to see a SW example in an app using a popular framework.
Reply to this email directly or view it on GitHub:
https://github.com/mozilla/high-fidelity/issues/65

Lol I totally posted the wrong link earlier...that's the timezone app not the podcasts app. I do believe I converted podcasts to ember-cli but may not have put it up on github. Will check on this tomorrow. Regardless, it's something I can take care of if I haven't already ¯_(ツ)_/¯

closing this - ember-cli-conversion branch has been added to the repo and will work on related bugs there


Update repo root directory on Coveralls

If I click any file in coverage list on Coveralls website, I can see an error:

SOURCE NOT AVAILABLE
The file "server/lib/closeIO.js" isn't available on github. Either it's been removed, or the repo root directory needs to be updated.

For example:
https://coveralls.io/builds/9884571/source?filename=server%2Flib%2FcloseIO.js

@CatSkald Did you find a solution for this issue?


Fix wrong closing bracket

![Test](Test) =16x16) is entirely matched by the pattern but I expect that only ![Test](Test) refers to the image.
On the other hand ![Test](Test] =16x16) would not match while ![Test](Test]) is the way to embed the image named Test].

Thanks to @JGSteinke for bringing this to my attention.


Pip-install fails

When following the install instructions and running:
pip install git+https://github.com/mhamilton723/FeatUp

I get the following error:
OS: Ubuntu 22.04
Python: 3.10
Collecting git+https://github.com/mhamilton723/FeatUp
  Cloning https://github.com/mhamilton723/FeatUp to /tmp/pip-req-build-woni10iy
  Running command git clone --filter=blob:none --quiet https://github.com/mhamilton723/FeatUp /tmp/pip-req-build-woni10iy
  Resolved https://github.com/mhamilton723/FeatUp to commit a5f9f56a973573a99504c09a69561f726dbe9d82
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [6 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/tmp/pip-req-build-woni10iy/setup.py", line 2, in <module>
          from torch.utils.cpp_extension import BuildExtension, CUDAExtension, CppExtension
      ModuleNotFoundError: No module named 'torch'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

try installing pytorch first accourding to
https://pytorch.org/get-started/locally/


Arch Tutorial

Arch Tutorial
Added the required Arch tutorial
I hope you will like it.

Please add the Hacktober fest label

Hey please merged it..... waiting for long


Saved model won't load: Unable to synchronously open object (bad local heap signature)

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
binary
TensorFlow version
2.16.1
Custom code
Yes
OS platform and distribution
windows 10
Mobile device
No response
Python version
3.12
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Model saved from Python 3.12 tensorflow 2.16.1
model.save('my_model.keras', overwrite=True)
After this the model does not load
Standalone code to reproduce the issue
model=tf.keras.models.load_model('my_model.keras', custom_objects=None, compile=True, safe_mode=True)

Relevant log output
Traceback (most recent call last):
  File "D:\Project\main.py", line 391, in <module>
    model=tf.keras.models.load_model('my_model.keras', custom_objects=None, compile=True, safe_mode=True)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\Project\venv\Lib\site-packages\keras\src\saving\saving_api.py", line 176, in load_model
    return saving_lib.load_model(
           ^^^^^^^^^^^^^^^^^^^^^^
  File "D:\Project\venv\Lib\site-packages\keras\src\saving\saving_lib.py", line 192, in load_model
    _raise_loading_failure(error_msgs)
  File "D:\Project\venv\Lib\site-packages\keras\src\saving\saving_lib.py", line 273, in _raise_loading_failure
    raise ValueError(msg)
ValueError: A total of 13 objects could not be loaded. Example error message for object <Sequential name=sequential, built=True>:

'Unable to synchronously open object (bad local heap signature)'

List of objects that could not be loaded:
[<Sequential name=sequential, built=True>, <TextVectorization name=text_vectorization, built=True>, <StringLookup name=string_lookup_1, built=False>, <Embedding name=embedding, built=True>, <Conv1D name=conv1d, built=True>, <Dropout name=dropout, built=True>, <Conv1D name=conv1d_1, built=True>, <Dropout name=dropout_1, built=True>, <GlobalMaxPooling1D name=global_max_pooling1d, built=True>, <Dense name=dense, built=True>, <Dropout name=dropout_2, built=True>, <Dense name=dense_1, built=True>, <keras.src.optimizers.adam.Adam object at 0x000001C5026B24E0>]

Some sample code
import tensorflow as tf
from tensorflow.keras.layers.experimental.preprocessing import TextVectorization
data = [
"The sky is blue.",
"Grass is green.",
]
Create vectorizer.
text_dataset = tf.data.Dataset.from_tensor_slices(data)
vectorizer = TextVectorization(
max_tokens=1000, output_mode='tf-idf', ngrams=None,
)
vectorizer.adapt(text_dataset.batch(1024))
Create model.
model = tf.keras.models.Sequential()
model.add(tf.keras.Input(shape=(1,), dtype=tf.string))
model.add(vectorizer)
print (vectorizer("sky is blue"))
Save.
filepath = "tmp-model.keras"
.tf not work, .h5 not work, .keras not work
model.save(filepath)
Load.
loaded_model = tf.keras.models.load_model(filepath)
loaded_vectorizer = loaded_model.layers[0]
print (loaded_vectorizer("sky is blue"))

Hi @Expert73 ,
Sorry for the dealy, I reproduced the code you shared. Here i am providing gist for how to save the model with TextVectorizer in tensorflow 2.16.1 for your reference. please go through it once.
Thank you!

Hi @Expert73 ,
Sorry for the dealy, I reproduced the code you shared. Here i am providing gist for how to save the model with TextVectorizer in tensorflow 2.16.1 for your reference. please go through it once.
Thank you!

Colab work fine. But in windows 10:
D:\project_1\venv\Scripts\python.exe D:\project_1\test.py  2024-03-21 20:54:53.563758: I tensorflow/core/util/port.cc:113] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable TF_ENABLE_ONEDNN_OPTS=0. 2024-03-21 20:54:54.181494: I tensorflow/core/util/port.cc:113] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable TF_ENABLE_ONEDNN_OPTS=0`.
2.16.1
2024-03-21 20:54:55.382961: I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
1/1 ━━━━━━━━━━━━━━━━━━━━ 0s 250ms/step - loss: 0.5000
Traceback (most recent call last):
File "D:\project_1\test.py", line 47, in
loaded_model = tf.keras.models.load_model("my_model_with_vectorizer.keras")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\project_1\venv\Lib\site-packages\keras\src\saving\saving_api.py", line 176, in load_model
return saving_lib.load_model(
^^^^^^^^^^^^^^^^^^^^^^
File "D:\project_1\venv\Lib\site-packages\keras\src\saving\saving_lib.py", line 192, in load_model
_raise_loading_failure(error_msgs)
File "D:\project_1\venv\Lib\site-packages\keras\src\saving\saving_lib.py", line 273, in _raise_loading_failure
raise ValueError(msg)
ValueError: A total of 1 objects could not be loaded. Example error message for object <keras.src.optimizers.adam.Adam object at 0x00000214B810A8A0>:
'Unable to synchronously open object (bad object header version number)'
List of objects that could not be loaded:
[<keras.src.optimizers.adam.Adam object at 0x00000214B810A8A0>]
Process finished with exit code 1`
Python 3.12
Tensorflow 2.16.1

After a lot of testing I found the problem. The problem is with the poetry package.
On python 3.12 the command
poetry add tensorflow
it executes without errors, but in fact there are problems with tensorflow-io.
That is, just
pip install tensorflow

Problem still with python 3.12

Hi @Expert73 ,
Sorry for the dealy, I reproduced the code you shared. Here i am providing gist for how to save the model with TextVectorizer in tensorflow 2.16.1 for your reference. please go through it once.
Thank you!

Colab work fine. But in windows 10:
D:\project_1\venv\Scripts\python.exe D:\project_1\test.py
2024-03-21 20:54:53.563758: I tensorflow/core/util/port.cc:113] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2024-03-21 20:54:54.181494: I tensorflow/core/util/port.cc:113] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2.16.1
2024-03-21 20:54:55.382961: I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
1/1 ━━━━━━━━━━━━━━━━━━━━ 0s 250ms/step - loss: 0.5000
Traceback (most recent call last):
  File "D:\project_1\test.py", line 47, in <module>
    loaded_model = tf.keras.models.load_model("my_model_with_vectorizer.keras")
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\project_1\venv\Lib\site-packages\keras\src\saving\saving_api.py", line 176, in load_model
    return saving_lib.load_model(
           ^^^^^^^^^^^^^^^^^^^^^^
  File "D:\project_1\venv\Lib\site-packages\keras\src\saving\saving_lib.py", line 192, in load_model
    _raise_loading_failure(error_msgs)
  File "D:\project_1\venv\Lib\site-packages\keras\src\saving\saving_lib.py", line 273, in _raise_loading_failure
    raise ValueError(msg)
ValueError: A total of 1 objects could not be loaded. Example error message for object <keras.src.optimizers.adam.Adam object at 0x00000214B810A8A0>:

'Unable to synchronously open object (bad object header version number)'

List of objects that could not be loaded:
[<keras.src.optimizers.adam.Adam object at 0x00000214B810A8A0>]

Process finished with exit code 1

Windows 10 x64 Python 3.12 Tensorflow 2.16.1
colab python 3.10.12 (!) Tensorflow 2.16.1
Update: test with python 3.10 and 3.11, tensorflow 2.16.1 in windows - work fine.
problem with python 3.12 (!)
how can I solve the problem?

you are the best bro!!

@Expert73 try Python 3.12.2.
Almost the same issue when I used Python 3.12.0 + Keras 3.1.1 + TensorFlow 2.16.1
ValueError: A total of 1 objects could not be loaded. Example error message for object <keras.src.optimizers.rmsprop.RMSprop object at 0x12732ec30>:

'Unable to open object (bad object header version number)'

List of objects that could not be loaded:
[<keras.src.optimizers.rmsprop.RMSprop object at 0x12732ec30>]

When I changed my Python version from 3.12.0 to 3.12.2, it worked well.

@Expert73 try Python 3.12.2.
Almost the same issue when I used Python 3.12.0 + Keras 3.1.1 + TensorFlow 2.16.1
ValueError: A total of 1 objects could not be loaded. Example error message for object <keras.src.optimizers.rmsprop.RMSprop object at 0x12732ec30>:

'Unable to open object (bad object header version number)'

List of objects that could not be loaded:
[<keras.src.optimizers.rmsprop.RMSprop object at 0x12732ec30>]

When I changed my Python version from 3.12.0 to 3.12.2, it worked well.

@Venkat6871

When I changed my Python version from 3.12.0 to 3.12.2, it worked well.

Thanks for the info. I checked too. Everything works great!

But one question. Why doesn't my code work? In python 3.10, 3.11 it works. In Venkat6871's version, I need to think about how to save the vectorizer layer into a separate file. In my version, this layer is saved in the model itself.
Type of error in my code here

Hi @Expert73 ,

Could you please confirm if this issue is resolved for you ? Please feel free to close the issue if it is resolved.

Thank you!


[Bug]: RuntimeError: Error(s) in loading state_dict for HuggingFaceWhisper

Describe the bug
size mismatch for _mel_filters: copying a param with shape torch.Size([80, 201]) from the checkpoint, the shape in the current model is torch.Size([201, 80]).
when i transpose the mel_filters tensor on the checkpoint, the model loaded but when transcribing gives a matrix multiplication error :
File /opt/conda/lib/python3.10/site-packages/speechbrain/lobes/models/huggingface_whisper.py:247, in HuggingFaceWhisper._log_mel_spectrogram(self, audio)
244 magnitudes = stft[..., :-1].abs() ** 2
246 filters = self._mel_filters
--> 247 mel_spec = filters @ magnitudes
RuntimeError: mat1 and mat2 shapes cannot be multiplied (3000x201 and 80x201)
I use torch.version '2.0.0+cu117'
It works when I reshape the feature_extractor.filter on speechbrain in /speechbrain/lobes/models/huggingface_whisper.py
but I don't know if this is a good way to fix it
Expected behaviour
loading the whisper model and transcribe audio
To Reproduce
pip install speechbrain transformers

from speechbrain.pretrained import WhisperASR

asr_model = WhisperASR.from_hparams(source="speechbrain/asr-whisper-large-v2-commonvoice-ar", savedir="pretrained_models/asr-whisper-large-v2-commonvoice-ar")
asr_model.transcribe_file("speechbrain/asr-whisper-large-v2-commonvoice-ar/example-ar.mp3")

Versions
torch.version '2.0.0+cu117'
Relevant log output
size mismatch for _mel_filters: copying a param with shape torch.Size([80, 201]) from the checkpoint, the shape in the current model is torch.Size([201, 80]).

Additional context
No response

Hello @Marwan-Ghouma,
Thanks for raising this issue.
The problem is coming from the transformer toolkit, basically they release a new version that is changing Whisper.
We are working on it! Could you please in the meantime downgrade your version of transformer please?
Many thanks.
Best.

Thanks, it works with transformers-4.26.0 !


examples: tweaks to demo_service

@ingve this solution is necessary to get the example working on GCE so I'll merge - but it makes the  service slower to start if you don't have a DHCP server running. Could you add a printf telling the user why we're waiting? e.g. waiting 5 sec. for DHCP, falling back to static config or something


git lfs issue

Hi Chen
Thanks for the excellent work.
I was trying to use the demo:
python demo.py --cfg ./configs/config_mld_humanml3d.yaml --cfg_assets ./configs/assets.yaml --example ./demo/example.txt
But I got this error:
You seem to have cloned a repository without having git-lfs installed. Please install git-lfs and run git lfs install followed by git lfs pull in the folder you cloned.
The error happens at this line:
self.text_model = AutoModel.from_pretrained(modelpath)
I installed git lfs, both before and after cloning the mld repository. But I still get the error related to git lfs. Any ideas what might be wrong?

Well, it seems that I had to run all the files in the prepare folder. Now it works. So I close this issue.

Hi, I am facing the same problem. Do you mean that after manually installed git lfs, I have to run the scripts in the prepare fold again?

Hi. As far as I remember, you only need run the scripts in the prepare folder. If you manually installed git-lfs in the environment, then just run all the prepare documents.

Yeah, I solved this. Thanks a lot!

If someone still sees it, you need to execute the script in the prepare folder in the project root directory, otherwise, it will download the file to the prepare directory


Make HttpClient public

There's no real reason to hide it, as it stops calling being able to customize or tweak the client later.
Also made it lazily instantiated.

Thank you!
@chrisbanes Do you need a new product version or does the snapshot version fit for you? I had my focus on another project in the last week.

snapshot works!


Code workflow

test

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

test


Getting Error: Package not found. with GITHUB_TOKEN

Here is the how the action is getting called:
Run<EMAIL_ADDRESS>  with:
    owner: abbvie-internal
    name: arch-ui-service
    token: ***
    tag: arch-ui-pr-2426

And I am seeing the error: Error: Package not found.
For the token, I am passing the secrets.GITHUB_TOKEN.
If I test this with octokit, making the same calls from my test node script.
import { Octokit, App } from "octokit";

const owner = "abbvie-internal";
const name = "arch-ui-service";

const octokit = new Octokit({ auth: `XXXXXXXXXXXXXXXXXXXXX` });

  for await (const response of octokit.paginate.iterator(
    octokit.rest.packages.getAllPackageVersionsForPackageOwnedByOrg,
    {
      package_type: "container",
      package_name: name,
      org: owner,
      state: "active",
      per_page: 100,
    }
  )) {
    for (let packageVersion of response.data) {
      console.log(packageVersion.metadata.container.tags);
    }
  }

This does find the package and returns metadata. The only difference is using a PAT as opposed to the GITHUB_TOKEN.
I have not yet tried switching the github action to use the PAT to see if that makes a difference.

Hello,
Can you mention the return code you are getting? I was getting a 204, which is genuine as the package got deleted and it couldn't return anything. However, I am using the PAT token added as Github secrets.
Regards,
Taniya

Using GITHUB_TOKEN would be more convenient. A little research on GitHub API is needed.

Does it need permissions in addition to packages: write?

I am getting a similar error, it says that it finds the image ID and the ID is also correct, but then it can't delete it.
Run<EMAIL_ADDRESS>🔎 search package version with tag 0.0.1-alpha.[13]
🆔 package id is #139386911, delete it...
Error: Package not found.

My token has the following privileges

I am not entirely sure whether this has the same context as the parent issue :)
Thank you in advance! 🫶

${{ secrets.GITHUB_TOKEN }} works fine.
permissions:
  contents: read # Checkout code
  packages: write # Push containers and manage tags

I got also the error Error: Package not found. and got it fixed because the name is not the repo name but the package name. This is unclear on the README
For me : https://github.com/wdes/mail-autodiscover-autoconfig/pkgs/container/mail-autodiscover-autoconfig%2Fmail-autodiscover-autoconfig
Gives: name: mail-autodiscover-autoconfig/mail-autodiscover-autoconfig
Gives: name: <repo-name>/<container-name>


Navigate between screens with Compose: Android Basics with Compose

In starter code of the Cupcake app, there is no "currentScreen" parameter in "@Composable
fun CupcakeAppBar" in java/com/example/cupcake/CupcakeScreen.kt
The text value of the TopAppBar title is hardcoded.
Therefore, it's hard to understand the step "7" in the pathway, because according to the tutorial, the existence of "currentScreen" parameter is expected. Also the text of the tutorial is a bit ambiguous. You don't know whether is CupcakeAppBar composable or CupcakeAppBar call in CupcakeApp composable meant.

Duplicate of #10

Hello @jelinekp,
Thanks for reaching out to us and sharing your concern. The steps are updated:

Please feel free to re-open the ticket, if the text of the tutorial is still unclear.


Feature: Adding contributors section to the README.md file.

There is no Contributors section in readme file .
As we know Contributions are what make the open-source community such an amazing place to learn, inspire, and create.
The Contributors section in a README.md file is important as it acknowledges and gives credit to those who have contributed to a project, fosters community and collaboration, adds transparency and accountability, and helps document the project's history for current and future maintainers. It also serves as a form of recognition, motivating contributors to continue their efforts.

@erikspears  Kindly assign me this #899 issue, I want to work on it. Thank You!!

Hi @Kalyanimhala ,
Is this a dynamic solution for adding and managing contributors?  Or does it require manual maintenance periodically?
Thanks,
--Erik

Yeah, This will encourage the other contributors to contribute in the Repo!
so its a generalized solution to add  contributors in the readme.md file.
Thanks!!

Thank you for the contribution, but that PR has been closed; it turns out we cannot take it at this time.


feat: additional opcache setting in check php.ini

Description
I think this additional needed to recommendation.
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

I have concerns.

Too many items in the output can be difficult to read. I want to display only the really important items.
The current output is too wide to see.
It is difficult to determine recommended values for OPCache settings because they are highly dependent on the environment.

+---------------------------------+---------+---------+-------------+------------------------------------------------------------------------------------------------------------------------------------------+
| Directive                       | Global  | Current | Recommended | Remark                                                                                |
+---------------------------------+---------+---------+-------------+------------------------------------------------------------------------------------------------------------------------------------------+
| error_reporting                 | 32767   | 32767   | 5111        |                                                                                |
| display_errors                  | 1       | 1       | 0           |                                                                                |
| display_startup_errors          | 1       | 1       | 0           |                                                                                |
| log_errors                      | 1       | 1       |             |                                                                                |
| error_log                       |         |         |             |                                                                                |
| default_charset                 | UTF-8   | UTF-8   | UTF-8       |                                                                                |
| memory_limit                    | 1G      | 1G      |             | > post_max_size                                                                                |
| post_max_size                   | 8M      | 8M      |             | > upload_max_filesize                                                                                |
| upload_max_filesize             | 2M      | 2M      |             | < post_max_size                                                                                |
| request_order                   | GP      | GP      | GP          |                                                                                |
| variables_order                 | GPCS    | GPCS    | GPCS        |                                                                                |
| date.timezone                   | UTC     | UTC     | UTC         |                                                                                |
| mbstring.language               | neutral | neutral | neutral     |                                                                                |
| opcache.enable                  | 1       | 1       | 1           |                                                                                |
| opcache.enable_cli              | 0       | 0       |             |                                                                                |
| opcache.jit                     | tracing | tracing |             |                                                                                |
| opcache.jit_buffer_size         | 0       | 0       |             |                                                                                |
| opcache.memory_consumption      | 128     | 128     | 728         | Increasing the configured memory size (MB) will improve performance by caching those files (consideration based on free space of memory) |
| opcache.interned_strings_buffer | 8       | 8       | 64          |                                                                                |
| opcache.max_accelerated_files   | 10000   | 10000   | 56000       | Increasing the configured accelerated will many cache files into OPCache hash table                                                      |
| opcache.max_wasted_percentage   | 5       | 5       | 15          |                                                                                |
| opcache.validate_timestamps     | 1       | 1       | 0           |                                                                                |
| opcache.revalidate_freq         | 2       | 2       | 0           |                                                                                |
| opcache.save_comments           | 1       | 1       | 1           |                                                                                |
+---------------------------------+---------+---------+-------------+------------------------------------------------------------------------------------------------------------------------------------------+

Probably a simple mention of opcache On/Off is enough.
Many people don't use the console when developing. Especially the newbies.
I think special attention to php.ini is not required. More questions about apache/nginx

This command is mainly used to check the PHP settings of production environments.

The Recommended column shows the recommended values for production environment. They may differ in development environments.
https://codeigniter4.github.io/CodeIgniter4/installation/running.html#checking-php-ini-settings

@ddevsr If you want to show opcache detailed settings, how about adding an argument?
E.g. php spark phpini:check opcache shows only opcache settings.

@kenjis @michalsn
$ php spark phpini:check

CodeIgniter v4.5.4 Command Line Tool - Server Time: 2024-08-22 09:50:55 UTC+00:00

+-------------------------+---------------------------------------+---------------------------------------+-------------+---------------------------------------+
| Directive               | Global                                | Current                               | Recommended | Remark                                |
+-------------------------+---------------------------------------+---------------------------------------+-------------+---------------------------------------+
| error_reporting         | 32767                                 | 32767                                 | 5111        |                                       |
| display_errors          | 1                                     | 1                                     | 0           |                                       |
| display_startup_errors  | 1                                     | 1                                     | 0           |                                       |
| log_errors              | 1                                     | 1                                     |             |                                       |
| error_log               | D:/Project/laragon/tmp/php_errors.log | D:/Project/laragon/tmp/php_errors.log |             |                                       |
| default_charset         | UTF-8                                 | UTF-8                                 | UTF-8       |                                       |
| max_execution_time      | 0                                     | 0                                     |             | The default is 30.                    |
| memory_limit            | 1024M                                 | 1024M                                 |             | > post_max_size                       |
| post_max_size           | 2G                                    | 2G                                    |             | > upload_max_filesize                 |
| upload_max_filesize     | 2G                                    | 2G                                    |             | < post_max_size                       |
| max_input_vars          | 1000                                  | 1000                                  |             | The default is 1000.                  |
| request_order           | GP                                    | GP                                    | GP          |                                       |
| variables_order         | GPCS                                  | GPCS                                  | GPCS        |                                       |
| date.timezone           | Asia/Jakarta                          | Asia/Jakarta                          | UTC         |                                       |
| mbstring.language       | neutral                               | neutral                               | neutral     |                                       |
| opcache.enable          | 0                                     | 0                                     | 1           |                                       |
| opcache.enable_cli      | 1                                     | 1                                     | 1           |                                       |
| opcache.jit             | tracing                               | tracing                               | tracing     |                                       |
| opcache.jit_buffer_size | 128M                                  | 128M                                  | 256         | Adjust with your free space of memory |
| zend.assertions         | 1                                     | 1                                     | -1          |                                       |
+-------------------------+---------------------------------------+---------------------------------------+-------------+---------------------------------------+

$ php spark phpini:check opcache

CodeIgniter v4.5.4 Command Line Tool - Server Time: 2024-08-22 09:52:13 UTC+00:00

+---------------------------------+---------+---------+-------------+----------------------------------------------------------------------------------+
| Directive                       | Global  | Current | Recommended | Remark                                                                           |
+---------------------------------+---------+---------+-------------+----------------------------------------------------------------------------------+
| opcache.enable                  | 0       | 0       | 1           |                                                                                |
| opcache.enable_cli              | 1       | 1       | 1           |                                                                                |
| opcache.jit                     | tracing | tracing | tracing     |                                                                                |
| opcache.jit_buffer_size         | 128M    | 128M    | 256         | Adjust with your free space of memory                                            |
| opcache.memory_consumption      | 728     | 728     | 512         | Adjust with your free space of memory                                            |
| opcache.interned_strings_buffer | 64      | 64      | 64          |                                                                                |
| opcache.max_accelerated_files   | 58000   | 58000   | 40000       | Find many files in your project (example: find your_project/ -iname *.php|wc -l) |
| opcache.max_wasted_percentage   | 15      | 15      | 15          |                                                                                |
| opcache.validate_timestamps     | 0       | 0       | 0           |                                                                                |
| opcache.revalidate_freq         | 0       | 0       | 0           |                                                                                |
| opcache.save_comments           | 0       | 0       | 0           |                                                                                |
+---------------------------------+---------+---------+-------------+----------------------------------------------------------------------------------+

This looks good, however, I wonder if we should keep all 4 opcache keys in the default table. IMO only the first 2 will be relevant now.

Recommendations by ChatGPT CodeIgniter 4 Assistant

When deploying CodeIgniter 4 (CI4) in a production environment, configuring PHP’s OPcache properly is essential to ensure optimal performance. Below are the recommended values for the specified OPcache settings that should be good for most CI4 users. However, certain values might need adjustment based on the specific needs of your application and server resources.
Recommended PHP ini settings for OPcache in a Production Environment

opcache.enable: 1 (or On)

Recommendation: Always enable OPcache in production to improve PHP performance by caching compiled script bytecode.

opcache.enable_cli: 0 (or Off)

Recommendation: Generally, you don't need OPcache for CLI scripts. However, if your application runs CLI scripts frequently, you might consider enabling it (1 or On).

opcache.jit: off

Recommendation: Just-In-Time (JIT) compilation can provide performance improvements for certain workloads, but it's generally safe to keep it off unless you've profiled your application and observed tangible benefits from JIT.

opcache.jit_buffer_size: 0

Recommendation: Set to 0 to disable JIT unless you've determined that enabling JIT is beneficial. If enabling JIT, you might set this value between 32M to 128M depending on available memory and workload characteristics.

opcache.memory_consumption: 128M to 256M

Recommendation: This defines how much memory OPcache can use. For most applications, 128M should be sufficient, but for larger applications with many scripts, 256M might be better.

opcache.interned_strings_buffer: 8M to 16M

Recommendation: This is used for storing interned strings. 8M is a good starting point, and 16M can be used for larger applications.

opcache.max_accelerated_files: 10000 to 20000

Recommendation: This defines the maximum number of files that OPcache can cache. A setting of 10000 should work for most applications, but for large codebases, 20000 might be necessary.

opcache.max_wasted_percentage: 5%

Recommendation: When the percentage of wasted memory (due to cache fragmentation) exceeds this value, OPcache will restart. A 5% threshold is generally recommended.

opcache.validate_timestamps: 0 (or Off)

Recommendation: Disable in production. This improves performance by not checking the script modification time on every request. If your deployment process doesn’t guarantee that updated files are reloaded (e.g., through a cache invalidation strategy), you might need to set this to 1 and use opcache.revalidate_freq.

opcache.revalidate_freq: 0 to 60

Recommendation: If opcache.validate_timestamps is 1, this setting controls how often (in seconds) OPcache will check for script changes. Setting this to 0 means it checks on every request, which is not recommended for production. Instead, use a value like 60 to reduce the performance hit.

opcache.save_comments: 1 (or On)

Recommendation: Keep comments enabled as they are often required by various frameworks and tools. Disabling this can cause issues, especially with Doctrine annotations and other comment-based parsing tools.

Additional Considerations:

Memory Resources: Ensure your server has enough memory to accommodate these settings. Adjust values like opcache.memory_consumption and opcache.jit_buffer_size based on your server’s available RAM.
Application Size: Larger applications might require higher values for settings like opcache.max_accelerated_files and opcache.memory_consumption.
Testing: Always profile your application with these settings to ensure they are optimal for your specific environment. What works well for one CI4 application might need tweaks for another.

Recommendations by Claude 3.5 Sonnet

As a PHP expert, I can provide recommended values for the PHP ini settings you've listed, specifically tailored for a CodeIgniter 4 production environment. These recommendations should be suitable for most CI4 users, but keep in mind that optimal values can vary depending on your specific application and server resources.
opcache.enable = 1
opcache.enable_cli = 0
opcache.jit = 1255
opcache.jit_buffer_size = 100M
opcache.memory_consumption = 256
opcache.interned_strings_buffer = 16
opcache.max_accelerated_files = 10000
opcache.max_wasted_percentage = 10
opcache.validate_timestamps = 0
opcache.revalidate_freq = 0
opcache.save_comments = 1

Let me explain each setting:

opcache.enable = 1: This enables OPcache for web requests, which is crucial for performance in production.

opcache.enable_cli = 0: OPcache is typically not needed for CLI operations, so it's usually disabled.

opcache.jit = 1255: This enables JIT compilation with a balanced setting. The value 1255 represents tracing JIT for all JIT-able functions.

opcache.jit_buffer_size = 100M: This allocates 100MB for JIT-compiled code. Adjust based on available memory, but 100MB is a good starting point.

opcache.memory_consumption = 256: Allocates 256MB for OPcache. Adjust based on your application size and available memory.

opcache.interned_strings_buffer = 16: Allocates 16MB for interned strings. This is usually sufficient for most applications.

opcache.max_accelerated_files = 10000: Allows caching of up to 10,000 files. Adjust based on the number of PHP files in your application.

opcache.max_wasted_percentage = 10: Restarts the OPcache when 10% of memory is wasted. This helps manage memory efficiently.

opcache.validate_timestamps = 0: Disables file timestamp checking. This improves performance but requires a server restart or OPcache reset when files are updated.

opcache.revalidate_freq = 0: Related to validate_timestamps. When set to 0 and validate_timestamps is off, it never checks file timestamps.

opcache.save_comments = 1: Preserves comments in the cached files. This is important for CodeIgniter 4, as it uses doc comments for annotations.

These values provide a good starting point for a CodeIgniter 4 production environment. However, you may need to adjust them based on your specific application requirements, server resources, and performance needs. Regular monitoring and tuning can help you find the optimal settings for your particular use case.

I guess there is no consensus on recommended values?


Express Middleware access to params

If I am using "application level middleware", I should be able to see all the server data within that middleware. For example I would like to trim each string (like Laravel). Or verify some special information in the application. But unfortunately in the "application level middleware" I can't see the parameters sent via 'POST' or 'GET' or other that belong to the application. I think I should be able to see it.

@albertcito Just to clarify, are you explicitly after the path parameter and not the data in a request payload? In that case it's tricky and currently unsupported because the path parameter may not be known until later when it's been actually matched, and currently it could be matched many times with different combinations of parameters. It could be done by using a custom router though, which added some restrictions and matched ahead of time before executing your middleware.
Can you help me understand or share an example of what you're trying to do?

I'm sorry it's my fault.. I didn't realize that I have to use body-parser library to be able to see the body arguments.
import { Request, Response, NextFunction } from "express";

const express = require("express");
const bodyParser = require("body-parser");
const app = express();

app.use(bodyParser.json());
app.use(bodyParser.urlencoded({ extended: false }));

app.use((req: Request, res: Response, next: NextFunction) => {
  req.body.user = req.body.user + " extraData";
  next();
})
app.post('/login', (req: Request, res: Response) => {

  var user_name=req.body.user;
  var password=req.body.password;
  res.end("User user = "+user_name+", password is "+password);
});

app.listen(3000,() => {
  console.log("Started on PORT 3000");
})


ATOM_HOME env var doesn't work (only partially)

hi
i'm on Win7 64bit and downloaded the newest
atom-windows.zip version 0.178.0 from
https://github.com/atom/atom/releases
to test the new ATOM_HOME env var feature.
i've set the env var ATOM_HOME to the path
D:\portable_programs\Atom\ATOM_HOME
and started atom.exe .
now in the folder
D:\portable_programs\Atom\ATOM_HOME
appeared only one dir "compile-cache" and
only one file "config.cson". it looks like
an incomplete part of the files and folders
that would normally be created on the first run
of atom.exe in the folder
C:\Users\ <user> \.atom
the content of "config.cson" looks different
than usual and contains the line
"exception-reporting"
the folder
C:\Users\ <user> \AppData\Roaming\Atom
was created in the same place, like without
a present ATOM_HOME env var. so for the folder
C:\Users\ <user> \AppData\Roaming\Atom
the redirection via ATOM_HOME does not
work in the actual Atom version 0.178.0 .
for the folder
C:\Users\ <user> \.atom
the redirection via ATOM_HOME env var
works partially and the files
.gitignore , init.coffee , keymap.cson ,
snippets.cson and styles.less and the
folder packages were not created, when
ATOM_HOME was set.
also i would expect that the files and folders of
C:\Users\ <user> \.atom
would be placed in the ATOM_HOME env var mode
in the same subfolder .atom like
<path in ATOM_HOME> \.atom
but they landed directly in
<path in ATOM_HOME> \

Did D:\portable_programs\Atom\ATOM_HOME exist before you launched Atom?

C:\Users\  \AppData\Roaming\Atom was created in the same place

This is expected, ATOM_HOME is for compile caches, installed packages & themes, local styles, config, snippets and keymaps.
C:\Users\ <user> \AppData\Roaming\Atom is where the browser's localStorage database is written to and other Chromium specific file.

Did the D:\portable_programs\Atom\ATOM_HOME folder exist before you launched Atom?
yes, i've created the folder before i started atom.exe .

now i've deleted first the folder
D:\portable_programs\Atom\ATOM_HOME
and then started atom.exe and it worked fine now when
D:\portable_programs\Atom\ATOM_HOME
was created by atom.exe .
and all expected sub folders and files were present too.
somehow i thought that
C:\Users\ <user> \AppData\Roaming\Atom
would be redirect also.
many thanks for the fast respond and info !

now i've deleted first the folder D:\portable_programs\Atom\ATOM_HOME

Yes, this folder is only populated by Atom with the template files when it doesn't already exist.

somehow i thought that C:\Users\  \AppData\Roaming\Atom would be redirect also.

Yes, I'll look into whether this can be added in, thanks for reporting this.

@MarjanTomas Is this still reproducible?
CC @damieng to Tag Windows https://github.com/atom/atom/issues/3560#issuecomment-217263907


package tomli-1.2.1-pyhd8ed1ab_0 requires python >=3.6, but none of the providers can be installed

Hi @thewchan !
Thanks for maintaining the recipe!
In a complex conda enviroment installation:
https://github.com/chapmanb/cloudbiolinux/blob/master/contrib/flavor/ngs_pipeline_minimal/packages-conda.yaml
I am facing an issue:
Executing transaction ... working... done
Encountered problems while solving:
 - package tomli-1.2.1-pyhd8ed1ab_0 requires python >=3.6, but none of the providers can be installed

$ conda list | grep python
python 3.7.6

3.7.6 > 3.6, do you have any idea why the error message?
After the message conda falls into infinite dependency conflict solving.
This installation of this environment has been done many times successfully in the past,
however, tomli was not part of it.
This is the first time one of the updated packages requested tomli, thus the issue surfaced.
Thanks in advance!
Sergey

Thanks @thewchan
We are on conda 4.10.3 and we are using mamba to speed-up the installation.
Out of the installer script, from the CLI mamba installs tomli ok, conda stalls.
We are also having similar issues with other packages - libtiff, libdeflate, so the issue is likely not related to tomli.


Get CLI on Windows package manager Chocolatey and automate publishing like we did with brew

Reason/Context
We want CLI to be easy to install for anyone on any OS, without a need to use Node/NPM
We have AsyncAPI already on MacOS package manager -> brew. Time to get it on Windows and I guess Chocolatey is most popular, although if you think not, please propose alternative.
Description

get CLI on given package manager
extend this workflow https://github.com/asyncapi/cli/blob/master/.github/workflows/bump-homebrew-formula.yml with separate job to get automated version updates in the given package manager

was implemented with https://github.com/asyncapi/cli/pull/847


errors when running LSTM on cuda for electricity dataset

I created the script electricity_2013_03_31 to run multiple time series in the electricity dataset. I have added:

Early stopping based on log-likelihood or mse of the validation set.
Rolling-window predictions for the test set.
A loop to go through each time series in the dataset by adding the time series index ts_idx as the showed in the following picture:

The code runs fine in CPU but it gives this error when running on cuda:

You can reproduce the error by running electricity_2014_03_31 in my repos at (https://github.com/van-dai-vuong/cuTAGI.git)

I have encountered the following errors as well when running the time_series_forecasting script provided in the examples folder:

I cannot run the script on GPU with a batch size of 1. I get a device memory allocation error: ValueError: Error in file: /home/dw/cuTAGI_DW/src/data_struct_cuda.cu at line: 573. Device memory allocation. However, on CPU the code runs fine.

When trying to run the same script with a higher batch size such as 20 or 32, the GPU execution results in the following:

Compared to CPU execution:

@van-dai-vuong @jamesgoulet @lhnguyen102

@davidwardan I am working on fixing the issue. I will get it fix by the end of the weekend. I'll let you know when it is done

Here is PR for the fix. I wanted to added @davidwardan and @van-dai-vuong for the reviewers but you need to accept my invite. Please take a look at the PR and confirm if it resolves your issue.

Thank you @lhnguyen102! I will look at it today and get back to you.


Remove duplicate items

The same tool health is being displayed multiple times (different timestamps) on a site. Remove duplicate items.
https://tool-watch.toolforge.org/

done


esp-idf/examples/bluetooth/bluedroid/classic_bt/hfp_hf   example  (IDFGH-2531)

I execute the 'hfp_hf.' example without any modification but i can't see the log outcome of 'Audio Connection and Disconnection'. Please refer my attachment file!!!

When i modified 'BR/EDR Sync(SCO/eSCO)' in menuconfig 0 into 1 then the following picture appears!!!
But 'handle hex' value didn't print value 181 but  value 180.
What do i add more configuration? and What does the diffierence between two values mean?
also when smartphone is connected to 'ESP32', can i see two devices are disconnected? can you let me know how to keep connecting between smartphone and ESP32?
Specification :
Board: nodemcu (esp-wroom-32)
version: ESP-IDF master
power: usb

Hi @ACJ-kor ,

BR/EDR Max Connection means the number of SCO links at your case.
Handle is just a number allocated by HCI from 180, it has no other meaning.
We have specific disconnected log with disconnecting reason.

Thanks
Abner

@ACJ-kor Thanks for reporting. Feel free to reopen if the issue still happens, or help open another issue for new issues. Thanks.


Incorrect behaviour FlexboxLayout in HorizontalScrollView

[x] I have searched existing issues and confirmed this is not a duplicate

Issues and steps to reproduce
This issue started only from version 2.0.1, earlier versions work correctly.
Expected behavior
Version 2.0.0

Version 2.0.1

So I want FlexboxLayout to be really wrap_content and scrollable in HorizontalScrollView
Version of the flexbox library
2.0.1
Link to code
Layout code - https://pastebin.com/ye59gGh1

Yea, something definitely broke in 2.0.1
I have a before and after of it and v2.0.0 in a horizontal scrollview, no wrap,

I think flex box lost over-flow... and I hope its not a feature 😕

To change  layout_flexShrink  may help.

Any updates? Not working with 3.0.0 also

Any updates? Not working with 3.0.0 also

Try layout_flexShrink

Any updates? Not working with 3.0.0 also

Try layout_flexShrink

I've tried - no result ):


Speed up setup script

A couple of measures to speed up the Setup.sh script.

Added an option to run the make commands with multiple async jobs.
Boost only compiles the used libraries, system.

It runs now in about 5min30s in our build machine (before it was 12min) :+1:

This change is

@marcgpuig is assigned because he is working on the windows version of this script

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

Comments from Reviewable

Setup.sh, line 153 at r1 (raw file):

pushd protobuf-source >/dev/null

git submodule update --init --recursive

You can add the "--depth=1" attribute here. More info.
Or you can merge it with the clone (using "--recurse-submodules") and it will spare the "--depth" attribute too.

Comments from Reviewable

Reviewed 1 of 2 files at r1.
Review status: 1 of 2 files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Review status: 1 of 3 files reviewed at latest revision, 1 unresolved discussion.

Setup.sh, line 153 at r1 (raw file):
Previously, marcgpuig (Marc Garcia Puig) wrote…
You can add the "--depth=1" attribute here. More info.
Or you can merge it with the clone (using "--recurse-submodules") and it will spare the "--depth" attribute too.

Done, I amended the commit. Also the -j doesn't seem to be available in Ubuntu 16.04

Comments from Reviewable

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

Comments from Reviewable


feat(bound-select-field): dirty indicator

I started doing this in BluePrint then realized it should probably have some first-class support in Beam. Shows an indicator if an input is dirty/has not been saved yet.

In the future I suppose we could add this to the other BoundXFields.

Totally fair. This entire ticket turned into Tyler Design Hour so yeah I should probably start tossing ideas back to Radhi for verification.


python3Packages.flit-scm: init at 1.7.0

Description of changes

Add the flit-scm Python build backend.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Ended up here searching for this package.
nix \
shell \
--expr \
'(
  with builtins.getFlake "github:cpcloud/nixpks/9f8a55aee51d118f81fb66af3691b359a306d00b";
  with legacyPackages.x86_64-linux;
  python3.withPackages (p: with p; [ flit-scm ])
)' \
--command \
python3 -c 'import flit_scm; print(flit_scm.__version__)'

Outputs:
0.1.dev1+g27ac37a.d19800102

Some metadata about the system


Disable time option

this jquery plugin has a DisabledDates option, is there a similar option that would disable certain times in the time picker?

Did you try parameter allowTimes?
http://xdsoft.net/jqplugins/datetimepicker/#allowTimes

Currently, the 'allowTimes' parameter removes all other time selections. Sometimes it would be nice to show all of them and just disable some of them.


[BUG] FM Radio frozen

Describe the bug
When I try to change the frequency (or do pretty much anything) in the FM Radio, it freezes the entire radio completely. Rebooting appears to be the only fix.
To Reproduce

Open the FM Radio
Press UP or DOWN
Try to exit

Expected behavior
The FM Radio should work, and I should be able to adjust the frequency.
Pictures/video

(it's just stuck there. looks incomplete?)
Details:

Firmware version: v.19.6 (also tested with v.19.5)
Bandwidth: any
Power: any
RxAGC: any
Gain settings (LNAs/LNA/PGA): default, probably
other: Pretty sure it worked when I built from a random commit around v.19.4 or so, but of course I didn't note which one it was or what I changed in the Makefile. Sorry about that.

Good bug report!
It is issue with loading the last played frequency from EEPROM that lacks validation. For now you might want to try pressing DOWN and being patient and maybe the fm chip will tune down to valid frequency, otherwise the EEPROM value needs to be changed to valid value.
This needs a FW fix though.


man - No manual entry for {anything}

There was no man by default and I tried installing man-db, man-pages, et al. manually. Even though these install, there is no /usr/share/man folder hence No manual entry for {anything}. Am I missing something?

https://bbs.archlinux.org/viewtopic.php?id=255015
Then set your MANPATH and reinstall man-pages. I'll fix this in the next release


KafkaLookupExtractorFactory: shutdown kafka consumer on close()

This call:
for (final MessageAndMetadata<String, String> messageAndMetadata : kafkaStream) {

never exits until the Kafka consumer is shutdown (or consumer.timeout.ms elapses). The consumer needs to be signaled to shutdown in close() so that the for loop exits and our Runnable can terminate, otherwise the JVM will not exit (the ConsumerFetcherThread created by the Kafka consumer is non-daemon).
This fixes #3531

I can confirm that with a version built from this branch, the issue I reported in #3531 no longer reproduces. Yay! 🎉

@drcrallen any more comments on this PR?

@drcrallen any more comments on this PR?

@dclim Now it's complaining it's called 3 times instead of 2. Could you check that?
https://travis-ci.org/druid-io/druid/jobs/170028487
https://travis-ci.org/druid-io/druid/jobs/170029074
https://travis-ci.org/druid-io/druid/jobs/170029747
occurs only in JDK8. interesting..

@navis there was a timing issue in the test, should be fixed in #3604


add documentation deep-dive into trained models

This should answer some of the questions that have come up about how to use NeuralGCM models.

merged in 789ed6f8bf0dc206e263f79058f6b3f8701613f3


Listiter impl

Added ListIter implementations for HashMap and OrdMap. Still unsure if handling key modifications the best way, but values should be good. (This is my first PR here)

I'm not very familiar with web assembly but I can't get the test to run on my computer. Not sure if there's additional setup I need to do it or possibly its having trouble on an M1-mac which I'm using.

I believe you're missing an entry in druid/examples/web/src/lib.rs for your example. Maybe that will solve the test failure?
(I abandoned a similar PR a while back glad to see this being picked up!)

Yes, that was exactly it. I just pushed the change adding to the web examples and all the tests ran successfully. Thanks.


Track pad sensitivity

On my laptop I found working with chart quite hard.
Zooming in and out with track pad is like flying with speed of light. Same on tvjs website.
It is well known behavior. Probably you can check how it was fixed in this repo https://github.com/openseadragon/openseadragon/pull/754
Was tested on windows laptop dell XPS 13 on latest version of chrome browser.

Also same behavior on quite old Lenovo laptop.

Also same behavior on quite old Lenovo laptop.

Any ideas on how to fix this?
Tested on my laptops, result: HP the trackpad is actually slower, Lenovo: works ok, but the panning is inverted.

Any ideas on how to fix this?
Tested on my laptops, result: HP the trackpad is actually slower, Lenovo: works ok, but the panning is inverted.

Take a look at facebook solution
https://github.com/facebookarchive/fixed-data-table/blob/master/src/vendor_upstream/dom/normalizeWheel.js

Take a look at facebook solution
https://github.com/facebookarchive/fixed-data-table/blob/master/src/vendor_upstream/dom/normalizeWheel.js

Also you can check this thread https://stackoverflow.com/questions/5527601/normalizing-mousewheel-speed-across-browsers

Also you can check this thread https://stackoverflow.com/questions/5527601/normalizing-mousewheel-speed-across-browsers

Jeez, that fb code doesn't look good. Can you experiment on you laptops?
https://github.com/tvjsx/trading-vue-js/blob/master/src/components/js/grid.js#L43
https://github.com/tvjsx/trading-vue-js/blob/master/src/components/js/grid.js#L378
I remember I was fighting with this to make it work on Mac Os. May be there is a quick fix for Windows (I guess that's the problem).

Jeez, that fb code doesn't look good. Can you experiment on you laptops?
https://github.com/tvjsx/trading-vue-js/blob/master/src/components/js/grid.js#L43
https://github.com/tvjsx/trading-vue-js/blob/master/src/components/js/grid.js#L378
I remember I was fighting with this to make it work on Mac Os. May be there is a quick fix for Windows (I guess that's the problem).

@oneart-dev This problem is hard to catch. I'll reopen the issue if you are planning to contribute.


List support

I needed to pick strings and ints from a list so I hacked this together. I'm sure it can be improved so any suggestions are welcome.
Usage:
char * ANIMATION_TYPE = "Type";
char * EQUALIZER = "EQ";
char * COLOR = "Color";
char * BRIGHTNESS = "Brightness";

// AUDIO
std::vector<String> eqModes = {"Normal", "Pop", "Rock", "Jazz", "Classic", "Bass"};
std::vector<String> playbackModes = {"All", "Shuffle"};

// ANIMATION
uint8_t animationBrightness = 7;
std::vector<int> brightnessTypes = {2, 3, 7, 9, 12, 30, 60, 100, 180, 255};

uint8_t animationIndex = 0;
std::vector<String> animationTypes = {"Ocean", "Pride", "BlendWave", "Sinelon", "Confetti"};

uint8_t colorIndex = 0;
std::vector<String> colors = {
    "Random",
    "Red",
    "Orange",
    "Yellow",
    "Green",
    "Aqua",
    "Blue",
    "Purple",
    "Pink"
};

// declare callback functions
void changeIntList();
void changeStringList();

extern MenuItem mainMenu[];

MenuItem mainMenu[] = {
  ItemHeader(),
  ItemStringList(ANIMATION_TYPE, animationTypes, 0, changeStringList),
  ItemStringList(COLOR, colors, 0, changeStringList),
  ItemIntList(BRIGHTNESS, brightnessTypes, 0, changeIntList),
  ItemStringList(EQUALIZER, eqModes, 0, changeStringList),
  ItemFooter()
};

LcdMenu menu(LCD_ROWS, LCD_COLS);

// initialize menu
menu.setupLcdWithMenu(
    DisplayRSPin,
    DisplayENPin,
    DisplayD4Pin,
    DisplayD5Pin,
    DisplayD6Pin,
    DisplayD7Pin,
    DisplayBacklightPin,
    mainMenu
);

void changeIntList() {
  uint8_t index = menu.getCursorPosition();
  MenuItem* item = menu.getItemAt(index);
  const char* menuLabel = item->getText();
  int selectedIndex = item->getSelectedIndex();
  int selectedLabel = item->getIntItems().at(selectedIndex);

  Log.info(F("%s: %d - %d" CR), menuLabel, selectedLabel, selectedIndex);

  if (menuLabel == BRIGHTNESS) {
    animationBrightness = selectedIndex;
    _neopixels->setBrightness(brightnessTypes.at(animationBrightness));
  }
}

I also needed to redraw the menu, like this:
// update menu
mainMenu[1].setSelectedIndex(animationIndex);
mainMenu[2].setSelectedIndex(colorIndex);
mainMenu[3].setSelectedIndex(animationBrightness);
mainMenu[4].setSelectedIndex(_player->equalizerMode);
menu.redraw();

The PR also includes a fix/workaround for #21

Hello, can you take a quick picture of how the lists are displayed and upload it here, I'm currently in a position where I can't test. Thanks

Sounds good. Might take a day or two.

Opened #26 for the redraw method introduced in this PR.

ps. I think you should release 2.0.0 as it is (it's beta now?), and use this pr as ideas for newer version, e.g. 3.0. I like how this library kept things minimal and it always should.

I have updated the CONTRIBUTING.md, please update this PR to reflect the guideline.

this sounds quite hostile hehe. this pr is a sketch or work in progress i was hoping we could discuss. i will review these guidelines

this sounds quite hostile hehe. this pr is a sketch or work in progress i was hoping we could discuss. i will review these guidelines

Since I don't have time to test, let GitHub do the testing, the most important part is including an example project, that's what I wanted you to see 😄


Model class name and items per page

Hi!
There are two questions.

Is there a setting for the number of lines on the page (items per page)?

The name of the model class (table in the database) becomes part of the address. Maybe the class has an alias so that the address can be substituted with an alias rather than the class name?

Hi @itoop2019!

Yes, you can set up grid options:

new EasyDataViewDispatcher({
    grid: {
       paging: {
        enabled: true // enable paging
        pageSize:  100 // default page size
        allowPageSizeChange: true, // add selector to change page size
        pageSizeItems: [20, 50, 100, 200] // page size options
    }
    }
}).run()

Unfortunately, there is no such possibility yet,  but we can consider adding it.

Many thanks, it works!
I will be very grateful if you add an alias.


Follow on questions from the Witness PR review

The following questions came up in the review of PR #136.  Let’s discuss and resolve them either by not adding them or adding a new PR.

[ ] @brianorwhatever : I don't think we say anything about whether witness can be disabled after it's been enabled or is this forbidden?
[ ] @brianorwhatever : Confirm the wording around “current or later” proofs of versions and whether it is in the right place.
[ ] @PatStLouis : Wouldn't just signing the versionId be sufficient since the versionId is a sort of hash of the entry? Meaning if the entry validation passes, a proof over the versionId only would be sufficient?
[ ] @PatStLouis : Does it make sense to instead only keep the proofs about the two most recent log update entries?

RE: Disable Witnesses.  I think we can add a line that allows for including a ”witness” : [] as valid. That entry would have to be approved by the active witnesses, and thereafter there would be no active witnesses.

This is dumb to have these as one issue.  Will move the other issues to new issues.  Doh!


[feat][test]: Squish Automated Test Framework - Dev Submit GUI Settings flow

What was the problem/requirement? (What/Why)
We currently do not have automated user-facing tests for the Submitters dialogue which would catch regressions/bugs from being released to customers.
What was the solution? (How)
Implement/add tests for the Submitters dialogue using Squish Automated Test Framework. This is a first set of automated tests for the Submitters dialogue, and this is PR focuses on the Dev Submit GUI - Settings flow/config.
What is the impact of this change?
Implement/add tests for the Submitters dialogue using Squish Automated Test Framework. This is a first set of automated tests for the Submitters dialogue, and this is PR focuses on the Dev Submit GUI - Settings flow/config.
How was this change tested?
I ran my tests locally and ensured that the tests passed with no issues.
Was this change documented?
I will have a quip doc/draft of Squish Automated framework, how to get set up with it, debug, etc.
Is this a breaking change?
No.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Will be closing this PR as I submitted a new one here: https://github.com/aws-deadline/deadline-cloud/pull/399


export: Fix sequence numbers when in dev mode

It would be nice to have export/import work correctly round trip in a dev environment.  It mostly works, with the following limitation:
Quoting @timabbott from #1567 with emphasis added by me:
"Oh, and a final issue is sequence numbers in the database -- after you do an import, you need to do some ALTER SEQUENCE operations or you won't be able to send messages. See https://github.com/timabbott/zulip/commits/base-numbers for an example of how to do this. Ideally, at least the --destroy-rebuild-database option would automatically reset sequence numbers to MAX(table) + 1 after an import. For actual production use, I expect we'll use something like that base-numbers commit and then import using the --import-into-nonempty option, but having something nice for local testing would be good."

It's possible the nicest solution to this would be to add full support for relocating IDs on import.

Does that actually address updating the sequences, though?  (The sequence problem is not about the import; it's about explicitly setting ids.)  Or are you implying that we insert all records without ids from the source db, and then re-map all the foreign keys?

Partially, but you're right my comment was unclear.  A nice way to do this would be to just update the sequences on one end of the migration (if the tool preplanned how much space it needed, you could imagine doing it at the start being a good model).  The problem with that approach is that we also need the server the data is being imported to not have used the IDs to be imported, and if you're importing what was the first realm on a server, it's very likely to be the case that some of the IDs you need were used by populate_db; thus we need to do this ALTER SEQUENCES partway through database setup...

This was done in a better way as part of the Slack import effort.  Huge thanks to @rheaparekh for her hard work on the sequence numbers problem.


Create cli bundle command

Creates a bundle command for the CLI desktop targets.
Currently this has only been confirmed to work for Windows (x64) and MacOs (arm). Linux targets have not been tested.

Sweet!


Installation fails when following instructions

In a clean new conda environment, installation fails using the instructions on the docs page.
The final error after pip install jmespath and then pip install exotedrf is

ERROR: Cannot install exotedrf==1.4.0 and exotedrf==2.0.0 because these package versions have conflicting dependencies.
The conflict is caused by:
exotedrf 2.0.0 depends on ray
exotedrf 1.4.0 depends on ray
... but no guidance is provided on what ray refers to.

The full error trace is below.
(exotedrf) user@computer:/directory/$ pip install jmespath
Collecting jmespath
Using cached jmespath-1.0.1-py3-none-any.whl.metadata (7.6 kB)
Using cached jmespath-1.0.1-py3-none-any.whl (20 kB)
Installing collected packages: jmespath
Successfully installed jmespath-1.0.1
(exotedrf) user@computer:/directory/$ pip install exotedrf
Collecting exotedrf
Downloading exotedrf-2.0.0.tar.gz (76 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Collecting applesoss==2.1.0 (from exotedrf)
Downloading applesoss-2.1.0.tar.gz (25 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Collecting astropy (from exotedrf)
Downloading astropy-6.1.4-cp313-cp313-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (10 kB)
Collecting astroquery (from exotedrf)
Using cached astroquery-0.4.7-py3-none-any.whl.metadata (7.2 kB)
Collecting bottleneck (from exotedrf)
Downloading Bottleneck-1.4.2-cp313-cp313-manylinux_2_5_x86_64.manylinux1_x86_64.manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (7.7 kB)
Collecting corner (from exotedrf)
Using cached corner-2.2.2-py3-none-any.whl.metadata (2.2 kB)
Collecting jwst==1.12.5 (from exotedrf)
Downloading jwst-1.12.5.tar.gz (18.5 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 18.5/18.5 MB 17.5 MB/s eta 0:00:00
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Collecting matplotlib (from exotedrf)
Downloading matplotlib-3.9.2-cp313-cp313-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (11 kB)
Collecting more-itertools (from exotedrf)
Using cached more_itertools-10.5.0-py3-none-any.whl.metadata (36 kB)
Collecting numpy (from exotedrf)
Using cached numpy-2.1.2-cp313-cp313-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (60 kB)
Collecting pandas (from exotedrf)
Downloading pandas-2.2.3-cp313-cp313-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (89 kB)
INFO: pip is looking at multiple versions of exotedrf to determine which version is compatible with other requirements. This could take a while.
Collecting exotedrf
Downloading exotedrf-1.4.0.tar.gz (69 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
Collecting applesoss==2.0.2 (from exotedrf)
Downloading applesoss-2.0.2.tar.gz (25 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
ERROR: Cannot install exotedrf==1.4.0 and exotedrf==2.0.0 because these package versions have conflicting dependencies.
The conflict is caused by:
exotedrf 2.0.0 depends on ray
exotedrf 1.4.0 depends on ray
To fix this you could try to:

loosen the range of package versions you've specified
remove package versions to allow pip to attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

Hi Ian,
I'm not able to reproduce this error, either on Mac or Linux. The installation proceeds without issue in a fresh conda env in both cases.

I'm really in trouble then, I guess!  Thanks -- we'll continue to troubleshoot.


[Fixed in 0.10.0] Exception in native call from JS (flat-list-mvcp on Android)

Flagging this breaking issue so that hopefully the setup docs can be amended or changes can be made to flat-list-mvcp.
My team has been dealing with a bug for the last few days associated with using this package and the wrong version of flat-list-mvcp that results in the following exception when you attempt to open a <MessageList /> on Android:
Exception in native call from JS

This particular issue arises with version 0.0.9 of flat-list-mvcp but does not occur if we switch back to version 0.0.5.
To replicate this issue, open the stream-chat-react-native/examples/TypeScriptMessaging example. Before building the project on Android, run the following command yarn add @stream-io/flat-list-mvcp --save which will bump the flat-list-mvcp version from 0.0.5 in the package.json to 0.0.9. Click around into a few of the chats and you will be able to consistently reproduce the above error (it may take a few seconds before appearing).
This issue has been previously flagged for flat-list-mvcp noted in this issue and was supposed to be fixed by this PR for version 0.0.9. I will be flagging it again as an issue on flat-list-mvcp but wanted to make sure that something was recorded here to ensure other developers have to spend days going over issues that we've been able to identify.
For developers, you can downgrade your version of flat-list-mvcp to 0.0.5 to resolve this issue by running yarn add<EMAIL_ADDRESS>For the GetStream team, this particular issue is not noted in the Setup Docs likely because all of these libraries have been updated very very recently. The docs currently only specify a required version for react-native-reanimated for installation and don't have any restrictions on the version for flat-list-mvcp. I would recommend specifying a version for the time being to be 0.0.5 until these libraries are more stable.
P.S. Our team really appreciates all of the work the GetStream team takes out of building incredible chat experiences into our apps! Thank you!
gz#10043

@lc3t35 Could you please upgrade to<EMAIL_ADDRESS>and let me know if you see this issue fixed? I have updated the native implementation to resolve this.

no more error with<EMAIL_ADDRESS>! congrats !

Also @BrianLi101 ^^

Unfortunatly I didn't wait long enough ... I still have the error with<EMAIL_ADDRESS>

Could you share some code? Want to see how I can reproduce it on my end

Here you are :
<FlatList
          data={momentsFiltered}
          keyExtractor={keyExtractor}
          renderItem={renderMomentItem}
          horizontal={false}
          numColumns={1}
          viewabilityConfig={viewConfigRef.current}
          onViewableItemsChanged={onViewRef.current}
          ref={setRef}
          onScrollToIndexFailed={onScrollToIndexFailedMoments}
          onEndReached={fetchMoments}
          onEndReachedThreshold={0.4}
          scrollEventThrottle={20}
          initialNumToRender={6} // https://reactnative.dev/docs/optimizing-flatlist-configuration
          bounces={false}
          windowSize={21}
          removeClippedSubviews
          maxToRenderPerBatch={10}
          updateCellsBatchingPeriod={100}
          showsVerticalScrollIndicator={false}
          ListFooterComponent={
            momentLoading ? <ActivityIndicator size='large' /> : renderFooter
          }
          maintainVisibleContentPosition={{
            autoscrollToTopThreshold: undefined,
            minIndexForVisible: 0,
          }}
/>

Thanks dev for the hard work, it's a great library.
I see the same crash on Android, sometimes when updating the data prop of the flat list.
Is there a way to simply catch the exception on native side as a hot fix / exception handling while you are working on the solution?

@lc3t35 @EndlezzCoding Just published a patch 0.10.1. There was a source for this issue, which is fixed now. Please upgrade and let me know if you still see the issue :)

ok for me 0.10.1, no more crash, thank you @vishalnarkhede

ConcurrentModificationException
hola yo uso
@stream-io/flat-list-mvcp": "^0.10.1",
me sale ese error cuando uso la aplicacion

ok para mí 0.10.1, no más accidentes, gracias @vishalnarkhede

hi, i get that error out of nowhere when i browse the app


fix maven_args in action-maven-publish@v1

Sorry I used a made-up argument by ai agent, this one is valid.

Codecov Report
Patch and project coverage have no change.

Comparison is base (f70259f) 0.00% compared to head (8443a74) 0.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@          Coverage Diff           @@
##           master    #222   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          74      74
  Lines        2513    2513
  Branches      275     275
======================================
  Misses       2513    2513

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

Seems it failed due to the node12--> node16
[deploy](https://github.com/nebula-contrib/ngbatis/actions/runs/6135530410/job/16649376835)
The following actions uses node12 which is deprecated and will be forced to run on node16: samuelmeuli/action-maven-publish@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Error:
node:child_process:935
    throw err;
    ^

Error: Command failed: mvn clean deploy --batch-mode --activate-profiles deploy --settings /home/runner/work/_actions/samuelmeuli/action-maven-publish/v1/settings.xml -Dmaven.javadoc.skip=true
    at checkExecSyncError (node:child_process:861:11)
    at execSync (node:child_process:932:15)
    at run (/home/runner/work/_actions/samuelmeuli/action-maven-publish/v1/index.js:23:9)
    at runAction (/home/runner/work/_actions/samuelmeuli/action-maven-publish/v1/index.js:83:2)
    at Object.<anonymous> (/home/runner/work/_actions/samuelmeuli/action-maven-publish/v1/index.js:89:1)
    at Module._compile (node:internal/modules/cjs/loader:1198:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1252:10)
    at Module.load (node:internal/modules/cjs/loader:1076:32)
    at Function.Module._load (node:internal/modules/cjs/loader:911:12) {
  status: 1,
  signal: null,
  output: [ null, null, null ],
  pid: 4259,
  stdout: null,
  stderr: null
}


Combine poynting tune and autotune

Closes #138
Closes #220
Closes #208
Closes #109
Should also test out #190 with these changes, and fix it if we can
edit:
Closes #190 (at least in my brief testing)

Note: "Syntax error" in LGTM is for Py2 only