How to run HeatMap with your best pretrained NABirds model?

Thank you so much for the beautiful code.
I'm trying to use your pretrained model best.pt on NABirds dataset.
First, I set the PATH to the pretrained model in NABirds_SwinT.yaml
then I run:
python heat.py --c ./configs/NABirds_SwinT.yaml --img ./vis/001.jpg --save_img ./vis/001/
But I get errors:

Building...
Traceback (most recent call last):
File "C:\Users\xxxxxx\Desktopxxxxxx\heat.py", line 100, in
model.load_state_dict(checkpoint['model'])
File "C:\Users\xxxxxx\anaconda3\envs\xxxxxxx\lib\site-packages\torch\nn\modules\module.py", line 1667, in load_state_dict
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for PluginMoodel:
Unexpected key(s) in state_dict: "combiner.conv_qk1.weight", "combiner.conv_qk1.bias".
size mismatch for combiner.adj1: copying a param with shape torch.Size([85, 85]) from checkpoint, the shape in current model is torch.Size([15, 15]).
size mismatch for combiner.param_pool0.weight: copying a param with shape torch.Size([85, 2720]) from checkpoint, the shape in current model is torch.Size([15, 480]).
size mismatch for combiner.param_pool0.bias: copying a param with shape torch.Size([85]) from checkpoint, the shape in current model is torch.Size([15]).
size mismatch for combiner.param_pool1.weight: copying a param with shape torch.Size([1, 85]) from checkpoint, the shape in current model is torch.Size([1, 15]).

It seems that your best,pt model is not using default SwinT model, the num_selects is not matching
And there are unexpected keys in your best.pt model: "combiner.conv_qk1.weight", "combiner.conv_qk1.bias"
I wish to know what modification I should make to load your pretrained best.pt.

I managed to make it work by :

Change num_selects in NABirds_SwinT.yaml to
num_selects:
layer1: 2048
layer2: 512
layer3: 128
layer4: 32
Delete keys "combiner.conv_qk1.weight" and "combiner.conv_qk1.bias" in best.pt model

Although the heatmap is now output as normal, I am concerned that this will affect the performance of the model. I hope to know a better solution.


Option to remove (or add) replay link from output

In order to have softer outputs, it would be nice to have an option (either passed as argument when doing python -m cgsubmit -t token or set via some kind configuration command) that removes (or adds, depending on what you want the default behaviour to be) the https://www.codingame.com/replay/{replayId} part of the output for lost games

done in release 1.1.0


Update P'ing-tung regions

Since there is no P'ing-chung in Taiwan, I think it should be P'ing-tung.
This PR update to the P'ing-tung regions

Thanks @sc0Vu!


HLS.js 1.1.1

Hi, can you please update hls.js to 1.1.1?
https://github.com/video-dev/hls.js/blob/master/MIGRATING.md

we can download and unzip videojs-hlsjs-plugin project then
edit package.json and change hls.js version into "1.1.1"
"dependencies": {
"hls.js": "1.1.1"
}
then
npm install
then
npm run build

This not work, videojs return for me error: VIDEOJS: ERROR: (CODE:2 MEDIA_ERR_NETWORK) HLS.js error: networkError - fatal: true - manifestLoadError
Object { code: 2, message: "HLS.js error: networkError - fatal: true - manifestLoadError" },
Via this way i can only update to hls.js 0.14.17

i think you should delete first /node_modules and call npm install again before npm run build

I noticed updating hls.js breaks "p2p-media-loader". Is there any ideas on why that happens?

We just open a PR to bump hls.js to version 1.3 : #115


Second try at specialized JSON: RapidJSON + custom assembly

This replaces #1162.
Forth may be good for binary formats (especially unruly ones like ROOT, where you really need the Turing completeness!), but parsing JSON just required too many instructions (at 7 ns per instruction). This PR implements a very simple language for encoding the JSON → Awkward rules, given a JSONSchema, and lets RapidJSON do the JSON parsing. These instructions are invoked by RapidJSON's SAX interface. It's called "JSON assembly" because it's basically a virtual assembly language.
Testing again with this example:
MULTIPLIER = int(10e6)
json_string = b"[" + b", ".join([
    b'[{"x": 1.1, "y": [1]}, {"x": 2.2, "y": [1, 2]}, {"x": 3.3, "y": [1, 2, 3]}],' +
    b'[],' +
    b'[{"x": 4.4, "y": [1, 2, 3, 4]}, {"x": 5.5, "y": [1, 2, 3, 4, 5]}]'
] * MULTIPLIER) + b"]"

we have

70.0 seconds in Python's json module (projected from a 1/10th dataset, since the full dataset would produce 20 GB of Python lists and dicts)
38.6 seconds with untyped ArrayBuilder—the state of the art if you have no schema
31.5 seconds with schema-specialized Forth code (trading ArrayBuilder navigation costs with Forth navigation costs)
10.4 seconds with RapidJSON + this simple language → Awkward Arrays
6.0 seconds for RapidJSON with no output

So getting just iterating over this simple language and writing out output costs 4.4 seconds, which is a little less than the time RapidJSON spends parsing it. Good enough.
For non-Awkward but JSON-schema users (who might be using Dask's "bag" API now), the value proposition is that using Awkward Array means they can load their data 7× faster, use 10× less memory, and then do fast vectorized operations on what has been loaded, rather than Python iteration.
Now I need to find out why Windows is failing.

Fixed the "Windows bug" (it was revealing a bug that was merely hidden on the other platforms) and tried a few things to get a bit more speed, such as merging identical switch cases and merging the two classes into a single class (that mattered for the Forth implementation...). No changes in speed. I'm leaving the switch cases merged (it's easier to read), but keeping the classes separate (I want to hide the RapidJSON dependency from the header files).

Another thing to try: each type of instruction should only be writing to one type of output buffer. Replace the single std::vector of abstract ForthOutputBuffets with a std::vector for each specialized subclass, which ought to avoid vtable indirection.
The output dtype, currently an argument for each assembly instruction, would have to become fixed. (Easy way to do that would be to just add assertions in the constructor. That way, the Python code can be unchanged.)
Also, replace std::shared_ptr with std::unique_ptr, as we already know that makes a difference.


ES6 style function

This patch adds a new syntactic sugar for functions:
(a) => ... is fun a => ...
(a,b) => ... is fun a b => ...
() => ... is fun () => ...
The updated testsuite shows how that looks like.
I will add a few improvements to the printer, for instance (a : int, b : int) => a + b is printed as ((a : int), (b : int)) => a + b.

Would we want to make function invocation and variant / type parameter syntax match in the same diff?

For the printer, it would better match how people write JavaScript to do:
(a: int, b: int) => a + b

rather than
(a : int, b : int) => a + b

In general though, this is a really exciting improvement in the syntax :)

The same syntax for application introduces a few problems:

simple_expr_list used by jsx in the grammar will need to be delimited (probably with comma)
while, for, if construct will likely need parentheses (to disambiguate if f(x): if f x or if (f(x)) ...).

"simple_expr_list used by jsx in the grammar will need to be delimited (probably with comma)"
Why would that be the case?

x (y) z is parsed as (x y) (z)

Just chiming in to say that I'd love to require parens when invoking function calls. I think that's going to be the biggest blocker for Reason adoption. Though I am aware it's quite an invasive change.
For example, this surprised me the other day: (Config.get "env" ^ "/item/public_token/exchange"). The string concat operator is actually lower precedence so that's equal to ((Config.get "env") ^ "/item/public_token/exchange"), which is really surprising. There are lots of little cases like that where forcing parens makes it a lot clearer: Config.get("env") ^ "/item/public_token/exchange".
Maybe that should be taken up in another PR though?

@jlongster The ^ makes sense to me, but that's arbitrary, it's just an habit. But I agree with the goal: I would like to require parens for function calls asap.

Requiring parens for function calls should probably go in the same diff/release as this PR. That would be best to avoid thrash.

Sorry for the noob question, but I checked out this branch, ran make build, and tried out ./refmt_impl.native on a file containing let x = (a) => a + 1;, and it failed w/ a syntax error. Also make test has a bunch of errors where the pretty printer isn't doing the es6 style.
Am I missing a step?

@jaredly there's also a parser directory inside the repo. Was that correctly built?

Incorporated into #1299

