[Feature request]How to associate editing to the right click

How to associate editing to the right click

1.Create a new file called 1.reg
2.Copy the following content to 1.reg file.
Windows Registry Editor Version 5.00

[HKEY_CLASSES_ROOT\*\shell\notepads]
@="open with notepads"
"Icon"="C:\\Your\\Icon\\Path\\notepads.ico,0"

[HKEY_CLASSES_ROOT\*\shell\notepads\command]
@="\"C:\\Users\\YourUsername\\AppData\\Local\\Microsoft\\WindowsApps\\Notepads.exe\" \"%V\""

3.Replace YourUsername and icon path. You can use the ico i converted or delete that line
4. Save it. Double click and import.

I have tried this method, but I will add 'Open with NotePads' to the right button of all files, how can I add it in the specified file format,for example BAT, CMD, etc.

I have tried this method, and it will add "Open with NotePads" to all files in the right button,how can I add it in the right button of the specified file format,for example BAT, CMD.
Thx


Execution reverted: swap more than you own

Describe the bug
“Execution reverted: swap more than you own” in swap feature.

Additional context
https://discord.com/channels/454734546869551114/1070556349039116328/1070563111817449532
https://discord.com/channels/454734546869551114/1070556636139241532/1070556725037518869
(2 cases in different chains)

Now an user can deposit usdc into lps.
Describe the bug
I assume this issue is related with previous one
”Execution reverted: ERC20: transfer amount exceeds balance”

Additional context
https://discord.com/channels/454734546869551114/1070544268466327622/1070553566823067688
(Again, only having issues with polygon)

Seems fixed now. I'm going to close it.


Error trying use gvm-cli from gvm-tools

Hello togehter. I installed the gvm-tools like here https://github.com/greenbone/gvm-tools. The tools shouldn't be run as root. If i try to run on a rasperry as user pi, i get the following error message
pi@core:~ $ gvm-cli --gmp-username admin --gmp-password admin socket --socketpath /var/opt/gvm/var/run/gvmd.sock --xml "<get_scanners/>"                                    [Errno 13] Permission denied
Any idea?
Thank you in advanced for your support and best greetings!
EDIT: I could solve the problem. It was neccessary to set the right to write for the user "pi" on the /var/opt/gvm/var/run/gvmd.sock file.

It was neccessary to set the right to write for the user "pi" on the /var/opt/gvm/var/run/gvmd.sock file.

It was neccessary to set the right to write for the user "pi" on the /var/opt/gvm/var/run/gvmd.sock file.


Add a service using the managed network type, the container gets the wrong IP

Rancher versions:
rancher/server: 1.6.13-rc3
rancher/agent:
Docker version: (docker version,docker info preferred)
17.06.2-ee-6
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Windows Server 2016 Standard
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
VirtualBox
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Windows
Steps to Reproduce:

Add service，'Network' select managed.
Create service
Check the IP address of the container.
Results:
The container gets the IP address of type 'NAT'.

Available with v1.6.13-rc4

Tested with Rancher/server:v1.6.13-rc4.
Added service page has the disabled managed network.


Quick Text Search: Duplicated entries added when typing after a pause

Testing #191499

open the vscode repo
open command palette, type % semant pause a second and continue typing ic

Note that index.ts has the same entry twice. The second entry is added when typing after the pause

Can't seem to repro, but I'll keep this open and look out for it.


mainnet0 snapshot sharing (Use at your own risk.)

Note: moving to https://github.com/Agoric/validator-profiles/wiki/Snapshots
These links were shared in discord. Use at your own risk.
Feel free to add more in further comments.

Danil Ushakov: My Agoric mainnet daily snapshot service
https://snapshots.stake2.me/agoric/
How to use:
https://github.com/k0kk0k/cosmos-snapshots-doc/blob/main/agoric.md
Maybe it will help someone to quickly restore the node in case of unforeseen circumstances

https://github.com/c29r3/cosmos-snapshots/blob/main/Agoric.md

Here is the state sync snapshot with the guide "how to use is".
https://github.com/Phantom1605/State-Sync-Snapshots#agoric
It seems to me that for purposes such as running a validator node or a sentry node, state sync is better than full data snapshot, as it will save a lot of disk space.

...state sync is better ...

Yes, here's hoping we get there: https://github.com/Agoric/agoric-sdk/issues/3769

Let's use https://github.com/Agoric/validator-profiles/wiki/Snapshots for this going forward.


