ReferenceError: process is not defined

Steps followed:
$ git clone https://github.com/beachio/chisel-parse-server-starter.git
$ cd parse-server-starter
$ npm install
$ npm start
$ npm install -g chisel-cms
$ chisel-cms --appId "SampleAppId" --serverURL "http://localhost:1337/parse"
Accessing http://localhost:9000 gives the following error in the browser console.
Uncaught (in promise) ReferenceError: process is not defined
at main.js:2
at c (main.js:2)
at Generator._invoke (main.js:2)
at Generator.next (main.js:2)
at qo (main.js:2)
at a (main.js:2)
at main.js:2
at new Promise ()
at main.js:2
at Qo (main.js:2)
Verified http://localhost:1337/parse and http://localhost:1337/dashboard are working fine.

It works when I clone the repo, build and run.
Instead of using NPM.


fix(Moddle): replicate set/get API for createAny elements

related to https://github.com/camunda/camunda-modeler/issues/3589

Thanks for the review, I'll take care of the patch release


SWP-95814 Create codeship build

Setting up CI for Splash fork of token-bucket library.  More details in the ticket
Dev Testing
CI passes

Why are we creating a separate repository as opposed to adding the code to toolbox?

Why are we creating a separate repository as opposed to adding the code to toolbox?

@chaitany6143 as we discussed separately, it's because I need to customize this (no longer supported) library to add support for 10/20 second rate increments (see #2 )

No configuration for php8?

I created a PHP8 config, but in my local testing, I was getting failures (when using Jet CLI to run the codeship steps).  I'll create a follow-up branch with PHP 8 build, but I may need some help getting that working.


Unable to add body/message in actions

Sentinl and Kibana version
6.2.3

Issue description
When configuring actions in reporters, the Body or Message field is gone, there is no where to input a message.

Reproducing the issue step-by-step
Create new reporter, try to add any action, body/message field is not there.

Expected results vs. actual results

Sample of data

Additional details

Duplicate of https://github.com/sirensolutions/sentinl/issues/473. The fix for this will be available today.

The fix is available in the release packages https://github.com/sirensolutions/sentinl/releases/tag/tag-6.2.3-3


Add max_ms parameter to parse function to limit execution time

Solves #4.
I was thinking it's also possible to change the code of configuration.rs to the following:
#[must_use]
	pub fn parse<'a>(&self, wiki_text: &'a str) -> crate::Output<'a> {
		crate::parse::parse(self, wiki_text, 0).unwrap()
	}

	#[must_use]
	pub fn parse_with_timeout<'a>(&self, wiki_text: &'a str, max_ms: u128) -> Result<crate::Output<'a>, &'static str> {
		crate::parse::parse(self, wiki_text, max_ms)
	}

and then modify the code I put in parse.rs so that if the max_ms parameter is 0, no limit is applied. That would mean that people who don't need to use the timeout code won't be affected if it's removed in future. I'm happy to implement that in this fork if it seems like a good idea.

Thanks for the PR.
I think in almost all cases the execution time should be limited. Maybe there is a good default that could be used in the parse function. Since you parsed a lot of wikimedia text, do you know the average execution time?
Then we could have two functions customizing this behaviour: parse_with_timeoutand parse_without_timeout.
I think it might be useful to have the parsed nodes and warnings still available even if the timeout occurred.
Either there could be a property in crate::Output something like errors or the return type is like Result<Output, Error> with the error being something like enum Error { TimedOut { execution_time: Duration, output: Output } }.
Instead of u128 use std::duration::Duration that seems more appropriate.
What do you think?

That all sounds good to me. I'll add some quick and dirty code to my Wikipedia dump parser to output an average parsing time to a CSV file.

I added those suggestions minus a few things. I'm not good enough with Rust to understand the lifetimes with Output, so I didn't include it in the return value. If anyone knows the correct way to do so, then I'll definitely add it.
In addition, I'm yet to have a figure for the average time to parse an article, but I stuck with 5 seconds as a solid default. Once I have a good figure for it, I'll update the default.

Thanks


Potential issue with multiple Ticks in example 8?

(First of all, I can't seem to be able to run example 8 since I've updated to 0.16, so had to disable DanDanDan/Easing).
In any case, I am looking at update function and to me it seems that there's is a potential problem with how Spin action is checked. And if state is Nothing, a Tick action is sent, but model is not updated. Is it possible that multiple Spins arrive before the first Tick?
I've tried to simulate this by sending Spin' from onClick, which in turn sends 2 Spins:
update msg model =
  case msg of
    Spin' ->
      ( model, Effects.batch [Effects.task (Task.succeed Spin), Effects.task (Task.succeed Spin)] )
    ...

Now each square turns all 180 degrees. Should not there be a special PreSpin state? Or maybe we can get current time instantaneously somehow?

Having subscriptions in 0.17 should make it much harder to have "simultaneous" animations going.


Compatible new field in write/lock cf with TiKV

What problem does this PR solve?
Issue Number: close #7212
Problem Summary:
Compatible new field(TXN_SOURCE_PREFIX,LAST_CHANGE_PREFIX)  in write/lock cf with TiKV.
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/cc @CalvinNeo @JaySon-Huang

/run-all-tests

/merge

/run-integration-test


add playlist folders support

Resolves #453

[!NOTE]
Spotify doesn't have an API to retrieve or modify playlist folders. In order to retrieve a folder structure a third party utility is required mikez/spotify-folders. Using this utility you need to create a compatible json file and put it in ~/.cache/spotify-player/PlaylistFolders_cache.json. You'll need to update this file every time you change your folder structure.

Demo
https://github.com/user-attachments/assets/7cbf8029-f9a8-43f7-b2c8-37b169bab459
Configuration
Follow the mikez/spotify-folders installation instructions. Then run
spotifyfolders > ~/.cache/spotify-player/PlaylistFolders_cache.json

It generates a full playlist folders structure and stores to a json file, like this:
{
  "type": "folder",
  "children": [
    {
      "type": "playlist",
      "uri": "spotify:playlist:00000000000000000000"
    },
    {
      "name": "Folder name",
      "type": "folder",
      "uri": "spotify:user:aaaaaaaaaaa:folder:bbbbbbbbb",
      "children": [
        {
          "type": "playlist",
          "uri": "spotify:playlist:111111111111111111111"
        },
        {
          "type": "playlist",
          "uri": "spotify:playlist:222222222222222222222"
        }
      ]
    }
  ]
}

How it works

spotifyfolders generates a json file with folder structure:

{
  "type": "folder", // root node is always a folder
  "children": [
    {
      "type": "playlist", // playlist type (contains uri)
      "uri": "spotify:playlist:00000000000000000000"
    },
    {
      "name": "Folder",
      "type": "folder", // folder type (contains name, uri and 0..N children)
      "uri": "spotify:user:aaaaaaaaaaa:folder:bbbbbbbbb",
      "children": [ /*...*/ ]
    }
  ]
}

Imagine we have a playlist folder structure like this:

playlist: Playlist 1
folder: Folder 1
    playlist: Playlist 1.1
    folder: Folder 1A
        playlist: Playlist 1A1
    playlist: Playlist 1.2
    folder: Folder 1B
        playlist: Playlist 1B1
        playlist: Playlist 1B2
folder: Folder 2
    playlist: Playlist 2.1
    playlist: Playlist 2.2

To represent it in a flat view like we have now, we need to introduce the following fields:

is_folder: bool — false for playlist, true for folder
current_level: usize — what playlist or folder it belongs to
target_level: usize — for folders: what level the folder points to. Unique for each folder.

3.1. Why must the target level be unique?

If the PlaylistFolders_cache doesn't contain some playlists, they'll go to a root folder level: (0, 0). Same for newly created playlists.

How it works

spotifyfolders generates a json file with folder structure:

{
  "type": "folder", // root node is always a folder
  "children": [
    {
      "type": "playlist", // playlist type (contains uri)
      "uri": "spotify:playlist:00000000000000000000"
    },
    {
      "name": "Folder",
      "type": "folder", // folder type (contains name, uri and 0..N children)
      "uri": "spotify:user:aaaaaaaaaaa:folder:bbbbbbbbb",
      "children": [ /*...*/ ]
    }
  ]
}

Imagine we have a playlist folder structure like this:

playlist: Playlist 1
folder: Folder 1
    playlist: Playlist 1.1
    folder: Folder 1A
        playlist: Playlist 1A1
    playlist: Playlist 1.2
    folder: Folder 1B
        playlist: Playlist 1B1
        playlist: Playlist 1B2
folder: Folder 2
    playlist: Playlist 2.1
    playlist: Playlist 2.2

To represent it in a flat view like we have now, we need to introduce the following fields:

is_folder: bool — false for playlist, true for folder
current_level: usize — what level the playlist or the folder belongs to
target_level: usize — for folders: what level the folder points to. Unique for each folder.

3.1. Why must the target level be unique?

If the PlaylistFolders_cache doesn't contain some playlists, they'll go to a root folder level: (0, 0).
https://github.com/aome510/spotify-player/blob/e61b6cfbd17bc9179d91110efb3d46c8c8bb78b6/spotify_player/src/playlist_folders.rs#L12-L21
Same for newly created playlists.

A playlist folder uri is not supported by the application at this time, but it's needed for proper playlist folder structuring. Moreover, I prepend the id with f for folders and u for up node, so folder's uri shouldn't be used in the API calls:

https://github.com/aome510/spotify-player/blob/e61b6cfbd17bc9179d91110efb3d46c8c8bb78b6/spotify_player/src/playlist_folders.rs#L54-L58
https://github.com/aome510/spotify-player/blob/e61b6cfbd17bc9179d91110efb3d46c8c8bb78b6/spotify_player/src/playlist_folders.rs#L65-L70

Thanks for the detailed explanation. I get the idea now.
Using level is confusing and level is not meant to be unique. (current_level, target_level) should be replaced (current_id, target_id) and should include a comment that a folder points to another folder represented by target_id.
In addition, I don't think it's a good idea to modify playlist to support representing folder because playlist and folder are separate identities. My suggestion is to define a separate struct for folder and another enum PlaylistFolderItem to represent either a folder or a playlist for use in action/UI state. IMHO, separating the data representation of playlist and folder also improves the code clarity and simplifies the implementation.

@aome510 I for me current_id is misleading because we also have a Spotify id "spotify:user:xxx:folder:yyy".  What about current_folder_id / target_folder_id and comment that it's a local id, not just a global Spotify id? Or current_folder_index / target_folder_index?
Separate struct is okay, will do so.

@aome510 for me current_id is misleading because we also have a Spotify id "spotify:user:xxx:folder:yyy". What about current_folder_id / target_folder_id and comment that it's a local id, not just a global Spotify id? Or current_folder_index / target_folder_index?