I suppose it's a bit late to comment on this now but one issue with (a) => ... vs fun a => ... is that the former isn't possible to highlight accurately with textmate grammars in general.
The problem is that you can't know whether (a) is a value or a pattern without looking ahead until you find a => or some terminating token like ), ], }, ;, etc.
As long as => is on the same line as the opening ( you can probably highlight it correctly with some effort but there's still the issue that the highlighting cannot be determined until the => is typed into the buffer. And if the => is not on the same line, the highlighting will just be wrong (it will show as a value).
In general, any syntax that requires looking ahead arbitrarily in order to determine the current parsing context is a problem for textmate grammars because there is no way to look beyond the end of the current line before making a choice.
This is an unfortunate limitation but I've come around to the view that it's an informative constraint as far as the language design is concerned because it also has an impact on how responsive and informative the diagnostic feedback (e.g., type inference, etc.) can be as you are editing.

how does javascript's highlighter work?

There are several different javascript highlighters implemented as textmate grammars and they all behave slightly differently. Last I checked, I believe the ones from the textmate repos, atom, sublime, and vscode are all maintained separately and give different results. And then there are several 3rd party ones available on the vscode extension marketplace and probably elsewhere too.
But none of the javascript highlighters I've ever seen have tried to be accurate down to the point where they are parsing the actual syntax like the reason one does. Mostly they just use approximate highlighting and tend to fail on certain corner cases, usually involving multiline syntax fragments. Of course the reason highlighter still fails in some cases too but that's due to bugs in the rules rather than through a conscious choice to parse approximately to begin with.
Javascript also doesn't have as many clear cut syntactic categories as reason does, so in their case this ambiguity that arises between patterns and values isn't really an issue to begin with. That's why there's this sort of loss of fidelity in removing the fun.
I suppose it may be an acceptable tradeoff to just try and do more approximate parsing in the reason highligher and give up on accuracy. But to do that, some of the stuff that is currently distinguished now will probably need to be simplified in order to appear consistent. That may have other consequences though. For instance, types can occur in patterns too…
One thing I've found is that once you start to make the highlighting parser more approximate, it generally becomes more difficult to maintain because it's harder to keep track of which cases need to be fudged in order to get good results. There's no way I know of to do this without a lot of hand tuning.


First pass at external tooling

This is tooling extracted from https://github.com/bountonw/translate and now used in https://github.com/bountonw/private-translate/pull/2 and https://github.com/bountonw/translate/pull/247.

@bountonw This is ready for review whenever you get a chance.


Windows Calico Felix leak Handles and exhaust Host's memory, Change NSSM to latest or alternatives

On Windows Node, Calico felix liveness/readiness checks create zombie processes and leave process handles that steadily consume host's memory.
Change to NSSM 2.25 or some other service manager such as Shawl (https://github.com/mtkennerly/shawl)
NSSM 2.24 has Handle leak known bug (https://nssm.cc/bugs)

Version 2.24: nssm leaks thread handles when the aplication is restarted. This bug is fixed in 2.25 pre-release builds.

Expected Behavior

Memory usage stable

Current Behavior

Memory usage by NSSM, or containerd-shim-runhcs-v1.exe (HostProcess installation) gets high (10GiB in a week)
Handle count of NSSM or containerd-shim-runhcs-v1.exe gets high (above 10k handles)
Using RAMMap (https://learn.microsoft.com/sysinternals/downloads/rammap), calico-node zombie processes can be found using 32kb (or similar) ram usage

Possible Solution

Use NSSM 2.25
Use Shawl as alternative

Steps to Reproduce (for bugs)

Install Calico using the script provided by calico (Either on the node or by hostprocess)

Context

I have tested Shawl and NSSM 2.25 and memory usage is very stable
No Handle leaks happen in either NSSM 2.25 or Shawl

Your Environment

Calico version 3.23.1
Orchestrator version (e.g. kubernetes, mesos, rkt): K8s 1.22.10
Operating System and version: CentOS 7.x / Windows Server 2019 17763.3113

First, I thought this was related to containerd, and opened a issue there(https://github.com/containerd/containerd/issues/7375), but found out that calico felix creating process and quickly killing the process can lead to zombie process

@narsimplyme Thanks for the details. I think we should upgrade to NSSM 2.25.

nssm 2.24-103 build might have fixed this issue too. I applied it and seems no leak.


Cannot start since version 22.3.4_microsoftstore (Wrong JRE PATH)

Description
Obviously, this folder shouldn't be here

DBeaver Version
Community Edition 22.3.4
Operating System
Windows 11 Pro
Database and driver
No response
Steps to reproduce

Upgrade to dbeaver 22.3.4 from Microsoft Store
Run dbeaver

Additional context
No response

Have you tried reinstalling DBeaver?

It is a bug in the latest release. We will fix it shortly

@serge-rider Are there any workarounds until a new release is published to the Microsoft Store?

New release should be available tomorrow.
The workaround is to install java on your machine or to download installer from dbeaver.io.

Thanks @serge-rider. Curiously, I do have java installed on my machine, specifically v17. Is there a specific set of versions that is required to run DBeaver?

On my machine, I can see the file specified in the error, but its in \jre\jre\bin\javaw.exe instead of \jre\bin\javaw.exe.
Tried moving the folders, but unable to figure out permissions.
Work is halted, need that update soon! Don't want to reinstall as I will loose all DB connections.

atm. there's no release with the fix on the microsoft store. only way that i progressed was instainlling the JDK linked in this github readme.md
https://adoptium.net/
the oracle one is "old".. after installing the openJDK from adoptium dbeaver is working again.

Should there be a sub-milestone for this? Hope we won't have to wait till the 3.5 release ;)

@carlos-garcia Can you elaborate a bit? Are you saying you were able to run the Windows Store version of DBeaver simply after installing the latest version of OpenJDK? Did you have any JDK installed previously? Did the installation of the JDK add something to your PATH or update your JAVA_HOME environment variables to get DBeaver to resolve a JVM?

On my machine, I can see the file specified in the error, but its in \jre\jre\bin\javaw.exe instead of \jre\bin\javaw.exe. Tried moving the folders, but unable to figure out permissions. Work is halted, need that update soon! Don't want to reinstall as I will loose all DB connections.

You can use regular Windows installer version and import data from the Microsoft Store version folder.

@carlos-garcia Thanks for the quick response and screenshot. Did the installation also create or set a JAVA_HOME environment variable? Curious why/how this workaround works for you. I already had a JDK installed on my machine and even after installing the OpenJDK version I am still unable to run DBeaver.

@miguellira my first straight-thing-to-do was downloading java from oracle.. but they just offer the 1.8 (which triggers the linked issue of requiring java 11) thats why after uninstalling the 1.8 from oracle, found the one in the readme.
i dont see any JAVA_HOME path on my env settings

@carlos-garcia Thanks again for the quick response and posting a screenshot of all your environment variable keys.
Not that it should matter but I'm assuming the previous screenshot of the value of your path environment variable was at the system level - correct?
I've tried mirroring your setup and am unable to launch DBeaver. Really hope that new version gets pushed to the Microsoft Store soon.
@ShadelessFox I know you closed a duplicate issue and stated the workarounds are to download and install the non-Windows Store version or install JRE 11 or greater.
Any idea why installing the Adoptium OpenJDK would work for @carlos-garcia and not for others?

@carlos-garcia What is the output of java --version on your machine? Is it similar to this:

weird.. got this..

@carlos-garcia If you're definitely running the Microsoft Store version of DBeaver and not the download version then you sir are one lucky person. 😄
Thanks again for sharing your setup.

Unfortunately we still cannot update the latest version.
Microsoft store is a tricky platform, updates are very slow and fail with weird diagnosis. And there is no tech support.
If the problem will persist we will have to disable app on MS store for some time (until it will be resolved).

@serge-rider That is unfortunate but understandable. Apparently, the install "JRE 11 or greater" workaround worked for at least one individual but doesn't work for myself and my peers. Any thoughts as to way the inconsistency? Shouldn't the presence of a JDK be enough to get it to run?

@carlos-garcia Still trying to figure out why your Microsoft Store install of DBeaver works and mine does not.
Can you do me a favor and look in your registry under Computer\HKEY_LOCAL_MACHINE\SOFTWARE\JavaSoft and see if you have an entry for Java Runtime Environment and if so, what is the latest key and its values?
This is what mine looks like; I'm guessing yours is different:

@serge-rider would it be possible for you to share the .appx from the 22.3.3 release?

@miguellira
I created a new registry entry \HKEY_LOCAL_MACHINE\SOFTWARE\JavaSoft\Java Runtime Environment\17
And by switching the current version in the Java Runtime Environment, I was able to open dbeaver

You can use regular Windows installer version and import data from the Microsoft Store version folder

Trying to help a few colleagues with that... But couldn't figure out where the Microsoft Store version folder is. Any pointers?
Thanks a lot, in advance.

You can use regular Windows installer version and import data from the Microsoft Store version folder

Trying to help a few colleagues with that... But couldn't figure out where the Microsoft Store version folder is (not mentioned in https://github.com/dbeaver/dbeaver/wiki/Workspace-Location). Any pointers?
Thanks a lot, in advance.

Replying to myself in the hopes that it might be helpful to others... I found the Microsoft Store folder holding DBeaver settings at
%USERPROFILE%\AppData\Local\Packages\DBeaverCorp.DBeaverCE_*\LocalCache\Roaming\DBeaverData
Copied the workspace6 directory from there into the settings location for the just-installed downloadable version and everything seems to work (connections, scripts, etc).

I installed this JDK https://learn.microsoft.com/en-us/java/openjdk/download
DBeaver from MS Markeplace is opening now.

@zhao-wuyan thank you so much for this.
For anyone who is unable to run after installing a JDK (many of us already had a JDK installed). You may have had a previous installation of Oracle Java and its registry setting is pointing to v1.8 which is causing DBeaver to fail to launch.
As @zhao-wuyan pointed out in his comment above, the fix is to create a new registry key with the version of the JDK you have installed and point the CurrentVersion to that location.
Here is my working setup using Amazon's Corretto's JDK:

@zhao-wuyan thank you so much for this.
For anyone who is unable to run after installing a JDK (many of us already had a JDK installed). You may have had a previous installation of Oracle Java and its registry setting is pointing to v1.8 which is causing DBeaver to fail to launch.
As @zhao-wuyan pointed out in his comment above, the fix is to create a new registry key with the version of the JDK you have installed and point the CurrentVersion to that location.
Here is my working setup using Amazon's Corretto's JDK:

Thank you @miguellira, this worked for me.

Installing JRE from adoptium did the trick for me

Visit https://adoptium.net/
Select Other Platforms and Versions
Use filters: OS: Windows, Architecture: x64, Package Type:JRE
Click on MSI button to download JRE
When installing, make sure all options are selected, this is a must

No support for Microsoft Store in the future?

 No support for Microsoft Store in the future?

This is temporarily I hope?

@twielens I sure hope so. But if you read the comment @serge-rider made early on in this issue, if it does come back to Microsoft Store it will be awhile.
The registry workaround worked for me and some of my colleagues. Good luck!

@twielens I sure hope so. But if you read the comment @serge-rider made early on in this issue, if it does come back to Microsoft Store it will be awhile.
The registry workaround worked for me and some of my colleagues. Good luck!

Yeah, I have a managed laptop, so I can only install software provided by the package manager. So installing a different JDK/JRE is not an option.

@serge-rider: Any clue on if DBeaver will be available in the store in the coming weeks? Is there any progress on that?

I installed this JDK https://learn.microsoft.com/en-us/java/openjdk/download DBeaver from MS Markeplace is opening now.

This Works!!
Tried it before doing any change to your regedit

I noticed the installer had a checked box to 'Include Java' as a component in the installation. I unchecked this since I already had JDK 17 installed. The application launched as it should.

I reckon the version the installer has, which would only be used by the app, and the version already installed on my machine, to be used by any java applet, were with odds with each other. This makes sense since the app can only use one JDK at a time.
So if you already have java jdk and jre installed, uncheck this option. It is not for you. The checkbox is there for workstations that are not already running java.

Unfortunately, after three weeks this bug hasn't been fixed, but fortunately, this works, thanks!

Eventually we have updated Windows Store version (although certification passed with warnings).
It should work now.

Eventually we have updated Windows Store version (although certification passed with warnings).
It should work now. No need to install Java or anything.

Yes! Thanks a lot! Don't have to rely on a dodgy run 7.3.0 version anymore!


Ensure that children of multicurated pages get properly multicurated as well

This performs a BFS traversal of multicurated pages to properly add references
to them in all the copied parents.
rdar://94798570
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[X] Added tests
[X] Ran the ./bin/test script and it succeeded
[X] Updated documentation if necessary

@swift-ci please test

@swift-ci please test


Generate insert statement without deprecation notices

We're trying to make the POLO gem work with Rails 5. Polo is a gem that generates INSERT statements for the records that already exist in the database.
After a cursory look through the codepaths, I can't figure out how to avoid the following deprecation notice:
DEPRECATION WARNING: Passing a column to `quote` has been deprecated. It is only used for type casting, which should be handled elsewhere. See https://github.com/rails/arel/commit/6160bfbda1d1781c3b08a33ec4955f170e95be11 for more information. (called from raw_sql at /Users/david/poetry/polo/lib/polo/sql_translator.rb:53)
Arel performing automatic type casting is deprecated, and will be removed in Arel 8.0. If you are seeing this, it is because you are manually passing a value to an Arel predicate, and the `Arel::Table` object was constructed manually. The easiest way to remove this warning is to use an `Arel::Table` object returned from calling `arel_table` on an ActiveRecord::Base subclass.

If you're certain the value is already of the right type, change `attribute.eq(value)` to `attribute.eq(Arel::Nodes::Quoted.new(value))` (you will be able to remove that in Arel 8.0, it is only required to silence this deprecation warning).

You can also silence this warning globally by setting `$arel_silence_type_casting_deprecation` to `true`. (Do NOT do this if you are a library author)

If you are passing user input to a predicate, you must either give an appropriate type caster object to the `Arel::Table`, or manually cast the value before passing it to Arel.

After looking through how Rails does it, and some other documentation, I've come up with the following few lines that should generate the INSERT statement, however the deprecation notice still happens (at the last line):
# `record` is an AR model instance
values = record.send(:arel_attributes_with_values_for_create, record.attribute_names)

values.each do |attribute, value|
  values[attribute] = record.class.arel_table.type_cast_for_database(attribute.name, value)
  # Also tried:
  # values[attribute] = record.class.connection.quote(record.class.arel_table.type_cast_for_database(attribute.name, value))
  # values[attribute] = Arel::Nodes::Quoted.new(record.class.arel_table.type_cast_for_database(attribute.name, value)))
end

model = record.class
scope = model.unscoped

im = scope.arel.create_insert
im.into model.arel_table

im.insert values
im.to_sql

It seems from this line that it's actually impossible not to call quote without a column, unless the value is an SqlLiteral. The deprecation notice talks about a Quoted but I'm not sure that's relevant for our usecase (I tried wrapping each value in a Quoted, then we got the error "can't quote quoted").
Is it possible to avoid the deprecation notice when generating an INSERT statement using Arel? I must be overlooking something but can't figure it out. Any help would be appreciated.

After rubberducking this here, I figured I should re-research the Rails 5 codepath. I first thought the subs/bindings are used for prepared statements but after further investigation that's not the case. I came up with the following which seems to do the trick:
      values = record.send(:arel_attributes_with_values_for_create, record.attribute_names)

      model = record.class
      scope = model.unscoped

      im = scope.arel.create_insert
      im.into model.arel_table

      substitutes, binds = scope.substitute_values(values)
      im.insert substitutes

      record.class.connection.to_sql(im, binds)

For posterity's sake: It actually does have something to do with prepared statements (doh), and can be fixed by surrounding the last statement with model.connection.unprepared_statement.


License?

Hi @dgobbi , what license are we using on the code? Is it GPL/MIT/etc? Or a custom one? I read the LICENSE file but it didn't say anything about basing it off an existing license.
Just wondering in case we want to include GNU tools for future improvements (say getopts, as an example).

It's the BSD 3-clause license (http://opensource.org/licenses/BSD-3-Clause).  No GNU stuff, I don't want to switch to copyleft.

Sounds good!


Set result to Query rather than last selected option

Using accessibleAutocomplete.enhanceSelectElement on a select element. If I choose a value from the dropdown eg Honda, then delete the field and enter something not an option eg blah blah from the select element. The value sent on submission is the last selected value (honda) but I want blah blah to be sent

Thanks for raising this @brendyhenry90. It looks like it's been reported previously with https://github.com/alphagov/accessible-autocomplete/issues/205 so I'm closing this issue.


Change index for "push" exercise to match with master README

@kayuweschmidt : can you please merge this? Also, we figured out some discrepancies in the steps. Will raise another pull request for fixing those

@eeny-meeny


[DevEx]: Update linting config to limit html to 3 attributes in a single line

Improvement Summary
We should make the line-lengths of html content more consistent by enforcing lint rules regarding the number of attributes per line and where they should go.
Detailed Description
The convention we've roughly followed for formatting html is as follows:

All attributes can go on a single line if there are <= 3 attributes
Otherwise it should go on multiple lines, with one line per attribute

Example:
  <!-- ✓ GOOD -->
  <MyComponent lorem="1"/>
  <MyComponent lorem="1" ipsum="2"/>
  <MyComponent
    lorem="1"
    ipsum="2"
    something="3"
    else="4"
  />

  <!-- ✗ BAD -->
  <MyComponent lorem="1"
               ipsum="2"/>
  <MyComponent lorem="1" ipsum="2" something="3" else="4" />

To configure this app-wide in an auto-fixable manner, we should use the vue eslint rules max-attributes-per-line and first-attribute-line-break.
We should set the max-attributes-per-line to singleline: 3 and multiline: 1 so that we allow up to 3 attributes when the entire component is in one-line and we only allow one attribute per line when breaking into multiple lines.
Then we should confvigure first-attribute-line-break to singleline: "beside" and multiline: "below" so that when all the attributes fit on a single line, they go next to the tag, and when they fit on multiple lines, they each go below.
These two rules together should enforce the style we've been using informally but in a way that will apply consistently across the entire app. In addition to configuring the rules as described, we'll need to format the code with npm run lint:fix to fix any violations of these rules app-wide.

Clarification Needed:
These two rules, will also affect other things, like every line that has more than three attributes will have all attributes split up, in some cases this might not be the effect that you wanted. Do you still want to proceed with this?

Clarification Needed:
These two rules, will also affect other things, like every line that has more than three attributes will have all attributes split up, in some cases this might not be the effect that you wanted. Do you still want to proceed with this?

Yes the intent is to require that a single line of html can have no more than 3 attributes, and tags with 4+ attributes be split into one line per attribute, so this
<!-- ✗ BAD -->
<MyComponent lorem="1" ipsum="2" something="3" else="4" />

becomes this:
  <!-- ✓ GOOD -->
  <MyComponent
    lorem="1"
    ipsum="2"
    something="3"
    else="4"
  />

Whereas for tags with 3 or fewer attributes we'd allow them in a single line like so:
<MyComponent lorem="1" ipsum="2"/>

We have roughly used this style already but it hasn't been automatically (and therefore consistently) enforced.
Does that answer your question? Is there another side effect of this change beyond what I've mentioned here that you think may be undesirable?

Yes, Refer to PR #416

Clarification Needed:
These two rules, will also affect other things, like every line that has more than three attributes will have all attributes split up, in some cases this might not be the effect that you wanted. Do you still want to proceed with this?

Yes the intent is to require that a single line of html can have no more than 3 attributes, and tags with 4+ attributes be split into one line per attribute, so this
<!-- ✗ BAD -->
<MyComponent lorem="1" ipsum="2" something="3" else="4" />

becomes this:
  <!-- ✓ GOOD -->
  <MyComponent
    lorem="1"
    ipsum="2"
    something="3"
    else="4"
  />

Whereas for tags with 3 or fewer attributes we'd allow them in a single line like so:
<MyComponent lorem="1" ipsum="2"/>

We have roughly used this style already but it hasn't been automatically (and therefore consistently) enforced.
Does that answer your question? Is there another side effect of this change beyond what I've mentioned here that you think may be undesirable?

Refer PR #416 for changes, unfortunately, my device will not support, electron, need someone to take it and actually check the effect from here, making sure all the test cases are passing.

they can run the Lint fix in my fork before making changes.

they can run the Lint fix in my fork before making changes.

The tests will be run in CI so that will confirm that there are no regressions. You can run npm run lint:fix to auto fix the linting errors. That shouldn't require electron, and then the cypress tests will run in GitHub actions after you push


Session 3: About teaching & cool things we all would like to share

Issue to discuss the full session 3. Please use new issues for single episode discussions and assignments to indicate that you are working on something.
Preliminary learning objectives:

Get to know the teaching philosophies of CodeRefinery instructors
Get to know the principle of co-teaching: How we do it and how you can too
Share your own favorite tools or teaching gem
...?

Initial schedule suggestion:

30 min: computational thinking
30 min: co-teaching (moved here from Session 2)
45 min: teaching philosophies
45 min: sharing teaching tips, tricks, tools etc by all

Instructor interests and materials documents

co-teaching.md: @dianaiusan , @blindij , @rkdarst , @stepas-toliautas
teaching-philosophies.md: @blindij , @rantahar , @HaoZeke , @stepas-toliautas
computational-thinking.md: @linguist89 , @wikfeldt

Host for this session: @samumantha
Coordinator for this session: @samumantha

Schedule request by @linguist89 : Computational thinking first thing in the morning.

Please add timing overview to https://coderefinery.github.io/train-the-trainer/guide/.

Suggestion added : https://github.com/coderefinery/train-the-trainer/pull/123

I added 5 mins outro to https://coderefinery.github.io/train-the-trainer/guide/ but now it does not add up. I did not want to just shorten other episodes but some will need to be shortened.

Thanks, will look into it!

Hei @stepas-toliautas, what does it take to finalize the co-teaching episode?
Would you take the lead on this episode?
And would you like to co-teach the teaching philosophies episode?

Hei @stepas-toliautas, what does it take to finalize the co-teaching episode? Would you take the lead on this episode? And would you like to co-teach the teaching philosophies episode?

What it takes -- mostly time to sit down for it :) most likely by Friday, might be today/ tomorrow as well.
Also, it definitely needs an experienced CR member to co-teach about co-teaching -- any volunteers?
For teaching philosophies, I'm very much OK to be the co-pilot, yes.

Sorry for the double posting.
Thanks!
I'll find you a co-teacher for the co-teaching episode :)

@linguist89 @blindij @stepas-toliautas
Is there anything that learners need to prepare/install/think about/ know to follow or understand your session on Day 3 that we should mention in the email going out tomorrow? Current email draft: https://github.com/coderefinery/train-the-trainer/issues/100#issuecomment-2302423742

I'll have to send this email to participants now as is. However, if something still comes up that would be good that participants know about beforehand, please let me know asap and we can send an additional email.

Thanks everyone for your work and presentations!
Please add your own feedback (wrt to teaching, organization, materials etc): https://github.com/coderefinery/train-the-trainer/issues/149


Using formly-skip-ng-model-attrs-manipulator to remove name attribute prevents formControl object from being populated

JSBin Link
http://jsbin.com/naqaxo/edit?js,output
Failing Test: formly-field.test.js
describe(`formly-skip-ng-model-attrs-manipulator`, () => {
  it(`should provide formControl when using formly-skip-ng-model-attrs-manipulator and no name is specified`, () => {
    scope.fields[0].template = `<input ng-model="model[options.key]" formly-skip-ng-model-attrs-manipulator />`;
    compileAndDigest();
    expect(field.formControl).to.exist;
  });
});

Details
I am using formly for a credit card form that will be processed by a third party provider(Stripe.js or Recurly.js) both of these payment providers advise against using a name attribute on sensitive fields in case the form is accidentally submitted to the server(from error in JS failure for JS to preventDefault() or otherwise) as it prevents those sensitive fields from being posted to the server with the rest of the fields, getting stored in logs and voiding certain levels of PCI compliance.
I've gotten around this by using a custom field template that uses the formly-skip-ng-model-attrs-manipulator attribute and has no name attribute. The issue is that when I do this the options.formControl object seems to be dependent on the name attribute and is therefore not populated which means I don't get access to the validation helpers like $valid, $pristine.
Interested to hear your thoughts on if this is something you think formly should be able to handle.

This isn't an angular-formly issue, it's actually an angular issue. The problem is that without the name attribute, there's absolutely no way for me to get access to the ng-model controller. One thing you might consider is creating a directive which requires the ngModel controller and then attaches it to scope.options.
Give me a second and I think I could give you a quick example.

Here you go! http://jsbin.com/mebiju/edit?html,js,output
Because this isn't something that we'll be implementing in the core of angular-formly, I'll close this issue. However, if you like to make a plugin for formly, that'd be way cool!

It completely makes sense that it's an angular thing now that you mention it. Thanks for the JSBin, very helpful. I'll have a look at turning it into a plugin as I'll be neck deep in formly stuff at work for the next week or 2 i think.

Awesome! If you want a good kick-start on building an open source library, I recommend you checkout my egghead.io series :-)

Oh I had actually just sent that link out to my team at work a couple of days ago, I didn't connect the dots that it was you who made them! I'll definitely be watching them, thanks for all your hard work!

:blush: yay!


Add support for Windows

It appears that only Linux and Mac are supported today

Hey @BradleyCulley,
We're archiving this repo as Concourse-Up turned into Control Tower.
We're unlikely to ever support the binary working on Windows, as one of the tools that we shell out to (the BOSH CLI) does not support Windows, even with MinGW installed.


feat(*): generate components from svg source

 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.


ConditionalCreate accidentally making contained resources.

When using a conditionalCreate with an ifNoneExist, if the resource already exists, and a bundle contains a reference to the existing resource described by a URN, the target resource will become contained in the referring resource, instead of the search resolving, and substituting the reference with the found PID from the database.
Exanple:

Create patient

{
"resourceType": "Patient",
"identifier": [
	{
		"system": "https://example.com/fhir/memberuniqueidentifier",
		"value": "123"
	}
]
}

Create a bundle with 2 entries:

One, a conditional POST on that same patient, with a full URL described by a URN.
Next, a conditional PUT on another resource, (e.g. Task), with a reference to the patient, defined by the fullUrl URN.

{
	"resourceType": "Bundle",
	"type": "transaction",
	"entry": [
		{
			"fullUrl": "urn:uuid:b348d0ca-337e-4554-8189-4d89386d67d3",
			"resource": {
				"resourceType": "Task",
				"language": "en-US",
				"identifier": [
					{
						"system": "https://example.com/fhir/taskidentifier",
						"value": "10100"
					}
				],
				"basedOn": [
					{
						"reference": "urn:uuid:9ffec273-d2c6-4b39-a23e-bd31f94afe32"
					}
				]
			},
			"request": {
				"method": "PUT",
				"url": "/Task?identifier=https%3A%2F%2Fexample.com%2Ffhir%2Ftaskidentifier|10100"
			}
		},
		{
			"fullUrl": "urn:uuid:9ffec273-d2c6-4b39-a23e-bd31f94afe32",
			"resource": {
				"resourceType": "Patient",
				"identifier": [
					{
                                               "system": "https://example.com/fhir/memberuniqueidentifier",
						"value": "123"
					}
				]
			},
			"request": {
				"method": "POST",
				"url": "/Patient",
				"ifNoneExist": "Patient?identifier=https%3A%2F%example.com%2Ffhir%2Fmemberuniqueidentifier|123"
			}
		}
	]
}

Expected behavior
When the Task is created, it should refer to the original Patient created, like so:
{
        "resourceType": "Task",
        "id": "1626",
        "meta": {
          "versionId": "1",
          "lastUpdated": "2021-08-05T05:22:25.353-07:00",
          "source": "#EBtcLOmk4oSFIRvb"
        },
        "language": "en-US",
        "identifier": [
          {
            "system": "https://example.com/fhir/taskidentifier",
            "value": "10100"
          }
        ],
        "basedOn": [
          {
            "reference": "Patient/1625"
          }
        ]
      }

Instead, it becomes a contained resource in the task, like so:

        "resourceType": "Task",
        "id": "1627",
        "meta": {
          "versionId": "1",
          "lastUpdated": "2021-08-05T05:22:47.759-07:00",
          "source": "#fMMdRu1IMywTlQdK"
        },
        "language": "en-US",
        "contained": [
          {
            "resourceType": "Patient",
            "id": "1",
            "identifier": [
              {
                "system": "https://example.com/fhir/memberuniqueidentifier",
                "value": "123"
              }
            ]
          }
        ],
        "identifier": [
          {
            "system": "https://example.com/fhir/taskidentifier",
            "value": "101019"
          }
        ],
        "basedOn": [
          {
            "reference": "#1"
          }
        ]
      },

Note the `"reference": "#1"``

Closed by #2880


Computing resources: limited availability and disconnection problems

Available computing resources:

colab
uni-cluster
bwHPC

doc

Colab:

RAM and GPUs only available at night (roughly after 7pm, then unavailable in the morning again)
unexpected disconnection after approx. 6 hours of running

Also Colab:
not using gpu for some reason

https://colab.research.google.com/notebooks/gpu.ipynb

UNI Cluster tutorial

UNI Cluster tutorial

uni-cluster hardware facts


Does module version 18.31.2 support EKS 1.25

This is more of a question than an issue.  Going through the examples (../terraform-aws-eks)/examples/) that were updated just days ago, it referenced EKS 1.24.  I don't see any mention about EKS 1.25.
Does module version 18.31.2 support EKS 1.25?  knowing that some of the newer feature of 1.25 may not be manipulated using 18.31.2, can the module 18.31.2 otherwise provision a working EKS 1.25 cluster?

Yes - there are no changes required in this module to support specific Kubernetes versions


v1.1 psi(0) == -inf, while v1.0 psi(0) == inf

psi(0) could be thought of as being +inf or -inf (or consequently undefined?).  The question raised with this issue is whether scipy wants to return a consistent value across versions, or at least document changes in behavior.
So right before submitting this, I found the PR https://github.com/scipy/scipy/pull/3902.  Despite the complete discussion of the matter, I decided to create the issue anyway to suggest that perhaps the documentation should mention this change in behavior across versions?
Reproducing code example:
$ python -c "import numpy; print(numpy.__version__)"
1.14.3
$ python -c "import scipy; print(scipy.__version__)"
1.0.0
$ python -c "import scipy.special; print(scipy.special.psi(0))"
inf

$ sudo pip install scipy==1.1 --force-reinstall
$ python -c "import numpy; print(numpy.__version__)"
1.14.3
$ python -c "import scipy; print(scipy.__version__)"
1.1.0
$ python -c "import scipy.special; print(scipy.special.psi(0))"
-inf             <------------  Note minus inf, different than with scipy v1.0

Scipy/Numpy/Python version information:
$ python -c 'import sys; print(sys.version_info)'
sys.version_info(major=2, minor=7, micro=12, releaselevel='final', serial=0)

To help make psi() more consistent, or at least document the change in behavior, I explored the psi() return values for inputs around 0, -1, -2 and -3. It might be nice if the known relation  psi(z+1) = psi(z) + 1/z holds for psi()'s return values.  Psi() under v1.0 for arguments 0, -1, -2 and -3 returns a consistent +inf, the value approached from the left on the negative real axis.  Psi() under v1.1 returns nan for inputs -1, -2 and -3, and returns -inf for an input of 0, where by the formula you'd expect possibly:
psi(0) = psi(-1) - 1 = nan - 1 = nan

On scipy 1.0:
$ python
>>> import scipy.special
>>> scipy.special.psi([[0 - 1e-5, 0, 0 + 1e-5],
...                    [-1 - 1e-5, -1, -1 + 1e-5],
...                    [-2 - 1e-5, -2, -2 + 1e-5],
...                    [-3 - 1e-5, -3, -3 + 1e-5]])
array([[  99999.42276789,              inf, -100000.57719922],
       [<PHONE_NUMBER>5723,              inf,  -99999.57718967],
       [<PHONE_NUMBER>5473,              inf,  -99999.07718606],
       [<PHONE_NUMBER>8695,              inf,  -99998.74385162]])
>>>

And on scipy 1.1:
python
>>> import scipy.special
>>> scipy.special.psi([[0 - 1e-5, 0, 0 + 1e-5],
...                    [-1 - 1e-5, -1, -1 + 1e-5],
...                    [-2 - 1e-5, -2, -2 + 1e-5],
...                    [-3 - 1e-5, -3, -3 + 1e-5]])
array([[  99999.42276789,             -inf, -100000.57719922],
       [<PHONE_NUMBER>5723,              nan,  -99999.57718923],
       [<PHONE_NUMBER>5473,              nan,  -99999.07718531],
       [<PHONE_NUMBER>8695,              nan,  -99998.74385087]])
>>>

The question raised with this issue is whether scipy wants to return a consistent value across versions, or at least document changes in behavior.

I see the change in values as a bugfix in this particular case. Documenting things never hurts though; I'd happily merge a PR documenting the changes.

It might be nice if the known relation psi(z+1) = psi(z) + 1/z holds for psi()'s return values

The current implementation is based on right/left limits, so we have
>>> sc.psi(0.0)
-inf
>>> sc.psi(-0.0)
inf

We only have one representation for negative integers and the limits don't agree, so we have to return NaN. I believe this is the "standard" way of doing things.

Closing-I probably should have noted the change in the release notes, but that ship has sailed now.


How to get the percentage of CPU usage in arm Cortex-R8 system

We are using threadx in arm Cortex-R8 SMP system and want to get the percentage of CPU usage druing running timer, we port the execution_profile_kit and found it not work properly. So we want to know whether execution_profile_kit support R8 SMP? If not, any other solution to work out the problem? thanks!

We are looking into the current status of the EPK for SMP ports. We will provide more information soon.

There is an experimental version of the Execution Profile Kit (EPK) for SMP here: https://github.com/azure-rtos/threadx/tree/master/utility/execution_profile_kit/smp_version


Add support for 'terraform show'

Example usage:
Show current state:
/tf/rover/rover.sh \
  -lz /tf/caf/landingzones/caf-terraform-landingzones/caf_solution/ \
  -level level3 \
  -tfstate aks.tfstate \
  -env foo \
  -a show [-json]

Show a plan file:
/tf/rover/rover.sh \
  -lz /tf/caf/landingzones/caf-terraform-landingzones/caf_solution/ \
  -level level3 \
  -tfstate aks.tfstate \
  -env foo \
  -a show [-json] /tmp/out.plan

Implements https://github.com/aztfmod/rover/issues/181

Hi Rover team, I've had to modify scripts/tfstate_azurerm.sh:other() to not pass -state by default. Not all 'other' commands take this arg, show being one. However I'm concerned this may break certain commands falling through to the other function that may require the state file?
If someone has some of these use cases would appreciate if they could confirm this patch is OK for their use case.

Thanks @brk3 . Going to include it in the next rover release and do the integration test into the tfc branch

thanks @brk3 for your contribution to Cloud Adoption Framework!

Tested and will need more rework so rolled back @brk3 ! Lets iterate for next month release.


Bad description of breaking TLS changes

This page doesn't mention that HttpClient and OUTBOUND connections are affected too - it gives the impression that it's just Kestrel when it talks about external ssl tools.
"either specify a CipherSuitePolicy value" -> There is no documentation ANYWHERE about how to do this and it does not seem possible to do it easily per HttpClient request/instance which is a huge mess in an API scenario where we need to connect to many different customer-supplied endpoints. After upgrading to net6 many of our customers broke because HttpClient on Linux now supports different ciphers than before. Also it is NOT correct that it uses the system ones, because the openssl CLI from the same system successfully connects to these sites with TLS 1.2, whereas HttpClient throws an exception that talks about ssl3 not working.
Also, what an ultra bad decision to make HttpClient behave differently on different systems - now the exact same code works on Windows 10, but breaks on production Linux servers!!!!!!!

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

ID: e68c35c6-eb84-64cb-7d8a-563fc78408fa
Version Independent ID: 7085f54b-ef50-51cd-f074-45fa6af43e02
Content: Breaking change: Default TLS cipher suites for .NET on Linux - .NET
Content Source: docs/core/compatibility/cryptography/5.0/default-cipher-suites-for-tls-on-linux.md
Product: dotnet-fundamentals
GitHub Login: @gewarren
Microsoft Alias: gewarren

Hi @karlra
We are facing the same issue.
Were you able to resolve it?
Thanks

@MaorDavidzon  We had to change openssl.cnf, there was no way around it.

Thanks @karlra
It works :)
I wonder if there is an applicative way to do it, so I won't need to change the OpenSSL conf

There isn't, which is really bad :(

@bartonjs Can you clarify how users can specify a CipherSuitePolicy value as a workaround, and also whether outbound connections should be explicitly mentioned?

The documentation does mention that it applies to HttpClient, which is only relevant to outbound operations.  ("... or higher-level operations, such as HTTPS via the HttpClient class.")
The type name apparently has a typo in it, since it is referring to https://learn.microsoft.com/dotnet/api/system.net.security.ciphersuitespolicy; but as it specifically pertains to the https://learn.microsoft.com/dotnet/api/system.net.security.sslserverauthenticationoptions.ciphersuitespolicy or https://learn.microsoft.com/dotnet/api/system.net.security.sslclientauthenticationoptions.ciphersuitespolicy properties.
SslClientAuthenticationOptions clientOpts = new SslClientAuthenticationOptions
{
    ...
    CipherSuitesPolicy = new CipherSuitesPolicy(
        new[]
        {
            TlsCipherSuite.TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384,
            TlsCipherSuite.TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384,
        }),
};

using (SslStream sslStream = new SslStream(networkStream))
{
    sslStream.AuthenticateAsClient(clientOptions);
    ...
}

Or, for HttpClient:
SocketsHttpHandler handler = new SocketsHttpHandler
{
    SslOptions =
    {
        CipherSuitesPolicy = new CipherSuitesPolicy(
            new[]
            {
                TlsCipherSuite.TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384,
                TlsCipherSuite.TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384,
            }),
    },
};

using (HttpClient httpClient = new HttpClient(handler))
{
    ...
}


Interpreted a Shared API router for all apps to register to.

See http://stackoverflow.com/questions/20825029/registering-api-in-apps#22684199 for the idea behind this code.
I hope you find this useful. It makes it easier for apps to register to one API endpoint

@spoetnik Thank you for taking the time to create and send over the pull request. However I'm going to close this PR without merging it. Being this project is just a starting point I'd prefer to leave it up to the developer if they want to incorporate something like this into their project.
Cheers and thanks again.


Docker failed

docker works well for me before I install scoop.
I am using native docker, and my windows build number is 1511, or 10586.589.
>docker info
An error occurred trying to connect: Get http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.24/info: open //./pipe/docker_engine: The system cannot find the file specified.

even after scoop install docker.

It is fine after reinstalling docker.


[WMS 11039] Minor update on instruction

[WMS11039] and [WMS9121]
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[x ] All filenames are lower case (including folders, images, files, etc.)
[x ] Filenames are descriptive
[ x] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ x] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ x] Image references in markdown contain an alternate text

Hi @kuanrcl, please resolve the current conflicts.

Hi Hope,
I have fixed the conflict, hope the merge is ok now. I am not really a power user of GitHub, hope this is good to go.
Thanks
Ryan


dx_utilities: dx_startThreadDetached, malformed function pointer?

I've started to do some work with a unit test framework that had some problems mocking this function.  Is the function pointer here malformed?
https://github.com/Azure-Sphere-DevX/AzureSphereDevX/blob/e521092897696637f1b16905f554a3eb562f48a5/include/dx_utilities.h#L44

to clarify, should the prototype be:
bool dx_startThreadDetached(void (*daemon)(void *), void *arg, char *daemon_name);

well, it does seem that the asterix in front of daemon is optional
it appears the rest of the function pointers in AzureSphereDevX do use this optional asterix
unfortunately, my test framework does require it

@gloveboxes, can you comment on this issue/question?
Thanks,
Brian

Hmmm, looking at https://man7.org/linux/man-pages/man3/pthread_create.3.html it seems you are right 😀
I've updated the EdgeDevX version as I can test on my laptop  https://github.com/Azure-Sphere-DevX/EdgeDevX/blob/b8e443f51e437728b0a6b809a2c300cdaa53fb0c/src/dx_utilities.c#L261.
Tomorrow I'll test on a Sphere to make sure it works as expected.
Thank you!
int pthread_create(pthread_t *restrict thread,
const pthread_attr_t *restrict attr,
void *(*start_routine)(void *),
void *restrict arg);

@CraigGabel @bawilless
I've fixed the return type to be dx_startThreadDetached(void *(*daemon)(void *), void *arg, char *daemon_name);
I've tested and pushed the change.
Thanks again!
@bawilless I've also updated the AzureSphereDevX.Examples to use the latest version of AzureSphereDevX lib.
Cheers Dave

Thanks for looking into this, Dave! :smiley:


Added TID for execsnoop & open

Hey, i'm from the kubescape team.
We would like to have TID as an additional enrichment to those events.
Thanks!

@afek854 before kicking the CI, would you mind squashing all commits into one please?

@afek854 I think you're missing a make ebpf-objects invokation: https://github.com/inspektor-gadget/inspektor-gadget/actions/runs/11707111312/job/32608767572?pr=3658

It seems it's missing an update to the unit tests too: https://github.com/inspektor-gadget/inspektor-gadget/blob/main/pkg/gadgets/trace/exec/tracer/tracer.go

It seems it's missing an update to the unit tests too: https://github.com/inspektor-gadget/inspektor-gadget/blob/main/pkg/gadgets/trace/exec/tracer/tracer.go

I updated the tests, but I noticed that some unittests fails on things I did not modify.
Can you be more specific please?

I updated the tests, but I noticed that some unittests fails on things I did not modify. Can you be more specific please?

I'm not sure why but the TestExecTracer/event_has_correct_paths unit test seems to be broken after this PR.
https://github.com/inspektor-gadget/inspektor-gadget/actions/runs/11763968090/job/32779521206?pr=3658

I updated the tests, but I noticed that some unittests fails on things I did not modify. Can you be more specific please?

I'm not sure why but the TestExecTracer/event_has_correct_paths unit test seems to be broken after this PR.
https://github.com/inspektor-gadget/inspektor-gadget/actions/runs/11763968090/job/32779521206?pr=3658

I fixed it, my bad


Unable to get the status of camera or microphone

I tried using this package on Windows 10 Enterprise version 1709. I am able to see all other sensor statuses,  except for the camera and microphone. Is there an issue with this window build or missing something?

Hi!
There is an issue (#23) with using special characters, like spaces, in a sensor name. Could that be the case for you?

I have attached the config screen information. I am not using space for names.

Thanks for the screenshot. Nothing wrong with the names like you said. Have you tried restarting the service? You can do this by killing it through Task Manager and restarting it from the start menu.

thanks, after killing from task manager

once it is started

do i need to check any registry or privacy, if this blocked by the system?

Could you maybe send me your log files? You can find them at "%localappdata%\Hass Workstation Service\logs".

please find the attached, i feel there should some setting in my laptop
that is preventing this from happening
On Tue, Feb 9, 2021 at 2:47 PM sleevezipper<EMAIL_ADDRESS>wrote:

Could you maybe send me your log files? You can find them at "%localappdata%\Hass
Workstation Service\logs".
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sleevezipper/hass-workstation-service/issues/38#issuecomment-776265395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOTIFJCATXXKTKMV7AYYB6LS6GUOJANCNFSM4XIHNWIA
.

 The registry keys the code is looking for don't exist in older versions

I feel my company laptop firewall blocking the access. Today I installed it on my old desktop it works as expected, this is not an issue with your software, but it can be information for others to check before the report this issue.

Thanks for the logs! I should make those messages a bit more descriptive. It doesn't seem like your firewall is blocking access. If the app is not connected to a broker, the sensors won't run so you wouldn't have those sensor failures in your logs.
It seems @ajobbins is correct. Your version of Windows 10 (1709) has no support for the privacy settings used by the app. It's possible that newer versions of Windows 10 Enterprise will work although I haven't tested that. I'd also recommend you to update to a newer version of Windows 10 because servicing has ended for version 1709.
I'll close this issue for now. If you still experience problems after updating, please create a new one!

My work laptop is (unfortunately) running W10 1809 (Which is still the latest LTSC release) and does not have these registry keys, so I am confident to use these sensors you will need 1903 or later.
@arungpillai ,as @sleevezipper says, 1709 is out of maintenance so you should be asking your IT department to upgrade to a supported version as you won't be getting any further security patches. Unfortunately for me, I'm likely stuck on 1809 for the foreseeable. There is no news on a new LTSC release and with 1809 in support for 10 years from release, my IT department won't be in a hurry to roll the next LTSC release until a while after it's released.
Kudos to you @sleevezipper for creating this app and finding/utilizing these reg keys - it's a great concept, just unfortunately some of us can't take advantage of it right now :(


Update readme to new style

Change the readme to make it easier to read and more pleasing

Small request for the updated README would be to include an image. When I look at this repo on HACS I can't really get a feel for what adding it offers me visually so it's hard to know if I want it or not.

Small request for the updated README would be to include an image. When I look at this repo on HACS I can't really get a feel for what adding it offers me visually so it's hard to know if I want it or not.

@funkymonkeymonk yes I plan on adding this into the new README

@funkymonkeymonk yes I plan on adding this into the new README


Increase the size of the item list editor popup

This makes it easier to edit large amounts of items.
This partially addresses https://github.com/godotengine/godot-proposals/issues/368.
Preview

Thanks!


Fn invocation after ticks removal

I have a time scale axis with ticks.. the domain changes on several actions, and I need to run some code after old ticks are removed. I can't seem to find a way to do accomplish that... is there something i'm missing or is this not supported?
Thanks

When you selection.call(axis), the code runs immediately, modifying the DOM. So if you want to run some code after updating the axis, you simply run your code after calling the axis. (If you use a transition, then you can use the transition end event.)

Thank you for your clarification


Not receiving email alerts for Azure container Registry from Azure Security center

Hi Team,
I have configured email notification in Azure security center . Security center is showing vulnerabilities for images in Azure container registry but we are not getting any email notifications for  the same.
Kindly advise

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

ID: 0a455a24-920e-f9f8-7a6f-60b5f4019acd
Version Independent ID: e6a4d8e2-0ada-e535-810f-5eb19e45ee1e
Content: Provide security contact details in Azure Security Center
Content Source: articles/security-center/security-center-provide-security-contact-details.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

@neetima09 Thank you for your query. We will investigate and update the thread.

@neetima09 Thanks for reaching out. Since you mentioned that you have configured the email alerts for Azure security center, I am assuming you can see emails for alerts for other Azure Services and not just for Containers. Please confirm if this is the case.
Also, it would be helpful to know what kind of vulnerabilities are being shown for containers (Low, medium, High) as you have option to show only the level you want and above that using this setting :

Also note that For each subscription the Azure Security center sends :
a maximum of four emails per day for high-severity alerts
a maximum of two emails per day for medium-severity alerts
a maximum of one email per day for low-severity alerts
If you still think the issue is only with Containers alerts, do let us know

Hi Team,
We have enabled security center for container registry only in pricing and
settings section.
After enabling, we have configured notification email on medium severity.
In container registry 72 images are scanned and images status in security
center is showing of high and medium severity.
Inspite of high and medium status for container registry in recommendation
section of security center, we are not getting any emails.
Kindly advise.
Regards,
Neetima
On Wed, 16 Sep, 2020, 1:34 pm VipulSparshMSFT<EMAIL_ADDRESS>wrote:

@neetima09 https://github.com/neetima09 Thanks for reaching out. Since
you mentioned that you have configured the email alerts for Azure security
center, I am assuming you can see emails for alerts for other Azure
Services and not just for Containers. Please confirm if this is the case.
Also, it would be helpful to know what kind of vulnerabilities are being
shown for containers (Low, medium, High) as you have option to show only
the level you want and above that using this setting :
[image: image]
https://user-images.githubusercontent.com/63580915/93309357-0d5c8d80-f821-11ea-94ab-722de402379d.png
Also note that For each subscription the Azure Security center sends :
a maximum of four emails per day for high-severity alerts
a maximum of two emails per day for medium-severity alerts
a maximum of one email per day for low-severity alerts
If you still think the issue is only with Containers alerts, do let us know
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/62573#issuecomment-693244962,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJZEB3SJMV2CRTT2O7KDPMLSGBWRXANCNFSM4RLLGCAQ
.

@neetima09  If that is the case I would request you to open a support ticket on this to investigate further as this might need more deeper investigation path. We will be closing this now, feel free to tag me in your comment if you have any questions.


Add showDialog api

Describe your changes
Screenshots [Optional]
Issue ticket number and link
Closes #380
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] I have added tests
[ ] I have added a changeset pnpm changeset add
[ ] I have added example usage in the kitchen sink app

Closing in favour of 387


fromUrl Renders odd pictures

Trying to use fromUrl from avataaars.io and this is how this: https://avataaars.io/?accessoriesType=Prescription02&avatarStyle=Circle&clotheColor=Blue03&clotheType=ShirtCrewNeck&eyeType=Dizzy&eyebrowType=UpDown&facialHairColor=Blonde&facialHairType=MoustacheMagnum&hairColor=BrownDark&hatColor=PastelRed&mouthType=Smile&skinColor=Brown&topType=ShortHairTheCaesar
looks like on my side

Any idea what's going on?
I am doing the following

<SVGImage [src]="avatar" height="256"></SVGImage>

import { fromUrl } from '@sergeymell/nativescript-svg';

const avatarUrl = generator.generateRandomAvatar(this.journal.ref_user_id);
console.log(avatarUrl);
this.avatar = fromUrl(avatarUrl);

Yeah, I've noticed that some images render broken. I don't know why this happens. Usually, our designers fix the files in this case. I will show them your file on Monday and I think they can help me to figure out what is wrong

@SergeyMell that's awesome, much appreciated :) haven't had the chance to try the original plugin that works only til {N} 6, cause i only been running 7.. :)

Hi @mm-psymbl, I've discussed with our designers. The problem actually happens because your SVG uses masks. Unfortunately for now SVGKit which is used for ios as a native library in this plugin has problems with rendering SVG with masks. Here you can find an appropriate issue.
As a possible solution, you can use SVGs with clip-path not masks (actually, these are two possible ways for SVG clipping). Our designers know about it and create images only this way.
Unfortunately, since this issue comes from the native library, for now, there is nothing to do on the plugin level. So I'll close the issue for now. However, you can reopen it if you disagree.


docs: add details about dev env setup

Proposed changes
Add more clarity to the Contributing docs about the development environment to allow contributors to get started more easily
Types of changes
What types of changes does your code introduce to Scuffle?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ x ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ x ] I have read the CONTRIBUTING doc
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ x ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, you may want to start the discussion by explaining why you chose the solution you did, what alternatives you considered, etc.

 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.


Create a Korok Seed guide depending on players lifelines

This feature helps the player to get the hint about korok seed locations when the lives are less

The feature has been added to the main branch


Reboot with TDD to get this up to speed

This contains structural improvements and tests based upon changes by @FND. Follow up after this PR:

#18: Expose a report function as a helper in faucet-pipeline-core and use it here instead of the private method.
#17: Remove the acorn dependency when ESLint releases a fix for eslint/eslint#11018
#19: Better errors
#11: Add a server/live server

@FND Navigation is now implemented, feedback welcome.

Thanks for your work and your feedback, @FND


Allow the namespace path to be container ID

In numerous cases, we need to share the container's namespace with
an existing container. In these cases, we have to resolve the namespace
file step by step. For example, the docker resolves it from the
information of the target container in several steps and saves it to the
config (now it is in-memory config, it will be runtime.json in future).
We can move the resolving operation to the oci implementations
and allow the namespace path to be container ID in the spec.
Reasons:

it really repsents "sharing the namespace with existing container".
(it is the most important reason.)
the administrator writes the runtime.json much conveniently and
less error. And it is better readability and the administrator
is easier to check the runtime.json which is written by himself
or by other human or by other tools.
we may have more orchestra tools than oci implementations,
the resolving operation is better in implementations than
in the tools.

Signed-off-by: Lai Jiangshan<EMAIL_ADDRESS>

I really don't want to have magic resolution in the runtime.json like this. If we consider this feature we should have the runtime resolve the naming.

This sounds useful, but I'd turn it around.  Instead of making it a
lookup-time thing, you could make it creation-time thing with a
pre-start hook like:
#!/bin/sh
STATE=$(cat) &&
PID=$(echo "${STATE}" | jq -r .pid) &&
ID=$(echo "${STATE}" | jq -r .id) &&
DIR="/run/opencontainer/containers/${ID}/ns" &&
mkdir "${DIR}" &&
for NAMESPACE_PATH in "/proc/${PID}/ns/"*
do
NAMESPACE=$(basename "${NAMESPACE_PATH}") &&
touch "${DIR}/${NAMESPACE}" &&
mount --bind "${NAMESPACE_PATH}" "${DIR}/${NAMESPACE}"
done
Then you could use:
"path": "/run/opencontainer/containers/{container-id}/ns/{namespace}"
in another container's runtime.json to join any of the initial
container's namespaces.

@laijs I agree with @philips. This sounds like a problem that can be solved using higher level tooling that generates the correct values in your runtime.json

Ok, lets close this one seeing that higher level tooling can generate these paths based on container IDs

On Thu, Oct 22, 2015 at 04:16:29PM -0700, Michael Crosby wrote:

Ok, lets close this one seeing that higher level tooling can
generate these paths based on container IDs

We should probably start collecting a list of these punted features,
so we can point people at a single document for out-of-scope features
(ideally with pointers to the higher-level tooling that covers that
use-case).  I'm happy to write up something along those lines if that
sounds useful, just tell me where you want it.


TextField will autoFocus？！

The thing is like this, when I upgrade to flutter 1.22.4 version, all the input boxes in the project will have the same problem, when I click the input box to input, click other places to close the keyboard, the cursor will automatically locate to the previous input Box and the keyboard pops up. This makes me very distressed. Does anyone know how to solve this problem?

when I upgrade to flutter 1.22.4 version

@mgd1427587890
You seem to be on old version of Flutter. Please upgrade to latest (2.2.3) that has gone through lot of updates and the behavior you mentioned may have already been fixed. So a good solution would be to upgrade your framework version.
Closing for now. If after upgrading, you still see the same behavior, provide your updated flutter doctor -v and a complete minimal reproducible code sample to properly address it.
Thanks.


Docker Integration Button Displays even when off

Environment
Docker
Version
0.12.3
Describe the problem
Updated from .12.2 > .12.3, my config has docker integration disabled but the icon still shows
Logs
too lazy as have already reverted to .12.2
Context
No response
Please tick the boxes

[X] I confirm that I attached the proper logs
[X] I've read the docs
[X] I've checked for duplicate issues
[X] I've tried to debug myself

You reverted to an older version with less features because a button is visible instead of hiding it with custom css ?
Anyways, we’ll look into it ;)

Ah yeah didn't think of that, I forgot there was a custom CSS box in the ui settings.


Feature Request: Scheduled ADMIN_DOWN for Servers

If it was possible for an Admin user to add a "maintenance" event to a server (Start time, stop time , required reason, username), that data could then be used so that when the server runs ORT within that time period the actual server status (if status == reported)  is overridden with ADMIN_DOWN.
This would enable ops people to pre-program the network maintenance and other stuff we need to work around, as well as give us a fantastic way (much more visible) to keep track of it for troubleshooting purposes.
In the self service world, this could even be taken a step further of just giving the permission to setup maintenance periods to the networking team directly.

@dneuman64 @jhg03a - is this still required with the implementation of https://github.com/apache/trafficcontrol/blob/master/blueprints/multi-interface-servers.md

Server interfaces doesn't address this request

I agree with Jonathan, the intention is to provide a mechanism by which TC itself can be "aware" of expected periods of downtime and route around them with no other operator intervention.

Server interfaces doesn't address this request

agreed. just wondered if a server having multiple interfaces reduced the need to even admin down servers for maintenance purposes. i guess not.

This ticket is predicated on queue and snap being safe for automation to perform.


Code organization (issue #8) and general enhancements

This is the full dataframe version that seems to work with in my deep learning project IRL. Sorry for the mix of things that have popped up here but it has been a little fixing as I found stuff needed for my actual work and adding an issue for each seemed like a hassle since most things are trivial.
The core changes are:

Split everything into extensions
Started updating the README
Changed travis to use run_all
Added core statistics that I find useful when designing networks (e.g. mode is important as this is the simplest classification network possible - always guessing the most common outcome)
Fixed torch serialization with tests
Rounding in output functions
Minor fixes to load_batch

Note: again - sorry for the size of this PR but it should be more readable as you merge the previous ones.

Could you please check your PR ? I lost the count with 2-3 last huge PR x)

Ok, resolved the conflict. It's a pity that git doesn't have a refactor feature since it is really hard to see what's new in the code since everything highlights as green. I hope that this will be the last major update, I may add a few minor features here and there but I'll try to keep to smaller PRs.
Let me know if you want to do a Google-Hangout session (or similar) and discuss sections of the code that you are uncertain about. I think the project has reached a maturity (my network is using it and learning stuff) and it may be good to notify the cheat sheet of the package. My email is<EMAIL_ADDRESS>

I am going to review all the code step by step and won't hesitate to ask you questions ;) If I have too many question I'll open a chat as you suggest but for now it's not necessary.
Awesome news to heard you're using it in your projects ^^ You really did an amazing job to take this project to another level !


Fixes godoc rendering

Follow up for https://github.com/operator-framework/operator-lifecycle-manager/pull/2933/files#r1132142964
Before:

After:

Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Bug fixes are accompanied by regression test(s)
[ ] e2e tests and flake fixes are accompanied evidence of flake testing, e.g. executing the test 100(0) times
[ ] tech debt/todo is accompanied by issue link(s) in comments in the surrounding code
[ ] Tests are comprehensible, e.g. Ginkgo DSL is being used appropriately
[ ] Docs updated or added to /doc
[ ] Commit messages sensible and descriptive
[ ] Tests marked as [FLAKE] are truly flaky and have an issue
[ ] Code is properly formatted

/cc tmshort

It's a comment. As long as go fmt ./... doesn't try to undo it...
/lgtm
/approve

Rebased. Please take another look

@perdasilva I had to rebase becaues github check was complaining that the branch is out of date. Could you please lgtm again? Nothing changed apart from rebase.

/lgtm


Fix conversion of Quint nullary polymorphic operators

Hello :octocat:
Here's another fix for the conversion from Quint. I was hitting problems whenever None was used to construct values of different types inside the same operator. Upon debugging, I realized this was specific to nullary operators (such as None), and that the builder itself enforces that the same name has the same type in a same scope. A concrete example:
pure def f(x) =
  pure val a = if (x > 2) Some(x) else None
  pure val b = if (x < 5) Some(true) else None

  (a, b)

One solution could be to use a polymorphic type (a type variable) for constructing the None name in TLA+, but that is too general and causes issues later on. So, instead, I wrote a new builder method polymorphicName that bypasses this check, and used that to construct names for nullary operators.
This is only a problem currently for nullary operators, as regular operators go through the application translation (not the name expression translation) and are dealt with differently.

[X] Tests added for any new code
[X] Ran make fmt-fix (or had formatting run automatically on all files edited)
[X] Documentation added for any new functionality
[x] Entries added to ./unreleased/ for any new functionality

Although my fix is sufficient to get over the conversion phase and produce valid TLA+, I see now that it is not sufficient to make snowcat happy:
error: internal error: while type checking in Apalache:
'unknown location,__QUINT_LAMBDA111's type annotation ((None({ tag: Str }) | Some(a36), a36) => None({ tag: Str }) | Some(a29)) is too general, inferred: ((None({ tag: Str }) | Some(a29), a29) => None({ tag: Str }) | Some(a29))'
Please report an issue: https://github.com/informalsystems/quint/issues/new

So more to investigate. This PR still unblocks things, so might be worth moving foward with it at this point. I'm not sure.

Yeah, the behavior of None is annoying in Quint. What I don't see is why this should be resolved at the level of TLA+. I thought that the Quint typechecker would assign proper types to the different instances of None. If this is the case, I believe that instead of propagating polymorphism to Apalache, I would rather produce different names for the different instances of None that carry different types. Otherwise, Apalache may not have sufficient context to infer the proper types.

I guess, this is the related issue: https://github.com/informalsystems/quint/issues/1451

I thought that the Quint typechecker would assign proper types to the different instances of None. If this is the case,

it is

I believe that instead of propagating polymorphism to Apalache, I would rather produce different names for the different instances of None that carry different types. Otherwise, Apalache may not have sufficient context to infer the proper types.

Hmmm but Apalache doesn't need to infer the types, as we annotate everything. However, in the end, it complains that our annotation is too general. I'll need to further debug to understand if it is actually too general and Quint has to provide a more specific type, or if Apalache is expecting something too strict.
Regarding the issue, I just confirmed that this PR fixed it. That spec works both with quint compile and quint verify. So that's more confirmation for me that this PR is worth moving foward with and that the "too general" issue is another problem to solve (probably the same as in https://github.com/informalsystems/quint/issues/1398)

The most important question for me is whether there was a reason for not allowing something like (F: Int)(...) and (F: Bool)(...) in the builder, unless F is defined as polymorphic. Perhaps, @Kukovec could help us to understand it.

The most important question for me is whether there was a reason for not allowing something like (F: Int)(...) and (F: Bool)(...) in the builder, unless F is defined as polymorphic. Perhaps, @Kukovec could help us to understand it.

I mean, the reason it's not allowed is simple, the basic notion of socpe-safety is that one name is only allowed to have one type in a given scope. For polymorphic operators, F() may have types Bool and Int in different instances, but F should have () => t in both.
Please don't add methods to the specifically scope-safe builder that makes it explicitly scope-unsafe, just use the unsafe builder.

Right! If we expect the types to be correct, we can just use UnsafeBuilder.

Ok, thanks for the input, it makes sense! Took me some time to figure out how to do it as I couldn't use point in Quint.scala. I ended up having to change the imports in unexpected ways, but it worked out in the end.

I will merge it right away to a cut a fresh release


Don’t use sqroot if not proven positiveness

Use ^(1/2) instead.

Principal root relation if we need a principal root of an expression is as follows:


genact: 1.0.0 -> 1.0.1

Description of changes
https://github.com/svenstaro/genact/compare/v1.0.0...v1.0.1
changelog

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package built:

genact

Thank you!


Added sort_heap  to algorithm file

I have added Heapsort file in algorithms file

Fixes #
Changes

-Just  added Heapsort  in algorithms file in C++
Checklist

[ yes] I have read CONTRIBUTING guidelines.
[ no] This is a typo fix.
[ yes] I am not updating any todo.txt files.

Thanks for your contribution but please stick to the format given in Contributing Guidelines


Rowname error with CIRI_DE_replicate command

Hi @Kevinzjy,
I am trying to execute the CIRI_DE_replicate command but I keep on getting the following error:
Error in [.data.frame(gene_mtx, , rownames(lib_mtx)) :
undefined columns selected
Calls: [ -> [.data.frame
Execution halted
[Sat 2021-03-06 12:08:08] [INFO ] Finished!
I think the error means that it cannot match the sample names listed in library_info.csv to the sample names in gene_count_matrix.csv. I have checked to verify that the sample naming in both files is the same, and the individual line responsible for subsetting in your CIRI_DE.R script  (gene_mtx <- gene_mtx[,rownames(lib_mtx)]) works just as expected when run in isolation on the same data sets. Would you have any insight into why the line is throwing an error in the context of your script? I suspect something may be off with my environment set-up, but I really have no clue.
Thank you!  Certainly let me know if you would need any more information to answer my question.
Prisca

Hi @Kevinzjy,
I was able to resolve this error. I noticed that an edit was added by @osagiei to the CIRI_DE.R file recently that would accommodate reading in of sample_IDs starting with a number. By git-cloning the most updated version of CIRIquant that included this edit, I was able to complete the differential expression successfully. Thanks!

Thanks, I will update the releases page soon.


NPE fix

Check that ControllerParams is non-null (should've had this in #19596)

Addendum to #14542 to catch failure early (in Java instead of in native code)


PC freezes

I build a new rig and tried using version 1.4.52 and 1.4.4 but everytime after I start the programm my pc freezes and I have to hard shutdown it. I used an old version on my ultrabook till now and had no issue. What can be the cause?

Same issue with mine too. The PC freezes whenever the program is opened!

I'm experiencing this issue as well. Latest version of DS4W on a Windows 7 machine.


fix manifest cache race

It occurred to me that the timestamp here needs to be pessimistic and use the time prior to initiating the IO call in order to avoid a race.

LGTM


May i ask if it  deploy on a new robot and fine-tuning it with only 1-5tracks？How is the generalization？

It won't work:( 1-5 trajectories (If I understand correctly, track means trajectory) are too few for the new robot.

It won't work:( 1-5 trajectories (If I understand correctly, track means trajectory) are too few for the new robot.

@lanlankilkil You can give it a shot, but it’s not likely to adapt the model to a brand new embodiment with just 1-5 trajectories. Let me know if you need more help! 😊


hotspots: Copy hotspots from other accounts with the same email.

Fixes this:
https://chat.zulip.org/#narrow/stream/9-issues/topic/disable.20helpful.20popup

If a new user signs up, the hotspots are copied from user profiles that are registered with the same email, so they are not shown to the user a second time.
If the user selects a user profile to copy the settings from only the viewed hotspots of this user are copied even if there are other profiles with the same email that have seen more hotspots.

The tutorial status is only updated if at least one hotspot is updated, this assumes that the tutorial status is never changed independently of hotspots.
Is this the desired behavior?
Testing plan:
The function test_signup_without_user_settings_from_another_realm and test_signup_with_user_settings_from_another_realm test the  behavior. Originaly I created another new test method for this feature, but it ended up being the same code like in test_signup_without_user_settings_from_another_realm so I removed it again.
Manually tested by running a server and creating different users in different realms.

@hackerkid Thanks a lot for this review! Your suggestion makes sense, I will update the code to implement this.
Thanks for the hint to the documentation, I already read that. I'm still wondering about the onboarding_steps, I can't find there usage in the code other than in the function copy_hotsots. So, when will they be updated and are the onboarding_steps representing the hotspots?

@mrchntia I don't think we make use of onboarding_steps anywhere. So we can just ignore it.

I implemented the suggestion and added tests for it. I also deleted the function copy_hotspots because it is not needed anymore.
I deleted the first commit, not sure if this was necessary.
@hackerkid could you review the changes?

Sorry for the late reply, I didn't find time to work over Christmas.
Thanks for the review. I'll do the changes and update the PR. :)

It seems like the test is failing for an unknown reason that is not related to my code. This PR https://github.com/zulip/zulip/pull/16997 will probably fix it.

It seems like the test is failing for an unknown reason that is not related to my code. This PR https://github.com/zulip/zulip/pull/16997 will probably fix it.

@hackerkid The failing frontend test is now fix and does not fail anymore. I ran ./tools/test-all locally and all tests are passed.
Because I didn't change anything on my branch force push only tells me that everyting is up-to-date therefore it doesn't run the CI tests again. Is there an other way to run the CI tests again so it shows that all checks are successful?

@hackerkid The failing frontend test is now fix and does not fail anymore. I ran ./tools/test-all locally and all tests are passed.
Because I didn't change anything on my branch force push only tells me that everyting is up-to-date therefore it doesn't run the CI tests again. Is there an other way to run the CI tests again so it shows that all checks are successful?

You can do this https://chat.zulip.org/#narrow/stream/43-automated-testing/topic/error.20checking.20links.20CI/near/825573 to repush and trigger CI again.

You can do this https://chat.zulip.org/#narrow/stream/43-automated-testing/topic/error.20checking.20links.20CI/near/825573 to repush and trigger CI again.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_08484417-a226-4ee5-be63-02dda5a337c9/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_08484417-a226-4ee5-be63-02dda5a337c9.git

Docs Build status updates of commit 60dc738:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-ServiceAccountCannotAccessRepository] Your repository https://github.com/VSC-Service-Account/docs-archive-test-source-1 has enabled Docs Build service account. It seems that at least one of the accounts, e2ebd3, does not have Read permission to the repo. Please ask your repo admin to grant correct permission to service accounts (refer to this document: https://review.docs.microsoft.com/en-us/engineering/projects/ops/engdocs/how-to-grant-service-account-permission-in-your-repository?branch=master#ops-e2e-ppe) and then retry the build. Don't know who to contact? Refer https://repos.opensource.microsoft.com/VSC-Service-Account/repos/docs-archive-test-source-1/permissions/ to get admin list of the repo if the repo is in one of the organizations Microsoft owns.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel