UnicodeDecodeError occurs when there is pickle binary data with decode_response=True

the thing is that:
>>> from redis import StrictRedis
>>> r=StrictRedis(decode_responses=True)
>>> r.set('data1',{'aa':11})
>>> r.get('data1')
throw UnicodeDecodeError: 'utf-8' codec can't decode byte...

I know it can r.set('data1',json.dumps({'aa':11})) or use r.hmset('data1',{'aa':11}) to slove, but that is example to describe it. there is more complex scene like session pickle to redis or other meanwhile also want to use decode_responses=True.
but if i don't use decode_responses=True. i have to do manual decode() everytime.
So,what can i do if there is pickle data when use decode_responses=True.
My solve is change the connection.py about line 279：
       if isinstance(response, bytes) and self.encoding:
            import pickle
            from pickle import UnpicklingError
            try:
                response = pickle.loads(response)
            except UnpicklingError as e:
                response = response.decode(self.encoding)
        return response

so,can this merge into master or there is better resolution?

decode_responses is designed to be used when you want to return data in a specific character set. It is not designed to automate type casting when dealing with dicts, lists, or your own data structures.
If you store both pickled data and encoded character set data (let's say UTF8) in your Redis instance, then I'd suggest creating two separate clients and connection pools to the same server:
redis_native_types = StrictRedis(decode_responses=True)
redis_pickled = StrictRedis(decode_responses=False)

Then use the appropriate client based on what data you're fetching/retrieving.

thinks @andymccurdy . I see it.  Your solution is very useful for me. thinks.


[Tables] Adding swagger.md

Adding swagger.md file and adding directive to change TableEntityProperties to a [][]byte

I think this should be in Draft?

I think this should be in Draft?

@chamons ready for your review now :)

How about naming it autorest.md or readme.md? Is there a reason for this specific name?

Changed to autorest.md, no reason for this specific naming.

@chamons 👍 ?


why the inference results are not aligned with the validation results?

Hello,  i used the weights saved after the training step to inference, but the results are not aligned with the results generated from the last validation step in training. So what is the reason for this phenomenon?

Hi. Typically, this is because the randomly sampled noises are different from the training stage.
Could you please provide more details? Like are you training MotionDirector on a single video or multiple videos?

Hi. Typically, this is because the randomly sampled noises are different from the training stage. Could you please provide more details? Like are you training MotionDirector on a single video or multiple videos?

Thanks for your reply, your answer helps, it is maybe different settings between training and inference. I have another question,  how is the generalization ability of MotionDirector? For example, if i use a custom dreambooth weight, which is different from training. Can the temporal weights trained from MotionDirector work under this situation?

If you use DreamBooth to only finetune the spatial layers, I think it is OK, just like the results shown here. If the temporal layers are also changed, I'm not sure what will happen. You can try it out. Looking forward to your insights.


Add C++ Backend For New Filter Option -- Chart Clear %

This adds in a new filter option -- a clear % range.
Charts are only matched in the filter if the player's best score percent on a chart falls between the lower and upper bounds.
This also respects the SSRNormSort option -- if it is enabled, that clear % is used, if it is disabled, the % of the judge used when the song was played is used instead.
This also introduces a small filter change -- certain filter options (currently duration and clear %) must be matched, even in a non-exclusive filter.
This is done because no-one wants to filter for "Charts that are (between 21-23 JS), (between 20-22 HS), or (have been cleared with at least 60%)", they want "Charts that are ((between 21-23 JS) or (between 20-22 HS)) and (have been cleared with at least 60%)".
The easiest way to test this new filter option is to modify FilterManager::ResetSSFilters() by temporarily adding in lines like the following at the end for testing:
SSFilterLowerBounds[NUM_Skillset+1] = 70;
SSFilterUpperBounds[NUM_Skillset+1] = 90;

TODO: The C++ front end for Til Death has not yet been made, so it is not yet possible to use the new filter range.
I will implement that functionality in a separate PR.

I finished the UI for this since it took a while and after looking at it I figured it would be an actual nightmare to deal with
testing took place in that UI
UI is restricted to number entry only, allowing input of between 1% and 99.999%
the decimal is automatically added after entering 2 numbers
so 1-10 are single digit percentages only (who is really going to want to filter with a bound in that range anyways)


Corporate Console doesn't show 'Signed By' and 'Download PDF' icon for a signed CCLA.

Steps:

Login to Corporate console and search for project 'PerfTestFoundation-Child1'

Actual Result: It shows CCLA is signed but 'Signed By' is empty and no download option shown.

Expected Result: 'Signed By' and 'Download' option should be displayed.

Sorry, I didn't realize you were going to do detailed functional testing on these generated user accounts. The signature records are missing the CLA Manager entry in the signature record (Signature ACL) as well as the company signatory name/email which signed the DocuSign document. Since we didn't actually sign the DocuSign document, I didn't fill in this field. I will need to go back and randomly assign a user to the existing records.  Additionally, I will need to explicitly set the Download URL in the signature record. V1 actually calculates the PDF path/URL. v2 seems to lack this logic and reads the download URL value from the record.  I will also need to update the generated records to set this value.
  I will work to make it more realistic by setting the CLA Manager info, signatory name/email, and properly setting the download URL.

Ran a migration script to set the CLA Manager name. I chose one randomly from the Staging database.

@dealako Ok. Thanks for generating the data, I can see Signed By name and PDF link to download.

However, I am unable to download the pdf and it shows this error as shown below. I believe its because the PDF doesn't exists in reality ? Please confirm.

For Document went stale, there is a separate ticket.
So, as updated by @ashav-github  we can mark it as QA Done.


isMAC() fixed

After combineMulAdd(), the opcodeName changes.  So isMAC() detects new opcodeName to verify whether it is MAC and isMAC() won't always return true.

Which fiction fuse mul with add into new muladd?

Function combineMulAdd(). But it's not used in this mapper in fact.

Function combineMulAdd(). But it's not used in this mapper in fact.

Can you please apply that combineMulAdd() on a fir kernel and show the two DFGs (before and after) here?

Function combineMulAdd(). But it's not used in this mapper in fact.

Can you please apply that combineMulAdd() on a fir kernel and show the two DFGs (before and after) here?

Fir kernel doesn't contain a mul and add that can be combined, So I show inttCore kernel (apply combine("mul","sub") before and after) instead.
I notice that combineMulAdd() is not as useful as combine("mul","add") or combine("mul","sub") or others, because it only combines Critical dfgNode.
Note that (19)mul and (21)sub are combined in InttAfterFuse.png and become (19)mulsub.
InttBeforeFuse.png

InttAfterFuse.png

Why 19 and 21 are combined rather than 20 and 21? Which pair has the higher priority to be combined in your opinion?
FIR kernel has a fmul followed by fadd. So that pair can also be combined, right?
The main purpose of this Pull Request (PR) is that we user enable MAC (or FMAC) in the GUI, all (or some of) the mul and add (or fmul and fadd) need to be combined.

I think (19)MulAdd means it is (19)Mul other than (20)Mul.
I notice it too. I think it is because fadd has a circule with phi, making this pair can't be combined.
I will check the launchUI.py and rewrite the code (including the above two). Traveling to school soon and will have plenty of time for programming :D

You didn't answer my first question. I was asking why 19 mul is chosen but 20 mul is not chosen.
Which line of code avoids such fusion?
Thanks for the contribution, take your time, and enjoy the holidays.

I will check the combine function later to answer your question, Sir. Right now, I have a busy time struggling with my relatives QAQ.

Why 19 and 21 are combined rather than 20 and 21? Which pair has the higher priority to be combined in your opinion?
FIR kernel has a fmul followed by fadd. So that pair can also be combined, right?
The main purpose of this Pull Request (PR) is that when users enable MAC (or FMAC) in the GUI, all (or some of) the mul and add (or fmul and fadd) need to be combined.

I print the chosen order of combine() function and find that 19 mul is analyzed before 20 mul, so it's chosen first.
I remove isCritical() from combineMulAdd(), so fmul can be combined with fadd now. Note that the changed code is not in this PR.
Since MAC is enabled by default, set combineMulAdd() positive will address your issue.

Below is Fir before and after combineMulAdd()(the one I remove judgment of critical nodes). Note that (6)mul and (7)add are combined and become (6)fmuladd.
firBeforeFuse.png

firAfterFuse.png

Task 1: Don't you think 20 should have higher priority to be combined? Any thoughts?
Task 2: Can you also provide a fusion function that fuse the add/compare/br/phi (i.e., 1/9/10/11) into single op? Similar to the add/phi (i.e., 0/7)?

Nice.

