[Q] How to get the number of seconds (including leap ones) since January 6th, 1980?

Hello Thierry,
I need to use the SCTE-35 XML table with splice_schedule() command.
According to the ANSI doc, the utc_splice_time is a 32-bit unsigned integer quantity representing the time of the signaled splice event as the number of seconds since 00 hours UTC, January 6th, 1980, with the count of intervening leap seconds
included.
Since it is different than the EPOCH time, have you developed a small piece of code able to get the above, or do you know any useful command able to do the same?
Or, is it enough to:

get the EPOCH time between 1970 and 1980
get the EPOCH time between the spliceSchedule and 1970
get the EPOCH time of (#2 - #1) + the EPOCH offset (As of today, 18 seconds. To be checked on IERS or here )

Thanks

Hi @movingDigital,
This is a very interesting question which brings other questions. This is why it will take some time to thing about it and make some tests and modifications.
First, checking the code, I realized there was a bug in the display of splice_schedule() commands. The 32-bit value was interpreted as a UNIX time (from Jan 1st 1970). I fixed it (except leap seconds). Only the display was affected.
Concerning your question, how to manipulate the SCTE 35 utc_splice_time 32-bit value, I initially thought we should be able to rely on the system time of the computer. Normally, "a time is a time" and an official time is a value which includes the leap seconds. Consequently, an operating system is supposed to manipulate official times. So, if date1 and date2 are system dates in seconds, the difference date2-date1 should include the leap seconds. This is unfortunately not the case.
In the Wikipedia article on leap seconds, we have a table of leap seconds. Let's consider two non-leap year (365 days): 2005 has a leap second on Dec 31 while 2006 has none.
How many seconds in a non-leap year, how many seconds in 2005 and 2006, on a macOS and a Ubuntu system (same results):
$ echo $((365 * 24 * 3600))
31536000
$ echo $(($(date '+%s' -d '2006-01-01 00:00:00 UTC') - $(date '+%s' -d '2005-01-01 00:00:00 UTC')))
31536000
$ echo $(($(date '+%s' -d '2007-01-01 00:00:00 UTC') - $(date '+%s' -d '2006-01-01 00:00:00 UTC')))
31536000

All the same, no leap second...
And Windows is not better. Using PowerShell on a Windows 10 20H2 system:
PS C:\> $y2005 = Get-Date -Year 2005 -Month 1 -Day 1 -Hour 0 -Minute 0 -Second 0 -Millisecond 0
PS C:\> $y2006 = Get-Date -Year 2006 -Month 1 -Day 1 -Hour 0 -Minute 0 -Second 0 -Millisecond 0
PS C:\> $y2007 = Get-Date -Year 2007 -Month 1 -Day 1 -Hour 0 -Minute 0 -Second 0 -Millisecond 0
PS C:\> (New-TimeSpan -Start $y2005 -End $y2006).TotalSeconds
31536000
PS C:\> (New-TimeSpan -Start $y2006 -End $y2007).TotalSeconds
31536000

So, yes, we have a problem I did not anticipated.
Let me think about the best way to solve it.
Stay tuned...

Hi @lelegard
Thank you so much once again for your swift response!
I see, noted on the above. Looking at how the utc_splice_time must be implemented in the encoder/transcoder when using spliceSchedule() command, I really wonder if all the vendors really follow the standard by the book, or not.
While I don't see any difficulty in getting the number of seconds (without the leap ones) since 6th January 1980, I am not so sure implementing the "leap seconds support" is easy, since leap second is unpredictable. I see two scenarios:

The system (encoder/transcoder) has access to the information
Or not

I am pretty sure that most of the encoder/transcoder do not have access to the information (= number of leap seconds since 6th January 1980).
Since SCTE-35 allows implementation to ignore the leap second (link here), I propose to keep it simple in your architecture by not taking care of the number of leap seconds. It would be too difficult to retrieve and maintain, since it means that TSDuck would probably need to remain connected to Internet 24/7 to make sure it is always up-to-date.
However, it may be nice to have a variable that we could use to set the number of leap seconds, in order to decorrelate the calculation of the UTC Splice Time, in having:
utcSpliceTime = (UTC Time Since 6th January 1980 without leap seconds) + (number of leap seconds)
Then, up to us to choose to leap seconds or not, based on the encoder/transcoder implementation.
What do you think?

Hi @movingDigital,

Since SCTE-35 allows implementation to ignore the leap second (link here)

The link points to a blog on "Understanding SCTE-35" and says this:

SCTE-35 allows implementation to ignore the leap second but it results in about 16 second error.

I don't see why the guy is allowed to say that. Maybe there is some more specific paragraph in SCTE 35 but the standard defines utc_splice_time in section 9.3.2 as follow:

utc_splice_time – A 32-bit unsigned integer quantity representing the time of the signaled splice event as the number of seconds since 00 hours UTC, January 6th, 1980, with the count of intervening leap seconds included. The utc_splice_time may be converted to UTC without the use of the GPS_UTC_offset value provided by the System Time table.

The expression "with the count of intervening leap seconds included" is pretty clear. There is no room for ignoring leap seconds.
Maybe the confusion comes from "may be converted to UTC without the use of the GPS_UTC_offset value provided by the System Time table". I do not see this as a permission to ignore leap seconds. I see this as an implementation guideline for systems outside the ATSC world. SCTE 35 comes from the US, an ATSC world, but is also used in DVB and ISDB zones. Time references are managed differently.

In DVB and ISDB, the time reference is a UTC (or UTC-related) time in MJD/BCD format in the TDT. It is my understanding that leap seconds are applied to UTC. Hence, a UTC time includes leap seconds or, otherwise said, the difference in seconds between two UTC times should include leap seconds.
In ATCS, the time reference is a GPS time in binary format in the STT. GPS time is a number of seconds since Jan 6 1980 without leap seconds (because the GPS clock is autonomous and uniformely increasing). The STT also contains a GPS_UTC_offset field which indicates the difference in seconds between the GPS and UTC time. This is where leap seconds are.

So, given the format and epoch value, the SCTE 35 time is clearly influenced by the ATSC time. But it emphasizes an important addition: "with the count of intervening leap seconds included". Knowing that, the sentence "may be converted to UTC without the use of the GPS_UTC_offset value provided by the System Time table" should be understood, in my opinion, as "you may use some other time reference than the ATSC STT". This is just my interpretation but it does not seem to allow to ignore leap seconds.

I really wonder if all the vendors really follow the standard by the book, or not.

That is the ultimate practical issue with standards: how far are they correctly unambiguously specified, understood and implemented? Even with extremely precise standards, this is an issue. It is even worse when there is some room for interpretation like here. And anyway, the strongest one wins as usual: when a big industrial actor has a wrong implementation, you are kindly but firmly requested to adapt in a "bug-to-bug" fashion since the big guy doesn't give a **** about your demonstration that its implementation is incorrect. Real-life experience, as you may imagine...
For a test toolbox such as TSDuck, to be used in multiple environments, including environments which do not correctly support standards, I think it is wise to support all modes, with and without leap seconds.
My plan is:

Add an XML "system time" configuration file to TSDuck (other such configuration files already exist for HF bands or LNB). In this file, we specify all known leap seconds (27 so far, not included the initial 10 from the pre-1972 era). Additional leap seconds will be added as IERS will define them.
In the computation of SCTE 35 utc_splice_time, leap seconds will be included by default since this seems to be the standard. Some option will disable them to cope with non-conformant equipments (a usual strategy in TSDuck).
In the XML definition of the splice information table, I will allow the dual definition of utc_splice_time: either a uint32 value which is used literally or a UTC date (YYYY-MM-DD hh:mm:ss) to be interpreted as a uint32 with or without leap seconds, depending on the option.
We may find other interesting usages to this configuration, beyond SCTE 35 utc_splice_time.

Any other suggestion?

Hi @lelegard , this is beautiful, I can't think of anything better.
I have been playing quite a lot with ESAM lately and the standard is relying on UTC Point as there is no PTS information provided (Because it is Out-of-band). The only cue_request_message type allowed is therefore splice_insert (Type 5), and will be inserted according to the desired UTC time. Which in the SCTE35 world is nothing but a splice_schedule (Type 4).
So I really like the idea of allowing the dual definition of utc_splice_time (with or without leap seconds) based on UTC date.
Thank you so much for your dedication once again!

The following modifications have been applied:

The configuration file tsduck.time.xml contains the list of officially registered leap seconds. Should there be any need to add other types of system time customization in the future, we will put them here.
The utc_splice_time attribute in <splice_schedule> entries in the XML definitions of <splice_information_table> now accepts either a 32-bit integer or a "YYY-MM-DD hh:mm:ss" string. In the former case, the binary value is set "as is". In the latter case, an SCTE 35 UTC time is computed. Since the "YYY-MM-DD hh:mm:ss" translation into a UTC time does not include the leap seconds (no operating system does this), the leap seconds are explictly added.
When a  <splice_information_table> is generated (table extraction in XML format), the value of the utc_splice_time attribute is a "YYY-MM-DD hh:mm:ss" string. To make sure that the corresponding date is the intended UTC date, the leap seconds are explicitly substracted before conversion from system time to text.
The option --ignore-leap-seconds has been added in all commands and plugins which manipulate tables. With this option, the leap seconds are no longer explicitly added or substracted.
In the TSDuck user's configuration file (see appendix A in the user's guide), the option leap.seconds can be set to false to enable this option by default.

Hi @lelegard
Thanks heaps! Will try this all ASAP. Would it be possible to add the "T" as well as the milliseconds in the UTC Time, in order to make sure to get a SCTE-35 marker frame accurate?
e.g. "2021-04-06T17:46:03.381Z"

The utc_splice_time field is a number of seconds. There is no millisecond and no frame accuracy in a splice_schedule() command. Only splice_insert() commands have a PTS time stamp with 90 kHz accuracy.
There is some inconsistency in the standard about splice_schedule() commands: the utc_splice_time is a number of seconds while the break_duration() in the same command is a number of PTS units.
My understanding is that splice_schedule() are optional (never seen any in practice) and used to warn that an event may occur around that time without guarantee, just to be prepared. But the actual event will be signaled on time with a splice_insert() command, using a PTS accuracy.
Using splice_schedule() as actual and unique mechanism to signal events seems impossible to me since SCTE 35 was originally designed for video splicing for which frame accuracy is crucial (unlike ad replacement in OTT content which is nothing more than program switching). A splice_schedule() command without a corresponding splice_insert() for the same event seems useless to me. Consequently, there is no frame accuracy in splice_schedule() because we just don't care.

