[client] Fix wrong comment

Motivation
Fix wrong comment.
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
NA
Related PRs
NA

@bors-libra r+

:pushpin: Commit 76ab084 has been approved by davidiw

:hourglass: Testing commit 76ab084d9ed36fd2df9c100cebfb154015758dca with merge 912f9bb38f7fa06fe31ed911f6d0ff60bfd1dd35...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: davidiw
Pushing 912f9bb38f7fa06fe31ed911f6d0ff60bfd1dd35 to master...


Change iframe_base_path to direct path

This should fix #19
https://fightforthefuture.github.io/battleforthenet-widget/iframe/iframe.html is being redirected to http://widget.battleforthenet.com/iframe/iframe.html

Thanks for tracking this down!


Update the gem upgrade doc

Now that performance stats are enabled by default there is no need to
enable performance monitoring manually, instead I've added a note about
performance monitoring being on by default.
Add some contact info if the user has trouble upgrading
Add a note about our upgrade from deprecated versions guide
Remove most current version note to keep the doc a little cleaner.

Before
After

Thanks for the review @thompiler! 🍔✨🍺


Audit browser compatibility

It looks like the browsers that don't support Web MIDI Player are Internet Explorer and Safari.

@Ksinia Great news! I've fixed the issue with Safari. Web MIDI Player version 1.3.0 should now work in this browser :)

@yvesgurcan Thanks, it works!

Woohoo! 🎉 How's the memory usage?


Add XCode 6.3 compatibility (UUID)

DVTPlugInCompatibilityUUID of XCode 6.3.

thx :+1:

You're welcome.
I noticed this is for 6.3.1 though. Not 6.3. Title is misleading.


Collapse NavigableCircuitContent functions

Now that the features are available in common code, BasicNavigableCircuitContent doesn't serve much purpose besides alternative defaults, which can by their very nature be changed.

The difference here is only in that the android function calls BackHandler before delegating, whereas, the common function expects the caller to handle the appropriate platform specific back handling.


vue v0.12.12

https://yadi.sk/i/CQ8bXcHIifrKs

Please be more specific - did this happen after you upgrade? What version were you using previously? What does your code look like? You can't expect people to help you just by posting a screenshot.

@yyx990803 previous version was 0.12.10, just updated, and there was an error

All the vue-router tests are working properly for 0.12.12, so I need a real demo to see what's wrong.

very strange here it works perfectly
http://jsfiddle.net/xyu276sa/10/
except for messages in the console
[vue-router] already installed.
[vue-router] v-link can only be used inside a router-enabled app.
[vue-router] <router-view> can only be used inside a router-enabled app.

but in the project, I have an error at this point
var $indexCtrl = new Vue

he looked a little bit, because of what a mistake ... as it turned out, it is certainly not because it can not find the element ID
error gave the router, causing all falls

error occurs after the announcement
Vue.use(VueRouter);

after I unplugged the router, the error is still there when the announcement
new Vue

Could this be due to the fact that I used nodejs?
because in php it had no such problems

If you are using the router you should start the app with the router. Closing until you can provide a repro with the actual error.

@yyx990803 I cited the example above, at the moment there is only the code that is provided in the example

it turns out there are two errors, one of vue, and the second from vue-router
and two identical error

You example isn't even using the router correctly, nor does it show the error you originally described.

@yyx990803 router code was taken from your example ... minor changes to it

You are mounting two Vue instances onto the same element #app. Your $indexControl should just be App.

even if I hook after the other selectors, or disable $indexControl the error still persists

Then it means the example is irrelevant to your error - and you need to provide one that actually shows the error.

the above link, I showed the code that there are now in fact used

But it's not showing the error in your screenshot. How many times do I have to explain this? I can't help you if the error only happens for you, because it means there's something wrong with your setup which is not shown.


Do we still need top-level groupings of annotations?

Do we, @ejhumphrey @justinsalamon @urinieto ?
At this point, they're redundant with the namespace fields.
Eliminating the top-level groupings would simplify the search logic #15 and make the schema automatically extensible for new tasks by adding new schema entries.
What we lose is a little coherence/human readability, but it's still encoded by the namespace field, so maybe it's not so bad?
Vote?

+1

I vote that we don't; however, there's no reason we can't provide
convenience properties / methods to simplify access.
e
On 1 Feb 2015 10:55, "Rachel Bittner"<EMAIL_ADDRESS>wrote:

+1
—
Reply to this email directly or view it on GitHub
https://github.com/marl/jams/issues/17#issuecomment-72370142.

The convenience method right now would look like
>>> beats = my_jam.search(namespace='beat')

We could provide sugar for that query, but I think it's simple enough that maybe we don't need to?

As long as it's documented with a couple of examples to get users started I think that's probably enough, with the bonus that you mentioned of making the format extensible without needing to modify the schema


fix: Bump version with one provided in pre-changelog-generation hook

I totally forgot about bumping version in the corresponding file. I should have tested it more thoroughly beforehand =\

This is not going to work as the version and the tag that you now provide to the script will always be empty. I will also check on how to solve this.

You are right. I forgot about the props completely, because they are not used in my case.


Support applicationTags.customTags for heartbeat

Address #39

@song-hao make sure you update the version in setup.py to 1.2.1

@haosong we have to merge this first, before merging https://github.com/wavefrontHQ/wavefront-sdk-python/pull/43


../../tests/dff_udp1.v line 11 - ERROR: syntax error, unexpected CLOSE_BRACKET - ')' - Parse failed

Issue Description
Getting following error
../../tests/dff_udp1.v line 11 - ERROR: syntax error, unexpected CLOSE_BRACKET
- ')'
- Parse failed

File ../../tests/dff_udp1.v obtained from https://www.hdlworks.com/hdl_corner/verilog_ref/items/UserDefinedPrimitives.htm
cat -n ../../tests/dff_udp1.v
     1	//https://www.hdlworks.com/hdl_corner/verilog_ref/items/UserDefinedPrimitives.htm
     2
     3	primitive Dff (q, d, clk, rst);    // sequential UDP
     4	  output q;
     5	  input clk, rst, d;
     6	  reg q;
     7	  initial q = 0;
     8	  table
     9	    // d  clk  rst : old q : q
    10	       ?   ?    0  :   ?   : 0;
    11	       0   R    1  :   ?   : 0;
    12	       1  (01)  1  :   ?   : 1;
    13	       ?   N    1  :   ?   : -;
    14	       *   ?    1  :   ?   : -;
    15	       ?   ?   (0?):   ?   : -;
    16	  endtable
    17	endprimitive

What problem / feature / issue are you raising? Describe in as much detail
as possible.
What should happen:
What should normal behaviour for the given input be?
../../tests/dff_udp1.v   - Parse successful

What actually happens:
../../tests/dff_udp1.v line 11 - ERROR: syntax error, unexpected CLOSE_BRACKET
- ')'
- Parse failed

Steps to re-create:
Describe exactly how to re-create the bug.
cd to build/debug
Obtain from https://www.hdlworks.com/hdl_corner/verilog_ref/items/UserDefinedPrimitives.htm and copy and paste file to ../../tests/dff_udp1.v
Then run
./src/parser ../../tests/dff_udp1.v
$> make clean
$> make all
$> ./src/parser ../../tests/dff_udp1.v

People to Notify:

@ben-marshall

Related Issues:

Verilog has symbols that can be used in the UDP table to improve readability
Symbol  Interpretation          Explanation
  b         0 or 1              Any known value
  r         (01)                0 -> 1 transition
  f         (10)                1 -> 0 transition
  p     (01) or (0x) or (x1)    Any positive edge, including unknowns
  n     (10) or (1x) or (x0)    Any negative edge, including unknowns
  *         (??)                Any transition

A workaround I have hacked the file to replace (01) with r with is the following,
but I believe the (01) notation serves a purpose in understanding the UDP when debugging
quickly.
cat -n ../../tests/dff_udp1a.v
     1	//https://www.hdlworks.com/hdl_corner/verilog_ref/items/UserDefinedPrimitives.htm
     2
     3	primitive Dff (q, d, clk, rst);    // sequential UDP
     4	  output q;
     5	  input clk, rst, d;
     6	  reg q;
     7	  initial q = 0;
     8	  table
     9	    // d  clk  rst : old q : q
    10	       ?   ?    0  :   ?   : 0;
    11	       0   R    1  :   ?   : 0;
    12	    // 1  (01)  1  :   ?   : 1;
    13	       1   r    1  :   ?   : 1;
    14	       ?   N    1  :   ?   : -;
    15	       *   ?    1  :   ?   : -;
    16	       ?   ?   (0?):   ?   : -;
    17	  endtable
    18	endprimitive

./src/parser ../../tests/dff_udp1a.v
../../tests/dff_udp1a.v 2879 /programming/ben-marshall/verilog-parser/src/verilog_parser.y Need to re-write this rule.
 - Parse successful
Freeing data for 117 memory allocations.
	Free'd 3446 bytes of 3526 bytes allocated.
	Bytes remaining: 80


Post 0.5.0 cleanup: provide manager, readme, changelog

fixes #21
fixes #17
updates some CHANGELOG links
update installation (add conda-forge links, update package name, etc.)
move area (main vs sidebar) to settings
bump version to 0.5.1 (nothing backwards-incompatible)

I don't anticipate any other changes, and the lack of the exposed token is blocking some downstream stuff, so will likely cut 0.5.1 (#23) once we merge this :rainbow: :unicorn: :fireworks: .

Thanks, @bollwyvl!


output disappears if resizing terminal?

When I make window smaller and then back to bigger.. things disappears from the terminal.
It seems to happen everytime.

This is (currently) the normal behavior of st, because one of its goals is not to reimplement tmux or other terminal multiplexers.
While this may be annoying sometimes, this can be easily solved by using tmux (or similar software), which handles resizing.

noice to know you can fix this with just using tmux

I'd actually like to have this "fixed" generally without tmux because I don't think it's unreasonable for st to keep information out of its borders for when windows change size. I don't have a fix for this now because I'm not familiar with that part of the source code, but if anyone else knows, feel free to open a PR or show another build with it because I would really like this feature.

maybe there is a terminal multiplexer that does just that: handles text overflow? i've been looking for a terminal multiplexer like that but didn't find anything.

I'll just note that if you're using the newer versions of st, it does indeed work with suckless's scroll program. It causes some other weird effects in zsh, so I haven't switched to it yet.

Yes, scroll seems very buggy, especially with zsh. Would really like to use it though

I'd actually like to have this "fixed" generally without tmux because I don't think it's unreasonable for st to keep information out of its borders for when windows change size. I don't have a fix for this now because I'm not familiar with that part of the source code, but if anyone else knows, feel free to open a PR or show another build with it because I would really like this feature.

