On hovering the text is now visible inside the button on page : meshery.io/subscribe

Signed-off-by: Utkarsh Mishra<EMAIL_ADDRESS>Description
On hovering the text is now visible inside the button on page : meshery.io/subscribe
This PR fixes #915
Notes for Reviewers
https://user-images.githubusercontent.com/67385503/205121429-09f0743d-3452-40df-b1d5-49ca49094649.mp4
Signed commits

[x] Yes, I signed my commits.

@UtkarshMishra12, LGTM, Thanks👌


[RFC] What should we do when we receive room announcement

Today we tested the room announcement feature with wechaty-puppet-padpro.
Seems like the room announcement is a special type of message, which mentions everyone in the room, and the data structure is different from the regular messages. So calling the mention() or mentionSelf() method on an announcement message will throw out errors with wechaty-puppet-padpro. I haven't got time to pull out the actually message content yet, will do this later, but before that, I wondered how should we handle such message, and would like to start the discussion about the way that we deal with such kind of message.
Shall we put all room members in the message.mention() call? Or we could have some other message type?
Please let me know what you think @huan

I will think that the announcement message should be seen from the user side as a normal mention message for themselves.

Okay, if we implement the feature in that way, then for Message.mention() function, we shall return back the whole list of the room members, is that correct?

Correct. That's the idea. Please go ahead.

Okay, will impl in that way. Close this issue.

Great.
Looking forward to this new feature, and thank you very much for keeping improving the Wechaty.
Cheers!

BTW: I'd like to keep this issue to be open so that you could be able to reference this issue in your Pull Request and commits.

This issue was originally opened for discussion of how to deal with room announcement message, and we have a conclusion, so this topic should be considered as closed.
But you want to track this issue's progress, so let's rename this issue and pivot the purpose of this issue to fix the room announcement message implementation

Thank you for renaming this issue and keeping track it.
Would love to know the following works related to this issue.

This issue is resolved in wechaty-puppet-padpro for version that is newer than v0.3.12
Can we close this issue? @huan

Of course. And it would be great if you could link the related issues & pull request to this issue before closing it.
Thanks for the contribution!

Okay, linked the issue and close this issue :)


Custom request retry strategy

Currently if request fails due to, let's say no connection error , the the framework will try to retry after 4 seconds then after 16 seconds (4^2), then after 64 seconds (4^3) and only after that the framework will give up and call request failed callback. It means user will see error message after 4+16+64=84 seconds. In some cases would be great to be able to specify custom retry logic. I have created a version with older PINRemoteImage version, tried to make a pull request, seems a lot has been changed, here is the link to that implementation. I saw that there is a way to skip retrying, it could be the solution in some cases, but overall would be great to have more control on that.

PR here: https://github.com/pinterest/PINRemoteImage/pull/386


Consider Tokens in Set Analyser

For now only the cards from the main set are considered. Tokens should also (optionally) be included (cli option).

Did that a while ago. Works now ^.^


Space requirements for Spaceeye Cloud Removal?

After setting up the sentinel-spaceye notebook environment & running the code, the process fails. Currently, I am using the recommended space of ~500GB. But this is clearly not enough. The cluster crashes at the preprocess_s2_improved_masks step. Current RAM: 16GB, Disk Space: ~500GB (100% consumed).
Need the Spaceye pipeline for cloud removal on a large number of data points (500), regions (roughly 20km2 in the area), and times (average period = 20 days). What are the suggested system requirements for this task?

Hello @madhavaggar, the recommended space would be to run the notebook with the sample inputs. Can you clarify what you mean by 500 data points? Do you mean 500 different regions, each with their own time period? Are these regions near each other?
For 20 days there should be on average 4 products for a Sentinel-2 tile. If all points require different products, this would mean downloading 2000 Sentinel-2 tiles, each tile using a few GB of disk space. If the regions are near each other (in the same tile) and the time range is the same, FarmVibes will automatically reuse the already downloaded data, so no recomputation will be made and the necessary disk space we be reduced accordingly.
It's hard to estimate exactly how much disk space a workflow will require, but if you provide a bit more information I can try to help.

I would first like to know about the sentinel_spaceye.ipynb notebook provided as a sample. It consumed approx. 500GB of disk space and crashed at the preprocess_s2_improved_masks step in the "Working with cloud masks" section.
Could you give specifics on how much space this pipeline uses (or should use) for the sample notebook provided? I believe that would help a lot of others figure out the disk space that will be required.
The pipeline I want to run has approx 500 data points spread apart in time and space. Each data point representing the centroid of a region has a variation of 0.05 either in Latitude or Longitude as compared to every other data point. So they are definitely not close to each other. Each region is approximately 100km^2 in radius.

Closing this issue for now. @madhavaggar, feel free to reopen it if you have more questions.


Clear input chars on long pressed

This feature is good for user experience in which they don't have to clear characters one by one.
Usage:
clearOnLongPressed is false by default, so it doesn't affect old package users who don't need it.
inputButtonConfig = InputButtonConfig(
  clearOnLongPressed: true,
  ...
);

https://user-images.githubusercontent.com/29684683/154835089-db4ce07f-7df5-473a-a1d8-00cd1a688a52.mp4

Thanks for adding that nice feature!


"bufferless drawing should not be supplied with buffers"

I'm doing bufferless drawing, but I'm getting this error message when I draw. It seems that this may be related to the way the command buffer is created:
AutoCommandBufferBuilder::primary_simultaneous_use(pipeline.device().clone(), queue.family()).unwrap()
	.begin_render_pass(framebuffer.clone(), false, vec![[0.0, 0.0, 1.0, 1.0].into()]).unwrap()
	.draw(pipeline.clone(),
		DynamicState::none(),
		Vec::new(),
		(),
		(),
	).unwrap()
	.end_render_pass().unwrap()
	.build().unwrap()

For the vertices argument of draw I provide an empty vector, because I have no actual buffer to provide. If I use anything else there such as () or None, I get a compile error:
   --> src/main.rs:409:6
    |
409 |                     (),
    |                     ^^ expected struct `std::vec::Vec`, found ()
    |
    = note: expected type `std::vec::Vec<std::sync::Arc<vulkano::buffer::BufferAccess + std::marker::Send + std::marker::Sync + 'static>>`
               found type `()`

I'm not sure what the proper way to handle this is. The compiler demands a Vec, but if I provide one then Vulkano gets upset with me.

@Rua I actually came across this issue myself. After digging I found that it was because the GraphicsPipelineAbstract implements the expected type. Right now you'd need to be explicit about what your GraphicsPipeline looks like, but currently I'm trying to investigate other ways round it

Has a solution been found for this?

The solution to this is deeply hidden inside the docs:

Note that the concrete type of the graphics pipeline using BufferlessDefinition must be visible to the command buffer builder for this to be usable.

https://docs.rs/vulkano/0.19.0/vulkano/pipeline/vertex/struct.BufferlessVertices.html
I ran into this issue because my Pipeline was of type Arc<dyn GraphicsPipelineAbstract> and the solution was to change this to
    Arc<
        GraphicsPipeline<BufferlessDefinition, Box<dyn PipelineLayoutAbstract + Send + Sync>, R>,
    >

where R is the renderpass type.
The draw command looks like:
        cbb.draw(
            self.pipeline.clone(),
            &dynamic_state,
            BufferlessVertices {
                vertices: 3,
                instances: 1,
            },
            desc,
            push_consts,
        )

Closing this as most of the APIs involved here no longer exist.


How to get logit during prediction

Reminder

[X] I have read the README and searched the existing issues.

System Info
If I use do_predict, how to  get legit?
Reproduction
do_predict
Expected behavior
No response
Others
No response

Duplicate of https://github.com/hiyouga/LLaMA-Factory/issues/4131


clang-tidy: fix existing errors on master branch

Description: Fix existing violations of performance-inefficient-vector-operation and readability-redundant-control-flow.
Risk Level: Low
Testing: existing
Docs Changes: N/A
Release Notes: N/A
Signed-off-by: Derek Argueta<EMAIL_ADDRESS>

@derekargueta  Thank you for the fix!


fix: preallocate slice

This PR fixes go slice preallocation, and adds prealloc lint rule.

always good to see minor performance improvements

@mnagaa thank you for your contribution!


"Invalid descriptor (capacitor-community/sqlite)" when installing<EMAIL_ADDRESS>with yarn

Hi,
I'm just trying out electric-sql and it seems there's an inconsistency between the peerDependencies and peerDependenciesMeta:
  "peerDependencies": {
    "@capacitor-community/sqlite": ">= 5.4.1",
   [...]
  },
  "peerDependenciesMeta": {
    "capacitor-community/sqlite": { "optional": true },
    [...]
  },

peerDependencies references @capacitor-community/sqlite, while peerDependenciesMeta references an invalid capacitor-community/sqlite. I think this is why installing electric-sql fails with yarn:
$ yarn add<EMAIL_ADDRESS>➤ · Yarn 4.0.1
➤ ┌ Resolution step
➤ │ Error: electric-sql@npm:0.7.0: Invalid descriptor (capacitor-community/sqlite)
    at nh ($HOME/.cache/node/corepack/yarn/4.0.1/yarn.js:140:109587)
    at cE.load ($HOME/.cache/node/corepack/yarn/4.0.1/yarn.js:140:125463)
    at hv.resolve ($HOME/.cache/node/corepack/yarn/4.0.1/yarn.js:678:5987)
    at async Bd.resolve ($HOME/.cache/node/corepack/yarn/4.0.1/yarn.js:141:1451)
    at async Bd.resolve ($HOME/.cache/node/corepack/yarn/4.0.1/yarn.js:141:1451)
    at async $HOME/.cache/node/corepack/yarn/4.0.1/yarn.js:210:7174
    at async Yy ($HOME/.cache/node/corepack/yarn/4.0.1/yarn.js:140:53916)
    at async ne ($HOME/.cache/node/corepack/yarn/4.0.1/yarn.js:210:7156)
    at async Promise.allSettled (index 10)
    at async Uc ($HOME/.cache/node/corepack/yarn/4.0.1/yarn.js:140:53250)
➤ └ Completed
➤ · Failed with errors in 0s 97ms

Installing<EMAIL_ADDRESS>works fine.

Same issue here

Thanks for the report, I've merged a fix and it will be out in the next release.

Awesome!  Could I ask approximately when you think the next release will come out, so I know if we should wait or work around it in the meantime?

We could do a quick patch release, right?

No stress - it works with npm so we have an easy workaround

Ah, so it turns out that it installs with npm, but when I actually try to use @capacitor-community/sqlite the error surfaces with npm too.  Would it be possible to push out a patch release?

Hey @ccapndave. It is likely that we'll publish a patch release this week. Stay tuned.


[curl] Fix tools

What does your PR fix? Fixes #12754

vcpkg_copy_tool_dependencies cannot copy dll files to the destinations, so update it as vcpkg_copy_tools().
Note: feature tools has passed with the following triplets:

x86-windows
x64-windows