Hi @lelegard ,
My bad, I got carried over from the ESAM/POIS OpenCable Specifications allowing a UTC Point precise to the milliseconds as described here  and here.

I got carried over from the ESAM/POIS OpenCable Specifications allowing a UTC Point precise to the milliseconds as described here and here.

I just had a look and it seems that the UTC in scheduled events has a seconds accuracy only in these ESAM documents. Copy/paste from these docs:
<signal:EventSchedule interval="PT5S">
  <signal:StartUTC utcPoint="2012-09-18T10:00:05Z"/>
  <signal:StopUTC utcPoint="2012-09-18T12:00:00Z"/>
</signal:EventSchedule>

Sorry, the thread was briefly closed by mistake (wrong button).

Hi @lelegard ,
You are right again... :-D I checked all the docs from CableLabs and indeed for both spliceSchedule() and spliceInsert(), there is not notion of milliseconds.
However, I have developed an ESAM server which is inserting an UTC Point such as "2021-04-06T17:46:03**.381Z**" and the encoder/transcoder I am using seems to take it well.


Update FixedMenuLayout.js

Keep maximum compatibility with https://semantic-ui.com/examples/fixed.html

Code from https://semantic-ui.com/examples/fixed.html

VS

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3316   +/-   ##
=======================================
  Coverage   99.89%   99.89%
=======================================
  Files         170      170
  Lines        2802     2802
=======================================
  Hits         2799     2799
  Misses          3        3

Continue to review full report at Codecov.

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


python310Packages.huggingface-hub: 0.9.1 -> 0.10.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.huggingface-hub is: "Download and publish models and other files on the huggingface.co hub"
meta.homepage for python310Packages.huggingface-hub is: "https://github.com/huggingface/huggingface_hub"
meta.changelog for python310Packages.huggingface-hub is: "https://github.com/huggingface/huggingface_hub/releases/tag/v0.10.0"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.10.0 with grep in /nix/store/mz8r8x1dk2fm2q4cny7wy9v6f1xpkqda-python3.10-huggingface-hub-0.10.0
found 0.10.0 in filename of file in /nix/store/mz8r8x1dk2fm2q4cny7wy9v6f1xpkqda-python3.10-huggingface-hub-0.10.0

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

19 total rebuild path(s)

19 package rebuild(s)

First fifty rebuilds by attrpath
openai-whisper
python310Packages.dalle-mini
python310Packages.datasets
python310Packages.huggingface-hub
python310Packages.openai-whisper
python310Packages.spacy-transformers
python310Packages.stanza
python310Packages.tokenizers
python310Packages.transformers
python310Packages.vqgan-jax
python39Packages.dalle-mini
python39Packages.datasets
python39Packages.huggingface-hub
python39Packages.openai-whisper
python39Packages.spacy-transformers
python39Packages.stanza
python39Packages.tokenizers
python39Packages.transformers
python39Packages.vqgan-jax

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mz8r8x1dk2fm2q4cny7wy9v6f1xpkqda-python3.10-huggingface-hub-0.10.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mz8r8x1dk2fm2q4cny7wy9v6f1xpkqda-python3.10-huggingface-hub-0.10.0
ls -la /nix/store/mz8r8x1dk2fm2q4cny7wy9v6f1xpkqda-python3.10-huggingface-hub-0.10.0/bin

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

2 packages failed to build:

python310Packages.spacy-transformers
python39Packages.spacy-transformers

16 packages built:

openai-whisper (python310Packages.openai-whisper)
python310Packages.dalle-mini
python310Packages.datasets
python310Packages.huggingface-hub
python310Packages.stanza
python310Packages.tokenizers
python310Packages.transformers
python310Packages.vqgan-jax
python39Packages.dalle-mini
python39Packages.datasets
python39Packages.huggingface-hub
python39Packages.openai-whisper
python39Packages.stanza
python39Packages.tokenizers
python39Packages.transformers
python39Packages.vqgan-jax

Maintainer pings

superseeded by #195514


Added :request-arrived field with nano time to request map

This is a very straight forward attempt to #273.
I'm had a look at the tests to see what I would have to add there, but I couldn't find something similar.
Please let me know if you had more in mind.

This seems fine, though I'm tempted to namespace the key under aleph, just to differentiate between the Ring keys and non-Ring keys.  If I did this, I should also do it for :keep-alive?.

The namespace prefix sounds like a good idea. Without the prefixing it's not always clear where a map entry in the ring map comes from.
One other thing I considered is using a proxy function instead of (System/nanoTime) directly. This would help during testing as you would be able to override the nano time and test for specific behaviour. Not sure if it would add some unnecessary overhead. This could also be achieved in other ways so I'm not certain adding it on Aleph level would be very beneficial, but as an integration test it could be useful.

I've decided to namespace both of them.  Thanks for your contribution, apologies it took so long to merge.


Change the name Test Suite to Test Suites in UI

CC @Sachin-chaurasiya and @ShaileshParmar11. This could be a beginner issue

Hey, I am a student and a new contributor to the project, can I be assigned this issue? I have read the project documentation and contribution statements. Could you please give me some instructions and point me in the direction of where to find the classes (or folders) concerned with the problem
Thanks.

Hey, I am a student and a new contributor to the project, can I be assigned this issue? I have read the project documentation and contribution statements. Could you please give me some instructions and point me in the direction of where to find the classes (or folders) concerned with the problem Thanks.

Hello @u7283219, Welcome to the community, I have assigned the issue to you.

@u7283219 , Here is the file https://github.com/open-metadata/OpenMetadata/blob/main/openmetadata-ui/src/main/resources/ui/src/components/MyAssetStats/MyAssetStats.component.tsx#L74 where you have to make the changes, let us know if you need any help.

Hey, Can you assign this to me? I am new to open source and want to start contributing

HI! I want to contribute to this issue as a first time contributor. Can you assign it to me?

@u7283219 let us know if you need help. @Suhas2002 and @ayushanand308, looks like this is already assigned. Let me know if you want to work on something else.

Yep, all good I have nearly completed the fix and will submit a pull request shortly.

This issue can be closed, the changes have been successfully merged.


Cannot read property 'children' of null

I am using the "Tabs with Slide Effect" pattern here inside of a dialog. When the dialog closes, it throws the error Cannot read property 'children' of null. Looking at the code at L198, slide is expected to be undefined but in my case it is null.
// ...
  getHeightSlide(index) {
    const slide = this.slides[index];
    // the line in question
    if (slide !== undefined) {
      const child = slide.children[0];
      if (child !== undefined) {
        return child.clientHeight;
      }
    }
    return 0;
  },
// ...

Intereseting.
Following the doc

Note that when the referenced component is unmounted and whenever the ref changes, the old ref will be called with null as an argument. This prevents memory leaks in the case that the instance is stored, as in the first example. Also note that when writing refs with inline function expressions as in the examples here, React sees a different function object each time so on every update, ref will be called with null immediately before it's called with the component instance.

Note that when the referenced component is unmounted and whenever the ref changes, the old ref will be called with null as an argument. This prevents memory leaks in the case that the instance is stored, as in the first example. Also note that when writing refs with inline function expressions as in the examples here, React sees a different function object each time so on every update, ref will be called with null immediately before it's called with the component instance.
https://facebook.github.io/react/docs/more-about-refs.html
Alright, I should fix this!

@oliviertassinari That should do it, thank you!
One sort of tangentially related issue with this and material-ui, since I notice you work on both :wink:. The transform used here breaks any fields that require a fixed overlay (i.e. SelectField). The fixed overlay will be positioned covering the first slide and will be unclickable for any other slide. This I think is due to the width of the slide container being fixed and allowing the slides to overflow.
A quick fix is to add style={ { transform: 'translate3d(0,0,0)' } } to each slide. That creates a new context for the fixed overlay to attach to.

Could you try https://github.com/oliviertassinari/react-swipeable-views/pull/21 for the original issue,

fixed overlay

I'm not sure I follow. What do you consider as a fixed overlay?

That PR fixes the original issue!
By "fixed overlay" I'm referring to the backdrop which you can click to dismiss the popup. It has position: fixed and is intended to fill the viewport. However, because this component uses transform: translate3d(), the backdrop of the SelectField component is affected by the following behavior: http://stackoverflow.com/questions/15194313/webkit-css-transform3d-position-fixed-issue.
To recreate the issue, put a select field inside of a swipeable view. Click the field and try to click away to close it. It will work inside of the first slide (if you click inside of the slide itself), but it will not work for any other slides since these are overflowing the container which the backdrop is filling. If you inspect the backdrop, it will be positioned over the first slide only, no matter which slide it originated from.
One way to fix that is to add transform: 'translate3d(0,0,0)' to each slide to create a new local coordinate system. It might also work to avoid overflowing slides out of the flex container and set it's width to 100% * number_of_slides, and set each child's width to 100% / number_of_slides.

I see, thanks for the explanation. I'm gonna try to reproduce this issue.

https://github.com/callemall/material-ui/pull/2150 was merged to address your issue with the SelectField.
Just wait the next release.

:+1: Great, thank you so much!


fix: fixed a typo in GetSubscribeFileResponseBody

There is a typo in the current code, which doesn't match the description of Get Docs events subscription status page in official document
A rough testing coded was written, but didn't find proper folder to place it though...

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

1.4.3 fixed.


Incorrect indent with template literal

From @unional on June 4, 2017 8:51

VSCode Version: 1.12.2
OS Version: OSX

test(`
`, _t => {
    console.log('wrong indent')
  })

// should be
test(`
`, _t => {
  console.log('wrong indent')
})

Copied from original issue: Microsoft/vscode#27998

Test in TS 2.3.4. The current formatting is inconsistent with the formatting of other multiline arguments such as:
test([
], _t => {
    console.log('wrong indent')
  })

which is formatted to:
test([
], _t => {
	console.log('wrong indent')
})


Implement AutocompleteSearchOperatorDsl

https://www.mongodb.com/docs/atlas/atlas-search/autocomplete/#std-label-autocomplete-ref

I'd like to work on it :)


Update go Version in all files

This pull request updates the go version in all files.

You are just spamming for open source contributions. The version you have updated to does not even exist yet. You have not even cloned and compiled this. Stop this.


Getmoreno.org

getmoreno.org
Moreno-Wallet-Cli
I would like to add the Moreno-Wallet-Cli Software. And the Monerod.
Moreno console program, and account manager.