Build(deps-dev): Bump cypress from 9.7.0 to 10.1.0 (#3628)

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

Merged in 05148c161ddb9521224f50939815922d6351b247


Fix bandaid for orphans

There's another open PR for this, but this one also fixes the docs and macros and passed on travis.

NB. Review the changes to Cargo.toml as you may prefer not to merge to the github dependencies (even though crates.io is broken for serialize at the moment).

@shadowmint Thank you! And thanks for the heads up on the dependencies. I think rustc-serialize is working now (albeit derive doesn't work without old_orphan_check), so I'll fix that and update crates.io.


Subnav on the left overlapps with the sticky Artsy icon

eh, I can live with this, who has their browser so small.


SRS error/issue with VCD

In VCD > preprocessing.py, around line 103 it takes in the SRS from the pipeline.quickinfo, but notes that if there are more than one SRS, it won't work.
       def _get_utm(pipeline: pdal.Pipeline) -> pdal.Pipeline:
            data = pipeline.quickinfo
            is_reader = [[k.split(".")[0] == "readers", k] for k in data.keys()]
            for k in is_reader:
                if k[0]:  # we are a reader
                    reader_info = data[k[1]]
                    bounds = reader_info["bounds"]
                    srs = CRS.from_user_input(reader_info["srs"]["compoundwkt"])

                    # we just take the first one. If there's more we are screwed
                    break

Encountered this error with Open Data LiDAR from DC, with a PCS and VCS (see image). Possibly we can make a workaround? If not, I will enhance the toolbox to warn people accordingly.

source of error: 2018 las data from here.

Possibly we can make a workaround? If not, I will enhance the toolbox to warn people accordingly.

Throw an error. Not out responsibility to normalize coordinate systems, and it is important that users provide cartesian data too.

Are Vertical Coordinate Systems / Datums off the table too?

understandable- will add this error/warning to toolbox


Add overloads for finalStateVia support

I've added:

An enum for FinalStateVia
the overloads to assume AzureAsyncOperation when not present (and therefore, should not be a binary breaking change)

I have not written tests for this.
You will have to:

Do something with the field  (see https://github.com/Azure/azure-sdk-for-net/compare/psSdkJson6...fearthecowboy:add-finalstatevia?expand=1#diff-f6fc4a3b85caf41f8836d1713180305bR140 )

After the client runtime is patched, I can update the c# generator to add the parameter to the call to the client runtime (patch is waiting...)

@shahabhijeet If you're sure...
We can close this, and if an issue pops up, it'll be ready.

@fearthecowboy yep let's close it.
The issue was always about knowing the response model and not the header.
So as of now if there are no plans to pass in multiple models pointing which model to be used for which header, we can close this issue.


Ver bump 1.2

Description
To go after #391
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@VachaShah @nhtruong need a new 1.2.0 branch for up coming release. This PR depends on #391 first and then has to be backported into the new branch when created.


Standardize difficulty levels to help sort exercises consistently on the site

Currently, we're sorting exercises by difficulty level and then name. The website bins displayed difficulty levels as easy, medium, and heavy. I think these correspond to levels 1 - 3, 4 - 6, and 7 - 10. By sorting on the difficulty level, the exercises are out of order.
The student sees something like this:
Hello World (EASY)
Two Fer (EASY)
Clock (EASY)
Eliuds Eggs (EASY)
So I'm proposing a two-part solution. First we take the current levels, see which band they fall under, and then replace that level with the middle of the band. So all exercises levels 1, 2, or 3 become level 2. All exercises levels 4, 5, or 6 become 5. All exercises levels 7, 8, 9, or 10 become 8. That makes implementing exercises easier since you don't need to consider the difference between individual levels. It also lets us now sort the exercises more cleanly. We have a jq one liner floating around that can sort the exercises. That makes it a cinch to sort the exercises if they ever get out of order. At the moment a few of the level 5 exercises aren't in alphabetic order. Manually fixing that becomes easier with this approach.
My earlier example when sorted this way looks like this:
Hello World (EASY)
Clock (EASY)
Eliuds Eggs (EASY)
Two FER (EASY)

Okay I've assigned you to do it. If you'd rather not, let me know.


Issue with N3Sky not being large enough

So I believe a lot of people are familiar with this issue

Turns out if you just make the render billboard larger it fixes this problem.

Just opening and closing this issue so the solution gets documented.

This issue was moved to ApexKO/OpenKO#59

Hey Stephen,
As I remember when I was going through this, the idea behind creating this billboard was to create an illusion of far plane fog, because the originally pixel and vertex fog wasn't good enough.
What you basically did, is covering the skybox while the position of the billboard was in its correct place. So that doesn't really resolve the issue, as the sky box have different colors from the fog.
I don’t have a setup of the new changes since 1-year I guess, so I'll try to have another look at this, as I always wanted to fix this one and got some time before school starts again.
Will keep this post update if I find anything.

You can still see the breaking line between the sky box and the billboard if you change the camera rotation a bit more up (F9):

So i guess that does the trick? :)
The actual problem here was when we were setting a texture to the billboard with colors using the SetTextureState. The whole idea of the billboard, is to create an illusion of fog, so in order to let the vertices get drawn by the DrawPrimitiveUP, directx is expecting us to use the SetTexture function and we simply set it to null as we don’t need one. The SetFVF diffuse macro in combination with xyz, tells it to spread the colors around the xyz coordinates of the vertices, since each billboard(in this case front and bottom) has 4 corners.
It’s about time for the 12xx clients, no? ^_^

also fixed /time command in server and client, so we can debug the sky or some other stuff(cool sunset ^_^):

@GiladRe the sunset looks beautiful! Nice job 😄

@SavvyNik Oh my bad..I just realized it's sunrise as i was doing /time 5:25 ^_^

Haha good catch! Now we wouldn't want to be wrong about the time of day since we got a time command lol


Plugin does not seem to work.

Which version of the plugin are you using?
[X ] Data Populator for Sketch
[ ] Data Populator for Adobe XD
Describe the bug
None of the commands do anything.
There are no windows that appear, no pop-ups... nothing.
To Reproduce
Steps to reproduce the behavior:

Go to 'Plugins'
Click on 'Data Populator'
Click on any of the commands availablele
See nothing

Expected behavior
For these actions to present windows that allow me to fulfill the action.

I have the same issue. As I have worked a lot wit Data Populator before, I feel like this happend with the newest sketch update.

I confirm the same issue with Sketch 85.1. This plugin is much more powerful than Sketch built-in data features. It would be awesome if there was a fix.

The same is happening to me. It worked for a few days and then stopped.

It seems this plugin used an internal class that no longer exists.
Console shows a this:

standard	12:07:08.428778+0200	Sketch	Possible Unhandled Promise Rejection: ReferenceError: Can't find variable: MSLayerArray at l (/Users/fbolte/Library/Application Support/com.bohemiancoding.sketch3/Plugins/data-populator.sketchplugin/Contents/Sketch/__src_sketch_commands_populateFromURL.js:1:60655) …

From the release notes of Sketch v84:

We removed the internal class MSLayerArray from our codebase. Code that dealt with layer arrays now uses regular NSArray objects instead. This change made our internal code cleaner and easier to maintain by removing an unnecessary layer of abstraction.
As mentioned in our Internal API page, this is an internal class that should never be used. But we are aware that some plugins use it, and we’re providing this information to help them migrate their code to use the new APIs.
Code expecting an MSLayerArray object, and using its .layers() method to get an NSArray with the layers, will now receive an NSArray object. The .layers() call is no longer needed.
Plugins relying on any of the internal methods in MSLayerArray will not work. They may even crash Sketch if they’re passing the returned value unchecked to other parts of the plugin code.
Check the table in the next section for examples of how to migrate your code.
We recommend using the JavaScript API where possible. If you feel something is missing from our API, please submit feedback and feature requests!.

Source: https://developer.sketch.com/plugins/updates/new-in-sketch-84
:-/

@theguywithhat @rreina @marybmichaels @fabianbolte @Buydens we just pushed a fix – please update. sorry for the inconvenience.


Uncomment check for texture files

Seems like I commented out some code and forgot to uncomment it.
Fixes #13.

@mergifyio backport iron


Replaced three_step with a fork/join test (which works)

Previously, three_step didn't work - its expected outputs were wrong.
I've made the outputs deterministic and added a "join" step at the end. And therefore renamed it.

👍

looks good, 👍


Front Matter support

For those who use Slim with Middleman or Jekyll I added syntax highlighting for page Front Matter.
Additionally I added newline as an option for rubyline parameter list which makes multiline parameters look better.

Thanks! :+1:


⚠ Bofrost.lu has degraded performance

In aead8e8, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1262 ms

Resolved: Bofrost.lu performance has improved in 1cc5564 after 6 minutes.


feat(tests): allow app to run matrix of experiments

Changes introduced
Inspired by https://github.com/erikbern/ann-benchmarks, try to run experiments as a matrix.
The experiments can be defined in a .yaml file where the parameters of the experiments are defined. A cartesian product between the parameters is done to run all the possible combinations.

hi, @JoanFM Good job!  I've only some minor suggestions. Would you please add some lines in the README for preparing the data. Plus, additional information about the CLI is also appreciated so that other contributors can quickly use your codes.

I have done some improvements in the documentation, let me know if you think there is something missing.

Related to Issue #2


UnicodeEncodeError

When running with sample data or my own data I am receiving the following error message:
UnicodeEncodeError: 'charmap' codec can't encode character '\u202f' in position 171: character maps to

Same error here, any solution or update?

I fixed this in a PR; you can pull and use the code directly so you don't have to wait for it to get merged.
Here are the exact lines that fix the issue: https://github.com/wjcarpenter/gvoice-sms-takeout-xml/pull/14/files#diff-1c088154e2e367badd78fb5dc7e7ba5902f2f76d2795580fea141afd97bae27bL172

Weird that Github didn't notify me about this issue (or I somehow overlooked it). Thanks @jensenhwa for the PR fix.


feat: add support for BGR888 image format

Hello,
I use the dev version of Sway with NVidia proprietary drivers and I'm unable to take a screenshot using Wayshot.
2024-01-08T16:56:40.302884Z ERROR libwayshot: No suitable frame format found
Error: NoSupportedBufferFormat

Turns out that with my setup, the frame format is BGR888, which is not supported yet. So here is a small PR that seems to make it work for me.
BGR888 is a 3-byte-per-pixel frame format with no transparency, unlike other formats used so far. So in convert_inplace, there is no room in the buffer to turn it into an RGBA image. As a workaround, I use a DynamicImage type in subsequent operations to handle both RGB and RGBA formats.
I'm aware this may not be the best solution, please let me know if there is anything I can do to improve this code :)

Hi I am on my phone so I can't give a full verdict yet. However the patch looks fine to me with just 1 concern.
Please correct me if my understanding is incorrect, but you're coercing the dynamic image into rgba8 - is it possible to let image crate pick the highest supported bit channel? Eg: 12 bit / 16 bit?

@Decodetalkers any comments?

Please correct me if my understanding is incorrect, but you're coercing the dynamic image into rgba8 - is it possible to let image crate pick the highest supported bit channel? Eg: 12 bit / 16 bit?

You're right. I've pushed a new commit to avoid that. Most of this happens in the rotate_image_buffer function. I was not able to turn a DynamicImage into some kind of generic buffer, so I went the other way around so that each branch of the match is a DynamicImage.

@Decodetalkers any comments?

Emm. I think it should be ok

Yep this seems fine to me.

I'll test the pr rq and go ahead with the merge

I'll test the pr rq and go ahead with the merge once I get home.

TY :)

Hey sorry for the late response! @Decodetalkers can you merge this PR? I'll mirror it to sourcehut when I'm home. Just drop a message in this thread once you merge.

Ok


[refactor] Remove unused MergePredicateRule

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[x] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

After This #10854, we use push down predicate scan rule to merge filter with scan node，so the merge predicate rule is useless.
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

run starrocks_admit_test


Enable query for creation order for links and attributes

Enable links and attributes to listed by lexographic order (the default) or creation order.
We already store a creation timestamp with the link and attribute json, so this shouldn't require any format changes.

Implemented in PR: https://github.com/HDFGroup/hsds/pull/225.


[CL][Swapping]: refactor and test CalcInAmtGivenOut

Background
Currently, our CalcInAmtGivenOut has precision and, possibly, logic issues. We should investigate, refactor and test this method.
The issue stems from potentially needing a different equation from CalcOutAmtGivenIn. This equation needs to be investigated and documented.
Acceptance Criteria

refactor the method
table-driven tests with Wolfram links

Closing as addressed in previous PRs


[BUG] Outdenting does not always work in 1.2 release

Describe the bug
Outdenting does not always work/
To Reproduce
I observed weird behaviour, I have this in outline in my vault(I removed some personal notes, but the structure of the outline is the same), I cannot outdent the last bullet points (I am using Outliner 1.2)
- Thinking in Systems
	-  Line 1
		- LIne 2
	- Line 3
	- Nie m liniowych systemow

	- Random link from Internet
		[Swasthi's Recipes - Indian food blog with easy Indian recipes](https://www.indianhealthyrecipes.com/)
	- Cannot move this line with Tab + Shift

When I start outdenting with Shift + Tab from Line 1  everything start working again, and I can out dent the other lines too
Desktop (please complete the following information):

OS: [e.g. iOS]
Obsidian Version: [e.g. 22] 0.12.3

Additional context
Add any other context about the problem here.

@danieltomasz Thank you for your contribution!
Should be fixed in https://github.com/vslinko/obsidian-outliner/releases/tag/1.2.1

hanks! @vslinko  I am really impressed with how fast you are fixing bugs and introducing new features!


strange connection using gosublime package

While working on a Go project I noticed the following:
lsof -i -P
displays:
COMMAND   PID  USER FD   TYPE             DEVICE SIZE/OFF  NODE NAME
gosublime 1320 me   18u  IPv4 0x69c51c56390cce47      0t0  TCP  <IP_ADDRESS>:53249->li396-29.members.linode.com:443 (CLOSE_WAIT)

... this goes away only when I exit Sublime Text 2 (Version2.0.2, Build 2221).
What's this for ?
Is it sending messages to the mother ship ?
thanks

AFAICT, li396-29.members.linode.com is packagecontrol.io but I don't recall the released version of GoSublime having any binary called gosublime (older private versions do) and why it would connect to the internet (unless you shared code on play.golang.org)

Sometimes it just says "Sublime" instead of "gosublime" ... perhaps just reinstalling sublime will work?


[SPARK-32659][SQL] Fix the data issue of inserted DPP on non-atomic type

What changes were proposed in this pull request?

Fix data issue when adding DPP on non-atomic type. for example: spark.range(1000)
.select(col("id"), col("id").as("k"))
.write
.partitionBy("k")
.format("parquet")
.mode("overwrite")
.saveAsTable("df1");

spark.range(100)
.select(col("id"), col("id").as("k"))
.write
.partitionBy("k")
.format("parquet")
.mode("overwrite")
.saveAsTable("df2")

spark.sql("set spark.sql.optimizer.dynamicPartitionPruning.fallbackFilterRatio=2")
spark.sql("set spark.sql.optimizer.dynamicPartitionPruning.reuseBroadcastOnly=false")
spark.sql("SELECT df1.id, df2.k FROM df1 JOIN df2 ON struct(df1.k) = struct(df2.k) AND df2.id < 2").show

It should return two records, but it returns null.
Replace Array with Set in InSubqueryExec to improve performance.

Why are the changes needed?

Fix data issue
Set‘s contains function has better performance: https://www.baeldung.com/java-hashset-arraylist-contains-performance

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Add new unit test.

Test build #127637 has started for PR 29475 at commit ffbed43.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32261/
Test PASSed.

Test build #127637 has finished for PR 29475 at commit ffbed43.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127637/
Test PASSed.

Test build #127670 has started for PR 29475 at commit 4df6496.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32294/
Test PASSed.

Test build #127670 has finished for PR 29475 at commit 4df6496.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127670/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32338/
Test PASSed.

Test build #127715 has started for PR 29475 at commit 3bb8f41.

Test build #127715 has finished for PR 29475 at commit 3bb8f41.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127715/
Test PASSed.

Test build #127894 has started for PR 29475 at commit d1db0bc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32520/
Test PASSed.

Test build #127894 has finished for PR 29475 at commit d1db0bc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127894/
Test PASSed.

Test build #127903 has started for PR 29475 at commit 7c4f2fb.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32529/
Test PASSed.

Test build #127909 has started for PR 29475 at commit f92a594.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/32535/
Test PASSed.

Test build #127903 has finished for PR 29475 at commit 7c4f2fb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127903/
Test PASSed.

thanks, merging to master/3.0!

Test build #127909 has finished for PR 29475 at commit f92a594.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/127909/
Test FAILed.


Add 1.4.0, 1.4.1, 1.4.2, 1.4.3 and 1.4.4 suport

Add 1.4.0 〜 1.4.4 support.
Checked download and install properly
in my machine (MacBook Pro 2019 Mid, Catalina).
Confirm and merge pull request, please.
Thanks and Regards.

thank you!


change to ubi8-minimal base with latest tag

fix #77

can we discuss this for a second? Why do we have to deviate from the operator-sdk default? @mathianasj

@sabre1041 thoughts?


reset lane arrows

fixes #632
Added interface to reset lane arrows back to default. Ignores lanes with outgoing lane connections.
I remove lane flags, call NetAI to recalculate lanes and finally push lane changes.
UI to delete lane arrows : select lane arrow tool -> select segment end -> press Reset all button.
Tests passed:

I tested with segment having lane connections on some lane or on all lanes. In the latter case the reset button should not show. In the first case resetting lane arrows ignores lanes with lane connections.
I tested it with incoming and/or outgoing lane arrows on the same lane. My code can successfully distinguish between them
I tested it with city to see cars obey reset lane arrows.

EDIT: Can someone please add this crowdin key?
 FILE:LaneRouting
"Button:Reset" => "Reset"

@kvakvs Not like it matters, but do you need sorted lanes for that? Or is there a cheaper way to get all lanes?

No I don't need the lanes to be sorted but I don't have another choice.
segment.Info does have m_sortedLanes but it has not been put to use in GetSortedLanes() (optimization idea?). You are right though: this is not a performance critical part of the code. there are only 3 to sort lanes anyways.

@kianzarrin Name it with the correct meaning, of course, i missed there "and" in the function doc. Just don't shorten it if a few more letters make it more clear.

Flickers when pressing reset button:

I go in to lane arrow tool and select this segment:

Looks correct.

But, wait.... Clicking the reset button actually selected a different road.
If I've got a segment selected, and I click reset, the already-selected segment should remain selected. I suspect the issue is related to the flickering observed in last comment.
Also, I think the whole lane arrow UI needs updating at some point. Are we making a rod for our own backs by adding reset via the UI? If we change to a new UI approach at later date, is it going to be weird for users if that reset button no longer exists?

@aubergine10  fixed it.
Unity says:
public static Rect Window(int id, Rect screenRect, GUI.WindowFunction func, string text, params GUILayoutOption[] options);

Returns Rect The rectangle the window is at. This can be in a different position and have a different size than the one you passed in.

but apparently the return value does not work when a button being clicked.
Damn if I did not have my HotReload patch it would have taken me ages !

Will test later this evening.
Does "Reset all" mean...

"for this segment" ? If so, less confusing caption would be "Reset"
"for this junciton" ? If so, less confusing caption would be "Reset junction"?

Reset all means reset everything you can see in the GUI.
OK i will change it to Reset.

BTW, if you have inclination to do so, some additional enhancements:

Allow user to simply press Delete / Backspace to reset the arrows for selected segment -- would be good to have this shortcut work consistent across all tools.
Maybe horizontally center the Reset button?

BTW, if you have inclination to do so, some additional enhancements:
1. Allow user to simply press `Delete` / `Backspace` to reset the arrows for selected segment -- would be good to have this shortcut work consistent across all tools.

we have  a GUI for resting here so there is not that much of a need. I like the idea of resting with delete but the Reset button has to give visual feedback that the user has pressed del. As you know from the other review I don't know how to use hotkeys so move it to #568

2. Maybe horizontally center the `Reset` button?

When I imagine the Reset button at the center, it may give the impression that it only belongs to the center lane. That is why I decided to put it on the side.

TBH I think at some point the lane arrows can get an overhaul so let's leave it as it is for now.
Once @kvakvs or @krzychu124 have done code review we can merge this.

Assume that entire tool workflow will change, you've seen some prototypes in the past, and assume that tool window will be redone too. Don't bother too much now.
I will keep all existing logic code where possible, but the UI will be thrown "oota windah", and made anew.

New translation is working.

This looks good, let me know if it's ok to merge - if so I'll merge today and, when #662 is merged, publish to LABS as 11.1-beta1 so users can start testing it.

Thanks @aubergine10 please merge it.
I also added delete hotkey. very simple code using Input.KeyDown(Delete).
pressing del will turn reset green as a visual feedback. same thing happens if you click green.
More sophisticated code can be handheld as part of #568. In any case the days of this GUI is numbered.

Just noticed there's some coments from kvakvs - did those get implemented?

I am going to test it later today too :)

All has comments from kvakvs  has been addressed now.

@kianzarrin Can you remove the log spam as per @krzychu124 comment above
Then this can be merged.


Update README with new ssh key path

It looks as though the ssh key file name was changed since the
README was written.  This patch will prevent Code Manager
configurations from pointing to the wrong path when following
the instructions.

@abottchen I'm not sure how you got a different key name but the name is currently correct as far as I can tell.
https://github.com/npwalker/pe_code_manager_webhook/blob/master/manifests/code_manager.pp#L8

oh I see I have it backwards. looks like you're removing .key from the end and that is correct


Locale FR not working and errors on issue-event structure

Gitea version (or commit ref): 1.3.0 or 1.3.0rc1
Git version:
Operating system: Debian 9
Database (use [x]):

[X] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
We used version 1.2.3 and eveything was fine.
When we tried 1.3.0 or 1.3.0rc1, our browsers showed bad structure on issue events as in #1413
Multiple browsers were affected (firefox, safari, chromium on OS X and epiphany, chromium on linux), only one was right, because it was a fresh install and never wen't on gitea... the browser is in french but gitea displays in english !
I cleared cookies in Epiphany and it switched to english and the structure went right.

Hi, could you give some screenshots?

Here is the bug in chromium:

If it can help:

this morning I test with a fresh FR install of chromium, the bug was there.
I used a "locale switcher" to set EN locale,
then I deleted gitea cookies and reload the page,
now it's in english, without the bug.

Fixed by  #3017


4466 maximum message channel

Fix #4466
Update the code checking the maximum message size to take the Juliet channels into account.

@marc-casperlabs should this be merged or closed?

Let me review this once again, please.

https://github.com/casper-network/casper-node/pull/4607 has the removal of the network settings.


Error while using your lib :/

Hello sir,
Thank you for your useful work. However, I can't figure out why, while using your lib, I'm getting the following error :
"stack":"TypeError: Cannot read properties of null (reading 'getValue')
at ExcelApp:133:51\n    at Array.map ()
at ExcelApp.getSharedStrings (ExcelApp:132:56)
at ExcelApp.parsXLSX (ExcelApp:112:24)
at new ExcelApp (ExcelApp:21:18)
at SpreadsheetAppp.getImages (SpreadsheetAppp:38:18)",
"message":"Cannot read properties of null (reading 'getValue')"
Would you mind pointing me in the right direction ?
Thank you again for your work.
My code (pretty basic) :
let file = DocsServiceApp.openBySpreadsheetId('<ID>')
console.log(file.obj.spreadsheetId)
let allImages = file.getImages()
The SpreadSheet id logged looks fine, the error "This file ID is not the file ID of Spreadsheet." is not triggered

At first I am getting the same error, and it seems this error has to do with other content on the sheet.
Watch this: https://www.loom.com/share/4254cf14032c40b0806554a7d607a7cd
By changing the text color of a parenthesis this error is gone but I am getting another error:

Here is the link to my test sheet:https://docs.google.com/spreadsheets/d/1OtlQNu7cKelnXBJk2gdCXHJkT0ixNugXfaBkQfQzxto/edit#gid=297574336
The script runs well with the first sheet ('test", however with second sheet, I am getting the above error.

Thank you for your answer. Indeed, if I clear the formatting of the sheet, the error is gone.
My workaround will be to make a copy of the sheet, clear the formatting, get the images and then delete the copy.
For your information, a dirty trick (but working) I was using in the meantime was to access the last spreadsheet revision, and fetch the published version for images.
function dirtyWorkaround(spreadsheetId) {
  //  get latest revision
  let revisions = Drive.Revisions.list(spreadsheetId).items
  let revisionId = revisions[revisions.length-1].id

  Drive.Revisions.update({
    published: true,
    publishedOutsideDomain: true,
    publishAuto: true
  }, spreadsheetId, revisionId)

  // Use of Cheerio lib to fetch revision
  const $ = Cheerio.load(
    UrlFetchApp
      .fetch('https://docs.google.com/spreadsheets/u/0/d/${spreadsheetId}/pub')
      .getContentText()
  );

  let images = [];

  $('img').each(function () {
    const imageBlob = UrlFetchApp
      .fetch($(this).attr('src'))
      .getBlob();
    images.push(imageBlob);
  });

  // Unpublish after use
  Drive.Revisions.update({
    published: false,
    publishedOutsideDomain: false,
    publishAuto: false
  }, spreadsheetId, revisionId);

  // Retourner le tableau d'images.
  return images;
}

I don't quite understand your code and I am not even sure where your code is supposed to be run. It seems to me that it is a javascript code instead of App script code, is it?
Have you also experienced the error in my screenshot? It seems that you have found the cause of this error and fixed it with your code. Did you?

Also, lots of SO questions are saying it is not possible to get blob of the overgridimages but obviously this library does. I am very curious how it manages to do it.

Hello,
I don't experienced the error in your screenshot. You should update the lib to the latest version tho, it might fix it.
Google App Script is javascript-based. You can paste my code in the Google App Script Editor. It will need the Drive service and the Cheerio lib (https://github.com/tani/cheeriogs) to run properly.
It published the spreadsheet as HTML, fetches the content using Cheerio, finds all image elements in the HTML, stores the image data in an array, and then updates the revision to be unpublished again. Pretty dirty I know^^

I think I am using the latest version

"It published the spreadsheet as HTML, fetches the content using Cheerio, finds all image elements in the HTML, stores the image data in an array, and then updates the revision to be unpublished again. Pretty dirty I know^^"
This sounds to me that it doesn't need the DocsServiceApp any more, right?


Example boilerplate for opensource projects

@jtdoepke  @Greatlemer let me know what you think - feel free to modify as needed.
Also let me know if you want to change the repo name.
The idea of this repo is to act as a boilerplate for any opensource (and ideally internal) projects we do.
I'm not sure whether we could do this with a cookiecutter or not (it may be overkill?). Keep in mind this boilerplate should not be specific to any language.
I'm also not sure what we should regarding the LICENSE file, as we'll have 2 or 3 options (but in most cases it'll be MIT).
I'd request people from the chapters to get involved on this as well (as soon as we send around our opensource policy docs).

.editorconfig and  .gitignore are so language-specific I'm not sure they should be included. Although we really do want them in every project. Not sure how to do that without breaking out into language-specific templates.

And I do think we should have a cookiecutter for the most common case: a Python package.

I've set an empty .gitignore.
I've kept .editorconfig and removed the ident_size (I'm not sure anything else in that file is language-specifc), but even if it is - editorconfig would support that (*.md, *.py etc). Let's see what teams think about .editorconfig.


No popup on reimport

Currently, when reimport is called, the importing popup still pops up.

Fixed thanks to mafiesto4! 😄


Default mapper neglects memory constraints

I think that the current version of the default mapper (or at least the one in control_replication) neglects the MemoryConstraints when it creates new instances.
I came up with the following workaround which seems to work in my case but I am not sure about potential side effects (feedback would be very appreciated):
diff --git a/runtime/mappers/default_mapper.cc b/runtime/mappers/default_mapper.cc
index 7ef3192..f041a0c 100644
--- a/runtime/mappers/default_mapper.cc
+++ b/runtime/mappers/default_mapper.cc
@@ -2095,7 +2095,12 @@ namespace Legion {
                 FieldConstraint(overlapping_fields,
                   index_constraints.field_constraint.contiguous,
                   index_constraints.field_constraint.inorder));
-            if (!default_make_instance(ctx, target_memory, creation_constraints,
+            Memory my_target_memory = target_memory;
+            if (creation_constraints.memory_constraint.is_valid())
+               // Change the target memory according to the constraint
+               my_target_memory = Utilities::MachineQueryInterface::find_memory_kind(machine, target_proc,
+                           creation_constraints.memory_constraint.get_kind());
+            if (!default_make_instance(ctx, my_target_memory, creation_constraints,
                          instances.back(), TASK_MAPPING, force_new_instances,
                          true/*meets*/, req, footprint))
               return false;

Can you provide a backtrace for how we're getting into the DefaultMapper::create_custom_instances call? If the kind of target_memory does not align with the memory constraint then something else has already gone wrong.

I'm getting into DefaultMapper::create_custom_instances from DefaultMapper::map_task https://gitlab.com/StanfordLegion/legion/-/blob/master/runtime/mappers/default_mapper.cc#L1580 . I believe that the main issue is that DefaultMapper::default_policy_select_target_memory, called at https://gitlab.com/StanfordLegion/legion/-/blob/master/runtime/mappers/default_mapper.cc#L1574, does not take the memory constraint as an argument and therefore always selects the memory with the best affinity to the target processor.
However, it was much easier for me to hack DefaultMapper::create_custom_instances which already filters the Layout constraints for each region requirement.

Yes, I agree that is an issue with the default mapper. The memory constraint needs to be a part of the mappers decision making when selecting a valid instance. What do you think @elliottslaughter @manopapad ?

It looks like default_policy_select_target_memory indeed doesn't consider memory constraints from the variant's TaskLayoutConstraintSet when selecting a target memory. Then later default_policy_select_constraints overrides any memory constraints coming from the variant https://gitlab.com/StanfordLegion/legion/-/blob/master/runtime/mappers/default_mapper.cc#L2179. I'll take a crack at fixing this.

https://gitlab.com/StanfordLegion/legion/-/merge_requests/297 fixes this bug, @mariodirenzo could you merge this commit into your legion branch and see if the default mapper works out of the box for you?

Works for me. Thank you very much

I think that this is not quite fixed.
I've got a reproducer on sapling at /home/mariodr/htr/unitTests/poissonTest. It needs to run on a GPU of a compute node by using the script run.sh in the folder (after sourcing my .bashrc). It will produce the error
[0 - 7f085febb840]   40.765700 {5}{runtime}: [error 67] LEGION ERROR: Invalid mapper output. Mapper Default Mapper on Processor 1d00000000000003 selected variant 70 for task initFFTplans (ID 5376). But instance selected for region requirement 1 fails to satisfy the corresponding Memory layout constraint. (from file /home/mariodr/legion/runtime/legion/legion_tasks.cc:1676)

If I apply the patch reported earlier in this issue, the program executes fine.

I think that this is not quite fixed.
I've got a reproducer on sapling at /home/mariodr/htr/unitTests/poissonTest. It needs to run on a GPU of a compute node by using the script run.sh in the folder (after sourcing my .bashrc). It will produce the error
[0 - 7f085febb840]   40.765700 {5}{runtime}: [error 67] LEGION ERROR: Invalid mapper output. Mapper Default Mapper on Processor 1d00000000000003 selected variant 70 for task initFFTplans (ID 5376). But instance selected for region requirement 1 fails to satisfy the corresponding Memory layout constraint. (from file /home/mariodr/legion/runtime/legion/legion_tasks.cc:1676)

If I apply the patch reported earlier in this issue, the program executes fine.

Can you use the LoggingWrapper to get some logging output from your mapper? Where you would do new MyMapper(...) instead do new LoggingWrapper(new MyMapper(...)), and run the final executable with -level mapper=2.

Can you use the LoggingWrapper to get some logging output from your mapper? Where you would do new MyMapper(...) instead do new LoggingWrapper(new MyMapper(...)), and run the final executable with -level mapper=2.

The reproducer mentioned in the earlier comment does not have a custom mapper. It just uses the default mapper.
Do you want me to modify the reproducer or to instrument the default mapper?
PS: If I write a custom mapper based on the default mapper and I overwrite default_policy_select_target_memory, everything works fine.

The reproducer mentioned in the earlier comment does not have a custom mapper. It just uses the default mapper.
Do you want me to modify the reproducer or to instrument the default mapper?
PS: If I write a custom mapper based on the default mapper and I overwrite default_policy_select_target_memory, everything works fine.

Please create a minimal custom mapper that just forwards everything to the default mapper, and add a LoggingWrapper to where that custom mapper is instantiated. I would like to see the output of -level mapper=2 for a failing execution, and also please add the following logging code to your mapper (I haven't tried to compile this snippet, it likely has syntax errors):
void MyMapper::map_task(const MapperContext ctx,
                        const Task& task,
                        const MapTaskInput& input,
                        MapTaskOutput& output) {
  DefaultMapper::map_task(ctx, task, input, output);
  if (std::string(task.get_task_name()) == std::string("initFFTplans")) {
    std::cout << "initFFTplans memory constraints:" << std::endl;
    for (unsigned i = 0; i < task.regions.size(); ++i) {
      MemoryConstraint mc = find_memory_constraint(ctx, task, output.chosen_variant, i);
      if (!mc.is_valid()) {
        std::cout << "(none)" << std::endl;
      } else {
        std::cout << mc.get_kind() << std::endl;
      }
    }
  }
}

Please create a minimal custom mapper that just forwards everything to the default mapper, and add a LoggingWrapper to where that custom mapper is instantiated. I would like to see the output of -level mapper=2 for a failing execution, and also please add the following logging code to your mapper (I haven't tried to compile this snippet, it likely has syntax errors):
void MyMapper::map_task(const MapperContext ctx,
                        const Task& task,
                        const MapTaskInput& input,
                        MapTaskOutput& output) {
  DefaultMapper::map_task(ctx, task, input, output);
  if (std::string(task.get_task_name()) == std::string("initFFTplans")) {
    std::cout << "initFFTplans memory constraints:" << std::endl;
    for (unsigned i = 0; i < task.regions.size(); ++i) {
      MemoryConstraint mc = find_memory_constraint(ctx, task, output.chosen_variant, i);
      if (!mc.is_valid()) {
        std::cout << "(none)" << std::endl;
      } else {
        std::cout << mc.get_kind() << std::endl;
      }
    }
  }
}

The output of the logging code is
initFFTplans memory constraints:
(none)
4

and you can find the log of the failing execution at /home/mariodr/htr/unitTests/poissonTest/mapper.log on sapling

The output of the logging code is
initFFTplans memory constraints:
(none)
4

and you can find the log of the failing execution at /home/mariodr/htr/unitTests/poissonTest/mapper.log on sapling

It looks like you are asking that the 2nd region requirement of initFFTplans be placed on zero-copy memory, is this correct? I can't find anything related to this in the directory of the reproducer.

It looks like you are asking that the 2nd region requirement of initFFTplans be placed on zero-copy memory, is this correct? I can't find anything related to this in the directory of the reproducer.

Tha is correct.
You cannot find the registration in that folder because the function that registers that task is defined in /home/mariodr/htr/src/Poisson/Poisson.cc

Tha is correct.
You cannot find the registration in that folder because the function that registers that task is defined in /home/mariodr/htr/src/Poisson/Poisson.cc

I think I found what's wrong, and have uploaded a fix to https://gitlab.com/StanfordLegion/legion/-/tree/bug/954b. Could you please try your reproducer on this branch (using the vanilla default mapper)?

I think I found what's wrong, and have uploaded a fix to https://gitlab.com/StanfordLegion/legion/-/tree/bug/954b. Could you please try your reproducer on this branch (using the vanilla default mapper)?

Yes, that works for me. Thanks

Yes, that works for me. Thanks


Coverage is showing as "None%" On Dashboard , when stop the session throug automation

Hi Michal,
When i Call [teco_url]/set_test_session_end in my suite teardown method (after the test execution is complete). Coverage is showing as "None%" On Dashboard , please refer to screen shot .
Note : if i stop the session manually the coverage is showing properly.

hi I'm aware of the issue, will be fixed in the new release-should be out in April.

Thank you Michal for updating, I'm excited to see new release.


Make metadata handling more rust ergonomic

We ran into this little rust issue with: https://github.com/integritee-network/worker/pull/1143#discussion_r1065449927
This is because we don't cleanly declare the rust types so that the compiler can infer all the types without us having to add some hints. This can easily be mitigated by changing some definitions:
In: https://github.com/integritee-network/worker/tree/master/core-primitives/node-api/metadata/src

pub trait NodeMetadata {
      type Error

      // implement all the getters that we directly implemented on the metadata

}

Then we can do in: https://github.com/integritee-network/worker/blob/master/core-primitives/node-api/metadata-provider/src/lib.rs

impl<NodeMetadata> AccessNodeMetadata for NodeMetadataRepository<NodeMetadata>
where
	NodeMetadata: NodeMetadata,
{
	type MetadataType = NodeMetadata;

	fn get(&self) -> Result<&Self::MetadataType>
	where
	{
		match self.metadata_lock.read().expect("Lock poisoning").deref() {
			Some(metadata) => Ok(metadata),
			None => Err(Error::MetadataNotSet),
		}
	}
}

And finally have a more rust-style interface everywhere in the code e.g.
node_metadata_repo.get()?.unshield_funds_call_indexes()?;

I have overseen that the read lock will be dropped at the end of the function, which implies that we can't return references to the metadata at the end of this function, which in turn would require to clone it every time. This is prohibitely expensive, so I we can't implement it like this.


Orcid pubs integration

Hey @ajkiessl, this is looking great! I'm just going to throw a couple of notes in about questions that I have or things that we might want to look at tomorrow as reminders to myself. No need to respond to anything here in GitHub if you'd rather just discuss in person.

@EricDurante  This is ready to be merged when you are ready.  We can discuss when you get back from vacation if you'd like.

Thanks, Alex!
This looks good. I'll go ahead and merge and deploy it now, and I'll give it a quick test in production.


[PCA10040] can not run bluetooth.bleadv

Describe the bug
I try the example bluetooth.bleadv@pca10040, and download to the nrf52 pca10040 develop board,my moblie can not scan the device,dos anyone try it?
P.S. I also try the Zephyr OS bleadv work.
environment

branch(version): master
Desktop OS(windows linux or macOS):linux
example: bleadv

Expected behavior
Screenshots
Additional context

@Tsai1005 bluetooth.bleadv@pca10040 need to flash to 0x00010000, and need bootloader, check: #806


lnd 0.12.0 can not connect

2021-01-16 14:44:03.648 [INF] LNDC: Creating lnd connection to localhost:10009
2021-01-16 14:44:03.649 [INF] LNDC: Connected to lnd
2021-01-16 14:44:03.661 [INF] LNDC: lnd version: v0.12.0-beta.rc5, build tags ''
2021-01-16 14:44:03.661 [ERR] LITD: Could not start subservers: error checking connected lnd version. at least version "v0.11.0, build tags 'signrpc,walletrpc,chainrpc,invoicesrpc'" is required

it looks like you compiled lnd yourself.  If you do so, you need to run make install tags="signrpc walletrpc chainrpc invoicesrpc".

I did not compile just downloaded latest binaries :
https://github.com/lightninglabs/lightning-terminal/releases/download/v0.3.4-alpha/lightning-terminal-linux-amd64-v0.3.4-alpha.tar.gz

I did not compile just downloaded latest binaries :
https://github.com/lightninglabs/lightning-terminal/releases/download/v0.3.4-alpha/lightning-terminal-linux-amd64-v0.3.4-alpha.tar.gz

thanks mate that fixed a problem .

thanks mate that fixed a problem .


review DiGeorge, velocardiofacial syndrome

MONDO_0008564 and MONDO_0018923 give velocardiofacial syndrome as an exact synonym yet
MONDO_0008644 is defined as velocardiofacial syndrome
What is necessary for clinical practice?
Should velocardiofacial syndrome  be removed as exact synonyms of DiGeorge and 22q11.2 deletion syndrome

@pnrobinson, @ahamosh, @ana-rath-orphanet - do any of you have any feedback/could you offer your clinical expertise?

22q11 deletion is a synonym for both velocardiofacial and DiGeorge syndrome (as both are caused by the deletion).  The difference between VCF (later recognition, not severe heart disease, not neonatal immunodeficiency) and DiGeorge is severity and age of recognition.  So both are 22q11 deletion, but the 2 syndromic names are not clinically identical.

Thanks! @ahamosh
Based on this I recommend @maglott's suggestion of removing the exact synonym (or demoting to RELATED) of VCF from MONDO:0008564 and MONDO:0018923. The first came from NCIT, we may want to pass this along to them. @ana-rath-orphanet I don't know if you want to do the same thing for https://www.ebi.ac.uk/ols/ontologies/ordo/terms?iri=http%3A%2F%2Fwww.orpha.net%2FORDO%2FOrphanet_567

I made the VCF synonyms related synonyms.
The following synonyms were related synonyms for DiGeorge syndrome, but I made them exact (because that seemed incorrect - let me know if I wrong and I can change them back to related):
DGS
Di-George syndrome
DiGeorge syndrome; DGS


Fix/int path dropdown

Jira
https://app.asana.com/0/1126340469288208/1139853633870908/f
https://app.asana.com/0/1126340469288208/1140132371899561/f
Summary
Refactor Interactive Pathways menu dropdown
Details
Previously, Micro Journeys utilized a web component which extended bolt-dropdown.   We've removed this and instead included a simpler version within interactive-pathways.js.   This allows for more design orthodoxy and ease of maintenance.
This has been largely patterned after the language selector on pega.com, with some concessions made in order to use existing mixins.   We may wish to revisit the underline, currently it disappears when the menu is active, as the combination of underline plus the dropdown itself felt broken.
How to test
Viewable here: /pattern-lab/?p=components-micro-journey-in-themes

@glassdimly  I've updated the PR with all feedback.   Note that the hover and divider line were present on my end, curious why they weren't showing for you.  LMK if they still aren't there.


Fix usage findDOMnode(deprecated)

Fixed #117  That can be used into minor release

I'm tempted to say just remove it entirely, since we only really support 0.14 and 15. AFAIK this API has been deprecated since 0.14.

I see, totally agree with your suggestion


Add note in release notes about sub-pages to apphooked pages.

Users may get 404 when going to subpages created with the toolbar menu or page-tree to existing apphooked pages.

Closed by #4753


Added outlier detection with iterativetrimming

Hi @jduerholt ,
I have added the robust GP outlier as in this paper (https://arxiv.org/abs/2011.11057

Thx, I let some comments.

I have answered them

Please also add a specs module as shown here: https://github.com/experimental-design/bofire/blob/main/tests/bofire/data_models/specs/surrogates.py and add the serialization and deserialization tests.

Furthermore, a notebook under tutorials would be nice in which the ITGP outlier detection stuff is demonstrated based on the ITGP paper.
Call it outlier_detection.ipynb


Clarifying #shared to reflect directly injected global identifiers

Addresses: https://github.com/MicrosoftDocs/powerquery-docs/issues/40

Docs Build status updates of commit ccf2635:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

query-languages/m/m-spec-sections.md
: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

Reviewed by Curt.
#sign-off


Support Deno alpine

using denoland/deno:alpine as a base image to run a devcontainer run in this error:
Error relocating /usr/local/lib/libgcc_s.so.1: _dl_find_object: symbol not found
Error relocating /usr/local/lib/libgcc_s.so.1: __cpu_indicator_init: symbol not found
Error relocating /usr/local/lib/libgcc_s.so.1: __cpu_model: symbol not found

reproduction

Hi 👋
Looking at your repro repo, you may want to update your lib local Feature to support it. I am afraid we won't be able to help with it.
Feel free to refer to https://containers.dev/implementors/features/ for more details.


litespeed error get request

litespeed server getting this error when i try to get method. how i can use get method for litespeed?
ERROR ::: Attack_Service ::: 404 :: Error Occured attackId : 496 *** { code: 404,   err:    { Error: read ECONNRESET        at TCP.onStreamRead (internal/stream_base_commons.js:111:27) errno: 'ECONNRESET', code: 'ECONNRESET', syscall: 'read' } } *** INFO ::: Attack_Service ::: Request Succeeded. RUDY attack failed attackId : 478 *** { status: 400,   message:    '\n<title> 400 Bad Request\r\n</title>\n
 400 Bad Request\r\n
\nIt is not a valid request!
\nPowered By LiteSpeed Web Server
\nLiteSpeed Technologies is not responsible for administration and contents of this web site!\n' }

The request is not valid. Litespeed returned in error It is not a valid request!


Lost of Session on Refresh (chrome macbook)

So on safari when i click some links and then i go back and refresh, i still have the ability to use the back button. I can go back to previous pages, i have visited. I can't go forward after a refresh.
However on chrome when i do this, i can't go forward or back.
Is this the correct behaviour? or is something wrong?
I am guessing under the hood it should be using window.history. which should give me those features.
I haven't tested any other OS yet.

i get true on both browsers for window.history && 'pushState' in window.history

Could you please provide a small demo application that demonstrates the behavior? I'm not quite understanding what you are describing.

ok i will take a little time to set it up.

Please re-open another issue when you have a test case we can look at.


Differences between rustic.el and rust-mode.el

These commits are not intended to be merged.
This demonstrates what differences remain between rustic.el and rust-mode.el after merging

https://github.com/brotzeit/rustic/pull/246
https://github.com/rust-lang/rust-mode/pull/406

(All the commits from #406 are included here as well.)
See https://github.com/rust-lang/rust-mode/pull/405 for an overview.
This pull-request was created by:

Applying the above pull-requests.

Evaluating:
(progn
  (copy-file "~/.emacs.d/lib/rustic/rustic.el"
             "~/.emacs.d/lib/rust-mode/rust-mode.el"
             t)
  (with-current-buffer
      (find-file-noselect "~/.emacs.d/lib/rust-mode/rust-mode.el")
    (save-excursion
      (goto-char (point-min))
      (save-excursion (replace-regexp ":group 'rustic" ":group 'rust-mode"))
      (save-excursion (replace-regexp "rustic-mode" "rust-mode"))
      (save-excursion (replace-regexp "rustic" "rust"))
      (save-excursion (replace-regexp "\"1.2\"" "\"0.6.0\"")))))

Manually spitting the differences across multiple commits.

r? @brotzeit
(rust-highfive has picked a reviewer for you, use r? to override)

@tarsius can you please open another pull request that only contains the commits that moves code to other files, this way I still got the diffs(thanks for that).

Ah you did that already, thanks :P

@tarsius can you please open another pull request that only contains the commits that moves code to other files, this way I still got the diffs(thanks for that).

Of course.
By the way, I meant to ask you how you perform the merges and offer some unsolicited advice on how to improve upon that. [I am guessing its one of the "merge" options offered by the Github web interface.] Since there were no new commits on master you could just have fast-forwarded master to the tip of the feature branch. Alternatively you could have created a merge commit. For some reason you picked a third option: create no merge commit (which could be used as a mechanism to signal who performed the integration into master) but for some reason you also did not just fast-forward.
 I would like to understand why you do that because I cannot see any benefits (except for the one that you could instead get by using a merge commit).
There is however the substantial drawback of the feature not actually being merged after such a "merge". One result of that is that you had to make this request, if you had just fast-forwarded, then the respective commits would automatically not have been considered part of this pull-request any more. Likewise I have to transplant some wip feature branches on the new "equal but not eq" history.
Also while Github displays such pull-requests as "merged" even though they haven't actually been merged, other tools don't know that the pull-request has been "merged" using Github.
I usually merge using proper "fast-forward" merges, i.e. no merge commit. I also use my Forge package, which records all tips of all pull-requests. The result is that despite using a linear history and in no way manipulating the contributed commits, the tips of merged pull-requests are still being recorded in the refs/pullreqs/N namespace and those refs are displayed in logs. With your merging style they are lost in time.
Um. That longer to describe than I had expected. I hope it was at least somewhat coherent.

Some images are probably easier to understand.
Lost in time:

Extreme counter-example:

Instead you recommitted each commit (changing the committer name and date). I would like to understand why you do that because I cannot see any benefits (except for the one that you could instead get by using a merge commit).

I was told by several people in the past that performing a "rebase and merge"(in github) is the way to go since you don't have the merge commit and get a cleaner history ?

I was told by several people in the past that performing a "rebase and merge"(in github) is the way to go since you don't have the merge commit and get a cleaner history ?

I've that option did the sensible thing ("if possible, then just fast-forward"), then I would fully agree. But because that isn't the case I perform all my merges using Magit/Forge. (I check out the pull-request and then us m i ("merge into"), which also performs some useful cleanup.)
Or in other words "rebase and merge": sure. "In github"? No, never perform any git operations using github, they usually break things in subtle ways.

But let's not focus on that right now (yes, I know I brought it up...). I should probably write a more coherent text about it at some time, including a howto on what I think people should be doing instead.

What do you want to do about the commands that rustic no longer defines in rustic.el:

*-beginning-of-defun
*-end-of-defun
*-indent-line + helper functions

Should we do the same for rust-mode and move them to a separate library as you did with rustic-interactive.el. (But i would name it rust-commands.el; *-interactive.el is unusual.) Or should we instead move these commands back into rust-mode.el?
I looked at the differences and they are not big, so after a bit of backward and/or forward porting rustic can use the commands from rust-mode. I guess that means moving the commands back into rust-mode.el would be preferable. In any case I would like to first decide which package we rearrange to make it more like the other.

This pull request helped a lot, thanks again.

Great :)


etcdctl should not return 0 when not properly used

Context: we use etcdctl in scripts with set -e mode.  It is expected that etcdctl returns something different of 0 when used with wrong arguments.
(I think standard exit status is 2?)
The issue:
etcdctl -vsdfa
[ ... display usage... ]
echo $?
0

It seems etcdctl is based on   codegansta/cli. There is a bug open there also for that :
https://github.com/codegangsta/cli/issues/276

@fridim Thanks for reporting. There is not much we can do for now. We will update the upstream pkg when the issue is fixed.

I would love to see someone put up a PR to convert to spf13/cobra which we're trying to use everywhere else :-)

@jonboulle I am interested. Is spf13/cobra the preferred choice for etcd?
I think either way, it's good to have one library for all cmd interface.

@gyuho We're trying to use it for all CoreOS Go projects: https://github.com/coreos/docs/blob/master/golang/README.md#cli

Got it. I have wanted to play with cobra. I will try this for etcd. Thanks @jonboulle !


Error on WebPack Task Runner / cmd

Hi,
need help here please,
I use  WebPack Task Runner to run the project from visual studio 2015,
same as running from cmd:
webpack -d
But I getting error:
C:\code\angular2-webpack\webpack.config.ts:20
  DllReferencePlugin,
               ^
TypeError: Cannot read property 'includes' of undefined
    at Object.<anonymous> (c:\code\angular2-webpack\webpack.config.ts:20:16)
    at Module._compile (module.js:570:32)
    at Module.m._compile (c:\code\angular2-webpack\node_modules\ts-node\src\index.ts:403:23)
    at Module._extensions..js (module.js:579:10)
    at Object.require.extensions.(anonymous function) [as .ts] (c:\code\angular2-webpack\node_modules\ts-node\src\index.ts:406:12)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at module.exports (C:\Users\wizardnet972\AppData\Roaming\npm\node_modules\webpack\bin\convert-argv.js:80:13)
    at Object.<anonymous> (C:\Users\wizardnet972\AppData\Roaming\npm\node_modules\webpack\bin\webpack.js:39:40)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.runMain (module.js:604:10)
    at run (bootstrap_node.js:394:7)
    at startup (bootstrap_node.js:149:9)
    at bootstrap_node.js:509:3

@wizardnet972 this is because the EVENT constant is based on which script you run with npm/yarn... to fix, you could just put something like:
const EVENT = process.env.npm_lifecycle_event || '';
or some other default string like 'prod' or 'aot:prod'

@wizardnet972 just added a commit that should address this as well: https://github.com/qdouble/angular-webpack2-starter/commit/2d5aeac66de1e2f50914c6a83294628d298270c0

Yes. Thank you @qdouble :)


Detectron only automatically caches URLs in the Detectron S3 bucket

python tools/infer_simple.py     --cfg configs/DensePose_ResNet101_FPN_s1x-e2e.yaml     --output-dir DensePoseData/infer_out/     --image-ext jpg     --wts https://dl.fbaipublicfiles.com/densepose/DensePose_ResNet101_FPN_s1x-e2e.pkl     DensePoseData/demo_data/demo_im.jpg
/home/SENSETIME/zhushiwei/anaconda3/lib/python3.6/site-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
Found Detectron ops lib: /home/SENSETIME/zhushiwei/anaconda3/lib/python3.6/site-packages/torch/lib/libcaffe2_detectron_ops_gpu.so
[E init_intrinsics_check.cc:43] CPU feature avx is present on your machine, but the Caffe2 binary is not compiled with it. It means you may not get the full speed of your CPU.
[E init_intrinsics_check.cc:43] CPU feature avx2 is present on your machine, but the Caffe2 binary is not compiled with it. It means you may not get the full speed of your CPU.
[E init_intrinsics_check.cc:43] CPU feature fma is present on your machine, but the Caffe2 binary is not compiled with it. It means you may not get the full speed of your CPU.
Traceback (most recent call last):
  File "tools/infer_simple.py", line 176, in <module>
    main(args)
  File "tools/infer_simple.py", line 117, in main
    args.weights = cache_url(args.weights, cfg.DOWNLOAD_CACHE)
  File "/home/SENSETIME/zhushiwei/sense_work/DensePose/detectron/utils/io.py", line 100, in cache_url
    'bucket: {}').format(_DETECTRON_S3_BASE_URL)
AssertionError: Detectron only automatically caches URLs in the Detectron S3 bucket: https://dl.fbaipublicfiles.com/detectron

i download  git clone https://github.com/Johnqczhang/DensePose $DENSEPOSE
run the code than i recieve the following code
how can i fix this problem?
thx!!!!

Hi @zsw667291, please change line-28 as _DETECTRON_S3_BASE_URL = 'https://dl.fbaipublicfiles.com'

Hi @zsw667291, please change line-28 as _DETECTRON_S3_BASE_URL = 'https://dl.fbaipublicfiles.com'

it works !!!
thanks a lot!


Allow data plane configuration at runtime

Changelog

Updates connlib parameter API_URL (formerly known under different names as CONTROL_PLANE_URL, PORTAL_URL, PORTAL_WS_URL, and friends) to be configured as an "advanced" or "hidden" feature at runtime so that we can test production builds on both staging and production.
Makes AUTH_BASE_URL configurable at runtime too
Moves CONNLIB_LOG_FILTER_STRING to be configured like this as well and simplifies its naming
Fixes a timing attack bug on Android when comparing the csrf token
Adds proper account ID validation to Android to prevent invalid URL parameter strings from being saved and used
Cleans up a number of UI / view issues on Android regarding typos, consistency, etc
Hides vars from from the relay CLI we may not want to expose just yet
get_device_id() is flawed for connlib components -- SMBios is rarely available. Data plane components now require a FIREZONE_ID now instead to use for upserting.

Fixes #2482
Fixes #2471

@roop Apple UI change screenshots

@pratikvelani Android screenshot:

@firezone/engineering Could you do another pass on this? It should be read to go in now.
@roop Is giving Apple the same runtime treatment config I gave to Android and CLI components. That'll come in another PR.


Does not validate for urls such as: http://servername01:8153/go/cctray.xml

This is a valid url and it gives the result: "url" is not a valid url
This is causing some problems for our application at the moment.

Sorry for the late reply. This has been fixed now on master.
Make sure you use the allowLocal option for the URL validator.

This is now released in 0.10.0


Add --console-log support

Fixes https://github.com/factoriotools/factorio-docker/issues/460
I don't know when that argument has been introduced, that's why the Available In section is currently blank

Ah, I checked: it's available since at least 0.12.35 which is the first version available in buildinfo.json.


detailed_workflow.ipyb collector.py not working

❓ Questions and Help
Thank you for making this tool.
I am learning the ropes and was going through [\examples\tutorial\detailed_workflow.ipynb](https://github.com/microsoft/qlib/blob/main/examples/tutorial/detailed_workflow.ipynb)
I am stuck at at cell where it is suppose to download extra data
if not p.exists():     !cd ../../scripts/data_collector/pit/ && pip install -r requirements.txt     !cd ../../scripts/data_collector/pit/ && python collector.py download_data --source_dir ~/.qlib/stock_data/source/pit --start 2000-01-01 --end 2020-01-01 --interval quarterly --symbol_regex "^(600519|000725).*"     !cd ../../scripts/data_collector/pit/ && python collector.py normalize_data --interval quarterly --source_dir ~/.qlib/stock_data/source/pit --normalize_dir ~/.qlib/stock_data/source/pit_normalized     !cd ../../scripts/ && python dump_pit.py dump --csv_path ~/.qlib/stock_data/source/pit_normalized --qlib_dir ~/.qlib/qlib_data/cn_data --interval quarterly
I just stays at stalls for a couple hrs and does not download the data. You can see the output here -
https://gist.github.com/vishalkhialani/1b15eb9a50511f67e05cf2d3d7835f75
I am on win11 and tried it on linux too but it did not help.
I tried to debug it but its taking very long as I am new to the tool.  Please can someone guide me on this.

I spent some time and I have narrowed down the issue to be with
\scripts\data_collector\utils.py
resp = requests.get(HS_SYMBOLS_URL.format(s_type=_k), timeout=None)

As http://app.finance.ifeng.com/hq/list.php?type=stock_a&class=ha is giving a 404.
The API has been either deprecated or moved.  I can't fix this as this will need the authors to find an alternative endpoint or the updated one.

Hi, @vishalkhialani , This issue has been resolved in PR 1758, please pull the latest code and retry.


Update orahapris.toml

My PRs: https://github.com/anoma/namada-testnets/pull/747 https://github.com/anoma/namada-testnets/pull/1234 https://github.com/anoma/namada-testnets/pull/1737
Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[x] Only one toml is added in this PR
[x] The file being added is indeed a .toml file
[x] The toml being added is to the correct folder, and only to the correct folder
[x] The eth_hot_key and eth_cold_key are present
[x] The email, discord, elements telegram, and twitter fields are present and valid

Successfully validated transactions.toml file for commit hash 98e0355


A Way to Query Localization Translations

Hi everyone,
I'm creating custom exception handling for my application, and I need BuildContext to show my translated error messages to my users. I find myself doing the following a lot.
This would be in some Widget
try {
// Something that can throw CustomException.
} on CustomException catch(e) {
String errorMessage;
if (e == CustomException.entryNotAvailable) {
errorMessage = context.l10n.entryNotAvailable;
} else if (e == CustomException.someOtherError) {
errorMessage = context.l10n.someOtherError;
}
// Repeated...
showErrorMessage(errorMessage);
} catch (e) {
 debugPrint(e.toString());
}

I want a method where I can pass a String  and get the correct translation from my .arb file like so
in .arb file
"entry_not_available": "This entry is not available",
"@entry_not_available": {
"code": "ERRORCODE123"
}

Using it like so
try {
// Something that can throw CustomException.
} on CustomException catch(e) {
String errorMessage = context.l10n.getTranslationByCode(e.code);
showErrorMessage(errorMessage);
} catch (e) {
 debugPrint(e.toString());
}

Here is my extension method for the sake of completeness
extension L10nBuildContext on BuildContext {
  MyL10n get l10n => MyL10n.of(this)!;
}

and in my CustomException I have
final String code;

  @override
  bool operator ==(Object other) {
    if (other is CustomException) code == other.code;
    return false;
  }

  @override
  int get hashCode => code.hashCode;

I hope this is clear. I don't know if you can add any property to the translation .arb like code above, but I don't see why not since it's just JSON
thank you

CC @shihaohong - perhaps we should optionally support generating a getter that returns the complete JSON object for an ARB translation?


LV2_Atom_Tuple has no member named size

Hi, I'm trying to build synthpod_alsa on a Raspberry Pi 3, and am getting the following build error:
[ 50%] Building C object bin/CMakeFiles/synthpod.alsa.dir/synthpod_nsm.c.o In file included from /home/pi/builds/synthpod/osc.lv2/osc.lv2/util.h:28:0,                  from /home/pi/builds/synthpod/osc.lv2/osc.lv2/writer.h:25,                  from /home/pi/builds/synthpod/bin/synthpod_nsm.c:27: /home/pi/builds/synthpod/osc.lv2/osc.lv2/util.h: In function âlv2_osc_body_unrollâ: /home/pi/builds/synthpod/osc.lv2/osc.lv2/util.h:437:3: error: âLV2_Atom_Tupleâ has no member named âsizeâ    LV2_ATOM_TUPLE_FOREACH(items, atom)    ^ /home/pi/builds/synthpod/osc.lv2/osc.lv2/writer.h: In function âlv2_osc_writer_packetâ: /home/pi/builds/synthpod/osc.lv2/osc.lv2/writer.h:432:3: error: âLV2_Atom_Tupleâ has no member named âsizeâ    LV2_ATOM_TUPLE_FOREACH(items, atom)    ^ /home/pi/builds/synthpod/osc.lv2/osc.lv2/writer.h:459:4: error: âLV2_Atom_Tupleâ has no member named âsizeâ     LV2_ATOM_TUPLE_FOREACH(arguments, atom)     ^ /home/pi/builds/synthpod/osc.lv2/osc.lv2/writer.h:467:4: error: âLV2_Atom_Tupleâ has no member named âsizeâ     LV2_ATOM_TUPLE_FOREACH(arguments, atom)     ^ bin/CMakeFiles/synthpod.alsa.dir/build.make:100: recipe for target 'bin/CMakeFiles/synthpod.alsa.dir/synthpod_nsm.c.o' failed make[2]: *** [bin/CMakeFiles/synthpod.alsa.dir/synthpod_nsm.c.o] Error 1 CMakeFiles/Makefile2:254: recipe for target 'bin/CMakeFiles/synthpod.alsa.dir/all' failed make[1]: *** [bin/CMakeFiles/synthpod.alsa.dir/all] Error 2 Makefile:137: recipe for target 'all' failed make: *** [all] Error 2
I saw an issue similar to this that you solved; it was apparently a bug in LV2 version 1.10. Can you help me out with this one? Thanks!

good catch, thanks for reporting, a bug in LV2 < 1.12, indeed
fixed in 81189de710103f1479a45460a54fadba356f9ca4

You're welcome! It compiles now, thanks for the fix.


Support long-poll for a more responsive initial device setup

If there was a long-poll mechanism to watch for updates to disk images, we should be able to make the device feel much more responsive during initial setup and, potentially, significantly improve the experiences of devices that are always on power.

Unhelpfully this probably requires us to spin up a secondary service, or move the main service over to Go or Node since Flask really doesn't lend itself to this.


Multiple configurations per project & version replacement variable

Projects can work with versions. If a version is finished it can be released. In many situations a version has multiple releases: first to a test environment, then to an acceptance environment, and finally to a production environment. Most of the time these releases are the same, but sometimes an issue is removed for some reason.
I wanted to use this plugin, and made two enhancements to it:

It can now support multiple release configurations per project. Before this, a project had to be unique in the release log configuration. Now (project,queue) has to be unique. With this, it is possible to make release queues for the releases to the different environments. Different mail recipients can already be configured depending if it is an test environment release, or a release to production, which is very nice. When creating a new release log, there is a dropdown to chose the release queue. If the release has no title template, there is a title field for a manual title, if it has a title template the title bar is hidden.
To make working with versions easier, there is a new replacement variable {{version}}. When this is used in the title template, a dropdown for the open (shared) version is shown. By choosing the version and the release queue, it is possible to quickly create a release for a specific environment. There is no integration with the issues in the release, i.e. choosing a version will not fill the issues for that release.

At the moment the code works but I wouldn't advice merging it just yet. I first wanted to discuss to see if such functionality would fit with your idea of what this plugin should do, and how to work with release queues and configurations. I tested it for a complete new setup. It is required to first create a release queue, then a configuration. Also it doesn't deal with the migration of existing queues/configurations to this new functionality. So it's a bit rough, but most functionality is there. If you think this fits with your idea of release logs, I can work on it a bit more and send a complete pull request later.

Great! I will test the branch later. Thanks so much.

it seems there's something error when I create a new release log, the error information is
FATAL -- :
ActionView::Template::Error (undefined method `version_id' for #<ReleaseLog:0x0000000b3fb848>):
    22:
    23:

    24:                 <%= form.select :version_id,
    25:                          options_from_collection_for_select(@versions, :id, :name, release_log.version_id) %>
    26:

    27:
    28:

  plugins/release_logs/app/views/release_logs/_form.html.erb:25:in `block in _plugins_release_logs_app_views_release_logs__form_html_erb__1638464797826983132_94026740'
  app/helpers/application_helper.rb:1030:in `labelled_form_for'
  plugins/release_logs/app/views/release_logs/_form.html.erb:5:in `_plugins_release_logs_app_views_release_logs__form_html_erb__1638464797826983132_94026740'
  plugins/redmine_workflow_enhancements/lib/workflow_enhancements/patches/action_view_rendering.rb:53:in `render_with_workflow_enhancements'
  plugins/release_logs/app/views/release_logs/new.html.erb:2:in `_plugins_release_logs_app_views_release_logs_new_html_erb___628557551839321263_94227680'
  lib/redmine/sudo_mode.rb:63:in `sudo_mode'

@danielvijge ， Thanks.
I rake db migrate and then test it again.
but there's still something wrong. could you tell me how to fix it?
the error is

F, [2016-06-01T23:27:27.139795 #12007] FATAL -- :
ActiveRecord::RecordNotFound (Couldn't find ReleaseLogQueue with 'id'=):
plugins/release_logs/app/models/release_log.rb:132:in queue_title_for_release_log'   plugins/release_logs/app/controllers/release_log_previews_controller.rb:26:in publish_notification'
lib/redmine/sudo_mode.rb:63:in `sudo_mode'

If there are errors, I'll take a look at it of course. Please note that for now, this only works on a new installation. It doesn't correctly migrate old data to be compatible with this feature. I'm still planning to write that as well.

@danielvijge ,
I see.
yesterday, I tested it on the new installation environment and old data environment, and I found there's same issue about it, I will confirmed it later.  But I'll take a look at it once I get around testing it with old data

Pushed some new commits that should fix backward compatibility issues with any release logs created using the old configuration. Old release logs are still displayed as a number in the overview, newer release logs are displayed by the release queue title.
Known issue is there is an error when creating a new release log where there are no Redmine versions at all for project.

@danielvijge , Thanks
I will test it later.


Suggestion: introduce Contribution Guidlines

Since I can't keep but notice some PRs turned down not because of quality of code per se, but due to failure to comply with unspoken contribution standards of this project, which include but are not limited to: commit messages, code styling, naming conventions, commentary or documentation coverage, etc. etc.; with deepest respect to the maintainer of this repository may I suggest introducing contribution guidlines, in a form of CONTRIBUTING.md file, a section in README.md, or any form or shape they find appropriate; in order to prevent any hindrances in acceptance of community contributions in the future.

Since you insist on that on multiples PR... e.g. https://github.com/ImagicTheCat/vRP/pull/636.
Those things will not solve magically because I put a CONTRIBUTING.md file. This file is useful for people wanting to read it, not for people trying their best to contribute on a project sometimes without knowing what they are doing because they are learning Lua/git/GitHub at the same time. And the project didn't receive enough contributions to make this mandatory either.
As you can see, people don't necessarily fix what I point out and that's when I have time to point it out.


Add panlint to rpm

fixes #133

does the rpm resolve all dependencies? in particular the colorama and prettytable ones?

No, I haven't found how you can do that.
Also, do we want to pull that in with the general panc rpm? Might be worth splitting off panlint into a independent python package?

well, the very least requirements is that the panlint works. i'm not sure how critical the colorama and prettytable are for the functionality, but if not critical, wrap the imports in try/except and deal with the missing modules in the code.
to add dependencies, have a look at https://github.com/quattor/CCM/blob/master/pom.xml#L278
if the rpms are in epel or main repo, it's ok
if not, we need to add them to the externals repo

As @jrha mentioned that he's working on panlint, I will hold off any refactoring until I see a PR.
prettytable can easily be made optional, colorama is only used for colors which can also be made optional.

I've updated this to also include the dependencies of panlint in the pan rpm. It's currently hard set to python2. Maybe we should split it off in a independent rpm with a python2 and python3 flavour?

With Python2 reaching EOL on January 1st 2020, we should probably only be targeting Python3.

panlint calls #!/usr/bin/env python -- do we believe it currently works on py2 and py3? We could continue to make it work on py2 but only ship py3 support in the rpm?

At this moment it works both under python2 and python3 but I'm all in favour of focusing on python3.

@wpoely86  - cool, so what's blocking here? Remove the merge commit from this branch, rebase on master, make it work with py3 and we can merge?

@hpcugentbot please test this please

I've verified that the Maven tests all pass correctly locally, the RPM is built with the correct dependencies and installs correctly.
Seems like Jenkins is having issues again.


Exit proxy gracefully with SIGTERM

Fix jasonacox/Powerwall-Dashboard#353

I will not have access to my pc for few days, feel free to update the commit by yourself.
On Saturday, September 23, 2023, Michael Birse wrote:

@mcbirse requested changes on this pull request.

@@ -344,6 +346,11 @@ def do_GET(self):
except:
log.error("Socket broken sending response [doGET]")

+def sigTermHandle(signum, frame):

sys.exit()

My recommendation/preference would be to use raise SystemExit per explanation here
--
Reply to this email directly or view it on GitHub:
https://github.com/jasonacox/pypowerwall/pull/49#pullrequestreview-1640788088
You are receiving this because you authored the thread.
Message ID: @.***

--
Sent from my Sailfish device

Thanks!! 🙏


Change routing_key config in VictorOps config to be a template string instead of string

I note from the docs (https://prometheus.io/docs/alerting/configuration/#<victorops_config>) that the routing_key config is just a string.
I was wondering if its possible to make this a tmpl_string so that people can change which key an alert is routed to by putting a label on the alert with that team's routing key. This would allow us to have teams self manage their notifications without my central team having to add new route configs for each team which is a lot of duplication where the only difference is the routing key.
Is this something that would be considered? If so I am happy to do a PR if someone could just point at the area of code I would need to change...
Many thanks.

It is already templated, the docs are out of date. Would you like to send a PR to fix the docs? It's the 'docs' repo.

Cool, thanks I have done the PR and will add the DCO sign off tomorrow.

Great!


show last hit time in GETMASK and LISTMASK

closes #6

looks like this
<jess> listmask
-bismite-test-   1: /^jess-test!/ (12 hits, 15h7m ago) WARN [a]


Using vgg or googlenet results in Value Error

I am trying to train with googlenet and vgg, alexnet and resnet training works. i am training on imagenet data, with 2 classes. the scripts fails here,
top1acc = tf.reduce_mean(
tf.cast(tf.nn.in_top_k(logits, labels, 1), tf.float32)).
the error.
ValueError: Tried to convert 'predictions' to a tensor and failed. Error: None values not supported.
upon inspection the logits in case of vgg or googlenet is none.

Have you set all the arguments right? I mean have you set --num_classes 2 ?
It is hard to say anything from the error.
Does this happen for every batch (starting from the first batch) or just some of them?
If it just happens for some of them, does this happen for a particular input image?

Yes i tried with with --num_classes argument, still the same. it happens as soon as the script hits the line described above. starting from the initial batch, i am new to tensorflow and ML, dont know how to investigate further, but the problem occurs in logits, while using googlenet or vgg the logits happens to be empty/none.

Have you changed anything in the code?
If you didn't, since you said you are working on the ImageNet dataset,  must be able to reproduce the problem by only having your --data_info file.
Can you share your input file for further investigation?

I found out what the problem was apparently the variable coming from args.model was none, catching else statement, its working now. Thank you.


expose SaveToFile symbol on windows

This pr is to expose the SaveToFile symbol on windows platform for tvm libs, same as other interface like SaveToBinary.

Please fix the code style(alignment) and this can be merged

Thanks @souptc , this is now merged


There are two entries on the left navigation for the Running Kubernetes on Google Compute Engine

One points to the page http://kubernetes.io/docs/getting-started-guides/gce/
The other points to: https://cloud.google.com/container-engine/docs/before-you-begin

No, one is Compute Engine and one is Container Engine. :)


DNMY - trying to tweak nlog output in appveyor

need to see and tweak how it looks in appveyor before merge
Add an nlog.config file to see more details of the messages in appveyor

Hm. Appveyor is still showing a raw output of just the message Issue in message handling loop for queue , region eu-west-1 from the statement Log.Error(ex, msg); in SqsNotificationListener line 201.
This is intended to make debugging easier, but is itself hard to debug. I don't know how appveyor is finding files and so if the nlog.config file is valid or is picked up by appveyor.

You can load the nlog config programmatically if you need, or indeed
configure nlog itself programmatically.
--
Sent from my phone. Please excuse typos and brevity, but never text speak.
On 9 Mar 2016 18:22, "Anthony Steele"<EMAIL_ADDRESS>wrote:

Hm. Appveyor is still showing a raw output of just the message Issue in
message handling loop for queue , region eu-west-1 from the statement Log.Error(ex,
msg); in SqsNotificationListener line 201.
This is intended to make debugging easier, but is itself hard to debug. I
don't know how appveyor is finding files and so if the nlog.config file
is valid or is picked up by appveyor.
—
Reply to this email directly or view it on GitHub
https://github.com/justeat/JustSaying/pull/180#issuecomment-194435597.

We might have to do that. It's not clear at present what it's doing - the output doesn't correspond to either the config file or the default layout mentioned in the online docs. It seems to be just {'${message}'}  so who set that?


add metrics

The docker compose file now starts a metrics gathering backend
The environment variables ENV_STATSHOST and ENV_STATSPORT allow the stats logging
to be controlled, if they are blank then it's disabled.
The environment variables in the docker compose should be modified before the
interface is made public (e.g. to add a non default password to the grafana frontend)

Test by running docker-compose up.
The grafana instance will need a datasource adding, http://graphite/ before the stats are available


Updated azurerm supportability message

Docs Build status updates of commit c202557:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-0.10.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-0.10.0)

docs-conceptual/azps-1.8.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-2.8.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-3.8.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-3.8.0)

docs-conceptual/azps-4.8.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-4.8.0)

docs-conceptual/azps-5.4.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-5.4.0)

docs-conceptual/azps-5.5.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-5.5.0)

includes/migrate-to-az-banner.md
:white_check_mark:Succeeded
View (azurermps-2.5.0) View (azurermps-4.4.1) View (azurermps-5.7.0) View (azurermps-6.13.0)

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 3f54463:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-0.10.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-0.10.0)

docs-conceptual/azps-1.8.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-2.8.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-3.8.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-3.8.0)

docs-conceptual/azps-4.8.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-4.8.0)

docs-conceptual/azps-5.4.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-5.4.0)