Other features are no need to test.

Also, one less patch to maintain, thanks to a proper usage of vcpkg built-ins!

/azp run

Related: #12276.


Dataset transformer improvements

Thoughts on Data Transformer Performance
The Problem:
The data transformer currently faces performance issues because every time we reopen it, all code cells are re-executed to restore the previous state. As the number of code cells in a notebook increases, this issue compounds, leading to longer loading times. Additionally, re-running all cells may not always replicate the exact state from when the notebook was last closed. For instance, if a random function was used to generate a column in a DataFrame, reopening the notebook would yield different results each time. Although we save the notebook's history, the kernel is shut down when we leave the node, necessitating the rerun of the entire history upon reopening. Checkpointing is not a viable solution, as the state is lost when the kernel session is terminated.
A Solution:
To address this, we can save the notebook history (as we already do) and, upon reopening the node, display the history without re-executing it. The Beaker kernel would be initialized with the input dataset(s) and their respective output dataset(s). For example:

A dataset (dataset1) is attached to the data transformer.
The transformer is opened, and the Beaker kernel is initialized with dataset1.
The agent modifies dataset1 (e.g., adds a column), resulting in a new dataset (dataset2).
Dataset2 is saved, the node is closed, and the session is terminated.
Upon reopening the node, the history is displayed but not re-executed.
A new session is initialized with dataset1 and dataset2 already defined.

This approach preserves the notebook's state without the need to rerun all cells, enhancing performance and ensuring consistency.

@YohannParis @mwdchang @mattprintz  Any thoughts above this above would be great 😄


[question] How to run eval_retry with task parameters

How would I pass task parameters when running failed run?
Example of full run:
logs = eval(
        tasks=task(grader_model=grader_model, dataset=dataset_obj, **kwargs),
        model=model,
        log_dir=output_folder,
        limit=samples_limit,
        max_connections=max_connections,
    )

When using eval_rerun it fails with error:
Code:
logs = eval_retry(
            tasks=["logs/2024-07-29T09-43-01-05-00_eval-tool-eval-multi-criteria-grader-list_3kqa33ubZsvviYHt6WQKxT.json"],
            log_dir=output_folder,
            max_connections=max_connections,
        )

Error:
TypeError: eval_multi_criteria_grader_list() missing 2 required positional arguments: 'grader_model' and 'dataset'

eval-retry needs to be able to fully re-execute the construction of the task (it can't serialise arbitrary tasks for re-execution as they could have all kinds of Python sauce that can't be saved).
If you do it this way then it should be able to reconstruct the task:
@task
def my_task(grader_model, dataset, **kwargs):
    return Task()

logs = eval(
        tasks=my_task,
        task_args=dict(grader_model=grader_model, dataset=dataset_obj) | kwargs,
        model=model,
        log_dir=output_folder,
        limit=samples_limit,
        max_connections=max_connections,
    )


Merge Request 3-7-2021

Reminder: Make sure to review CONTRIBUTING.MD and make sure you've sent in your signed contributor agreement.
Reminder: Add issue number for work item (e.g. #123)

Done


[TC-GC-2.3] need YAML support to send commands in  between commissioning process

Summary:
Need YAML support for Step5 :How do we send command in  between commissioning process
Testplan screenshot:

This issue was created by me using a account that i had to delete due to access issues, I will continue to follow these issues with my current id.

@manjunath-grl this should be possible if you do not do the WaitForCommissionee command. You will have to run each of the commands to pair.

@krypton36 @manjunath-grl Can you guys sync up on the next steps here please ? Creating a list for the items that we need  from the SDK team post 1.0, so want to have clear requirements for each task

@manjunath-grl , were you able to sync up ?


help with one click trainer

Hi, I was trying to create a LORA but I keep getting this error
2023-03-20 04:40:56.630426: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2023-03-20 04:40:57.674789: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/lib64-nvidia
2023-03-20 04:40:57.675017: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/lib64-nvidia
2023-03-20 04:40:57.675048: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
2023-03-20 04:41:00.803315: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2023-03-20 04:41:01.503107: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/lib64-nvidia
2023-03-20 04:41:01.503212: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/lib64-nvidia
2023-03-20 04:41:01.503248: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
prepare tokenizer
update token length: 225
Train with captions.
loading existing metadata: /content/drive/MyDrive/training_dir/meta_lat.json
Traceback (most recent call last):
File "/content/kohya-trainer/train_network.py", line 663, in
train(args)
File "/content/kohya-trainer/train_network.py", line 94, in train
train_dataset_group = config_util.generate_dataset_group_by_blueprint(blueprint.dataset_group)
File "/content/kohya-trainer/library/config_util.py", line 368, in generate_dataset_group_by_blueprint
dataset = dataset_klass(subsets=subsets, **asdict(dataset_blueprint.params))
File "/content/kohya-trainer/library/train_util.py", line 911, in init
assert len(abs_path) >= 1, f"no image / 画像がありません: {image_key}"
AssertionError: no image / 画像がありません: 10
Traceback (most recent call last):
File "/usr/local/bin/accelerate", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.9/dist-packages/accelerate/commands/accelerate_cli.py", line 45, in main
args.func(args)
File "/usr/local/lib/python3.9/dist-packages/accelerate/commands/launch.py", line 1104, in launch_command
simple_launcher(args)
File "/usr/local/lib/python3.9/dist-packages/accelerate/commands/launch.py", line 567, in simple_launcher
raise subprocess.CalledProcessError(returncode=process.returncode, cmd=cmd)
subprocess.CalledProcessError: Command '['/usr/bin/python3', '/content/kohya-trainer/train_network.py', '--output_name=adrcl', '--pretrained_model_name_or_path=/content/pretrained_model/anything-v3-fp32-pruned.safetensors', '--vae=/content/vae/anime.vae.pt', '--train_data_dir=/content/drive/MyDrive/ADR/5_adrcll', '--in_json=/content/drive/MyDrive/training_dir/meta_lat.json', '--output_dir=/content/drive/MyDrive/training_dir/output', '--network_dim=128', '--network_alpha=128', '--network_module=networks.lora', '--unet_lr=0.0001', '--text_encoder_lr=5e-05', '--optimizer_type=AdamW8bit', '--learning_rate=2e-06', '--lr_scheduler=constant', '--lr_warmup_steps=250', '--dataset_repeats=10', '--resolution=512', '--keep_tokens=1', '--lowram', '--mixed_precision=fp16', '--save_precision=fp16', '--save_n_epoch_ratio=3', '--save_model_as=safetensors', '--train_batch_size=4', '--max_token_length=225', '--max_train_epochs=20', '--clip_skip=2', '--logging_dir=/content/training_dir/logs', '--log_prefix=adrcl', '--shuffle_caption', '--xformers']' returned non-zero exit status 1.
how do I solve it?... I'm not a programmer so, please I'd apreciate if you could explin like if I were a 5 yo.

I'm sorry, I don't have any plans to continue the Fast Kohya Trainer project (for now).

The code is getting longer in one cell, and my device is not powerful enough to handle it, (I run most of things in small laptop), which is causing lag.

It's really hard to maintain the 1-click cell colab.

There is another good 1-click cell colab project maintained by HollowStrawBerry, which is integrated with Google Drive. It based on my notebook (?) so you may get the same experience, or even better.You can check it out here: https://github.com/hollowstrawberry/kohya-colab


type declaration for errors are wrong

The errors are exposed in the declaration file at top level.
https://github.com/QuorumDMS/ftp-srv/blob/e8efe321266da074e65ce9d10ffdb52c1733a517/ftp-srv.d.ts#L44
But they are actually exported under the ftpErrors.
https://github.com/QuorumDMS/ftp-srv/blob/e8efe321266da074e65ce9d10ffdb52c1733a517/ftp-srv.js#L8
Should wrap the errors in the type definition inside a namespace to match the code.

Currently this can be used as a workaround:
import { ftpErrors } from 'ftp-srv';
new ftpErrors.GeneralError('Invalid username or password', 401)

Be sure to surpress type and lint errors.


tmp: break firebase messenger

[!CAUTION]
Do not merge this.

Closed as it was just for a test.


Project won't build: missing directory. Help

Project will not build anymore after installing react-native-maps. Here's the error I got when trying to run my .xcworkspace in:
ld: warning: directory not found for option '-L/Users/name/Library/Developer/Xcode/DerivedData/project-cogbqkfqcwttvxaipnswqlsfewpq/Build/Products/Debug-iphonesimulator/React'
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Environment
OS: macOS High Sierra 10.13.3
Node: 8.9.1
Yarn: 1.3.2
npm: 5.7.1
Watchman: 4.9.0
Xcode: Xcode 9.2 Build version 9C40b
Android Studio: Not Found
Packages: (wanted => installed)
react: 16.2.0 => 16.2.0
react-native: 0.53.0 => 0.53.0
Expected Behavior
expected project to compile
Actual Behavior
Getting error above. Any help would be much appreciated

It looks like you need to clean your XCode project & remove the folders inside DerivedData

I'm still having this issue. I've cleaned my project.

Same here


Project pages: updates for geosmart

This PR simplifies the structure of our Jupyter Book project pages. It is intended to support specific ways we plan to run GeoSMART, but also incorporates changes that we've been introducing to project structures in the past year. Some of these changes will eventually need to be added back into our hackweek website template.

@spestana can you assist in linking this project roadmap to your new project template? I also need help updating the roadmap in general.


Theme system

Is it possible to get some explanation on the theme system? And how to change it etc?

Hey, thank you for your interest in the project. Are you able to clone and run the project locally? During the install process you are able to pass in your own theme object that contains the properties needed to style it your own way.

Hey, thank you for your interest in the project. Are you able to clone and run the project locally? During the install process you are able to pass in your own theme object that contains the properties needed to style it your own way.

yes, I understand, but If I'm using for example just the Table component it starts complaining about not having the theme, i basically have to pass the theme-override prop every time i call a component

Do you have an example of how you are including it?

in nuxt for example, I pull a component like this:
import { Button } from '@advanced-data-machines/tailwindcss-vue';
...
components: {
    Button,
},
...

The best way to handle that is just install the specific component and override the theme.
import { Button } from '@advanced-data-machines/tailwindcss-vue';
const custom = {
    theme: {
        TvButton: {
            ...
        }
    }
};
Vue.use(Button, custom);

Can also install just an array of components.
import Twv, { Button } from '@advanced-data-machines/tailwindcss-vue';

const components = [Button];
const custom = {
    theme: {
        TvButton: {
            ...
        }
    },
    components
};
Vue.use(Twv, custom);

This way you handle the config once.
Let me know if you have other issues or errors.


Mass Actor spawning not working in other projects

Hi, I am currently exploring the MassSample project and I have to say great job getting everything set up as it is.
I am currently struggling a bit to get Actors to spawn in the MassSpawner.
In the Sample project provided, I can spawn Actors in the MassSpawner as expected.
However, I tried to do the same in my own project but nothing happens when I start the level.
I created a MassSpawner and added the EnityConfig with an Actor added and Env Query but when I start the game no Actors are spawned. I am also not getting any error messages.
It works with Entities though. Did I miss something that has to be enabled/ added for the MassSpawner to work with Actors too? (There is not much documentation so I can't really look stuff up)
I am quite new to the Mass system and apologize if I overlooked something obvious.
I would appreciate all the help.

It's possible the actor is missing a mass agent component... That's my first guess but I haven't really done a lot of agent stuff yet.


Update dhall-generic to 0.3.74

Updates us.oyanglul:dhall-generic from 0.3.67 to 0.3.74.
GitHub Release Notes
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "us.oyanglul", artifactId = "dhall-generic" } ]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Codecov Report