Please sto spamming nonsense
+invalid


[VS Code] Could not load file or assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'. The system cannot find the file specified.

Error:
Could not load file or assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'. The system cannot find the file specified.
Steps to Reproduce:

Signed in with github (I never signed up, so maybe that's the problem)
on going back to vs code the problem was there

Version Data

extensionName
VSLS

extensionVersion
0.2.206

protocolVersion
2.1

applicationName
VSCode

applicationVersion
1.21.1

platformName
MacOS

platformVersion
16.5.0

@happysalada Thanks for reporting this!  Two things:

This is probably a duplicate of #112. We've found that fully restarting Visual Studio Code if multiple windows were open when you installed can solve this and if not simply uninstalling and reinstalling the extension can fix it.  Can you verify if that solves the issue for you?

Regarding your "I never signed up" comment above: You do not need to have signed up or be accepted into the preview to join someone else's collaboration session as a guest. However, you'll need to be accepted into the preview to share / host a session yourself.  If you email<EMAIL_ADDRESS>after you sign up we can help you with that.

@Chuxel thanks that was fast!
restarting a couple of times worked fine, thanks!


strings.Split an empty string should get a zero length slice, but the return slice's len is 1

What version of Go are you using (go version)?

$ go version
go version go1.12.5 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
widon@widon-PC:~$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/widon/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/widon/go"
GOPROXY=""
GORACE=""
GOROOT="/home/widon/soft/golang/go"
GOTMPDIR=""
GOTOOLDIR="/home/widon/soft/golang/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build437062802=/tmp/go-build -gno-record-gcc-switches"

What did you do?
2 lines of code:
ret := strings.Split("", " ")
fmt.Println(len(ret), ret)
run this code get:
1 []

What did you expect to see?
strings.Split an empty string should get a zero length slice
What did you see instead?
the return slice's len is 1

Thanks for the report. This is working as documented.
The doc for Split says:

If s does not contain sep and sep is not empty, Split returns a slice of length 1 whose only element is s.

Since s (in your case, "") does not contain sep (in your case, " "), the returned slice has length 1 and it has s (i.e.,  ""), inside.
Closing here, since this is working as intended.


Quickstart Missing signout-redirect.js from wwwroot

Love the work guys. Slight issue with a quickstart seed that can be fixed in seconds
Quickstart 6: IdentityServer and ASP.NET Identity
https://github.com/IdentityServer/IdentityServer4.Samples/tree/dev/Quickstarts/6_AspNetIdentity/src/IdentityServerWithAspNetIdentity
In the source code, wwwroot folder is missing signout-redirect.js. So if users request automatic redirect it won't work because LoggedOut.cshtml cannot find the file. Not sure if it is intentional or not (still learning about ID4).
Simple fix is to copy the signout-redirect.js from any other quickstart into the folder and it works.

Done

Sorry @brockallen different timezones lol.
For other peoples benefit another solution is to implement following few steps in the Logout model to bypass the need for the pageload (thus javascript file):
    [HttpPost]
    [AllowAnonymous]
    [ValidateAntiForgeryToken]
    public async Task<IActionResult> Logout(LogoutInputModel model)
    {
        // existing code

        // if you don't want identity server to load the LoggedOut page you can redirect automatically
       // check if there is a URI to redirect to
        if (!vm.PostLogoutRedirectUri.IsNullOrEmpty())
        {
            // if there is a URI then redirect
            return Redirect(vm.PostLogoutRedirectUri);
        }

        // otherwise continue on and display the logout page
        return View("LoggedOut", vm);
    }

For other peoples benefit another solution is to implement following few steps in the Logout model to bypass the need for the pageload (thus javascript file):

I'd discourage this approach, as it prevents the signout callback iframe from having a chance to load.

Interesting didn't consider that. Thanks will review my approach.


Support iOS Cloud Photo Library

Will you be adding support for this? seems to break with a _codec issue if image has not been downloaded to the device! Thanks for all the hard work!

supported #9


Update marketplace publishing description

Suggest updating the "how to publish" portion which says to create a SaaS listing; however, SaaS is not a supported offer type in Azure Gov Marketplace yet.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9cc42c94-0dc7-6979-9ffa-393ce78535f6
Version Independent ID: e618950d-5262-4d65-f32a-159b12cc5c2c
Content: How to accelerate your journey to compliance with Azure - Azure Government
Content Source: articles/azure-government/compliance/documentation-accelerate-compliance.md
Service: azure-government
GitHub Login: @todorgb
Microsoft Alias: todorb

@BrianLevenson Thank you for bringing this to our notice, we will work on this and update the document accordingly. Thank you

@BrianLevenson  Thanks again for bringing this to notice and sharing your feedback , raised a PR to reflect the changes. Will work with @todorgb to have it reviewed and sign off to merge the PR, so that the document reflects accordingly.
At this point it appears that there's no further edits or updates needed to the documentation, other than what is listed above. We will go ahead and close this issue for now, however if you have any further feedback please tag us in your reply and will be happy to reopen the thread and continue the conversation. Thank you


repeat buttons of "organize imports" and "sort imports"

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
when I open a TypeScript file with serval import statements and click the "flash" icon near the row number, the "Organize Imports" and "Sort Imports" selection appears two times repeatedly.
Environment
zed: 0.76.1
system: macOS 13.2.1 (22D68)
arch: Apple M1 Pro ARM64
If applicable, add mockups / screenshots to help explain present your vision of the feature

If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

Duplicate of #969

A fix for this should be hitting stable soon.


RclusTool installation freezes

Describe the bug
The installation of the package RclusTool freezes just after this message:

** byte-compile and prepare package for lazy loading

To Reproduce
Steps to reproduce the behavior:

nix-shell -p pkgs.rPackages.RclusTool

Additional context
I am the main maintainer of this package.

This issue may be related to the fact that our package requires a X server (issue already noticed with mac os X).
I have some difficulties to build the package with Nixos.This may be related too :

cat RclusTool.Rcheck/00install.out

installing source package ‘RclusTool’ ...
** using staged installation
** R
** inst
** byte-compile and prepare package for lazy loading
Error in loadNamespace(j <- i[[1L]], c(lib.loc, .libPaths()), versionCheck = vI[[j]]) :
there is no package called ‘lattice’
Calls:  ... loadNamespace -> withRestarts -> withOneRestart -> doWithOneRestart
Exécution arrêtée
ERROR: lazy loading failed for package ‘RclusTool’
removing ‘/home/hebert/programmation/R/RclusTool/RclusTool.Rcheck/RclusTool’

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

system: "x86_64-linux"
host os: Linux 5.4.35, NixOS, 20.03.1619.ab3adfe1c76 (Markhor)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.4
channels(root): "nixos-20.03.1619.ab3adfe1c76"
channels(hebert): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

I took a look at this and attached to the R process during the byte compilation. Looks like it's stuck in an infinite loop that continuously looks through the r-tcltk2 package files. Other packages that depend on tcltk2 (PDE) seem to work fine, so something specific to RclusTool is triggering this.

Yeah it was missing from the list of packages requiring X.

Thanks for your answer. I will think about giving another chance to Nixos. (I went back to Linux, because I couldn’t use R and its packages properly…)
Regards,
Pierre Hébert

Le 7 sept. 2021 à 08:35, Justin Bedő @.***> a écrit :
Yeah it was missing from the list of packages requiring X.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/NixOS/nixpkgs/issues/87598#issuecomment-914031467, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACSLVX5FC2RMSVFM4PHXV4TUAWXCHANCNFSM4M533CKA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

No worries, you can always run Nix on top of your current distribution and see if R is suitable for your needs without installing NixOS. A healthy R ecosystem is very important to me as we use R heavily in my lab. Tag me in on any R related issues you might have and I'll do my best to resolve them.


Postgres locking DAO

Pull Request type

[ ] Bugfix
[x] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] WHOSUSING.md
[ ] Other (please describe):

Changes in this PR
Implement Postgres locking DAO
Fixes the issues observed in https://github.com/conductor-oss/conductor/pull/81
A remaining issue is the fact that lock leases may not cover the full time the thread want to hold the lock is covered (e.g., long GCs, ...), this issue is present in the Redis lock implementation as well and would be solved the alternative outlined below.
Alternatives considered
Postgres has row level and table level locks, in addition to application-defined locks through advisory locking. Both of these are on the transaction level, which does not work with the current architecture of Conductor as transaction are handled at the lowest level (i.e., per database operation). To allow this to work, a larger overhaul is required where transaction are managed at the service level.

@VerstraeteBert Looks like the build is failing because there's already a V9 migration - renaming to V11 should do it


fix: stop propagating request to backend if not valid

Issue
https://github.com/gravitee-io/issues/issues/7301

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 1.2.1-issues-7301-stop-propagate-request-3-10-SNAPSHOT
download it from Artifactory here

:tada: This PR is included in version 1.2.1 :tada:
The release is available on:

1.2.1
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.2 :tada:
The release is available on:

1.3.2
GitHub release

Your semantic-release bot :package::rocket:


KAFKA-13602: Removing Multicasting partitioner for IQ

Follow up PR for KIP-837. We don't want to allow multicasting for IQ. This PR imposes that restriction.

Merged to trunk and cherrypicked to 3.4


chore(travis): Use Xenial

This migrates the Travis build to run on Ubuntu 16.04 Xenial.
review?(@dahlbyk): This fixes the build to pass again.

Do we need this PR? It looks like you included this change in PR #673.

I’d say merge this and then #673.


Add Finishing state after which state machine stops.

https://github.com/nsk90/kstatemachine/commit/abfad102a0041dd15458b804bc5cfc54394da5c2


It is possible to use sessionStorage instead of cookies

I want to use sessionstorage to save the token so that it is deleted when closing the browser tab, is it possible?

@alfre150 Follow #97.


LISTER

Will the "LISTER: Neighbor Decoding for Length-Insensitive Scene Text Recognition" be introduced?

The code and data are on the way.

waiting ✨

Code has been released.


Flutter Refactor Shortcut abilities

Can we have the ability to apply shortcut to refactoring options? Such as wrapping a widget in a column with the shortcut "ctrl+alt+c" or something similar?
I think it would be cool to have the option to refactor it if your selecting a widget, but also if you were on a blank line to automatically insert the widget associated with the shortcut.

Please file the issue against the Flutter extension.
It is possible to assign key bindings to refactorings.


Triggerkeyvault on proxy changes

