Using the new Zamg Datahub Api?

Is your feature request related to a problem? Please describe.
The problem is, that with the current Api (OGD) there are only a few stations availible.
Describe the solution you'd like
Using the new Api provided by Zamg, would support all stations. (Doku, Example)
Describe alternatives you've considered
I think there is no alternative solution.?
Additional context
Example Request:
https://dataset.api.hub.zamg.ac.at/v1/station/current/tawes-v1-10min?parameters=TL,P,FFAM&station_ids=11035
to request Temperature, Pressure and Windspeed.

Hi,
Thanks for the hint to the new API!
You are right would be a very good improvement to have all stations available in HA.
I will have a look on it.
my base intention was just to update the zamg integration in HA, to have unique ids on the sensors to change icons, names etc. in the GUI.
but the effort to do this got a little bit out of control..
BR

Hi @LuisThe0ne,
The hint to use the new datahub API was pretty good ;-)
i had moved over the zamg client to the datahub API.
Thanks!

Thank you very much for implementing the new data API!
Do you have any plans to also have a look at the forecast API, if they gonna release it Q1 2023 ?

Thank you very much for implementing the new data API!

No Problem!

Do you have any plans to also have a look at the forecast API, if they gonna release it Q1 2023 ?

Currently not, but if it's released we will see ;-)

Now there is some publication for upcoming weather information, but they only distribute it through ftp and as a package for the whole region (45.5 - 49.48 °N, 8.1 - 17.74 °E) in a 1*1km grid.
Here is the link: https://data.hub.zamg.ac.at/dataset/inca_fcst-v1-15min-1km


Acrylic and Glass dont seem to be working