Merging #181 (82d0d95) into master (1e0bc28) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #181   +/-   ##
=======================================
  Coverage   98.27%   98.27%
=======================================
  Files           1        1
  Lines          58       58
  Branches        2        2
=======================================
  Hits           57       57
  Misses          1        1

Continue to review full report at Codecov.

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

Superseded by #184.


Do not supply --crate-version flag to rustdoc if present in RUSTDOCFLAGS

Checks for the flag in extra compiler arguments as well

r? @ehuss
(rust_highfive has picked a reviewer for you, use r? to override)

Applied the suggestions as separate commits for easier reviewing, will squash after approval

@bors r+

:pushpin: Commit c9e3141b0fbf1bf49504947c7037bde2d2b0bf2a has been approved by ehuss

:hourglass: Testing commit c9e3141b0fbf1bf49504947c7037bde2d2b0bf2a with merge 526aaccfdb211a2be2895a9f53ad810159c7e73f...

:sunny: Test successful - checks-azure
Approved by: ehuss
Pushing 526aaccfdb211a2be2895a9f53ad810159c7e73f to master...


Deeper matches should win

@mcollina The following test is failing, with the smaller patter being matched:
test('deeper matches should win - #', function (t) {
  t.plan(1)

  var instance = bloomrun()

  instance.add({ role: 'auth', cmd: 'login' }, 42)
  instance.add({ role: 'auth', cmd: 'login', retry: 'true' }, 24)

  t.equal(instance.lookup({role: 'auth', cmd: 'login', retry: 'true'}), 24)
})

If you reverse the patterns the test passes. This is because the code in deep match is only validating the params on the pattern. If a smaller pattern fully matches a larger pattern and it was inserted first it wins because our check if for full match on pattern. See https://github.com/mcollina/bloomrun/blob/master/lib/deepMatch.js#L16-L18
Any suggestions? The only way I can see right now is to sort the matching buckets.data by length, that way we are always looking at the larger matches first. This of course would require a major bump because we are going against insertion order.
Any other suggestions?

I'd say put an option on the constructor, and make it non default to havr
the resorting on add.
What do you think?
Il giorno mer 18 nov 2015 alle 11:29 Dean McDonnell <
<EMAIL_ADDRESS>ha scritto:

@mcollina https://github.com/mcollina The following test is failing,
with the smaller patter being matched:
test('deeper matches should win - #', function (t) {
t.plan(1)
var instance = bloomrun()
instance.add({ role: 'auth', cmd: 'login' }, 42)
instance.add({ role: 'auth', cmd: 'login', retry: 'true' }, 24)
t.equal(instance.lookup({role: 'auth', cmd: 'login', retry: 'true'}), 24)
})
If you reverse the patterns the test passes. This is because the code in
deep match is only validating the params on the pattern. If a smaller
pattern fully matches a larger pattern and it was inserted first it wins
because our check if for full match on pattern. See
https://github.com/mcollina/bloomrun/blob/master/lib/deepMatch.js#L16-L18
Any suggestions? The only way I can see right now is to sort the matching
buckets.data by length, that way we are always looking at the larger
matches first. This of course would require a major bump because we are
going against insertion order.
Any other suggestions?
—
Reply to this email directly or view it on GitHub
https://github.com/mcollina/bloomrun/issues/23.

Yup, I'm happy with that. I'll get it done tonight.


Notification bar has no title

When the notification is received, the notification bar shows app's icon but no title, when I slide down, then the app icon & title & message show. Tested other phones having earlier Android versions with the same app setup using the same notification process and the title shows.
phonegap-plugin-push 1.8.0
android-minSdkVersion 14
PhoneGap Cloud Build (iOS / Android / Windows) cli-6.3.0 (4.2.0 / 5.2.1 / 4.4.1)
Windows PhoneGap Desktop (PhoneGap.exe) 0.1.9
Android PhoneGap Developer 1.7.6
Android 6.0.1 – Marshmallow
Phone Motorola DROID MAXX 2

@NateShoy I confess that I don't actually understand what the problem is. Can you take a screen shot of what you expect to see from another phone and then compare it to the one that has a problem?

Sure, here are both images: working and not working. TY

@NateShoy okay, I see what you are talking about now. As of Android L the title text is no longer shown. See https://developer.android.com/reference/android/app/Notification.html#tickerText


Save Post/Bookmarks does not work

Expected behavior
Bookmarks page (Saved Posts)
Actual behavior
Nothing
How to reproduce
/bookmarks  or try saving a post

Browser:  Chrome
Operating system: MacOS High Siera

Because this is not a crucial feature we may just want to remove this for now.


[Snyk] Upgrade async-mutex from 0.3.0 to 0.3.1

Snyk has created this PR to upgrade async-mutex from 0.3.0 to 0.3.1.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2021-02-22.

Release notes

Package name: async-mutex

0.3.1 - 2021-02-22Changelog, bump version.

0.3.0 - 2021-02-05Bump version.

    from async-mutex GitHub release notes

Commit messages

Package name: async-mutex

93f1ff7 Changelog, bump version.
3aad935 Bump version.
eed47b0 Merge pull request #34 from cantoine/clear-timeout-fix
38e59ba Add clearTimeout and tests

Compare

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

Coverage decreased (-0.02%) to 72.218% when pulling ab51102b0b3a077633d1489c9ea49e1f01a99c66 on snyk-upgrade-eff5bbc60d337a7664acf417c182f5cd into d33daa1669925c42c789cf4b786fa9c4453337cb on develop.


Re: Verification of lema - definition compatibility

Please verify the definitions of the following lemmae:
urn:cite:perseus:latlexent.lex6628.1 Phlegra "the volcano of Vesuvius"
<proposal of definition: "name of a country in Macedonia"
urn:cite:perseus:latlexent.lex6629.1 Phlegraeus "a country of Macedonia"
<proposal of definition: "adjective of belonging to or of Phlegra"

Not relevant for our project because of #19 .


e2e logging: Hard to follow workload events

The current logs make following single workload events hard:
Example logs
2024-10-22T16:15:37.201+0530	INFO	dractions/actions.go:115	enter Failover appset-deploy-cephfs-busybox
2024-10-22T16:15:37.273+0530	INFO	dractions/retry.go:67	drpc appset-deploy-cephfs-busybox is ready
2024-10-22T16:15:37.432+0530	INFO	dractions/actions.go:187	update drpc appset-deploy-cephfs-busybox failover to rdr-dr1
2024-10-22T16:15:42.462+0530	INFO	dractions/retry.go:124	drpc appset-deploy-rbd-busybox phase is FailedOver
2024-10-22T16:15:47.312+0530	INFO	dractions/retry.go:124	drpc subscr-deploy-rbd-busybox phase is FailedOver
2024-10-22T16:16:12.663+0530	INFO	dractions/retry.go:124	drpc appset-deploy-cephfs-busybox phase is FailedOver
2024-10-22T16:18:12.705+0530	INFO	dractions/retry.go:67	drpc appset-deploy-rbd-busybox is ready

We need this format:
timestamp level test-id message

Example logs converted to wanted format:
2024-10-22T16:15:37.201+0530 INFO  [appset-cephfs] enter Failover
2024-10-22T16:15:37.273+0530 INFO  [appset-cephfs] drpc is ready
2024-10-22T16:15:37.432+0530 INFO  [appset-cephfs] starting failover to rdr-dr1
2024-10-22T16:15:42.462+0530 INFO  [appset-rbd] drpc phase is FailedOver
2024-10-22T16:15:47.312+0530 INFO  [subscr-rbd] drpc phase is FailedOver
2024-10-22T16:16:12.663+0530 INFO  [appset-cephfs] drpc  phase is FailedOver
2024-10-22T16:18:12.705+0530 INFO  [appset-rbd] drpc is ready

Notes:

Short workload names without repeated parts (deploy, busybox)
aligned with spaces, not tabs for more compact logs

@raghavendra-talur what do you think?

Having shorter names requires more work. The basic changes were completed in #1628


Removing talents (with bonus to attributes) doesn't remove the bonus

Hello guys,
Just to point out that when you add, for instance, the Talent "savvy" to a character, it applies the +5 bonus to intelligence, but when you remove it, the bonus remains.
As bonuses given by talents are not tracked as "specific" in the character sheet, this could quickly lead to wrong character sheets with 5 free augmentations each time.
Is this mechanic intended ?
Good job anyway. ;)

The bonus should be removed so this is a bug, thanks.

Seems to work with the last update...
Thank you !


Evaluate the metrics question

Thank you for your work. I used the 26.3M pretrained weights you provided, but the driving score I obtained is somewhat low. I'm having some problems evaluating metrics.Could you please confirm if the weights you provided were trained after using PlanKD? Is there anything I should pay attention to during the evaluation process?

Thank you for your question. I can confirm that the weights are accurate. The evaluation results on Town05 Long and Town05 Short in the CARLA simulator often exhibit significant variance. Therefore, if your score is somewhat lower or higher (e.g., ±10), this is normal. You can try to run multiple times for evaluations to observe a more consistent average score.


Update README.md

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)
How do we know it's really free?
For book lists, is it a book?
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

thanks, and welcome to free-programming-books!


Germanium OKR: Content review: Delete Video topics for MVC & Web Forms on AspNetDocs repo

The following applies to the dotnet/AspNetDocs repo, not dotnet/AspNetCore.Docs, but we are tracking here:
Part of the FY24 OKR related effort to archive or remove content that is no longer useful.
Content deletion Candidates for repo dotnet/AspNetDoc:
162 video topics under MVC and Web-Forms: Nearly all have broken video links and videos are no longer available. Traffic to these is almost non-existent.
All 72 Video topics including and under https://learn.microsoft.com/en-us/aspnet/mvc/videos/
All 90 topics including and under https://learn.microsoft.com/en-us/aspnet/web-forms/videos/

Holding off on reQuest label until I hear what labels we are to use for these and where/how we want them tracked in DevOps.


Fixing README.md to update region parameter in country constructor to a working example

Before the README.md region settings has been mixed with two different countries.
$holidays = Holidays::for(Austria::make(region: 'de-bw'))->get();

Austria has called a German region. I´ve adjusted it to the German working example and uppercased the region parameter as it is in the German class.

Thanks!


minor update to allow stronger assertion about sidecar to waypoint