So I don't need to manually queue this anymore. I prefer to only merge updates to eng/common/testproxy/target_version.txt after a completed CI run from every major language that uses the proxy.
This PR just ensures that I don't need to remember to queue it manually anymore.

@scbedd

i would make the same change to storage also

Finally got a chance to get back to this. Added!


exclude credits by credit_filter

Filter equals charge type exclude only "Credit"

But it can not fix Tax. And Tax will show real tax

fixed by cd062966b57382baad1fa1f86b46a569a0c7e12d

I only found PR after implementing this myself because needed it for Herring. Will be checking pull requests next time before jumping into implementation


DDP GPU memory imbalanced

🐛 Bug
Creating an issue for the problem described in https://github.com/PyTorchLightning/pytorch-lightning/discussions/6568.
When using DDP on a single node with 8 GPUs the memory usage is highly imbalanced with one using 98% of the memory while the remaining GPUs just use around 70%.
A solution to this might be https://discuss.pytorch.org/t/extra-10gb-memory-on-gpu-0-in-ddp-tutorial/118113.
cc @justusschock @kaushikb11 @awaelchli @akihironitta @rohitgr7

Hi @lukasfolle Can you provide a reproducible script for that?
The solution you mentioned is already implemented in https://github.com/PyTorchLightning/pytorch-lightning/blob/f79d75d4b5e50eb3e37073606eba5ca10d48c99a/pytorch_lightning/strategies/ddp.py#L119 and https://github.com/PyTorchLightning/pytorch-lightning/blob/master/pytorch_lightning/accelerators/gpu.py#L45

Did this change recently?
The script I am running is this (with 8 GPUs on one node): https://github.com/facebookresearch/fastMRI/blob/main/fastmri_examples/varnet/train_varnet_demo.py

@lukasfolle Is there any chance that you could provide a minimal reproducible script or a dummy dataset providing similar outputs?
I would prefer not to download tons of data for trying this.

Did this change recently?
No, this should be quite "old" actually.

The solution you mentioned is already implemented in

I don't see the empty_cache call, but I don't have the deepest understanding of the topic, so it might not be necessary here.
With this code you should be able to reproduce it:
import os
import torch
from torch.utils.data import DataLoader, Dataset

from pytorch_lightning import LightningModule, Trainer

class RandomDataset(Dataset):
    def __init__(self, size, length):
        self.len = length
        self.data = torch.randn(length, size)

    def __getitem__(self, index):
        return self.data[index]

    def __len__(self):
        return self.len

class BoringModel(LightningModule):
    def __init__(self):
        super().__init__()
        self.layer = torch.nn.Linear(32, 2)

    def forward(self, x):
        return self.layer(x)

    def training_step(self, batch, batch_idx):
        loss = self(batch).sum()
        self.log("train_loss", loss)
        return {"loss": loss}

    def validation_step(self, batch, batch_idx):
        loss = self(batch).sum()
        self.log("valid_loss", loss)

    def test_step(self, batch, batch_idx):
        loss = self(batch).sum()
        self.log("test_loss", loss)

    def configure_optimizers(self):
        return torch.optim.SGD(self.layer.parameters(), lr=0.1)

def run():
    dataset_train = RandomDataset(32, 64)
    dataset_val = RandomDataset(32, 64)
    sampler_train = torch.utils.data.DistributedSampler(dataset_train)
    sampler_val = torch.utils.data.DistributedSampler(dataset_val)
    train_data = DataLoader(dataset_train , batch_size=2, sampler=sampler_train, num_workers=64)
    val_data = DataLoader(dataset_val , batch_size=2, sampler=sampler_val, num_workers=64)

    model = BoringModel()
    trainer = Trainer(
        default_root_dir=os.getcwd(),
        gpus=8,
        accelerator='ddp',
        replace_sampler_ddp=False,
    )
    trainer.fit(model, train_data, val_data)

if __name__ == "__main__":
    run()

Hi @lukasfolle , I unfortunately cannot reproduce this.
I have only 2 GPUs, but with your script I trained for 1000 epochs and the output is as follows:

So substracting the two x-server processes, the gpus are at both at 2103 MB (i.e. perfectly on par).
Which lightning version are you using?

Thank you for checking this. I will try to dig deeper into the issue and update here if I notice anything further.
I freshly installed pl.

Thank you for checking this. I will try to dig deeper into the issue and update here if I notice anything further. I freshly installed pl.

Hello，I am also suffering from this problem. Is there any solution now

I also am interested in the solution here


Link on the logo is confusing

Clicking on the logo of Hackdash takes people away from the sub.hackdash where they are, this is confusing for most of people.
I propose clicking on the hackdash logo AND on the sub.hackdash title, get's to the home of the sub.hackdash
Another link, probably at the bottom, should go to the main root hackdash page.
In the dbootcamp UY, lot's of people got confused and made new sub.hackdash pages instead of new projects inside dbootcamp.hackdash

Still "feature" is really annoying

This has been splited into a different approach with the new front-end.
Hope it is more clear now. If it's still annoying we can re-think it.


Are we going to upgrade dependencies or wait for the reactjs.org to upgrade it?

Greetings everyone?, There is dependabot's alert saying critical severity for other localization projects. Do we need to wait for the reactjs.org to upgrade it ? or we can upgrade it by ourselves(localization contributors)?

Please don't update dependencies.


Monitoring - the app cluster-monitoring is stuck on installing state

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

run Rancher: v2.3.0-alpha1
add a REK ec2 cluster (3 etcd, 2 control plane, 3 workers)
enable the cluster monitoring

Result:

the app cluster-monitoring is stuck on installing state but all workloads are active and the cluster monitoring is functional.

Other details that may be helpful:

turn the cluster monitoring off and on will get the app re-deployed and eventually being active
It does not happen all the time, for example, it does not happen on another EC2 cluster  (3 etcd, 1 control plane, 3 workers) on the same Rancher server

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
v2.3.0-alpha1
Installation option (single install/HA):
single node

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.6", GitCommit:"ab91afd7062d4240e95e51ac00a18bd58fddd365", GitTreeState:"clean", BuildDate:"2019-02-26T12:59:46Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.5", GitCommit:"2166946f41b36dea2c4626f90a77706f426cdea2", GitTreeState:"clean", BuildDate:"2019-03-25T15:19:22Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

 18.9.6

Does refresh help?

not sure if it is related to https://github.com/rancher/rancher/issues/20202#issuecomment-492062949
Can you test again 2.2.3, please? The fix is not in v2.3.0-alpha1

The bug fix is validated on Rancher: master beffccd
Steps:

enable and disable the cluster monitoring on several different clusters

Result:

did not see the state of the cluster-monitoring app being stuck on installing  anymore
occasionally saw the following situation, but it goes normal after refreshing the page  (the same observation result on v2.2.4-rc1 )


Panic when clicking on 'Challenge' -> 'Traffic signal survivor'

Clicking on 'Challenge' -> 'Traffic signal survivor' always crashes my abstreet.
OS: Debian 10
Version: v0.2.30
SpawnOverTime each agent (10)... 0.0000s

BorderSpawnOverTime (41)... 0.0002s
Generating scenario small_run... plus 0.0001s
Generating scenario small_run took 0.0004s
[2021-02-14T16:11:43Z WARN  sim::make::scenario] 0 of 1,740 people have nonsense schedules
Instantiating small_run...

trips for People (1,740)... 0.0006s

seed parked cars (247)... 0.0003s

spawn trips (1,740)... 0.0010s
Instantiating small_run... plus 0.0010s
Instantiating small_run took 0.0029s
screensaver traffic... plus 0.0091s
screensaver traffic took 0.0124s

- screensaver traffic took 0.0124s
  - Generating scenario small_run took 0.0004s
    - SpawnOverTime each agent (100)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - SpawnOverTime each agent (10)... 0.0000s
    - BorderSpawnOverTime (41)... 0.0002s
    - ... plus 0.0001s
  - Instantiating small_run took 0.0029s
    - trips for People (1,740)... 0.0006s
    - seed parked cars (247)... 0.0003s
    - spawn trips (1,740)... 0.0010s
    - ... plus 0.0010s
  - ... plus 0.0091s
load ../data/system/seattle/maps/montlake.bin... plus 0.0196s
load ../data/system/seattle/maps/montlake.bin took 0.1838s

- load ../data/system/seattle/maps/montlake.bin took 0.1838s
  - Read ../data/system/seattle/maps/montlake.bin (3)... 0.0545s
  - draw_map took 0.1097s
    - make DrawRoads (590)... 0.0001s
    - make DrawLanes (2,848)... 0.0053s
    - make DrawIntersections (445)... 0.0001s
    - generate unzoomed roads and intersections took 0.0020s
    - make DrawBuildings (3,080)... 0.0324s
    - upload all buildings took 0.0099s
    - make DrawParkingLot took 0.0012s
    - make DrawAreas (67)... 0.0000s
    - upload all areas took 0.0000s
    - create quadtree took 0.0584s
    - ... plus 0.0002s
  - ... plus 0.0196s
[2021-02-14T16:11:45Z WARN  usvg::svgtree::parse] Failed to parse fill value: ''. Fallback to black.
thread 'main' panicked at 'Two buttons in one Panel both use action Traffic signal survivor', widgetry/src/widgets/mod.rs:646:17
stack backtrace:
   0: rust_begin_unwind
             at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:475
   1: std::panicking::begin_panic_fmt
             at /rustc/18bf6b4f01a6feaf7259ba7cdae58031af1b7b39/library/std/src/panicking.rs:429
   2: widgetry::widgets::Widget::get_all_click_actions
   3: widgetry::widgets::Widget::get_all_click_actions
   4: widgetry::widgets::Widget::get_all_click_actions
   5: widgetry::widgets::Widget::get_all_click_actions
   6: widgetry::widgets::panel::PanelBuilder::build_custom
   7: game::challenges::ChallengesPicker::make
   8: <game::challenges::ChallengesPicker as widgetry::app_state::State<game::app::App>>::event
   9: widgetry::app_state::App<A>::event
  10: <std::panic::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once
  11: widgetry::runner::State<A>::event
  12: widgetry::runner::run::{{closure}}
  13: winit::platform_impl::platform::sticky_exit_callback
  14: winit::platform_impl::platform::x11::event_processor::EventProcessor<T>::process_event
  15: winit::platform_impl::platform::x11::EventLoop<T>::run
  16: winit::platform_impl::platform::EventLoop<T>::run
  17: winit::event_loop::EventLoop<T>::run
  18: widgetry::runner::run
  19: game::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