Hi, seems issue related with you. Check if [https://github.com/arshx86/Themer/releases/tag/v2.0](sample app) has the same issue, if yes it can be related with windows settings.


Contact form - how does it work

I'm trying to configure a contact form using a Dimension theme, and I really don't know where I can configure where te message is sent. Anyone can help me with this issue. I assume that all the forms in different themes works same way. I found that there is form.meta but where it is processed? I thought that there should be some settings in config.yml but no..

There's no built-in contact form support in Pico. You'll have to either add a 3rd-party plugin or use some external contact form service.

At content-sample/index.md of the Dimension Theme you can find in the Meta Section a link to an 3rd Party Service (Formbucket)

Great thanks for that I actually didn't notice it :)


Added dot and cos merge modes

Two useful merge modes: dot product merge and cosine merge.

The way i had implemented the shape is not elegant at all :D try something like that instead
 elif self.mode =='dot':
       return tuple(input_shapes[0][0], 1)

@dbonadiman  I included shape checking.. But is it necessary? sum. average, elementwise multiplication all requires vectors of same shape.. but shape checking is not done for any of them. Should not shape checking implemented in general for all these modes? Like:
if mode in ['sum','ave','mul','cos','dot']:
shape=layers[0].output_shape
for l in layers:
if l.output_shape != shape:
raise Exception(mode + " merge requires layers of same output shape")
If yes, then it should be a separate PR

Done. Shape checking has been proposed in a separate PR #881 .
Removing shape checking for dot and cos in this one.

Does the "dot" merge mode cover the case where one side is reshaped to a m x n matrix and the other side is a length n vector? I've found this occasionally useful (though it would scale to larger matrices if we had some way to represent them in a factorized form).

@iskandr  No it doesnt, as of now. Infact, I am waiting for #881 to be merged to add strict shape checking. What you are proposing could be easily accomplished, once this PR is merged. Simply repeat the n vector using RepeatVector layer m times to get a mxn model, this can then be merged with your other mxn model using dot or cos mode.

This is neat. But I think we would need to be able to specify the axis of the merge. For instance, with timeseries (3D tensors), sometimes you will want to dot the features (axis=2), and sometimes you will want to dot the time data points (axis=1).
So I see a need for an axis keyword argument. For a cleaner API, we could remove the concat_axis argument and use the same axis argument for all merge modes that use an axis.
Also, I believe we can implement these new merge modes without scan.

@fchollet  The default value of concat_axis is -1, whereas the default value for axis that makes sense for dot merge is 2. What do we do about this? What about:
if mode == 'dot':
    if axis == -1:
         axis = 2

But it doesnt "feel" right.

 Automatically merge-able PR : #891

Right. The Numpy API allows the axes argument to be an int, a tuple of 2 lists / arrays of same length. In this context it's clearer to use a different keyword.


Unknown Symbol with config file

Hi,
Thanks for developing and publishing this plugin. I can see it being very helpful to our development.
I am trying to use the plugin on Ubuntu 18.04 with the following config file:
https://github.com/ament/ament_lint/blob/master/ament_uncrustify/ament_uncrustify/configuration/ament_code_style.cfg
I changed my user settings to point to the uncrustify executable and to the config file as follows:
 "uncrustify_executable" : "/usr/bin/uncrustify",
  "uncrustify_config" : "/home/freki/rover_wss/ros2_ws/src/simple_rover_locomotion/ament_code_style.cfg",
Unfortunately, I'm receiving the following error:

Using uncrustify from the command line with the same config file does not throw an error and works flawless.
uncrustify -f src/simple_rover_locomotion.cpp -o output.cpp -c ament_code_style.cfg
Using your plugin with a simpler config file also worked.
Do you have any idea where the issue might be?
Thanks a lot,
Miro

hi~ this plugin just run below command with stdin/stdout source content when you run format source
Uncrustify.py line 285: command = [program, "-l", lang, "-c", config]
for your config it will be
/usr/bin/uncrustify -l CPP -c /home/freki/rover_wss/ros2_ws/src/simple_rover_locomotion/ament_code_style.cfg`
You can check sublime's console to see the command log which
Uncrustify.py line 291: print("> " + running + " ...")
I saw your log, the uncrustify already called and ament_code_style.cfg had assigned. but uncrustify return errors about bad config, it should because the uncrustify problem because this plugin just pass the path of config as param to uncrustify, plugin didn't mod/load config...
if you google 'uncrustify retutrn unknow symbol' someones has problem like this because the old uncrustify (maybe the file format of  ament_code_style.cfg?)
please use
'/usr/bin/uncrustify -l CPP -c /home/freki/rover_wss/ros2_ws/src/simple_rover_locomotion/ament_code_style.cfg -f src/simple_rover_locomotion.cpp -o output.cpp'
test again

Hi Obxyann!
Thanks, I discovered that I had multiple versions of uncrustify installed.
With running whereis uncrustify I found that ROS installed a uncrustify here: /opt/ros/eloquent/bin/uncrustify. Changing the config to that made it work in sublime!
Thanks for the pointers!


Conserve type attributes in the union builder

Right now the union builder doesn't keep type attributes for the union members.

This was fixed a while ago.


Update Euro 2012 stats- PLAYERS EXTRA DATA.csv

http://www.uefa.com/uefaeuro/season=2012/statistics/round=15172/players/type=yellowcards/index.html

Thanks again!


[RFC] Configurable popup keepInViewPort logic

Feature Request
The Popup component should take an optional positionFallbacks prop which would be an ordering of positions to fall back on if the chosen position is outside the viewport
Problem description
I have an issue where there is a row of popups across the middle of a page where the position prop is set to bottom center. The popover on the left edge of the page is displaying with a position of top left instead of bottom left which would be closer to the position of all the other popups.
Proposed solution
The Popup component should take an optional positionFallbacks (or similarly named) prop. This prop would be an array of other positions that would override the default ordering of the POSITIONS array used in setPopupStyle().
Example:
const fallbacks = ['bottom left', 'bottom right', 'top center', 'top left', 'top right', 'right center', 'left center', 'bottom center']

<Popup
  position='bottom center'
  positionFallbacks={fallbacks}
/>

Alternatively, logic could be added to setPopupStyle() to select the next best position automatically. I spent some time thinking about a possible set of rules to implement this, but I think any ordering is ultimately somewhat subjective and that it's best to just to let the user set the order they want.

I spent some time thinking about a possible set of rules to implement this, but I think any ordering is ultimately somewhat subjective and that it's best to just to let the user set the order they want.

Agreed.  Due to this, I think it is best to work this into a callback somehow and let the user handle it.  Our history shows there is no end to options and configuration that user's will want for components.  I'm very focused on simplifying component props.
I'd like to support this, but not with more hard-coded logic.  Any ideas?

I think it is best to work this into a callback somehow

Could you clarify what you mean by this? Do you mean integrating this into the onMount prop or some other prop like it?
I can understand wanting to keep the number of props to a minimum, but I think either adding a positionFallbacks prop or modifying the position prop to optionally take an array would be a reasonable solution.

It seems fallbacks are reasonable.  It may conflict with keepInViewport as providing fallbacks seems to infer the user wants the Popup to be kept in the viewport.
@layershifter @brianespinosa any thoughts on this?

If we are specifically trying to use these fallbacks to keep the popup in the viewport, maybe we make keepInViewport enumerated instead of boolean and allow the user to specify fallbacks there? Not sure if there would be a way to reduce those declarations to single strings versus pairs. Then let the viewport determine whether the popup should get a verical/horizontal override based on viewport/popup position.
This is mostly woodshedding. I think allowing position to be an array could be a better solution if we can envision a use case where the user may not be using this solution to keep a popup in the viewport.

I also vote for a way to configure it.
Use case is that I want my popups with the arrow in the "bottom center", but if there is no space at the bottom of the page I would like it to be "top center". But currently there is no way of doing it :(

In the meantime this is what I did.
`import { Popup } from "semantic-ui-react";
import _ from "lodash";
import Proptypes from "prop-types";
class CustomPopup extends Popup {
static propTypes = {
...Popup.propTypes,
positions: Proptypes.array
};
static defaultProps = {
    ...Popup.defaultProps,
    positions: [
        "top center",
        "bottom center",
        "top left",
        "top right",
        "bottom right",
        "bottom left",
        "right center",
        "left center"
    ]
};

setPopupStyle() {
    if (!this.coords || !this.popupCoords) return;
    let position = this.props.position;
    let style = this.computePopupStyle(position);
    const { keepInViewPort, positions } = this.props;

    if (keepInViewPort) {
        // Lets detect if the popup is out of the viewport and adjust
        // the position accordingly
        const preferedPositions = _.without(positions, position).concat([position]);
        for (let i = 0; !this.isStyleInViewport(style) && i < preferedPositions.length; i += 1) {
            style = this.computePopupStyle(preferedPositions[i]);
            position = preferedPositions[i];
        }
    }

    // Append 'px' to every numerical values in the style
    style = _.mapValues(style, value => (_.isNumber(value) ? `${value}px` : value));
    this.setState({ style, position });
}

}
export default CustomPopup;`
it's not perfect but hey it works.
setPopupStyle is the same as the source code except minor tweaks the take positions from the props.

Good news, this feature is already available 🐱

We switched to Popper.JS in #3532<EMAIL_ADDRESS>and added access to popperModifiers<EMAIL_ADDRESS>so this behavior can be completely configurable 🚀
You can check for more information:
https://popper.js.org/popper-documentation.html#modifiers..flip.behavior
https://popper.js.org/popper-documentation.html#modifiers..preventOverflow.priority
Disable flipping
<Popup
    content='Hello'
    open
    position='top center'
    popperModifiers={{
      flip: { enabled: false },
    }}
    trigger={<Button content='Already Open' />}
  />

Flip to right instead of bottom
import React from 'react'
import { Button, Popup } from 'semantic-ui-react'

const PopupExampleDefaultOpen = () => (
  <Popup
    content='Hello'
    open
    position='top center'
    popperModifiers={{
      flip: { behavior: ['top', 'right'] },
    }}
    trigger={<Button content='Already Open' />}
  />
)

export default PopupExampleDefaultOpen


fix: Unhandled exception for Any metric type

SUMMARY
The metric is a list of Any, so that may or may not have the attribute get. This pull request adds a check for that and avoids the APIs using the same throwing internal server error.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
NA
TESTING INSTRUCTIONS
NA
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@villebro can you please check this?

@villebro please check.

@villebro applied the requested changes. Please check.


[Python] SEGFAULT when casting a slice of a fixed with binary array to binary

Describe the bug, including details regarding any error messages, version, and platform.
import pyarrow

arr = pyarrow.array(
    [b'a',b'b',b'c'],
   pyarrow.binary(1),
)
arr.slice(1).cast(pyarrow.binary())

Job 1, 'python' terminated by signal SIGSEGV (Address boundary error)

Running on an M1 Mac with pyarrow==15.0.2.
Component(s)
C++, Python

take

Issue resolved by pull request 41452
https://github.com/apache/arrow/pull/41452


Send peer deltas in MapResponse

Summary
Implement the sending of peer list deltas in MapResponse, rather then sending the full list each time. Newer versions of tailscale support this, but it had not yet been implemented in Headscale and was left as a TODO. Sending peer list deltas reduces data usage of the control protocol, especially for larger tailnets.
This also fixes a known bug described in #1383, where a peer would not be removed from the network if it is deleted and there are only 2 peers in the network.
This is implemented in a way intended to later be expanded to other deltas being sent in MapResponse. It introduces a new struct mapResponseStreamState that is forwarded to sequential calls to generate a MapResponse. Over time, this struct can evolve to track more state to move more of the MapResponse to supporting the deltas.
Closes #1383
Testing
This was tested locally by creating a small tailnet with 2 peers, and deleting one to verify that the other peer is notified of the deleted peer.
CI unit and integration tests should give reasonable assurance of no other regressions with this change.

Looks sensible 👍

just a reminder, need to ensure there's no delta may miss if there's any network unstable.

just a reminder, need to ensure there's no delta may miss if there's any network unstable.

@gps949, this approach should work as the state that determines if we should send the delta is stored in a function that only lives while the connection is alive.
That said, it would be have released as testing for a while.

@kev-the-dev cna you rebase to main?

if you can rebase this of master and fix the lint error, that should be good.

Oh I think it's related to the integration test testing multiple versions of tailscale client, some of which don't have support for the map deltas. We'll need to either drop support for these versions or introduce code to do different things depending on client version

When a maintainer gets the chance, I'm hoping the next run of the integration tests with new debug lines will clarify what versions of tailscale this is failing for

Oh I think it's related to the integration test testing multiple versions of tailscale client, some of which don't have support for the map deltas. We'll need to either drop support for these versions or introduce code to do different things depending on client version

Most of them seem to pass, so that would be strange, I would expect to see more of them failing. It would have to be done with CapVersion

Given it was 1.36.2 that failed, I think it wasn't an issue with Tailscale versions. My current guess is it's just a timing issue.. the test sleeps for a fixed 30 seconds. Depending on CPU load of CI, etc that may not work. I updated test to explicitly wait for the condition we're looking for.

Hm now failures with message:
=== FAIL: integration TestTailscaleNodesJoiningHeadcale/create-tailscale (unknown)
2023/05/16 07:38:36 failed to create tailscale node: could not start tailscale container (version: 1.30.2): : The command '/bin/sh -c apt-get update     && apt-get install -y gnupg curl ssh dnsutils ca-certificates     && adduser --shell=/bin/bash ssh-it-user' returned a non-zero code: 100
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0xecb78b]

I can't imagine this being related to the PR... perhaps a re-run?

It is not the best timing, we started quite a large code reorg, and we will likely not cut a new version for quite some time. We will have to discuss it a bit.
The second part of the code reorg is in https://github.com/juanfont/headscale/pull/1458, and that brings in the biggest changes. However, the changes touches so much of the code that I suspect we will run it as beta for quite some time, if not alpha.


Add vertex_normals

I had a little bit of spare time and wanted to move this from pygfx.utils.normals_from_vertices to here to give it a proper place and documentation.

Is this linear algebra though? I am not against putting that here, but maybe it's good to discuss/decide what belongs in pylinalg and what not. E.g. do triangulation techniques also have a place here?
Also, while we're touching this code, would it make sense to support quads too? (Either by means of another function, or by detecting quads in the function itself.)

Is this linear algebra though? I am not against putting that here, but maybe it's good to discuss/decide what belongs in pylinalg and what not. E.g. do triangulation techniques also have a place here?

The math involved is definitely linear algebra. I think it belongs. But I do not know where to "draw the line".

Also, while we're touching this code, would it make sense to support quads too? (Either by means of another function, or by detecting quads in the function itself.)

I guess it does!

I've got heaps and heaps of functions that fall in this category that I would like to add here. Many of them are well suited/sufficiently generic for broad re-use. I'm a little tired of copy pasting them into all my projects. :)
I've had a think on this, and I feel like it definitely belongs, but perhaps we should create a subpackage to keep computational geometry functions grouped together, separately from the existing linear algebra functions.
What do you think of introducing a subpackage called pylinalg.geom or something, to keep it terse?

A subpackage could work. I think it all depends on how close we can consider such functions to linear algebra.
I think my main concern is that such functions look more like algorithms, and for some of these, Cython or Numba may be helpful to make the implementation performant, which might be a reason to bundle these in a separate package.


Change download button icon

There's a glyph for download. The current one is dropdown menu. I was hoping you were going to offer me an ASCII art rendering of a horse, and now I'm sad.

It was going to allow you to download it in different formats, e.g. PC, Mac, Amstrad.
Now using download-image-alt instead as per suggestion.


Consider removing intelligent build (shelling to pangyp).

I'm leaning towards getting rid of the intelligent build process soon. It's caused issues before (now we call out to pangyp instead of node-gyp, fortunately), but a lot of the benefit seems like it's meant to be handled by other workflows, eg apm handling building native modules (or gyp defines, etc for electron apps with different names/versions). Not sure how nw.js handles it, but I'm betting an nw-gyp rebuild probably will redo your whole structure, invalidating the need for this and a lot of the complexity it adds. Basically we handle this, but other native modules still don't, and it feels like that responsibility is shifting towards the package management side (I guess it was just kind of up in the air before, tbh). I still wanna pre-generate/fetch the deps, but shelling out to the build ourselves can probably go away, and reliability AND portability would benefit (along with simpler code).
Anybody have any thoughts or opposition?

@tbranyen @johnhaley81 @orderedlist
Anybody have thoughts on this? If nobody has any complaints I might throw this in for a 0.5 release.

:+1: I feel like it messes up a lot of things in nwjs/electron land.

https://github.com/nodegit/nodegit/issues/574
Like that. I'm going to go through with this tonight unless someone has complaints. Too many issues, it's really not needed anymore. Even more recent versions of node 10 will auto call the appropraite node-gyp version.

Yeah this isn't ever going to be a thing until npm3+ is long since the only standard


Feature selection by mutual information

Closes https://github.com/dwavesystems/demos/pull/42

I don't see any unit tests for this code. Will that be added in at a later time?

Since we are moving the demos to a new GitHub account, I would prefer a clean switch from directories to submodules. The alternative would be to let this repo to be made up of mostly submodules with one or two directories waiting on their PRs; however, I don't like this mixed history because eventually, the directory will be removed and replaced with a submodule (but at a different point in time).
I have recorded the comments in this PR and will apply them in the new submodule.


Reversibility and GPRs of MAR04567 and MAR04568

Current Situation
Both reactions are reversible:
MAR04567: sedoheptulose-1,7-bisphosphate + ADP + H+ <-> sedoheptulose-7-phosphate + ATP
MAR04568: sedoheptulose-1,7-bisphosphate + UDP + H+ <-> sedoheptulose-7-phosphate + UTP
and both have the same GPR: ENSG00000067057 or ENSG00000141959 or ENSG00000152556 (three isozymes of phosphofructokinase (PFK))
Problems With The Current Situation
I find their reversibility suspicious, since that would imply that cells can produce ATP by removing a phosphate from sedoheptulose-1,7-bisphosphate (S17BP), which is not a method of ATP production I've ever heard of in humans. Also, it allows this loop of arbitrarily large fluxes with no inputs or outputs to form with MAR04379, MAR04375, MAR04355, and MAR04565:

Same as above but with names instead of IDs:

The only reference associated with MAR04568 does not mention sedoheptulose-7-phosphate (S7P) at all; it's about how yeast PFK acts on fructose-6-phosphate. One of the references associated with MAR04567 shows that rabbit PFK can phosphorylate S7P to S17BP, but says nothing about the reverse reaction. The only other reference associated with MAR04567 claims that aldolase and PFK can form S17BP and that fructose-1,6-bisphosphatase (FBPase) can dephosphorylate it to S7P, but it appears to just be the abstract of a paper that is otherwise not available online, so it's not clear if they found that FBPase could form ATP or just cleave the phosphate and release it as a free inorganic phosphate (also it was talking about rats and not humans).
I found this paper that found that human F16BPase can hydrolyze S17BP to S7P and generally gives me the impression that they did not also observe ADP or UDP simultaneously being phosphorylated to ATP/UTP. This paper says that PFK catalyzes the S7P -> S17BP reaction and specifically says that F16BPase catalyzes S17BP -> S7P + Pi, although it's not exactly clear which organism they're talking about (they mention several different mammals throughout the paper).
Proposed Changes

[ ] Make MAR04567 and MAR04568 irreversible and switch their reactants and products:
MAR04567: sedoheptulose-7-bisphosphate + ATP -> sedoheptulose-1,7-bisphosphate + ADP + H+
MAR04568: sedoheptulose-7-bisphosphate + UTP -> sedoheptulose-1,7-bisphosphate + UDP + H+
with IDs instead of names:
MAR04567: MAM02884c + MAM01371c -> MAM02883c + MAM01285c + MAM02039c
MAR04568: MAM02884c + MAM03130c -> MAM02883c + MAM03106c + MAM02039c
Their GPRs can stay the same, since those genes are all PFK isozymes, which is apparently known to catalyze this reaction in humans, but only in the ATP/UTP-consuming direction (source, source).

[ ] Add a new irreversible reaction for the hydrolysis of S17BP into S7P and phosphate catalyzed by the human FBPase isozymes FBP1 and FBP2 (ENSG00000130957 or ENSG00000165140, the same GPR as MAR04377): MAM02883c + MAM02040c -> MAM02884c + MAM02751c

@Devlin-Moyer thanks again for the detail investigation and report!

Yea after another round of searching, I can't seem to find any papers that actually demonstrated direct conversion of S7P to S17BP in humans; everyone just keeps citing the same two papers that observed it in rabbit and rat liver extracts. Also, as this review points out, those papers were using liver extracts that were enriched in PFK but also had plenty of other enzymes around, and they never specifically determined that PFK was catalyzing the direct phosphorylation of S7P to S17BP.
So maybe the best course of action here is to remove MAR04567 and MAR04568 entirely and replace them with a single reaction that converts S17BP to S7P + Pi with FBP1 or FBP2, since that actually seems to have been directly observed to happen with human enzymes (source, source)

Yea after another round of searching, I can't seem to find any papers that actually demonstrated direct conversion of S7P to S17BP in humans; everyone just keeps citing the same two papers that observed it in rabbit and rat liver extracts. Also, as this review points out, those papers were using liver extracts that were enriched in PFK but also had plenty of other enzymes around, and they never specifically determined that PFK was catalyzing the direct phosphorylation of S7P to S17BP.

This study reports elevated S17BP level in human Hep G2 cells as a response to oxidative stress. It also provided alternatives routes of S17BP formation in PPP pathway, including the one from S7P catalyzed by phosphofructokinase but without citation!

So maybe the best course of action here is to remove MAR04567 and MAR04568 entirely and replace them with a single reaction that converts S17BP to S7P + Pi with FBP1 or FBP2, since that actually seems to have been directly observed to happen with human enzymes (source, source)

Not exactly sure about removing MAR04567 and MAR04568. But it makes sense to have one that converts S17BP to S7P + Pi with GPR of FBP1 or FBP2

@Devlin-Moyer how about this proposal for follow-up implementation:

remove MAR04568, because there is no evidence;
make MAR04567 irreversible as RHEA:31444 that is catalyzed by PfkA in E. coli - same plan as above; because it shouldn't be reversible;
add new reaction: H2O + S17BP -> S7P + Pi to cytosol

Only old references can be associated with step 2 (Biochem J. 1974 138:71-6) and 3 (Biochem J. 1978 176:257–282), would be good to find more recent ones.

Sounds good to me; I have tried to find more recent references relating to these reactions and failed; all the recent papers that mention this reaction I can find are talking about how human gut microbes use it and now how human cells use it

would you like to implement this proposal that you initialized @Devlin-Moyer

I could try but I'm not entirely certain that I'd do it right; I don't know a ton about using Git/GitHub and am not confident that I've fully grasped how all the different files here fit together

then give a try - just ask if you need help

@Devlin-Moyer looking closer at the figure, I'm wondering how you obtained it. It looks Escher-like (curved lines), but also with a layout that is algorithmic-like (aligned nodes). Could you share some details about it?

Yep that's just an ordinary Escher map that I spent a potentially unreasonable amount of time fiddling with to try and get everything laid out cleanly; no layout algorithms were involved (I would love it if there were such an automated layout algorithm for making Escher maps, but I've never found one, and I get why; it seems like it'd be quite difficult to create)

Thanks @Devlin-Moyer for the info. At Metabolic Atlas we have been thinking of the best of doing this for some time, and have started some tiny steps in this direction but there is a long way to go.

feel free to reopen


Parts of plot, plot history window not localized

Tested with French language.
These words appear in tool window caption / content. They are correctly localized in the menu item (circled blue).
RPlot
Device
Active

Here's additional text from plot and plot history.

All strings are in resx files already

No repro in latest build:


Fix BadZipFile Error

Fix "BadZipFile Error" when build docker. "BadZipFile Error" is same as mmcv #1352
(base) ➜  mmdetection git:(master) docker build -t mmdet docker/
Sending build context to Docker daemon  7.168kB
Step 1/15 : ARG PYTORCH="1.6.0"
Step 2/15 : ARG CUDA="10.1"
Step 3/15 : ARG CUDNN="7"
Step 4/15 : FROM pytorch/pytorch:${PYTORCH}-cuda${CUDA}-cudnn${CUDNN}-devel
 ---> bb833e4d631f
Step 5/15 : ENV TORCH_CUDA_ARCH_LIST="6.0 6.1 7.0+PTX"
 ---> Using cache
 ---> db65124ef61d
Step 6/15 : ENV TORCH_NVCC_FLAGS="-Xfatbin -compress-all"
 ---> Using cache
 ---> d83a99117aa2
Step 7/15 : ENV CMAKE_PREFIX_PATH="$(dirname $(which conda))/../"
 ---> Using cache
 ---> 7bfd23a73348
Step 8/15 : RUN apt-get update && apt-get install -y ffmpeg libsm6 libxext6 git ninja-build libglib2.0-0 libsm6 lib
xrender-dev libxext6     && apt-get clean     && rm -rf /var/lib/apt/lists/*
 ---> Using cache
 ---> 97e88db9944a
Step 9/15 : RUN pip install mmcv-full==1.3.17 -f https://download.openmmlab.com/mmcv/dist/cu101/torch1.6.0/index.ht
ml
 ---> Running in c298acc26563
Looking in links: https://download.openmmlab.com/mmcv/dist/cu101/torch1.6.0/index.html
Collecting mmcv-full==1.3.17
  Downloading https://download.openmmlab.com/mmcv/dist/cu101/torch1.6.0/mmcv_full-1.3.17-cp37-cp37m-manylinux1_x86_
64.whl (41.7 MB)
ERROR: Exception:
Traceback (most recent call last):
  File "/opt/conda/lib/python3.7/site-packages/pip/_internal/cli/base_command.py", line 186, in _main
    status = self.run(options, args)
  File "/opt/conda/lib/python3.7/site-packages/pip/_internal/commands/install.py", line 331, in run
    resolver.resolve(requirement_set)
  File "/opt/conda/lib/python3.7/site-packages/pip/_internal/legacy_resolve.py", line 177, in resolve
    discovered_reqs.extend(self._resolve_one(requirement_set, req))
  File "/opt/conda/lib/python3.7/site-packages/pip/_internal/legacy_resolve.py", line 333, in _resolve_one
    abstract_dist = self._get_abstract_dist_for(req_to_install)
  File "/opt/conda/lib/python3.7/site-packages/pip/_internal/legacy_resolve.py", line 282, in _get_abstract_dist_fo
r
    abstract_dist = self.preparer.prepare_linked_requirement(req)
  File "/opt/conda/lib/python3.7/site-packages/pip/_internal/operations/prepare.py", line 482, in prepare_linked_re
quirement
    hashes=hashes,
  File "/opt/conda/lib/python3.7/site-packages/pip/_internal/operations/prepare.py", line 287, in unpack_url
    hashes=hashes,
  File "/opt/conda/lib/python3.7/site-packages/pip/_internal/operations/prepare.py", line 164, in unpack_http_url
    unpack_file(from_path, location, content_type)
  File "/opt/conda/lib/python3.7/site-packages/pip/_internal/utils/unpacking.py", line 252, in unpack_file
    flatten=not filename.endswith('.whl')
  File "/opt/conda/lib/python3.7/site-packages/pip/_internal/utils/unpacking.py", line 114, in unzip_file
    zip = zipfile.ZipFile(zipfp, allowZip64=True)
  File "/opt/conda/lib/python3.7/zipfile.py", line 1258, in __init__
    self._RealGetContents()
  File "/opt/conda/lib/python3.7/zipfile.py", line 1325, in _RealGetContents
    raise BadZipFile("File is not a zip file")
zipfile.BadZipFile: File is not a zip file
The command '/bin/sh -c pip install mmcv-full==1.3.17 -f https://download.openmmlab.com/mmcv/dist/cu101/torch1.6.0/
index.html' returned a non-zero code: 2

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

Hi @DIYer22 ,
Could you fix the lint issue so that we can merge this PR?

@DIYer22 Thanks for your contribution! Please install pre-commit and then run pre-commit run --all-files to fix the lint.


Change pay button in checkout being disabled to always enabled with errors thrown if necessary

Currently if some information is missing or incorrect in checkout the pay button is disabled, not providing much info to the user about the particular reason. It'd make more sense to always enable the button and in case some info is missing or incorrect before making request to the api throw alert saying what is wrong.

@bmigirl is this related https://github.com/saleor/react-storefront/issues/377 ?


[google_maps_flutter] embedded_views_preview not required since v1.0.0

The release notes state that io.flutter.embedded_views_preview is not required since version 1.0.0.
Thus removed this from the example, to keep things clean and tidy ✨

@googlebot I signed it!

Landing on red since the two red runs are https://github.com/flutter/flutter/issues/76805


Plugin for Kong as API Gateway to proxy S3

Summary
Whether or not kong can make a plugin to upload file into S3 or download file from S3 and cache it?
For Example:
https://github.com/jamesmarlowe/lua-resty-s3
Kong can upload the file and send s3 private url to backend, receive a s3 url from backend to download the file and cache it.
I am not sure that the idea make sense as kong plugin?

Who would use it and why?

We've recently used same approach (nginx + lua scripts) to implement a s3 proxy so we don't need to include aws sdk on client-side anymore to upload files to s3. So I would say that could be a useful plugin. But I'm not sure about caching, maybe using a cdn would be a better choice.

Could be a neat core integration feature for Cassandra/Postgres DB backups(which is what we use S3, for but we use a tool called cassandra-snapshotter that is opensource to actually orchestrate that).

@jeremyjpj0916 we generate temp auth credentials (on server-side where we don't have problem to import aws sdk) to grant s3 access for client, then client calls the proxy sending these credentials and we generate aws signature using slightly modified version of this script : https://github.com/Kong/kong/blob/master/kong/plugins/aws-lambda/v4.lua but that would be the idea.

Would be convenient if we could use this feature to host a static website, since there is no easy way to do this currently

Anyone knows about, how to integrate https://github.com/anomalizer/ngx_aws_auth this with kong?

We have a scenario involving Kong and S3 buckets too, although is to Get objects.

Although, when Kong sits in front of a bucket, either Kong gets through s3 "website" endpoint or directly with s3 endpoint (kind of the same behavior). Those requests are unsigned GETs, it means you cannot write a bucket policy to cover "let only that IAM role to get the objects" policy, because those unsigned GETs does not carry information to authenticate the caller (to be simply: the unsigned GETs does not have "authorization" header).
So, with my plugin https://github.com/carnei-ro/kong-plugin-s3, based on aws-lambda, I am signing the GET requests, solving this headache.
So far, I only cover GETs... it is an alpha stage. PRs are welcome to cover e.g. "POST"/"PUT" and "DELETE" requests/operations.


No option for xcodebuild option "-parallel-testing-enabled"

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

It's pretty strange that fastlane lets me set -parallel-testing-worker-count using concurrent_workers, but I have to use xcargs: "-parallel-testing-enabled YES" to actually enable parallel testing. I would expect that option to be configurable via fastlane as well.

This is quiet a common issue


Feature request: Filter multiple tags

It would be nice if it was possible to filter multiple tags in map view mode. I have notes containing the tag of a month ("#june") and some of them also contain tags like "#hotel". Is there a way to show only notes that contain both "#june" and "#hotel" when showing the map? When you select multiple tags, it seems it searches for notes containing "#june" OR "#hotel" - i'm looking for a way to show notes containing "#june" AND "#hotel". Thanks a lot in advance! 😊

Is the SearchComponent class the same as the filter entry in the graph view?
If it is I think it would make sense to switch to that.
That way Obsidian can do the work to find the files and it will support OR and all the other things that it supports without any extra work.

SearchComponent is unfortunately just a text box with an X, it does not offer any search functionality. I've asked the Obsidian devs to open some search functionality in the API for a long time now (https://github.com/obsidianmd/obsidian-api/issues/22).
I've given up and spent lots of time to rewrite it from scratch, allowing search operators, logical operators, auto-suggest etc.
I'm in advanced stages (only README updates are missing) of releasing a major 2.0.0 version of Map View with this big new functionality.

Added in the just-released 2.0.0 version.


Calendar time select scrolling performance issue

Tooltips inside scroll select element slow down scrolling.

This happens only in dev
https://github.com/popperjs/react-popper/issues/419


manual installation required, no devices or entities found

Hi, i wanted to switch to this integration;
there was no auto-config, the manual worked but no devices or entities found.

When you download the current diagnostics, there are correct values in it:

Do you have an idea?
Thanks in advance

If you use HA >= 2023.8 please ensure to use the latest prerelease of this integration.
Please read the readme of the latest prerelease.

thanks, it works now!


SAS token for Blob Index Tags

Is your feature request related to a problem? Please describe.

Azure CLI doesn't support Blob Index Tags permissions t f x. It throws validation error.
az storage account generate-sas --account-key xxx --account-name myaccount --expiry 2020-01-01 --https-only --permissions rwdlactfx --resource-types cos --services b

valid values are (a)dd (c)reate (d)elete (l)ist (p)rocess (r)ead (u)pdate (w)rite (a)dd (c)reate (d)elete (l)ist (p)rocess (r)ead (u)pdate (w)rite or a combination thereof

Describe the solution you'd like

Add t f and x to acceptable values for --permissions.
Describe alternatives you've considered

Additional context

Works with PowerShell and is well documented with REST.
$storage = Get-AzStorageAccount -Name myaccount -ResourceGroupName mygroup
New-AzStorageAccountSASToken -Service Blob -ResourceType Service,Container,Object -Permission rwdlactfx -Protocol HttpsOnly -StartTime (Get-Date) -ExpiryTime (Get-Date).AddDays(1) -Context $storage.context

storage

Note: this is kind of a bug as it is lack of support of an Azure feature which should have been supported since day one.
@yonzhan Given the apparent simplicity of the solution I believe it should be addressed sooner. Is that possible?

Looks like Azure Portal haven't supported t, f, x either:

Azure Storage Explorer only supports t, x:

this is kind of a bug as it is lack of support of an Azure feature which should have been supported since day one.

I agree. Per https://docs.microsoft.com/en-us/rest/api/storageservices/create-service-sas#permissions-for-a-directory-container-or-blob, t, f and x are just literal characters SDK should be able to bypass the validation.
I wrote a simple script to help generate the SAS token with t, f, x permissions:
import requests
from azure.storage.common._common_conversion import _sign_string
from azure.storage.common._serialization import url_quote

# string_to_sign is constructed following
# https://docs.microsoft.com/en-us/rest/api/storageservices/create-service-sas#constructing-the-signature-string

key = 'kyZcwi...'
query = ('sp=racwdxltf&'  # signedPermissions
         'st=2021-11-19T00%3A00%3A00Z&'  # signedStart
         'se=2021-12-01T00%3A00%3A00Z&'  # signedExpiry
         'sv=2020-12-06&'  # signedVersion
         'sr=c'  # signedResource
         )
string_to_sign = ('racwdxltf\n'  # signedPermissions
                  '2021-11-19T00:00:00Z\n'  # signedStart
                  '2021-12-01T00:00:00Z\n'  # signedExpiry
                  '/blob/jlst1007/con1\n'  # canonicalizedResource
                  '\n'  # signedIdentifier
                  '\n'  # signedIP
                  '\n'  # signedProtocol
                  '2020-12-06\n'  # signedVersion
                  'c\n'  # signedResource
                  '\n'  # signedSnapshotTime
                  '\n'  # signedEncryptionScope
                  '\n'  # rscc
                  '\n'  # rscd
                  '\n'  # rsce
                  '\n'  # rscl
                  )
sig = url_quote(_sign_string(key, string_to_sign))
sas = f"{query}&sig={sig}"
print(sas)
r = requests.get(f'https://myst.blob.core.windows.net/con1/test.txt?comp=tags&{sas}')
print(f"{r.text}")

@jiasli Interesting. Portal supports it on Storage account level though.

@Tbohunek I also noticed that! 😃 It's just generating container SAS doesn't support those.

Why has this been added to a milestone and removed multiple times?

This is blocked by Python SDK gap between track1 and track2. We will add support as soon as Python SDK fix


TypeError: undefined is not a constructor (evaluating 'new brorand.Rand()')

I am using react-native-crypto with rn-nodeify. but got following error
Requiring module "react-native-crypto/index.js", which threw an exception: TypeError: undefined is not a constructor (evaluating 'new brorand.Rand()')

Hi, we are working on RN Ethereum starter and just had the same issue with importing ethereumjs-wallet lib.
Adding one line with "require("crypto")" above the import fixed the problem.
See here - https://gitlab.com/sashamihaha/rn-ethereum-starter/blob/dev/App.js#L5

Did you ever figure this out @Kishanjvaghela ?

following

@Kishanjvaghela does @Dobrokhvalov's fix work for you? If not, send me a link to your project, I can take a quick look.
@Dobrokhvalov maybe we should add require('crypto') to the crypto section of shim.js?
Currently it only gets run when not remotely debugging.

I was just able to fix by adding the following to shim.js
global.crypto = require('crypto') global.crypto.getRandomValues = function(byteArray) {   for (let i = 0; i < byteArray.length; i++) {     byteArray[i] = Math.floor(256 * Math.random());   } }
Before adding the getRandomValues method, the app would only work when remotely debugging

@claykohut oof, you don't want to do that! rn-nodeify already shims getRandomValues with a secure implementation backed by either the iOS/Android native rng, or sjcl's implementation seeded with the native rng. Just make sure to import shim.js in your project, as early as possible

Thanks for the heads up @mvayngrib ! Any idea why my shim.js didnt already have that in it?
I've now manually copied the shim.js you linked to over to my shim.js -- previously it looked like:

if (typeof __dirname === 'undefined') global.__dirname = '/'
if (typeof __filename === 'undefined') global.__filename = ''
if (typeof process === 'undefined') {
  global.process = require('process')
} else {
  var bProcess = require('process')
  for (var p in bProcess) {
    if (!(p in process)) {
      process[p] = bProcess[p]
    }
  }
}

process.browser = false
if (typeof Buffer === 'undefined') global.Buffer = require('buffer').Buffer

// global.location = global.location || { port: 80 }
var isDev = typeof __DEV__ === 'boolean' && __DEV__
process.env['NODE_ENV'] = isDev ? 'development' : 'production'
if (typeof localStorage !== 'undefined') {
  localStorage.debug = isDev ? '*' : ''
}

yea, i forget when i added the crypto stuff to shim.js. rn-nodeify doesn't overwrite your shim.js, so I guess u missed out on an update. rn-nodeify should probably export the shim in a better way, e.g. recommend apps to require it via import 'rn-nodeify/shim'

Not sure if everyone has got this working yet but I had the same problem and after messing with it for a while I got it working.

rm -rf node_modules
rm shim.js
npm install
rn-nodeify --hack --install

And it worked! A couple things to note.

I had hacked bignumber.js (that's required by web3.js) a while back to work with the proper crypto library, but removed the hack since rn-nodeify solves the issue.
When building from xcode on a simulator or device I made sure to do a hard clean of the build directory (option + shift + command + k)

I'm not 100% sure what combination of all of this worked but it's currently working for me.

looks like this is now working for people. The crypto shim has been moved over to a more sensible place in react-native-crypto, where it's loaded automatically. If this is still an issue for someone, we'll reopen

it shows me a blank white screen . No errors though.
this is how my shim.js looks
if (typeof __dirname === 'undefined') global.__dirname = '/'
if (typeof __filename === 'undefined') global.__filename = ''
if (typeof process === 'undefined') {
  global.process = require('process')
} else {
  const bProcess = require('process')
  for (var p in bProcess) {
    if (!(p in process)) {
      process[p] = bProcess[p]
    }
  }
}

process.browser = false
if (typeof Buffer === 'undefined') global.Buffer = require('buffer').Buffer

// global.location = global.location || { port: 80 }
const isDev = typeof __DEV__ === 'boolean' && __DEV__
process.env['NODE_ENV'] = isDev ? 'development' : 'production'
if (typeof localStorage !== 'undefined') {
  localStorage.debug = isDev ? '*' : ''
}

Am i missing anything?

@sreeksp not enough information :) Did you check for errors both in the dev tools console and in the react-native packager's?

Hello!
I've resolved this issue by importing crypto just after shim.js
This is my app.js
import "./shim";
import crypto from "crypto";
import React from "react";
import App from "./js/App";

export default class App1 extends React.Component {
  render() {
    return <App />;
  }
}

Thanks @dylanbathurst and @ahmb84 , a combination of those 2  suggestions helped me out.

I've tried all the suggestions here a number of times, with no results. I have no idea what the problem is. "react-native": "0.51.0", "react-native-crypto": "^2.1.2"

@micimize did you follow the suggested workflow from the readme? Can you run the sample project at https://github.com/mvayngrib/rnctest2 ?

@mvayngrib Got it working. What ended up working was deleting all local files, running yarn cache clean, deleting everything in my $TMPDIR, and reinstalling everything. I still have no idea what the problem was, but I don't think it was any of the shim toolchain.
FWIW, I'm now using babel transforms in place of rn-nodeify and it's working fine for my limited use case:
{ // .babelrc
  "presets": ["react-native"],
  "plugins": [
    "babel-plugin-react-native-nodeify-hack",
    ["rewrite-require", {
      "aliases": {
        "crypto": "react-native-crypto",
        "stream": "stream-browserify",
        "vm": "vm-browserify"
      }
    }]
  ]
}

// index.js
require('crypto')
import process from 'process'
import buffer from 'buffer'
// ...

@micimize great to hear! Yes, for some cases rewrite-require is enough. But due to a number of incompatibilities between the react-native packager and webpack/browserify, there's a number of hacks/patches rn-nodeify uses (see https://github.com/tradle/rn-nodeify/blob/master/pkg-hacks.js)

This PR fixed this issue for me.
My project is using react-native-bip39 and react-native-bitcoinjs-lib, and I was able to get this working using that fix along with the babel transforms you suggested @micimize.
For anyone also running into this issue, that doesn't want to use rn-nodeify for whatever reason, here is the process that worked for me:
Install
yarn add all of these subsequent modules.

assert
babel-plugin-rewrite-require
react-native-bip39
react-native-bitcoinjs-lib
react-native-crypto
react-native-os
react-native-process-shim
react-native-randombytes
react-native-stream
vm-browserify

then, manually add this line to your dependencies in package.json:
  "brorand-shim": "https://github.com/HoardInvest/brorand",

We will later use this to alias brorand calls to this shimmed version,
which will resolve the new brorand.Rand() error.
Link
After install, react-native link each of these modules.

react-native-os
react-native-randombytes

File changes
Process shim
this shim must be loaded before bip39 or the bitcoin lib,
to correctly initialize the "process" global variable that
is expected to be available by their dependencies
import 'react-native-process-shim';

.babelrc
With babel-plugin-rewrite-require, we can redirect any of the
require or import calls of the alias we define to the module we
give.
We will use the normal shims for crypto, stream, and vm, but
add our new brorand shim here as well. We don't need to alias
assert, since that is available just through having it as one
of our dependencies.
{
  // the rest of your .babelrc file

  "plugins": [

    // other plugins you might have

    ["rewrite-require", {
      "aliases": {
        "brorand": "brorand-shim",
        "crypto": "react-native-crypto",
        "stream": "react-native-stream",
        "vm": "vm-browserify"
      }
    }]

  ],
}


Image failed to load

Expected Behavior

show the images
Current Behavior

can't show the images
Possible Solution

Steps to Reproduce

I run the example in this project.
Context (Environment)

"react": "16.8.3",
"react-native": "0.59.1",
"react-native-fast-image": "^5.2.0",
"react-native-masonry": "^0.5.0-alpha.3"

Detailed Description

Possible Implementation

Android or iOS?

@brh55 Android,

I see the same yellow box on iOS. However, the image loads.


Style guide and commit standards

Adds a code style guide, including standards for commit messages
I think it makes sense that this document would replace CONTRIBUTING.md once considered complete and correct.

We should probably have something on accessing individual request headers. From https://github.com/openresty/lua-nginx-module#ngxreqget_headers:

Note that the ngx.var.HEADER API call, which uses core $http_HEADER variables, may be more preferable for reading individual request headers.

What about function calling with string and table?
require "something"
-- vs.
require("something")
-- vs.
require"something"

And same for table?

Proposed an alternative, more complete version at #2569

Done in #2569


When execute command got this error.

Facing this issue in laravel 5.1 Can some tell me how to fix this?
[Symfony\Component\Debug\Exception\FatalThrowableError]   Call to undefined method Illuminate\Foundation\Application::bootstrapPath()

Hi, hassanuos just add this
$this->app->register(\Mpociot\ApiDoc\ApiDocGeneratorServiceProvider::class);
on the AppServiceProvider.php under function register.


Upgrading to React 18 breaks remove event listeners.

Describe the bug
Upgrading to React 18 breaks remove event listeners.
To Reproduce
I am using React-18, react-infinite-scroller, and react-router
Let's say I have a react-infinite-scroller on /user-list route. All works fine with scrolling and loading data.
Now when you click on one of the users and go to the next /user-details route, it breaks while trying to remove the old routes event listeners.
Screenshots

Device (please complete the following information):

OS: Windows 11
Browser Chrome
Version 100.0.4896.75 (Official Build) (64-bit)

I tried reproducing this, but couldn't hit that error. Here's my code: https://github.com/ruipserra/react-infinite-scroller-repro-296
@akhilxnair are you able reproduce the error with my testing code? Can you fork it and change it to hit the error?

设置useWindow={false}，就会出现问题

The same issue. Not working with react 18.

The same issue. Not working with react 18.

Hi, I am facing same issue, were you able to find the solution?

@andyogaga Nope, we postponed the upgrade process till everything was stable.
Will try it again in a month.

I just tried with useWindow = false and it worked on my end, as suggested by @foreverzwl

We upgraded all our packages now and faced no issues this time.
I'll try to find out what fixed the issue, maybe it was fixed by react minor version upgrade.


Update communities

Without those changes:
44893 unique nodes
847 duplicates skipped

With those changes:
46565 unique nodes
2915 duplicates skipped

deployed


Blinn specular is too intense

Godot version
3.3.2
System information
gles3, ubuntu, intel i5 8350u
Issue description
Blinn

Phong

Schlick ggx

Lambert diffuse, Specular 0.5, Metallic 0.0, Roughness 1.0
Tonemap filmic, exposure 1.0, white point 16.0
Needless to say, this result is unusable. It looks like the Blinn and Phong are not normalized at all, and the Blinn is boosted some too.
Steps to reproduce

Create a meshinstance and a directionallight
give the meshinstance a sphere and a spatialmaterial
set the albedo to something low, like 64,64,64
alternate between specular shading modes to see significant differences between algorithms

Minimal reproduction project
No response

Just testing with a PlaneMesh you can see that both the Blinn and Phong settings are very bright when using a rough material. In both cases they indeed look like they are not being normalized at all.

This turned out to be more complex than I thought (but also should have been more obvious). Physically Based specular Materials are typically the product of 3 functions:

a normal density function
a geometric function (masking function)
a Fresnel function

Right now we use all three in the GGX shading mode, but for Blinn and Phong we just use the NDF. The surface albedo of a material is only references in the Fresnel function. This means that neither Blinn nor Phong take the surface albedo into account. This is closer to the behaviour of blinn and phong in non-PBR materials. But it does result in very bright highlights on dark materials. Essentially, the Blinn and Phong models are non-PBR, but they are energy conserving. They are much cheaper than the GGX model but they do not look very similar.

Fixed by #51410.


Does typeguard work with expresssions, i.e. ev.target?

A cast appears to work with the expression, ev.target.
         if(ev.target instanceof Node) {
             var n : Node = <Node>ev.target;
             n.appendChild(document.getElementById(data));
         }

A typeguard appears to work with the simple variable, t.
         var t = ev.target;
         if(t instanceof Node) {
             var n : Node = t;
             n.appendChild(document.getElementById(data));
         }

A typeguard appears to fail with an expression, ev.target or (ev.target).
         if(ev.target instanceof Node) {
             ev.target.appendChild(document.getElementById(data));
         }

This is by design currently (the spec section on type guards only mentions narrowing variables and properties, not dotted names). We can consider augmenting the narrowing functionality but there are concerns about side effecting function calls.

+1

closing in favor of https://github.com/Microsoft/TypeScript/issues/3812


Localize in known map

can probably use AMCL to localize in known map: https://github.com/ros-planning/navigation2/tree/main/nav2_amcl

Can also set n_particles=1 (see simulation.cpp)


Improve design of client class methods

As pointed out in issue #55, the method signatures for the Client class are odd to look at.
The entire setup of methods appears a bit bolted on and redundant, and should be improved.

With the introduction of  zmq to the project, I'm looking to deprecate the old client interface altogether.


[jvm-packages] support missing value when constructing dmatrix with iterator

Hi @trivialfis, Could you help review it and merge it?


Detect process failure and restart it

Still partially exists, but we'll investigate in with other issues.


Lower bound constrained time constraints missing the upper bound submitted through the GeoTools pluggin inefficient

The upper bound is left unbounded if not provided in a query.  The lower bound is constrained between two values. A broad index search is then initiated.
Example Query:  start between '1/12/2015' and '1/14/2015'.

As peel through this, the fix is relatively easy.  However, it does not address a bigger issue.  I way want multiple temporal indices over different time attributes of a feature.  For example, I may want one ot start and end date and another on creation date.  How this achieved?
My thought is that we continue to use time descriptors with an optional index identifier.  This states that I want to use index 'x' with these attributes.  Our ability to name indices that may have the exact configuration (strategy/model) is critical, as we do not want the data indexed two different ways in the same index table.   The data adapter would be armed with one or more time descriptors, one for each index.
Armed with this, how does this change the rest of the code?  When CQLQuery is inquired for a Query (SpatialQuery), it returns one based on time constrained derived from the CQL and aligned with the time descriptor attributes.  (method is List getIndexConstraints(
final NumericIndexStrategy indexStrategy )).
Change this method to accept an Index, not just the strategy, so the ID is present.  Using the ID, look up the time descriptor that matches.  If none matches, use a default (one provided without id which is often the one inferred from the feature type).  Using  specific time descriptor for the index, pull out the relevant constraints from the query. This concept moves some of the logic performed in the constructor into this method.  It also forces the retention of the time descriptors in the instance (client side only) unless we also change the getIndexConstraints method to accept the adapter as well I think this is not a good idea  since Time Descriptors is specific to one adapter.  We could generalize time descriptors-- have the adapter return the fieldIds that a given index (and its  CommonIndexModel).
What is really nice is that the query optimizer can choose which indices is presented to this method.
I need to ExtractTimeVisitorFilter to handle 'or' different to build up the hierarchy of pairs.   This can used by the CBO to see which constraints CONSTRAIN the data the most and thus lead to the best index.   Alternatively, I just repeat the use of the visitor given time descriptors.  This is more in line with what is done today.  This would result in multiple 'parse' steps for each temporal index.  Not desirable, although, not  horrible since we are not likely to see more than two or three different temporal indices.
The QueryIndexHelper need not change much.  It can be given sets of constraints and a time descriptor, providing a 'trimmed' range based on stats. Armed with histogram stats, a true indication of constrained rows can aid in the final decision for the appropriate index.


ELF loader: Refactor and fix multiple issues.

@cfcs found multiple issues in our ELF loader (#382, #383). This code is quite old and has been in need of review for some time, so here is a first cut of refactoring along with fixes for the reported issues.
Summary of changes:

Validate additional ELF header fields in ehdr_is_valid(), including sizes of dependent headers.
Use types from elf.h throughout.
Move ehdr to heap-allocated memory.
Reduce the number of intermediate variables and use naming from elf.h throughout.
Use p_vaddr instead of p_paddr for determining the load address.
According to the spec, this is what we should be doing. This has worked so far due to current toolchains setting p_paddr = p_vaddr.
Add extra checks to ensure that e_entry is valid, and that p_memsz is not less than p_filesz.

More refactoring to come, likely as part of this PR.

@cfcs Ok, I've done another pass in abeca34. This makes the ELF loader even more strict -- see the commit message for details. I've also implemented adjusting the segment start address for alignment.
I also deliberately disallow overlapping segments in the ELF loader. As a side effect of this, I found that clang (FreeBSD) was generating such an executable for test_tls due to missing TLS sections in our linker script, so I fixed that -- I think -- at least enough to make the test pass. I'm still not convinced that we are actually doing enough/the right thing regarding TLS but don't want to get too distracted by that particular rabbit hole right now.
Please review.

Note that the OpenBSD test is actually successful, but is showing as pending due to random surf- build/node.js brokenness.


Add CI plan

As a developer, I'd like to build and stage the app to CF on every incremental commit.

Added with commit 1951414e4107222f742a19821a609ae5c0b1ef9a


Implement fiat pair converter plugin

Since the CCXT library does not have a forex exchange connected to it #48 , we will need to implement another solution to convert between different kinds of fiat. exchangerate.host seems to be a free and easy to implement solution for the time being.
We will need to create a simple converter plugin that makes use of the JSON calls and not require other packages.

This will need to be rolled into the CCXT plugin or the abstract pair converter plugin itself.


Example Resource Group?

"Resource Groups	Subscription	Create a new resource group for use by other artifacts within the blueprint. These placeholder resource groups enable you to organize resources exactly the way you want them structured and provides a scope limiter for included policy and role assignment artifacts as well as Azure Resource Manager templates."
Would be nice to have an example of how to include a Resource Group only template in a BluePrint.

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

ID: b1b33f46-674b-d3ef-7644-4422927625b3
Version Independent ID: b036a530-81bd-1d4b-d099-d485f7098ab4
Content: Overview of Azure Blueprints
Content Source: articles/governance/blueprints/overview.md
Service: blueprints
GitHub Login: @DCtheGeek
Microsoft Alias: dacoulte

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@ronbok Thanks for the question and interest in Blueprints! The existing Quickstart includes using several artifacts. If you wanted just a Resource Group, you could follow the Quickstart and omit other artifact steps.  To help me understand, what is the use-case for having a Blueprint with only a Resource Group and no other artifacts?
@MicrosoftDocs/azure-cxp-triage #in-progress

@ronbok - Just checking in... was my suggestion of using the existing Quickstart, but skipping the other artifact steps, useful for what you were looking for? Still looking to understand the use-case of what you have asked for.  Thanks!

Sorry, no.
The issue I think I was trying to surface is that there’s no ARM Template capability to create just a Resource Group.
From: David Coulter<EMAIL_ADDRESS>Sent: Friday, October 19, 2018 11:06 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Bokleman, Ron<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Example Resource Group? (#16745)
@ronbokhttps://github.com/ronbok - Just checking in... was my suggestion of using the existing Quickstart, but skipping the other artifact steps, useful for what you were looking for? Still looking to understand the use-case of what you have asked for. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/16745#issuecomment-431393999, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHHxLK1gwY661WsCYGfl8G5bxfp-OdPVks5umepRgaJpZM4XZ4Xa.

@ronbok I'm confused why you need/want a Resource Manager template for just creating a Resource Group when a Resource Group is a built-in artifact within Blueprints. Why does it need to be a template artifact and not just use the Resource Group artifact?

Because … all too often want to create a series of Environment specific RGs in advance to support a project and then set RBAC on them for the Development teams and/or other things like Service Principles for Azure DevOps pipelines.
Thus for any project for a given client there would be ‘Development, Test, & Production’ RGs.
From: David Coulter<EMAIL_ADDRESS>Sent: Friday, October 19, 2018 11:45 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Bokleman, Ron<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Example Resource Group? (#16745)
@ronbokhttps://github.com/ronbok I'm confused why you need/want a Resource Manager template for just creating a Resource Group when a Resource Group is a built-in artifact within Blueprints. Why does it need to be a template artifact and not just use the Resource Group artifact?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/16745#issuecomment-431407780, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHHxLFyCEEL_Mw1U8Vuop0eUcVMFVBFEks5umfOPgaJpZM4XZ4Xa.

@ronbok Blueprints can help with this!  When you create the Blueprint, configure a Resource Group artifact, any needed role assignments, and then the Resource Manager template that has the rest of your configuration.  Then, just deploy the Blueprint to your Subscription and it does everything together, Resource Groups included!
Since this was more a question about Blueprints, I'm going to have the team change the label.  Since this is the answer to what you need, I'm also going to have them close it.  Feel free to ask more questions about docs and check out the User Voice for Azure Governance if you have product asks.  Thanks again!
@MicrosoftDocs/azure-cxp-triage Please change label to 'product-question' and #please-close.


Bug: DeadlineCloudSubmitter-2<IP_ADDRESS>-linux-x64-installer.run in unattended mode installs to /root/DeadlineCloudSubmitter in installScope system

Expected Behaviour
Perhaps this is something that I am misunderstanding, but when I install with --installscope system I thought that the submitter would get installed to /opt/DeadlineCloudSubmitter, I could have sworn it used to do that? But maybe that was my imagination.
I would expect a --installscope user to go to "$HOME/DeadlineCloudSubmitter" and a system scope to go to somewhere like "/opt/DeadlineCloudSubmitter".
Current Behaviour
It appears to maybe ignore the --installscope setting and always go to "$HOME/DeadlineCloudSubmitter" so when I run the --installscope system as a sudo user, it goes into "/root/DeadlineCloudSubmitter"
Reproduction Steps
Install DeadlineCloudSubmitter in an unattended mode using the code snippet below
Code Snippet
./DeadlineCloudSubmitter-2<IP_ADDRESS>-linux-x64-installer.run --unattendedmodeui none --mode unattended --enable-components deadline_cloud_for_maya,deadline_cloud_for_nuke,deadline_cloud_for_houdini,blender_integrated_submitter --installscope system

Yea, this is definitely not the intended behaviour. We were able to reproduce this on our end on Linux.

Was just double checking the main installation component, and it sets installdirprefix based on the installation scope, which is also what's set on the GUI. If it's system, it'll user platform_install_prefix, which according to https://releases.installbuilder.com/installbuilder/docs/installbuilder-userguide/variables.html only shows:
    platform_install_prefix: The platform specific default installation path
    Example values:

        /home/user, C:\Program Files, /Applications

This would indicate to me that it's not just an unattended issue. Probably have to manually change that value for Linux.

Thanks @epmog
I'll manually set it in the meantime!


fix py & dependencies version for build

fix error like: cannot import name 'soft_unicode' from 'markupsafe'

Codecov Report

Merging #13 (b58bb7c) into master (a237778) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #13   +/-   ##
=======================================
  Coverage   77.77%   77.77%
=======================================
  Files           4        4
  Lines         135      135
=======================================
  Hits          105      105
  Misses         30       30

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 a237778...b58bb7c. Read the comment docs.


2.0.26 in OpenWrt

Resolve process in 2.0.26 is very slowly. Dnsleaktest shows 1-3 servers while in .25 was 10-15+. dnsmasq and .toml configuration from Wiki and didn't change since .25.

Is that using DNSCrypt? DoH? Both?

What CPU is your router running?

Is that using DNSCrypt? DoH? Both?
What CPU is your router running?

Both.
TP-Link TL-WDR3600 (Atheros AR9344), OpenWrt built from sources: "openwrt-18.06" branch v194.2-r7853-f6de1fa6c6. Uptime is 45min, load average from LuCI is 0.01, 0.02, 0.00.

I think I found the issue. It's not exactly related to OpenWRT, but rather the Mips CPU of these routers.
In version 2.0.26, some cryptographic operations have been switched to using a new library (CIRCL, by Cloudflare).
Unfortunately, that library doesn't work properly on big-endian CPUs.
I've switched back to the previous library and released version 2.0.27.
As a bonus, version 2.0.27 is a little bit smaller than 2.0.26.
Give it a try!


Mod Maker Feature - Standalone installer needed.

It doesn't work for some reason.
I try to mod Darkest Dungeon 2 and UABEA is the only tool that allows me to mod the game (for now).
It was Godsend, but without the installer feature, I've to force everyone who wants to use my mod to replace the whole asset file just to see a change for a single 2D texture.
This brings up compatibility issues, it can't show more than one mod and is forced to replace each other out, rather discouraging people who want to jump into the modding scene. I also have to sacrifice my previous mod for a future mod that changes entirely different texture2D.
Is it possible to fix this?

Duplicate of #86. You can mod assets files for the time being and install them through command line as explained in that issue. Bundles can't be modded due to a bug in the AssetsTools.NET library which will be fixed when I have time to review the pull request. As for why the standalone installer isn't supported, I'd at least want to find a cross platform solution for installing (and newer versions of .NET are hard to package in one executable file).


Accessibility Audit - Carbon Website: Components - Tabs

This epic is tied to a larger "release" that is meant to track the progress of the Accessibility Audit for the Carbon Website. We will audit all of the pages of the carbon website based on AVT 1, AVT 2, & AVT3 - which can be found here.

AVT 1: will audit the site against the IBM Accessibility Checker - browser extension

AVT 2: will audit the site against accessible keyboard interactions

AVT 3: will audit the site against screen readers

Legend:

🌶 fully audited - on all three AVT levels

:one: 🔍 AVT 1 audited
:two: 🔍 AVT 2 audited
:three: 🔍 AVT 3 audited

:boom: issues and documentation - on all three AVT levels are complete

:one: 🚨 AVT 1 issues submitted
:two: 🚨 AVT 2 issues submitted
:three: 🚨 AVT 3 issues submitted

:sparkles: tickets are triaged

Current status of this page: 1⃣ 🔍 AVT 1 audited & 🚨 AVT 1 issues submitted

Components - Tabs

Usage: 8 issues
Style: 3 issues - no new issues
Code: 8 issues
Accessibility: 3 issues - no new issues

Closing due to inactivity. Will reopen if/when ready to be worked on again.


(DTR Generators) Reset Button Not Working

Currently (v1.2.0-beta.1), the reset form button is not working. The fields aren't cleared and the JO list isn't removed. The generated DTR also remains.

Fix is applied in beta version release v1.1.0-beta.1. As no issues are found, this will be closed and is now guaranteed to be in the release v1.1.0.


used  boolean instead of int to avoid TLE and optimized code using Sieving till root

Description of Change

used  boolean instead of int to avoid TLE for large N in sieve_of_eratosthenes.cpp since boolean takes 2 bytes whereas int takes 4 bytes and also optimized code using Sieving till root
Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] Sort by alphabetical order
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

@mishraabhinn Should I make issue first about previous code because previous sieve code was running for N time in the loop but It can be optimized by running sqrt(N) which gives the same result. Also, the previous code was using int which was taking 4 bytes but can be improved by using boolean which takes 2 bytes.

I got the point and agree on it.
You are getting an error because std:: is not used before cout, cin etc.  Do that.
Don't use using namespace std.
Indent your code accordingly , so it conforms to Google's coding style guides.
You can use clang-format or do that manually.

as an addendum to @mishraabhinn's nice review, please note:

this repository is for educational purposes and not to provide 100% performance optimized code
hence, if you would like to submit a different implementation for an existing algorithm, please submit it as a new file in the corresponding folder.

thank you for the suggestion @kvedala @mishraabhinn I will add a different implementation for the existing algorithm and for this PR I will revert my code and will remove bits/stdc++.h from existing sieve_of_eratosthenes.cpp because it is importing bits/stdc++.h

thank you for the suggestion @kvedala @mishraabhinn I will add a different implementation for the existing algorithm and this PR I will revert my code and will also remove using namespace std; from existing sieve_of_eratosthenes.cpp  and add std:: wherever it is needed

thank you for the suggestion @kvedala @mishraabhinn I will add a different implementation for the existing algorithm and this PR I will revert my code and will also remove using namespace std; from existing sieve_of_eratosthenes.cpp and add std:: wherever it is needed

Thank you @iamrajiv. You can simply close this PR and start afresh.


"Supported OS versions" link gets 404

Under the "Prerequisites" heading, "Supported OS versions" is a bad link.

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

ID: b3462131-c17c-c9e1-106c-390e0bdb523d
Version Independent ID: bd203f82-3725-6df6-1aef-9ffad92fa7fa
Content: Publish a bot to Azure from Bot Framework Composer
Content Source: composer-docs/how-to-publish-bot.md
Service: bot-composer
GitHub Login: @emgrol
Microsoft Alias: v-eolshefski

@BruceHaley Link fix should be live sometime after noon.


fix: resolve _root_ in macro, syntax, elab

This is not just a convenience feature: Because _root_ is resolved in def, if we do not also resolve it in macro there will be a mismatch between the syntax kind used for the definition and the one used for the parenthesizer / formatter. This comes up in practice for register_simp_attr, which is defined like:
namespace Lean.Meta
...

macro (name := _root_.Lean.Parser.Command.registerSimpAttr) doc:(docComment)?
  "register_simp_attr" id:ident : command => ...

leading to a parser that puts the syntax kind Lean.Meta._root_.Lean.Parser.Command.registerSimpAttr on generated nodes, and a parenthesizer that is expecting to find syntax kind Lean.Parser.Command.registerSimpAttr.

Ping on this one, it is the cause of a bug in mathport. How should I address the issue about name joining?

there should definitely be a name function to do this rather than reinventing it all over the place.

Do you want to add such a function (possibly #2341)? Then I think this is good to go. Actually, why do we need the stage 0 update?

@Kha Should this PR wait on #2341? I'm fine with landing that and updating this to use it.

@Kha perhaps the difference between the two CI runs answers your question about why the update-stage0 is needed.

@Kha Should this PR wait on #2341? I'm fine with landing that and updating this to use it.

Could you cherry-pick it into this PR (and adjust if desired)? I would prefer it to be part of a PR with clear impact


Support single page custom handlers

Issue https://github.com/tipsy/javalin/issues/812

Thanks, nice work @pgross41 !

This has been released as part of 3.7.0 now!
https://javalin.io/news/2020/01/01/javalin-3.7.0-released.html


Questions about learning rate and batch size

In the general configuration, Lr=0.02, the batch size on each GPU is 16, your training plan uses 8 GPUs, I use four GPUs for training, does it mean that my batch size is the second of when I use 8 blocks One part, still said that it will not be affected?
I found that when I use 4 GPUs, the network training and convergence speed is very fast, but when I use two GPUs to retrain from the breakpoint, the loss increases than before, and the training speed is about 10 times slower. Logically speaking, it should be about twice. Another point is that I use two cards for training, do I need to set Lr to half of the 4 cards?

Another point is that I use two cards for training, do I need to set Lr to half of the 4 cards?
sure

Another point is that I use two cards for training, do I need to set Lr to half of the 4 cards?

yes

It means that if the lr of 8 cards is 0.02, then I need to set it to 0.005 when I use 2 fast cards?

It is reasonable that the setting of distributed training should not be so, because I think the data on each card is only shared when calculating loss, and is isolated at other times, so why is the total batch size numGPU*oneGPU batch?

It is reasonable that the setting of distributed training should not be so, because I think the data on each card is only shared when calculating loss, and is isolated at other times, so why is the total batch size numGPU*oneGPU batch?

Could you explain your question more clearly?

Feel free to reopen the issue if there is any question


Add level for ul/ol and index for li

Allows for additional customisation of nested lists
Addresses #4

Thanks for the PR. Unfortunately this would require users to be aware of the level and index props in all cases to avoid passing the props down to React elements. This goes against the primary intent of this library which is to eliminate the cognitive overhead and documentation peeping that's required to use @sanity/block-content-to-react directly.
For example, the following is safe today,  but would not be with this change:
<PortableText
  serializers={{
    ul: props =>
,
    li: props =>
,
  }}
/>

I would be more comfortable accepting a list serializer and using it as a fallback when ul or ol are not provided, and passing the level prop to it. I don't believe there's a good solution for passing the index prop to li without adding an includeIndices option that works for all types (which gets messy).
This said, you can likely get by without either level or index using CSS provided you aren't treating lists at different levels with different behavior. If you are rendering components or changing behavior conditionally based on the level, you could solve this using Context. E.g.:
const LevelContext = React.createContext(0)

const List = (props) => {
  const level = useContext(LevelContext)

  return (
    <LevelContext.Provider value={level + 1}>
      {level > 0 ? <NestedList {...props} /> : <TopLevelList {...props} />}
    </LevelContext.Provider>
  )
}

@coreyward Fair enough. I think my use case calls for the lower level library. I like the light and straightforward approach you're taking and I think it's the right thing to keep the levels/indexes out of it.


Correct a word on line 56

@Keatnuxsuo : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


hello,utilities of merge lora have some issues

./venv/Scripts/python.exe "networks\merge_lora.py" --save_precision float --precision float --save_to "F:/3.safetensors" --models "F:/1.safetensors" "F:/2.safetensors" --ratios 0.75 0.25
Traceback (most recent call last):
File "G:\anaconda3\envs\base310\lib\site-packages\gradio\routes.py", line 337, in run_predict
output = await app.get_blocks().process_api(
File "G:\anaconda3\envs\base310\lib\site-packages\gradio\blocks.py", line 1015, in process_api
result = await self.call_function(
File "G:\anaconda3\envs\base310\lib\site-packages\gradio\blocks.py", line 833, in call_function
prediction = await anyio.to_thread.run_sync(
File "G:\anaconda3\envs\base310\lib\site-packages\anyio\to_thread.py", line 31, in run_sync
return await get_asynclib().run_sync_in_worker_thread(
File "G:\anaconda3\envs\base310\lib\site-packages\anyio_backends_asyncio.py", line 937, in run_sync_in_worker_thread
return await future
File "G:\anaconda3\envs\base310\lib\site-packages\anyio_backends_asyncio.py", line 867, in run
result = context.run(func, *args)
File "F:\python\ai painting\sd-webui启动器【一键启动器】\train\script\kohya_ss\library\merge_lora_gui.py", line 60, in merge_lora
subprocess.run(run_cmd)
File "G:\anaconda3\envs\base310\lib\subprocess.py", line 503, in run
with Popen(*popenargs, **kwargs) as process:
File "G:\anaconda3\envs\base310\lib\subprocess.py", line 971, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "G:\anaconda3\envs\base310\lib\subprocess.py", line 1440, in _execute_child
hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2]

Hi, i think you're asking in the wrong repo

Thank you for your reply


Custom Surveys for Washington Commons Project

We're working with the Washington commons project to implement their custom surveys.
Right now I have the xls and xml files for the surveys they sent over. I have noted some extra options in the demographic survey that might be a mistake, so I need to ask about those and make sure the survey is final before I go through with getting the json files, since those are more labor intensive to generate.

I have tested this out by loading in the emulator:

saw custom demographics survey instead of default
trip labels with custom survey
when survey is filled, the "1 mode, 1 purpose" text will show in the label

I went through the spreadsheets, updated names that were still "option_x" and several that were confusing -- some of the onboarding questions were using the same keys for options in questions that were very different
The names should all be intuitive now, and match the labels they represent

This is the first one, so it is OK to be so hands-on. But I do wonder what, if anything, we can do to clean up when we scale. We can't do this for every partnership, I think. Should we just allow them to ahead with the bad option labels, and let them find it in testing? Should we include this in validation?

We're going to have to make sure that the path to the surveys is valid, and if a trip survey there is a way to populate the text in the button after the user labels (this was one of the things that took some testing). I think that anything we can do to validate or run it against some phone tests is going to be very helpful.
We can try and validate that nothing is the "option_X" labels, but that won't catch the ones that were just 'weird' like I found in the spreadsheet. Some of it will probably fall to testing and we can try and warn people -- maybe put some instructions for writing surveys somewhere?


[Star Wars Saga Edition] Bugfix for skill notes in darkmode not displating

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Set the colour for skill notes so that they appear when in dark mode and light mode.

Character Sheet Info Roll20 Internal Use only.


router-link will add a parent route path

The <route-link> tag will add a parent route path after clicked, but the <a> tag will not.

before click

after click

Please ask questions on the forum or StackOverflow.


unquote parameters

for example load_module "modules/ngx_stream_module.so";
we will get a quoted string now
{
        "Block": null,
        "Name": "load_module",
        "Parameters": [
            "\"modules/ngx_stream_module.so\""
        ]
}

@virusdefender  thanks for reporting. You can trim it before using. This needs some more developements to keep parameters with sth like quoted flag. Because those are needed to be quoted on dumping. I will think on it to find an easy way without a gib refactor but I would like to keep as is. (if you have any advice, do not hesitate to open a PR)

closing this, I think unfolding should not be a part of parsing.


I cant get the integration to showup on lates home assistant build

I installed via hacs but nothing shows up anywhere

HACS

Navigate to integrations section.
Click "Explore & Add Repositories" in the bottom right corner.
Search for "Baby Buddy".
Click "INSTALL THIS REPOSITORY IN HACS".
Click "Install".

did that, forgot to hard refresh my browser. all good


[FEATURE] Clear all messages button

Is your feature request related to a problem? Please describe.
A button to clear all messages.
Describe the solution you'd like
User should have a one click option to delete all messages
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@MitulSonagara May I work on this issue


AstroDAO is not compatible with 2FA

Getting gas limit exceeded error.

Via VG:
I think I know the source of issue
https://explorer.near.org/transactions/3atxf8WEVQd9AUq3X8esheQ1bSdfPbvpABGn3qng9qMN
looks like Astro UI attaches 250 Tgas for the add_proposal call
but wallet also gives 250 Tgas for whole 2FA call, so there is no space
it is also definitely an overkill, e.g. here  250 Tgas attached 7 Tgas used:
https://explorer.near.org/transactions/2KhQFAETK39NzFpsFyH2UdZz4934FHfNWkPhd4vA1nzj

Is confusing because gas is not measured in NEAR. Instead, measure in Tgas (same as it shows in explorer).
Also confusing that it's not obviously a text box which can be edited.
Default value should be low (100 Tgas).

Verified on dev. Fixed. Now TGas default value is 0.1

Verified on testnet. Fixed


Update .gitignore to ignore editor artifacts

This update tells git to ignore backup files, generated by emacs and vim while a file is being edited.

Editor specific ignores should generally be in .git/info/exclude instead, as they are not relevant to the project as a whole.

I agree with @CryZe, I don't think .gitignore is the right place for editor artifacts (and the same goes for platform-specific special files). I appreciate the PR, but generally I just don't think this is a problem. It annoys a select few viewers of the stream far more than it annoys me; in fact if I wasn't livestreaming this wouldn't even come up :)

I think what I'll do though is just fix my local config and keep this open so I can mention it on the stream.

for (local-)system dependent stuff like this, you might be better off with a global excludesFile  configured in ~/.gitconfig.
From https://git-scm.com/docs/gitignore:

Patterns which a user wants Git to ignore in all situations (e.g., backup or temporary files generated by the user’s editor of choice) generally go into a file specified by core.excludesFile in the user’s ~/.gitconfig. Its default value is $XDG_CONFIG_HOME/git/ignore. If $XDG_CONFIG_HOME is either not set or empty, $HOME/.config/git/ignore is used instead.


building package-lock.json with npm v7

Closes: https://github.com/oceanprotocol/ocean.js/issues/699
Changes proposed in this PR:

Deleting package-lock.json and rebuilding package-lock.json with npm v7

This had previously been done here: https://github.com/oceanprotocol/ocean.js/blob/7b6621d69e58708819c5cd0cce92451ac8489d1a/package-lock.json#L4 but it was overwritten


Debian / Ubuntu base conda-notice.txt error

Issue
Hello,
It seems that the base Debian / Ubuntu images add the following code to /etc/bash.bashrc and /etc/zsh/zshrc:
if [ -t 1 ] && [ "${IGNORE_NOTICE}" != "true" ] && [ "${TERM_PROGRAM}" = "vscode" ] && [ "${CODESPACES}" != "true" ] && [ ! -f "$HOME/.config/vscode-dev-containers/conda-notice-already-displayed" ]; then
    cat "${DEV_CONTAINERS_DIR}/conda-notice.txt"
    mkdir -p "$HOME/.config/vscode-dev-containers"
    ((sleep 10s; touch "$HOME/.config/vscode-dev-containers/conda-notice-already-displayed") &)
fi

However, the file ${DEV_CONTAINERS_DIR}/conda-notice.txt isn't present by default and triggers an error when opening a terminal for the fist time:
cat: /conda-notice.txt: No such file or directory
vscode ➜ /workspaces/condabug $

Additional information

VSCode Version: 1.71.0
Local OS Version: macOS Monterey 12.5.1
Local chip architecture: x86
Reproduces in: Remote - Containers
Name of Dev Container Definition with Issue:  at least base:bullseye and base:focal

Steps to Reproduce:

Create a devcontainer using the following config files below
Open up a new bash terminal window
Observe the warning cat: /conda-notice.txt: No such file or directory

Dockerfile
ARG VARIANT=bullseye
FROM mcr.microsoft.com/vscode/devcontainers/base:${VARIANT}

devcontainer.json
{
  "name": "Conda warning test",
  "build": {
    "dockerfile": "Dockerfile"
  },
  "remoteUser": "vscode"
}

Workaround
Add
ENV IGNORE_NOTICE=true

to your Dockerfile
Impact
Very low, it only manifests on first startup of the terminal.

@samruddhikhandale @joshspicer Is this a regression for https://github.com/devcontainers/images/tree/main/src/anaconda ?

Fix - https://github.com/devcontainers/features/pull/135