I think there are priorities for fusion. The ones in the critical path has the highest priority, say in your attached figure, the phi is not able to be fused with add to decrease the RecMII to 1, because the add is fused with mul first. For the 13 to 25 case, the ones that can decrease the overall height should have higher priority (agree? This may help the mapping. But I think you don't need to implement this case for now). Please focus on increasing the priority of the fusion on critical path first.
Rename DFG::combineAddCmpBranchPhi() to combineForIter()? The function should also combine 0 and 7.
Can you try to provide some tests for your modifications?
Thanks

So the purpose of combineForIter() is to combine any kind of critical path, no matter it's add/compare/br/phi or add/phi (those two are the most common ones but there exit other critical paths)? In that way, I need to change the logic of combineForIter().
I also want to know if this task is relevant to the mail you sent to us (Help needed ...) before or not.

Yes, we want to fuse all the nodes within the critical cycle. However, we also want to make it parameterizable, for example, some cases, we may not want to over fuse them. So we can start from the above two cases I guess?
Yes, this is related to the mail. Fusion is a critical step towards better mapping.

Cool, great progress. Recursive way is very popular for pattern recognition. I am fine with either recursive or iterative.
It would be perfect to also fuse the pattern in the non-critical paths. But if you think that is not trivial engineering effort, we can leave it to another PR.

DFG::combineIter() and DFG::combineForIter() is made to fuse the pattern in DFGs regardless the order or size of the pattern. I update the codes and you can merge the PR if there is no question.
Here is an example of DFG of fir after fusion where the pattern is "add -cmp-br-phi" and "phi-add".

combineForIter() is to accept fusion pattern that user provides and it will call combineIter().
combineIter() is a recursive function that fuse patterns.
Here is an example of code of fusion pattern "add -cmp-br-phi" and "phi-add".
    list<string>* combinePattern1 = new list<string>[4];
    combinePattern1 -> push_back("add");
    combinePattern1 -> push_back("icmp");
    combinePattern1 -> push_back("br");
    combinePattern1 -> push_back("phi");
    list<DFGNode*>* cPatternNodes1 = new list<DFGNode*>[4];
    combineForIter(combinePattern1, cPatternNodes1);

    list<string>* combinePattern2 = new list<string>[2];
    combinePattern2 -> push_back("phi");
    combinePattern2 -> push_back("fadd");
    list<DFGNode*>* cPatternNodes2 = new list<DFGNode*>[2];
    combineForIter(combinePattern2, cPatternNodes2);

I update combineForIter() with one parameter and fix the name issue. The result DFG is the same as before.
    list<string>* combinePattern1 = new list<string>[4];
    combinePattern1 -> push_back("add");
    combinePattern1 -> push_back("icmp");
    combinePattern1 -> push_back("br");
    combinePattern1 -> push_back("phi");
    combineForIter(combinePattern1);

    list<string>* combinePattern2 = new list<string>[2];
    combinePattern2 -> push_back("phi");
    combinePattern2 -> push_back("fadd");
    combineForIter(combinePattern2);


BIOS beep on every bar collision

It's gonna be too annoying during ball penetration phase (when a point is scored every frame).


bitcoind-abc: 0.29.6 -> 0.29.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/bitcoin-ABC/bitcoin-abc/releases.
meta.description for bitcoind-abc is: Peer-to-peer electronic cash system (Cash client)
meta.homepage for bitcoind-abc is: https://bitcoinabc.org/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.29.6 with grep in /nix/store/xk2fv4vprd7p52mh98ywg24f9hd7a968-bitcoind-abc-0.29.6
found 0.29.6 in filename of file in /nix/store/xk2fv4vprd7p52mh98ywg24f9hd7a968-bitcoind-abc-0.29.6

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

bitcoin-abc
bitcoind-abc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xk2fv4vprd7p52mh98ywg24f9hd7a968-bitcoind-abc-0.29.6 \
  --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 bitcoind-abc https://github.com/r-ryantm/nixpkgs/archive/7eb5267703bee492a002f80a98453b97f9a9f12a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/7eb5267703bee492a002f80a98453b97f9a9f12a#bitcoind-abc

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xk2fv4vprd7p52mh98ywg24f9hd7a968-bitcoind-abc-0.29.6
ls -la /nix/store/xk2fv4vprd7p52mh98ywg24f9hd7a968-bitcoind-abc-0.29.6/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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

bitcoin-abc
bitcoind-abc

Maintainer pings
cc @Lassulus for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/bitcoin-ABC/bitcoin-abc/releases.
meta.description for bitcoind-abc is: Peer-to-peer electronic cash system (Cash client)
meta.homepage for bitcoind-abc is: https://bitcoinabc.org/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.29.7 with grep in /nix/store/f6z9237lc3fjwzv5lvj5k50b7fy6rv4x-bitcoind-abc-0.29.7
found 0.29.7 in filename of file in /nix/store/f6z9237lc3fjwzv5lvj5k50b7fy6rv4x-bitcoind-abc-0.29.7

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

bitcoin-abc
bitcoind-abc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/f6z9237lc3fjwzv5lvj5k50b7fy6rv4x-bitcoind-abc-0.29.7 \
  --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 bitcoind-abc https://github.com/r-ryantm/nixpkgs/archive/2466aa7a209b518a05dafa52dd262fda1808b719.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/2466aa7a209b518a05dafa52dd262fda1808b719#bitcoind-abc

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/f6z9237lc3fjwzv5lvj5k50b7fy6rv4x-bitcoind-abc-0.29.7
ls -la /nix/store/f6z9237lc3fjwzv5lvj5k50b7fy6rv4x-bitcoind-abc-0.29.7/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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

bitcoin-abc
bitcoind-abc

Maintainer pings
cc @Lassulus for testing.


Figma Desktop App out of date

When I start the application, figma tells me
"Figma Desktop App is out of date! Please download and install the latest version of the Figma Desktop App."
Updating the snap package does not yield any updates.

I have the same issue in Linux Mint 19.3, uninstall and download it from snap again and still displaying the same message

I have the same issue in Ubuntu 20.04 LTS. Snap or PPA same error.


Use reciprocal of tempo when syncing time signals to Transport

Previously, signals with time units were synced to Transport by connecting it to the tempo signal, which has frequency units. This unit mismatch caused incorrect behavior, e.g. a time signal would double instead of halving as expected when bpm doubles. Now, signals with time units are connected to the reciprocal of the tempo signal instead, which also has time units.
fixes #879

On further thought, this PR has an issue whereby the new intermediate signal operators don't get garbage collected when the signal is unsynced. I'll fix this shortly.

Great PR! thanks you!


update new files only?

Jiaqi:
I am curious if your plugin does update new files only?
in other words, if source & target file has same time stamp,
will your wagon run upload anyway?
thanks
Andrei.

Reading code in https://github.com/jiaqi/cym2/blob/master/awss3/src/main/java/org/cyclopsgroup/cym2/awss3/S3Wagon.java, which I completely forgot, I don't think it checks timestamp. Though I am not 100% sure about the base class.
I often make a few file changes and call wagon to upload entire site to S3. According to logging, it does looks like maven iterate through every file no matter it's changed or not and upload them. There's noticeable delay during upload of each file. So I believe it always upload all files without looking at timestamp.
Regards
Jiaqi

ok; I guess I have to steal your code an play with it a bit
to see if I can convince maven/wagon get smart :-)

Good luck on that. There are several gotchas I want to bring out, which you may already know.

Don't easily trust timestamp. Time on each OS may drift or may not make sense at tall. If you want it to be selective and checking last modified time, make it optional so that user can still choose to upload everything if he wants. Checksum is much better in this case.
I think AWS Java SDK has a smart putObject method, which takes File as input and does some checking and preparation that it can't do for InputStream. I'm not sure how much optimization it does through.
The current wagon class extends StreamWagon, which may not give you much room to improve. To be really smart, you probably have to throw away StreamWagon and do a lot of work by yourself.

great ideas, thank you for sharing;
I will keep you posted if I come up with a solution;


The library has not been transpiled from ES6 to ES5 for browser support

Describe the bug
The main reason raising the issue is that common approach for such npm packages is being supportable by any browser. One of the way to reach it out is to transpile code written in ES6 to ES5 before moving it to npm repo. I know that this is a discussable question whether it should be consuming developer's or the library developer's responsibility of transpiling the library. But as easier to use your product for your customers as better for you, right?
To Reproduce
As the result, the outgoing error SCRIPT1028: Expected identifier, string or number appears using EDGE browser any version. ES6 rest operator has not been transpiled to ES5, and EDGE doesn't support rest operator unlike Chrome, Firefox etc.
Code reference: /react-calendar-timeline/src/lib/utility/calendar.js:624
const newItem = {
    ...item, //  <----  SCRIPT1028: Expected identifier, string or number
    [keys.itemTimeStartKey]: itemTimeStart,
    [keys.itemTimeEndKey]: itemTimeEnd,
    [keys.itemGroupKey]: isDragging
      ? _get(groups[newGroupOrder], keys.groupIdKey)
      : _get(item, keys.itemGroupKey)
  }

Expected behavior
The code of the library is transpiled to ES5 (on npm).
Library Version
any
Desktop (please complete the following information):
To see the example of caused issue:

OS: Windows 10
Browser: EDGE
Version: any

As a workaround follow the solution - https://github.com/babel/babel-loader/issues/171
Summary:

Add 'exclude but one' rule to your webpack configuration file:

const rules = [
       ...,
       {
              test: /\.(js|jsx)$/,
              loader: 'babel-loader',
              exclude: /node_modules\/(?![react\-calendar\-timeline])/
       },
       ...
]