********************************************************************************
Wrote ../data/player/camera_state/seattle/montlake.json
Crash! Please report to https://github.com/a-b-street/abstreet/issues/ and include all output.txt; at least everything starting from the stack trace above!

At 00:00:00.0

Camera:
{ "cam_x": 12821.666000000001, "cam_y": 14453.667, "cam_zoom": 10 }

No edits

Crash! Please report to https://github.com/a-b-street/abstreet/issues/ and include all output.txt; at least everything above here until the start of the report!
********************************************************************************

Thanks for reporting! Fixed in 0f1c30a889bbcf1d3619bee388168b42da6eba45, it'll be in the next release in 1-2 hours

Fixed in https://github.com/a-b-street/abstreet/releases/tag/v0.2.31. Let me know if you hit more problems!


Angular2 Snippets

Carrying over from https://github.com/mgechev/angular2-style-guide/issues/15.
The Angular2 snippets package for Sublime Text are ready.
The PR for publication to Package Control is pending. https://github.com/wbond/package_control_channel/pull/5482.
I noticed mention of snippets in the Style Guide, not sure if this is worth mentioning. Feedback, contributions, and constructive criticism is welcome.

A2 snippets are listed in the official A2 guide which I have migrated to the A2 repo


feature: cmd queue, kill worker

hi @ireznice and @panjan, I already wrote to @h4rr0d that I expect current solutin with sidekiq is not ideal. Please discuss it with @h4rr0d.


Publicize existence of tool

Obviously is a good thing if Zotero users who need something like this know of its existence.
List of relevant threads I found:

https://forums.zotero.org/discussion/35987/import-old-zotero-bibliography-from-word-field-codes-still-present-into-library/
https://forums.zotero.org/discussion/6552/extract-bibliography-from-document/
https://forums.zotero.org/discussion/20310/how-to-import-word-bibliography-into-zotero (in particular https://forums.zotero.org/discussion/comment/212347/#Comment_212347)
https://forums.zotero.org/discussion/57927/recover-zotero-library-for-a-manuscript
https://forums.zotero.org/discussion/58770/how-to-import-citations-bibliography-from-microsoft-word-back-into-zotero
https://forums.zotero.org/discussion/57186/creating-zotero-database-entries-from-document-with-zotero-links

Posted some comments in the identified threads. Tweeted about it too: https://twitter.com/rintzezelle/status/786997733433409537


You may have accidentally replaced some files in the repo with the wrong mod

In 927e7b1, some files were replaced with reference to "FactoryPlanet" which doesn't seem right. Try git revert if this isn't right?

Weirdly, version 1.0.5 on GitHub releases reflects this problem but the Factorio releases doesn't seem to.

lol nice catch, I'll try and redo it

I think it's all correct now


Fix within/writable spelling in .\src\USBHost and .\src\USBHost3GModule

The current status on the Readme page states that the Spelling Check is not passing.

This PR resolves issues in https://github.com/arduino-libraries/Arduino_USBHostMbed5/actions/runs/6708967539 to make sure that the spell check passes https://github.com/arduino-libraries/Arduino_USBHostMbed5/actions/runs/6723001780

We should bump the version number in the next release, since the function rename could potentially be a breaking change.


Check in generated code and update according to buf lint

@shashank11p could you please explain why we need this change? Also, do we truly need to add the gen/go folders? Can't we just use go? Also the go package is option go_package = "github.com/hypertrace/goagent/config/v1"; so I am not sure this directory structure will work.

The gen/go was my suggestion as follows a pattern we've used elsewhere to make it explicit that this is generated code and shouldn't be hand modified.
We'll also need to update the package structure to match the directory structure.

The gen/go was my suggestion as follows a pattern we've used elsewhere to make it explicit that this is generated code and shouldn't be hand modified.

Every file includes a identifiable suffix (pb.go) and a first line stablishing the file is generated, I don't think we need the package name.

The gen/go was my suggestion as follows a pattern we've used elsewhere to make it explicit that this is generated code and shouldn't be hand modified.

Every file includes a identifiable suffix (.pb.go) and a first line stablishing the file is generated, I don't think we need the package name.

fair enough.  let's drop the gen

I would go with /go for the package. Does that make sense @pavolloffay?
On Wed, 14 Apr 2021, 07:08 Shashank Patidar, @.***>
wrote:

@.**** commented on this pull request.
In proto/hypertrace/agent/config/v1/config.proto
https://github.com/hypertrace/agent-config/pull/62#discussion_r612939637
:

@@ -98,20 +98,24 @@ message DataCapture {

// PropagationFormat represents the propagation formats supported by agents
enum PropagationFormat {
+

// Default to none. Agent will use its default propagation format
PROPAGATION_FORMAT_UNSPECIFIED = 0;

Sure
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hypertrace/agent-config/pull/62#discussion_r612939637,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYARQANLNFXQBG4UWRE3TIUPNPANCNFSM42WQA5SQ
.

I think we can go with this. I will first test in goagent.

Ping @davexroth @shashank11p @prodion23 this is the best I could get, please check it out.


HTML fixtures load but JSON fixtures do not

I have JSON fixtures in this path:
spec/javascripts/fixtures/json/
I have html fixtures in: spec/javascripts/fixtures and those load just fine. However, when I try to use the JSON api eg:
loadJSONFixtures(...)
getJSONFixtures(...)
or any other of the available API methods to get JSON fixtures; it causes an internal server error in the jasmine server.

Hello,
I have the same kind of problem, using jasmine-query with gulp
package.json
...
  "devDependencies": {
    [...]
    "gulp": "^3.9.0",
    [...]
    "jasmine": "^2.5.2",
    "jasmine-core": "^2.5.2",
    "jasmine-jquery": "^2.1.1",
    "karma": "^1.3.0",
    "karma-chrome-launcher": "^2.0.0",
    "karma-jasmine": "^1.0.2",
    "karma-phantomjs-launcher": "^1.0.2",
    [...]
  }

karma.conf.js
module.exports = function(config) {
    'use strict';

    config.set({
        autoWatch: true,
        basePath: GLOBAL.dirname,
        frameworks: [
            'jasmine',
        ],
        files: [
            { pattern: 'app/sources/locales/**/*.json', watched: false, served: true, included: false },

            // BOWER
            // bower:js
            'bower_components/jquery/dist/jquery.js',
            'bower_components/jquery-ui/jquery-ui.js',
            'bower_components/i18next/i18next.js',
            // endbower

            // NPM
            'node_modules/jasmine-jquery/lib/jasmine-jquery.js',

            // EXTERNAL
            'https://www.gstatic.com/charts/loader.js',

            // JS
            'app/sources/js/plugins.js',
            'app/sources/js/main.js',

            // TEST
            'app/test/mock/**/*.js',
            'app/test/spec/**/*.js',

            // fixtures
            { pattern: 'app/test/mock/**/*.json', watched: true, served: true, included: false }
        ],
        proxies: {
            '/locales/': GLOBAL.dirname + '/app/sources/locales/'
        },
        exclude: [],
        port: 8080,
        browsers: [
            'PhantomJS',
        ],
        plugins: [
            //'karma-chrome-launcher',
            'karma-phantomjs-launcher',
            //'karma-coverage',
            'karma-jasmine',
        ],
        singleRun: false,
        colors: true,
        logLevel: config.LOG_INFO,
        //reporters: ['progress', 'coverage'],
        //reporters: ['progress'],
    });
};

I solved my problem in... reading the doc !

Note: If you are running your test with Karma, remember that your files are served from a base/ directory, so your path should be configured to: jasmine.getFixtures().fixturesPath = 'base/my/new/path';.

I just add base/ to my path:
jasmine.getJSONFixtures().fixturesPath = 'base/' + myCustomPath;


fixes invalid translation on startup

Fixes #4904

LGTM

Coverage remained the same at 89.227% when pulling facea53380025418a61478da3e046c52a15d6819 on czpython:fixes/issue-4904 into 89778f2623bc346506b1c09c03b257177ef87cd2 on divio:release/3.2.x.


Implementation of fmi2Unit and fmi2SimpleType.

I wanted to try working with Units and these changes seem to allow for it :)
The pull request defines the and exports the mutable types fmi2Unit and fmi2SimpleType which were marked as ToDo.
It also implements BaseUnit and DisplayUnit according to spec, but does not export them.
Up for debate:

Types of the fields are deduced from the specification and up for debate.
Base.show for BaseUnit is modified to print a pretty Unicode String.
I have defined const SI_UNIT_STRINGS = ("kg", "m", "s", "A", "K", "mol", "cd", "rad") for convenient looping, also in FMIImport.

Thank you very much, we will check the PRs and merge if everything is ok.
Much appreciated 👍
@JoKircher


Some weights of VLT5VQA were not initialized from the model checkpoint at t5-base and are newly initialized

I'm running your VQA model on Google colab and I seem to get an error when loading the model weights:
Building Model at GPU 0
Some weights of VLT5VQA were not initialized from the model checkpoint at t5-base and are newly initialized: ['encoder.visual_embedding.feat_embedding.0.weight', 'encoder.visual_embedding.feat_embedding.0.bias', 'encoder.visual_embedding.feat_embedding.1.weight', 'encoder.visual_embedding.absolute_vis_pos_embedding.0.weight', 'encoder.visual_embedding.absolute_vis_pos_embedding.0.bias', 'encoder.visual_embedding.absolute_vis_pos_embedding.1.weight', 'encoder.visual_embedding.obj_order_embedding.weight', 'encoder.visual_embedding.img_order_embedding.weight']
You should probably TRAIN this model on a down-stream task to be able to use it for predictions and inference.
Model loaded from  snap/pretrain/VLT5/Epoch30.pth
_IncompatibleKeys(missing_keys=[], unexpected_keys=['encoder.visual_embedding.layer_norm.weight'])
Model Launching at GPU 0
It took 3.0s

EDIT: This also happens when using the VLT5Pretraining:
Some weights of VLT5Pretraining were not initialized from the model checkpoint at t5-base and are newly initialized: ['encoder.visual_embedding.feat_embedding.0.weight', 'encoder.visual_embedding.feat_embedding.0.bias', 'encoder.visual_embedding.feat_embedding.1.weight', 'encoder.visual_embedding.absolute_vis_pos_embedding.0.weight', 'encoder.visual_embedding.absolute_vis_pos_embedding.0.bias', 'encoder.visual_embedding.absolute_vis_pos_embedding.1.weight', 'encoder.visual_embedding.obj_order_embedding.weight', 'encoder.visual_embedding.img_order_embedding.weight']

You can ignore the first warning, as the encoder.visual_embedding.layer_norm.weight is not used in pretraining/finetuning.
And Some weights of VLT5Pretraining were not initialized from the model checkpoint at t5-base is normal, as those visual projection parameters are added on top of original T5 weights.


Signing in with Azure China & Azure Germany accounts fails to create function apps according to the notification

OS: All
Build Version: 20190905.2
Regression: Not a Regression
Repro Steps:

Sign in with Azure China account.
F1 --> Type and select "Azure Functions: Create Function App in Azure...".
Complete the wizard with available name -->Windows OS --> One runtime --> Select "China East" location.

Pop up an error "...List of available regions for the resource type is 'chinaeast2'"

Try again to create a new function app with 'chinaeast2'.
Check the result.

Expect:
Succeed to create the function app with the 'chinaeast2'.
Actual:
Fail to create with an error "Requested features are not supported in region".

More Info:

Pop up an error when creating a function app signing in with Azure Germany account with "Germany Central & Germany Northeast" locations.

We create function app by the command "Azure Functions: Create Function App in Azure...", not in the process to deploy a project to a new function app.

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.\n\nHappy Coding!

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


Button group with outlined buttons & dropdown does not round if dropdown is last

    <button class="btn btn-outline-secondary" type="button" id="dropdownMenuLink" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false" title="Mod tools"></button>

      Edit

      Remove

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via JS Bin or CodePen and report back with your link, Bootstrap version, and specific browser and OS details.

Pen
Bootstrap: v4.0.0-beta
Browser: Chrome v61 + all chrome-like
OS: Windows 10 v1703 (build 15063.540)

Hi @aolko you are missing the class .dropdown-toggle on class="btn btn-outline-secondary dropdown-toggle"

it is mandatory, but then what if i don't want :after arrow at all?

@aolko that's my point :) I though we should make the caret optional.

fixed the pen

they are a PR for that @aolko (make the caret optional)

Found it ! #22926
Thanks @Johann-S  😄

#22926 was merged, so closing.


Create Github Pages website

Create GitHub Pages website. Add GitHub Pages website to homepage repository

Added GitHub Pages website to homepage repository


Setting the cpu and memory stats of containers in nanoseconds

Signed-off-by: Abhinandan Prativadi<EMAIL_ADDRESS>

LGTM


New Animation Added

I have added a new animation to the buttons "Add new Task" and "All".
Please check and merge (Issue #181 )
https://user-images.githubusercontent.com/74719293/194695012-2d23467b-9b60-442d-9d0a-7f58d0e2e456.mp4

please add the screenshot of the ui change

i have attached a video file in the previous comment


F-Prefix Syntax Errors

I'm getting syntax errors lines 30, 32, 35, 44.
Here is the output:
>>> from scrapy_selenium import SeleniumRequest Traceback (most recent call last):   File "<console>", line 1, in <module>   File "/usr/local/lib/python2.7/site-packages/scrapy_selenium/__init__.py", line 2, in <module>     from .middlewares import SeleniumMiddleware   File "/usr/local/lib/python2.7/site-packages/scrapy_selenium/middlewares.py", line 30     webdriver_base_path = f'selenium.webdriver.{driver_name}'
When running through some linters I get that the issue seems to be coming from the f-prefix that was was introduced in Python 3.  I'm getting this error on Python 2.7.

I just went through some testing and noticed that this package is only compatible with Python 3.  I've adjusted my code accordingly.  Can you add a note to the main package docs noting compatibility?
Either way, thanks for the the package. It is awesome!


Add ability to clear cache to allow real-time refresh of search results.

Pull Request
What does this PR do?
Fixes #790
Implements clearCache method on InstantMeiliSearchInstance to allow realtime updates to search results. This allows algolias refresh example to work out of the box. (https://www.algolia.com/doc/guides/building-search-ui/going-further/improve-performance/vue/#caching)
PR checklist
Please check if your PR fulfills the following requirements:

[X ] Does this PR fix an existing issue?
[X ] Have you read the contributing guidelines?
[X ] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

bors merge

i think this feature is missing in the official documentation. would be great to add that.

#893 :) thanks!


check licence plate duplication towards db when creating

pr: https://github.com/bcgov/platform-services-registry/pull/3284


Document showing

Users should be able to show off their documents to others by providing a permanent link to the document.

For UX, this should work similar to Google's UX:
Who can View This Document?

Only you
Anyone with the link
Anyone who views your profile


workaround url.hostname bug for IPv6 literals - fixes #3

Here's a possible workaround for #3. A proper fix would be to rewrite using the standard URL module, but it looks to be a major undertaking given how different those APIs are (You can't create any invalid URL objects, like you could in the legacy variant).

ping @watson.

re-ping @watson.

Hi @silverwind
Thanks a lot for investigating this bug and coming up with a fix.
I've been wanting to use the WHATWG URL parser as well, but as you discovered, it does not support partial URLs. I think your solution here is a good middle ground.
I've published your fix to npm in version 1.2.2.


sound notificaiton rest period

Hey guys!
I'd like to set a rest period when initializing the pushnotification plugin, for example:
_push = PushNotification.init({
      android: {
        restperiod: {start: "12:30", end: "22:30"}
      },
      ios: {
        restperiod: {start: "18:30", end: "00:30"}
      });

So, if a new notification arrives on my device and the current smartphone time is in the restperiod, the notification volume will be turned OFF
Is this somehow possible?
Thanks! :)

This is kinda a duplicate of #69

Oh, okay, closing this one!


Documentation: Indicate supported Python versions

Hi!
Thanks for your lovely tools.
I believe it would be great for dissemination to be upfront with the supported Python versions.
I fished out from a few issues like e.g. https://github.com/fastai/nbdev/issues/454 that 3.9 wasn't supported in April 2021.
Is that still the case? (A few trials seem to indicate that it still doesn't work).
I cannot find any statements about supported version in the docs, and I think to prevent new users from getting frustrated trying it, it would be good to clearly state them somewhere upfront in the docs.
If you let me know where this should appear I can submit a PR.

Oh, and another thing I'm not sure I understand: Is a py38 nbdev env able to safely compile a py39 package, if it doesn't use any new 3.9 features? Or are there things breaking underneath still?

For me python 3.9.7 is fine despite having "declared" python 3.8 in settings.ini and.github/workflows/main.yml (I'm not using any python 3.9 feature). Anyway, it would be nice python 3.8+ official support.


Remove duplicated images

icon-57.png is the same as icon.png
icon-120.png is the same as<EMAIL_ADDRESS>icon-152.png is the same as<EMAIL_ADDRESS>

Thanks for the PR.
Those sizes have been added following issue #39 & PR #52. Are you sure that removing them won't cause any problem?
I saw today that icon-57.png, icon-60.png, icon-120.png & icon-152.png are considered as "Unassigned" in Xcode, so they indeed may not be required.

Yep, same here. they are unassigned. They are not required.

Let's remove them, we'll see how this goes :game_die:


Data preprocessing with coarse graining does not seem to work

Following the readme instructions I have downloaded the crossdocked, unzipped it and am trying to run the preprocessing script on it with and without the flag --ca_only.
Running
python process_crossdock.py my_data_dir --no_H

runs without errors, but running
python process_crossdock.py .data --no_H --ca_only

fails, giving the error
KeyError "'R' not in amino acid dict (.data/crossdocked_pocket10/WNK1_HUMAN_202_483_0/5tf9_A_rec_5wdy_a6s_lig_tt_min_0_pocket10.pdb, .data/crossdocked_pocket10/WNK1_HUMAN_202_483_0/5tf9_A_rec_5wdy_a6s_lig_tt_min_0.sdf)" WNK1_HUMAN_202_483_0/5tf9_A_rec_5wdy_a6s_lig_tt_min_0_pocket10.pdb WNK1_HUMAN_202_483_0/5tf9_A_rec_5wdy_a6s_lig_tt_min_0.sdf
#failed: 10: 100%|█████████████| 10/10 [00:00<00:00, 128.31it/s]
Traceback (most recent call last):
  File "/home/stratis/repos/DiffSBDD/process_crossdock.py", line 364, in <module>
    lig_coords = np.concatenate(lig_coords, axis=0)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: need at least one array to concatenate

It looks like in the second case, the script is failing to find certain entries in the amino acid dict and is skipping all protein-ligand complexes, resulting in an empty list for lig_coords which can't be concatenated. Looking at the dataset_params dictionary, it seems that there's two sets of preprocessing parameter settings crossdock_full and crossdock. Changing line 24 in the preprocessing script from dataset_info = dataset_params['crossdock_full'] to dataset_info = dataset_params['crossdock'] and running the preprocessing with --ca_only works without any errors, but I'm not sure the resulting data is correctly preprocessed. Is there something wrong with the preprocessing script or am I doing something wrong on my side?

Hi Stratis,
I think the process_crossdock.py file is indeed outdated and should be updated. As far as I can tell, your solution should be fine as a temporary fix because dataset_params['crossdock'] contains the correct amino acid types required for the coarse-grained model (maybe @yuanqidu can confirm). We will try to upload a correct version as soon as possible.
Sorry for the inconvenience!


Is it possible to disable logging to stdout/stderr?

Is your feature request related to a problem? Please describe.
Currently camouflage logs to both camouflage.log and stdout/stderror.
Describe the solution you'd like
Not sure if this is already possible, but I would like to configure these separately. Ideally stderr would only contain logs at the error level, while camouflage.log would contain info and above.
Additional context
Currently I launch camouflage as a sub-process from Python. Since camouflage is left running during the duration of the tests, I redirect stdout/stderr to /dev/null to avoid the possibility of the subprocess PIPE from filling up and blocking.
popen = subprocess.Popen(["camouflage", "--config", "config.yml"],
                            cwd=root_dir,
                            stderr=subprocess.DEVNULL,
                            stdout=subprocess.DEVNULL,
                            preexec_fn=_set_pdeathsig(signal.SIGTERM))

While this works, and I can print the contents of camouflage.log in the event of an error, this fails to cover the cases where camouflage is unable to launch such as not being installed, or the system having some other issue.

I poked around, it appears the console transport is always present:
src/logger/index.ts:
  transports: [
    new winston.transports.Console(),
    new winston.transports.File({
      filename: path.join(process.cwd(), "camouflage.log"),
    }),


[bug|windows] Infinite exit loop after setting the Monitor Trap Flag (MTF)

At the moment, there is an issue, where, after setting the MTF, the RIP ends up in some kernel function (in ntoskrnl.exe).
This has the side effect, that the actual instruction, that we wanted to single step, not being executed, resulting in an infinite loop between EPT Violation and MTF exits.
Note that the above is only the case if thrashing was detected.
I haven't managed to find a solution yet, but a user on the OSR Forum seems to have a similar issue caused by interrupts.
Semantics of Monitor Trap Flag (VT-x)

Fixed.

Did you ever fix this? We are seeing something similar with RIP being somewhere else and you end up with an infinite loop. Not really sure of the right way to fix this.

@rianquinn
I managed to find a passable solution for my (this) case.
First I reduced the amount of logging instructions on the "hot" path, and only output to serial if something unexpected happened.
The second thing I did, is to NOT flip between pages (change the physical address), but instead change the EPTP (pointer). This reduces the "thrashing"/exits significantly.
I don't know if that is a viable solution for you though.

@rianquinn I still use MTF to switch between the shadow-page with the injected INT3 and the normal view exactly the same way @Randshot solved his issue: switching EPTPs.

Possibly, but from my end it doesn't really matter. At the MTF exit the EPT view is switched back to shadow page, so after the interrupt is handled the OS would just trap again when it resumes execution where it left off. This may appear as the trap happening 2x in the logs but otherwise it's not a problem.

Yeah, I also am using MTF for cases where an instruction tries to read memory on the same page.


Publish images using ocm-ci-image-mirror Prow workflow

For robustness of image publishing, we should switch to the new ocm-ci-image-mirror Prow workflow.
Our current use of pull:
https://github.com/openshift/release/blob/master/ci-operator/config/stolostron/submariner-addon/stolostron-submariner-addon-release-2.6.yaml#L22
Example of push:
https://github.com/openshift/release/blob/master/ci-operator/config/stolostron/search-v2-api/stolostron-search-v2-api-release-2.6.yaml#L75
More info from Gurney:

A little background: we have two different styles of image mirroring.  One is a "push" model that uses the ocm-ci-image-mirror workflow (here's an example of its use in search-v2-api).  This is our preferred mirror method and the only non-lossy mirroring method available.  Otherwise, we fall back on a "pull" mirroring model wherein quay-retag (the job linked above) tries to mirror the build from a floating tag in the CI registry.  Pull was the default in ACM 2.3 until ocm-ci-image-mirror was created during 2.4's development cycle.  The pull model is susceptible to timing issues and inconsistency issues between the mirrored content and the actual build.  I can go into more detail if you're curious.
It looks like submariner-addon is still using the pull model (it doesn't use the ocm-ci-image-mirror workflow).  For now, we can try re-running the initial bad quay-retag run and hope that the image to mirror is there this time.  In the longer-term, we probably want to move submariner-addon to the push model using ocm-ci-image-mirror to ensure that content is consistent and we don't hit any timing issues.  The docs on how to configure this can be found here.

I can't assign myself?

/assign @dfarrell07

Tracked in https://issues.redhat.com/browse/ACM-3670


typo

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "sergey-alekseev-patch-1" git@github.com:sergey-alekseev/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so you do not need to open a new one.


fix(RedHat): compare advisories by severity

Description
There are cases, when advisory have different severity for different platforms.
It is also required to compare recommendations by severity.
See more here

fixed


Update words-to-follow.json

Add "git", "github" in Persian and English

Thanks! Why I didn't think of that before? :)

Thanks! Why I didn't think of that before? :)


casting full 360 degree view of vr headsets (android based)

Hi
Is there any way to cast and record one single full 360 degree view of what is playing on Oculus Go ?
thanks

Not with scrcpy, which captures the surface (what is rendered).

Not with scrcpy, which captures the surface (what is rendered).
do you know any other application which can make it?

No, I don't. (I would have said it)


I get this error for env variables which are defined in one step and then used in another step through appending to $GITHUB_ENV

@Heronaixes               I get this error for env variables which are defined in one step and then used in another step through appending to $GITHUB_ENV
      - name: Main version
        run: |
          cd main
          MAIN_VERSION=$(npm exec -c 'node -e "console.log(process.env.npm_package_version)"')
          echo "MAIN_VERSION=$MAIN_VERSION" >> $GITHUB_ENV
          cd ..

      - name: Branch version
        run: |
          cd branch
          BRANCH_VERSION=$(npm exec -c 'node -e "console.log(process.env.npm_package_version)"')
          echo "BRANCH_VERSION=$BRANCH_VERSION" >> $GITHUB_ENV
          cd ..

      - run: |
          echo "Main: ${{ env.MAIN_VERSION }}\nBranch: ${{ env.BRANCH_VERSION }}"

      - name: Fail if version is not bumped
        run: if [ ${{ env.MAIN_VERSION }} == ${{ env.BRANCH_VERSION }} ]; then exit 1; else exit 0; fi

Originally posted by @einarpersson in https://github.com/cschleiden/vscode-github-actions/issues/101#issuecomment-1293892482

This is not something that's currently supported. Might be possible, but not planned at the moment. Parsing out whatever is echo'd into $GITHUB_ENV is beyond the scope of the extension for now.


Security Scan:Oct: IBM Supported security scan and analyze the results

All scan (app/net) which can be performed for BMC .
Will request Nanda to run the test and one of us can analyze the test results and open issues accordingly.

@prkatti1 As you have already run this test cases manually, For Nov. Can we focus on automating the same?


read authzexpiry duration from sa configured and run cleanup based on it

Signed-off-by: mabhi<EMAIL_ADDRESS>What does this PR change?

This PR reads a new label introduced for SA named authz-expiry that is check  for staleness during regular cleanup sweeps and if it happens to be older than the current time during the sweep ticker, the SA and its dependent resources are cleaned up

Does the PR depend on any other PRs or Issues? If yes, please list them.

135
Actual feature 98

Checklist
I confirm, that I have...

[x] Read and followed the contributing guide in CONTRIBUTING.md
[ ] Added tests for this PR
[x] Formatted the code using go fmt (if applicable)
[ ] Updated documentation on the Paralus docs site (if applicable)
[ ] Updated CHANGELOG.md

changes in new PR


Pahtes seem not to be send to XFILES_CONTEXTCMD

To test the functionality of XFILES_CONTEXTCMD I've created this script:
#!/usr/bin/sh

notify-send "triggered"
echo "$@"
notify-send "$@"

When I rightclick in xfiles the notify-send command gets triggered, but the pathes seem not to be there.
Am I missing something?

Have you selected a file or right-clicked on a file?
The context command is called with the selected files as arguments, when there's no selected files, the argument list is empty.
There's also two issues happening here:

echo "$@"

xfiles redirects the stdout of the invoked command to /dev/null, you can try echoing to stderr or into a file to check whether it is working.

notify-send "$@"

If you have more than two files selected, the script will be called with more than two arguments, and notify-send only likes two or less arguments.
$ notify-send a b c d
Invalid number of options.


Rule with polymorphic relationship fails

Here's a test which fails https://github.com/mrloop/cancancan/commit/aba0a064838366739b3f1f1576f5ec2aa337a4b1. Initial error for polymorphic test was uninitialized constant VideoEncoding::VideoEncodable.
Solving this with this commit, https://github.com/mrloop/cancancan/commit/79079125d07571a3a67571bf2068ed7eae6c7a1a, and now have a problem eagerly loading polymorphic relationship. Can not eagerly load the polymorphic association :video_encodable
Thinking I need to update https://github.com/mrloop/cancancan/blob/polymorphic/lib/cancan/model_adapters/active_record_4_adapter.rb#L15 somehow, any suggestions welcome.

Closed because of starvation.
Please feel free to reopen if this issue is still valid or interested in solving with a PR.
Thank you.


Unable to debug on physical device

Thank you very much for creating this plugin. I really like it.
However, I've found that I can't debug on a physical device. Is there something wrong with my configuration?
My plugin version is 0.1.33.
I'm using Xcode 15.2, and my phone's system is iOS 17.6.1.
My launch.json configuration is as follows:
{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "sweetpad-lldb",
            "request": "launch",
            "name": "Attach to running app (SweetPad)",
            "preLaunchTask": "sweetpad: launch",
        },
    ],
}

When I use VSCode's debug feature, the Terminal outputs:
Result bundle written to path:
Build Succeeded

🚀 Executing command:
xcrun devicectl device install app --device iosdevice-00008020-000B34D6012A003F /Users/myname/Library/Developer/Xcode/DerivedData/myapp-djwiypcpaflfvkbkckuttoyimesk/Build/Products/Debug-iphoneos/myapp.app

ERROR: The specified device was not found. (com.apple.dt.CoreDeviceError error 1000.)

🚫 Command returned non-zero exit code

When I use the command: SweetPad: Build & Run (Launch), the Terminal outputs:
Result bundle written to path:
Build Succeeded

🚀 Executing command:
xcrun devicectl device install app --device 00008020-000B34D6012A003F /Users/myname/Library/Developer/Xcode/DerivedData/myapp-djwiypcpaflfvkbkckuttoyimesk/Build/Products/Debug-iphoneos/myapp.app

21:59:41  Acquired tunnel connection to device.
21:59:41  Enabling developer disk image services.
21:59:41  Acquired usage assertion.
1%... 2%... 3%... 4%... 5%... 6%... 7%... 8%... 9%... 10%... 11%... 12%... 13%... 14%... 15%... 16%... 18%... 19%... 20%... 21%... 22%... 23%... 24%... 25%... 26%... 27%... 28%... 30%... 31%... 32%... 33%... 34%... 35%... 36%... 37%... 38%... 39%... 40%... 41%... 42%... 43%... 44%... 45%... 46%... 47%... 48%... 49%... 50%... 51%... 52%... 53%... 54%... 55%... 56%... 57%... 59%... 60%... 62%... 66%... 68%... 72%... 74%... 76%... 80%... 84%... 88%... 92%... 96%... Complete!
App installed:
? bundleID: com.myapp.ios
? installationURL: file:///private/var/containers/Bundle/Application/689DB382-C1B9-4626-B4D7-56ADEA7B5021/myapp.app/
? launchServicesIdentifier: unknown
? databaseUUID: E1D4A799-1D0D-4931-A385-74A49B4B3FD7
? databaseSequenceNumber: 1268
? options:

🚀 Executing command:
xcrun devicectl device process launch --json-output '/Users/myname/Library/Application Support/Cursor/User/workspaceStorage/f8d9db3c1fcf7403dae4fe89e60093e3/sweetpad.sweetpad/_temp/json_f54f27f0' --terminate-existing --device 00008020-000B34D6012A003F com.myapp.ios

22:00:12  Acquired tunnel connection to device.
22:00:13  Enabling developer disk image services.
22:00:13  Acquired usage assertion.
Launched application with com.myapp.ios bundle identifier.

App launched on device with PID: 872
✅ Task completed

When using VSCode's debug feature, the --device parameter is: iosdevice-00008020-000B34D6012A003F (may be wrong)
When using SweetPad's debug feature, the --device parameter is: 00008020-000B34D6012A003F
Additionally, when I use SweetPad: Get app path for debugging, it pops up an error:
Sweetpad: No last launched app path found, please launch the app first using the extension

I've noticed that when running on a physical device, build.lastLaunchedAppPath is not updated. I'm not sure if this is related.
I would greatly appreciate if you could look into this issue. I'm really looking forward to debugging on a physical device.

Hi! Actually debugging currently doesn't work on physical devices. You can only launch app on physical devices.
To fix error that you've mentioned please update the extension to the latest version and try to launch using ▶ button on the extension panel.
https://sweetpad.hyzyla.dev/docs/devices

Thank you very much, I can run it on a physical device.
I would be extremely grateful if you could explain the challenges of debugging on a physical device.
I'd like to try implementing this feature and hope to be of some help.

That would be amazing @wr-fenglei! I was trying to run it on the device today, and having debug support would be wonderful!

In the latest version (0.1.34), I've added the --console option for launching on physical devices. This option will redirect stdout to the extension, allowing you to print debug information on a physical device.

--console — Attaches the application to the console and waits for it to exit. devicectl will wait for the app to terminate. Catchable signals sent to devicectl are forwarded to the app. If the app is not already running, its standard streams will be connected to devicectl's standard streams.

I'd like to try implementing this feature and hope to be of some help.

Thank you! That would be amazing! 💜
I think a good starting point for configuring proper debugging with LLDB is to check this comment in the Flutter project: https://github.com/flutter/flutter/issues/133465#issuecomment-2159512125. From my understanding, launching an LLDB session with devicectl works only with Xcode 16 and requires running these commands in the LLDB session.
(lldb) device select <device-identifier>
(lldb) device process attach --pid <pid>

In my extension, I heavily rely on the CodeLLDB extension for LLDB debugging and my extension just provide the correct options to this extension based on project information. Most of the logic is located in the DebuggerConfigurationProvider class. Additionally, I’ve prepared a document on how to debug the extension on a local machine. This is also applicable for development, but rememeber to fork the project instead of just cloning it: Debugging extension.
If you have any other questions or ideas, don't hesitate to contact me

I'm able to debug app using folowing patch and vscode-ios-debug extension (patched for Xcode 16): https://github.com/sweetpad-dev/sweetpad/pull/76

@kvaster This is great! I'll upgrade to Xcode 16 soon and give it a try.

@kvaster i've tried but not still not working, with a toast could not find pid for <Bundle ID>.

part of my launch.json with bundle id copied from plist file

running app from sweeptpad extension
F5 to run above attach command
Is any thing i missed ?

@bingli-minimax Try replacing  with your app's actual Bundle ID, for example: com.example.myapp

@bingli-minimax Try replacing with your app's actual Bundle ID, for example: com.example.myapp

Actually, i have set my own app Bundle ID and still not working. The Bundle Id in post above is showing example because i do not want to show the app info.

How it works at my place.
You need to install latest sweetpad and vscode-ios-debug. If you're using iOS >= 17 (Xcode >= 16) then you need to install patched version of vscode-ios-debug. Patch is here: https://github.com/nisargjhaveri/vscode-ios-debug/pull/25
Open your project, choose proper schema and configuration and press build and run in sweetpad pane. Make sure you device is unlocked in that moment.
After app will be launched and you'll be able to see app's logs, start debugging from Run and Debug pane. Please make sure your app is running and device is unlocked in that moment.
This is how it works at my place. And I'm using Xcode 16.1, tested also on Xcode 16. And I've not tested it without patch and on Xcode < 16 (iOS < 17).

I've attempted and released a fix in v0.4.1 of vscode-ios-debug for iOS 17+ (with Xcode 16+) support. Thought it might be useful here.

Great! It's really easy now to launch debug. With following config you may install, launch and debug:
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "iOS Debug",
            "type": "lldb",
            "request": "launch",
            "program": "${command:sweetpad.debugger.getAppPath}",
            "iosBundleId": "YOUR BUNDLE ID",
            "iosTarget": "select",
            "preLaunchTask": "sweetpad: build"
        }
    ]
}

@nisargjhaveri , I don’t know how you do this, but it works like magic! 🤩
I’ve added the same logic to the Sweetpad extension https://github.com/sweetpad-dev/sweetpad/blob/main/src/debugger/provider.ts#L77C7-L102C12. Could someone test how it works on your projects and devices? It works on my iPad but randomly fails on my iPhone.


0.11.3 release notes

Codecov Report

Merging #1236 into release/0.11.3 will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           release/0.11.3   #1236   +/-   ##
==============================================
  Coverage            34.2%   34.2%
==============================================
  Files                  36      36
  Lines                2362    2362
==============================================
  Hits                  808     808
  Misses               1451    1451
  Partials              103     103

Continue to review full report at Codecov.

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


Engagement/jgilmore/12536 build vi receiver settings

This PR adds a receiver transform for Virgin Islands and updates the SR sender transform to lookup loinc code descriptions for order and result codes if not present in the received message from SR.
If you are suggesting a fix for a currently exploitable issue, please disclose the issue to the prime-reportstream team directly outside of GitHub instead of filing a PR, so we may immediately patch the affected systems before a disclosure. See SECURITY.md/Reporting a Vulnerability for more information.
Test Steps:

run ./gradlew testIntegration and ./prime test on local

Changes

Include a comprehensive list of changes in this PR

Checklist
Testing

[x ] Tested locally?
[ x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[x ] Added tests?

Linked Issues

Fixes #12536

When I run ./gradlew testIntegration I'm getting gov.cdc.prime.router.fhirengine.translation.hl7.SchemaException: Specified lookup table not found... Is there something else I need to do on local to get this to behave properly?

When I run ./gradlew testIntegration I'm getting gov.cdc.prime.router.fhirengine.translation.hl7.SchemaException: Specified lookup table not found... Is there something else I need to do on local to get this to behave properly?

@adegolier Should have loaded the table when you did a clean package.. Try ./gradlew reloadTables and see if that still gives you the error.

Yep, that did it! Might be an apple chip thing, idk


Implement function to randomly generate dim to semi-bright colors

Hi!
I'd like to contribute if you're open to it, but would need a bit more info on what you're looking for here?

Hi!
I'd like to contribute if you're open to it, but would need a bit more info on what you're looking for here?

hello!
ok so the issue is I have this code snippet that generates a random color in hex that I assign to each label of a person that joins the chat group
def rand_color(def_color='#a5d6a7'):
    r = lambda: random.randint(0, 255)
    return '#%02X%02X%02X' % (r(), r(), r()) if r else def_color

the problem is that, it sometimes generates darker colors which makes the message content unreadable, i'm looking for a better way to approach this issue as currently messages are black text on light backgrounds

Hey! See above - forked and had a look at this my side, think the code in that pull request should be nearer to what you want.
I changed from the lambda function you had previously which generates RGB values into a function which first generates HSL values.
As HSL specifically pinpoints the 'lightness' of colours, I just used a random.uniform() pick method; lightness is expressed between 0 and 1 with 0 being closer to black and 1 closer to white. I just asked it to pick an l value between 0.5 and 1 so we can be sure the generated colour is on the brighter side.
I additionally set the lowest value of the saturation (s) to 0.2 - below this the colours get a bit murky and tend to be more grey, but obviously feel free to play with it and adjust to how you see fit.
It's a bit dirty from there as I decided to keep the final value of the function as a HEX to make sure it didn't break anything further down the line - I kept your original implementation to convert RGB to HEX, as I couldn't find a way to convert directly from HSL. As such, we now convert twice - HSL -> RGB -> HEX.
NB: I tested the rand_color function separately as I wasn't able to get the full ChatApp running on my machine for some reason (line 32 in Server.py hangs, I'm still looking into why that is), so I hope it works! I figured this was a quick win which I could probably do without running the full test, but of course those are 'famous last words'...

Right!! you can control lightness using HSL! how did I miss that! makes so much sense
Anyway, I tested. The new rand_color generates lighter colors that match the black perfectly. I'm sure we can look into how to convert directly from HSL to HEX down the line but for now this works just as I wanted 👍. Your explanation was really clear
Concerning the Line 32 in the server. I forgot to add a print message showing that the server is running. So it was probably running. You should only see something pop up if  ClientCode is run and you've connected unless there was a specific error you got
The only problem I know exists is the path to the other modules since they're in different folders. Pycharm is able to find them but running on desktop throws errors. This could also have been the issue.
Screenshot added

Awesome, I'm so glad it worked well :0)
Noted on the server messaging! I will have another go.
If there's anything else you'd like help on feel free to chuck it my way - I'm just graduating from a degree so have a lot of time on my hands over the summer!

Oh nice. I'll be graduating in September sadly
also as a fix to the code not working if you run it locally without any IDE
Add these 3 lines to the ClientCode.py and it should work
import os
import sys

sys.path.append(os.path.join(os.path.dirname(__file__), '../'))

I'll be pushing this update when I get back to my PC or you can do that :)
since you're free you could take a look at some of the issues opened and we can talk about the way forward


fix for ownedtv plugin

own3d.tv was not working for me, because the regular expression used for finding the channel id did not match. I altered the RE to again find the channel id.

Thanks :)


Fixes issues in tf.contrib.keras.utils.Progbar

In version 1.5 of Tensorflow, if Progbar's target is set to None, internally it gets set to -1.
Changed code that referenced self.target is not None to self.target != -1, self.target is None to self.target == -1.
Also, ProgBar is printing twice as reported here https://github.com/tensorflow/tensorflow/issues/16538. Fixed duplicate text

Never mind, we are syncing the progbar internally. It will show up on GitHub shortly.
Thanks for the PR regardless!