Extracted description broken

Extracted description of lineBreaks is broken, look here: https://github.com/npalm/action-docs-action/blob/develop/README.md#inputs

I have discovered that the description is not broken but simply the | is treated as column separator, so the solution is to replace | with |

@ale5000-git thanks for creating the issue, but I think I don't got the issue. Please can you explain in a bit more detail the problem. Or create a PR?

The sources of your README.md:
| parameter | description | required | default |
| - | - | - | - |
| readme | Readme file to update. | `false` | README.md |
| actionFile | THe action definition file. | `false` | action.yml |
| tocLevel | TOC level used for the headers. | `false` | 2 |
| lineBreaks | Line breaks to be used in updated readme (LF|CR|CRLF). | `false` | LF |

The "|" character inside "(LF|CR|CRLF)" is treated by the markdown parser as column separator and not as normal character so it is disaplayed wrongly, look here (at the lineBreaks description): https://github.com/npalm/action-docs-action/blob/develop/README.md#inputs

The  action-docs-action shouldn't leave "|" characters in descriptions as is but replace them with | to avoid this problem.

@npalm
You fixed the effect without the cause.
The change should be placed in the action that update the readme.
You're readme was just an example, it happens on all projects using your action.

@npalm
Any update on this?


Unable to include "shadows" parameter in mark properties with Toolbar and Toolbar 2.0 (Beta)

Sketch v.45.1 (43504)
Sketch Measure v.2.6.7
Reopening issue from "Unable to mark properties with Toolbar and Toolbar 2.0 (Beta) #326"
Thank you for fixing and everything works except one parameter.
The only issue is the shadow parameter. If you choose check shadows when marking properties with the following steps, the same outcome happens; No response or empty folder.
_Toolbar and Toolbar 2.0 (Beta) are visible. Mark functions work except for "properties". Attempted to Mark Properties from Toolbar, Toolbar 2.0 (Beta), and from the Plugin menu. Was not successful
Toolbar -> Tap Mark Properties -> No response
Toolbar 2.0 (Beta) -> Tap Label - Mark Properties (one of the 4 icons) -> Dialogue box opens up for property parameters -> #properties folder created but no properties shape and layers.
Plugin Menu -> Tap Mark Properties -> No response_

Old toolbar you need to know, " Hold down the ⌥ key and click the sizes button. "


Issues with rbenv and git sources

Description
Reproduction steps
I have a git gem in my gemfile, during booting the ruby-lsp seems to instruct bundler to look into the wrong spot for that.

The path /home/user/.local/share/gem/ruby/3.2.0/bundler/gems/gem_name-5ef8fd7a5205 does not exist. (Bundler::PathError)

/home/user/.local/gem does in fact not exist on my machine. The correct path would be /home/user/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/bundler/gems/gem_name-5ef8fd7a5205/. With bundle exec irb I am able to correctly require and use the gem.
Here's the relevant part of the activation script:
{:env=>
  {:SHELL=>"/bin/bash",
   :RBENV_HOOK_PATH=>
    "/home/user/.rbenv/rbenv.d:/usr/lib/rbenv/rbenv.d:/usr/local/etc/rbenv.d:/etc/rbenv.d:/usr/lib/rbenv/hooks",
   :RBENV_SHELL=>"bash",
   :PWD=>"/home/user/Documents/project",
   :HOME=>"/home/user",
   :RBENV_ORIG_PATH=>
    "/home/user/.rbenv/shims:/home/user/.rbenv/shims:/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/lib/jvm/default/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl:/home/user/.local/bin:/home/user/.local/bin",
   :USER=>"user",
   :RBENV_DIR=>"/home/user/Documents/project",
   :RBENV_VERSION=>"3.2.2",
   :PATH=>
    "/home/user/.rbenv/versions/3.2.2/bin:/usr/lib/rbenv/libexec:/home/user/.rbenv/shims:/home/user/.rbenv/shims:/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/lib/jvm/default/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl:/home/user/.local/bin:/home/user/.local/bin",
   :RBENV_ROOT=>"/home/user/.rbenv"},
 :yjit=>true,
 :version=>"3.2.2",
 :home=>"/home/user/.local/share/gem/ruby/3.2.0",
 :default=>"/home/user/.rbenv/versions/3.2.2"}

If I revert #1811 it works again. Perhaps it needs to check for the presence of Gem.user_dir before adding it to the env?

Thank you for reporting this. I might have misunderstood exactly how rbenv sets up gem environment variables.
Can you please print these in your terminal (with rbenv activated)? I'm trying to understand if rbenv is setting the environment variables or if relies on Ruby's defaults.
echo $GEM_HOME
echo $GEM_PATH
echo $PATH

For the PATH, what I need to know is: does rbenv add gem bin directories to the path? Or does it add some shims directory into the path?
For example, chruby adds directories like ~/.gem/ruby/3.2.2/bin to the PATH, so that you can invoke gem executables. Asdf on the other hand, doesn't add those and instead adds its shims directory to the PATH, where they have a shim for every gem executable (~/.asdf/shims).

Now that's a quick turnaround! With the extension from that PR the LSP is able to again start up without errors in the project that contains the git source. This is great, thank you.


[websocket-nio]Fix failures with continuation frames

@nethraravindran Can you please rebase against the latest websocket-nio branch?


chore: suggest autofix when #guard_msgs errors

Not certain this is a good idea, I'll let reviewers evaluate.
(I'm a bit worried about encouraging people to blindly update the message!)
Suggested on zulip.

nvim users feeling left out is a feature, not a bug, but yes, I agree overall.


rshell: 0.0.30 -> 0.0.31

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for rshell is: "Remote Shell for MicroPython"
meta.homepage for rshell is: "https://github.com/dhylands/rshell"
meta.changelog for rshell is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/9v45dfqf9n8rg1zc8frgxfrfb5qm4zxv-rshell-0.0.31/bin/pyboard passed the binary check.
/nix/store/9v45dfqf9n8rg1zc8frgxfrfb5qm4zxv-rshell-0.0.31/bin/.rshell-wrapped passed the binary check.
/nix/store/9v45dfqf9n8rg1zc8frgxfrfb5qm4zxv-rshell-0.0.31/bin/rshell passed the binary check.
/nix/store/9v45dfqf9n8rg1zc8frgxfrfb5qm4zxv-rshell-0.0.31/bin/.pyboard-wrapped passed the binary check.
4 of 4 passed binary check by having a zero exit code.
4 of 4 passed binary check by having the new version present in output.
found 0.0.31 with grep in /nix/store/9v45dfqf9n8rg1zc8frgxfrfb5qm4zxv-rshell-0.0.31
found 0.0.31 in filename of file in /nix/store/9v45dfqf9n8rg1zc8frgxfrfb5qm4zxv-rshell-0.0.31

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

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
rshell

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9v45dfqf9n8rg1zc8frgxfrfb5qm4zxv-rshell-0.0.31 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

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

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

rshell

Maintainer pings
cc @c0deaddict for testing.

Result of nixpkgs-review pr 153000 run on x86_64-linux 1

1 package built:

rshell


Remove object spread

Fixes #23
I had to update the unpkg refs because redux 4.0.0 is different.

Hi guys, when can it be published to npm? I see the latest version (0.8.2) still uses object spread.


THRIFT-4752: Added cmake package configuration for cpp and compiler

This pull request is loosely inspired by the cmake documentation and by the hunter-package cmake package configuration at https://github.com/hunter-packages/thrift/.
The PR adds support for the cpp library, transitive library dependencies and and the thrift compiler in a cmake package configuration.
With the PR in place, users can
find_package(thrift)

without the need for a FindThrift.cmake script.

Merged #1714 instead of this one.

Whoops I closed the wrong one.

A reminder this needs to be rebased.

@emmenlau Hi, where is your fork? I want to make some change in your PR :) thanks :)

Dear @soroshsabz please find it here: https://github.com/BioDataAnalysis/thrift/tree/emmenlau_add_cmake_package_configuration

Closed by #1748


Jeremy's list

[x] Section 2.2 WIS2 to GTS Gateway
[ ] Section 3.1.2 RTH (Maintenance and operation of MSS) ... with @golfvert
[x] Section 5.3 GISCs (Management of WIS Centres) ... with @efucile

Stuff is done. New issues to be raised for outstanding work.


[RFC] CDC library

Type of issue: feature request
Impact: API addition (no impact on existing code)
Development Phase: proposal
For a PnR friendly RTL, I think we should start to consider how to add Clock-domain-checking library.
Here are my simple ideas to this problem.
for registers in the circuit, they have 4 types:

under a clock
under a clock and correspond synchronous reset
under a clock and full asynchronous reset
under a clock and asynchronous reset synchronous deassertion reset.
RTL of "full asynchronous" and "asynchronous reset synchronous deassertion"  are same:
https://scastie.scala-lang.org/6HKMD9YbSxGeZxP0oroEtw