Please provide a description of this PR:
Removes a weakened assertion for ambient sidecar to waypoint test

@ilrudie: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
99944af84c62fe13b4ba7e1e504d3a00e2f16fd0
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


It's not working, because without wrap by

The current code without  and callback and error handling.
so the repo is not working.

Have you tried the demo application (https://next-auth-example-sign-in-page.vercel.app/auth/signin)? OAuth (Github) login works for me.
Are you trying to use a different type of provider?

it works.
I mean when accessing an unauthorized page, it redirects to Singin.tsx with the callback.
I think the repo is not for that?

Correct, the repo is simply for showing the custom sign-in page as an example.
Going to close this then :+1:


javascript: uses unsafe-eval

Howdy,
Something recently came up with lightstep's javascript tracer implementation that I wanted to relay here. The protobuf library for browser javascript uses eval, creating a csp violation. There are currently two open tickets with no response: https://github.com/protocolbuffers/protobuf/issues/5464 and https://github.com/protocolbuffers/protobuf/issues/6770. There's an alternative protobuf implementation called protobuf.js but it also has the same issue: https://github.com/protobufjs/protobuf.js/issues/593
Not really sure what there is to do with this, but it is a potentially massive gotcha depending on how strict the end user's security policies are.

Based on description it looks like a JS library issue, not a schema problem.  So I've transferred this issue

For Node, the content security policy I don't think would apply, so this is really only relevant to browser-specific code.
But I don't think we should be using protos directly in any code that gets shipped to the browser. What the OTel Collector exporter should do instead is to just essentially vendor the TS types for the JSON that will get sent to the collector (that corresponds via grpc-gateway to the protos). See this types file in OpenCensus Web.
cc/ @obecny
Does this seem right? Or am I missing something here?

From what I see now the opentelemetry-js' is not using any package in **browser** that depends on google-protobuf. The only package that uses it is opentelemetry-plugin-grpc` so far. But this package is the node.js package only. I'm not sure about electron apps - maybe this is something that should be checked separately although I think it should not cause any issues.

There's really no reason for OpenTelemetry to use any protos unless protos are already being used.


[Bug]: Chart does not render*

Would you like to work on a fix?

[x] Check this if you would like to implement a PR, we are more than happy to help you go through the process.

Current and expected behavior
*I understand that you said "if you are experiencing an issue with charts rendering, please create a related issue in Chart.js repository." However, this code is a direct port from your scatter example but it only works in code-sandbox and not in a local sveltekit repo.
I would expect svelte-chartjs to work in a sveltekit app but I have been unable to discover a working example that uses sveltekit nor have I been able to get mine to work with sveltekit.
Reproduction
https://github.com/taylorjdawson/svelte-chartjs-example
chart.js version
4.3.0
svelte-chartjs version
3.1.2
Possible solution
Sveltekit uses ssr so that could potentially be causing issues. I am unsure if this library accounts for ssr?

Confirming issue

For SSR you could check out:
https://github.com/Rich-Harris/pancake
or: https://layercake.graphics/

Did anyone find a fix for this? Been unable to even render a line chart within svelte.

Running into the same issue with SvelteKit and svelte-chartjs

same here with sveltekit :/

Is this still a problem for anyone? I'm able to use all the charts in my SvelteKit project.

Is this still a problem for anyone? I'm able to use all the charts in my SvelteKit project.

Yes :( , even with SSR turned off, charts don't render. Any solution?

I can't make this work either, it just doesn't work, I can't even get the examples working

Similarly, I can't get even the examples working by directly copy and pasting.

work fine on sveltekit without this library, I using it.

Could you share some working code example?


⚠ Jenkins Plugins Artifactory has degraded performance

In 47d8455, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1233 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 71eac08 after 34 minutes.


Can't handle empty pgbouncer logs in incremental mode

Caveat: I'm a bit of a pgbadger n00b. Please be gentle with me if I've missed something obvious.
I have a test server (which collects logs via syslog) which sometimes has a day go by with no connections. Thus, the only thing in there are stats and one SIGHUP, eg:
May 19 05:11:01 hostname pgbouncer[7360]: stats: 0 xacts/s, 0 queries/s, in 0 B/s, out 0 B/s, xact 0 us, query 0 us, wait 0 us
May 19 05:11:01 hostname pgbouncer[7360]: got SIGHUP, re-reading config
May 19 05:12:01 hostname pgbouncer[7360]: stats: 0 xacts/s, 0 queries/s, in 0 B/s, out 0 B/s, xact 0 us, query 0 us, wait 0 us

If I use a standard incremental command line:
/srv/pgbadger/pgbadger-11.8/pgbadger --incremental emptylogs/pgbouncer.log -O $PWD/emptyoutput/
I get:
FATAL: unable to detect log file format from emptylogs/pgbouncer.log, please use -f option.
    - Error at line 17888

The problem is that I want to run pgbadger on both postgres and pgbouncer logs, so I can't use -f pgbouncer.
Things I've tried:

Running pgbadger twice, on the postgres / pgbouncer logs separately (with -f)
Using -f syslog - that seems to assume it's a postgres syslog file
Number 1 seems to work... sometimes. If I run it on the pgbouncer log first, it does what I'd expect (this is on logs that do have queries in, btw) (-q for brevity):

$ /srv/pgbadger/pgbadger-11.8/pgbadger -q --incremental fulllogs/postgres.log -O $PWD/fulloutput/
$ cat fulloutput/LAST_PARSED
2022-05-19 09:06:26     777423  May 19 09:06:26 hostname postgres[redacted]
pgbouncer       2022-05-19 09:06:26     0       May 19 09:06:26 hostname pgbouncer[redacted]
$

But if I run it on postgres first, it seems to ignore the pgbouncer logs (there's no output even without -q):
$ /srv/pgbadger/pgbadger-11.8/pgbadger -q --incremental fulllogs/pgbouncer.log -O $PWD/fulloutput/
$ cat fulloutput/LAST_PARSED
2022-05-19 09:06:26     777423  May 19 09:06:26 hostname postgres[redacted]
$

The only sure-fire way I can see at the moment to process all the logs is to have two separate output directories for pgbouncer and postgres. This would give me all the info I need, but isn't quite as convenient.
Please let me know if I've missed something glaringly obvious!

Syntax: `logfilename:format'  can be used when there is multiple log format to process and they are not autodetected.
In your can use the following syntax:
/srv/pgbadger/pgbadger-11.8/pgbadger -q --incremental -O $PWD/fulloutput/   fulllogs/postgres.log emptylogs/pgbouncer.log:pgbouncer

PERFECT! I knew you would have thought of everything :smile: I'll leave this open in case you want to investigate anything else in it, but feel free to close it from my point of view.
Thank you!

Damn. It worked fine in my test setup, but in the actual setup I'm getting the files from a remote server:

If I add :pgbouncer on the end, I get:
Unmatched '.
FATAL: can't get size of remote file, please check what's going wrong with command:  "ls -l '/srv/logdir/pgbouncer.log
'"
    - Error at line 18406

It does the same even if I remove the wildcards:
Unmatched '.
Unmatched '.
FATAL: can't get size of remote file, please check what's going wrong with command:  "ls -l '/srv/logdir/pgbouncer.log
'"
    - Error at line 18406

Seems there's a newline in there, somehow. This could well be a tcsh problem (this user, sadly, uses tcsh and I can't change that). I'm just guessing from the "Unmatched" lines. If it is, I fully understand you walking away from it: tcsh is, after all, not used many places anymore.

I think I've found the problem (forgive me, perl isn't my strongest suit). You chomp the @given_log_files, but if it has a :pgbouncer on the end, the newline is in the middle. I'll see if I can get a fix together and put in a pull request later.

Right got it, commit b27e593 might solve this issue.


"bloom-ds-zero-inference.py" works but "inference_server.cli --deployment_framework ds_zero" fails

deepspeed --num_gpus 4 bloom-inference-scripts/bloom-ds-zero-inference.py --name /raid/data/richardwang/bloomz --cpu_offload worked and gave me inference output. /raid/data/richardwang/bloomz is a downloaded copy of bigscience/bloomz
However python -m inference_server.cli --model_name /raid/data/richardwang/bloomz --model_class AutoModelForCausalLM --dtype bf16 --deployment_framework ds_zero --generate_kwargs '{"min_length": 100, "max_new_tokens": 100, "do_sample": false}' failed with error message:
Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/richardwang/transformers-bloom-inference/inference_server/cli.py", line 43, in <module>
    main()
  File "/home/richardwang/transformers-bloom-inference/inference_server/cli.py", line 18, in main
    model = ModelDeployment(args, True)
  File "/home/richardwang/transformers-bloom-inference/inference_server/model_handler/deployment.py", line 54, in __init__
    self.model = get_model_class(args.deployment_framework)(args)
  File "/home/richardwang/transformers-bloom-inference/inference_server/models/ds_zero.py", line 51, in __init__
    self.model = get_hf_model_class(args.model_class).from_pretrained(args.model_name, torch_dtype=args.dtype)
  File "/home/richardwang/venv/llamaenv/lib/python3.8/site-packages/transformers/models/auto/auto_factory.py", line 464, in from_pretrained
    return model_class.from_pretrained(
  File "/home/richardwang/venv/llamaenv/lib/python3.8/site-packages/transformers/modeling_utils.py", line 2357, in from_pretrained
    init_contexts = [deepspeed.zero.Init(config_dict_or_path=deepspeed_config())] + init_contexts
  File "/home/richardwang/venv/llamaenv/lib/python3.8/site-packages/deepspeed/runtime/zero/partition_parameters.py", line 657, in __init__
    _ds_config = deepspeed.runtime.config.DeepSpeedConfig(
  File "/home/richardwang/venv/llamaenv/lib/python3.8/site-packages/deepspeed/runtime/config.py", line 808, in __init__
    self._configure_train_batch_size()
  File "/home/richardwang/venv/llamaenv/lib/python3.8/site-packages/deepspeed/runtime/config.py", line 991, in _configure_train_batch_size
    self._batch_assertion()
  File "/home/richardwang/venv/llamaenv/lib/python3.8/site-packages/deepspeed/runtime/config.py", line 926, in _batch_assertion
    assert (
AssertionError: Train batch size: 0 has to be greater than 0

Environment:
torch              1.12.0+cu113
deepspeed          0.8.2
deepspeed-mii      0.0.4
transformers       4.26.1

I have check that both bloom-inference-scripts/bloom-ds-zero-inference.py and inference_server/models/ds_zero.py call the same thing AutoModelForCausalLM.from_pretrained(model_name, torch_dtype=torch.bfloat16) How can I make it work to start up a server for bloomz with deepspeed inference?

this is weird. Ill look into this one

Hi @richarddwang,
I also want to load a local saved bloomz model for inference. However, when I tried to load my local bloomz checkpoint. it often gives me the following error:
Traceback (most recent call last):
  File "bloom-inference-scripts/bloom-ds-zero-inference.py", line 66, in <module>
    tokenizer = AutoTokenizer.from_pretrained(model_name)
  File "/usr/local/lib/python3.7/dist-packages/transformers/models/auto/tokenization_auto.py", line 642, in from_pretrained
    tokenizer_config = get_tokenizer_config(pretrained_model_name_or_path, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/transformers/models/auto/tokenization_auto.py", line 499, in get_tokenizer_config
    _commit_hash=commit_hash,
  File "/usr/local/lib/python3.7/dist-packages/transformers/utils/hub.py", line 420, in cached_file
    local_files_only=local_files_only,
  File "/usr/local/lib/python3.7/dist-packages/huggingface_hub/utils/_validators.py", line 112, in _inner_fn
    validate_repo_id(arg_value)
  File "/usr/local/lib/python3.7/dist-packages/huggingface_hub/utils/_validators.py", line 161, in validate_repo_id
    "Repo id must be in the form 'repo_name' or 'namespace/repo_name':"
huggingface_hub.utils._validators.HFValidationError: Repo id must be in the form 'repo_name' or 'namespace/repo_name': '/raid/data/richardwang/bloomz'. Use `repo_type` argument if needed.
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 2378) of binary: /usr/bin/python3

It seems like the code does not support loading local checkpoints. May I know what change did you make to allow it to load local checkpoints? I have tried neither bloom-inference-scripts/bloom-ds-inference.py nor bloom-inference-scripts/bloom-ds-zero-inference.py works
Thanks a lot:)

Hi @richarddwang,
I also want to load a local saved bloomz model for inference. However, when I tried to load my local bloomz checkpoint. it often gives me the following error:
Traceback (most recent call last):
  File "bloom-inference-scripts/bloom-ds-zero-inference.py", line 66, in <module>
    tokenizer = AutoTokenizer.from_pretrained(model_name)
  File "/usr/local/lib/python3.7/dist-packages/transformers/models/auto/tokenization_auto.py", line 642, in from_pretrained
    tokenizer_config = get_tokenizer_config(pretrained_model_name_or_path, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/transformers/models/auto/tokenization_auto.py", line 499, in get_tokenizer_config
    _commit_hash=commit_hash,
  File "/usr/local/lib/python3.7/dist-packages/transformers/utils/hub.py", line 420, in cached_file
    local_files_only=local_files_only,
  File "/usr/local/lib/python3.7/dist-packages/huggingface_hub/utils/_validators.py", line 112, in _inner_fn
    validate_repo_id(arg_value)
  File "/usr/local/lib/python3.7/dist-packages/huggingface_hub/utils/_validators.py", line 161, in validate_repo_id
    "Repo id must be in the form 'repo_name' or 'namespace/repo_name':"
huggingface_hub.utils._validators.HFValidationError: Repo id must be in the form 'repo_name' or 'namespace/repo_name': '/raid/data/richardwang/bloomz'. Use `repo_type` argument if needed.
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 2378) of binary: /usr/bin/python3

It seems like the code does not support loading local checkpoints. May I know what change did you make to allow it to load local checkpoints? I have tried neither bloom-inference-scripts/bloom-ds-inference.py nor bloom-inference-scripts/bloom-ds-zero-inference.py works
Thanks a lot:)

Hello, I successfully used the zero3 stage acceleration for deepspeed inference and used two gpus, but I found that using two gpus did not perform data parallelism or model parallelism. Each apus asked the same question. How should I solve this problem? Thank you for your reply.

this repo is no longer being maintained @sevenandseven
I suggest using vLLM or TGI


Fix crash when invalid input was provided to ragged_tensor_to_variant

This PR fixes #59084 by return error in case invalid shape set_dim is called.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Thanks @cantonios . The PR has been updated.
In the past I did notice that internal tests may have different error codes. Not sure the change will cover all scenarios in internal tests. Please let me know if there are additional failures in internal tests.

Here are the internal errors, @yongtang  can you please verify ? Thank you!
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/tensorflow/python/framework/test_util.py", line 1526, in decorated
run_eagerly(self, **kwargs)
File "/tensorflow/python/framework/test_util.py", line 1510, in run_eagerly
f(self, *args, **kwargs)
File "/tensorflow/python/ops/ragged/ragged_tensor_test.py", line 2019, in testToVariantInvalidInputs
self.assertRaisesRegex(
AssertionError: "must be less than 0, got 0|Shape must be at least rank 1 but is rank 0" does not match "{{function_node _wrapped__RaggedTensorToVariant_RAGGED_RANK_1_device/job:localhost/replica:0/task:0/device:CPU:0}} Requires the first element of nested_row_splits[0] to be 0 but is 150 [Op:RaggedTensorToVariant]"

There might be some skew in Graph mode (shape inference error) vs eager mode (kernel check) error messages. Maybe look at the shape inference for this particular op?

Hi @yongtang Any update on this PR? Please. Thank you!


Replace with geojson_utils.NewCoordinateFromLatLons with local code

Probably a custom Coord struct not that the universe needs one but it would be one less package to import.

Something something something reconcile with protobuf definition used by whosonfirst/go-whosonfirst-spatial-grpc:
https://github.com/whosonfirst/go-whosonfirst-spatial-grpc/blob/master/spatial/spatial.proto

This is now part of the go-whosonfirst-spatial/geo package.


Difficulty syncing replicator

What is the bug?
I am running the postgres replicator on the Railway Pro plan. It never completes syncing and eventually gets stuck like in the screenshot below:

How can it be reproduced? (optional)
Set up a hub in railway service that uses hub-monorepo with the following env variables with redis and postgres services
HUB_HOST=nemes.farcaster.xyz:2283
HUB_SSL=true
NIXPACKS_PKGS=... python3 gcc rustup
NIXPACKS_INSTALL_CMD=yarn install
NIXPACKS_BUILD_CMD=yarn build
NIXPACKS_START_CMD=cd apps/replicator && yarn start
POSTGRES_URL=${{Postgres.DATABASE_URL}}
REDIS_URL=${{Redis.REDIS_URL}}

Additional context (optional)
After initially getting stuck, restarting the replicator backfills from scratch again, which makes it take even longer. I think snapshot sync would help to at least get faster iteration and feedback loops on the replicator since re-syncing takes a very long time.

Update: managed to sync after removing many of the constraints

here is the linked issue mentioned in the cast above: https://github.com/farcasterxyz/hub-monorepo/issues/1556

Sorry for the delay in response. We don't recommend using the replicator any more due to multiple issues with it. We recommend using the Shuttle project if you'd like to sync. Thanks!


Simplified system specifications reporting

Coverage remained the same at 100.0% when pulling 5622b9e33686bdb42d09d470579d373dc6fa4226 on specification-utils into b05e764d62d6599eb9c5460df0fe820a30ca6940 on master.

Coverage remained the same at 100.0% when pulling 5622b9e33686bdb42d09d470579d373dc6fa4226 on specification-utils into b05e764d62d6599eb9c5460df0fe820a30ca6940 on master.

Coverage remained the same at 100.0% when pulling 5622b9e33686bdb42d09d470579d373dc6fa4226 on specification-utils into b05e764d62d6599eb9c5460df0fe820a30ca6940 on master.

Coverage remained the same at 100.0% when pulling 5622b9e33686bdb42d09d470579d373dc6fa4226 on specification-utils into b05e764d62d6599eb9c5460df0fe820a30ca6940 on master.

Codecov Report

Merging #143 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #143   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           9      9
  Lines         267    267
=====================================
  Hits          267    267

Coverage remained the same at 100.0% when pulling 0cfb354e0d9d8b554b73a8421b61e9587c93ea24 on specification-utils into e8487be67e5853387565ea9a4588f13a18d12d4f on master.

Coverage remained the same at 100.0% when pulling 3107b6a70a8b229f401b289a65586193061710ff on specification-utils into 9defd87a783928e057cf0456aff2097de8ebe4d0 on master.

Coverage remained the same at 100.0% when pulling 3107b6a70a8b229f401b289a65586193061710ff on specification-utils into 9defd87a783928e057cf0456aff2097de8ebe4d0 on master.

Coverage remained the same at 100.0% when pulling 3107b6a70a8b229f401b289a65586193061710ff on specification-utils into 9defd87a783928e057cf0456aff2097de8ebe4d0 on master.


App migration FAQ

This PR adds the instructions on how to migrate an app from one organization to another.

Hadn't thought about moving this to the public documentation, but I don't have any objections. I've asked for more reviews on this, so we can discuss.

Sure, one of the customer who contacted us recently about the migration said he could not find any information around it in the docs, so i thought if might be a good idea to add.


[SVSM] Failed to launch /init

I followed procedures in INSTALL.md and built linux host, guest kerns,  igvm(1.6), QEMU, but when i run lunch-qemy.sh from SVSM, i do see " [SVSM] Failed to launch /init" and that leads to kernel crash.
Any suggestions. Attached complete log.
Launch QEMU command
$SUDO_CMD
$QEMU
-enable-kvm
-cpu EPYC-v4
-machine $MACHINE
-object $MEMORY
-object sev-snp-guest,id=sev0,cbitpos=51,reduced-phys-bits=1,init-flags=5,igvm-file=$IGVM
-smp 4
-no-reboot
-netdev user,hostfwd=tcp::10022-:22,id=vmnic -device e1000,netdev=vmnic,romfile=
$IMAGE_DISK
-nographic
-monitor none
-serial stdio
$DEBUG_SERIAL
$QEMU_EXIT_DEVICE
$QEMU_TEST_IO_DEVICE
char device redirected to /dev/pts/5 (label serial1)
[Stage2] COCONUT Secure Virtual Machine Service Module (SVSM) Stage 2 Loader
[Stage2] Order-00: total pages:    15 free pages:     0
[Stage2] Order-01: total pages:     0 free pages:     0
[Stage2] Order-02: total pages:     1 free pages:     1
[Stage2] Order-03: total pages:     1 free pages:     1
[Stage2] Order-04: total pages:     0 free pages:     0
[Stage2] Order-05: total pages:     0 free pages:     0
[Stage2] Total memory: 108KiB free memory: 48KiB
[Stage2]   kernel_region_phys_start = 0x0000008000000000
[Stage2]   kernel_region_phys_end   = 0x0000008001000000
[Stage2]   kernel_virtual_base   = 0xffffff8000000000
[SVSM] COCONUT Secure Virtual Machine Service Module (SVSM)
[SVSM] Order-00: total pages:    48 free pages:     0
[SVSM] Order-01: total pages:     0 free pages:     0
[SVSM] Order-02: total pages:     1 free pages:     1
[SVSM] Order-03: total pages:     1 free pages:     1
[SVSM] Order-04: total pages:     1 free pages:     1
[SVSM] Order-05: total pages:   106 free pages:   106
[SVSM] Total memory: 13872KiB free memory: 13680KiB
[SVSM] Boot stack starts        @ 0xffffff800025a000
[SVSM] BSP Runtime stack starts @ 0xffffff000020c000
[SVSM] Guest Memory Regions:
[SVSM]<PHONE_NUMBER>00000000-000000000080000000
[SVSM]<PHONE_NUMBER>00000000-000000000280000000
[SVSM] Invalidating boot region<PHONE_NUMBER>00000000-0000000000000a0000
[SVSM] Invalidating boot region<PHONE_NUMBER>00100000-000000000000356a00
[SVSM] Invalidating boot region<PHONE_NUMBER>00357000-00000000000035a000
[SVSM] 4 CPU(s) present
[SVSM] Launching AP with APIC-ID 1
[SVSM] AP with APIC-ID 1 is online
[SVSM] Launching AP with APIC-ID 2
[SVSM] Launching request-processing task on CPU 1
[SVSM] AP with APIC-ID 2 is online
[SVSM] Launching AP with APIC-ID 3
[SVSM] Launching request-processing task on CPU 2
[SVSM] AP with APIC-ID 3 is online
[SVSM] Brought 3 AP(s) online
[SVSM] Launching request-processing task on CPU 3
[SVSM] FW Meta Data
[SVSM]   CPUID Page   : 0x0080e000
[SVSM]   Secrets Page : 0x0080d000
[SVSM]   CAA Page     : 0x0080f000
[SVSM]   Pre-Validated Region 0x0000000000800000-0x0000000000809000
[SVSM]   Pre-Validated Region 0x000000000080a000-0x000000000080d000
[SVSM]   Pre-Validated Region 0x0000000000810000-0x0000000000820000
[SVSM] Validating 0x0000000000800000-0x0000000000809000
[SVSM] Validating 0x000000000080a000-0x0000000000820000
[SVSM] Flash region 0 at 0x00000000ffc00000 size<PHONE_NUMBER>00400000
[SVSM] Size of OBJECT = 1204
[SVSM] Size of components in TPMT_SENSITIVE = 744
[SVSM]     TPMI_ALG_PUBLIC                 2
[SVSM]     TPM2B_AUTH                      50
[SVSM]     TPM2B_DIGEST                    50
[SVSM]     TPMU_SENSITIVE_COMPOSITE        642
[SVSM] MAX_CONTEXT_SIZE can be reduced to 1264 (1344)
[SVSM] VTPM: Microsoft TPM 2.0 initialized
[SVSM] [CPU 0] Virtual memory pages used: 0 * 4K, 0 * 2M
[SVSM] VMSA PA: 0x8000f66000
[SVSM] Launching Firmware
[SVSM] Launching request-processing task on CPU 0
[SVSM] Failed to launch /init  ----- This message lead to crash
SecCoreStartupWithStack(0xFFFCC000, 0x820000)
Register PPI Notify: DCD0BE23-9586-40F4-B643-06522CED4EDE
Install PPI: 8C8CE578-8A3D-4F1C-9935-896185C32DD3
...............................
SetUefiImageMemoryAttributes - 0x000000007F2B3000 - 0x0000000000004000 (0x0000000000000008)
SetUefiImageMemoryAttributes - 0x000000007F2AE000 - 0x0000000000005000 (0x0000000000000008)
KVM: unknown exit reason 24
EAX=00000000 EBX=00000000 ECX=0000020f EDX=00000000
ESI=00000000 EDI=00000000 EBP=00000000 ESP=00000000
EIP=00000000 EFL=00000002 [-------] CPL=0 II=0 A20=1 SMM=0 HLT=0
ES =0000 00000000 00000000 00000000
CS =0000 00000000 00000000 00000000
SS =0000 00000000 00000000 00000000
DS =0000 00000000 00000000 00000000
FS =0000 00000000 00000000 00000000
GS =0000 00000000 00000000 00000000
LDT=0000 00000000 00000000 00000000
TR =0000 00000000 00000000 00000000
GDT=<PHONE_NUMBER>000000 00000000
IDT=<PHONE_NUMBER>000000 00000000
CR0=80010033 CR2=0000000000000000 CR3=0000000000000000 CR4=00000668
DR0=0000000000000000 DR1=0000000000000000 DR2=0000000000000000 DR3=0000000000000000
DR6=0000000000000000 DR7=0000000000000000
EFER=0000000000000d00
Code= ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ??
qemu-system-x86_64: terminating on signal 2
SVSM latest commit information.
/coconut-svsm/svsm$ git log -1
commit c3da88d2ce90ea3ef74e9a8cf54df08252c86c4c (HEAD -> main, origin/main, origin/HEAD)
Merge: 2d55a4b 0a05dcc
Author: Jörg Rödel<EMAIL_ADDRESS>Date:   Sun Jun 16 09:34:12 2024 +0200
Merge pull request #383 from p4zuu/clippy

igvmbuilder: remove firmware field in GpaMap

qemu_06162024.log

It's my setup issue and [SVSM] Failed to launch /init --> this message is not reason for my guest VM bootup.

[SVSM] Launching Firmware
[SVSM] Launching request-processing task on CPU 0
[SVSM] Failed to launch /init

I want to know this message if this error message is critica?


Or 1843 merge internal audit to main

For comparison, I will close. Please ignore

rebased the main


Adapt hardening to AM62 (scarthgap)

See commit messages for details.
Note: the implementation was tested on an "open" device only.

@lucas-akira tagging you to keep you informed...

Think the table here: https://github.com/toradex/meta-toradex-security/blob/kirkstone-6.x.y/docs/README-secure-boot.md?plain=1#L68
Needs to also be updated to reflect the TI specific TDX_K3_HSSE_ENABLE option.

Moved to draft to fix an issue I didn't notice before.

Think the table here: https://github.com/toradex/meta-toradex-security/blob/kirkstone-6.x.y/docs/README-secure-boot.md?plain=1#L68
Needs to also be updated to reflect the TI specific TDX_K3_HSSE_ENABLE variable. To clarify the table should at last mention the interaction with the TI variable as well.

Good point. I'll do that.


Caching doesn't seem to apply to dashboards

Caching works for slices, but for dashboards it looks like a new query is triggered on every reload.
For long running queries, or for dashboards with many slices, this can become a problem.

It works on my side. If you refresh a dashboard you don't see a red refresh button?

No, I don't see a red refresh button. This is not a Druid connection, but a SQLAlchemy connection, maybe that is the difference?
The caching works fine for slices, but on the dashboards a new query is triggered each time.

After further testing, I think this issue is invalid. Not sure what was causing problems before.


cnf() reorders the variables

In this JUnit test case the cnf() method reorders the variables and the JUnit test fails:
	public void testLogicNGCNF() {
		final FormulaFactory f = new FormulaFactory();
		final Variable x = f.variable("x");
		final Variable y = f.variable("y");
		final Literal notX = f.literal("x", false);
		final Literal notY = f.literal("y", false);
		// x & ~y | ~x & y
		final Formula formula = f.or(f.and(x, notY), f.and(notX, y)).cnf();
		assertEquals("(x | y) & (~x | ~y)", formula.toString());
	}

See: Symja JUnit tests

Hi,
LogicNG doesn't want to make any guarantee about the order of operands. I think in this case the order is just due to the internal implementation of the cnf algorithm. However the order can also be different because the formula factory caches all formulas it creates. See this example:
final FormulaFactory f = new FormulaFactory();
final Formula or1 = f.or(f.variable("x"), f.variable("y"));
final Formula or2 = f.or(f.variable("y"), f.variable("x"));
System.out.println(or1); // x | y
System.out.println(or2); // x | y
System.out.println(or1 == or2); // true

The second formula or2 is the same object as or1 since LogicNG recognizes, that both are semantically equal.
Btw. LogicNG even applies some simplifications like absorption, removal of duplicate operands, and some more, when formulas when they are created. For example
System.out.println(f.or(f.variable("x"), f.variable("x"), f.falsum())); // x

Hope this helps a bit to understand the way LogicNG works. ;)

So if I have more than 2 variables and create a "big conjunctive normal form" the variables in every single clause could have different orderings?
And if I'm using Java Formula#equals() method for two arbitrary "mixed variable forms of cnf()" which are "logical identical" get true as result?

If I have more than 2 variables and create a "big conjunctive normal form" the variables in every single clause could have different orderings?
Yes.
You don't even need to create a CNF. The variable order inside each clause is not guaranteed to be as it was in the input. See the example from @SHildebrandt above.

And if I'm using Java Formula#equals() method for two arbitrary "mixed variable forms of cnf()" which are "logical identical", I get true as result?
If you mean "logical identical" as in syntactically identical, then yes.

factory.and(factory.variable("x"),factory.variable("y").equals(factory.and(factory.variable("y"),factory.variable("x")) returns true
If you mean semantically equivalent, then the answer might be false. You would need to call the solver for that question.

The equals method for formulas in LogicNG takes some trivial mathematical rules like commutativity or absorption into account.
That means, that formulas like "A & B" and "B & A & A" are considered equals.  But it is not a full semantical equivalence check.  Like @d-bischoff said: therefore you would have to call a SAT solver.
The formula factory in LogicNG is there to minimize heap consumption.  Therefore, formulas are not created twice, if they are equal.  Therefore if you create a formula "A & B" and afterwards you create a formula "B & A" with the same factory, the second one is not generated again, but the first one is returned.

So this is only a problem of the Formula#toString() method for easier readability?

The described phenomenon is not only a property of Formula#toString(). Internally the variables are also re-sorted if you will. However this is not important in any case since the formulas are semantically equivalent. So I don't view this as a problem at all.


Improve DB queries

@steventux Yeah I agree with you, I will add it into the test environment so when we run tests we can catch these N+1 queries during testing.


[JUJU-1437] Make packages internal

The parse, reflect and testing packages are hereby made internal.
This keeps the visible API surface of the package clean.

LGTM


patch: Vertical Stacking(antonholscher) + Dynamic content(TAAPArthur)

Apply multiple patches:
Vertical Stacking(antonholscher) + dynamically updated the displayed notification(TAAPArthur)
I created this patch because i couldn't apply these 2 separated

How do you use this patch? How do you assign an ID to a notification and update the content? The vertical stacking works but I can't figure out the dynamic content part.

env HERBE_ID=/msgid herbe "typeyourdynamicmessagehere"

Hello, how can I apply the Xresource patch for this? Can u help me?

Hello, how can I apply the Xresource patch for this? Can u help me?

It's been a while since I patched herbe, so I don't remember if I had to make any special modifications to make the two patches work together, but you can check out the history of my repository to see exactly how I applied them. I first applied this patch and then I applied the xresources one over top.
https://github.com/mlscarlson/herbe/commits/herbe


DRAM energy in cacti_wrapper.py

The cacti_wrapper.py has references for DRAM energies of LPDDR and GDDR5. For LPDDR4, it is listed as "public data" and no references for rest of the memory types. Is it possible to get the references from where the energy formulas for LPDDR4, DDR3 and HBM2 were derived from?
Thank you

Thank you


[Python 3] New API change to the use of color_cycler

In Python 3, it seems matplotlib has removed the old usage of color_cycler like what is at line 118 in logSummaryReader.py.
According to the suggested new API usage,  line 118 in logSummaryReader.py should be modified to
color_cycle = ax._get_lines.prop_cycler

And line 126 in the same file should be modified correspondingly  to:
... color=next(color_cycle)['color'], ...

Also, line 20 in mesnardo.mplstyle is deprecated in Python 3, though it has not been deleted yet. The new usage is:
axes.prop_cycle : cycler('color',
    ['348ABD', '7A68A6', 'A60628', '467821', 'CF4457', '188487', 'E24A33'])

color_cycle is deprecated in the new version of matplotlib available in the Anaconda distribution (both Python 2 and 3).
I update the different files.
Thank you @piyueh


Remove patch revision from package version

Issue description
The version should not change when patches are applied. See https://github.com/NixOS/nixpkgs/issues/73228#issuecomment-552526517 for reference on the topic.
There are some packages that have some addition to the release version number for patches.
This issue is to clean up our metadata.
Steps to reproduce
I have created a list of all versions containing - and _:
https://gist.github.com/davidak/54c8222e63b229a30574d6e0a1263125
But in many cases it seems to be the upstream version.

Is this issue about cleanup of version attribute or cleanup of derivation name?

The goal is that the version attribute matches the upstream release version.


Best Unit?

Should we displace prices in GB / Mo, or TB / Mo?
19,000 SC / TB / Mo is easier for bulk users, because they typically measure their data in TB.
But, 19 SC / GB / Mo seems more... relatable? 19,000 is a big number. But we should also consider a future where SC is 3x as valuable and the price is 1/2. Now you're looking at prices of 2 and 3 SC / GB / Mo, which means you have very little precision.
But, 2,000 SC / TB / Mo still seems like a huge price. I don't think we want people advertising that they are that expensive. It makes the system feel expensive, which is not really the goal.
I'm uncertain.

Precision will be more important once people start using other tools to price their storage.
Can you specify a decimal price?

Is this just for display, or also entry? If the latter (as well) you may
want to consider the consequences of data entry errors.
For instance, entering 19 SC/TB is a more severe error than entering 19000
SC/GB, since the former is likely to have an immediate practical effect
while the latter will have no effect (because the storage will be orders of
magnitude more expensive than the competition).
On Tue, Apr 26, 2016 at 2:56 PM, David Vorick<EMAIL_ADDRESS>wrote:

Should we displace prices in GB / Mo, or TB / Mo?
19,000 SC / TB / Mo is easier for bulk users, because they typically
measure their data in TB.
But, 19 SC / GB / Mo seems more... relatable? 19,000 is a big number. But
we should also consider a future where SC is 3x as valuable and the price
is 1/2. Now you're looking at prices of 2 and 3 SC / GB / Mo, which means
you have very little precision.
But, 2,000 SC / TB / Mo still seems like a huge price. I don't think we
want people advertising that they are that expensive. It makes the system
feel expensive, which is not really the goal.
I'm uncertain.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/NebulousLabs/Sia/issues/1109


Update http4s-client, http4s-core, ... to 0.23.11

Updates

org.http4s:http4s-client
org.http4s:http4s-core
org.http4s:http4s-dsl

from 0.23.0 to 0.23.11.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #343.


Rewrite using json3

When I first wrote this package, I was pretty inexperienced with Julia and didn't know best practices/how to make my code not ugly.
This rewrites most of the rail functions to utilize JSON3 and take a more Julian way of constructing the DataFrames.
Some light documentation has been added to some functions, but I'll save that for another PR.

Love to see it.


No catch on listen

The test works locally with privileges, I'm able to merge it but we should disable tests and find a proper solution with them to be executable by any.


Deferred sending: Send events over Web-Worker

discarded. This should be a tracker's responsibility and internal implementation.


Package request: Salmon

Salmon is a bioinformatics tool to produce a highly accurate, transcript level quantification estimates from short-read RNA-seq data.
It's mostly written in C++. There's a similar tool already in nixpkgs (Kallisto), but they both have a few unique features, so packaging Salmon is still useful. It's also quite a popular tool.

homepage URL: https://salmon.readthedocs.io/en/latest/index.html
source URL: https://github.com/COMBINE-lab/salmon
license: GLP v3.0
platforms: linux, others?

I started packaging it, and made good progress. It's a way for me to learn Nix, but may need a few pointers to get it done

Open a draft PR, then.


JIRA-469: As a user I want to be able to subscribe to categories

Acceptance Criteria:
When a user hovers over a category they are not subscribed to:
small text appears next to it to subscribe to the category.
When the small text is clicked it creates a subscription to the category on the backend
The front end should somehow indicate the categories that the user is subscribed to

Note: keep in mind 508 and perhaps style the text so it is hidden but still in the dom and readable by screen readers

Maybe something like this for the hover?

I like the idea of an icon instead, but not sure if it would be obvious what it did :(

Will need to add Piwik tracking to the Subscribe link. Possibly have a generic 'subscribe' metric with 'Categories' (and tags and etc) being the sub categories.


Add Day Of Year Function To OpenSearch

Description
Adds the day_of_year function to the SQL plugin. It behaves like the dayofyear function which already exists in the new engine, and both are aligned with MySQL
Issues Resolved
#722
Check List

[X] New functionality includes testing.

[X] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[X] New functionality has user manual doc added

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

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

Codecov Report

Merging #1089 (0c6dcae) into 2.x (fef20f8) will decrease coverage by 35.51%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##                2.x    #1089       +/-   ##
=============================================
- Coverage     98.27%   62.76%   -35.52%
=============================================
  Files           343       10      -333
  Lines          8542      658     -7884
  Branches        544      119      -425
=============================================
- Hits           8395      413     -7982
- Misses          142      192       +50
- Partials          5       53       +48

Flag
Coverage Δ

query-workbench
62.76% <ø> (?)

sql-engine
?

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

Impacted Files
Coverage Δ

...c/main/java/org/opensearch/sql/expression/DSL.java

...arch/sql/expression/datetime/DateTimeFunction.java

...h/sql/expression/function/BuiltinFunctionName.java

...rage/querybuilder/SeriesSelectionQueryBuilder.java

...java/org/opensearch/sql/utils/ExpressionUtils.java

...ch/sql/data/utils/NullsFirstExprValueOrdering.java

...search/sql/planner/logical/LogicalAggregation.java

...search/sql/opensearch/storage/OpenSearchIndex.java

...arch/sql/expression/aggregation/SumAggregator.java

...org/opensearch/sql/planner/DefaultImplementor.java

... and 343 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Improve Command Handling and Error Messages in index.ts

Description:
Problem Statement:
The current implementation of command handling in index.ts lacks detailed error messages and robust handling for various command scenarios. This can lead to confusion for users when commands fail or are used incorrectly.
Proposed Changes:

Enhanced command handling logic in index.ts.
Improved error messages to provide more context and guidance to users.
Updated the README.md to reflect changes in command usage and provide examples.

Changes Made:

Updated src/index.ts to include:

Improved error handling for command execution.
Clear and descriptive error messages for common command issues.

Updated README.md to:

Include new command usage examples.
Provide additional context for the changes made to command handling.

Additional Context:
These changes aim to improve the user experience by making command execution more intuitive and errors easier to understand and resolve. This should help reduce the learning curve for new users and improve overall usability.
Steps to Reproduce:

Attempt to use a command with incorrect parameters in the current version.
Observe the error message and handling.
Compare with the updated version for improved feedback and guidance.

Expected Outcome:
Users should experience more informative error messages and smoother command handling, leading to a better overall experience with the tool.

kindly assign this issue to me!!!!!!


Add a hash map for storing groupBy partition index

Improves groupBy performance by approx 15%

+1 I believe this is merge-able

Before we merge can we add comments to indicate the purpose of the second hash map. It wouldn't be obvious out of context.

@xvrl : comment added


AWSCredentialsProvider.Credential_steps(region=XXX) not working

Not sure if this is a bug or working as designed.
aws_creds: AwsCredentialsProvider = AwsCredentials.from_git_hub_secrets(
            access_key_id="AWS_ACCESS_KEY_ID_DEV_EU_WEST_1",
            secret_access_key="AWS_SECRET_ACCESS_KEY_DEV_EU_WEST_1",
        )
aws_creds.credential_steps(region="eu-west-1)

Produces:
 - name: Authenticate Via GitHub Secrets
   uses: aws-actions/configure-aws-credentials@v4
   with:
      aws-region: us-west-2
      role-duration-seconds: 1800
      role-skip-session-tagging: true
      aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID_DEV_EU_WEST_1 }}
      aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY_DEV_EU_WEST_1 }}

It seems the setup is always the same so can patch it at the end of things
def update_region_git_hub_secrets(self, env: Environment, asset: str) -> JsonPatch:
       return JsonPatch.replace(f"/jobs/{asset}/steps/2/with/aws-region", env.region)

assets_to_update = ["Assets-DockerAsset1", "Assets-DockerAsset2", "Assets-FileAsset1", "Assets-FileAsset2", "Assets-FileAsset3"]
for asset in assets_to_update:
   patch = self.update_region_git_hub_secrets(env=env, asset=asset)
   deploy_workflow.patch(patch)

Hi @hogbinj, I'm not certain what the unexpected behavior is. Do you mind laying it out a bit clearer for me?

if you leave it as standard and your credentials are in a different AWS Region than us-west-2 or whatever then it cannot retrieve them and the job fails


Output stats to console and Slack

Steps:

[x] Refactor getting projects
[x] Output simple stats to console
[x] Output simple stats to Slack

Later:

[x] Output more complex stats to Slack


feat: [DONT MERGE] Testing SSH Tunnel

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up


Added ssp extension automatically for saving project in linux

In linux, file dialog doesn't add extension when saving, that caused save
project crash.

Would it be better to use completeSuffix ?

Hey! use those functions where? completeSuffix and suffix are for getting the suffix.. in this case we need to set the suffix. you mean there is a function for that?

Yes, look at QFileInfo(fileName).suffix() in MainWindow.cpp

In that code you are getting the suffix to see if it is a ssp project. I would just use suffix because you don't expect your projects to have multiple extensions. If you use suffix and the project ends in ssp, then you try to open it. However, i don't think this is related to this PR, is it?. Should we continue discussing it on a sepparate issue?

Yes, it is correct

Hi! is any change required for this PR?

Hi, plz use suffix instead of endsWith

Ohhh i understand now. Sorry.

Done!

Thanks!


datapkg_to_sqlite fails to load all of EPA CEMS

After doing a full ETL of all years and states in CEMS, the datapkg_to_sqlite script doesn't seem to load all of that data into the SQLite database. Rather, it only loads the last year of data into the database. However, the process terminates quickly, so it's probably not even attempting to load all the data. Suspect it's an issue with the iteration and/or partitioning...

Hmmm I tried to recreate this issue with a grouped table and am getting a similar result. For me only the last resource described in the datapackage.json file is showing up in the sqlite database. @roll, is it possible that for the grouped table sqlite export is not appending the parts of the grouped table but just clobbering each of the earlier groups.
I generated data packages for CEMS with these parameters:
epacems_years: [2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013, 2014]
epacems_states: [ID,MT,CO,TX,IL]

It is being partitioned by state and year so each tabluar resource for the
hourly_emissions_epacems table is a part of the same group. Here is a description of one of the resources:
       {
            "profile": "tabular-data-resource",
            "name": "hourly_emissions_epacems_2005_id",
            "path": "data/hourly_emissions_epacems_2005_id.csv.gz",
            "title": "hourly_emissions_epacems",
            "encoding": "utf-8",
            "mediatype": "text/csv",
            "format": "csv",
            "dialect": {
                "delimiter": ",",
                "header": true,
                "quoteChar": "\"",
                "doubleQuote": true,
                "lineTerminator": "\r\n",
                "skipInitialSpace": true,
                "caseSensitiveHeader": false
            },
            "schema": {
                "fields": [
                    {
                        "name": "plant_id_eia",
                        "type": "integer",
                        "description": "The unique six-digit facility identification number, also called an ORISPL, assigned by the Energy Information Administration.",
                        "format": "default"
                    },
## a bunch of deleted field descriptions
                            ]
                        }
                    }
                }
            ],
            "bytes": 229820,
            "hash": "sha256:12d2998a68c0689c19061e001910519c7a19dfe6be3fbc8444f914d9a9ff4c9f",
            "created": "2019-10-22T19:58:40Z",
            "start_date": "2005-01-01",
            "end_date": "2014-21-31",
            "group": "hourly_emissions_epacems"
        },

@cmgosnell
Could you please share something I can run to debug this issue?

No, still getting the error with the most recent versions of the datapackage libraries. A very simple version with a couple of resources in a group seems to work as expected, but the simplest PUDL output that tests the behavior doesn't work. I'm trying to simplify that resource group output one step at a time until I get to a minimal example to share with you.

Hey @roll! this was totally just our mistake, so don't worry about it. We are making a bunch of data source specific data packages and then squishing them together into one package, I set up a process for determining how to generate a new data package without duplicating elements of the metadata.... but I messed up and it was only grabbing one of the CEMS resources. It was a very simple fix once we figured out what was happening.


Zone renting separated from taxes

This update adds majorly a zone renting for either admin nations or player nations which runs separately from taxes. Alos adds zone balances to automatically pay the rent.
Together adds a cosmetic thing:Display names to nations, separated from real names, this ones can have any character and length, and also can be repeated between nations and zones, after all it's just a name for displaying.

Very cool feature, a lot of Nations users will like this improvement.


ER Editor allways closed

When I try put my PS4 save file on ER Editor, the program allways closed.
Any clue to fix that?

Its an old version. Im working on a new release
duplicate of #43


Simplify the code that resets the value of the pageNumber element in PDFViewerApplication.setInitialView

Another very small piece of viewer.js clean-up.
After the creation of PDFViewer, its _resetView method takes care of resetting, among other things, the page number property. Hence we don't need to set pdfViewer.currentPageNumber = 1; here any more, and the comment is no longer accurate either.

/botio-windows preview

Looks good, thank you.


Upgrade all possible dependencies

Upgrade + pin all dependencies
Improve build commands

https://capacitorjs.com/docs/updating/5-0
https://nextjs.org/docs/app/building-your-application/upgrading/version-14#upgrading-from-13-to-14
Addresses #54, #41, and #43
Note: I was unable to upgrade react-router and react-router-dom from 5.x to 6.x at this time due to the peer dependency requirements of ^5.x from @ionic/react-router

Do you guys have permission to merge/update? If so let me know what I have to do to enable that. I won't be able to review contributions but trust you guys know what's best anyways

@mlynch yes, I do! I was waiting on review just as a general best practice, but will go ahead and merge this one in. Thanks!

Still a few dependencies out of date. Unfortunately, some of them require major updates.

Still a few dependencies out of date. Unfortunately, some of them require major updates. How we can fix that ? We can't because of ion-router?

correct

Note: I was unable to upgrade react-router and react-router-dom from 5.x to 6.x at this time due to the peer dependency requirements of ^5.x from @ionic/react-router


Build on travis

Hello,
I was having issues building from source on Ubuntu because of libraries and permission changes not mentioned in the documentation.
Took about 11 builds to get it right. But now it builds fine on a basic ubuntu install.
I have added those changes to the document as well as created travis-ci build Results.
I hope you find this update useful.
ONION: DoPhPRHvZhgjTV4mSo7w5z6pDWqUn3LpXd

Thanks for adding my pull request. Is there any chance you could add me to the AirDrops? I was really hoping to be a part of this project more.
DoPhPRHvZhgjTV4mSo7w5z6pDWqUn3LpXd


Market aid book updating and order management

There is still a lot more to add and a long road of improvements ahead for rubi-bots. This PR includes the first two working strategies* w/ the added caveats that there are many bugs but I want to include the progress. Here is an overview of what I think needs to be added next:

[ ] Testing and verifying the tail-off module in the RiskMinimized strategy
[ ]  Implementing an inventory management model in TargetVenueOutBid for smarter liquidity provisioning
[ ]  Adding entry points for custom strategies that can fork the low-level repo to always have access to utility functions
[ ]  In general, lots of code clean-up work and iterations for modularity should be made... onwards and upwards

Also note that this only works full-stack on Mumbai atm


I've now changed it :)


Increment version for ai/azopenaiassistants releases

Increment package version after release of ai/azopenaiassistants

@richardpark-msft Are you waiting on anything?

@richardpark-msft Are you waiting on anything?

Yes.

/check-enforcer override


How to deal with the conv in SE module？

same question

any updates?

Looking forward your reply, any suggestions? @keyu-tian

@RainFrost1 @will1973 @yxchng SE module contains two linear layers (channel-wise) and one global average pooling. No changes required to the linear layers. As for the global pooling on a masked feature map, one should calculate the mean value of non-masked positions only. To implement this you can use our function _get_active_ex_or_ii in https://github.com/keyu-tian/SparK/blob/main/pretrain/encoder.py#L30 like:
class SparseGlobalAveragePooling(nn.Module):
    def forward(self, x):   # shape: BCHW
        B, C, H, W = x.shape
        non_masked_positions = _get_active_ex_or_ii(H=H, W=W, returning_active_ex=True)
        mean = (x * non_masked_positions).sum(dim=(2,3), keepdims=True) / non_masked_positions.sum(dim=(2,3), keepdims=True)
        return mean         # shape: BC11


EnsoCompilerTest to verify compatibility of parsers

Pull Request Description
Adding new compatibility test EnsoCompilerTest to verify the new Rust based parser can produce the same IR as the original AST based one. The simplest way to execute the test from an empty repository is:
enso$ sbt bootstrap
enso$ sbt "testOnly *EnsoCompilerTest"

There are GitHub Actions run on Linux as well as run on Windows that show EnsoCompilerTest is being executed by the CI (good, as that means .so was properly built and linked to the JVM running the test). The linux as well as windows runs also demonstrate that failures in the EnsoCompilerTest suite fail the CI.
Important Notes
Right now there are five test failures - waiting for @kazcw to make sure codeRepr() doesn't contain spaces. However, as this PR is more about the infrastructure, I am disabling the currently failing tests in 031169b
Checklist
Please include the following checklist in your PR:

[ x ] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[ x ] Unit tests have been written where possible.

[ci no changelog needed]

Any idea what might be wrong with my environment? I think I have the right version of Graal.

Looks like there is some debris from previous builds - try sbt clean first. Or (to be sure):
enso$ rm -rf *; git checkout -f .

That worked! I had done a ./run git-clean, which I thought would be just as comprehensive


Update Facilities.md

replace link to github project as it seems one the most active source compared to the previous dead link and other site http://sharparchitecture.github.io/ which looking a bit outdated

Thanks


fix: bump cornerstone-tools to latest version

Closes #917
Related: https://github.com/OHIF/Viewers/pull/918

CC: @swederik

We've been hammering the dcmjs demo PACS pretty hard. Slow response time is causing failing E2E tests intermittently.


bug: ion-select always show options at start instead of showing selectedItem

Bug Report
Ionic version:
[x] 5.x
Current behavior:
1- The list of options always appear to be at start even if the selectedItem is at last postion when there are more options (list becomes scroll-able).
2- The options list does not close if i click/select, the already selectedItem, i need to click somewhere else to close it.
Expected behavior:
1- if the last item or any other item which gets selected, then the selected option should be shown right away instead of the need to scroll down and see where the selected item is.
2- The options list needs to get close if selectedItem is selected/clicked again, it happens in all other UI kits.
Steps to reproduce:

Related code:

https://stackblitz.com/edit/ionic-v4-angular-tabs-22m8i9?embed=1&file=src/app/tab1/tab1.page.html
insert short code snippets here

Other information:

Ionic info:

Ionic CLI                     : 6.2.2
Ionic Framework               : @ionic/angular 5.0.4
@angular-devkit/build-angular : 0.803.25
@angular-devkit/schematics    : 8.3.25
@angular/cli                  : 8.3.25
@ionic/angular-toolkit        : 2.2.0
NodeJS : v12.16.1
npm    : 6.14.2
OS     : Windows 10
insert the output from ionic info here

Thanks for the issue. I am going to close this as a duplicate of https://github.com/ionic-team/ionic/issues/19296.


Proxy support

As the presence of an HTTP proxy is common in a company, could you please provide proxy support ?
I quickly tested the following configuration in Autounattend.xml (eval-win7x64-enterprise)  but it failed:
    <settings pass="oobeSystem">
        <component name="Microsoft-Windows-IE-ClientNetworkProtocolImplementation" processorArchitecture="amd64" publicKeyToken="31bf3856ad364e35" language="neutral" versionScope="nonSxS" xmlns:wcm="http://schemas.microsoft.com/WMIConfig/2002/State">
          <POLICYProxySettingsPerUser>0</POLICYProxySettingsPerUser>
          <HKLMProxyEnable>true</HKLMProxyEnable>
          <HKLMProxyServer><IP_ADDRESS>:3128</HKLMProxyServer>
        </component>
...

Error :

Exception calling "DownloadFile" with "2" argument(s): "Unable to connect to remote server"
BITSAdmin is deprecated and is not guaranteed to bo available in future versions of Windows... then endless waiting.

Note: The proxy doesn't need user/pass.
Doc: https://technet.microsoft.com/en-us/library/cc748894(WS.10).aspx
Thanks!

Ahh.. I didn't saw the waiting PR #85 ..

Hey 2xyo, you still interested in Proxy support?
I have a few things I want to improve wrt relocating templates (since I'm not married to GNU Make or anything), but daxgames linked to a good base and so it seems like a good first feature to add especially since you've been waiting for like what...3 years?

Yes please!

Ok. I'll start working on how to figure out an elegant way to accomplish that. I'll update this with a PR that I'm working on when I make progress. Thanks.