If you use Babel 7, you should switch .babelrc to babel.config.js (if you haven't done it before). https://babeljs.io/docs/en/configuration#babelconfigjs

@EugeneBugor you can use the transpiled version from react-calendar-timeline/lib


[Production & Staging] Dropbox add-on does not show folder picker on auth

Steps

Go to a project you have admin privileges on
Go to the project settings page
Enable the dropbox add-on
Click to import the access token
If the folder picker appears, refresh the page. If not, Deauthorize and import the access token again
Attempt to click on 'change' to change the selected folder (which is none)

Expected
When importing the access token, the folder picker would appear or an error message would be shown to the user if the token is invalid. Clicking to 'change' the folder would show the folder picker to the user.
Actual
The folder picker does not appear, clicking the 'change' button to envoke the folder picker below to change the folder does no action.

@lbanner  @GaryKriebel
Can you please assign me?

You got it!

Resolved in https://github.com/caneruguz/treebeard/pull/81,
defer until @caneruguz's next feature PR for OSF goes in

@lbanner the PR @samchrisinger is referring to has been merged and it is in production. Can we test this and close based on result?

:+1:, this is already tested

OK good -- I'll close -- TY


add dependabot.yml for C3 framework clis updates

resolves #3805

Based on my testing this should work as expected, but I think we can only be 100% sure of it after we merge and check how it behaves.

Question, as you can see here with this we get one PR per dependency, although this is nice since it gives us more flexibility/granularity it could be annoying if we end up with too many smaller PRs
I should be able to group all the PRs together via the groups config
But that could be more problematic/more cumbersome to deal with, for example in the case C3 has an issue with some but not all upgrades (we'd have to manually create/separate different PR for different updates)
Thoughts?

@dario-piotrowicz as discussed, during the initial testing period (1-2 weeks) let's have it send individual PRs and based on how often they "just work" and can be merged without hassle we'll switch to weekly update cadence and potentially group all updates into a single PR. We need more data to make that decision, so let's get that first.


Deprecate the azure mixin for az+arm

What does this change
Keep the old page around just to tell people to use az+arm instead. I've added a placeholder page for the arm mixin. Once I have examples for the arm mixin, we can update it to match the other pages.
What issue does it fix
None, just general housekeeping.
Notes for the reviewer
I like lucky charms. ☘
Checklist

[ ] Unit Tests
[x] Documentation

[ ] Documentation Not Impacted

/azp run porter

/azp run porter

I kicked the build and it really passed. It's just showing up twice (once failed and once passing). So merging with my super powers. 💪


Checksum/GPG signature for draw.war files

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Is your feature request related to a problem? Please describe.
Currenctly there's no way to verify the integrity of the downloaded draw.war files.
E.g. on https://github.com/jgraph/drawio/releases/tag/v13.4.0
Describe the solution you'd like
Could you please add a checksum file and or GPG sign the file (not sure it that is possible with .war files).
Many thanks.

Out of date since we don't develop the java back-end any longer.


update .gitignore to ignore .vs folder

I update .gitignore to ignore .vs folder, because I often open this repo in Visual Studio, not just in any other text editors or other IDE.

Docs Build status updates of commit 4ad7e2c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.gitignore
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@JonDouglas
Could you review this simple PR?


CI tests are failing

Possible cause: newer versions of the packages gdal-bin, libgdal-dev and/or python3-gdal are installed compared to earlier.
See also #4
Solution: adapt testcases to work with these package versions

Protobuf upgraded, issue resolved.


dep --help doesn't work if not in GOPATH

$ dep --help
project not in a GOPATH$

It should be possible to get help whereever the command is run from.
(also the error should print a newline)

seems like a fun one for a new contributor :)

Can I try to take a stab at this? Would like to get a PR in to learn the process, but have been waiting on something with a smaller scope for my first one.

yes! please!
On Tue, Jan 24, 2017 at 10:36 AM Kris Nova<EMAIL_ADDRESS>wrote:

Can I try to take a stab at this? Would like to get a PR in to learn the
process, but have been waiting on something with a smaller scope for my
first one.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/golang/dep/issues/125#issuecomment-274894323, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABYNbD2Dnd1PxSFF1DSrCHECWPph6ed0ks5rVkSbgaJpZM4LsgAR
.

for some context on what the problem may be, I think this is calling newContext() before displaying output :)


Rebrand from "aut" to "autonity-cli"

The project name "aut" can be improved. Feedback received is that it is ambiguous when used in conversation, and often needs to be qualified further with the redundant "tool" suffix.
This change is simultaneous with repo URL change and shortly publishing the tool as a PyPI package.
Also, expand introduction in the README and use the term "command-line" consistently.

I think we're good


Failed: Update autorest version to prerelease 3.0.0-alpha.20240522.6

Triggered from pull request: https://github.com/Azure/autorest.csharp/pull/4432

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.NotificationHubs


FI-1536-sort-presets

Sorts presets alphabetically, with None at the beginning of the list.

Issue raised in #175.


Fix #300: Stream sync get object responses

@andrewbaxter thank you for the fix, I've been vaguely aware of it

Thanks for the review and merge! :bow:


Something funny going on

Hey --
It seems like there something funny going on here in the interaction between hungarian and lapjv.  If you run lapjv.lapjv(cost) after running hungarian.lap(cost), then it runs very fast (< 1 second, as reported in your notebook).  However, if you run lapjv without first running hungarian, it takes way longer (> 20 seconds on my machine).  Looking closer, hungarian.lap(cost) actually modifies cost, which is unexpected.  Any ideas?
~ Ben

thanks! i suspected something like this was happening but never got too deep into it.
before the core of the JV algorithm kicks in, the cost matrices go through an initial set of "reduction" steps to convert them to a form that is easier to work with. if the hungarian algorithm already did some complex reductions (which i think it did) then the JV algorithm can run faster.
i changed the notebook to use np.copy to avoid this mistake.
but if you really want better performance out of the JV algorithm, make sure to cast to integers first and you should get back most of that speed. sorry for misleading!
closed by 8993036fc04176779f3f0d368a6efce7e1b36e5d


ScrollView Moving to initial position and then coming to the position required  if we are using scrollTo.

Description
We are using scrollview and needed to focus on some Item in the scrollView. We have used scrollTo to move scrollView to a position and made animation to true. But when the scrollTo is working first it is moving to the initial position and then coming to the position required. Working fine on Apple tv but causing issue in Android tv
Version
0.68.2-2
Output of npx react-native info
System:
OS: macOS 13.4.1
CPU: (8) x64 Intel(R) Core(TM) i5-1038NG7 CPU @ 2.00GHz
Memory: 185.94 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 16.18.1 - ~/.nvm/versions/node/v16.18.1/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 8.19.2 - ~/.nvm/versions/node/v16.18.1/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.12.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
Android SDK: Not Found
IDEs:
Android Studio: 2022.2 AI-222.4459.24.2221.10121639
Xcode: 14.3.1/14E300c - /usr/bin/xcodebuild
Languages:
Java: 11.0.19 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: Not Found
react-native-macos: Not Found
react-native-tvos:  0.68.2-2
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Add few touchable inside a scrollview and try to move scrollview using scrollTo
Snack, code example, screenshot, or link to a repository
NA
Uploading testing.webm…

You might try doing the scrollTo within a setTimeout. See related issue https://github.com/facebook/react-native/issues/6849#issuecomment-225124536

Also, if you're not already doing it, you might try setting scrollEnabled={false} on the scrollView to disable the default Android scrolling behaviour (since you're doing it imperatively with scrollTo(). I had some really jittery scrolling experiences when using scrollToIndex() and scrollEnabled=true at the same time.


Fix E2E tests and namespace deletion validation

When deleting a parent namespace, we should allow it if none of its
children are sub-namespace.
In the E2E test, I moved the webhook deletion out of AfterEach, so that
if the user doesn't have HNC_REPAIR path set, it won't delete the
webhook.
Since we changed the deletion validator, one of the tests for issue #716
does not apply anymore. I deleted it in this PR.
Fix #1269
Tested: make test-e2e

/assign @adrianludwin
/assign @yiqigao217


Build is broken again

It looks like Ultimate Guitar changed the tab type again.

Fixed in version 1.0.1 🎉 !


[FLOC-4016] Add a place to hang persistent non-discoverable state.

https://clusterhq.atlassian.net/browse/FLOC-4016

Seems reasonable and I'm glad we're tackling this bothersome AWS annoyance.
One concern though, this PR adds complexity in some of our core logic, and it's hard to evaluate in isolation. Any chance you could add some comments for the new classes and arguments to show how they will be used? I'd like to review it after those comments are in-place.

Thanks review. I've been looking at this code long enough that having a fresh pair of eyes looking at it is useful to see where things aren't clear.
The big distinction that I've tried to clarify is that there is "configuration", "discoverable state", and "non-discovreable (or persistent) state". We previously only had the first two. The third is for when flocker makes a decisions (for example) and needs a record of it, because the underlying system can't consistently report it.
Looking ahead: when we create a blockdevice volume, we record the association of it with the dataset it is for. This is basically a choice the deployer is making: this volume I created is going to be associated with this dataset; there is also some metadata on the volume, but there isn't a way of preventing multiple volumes from being created with that metadata. This way, if the backend doesn't immediately report that volume, we don't continue creating new volumes for that dataset.

Thanks for the comments, makes it much more easy for me to understand the motivation. I probably could have gotten there myself if I had thought long enough, but it's much nicer to just read the documentation since you already thought it through :).
My 1 concern is that we have a fair amount of code that assumes that a Deployment is 1:1 with cluster configuration (git grep Deployment | grep configuration shows things like:
flocker/node/functional/test_deploy.py:        desired_configuration = Deployment(nodes=[
and
flocker/control/_protocol.py:        :param Deployment configuration: The desired configuration for the
It seems to me we might be taking on a bit of tech debt with this change. At least at some level it seems like it might be a bit harder to understand how flocker works by grepping through our code. You might see one of the above descriptions and assume that Deployment only consists of flocker-user-supplied-configuration.
However, most of the references are in tests, I haven't done a full audit, but I don't think this breaks any uses of Deployment, and the cleanup is sort of under the realm of the XXX that you added to the code. So... I don't think that should block this PR. At your discretion, open a FLOC-XXX ticket to go along with your XXX so we have a place to add comments and discuss what cleanup if any should occur to offset any confusion between "configuration" and "persistent state" in the documentation around the flocker codebase.
Looking forward to seeing the interface for changing the persistent state from the dataset agents. :smile: Thanks for entertaining my review comments.

Also, merge when test are green.

It seems to me we might be taking on a bit of tech debt with this change.

Yes. But I think the cost is worth it.

Thanks for entertaining my review comments.

Thanks for making them, I'm much happier with the documentation than before.

My 1 concern is that we have a fair amount of code that assumes that a Deployment is 1:1 with cluster configuration [...]

I suspect that

most of these places also assume that there isn't any non-discoverable state.
most of these places don't care about any state (I think most state is likely at least somewhat deployer specific, and so anything that cares about state will need to be adjusted to deal with it anyway.

I probably could have gotten there myself if I had thought long enough, but it's much nicer to just read the documentation since you already thought it through :).

I'm glad you didn't since it gave me the chance to improve the code in response to your confusion.


Detect delegate to self

I discovered some code like this:
delegate :foo, to: self

foo is a class method. So calling foo in an instance method would be equivalent to self.class.foo.
I can't see how this confusing indirection would ever make sense.
Any thoughts? Should rubocop-rails detect this?

To check how common this is, I grepped the set of apps from real-world-rails and only found a handful of occurrences:
discourse/app/models/category.rb
110:  delegate :post_template, to: 'self.class'

e-petitions/app/models/constituency.rb
17:  delegate :query, :example_postcodes, to: "self.class"

foreman/app/models/compute_resources/foreman/model/ec2.rb
7:    delegate :security_groups, :flavors, :zones, :to => :self, :prefix => 'available'

foreman/app/models/subnet.rb
104:  delegate :supports_ipam_mode?, :supported_ipam_modes, :show_mask?, to: 'self.class'

solidus/core/app/models/spree/promotion_code/batch_builder.rb
24:    delegate attr, to: self

zammad/lib/mixin/rails_logger.rb
7:    instance_delegate [:logger] => self


use updated dockerdns

Upstream dockerdns was using a very old version of the docker-py libraries that could not handle some of the weirder shit docker does. This fixes that.
To test this you need to update to the latest docker (26 and above)
just kaiser up any project.

@sikachu can you help us review this PR, please?


Kaufman's Adaptive Moving Average (KAMA) - calculation issue

import yfinance as yf
from pandas_ta.overlap import kama
import mplfinance as mpf

ticker = yf.Ticker("SPY")
minute_data = ticker.history(period="7d", interval="1m")

kama_results = kama(minute_data["Close"])
addplot = [
    mpf.make_addplot(kama_results, color='blue')
]
mpf.plot(minute_data, volume=True, type='candle', addplot=addplot)

I tried using the Kama indicator (snippet and picture above), seems like there's a bug with the calculate the Kama. The initial Kama value relies on the previous value which I believe from the snippet below returns zero for the first value.
https://github.com/twopirllc/pandas-ta/blob/bc3b292bf1cc1d5f2aba50bb750a75209d655b37/pandas_ta/overlap/kama.py#L34
This doc was referenced on a comment in the kama.py file and suggests that "the first KAMA is just a simple moving average".
Is there anyway we can add another arg for starting_kama or generate some sma based on the "slow" length?

Hello @onachassi,
I added sma as the default MA mode for kama on the development branch if you want to test it out before I update the main branch. Note: Not all mamodes are applicable due to their individual bootstrap lengths, but the majority are. See help(ta.kama) after installing the development branch.
$ pip install -U git+https://github.com/twopirllc/pandas-ta.git@development

Please let me know if it works as intended now. Thanks for noticing the Issue. 😎
Kind Regards,
KJ

@twopirllc I'm getting the same thing as before. Taking a look at the development branch I don't think anything has changed: https://github.com/twopirllc/pandas-ta/blob/development/pandas_ta/overlap/kama.py
Am i missing something?

https://stackoverflow.com/a/73619226/1233573


Question: Limiting the size of memory use

Wanted to know how to limit size of the RAM influxdb can use, then it start using Memory Mapped IO (disk) for additional space? We have huge data set, but users shall be running influxdb in limited capacity laptops.

@gopalakrishnan-subramani There is currently no method for controlling InfluxDB's memory usage. Running out of memory is a pretty common issue when using expensive queries, so I'd recommend your users exercise caution when attempting to query results over a large dataset.
That being said, the tsm storage engine (default with 0.10.0) is extremely memory efficient, and we have been seeing much less issues with memory since the release. You may still see issues if attempting to run queries over a large number of series on underpowered machines, though. Also if performance is not critical for your use-case, then you can also utilize swap space to prevent InfluxDB from triggering the system's OOM-killer. You can also disable Linux's memory overcommit settings if needed.
Regulating memory use will also be a target of development with the next few releases, so we should start to see improvements in this area relatively soon.


[PowerAutomate-TestEngineModule - Feature]: Sample Loop

Is your feature request related to a problem? Please describe.
Extend on the basic example to demonstrate how to unit test a loop.
Describe the solution you'd like
Implement new samples that demonstrate how to create simple and edge cases for loops
Describe alternatives you've considered
No response
Additional context?
No response

Added testPlan.fx.yaml that currently fails today as it only evaluates actions but not the execution of foreach
Expected count should be 6

Initialize_variable, called once (Set the count to 0)
Initialize_variable_1, called once (Set array to ["A","B","C"])
Apply_to_each, called once
Increment_variable, called three times (Increment count 1, 2, 3)

Assert

Verify that the total number of actions 6 and number of calls per action
Value of Counter is 3


Please, add an easy way to copy the text that generates.

It would be great to be able to copy the text it generates. Not always we are looking for code, but for analysis, explanations and critique+improvements.
But it's great. Can't complain.

v1.0.6 now has the ability to export tasks to markdown. I hope this helps you get what you were looking for, but if not feel free to re-open with more specifics.


In Firefox label and star icons have gray background that does not look attractive

In the pre-release beta deploy in Firefox the My Resources graphics have a gray background.

Changed the style to accommodate for this browser.

Verified fixed.  Thanks!


Call description notes

Should this be for every instance where a proposal is listed? Or is this referring to the "Set Pairing" page where it's already been added? There's also a number of locations where the proposal information just doesn't fit well on the standard grid row layout and could use some customized displaying I could implement.


Gzip compression

In anticipation of v0.7 support for gzip compression, may I suggest that it is implemented in a way that goes one step further than Cowboy. Instead of a binary on/off option, Bandit could allow the user to set the gzip compression level, like Nginx and Caddy do, as follows for Phoenix:
config :prj, Endpoint, http(s): [gzip: 0-9]
The default :gzip value would be zero (no compression). There would be no pre-selected compression level, and that is a good thing. Nginx's default seems to be 1, Caddy's 6. The Bandit documentation could recommend a positive value or range (e.g. 3 or 1-6), but leave the actual decision to the user. In the future, there could also be other options, such as:
config :prj, Endpoint, http(s): [zstd: 0-22]
In case of conflict, e.g. http(s): [gzip: 2, zstd: 4], the obvious resolution would be to default to gzip.
I hope this helps.

Since http compression only compresses the body and seems to be the same between http 1 & 2 couldn't we implement compression as a Plug without modifying the server?
I played around a bit with it yesterday and it seemed to work without problems (https://github.com/moogle19/plug_compression).
But your suggestion to make the compression configurable sounds good either way.

@kotsius absolutely! The goal for 0.7 is twofold: first, add a couple of missing things around the project (like compression), and second, make a lot more (non-policy) things configurable. You can expect to see (these are my rough internal notes):

Compression in HTTP/1 (and 2?)
WebSockets over H2?
Upgrade to h2c via Upgrade header
Configurable limits on reading (header count/length, uri / body / frame sizes etc)
Enhance configuration

Add support for defining permitted protocols / upgrades
Add support for connection level recv timeouts in TI
Add support for changing HTTP2 config values
Add Plug.SSL based support for TLS1.3
Add easy levers for cert config?
Add config point to show packets sent / received (via telemetry)
Add config to shutdown delay (TI todo)
Add config to WS compression level / direction / global disable (also benchmark this to see if it’s slower. It seems it is)
Add support for max_frame_size on WS
Explore max size limits for H1 & H2

Overall readme and docs

In terms of implementation, I agree that a Plug based approach for this should be possible. I'm thinking about refactoring the Plug pipelines within the HTTP/1 and H2 implementations to use a common pipeline, which would make things like this really easy. The main difficulty with it is the semantic differences between the two protocols surrounding transfer encoding abstractions. We'll see how it goes.


Create no-jquery-selector rule

Currently, $('.some-class') to get a DOM element and this.$() to get a component's element aren't caught, yet they are probably the biggest usage of jQuery.
Added more error messages for selector vs method jQuery usage and of course some tests.
@chadhietala @trentmwillis @lynchbomb @scalvert

Thoughts on creating a new rule for linting against jQuery in its entirety, in-which this example might align closer to? The no-jquery-method rule is intentionally for linting against specific blacklisted jQuery methods.

So I thought about that, but figured there wouldn't be a lot of cases where devs didn't want a method used ($('.elem).addClass('.class')), but were okay with $('.elem).
But I agree, this should prob be another rules for simplicity's sake.

Marc is on vacation, can one of @chadhietala @trentmwillis @scalvert take a look?


feat: import toolchains to use for copy actions

Register the "coreutils" toolchains using definitions imported from the bazel_lib Bazel repository in order to allow use of this module with bazel_lib version 2 and later.

Type of change
New feature or functionality (change which adds functionality)
Test plan
Covered by existing test cases
See aspect-build/rules_esbuild#165 for precedent.
Fixes #134.

The CI failure with Bazel 5.3.2 looks familiar:
ERROR: /home/runner/.cache/bazel/_bazel_runner/3583e71b01fa1c6a4fe6bf126ed74da3/external/aspect_bazel_lib/lib/private/BUILD.bazel:253:12: no such target '@local_config_platform//:constraints.bzl': target 'constraints.bzl' not declared in package ''; however, a source file of this name exists.  (Perhaps add 'exports_files(["constraints.bzl"])' to /BUILD?) defined by /home/runner/.cache/bazel/_bazel_runner/3583e71b01fa1c6a4fe6bf126ed74da3/external/local_config_platform/BUILD.bazel and referenced by '@aspect_bazel_lib//lib/private:platform_utils'

We'll have to green this up before landing. I'll look deeper later tonight or tomorrow when I have some time.

The CI failure with Bazel 5.3.2 looks familiar:

Just by way of the error message, it sounds like it may be related to bazelbuild/bazel#15175, though in this case the failure was with Ubuntu Linux, not macOS.
Reading further, there is aspect-build/bazel-lib#392. You noted in https://github.com/aspect-build/bazel-lib/issues/392#issuecomment-1464012521 that there's a workaround that my patch here undid, by way of removing the "override_local_config_platform" attribute value supplied to aspect_bazel_lib_dependencies. That was me just aping what I saw in the similar patches against rules_esbuild and rules_ts. Should I try adding that attribute back in?

Should I try adding that attribute back in?

Commit 7848eb2e5b7a2167d6c0abddf9960a566d0c331f adds that "override_local_config_platform" attribute back into the call to aspect_bazel_lib_dependencies in the WORKSPACE file.

Can we please allow this most recent commit to pass through the CI workflow? I'd like to see whether it resolves the previous problem.

Commit 7848eb2 adds that "override_local_config_platform" attribute back into the call to aspect_bazel_lib_dependencies in the WORKSPACE file.

Well, clearly that didn't work. That attribute is no longer known to aspect_bazel_lib_dependencies.

I removed that previous attempt to use the "override_local_config_platform" attribute (per https://github.com/aspect-build/rules_webpack/pull/135#issuecomment-1859238445).
Do we need to continue supporting Bazel version 5.3.2 in our CI workflow?

I'm working on a PR to bump all the bazel deps and test only on Bazel 7 & 6. That will drop Bazel 5 from the CI matrix.


locally run failed

File: Error: The constructor now takes in a list of Node objects. Since you are passing in a list of Document objects, please use from_documents instead.
after I uploaded a PDF file,I got this. any help? thanks!

May I ask what type of file you are uploading? pdf? or epub?

PDF发自我的 iPhone在 2023年4月7日，14:19，guangzhengli @.***> 写道：
May I ask what type of file you are uploading? pdf? or epub?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

At present, I don't reproduce this issue locally. The error was caused by the llama_index create index function. I upgraded llama_index to the latest version and modified the from_documents method. Please test whether pulling the image again is successful. Thank you, and best wishes.

File: Error: The constructor now takes in a list of Node objects. Since you are passing in a list of Document objects, please use from_documents instead.
after I uploaded a PDF file,I got this. any help? thanks!

Add from_documents after passing your vector index classifier, for example;
change this:-
index = GPTSimpleVectorIndex(documents)
adding from_documents, it will be:-
index = GPTSimpleVectorIndex.from_documents(documents)
I suppose it will work just fine.

It seems your OpenAI key is incorrect, do you set the env with OPENAI_API_KEY=sk-xxxx ?

It seems your OpenAI key is incorrect, do you set the env with OPENAI_API_KEY=sk-xxxx ?

thanks. I clear browser cash and set os.environ['OPENAI_API_KEY'] =
problem fixed!

Hey @guangzhengli @Ilya88Weber I faced the same issue mentioned at the top of the issue by @michael7908  where
File: Error: The constructor now takes in a list of Node objects. Since you are passing in a list of Document objects, please use from_documents instead.
I resolved that, Thanks to this thread.
However, I am getting this error now
ImportError: tiktoken package not found, please run pip install tiktoken`
Can someone please help


Reload available commands on SIGUSR2 signal

If user doesn't want to wait on the periodic commands reload they can now also send SIGUSR2 signal to the bot process which will cause it to reload the commands without a restart.

Ok. We need some coding conventions.
https://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml
I don't care what the conventions are as long as they can be automated and violations identified by a linter.

Merged master to remove conflicts.


DotNet: forward all headers to client response

added logic to forward all headers that are allowed from the server response to the client response.
proposed fix for #67

As a result of this... duplicate header values are being passed back to the client.  We are operating in a 'Integrated Windows Authentication' environment and I can see both the server and persistent-auth headers are duplicated.  Was this intentional and do we see any potential impacts with this scenario?  I did not see any application functionality degraded, but did not look right.  Thanks!

@pfoppe - You're correct. The proxy needs to be smarter about header values, some should be passed through, some should not.  For example Server should not be passed on and duplicated.


json column query

i have looked issue 118 and JSON columns
i have a table conceptmap and a jsonb type column 'content',which store object like this
    {
        "version_id": "0fea943e6f1ba6433914faf593104836250ab627",
        "logical_id": "v3-medication-dispense-status",
        "resource_type": "ConceptMap",
        "updated": "2015-07-27T08:36:01.968571+00:00",
        "published": "2015-07-27T08:36:01.968571+00:00",
        "category": null,
        "content": {
            "id": "v3-medication-dispense-status",
            "url": "http://hl7.org/fhir/cm/v3/medication-dispense-status",
            "date": "2015-04-09T07:11:27+00:00",
            "meta": {
                "versionId": "0fea943e6f1ba6433914faf593104836250ab627",
                "lastUpdated": "2015-07-27T08:36:01.968571+00:00"
            },
            "name": "v3 map for MedicationDispenseStatus",
            "text": {
                "div": "
too costy
",
                "status": "generated"
            },
            "status": "draft",
            "contact": [
                {
                    "telecom": [
                        {
                            "value": "http://hl7.org/fhir",
                            "system": "url"
                        },
                        {
                            "value"<EMAIL_ADDRESS>                            "system": "email"
                        }
                    ]
                }
            ],
            "element": [
                {
                    "map": [
                        {
                            "code": "active",
                            "codeSystem": "http://hl7.org/fhir/v3/ActStatus",
                            "equivalence": "equal"
                        }
                    ],
                    "code": "in-progress",
                    "codeSystem": "http://hl7.org/fhir/medication-dispense-status"
                },
                {
                    "map": [
                        {
                            "code": "suspended",
                            "codeSystem": "http://hl7.org/fhir/v3/ActStatus",
                            "equivalence": "equal"
                        }
                    ],
                    "code": "on-hold",
                    "codeSystem": "http://hl7.org/fhir/medication-dispense-status"
                },
                {
                    "map": [
                        {
                            "code": "completed",
                            "codeSystem": "http://hl7.org/fhir/v3/ActStatus",
                            "equivalence": "equal"
                        }
                    ],
                    "code": "completed",
                    "codeSystem": "http://hl7.org/fhir/medication-dispense-status"
                },
                {
                    "map": [
                        {
                            "code": "nullified",
                            "codeSystem": "http://hl7.org/fhir/v3/ActStatus",
                            "equivalence": "equal"
                        }
                    ],
                    "code": "entered-in-error",
                    "codeSystem": "http://hl7.org/fhir/medication-dispense-status"
                },
                {
                    "map": [
                        {
                            "code": "aborted",
                            "codeSystem": "http://hl7.org/fhir/v3/ActStatus",
                            "equivalence": "equal"
                        }
                    ],
                    "code": "stopped",
                    "codeSystem": "http://hl7.org/fhir/medication-dispense-status"
                }
            ],
            "publisher": "HL7 (FHIR Project)",
            "description": "v3 Map (ActStatus)",
            "resourceType": "ConceptMap",
            "sourceReference": {
                "reference": "http://hl7.org/fhir/vs/medication-dispense-status"
            },
            "targetReference": {
                "reference": "http://hl7.org/fhir/v3/vs/ActStatus"
            }
        }
    }

postgrest-<IP_ADDRESS>
when i query ,it all failed.

http://localhost:3000/conceptmap?content->>logical_id=eq.v3-medication-dispense-status

http://localhost:3000/conceptmap?content->content->>status=eq.draft

but the following is ok
http://localhost:3000/conceptmap?content->>status=eq.draft
http://localhost:3000/conceptmap?logical_id=eq.v3-medication-dispense-status

first i think the doc describe here need some fix ,
second when a column is a jsonb type,perhaps add column name is  more solid when there are more than one jsonb type column in one table.

That is surprising. Could you try it again on version 0.2.12?

@wanghaisheng is this still a problem in v<IP_ADDRESS>?

Closing, no reply for a long time. Please reopen if this is not yet fixed.


refactor: add the parameter input implicity with type any

PR Checklist
Please check to confirm your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/angular/angular-cli/blob/main/CONTRIBUTING.md#-commit-message-guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

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

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

Thanks for this change however explicit any are not allowed.


ImgHTMLAttributes should has a loading propety

What problem does this feature solve?
export interface ImgHTMLAttributes extends HTMLAttributes {
...
loading?: "lazy" | "eager" | "auto";
}
What does the proposed API look like?
export interface ImgHTMLAttributes extends HTMLAttributes {
...
loading?: "lazy" | "eager" | "auto";
}

It's an experimental attribute

ok,确实
mdn没看到说明
can i use 倒是说的比较明确
希望这个快点标准化吧，哈哈
我程序自己加个polyfill和重定义这个好了


IDEAS

Vector option, like for adobe
Extract lineart like ibis paint when you export a photo
Rounded lines for the shapes, plus more shapes if you feel like it

I did, but to be honest, I thought it would've extracted the lineart, it kind of did, but not in the way I expected, but thanks for telling me how to :)

THANK YOUUU, It seems really helpful


Simpler docker-compose setup for tests

To run tests locally, a contributor needs to setup a gitlab instance (which is totally fine).
The suggested docker-compose is in gitlab_tests/test_v91/docker-compose.yml
I think we could leverage the documentation that Gitlab has and use one of their examples and docker containers:
https://docs.gitlab.com/omnibus/docker/README.html#install-gitlab-using-docker-compose
This should free up maintenance of those containers and we end up using the official gitlab ones.

My only issue with the Gitlab docker containers is that they don't go back as far as this library states we support. The Gitlab one only goes back to 8.13.6-ce.0 well the docker-compose file that I have in test goes back a bit further. It's this one https://github.com/sameersbn/docker-gitlab

I can spend some cycles to try and convert.
Is the issue supporting the the different API versions as they are related to gitlab versions?

We have a question that could be a bug open for as far back as 7.1

As to understand, is that gitlab version supported?
What I'm trying to understand, is the library supporting all versions of the gitlab API or only v2 (or some version) and higher?
So if the support matrix is based on gitlab API versions, then the code, I would guess, is positioned in a way to handle different gitlab API versions separately. At a glance, it's hardcored to v3 and I don't see a way to specify my API version.
I'm not questioning the design just trying to better understand the requirements. I want to ensure I continue the projects bison appropriately with any PR / suggestion I have.

As I thought about this a little, it reminded me of a project I worked on (https://github.com/gogoair/pyasgard) that dynamically built all the api endpoints available based on a file (endpoint.py). The idea could easily be adapted here to support multiple api versions (by adjusting the endpoint logic), I think. Again, this all depends on the support matrix. Is it gitlab versions or gitlab api versions?
Thanks for hearing me out.

My plan is to not drop support for anything already supported and currently 7.1.X is listed as supported with missing functionality.
We will need to add some way of supporting both V3 and V4 of the API before August roles around. I just need to review the API V4 docs to see how different they are.
Your PR's / suggestions are all very valid and appreciated. Also please do question the design if you find something you feel is wrong or out of place. Feed back can only make this project better.

My goal at the moment is to mimic the API for a version of Gitlab in the unittests using mock data.
What I have been doing as we add new tests is remove the mock calls and do a quick integration test with the version I have currently and use that data in the mock. This will allow us to quickly run our tests against all the different version when we update things. Though it doesn't provide the much needed integration tests.


InfluxDB Repository

The new influxdb repository does not seem to be keeping up to date with the latest telegraf releases.
https://repos.influxdata.com/centos/7/x86_64/stable/

@rossmcdonald any ideas? What do we need to do to keep Telegraf updated in these repos?

Sorry about that! I haven't been keeping up to date with the most recent Telegraf releases. It should be fixed now.
@sparrc We're going to try and implement some automated way of keeping the releases up-to-date, but, in the meantime, just ping me when there's a new release and I'll get it added.

Hi Ross, telegraf 0.10 has been released a couple days ago, could you add it to the repos? Thanks!
https://influxdata.com/blog/announcing-telegraf-0-10-0/

@gfk, 0.10 is a breaking build, which is why we're holding off on updating.

@gfk The 0.10.0 packages are now available through the repository.


Bracket Pair Colorization broken with Elixir

VS Code Version: 1.79.2
OS Version: macOS Ventura 13.4

Steps to Reproduce:

Enable Bracket Pair Colorization
Use any syntax highlighting tool for Elixir, such as ElixirLS
Open any Elixir file with .ex or .exs extension.
The brackets colors won't always match (that happens as well with the do endpairs)

The problem is probably coming from ElixirLS and not VSCode. Closing until I can be sure.


Install command returns an error.

The command seems not to work for me. Basic Skeleton install of Svelte Kit.
I've only run the init of Svelte Kit npm init svelte@next . then run npx svelte-add mdsvex
The console then reports the following
[ info ]  Applying preset svelte-add/mdsvex.
[ error ]  Could not clone svelte-add/mdsvex.

    at Object.action (/Users/davidfitzgibbon/.npm/_npx/76ab19934dff27c3/node_modules/simple-git/src/lib/plugins/error-detection.plugin.js:30:33)
    at PluginStore.exec (/Users/davidfitzgibbon/.npm/_npx/76ab19934dff27c3/node_modules/simple-git/src/lib/plugins/plugin-store.js:21:33)
    at /Users/davidfitzgibbon/.npm/_npx/76ab19934dff27c3/node_modules/simple-git/src/lib/runners/git-executor-chain.js:95:45
    at new Promise (<anonymous>)
    at GitExecutorChain.handleTaskData (/Users/davidfitzgibbon/.npm/_npx/76ab19934dff27c3/node_modules/simple-git/src/lib/runners/git-executor-chain.js:93:16)
    at GitExecutorChain.<anonymous> (/Users/davidfitzgibbon/.npm/_npx/76ab19934dff27c3/node_modules/simple-git/src/lib/runners/git-executor-chain.js:77:46)
    at Generator.next (<anonymous>)
    at fulfilled (/Users/davidfitzgibbon/.npm/_npx/76ab19934dff27c3/node_modules/simple-git/src/lib/runners/git-executor-chain.js:5:58)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)

Here's the package.json in case that's useful:
{
  "name": "~TODO~",
  "version": "0.0.1",
  "scripts": {
    "dev": "svelte-kit dev",
    "build": "svelte-kit build",
    "preview": "svelte-kit preview",
    "lint": "prettier --check --plugin-search-dir=. . && eslint --ignore-path .gitignore .",
    "format": "prettier --write --plugin-search-dir=. ."
  },
  "devDependencies": {
    "@sveltejs/kit": "next",
    "eslint": "^7.22.0",
    "eslint-config-prettier": "^8.1.0",
    "eslint-plugin-svelte3": "^3.2.0",
    "prettier": "~2.2.1",
    "prettier-plugin-svelte": "^2.2.0",
    "svelte": "^3.34.0"
  },
  "type": "module"
}

thanks for any help!

It is possible you need to install git.

Please let me know if that was the right solution!
In the (near) future, installing git won't be a requirement because we are replacing the internals of the package so that the dependency is no longer needed.

Thanks for the super quick reply!
Just installing XCode to install git and I'll report back then. I had installed the GitHub app, which it seems doesnt install a globally accessible git.
I'll report back!

Confirmed, that worked immediately! Thank you so much for the quick and clear reply!


readme part of Configuration, lua grammatical errors

Please elaborate, it's not clear what you mean.

In your readme doc：
{
-- triggers RsyncUp when fugitive thinks something might have changed in the repo.
fugitive_sync = false,
-- triggers RsyncUp when you save a file.
sync_on_save = true
-- the path to the project configuration
project_config_path = ".nvim/rsync.toml"
-- called when the rsync command exits, provides the exit code and the used command
on_exit = function(code, command)
end,
-- called when the rsync command prints to stderr, provides the data and the used command
on_stderr = function(data, command)
end,
}
Whether need insert , to the line end of "project_config_path = ".nvim/rsync.toml""  and "sync_on_save = true"?
Maybe cause lua fault?

Yes, seems I missed that.

Keeping this open until I have fixed this.


docker images for magicdrive

Hey,Could any one give me a docker images for v100 of python enviroment, because my v100 server could not connect to the internet. thanks a lot.

Sorry, I do not have a gpu server to test dockerfile or build docker image.


parquet1.10.0 support zstd

Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?

@singer-bin Parquet 1.10 is pretty old (Jan 2019), and no new releases will be done for this version. Thanks for raising this PR, but I'm going to close it. I would highly recommend upgrading to a later version of Parquet-Java.


update ADM-dateTimePicker to angular 2

hi, could you please update this package for angular 2 ?

Hi Morteza,
Yes, I just need more free time! :)


Milestone 1: Animal Table

In this pull request

An animal table was created for vet-clinic using the SQL query in schema.sql.
Data was inserted into the table using the queries in data.sql.
Queries made into the table was done using the action in queries.sql

Screenshots to demonstrate the queries are in the comment section below.

Queries Screnshot


Fix a notification message when a guest user tries to create a new re…

…pository
Motivation:
A guest user can be confused because a notification when he or she tries to create a new repository is wierd like below.
You must be GUEST of project '...'
Modifications:

Fix the notification message.
Refactor decorators using a custom decorator annotation.

Replace @Decorator(...) with @RequiresRole, @RequiresReadPermission, @RequiresWritePermission and @RequiresAdministrator.

Result:

Closes #299

Thanks, reviewers!


Use price selection with Product Projection Search service

Description
Currently it's not possible to enable Price Selection when using Product Projection Search (or other services). It is possible to send priceCurrency, priceCountry etc through the REST API, but this SDK doesn't allow sending those at least from what I have figure it out so far.
Expected Behavior
It should be possible to enable Price Selection using the NodeJS sdk.

What do you mean under "Price Selection"? Like "Give me products with price more than 256 cents"?
Something like this masterVariant(prices(value(centAmount > 256)))?

No, according to CT API docs, price selection finds the best matching price from the prices array. For example if I pass in "priceCountry=US&priceCurrency=USD" the resulting variant will include a "price" key in addition to "prices" array.

@LEQADA for example this query returns the correct result now: https://api.sphere.io/PROJECT_KEY/product-projections/search?priceCurrency=EUR&priceCountry=FI

Now I'm confused. Do you want to say "Give me only those products which has price with priceCurrency=EUR and priceCountry=FI"?

@LEQADA No, im not trying to "filter" products. Im trying to enable "Price Selection". See this section of the docs and keys: priceCountry, priceCurrency. Basically we need to send a few more query strings.
and this is how the variant response looks like with those options:

@sallar, thanks for your explanation, I hope it clears it
...and I look into this today and confirmed that it's missing from the SDK, We will prioritize and add it here and new SDK #154

@sallar oh, now I understand what you mean.
Maybe use this kind of GraphQL query instead?
query Sphere {
  products {
    results {
      masterData {
        staged {
          allVariants{
            price(currency:"EUR", country:"DE"){
              id
              value{
                centAmount
              }
            }
          }
        }
      }
    }
  }
}

As @sallar pointed out price selection is currently not supported by neither this nor new nodejs SDK.

thx for releasing it


Add support for textEdit since insertText is deprecated

Related issue: #27
insertText is deprecated because it is too open to interpretation and not precise enough. Instead we should support textEdit

insertText deprecation: https://github.com/microsoft/language-server-protocol/blame/56e4fc292c121118ac520c8a19688c85066ed0bb/specification.md#L2946
textEdit details: https://github.com/microsoft/language-server-protocol/blame/56e4fc292c121118ac520c8a19688c85066ed0bb/specification.md#L434

As of 3.15 (and upcoming 3.16) insertText  is no longer deprecated. The spec now only recommends using texEdit and a lot of other are being introduced to this APIs

Good point, updated the title


Kilo to Liberty upgrades fail on bootstrap-ansible.sh due to pip

What version (commit) do you encounter this issue on?
r11.1.8 and master
What type of issue is this (bug, feature, documentation, regression)?
bug
What is the expected behavior?
bootstrap-ansible.sh updates Ansible to the appropriate version on upgrade.
What is actually happening?
bootstrap-ansible.sh fails when upgrading to Liberty because it is using the Kilo version of the repo servers andthey do not have the required version of paramiko.
http://jenkins.propter.net/job/RPC-AIO/5460/consoleFull
How can we reproduce this issue?
Attempt to upgrade from a deployment of r11.1.8.
How can we verify your issue is addressed?
bootstrap-ansible.sh successfully finishes.
If this is a large issue, what can be done immediately to alleviate the issue while working on a longer-term solution?
What is your evaluation on the importance of this issue, and why?
High, this is currently blocking the upgrade check on Liberty branches.
What branches / milestones do you believe this issue should target?
The list below is what is currently supported for backports
Remove branches (lines) you do not wish to target.

[ ] Master
[ ] liberty-12.2
[ ] liberty-12.1

I am looking to address this issue with https://review.openstack.org/#/c/314047/


Sending Context Information

Hi there,
awesome project, loved it when I saw it at the AI Summit.
As mentioned there, it would be support awesome if stuff could be send to a bot, that is technically not a user input.
I am the lead developer for EDDI (http://github.com/labsai/EDDI); with it, it is possible to hand over context information as String, JSON or Expressions (syntax aware String representing semantics)
I would write an adapted to EDDI, but would need a possibility to define the context values in the script.
Example of such context information:
type String; Key: firstTimeLogin Value: true
type JSON; Key: userInfo Value: {'username':'John'}
Cheers,
Gregor

Hi Gregor,
We do the same thing at Aiden. What we do is we have special developer commands such as __login_as, which are hardcoded regexes where we can pass the state of the user before the test. We just put them at the top of the test file, so this could be one option. This has been fine in our case.
Another option which is closer to what you are meaning I believe is to have some kind of option in the test file itself which is like Context, where you specify a JSON, and we send this over some side channel. This would mean adding a method to the ClientInterface class such as sendMetadata.
The first option is enough for us at Aiden, but if you think the second one might work better in your case, it would be awesome to fork it and merge that feature back in. My intuition is that it should be quite an easy patch, and I can help you if needed.

probably the second option would be better, but in our case we would send it in the same channel as the input, just differently tagged such as:
{'input':'someUserInput', :context':{'someKey':'someValue'}}


Local Console/Portal

I want to be able to run something like defang console and have it launch a UI where I can see everything that's happening in my environment right now: builds, running applications, resources, logs, etc.

Could be done by hosting Fabric rpcs in the CLI and point Portal to localhost


TrueBeam v3.0 MPC Changes

TrueBeam v3.0 has some changes to MPC, and what was previously called "Enhanced MLC Check" is now called "Collimation Devices Check". This includes some additional jaw tests where jaw position is tested at 4 different positions per jaw, as well as a determination of jaw parallelism (in degrees) at each position, along with max parallelism for each jaw.
While existing MLC metrics appear unchanged, the folder naming convention has now changed from EnhancedMLCCheckTemplate6x to CollimationDevicesCheckTemplate6x

Changes made to mpc.py here: https://github.com/ash-cullen/qcpump/commit/bb1b1cac7085e979bcfeeef44ab57a171747804b
I tested this and these changes work at my hospital.
Note that new tests need to be created as checks are under the CollimationDevicesGroup, and MLCs are tests at 3 positions.

Thanks so much Ash!   I'll apply that fix and get a new release out soon.

Updated in v0.3.15. Thanks again.


Added generic "property filtering" to /missions and /runtimes

It would be nice to have a test in BoosterCatalogEndpointIT with this new feature as well

Ike Plugins (test-keeper)
Thank you @quintesse for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin - for example exclude certain file types so if PR contains only them it won't be checked.


Add configure option to disable libparsebgp warnings

We currently have the parser configured to log any parse failures to aid with debugging the v2 beta, but these can be annoying, so this allows advanced users to disable the warnings.
Run configure with the --disable-parser-warnings option to disable the parser warnings.

Tested on fresh vm, the added flag works as designed.

Test example: bgpreader -w<PHONE_NUMBER>,1539561600 -t updates
Version configured without the flag would give out warnings at top (the first few lines of output).


Lock goroutine to OS thread while changing NS

Prevents an issue where the goroutine may jump to a new OS thread during
execution putting it into a mount/network NS that is unexpected.


clone on read from cache

@metacosm @adam-sandor So as discussed here:
https://github.com/java-operator-sdk/java-operator-sdk/pull/319
this PR should fix the mentioned issue an possible others, can we merge this? and close the issue and the other PRs?

@metacosm @adam-sandor So as discussed here:
https://github.com/java-operator-sdk/java-operator-sdk/pull/319
this PR should fix the mentioned issue an possible others, can we merge this? and close the issue and the other PRs?

@csviri sorry I checked the wrong link! I have tested my PR not yours yet.

@csviri sorry I checked the wrong link! I have tested my PR not yours yet.

Hmm, cloning by serializing and deserializing might have quite a performance impact and I'm not sure it wouldn't actually be faster to always retrieve the last version from the cluster…

Hmm, cloning by serializing and deserializing might have quite a performance impact and I'm not sure it wouldn't actually be faster to always retrieve the last version from the cluster…

Hmm, cloning by serializing and deserializing might have quite a performance impact and I'm not sure it wouldn't actually be faster to always retrieve the last version from the cluster…

@metacosm
Retrieving from the cluster does also deserialization. But note that these are small objects (not huge json docs usually) this can be measured in micro seconds (we can actually log it on trace level). (https://www.ericdecanini.com/2020/10/13/benchmarking-gson-vs-jackson-vs-moshi-2020/)
Making an API call on every event we received / processing of controller is not a good idea. You will have to make or maintain a TCP connection to the K8S API server, while we have anyways the latest resource from the watch always. Also note that we try to minimize the K8S API, so we don't make a DOS attack on our own.
( This would mean API call per controller execution, where the execution might not be triggered by a custom resource change. Imagin you have 1000 CRs an receive an event from a Deployment Event source for each, you would do 1000 request for K8S api while there is no request needed at all, since the CustomResource is not changed )
I think its quite high priority to not do polling on K8S API in general.

Hmm, cloning by serializing and deserializing might have quite a performance impact and I'm not sure it wouldn't actually be faster to always retrieve the last version from the cluster…

@metacosm
Retrieving from the cluster does also deserialization. But note that these are small objects (not huge json docs usually) this can be measured in micro seconds (we can actually log it on trace level). (https://www.ericdecanini.com/2020/10/13/benchmarking-gson-vs-jackson-vs-moshi-2020/)
Making an API call on every event we received / processing of controller is not a good idea. You will have to make or maintain a TCP connection to the K8S API server, while we have anyways the latest resource from the watch always. Also note that we try to minimize the K8S API, so we don't make a DOS attack on our own.
( This would mean API call per controller execution, where the execution might not be triggered by a custom resource change. Imagin you have 1000 CRs an receive an event from a Deployment Event source for each, you would do 1000 request for K8S api while there is no request needed at all, since the CustomResource is not changed )
I think its quite high priority to not do polling on K8S API in general.

@csviri sorry I checked the wrong link! I have tested my PR not yours yet.

Uhh I saw this now, could you test with this PR pls.

@csviri sorry I checked the wrong link! I have tested my PR not yours yet.

Uhh I saw this now, could you test with this PR pls.

@csviri retrieving from the cluster does indeed deserialize but here we're serializing then deserializing… Also, the process is not a perfect clone i.e. not all state is necessarily serialized (though the controller should be able to reconstruct that state from the "clean" copy).
I do agree that we should merge this fix if it indeed addresses the issue but I'm not convinced that this caching strategy is the proper one and we might indeed need to re-think some things.

@csviri retrieving from the cluster does indeed deserialize but here we're serializing then deserializing… Also, the process is not a perfect clone i.e. not all state is necessarily serialized (though the controller should be able to reconstruct that state from the "clean" copy).
I do agree that we should merge this fix if it indeed addresses the issue but I'm not convinced that this caching strategy is the proper one and we might indeed need to re-think some things.

@metacosm  Sure, happy to discuss alternatives. Pls organise a meeting in case.
Although I think caches have a significant roles here also in event sources to avoid polling.
Hopefully can finish the blog this week, this is touched too, ehh cannot find  enough time :/

@metacosm  Sure, happy to discuss alternatives. Pls organise a meeting in case.
Although I think caches have a significant roles here also in event sources to avoid polling.
Hopefully can finish the blog this week, this is touched too, ehh cannot find  enough time :/


[staging] discrepancies in dropdown menus in header

Steps to Reproduce

Click on Tools menu
Compare Submenu to design
Click on About & Help menu
Compare Submenu to design

Actual Results
In Tools menu is listed FAQ instead of API
In About & Help menu is listed API and Submit Feedback
Expected Results
In About & Help menu should be listed Report Issue on GitHub and Ask Questions on GitHub
Design

Evidence

Thats it from my end. Staging reflect the latest designs for the about and help menu.

(Updated the github project automation so closing issues will be moved to "Completed" instead of "High severity" going forward 😃)


context update exception

Describe the bug
Hi everyone,
I am trying to perform an MD simulation of a solute-in-solvent system. I need to keep freezed the geometry of the solute and to do this I am relying on a script (rigid.py) from Peter Eastman which implements rigid bodies. To avoid intramolecular interactions I am acting on force.getExceptionParameters() to set both charges and LJ parameters to 0.0. Unfortunately when I try to update the context I get the following error:
Exception: updateParametersInContext: The number of non-excluded exceptions has changed
I have also noticed that if the parameters are already 0.0 no exception is raised.
To Reproduce
Pleas find attached a code to reproduce the error.
error.zip

@matteo-ambrosetti : This is an OpenMM specific problem, so it's best to use the OpenMM issue tracker for further questions, but there's an easy fix for your problem:
OpenMM treats charge and LJ epsilon parameters of 0.0 as exceptions, so the better approach is to instead set this to a small number (e.g. 1.0e-8) that is still non-zero.
Also, it's best to only change LJ epsilon, and not sigma since this can sometimes cause numerical instability because sigma appears in the denominator of some expressions.

I believe this can be closed, but please re-open if you need further assistance with openforcefield.

Thanks @jchodera, and sorry for the delayed answer. I am aware of the solution of using a small number instead of 0.0 (which is what I have done in error.zip), but it doesn't work for me anyway. I will move this issue to the OpenMM issue tracker.


.hide class implementation and side effects

In my current project I faced a performance issue because of a CSS3 animation, basically we noticed lot of CPU usage. Turns out that the CSS3 animation (think about it as a loading spinner that is hidden once data is loaded) was being performed even when the DOM element containing the spinner wasn't visible.. So I investigated a bit and I realized that this is the actual .hide implementation:
.hide {
  position: absolute !important;
  height: 1px;
  width: 1px;
  overflow: hidden;
  clip: rect(1px, 1px, 1px, 1px);
}

Turns out that this way, CPU is still being used as if the CSS3 animation would be visible.
On the other side, a simple implementation like
.hide {
  display: none;
}

works as expected: the browser recognizes that the DOM element is not visible, so it doesn't perform any computation.
I'd just like to understand the reasons behind your implementation choice, and warn you about the side effects it may have when multiple CSS animation are being hidden in the DOM with it.

.hide is for accessibly hiding elements. You'd want to use .display-none for your use case.

Yep, thanks @jxnblk , I've just seen the other class and I read something about the accessibility thing.
Even if this is a bit of a convention, I am not really sure about if such a specific behavior should be addressed with a class name a bit too common and used by another frameworks in the other way, or if the readers should be aware of hidden elements, but that's another point :)


(FACT-1825) Update cmake definitions to allow skipping of test dir

The cfacter gem does not need to include the tests directory, nor should it.
If we allow builds to specify NO_FACTER_TESTS and skip the test dir we can
make the gem smaller

CLA signed by all contributors.

Hi @McdonaldSeanp, I'm gonna go ahead and close this. Feel free to reopen if you'd like to continue the work.


Publish main to live on 11/10 @ 10:30 am

PR Summary

PR Checklist

[ ] Descriptive Title: This PR's title is a synopsis of the changes it proposes.
[ ] Summary: This PR's summary describes the scope and intent of the change.
[ ] Contributor's Guide: I have read the contributors guide.
[ ] Style: This PR adheres to the style guide.

Learn Build status updates of commit d3a01cd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/smbshare/New-SmbShare.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/smbshare/Reset-SmbClientConfiguration.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/smbshare/Reset-SmbServerConfiguration.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/smbshare/Set-SmbClientConfiguration.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/smbshare/Set-SmbServerConfiguration.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Arch arm float unsharing rework

The pull requests addresses #15115 for enhancing the implementation of Un-shared Floating Point Services for ARM.

Tested with tests/kernel/fp_sharing on frdm_k64f nrf52_pca10040, and sam_e70_xplained


OpenGL is dumb with vsync

Related to #5 - when vsync is enabled, opengl will use seemingly excessive cpu, even with manual framerate limiting, until any framerate limiting results in <30fps (at which point it's unpleasant to use anyway).  I tend to get around 12% cpu usage regardless of what is being drawn.
This appears to be known behavior of gl drivers (at least of nvidia's), as the driver basically spins on the cpu until the swap interval is reached.  It is claimed that this doesn't actually matter as it will yield cpu to any application that needs it, so it's basically only appearing to use a lot when nothing else is going on.
It is interesting that disabling vsync will drop the cpu to the expected "near nothing" values, and manual fps limiting on top of that can drop it still further.
It might be worth disabling vsync entirely and manually capping to something (in the 40-100 range seems reasonable).  I haven't noticed any artifacting when running around 200-300 and dragging windows around, but even if it happens it's probably not a significant issue for a ui like this.

As explained in #5, assorted methods for framerate limiting have been added and can be used to effectively bypass the high cpu issue with OpenGL.
If you must use OpenGL, set a manual FramerateLimit on your SimpleImGuiScene object (anywhere from ~50-120fps is probably fine, but experiment), and set PauseWhenUnfocused to true if you do not need background rendering.


Use _index.md

There is a pattern from other projects
Will it work in Jekyll?
If I have frontmatter in it should be okay.
Check that the path resolves too in the URL though.

That is just a Hugo thing
It is hard in Jekyll
https://help.github.jp/enterprise/2.11/user/articles/files-that-start-with-an-underscore-are-missing/
https://jekyllrb.com/docs/configuration/options/
Even when using include: option for _index.md the file gets rendered as _index.html. So need a permalink option that will work.

include:
  - _index.md


Reduce amount of dependencies

Reduce the amount of dependencies used by changing from futures to futures-util and removing enum_dispatch as it's only used for one single enum.
A naive comparison with cargo tree --all-features | wc -l:
Before: 289
After: 263
Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Sure no problem, I think the biggest reduction comes actually from futures-util as all the transitive dependencies of enum_disptach are already on the dependency graph due to other deps.

Sure no problem, I think the biggest reduction comes actually from futures-util as all the transitive dependencies of enum_disptach are already on the dependency graph due to other deps.

Thank you very much!

Thank you very much!


Remove reuseForks to fix surefire forks crashing during build

I thought the default was true?

I've removed T 1.5C and replaced it with T 6. Seems to be stable now.


SampleGen: Update the generated CLI to use consistent --param-name="" flags for all languages

Today, we vary the CLI flags used to pass parameters to generated samples (when run) across the languages:

Python: --file_path="image.png"
PHP: --filePath="image.png"

I advocate that these are all changed to follow a --file-path="image.png" dash-es convention
Reasoning:

Almost all common *nix tools follow a standard --foo-bar= naming convention, so this does not conflict with any particular programming language's convention
It will be somewhat useful for any users who happen to run samples in many languages
It will be very, very useful for any sample developers who are generating samples in many languages
It will allow tools which test samples in many languages to use a common interface across the CLIs for executing samples for testing

Agreed that the CLI argument convention (a) should be the same across languages, and (b) should adhere to an accepted standard.
I'd be OK with either snake_case or dash-case, but I prefer snake_case because IMO it's much more prevalent (and also what we use in YAML, etc.).

Hmm, a cursory look at a few man pages suggests that both snake_case and dash-case are in wide use, but perhaps dash-case is used a bit more....

For what it's worth, when gapic supports C++, and if that support uses Abseil to handle CLI flags, then snake_case would be preferable.

if that support uses Abseil to handle CLI flags, then snake_case would be preferable.

Hoping we won't run into issues with languages where the opposite is true
The only reason why I would like to see snake_case is because it provides a nice, deterministic mapping from YAML <--> CLI, which I would like a lot!

It sounds like we should be able to do either. My personal preference is snake_case as well. We can go with that if that's the majority opinion. @hzyi-google ?

OK let's do snake_case.

@devbww I don't actually think the casing here would cause any troubles for Abseil though. The CLIs here are only used for generated samples, for which we expect to be standalone executable, not a generated "package" to be managed by any dependency management tool. but I can never tell :/
We don't see sanke_case causing any problems to existing languages, at least not for now, so let's just do snake_case.


Creating new PR results in collapsed description box

Extension version v0.39.2022031709
OS: Windows 10
On VS Code Insiders, when I create a new PR, the description box seems to be collapsed.
If I double click on the description box to edit the message, I see the original message I entered.
If I press OK after that, the rendering is fine again.

Can you share a screen shot of what you're seeing? It looks ok to me:

I failed to repro the issue.
What was supposed to happen in order to repro the issue was the following:

Fill in a description and hit the Create button.
The pull request view shows up. The first comment, which contains the PR description, is shown below surrounded by a black outline.
:bug: In the original issue, the first comment would be collapsed and show just a thin rectangular border instead of a full comment.


loader : Log upsert and remove route errors

Fixes : #15282
Signed-off-by: Gaurav Genani<EMAIL_ADDRESS>

test-me-please

test-1.20-4.19

I opened https://github.com/cilium/cilium/issues/15572 for the new flake. Merging.


adding node selector to the controller to force it run on metadata se…

…rver nodes

rwkarg
I signed the agreement , what do i have todo in order for that to get merged ?

Hey, can you rebase this?


How can I test the outcome of a publication?

I want to test if the outcome of my query is good. How can I do this using a test-framework?
I've tried using johanbrook:publication-collector, which works for https://github.com/percolatestudio/publish-counts a.o. but I couldn't get it work with your plugin ...
Writing full-app-tests seems to heavy to me, since I'm sure there's also another way ...

Could you reproduce the issue with johanbrook:publication-collector for me? A test repo would be greate

I see that you use Meteor.defer() when executing the joins. https://github.com/nlhuykhang/meteor-publish-join/blob/master/src/server/Store.js#L26
This leaves me in an odd state, because I never know when this method will finish. Instead, I'd have to wait for it. Let's give it a try.
In a PR I'm gonna push soon, I've started on an option, that you can set to the server. It will change the behavior that the function just runs once - without re-calling. This also makes it quite suitable to a functional test using johanbrook:publication-collector. If you want to do more complex testing, you can always start awaiting the result and go from there.

Fixed testing in #6


Users want to receive unwrapped ETH on destination

Describe the bug
Today, if you bridge from Ethereum we auto-wrap on Origin to WETH.
The user receives WETH.
This is

Different than nxtp-v1
A huge pain since you cant use ETH as gas on your destination

Scoping:
Solution Design

Create an xReceiver contract that will receive destination-side calldata and call the WETH contract withdraw function.
Update deploy scripts to deploy this contract on all chains that support native ETH (i.e. rollups).
Modify SDK to create xcall with the proper calldata to call the xReceive contract on execute (fast path). This should be configurable to receive ETH or WETH.

Acceptance Criteria

[ ] Bridge transactions from the UI into ETH receive unwrapped ETH.


Beta feedback: voting period time examples

Overview

When entering “Voting Period” time, it would be nice if there were a dynamic message that displayed how long the input number is. For example
Entering “65” shows “1 hour 5 minutes”
Entering “2160” shows “1 day 12 hours 0 minutes”
Entering “10100” shows “1 week 0 days 0 hours 20 minutes”

what we have is fine for now, this can be included in a rethink once we get more user feedback


Rewrite 3D touch preview code

Currently, it just pushes whatever the VC is to the navigation stack. Instead, it should figure out what type of content it is and display it accordingly (modal, popover, etc)

Fixed


Riot to Telegram not working

Hi,
This seems to work for me from telegram to riot, but messages are not going from riot to telegram. Any advice or is this an issue?

This is happening to me too.
@evok3d is your room encrypted? Mine is.

Telematrix does not support end-to-end encrypted rooms.

screenshot of the settings:
https://prnt.sc/ghwnl0
it is not encrypted to be honest. I am using t2bot.io implementation. If you have a different solution, please do let me know.
Thank you for your help.

 I've just created my own t2bot link (haven't really had the time and energy to maintain my own homeserver) and it seems to work fine


Chequebook factory: 0xc2d5a532cf69aa9a1378737d8ccdef884b6e7420 What is the code of the contract？

Chequebook factory: 0xc2d5a532cf69aa9a1378737d8ccdef884b6e7420 What is the code of the contract？

It seems that for this, you're referring to the ChequeBookFactory contract that's been deployed to xDai Chain. For exploring xDai chain, please refer to https://www.blockscout.com/.
Therefore, you can see the deploy contract at https://blockscout.com/xdai/mainnet/address/0xC2d5A532cf69AA9A1378737D8ccDEF884B6E7420/contracts. I do note however that the contract source code hasn't been uploaded / verified.
@significance Not sure who to refer this to for verification on blockscout.com?

the source code is at https://github.com/ethersphere/swap-swear-and-swindle/blob/master/contracts/SimpleSwapFactory.sol
unfortunately verification does not work. flattened verification does not work most likely because of the abicoder pragma in some files and the more advanced verification does not work due to an internal server error on blockscouts side.