However a synchronizer should be implemented like this.
https://scastie.scala-lang.org/gAUzcR5iQXSCXVXwGgEATA
These two asynchronous reset have two types of reset, but a same chisel type.
Basically, my idea is annotate clock and reset and let FIRRTL transform to resolve and check them.
I wanna RFC for some comments on what to annotate, and what to check.
Here are something I can come up with:

annotate clock frequency.
annotate clock - synchronous reset relationship
annotate reset generation IO or Module.
annotate asynchronous type.
annotate false path(from register to register?)
annotate safe cross clock domain Module.(AsyncQueue, RationalQueue etc.)
check register under different clock are not crossing.
check register under same clock - synchronous reset pair are not crossing to other clock - synchronous reset pair.
check asynchronous resets are from a reset generation IO without passing any combinational path.

Output should be a set of annotations and can have a vendor specific Transform to generate SDC files.

I like this idea. One concern I have about approaches I have heard in the past is if we derive the CDC from the FIRRTL directly (like, "this register is synchronous to this clock"), that is basically exactly what the SDC lint checks will do, therefore your lint checks dont actually check anything anymore. You really want a human in the loop saying something about their design intent, then a check that the hardware actually does that.
Haven't read through your proposal in detail, just noting one solution I would not prefer.

I like this idea. One concern I have about approaches I have heard in the past is if we derive the CDC from the FIRRTL directly (like, "this register is synchronous to this clock"), that is basically exactly what the SDC lint checks will do, therefore your lint checks dont actually check anything anymore. You really want a human in the loop saying something about their design intent, then a check that the hardware actually does that.
Haven't read through your proposal in detail, just noting one solution I would not prefer.

You probably also want:
10. the clock domain of any toplevel input signal (or if it is asynchronous)

You probably also want:
10. the clock domain of any toplevel input signal (or if it is asynchronous)

@sequencer, when we want to do clock domain analysis - in my opinion - the biggest problem are special cases where clocks or resets are cast to something else. Could you try and see if the asClock or asAsyncReset primops are used anywhere in your company's code base? Having some use cases would help us decide how to deal with them when trying to extract the clock domains.

@sequencer, when we want to do clock domain analysis - in my opinion - the biggest problem are special cases where clocks or resets are cast to something else. Could you try and see if the asClock or asAsyncReset primops are used anywhere in your company's code base? Having some use cases would help us decide how to deal with them when trying to extract the clock domains.

@azidar made something related to these issues a long time ago to make the "designer inspection" part simpler.
It wasn't perfect but it did help catch a few issues at the time.
https://github.com/chipsalliance/firrtl/tree/master/src/main/scala/firrtl/passes/clocklist

@azidar made something related to these issues a long time ago to make the "designer inspection" part simpler.
It wasn't perfect but it did help catch a few issues at the time.
https://github.com/chipsalliance/firrtl/tree/master/src/main/scala/firrtl/passes/clocklist

I like this idea. One concern I have about approaches I have heard in the past is if we derive the CDC from the FIRRTL directly (like, "this register is synchronous to this clock"), that is basically exactly what the SDC lint checks will do, therefore your lint checks dont actually check anything anymore.

Yes, clock tree analysis will be executed during synthesis flow. checking CDC error is redundant, but annotate relationship for different clock/reset the target.

You really want a human in the loop saying something about their design intent, then a check that the hardware actually does that.

Yes, what I really wanna do is: try to gather "what will a human say during this loop", abstract them to APIs, annotate to target Data.
Chisel only handle synchronous circuit, and SDC for synchronous circuit won't be too much, I think we only need to gather the clock and reset relationship.

I like this idea. One concern I have about approaches I have heard in the past is if we derive the CDC from the FIRRTL directly (like, "this register is synchronous to this clock"), that is basically exactly what the SDC lint checks will do, therefore your lint checks dont actually check anything anymore.

Yes, clock tree analysis will be executed during synthesis flow. checking CDC error is redundant, but annotate relationship for different clock/reset the target.

You really want a human in the loop saying something about their design intent, then a check that the hardware actually does that.

Yes, what I really wanna do is: try to gather "what will a human say during this loop", abstract them to APIs, annotate to target Data.
Chisel only handle synchronous circuit, and SDC for synchronous circuit won't be too much, I think we only need to gather the clock and reset relationship.

in my opinion - the biggest problem are special cases where clocks or resets are cast to something else. Could you try and see if the asClock or asAsyncReset primops are used anywhere in your company's code base?

Yes, that's what I really care about, I discussed with some experienced PnR engineers, they said there will always be some  ClockGen and ResetGen Module to do commutation on clock/reset signal, rather than directly pull a Clock signal, cast them to Bool, do some combinational logic, and cast back to Clock, they are forbidden to do this. So I think if our Clock cast and Reset cast have a bad design for PnR guys?

in my opinion - the biggest problem are special cases where clocks or resets are cast to something else. Could you try and see if the asClock or asAsyncReset primops are used anywhere in your company's code base?

Yes, that's what I really care about, I discussed with some experienced PnR engineers, they said there will always be some  ClockGen and ResetGen Module to do commutation on clock/reset signal, rather than directly pull a Clock signal, cast them to Bool, do some combinational logic, and cast back to Clock, they are forbidden to do this. So I think if our Clock cast and Reset cast have a bad design for PnR guys?

the clock domain of any toplevel input signal (or if it is asynchronous)

In my understanding, annotating a clock anywhere inside the circuit will propagate the metadata to whole clock tree. So maybe no need to annotate toplevel input signal, aka, the source of clock tree. We just need to annotate a clock node inside the clock tree and let CircuitAnalysis transform automatically spread the metadata.

the clock domain of any toplevel input signal (or if it is asynchronous)

In my understanding, annotating a clock anywhere inside the circuit will propagate the metadata to whole clock tree. So maybe no need to annotate toplevel input signal, aka, the source of clock tree. We just need to annotate a clock node inside the clock tree and let CircuitAnalysis transform automatically spread the metadata.

@ azidar made something related to these issues a long time ago to make the "designer inspection" part simpler.
It wasn't perfect but it did help catch a few issues at the time.

That's a good start point to do this! We already have CircuitGraph and ConnectionGraph to do connection analysis, I think the rest of thing is try to understand the real need, design API, and work on the transform, the most hard part are perfectly done by Adam ;p

@ azidar made something related to these issues a long time ago to make the "designer inspection" part simpler.
It wasn't perfect but it did help catch a few issues at the time.

That's a good start point to do this! We already have CircuitGraph and ConnectionGraph to do connection analysis, I think the rest of thing is try to understand the real need, design API, and work on the transform, the most hard part are perfectly done by Adam ;p

Yes, that's what I really care about, I discussed with some experienced PnR engineers, they said there will always be some ClockGen and ResetGen Module to do commutation on clock/reset signal, rather than directly pull a Clock signal, cast them to Bool, do some combinational logic, and cast back to Clock, they are forbidden to do this.

Could you build the firrtl of some of your designs and grep for asClock or asAsyncReset?
I am just wondering if people are using these today, or if all the clock and reset generators are written in Verilog.

Yes, that's what I really care about, I discussed with some experienced PnR engineers, they said there will always be some ClockGen and ResetGen Module to do commutation on clock/reset signal, rather than directly pull a Clock signal, cast them to Bool, do some combinational logic, and cast back to Clock, they are forbidden to do this.

Could you build the firrtl of some of your designs and grep for asClock or asAsyncReset?
I am just wondering if people are using these today, or if all the clock and reset generators are written in Verilog.

In my understanding, annotating a clock anywhere inside the circuit will propagate the metadata to whole clock tree. So maybe no need to annotate toplevel input signal, aka, the source of clock tree. We just need to annotate a clock node inside the clock tree and let CircuitAnalysis transform automatically spread the metadata.

The reason I suggested to annotate the clock domain of the inputs is to check whether there is a clock domain crossing of a data signal. If you think about a circuit in firrtl, we can easily determine the clock domain of any signal that comes from a register since it will be the same as the register's. However, for input to the top level module (and also for outputs of BlackBoxes) we do not know the clock domain and thus it needs to be annotated in order to do CDC checking.

In my understanding, annotating a clock anywhere inside the circuit will propagate the metadata to whole clock tree. So maybe no need to annotate toplevel input signal, aka, the source of clock tree. We just need to annotate a clock node inside the clock tree and let CircuitAnalysis transform automatically spread the metadata.

The reason I suggested to annotate the clock domain of the inputs is to check whether there is a clock domain crossing of a data signal. If you think about a circuit in firrtl, we can easily determine the clock domain of any signal that comes from a register since it will be the same as the register's. However, for input to the top level module (and also for outputs of BlackBoxes) we do not know the clock domain and thus it needs to be annotated in order to do CDC checking.

However, for input to the top level module (and also for outputs of BlackBoxes) we do not know the clock domain and thus it needs to be annotated in order to do CDC checking.