That also works. I mean Spotify ID doesn't really apply to folder, so we don't really need to differentiate here. Comments/documentations will definitely make it more clear though.

@aome510 done

@aNNiMON sorry for the delay (I've been kinda busy lately). I've reviewed the PR again and pushed a commit to simplify the codes and processing logic further. Can you review the comment and test my changes to ensure that nothing changes in terms of functionalities? I don't have folders so couldn't really test it myself.

@aome510, everything works fine. 👍  Thanks


Unused variable warning CS0219 is not given if the variables are defined using tuple deconstruction

Describe the bug
"Unused variable" warning CS0219 is not given if the unused variables are defined using tuple deconstruction.
To Reproduce
In the following code snippet
let a = 1;
let (b, c) = (2, 3);

I get the warning for variable a but not for b and c.
Expected behavior
The warning shows for all unused variables regardless of how they are defined.
System information

0.16.2104138035

Given that this is a C# error it should actually not show up at all (in VS Code I think it might since it would depend on the C# extension there). The Q# compiler at this time does not yet generate warnings for unused variables at all.

Transferring to runtime, since we should suppress all C# warnings.


test: upload local version of federation controller to kind clusters

set the HUB and TAG variables when running kind_provisioner.sh and use the variables where kind load is run

I do not see a use case to customize name for the test image. I intentionally use TAG=test, because when tag is latest and the imagePullPolicy is not set explicitly, then Kubernetes always pulls the image. But if we will change it permanently or someone will customize the pull policy to Always, we would not notice that we do not use the local version of the controller.

set the HUB and TAG variables when running go test and continue to use them in the helm upgrade command

I restored using HUB and TAG in the test. I forgot that I intentionally used --set for these options.

remove image.tag.test from federation_controller.yaml

Done.

delete build-test-image target and instead use docker-build

I think the first reply answers this point.

I intentionally use TAG=test, because when tag is latest and the imagePullPolicy is not set explicitly, then Kubernetes always pulls the image. Our default is "IfNotPresent", but if we will change it in the future or someone will customize the pull policy to Always, we would not notice that we do not use the local version of the controller.

Ah, TIL images with the latest tag were treated like this! Thanks! My comment was more about having a single source for the HUB and TAG values that gets passed appropriately to the script/test. I can now see the benefit of hard-coding the TAG variable, and I suppose there's no real need to customise the HUB variable since this will always be a local image.


Release to foxy

Feature request
Release to foxy
Description
We would like to use it as an installable package in foxy. Guess it would still be useful to many?

Implementation considerations

Alternatives
Users have it to integrate it on their own for foxy.

Additional information

You can find eloquent and foxy releases on this package in the rmf package repository:
http://rmf.servehttp.com
Instructions on how to use it are documented in the book here:
https://osrf.github.io/ros2multirobotbook/#setup-sources-and-installation-of-rmf
However, since we are working on the process of releasing open-rmf packages for galactic into the official ros repos, it might make sense to do it for foxy too. I will leave this open until a decision is made on that direction.

Thx. I think it is already released to ros2 for galactic and rolling (at least the packages are there and the gbp repos exist). At least that is why we noticed it as some package uses it already and fails on foxy of course.

This should be already released into foxy:
https://build.ros2.org/view/Fbin_uF64/job/Fbin_uF64__ament_cmake_catch2__ubuntu_focal_amd64__binary/
Will proceed and close the issue.

Thx. Just checked it now appears in rosdep for foxy. The package itself seems to be not yet released to the official apt repo and probably will take until the next update cycle.


Error: PrismaClient is unable to be run in the browser.

Hello all,
I am having the same issue, and not running PrismaClient on the frontend. I have a file in the /api/file, and when I tried making a POST request I got this error:
Error: PrismaClient is unable to be run in the browser.
In case this error is unexpected for you, please report it in https://github.com/prisma/prisma/issues

I also got this message in the console:
error - node_modules/.prisma/client/index-browser.js (169:0) @ <unknown>
error - PrismaClient is unable to be run in the browser.
In case this error is unexpected for you, please report it in https://github.com/prisma/prisma/issues
null

Is there a solution to this issue?

Originally posted by @PublicTrades in https://github.com/prisma/prisma/issues/19500#issuecomment-1605283701

Duplicate of https://github.com/prisma/prisma/issues/19500#issuecomment-1605283701

@janpio - Don't close my issue unless you have an answer for me. First off, my issue is not a duplicate of #19500 - Issue with that was PrismaClient was NOT setup on the server side - a simple google would provide them an answer to resolve that issue. I have it set up on the server side, and was still throwing error. I couldn't find any way to resolve it so, I'd appreciate you leave this ticket the way it is so I can get folks who can help point me in the right direction, it is NOT  duplicate of that other one. Thank you.
P.S. If you gonna do shit like that at least provide your two cents, Just don't hop on the machine closing ticket left and right.

Then don't post the exact same message multiple times, especially not by creating an issue by just putting the comment into it. Doing so means that different people will see your message in multiple places, and potentially do double the work.
If you think your problem is not related to #19500, then please delete your comment there and open a new issue and fill in the full bug issue template and provide all the information it asks for. Otherwise we will not be able to help you.
Also: Please watch your tone. Your message here is inappropriate and we will not accept that around here. If you want help, be nice and follow the rules. Thank you.

Little @janpio got a little sensitive huh ‘Mr. Prisma’, couldn’t take punch - So what I thought the other thread’s was  directly related to my issue.
Have at it if it makes you feel better, close the issue, better yet, shut down GitHub.


Resource update function typed wrong in Flow

It accepts a function (data) => resourceData in stead of resourceData directly

Closed in #54


Bugfix mutable default arguments

this PR fixes up the same bug (mutable default arguments) scattered around the codebase.
Additionally, it changes the Travis config to execute a search for mutable default arguments in the code. If it finds one, fails the run.
Please, do not squash the PR, it's been rebased already and I want to keep the bugfix and the Travis config change in separate commits. Thanks!

This PR also seems to run and makes sense to me. Will merge without squashing. I do have a question though: What does code like this do?
species: List[Union[Species,str]]
This kind of python is new to me.

This PR also seems to run and makes sense to me. Will merge without squashing. I do have a question though: What does code like this do?
species: List[Union[Species,str]]
These are python type hints. Right now, it doesn't do anything. (if you use VScode or Pycharm, then it show potential errors based on the type hints. Example:

This kind of python is new to me. @zoltuz
I'm adding these in gradually, feel free to ignore them for now.

There were merge issues. To fix them, I this PR into my dev branch, got it working locally, then pushed dev.


fix(format): double percentage '%' for nvim components

relate: #98

[ ] test on lua-language-server
[ ] test on rust-analyzer
[ ] test on ruby-lsp


Console app to instantaneously read midi commands via fader

Hello, I'd like to create a console app that instantaneously reads commands from a midi device. This device is not musical, it only sends note on/off or fader control changes. Is it even possible for this framework to handle fader events? Thanks!

Hi, have a try of the MIDI in demo in the NAudio demo app and see if that meets your needs.

I created my own Windows form app that reads input from an Akai APC20 and sends to my lighting program, Daslight. This lighting program can accept input from any midi device, however, it did not behave how I wanted it to. I even purchased Bome Midi Translator Pro and after many months of experimenting and using live at shows - this translator was too difficult to use and configure. Anyways, if you or anyone sees this and is interested in the Windows form app that translates MIDI, let me know.


Add Generic Datatypes

Is your feature request related to a problem? Please describe.
Currently I'm implementing some device with a magnetic sensor. But since there is no data type for magnetic flux or something similar, I would need at least a workaround, but there is also not a generic data type for non-binary values.
Describe the solution you'd like
I would like to have a set of generic data types, someone can use according to their needs. These are:

unsinged: uint8_t, uint16_t uint32_t and uint64_t
singed: sint8_t, sint16_t, sint32_t and sint64_t
floating point: float (4-Byte), double (8-Byte)

Additional context
I also realized, that some datatypes are defined as uintX, like voltage, but voltage can also go negative...

The raw sensor should be able to receive any number as a hexadecimal coded string. It doesn't say if there should be a leading "0x" in the string though.

The count sensors are available in uint8, uint16 and uint32, which you can use for any (positive number). The raw sensor is a special case, where you have to specify the length of the bytestring first (in the 2nd byte). However, the raw sensor is intended to send a bytestring as is, there won't be any translation to a number in e.g. Home Assistant. But the count sensor should be ok for the unsigned inters at least.
Feel free to add a specific type if you want, by creating a PR.

I understand, that there is a raw sensor, but this does still not address the issue, that some of the sensor values, currently defined only positive, can often go negative (voltage, power, current...). Is it intended to be solved in the future or does someone need to always work around it and introduce quite some overhead on the receiving side?

We can add a count sensors that can go negative (signed int), to at least solve it with a generic sensor type. It is also possible to add some singed int sensors for specific types. But I don't think negative voltage is used very often.

A generic signed int sensor would already serve well. You should also think about integrating a signed power sensor. Thinking about households with solar power and battery storage, a power sensor that measures positive and negative power is needed in every place (main meter, battery sensor,...) and it is totally necessary, if you want to control the power generation in a way, that you get net zero for the main net meter.
Thanks for taking it into account!

Added in 3.10.0


Unsub xpath

This is a great refactor for simplifying the code I've been working on. I also figured out that we were doing some weird things with text field value attribute vs property.

Pull Request Test Coverage Report for Build 1829

168 of 168 (100.0%)  changed or added relevant lines in 8 files are covered.
13 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.05%) to 97.812%

Files with Coverage Reduction
New Missed Lines
%

lib/watir/locators/element/locator.rb
1
99.26%

lib/watir/capabilities.rb
12
84.44%

Totals

Change from base Build 1827:
0.05%

Covered Lines:
2235

Relevant Lines:
2285

💛  - Coveralls

Pull Request Test Coverage Report for Build 1829

168 of 168 (100.0%)  changed or added relevant lines in 8 files are covered.
13 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.05%) to 97.812%

Files with Coverage Reduction
New Missed Lines
%

lib/watir/locators/element/locator.rb
1
99.26%

lib/watir/capabilities.rb
12
84.44%

Totals

Change from base Build 1827:
0.05%

Covered Lines:
2235

Relevant Lines:
2285

💛  - Coveralls

Pull Request Test Coverage Report for Build 1829

168 of 168 (100.0%)  changed or added relevant lines in 8 files are covered.
13 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.05%) to 97.812%

Files with Coverage Reduction
New Missed Lines
%

lib/watir/locators/element/locator.rb
1
99.26%

lib/watir/capabilities.rb
12
84.44%

Totals

Change from base Build 1827:
0.05%

Covered Lines:
2235

Relevant Lines:
2285

💛  - Coveralls


Add --upgrade-package from pip-compile

pip-compile's "--upgrade-package" option allows upgrading only specific package(s), either to the newest version or to a specific version. This feature would also be useful in pip-compile-multi.