docs-conceptual/azps-5.5.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-5.5.0)

includes/migrate-to-az-banner.md
:white_check_mark:Succeeded
View (azurermps-2.5.0) View (azurermps-4.4.1) View (azurermps-5.7.0) View (azurermps-6.13.0)

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Make props import consistent

Describe your changes
made all props consistent
Screenshots - If Any (Optional)
Link to issue

Closes #229
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] Followed the repository's Contributing Guidelines.
[x] I ran the app and tested it locally to verify that it works as expected.
[x] I have checked my code with an automatic accessibility tool such as Axe Dev Tools or Wave
and it shows no errors.

Additional Information (Optional)
Any additional information that you want to give us.

hey @ctoffanin, made requested changes


Create model slots

Found while investigating error in https://github.com/starlite-api/starlite/pull/401
Change Summary
In v1.9.2 __slots__ could be passed to create_model, I'm not sure if it was honoured or ignored, but it didn't cause an error.
In v1.10.0a2 however it caused a warning, and if that warning was ignored, an error creating the model.
With this change a RuntimeWarning is raised, and __slots__ is then ignored.
Related issue number
Caused by #3946 I think.

please review.

Except the failing CI, LGTM 👍

I'll wait for @hot123s (author of #3946) and @PrettyWood to review, then I think we're ready for a final pre-release - v1.10.0b1.