Yes, I got it, that's what we need to annotate too!

However, for input to the top level module (and also for outputs of BlackBoxes) we do not know the clock domain and thus it needs to be annotated in order to do CDC checking.

Yes, I got it, that's what we need to annotate too!


Empty results on univariate dataset scripts

Hello, thanks a lot for your effort into this comprehensive forecasting benchmark.
I tried to run for univariate dataset "weather_dataset_3.csv" (downloaded from your google drive link) but I get empty results. For example,  running
python ./scripts/run_benchmark.py --config-path "fixed_forecast_config_daily.json" --data-name-list "weather_dataset_3.csv"  --model-name  "time_series_library.Triformer"  --model-hyper-params "{\"d_model\":16,\"d_ff\":16,\"factor\":3}" --adapter "transformer_adapter" --save-path "daily"   --gpus 0 1  --num-workers 1 --timeout 60000
trains successfully the model with no errors and creates the zip file. However, when unzipping the test .csv file it is empty without metric results (see figures below).
For rolling_forecast as below, I get all the results.
python ./scripts/run_benchmark.py --config-path "rolling_forecast_config.json" --data-name-list "Weather.csv" --strategy-args '{"horizon":96}' --model-name "time_series_library.Linear" --model-hyper-params '{"d_ff": 64, "d_model": 32, "horizon": 96, "seq_len": 512}' --adapter "transformer_adapter"  --gpus 0  --num-workers 1  --timeout 60000  --save-path "Weather/Linear"

What can be the cause for this?
Besides this issue, what are your recommendations/guidelines for running the benchmark on a custom dataset? e.g., a) data format, b) config file update, c) hyperparameter tuning

Thanks.

python ./scripts/run_benchmark.py --config-path "fixed_forecast_config_daily.json" --data-name-list "weather_dataset_3.csv"  --model-name  "time_series_library.Triformer"  --model-hyper-params "{"d_model":16,"d_ff":16,"factor":3}" --adapter "transformer_adapter" --save-path "daily"   --gpus 0 1  --num-workers 1 --timeout 60000
Thank you for your attention and question. I ran the same command, but I got the result as follows:

For the second question, we will provide a dedicated tutorial later on.
You can now follow the following tutorial to use your own dataset.
Step 1: Convert your own dataset to our format, which is a 3-column long table. You can open a multivariate dataset or a univariate dataset to view it.
Step 2: Add your dataset information to the FORECAST-META.csv file. Please note that you only need to fill in attributes such as filename, freq, if_univariant, size, and length. Please fill in the size as large, and the length is the original length of your dataset, not the length of the dataset converted to our format.
Step 3: Specify your own dataset name in the command, such as: python ./scripts/run_benchmark.py --config-path "fixed_forecast_config_daily.json" --data-name-list "your dataset name" --model-name "time_series_library.Triformer" --model-hyper-params "{"d_model":16,"d_ff":16,"factor":3}" --adapter "transformer_adapter" --save-path "daily" --gpus 0 1 --num-workers 1 --timeout 60000

For the second question, we will provide a dedicated tutorial later on.
You can now follow the following tutorial to use your own dataset.
Step 1: Convert your own dataset to our format, which is a 3-column long table. You can open a multivariate dataset or a univariate dataset to view it.
Step 2: Add your dataset information to the FORECAST-META.csv file. Please note that you only need to fill in attributes such as filename, freq, if_univariant, size, and length. Please fill in the size as large, and the length is the original length of your dataset, not the length of the dataset converted to our format.
Step 3: Specify your own dataset name in the command, such as: python ./scripts/run_benchmark.py --config-path "fixed_forecast_config_daily.json" --data-name-list "your dataset name" --model-name "time_series_library.Triformer" --model-hyper-params "{"d_model":16,"d_ff":16,"factor":3}" --adapter "transformer_adapter" --save-path "daily" --gpus 0 1 --num-workers 1 --timeout 60000

a) If you want to compare the performance with the univariate algorithm in the paper, you should follow the configuration in the existing configuration file and not modify the forecasting horizon and look back window size. The hyperparameters of the algorithms themselves, such as training epochs and lr, can be modified.
b) Whether to perform rolling forecast is another usage scenario and also reflects another ability of the algorithm. You can decide according to your needs.

For each algorithm on a dataset, we ran multiple sets of hyperparameters and reported the performance results under the best parameters in the paper. The univariate scripts provide examples of batch evaluating univariate datasets.
We suggest that you use the settings in the config file, including seed. You can also write your own config file according to your needs

Hi, If you have any further questions, please feel free to report them to us. If the problem has been resolved, please close the issue.


Could pls support cors ？

I spoke to this in my introductory blog post on this demo, but CORS support is something you would need to handle server-side. https://rossta.net/blog/building-a-pdf-viewer-with-vue-part-1.html#requirements
In the case of this demo, I've uploaded PDFs to a free account on Filestack and have set it to accept uploads from any origin.


Error:Could not get unknown property 'bintray_user' for object of type com.novoda.gradle.release.PublishExtension

will you please post answer to this error.(I am getting error while setup the project)

bintrayuser =""
bintraykey=""


Compatibility Issue with GridSample and ONNX Opset 15 on NXP i.MX 93 NPU

Hi,
I'm currently working on deploying the LightGlue model from this repository on the NXP i.MX 93 NPU using the EIQ toolkit. However, I've encountered a challenge: the toolkit supports only ONNX opset 15, which seems not to include support for the GridSample layer used in the PyTorch model.
Could you suggest any alternatives to the GridSample operation that would be compatible with ONNX opset 15, or any other workaround that might help in converting and running this model on the specified NPU?
Cheers
Paul

I am not experienced in ONNX, so I can not really help here. The grid sample in python is basically an interpolation of the descriptor map, you could try to port the interpolation from PixLoc.


Added Model for Time Series Classification

I have added the code for a basic RNN Model for the task of time series classification.
> data, blocks = load(datarecipes()["ecg5000"]);
> task = TSClassificationSingle(blocks, data);
> model = FastAI.taskmodel(task);
> traindl, validdl = taskdataloaders(data, task, 32);
> callbacks = [ToGPU(), Metrics(accuracy)];
> learner = Learner(model, tasklossfn(task); data=(traindl, validdl), optimizer=ADAM(), callbacks = callbacks);
> fitonecycle!(learner, 10, 0.033)

As I discussed with @darsnack, the idea is to add an encoding to do the reshaping to (features, batch, time) instead of doing it inside on the RNN Model. Working on that right now.
Have remove the type from StackedLSTM as it was redundant.

I have also added a multivariate classification dataset and verified the training loop is working fine for it.

Hey guys, @ToucheSir @darsnack @lorenzoh. I have updated the notebook. This PR essentially contains code to load regression datasets and the notebook for the classification task.
I will start adding the Inception Time Model in the next PR as we discussed during our weekly call.

Great progress in my absence and cool to see the training working!
Are the outstanding comments being addressed in #256 or could this be merged individually beforehand?

Great progress in my absence and cool to see the training working!
Are the outstanding comments being addressed in #256 or could this be merged individually beforehand?

I am resolving the outstanding comments in #256 , some of them have been resolved and a couple are left, which I will resolve by end of day. This can be merged then I believe.

I wasn't aware #256 was also working on the StackedRNN model. It would be preferable to merge those changes here so that PR stays more focused on the InceptionTime. If not, we can just close this one and focus on that one (I'm interpreting your comment to mean that the changes in #256 completely subsume this PR).

You'd likely have to rebase #256 after merging this PR if you've made changes to the same functions/types in that one. If that doesn't sound like a problem, we can merge this first.


Update QC report based on CITE-seq post-processing

Following changes made to incorporate CITE-seq post-processing into the pipeline, we'll need to update QC report.
qc_report.rmd

Update relevant phrasing throughout

It would be nice to be able to have logic for whether certain text appears, for example here: https://github.com/AlexsLemonade/scpca-nf/blob/6953ae72d0b8600b09bc8d50205260bb611a4e7e/templates/qc_report/qc_report.rmd#L473

The string for scpca_filter_method metadata has now changed, so think about that...
Some changes needs to happen to reflect that filtering in the _processed.rds object is filtered for both RNA and ADTs.
Update the basic_statistics table appropriately. Needs an if to check for CITE-seq and rows for more stats https://github.com/AlexsLemonade/scpca-nf/blob/6953ae72d0b8600b09bc8d50205260bb611a4e7e/templates/qc_report/qc_report.rmd#L233-L238
TBD: Should this plot only show RNA-based filtering? As written, it will include ADT-based filtering as well, which seems reasonable to me if there are ADT counts! Just need to update text (and maybe make the text conditional) https://github.com/AlexsLemonade/scpca-nf/blob/6953ae72d0b8600b09bc8d50205260bb611a4e7e/templates/qc_report/qc_report.rmd#L442-L450