Hi Jonathan. Thanks for pointing to that option. Currently here is what I do to upgrade some specific package(s):
# Remove a package and all its direct dependencies from all lock files:
sed -i '/applog/d' requirements/*.txt
# Recompile lock files without upgrading remaining packages:
pip-compile-multi --no-upgrade

(SO answer for sed syntax)
But I see the benefits in supporting this option. @kylegibson requested the similar thing in #109.
How do you feel about opening a PR with this feature?

I have opened a pull request. Several tests fail, complaining about package versions not matching. I don't think those failures are related to my changes.

That’s very cool, thanks! I’ll take a look after Monday.
On Fri, May 24, 2019 at 6:57 PM Jonathan Rogers<EMAIL_ADDRESS>wrote:

I have opened a pull request. Several tests fail, complaining about
package versions not matching. I don't think those failures are related to
my changes.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/peterdemin/pip-compile-multi/issues/139?email_source=notifications&email_token=AALBVWSQ3NKPXQDMYEA5HXLPXBXG7A5CNFSM4HMVA2L2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWGXEUQ#issuecomment-495809106,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALBVWTLEMM6W2HPN6PW3LLPXBXG7ANCNFSM4HMVA2LQ
.

--
С уважением, Пётр Дёмин.


IndexError: string index out of range

Hi,
Thanks a lot for your program.
I have a fasta file which is giving the above error at the line 542 of the code.
Traceback (most recent call last):
File "", line 1, in
File "/home/pkokate/miniconda3/lib/python3.7/site-packages/dinuq/sdu.py", line 542, in SDU
dint = str(sequ[d] + sequ[d+1])
IndexError: string index out of range
melanogaster.txt
Thanks for your help.

This error was caused by having stop codons or non-coding sequences in the fasta file provided. DinuQ v1.1.1 now has more thorough troubleshooting messages for identifying such problems in the dataset.

Hi,
I hope you are well.
Thanks for the update. I might be using it in the future.
Thank you for your help.
with best regards
yours sincerely,
Prajakta
On Fri, 30 Oct 2020, 05:36 spyros-lytras<EMAIL_ADDRESS>wrote:

This error was caused by having stop codons or non-coding sequences in the
fasta file provided. DinuQ v1.1.1 now has more thorough troubleshooting
messages for identifying such problems in the dataset.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spyros-lytras/dinuq/issues/5#issuecomment-719448077,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANCSX2IGARB27Q4O2VTSETDSNKCK3ANCNFSM4OYYAETA
.


put new testing rewrite behind an experiment

adds in experiment for ramp of the new testing rewrite

@luabud and @cwebster-99 does this look correct for adding in the experiment? I want to make sure I am using the correct experiment name. After this is merged what are the next steps?

it looks right to me! once it's merged (or even before that when we have a vsix that we can test out), we can manually test to see if it's working as we expect 😊


Leverage JSON schema as the source of truth for the structure of your tags

Analyze incompliances, e.g. env = [dev, qa, prod] but a tag is used with development.
Go into a mode where you can set tags using the JSON schema, e.g. single-select could use the Cliffy selector (suggestions thingy) to select the right values.

Closed due to lack of community interest - feel free to revive discussion at https://github.com/meshcloud/collie-cli/discussions


Unit Testing Action Workflow

Hi all, as we need to create two unit testing (one for calculating the actual frequency and another for the peak frequency) py files, Can we do these two unit testings in one single action workflow file or add two separate unit testing action workflow files?

@Mosaddeq107 I think, we need to do them separately. BTW can you share what values you got for the actual and peak frequencies?

Yeah, we need to do those separately, I think. @Mosaddeq107 which part are you going to do?


[IMP] pylint-odoo: upstream pre-commit hook

Updating as requested in https://github.com/OCA/pylint-odoo/issues/360#issuecomment-1217528607.
cc @moylop260
Please bear in mind that this could potentially put some stable branches (14+) in red. It shouldn't do it, because config is not changed. But it could happen.

@sbidoul this is needed for being compatible with Python 3.10, which is the default on Ubuntu 22.04. I think we should deploy it on at least v14 and v15, being that ones the branches under support when 16.0 arises.

OK, let's merge this.
Do you guys prefer me to push a beta tag of the template? So you can test on controlled repos before a final release?

As we can update the template selectively on the repo we want, I don't think we need such extra careful on this and I would publish directly a new full version.

OK, you got https://github.com/OCA/oca-addons-repo-template/releases/tag/v1.8.0

@sbidoul do you think we can deploy with your script to all 14.0/15.0 OCA branches?

I think it's better to update manually a couple of repos and see there's no regression before doing that.

Deployed on https://github.com/OCA/delivery-carrier/pull/526. I will test some other repos.

It went OK also in the most conflicting OCA repository: https://github.com/OCA/sale-workflow/pull/2172, so I think it's safe to deploy this everywhere in 14.0 and 15.0.


JSON arrays

Is it possible to replace multiple values in a json array?
Example:
{
  "bindings": [
    {
      "type": "httpTrigger",
      "direction": "in",
      "webHookType": "genericJson",
      "name": "req"
    },
    {
      "type": "http",
      "direction": "out",
      "name": "res"
    },
    {
      "type": "blob",
      "name": "name2",
      "path": "path2",
      "connection": "connection_storage",
      "direction": "in"
    },
    {
      "type": "blob",
      "name": "name2",
      "path": "path2",
      "connection": "connection_storage",
      "direction": "in"
    }
  ],
  "disabled": false
}

In this json file i want to replace the connection attribute of all elements in the bindings array.
I tried this
Transformation file:
{
  "bindings/connection": "test"
}

Result:
{
  "bindings": {
    "connection": "test"
  },
  "disabled": true
}

You're right, there is no option to do this yet. I'll add it in further versions.

Latest fixes were released to Nuget and VS Marketplace.

Could you update the docs to show how exactly we can transform elements in arrays?


core setup - Add unittests for SetupBuilder

Issue - https://github.com/mediapills/dlab/issues/18

Moved in to master


Fix a couple of new Elixir 1.4 warnings

In Elixir 1.4, local arity 0 functions which are called without parentheses raises a warning. Just a quick PR to satisfy these warnings by explicitly using parentheses where applicable.

PS: On this topic, it seems that there's a lot of warning fixes in master that haven't been released yet. Any ETA on this? EG: Swapping out Map for HashDict

Thanks for the PR @lukerollans! Yep there are a few changes in master we need to get done. No ETA, but it should be soon! Just need to make sure everything works as expected in a few of our projects before releasing


Incompatible with other SilverShop modules

The requirement for your silvershop/core dev-merged repository version makes this difficult to install if you are using other SilverShop modules such as silvershop/discounts, silvershop/shipping etc as these require silvershop/core ^3
composer displays:
Your requirements could not be resolved to an installable set of packages.
Problem 1
- silvershop/shipping 2.x-dev requires silvershop/core ^3 -> satisfiable by silvershop/core[3.x-dev].
- silvershop/shipping 2.x-dev requires silvershop/core ^3 -> satisfiable by silvershop/core[3.x-dev].
- silvershop/shipping 2.x-dev requires silvershop/core ^3 -> satisfiable by silvershop/core[3.x-dev].
- don't install silvershop/core dev-merged|remove silvershop/core 3.x-dev
- don't install silvershop/core 3.x-dev|don't install silvershop/core dev-merged
- don't install silvershop/core dev-merged|don't install silvershop/core 3.x-dev
- don't install silvershop/core 3.x-dev|don't install silvershop/core dev-merged
- Installation request for silvershop/core dev-merged -> satisfiable by silvershop/core[dev-merged].
- Installation request for silvershop/shipping (locked at 2.x-dev, required as master) -> satisfiable by silvershop/shipping[2.x-dev].

You can use the following workaround to install the module and make composer accept the custom repository (see composer docs):
"repositories": [
    {
        "type": "vcs",
        "url": "https://github.com/xini/silvershop-core"
    }
],
"require": {
    "silvershop/core": "dev-merged as 3.x-dev"
}

In the meantime, my two pull requests have been accepted by the main repo though. You no can just remove the custom repository and require 3.x-dev from the main repo. I will update the module asap.


New transcription

EXPLANATION:
I have found a better way to transcribe the video. Previously, we had first downloaded the video as an audio file and then transcribed it. now all this is skipped and the video is transcribed directly. (The duration is a few seconds).
This way is clearer (only one file) and more runtime efficient.
USE:
To use the function you have to call get_transcription(url) and pass the desired url.

If this is really better it is okay. It was not about the time it takes, it was more about the accuracy.
If this new function is only working when there is a transcription available,  it would be inferior to the previous version.
It is normal that an AI takes a bit of time to transcribe everything, especially with a larger model, when it is running locally.
We can test it, if it is suitable for every video on youtube, especially one which has no subtitles.
The previous method would work on every audio file or video file, so it would not be limited to youtube.


Fix relative links

Fixing more classes of bad relative links after #93.
Tried to find instances of the following relative links that don't work anymore:

/pricing
/develop
/services
/marketplace
/manage
/support

Note: I only tested that a few of the replacements were indeed of broken links for each class of link.
Seems like some automation around bad links would be useful. I'm also not sure if there are other tools that could help (e.g. redirect rules/logic somewhere) also.

Hi @obsoleted, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Andre Price). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@obsoleted : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@obsoleted - Hi Andre, we appreciate the fixes. There are some quality processes we still need to operationalize. I'm going to pull these updates over to our private repo so the changes can be validated. Thanks very much!

@obsoleted @tysonn  Thanks for the contributions, Andre. Moved content to private repo pull request #2635.

#sign-off

@Juliako Hi Julia Thanks for the sign off. I'm closing this pull request because this content was moved to the private repo to validate changes. Can you please review the changes in 2635 in the private repo after validation is finished?


Chem: Molecular filter

[x] Transfer molfilter to package.
[x] Wait untill filter problem is fixed
[x] Add substructure highlighting
[ ] Text search
[ ] Other sketchers
[ ] "Favourites"

Continues in #132.


error: invalid command 'bdist_wheel'

ub1404@ub1404-A:~/github/tensorflow$ bazel-bin/tensorflow/tools/pip_package/build_pip_package /tmp/tensorflow_pkg
Wed Nov 25 02:32:35 EST 2015 : === Using tmpdir: /tmp/tmp.lVOy4pRiQ5
/tmp/tmp.lVOy4pRiQ5 ~/github/tensorflow
Wed Nov 25 02:32:35 EST 2015 : === Building wheel
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: setup.py --help [cmd1 cmd2 ...]
or: setup.py --help-commands
or: setup.py cmd --help
error: invalid command 'bdist_wheel'

pip install wheel

ub1404@ub1404-A:~/github/tensorflow$ sudo pip install wheel
[sudo] password for ub1404:
The directory '/home/ub1404/.cache/pip/log' or its parent directory is not owned by the current user and the debug log has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want the -H flag.
The directory '/home/ub1404/.cache/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want the -H flag.
You are using pip version 6.0.8, however version 7.1.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
The directory '/home/ub1404/.cache/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want the -H flag.
Collecting wheel
Downloading wheel-0.26.0-py2.py3-none-any.whl (63kB)
100% |################################| 65kB 837kB/s
Installing collected packages: wheel
Successfully installed wheel-0.26.0
ub1404@ub1404-A:~/github/tensorflow$
ub1404@ub1404-A:~/github/tensorflow$
ub1404@ub1404-A:~/github/tensorflow$ sudo bazel-bin/tensorflow/tools/pip_package/build_pip_package /tmp/tensorflow_pkg
Wed Nov 25 03:06:54 EST 2015 : === Using tmpdir: /tmp/tmp.6aZe9fEjxd
/tmp/tmp.6aZe9fEjxd ~/github/tensorflow
Wed Nov 25 03:06:54 EST 2015 : === Building wheel
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: setup.py --help [cmd1 cmd2 ...]
or: setup.py --help-commands
or: setup.py cmd --help
error: invalid command 'bdist_wheel'
ub1404@ub1404-A:~/github/tensorflow$

pip install wheel  can sovle this issue.
I used pip2 install wheel since my installed both py2 and py3

I encountered the same problem and worked it out. It seems that python can't find wheel module.
Does you use pyenv-virtualenv? you should install wheel on global python
pyenv global 3.6.0
pip3 install wheel

anyone still having this problem? I have wheel installed in both pip2 and pip3, still the command fails
pablo@batman tensorflow (master) $ pip2 list
DEPRECATION: The default format will switch to columns in the future. You can use --format=(legacy|columns) (or define a format=(legacy|columns) in your pip.conf under the [list] section) to disable this warning.
certifi (2<IP_ADDRESS>)
chardet (3.0.4)
cryptop (0.1.0)
idna (2.5)
mercurial (4.3.1)
numpy (1.13.1)
pip (9.0.1)
requests (2.18.2)
setuptools (32.1.0)
six (1.10.0)
urllib3 (1.22)
wheel (0.29.0)
yasm (0.0)
pablo@batman tensorflow (master) $ pip3 list
DEPRECATION: The default format will switch to columns in the future. You can use --format=(legacy|columns) (or define a format=(legacy|columns) in your pip.conf under the [list] section) to disable this warning.
pip (9.0.1)
setuptools (32.2.0)
virtualenv (15.1.0)
wheel (0.29.0)
pablo@batman tensorflow (master) $ bazel-bin/tensorflow/tools/pip_package/build_pip_package /tmp/tensorflow_pkg
Tue Sep 12 21:52:39 -03 2017 : === Using tmpdir: /var/folders/p2/1n2xjbq93hn4sd7lgj9mv4c80000gn/T/tmp.XXXXXXXXXX.TBZdyy7q
~/Documents/work/projects/forks/tensorflow/bazel-bin/tensorflow/tools/pip_package/build_pip_package.runfiles ~/Documents/work/projects/forks/tensorflow
~/Documents/work/projects/forks/tensorflow
/var/folders/p2/1n2xjbq93hn4sd7lgj9mv4c80000gn/T/tmp.XXXXXXXXXX.TBZdyy7q ~/Documents/work/projects/forks/tensorflow
Tue Sep 12 21:52:41 -03 2017 : === Building wheel
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
   or: setup.py --help [cmd1 cmd2 ...]
   or: setup.py --help-commands
   or: setup.py cmd --help

error: invalid command 'bdist_wheel'

I have installed wheel but I get the following error yet. What should I do?
Wed Nov 22 23:25:09 CST 2017 : === Using tmpdir: /var/folders/vt/mp6zqnxd095bt5m88tnf0gq00000gn/T/tmp.XXXXXXXXXX.xwIk35fo
~/software/tensorflow/bazel-bin/tensorflow/tools/pip_package/build_pip_package.runfiles ~/software/tensorflow
~/software/tensorflow
/var/folders/vt/mp6zqnxd095bt5m88tnf0gq00000gn/T/tmp.XXXXXXXXXX.xwIk35fo ~/software/tensorflow
Wed Nov 22 23:25:11 CST 2017 : === Building wheel
error in tensorflow setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected ',' or end-of-list in backports.weakref >= 1.0rc1; python_version < "3.4" at ; python_version < "3.4"

I was having this problem and for me it was not sufficient to simply execute "sudo pip install wheel".
To investigate further I looked into the bazel-bin/tensorflow/tools/pip_package/build_pip_package script to see how it was loading python, where I found these lines:
# Before we leave the top-level directory, make sure we know how to
# call python.
source tools/python_bin_path.sh

The shell script sourced there is the one liner:
export PYTHON_BIN_PATH="/Library/Frameworks/Python.framework/Versions/2.7/bin/python"
Which is the python version I selected when configuring my tensorflow build.
The pip on my path, however, was for python 3.5, which you can see by calling pip --version or which pip.   So my calls to sudo pip install wheel were only installing to the 3.5 version.  The solution for me was to explicitly call the version of pip for the version/install of python you are building tensorflow with.  In my case, that was:
sudo /Library/Frameworks/Python.framework/Versions/2.7/bin/pip install wheel
After that, the build_pip_package script worked for me.

Maybe use chown you_user_name /path/to/project/evn/* -R
it was good for me!

if you're on ubuntu and it doesn't work, see this:
https://stackoverflow.com/a/59596814/2353085
sudo apt-get install gcc libpq-dev -y sudo apt-get install python-dev  python-pip -y sudo apt-get install python3-dev python3-pip python3-venv python3-wheel -y pip3 install wheel

@relh
I builded a script to have various environments in one project DEV, QA, etc, in each run, I delete the .temp folder, where is the venv for each environment, and I get the error error: invalid command 'bdist_wheel',  I realize that the error disappears if I don't delete the folder, but then I add the install pip3 install wheel before install the other libs and that resolved the problem too.
#!/bin/bash

# Time tracker Start
echo START: `date`

# ..:: DEV

ENVIRONMENT='dev'

ENVIRONMENT_DIR="/${ENVIRONMENT}"

ENVIRONMENTS_DIR="/.temp"

PROJECT_DIR="/project"

PWD="$(pwd)"

# Define environment dependencies FOLDER
VENV_DIR="${PWD}${ENVIRONMENTS_DIR}${ENVIRONMENT_DIR}"

# Clear environment dependencies FOLDER
if [ -d "$VENV_DIR" ]; then
    rm -rf $VENV_DIR
fi

# Create venv directory
python3.6 -m venv $VENV_DIR

# Activate venv directory
.  .${ENVIRONMENTS_DIR}${ENVIRONMENT_DIR}/bin/activate

# https://github.com/tensorflow/tensorflow/issues/348
# Issue ref: https://stackoverflow.com/a/59596814/2513972
pip3.6 install wheel

# Install requirements
pip3.6 install -r "requirements_${ENVIRONMENT}.txt"

export PYTHONPATH="${PYTHONPATH}:${PWD}${PROJECT_DIR}"

# Time tracker End
echo END: `date`

repo: https://github.com/romelgomez/python-project-with-various-environments-example


Added a new FAQ

New FAQ of we cant RDP into a windows pod/container

@PradeepKommaraju : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thank you for the feedback and sorry for the delay.
I believe this fix is no longer relevant.
#please-close


ci: use reusable workflow to publish project to PyPI

Summary of Changes
Use reusable workflow to publish project to PyPI.

🦙 MegaLinter status: ✅ SUCCESS

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by


persistent clusters enhancements

Use CLUSTER_NAME to write out location of the persistent cluster build directory for increased flexibility on multi persistent cluster deployments
Enhanced logic to allow e2e deployment of applications and CAM configuration without migrating workloads, necessary for demo clusters
Adjust E2E deploy only logic
Add persistent cluster destroy script to be called for rebuild Jenkins jobs
Depends on : https://github.com/konveyor/mig-e2e/pull/81

LGTM


[feat] StableMemory library

The StableMemory API for (almost) raw access to IC stable memory.
Still assumes 32-bit IC stable memory API (offsets are 32-bit)

Line 32 also, please!

Oops, done!


Twitter Login Server example is wrong

I've asked it here: http://stackoverflow.com/questions/27570528/sign-in-twitter-oauth-without-sessions-aka-token-secret-angularjs-fail

@eguneys, @sahat  I am also having difficult getting Twitter login to work. I am using Node.js for my server.


virtual network manager - initial commit

@mbender-ms

Please don't include the MD file in the code-sample repo. Once the code is merged, I'll generate the article via an app I wrote that uses OpenAI.
Resource names should be produced from the random_pet resource instead of relying on the customer to supply a valid value in a variable.
Why are you using the tag attribute when it doesn't seem to add anything to the sample?
All resources and vars should also be listed in the README file
The README file points to the wrong article in the "Example" section.
The outputs.tf file needs to display any randomly generated value as well as any values that are necessary for the customer to work with the newly created resources. For example, the "Verfiy the Results" section should show the AzCLI and AzPS command necessary to show what was created in the article. To that end, the sample needs to output any necessary arguments to those command calls.
See the following sample for how to define the providers' version constraints: https://github.com/Azure/terraform/blob/master/quickstart/101-key-vault-key/providers.tf
All variables need to have a type, description, and default.
All variables need to be snake case (this sampel is using camel casing).
By convention, the "depends_on" should be the last line of any resource and be preceeded by a blank line.

Hi @TomArcherMSFT - I implemented you feedback above and I've tested the files with the new code suggestions from the email thread. Can you review and let me know next steps? Currently working on Article to go with it (not ready for review). https://github.com/MicrosoftDocs/azure-docs-pr/pull/239027

Hi @TomArcherMsft - I implemented you feedback above and I've tested the files with the new code suggestions from the email thread. Can you review and let me know next steps? Currently working on Article to go with it (not ready for review). MicrosoftDocs/azure-docs-pr#239027

Thanks @mbender-ms. I'll look this over this morning. Please do not work on the article as we generate quickstarts from a python app using openai. I'll create the article once the code is merged and assign the PR to you for review.

This looks really good, @mbender-ms .
The main thing I noticed is that any Azure resource name that needs to be unique should be obtained from a random value generator such as random_pet. Otherwise, the E2E testing pipeline will fail.
@lonegunmanb @stemaMSFT @grayzu Could you please review Michael's PR? Thanks!

@TomArcherMSFT Does that mean that each unique name needs to have a random resource created? After I made the changes to use random_pet, I thought that may not have been the way the pipeline needs it.

@lonegunmanb I removed all the dependencies and implemented all the changes. Verified it works as expected.

@lonegunmanb Let me know if I got it merged correctly. Thanks!


file issue: Should not generate a hyperlink if the backend didnt provide one

Only Slack and Gitter currently have a URL for each message; Telegram and other backends return '', which results in an empty URL, a self-ref.
Opened by @jayvdb at Zulip-GitHub

markdown used here: Opened by @jayvdb at [Zulip-GitHub]()

Blocked, waiting for backend patches to be merged, as that will mean less patches.


Update coffee file

Changelog-entry: Update coffee file

Can one of the admins verify this patch?


installer: Update installation image for DMG installer.

Rename Zulip-desktop to Zulip
Try new design (maybe add Zulip's logo)

@akashnimare

The Zulip logo kinda looks odd. Can you reduce the size and paste it here?

Updated, this is how it looks now -


updated the wino_bias dataset

Updated the wino_bias.py script.

updated the data_url
added different configurations for different data splits
added the coreference_cluster to the data features

Hi @JieyuZhao ! Have you had a chance to add the different configurations ?
Thanks again for your help on this !

Hi @JieyuZhao ! Have you had a chance to add the different configurations ?
Thanks again for your help on this !

Hi @lhoestq  Yes, I've updated the code. Now the configuration will have dev/test splits.

Hi @JieyuZhao ! Have you had a chance to add the different configurations ?
Thanks again for your help on this !

Hi @lhoestq  Yes, I've updated the code. Now the configuration will have dev/test splits.

Cool thanks !
This looks perfect this way.
Now we just need to update the dataset_infos.json (it contains the metadata of the dataset) and add dummy data to be able to test this script automatically.
To update the dataset_infos.json you just need delete the current one at ./datasets/wino_biais/dataset_infos.json, and then run this command:
datasets-cli test ./datasets/wino_biais --save_infos --all_configs --ignore_verifications

To add the dummy data there's also a tool to add them automatically.
First delete the folder at ./datasets/wino_biais/dummy and then run
datasets-cli dummy_data ./datasets/wino_biais --auto_generate --match_text_files "*conll" --n_lines 15

Let me know if you have questions :)
Also don't forget to run make style to format the code properly.

Thanks for the instruction! I've updated the metadata and the dummy data and also do the formatting. Please let me know if more is needed. :)


Create vendor neutral CRDs for Network Functions

Create vendor neutral CRDs and Go types for Network functions.
See this document for reference and details.
As part of this we need to define CRDs for SMF, UPF and AMF.
Following are the task list

[x] Create CRDs for UPF (UPF class and deployment)
[x] Create CRDs for SMF (SMF class and deployment)
[x] Create CRDs for AMF (AMF class and deployment)

This is done.


Add support for Python 2, fix AttributeError: 'module' object has no attribute 'quote'

Apparently, the shlex.quote method was moved from pipes at some point in the history of Python’s standard library, so fall back to importing it from there, as suggested on SO.
If some version of Python out there has neither of those, the error message would be ImportError: cannot import name quote.
Attribution for the SO snippet is in the commit message, if anyone cares.

Oh noes! This PR has sat for two months!?
Merged. Thanks for contributing.


Feature: Color override setting for file/directory indicator column

Attempt at fixing #574 . Provide two new theme options:
file-indicator: 43
directory-indicator: 65

(numbers in example are arbitrary, and can be any color currently supported)
I thought one for each made more sense than a common one. Feel free to disagree.
The indicators use the theme in the file if found, if not they use the defaults for file/directory names. LS_COLORS color is only used if no theme setting is provided. This is based upon the following comment in the feature suggestion, which I found reasonable:

I suggest an optional setting in the theme yaml. If this field is absent, fall back to the file type color from LS_COLORS.

I was unsure on how to test the LS_COLORS bits, so have briefly tested that manually. Please tell me if I'm missing something and there are good ways of writing tests for them 🙂
In general I'm fairly new to Rust, so any feedback is greatly appreciated! 🙂

TODO

[x] Use cargo fmt
[x] Add necessary tests
[x] Add changelog entry
[x] Update default config/theme in README (if applicable)
[x] Update man page at lsd/doc/lsd.md (if applicable)

Codecov Report

Merging #725 (6d7fc74) into master (d538d74) will decrease coverage by 0.84%.
The diff coverage is 76.47%.

@@            Coverage Diff             @@
##           master     #725      +/-   ##
==========================================
- Coverage   88.31%   87.46%   -0.85%
==========================================
  Files          41       41
  Lines        4500     4586      +86
==========================================
+ Hits         3974     4011      +37
- Misses        526      575      +49

Impacted Files
Coverage Δ

src/color.rs
52.58% <70.27%> (+3.71%)
:arrow_up:

src/color/theme.rs
64.57% <91.66%> (+1.99%)
:arrow_up:

src/meta/filetype.rs
82.85% <100.00%> (ø)

src/display.rs
83.93% <0.00%> (-8.90%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I think it would be better to keep just one color. This feature will be mostly if folks want to make everything consistent (you can see something similar in the original issue). When they want custom colors, I think it will be better to match LS_COLORS.
PS: If we were to go the route of implementing separate colors we would need more than just file and dir colors, for example symlinks, block devices etc.


Ordering of reviews

Instead of always having a random order it would be nice to have some alternatives.
For example "longest available" or "lowest SRS stage" to be able to do the more important ones first in case you don't have time to do all reviews at once.

Hello @oguzgelal, hope you are doing fine!
Just tried my hand at implementing a Review Order option #85.
It would be wonderful if you could take a look, and any feedback is highly appreciated 😄  (especially since I'm a back-end developer so not sure what the best-practices here!)
Cheers mate, congrats on your app btw, it's awesome!

I’ll close this since this has been merged


Extend plugin API to support transfer handlers

What this PR does / why we need it
The OCM library already provides an extension point to provide transfer handlers
controlling the transport of component versions from one environment into another one.
A transfer handler controls:

how to handle changes in component versions when re-transporting them
how to resolve component references and whether they should be transported, also
whether sources and resources should be transported by value or by reference.

So far, there were two implementations:

a standard implementation working on the standard transfer options, like --recursive or --copy-resources
a spiff++ based implementation based on the standard handler adding the capability to influence
the behaviour based on a spiff script.

On the command line interface always the spiff handler was used which works like the standard handler if no script
is configured.
This PR adds three new features:

it supports a hierarchical naming scheme to denote transfer handlers (like for download and upload handlers),
which can be used un the command line to select and configure supported transfer handler types.
it extends the Plugin API to support transfer handler. This way a plugin may implement the transfer handler interface to
provide an arbitrary rule set controlling the transport process.
A general configuration option for handlers allows for passing additional handler configuration

Similar to the uploaders and downloaders the naming scheme for transfer handlers looks as follows:

ocm/<builtin handlers>:  used for the already existing  standard and spiff handlers.
Additional handlers may now add themselves to appropriate names to be usable from the command line.
plugin/' A dedicated handler maps plugin provided transfer handlers into the naming scheme.
It uses the sub scheme <plugin name>[/<handler name>]. A plugin may provide multiple (named) handlers.

Which issue(s) this PR fixes

Since I've been struggling to see the plugin architecture since it is quite scattered throughout the library, is there documentation providing an overview?
Thus, the plugin implementer side including the role of the Plugin Descriptor and the plugin implementation support library (Plugin Programming Interface (PPI)) or a raw command implementation, as well as the plugin caller side including Plugin Adapter , the registrations and the individual extension point specific plugin packages AND how all these parts fit together.

epic: https://github.com/open-component-model/ocm/issues/1130


Perf: precompute width and height calculations

With this change, factors for width and height calculations are precomputed in the constructor. This prevents their recalculation every time getPoint is called.
In the ImagePainter, where getPoint() is executed tens of thousands of times, this can make a significant difference.
Benchmark Results for ImageShapeBench:
+-----------+---------------------+--------+------------+--------+------------+------+-----------+----------+
| suite     | date                | php    | vcs branch | xdebug | iterations | revs | mode      | net_time |
+-----------+---------------------+--------+------------+--------+------------+------+-----------+----------+
| <current> | 2023-11-11 18:56:52 | 8.2.12 | painter    | false  | 4          | 100  | 109.289ms | 10.898s  |
| main      | 2023-11-11 18:50:03 | 8.2.12 | main       | false  | 4          | 100  | 120.271ms | 12.091s  |
+-----------+---------------------+--------+------------+--------+------------+------+-----------+----------+

<current>
+-----------------+--------------------+----------+-----------+-----------+-----------+--------+-----------+
| benchmark       | subject            | memory   | min       | max       | mode      | rstdev | stdev     |
+-----------------+--------------------+----------+-----------+-----------+-----------+--------+-----------+
| ImageShapeBench | benchImageShape () | 12.327mb | 108.381ms | 109.518ms | 109.289ms | ±0.44% | 480.182μs |
+-----------------+--------------------+----------+-----------+-----------+-----------+--------+-----------+

main
+-----------------+--------------------+----------+-----------+-----------+-----------+--------+---------+
| benchmark       | subject            | memory   | min       | max       | mode      | rstdev | stdev   |
+-----------------+--------------------+----------+-----------+-----------+-----------+--------+---------+
| ImageShapeBench | benchImageShape () | 12.327mb | 120.072ms | 122.903ms | 120.271ms | ±0.96% | 1.162ms |
+-----------------+--------------------+----------+-----------+-----------+-----------+--------+---------+

Note:  I have removed this check:
if ($width === 0.0 || $height === 0.0) {
    return null;
}

I'm wondering if it has any real use case. Does it? If so, I can bring it back.

The 0 checks are to catch division by zero

@dantleech Yes, but I thought this would never happen in a real case. Bringing back this check made things too verbose (I've pushed the changes here), I think. So, I'm closing this PR for now.

Probably this bottleneck will become negligible in most cases with #85


Add Label to Column Name in Tool Panel for Selection

In the Tool Panel when hiding/showing columns, the icon is the only location that you can click/tap.  Would be nice to have a label on the column name as well so clicking/tapping the column name would also toggle the hide/show selection.
Thank for all the good work on ag-grid.

Here is a working less hack. Tested in chrome.
.ag-tool-panel-container {
#borderLayout {
#centerRow {
#center {
.ag-list-selection {
.ag-list-item {
height: 36px;
line-height: 36px;
.ag-visible-icons {
width: 100%;
display: block;
position: relative;
z-index: 2;
}
}
.ag-list-item>span>span:last-child {
position: relative;
z-index: 1;
top: -36px;
left: 20px;
}
}
}
}
}
}

thanks for the feedback. i have big plans for the sidebar - so i will be keeping from the finer details you mention as it's not ready for polishing. on item i want to do is allow multi select, and then action on a group (eg pick 10 cols and set them visible all at once, or drag 10 cols onto the grid at the same time, or drag a group of cols at the same time). i want to get all these bigger features in first as they will impact then the smaller user interaction details.

clicking the label now shows / hides the column


Add capability to attach additional security groups to ALB

Description
This change is to allow a list of security group IDs to be added to the load balancer. This will enable externally created security groups to be passed through to the ALB module.

Can we get a code review here? I could use that feature.

v2.2.0 has been just released with this feature.
Thanks @amittag!


FragmentPermissionDispatcher is not generated another fragment function on same package

Expected

Generate FragmentA.readAndReceiveSmsWithPermissionCheck() and FragmentB.readAndReceiveSmsWithPermissionCheck() on same package

Actual

Only generate FragmentA.readAndReceiveSms()

Environment

PD 3.2.0 and PD Processor 3.3.1

@eyedol @punchdrunker @snkashis @egor-n @hotchemi

resolved?


Automatically refresh "Running" status

The "Running" status for an SCIO pipeline does not automatically refresh in the Projects dashboard or the Project page. The problem is that you may think the pipeline is still running after it has finished.  A periodic refresh every 30 seconds could suffice.

Implemented in https://github.com/nexB/scancode.io/pull/428
From https://github.com/nexB/scancode.io/blob/main/CHANGELOG.rst

Implement run status auto-refresh using the htmx JavaScript library. The statuses of queued and running pipeline are now automatically refreshed in the project list and project details views every 10 seconds. A new "toast" type of notification is displayed along the status update.

@mjherzog "Running" and "Queued" are now refresh every 10s.


Could not find com.jetbrains.plugins:com.intellij.java:. when calling plugins()

What happened?
I am trying to upgrade an IJ plugin to 2.x.  I have tried a number of things including different Gradle plugin versions, 2.0.0, 2.1.0, 2.1.1-SNAPSHOT, using the exact code provided in this documentation, https://plugins.jetbrains.com/docs/intellij/tools-intellij-platform-gradle-plugin-migration.html#intellijplugins, as well as nesting a Gradle directory into another one, closer to where the Java code is, as well as calling different permutations of the plugins() method, that is plugins(listOf("com.intellij.java")) yields the same result for me.
Is there an example 2.x project that I could get running locally to help migrate my plugin?
Relevant log output or stack trace
Could not find com.jetbrains.plugins:com.intellij.java:.
     Required by:

(The same can be said for any of the output from `./third_party/gradlew printBundledPlugins`)

Steps to reproduce
See what happend above.  Following directions here: https://plugins.jetbrains.com/docs/intellij/tools-intellij-platform-gradle-plugin-migration.html for the Flutter IJ plugin.
Gradle IntelliJ Plugin version
2.0.0
Gradle version
8.2.1
Operating System
macOS
Link to build, i.e. failing GitHub Action job
No response

The com.intellij.java plugin is a bundled plugin. Instead of using plugins() dependency helper, you should use bundledPlugins()

That was it, thank you.


fix: don't fetch old values when changing data

This still has a problem where sometimes metrics can't be written to the table directly and it has some delay. I did not get to fully debugging anymore. But this is still strictly better than what we had before.

Seems like we have a rule that prevents this merge.


Enforce docs ids _changes filter optimization limit

It turns out changes_doc_ids_optimization_threshold limit has never been applied for the clustered changes feeds. So it was effectively unlimited. This commit enables it, and also adds tests to ensure the limit works.
Since we didn't have a good Erlang integration test suite for clustered changes feeds, which allowed this case to slip through the cracks, add a few more tests along the way to test the majority of parameter combinations which might interact: sharding single shards vs multiple, continuous vs normal, reverse, row limits etc.
The previous limit was 100, but since it was never actually applied it's equivalent not having one at all, so let's pick a new one. I chose 1000 noticing that at Cloudant, close to 3000 we had fabric timeouts on a busy cluster, so that seemed too high. And 1000 seemed about the ballpark of the what size of _bulk_get batch might be. Adding a benchmarking eunit test https://gist.github.com/nickva/a21ef04b7e4bdbed5fdeb708f1d613b5 showed about 50-75 msec to query batches of 1000 random (uuid) doc_ids for Q values 1 through 8.

Hi,
seeing this errors from build-report under Windows:
Errors
======

chttpd_changes_test:97 with (t_design_filter)
---------------------------------------------

::in function chttpd_changes_test:t_design_filter/1 (test/eunit/chttpd_changes_test.erl, line 373)
in call from eunit_test:run_testfun/1 (eunit_test.erl, line 71)
in call from eunit_proc:run_test/1 (eunit_proc.erl, line 531)
in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 356)
in call from eunit_proc:handle_test/2 (eunit_proc.erl, line 514)
in call from eunit_proc:tests_inorder/3 (eunit_proc.erl, line 456)
in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 346)
in call from eunit_proc:run_group/2 (eunit_proc.erl, line 570)
**error:{assertEqual,[{module,chttpd_changes_test},
              {line,373},
              {expression,"Pending"},
              {expected,2},
              {value,0}]}

chttpd_changes_test:97 with (t_docs_id_filter_over_limit)
---------------------------------------------------------

::in function chttpd_changes_test:t_docs_id_filter_over_limit/1 (test/eunit/chttpd_changes_test.erl, line 409)
in call from eunit_test:run_testfun/1 (eunit_test.erl, line 71)
in call from eunit_proc:run_test/1 (eunit_proc.erl, line 531)
in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 356)
in call from eunit_proc:handle_test/2 (eunit_proc.erl, line 514)
in call from eunit_proc:tests_inorder/3 (eunit_proc.erl, line 456)
in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 346)
in call from eunit_proc:run_group/2 (eunit_proc.erl, line 570)
**error:{assertEqual,[{module,chttpd_changes_test},
              {line,409},
              {expression,"meck : num_calls ( couch_changes , send_changes_doc_ids , 6 )"},
              {expected,0},
              {value,1}]}

chttpd_changes_test:97 with (t_design_filter)
---------------------------------------------

::in function chttpd_changes_test:t_design_filter/1 (test/eunit/chttpd_changes_test.erl, line 373)
in call from eunit_test:run_testfun/1 (eunit_test.erl, line 71)
in call from eunit_proc:run_test/1 (eunit_proc.erl, line 531)
in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 356)
in call from eunit_proc:handle_test/2 (eunit_proc.erl, line 514)
in call from eunit_proc:tests_inorder/3 (eunit_proc.erl, line 456)
in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 346)
in call from eunit_proc:run_group/2 (eunit_proc.erl, line 570)
**error:{assertEqual,[{module,chttpd_changes_test},
              {line,373},
              {expression,"Pending"},
              {expected,2},
              {value,0}]}

chttpd_db_test:329 should_return_409_for_put_att_nonexistent_rev
----------------------------------------------------------------

::in function chttpd_db_test:'-should_return_409_for_put_att_nonexistent_rev/1-fun-2-'/1 (test/eunit/chttpd_db_test.erl, line 330)
in call from eunit_test:run_testfun/1 (eunit_test.erl, line 71)
in call from eunit_proc:run_test/1 (eunit_proc.erl, line 531)
in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 356)
in call from eunit_proc:handle_test/2 (eunit_proc.erl, line 514)
in call from eunit_proc:tests_inorder/3 (eunit_proc.erl, line 456)
in call from eunit_proc:with_timeout/3 (eunit_proc.erl, line 346)
in call from eunit_proc:run_group/2 (eunit_proc.erl, line 570)
**error:{badmatch,{error,connection_closed}}

Thanks for investigating, @big-r81. Does it happen on every run or is it random?
I could see how {expression,"Pending"}, {expected,2}  might happen, I think in case of a Q=8 we made too strong of an assertion that Pending will always be 2, it doesn't have to unless Q=1. I think we can relax that to Pending is an integer >= 0 and < 7.
The other ones are bit unexpected:
{expression,"meck : num_calls ( couch_changes , send_changes_doc_ids , 6 )"},
Indicates somehow we failed to set the config value or didn't pass the parameters correctly through in the test request. See if you run a few times in a row if keep seeing this error. I checked on other architectures and macs, arm64 and various other OS seem to pass this test consistently.

in function chttpd_db_test:'-should_return_409_for_put_att_nonexistent_rev/1-fun-2-'/1 (test/eunit/chttpd_db_test.erl, line 330)
in call from eunit_test:run_testfun/1 (eunit_test.erl, line 71)

This ones looks rather generic and points to something being off with the network code and sometimes it fails unexpectedly.  {error,connection_closed} comes from ibrowse and just indicates that the connection Erlang process suddenly died. Wonder if we could add more debug logic to ibrowse or use Wireshark and the like to see what the connect state was and what the lower level error was.


optional libjpeg-turbo support

This is in reference to #4807.

Can one of the admins verify this patch?

@jart @tensorflower-gardener @keveman BTW, in #4807, I was asked to provide some benchmarks. When using a jpeg-decode-only graph source here and limiting processing to 1 thread, I got 1,257 images/sec on a machine with an SSD and an Intel Core i7-5930K CPU @ 3.50GHz with libjpeg-turbo, and 632 images/second with libjpeg. This represents a 1.99x speedup, which is consistent with the expectations based on the benchmarks provided by the libjpeg-turbo team. The range of speed-up factors for x86-64   is 1.89 to 4.31, depending on image characteristics, and my test shows that the speed-up factor is indeed in that range. For other platforms such as ARM v8, the speed-up factors are basically the same as for x86-64 (see here). A very exhaustive spreadsheet with lots of benchmark scenarios can be found here. My finding of a 1.99x speedup with the library called from TensorFlow itself should be convincing enough that this constitutes a major improvement.

Jenkins, test this please.

We're grateful that you took the time to send us this pull request @mkolod. I've just written up a change internally that upgrades TensorFlow to libjpeg-turbo, using the secret build knowledge other Googlers have figured out before me. When my change gets exported to GitHub (in probably a week tops) #4807 will be updated automatically.


updated dependencies

updates dep versions

Hi @Stallon-niranjan and thank you for the PR.  Would it be possible to restrict this PR to only the commit updating the dependencies.yaml file?  A separate PR explaining the fabio/questdb.nomad files in the aws multi-node scenario would be preferred, to gather context on those changes.
As for the dependency bumps, I gave it a quick try and discovered a few sha256sum discrepencies: the hashes for consul, vault, traefik and bindle didn't match those of the linux/amd64 binaries -- I wonder if you had been using a host with a different os/arch combo?  In any case, we'd want the committed hashes to be for linux/amd64.  Also, I'm seeing issues running bindle v0.8.2 on account of error while loading shared libraries: libssl.so.3.  Did you encounter the same?

libssl.so.3

I did not face issue directly with SSL , but faced spin cloud login --url , login errors

okay, will do dependency.yaml, exclude the questdb/fabio


Unknown Light model "Paul Neuhaus" RGBW

Hi Erik
similar to #360 this is another unknown light model for a Paul Neuhaus RGBW lamp. Any ways to fix this? Using the latest homebridge-hue and hue firmware:
Apr 08 21:20:09 raspi homebridge[21763]: [2019-4-8 21:20:09] [Hue] Philips hue: /lights/9: warning: using default colour gamut for unknown light model {"state":{"on":false,"bri":1,"hue":7072,"sat":171,"effect":"none","xy":[0.4031,0.392],"ct":275,"alert":"select","colormode":"ct","mode":"homeautomation","reachable":true},"swupdate":{"state":"notupdatable","lastinstall":null},"type":"Extended color light","name":"Deckenleuchte","modelid":"NLG-RGBW light ","manufacturername":"Neuhaus Lighting Group ","productname":"Extended color light","capabilities":{"certified":false,"control":{"colorgamuttype":"other","ct":{"min":275,"max":295}},"streaming":{"renderer":false,"proxy":false}},"config":{"archetype":"classicbulb","function":"mixed","direction":"omnidirectional"},"uniqueid":"00:15:8d:00:02:41:01:10-02","swversion":"1001-0007"}
Regards
Greg

Can you run ph probe -v /lights/9to see if we can determine what range of colour it supports?
"ct":{"min":275,"max":295}

For an extended colour light, it supports a very small range of colour temperatures.  I wonder if the reported values are correct.
"modelid":"NLG-RGBW light ","manufacturername":"Neuhaus Lighting Group "

Do these end in a space character?

Can you run ph probe -v /lights/9to see if we can determine what range of colour it supports?
"ct":{"min":275,"max":295}

For an extended colour light, it supports a very small range of colour temperatures. I wonder if the reported values are correct.

Yes color adjustment at a dimmed down level only supports very few color variances, at a dimmed up level it's more.
ph probe: /lights/9: cool ... done
ph probe: /lights/9: warm ... done
ph probe: /lights/9: red ..........................^C

It's stuck "forever" at red, at least I waited like three minutes. Also tried with -t 1 and higher.

"modelid":"NLG-RGBW light ","manufacturername":"Neuhaus Lighting Group "

Do these end in a space character?

Seems so, it's a 1:1 copy&paste from console

By default, ph probe will timeout  after five minutes.  To change that, use
ph probe -v -t 1 /lights/9

to set the timeout to one minute.  See ph probe -h.
The standard ph timeout
ph -t 1 probe -v /lights/9

sets the timeout per bridge request.  ph probe does a bridge request every five seconds (on each dot) to see if the bridge reports different xy values.

Ok, here we go:
ph probe: /lights/9: cool ... done
ph probe: /lights/9: warm ... done
ph probe: /lights/9: red ............... timeout
ph probe: /lights/9: green ............... timeout
ph probe: /lights/9: blue ............... timeout
{
  "manufacturername": "Neuhaus Lighting Group ",
  "modelid": "NLG-RGBW light ",
  "type": "Extended color light",
  "bri": true,
  "ct": {
    "min": 275,
    "max": 295
  },
  "xy": {
    "r": [
      0.9961,
      0.0001
    ],
    "g": [
      0.0001,
      0.9961
    ],
    "b": [
      0.0001,
      0.0001
    ]
  }
}

OK, I'll suppress the message in the next version, but that won't change the fact that we'll still be using the default gamut.


make the restore action unload and reload the solution

fixes #64 by making the restore also unload.  We've seen VS locking files and it shouldn't hurt to be proactive here.

thx!


use grpc probe to query health status of Ray 2.2 and above

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
https://github.com/grpc-ecosystem/grpc-health-probe
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Close this one in favor of #1554.


Remove not used imports (#6796)

Pull request opened by the merge tool on behalf of #6796

Merged into branches iso6 in 27d4830d1bbedf89d61277d6e54f1d4e2ada6fe6


gnosis safe rebrand assets updated

What does this PR do and why?
Fixes - https://github.com/jbx-protocol/juice-interface/issues/1924
I've used https://github.com/svg/svgo to optimize SVGs since I don't use a mac ( from 2kb to 1kb ) the same size as old assets were, it is a reputable repo from what I know 🍻
I've tried SVGs before and after optimizing and the quality is the same.
Screenshots or screen recordings
https://user-images.githubusercontent.com/18723426/188650577-03e3221d-c6aa-43d7-9c21-0bbc0152d6d6.mov
Acceptance checklist

[x] I have evaluated the Approval Guidelines for this PR.
[x] I have tested this PR in all supported browsers.
[x] I have tested this PR in dark mode and light mode (if applicable).

@NakedFool great job here, thanks for jumping on it so quickly. Merging 🎉


Cherrypick #892 to release/1.0

Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

/lgtm

known test framework flake. So annoying.

/test pull-cri-containerd-node-e2e


Is it possible to use an input ref for to hold keyboard values?

I am using the latest version .
My issue is that I want to use the virtual keyboard and the device keyboard at the same time.
In the examples to do that you need to update the keyboard value when the value of the input changes so virtual keyboard and HTML input are synced.
Is there any way to have one place of truth to hold the value?
For example to pass the input ref into the virtual keyboard.

Hello @seloner,
Long story short, this is not possible.
The logic of simple-keyboard (the vanilla JS library) is based around handling the input within the library (keyboard.input object) and giving options to the user to get/set this value through simple-keyboard's callbacks or methods.
The main reason is that in simple-keyboard/vanilla JS there's no two way binding. So if you're typing on an input element directly it's already expected to use the change event in order to update any places that depend on that value.
react-simple-keyboard is just a wrapper for the vanilla JS library, so it follows the same pattern.
Hope that makes it a bit clearer.
Francisco Hodge


Remove arg name on call self.router.url_path_for()

Trace wit using on last fastapi version (0.95.1) and python version 3.10

Without name= calling


Timesheet - 20/04/2021 - Manish Ram

Task 1: Timesheets
Reviewed timesheets
Proof of Work

Commented "Approved" for approved timesheets or left other comments to ask for additional Infos to the timesheet issues dated 19/04/2021
https://github.com/thenewboston-developers/Contributor-Payments/issues

Time Spent
1 hour

Task 2: Audit Discuss

Audit related minor discussion
Discussed with Rajput for the new separate channel for Projects discussion for Auditors.

Proof of Work

Can be verified by Audit Team Lead and Raj

Time Spent
0.5 hour

Task 3: Projects

Review project proposal:

tnbuser.com / tnbeverything.com

Removed discontinued projects and labeled them.

Proof of Work

Can be verified by Audit Team Lead
https://github.com/thenewboston-developers/Projects/issues/181

Time Spent
1 hour

Total Time Spent
2.5 hours
Account Number
0f92e975101eb96cea80dce430d74efd3172fa229377818d467ded85dcf2ab6c

@itsnikhil

Approved

paid


True fix of #23 BUG: sh metracer.sh -v 30899 "MoreTools|Account" "init" bug

Bug #23 is workarounded.
Need to find the real root cause and fix it.

now there is no promiscuous ClassLoaders instrumentation
PatternMatchedMethodMutator now uses getSystemClassLoader + Class.forName + reflection - more robuts


Database Authentication

Reproduce the DB+JWT JHipster authentication. #12
Done

The user send its credentials to an open endpoint.
Quarkus has no programatic way to invoke an authenticate method so there is a dedicated AuthenticationService which does  the Spring UserDetailsService and AuthenticationManagerBuilder job (ie loading a domain user with entities from a DB and checking BCrypted hasshed passwords
Once authenticated, the application forges a JWT. Quarkus is more rigourous on the security algo and thus RSA keys are mandatories (where JHipster can work with a base64 secret).
The app embeds a key pair (they are coming from Quarkus JWT guide).
Finally the app can return a JWT to the frontend for the next requests.
The app is configured with quarkus-smallrye-jwt which automatically intercept the request, validate the token, load the user details into the context and apply the endpoint authorization (versus the manual JWTFilter present in Spring JHipster)

Among the way I found couple of issues.

the NONSTRICT_READ_WRITE  caching option works nice when I run the app but fails during the test (https://github.com/quarkusio/quarkus/issues/6492).
Meanwhile I switch to READ_ONLY just to move on and I track an issue in the current repo
Quarkus as an injection issue with JUnit which makes impossible to inject properly a repository. Quarkus's team is working hard on this limit.
Meanwhile I manually look for a bean in CDI like:UserRepository userRepository = CDI.current().select(UserRepository.class).get();, it's not ideal but again it permits to move on.
There is no version of Zalando problems for JAX-RS and the default JAX-RS exception handing is way too limited. IMHO, Integrating Zalando is not a big issue but this PR was already huge enough.

Bottom line, I have to polish some part but I succeed to authenticate via the frontend againt a Quarkus Server, and that's cool!!

@amanganiello90 I would be more than happy if you could have a look to the PR, please.

@amanganiello90 I would be more than happy if you could have a look to the PR, please.

No problem, I will check it :)

The problem is the prettier.. I can do a PR to add it. After, before pushing, you have to execute on the generator main dir:

npm run lint-fix

Otherwise we can add a commit hook to run automatically it.

I'm always getting trouble with prettier...  If you have a solution it would be great!

I'm always getting trouble with prettier... If you have a solution it would be great!

Yes, I have already added a prettier with #19 .. Now all is formatted and an hook check is run before git commit. If it fails, execute the lint fix command:

npm run lint:fix

The previous correct the formatting errors, but if you have violated some eslint rules (such as variable unused or undeclared), you have to fix it manually (lint tells you where lines there are errors).


New version: Wondershare.Fotophire.SlideshowMaker version 10.0.3

Created by 🥟 Dumplings in workflow run #7851.

Log
Updated: 1.0.3 → 10.0.3
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
The manifests for the last version 1.0.3 will be removed. Reason: No installer URL is changed compared with the last state while the version is updated
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Enhancing the PHP code templates

Hello,
very nice tool!
My suggestion:
In case of
...
"postData": {
	"mimeType": "application/x-www-form-urlencoded",
	"params" : [
	    {
	        "name": "params[IDENTIFIER]",
	        "value": "DUMMY_ACCOUNT"
	    },
	    {
	        "name": "params[OPERATIONTYPE]",
	        "value": "create"
	    },
}
...

it would be nice that in the php code to have something like
...
CURLOPT_POSTFIELDS => http_build_query([
    'params[IDENTIFIER]' => 'DUMMY_ACCOUNT',
    'params[OPERATIONTYPE]' => 'create',
])
...

instead of
...
CURLOPT_POSTFIELDS => "params%5BIDENTIFIER%5D=DUMMY_ACCOUNT&params%5BOPERATIONTYPE%5D=create"
...

makes sense .. did you want to take a stab at making this change, and claim all the fame and glory? :)


Method to re-format HTML text, received as plain text and its entities.

This PR focuses to solve Issue #260 and as a result let's you recover text messages from updates to HTML marked up strings.

Hi @Arengo, this is awesome. I accept your suggestions in principle. But if it's going to handle bold and italic in HTML, for completeness it should also handle other decorations in HTML and Markdown. I will accept your submission after some time, after I decide how to integrate the whole thing (including HTML and Markdown) into telepot. Thank you!

Thanks for the feedback. If you request changes I am keen to address them, as soon as you want to tackle this. I am running this modification on my telepot application right now and it works flawlessly. I agree that additional decorations should be handled as well, but I am unsure if all - like mentions (@) or hashtags (#) - are working the same with text messages like the current HTML are doing. However, HTML and Markdown themselves should be no problem to address in this method.

Added support for Markdown and included the common HTML and Markdown entities.

@SiteEffect (did you change your name), thanks for the effort, but the complications are more than that. What if the text contains special characters (e.g. >, <, & in HTML, _, * in Markdown)? After applying the entities to the string, the user should reasonably expect that all special characters are also properly escaped.
Escaping them before applying entities may screw up the offsets. Escaping them after applying entities is hard because you got a lot more special characters mixed in. If you want a function to apply entities in a reasonable way, that function should also take care of escaping special characters properly.
Given all that extra work, I would prefer the entity-applying function to be in a separate module. I am thinking of implementing that myself. It's up to you if you want to go ahead yourself. :smile: Thanks anyway.

This is my preliminary version for markdown, along with a test at the end. Note that I split the string into a list of chars before escaping, so as not to affect the offsets in entities. Another subtlety is that special characters contained within an entity should not be escaped (this may be different in HTML).
def format_markdown(string, entities):
    # incomplete
    char_map = {'*': '\\*',
                '_': '\\_',}

    formatter = {'bold':      lambda s,e: '*'+s+'*',
                 'italic':    lambda s,e: '_'+s+'_',
                 'text_link': lambda s,e: '['+s+']('+e['url']+')',
                 'code':      lambda s,e: '`'+s+'`',
                 'pre':       lambda s,e: '```text'+s+'```'}

    def inside_some_entities(i):
        return any(map(lambda e: e['offset'] <= i < e['offset']+e['length'], entities))

    # Escape special characters
    seq = map(lambda c,i:
                  char_map[c]
                      if c in char_map and
                          not inside_some_entities(i) else c,
              list(string),          # split string to char sequence
              range(0,len(string)))  # along with each char's index

    offset = 0
    result = ''

    for e in entities:
        f,n,t = e['offset'], e['length'], e['type']

        result += ''.join(seq[offset:f])

        if t in formatter:
            result += formatter[t](''.join(seq[f:f+n]), e)
        else:
            result += ''.join(seq[f:f+n])

        offset = f + n

    result += ''.join(seq[offset:])
    return result

entities = [
    {'type': 'bold', 'offset': 0, 'length': 3},
    {'type': 'italic', 'offset': 3, 'length': 3},
    {'type': 'text_link', 'offset': 9, 'length': 6, 'url': 'www.yahoo.com'},
]

print format_markdown('a_bcdefh*ijklm*nopq', entities)

I have added this functionality to package telepot.text in the latest release of telepot. Thanks for suggesting.


[Config] Ignore files

Is there any plans for Now to be able to ignore files uploaded? I know it allows to force upload .gitignore files, but the opposite would be rad.
Use case would be if a project has a React Storybook associated with it, but you don't want the storybook and all the assets associated with it on your now instance.

You can use .npmignore as the filter config as well, which might be suitable in your case.
Additionally, you can specify whitelist files to upload on you package.json or now.json, using the file field or now.files field.
https://zeit.co/docs/features/now-cli#selecting-files-and-directories-to-be-uploaded
{
  files: [...]
}


Update front-door-health-probes.md

@teresayao : Thanks for your contribution! The author(s) have been notified to review your proposed change.


xarchiver: <IP_ADDRESS> -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/xarchiver/versions.
meta.description for xarchiver is: GTK frontend to 7z,zip,rar,tar,bzip2, gzip,arj, lha, rpm and deb (open and extract only)
meta.homepage for xarchiver is: https://github.com/ib/xarchiver
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/v88f68hnrhwylwvai1pcxir545sjnhbl-xarchiver-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/v88f68hnrhwylwvai1pcxir545sjnhbl-xarchiver-<IP_ADDRESS>

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
xarchiver

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v88f68hnrhwylwvai1pcxir545sjnhbl-xarchiver-<IP_ADDRESS> \
  --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 xarchiver https://github.com/r-ryantm/nixpkgs/archive/565e1627b7ba181d95d3550614545a2d0965a635.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/v88f68hnrhwylwvai1pcxir545sjnhbl-xarchiver-<IP_ADDRESS>
ls -la /nix/store/v88f68hnrhwylwvai1pcxir545sjnhbl-xarchiver-<IP_ADDRESS>/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:

xarchiver

Maintainer pings
cc @domenkozar for testing.

@ofborg build xarchiver xarchiver.passthru.tests


The old readback function was designed exclusively for strict mode, and used special reference nodes to ensure the argument was normalized. This approach doesn't work on lazy mode, because the interactions with the special reference nodes will never be reached in normalization.
In lazy mode, one simply needs to call normal_from on a wire to ensure it's normalized; you don't need to do anything with special reference nodes.
This PR just makes it so that the readback function chooses between those two approaches based on M::LAZY.

I'm not sure what you mean.
The old readback function was designed exclusively for strict mode, and used special reference nodes to ensure the argument was normalized. This approach doesn't work on lazy mode, because the interactions with the special reference nodes will never be reached in normalization.
In lazy mode, one simply needs to call normal_from on a wire to ensure it's normalized; you don't need to do anything with special reference nodes.
This PR just makes it so that the readback function chooses between those two approaches based on M::LAZY.

I understand it now. Sorry, all this IO code is a bit hard to understand without an explanation


fix vmname area

Thanks for the PR. This module has been replaced by https://github.com/Terraform-VMWare-Modules/terraform-vsphere-vm.


methodNameBuilder no longer exposes name

Description
Trying to upgrade to the latest version and I noticed that the signature for methodNameBuilder has changed to now just passing a single operation object.
That's fine, however the operation object doesn't appear to have name on it:
interface IROperationObject {
  body?: IRBodyObject;
  deprecated?: boolean;
  description?: string;
  id: string;
  parameters?: IRParametersObject;
  responses?: IRResponsesObject;
  // TODO: parser - add more properties
  // security?: ReadonlyArray<SecurityRequirementObject>;
  // servers?: ReadonlyArray<ServerObject>;
  summary?: string;
  tags?: ReadonlyArray<string>;
}

If i console.log the operation object out, name is there, so this just looks like an accidental omission on the type?
Reproducible example or configuration
https://stackblitz.com/edit/hey-api-client-fetch-example
OpenAPI specification (optional)
No response
System information (optional)
No response

I think you should be able to do if ('name' in operation) {} to narrow it down as the object is a union with the experimental parser type which you shared above

The previous name now seems to be id.


Docs

Initial skeleton of docs for aurora. At this this stage, I don't have them yet successfully deploying.
It is currently deployed on read the docs: https://aurorapy.readthedocs.io/en/docs/

When I was working on the GPG website, I found a nice deploy to GitHub-pages workflow that was very simple. It depends on what you all want to do, but it was very simple.
It essentially deploys from the workflow to a new branch on the same repo (called gh-pages). It is completely separated from the main branch, but it makes it easy to gave it all in the same repo. We can then enable GitHub Pages on this repo, and the docs will then be hosted (to start) at simpeg.github.io/aurora

we could also do a similar setup and have it at aurora.simpeg.xyz too

@jcapriot: moving it over to pages would be great!


TextBinding unbind needs to add original el.textContent back in place

The TextBinding template needs to be reset upon unbind. This comes in handy when dealing with TextBinding in a nested view. I've demonstrated this by explicitly unbinding the view in the JSFiddle below.
https://jsfiddle.net/havessex/1/

@tanshul22 Your example is pretty cool, but very complex.
Can you boil it down to just a rv-text that is bound, then unbound? And tell us what behavior you have an issue with?
I am interested in helping you. But your example is much too complex to isolate rivets alone as the issue.

Thanks for looking into this @Duder-onomy. This issue is exasperated by #622 and thus the update to rv-if.unbind was added to my original example.
I've created a much simpler version of the bug here. As you'll notice rv-text perform fine upon a re-bind but TextBinding does not.
The solution that works for me is to unbind a TextBinding as follows:
TextBinding.prototype.unbind = function() {
      var _ref1, _ref2, _ref3, _results;
      _ref1 = this.view.templateDelimiters;
      _ref2 = this.options.dependencies ? ' < ' + this.options.dependencies.join(' ') : '';
      _ref3 = this.options.formatters.length ? ' | ' + this.options.formatters.join(' | ') : '';
      _results = TextBinding.__super__.unbind.apply(this, arguments);
      this.el.textContent = _ref1[0] + this.keypath + _ref2 + _ref3 + _ref1[1];
      return _results;
    };

I'm curious : what is the use case where you need to unbind the view and rebind it later? When i unbind a view it's to overwrite the template before performing a new bind
Nevertheless your solution will work, but I would do it like this for simplicity reasons
class Rivets.TextBinding extends Rivets.Binding
  constructor: (@view, @el, @type, @keypath, @options = {}) ->
    [...]
    @originalTextContent = @el.textContent

  unbind: =>
    super
    @el.textContent = @originalTextContent

I did not test it, but it should do the job.

Hi @jccazeaux. I like your suggestion of saving off the originalTextContent. I'll give that a shot.
To answer your question, the real problem is better described in the original example which wraps up the text elements inside an rv-if which in my application often gets bound and unbound. Hope that clarifies the concern.

I updated your fiddle here
Basically i replaced unbind with application.view.isFullScreen = false to force rv-if to unbind. In this scenario there is no bug.
It makes sense to reset the template on unbind but we must be sure to not reset it too soon.

 I can no longer recreate the original bug in my app. I have several nested if blocks and this was a corner case. I did my best by replicating it in the original fiddle but the unbind in the fiddle was just a simulation of an rv-if unbinding.
I'll keep experimenting and keep this thread updated with my findings. Thanks for helping out.


For Employee Role - On Add Timesheet page, user can Submit the timesheet even after it is APPROVED.

Updated the code to implement a mechanism to check for the Timesheet status and enable/disable the fields to restrict user from updating the information after the timesheet is Approved.


add hubspot analytics to docs

@jseldess
Can you add HubSpot analytics code to all the docs pages?
`

`
Instructions for how to add it are available here although I think it's pretty intuitive.
Thank you!

Adding code to all pages. @jess-edwards, please verify that it's working.

@jseldess This is working. Thanks for adding the code!
I have an SEO tool we can use on the docs pages at some point.