okay, I want to get the (hopefully) last pre-release out, so I'm going to merge this.


pressing CMD+v on latest update produces a workspace zoom-in instead of a Paste of data.

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.75.0
OS Version: macOS 12.6.1

Steps to Reproduce:

Pressing CMD + V produces a workspace zoom in instead of pasting the data.

This started to happen as soon as VSCode was updated. Data is still able to be pasted using Edit > Paste from the menu. The Paste keybinding is still the default CMD+v

Upon further research, it seems that somehow a Keybinding for command+V was added for the zoom in. This was not added by me, and it happened just after updating

I had a similar issue in which a few of my keybindings had random new ones added against them. I didn't do anything, just started my day as usual and found CMD+V, CMD+[, CMD+/ etc were either unresponsive or doing the wrong thing. They were bound to things like "Unfold all" and "Go to end of line" etc. I didn't add them or change anything about my install. The source for these was "User" if that means anything, but I did absolutely nothing. I disabled all my extensions and nothing fixed it except manually removing the erroneous keybindings.
Also on 1.75.0.
MacOS 13.2 (22D49

Here's another one I just noticed broken. For some reason CMD+X has been mapped to "go forward". It does nothing when I press it. Removing the "go forward" binding fixes cut.

Bug also exists for me. I have a custom keybinding for editor zoom in: ⌘ +
Today I started my machine and the keymapping was automatically changed to: ⌘ X
If I want to change the binding again to ⌘ +, VS Code displays cmd+[BracketRight] and when I hit enter, the keybinding is ⌘ X again.
Seems that the + is mapped to X somehow, but only if I reassign a shortcut in VS Code. Everywhere else my keyboard layout is totally fine. I'm on a mac using a german layout.
What I also find out is if I combine Backspace with + the plus sign gets mapped correctly. I will change the binding for now, but really a strange issue.

Duplicate of #173325


Refactor the examples to replace the auxiliary contracts to TestEngine calling scripthash usage

Summary or solution description
Removed auxiliary_contract and auxiliary_contract_2.
Platform:

OS: Windows 10 x64
Python version: Python 3.7

Task linked: CU-2wuw8vg Refactor the examples to replace the auxiliary contracts to TestEngine calling scripthash usage