There is a build that is capable of solving this problem: https://github.com/juliusHuelsmann/st-history-vim.
On the release page (https://github.com/juliusHuelsmann/st/releases) you will find all the diff files required.
The diff file with the name of "st-columns" is the one which will fix the resizing problem. But there are a few caveats:

It requires the "st-history" diff to function. Which I believe (I could be wrong) conflicts with the scrollback patch you can find on st's patches page.
Since the st-history forces you to get rid of the scrollback patch, the only way to get a scrollback function to work with the other two patches (st-columns and st-history) is to get juliusHuelsmann's custom scrollback patch. Which is called "st-meta-scrollback".

From the very brief testing I've done with it, in an st (0.8.4) build with no patches but the three mentioned, the columns patch seems to work as expected, the history patch seems to function just as the history implementation of the original scrollback patch, the custom scrollback patch sometimes have the strange behavior where after scrolling up the history and then typing something, it will not scroll back down to the current shell line. This could be related to a function implemented by this custom scrollback of which I don't really know what it is supposed to do. This function also happens to share a shortcut with vanilla dwm's incnmaster function and your personal dwm build's setlayout function, so you might want to change that.
Custom scrollback historyQuit function shortcut:
{ MODKEY,               XK_i,           historyQuit,    {.i =  0} },
dwm's incnmaster function shortcut:
{ MODKEY,                       XK_i,      incnmaster,     {.i = +1 } },
Your build's setlayout function shortcut:
{ MODKEY,			XK_i,		setlayout,	{.v = &layouts[6]} }, /* centeredmaster */
I've looked at the README of the repository containing the releases and also at the one on the development repository and could not find anything mentioning a "patch_column" that can work with the original scrollback patch, unfortunately.
I lack the time and know-how to tinker with these patches and try to come up with one that can work with the original scrollback patch. But for anybody willing to give it a shot, perhaps a diff file from when vim-browse was still relying on the original scrollback could give some insight or basis of comparison: https://st.suckless.org/patches/vim_browse/st-vimBrowse-20200604-295a43f.diff.

This long-standing annoyance has been solved by the upgrade to 0.8.4 alongside with the integration of scroll as an automatically installing gitmodule.
Enjoy.

Being using this without updating for two years :D let see what niceties it brings with it.

Hey @LukeSmithxyz , have you been using the scroll tool? Are there any issues you encountered? Because I just read in their site (under "What is the state of scroll?") that it has some issues: https://tools.suckless.org/scroll/.


Make boolean compiler options in harness true only if they say 'true'

Today, if you say
//@declaration: false

in a test, it will emit declarations! The harness just checks for the presence or absence of these options. It does not respect false.

:+1:

:+1:


Makes this Github Action target node16 or newer

Same description as https://github.com/emulator-wtf/invoke/pull/3 😄

@madisp please dont miss any line 😹


flask inner server console display in a mess

<IP_ADDRESS> - - [12/Jul/2017 23:30:15] code 400, message Bad request version ('\xcc\xa9\xcc▒▒\xcc\x14\xcc\x13\xc0+\xc0/\xc0') <IP_ADDRESS> - - [12/Jul/2017 23:30:15] "\xbe\xba\xe5\x9b▒▒K\xe3                                                              \\xf5-U\xe3ou\xbe&M\x83O\xd3cTN\xf0V\xf4\xef\xa2\x95\xcc\xa9\xcc▒▒\xcc\xcc\xc0+\xc0/\xc0" 400 - <IP_ADDRESS> - - [12/Jul/2017 23:30:15] code 400, message Bad request version ('\xcc\xa9\xcc▒▒\xcc\x14\xcc\x13\xc0+\xc0/\xc0')
When the client connect to the flask server, it just print something above. My computer has always work well with flask inner server console display. And i cannot figure out what's wrong with it.

@linkmyth this needs more details, which fronted are you using, and are you trying to use https  protocol?

I am pretty sure this is happening because you are trying to access it using https, I'm closing this.


Syndicate fax machine doesnt start with the emagged component

Description
The syndicate long range fax machine, by default, doesnt start out emagged due to changes in the fax machine component. Unless it is manually emagged, it will not show up for any emagged fax machines.
Reproduction

Spawn a syndicate long range fax machine.
Emag a station fax machine
Observe how the emagged fax machine cant see the syndicate one until its also emagged

Additional context
It should be a simple and easy fix by modifying this bit of code that was made obsolete by the EmaggedComponent, and adding that as a component instead. I'd do it myself but I'm not at a computer.

Fixed in https://github.com/space-wizards/space-station-14/pull/23291


implement exemptFeatures for conformance profile

Fixes: https://github.com/kubernetes-sigs/gateway-api/issues/2513
What type of PR is this?

What this PR does / why we need it:
Implement the ExemptFeatures setting in Experimental Conformance Profile
Which issue(s) this PR fixes:

Fixes # https://github.com/kubernetes-sigs/gateway-api/issues/2513
Does this PR introduce a user-facing change?:

Implemented the `ExemptFeatures` field for Experimental Conformance Profiles

/test pull-gateway-api-verify


It is difficult to determine what Hack Cambridge is from /apply

This year, many ads for Hack Cambridge are linking straight to /apply, to reduce friction in the process.
However, landing on it, if you don't know what Hack Cambridge or even what a hackathon is, it does not provide much information:

Maybe we should add some of the content on the home page, or at least a more compelling reason to visit the home page?
A useful follow-up to this may be to look at the analytics and see if this has affected application rates.

@jmalcic assigning to you as discussed in the meeting


Add benchmarks

Add Criterion benchmarks to the project

Vector initialization benchmark tests https://github.com/chrissimpkins/vectora/pull/6


Ask for credentials separately

Hi,
I created this fork since the way I'd like to work with my Simplenote credentials doesn't seem to work at the moment. I'd like to store my email/username in my .vimrc, but not my password. However, if I don't set g:SimplenotePassword the plugin always asks for both username and password.
My changes to set_cred will ensure that only the part of the credentials is asked for that isn't specified. Please let me know if there are any problems with the changes.
@mrtazz @atomicules I'm assuming that you guys are the best people to review this PR!?
Cheers

Hi. Thanks! I will take a look (hopefully) soon.

Hi, thanks for going to the length of writing tests. How are you intending that those are run?
Admittedly I haven't looked in detail, but I guess there is a reason why you couldn't user Vader?
Apart from quibbles over the tests this looks like a good addition though.

Hey, thanks for having a look.
I'm not sure about running the tests automatically. It seems https://github.com/mrtazz/simplenote.py is using travis ci for that purpose. Maybe that's an option here, too? How are the Vader tests run?
I haven't used Vader for 2 reasons:

I have no experience with it (lame excuse, I know 😄 ).
My changes only affected 1 function and I strongly believe in keeping the scope of tests minimal (to keep them fast).
However, if a few Vader tests are more in line with the general consensus, I'd be happy to spend some time writing them.

I've merged that in without the tests for now.

Great, thanks!


Copy Failed, Please Try Later Error in PowerToys Run even when it actually copies

Microsoft PowerToys version
0.84.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce

Open PowerToys Run and try to perform any arithmetic calculation. It gives an option to copy this number to the clipboard. When you press Enter or left click the error pops up even though it does actually copy the number. Also to note: Computer freezes and doesn't let me do anything until I acknowledge the error.
✔ Expected Behavior
Not show the error
❌ Actual Behavior
Shows the error popup
Other Software
Nothing as of yet but I will try seeing if opening certain app along with powertoys run leads to this issue.

Found the cause: Parsec was the issue. I do want to run both together, is there anyway we can resolve this?


Will need an association table

CompetitionParticipants (id, comp_id, user_id, created)
Comp_id and user_id should be a composite unique key (user can only join a competition once)

https://github.com/ReyMArias/IT202009/pull/45


Add introduction to streamlit_app.py

Add an intro to the document.
Edit the variable intro in streamlit_app.py
intro = '''
This is some text
'''
st.write(intro)

And include the text you built in the Google Docs. You can decide two summarize in one or two paragraphs or leave as bullet points.

print("The Human BioMolecular Atlas Program (HuBMAP) aims to create an immersive experience of a 3D-map representation of the human body, improving access to data of the human body and developing methods for tissue interrogation applicable to other research areas. To commence its first phase, HuBMAP achieved significant highlights, including standardizing protocols, innovating imaging and sequencing techniques, and establishing reliable data. These efforts led to the creation of high-resolution molecules and of the nine major organ systems. Researchers are working together to expand the map from a 2D to a 3D environment, including factors such as age and ethnicity.
Looking ahead, the main objective of HuBMAP is to provide freely accessible data via its online portal. The program focuses on investigating changes in individual cells and detecting diseases to prevent the harms of healthy aging. Understanding these details will help scientists develop better drugs targeting multiple organs and predict disease outcomes and progression more accurately in clinical settings. The HuBMAP production phase (2022–2026) has several goals. Additionally, it aims to generate reference datasets using new technologies, emphasizing building 3D maps and collecting data from diverse donors representing a range of demographic features (sex, race or ethnicity, and age). Its main purpose is to overcome challenges and gaps. Finally, it aims to improve metadata standards, analytical and visualization tools, and data integration and interpretation to enhance the tissue atlases of the human body through collaborative efforts.")
st.write (intro)

@jseoku
text = '### Dataset types'
st.write(text)

text = '### Dataset types'
st.write(text)

text = '''
## Assessment of public tool - Organ VR
### Method
### Setup
### Assessment
---
Copyright © 2024 Carnegie Mellon University. All Rights Reserved.

The [Biomedical Applications Group](https://www.psc.edu/biomedical-applications/) at the [Pittsburgh Supercomputing Center](http://www.psc.edu) in the [Mellon College of Science](https://www.cmu.edu/mcs/) at [Carnegie Mellon University](http://www.cmu.edu) and the Data Science Group at CS Scholars 2024.
'''
st.write(text)

this is fine but you need to edit the file streamlit_app.py and it there. If you have questions, do not hesitate to ask.


Fix margins of Start with data page according to the design

Fix margins according to the design
In the current implementation, we don't have any padding. I suggest implementing it according to the design since having full width like we currently have is not visually recommended and it affects readability, especially on larger screens, as the eye has to travel a longer distance.

Can you please assign this task to me?


[Breaking change] Account for field promotion to Null when computing reachability based on equality tests

Background
Normally, reachability analysis understands that comparing two values with static type Null is guaranteed to succeed, e.g.:
int test() {
  if (null == null) {
    return 0;
  }
  // No `return` statement but that's ok because this code path is unreachable
}

This works even if one of the values being compared is Null solely because it got type promoted via an is test:
int test(int? value) {
  if (value is! Null) return 0; // promotes `value` to type `Null`
  if (value == null) { // guaranteed to succeed
    return 0;
  }
  // No `return` statement but that's ok because this code path is unreachable
}

(Note that an is or as test is the only way to promote to the type Null. Type promotion doesn't promote values to the type Null based on an == check because it isn't really that useful to do so. And we have an analyzer warning that discourages people from using is Null or is! Null, so probably this doesn't arise very often in practice).
Unfortunately, due to a bug in how reachability analysis tracks field type promotion, it doesn't work if the value being compared is a promoted field:
class C {
  final int? _f;
  C(this._f);
}
int test(C c) {
  if (c._f is! Null) return 0; // promotes `c._f` to type `Null`
  if (c._f == null) { // reachability analysis fails to understand that this is guaranteed to succeed
    return 0;
  }
  // Error: missing `return` statement
}

Proposed change
The above-described bug will be fixed, so that if one of the values on the left or right side of == or != is promoted to the type Null, this promoted type will be used to determine reachability of the code that follows.
Rationale
Fixing this issue makes the reachability analysis of == and != tests dependent solely on the static types of the expressions being compared. This will allow the implementation of reachability analysis to be simplified.
Expected impact
This change is expected to be extremely low impact, for the following reasons:

Field promotion does not occur very often
Promoting to the type Null is rare (and discouraged)
Comparing a value to null when it is known to have type Null is not particularly useful
Usually, it is low impact for a code path that was previously considered reachable to be reclassified to unreachable, since for the most part, code being unreachable only results in a warning.

However, it's conceivable that a code path being unreachable could result in a change to type promotion behavior, which could in turn lead to type errors later in the program. Here is an example of how that could occur:
class C {
  final int? _f;
  C(this._f);
}
void test(C c, int? x) {
  if (c._f is! Null) return; // promotes `c._f` to type `Null`
  if (x == null) return; // promotes `x` to type `int`
  if (c._f != null) {
    // considerd reachable today; with the fix, will be considered unreachable
    x = null; // demotes `x` back to `int?` today; with the fix, will have no effect
  }
  var y = x; // inferred type is `int?` today; with the fix, will be `int`
  y = null; // ok today; with the fix, will be an error.
}

(Note that this example is quite contrived; I haven't come up with a real-world example where there is a problem.)
Mitigation
In the unlikely event that this change affects the behavior of your code, it will affect it by way of changing an inferred type (as in the example above). If this happens, you can restore the old behavior by adding explicit types to the program. (For example, in the code above, the old behavior could be restored by changing var y = x; to int? y = x;.)

@vsmenon @leonsenft @Hixie breaking change review.

@leafpetersen - are you supportive of this change?

Fine by me; if it was up to me we'd just remove var entirely. :-)

LGTM

lgtm

LGTM


New package: HMatrices v0.1.0

Registering package: HMatrices
Repository: https://github.com/WaveProp/HMatrices.jl
Created by: @maltezfaria
Version: v0.1.0
Commit: e67f1ecada85e71ca5d4b54fa2aeaa5dde5997e7
Git reference: HEAD
Description: A Julia library for hierarchical matrices

Hi,
Automerge has halted due to an existing package with a very similar name (HSSMatrices). I am reluctant to change the package's name, which I think is descriptive and provides an accurate description of the hierarchical matrix format implemented. The issue is that the names of various hierarchical matrix format are very similar (e.g. HMatrix, HSSMatrix, H2Matrix, HODLRMatrix, etc), so trying to name the package according to the implemented format causes a conflict.
Is it acceptable to keep the name as such and manually merge this? If not, any suggestions on how to proceed?
Thanks.

To be honest, I think HierarchicalMatrices would be a better name, instead of trying to guess what the H stands for

To be honest, I think HierarchicalMatrices would be a better name, instead of trying to guess what the H stands for

That name is already taken by another package implementing more of an abstract interface to hierarchical matrices. I guess I could go with HierMatrices then if it seems more explicit.


[BUG] Build fails with missing dependency build-tools

☠ ./gradlew build

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'opensearch-sql'.
> Could not resolve all artifacts for configuration ':classpath'.
   > Could not find org.opensearch.gradle:build-tools:1.0.0-rc1.
     Searched in the following locations:
       - file:/home/me/.m2/repository/org/opensearch/gradle/build-tools/1.0.0-rc1/build-tools-1.0.0-rc1.pom
       - https://repo.maven.apache.org/maven2/org/opensearch/gradle/build-tools/1.0.0-rc1/build-tools-1.0.0-rc1.pom
       - https://jcenter.bintray.com/org/opensearch/gradle/build-tools/1.0.0-rc1/build-tools-1.0.0-rc1.pom
       - https://plugins.gradle.org/m2/org/opensearch/gradle/build-tools/1.0.0-rc1/build-tools-1.0.0-rc1.pom
     Required by:
         project :

Hi @faxm0dem the build failure is because the opensearch dependencies (v1.0.0-rc) were not available in the maven repository yet since the GA was not officially released until this Monday. The good thing is we just released v<IP_ADDRESS>. Please checkout the latest source code and try to build against v<IP_ADDRESS>, this should work fine.

Thanks. I tried, but I still get the same error. I see there are many tags that resemble <IP_ADDRESS> maybe I checked out the wrong one ?
tag <IP_ADDRESS>
> git status
HEAD detached at <IP_ADDRESS>
nothing to commit, working tree clean
> ./gradlew build

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'opensearch-sql'.
> Could not resolve all artifacts for configuration ':classpath'.
   > Could not find org.opensearch.gradle:build-tools:1.0.0.
     Searched in the following locations:
       - file:/home/fwernli/.m2/repository/org/opensearch/gradle/build-tools/1.0.0/build-tools-1.0.0.pom
       - https://repo.maven.apache.org/maven2/org/opensearch/gradle/build-tools/1.0.0/build-tools-1.0.0.pom
       - https://jcenter.bintray.com/org/opensearch/gradle/build-tools/1.0.0/build-tools-1.0.0.pom
       - https://plugins.gradle.org/m2/org/opensearch/gradle/build-tools/1.0.0/build-tools-1.0.0.pom
     Required by:
         project :

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1s

tag v<IP_ADDRESS>
> git status
HEAD detached at v<IP_ADDRESS>
nothing to commit, working tree clean
> ./gradlew build

FAILURE: Build completed with 2 failures.

1: Task failed with an exception.
-----------
* Where:
Build file '/tmp/sql/build.gradle' line: 55

* What went wrong:
A problem occurred evaluating root project 'opendistro-sql'.
> Failed to apply plugin [id 'elasticsearch.esplugin']
   > JAVA_HOME must be set to build Elasticsearch. Note that if the variable was just set you might have to run `./gradlew --stop` for it to be picked up. See https://github.com/elastic/elasticsearch/issues/31399 details.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.
-----------
* What went wrong:
A problem occurred configuring root project 'opendistro-sql'.
> Must specify license and notice file for project :

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

* Get more help at https://help.gradle.org

BUILD FAILED in 0s

@chloe-zh This is still an issue with all plugins. Do you plan to release org.opensearch.gradle:build-tools in Maven Central? If not, how would one provide it in a local repository?

@chloe-zh This is still an issue with all plugins. Do you plan to release org.opensearch.gradle:build-tools in Maven Central? If not, how would one provide it in a local repository?

Yes, https://github.com/opensearch-project/opensearch-build/issues/20.

@faxm0dem For now, follow the sequence in https://github.com/opensearch-project/sql/blob/main/.github/workflows/sql-test-and-build-workflow.yml#L26 and https://github.com/opensearch-project/opensearch-plugins#building-plugins-with-opensearch

Closing this for now. Feel free to reopen if you need anything. Thanks!


Improved texts

As agreed on Discord, here is an update of files with some text improvements.
2458180650 - Ancient Rim Core.zip
Only changed files are attached.

Fixed in 2334191
Thanks!


VM Contents too Vague

So I get -

Each image contains the recommended feature set for that Visual Studio edition. Generally, the installation includes:
All available workloads, including each workload’s recommended optional components

The problem is, I have no idea what that means. Specifically, I'd like to know what SDKs come installed by default. (Since it's very sadly not simply all Windows SDKs)
I get that maybe this page itself can't have that info - because it changes - but if that information lives anywhere, I'd appreciate getting a pointer to it.

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

ID: eb992634-7ced-e225-f090-8f6c1c78d589
Version Independent ID: 8adca596-4bd5-5435-3df8-371c1676e27a
Content: Using Visual Studio on an Azure virtual machine - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/using-visual-studio-vm.md
Service: virtual-machines
GitHub Login: @cathysull
Microsoft Alias: cathys

@mandalorianbob Thanks for the question. We are investigating the issue and will update you shortly.

@mimckitt
Can you please check and add your comments on this doc update request as applicable.

#please-close


AttributeError: can't set attribute

Hello,I'm running your project and an error occured:
Traceback (most recent call last):
File "/home/PycharmProjects/gymdemo/mobile_robot_rl/run.py", line 71, in
run_vision_env(4)
File "/home/PycharmProjects/gymdemo/mobile_robot_rl/run.py", line 15, in run_vision_env
length=history_length)
File "/home/PycharmProjects/gymdemo/mobile_robot_rl/common/env_wrapper.py", line 150, in make_env
env = FrameStack(env, length, nchw_format)
File "/home/PycharmProjects/gymdemo/mobile_robot_rl/common/env_wrapper.py", line 104, in init
value.shape = generate_shape(shp)
AttributeError: can't set attribute
Do you know how to solve it？
Thanks in advance :)

@Murphy103 Hello, sorry for late answer. I'll look a this in the next month, is it ok for you?


:bug: useNetwork doesn't detect network change (starknet-react v1.0.2)

The useNetwork hook doesn't detect network change.
https://github.com/apibara/starknet-react/blob/a725fa891960f95a0dbf8994a420ff23f2738e34/packages/core/src/hooks/network.ts#L31
Replacing line 31 with something like this could fix the bug.
use [chain, setChain] = useState<Chain>(); // TODO: retrieve the default network value
useEffet(() => {
    if (!library) return;
    library.getChainId().then((chainId) => setChain(chainById(chainId)));
  }, [library])

You're right. What's happening is that the query doesn't have a key so it won't refresh. Fix incoming.


Chef::Node.load(node_name) giving configuration errors while interacting with chef server unless Chef::Knife.run() is run first

Description
Before starting my app, I am running in command line - knife ssl fetch, since I have a self-signed certificate. If I try to run Chef::Node.load("node_name"), the command fails with error

[2016-09-10T12:08:49+00:00] WARN: Failed to read the private key /etc/chef/client.pem: #
Chef::Exceptions::PrivateKeyMissing: I cannot read /etc/chef/client.pem, which you told me to use to sign requests!
.
.

I have to run any knife command likeChef::Knife.run(%w(node list), options = {}) first and then run Chef::Node.load("node_name") to make it work
Chef Version
chef (12.1.2)
Platform Version
root@a8f2e0b96f36:/app# uname -a
Linux a8f2e0b96f36 3.10.0-327.28.3.el7.x86_64 #1 SMP Thu Aug 18 19:05:49 UTC 2016 x86_64 GNU/Linux
root@a8f2e0b96f36:/app# cat /etc/*-release
PRETTY_NAME="Debian GNU/Linux 8 (jessie)"
NAME="Debian GNU/Linux"
VERSION_ID="8"
VERSION="8 (jessie)"
ID=debian
Replication Case

run knife ssl fetch from command line from pwd
Certificates are successfully fetched.
Enter irb console (from same dir) with require chef done (and self-signed chef server certificates, config etc. all present in appropriate directory .chef/trusted_certs and .chef/knife.rb)
Run command Chef::Node.load("some-valid-node-name", options = {})
Command will FAIL with output:

[2016-09-10T12:19:32+00:00] WARN: Failed to read the private key /etc/chef/client.pem: #
Chef::Exceptions::PrivateKeyMissing: I cannot read /etc/chef/client.pem, which you told me to use to sign requests!
 and some stacktrace is also present here - attached in gist link https://gist.github.ibm.com/bhavya-maheshwari/6e4597dbe0e7c0980dbad7c40b4604cb

Run command Chef::Knife.run(%w(node list), options = {}) and now same command in Step 3.
Command will SUCCEED with obj details dumped into stdout.

Client Output
The relevant output of the chef-client run or a link to a gist of the entire run, if there is one: https://gist.github.ibm.com/bhavya-maheshwari/6e4597dbe0e7c0980dbad7c40b4604cb
Stacktrace
https://gist.github.ibm.com/bhavya-maheshwari/6e4597dbe0e7c0980dbad7c40b4604cb
NOTE: CHEF CLIENT BUGS ONLY
This issue tracker is for the code contained within this repo -- chef-client, base knife functionality (not
plugins), chef-apply, chef-solo, chef-client -z, etc.

Server issues
ChefDK issues
Cookbook Issues (see the https://github.com/chef-cookbooks repos or search Supermarket or GitHub/Google)

Thanks for filing a GitHub Issue.
It looks like you've not configured Chef with its configuration file before you've tried to use it. You should do this by using Chef::WorkstationConfigLoader or similar. have a look at https://github.com/chef/chef/blob/master/lib/chef/knife.rb#L406 for how knife does it.
At Chef, we use GitHub Issues to track bugs and feature requests. Questions like "How do I .... with Chef?" are better answered by our awesome community on the Chef mailing lists, or on StackOverflow. (Please pick only one place to ask your question, however, as it's considered impolite to post the same question to multiple forums.)
You can find information about joining and asking questions on our mailing lists here. On StackOverflow, please tag your question with the "chef" label so that it shows up in this list.
We will be closing your issue. Please get in touch with the community via the above mechanisms in order to find a solution to your problem.


[WIP] recent senders: Add sort by overall message recency.

If both the users have neither posted on the current topic nor
in the current stream, then as a tiebreaker, check whether
who have posted recently in the organization as a whole.
Fixes: #5956.
(Test to be updated).

Before updating tests I want to mention which test cases I'm gonna cover:
When recency is checked on the basis of topic:

[ ] Both have posted in current Stream and Topic.
[ ] Both have posted in current Stream but only one has posted in the current topic.
[ ] Only one has posted in current Stream and that one has posted in the current topic also.

When recency is checked on the basis of the whole stream.

[ ] Both have posted in current Stream but no one has posted in the current topic.
[ ] Only one has posted in current Stream but no one has posted in the current topic.

When recency is checked on the basis of the recent message in the whole organization.

[ ] Neither of them has posted in the current stream.

And

[ ] Other cases like neither of them have posted in the organization.

This is looking great so far! And those test cases seem good to me. Some to consider

"only one person has posted in the org", that I assume you were covering other "Other cases like ..".
A has posted more recently in the stream but B more recently in the topic
A more recently in the org but B more recently in the stream

"only one person has posted in the org", that I assume you were covering other "Other cases like ..".

Actually other cases only happen when no one has posted in the organization, and above case is resolved by overall sort.
I have updated the tests and reorganized them in each case trying to cover all the cases in a systematic manner (tell me If the previous order of cases was better).

Great! Feel free to remove the WIP when this is ready for review; have assigned Steve for final review (may take a few days, due to the holiday).


Enable subPath as a variable for nextcloud chart.

What this PR does / why we need it:
Enable nextcloud data path to use a user-defined subpath.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md

@scottrigby -- could you review?

Still looking for someone to approve this PR...

Hi @carpenike ,
thx for your PR. Can you please explain a bit why we should want to make the subpath for /var/www/html editable?

My understanding is the subpath enables the configuration of a sub folder on the PV. So if their is a shared PV, ie /data on the host, the subpath would mount /data/nextcloud to the /var/www/html path in pod.

I´m with you. One point I don´t get at the moment is why you suggest an editable subpath-name only for /var/www/html?
What´s with the following subPaths?

/var/www/html/data
/var/www/html/config
/var/www/html/custom_apps
/var/www/html/themes

@scottrigby from my site, we should close this PR. No more response and forked branch got removed.


removed solution from comparison operators exercise and added missing input field to its last exercise

fixes #496.

Wonderful!


Refactor header / base site into separate template to avoid code duplication

Since this is the same between pages on the dashboard


add optional load_balancers option

I noticed #25 already proposed this change, but it doesn't seem to be able to merge + provides additional changes. ECS's limitation of load balancing one port per service is the main justification for this change.

@jamesalbert would you mind updating the README as well, once thats all set, I'll merge this in!

@tfhartmann Nice, sounds good 👍 pushed just now

🏆  Thanks!


CEF 61 is due to close without races

https://github.com/NISystemsEngineering/CEF/issues/61

hrm, now I'm seeing something else. But close without races does need to have a slightly different wait mechanism to be more performant.


Plugin 'InnoDB' registration as a STORAGE ENGINE failed.

This appears to be a critical bug related to the mysql container and is very much related to Mariadb plugin 'InnoDB' init function returned error..
$ docker-compose up --build mysql
Building volumes_data
Step 1 : FROM debian:jessie
 ---> 1b088884749b
Step 2 : MAINTAINER Mahmoud Zalt<EMAIL_ADDRESS> ---> Using cache
 ---> 30d4a45a7746
Step 3 : CMD true
 ---> Using cache
 ---> f6d96a5ebe00
Successfully built f6d96a5ebe00
Building mysql
Step 1 : FROM mysql:latest
 ---> 1195b21c3a45
Step 2 : MAINTAINER Mahmoud Zalt<EMAIL_ADDRESS> ---> Running in 21df3c2c63a9
 ---> a7edaddb3716
Removing intermediate container 21df3c2c63a9
Step 3 : CMD mysqld
 ---> Running in 7ff8b0c55105
 ---> 23b086ba17ad
Removing intermediate container 7ff8b0c55105
Step 4 : EXPOSE 3306
 ---> Running in 6f1efbf5c18b
 ---> b264141244d8
Removing intermediate container 6f1efbf5c18b
Successfully built b264141244d8
Starting laradock_volumes_data_1
Creating laradock_mysql_1
Attaching to laradock_mysql_1
mysql_1               | 2016-08-03T16:48:50.612224Z 0 [Note] mysqld (mysqld 5.7.13) starting as process 1 ...
mysql_1               | 2016-08-03T16:48:50.617998Z 0 [Note] InnoDB: PUNCH HOLE support available
mysql_1               | 2016-08-03T16:48:50.618059Z 0 [Note] InnoDB: Mutexes and rw_locks use GCC atomic builtins
mysql_1               | 2016-08-03T16:48:50.618067Z 0 [Note] InnoDB: Uses event mutexes
mysql_1               | 2016-08-03T16:48:50.618133Z 0 [Note] InnoDB: GCC builtin __atomic_thread_fence() is used for memory barrier
mysql_1               | 2016-08-03T16:48:50.618138Z 0 [Note] InnoDB: Compressed tables use zlib 1.2.8
mysql_1               | 2016-08-03T16:48:50.618141Z 0 [Note] InnoDB: Using Linux native AIO
mysql_1               | 2016-08-03T16:48:50.618373Z 0 [Note] InnoDB: Number of pools: 1
mysql_1               | 2016-08-03T16:48:50.618463Z 0 [Note] InnoDB: Using CPU crc32 instructions
mysql_1               | 2016-08-03T16:48:50.619780Z 0 [Note] InnoDB: Initializing buffer pool, total size = 128M, instances = 1, chunk size = 128M
mysql_1               | 2016-08-03T16:48:50.625443Z 0 [Note] InnoDB: Completed initialization of buffer pool
mysql_1               | 2016-08-03T16:48:50.626874Z 0 [Note] InnoDB: If the mysqld execution user is authorized, page cleaner thread priority can be changed. See the man page of setpriority().
mysql_1               | 2016-08-03T16:48:50.637796Z 0 [ERROR] InnoDB: The Auto-extending innodb_system data file './ibdata1' is of a different size 0 pages (rounded down to MB) than specified in the .cnf file: initial 768 pages, max 0 (relevant if non-zero) pages!
mysql_1               | 2016-08-03T16:48:50.637973Z 0 [ERROR] InnoDB: Plugin initialization aborted with error Generic error
mysql_1               | 2016-08-03T16:48:51.238477Z 0 [ERROR] Plugin 'InnoDB' init function returned error.
mysql_1               | 2016-08-03T16:48:51.238511Z 0 [ERROR] Plugin 'InnoDB' registration as a STORAGE ENGINE failed.
mysql_1               | 2016-08-03T16:48:51.238517Z 0 [ERROR] Failed to initialize plugins.
mysql_1               | 2016-08-03T16:48:51.238519Z 0 [ERROR] Aborting
mysql_1               |
mysql_1               | 2016-08-03T16:48:51.238530Z 0 [Note] Binlog end
mysql_1               | 2016-08-03T16:48:51.238678Z 0 [Note] Shutting down plugin 'MyISAM'
mysql_1               | 2016-08-03T16:48:51.238698Z 0 [Note] Shutting down plugin 'CSV'
mysql_1               | 2016-08-03T16:48:51.239062Z 0 [Note] mysqld: Shutdown complete
mysql_1               |
laradock_mysql_1 exited with code 1

Can't say exactly which ones, but I removed a lot of images from related laradock builds and retried the above and it now starts without errors! :)


EarlyStopping with restore_best_weights=True makes TypeError: 'NoneType' object is not subscriptable

EarlyStopping with restore_best_weights=True makes TypeError: 'NoneType' object is not subscriptable.
When restore_best_weights=False no exception.
System information

Keras version:  2.3.1
Python version:  3.6

Stack trace:
  File "/.../.venv/lib/python3.6/site-packages/keras/engine/training.py", line 1239, in fit
    validation_freq=validation_freq)
  File "/.../.venv/lib/python3.6/site-packages/keras/engine/training_arrays.py", line 216, in fit_loop
    callbacks.on_epoch_end(epoch, epoch_logs)
  File "/.../.venv/lib/python3.6/site-packages/keras/callbacks/callbacks.py", line 152, in on_epoch_end
    callback.on_epoch_end(epoch, logs)
  File "/.../.venv/lib/python3.6/site-packages/keras/callbacks/callbacks.py", line 834, in on_epoch_end
    self.model.set_weights(self.best_weights)
  File "/.../.venv/lib/python3.6/site-packages/keras/engine/network.py", line 516, in set_weights
    own_weights = weights[:num_param]
TypeError: 'NoneType' object is not subscriptable

Process finished with exit code 1

i have same problem

Same problem running on Google Colab

+1

Found the "problem".
It makes sense it is None because it found no better model than the baseline in my case.
I got rid of "baseline=1.0" and now it works.

@iirekm Moving this issue to closed status as there has been no recent activity, in case you still face the error please create a new issue.Thanks!


new tab

can you re-add the ability to open a new Google tab in an about:blank format

Yes, im working on adding that feature back. I had to restart the codebase from scratch as it was too complex and clunky so many features are being rewritten. My apologies


Use event in main view

This PR includes:

use of events to update, add or delete the layer and sources
use the layers tree to handle the level of each layer in the map
start refactoring mainview.tsx to move it then to an base View. It adds functions setSource, removeSource, updateLayers, updateLayer and removeLayer that could be implemented by a plugin (see https://github.com/QuantStack/jupytergis/issues/7 for reference)

Thanks!

It adds functions setSource, removeSource, updateLayers, updateLayer and removeLayer

I wonder if the APIs between sources and layers could be symmetrical:

addSource/updateSource/removeSource
addLayer/updateLayer/removeLayer

I wonder if the APIs between sources and layers could be symmetrical:

I agree.
MapLibre uses a beforeId value to insert the layer at the correct level.
In the addLayer function, we should probably  use a index number to be more generic for other frameworks, and compute the beforeId value in the function, in the case of MapLibre.

index would be the index at which to insert the layer in the list? If so, this is not ideal in RTC since layers could have been removed in the meantime and the index could be out of range.

index would be the index at which to insert the layer in the list? If so, this is not ideal in RTC since layers could have been removed in the meantime and the index could be out of range.

The index is the index of the layer in the map tool (view), not in the list of layer in the shared document.
This index is computed when the layersTree change, accordingly to the position of the layer in possible nested groups.

Got it, thanks!


typo fix in changelog

Thanks!

Coverage remained the same at 100.0% when pulling f4bd228d7a36097efdc8683dc16ac359e53564a2 on appatoir:patch-1 into 7952e81b6062b3469dfb303591f105b2dfc74fc8 on zemirco:master.


$releasever instead of splitting platform_version

I think it's better to let YUM figure it out than to compute it with Ohai.

+1


Cocoapods is still dispensing v3.1.3

SwiftyJSON on CP

Fix: amend release 3.1.4 with the correct version in SwiftyJSON.podspec.
Note: Sorry couldn't create a PR as the fix should be applied to a release. Please correct me if I am wrong.

I second this post!

Sorry for late reply. I had just pushed 3.1.4 to CocoaPods. Feel free to hit me up if there is still an issue. Closing. 🚢


Precision Error

Looks like all numbers are coersed to Number inside, which is not exactly correct if you parse strings.
numeral('1.0000000000000000000001').format('0.00000000000000000000000000')
gives "1.00000000000000000000000000" instead of "1.00000000000000000000010000"

numeral(0.3).format("0,0.00000000000000000") gives 0.29999999999999999
which is incorrect.
Please fix this.


Search: using ripgrep ignores any external file that I have open

This is a regression from compared to not using rip grep: any file that is opened is considered when searching even if the file is not loaded. To reproduce:

open a out of workspace file
click on another file inside the workspace
reload window
search for a term that is inside the out of workspace file

=> ripgrep will not find it, but the classic search does

Works fine.
@sandy081 @roblourens I wonder if we should remove a search result once you close the editor. Of course only for files which are not inside the workspace. Once you retrigger the search after closing such a file, the result is not showing up again.


Changing minor tolerance to pass linalg_grad_test on ppc64le

Fixes: https://github.com/tensorflow/tensorflow/issues/19935
Thanks!

@martinwicke  Could you please review this PR.

@rmlarsen As per your suggestion I have updated the PR , please review. Thanks !

Nagging Assignee @martinwicke: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@rmlarsen, Can you please review the updated pull request ?

Nagging Assignee @martinwicke: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @martinwicke: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Fixed in https://github.com/tensorflow/tensorflow/commit/422158776bcd9ffbde485610fdd3af498a2d5669, hence closing.


Fix nightly build failure (Added RunIgnored)

The failure was caused by https://github.com/rust-lang/rust/pull/55053
Closes #129

I'm not sure about the failure on AppVeyor, it seems unrelated?
Also Travis CI doesn't seem to pick up the PR


Exception in sample App.

Just run sample. Samsung Galaxy S5. Android 5.0
03-19 17:16:34.569 E/mono    (30193): Unhandled Exception:
03-19 17:16:34.569 E/mono    (30193): System.InvalidOperationException: Collection was modified; enumeration operation may not execute.
03-19 17:16:34.569 E/mono    (30193):   at System.Collections.Generic.List1+Enumerator[AltBeaconOrg.BoundBeacon.Beacon].MoveNext () [0x00000] in <filename unknown>:0  03-19 17:16:34.569 E/mono    (30193):   at System.Linq.Enumerable.All[Beacon] (IEnumerable1 source, System.Func`2 predicate) [0x00000] in :0
03-19 17:16:34.569 E/mono    (30193):   at AndroidAltBeaconLibrary.Sample.MainActivity+<>c__DisplayClass20.b__1c () [0x00003] in c:\Users\POLKOVNIK\Documents\Android-AltBeacon-Library\Samples\Android\AndroidAltBeaconLibrary.Sample\MainActivity.cs:259
03-19 17:16:34.569 E/mono    (30193):   at System.Threading.Tasks.TaskActionInvoker+ActionInvoke.Invoke (System.Threading.Tasks.Task owner, System.Object state, System.Threading.Tasks.Task context) [0x00000] in :0
03-19 17:16:34.569 E/mono    (30193):   at System.Threading.Tasks.Task.InnerInvoke () [0x00000] in :0
03-19 17:16:34.569 E/mono    (30193):   at System.Threading.Tasks.Task.ThreadStart () [0x00000] in :0
03-19 17:16:34.569 E/mono    (30193): --- End of stack trace from previous location where exception was thrown ---
03-19 17:16:34.569 E/mono    (30193):   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x00000] in :0
03-19 17:16:34.569 E/mono    (30193):   at System.Runtime.CompilerServices.TaskAwaiter.GetResult () [0x00000] in :0
03-19 17:16:34.569 E/mono    (30193):   at AndroidAltBeaconLibrary.Sample.MainActivity+d__24.MoveNext () [0x0004c] in c:\Users\POLKOVNIK\Documents\Android-AltBeacon-Library\Samples\Android\AndroidAltBeaconLibrary.Sample\MainActivity.cs:257
03-19 17:16:34.569 E/mono    (30193): --- End of stack trace from previous location where exception was thrown ---
03-19 17:16:34.569 E/mono    (30193):   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x00000] in :0
03-19 17:16:34.569 E/mono    (30193):   at System.Runtime.CompilerServices.TaskAwaiter.GetResult () [0x00000] in :0
03-19 17:16:34.569 E/mono    (30193):   at AndroidAltBeaconLibrary.Sample.MainActivity+<>c__DisplayClass7+<b__5>d__9.MoveNext () [0x00031] in c:\Users\POLKOVNIK\Documents\Android-AltBeacon-Library\Samples\Android\AndroidAltBeaconLibrary.Sample\MainActivity.cs:184
The thread 'Unknown' (0x7) has exited with code 0 (0x0).

Thanks for the heads up @MAX-POLKOVNIK. I'm improperly modifying the collection while iterating over it.

@MAX-POLKOVNIK - Just pushed a fix to the Xamarin.Android sample. Hope that helps.


Add core-js

15.1k stars on GitHub: https://github.com/zloirock/core-js
Example website: https://www.bbc.com/

Thanks!

It does not work with actual versions. See https://github.com/zloirock/core-js/blob/master/scripts/usage.mjs#L33-L35 and https://gist.github.com/zloirock/7331cec2a1ba74feae09e64584ec5d0e


Nad lib Squashed

https://github.com/nephio-project/nephio/pull/67

https://github.com/nephio-project/nephio/issues/53

/approve

/lgtm


python310Packages.sabctools: 7.1.2 -> 8.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.sabctools is: C implementations of functions for use within SABnzbd
meta.homepage for python310Packages.sabctools is: https://github.com/sabnzbd/sabctools
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.0.0 with grep in /nix/store/is6c5wmax0dxgdqpvanwkcixxvmz4xxl-python3.10-sabctools-8.0.0
found 8.0.0 in filename of file in /nix/store/is6c5wmax0dxgdqpvanwkcixxvmz4xxl-python3.10-sabctools-8.0.0

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.sabctools
python311Packages.sabctools
sabnzbd

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/e81e8bdd723a083371743a1438f4bc13a8e6dcc6#python310Packages.sabctools

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

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

3 packages built:

python310Packages.sabctools
python311Packages.sabctools
sabnzbd

Maintainer pings
cc @adamcstephens for testing.

This will require sabnzbd 4.2.0 to be released. It is currently in Alpha.

Oops, reverting.

Sorry for not seeing your comment.

No problem. I wish the tests would run on this, but they don't because sabnzbd depends on the unfree unrar. :(


Cleanup Tests

Tests have been updated to be a bit more consistent. There are no functional changes.

Hi @MarkEWaite do you have some time to check my answers/questions to your review, so we could get this pull request done?


Foundry script breaks on Base network for certain transactions signed with Ledger (error: chainId 0)

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (bff4ed9 2023-08-29T00:32:25.634676000Z)
What command(s) is the bug in?
forge script
Operating System
macOS (Apple Silicon)
Describe the bug
When using forge script to deploy on the Base network, on certain transactions its sending chainId 0 to ledger, which makes it fail with error:

==========================

Chain 8453

Estimated gas price: 0.109494488 gwei

Estimated total gas used for script: 5370493

Estimated amount required: 0.000588039381342584 ETH

==========================

###
Finding wallets for all the necessary addresses...
##
Sending transactions [0 - 2].
2023-09-12T11:21:32.611137Z ERROR coins_ledger::transports: Received error from device err="Ledger device: APDU Response error `Code 6985 ([APDU_CODE_CONDITIONS_NOT_SATISFIED] Conditions of use not satisfied)`"
2023-09-12T11:21:32.611366Z ERROR ethers_signers::ledger::app: error=Ledger device: APDU Response error `Code 6985 ([APDU_CODE_CONDITIONS_NOT_SATISFIED] Conditions of use not satisfied)`

Transactions saved to: /Users/SomeContractPath/8453/run-latest.json

Sensitive values saved to: /Users/SomeContractPath/8453/run-latest.json

Error:
Failed to sign transaction

Context:
- Ledger device: APDU Response error `Code 6985 ([APDU_CODE_CONDITIONS_NOT_SATISFIED] Conditions of use not satisfied)`
make: *** [deploy-payloads-controller-chain] Error 1

Have not been able to isolate the cases for when this happens. But provably similar problem to this issue: https://github.com/foundry-rs/foundry/issues/4362

Hi @sendra thanks for reporting. Have you experienced issues around this more recently? We've since migrated from Ethers to Alloy. If so, would greatly appreciate a minimal reproduction if possible.
The root cause of #4362 is still open unfortunately: https://github.com/LedgerHQ/app-ethereum/issues/409

@sendra we're going to optimistically close this one as an workaround is already implemented in alloy. Please reopen if still an issue. Thank you!


ffq crashes when run has no data

Example
ffq -t GSE GSE75790
Should handle
INFO Parsing run SRR3729917
ERROR 'NoneType' object has no attribute 'text'

Gracefully and should continue parsing the rest of the runs instead of stopping.

Turns out the SRX was located in a different place for this SRR. Still need to handle the case where there are Nonetypes for the Study/Run/Experiment

Crash fixed df90a671bc0ad049c0545f7fd4eb533004858011.

Merged to master https://github.com/pachterlab/ffq/pull/10


atom_feature="symbol" is only available for class GCPNGeneration() ?

     ...
    @torch.no_grad()
    def generate(self, num_sample, max_resample=20, off_policy=False, max_step=30 * 2, initial_smiles="C", verbose=0):
        is_training = self.training
        self.eval()
        graph = data.Molecule.from_smiles(initial_smiles, kekulize=True, atom_feature="symbol").repeat(num_sample)
     ...

Is there any use for limiting the graph features during generation to symbols?
Does this mean that generation will not work for anything other than symbols?
Actually, reonforcement_learning can finish in less than a long time for "symbols."
But, it can spend long time with, for instance, "explicit_property_prediction".
I do not know the reason...

I noticed that, when splitting up finalized molecular graphs into subgraphs, there is no refeaturization in the torchdrug implementation of GCPN.
For some features this can be troublesome, e.g. number of implicit hydrogen atoms, as these node features depend on neighbourhood of a node.
As such, I think that you should be careful about which features you include in the node embeddings.


Sync upstream

Sync to 2d0d2676fee333235367c0f53f69206e769f784d.

The windows 8.44 build is failing but that's beyond our control - something to do with happy. Landing anyway. If it persists into the future, I'll take it out of the build matrix.


Cleanup cirque_tests.sh

Problems

cirque_tests.sh is a bash script, but doesn't take advantage of bashisms
cirque_tests.sh uses sudo

Summary of Changes
correct the above

Fixes #3168

@saurabhst @jelderton @hawk248 @BroderickCarlin ?


fix: case sensitivity on ChargeType enum

Closes #243

Codecov Report

Merging #244 (223050d) into master (7a1be27) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #244   +/-   ##
=======================================
  Coverage   99.68%   99.68%
=======================================
  Files          37       37
  Lines        2223     2223
  Branches      116      116
=======================================
  Hits         2216     2216
  Misses          5        5
  Partials        2        2

Impacted Files
Coverage Δ

src/sections/finances/codec.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Certain nodes are only seeing service agreement bids of a specific ask (ie, node is only seeing .17 publishes when .24 publishes are also being offered)

Issue description
Random nodes on the network are only seeing 1 range of ask offers instead of all ranges. This affects multiple node operators.
Expected behavior
Nodes are able to bid on all service agreements regardless of ask price. If there are publishes for .17 and .24 then if the node has an ask of .24, it should be eligible to earn the .24 pubs and not only get stuck with the .17 pubs it will never earn anyway.
Actual behavior
The node only sees .17 publish attempts and does not have the opportunity to be r0 for .24 pubs. vice versa a node only sees .24 and does not see .17.
Steps to reproduce the problem

grep "service agreement bid"
See attached screenshot. The node on the right (green) can see only pubs with 0.24-0.25 bids (do we expect to see service agreement bid records in case bid is below node ask? i'd assume yes)
node on the left (red) can see only pubs with 0.17-0.18 bids and see zero 0.24-0.25 bids

Specifications

Node version: 6.0.6
Platform:
Node wallet:
Node libp2p identity:

Contact details

Email: Node Alliance

Error logs
.24 only node:
Dmitry_only_0.24_pubs_node.txt
.17 only node:
Dmitry_only_0.17_pubs_node.txt
Disclaimer
Please be aware that the issue reported on a public repository allows everyone to see your node logs, node details, and contact details. If you have any sensitive information, feel free to share it by sending an email to<EMAIL_ADDRESS>

I'm seeing the same behaviour on my nodes.
BRX, BRX88, BRX89 are only seeing 0.24 bids
BRX86, BRX87 are only seeing 0.17 bids
All 5 of them have a service ask of 0.24 but only the first 3 mentionned are able to win any commits.
This started happening about 30 hours ago. It's as if nodes are split into 2 groups, ones that can only see 0.17 service bids and the other group only 0.24.
otnode_log_BRX.txt
otnode_log_BRX86.txt
otnode_log_BRX87.txt
otnode_log_BRX88.txt
otnode_log_BRX89.txt

only 0.24 nodes:
79	QmNqdyK4dAZg97Dy8XRZ8VAUppLJeBJdpdkUq1u8iowuUS	Amado2
78	QmUSHCxxisiGipuMNqigcU64YgwiTjU9nTnvZkPPtWGeBy	        Amado3
80	QmPx8gKqABFQuzYTHD7Ka9uvbFvmg7hfezU8BYcUahvVUC	Amado4
only 0.17 nodes:
77	QmQSmxjJL38oT6D8UHoyJzg9zhFmUtteH1uepFUZVUPKgD	Amado1

only .0.17 node:
QmSjLxW54uyPTKtJ3LZcNCmjZReKA2PyieHuuGKkkhA5vT
My only node :)

I have the same issue
Qmb44HMCYoQPkNfxxVE6rDtYv3uoKZJkp52cpfQi72BXqD only sees jobs with approx 0.17
QmTjs8wGS297NsQ9B3D42E11dUWAR8P3XsikvuKkJy8qFD only sees jobs with approx 0.24

I have the same issue, as described by CosmiCloud
Identity ID: 51
Network ID: QmcmsZ76Yrm3sf6jZx7SX83W3CnWwadfYQ6YerefWAZuuH
Zhe node is only seeing 0.17 bids.
My other three nodes are functioning as expected.

Thanks for the report, we're investigating it. I will get back to you with our findings or if some additional information is needed from you


rpcclient: SendRawTransaction error codes

This PR adds a new error type SendRawTransactionRejectError that is returned from the SendRawTransaction command in case the JSON error we receive from the RPC server contains a well know RejectCode. It is used to wrap this reject code, such that callers more easily can inspect the reason for rejection.
TODO:

[ ] bitcoind compatability?

Replaced by https://github.com/lightningnetwork/lnd/pull/594


[WIP] JSON Serialization: Parse responses fully

Summary
This is a very early snapshot showing what I'm thinking we could do with JSON Serialization. We already have some of the responses from core being parsed, but in many cases the deeper we get into the JSON structure, the more likely we just punt a [String: AnyObject] out to the rest of the codebase.
My thinking is that we'd never let any code other than the RPCSending know about the JSON structures coming from core. They'd all be parsed into strongly-typed structs or enums.
Note: I just realized I've got my Xcode settings to use tabs instead of spaces. I'll fix that shortly.

Related Issues
This is somewhat related to https://github.com/xi-editor/xi-mac/issues/102. However, I don't see this adding too much in the way of performance. I see this as a way to make the xi-mac codebase cleaner and less fragile.

Checklist

[ ] Ensure we don't pass any dictionary-like JSON objects out of the network "layer" of xi-mac
... more here eventually :)

Review Checklist

[ ] I have responded to reviews and made changes where appropriate.
[ ] I have tested the code
[ ] I have updated comments / documentation related to the changes I made.
[ ] I have rebased my PR branch onto xi-mac/master.

It's very great to have you contributing this to xi. I look forward to learning from your work @jeremywiebe !

Oh good catch! Thanks for pointing that out.
On Tue, 19 Mar 2019 at 21:43, Ryosei Shinoki<EMAIL_ADDRESS>wrote:

@rshinoki commented on this pull request.
In Sources/XiEditor/Core/Types.swift
https://github.com/xi-editor/xi-mac/pull/444#discussion_r267184026:

+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.

+import Foundation
+
+enum UpdateOperationType: String {

typealias RawValue = String

case Copy = "copy"
case Invalidate = "invalidate"
case Insert = "ins"
case Update = "update"
case Skip = "skip"

The coding style of the enum case should be lower camel case. In Swift
guideline and other companies' coding style, enum case is lower camel case.
https://docs.swift.org/swift-book/LanguageGuide/Enumerations.html
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xi-editor/xi-mac/pull/444#pullrequestreview-216525948,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEtUlhZ5ADuCFfNIjaScWODe8kbhQ-Oks5vYbyKgaJpZM4bhRV2
.

--
Jeremy Wiebe
<EMAIL_ADDRESS>

@jeremywiebe Let me know if you want a review, as I might be doing stuff which could introduce some more work for you :)

@nangtrongvuon I don’t feel like I’m fully settled on how I’m going to be organizing everything yet. But feel free to post comments on what you see so far. Based on what you’re planning it might change how I end up doing this. :-)

I think this looks good, thanks a bunch @jeremywiebe for taking this on. @mmatoszko happy to have you take a closer look this week; I'll do another final pass when you're satisfied, but I don't expect to have any issues.

Thanks for the review @cmyr and @mmatoszko!


TypeScript version of Contoso Product

hey @garrytrinder this is ready to be reviewed and merged. CC @BobGerman

Apologies for the delay on this @rabwill,
Thanks for making the changes in from the previous comments, there are a few things that I think we need to address before we merge.
I was able to get this working fine in Teams, but when using it in Copilot, I was unable to go through the authentication flow (when I click on the Sign in button) it does nothing.

Thanks @garrytrinder I have addressed most of the issue you have raised except testing in copilot. I need to repro this before I can fix the issue. Thanks for your patience!


#853 - Allow calling 'scope' on an existing ScopeBuilder object.

This fix prevents an UnsupportedOperationException being raised by the implementation of the wrapped List upon calls to the scope method of ScopeBuilder.

@mdsell nice, thanks for contribution!


option relax "wrap the /regexp/ literal in parenthesis" warning

/[0-9]/.exec('123') throwing a warning is pretty annoying.

Fixed in #1433


Extensions for F40

The following need to be done for F40:

logomenu
gsconnect
tailscale QS
caffeine?
dashtodock?

And the following are good to add once supported:

blur-my-shell

These are all in bazzite so we'll get those updates, just placeholding them here.

[x] logomenu
[ ] caffeine
[ ] tailscale qs

Unless I'm missing something, caffeine and Tailscale QS are already on the image.
Have PRd blur-my-shell since it's now being successfully installed

Have loaded a VM and can confirm all except GSConnect and Tailscale are working:
Blur:

![image](https://github.com/ublue-os/bluefin/assets/46304672/581865d6-81a0-4faf-af98-3a6fbc33cfd9)

Caffeine:

![image](https://github.com/ublue-os/bluefin/assets/46304672/e3d35114-bb0f-4673-b475-964eb87a2852)

Tailscale:

Logo Menu:

Dash to Dock:

GSConnect:

They are all on the image, but some incompatible with the current GNOME version.  This will sort itself out on it's own, but will leave this issue open.

Confirmed the above with fresh iso install on bare metal.

Tailscale QS appears in my quick settings now.
GSConnect has been updated.


Random files deleted when there are lots of files

I'm experiencing a random cancellation of files (only locally luckily) of files already pulled.
I have a folder with more than 400 files and above this number the pull command deletes legitimate files pulled in previous downloads. I can easily do a pull of that particular file deleted but it is still annoying.
The file deleted doesn't seem to follow a pattern, it is just one of the already downloaded files.
Thanks

Sure, sorry, I did in the other issue I opened which I guess it could be correlated
I'm using:
drive version: <IP_ADDRESS>
Commit Hash: '8c26f34b24e0e681acfaffee4149edbbd0aec6a2'
Go Version: go1.7.5
OS: linux/amd64
BuildTime: 2017-02-03 22:13:30.854342991 +0100 CET
on Manjaro Linux


Fix: Update social share text

What changed? Why?

removed link from social share text to ensure URL metadata is to user's name

Notes to reviewers
How has it been tested?

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


Link to skip nav page is broken

On https://ux.redhat.com/components/, the link to the Skip navigation page is currently https://ux.redhat.com/components/skip-nav. It should be https://ux.redhat.com/components/skip-navigation/.

Fixed in #246


Add built-in column-specific transformers to TableVectorizer

Fixes part of #554
As discussed in #580, adds the column_specific_transformers parameter to the TableVectorizer.
This allows this kind of functionality:
from skrub import TableVectorizer, MinHashEncoder
from sklearn.compose import make_column_transformer

make_column_transformer([
    (
        MinHashEncoder(),
        ["PRODUCTTYPENAME"],
    ),
    remainder=TableVectorizer(),
])

from the TableVectorizer directly, with this syntax:
TableVectorizer(
    column_specific_transformers=[
        (MinHashEncoder(), ["PRODUCTTYPENAME"])
    ],
)

When the assignements need to be named (e.g. for a grid-search), the user can specify a name (same syntax as the ColumnTransformer):
TableVectorizer(
    column_specific_transformers=[
        ("mh_product_type", MinHashEncoder(), ["PRODUCTTYPENAME"])
    ],
)

You have failing tests. Can you address them please

Thanks for reminding me :)
The PR should be ready to be reviewed/merged now. The implementation is complete and the tests are done.

The grid-search doesn't work as expected, there is something different between the ColumnTransformer and the TableVectorizer, but I don't know what yet. I think you had a concerned about set_params @glemaitre?

So the feature has been ready for a while, but this example is blocking. Since it doesn't work and needs some more work, I've put it into a temporary directory, so it's not accessible on the website, but so we don't lose the code. I'll open an issue in a few minutes to fix that.


Add new method in Headers.kt to allow non ASCII chars in both header …

I'm working on a fuzzing tool called CATS: https://github.com/Endava/cats that use okhttp as underlying http transport library. Multiple HTTP headers fuzzers will send different non-ASCII characters in both header names and values. Other fuzzers are prefixing/suffixing header values with spaces. The current addUnsafeNonAscii() doesn't allow non-ASCII in header values and also does trimming of values. This PR introduces an additional method that allows non-ASCII in both header names and values and doesn't do any value trimming.

I’m sorry to say that OkHttp is absolutely not designed to be used as a fuzzer at the HTTP layer, and you’re going to have a bad time if you want to use it like that. We want to send well-formed HTTP requests, and want to avoid APIs that create malformed HTTP requests, even if you’re doing so deliberately.
That said, it is totally valid to use OkHttp to fuzz at the application layer. For example, to test for SQL injection or with request bodies that use invalid JSON.
On this PR specifically – most servers split the HTTP layer (implemented by nginx, Jetty, etc.) from the application layer. Are you interested in fuzzing a specific layer?

... I just read the linked GitHub project and it looks like you’re working at the application layer. That’s good news for both of us.
I don’t want to brick-wall this PR, but I do want to advise that we’re disinterested in following this to its logical conclusion. For example, OkHttp is not going to be very helpful if you wanna send say, malformed HTTP/2 frames.

Mind updating HeadersJvmTest.kt ?

Yes, apologies, missed that.

(Sorry for not flagging that sooner.)

I will probably close this for now. I reached additional cases which will require OkHttp to relax some constraints, which is probably not compatible with how OkHttp was designed. I'll might raise a future PR.


Feat/ipfs image upload writeup

Created document writeup on image upload

@doriancrutcher can you check, all the files you added are empty? 🤔


[Snyk] Security upgrade python from 3.11.7-slim-bullseye to 3.11.9-slim-bullseye

Snyk has created this PR to fix 2 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

Dockerfile

We recommend upgrading to python:3.11.9-slim-bullseye, as this image has only 75 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Integer Overflow or Wraparound SNYK-DEBIAN11-ZLIB-6008961
  264

CVE-2024-37371 SNYK-DEBIAN11-KRB5-7411316
  232

CVE-2024-37371 SNYK-DEBIAN11-KRB5-7411316
  232

CVE-2024-37371 SNYK-DEBIAN11-KRB5-7411316
  232

CVE-2024-37371 SNYK-DEBIAN11-KRB5-7411316
  232

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

https://github.com/equinor/sumo-fmu-to-resqml/commit/2acdee41b20f2a4adae32b8d000ae5988ecda44e closes


Miss icons in the 'Manage Access' dialog for one ADLS Gen2 blob container

Storage Explorer Version : 1.29.0-dev
Build Number : 20230327.1
Branch: main
Platform/OS: Windows 10
Architecture: ia32
How Found: From running test cases
Regression From: Previous release (1.28.1)
Steps to Reproduce

Expand one ADLS Gen2 storage account -> Blob Containers.
Right click one blob container -> Click 'Manage Access Control Lists...'.
Check whether no icon is missed in the 'Manage Access' dialog.

Expected Experience
No icon is missed in the 'Manage Access' dialog.

Actual Experience
Miss icons in the 'Manage Access' dialog.

Additional Context
This issue does not reproduce on Linux Ubuntu 22.04/MacOS Ventura 13.2.1 (Apple M1 Pro).

Verified this issue on the build main 20230328.3. Fixed. So, we are going to close it.


[OC-3798] Upgrade Cloudera theme for Hawthorn.1

Applies fixes related to template changes for Hawthorn.

Wrong branch..


Support for High Availability

Love this repo! Would be nice if this supported a high availability set up of pi-hole. Was considering trying to do this myself but think it's a little over my head with my current ansible experience. This can be accomplished using gravity sync and keepalived.
Basically think there would be a primary host option and a list of secondary hosts, each host would be configurable to have the specific services you want, with keepalived being optional for tieing the host to a vip, and gravity sync being optional to keep the instances synced.
https://github.com/vmstan/gravity-sync
https://keepalived.readthedocs.io/en/latest/introduction.html

At some point I may work on Pi-hole HA, but probably not as part of this particular repo—I think it's a little more specialized and use-case-specific than the goal of this project, to be a one-pi-does-it-all solution.


Layout of landing page: suboptimal responsiveness

For certain devices the layout will not be optimal.

no one seems to be working on this. Closing. Please feel free to reopen.


Converting Graph to matplotlib graph or Saving graph as a image

Hi,
I want to convert the graph into the matplotlib graph format or save the graph into the image, How can I achieve this because the current visualization function is not returning enough information to do so. Getting error while accessing the graph via rasterio. i.e.  TypeError: Image data of dtype <U44560 cannot be converted to float
Another error that I got while plotting charts[0] is :-   AttributeError: 'ChartExecutorResponse' object has no attribute 'savefig'

@victordibia can you have a look into this :)

Hi @centa-furqan ,
The ChartExecutor response is a text based representation to make serialization straightforward (send results over a web api). The raster field is a base64 encoded string.
You can convert it to an image and save as follows:
import base64
with open("chartn.png", "wb") as fh:
    fh.write(base64.decodebytes(charts[0].raster.encode()))

Overall,  thanks for pointing this out. I'll add a savefig method to the ChartExecutor object.


modify the configuration about crio.image

Thanks! Can we have separate
crio/install.sh
crio/install-zh.sh
as well?
Then, perhaps we can have separate CIs for the zh versions of containerd and crio. Thanks!

Thanks! Can we have separate
crio/install.sh crio/install-zh.sh
as well?
Then, perhaps we can have separate CIs for the zh versions of containerd and crio. Thanks!

Of course. I will submit it again when i'm done.

Great. Mark this PR as draft please. When it is ready, we will review and merge. Thanks.

Great. Mark this PR as draft please. When it is ready, we will review and merge. Thanks.

Yeaph, I have separated CI for the zh versions of crio as well. Could you approve the github action for me?Thanks.


Design Meeting Notes, 12/10/2021

"Correlated Union Types"
https://github.com/microsoft/TypeScript/issues/30581
type NumberRecord = { kind: "n", v: number, f: (v: number) => void };
type StringRecord = { kind: "s", v: string, f: (v: string) => void };
type BooleanRecord = { kind: "b", v: boolean, f: (v: boolean) => void };
type UnionRecord = NumberRecord | StringRecord | BooleanRecord;

function processRecord(record: UnionRecord) {
  record.f(record.v); // error!
 // error msg in TS3.2 and below: can't call union of functions
 // error msg in TS3.3 and up: record.v not assignable to never
}

We want to be able to feed the callback with the original value; but now v is a union type and f's parameter type is an intersection; that's not precise for v, and that intersection now only accepts never.

People have speculated that maybe we need existential types so that we can say "this function takes whatever type v has" and can feed it through.

Can try to re-model this in the form of a re-indexed mapped type.
interface TypeMap {
    "n": number;
    "s": string;
    "b": boolean;
}

type UnionRecord<P extends keyof TypeMap> = { [K in P}]: {
        kind: K;
        v: TypeMap[K];
        f: (p: TypeMap[K]) => void;
    }
}[K];

function processRecord<K extends keyof TypeMap>(record: UnionRecord<K>) {
    record.f(record.v);
}

With some improvements of inference and apparent types, we can enable most of these cases - 2 improvements on the scale of what feel like bug fixes.

First is inference to the IIFE-like mapped object pattern from above.
Second is how we present the apparent members of certain types when you index into an already-instantiated mapped object type (a "table type") with some generic key.

{ [P in K]: Foo<P> }[X] => Foo<X>
Does that mean that if you aren't using a type that was declared as a mapped type, then we won't get a more precise type?

Yes, if you leave off types on these objects.

I guess we don't have an easy way to correlate.
We could try to do something there...but that's a bit of a stretch.

This is a weird pattern. Would like to not have to teach people to write this.

Modules Migration Status Update
https://github.com/microsoft/TypeScript/pull/46567

A lot of time running tests.
A lot of time is going into getters for every single binding.
Had TypeScript generate to CJS, had esbuild run on top of that.
Swapped TS to generate ESM, and esbuild generates the same sorts of bindings - so no optimization there.
25 minutes to run the full tests! Thanks to the multiple levels of indirections of calling functions.
Could switch to ES3 emit to avoid accessors.
Also could switch to native ESM emit.

No telling how slow ESM live bindings would be.

Could avoid indirect barrel imports.
Looked at bundler front-ends that are largely esbuild wrappers.

But these tend to push for the optimized build to use Rollup.

[[Whole discussion on what the right shipping format and layout to send.]]
Dogfooding with modules showed us just how slow this is.

Even getting complaints on tslib: https://github.com/microsoft/tslib/issues/165

Conclusion

Investigate the __createBindings issue.
Investigate alternative bundlers.

Ensuring Modules Conform to a Type
https://github.com/microsoft/TypeScript/issues/38511

Lots of people would like to be able to ensure "this file has this sort of shape."

Then "I would like this folder to describe this shape".

export implements SomeType that
PR suggests everything should be contextually typed.
You can emulate this today with a self-import.

Without contextual typing, this is possibly just sugar that saves you 2 lines. Maybe not as compelling.

This links to typing default exports - is that the real feature?
Out of time - revisit in the new year.

PR up for correlated union types is up https://github.com/microsoft/TypeScript/pull/47109

What is the difference between #46567 and #35561?

#46567 builds on #35561

Hello! I'm the developer behind esbuild. I noticed that you're running the TypeScript compiler through esbuild, so I couldn't help trying it out myself. I've been meaning to do cross-module enum inlining so I did a quick implementation and tried it out on the branch in #46567. Here's a simple benchmark of the result:

lib/tsc.js
esbuild 0.14.6
esbuild 0.14.7
fixes + esbuild 0.14.6
fixes + esbuild 0.14.7

Type check time
2.96s
5.31s (1.79x)
4.94s (1.67x)
3.45s (1.17x)
2.95s (1.00x)

These tests measure the time to type check the Rollup code base (with fixes for type errors due to the TypeScript upgrade). As you can see, bundling with esbuild after inlining enums results in a TypeScript compiler build that's the same speed as the one in lib/tsc.js.
But I had to fix a bug in that branch that was slowing things down a lot (the "fixes" columns in the table):
diff --git a/src/compiler/corePublic.ts b/src/compiler/corePublic.ts
index 11c5416b54..4334edad7f 100644
--- a/src/compiler/corePublic.ts
+++ b/src/compiler/corePublic.ts
@@ -138,6 +138,7 @@ namespace NativeCollections {
     export function tryGetNativeMap(): MapConstructor | undefined {
         // Internet Explorer's Map doesn't support iteration, so don't use it.
         // eslint-disable-next-line no-in-operator
+        const Map = globalThis.Map as any;
         return typeof Map !== "undefined" && "entries" in Map.prototype && new Map([[0, 0]]).size === 1 ? Map : undefined;
     }

@@ -147,6 +148,7 @@ namespace NativeCollections {
     export function tryGetNativeSet(): SetConstructor | undefined {
         // Internet Explorer's Set doesn't support iteration, so don't use it.
         // eslint-disable-next-line no-in-operator
+        const Set = globalThis.Set as any;
         return typeof Set !== "undefined" && "entries" in Set.prototype && new Set([0]).size === 1 ? Set : undefined;
     }
 }

This code was picking up on the local versions of Map and Set instead of the global versions, so it was always installing the shim version and never using the native version. Using the native Map and Set is a 50% speedup. I assume you'll figure this out too if you haven't already, but I thought I'd save you the trouble in case it's helpful.
Anyway just wanted to share this result and bug fix. It's totally up to you whether or not you use esbuild, so no pressure at all. I imagine there are concerns about wanting to dogfood the TypeScript compiler.  The only catch is that we don't always have globalThis defined in older JS versions, so I think you'll have to try something like
const localGlobal =
    typeof globalThis !== "undefined" ? globalThis :
    typeof global !== "undefined" ? global :
    typeof window !== "undefined" ? window;
const Map = localGlobal.Map as any;

I think one other thing to consider is that this is something the conversion script kept working, but silently wrong. Seems like nested namespaces and places where we use local declares might be something to watch out for.

@evanw Thanks for that and sorry that I got to it so late.  I'm guessing that you were running esbuild directly on the TS sources--?  I'm using it with the CJS output of the compiler, and that has some differences in runtime -- specifically, my concern was the time it takes to run the full set of TS tests...  First, I didn't see any change when bumping the version (which is probably expected, given that it's the cjs output).
Second, the Map/Set bug didn't actually affect the cjs output since it doesn't create a global but uses exports.Map directly -- I did try at some point to run esbuild directly on the ts sources which was very slow, so I'm wondering if that could have caused some of the cost (especially if comes on top of the getter cost).  In any case, it is definitely a bug, and I just opened #47409 for that.

I'm guessing that you were running esbuild directly on the TS sources--?

Yes, that's correct. I did that because you need to run esbuild on the TS sources to take advantage of TS features such as enum inlining, so that's the fastest way to run the code.

RE: #38511

You can emulate this today with a self-import

Can you explain this with an example please? I don't fully understand how importing self can achieve the same thing as proposed.

This links to typing default exports - is that the real feature?

Apparently export default: Type <thing> syntax was introducing so much syntax ambiguity that this request has been stalled (https://github.com/microsoft/TypeScript/issues/3792).  With export implements we can type the default export without limitations mentioned there:
export implements { default: number };
export default 1

Can you explain this with an example please? I don't fully understand how importing self can achieve the same thing as proposed.

In theory you can write
import * as self from "./self.js"
const _: ExpectedType = self;


Documentation conflict for automatic creation of pull secrets

What happened?
Using the information detailed in https://www.devspace.sh/docs/configuration/images/pull (6.x, latest) that say to use the field createPullSecrets to disable automatic creation of pull secrets, pull secrets are still created.
The 5.x docs https://www.devspace.sh/docs/5.x/configuration/images/pull-secrets say to use the field createPullSecret, which does disable automatic creation of pull secrets.
What did you expect to happen instead?
Documentation that indicates createPullSecrets should be updated to the working createPullSecret.
How can we reproduce the bug? (as minimally and precisely as possible)
Setting the plural setting createPullSecrets to false when running a build will still create pull secrets. Using the singular version will prevent creation.
My devspace.yaml:
version: ...
...

Local Environment:

DevSpace Version: 6.3.2
ARCH of the OS: AMD64
Kubernetes Cluster:
Cloud Provider: google
Kubernetes Version: v1.25.11-gke.1700

Anything else we need to know?
This can easily lead to leaked credentials of a user that believes they've taken measures to prevent their creds from being uploaded to a remote server.

@LISTERINE Hello! Thanks for reporting this. We'll get the documentation updated.
FWIW, the schema validation should fail and prevent the secrets from being leaked. At least that's what I observed when trying it with this devspace.yaml:
version: v2beta1

images:
  default:
    image: myregistry.foo.bar/myusername/devspace
    createPullSecrets: false

Output:
info Using namespace 'pull-secrets'
info Using kube context 'kind-kind'
fatal error loading config: yaml: unmarshal errors:
  Parsed Config:
    1: images:
    2:     default:
    3:         createPullSecrets: false
    4:         image: myregistry.foo.bar/myusername/devspace
    5: name: pull-secrets
    6: version: v2beta1
  Errors:
    line 3: field createPullSecrets not found in type latest.Image (line 3: createPullSecrets: false)

If you have an example devspace.yaml where this didn't occur please share and we'll take a look!

Thanks for the update!


Update tail.js

Removing the error thrown in "latestPosition()". The application is can't catch this exception so it'll crash. An example scenario would be to tail a file and then delete the file from the system.

Hi @andrewnguyen92 ,
In that scenario, when the library cannot access the underline file anymore, rethrowing the exception is the expected behaviour.
If the exception is just swallowed, tail could stay stuck forever (no file to read but also no exiting).
Can you share your specific scenario where swallow the exception would be better?

Hi Luca, would that case be handled by the previous line that emits the error event? I am running tail on a file that may get deleted and recreated shortly thereafter in the background. Removing this exception throwing and setting up an event handler for my Tail object on the "error" event allows my application to cleanly handle this scenario without crashing. Is there another way for the application to catch this exception?

you app should handle the exception on its side:
try {

 tail = new Tail("fileToTail");
 tail.on("line", function(data) {
  console.log(data);
});

tail.on("error", function(error) {
  console.log('ERROR: ', error);
});
} catch (ex) {
 //check for file to come back first
// create a new instance of tail on the newly created file
}

Hey Luca,
I've tried that previously but that fails to catch the exception. This is what I get when the file is deleted and the application crashes:
ERROR:  size check for ./file.log failed: Error: ENOENT: no such file or directory, stat './file.log'
/home/user/test/node_modules/tail/lib/tail.js:78
            throw err;
            ^

Error: ENOENT: no such file or directory, stat './file.log'
    at Object.fs.statSync (fs.js:948:11)
    at Tail.latestPosition (/home/user/test/node_modules/tail/lib/tail.js:74:23)
    at Tail.change (/home/user/test/node_modules/tail/lib/tail.js:118:22)
    at Tail.watchEvent (/home/user/test/node_modules/tail/lib/tail.js:183:18)
    at FSWatcher.watcher.fs.watch (/home/user/test/node_modules/tail/lib/tail.js:143:101)
    at emitTwo (events.js:126:13)
    at FSWatcher.emit (events.js:214:7)
    at FSEvent.FSWatcher._handle.onchange (fs.js:1364:12)

Looks like the function throwing the exception is called as a callback by fs.watch, so I don't believe there's a way to catch the exception in that case.

tracked with issues #129
Will be fixed with v2.2.1. Tail will emit an error event if a file is deleted.


traj_collection.to_line_gdf() removes some observations

I've a dataset of 965931 rows (points) and grouping by traj_id I have 1489 trajectories.
Then I've used:
traj_collection = mpd.TrajectoryCollection(df_geopandas_long_traj_mp, 'id', min_length=0, crs = 'epsg:32632')
where df_geopandas_long_traj_mp  is the dataset mentioned before.
Using traj_collection.to_line_gdf() I get the correct number of trajectories, 1489, but a smaller number of points (964442).
I've noticed that considering 965931 - 964442 i get the number of trajectories.
How does it work? Why are there missing points?
UPDATE:
I've discovered that it discards the first frame of each trajectory and I don't understand why.
It seems that to_line_gdf() has problems. Is it possible that it skips the first row?

I've discovered that it discards the first frame of each trajectory and I don't understand why

Dear Lorenzo,
Imagine a toy trajectory with n=3 points. When we connect consecutive points, we get n-1=2 lines.
to_line_gdf() creates lines.
to_point_gdf() returns the points.
Hope this helps.


About the reltol parameter

Hello, thanks for your excellent work!
But I have one question.
In model/pred_illuDecomp_layer.py, there is a function pinv(A, reltol=1e-6). I found that if I reserve this line:
s = tf.boolean_mask(s, s>atol)
the shape of s may become (2,) and then the shape of s_inv becomes (2, 2), which will raise error when do
tf.matmul(s_inv, tf.transpose(u))
So I want to know why need to clear entries lower than reltols_max ?
Does it matter if I don't clear entries lower than reltols_max ?
Thank you!

Oh, it maybe because the mask I put into the network is 0,1 form. After I convert it to 255 form, the problem does not happen anymore.