cite_qc.rmd

Also select target type https://github.com/AlexsLemonade/scpca-nf/blob/6953ae72d0b8600b09bc8d50205260bb611a4e7e/templates/qc_report/cite_qc.rmd#L38

TBD: We end up creating this column if users didn't provide. Do we want to tweak the implementation to add a metadata indicator if this information was user-provided?

CC @allyhawkins @jashapiro , tagging (this is a pun) if you have thoughts on some of the TBD items above, or anything else! This might not be an exhaustive list of changes, but it's enough to get started and more will gel once I start getting going.

TBD: Should this plot only show RNA-based filtering? As written, it will include ADT-based filtering as well, which seems reasonable to me if there are ADT counts! Just need to update text (and maybe make the text conditional)

I think it is fine to show all of the filtering that was applied: I will note though that I think we can just use the values in filtered_sce rather than checking whether they are present from processed_sce? We do put those labels in both SCE files, correct?

TBD: We end up creating this column if users didn't provide. Do we want to tweak the implementation to add a metadata indicator if this information was user-provided?

I assume you mean the target column here: adding an indicator is fine, but I'm not sure it is needed: the reason to include this info is to indicate how it was used, which I think the column alone does cover.

I think it is fine to show all of the filtering that was applied: I will note though that I think we can just use the values in filtered_sce rather than checking whether they are present from processed_sce? We do put those labels in both SCE files, correct?

:100:, will clean this up!

Closed with #343


CANNOT_EDIT_HOSTSFILE

Microsoft PowerToys version
0.70.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Hosts File Editor
Steps to reproduce
I open the tool, but I can not add anything to the Hosts file.

system info:
版本	Windows 11 Professional Workstaition
版本	23H2
安装日期	2023/5/26
操作系统版本	25375.1
体验	Windows Feature Experience Pack 1000.25350.1000.0
[Some of the information is Chinese]
✔ Expected Behavior
Edit the HOSTS file with this tool.
❌ Actual Behavior
I can not edit the HOSTS file with this tool.
Other Software
No response

@Github-Luke-Zhang
Pleas try if it helps to switch the content of the first text input with the one in the second input.


Fitting an IRT model on the SQuAD dataset

Thanks for your responses to issue #9 @EntilZha and @jplalor! We saw the description underneath the from_jsonlines() function, but are still unsure of how to convert the SQuAD dataset into the format required for the function (since the function description asks for the dataset to be in a specific format and I'm not sure if extracting just QA pairs and their corresponding labels from the SQuAD dataset would be enough). Could one of you please do a small test run with maybe 5 rows from the SQuAD dataset as an example?

Thanks for reaching out! It would help to know about what you are aiming to accomplish, but here is one way to think about what input is required. Imagine a matrix where rows are subjects/systems/submission, columns are items/examples/questions, and entries are whether that particular subject got that item correct.
So doing this using only the squad questions + answers, doesn't quite make sense, you also need the scored predictions of at least several systems. The data below is the json lines version of the matrix I described, where each row is one squad system's scored predictions on the dev set (we can't release test data).
For SQuAD development data, you can find the py-irt compatible file here https://obj.umiacs.umd.edu/acl2021-leaderboard/data/squad-pyirt.jsonlines. You should be able to train the model with something like: py-irt train 4pl ~/Downloads/squad-pyirt.jsonlines /tmp/test-4pl/

Hi @EntilZha! Thanks for your response! That makes a lot of sense. The overall objective that we are trying to achieve is improving QA systems. Specifically, we are trying to use IRT to identify errors in training questions, correct those errors and retrain the system to ideally see better results :)
After reading your response, this is what I gather: From the scored predictions of several systems on the SQuAD data, we will be able to do some exploratory data analysis to see on which questions systems consistently fail to answer/answer incorrectly and hopefully try to find errors in questions from there?

Actually we all get the same problem when running  the py-irt train 4pl ~/Downloads/squad-pyirt.jsonlines /tmp/test-4pl/  command and the error is :'py-irt: command not found'
Do you have any idea why this happens? And could you please tell us how to fix it ?
Thank you very much

Are you installing with pip or from the GitHub master branch? The CLI isn’t on PyPI yet so for now have to install from master.

We tried installing with PyPi and running the command, but it didn't work (because of the reason above). So, we cloned the repo and ran the cli.py after much debugging. The good news is that the model fits well on the jsonlines data that you linked to, above and we are able to see results. The bad news is that the file structure is slightly haphazard and some files need to be moved from sub-directories to the parent directory in order for cli.py to run and the model to train on the SQuAD dev data. I'll link the other issue I open regarding the file structure and update the docs and submit a pull request by tonight. Thanks for the help @EntilZha!

FYI, you can also pip install from github branches directly like here https://adamj.eu/tech/2019/03/11/pip-install-from-a-git-repository/
Its hard to give suggestions about the issues you ran into, without knowing more about the errors. Generally, its really import to list these things:

What did you try to run (e.g., run py-irt)
What did you expect to happen
What actually happened
Context information, what version of python, from what directory, etc.

It might not be necessary to change the file structure, but I can't really advise without more info

Thanks for the response, @EntilZha! Apologies for not detailing the issue as outlined by you. I'll keep that in mind for future issues. To summarize this issue as per given outline:

What I tried to run: Tried installing py-irt directly from PyPi using the command pip install py-irt. Installation happened successfully, but when trying to run the command you provided: py-irt train 4pl ~/Downloads/squad-pyirt.jsonlines /tmp/test-4pl/, it threw an error saying bash: py-irt: command not found.
What I expected to happen: Expected the above command to run successfully, model training to start correctly, and epochs and training loss to show up without errors or other issues.
What actually happened:  bash: py-irt: command not found showed up with no other output. (Possibly due to what you said earlier with cli.py not being on PyPI yet)
Context: Trying to use the py-irt package to better understand training data from the NQ dataset to try and fix training errors with the end goal of improving QA systems as part of a research internship project with Dr. Jordan Boyd-Graber. Python version: 3.6.9. Created a directory and virtual env using venv for this project, did a pip install inside this directory.

When this did not work, we moved cli.py out of the /py_irt/ directory and into the main directory inside the repo, and ran this command in the main directory: python cli.py train 4pl ~/path/to/squad-pyirt.jsonlines /where/you/want/to/store/output_4pl. This made it work and the model works well with the following output:

Since it worked this way, for the benefit of future users, until cli.py is deployed to PyPI, I'm updating the docs and the file structure in this manner on my fork of the repo as a temporary fix to this issue. Should I open a new issue about the bash; py-irt; command not found error so that it may be addressed in detail there?
Please let me know your thoughts/suggestions/queries regarding the same! Thanks!

Re docs: I think the problem you ran into is installing from PyPI versus the master branch. I think we should:

Deploy a new version to PyPI, @jplalor has the credentials, so I can't do that. It should be relatively easy to configure poetry to do that and then run poetry build followed poetry publish after bumping version numbers.
Update the docs to indicate what versions the CLI is available on, the best thing would be a changelog indicating this
Add a FAQ to the docs and add this question there

The reason moving the cli.py works is that this is the source file with the CLI, but it is by installing with poetry install that makes it available as py-irt through the command specs in pyproject.toml.
If you could open an issue that can serve as a feature request for doc additions, that would make it easier to track docs we have yet to add

#11 takes care of 1 and 2 above, 3 is still todo


Icon not centering in Angular Material

When setting the size of an icon it was not being centered in an element when it should have been (using layout-align="center center"). Setting the icon size to "48" produced an icon with the correct size, but the parent ng-md-icon element still had css properties (height/width) set to 24px.
A potential solution is as follows around line 38:
                    // size
                    if (attr.size !== undefined) {
                        size = attr.size;
                    }
changes to:
                    // size
                    if (attr.size !== undefined) {
                        size = attr.size;
                        element.css('height', size + 'px');
                        element.css('width', size + 'px);
                    }
This inlined the css styles and fixed the issue.
Cheers
Simon

That's a good idea, @essesse.


fix: access To, Cc, Bcc elements via rows if no previous textarea selector

Something to be fixed - only works in English
Fixes #654

Sorry for the ultra-slow feedback, but now I've taken a basic look-over and think we have somethink to work on.
Let me know if you have any questions :)

Fixed all the comments also dropped "recipient_change_new" stuff - now it is in one single event for both new and old Gmail contact mode. Thankfully I have access now to both versions.

Fix should be available in new release<EMAIL_ADDRESS>Thanks @huksley !


use context size from model when loading last session

Describe the bug
llama.cpp cache/session file cannot be loaded due to small capacity.
I'm trying to use the new cache feature in order to speed-up CPU inference, however I cannot get it to work using full context.
The default context size is set to 512 in go-llama.cpp and it should be overridden by the model's context size, however that doesn't seem to be the case.
I have changed the C code to use the model's context size instead of using the parameters.
References

Original PR: #70
llama.cpp line no: https://github.com/ggerganov/llama.cpp/blob/7552ac586380f202b75b18aa216ecfefbd438d94/llama.cpp#L2844

Good catch, thanks!


Generate only requested platform directories on create

Description

For flutter create -t app, generate the directories/plugins only for the requested platforms.
For other commands that regenerate various files (plugins, create missing template files, etc), only do so in existing platform directories.
Replace the checkProject concept by explicitly requesting platforms based on user flags, existing directories, and whether features are on, as appropriate.

Related Issues
Fixes https://github.com/flutter/flutter/issues/67346
Fixes https://github.com/flutter/flutter/issues/60023
Related:
https://github.com/flutter/flutter/pull/31342
https://github.com/flutter/flutter/pull/59507
Tests
Added --platform app create tests.  Update existing plugin, project, cocoapods tests which already exercise these code paths.
Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

Why were we getting the warning about the android embedding though? even if the project wasn't there or was partially created, we should probably default to not showing a warning

Before this PR the android (and ios directories) were incorrectly created during the plugin registrant generation step.  After a flutter create the only thing in android was GeneratedPluginRegistrant.java and the only thing in ios was GeneratedPluginRegistrant.{h,m}.
https://github.com/flutter/flutter/blob/345188d40e576ffb6c97fee60446af58b8c9965a/packages/flutter_tools/lib/src/project.dart#L860-L862
is guarded by a check if the android directory exists (it did, but only contained GeneratedPluginRegistrant.java) and returned v1 because the app manifest didn't exist.
After this PR, the android directory is not created during the during the plugin registrant generation step, so the embedding version is never checked.


Support for list-type of backend.images in tasks.py _create_and_start_container() to fetch RepoTags

We're testing Nexus 3.2-hosted Docker registries (moving away from a legacy v0.6.8 registry), and found that while service image pulls worked correctly, service start failed with an error indicating a NoneType variable was being iterated.  I traced the code down to the following code in tasks.py (_create_and_start_container):
not filter(                     lambda i: self.container.image in i['RepoTags'],                     self.container.ship.backend.images(image['repository']))
The issue appears to be i['RepoTags'] would resolve to None and cause the exception.  When inspecting the data returned by self.container.ship.backend.images(image['repository']) was in fact a list of dicts (containing RepoTags).  In order to accommodate the list, I'm proposing to change the code to something like this:
pull_image = True
backend_images = self.container.ship.backend.images(image['repository'])
if isinstance(backend_images, list):
    for image_repo in backend_images:
        if image_repo['RepoTags']:
           if self.container.image in image_repo['RepoTags']:
               pull_image = False
else:
  pull_image = not filter(
                    lambda i: self.container.image in i['RepoTags'],
                    self.container.ship.backend.images(image['repository']))

if self._refresh or pull_image:
    PullTask(self.o, self.container, self._registries,
                         standalone=False).run()

With this change, service start appears to work with both legacy and new repositories that we've tested.

Oh I believe I just fixed this very recently, it's just not released: https://github.com/signalfx/maestro-ng/commit/6fdb9b4b93ffed7b0b3bd5205389bb8f9fbbb9e5. Can you test with master and check that the fix I have works in your situation?

Excellent -- this seems to work nicely!

Cool. I'll close this and do a release soon.


Update Final-Visualization.md

@12simu please pull from main branch and make PR

I made the pull request in main branch itself .


Unclear on how to upload files to Aspera Shares

The documentation (README.md and Manual.pdf) is a bit unclear on how I can use ascli to upload to Aspera Shares server.
I am trying using the method below:
$ ascli --insecure=yes shares files upload --to-folder='/! my-share-name' testfile.bin

WARNING: Ignoring certificate for: https://[REDACTED]:443. Do not deactivate certificate verification in production.
ERROR: Other(RuntimeError): no FASP installation found--=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=---=-- 0% 0 Mbps  ETA: ??:??:??
Please check manual on how to install FASP.
Use --log-level=debug to get more details.

The configuration is below
---
config:
  version: 4.19.0
aspera_shares:
  url: 'https://[REDATED]'
  username: 'username'
  password: 'password'
default:
  shares: aspera_shares

As I understand it, FASP is some other product from IBM.
Is it mandatory to purchase & install this product as well, in order to be able to upload files to Aspera Shares?
I am able to list the share, and use mkdir... but uploading seems to not work.
Any ideas?

install ther SDK:
ascli conf ascp install

as described in manual ...

Thank you for the very quick reply.
Will have to try that, but it seems that this URL is blocked by our firewall.
Is there any other place we can get the SDK from?

The next version will download using a different URL.
Beta is available here:
https://eudemo.asperademo.com/download/aspera-cli-4.20.0.pre.gem
you can get the list of SDKs for platforms here:
https://raw.githubusercontent.com/IBM/aspera-cli/refs/heads/develop/docs/sdk_location.yaml
or here:
https://developer.ibm.com/apis/catalog/aspera--aspera-transfer-sdk/downloads/downloads.json
Next version will use these.
To get the SDK for version 4.19 you can get here:
https://eudemo.asperademo.com/download/sdk-1.1.3-c6c7a2a.zip
If you can't download from where you are, then download somewhere else, and install from the file:
ascli conf ascp install --sdk-url=file:///sdk-1.1.3-c6c7a2a.zip

After some workarounds I was able to get the sdk and install it.
However to make our lives easier we have decided to go with the Docker image instead.
But we are facing the same issue both with docker image and "from scratch" installation.
I am able to list all the files, but I am unable to download or upload anything.
upload:
W, [2024-10-31T14:50:37.937741 #18]  WARN -- : An error occurred during transfer: Error establishing HTTP connection (check HTTP port and firewall)
E, [2024-10-31T14:50:37.937785 #18] ERROR -- : non-retryable error
E, [2024-10-31T14:50:37.937835 #18] ERROR -- : Transfer thread error: Aspera::Transfer::Error:
Error establishing HTTP connection (check HTTP port and firewall):
/usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/agent/direct.rb:246:in `start_transfer_with_args_env'
/usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/agent/direct.rb:368:in `block in transfer_thread_entry'
/usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/resumer.rb:45:in `block in execute_with_resume'
/usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/resumer.rb:41:in `loop'
/usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/resumer.rb:41:in `execute_with_resume'
/usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/agent/direct.rb:367:in `transfer_thread_entry'
/usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/agent/direct.rb:86:in `block in start_transfer'
D, [2024-10-31T14:50:37.937991 #18] DEBUG -- : EXIT (transfer)
D, [2024-10-31T14:50:37.938236 #18] DEBUG -- : all transfers joined
D, [2024-10-31T14:50:37.938418 #18] DEBUG -- : (value) get notify_to=
ERROR: Transfer: Error establishing HTTP connection (check HTTP port and firewall)
/usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/agent/direct.rb:246:in `start_transfer_with_args_env': Error establishing HTTP connection (check HTTP port and firewall) (Aspera::Transfer::Error)
  from /usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/agent/direct.rb:368:in `block in transfer_thread_entry'
  from /usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/resumer.rb:45:in `block in execute_with_resume'
  from /usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/resumer.rb:41:in `loop'
  from /usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/resumer.rb:41:in `execute_with_resume'
  from /usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/agent/direct.rb:367:in `transfer_thread_entry'
  from /usr/local/bundle/gems/aspera-cli-4.18.0/lib/aspera/agent/direct.rb:86:in `block in start_transfer'

For download it seems it is trying to connect to random localhost port everytime:
ascp: required management connection to port localhost:48467 failed.

Is port 33001 required for non-ssh transfers?

TCP and UDP ports 33001 are not required to download using HTTP fallback.
From the error you show, it sems that it falls back to HTTP download, but it does not manage to connect.
The random port 48467 is allocated (listened) by ascli and provided to ascp (option -M) so that ascp connects-back to that management port and then ascli gets transfer events on that channel. (that's ascp internal thing).
To troubleshoot your issue here, maybe check why ascp cannot connect to the server from within the container, something like : podman run -it container-name-here bash and then from inside you can either execute ascli, or ping or curl, etc... to see what's happening.


node went down when channels are pending force closure

Background
Unfortunately my node went down a couple of days ago, I was able to sweep most funds using an SCB, however before my node went down I had to two force closures in flight that were initiated by me, what would happen to these funds? The closure transactions hit a few confirmations before the node went down but they haven’t hit the maturity block yet. What would happen to the stuck as of now in those closures?
Your environment

version of lnd v0.13.1-beta
which operating system (uname -a on *Nix): Ubuntu
version of btcd, bitcoind, or other backend
any other relevant environment details: bitcoind pruned

Expected behaviour
Funds returning back to onchain wallet.
Actual behaviour
Unknown yet, funds still locked in the force closure.

Do you have an SCB from before you force closed the channels? Then you should be able to normally recover the funds by using that SCB.
Unfortunately lnd used to remove the channels too early from the SCB file (this was fixed with #5528 which will be in 0.14) when a local force close was issued. So if you don't have an older SCB you might need a more manual approach. I was planning on adding a new functionality for this to chantools, so please let me know if you are affected.

Do you have an SCB from before you force closed the channels? Then you should be able to normally recover the funds by using that SCB.
Unfortunately lnd used to remove the channels too early from the SCB file (this was fixed with #5528 which will be in 0.14) when a local force close was issued. So if you don't have an older SCB you might need a more manual approach. I was planning on adding a new functionality for this to chantools, so please let me know if you are affected.

I have an older SCB before the force closures, I ran that one and the channels show up in pending force closure, do I have to wait till the maturity block for the funds to return because I initiated the closure or will I be able to sweep the funds?
Also, adding that feature to chantools would be great.

Ah okay, that is good! Then lnd will sweep the funds as soon as the time lock expires. There is no way around the time lock, no.
I'm closing the issue as the initial problem was solved in #5528 and you'll get the funds back soon.
Feel free to comment again if something doesn't work as expected.


Add book versions

You can find them in the figma file marked with green borders
Designs:

Done!


React

This is a working PR of a few things:

Move from npm to yarn (hence the giant yarn.lock file)
Begin the process of moving all material-ui components to use google's implementation

Coverage remained the same at 99.794% when pulling 2d64d0b9add6b4fd142133556f63e4567ea5ac01 on react into 25c118dd0cf4e38e7458f51cfbf89682e67fc143 on rails.

Coverage remained the same at 99.794% when pulling 0a2ad7523d8ca14197f58eb880bf1a076acfc68a on react into 25c118dd0cf4e38e7458f51cfbf89682e67fc143 on rails.

Coverage remained the same at 99.794% when pulling 90ea9b3db6426ae2f7ada21b71796e22c07f356c on react into 25c118dd0cf4e38e7458f51cfbf89682e67fc143 on rails.

Coverage remained the same at 99.794% when pulling 90ea9b3db6426ae2f7ada21b71796e22c07f356c on react into 25c118dd0cf4e38e7458f51cfbf89682e67fc143 on rails.

Coverage remained the same at 99.794% when pulling 61210460ee8e5fd2697940e1097bbb8444a2ad07 on react into 25c118dd0cf4e38e7458f51cfbf89682e67fc143 on rails.

K. So a few things are happening here. In the spirit of maintainability, the react code is following a new convention. (requiredreading).
This app is no longer supporting Safari's 'home screen web app thing'. There's a significant performance decrease. I can't find anything more relevant than 2011 on the matter, so it's not worth it.


[FEEDSTOCK REQUEST] ollama

Describe the package you'd like added
ollama has become a popular easy-to-setup environment for LLMs: https://github.com/ollama/ollama
Describe how this package fits in with the project
GenAI/LLMs fit to core values in Open-CE.
Additional context
Required internally and by some customers. Llama.cpp (https://github.com/open-ce/open-ce/issues/1179) is related.

Moving to a later release.


removed navigation store's embedded property

Fixes

Fixes #2159 by @YashGaur000
Description
The code checks if the query parameter "embedded" exists and has a value of "true". If it does, it sets the "isEmbedded" property in the "navigation store" to true. But we needed to remove this code snippets,

Testing Instructions

Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Why is there a need to delete it? Can you help me with this, I'm new to open source.

Deleted frontend/src/plugins/url-change.ts and from the plugins array in nuxt.config.ts also.
*Why is there a need to delete it? Can you help me with this, I'm kinda new to open source.

Sure, @YashGaur000! I'm sorry I didn't give more details in the issue. The isEmbedded property is used in several places, and you've removed it from them. Previously, Openverse was hosted inside the wordpress.org iframe. Now, the old URL (https://wordpress.org/openverse) is only used for redirect to the new domain, https://openverse.org
frontend/src/plugins/url-change.ts is the file that was used to sync the URL between the iframe and the Openverse app. It is not needed anymore.
I've removed the unused tests, and now this PR is ready 🚀 Thank you for your contribution, and welcome to Openverse!

Thank you for your guidance and for making me understand the these issue needed to be altered. And Thank you for welcoming me in openverse.

Can you please tell me why this other review is pending? And will it even be reviewed?

Thank you so much to approve this pull request @obulat. Can I know when
this pull request be merged? And how do I become the contributor in
openverse?
On Tue, May 30, 2023 at 11:25 AM Olga Bulat @.***>
wrote:

@.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
https://github.com/WordPress/openverse/pull/2212#pullrequestreview-1450117733,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYYJGB6POEVFG4ZDCARAUDLXIWDTTANCNFSM6AAAAAAYR7EYRA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Can I know when this pull request be merged?

Our policy is that every PR needs 2 approvals to be merged, @YashGaur000.

When will the other one review it? Will she even review it?
On Tue, 30 May, 2023, 19:28 Olga Bulat, @.***> wrote:

Can I know when this pull request be merged?
Our policy is that every PR needs 2 approvals to be merged, @YashGaur000
https://github.com/YashGaur000.
—
Reply to this email directly, view it on GitHub
https://github.com/WordPress/openverse/pull/2212#issuecomment-1568484615,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYYJGB5GITMHDV4U6WBA6RLXIX4GVANCNFSM6AAAAAAYR7EYRA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Okay. Thank you to inform me. I appreciate that a lot.
On Fri, 2 Jun, 2023, 09:55 Dhruv Bhanushali, @.***>
wrote:

@.**** approved this pull request.
LGTM, thanks for the contribution @YashGaur000
https://github.com/YashGaur000! Please also note that you're free to
work on another issue while a PR is awaiting a second approval.
—
Reply to this email directly, view it on GitHub
https://github.com/WordPress/openverse/pull/2212#pullrequestreview-1456628264,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYYJGB7OELQIJCZQCZ3247DXJFTMXANCNFSM6AAAAAAYR7EYRA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Bug using plugin datatables.net-colreorder-bs4

I realized that when trying to change the column positions the error below appears:
Uncaught TypeError: Failed to execute 'insertBefore' on 'Node': parameter 1 is not of type 'Node'.

I looked for the bug in the Datatables community and found this discussion:
https://datatables.net/forums/discussion/59248/colreorder-failed-to-execute-insertbefore-on-node-parameter-1-is-not-of-type-node
Then I realized that there are actually two "tr" in the code generated by the component:

When I remove the void "tr", the error no longer happens.
I used this code below in the function @table-created="doCreated"
doCreated(comp) {
        this.$refs.table.$el
                .querySelector(".dataTables_scrollHead table thead tr")
                .remove()
}

BTW, thanks for developing this Vue wrapper component. Works great!

I notice the extra empty row in the thead too.  Seem to be bug in the base code when you define your own thead, which I feel iffy about patching within this component.  Since this component allow for defining your own thead, I will keep this issue open while I ponder more on if I should patch this.

Your example just got a lot better, tks!
I did a test with the code above, but it only worked by changing "querySelector" to "querySelectorAll"

Quite annoying. I noticed it as well at some point using datatables.net iwthout this Vue wrapper. Of course should report to datatables.net author.

Stale issue.


Git branches containing / not pollable

Hi,
i have the following issue: I have feature branches that follow the feature/bugfix/hotfix pattern e.g.
feature/bling-bling
bufgix/fix-something
when the multibranch plugin creates subjects for these the same branch names show up in the "branches to build".
The git plugin polling mechanism does not detect these branches. The branch should be fully qualified to be detected, e.g. "origin/feature/bling-bling".
This works for branches without / in the name, because the Jenkins git plugin adds wildcards to branches  if they do not contain slashes. A branch named "fix-something" would be resolved as "*/fix-something".
However, a branch named "feature/bling-bling" will be resolved as "feature/bling-bling" and will not be work (it should be "origin/feature/bling-bling").
I would suggest a solution but would like to get your input before I submit a PR:
Instead of specifying the plain branch name in "branches to build", specify the fully qualified branch name, including "origin/".
What do you think?
(There is an issue in the git plugin related to this, https://issues.jenkins-ci.org/browse/JENKINS-27332, and they will probably fix it, but IMO it is unwise to rely on these mysterious inner workings)
Regards,
Holger

@holger-s  I've found the same issue you pointed out. In my case, a branch release/0.1 is detected (a new job is created), but polling doesn't work for that branch in particular. If I create a new "standard" (not multibranch) job for any ( * ) branch, the polling works with all branches including release/XX ; so I guess it's not a git-plugin issue.
Did you finally solve the problem? I'd like to keep using the multibranch plugin, but I can't do it with this issue ... :/

we introduced a new rule, that strips the "/" out of the branches:
feature/new-stuff => f-new-stuff
very specific, but worked :)

Hehe I guess fixing the human part is a valid workaround. Thanks! :)


Incorporate OpenAI PoC into dnceng

Resolves dotnet/dnceng#347.
Incorporate @jlipscomb071's Proof of Concept into the dnceng repository.
Things to note:
The PoC uses .NET 7 (instead of the repo-standard .NET 6) to take advantage of a more mature Blazor. I limited the runtime use of .NET 7 to the PoC code. The rest of the project uses .NET 7 tooling but still targets the .NET 6 runtime.
I've added deployment steps to the pipeline. This only happens during main-branch builds as there is (currently) no "production" environment. Testing this was tricky but I believe I covered all new code in build #20230706.8.

This PoC has code of features that will not exist in the next PR. There's a sub directory inside of the "frontend" directory called "pages". Inside this sub directory, every razor page and C# file that's not the "Index" corresponds to a feature that existed in the original template of this code. The only feature that will exist in the final PoC version is the "Chat" feature.

I briefly went through the code out of my own curiosity and I think the code is of good quality. I appreciate the usage of modern technologies.
My only concern is whether we really need all this code and a new service for this purpose? How does this tie together with the latest Azure announcement that allows you to train ChatGPT with custom data directly from the Azure portal? https://learn.microsoft.com/en-us/azure/cognitive-services/openai/use-your-data-quickstart?tabs=command-line&pivots=programming-language-studio
Is this the same thing but a service is needed?

Hi Přemek,
It does appear the new Azure Service directly mirrors the functionality of this project. I plan to have some more conversations and do some comparisons to determine which approach would be better long term.

@jlipscomb071 looks like the tests are failing in CI. Are they passing on your local machine?

@jlipscomb071 looks like the tests are failing in CI. Are they passing on your local machine?

Should now be resolved.

Have there been any changes to the deployment bits since the test build on the 6th?

Have there been any changes to the deployment bits since the test build on the 6th?

No, no changes.

Closing for now. Will reopen when we sort out next steps.


Refer to sample filter project in the docs.

Is your feature request related to a problem? Please describe.
Potential Filter Authors could use a quick way into the code when they stumble on our github/site
Describe the solution you'd like

A link from the front github README.md to the sample filter project README
A link from the website docs to the sample filter project README (could be on the website frontpage https://kroxylicious.io/ or at https://kroxylicious.io/kroxylicious)

FYI: the front website repo is this one, and the https://kroxylicious.io/kroxylicious page is generated from here

A link from the website docs to the sample filter project README (could be on the website frontpage https://kroxylicious.io/ or at https://kroxylicious.io/kroxylicious)

How about in the docs under Custom Filters > Protocol Filters > An example protocol filter? Or perhaps we could have a new "Sample Filters" section under Custom Filters (instead of/in addition to the two empty sections for example filters), with a view to maybe having a sample net filter in kroxylicious-sample in the future too?


Rate function not available

It seems that the Rate function has been removed as it doesn't work throwing error that Rate not available

Rate function exists but it's not available since it was not released. Will do. Thanks!

@tahseenjamal its resolved now.
There is some error in the example will update it. You can try the following:
package main

import (
	"fmt"

	gofinancial "github.com/razorpay/go-financial"
	"github.com/razorpay/go-financial/enums/paymentperiod"
	"github.com/shopspring/decimal"
)

func main() {
	fv := decimal.NewFromFloat(-3000)
	pmt := decimal.NewFromFloat(100)
	pv := decimal.NewFromFloat(2000)
	when := paymentperiod.BEGINNING
	nper := int64(4)
	maxIter := int64(100)
	tolerance := decimal.NewFromFloat(1e-6)
	initialGuess := decimal.NewFromFloat(0.1)

	rate, err := gofinancial.Rate(pv, fv, pmt, nper, when, maxIter, tolerance, initialGuess)
	if err != nil {
		fmt.Printf("error",err.Error())
	} else {
		fmt.Printf("rate: %v ", rate)
	}
  fmt.Printf("done")
	// Output:
	// rate: 0.06106257989825202
}

@gyanesh-m Worked. Thanks
Actually, I asked because I have also written same function in C long back using Newton Raphson. But instead of simple initial guess, I use a derived computation value as initial guess that makes result accurate in lesser time and avoid any borderline cases, which can leave iteration in loop or incorrect answer


Document Insurance Eligibility Checks

Outline

Intro

Describe the purpose of doing an eligibility check
Describe the information required

Patient info
Patient Insurance Coverage (Coverage resource)

Link to Patient Insurance

Provider Info (in-network, out-of network)

Link to Provider Networks doc

Procedure / Service Provided + Diagnosis (should be on the Encounter)

Link to Operations Blog post, "Service Menu" section

Describe the process of eligibility checks, gathering information about request, sending to insurer / or service provider, receiving response

Creating a CoverageEligibilityRequest

Table with key elements (including recommended code system). CPT codes for productOrService

Sending a CoverageEligibilityRequest

Example of how to send a request (TBD)

Receiving a CoverageEligibilityResponse

Table with key elements

Supporting info

Opkit is a company that specializes in eligibility checks. See their marketing for example use case
FHIR Financial Resource Lifecycle - The "Relative Order of Use" would be valuable to turn into a Mermaid diagram

@ksmith94 let's remove the Opkit reference in favor of Candid Health


json/yaml highlighting in diff dialog

no highlighting now

one way is to do it through temp files (those would have extensions), but don't like it at all...


Ability to export/import input prices

As a user I wish to be able to share my calculator inputs and results with other people I am playing with.
To make this easier it would be nice to be able to export the locally stored cached data into a text file that I can send to other players.
Then,
As a user I wish to be able to take a text file and upload it to my version of eco-calc to receive the same local data as what was sent to me.
Possible problems/pitfalls:

Overwrite all current local data, or overwrite values that are set to 0, or a hybrid. Possible solutions include
a. Always overwrite all current local data
b. other options in between such as on import choose overwrite all vs overwrite 0s, vs overwrite overlap
c. Allow importer to pick and choose values to import

Hi, we try to make a colony with standardised price, and sharing all price is the key of sucess.
the import/export feature is just critical for us, we can't share the prices for advance ressource without hours of individual settings....
import export would allow to saves days of work, and make it so much easy for different play !
Thank you very much :)

Hi @Enderswood if you want to import/export right now you can copy/paste the values from F12 (dev console) Application -> Local Storage -> https://eco-calc.com

I am planning to add actual import/export when I have a chance but it will basically just be doing this.

Would be awesome, I will look at it, F12 is totally fine for me, but I know most of my fellow citizen would feel totally lost lol

I have implemented this feature with full override of all current data, except for locale.


Wrong IP address used in session manager

The incorrect IP address is retrieved by the session manager when the site is behind Cloudflare or another similar tool.
it doesn't use the http_x-forwarded-for server variable when it is available, instead always defaulting to the remote_addr server variable. This variable is used by various different services, and is a standardised response method.
The remote_addr variable is automatically calculated by the server, but when the server is only receiving requests from a third party rather than directly from the browser (as this is what Cloudflare does!) then the remote_addr variable is not correctly able to distinguish one user from another based solely on their ip addresses, meaning 1000 sessions may all appear to come from the same IP address but belong to different users, which would get very confusing very quickly.
My recommendation would be to add a simple if statement anywhere the IP address is used to check for the existence of the http_x-fowarded-for variable and then use it if it exists, or remote_addr if it doesn't.
There may be some services which this doesn't work for.

Duplicate of #4003.


"That API call is not allowed for the authenticated user"

I get this error while trying to access the API with your library, right after the user authenticates successfully and generates a request token:
"That API call is not allowed for the authenticated user"

Is there something that needs to be enabled on my personal Zerodha a/c to sort this out?

Please read the documentation carefully. We have mentioned that  publisher type apps are only for basket order execution and you won't get full api access. Please create a app of type Full kite connect to access all apis.

Can you point me to the documentation where this line is mentioned? The point of an API Doc is to state such things explicitly.
Also, how do you expect a developer who's simply testing an API to pay 2000 credits? Shouldn't the API come with the Kite account?

@gopi-ar Not sure I understand your question, but Publisher is free of cost. Kite Connect APIs have a monthly subscription fee. The actual API docs (obviously) only deal with technical and not pricing or business details.
But, the difference between various kinds of APIs are clearly explained on the Kite homepage https://kite.trade
Please refer to the various sections -- "Kite Connect for individuals", "Kite Connect for startups", and "Kite Publisher".
Underneath each section, the details are further broken down with concise explanations (how, what, why ..). There's also a "Are there any fees?" section, which is what you seem to be referring to.


Russian translation of Welcome and Basics

Basics is in progress (12 files translated, 7 files left)

Wow that looks great :)
Short remarks

source code length is supposed to be shorter than 48 chars (that's why Travis is failing)
you might want to add a short Readme file in the ru folder containing the commit hash of the latest translated version and maybe whom to ping for review of new contributions
Unfortunately I don't speak any Russian, but I think it would be good if someone who could have an eye on your nice work. Maybe @buggins?

@stonemaster anything from your side?
Keep going!

I agree with @wilzbach and of course a big thank you for your hard work!

Ok, I'll fix it. What 48 chars mean exactly: 4 spaces + 44 chars of code, or 4 spaces + 48 chars of code?
Ok
These D-stars know Russian also: @Dicebot, @CyberShadow, @9il, @DmitryOlshansky. Maybe somebody of them?

1.) That means the total number of chars aka the total column length. It is necessary that code still looks fine on mobile devices btw..

This is huge amount of work. Will try to proof-read as I time permits. Thanks @dmi7ry!

FYI I have submitted a PR to fix the "character" counting for Russian text: https://github.com/stonemaster/dlang-tour/pull/386

In order for the New Travis checks to be run, either @wilzbach or @stonemaster needs to manually rerun it, as Travis doesn't update when master changes.

In order for the new Travis checks to be run, either @wilzbach or @stonemaster needs to manually rerun it, as Travis doesn't update when master changes.

I think @dmi7ry should be able to rerun it himself? (triggered Travis)

@dmi7ry just rebase on master (upstream/master) and the travis build will
be started automatically.
Am 21.07.2016 18:33 schrieb "Sebastian Wilzbach"<EMAIL_ADDRESS>
In order for the new Travis checks to be run, either @wilzbach
https://github.com/wilzbach or @stonemaster
https://github.com/stonemaster needs to manually rerun it, as Travis
doesn't update when master changes.
I think @dmi7ry https://github.com/dmi7ry should be able to rerun it
himself? (triggered Travis)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stonemaster/dlang-tour/pull/384#issuecomment-234309800,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZUwpOYvQbv9lRLFvnZElmR616yavU9ks5qX59kgaJpZM4JPvdk
.

You can only rerun it manually if you're part of the repo. In order for him to rerun it he would have to add a new commit or rebase.

I finished Basics and changed formatting of code to 48 chars. But check agian failed. How do I know the exact place (file, line) and reason of the fail?

I think I found how I can see logs

Unable to open file public/content/ru/gems/index.yml for YAML loading: public/content/ru/gems/index.yml: No such file or directory

I think I found how I can see logs

Every section has an index file that gives the ordering of the files and contains semi-global information like the section title, see e.g.
https://github.com/stonemaster/dlang-tour/blob/master/public/content/en/gems/index.yml

Hi, @Dicebot. Thank you for the fixes. Yes, this is initial translation base. I'm not a professional translator, I study English (and sorry, I writing bad on English) and D and for me is difficult fully switch between languages when I do lot of translation. I want check these texts again in some time, then I'll do them more native (or someone with fresh look can do it).

There is one much more pressing issue though - I have noticed quite some imprecise or even plain wrong statements in original English text of the tour (I haven't read it before). Looks like it needs careful review and fixup on its own - I will try looking into it but that is non-trivial time investment.

Yes reviewers for the English content are highly wanted :)
In fact I don't know how many experienced D developers have already gone through the DLang Tour, but definitely too few.

Hi, @Dicebot. Thank you for the fixes. Yes, this is initial translation base. I'm not a professional translator, I study English (and sorry, I writing bad on English) and D and for me is difficult fully switch between languages when I do lot of translation. I want check these texts again in some time, then I'll do them more native (or someone with fresh look can do it).

@dmi7ry thanks a lot for your hard work & it would be a shame to see it being stalled here for so long.
How about you split it into a PR for every page, s.t. it's easier for you and your reviewers to keep track of pages that need further attention & we can already start to merge pages?
Also I think it's a lot easier to split the reviewing work if we have separate PRs.
I am not sure how familiar you are with git, but it's not that hard and even scriptable - I use the following to automate this for the German translation:
It's a bit opinionated as it assumes that you start from the master branch and have no commits of the to-be-added files in master, but it will create a new PR for each unstaged file with changes automatically.
#!/bin/bash

prefix="[german]"
files=$(git status --porcelain | cut -c4- | tr '\n' ' ')

# optional: you only need to add the upstream once
#git remote add upstream git@github.com:stonemaster/dlang-tour.git
# just in case the repo got a bit outdated
# git fetch && git fetch upstream && git merge --ff-only upstream/master

# just in case there are any staged files
git reset master

for filename in $files ; do
	# for simplicity skip index.yml
	if [ $(basename "$filename") == "index.yml" ] ; then
		continue
	fi
       # skip other files laying around - (e.g. the script itself)
	if ! grep -q "public\/content\/.*" <<< "$filename" ; then
		continue
	fi
	git checkout master
	echo "Adding: $filename"
	shortFileName=$(echo "$filename" | sed 's!public/content/!!')
	git checkout -b $shortFileName
	git add "$filename"
	commitMessage="${prefix} ${shortFileName}"
	git commit -m "$commitMessage"
	git push --set-upstream origin $shortFileName

       # in case you don't have hub, comment and submit the PR via the webinterface
	hub pull-request -m "$commitMessage"

	# if you want to execute it step by step in the beginning
	exit
done

Hi, @wilzbach. I'm on Windows, so can't use your way. But I understand. Ok, will be done, but not very soon. Right now I'm very busy and can't do anything within current month.

Hi, @wilzbach. I'm on Windows, so can't use your way. But I understand. Ok, will be done, but not very soon. Right now I'm very busy and can't do anything within current month.

Ok, I hope you find time soon. I will close this PR as you now can make new PRs against the Russian language repository:
https://github.com/dlang-tour/russian


docs: Document new features and config changes for 2.0.0

Closes #928 .
Will have to make sure for 3.0.0 we update this document in every PR. I kept track of config changes in #928 and just went through the PRs now to find all the new features. Hope I didn't miss anything.
Biggest part of the release notes is trying to explain how this can break configs.
The other 2.0.0 PRs that are still open are internal code changes so have no impact on this.

In the process of releasing it now 😃
New features are definitely welcome, but will be for v2.1 or v3 (depending on whether they involve breaking API changes).
Note that such release can happen relatively fast, so I don't expect much delay once these are finished.


show author time insteadof commit time?

Suppose I have a feature branch and have some old commits, then I use git rebase develop to rebase on develop branch. These old commits should not be listed when I git standup.

Thank you for opening the issue, you can use -R flag which will show the committer date instead of author date
git standup -R


Master

Purpose

...

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

What to Check
Verify that the following are valid

...

Other Information

Looks like credentials are in web.config. The one new sample appears to be a duplicate of an existing sample. I'm in the process of a major update to all samples, so I will not be merging this PR in at this time.

Looks like credentials are in web.config. The one new sample appears to be a duplicate of an existing sample. I'm in the process of a major update to all samples, so I will not be merging this PR in at this time.


Suggest Including S3 Bucket Policy Requirements

Took me quite a bit of tinkering to figure out the correct policies for the S3 Bucket User. I might even be over granting still. The biggest hang up I was having was PutObectAcl.  The errors messages didn't really describe what was missing, so I had to tinker around and check the code. Also might want to mention that you have to turn Static Website hosting on. (common sense, but it could help somebody.)
Here is my current policy, but I think it might more permissive than needed.
"Effect": "Allow",
"Action": [
"s3:AbortMultipartUpload",
"s3:DeleteObject",
"s3:DeleteObjectVersion",
"s3:ListBucket",
"s3:PutObject",
"s3:PutObjectAcl",
"s3:GetObject",
"s3:GetObjectVersion"
],

s3:PutObjectAcl should be the only policy required as far as I know. You're right we should note this somewhere.

Thanks for the hint! The Static Website part was not obvious to me.

Strange, I forgot to mention it above but "Static Website" should not need to be enabled for Lookerbot to work.


bdd list

Codecov Report

Merging #96 (244244c) into main (f960161) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #96   +/-   ##
=======================================
  Coverage   95.71%   95.71%
=======================================
  Files           5        5
  Lines         257      257
=======================================
  Hits          246      246
  Misses         11       11

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 f960161...244244c. Read the comment docs.


NameError: name 'v' is not defined

I get this error when running the script:
Traceback (most recent call last):
File "antivmdetect.py", line 50, in
dmi_info['DmiBIOSReleaseDate'] =  "string:" + v['Relase Date']
NameError: name 'v' is not defined

Hi @Mart1neZ

Lets try this:

Do you get a "Release date" in the output from the following command; sudo dmidecode -t0 ?

Also:

Are you running version 1.9?
Can you send the output of: pip3 list | grep py-dmidecode ?

I think this is a good start
/Mikael

@Mart1neZ Did you get a chance to try the above?

Yeap, that's what you get when you don't specify version of library you depend in requirements.txt. Things will break when your library decided to push breaking changes.
import dmidecode

should changed to this:
from dmidecode import DMIDecode
dmidecode = DMIDecode()

I'll be soon open a pull request with this issue fixed once I finished testing it.