[Bibox] Fixed get balances

@timmolter Please have a look at 1a654ba. Any idea why the compilation fails and forces me to do that? I would expect the lombok dependency to be resolved with the dependency to core...

It's in the root pom though, what should we do?

actually, they way you did it is correct. It's only defined in the parent pom, and only the modules that depend on it should explicitly define it.

ok so I think you can merge :)


Unmet plugin dependencies

With latest checkout of girder and Minerva, I am getting unmet dependencies: (Message below). But no-where (at least in my quick look), I could find what I missing.,
Minerva Geospatial ApplicationVersion 0.1.0EXPERIMENTAL
An application for geospatial analytics.
This plugin has unmet dependencies.

@aashish24 when do you see this?  from git?  pip?  python -m girder? npm?

My guess is that you're missing the romanesco plugin.

My guess is that you're missing the romanesco plugin.

@jbeezley you are probably right (although I thought I had it installed and setup last time). One thing was that there was no error message visible on system console or web-console. Also, Girder UI just had the message but no further information. I am wondering if we can make it more clear? May be it is a Girder issue.

You were definitely missing one or more of romanesco/jobs/gravatar. I added that message because Girder was previously throwing a KeyError backtrace in the web console.
I'll add a PR to Girder that will state the missing top level dependencies in the error message.

I'll add a PR to Girder that will state the missing top level dependencies in the error message.

@danlamanna that would be most useful! Please link it to here so that we can close this issue.

This has been merged into Girder master.  We'll get the behavior when we upgrade to Girder v1.4.2.

related to https://github.com/Kitware/minerva/issues/276

Fixed by #302.


Missing line_number in err method call

Describe the bug
In the parse_kvn_oem method, if the CCSDS_OEM_VERS keyword is missing as the first line, the err method is called but is missing the line_number parameter value. This results in a TypeError being thrown.
https://github.com/bradsease/oem/blob/main/oem/parsers.py#L53
PR for fix: https://github.com/bradsease/oem/pull/81
Steps to Reproduce
Provide any OEM file that does not have the CCSDS_OEM_VERS keyword as the first line
Python/Package Version Information
Python: [e.g. 3.5.1] 3.11.6
oem: [e.g. 1.0.0] 0.4.0

Thanks for the fix, @josephgruber!


Grpc module became useless

Expected Behavior
Grpc module like in previous version
Actual Behavior
Why you decided to change ui from message list to "draw all messages in one screen"? Js can't handle this amount of data and stuck always
Reproduction Steps
No response
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information

Insomnium Version
0.2.3-a
What operating system are you using?
Ubuntu
Operating System Version
Ubuntu 23.04
Installation method
Download from releases
Last Known Working Insomnium version
No response

Can you provide some more information about what you are doing here, and if it worked at any version in the past?
I tested this with lots and lots of messages without seemingly any performance issues:


deps: Update github.com/alecthomas/chroma v0.9.4 => v0.10.0

I think I was overly concerned with this change.

My take exactly.

I think this will have minimal style side effects.
I think I understand why they wanted to do this.
So, given that Chroma is such a great upstream library (extremely well maintained), I don't think we can ... complain about it ... So I suggest we just merge, but I wanted a second opinion.


Submenu doesn't adjust their position when there is no space available

Bug report
Current Behavior
Submenu's get cropped instead of rearranging when there is not enough space for it to be completey shown.

Expected behavior
Submenu should auto adjust it's position when there is not enough space where it is supposed to open

Reproducible example
Easily reproducible on radix docs for dropdown menu.

CodeSandbox Template
Suggested solution

Additional context

Your environment

Software
Name(s)
Version

Radix Package(s)

React
n/a

Browser

Assistive tech

Node
n/a

npm/yarn

Operating System

Hi @raviteja83
Yeah we are aware of this.
Here are a few related issues: #945, #973, #557.
I will close it as a duplicate of #945 for now.


Add privacy statement

Closes #16

@KiOui we should add a cronjob to the docker that executes ./manage.py dataminimisation every night


fix(scss): show logos of job-listings page correctly

Types of changes

[x] Bugfix
[ ] New feature
[ ] Refactoring
[ ] Breaking change (any change that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Other (please describe)

Description
The original width is insufficient to show some logos with larger width. This commit will unset the width restriction.
A bonus commit is to update .gitignore.
Steps to Test This Pull Request
Steps to reproduce the behavior:
Go to job-listings page to check the logos are shown correctly.
Expected behavior
The width is sufficient to show a logo, e.g.

More Information
Screenshots
The buggy frontend examples

This one is currently in the production site.

Because the buggy logo looks are already in the production site, I have enabled "allow maintainers to modify this pull request". Please feel free to tweak my PR directly if you have a better implementation thoughts. Let's land it ASAP.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #898   +/-   ##
=======================================
  Coverage   76.25%   76.25%
=======================================
  Files          79       79
  Lines        2927     2927
=======================================
  Hits         2232     2232
  Misses        695      695

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 74ac651...7d16e12. Read the comment docs.


Design beneficiary tags

There are a few options to consider:
1 Beneficiary tags only stored against beneficiary object
2 Ben tags copied to meetings as a separate array
3 Ben tags included within meeting's tag field
I think 3 is the best option as reporting (the slowest query) remains simple to perform and fast. Beneficiary tags will change infrequently so keeping tags up to date on the meeting objects should not be hard. We do need to be able to tell the frontend which tags are beneficiary tags and which are meeting tags though, could:
A Keep separate array on a meeting detailing which tags are ben tags
B Look up tags from beneficiary
C Keep a tag, meetingTag and benTag arrays.
B will be easiest, but would need a separate lookup when ever a meeting is obtained. A should be more performant and won't be more effort than storing that tags in the meetings anyways. C could be tricky when deleting tags to know whether they should be deleted from tag as well.
So decided on 3A.

Introduce beneficiary object and query, should just have tags, org and an ID atm

Change beneficiaries query to use beneficiary storage as well as meetings

Add beneficiary mutation which allows setting tags, this should update ben tags in ben's meetings

Meeting object should have a tag, meetingTag and benTag arrays so the frontend can format them correctly to the user.

addMeeting and addRemoteMeeting should look up beneficiary tags and set tag and benTag for the meeting

editMeetingTags should not allow the editing of tags in tag which feature in the benTag array

report, exportReport, GetFilteredOSMeetingsInTimeRange and GetFilteredOSMeetingsForBeneficiaries won't change

suggestQuestionnaireTags won't change - i.e. keep looking at record tag arrays

suggestTags for beneficiary, should base suggestions on meeting tags only. This should get all tags against a ben's meetings then remove the ben tags from that set.

tags should expose ben and meeting tags, should look for tags in beneficiary collection as well as records

 Beneficiary Tags


Touch to turn page on Windows not working in full screen view

Using Thorium 3 on Surface Pro 11 (ARM). When the reader is not in fullscreen mode but the window is maximized, the < > buttons are visible. They operate correctly to change pages forward or backward.
When the reader is in fullscreen mode, the < > buttons are not visible and neither tapping nor swiping the margins will change pages.
Windows 11 24H2, Snapdragon X 64-bit ARM

Hello, this is not the correct GitHub repository to post Thorium issues (this is the documentation source, not the application source code).
Could you please re-post at https://github.com/edrlab/thorium-reader/issues
Thank you :)
PS: touch / single-finger "swipe" gesture is (should be) supported, I tested it on an old Microsoft Surface Pro 3 hybrid tablet computer.


AimEntity gate errors upon being wired to a prop

Steps to reproduce:

Spawn AimEntity gate
Wire from input 'Ent' to a prop

Produces error
[Wiremod] lua/wire/gates/entity.lua:680: attempt to call method 'GetEyeTraceNoCursor' (a nil value)
  1. output - lua/wire/gates/entity.lua:680
   2. CalcOutput - lua/entities/gmod_wire_gate.lua:119
    3. unknown - lua/entities/gmod_wire_gate.lua:101 (x21)

Needs an IsPlayer check for the Ent input


Update versions.csv

i suspect doing this without actually loading any of the data will fill the website with a bunch of empty space (or, break it entirely)

Like eevee said, there's no point in doing this before the rest of the data is added.

is there a way to add the OR AS data?

Unfortunately the 3DS carts are encrypted so it's not possible to do a direct dump of the game data like it was for previous versions. For X/Y and now OR/AS it had/has to be done manually which takes time.

Actually, it's very possible and relatively fast.  Serebii does it every time a game comes out for his Pokédex. Project Pokémon will provide the data if they are asked for it, I'm pretty sure they will contribute.

It's true, at this point many others have done the work and could probably provide the data for this project. But that's why it couldn't happen right when OR/AS was released.

I'm sure Project Pokémon would willingly contribute what they have, but @eevee already knows how to contact us when the time comes.

I had the impression the main reason the OR/AS data isn't up was just that Zhorken left veekun and nobody's stepped up to fill the vacuum they left.

Pretty much.


Add some more basic features to the site

Some more basic features can be added without increasing much work for beginners like:
Adding an image/logo to the title of the web page

Can I work on this issue?
@chrisdavidmills

This is not needed. It is just supposed to be a basic example, not anything like a proper web page.


oneshot: 1.2.0 -> 1.3.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/oneshot/versions.
meta.description for oneshot is: "A first-come-first-serve single-fire HTTP server"
meta.homepage for oneshot is: "https://github.com/raphaelreyna/oneshot"
meta.changelog for oneshot is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.3.1 with grep in /nix/store/qdk93w28h8qmsajz5amhxbdfk6s59d0f-oneshot-1.3.1

directory tree listing: https://gist.github.com/83354a4cfbe12f89c900c1a560e0d731

du listing: https://gist.github.com/bea35b667e1a27d329079ba8cf45baa5

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
oneshot

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qdk93w28h8qmsajz5amhxbdfk6s59d0f-oneshot-1.3.1
ls -la /nix/store/qdk93w28h8qmsajz5amhxbdfk6s59d0f-oneshot-1.3.1/bin

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

1 package built:

oneshot

Maintainer pings
cc @edibopp @kalbasit for testing.


Refacto the transactions updater architecture

How it works now
The current updater is running in background via redux-saga. An update is launch each time an UPDATE_TASK action is dispatched in the store.

getTxsGenerator gets the account to update and extract all the addresses (from the current state of the store). Then it launches several LDK generators (txsFetchGenerator) to fetch and unblind transactions from the explorer.
The results (= the unblinded txs as TxInterface) of all generators are sent to redux-saga channel.
That channel is connected to a txsTransformerWorker which is a function converting ldk.TxInterface (from the channel) to an internal interface named TxDisplayInterface.
TxDisplayInterface has a transfers member used in UI to compute the user-friendly informations about the transaction (Inbound & outbound values). In order to do this, the worker is recomputing all the wallet scripts from state and send them with the tx in getTransfers function.
the TxDisplayInterface object is the one persisted in redux state. once fetched, unblinded and computed, TxInterface is not accessible anymore by the UI.

The problem
The steps (1) and (4) are really sensitive because they need the wallet scripts. But those scripts come from the accounts restorerOpts (the last internal/external indexes) from state. If for any reason, the state is not up to date in background during the execution of step (1) or (4), the TxDisplayInterface will be wrong and UI will display bad informations to the user.
Proposals
I see two refactors to fix the update txs process:
A: We persist TxInterface in the redux state instead of TxDisplayInterface, getTransfers is executed only while needed in the UI. It means the UI reflects exactly the current state value (inbound/outbound are computed when user is looking at tx history, not during the update step).
B: Add an event based unblinder worker. Fetching & unblinding would be triggered by the UPDATE_TASK exactly like it works currently. But INCREMENT_*_INDEX actions could also trigger an unblind process, iterating through persisted TxInterface in the state and trying to unblind outputs associated with the address' script.
cc @tiero @altafan

I think we should explore a deep refactor of current architecture eventually, I think B option is the browser extension way (ie. event driven) to have less coupling between components and most importantly to work nicely in a context where users can open/close browser etc...

What is the impact of A)? It sounds like a breaking change for a user updating from a previous version. He would have a list of TxDispalyInterface in the redux store, while marina would now expect a list of TxInterface. We may want to have both instead, but I honestly prefer option B).
Incementing the index means generating a new address, and that must be known by the component that fetches and unblinds utxos for sure.

What is the impact of A)?  It sounds like a breaking change for a user updating from a previous version. He would have a list of TxDispalyInterface in the redux store, while marina would now expect a list of TxInterface.

for the user nothing because he is not using directly the store. A needs some migration logic but we already did it in the past (basically, u clean the whole txs state to force refetch all at the next update).

We may want to have both instead, but I honestly prefer option B).

Was not clear my bad but B needs A because TxDisplayInterface does not have enough data to handle unblinding so to implement B we need a change in the transactions state.

closed by #436


Default value of line delimiters when a patch is applied (#228)

Fixes #228
Since a structured patch does not define its line delimiters (#157), this change is needed to prevent an error when the returned value of structuredPatch is used in applyPatch. The current API does not change:
const oldContent = 'ANYTHING';
const newContent = 'SOMETHING';
const patch = structuredPatch('test.txt', 'test.txt', oldContent, newContent);

expect(applyPatch(oldContent, patch)).to.equal(newContent);

Hmm. This PR is roughly a duplicate of https://github.com/kpdecker/jsdiff/pull/384. I have the same doubt about it as I do about #384, though: wouldn't it make more sense to update structuredPatch to include a linedelimiters property in its output, thus making its return type consistent with parsePatch, rather than modifying applyPatch to accommodate the (arbitrary-seemingly, probably accidental?) inconsistency between the two patch formats?

I provisionally plan to fix this by fixing structuredPatch, and to leave applyPatch alone (and close both this and #384), but I am interested in hearing out any argument for why modifying applyPatch would be better!

After looking more closely, I think we should just rip out all the linedelimiters logic from parsePatch and always assume, in there and in applyPatch, that the line delimiter is a \n. The stuff parsePatch is doing, letting you have a patch where lines of the patch are delimited by, say, vertical tab characters, is inconsistent with the diff and patch Unix tools, seems kinda pointless to me, and breaks jsdiff's ability to correctly parse a unified diff format diff generated using diff -u foo bar if foo or bar contained any of the characters (like vertical tabs) that jsdiff treats as line breaks in the diff but that the diff and patch CLI tools just treat as ordinary characters.
I'm gonna merge this, and then put up another PR fixing the linedelimiters stuff.


Cannot play WAV files

trying to use wav files and i can't get them to work no matter what using phiola. Plays in any other media player just fine, so I'm unsure what could be going on. I am running these through the CLI.
The first file I tried is 353kbps, int16, 22050Hz mono, no dice
then the second file i tried, something more modern, 2822kbps int32 44100Hz stereo, also wouldn't work.
Both files output the same error:
01:27:11.720 #25968 ERROR  wasapi: *1: open(): unsupported format

Any help would be massively appreciated!

Hi! Looks like phiola can't get the audio format that is set by default in your Windows audio settings. Let's find out why is that. Here are a few options:

Execute
phiola -D ....wav | findstr wasapi:
This will print several lines to you terminal.  Please paste here the line wasapi: *1: opening device ....
Execute
phiola -D ....wav -exclusive | findstr wasapi:
It's the same but now trying to open the audio device exclusively.
What's the default format for your audio device?  Try to set another format. (See https://integraudio.com/default-audio-format-windows-10/)

Lines + Error

08:40:52.738 #3272 DEBUG  wasapi: *1: opening device #0, int16/22050/1, flags:80
08:40:53.576 #3272 DEBUG  wasapi: *1: opening device #0, float32/48000/8, flags:80
08:40:54.112 #3272 ERROR  wasapi: *1: open(): unsupported format

No change running -exclusive, same output as above

My Default Format is "8 channel, 16 bit, 480000 Hz", which I believe is unique to my corsair headset. Setting it to any of the other available formats in 2 channel mode makes it work just fine.

Something I will note: Now, with the channel change, I can play all of the wav files, however, any sounds with a different Hertz from my Default Format have a considerable delay compared to the other formats. a little over half a second!

in 2 channel mode makes it work just fine.

Interesting!  I think your audio device doesn't support float32/48000/8 that phiola is trying to use but supports int16/48000/8.  This is surprising to me because WASAPI should convert float32->int16 internally, and phiola relies on this behaviour.  We can test this theory:

Change your default format back to int16/48000/8 as it was before.
Convert your audio file:

phiola convert ....wav -o test.wav -af int16 -chan 8 -ra 48000

Now try to play the new file:
phiola -D test.wav | findstr wasapi:

It should print opening device #0, int16/48000/8 and then play audio just fine.


Workflow that will compile font files

Hi :wave:!
I created a workflow that compiles the Featherity font in various formats, including TTF, Woff2, SVG Spritesheet and CSS for web use 🤗
I have tried it and there is no apparent problem 😄

https://github.com/featherity/featherity/issues/54

I accepted the suggestions and the YAML got corrupted, I already fixed it 😅

Is there a way to get the link to the artifact created for a given commit?

@fabricedesre here is the link.
I didn't test it yet, so I'm curious if you see some improvements.

Overall it's working, but some icons are not rendered properly as part of the font. Here's the "phone-call" one for instance:

Wdym, that's a feature (jk)

Oh wow that's look like a mess.
I think the convertion to paths is not going right. Maybe @yanquisalexander knows what is going on?

Oh wow that's look like a mess.
I think the convertion to paths is not going right. Maybe @yanquisalexander knows what is going on?

I'd dare say it's probably something in the SVG
I have tried the same workflow with other sources (Bootstrap Icons to mention one) and this does not happen :thinking:

Although I think it would be useful to highlight this answer...
https://github.com/featherity/featherity/issues/54#issuecomment-663731375

I see, the phone-call has merged path. I think we can fix that one soon.
For the other icons, I saw that the rounded joins are not possible font.
According to the fontcustom documentation we need to first covert stroke-paths to outlines.
So maybe we can use this library: outline-stroke. That will convert all the icons to outline strokes.


[ENHANCEMENT] Delete old replit files

Delete old replit files from the bot folder.

Closed with 4728f2e.


dont working in ie11

sets wrong value

Nice catch! Looks like an issue with getBoundingClientRect not working in IE. I'll fix it later today.

Should be fixed @helight59

thanks!


there is no file named 'UNet_plan_and_preprocess'

i only find 'nnUNet_plan_and_preprocess.py'

Please see the readme in https://github.com/MIC-DKFZ/nnUNet/tree/nnunetv1
After installing nnunet, you can run the command named UNet_plan_and_preprocess in bash, which is defined in nnUNet_plan_and_preprocess.py

in setup.py, i only find 'nnUNet_plan_and_preprocess = nnunet.experiment_planning.nnUNet_plan_and_preprocess:main', i cannot find command about: UNet_plan_and_preprocess

Please see the readme in https://github.com/MIC-DKFZ/nnUNet/tree/nnunetv1 After installing nnunet, you can run the command named UNet_plan_and_preprocess in bash, which is defined in nnUNet_plan_and_preprocess.py

in setup.py, i only find 'nnUNet_plan_and_preprocess = nnunet.experiment_planning.nnUNet_plan_and_preprocess:main', i cannot find command about: UNet_plan_and_preprocess

thanks for your contribution :), running 'nnUNet_plan_and_preprocess -t XXX -pl2d None' could solve this problem, for this issue i really want to ensure if there is a file named UNet_plan_and_preprocess.py instead of nnUNet_plan_and_preprocess.py


Update search SDK and apollo version

graphql 16.4.0 -> 16.9.0
@apollo/client 3.5.10 -> 3.10.8
Updated search-sdk using graphql-codgen
Removed searchResultCountAndRelatedCount query as it was not used.
Added searchResultItemsAndRelatedItems query for Apps (was previously using searchResultRelatedItems). They are using both search result items and related items from the query. And the searchResultRelatedItems query should only return the related items (and with these changes it will only return related items now).

/hold

FYI @jlpadilla

/lgtm

/unhold


Unable to Arm/Disarm

The integration allows me to see the current status of the panel, but arming/desarming generates an error.  This is what I see in the logs:
Error executing service <ServiceCall alarm_control_panel.alarm_disarm: entity_id=['alarm_control_panel.total_connect'], code=>
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/core.py", line 1002, in _event_to_service_call
    await service_handler.func(service_call)
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/alarm_control_panel/__init__.py", line 140, in async_alarm_service_handler
    yield from getattr(alarm, method)(code)
  File "/usr/lib/python3.5/asyncio/futures.py", line 380, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.5/asyncio/tasks.py", line 304, in _wakeup
    future.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/alarm_control_panel/totalconnect.py", line 94, in alarm_disarm
    self._client.disarm()
  File "/srv/homeassistant/lib/python3.5/site-packages/total_connect_client/TotalConnectClient.py", line 264, in disarm
    deviceId = self.get_security_panel_device_id(location)
  File "/srv/homeassistant/lib/python3.5/site-packages/total_connect_client/TotalConnectClient.py", line 149, in get_security_panel_device_id
    raise Exception('No security panel found')
Exception: No security panel found

If that helps, I am able the arm the panel with curl, using these two calls:
https://rs.alarmnet.com/TC21API/TC2.asmx/AuthenticateUserLogin?userName=<user>&password=<pass>&ApplicationID=<application_id>&ApplicationVersion=3.0.32
https://rs.alarmnet.com/TC21API/TC2.asmx/ArmSecuritySystem?SessionID=<session_id>&LocationID=<location_id>&DeviceID=<device_id>&ArmType=1&UserCode=-1

Thanks! What value do you use for the Device ID?

I don't remember where I found that number.

ok, I'll take a poke at this when I get home from work tonight. If you happen to come across a DeviceName for that panel (it will probably be in the same chunk of info that you originally got the DeviceId), let me know.

I remember now that it's from the API function GetSessionDetails.  The DeviceName is "Lynx Touch-WiFi"

Fix is merged in #15 and uploaded to Pypi, version still needs to be bumped in HASS


Invalid code generated by loop (break inside a closure)

(ps:ps
(defun foo (t1 t5)
  (loop for t2 in t1
     for t4 in t5
     collect (loop for t3 in t2 collect t3))))

generates
function foo(t1, t5) {
    __PS_MV_REG = [];
    return (function () {
        var _js54 = t1.length;
        var _js56 = t5.length;
        var collect57 = [];
        var FIRST58 = true;
        for (var _js53 = 0; _js53 < _js54; _js53 += 1) {
            (function () {
                var t2 = t1[_js53];
                var _js55 = FIRST58 ? 0 : _js55 + 1;
                if (_js55 >= _js56) {
                    break;
                };
                var t4 = t5[_js55];
                collect57.push((function () {
                    var _js60 = t2.length;
                    var collect61 = [];
                    for (var _js59 = 0; _js59 < _js60; _js59 += 1) {
                        var t3 = t2[_js59];
                        collect61.push(t3);
                    };
                    __PS_MV_REG = [];
                    return collect61;
                })());
                __PS_MV_REG = [];
                return FIRST58 = null;
            })();
        };
        __PS_MV_REG = [];
        return collect57;
    })();
};

Note that the entire outer-loop body is inside a js function but it still contains break statements.
Reverting e6489e091533eaa830a4b50b622915584c651c13 fixes this issue.
This is reduced from a regression in real-world code.

I recorded this issue in https://gitlab.common-lisp.net/parenscript/parenscript/commit/526346549f0342d67947d360a8f7ba9fc7b09e54
I am going to delete this GitHub project (I am minimizing use of
GitHub since Microsoft now owns this website). You can follow up on
the mailing list:
<EMAIL_ADDRESS>Subscription instructions:
https://mailman.common-lisp.net/listinfo/parenscript-devel
Or on the common-lisp.net GitLab instance:
https://gitlab.common-lisp.net/parenscript/parenscript


Version v8.0.0 is not working on npm

🐛 Bug Report
When you install the version v8.0.0 with npm it's throwing a workspace:* issue.

To Reproduce
Install the library @svgr/core with npm and the latest version.
Expected behavior
Install the dependency with all the package managers, not only with pnpm (works good).
Run npx envinfo --system --binaries --npmPackages @svgr/core,@svgr/cli,@svgr/webpack,@svgr/rollup --markdown --clipboard
Paste the results here:
## System:
 - OS: macOS 13.4.1
 - CPU: (8) arm64 Apple M1 Pro
 - Memory: 80.63 MB / 16.00 GB
 - Shell: 5.9 - /bin/zsh
## Binaries:
 - Node: 20.2.0 - /opt/homebrew/bin/node
 - npm: 9.6.6 - /opt/homebrew/bin/npm

I don't see any problem. Could you please point it more precisely?


fix: pass an empty string in case no category is available

Description
This PR resolves the issue where we are querying for artworks with an empty category resulting on the query failing and breaking my collection artwork screen.
PR Checklist

[ ] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[ ] I hid my changes behind a feature flag.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Fix issue breaking my collection artwork screen - kizito, mounir, ole

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Closing in favour https://github.com/artsy/eigen/pull/7170#pullrequestreview-1063546340


import method name is a reserved keyword

Prerequisites

[ ] Are you running the latest version? -> 3.9.4
[ ] Are you reporting to the correct repository?
[ ] Did you perform a cursory search?

For more information, see the CONTRIBUTING guide.
Description
I'm currently integrating cornerstone tools in a typescript project and I face a problem declaring function "import". Is it possible to rename it with another method name which is not a reserved key word ?
Thanks

I've come across this, and it does stink. I find myself doing:
// If Babel transpiles this reasonably
import { import as csTools } from 'cornerstone-tools'

const someLibraryFunc = csTools('drawing/someDrawingFunc');

// Or...
import cornerstoneTools from 'cornerstone-tools'

const someLibraryFunc = cornerstoneTools.import('drawing/someDrawingFunc');

If Typescript is yelling at you for both of the above, we could try adding an aliased export? Unfortunately, we can't completely rename it without introducing a breaking change and a major version bump.

Thanks for your answer, an alias could be a nice workaround. We could choose a method name like 'get' or 'retrieve' ? What do you think about this naming?

@JamesAPetts a full rename may be worth considering for v4.
@Igaudez, naming things is hard. Something more indicative of the behavior might be:
getModule, importModule
Or to better indicate that these are more prone to changing than top level exports:
getInternalModule

getInternalModule gets my vote. Lets do this for v4, even:
import { import as csTools } from 'cornerstone-tools' makes even our examples difficult to follow sometimes. Quilljs doesn't do everything right.
I'll add this to the Kanban board.

We went with importInternalModule haha. Merged in vNext in #958. I'll close this for now, this can't be changed until 4.0 as its a breaking change.

importInternalModule is fine. Thx

We went with importInternalModule haha. Merged in vNext in #958. I'll close this for now, this can't be changed until 4.0 as its a breaking change.

Per @JamesAPetts's comment, you'll see this change when v4 drops. If you can't wait, please feel free to PR it as an alias'd export and I'll merge the change.


Infographic on donate page

This adds an infographic to the donate page. On mobile, most elements are hidden so that the donate button isn't pushed super far down the page. Here's how you can expect it to look above 640px:
![Donate page]
(https://cloud.githubusercontent.com/assets/697973/12068801/bccbdb3a-afca-11e5-81b7-530f7155cc06.png)
Here's a breakdown of the changes:

donate.html.erb: new infographic markup
donate.css.scss: new donate page stylesheet
modules.css.scss: adjustments to existing infographic styles
about.html.erb: adjustment to existing infographic markup
lots of new SVGs in infographic/ image folder

fyi @corinnepw !

Thanks lady! Pushing this now then adding more content later :) One suggestion, @lzacharkow, is considering another "donate" button higher up on the page...any thoughts on that?

Oh my word I just died of awesome 💗💗💗💗💗

<3 can't wait to see this live! If possible, please update the 54 to be 52 match the 52 chapters referenced earlier.

Yup! Merged into staging for now and will fix the dynamic chapter count with the upcoming content changes :+1:

:clap: :clap:

🙌
On Wed, Jan 6, 2016 at 9:02 AM, Corinne<EMAIL_ADDRESS>wrote:

[image: :clap:] [image: :clap:]
—
Reply to this email directly or view it on GitHub
https://github.com/girldevelopit/gdi-new-site/pull/397#issuecomment-169330344
.

--
Vanessa Hurst
Co-Founder | Girl Develop It http://www.girldevelopit.com | @girldevelopit
http://twitter.com/girldevelopit
Girl Develop It http://girldevelopit.com/ teaches women to code in over
54 cities! Help us celebrate five outstanding years by supporting #GDIday
on January 12, 2016
https://www.thunderclap.it/projects/36239-help-more-women-learn-to-code.
Women coders break new ground, and stereotypes - TODAY show
http://www.today.com/video/today/53924117

@kstack7 I think that's a great idea! A quick win would be to add a donate button above or below the paragraph beneath the map. If we could skim this to a sentence, that would be even better for quick understanding, and hopefully quick donating :)


[BUG] Status Effect Conflict

Status Effect conflicts with vanilla inventory.

Fixed by moving the effect list to the right. Technically a new bug, but right now I don't care enough to fix.


onLoad onProgress not working IOS

I have used react-native video for playing audio files, it is working fine on android , but on ios it is not able to fire onLoad and onProgress functions which I am using to get duration and progress. Below is my code.
<Video source={{ uri: songPlaying.url }}
          ref="audio"
          volume={this.state.muted ? 0 : 1.0}
          muted={false}
          paused={!this.state.playing}
          onLoad={this.onLoad.bind(this)}
          onProgress={this.setTime.bind(this)}
          onEnd={this.onEnd.bind(this)}
          resizeMode="cover"
          repeat={false} />

 onLoad(params) {
    this.setState({ songDuration: params.duration });
  }

//called on onProgress to set time.

 setTime(params) {
    if (!this.state.sliding) {
      this.setState({ currentTime: params.currentTime });
    }
  }

This is still an issue in 2020, as of v5.0.2
To clarify: the onLoad and onProgress events do not fire on iOS when the player is audio-only.

I have this issue as well.


examples: Use ECS entities for individual instances in shader_instancing

Objective

The shader_instancing example uses a component representing many instances stored on one entity, which is perhaps surprising given users will generally think about an instance as an entity. Make instances be represented by separate ECS entities.

Solution

Identify instances by a ColorMaterialInstanced component that contains the colour of the instance
Use the translation and scale (x-component of scale only) from the entity transform as the instance position and scale.
When preparing instances, do it per-view and create an InstanceBuffer that contains a BufferVec<InstanceData>. For each instance entity, push the appropriate InstanceData onto the BufferVec and store the instance index that is returned in a InstanceIndex on the instance entity. Finally store the InstanceBuffer on the view entity
Draw each instance with a separate draw command using the InstanceIndex component

Possible extension

I was thinking we could maybe add a toggle to switch between drawing each instance with a separate draw command using the instance index, and drawing the whole batch using one command based simply on the number of instances. Alternatively we could actually try to implement batching properly for this, or a separate example.

This is a significant improvement to the instancing example. How difficult would it be to fully support the entity's Transform instead of only using position and uniform scale? Fully supporting it would probably make this example cover 99% of the cases where someone needs instancing.

Oh nice! I agree with Aevyrie's question, but even as is this looks like a nice improvement.

I like the new example but as mentioned in the discord, it showcases a different use case of instancing.
The example before this PR was "I have a lot of the same mesh with slightly different parameters and want to keep the number of draw calls low", and the new code is "I have per-mesh data that I want to access in the shader, and I'm using an instance buffer for that".
Both would be nice as an example, and I would prefer to keep this example and improve on it separately (by somehow having multiple entities in the main world be extracted and batched into a single instanced draw call).

I like the new example but as mentioned in the discord, it showcases a different use case of instancing.
The example before this PR was "I have a lot of the same mesh with slightly different parameters and want to keep the number of draw calls low", and the new code is "I have per-mesh data that I want to access in the shader, and I'm using an instance buffer for that".
Both would be nice as an example, and I would prefer to keep this example and improve on it separately (by somehow having multiple entities in the main world be extracted and batched into a single instanced draw call).

Both would be nice as an example, and I would prefer to keep this example and improve on it separately (by somehow having multiple entities in the main world be extracted and batched into a single instanced draw call).

Thanks for clarifying how this PR changes the use case. I agree that we should keep the existing example as-is and add the content of this PR as a new example. I'd also appreciate more comments explaining the "why" in both examples, aimed at people who are using these examples to learn.

Leave Comment

I like the new example but as mentioned in the discord, it showcases a different use case of instancing. The example before this PR was "I have a lot of the same mesh with slightly different parameters and want to keep the number of draw calls low", and the new code is "I have per-mesh data that I want to access in the shader, and I'm using an instance buffer for that".
Both would be nice as an example, and I would prefer to keep this example and improve on it separately (by somehow having multiple entities in the main world be extracted and batched into a single instanced draw call).

With just s small change to how the queuing and draw command is done, all entities could be drawn in one draw call. But it requires knowledge that the batching can be done.

I can definitely add in the full transform as I have done for moving MeshUniform to an instance buffer on a branch as discussed in #4288 .
I agree that we should have examples of instancing, batching, and both combined. I think we need to come up with representative cases for each.

@alice-i-cecile @JMS55 @superdump I propose we close this as stale, as recommended in the Triage Doc.


download model weights once in a Drive

model weights are downloaded in each of the works on the cloud. We can download it in a single Drive and share it with all the works

Any updates on this one?

yes, the model weights are only downloaded on the model serving work machines.

But it's downloaded for each worker every time, no?

yes, the model weights are downloaded only for the Model serving workers. Even with Drive it will have to load the model to the serving worker.


PBRMaterial to dark under safari (DDS conversion to int loses hdr info)

IOS shows PBR materials darker than other browsers. All other features of the PBR material are working correctly only the colors are to dark.
I tested Chrome, Firefox, Edge and Android-Chrome. Only IOS has that issue.
Playground: http://www.babylonjs-playground.com/index.html#FTD1E8
The left material is a PBR material and the right one is a StandardMaterial as a color reference.
Both materials should look identical.
Expected result (firefox):

Current result (ios):

I have the same issue of my iPad. Pinging @sebavan

So it all comes down from the conversion of the environment from float to int loosing the hdr information contained in the environment texture.
I will make a fix tomorrow for this.
Thanks again for reporting.

This will be available in tomorrow s nightly thanks @TrevorDev

Problem persists using "EquiRectangularCubeTexture".
Need a playground?

We need a discussion in the forum :) forum.babylonjs.com


firefox support

Hello,
We have been considering this for a long time now and it is good that you have raised it. I can share that we plan to work this in the future but cannot commit to a specific time line at the moment. Stay tuned as this issue should be updated in case of news.
Best regards,
Jordan

@jdichev is this it? https://addons.mozilla.org/en-US/firefox/addon/ui5-inspector/
there doesn't seem to be any mention of the firefox version anywhere, perhaps https://sap.github.io/ui5-inspector/ should be updated to link to it if the user is on firefox?


Fix issue where keys are getting cleared

Keys are getting cleared when steamcmd sends [0m back to client.
This occurs after the keys have already been loaded so we can safely ignore it
Not saying this is the best fix as my experience with steamcmd and Rust is quite limited but it worked on my machine & Steam account

Sorry didn't see this. Will merge on test success

Thanks!


Error in post-install script oraclejdk10

scoop install oraclejdk10
Installing 'oraclejdk10' (10.0.1-10) [64bit]
Downloading http://download.oracle.com/otn-pub/java/jdk/10.0.1+10/fb4372174a714e6b8c52526dc134031e/jdk-10.0.1_windows-x64_bin.exe#/dl.7z (390,2 MB)...
Checking hash of jdk-10.0.1_windows-x64_bin.exe... ok.
Extracting... done.
Running pre-install script...
Linking ~\scoop\apps\oraclejdk10\current => ~\scoop\apps\oraclejdk10\10.0.1-10
Running post-install script...
rm : Не удается найти путь "C:\Users\MY_USER\scoop\apps\oraclejdk10\current\tools.zip", так как он не существует.
строка:1 знак:1

rm "$dir\tools.zip"

  + CategoryInfo          : ObjectNotFound: (C:\Users\baimur...rrent\tools.zip:String) [Remove-Item], ItemNotFoundException
  + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.RemoveItemCommand

--IN ENGLICH--
Cannot find path as it doesn't exist

Shouldn’t this be filed at https://github.com/se35710/scoop-java/issues ?

I have reverted the manifest to removing the tools.zip file in the pre-install step instead. Should work now, when $dir does not change to current instead of $version.

I think this was resolved already. Could be closed.

Yes, and the oraclejdk10 does not exist anymore.

Implemented in java bucket


Dbarok/athena dbt utils

This is a:

[ ] bug fix PR with no breaking changes — please ensure the base branch is master
[x] new functionality — please ensure the base branch is the latest dev/ branch
[ ] a breaking change — please ensure the base branch is the latest dev/ branch

Description & motivation
In our team we use dbt extensively hence we found it useful to have dbt-utils available also for Athena.
Checklist

[ ] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered)

[ ] BigQuery
[ ] Postgres
[ ] Redshift
[ ] Snowflake
[x] Athena

[ ] I have "dispatched" any new macro(s) so non-core adapters can also use them (e.g. the star() source)

[ ] I have updated the README.md (if applicable)

[ ] I have added tests & descriptions to my models (and macros if applicable)

[ ] I have added an entry to CHANGELOG.md

Hi thank you very much for dbt-utils, I added these changes so that we can use this also on Athena. Doing this for the first time so I'll appreciate any feedback or comments. Cheers! :)

@dbarok This looks awesome! I'm glad you were able to add Athena support with just a handful of macros.
We're trying to be really careful about the scope of this package, which has ballooned significantly over the past few years. As such, we've invested in functionality that enables adapter-specific macros to live in external "shim" packages. These packages can be installed alongside dbt-utils and, by setting a project-level var (before v0.19.2) or dispatch config (v0.19.2 and going forwards), users can tell dbt to look for implementations of dbt_utils macros in these "shim" packages, too.
You can check out some example "shim" packages that are already up & running:

spark-utils
tsql-utils
materialize-dbt-utils

A hypothetical athena-utils package would be responsible for:

Defining just the athena__ version of each macro
Setting up CI to run integration tests against an Athena cluster

How does that sound to you?

Great thanks for the explanation and the advice @jtcohen6 🙂 ! I will try to do it that way then.

Thank you @dbarok for your work! I implemented the suggested solution of @jtcohen6 in the repository https://github.com/lalalilo/athena-utils
Please note that there are no integration tests for now, but it's a first draft and any PRs are welcome!
Thanks again!

Thanks for moving to the separate package!! Going to close this PR.


Add Helm chart for AKS managed cluster

Add Helm chart for AKS managed cluster
Note: this is originated from the following repo: https://github.com/nellyk/azure-managed-cluster-capz-helm
Co-authored-by: Nelly Kiboi<EMAIL_ADDRESS>Co-authored-by: Alessandro Vozza<EMAIL_ADDRESS>

Some whitespace cleanup required before the linter is happy:
charts/azure-managed-cluster/values.yaml
  Error: 10:1 [trailing-spaces] trailing spaces
  Error: 21:16 [trailing-spaces] trailing spaces
  Error: 45:1 [trailing-spaces] trailing spaces
  Error: 57:17 [colons] too many spaces after colon
  Error: 61:16 [colons] too many spaces before colon
  Error: 61:18 [trailing-spaces] trailing spaces
  Error: 77:26 [trailing-spaces] trailing spaces
  Error: 79:10 [trailing-spaces] trailing spaces
  Error: 83:21 [trailing-spaces] trailing spaces
  Error: 106:6 [indentation] wrong indentation: expected 4 but found 5
  Error: 108:12 [colons] too many spaces before colon
  Error: 111:2 [comments] missing starting space in comment
  Error: 111:43 [trailing-spaces] trailing spaces
  Error: 115:10 [trailing-spaces] trailing spaces
  Error: 117:16 [comments] too few spaces before comment
  Error: 125:23 [trailing-spaces] trailing spaces
  Error: 126:6 [indentation] wrong indentation: expected 6 but found 5
  Error: 1[30](https://github.com/mboersma/cluster-api-charts/actions/runs/9025955495/job/24818320639?pr=4#step:7:31):6 [indentation] wrong indentation: expected 6 but found 5
  Error: 1[31](https://github.com/mboersma/cluster-api-charts/actions/runs/9025955495/job/24818320639?pr=4#step:7:32):19 [trailing-spaces] trailing spaces
  Error: 147:22 [trailing-spaces] trailing spaces

/hold cancel

lgtm
let's merge this and address any follow-up items in a follow-up PR
rapid iteration ftw

helm repo add capi https://mboersma.github.io/cluster-api-charts
helm install <name> capi/azure-managed-cluster


Manage an Inherited Process...

[Enter feedback here]
To me (and many others, this includes COMPARING different processes... One example... Two different inherited processes from the Agile Templates:  Sales-Agile and Manufacturing Agile --- How do I compare the differences...

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

ID: f5969861-6f6b-3da1-7eb3-2ee278110937
Version Independent ID: ae1c687d-7c93-ba01-f63c-2f1df95ed873
Content: Create and manage an inherited process - Azure DevOps Services
Content Source: docs/organizations/settings/work/manage-process.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@dcorbin-wintellect - Thanks for your feedback; we appreciate it. To make sure we have all the information needed to address your issue, please let us know which article you’re referencing, or provide more context around the general task.

The original post contains a link to the page, so I am confused about what you are missing….
To repeat what I posted…   I have created an inherited process, now I need to manage that,, and a key part is being able to compare my inherited with the base (out of box) or other modified versions….. Without out this it is impossible to manage said modifications – so I would expect this to be addressed (content or link) on the page in question.
@.***
@.***
From: Tim Sherer @.>
Sent: Thursday, October 21, 2021 11:36 AM
To: MicrosoftDocs/azure-devops-docs @.>
Cc: David Corbin @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-devops-docs] Manage an Inherited Process... (Issue #11509)
@dcorbin-wintellecthttps://github.com/dcorbin-wintellect - Thanks for your feedback; we appreciate it. To make sure we have all the information needed to address your issue, please let us know which article you’re referencing, or provide more context around the general task.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-devops-docs/issues/11509#issuecomment-948730076, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AT3Q5VFXXON3QX6MR4CB2STUIAXPZANCNFSM5GOGYNTA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


possibility of host/service group awarenaes

hi all
is there any future plan to implement host or service group awareness on the front end.
Thank you

+1, this would be useful


themes

Would it be possible to have a settings option for setting the theme to dark?

The policy of the project is to keep it simple, so no, please override the template to do that.


custom_data_method controller context

maybe its just me but how do u guys use the custom_data_method because it seems i dont have any controller access. in which context is custom_data_method called? i only can find the notifier class.
maybe i use it awfully wrong but atm it seems it can only be filled with static data. cant find any dynamic examples

@bhh you are right. that method doesn't receive actually any request or context. it can however useful if you need some dynamic data from the host or something.
we have in our roadmap improve that so the defined method receives the current scope, where you will be able to get the request or other data.
Thanks for remind us this :smile:

@jondeandres additionally to scope it would be nice to have access (and it should be easier to do) to current exception. For instance I'm dealing with RestClient::UnprocessableEntity exceptions and the actual error message is attached to body. Currently I'm sending the error with this code but it's pretty ugly and not thread safe:
  config.custom_data_method = lambda do
    if RestClient::Exception === $!
      {
          status_code: $!.http_code,
          errors: (JSON.parse($!.http_body) rescue $!.http_body),
      }
    end
  end

Good point @fab, I agree with you. We'll notice you when we progress on this.
Thanks both!

Wrong @fab. I think you meant @fabn

Yah, sorry @fab :smile:

No worries. It happens every now and then. Probably also because I've contributed to this gem in the past :)

Has work been done on this, or do you have a few files you can point me to so I can get started with it?

@saulcosta18 we know having access to the request in order to send custom data is a very requested and we have plans to do this soon.
The idea would be set a new key in the scope generated by the middleware, for ex in rollbar-gem/lib/rollbar/middleware/rails/rollbar.rb and evaluate that value, that should be a Proc in lib/rollbar.rb, on #build_payload method.
We plan to do as soon as we can. Sorry for the inconveniences.

Any movement on this? It would be good if we could just pass a symbol of a controller method name that would get called.

@ajsharp I am working on implementing this right now. Should have a pull request ready sometime this week.

Does this solution only work for caught exceptions that are reported manually. Is there a way to include additional controller data for exceptions that get reported automatically?

@siav92  e.rollbar_context may do what you need: https://docs.rollbar.com/docs/ruby#error-contexts
This assumes you have access to the error object at some point while you also have the controller data. For example, you are raising the exception, or you are rescuing and re-raising it.
Also, it doesn't get sent to custom_data_method, but is added to the payload as is. (It is merged with the result of custom_data_method and any other custom data.)


Proposal for new cookbook version

Hi,
This is a proposal for a new cookbook version which would utilise the Atlassian JIRA installer and enable a standalone JIRA installation without depending on any external cookbooks.
Please take a look, thanks!

Hi @rveznaver,
There is another JIRA cookbook which already has support for the JIRA installer: https://github.com/bflad/chef-jira
You may wish to integrate your work with the work already completed there? There has been recent activity on https://github.com/bflad/chef-confluence, but chef-jira could do with some love in addition to the couple of PRs I've submitted to it.

Thanks @gsreynolds,
I thought that one was not maintained any more (I also haven't found it on supermarket).

Looks cool. I will review, thanks! I agree that the bflad/chef-jira cookbook seems to be abandoned.

I have ceased maintaining this cookbook and given over the jira namespace to these folks: https://github.com/afklm/jira


Add multiindex and multiindex column test for Expanding functions

Codecov Report

Merging #1004 into master will decrease coverage by 2.51%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1004      +/-   ##
==========================================
- Coverage   94.79%   92.28%   -2.52%
==========================================
  Files          34       34
  Lines        6568     6568
==========================================
- Hits         6226     6061     -165
- Misses        342      507     +165

Impacted Files
Coverage Δ

databricks/koalas/usage_logging/__init__.py
24.54% <0%> (-72.73%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
78.72% <0%> (-8.52%)
:arrow_down:

databricks/conftest.py
91.48% <0%> (-6.39%)
:arrow_down:

databricks/koalas/utils.py
95% <0%> (-3.13%)
:arrow_down:

databricks/koalas/frame.py
93.84% <0%> (-2.42%)
:arrow_down:

databricks/koalas/groupby.py
89.34% <0%> (-2.05%)
:arrow_down:

databricks/koalas/namespace.py
85.71% <0%> (-1.56%)
:arrow_down:

databricks/koalas/plot.py
93.33% <0%> (-0.96%)
:arrow_down:

databricks/koalas/testing/utils.py
77.5% <0%> (-0.63%)
:arrow_down:

... and 1 more

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 f302c6d...395f9a3. Read the comment docs.

I'm just merging ... it's just a test.


Not able to upgrade CLI

I have installed Maestro via script. As on December 14, 2022, it installed version 1.9.0. I tried to upgrade CLI by these instructions. It is still same version. Because of this version, I am not able to run record & studio functionality. I am testing this on WSL2.

@UnitedWithCode are you still experiencing issues with upgrading the CLI?

Hi @UnitedWithCode, I'll go ahead and close this out for now. Please file a new bug report here if you are still experiencing issues!


added comments font size option

Added a setting to switch comments font sizes.
SettingsActivity:

ListPreference:

Small (default):

Medium:

Large:

Awesome! I'll run a build of this really quick later today and merge it in, thanks!

Thanks! I love this app, it's the best Hacker News app in Android, can't live without it.
I had been looking to learn how to use ViewHolder effectively and also learn to use RxJava, so I've  been learning a lot from your app, thanks for the hard work!

Hey man, thanks! Look forward to the refactor branch going into upstream soon, I've been doing a lot of work to clean up the architecture. Unfortunately, got a little busy today, so I'll test your PR as soon as I can.

I've merged this to my local and will run it over the next day to make sure everything is alright. Luckily it merged cleanly with the refactor, so it looks good so far!

Looks good!
Small conflict in strings, but that was an easy fix, thanks for the contribution!

By the way, I've made some stylistic changes to the code and also renamed the preference to "Comment Text Size". These changes can be seen here:
https://github.com/dinosaurwithakatana/hacker-news-android/commit/1c2d164fa0e989365028b63f57f208e7658f4d2f

You're welcome, I'm glad I could help :)


feat: liquidity mining for xyk pallet

Description
This PR adds liquidity mining support for xyk pools.

Just small stuff otherwise LGTM. Did we do benchmarks yet?

Yes,  committed weights are from new benchmarks with insufficient assets


Do not take example data

Example data should not be used to backfill the object. It is there
for documentation - unlike default property.
Fixes #66

+: this also causes unexpected behavior with doing a PUT request with undefined fields that are not required. If they have an example, the PUT response body contains the example rather than the intended object.

I agree with this change, but I won't merge it yet, since it's a breaking change.  I'll include it in the next major release though.

Replaced by #99


[flutter_tools] fix top web crasher

Fixes the error:
StateError: Bad State: cannot add event after closing
at  _streamController.add
at  _StreamSinkWrapper.add
at  AppConnection.runMain
IIRC this was added before we added the isolate start subscription and auto run main in https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/isolated/resident_web_runner.dart#L601, so it should not be necessary any more

This code had the odd behavior that if the stream emitted more than one even we would try and call runMain again, which caused the crash. I believe this was a hold over from before we used the isolate start even to invoke that automatically

This one needs to bake for a while before we consider cherry picking


[SPARK-34543][SQL] Respect the spark.sql.caseSensitive config while resolving partition spec in v1 SET LOCATION

What changes were proposed in this pull request?
Preprocess the partition spec passed to the V1 ALTER TABLE .. SET LOCATION implementation AlterTableSetLocationCommand, and normalize the passed spec according to the partition columns w.r.t the case sensitivity flag  spark.sql.caseSensitive.
Why are the changes needed?
V1 ALTER TABLE .. SET LOCATION is case sensitive in fact, and doesn't respect the SQL config spark.sql.caseSensitive which is false by default, for instance:
spark-sql> CREATE TABLE tbl (id INT, part INT) PARTITIONED BY (part);
spark-sql> INSERT INTO tbl PARTITION (part=0) SELECT 0;
spark-sql> SHOW TABLE EXTENDED LIKE 'tbl' PARTITION (part=0);
Location: file:/Users/maximgekk/proj/set-location-case-sense/spark-warehouse/tbl/part=0
spark-sql> ALTER TABLE tbl ADD PARTITION (part=1);
spark-sql> SELECT * FROM tbl;
0	0

Create new partition folder in the file system:
$ cp -r /Users/maximgekk/proj/set-location-case-sense/spark-warehouse/tbl/part=0 /Users/maximgekk/proj/set-location-case-sense/spark-warehouse/tbl/aaa

Set new location for the partition part=1:
spark-sql> ALTER TABLE tbl PARTITION (part=1) SET LOCATION '/Users/maximgekk/proj/set-location-case-sense/spark-warehouse/tbl/aaa';
spark-sql> SELECT * FROM tbl;
0	0
0	1
spark-sql> ALTER TABLE tbl ADD PARTITION (PART=2);
spark-sql> SELECT * FROM tbl;
0	0
0	1

Set location for a partition in the upper case:
$ cp -r /Users/maximgekk/proj/set-location-case-sense/spark-warehouse/tbl/part=0 /Users/maximgekk/proj/set-location-case-sense/spark-warehouse/tbl/bbb

spark-sql> ALTER TABLE tbl PARTITION (PART=2) SET LOCATION '/Users/maximgekk/proj/set-location-case-sense/spark-warehouse/tbl/bbb';
Error in query: Partition spec is invalid. The spec (PART) must match the partition spec (part) defined in table '`default`.`tbl`'

Does this PR introduce any user-facing change?
Yes. After the changes, the command above works as expected:
spark-sql> ALTER TABLE tbl PARTITION (PART=2) SET LOCATION '/Users/maximgekk/proj/set-location-case-sense/spark-warehouse/tbl/bbb';
spark-sql> SELECT * FROM tbl;
0	0
0	1
0	2

How was this patch tested?
By running the modified test suite:
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly *CatalogedDDLSuite"

@dongjoon-hyun @HyukjinKwon Could you review this PR. It fixes similar issue as https://github.com/apache/spark/pull/30615 which you have reviewed already.

Test build #135474 has started for PR 31651 at commit 6a6de20.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40054/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40054/

Test build #135474 has finished for PR 31651 at commit 6a6de20.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

jenkins, retest this, please

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

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

Test build #135476 has started for PR 31651 at commit 6a6de20.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40056/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40056/

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

Test build #135476 has finished for PR 31651 at commit 6a6de20.

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

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

@cloud-fan May I ask you to take a look at this small fix.

thanks, merging to master/3.1/3.0!

Hi, guys.
This broke branch-3.0 compilation.
/home/runner/work/spark/spark/sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala:830: type mismatch;
[error]  found   : org.apache.spark.sql.types.StructType
[error]  required: Seq[String]
[error]           table.partitionSchema,
[error]                 ^

Sorry, but I'll revert this commit first to recover branch-3.0 and unblock the other PRs.
To @MaxGekk , could you make a backporting PR to branch-3.0, please?
cc @sunchao

Here is the backport to branch-3.0: https://github.com/apache/spark/pull/31672

Thank you, @MaxGekk .


GDAX get order by id issue

tradeService.getOrder("f89c7a18-55cf-49e4-843f-80f4f77ebefb");
this will throw the following error:
java.lang.ArithmeticException: Division undefined
at java.math.BigDecimal.divide(BigDecimal.java:1741)
at org.knowm.xchange.gdax.GDAXAdapters.adaptOrder(GDAXAdapters.java:188)
at org.knowm.xchange.gdax.service.GDAXTradeService.getOrder(GDAXTradeService.java:101)

tradeService.getOrder("31f2e557-d8eb-4f61-8e2d-7a557ea78a76");
this will throw the following error:
java.lang.NoSuchMethodError: org.knowm.xchange.dto.trade.LimitOrder.(Lorg/knowm/xchange/dto/Order$OrderType;Ljava/math/BigDecimal;Lorg/knowm/xchange/currency/CurrencyPair;Ljava/lang/String;Ljava/util/Date;Ljava/math/BigDecimal;Ljava/math/BigDecimal;Ljava/math/BigDecimal;Ljava/math/BigDecimal;Lorg/knowm/xchange/dto/Order$OrderStatus;)V
at org.knowm.xchange.gdax.GDAXAdapters.adaptOrder(GDAXAdapters.java:212)
at org.knowm.xchange.gdax.service.GDAXTradeService.getOrder(GDAXTradeService.java:101)

The top one (divide by zero) was fixed in commit 80168395f4c97408979c263153a9d8ff30217441

The bottom error looks like a transitive dependency issue between xchange-core and xchange-gdax. Use "mvn depdendency:tree -Dverbose=true | grep conflict" to check for issues.

Also I think in the method GDAXAdapters.adaptOrderStatus() it should be order.getFilledSize().compareTo(order.getSize()) < 0

Is this still in scope given we have moved to CoinbasePro?


issue with map_view version 1.2.50

The Android Gradle plugin supports only Kotlin Gradle plugin version 1.2.51 and higher. Project 'map_view' is using version 1.2.50.

I have the same issue with map_view when building

Probably related to https://github.com/apptreesoftware/flutter_google_map_view/issues/137 and https://stackoverflow.com/questions/53510048/adding-map-view-to-my-flutter-app-is-not-successful/53515770#53515770

hey guys, regards from Costa Rica,
you can update their kotlin file at <flutter_dir>.pub-cache\hosted\pub.dartlang.org\map_view-0.0.14\android
ext.kotlin_version = '1.2.51' (compile)
at least I have not find any issue by doing this.

hey guys, regards from Costa Rica,
you can update their kotlin file at <flutter_dir>.pub-cache\hosted\pub.dartlang.org\map_view-0.0.14\android
ext.kotlin_version = '1.2.51' (compile)
at least I have not find any issue by doing this.

you can update their kotlin file at <flutter_dir>.pub-cache\hosted\pub.dartlang.org\map_view-0.0.14\android
ext.kotlin_version = '1.2.51' (compile)
at least I have not find any issue by doing this.

@LatinaIsaac  This solution worked for me.
FAILURE: Build failed with an exception.
It is handled by given solution in here.[https://github.com/apptreesoftware/flutter_google_map_view/issues/129#issuecomment-438165577]
Thank you all.

I have tried above solutions. But i get an error as:
"FAILURE: Build failed with an exception.

What went wrong:
Could not resolve all files for configuration ':map_view:androidApis'.

Failed to transform file 'android.jar' to match attributes {artifactType=android-platform-attr} using transform PlatformAttrTransform
java.io.FileNotFoundException: C:\Users\Harsh Goyal\AppData\Local\Android\sdk\platforms\android-27\android.jar (The system cannot find the file specified) "

Any help please!!!


Spyder 4.2.1 installed with windows installer: how do we attach spyder-terminal

Description of your problem
I installed Spyder 4.2.1 using the new Windows installer. How do I attach spyder-terminal? Anaconda Navigator still thinks I'm running 4.1.4
Versions and main components

Terminal Version:
Spyder Version: 4.2.1
Python Version: 3.7.9
Operating system: Windows 10

Hi @dkinneyBU,
You'll need to create a new conda environment to install the dependencies that the installer doesn't bring. So in this new environment please install spyder-terminal. Then you can go to Preferences > Python Interpreter > Python Interpreter and choose the option Use the following python interpreter and in the combobox you should be able to see your new environment to connect to Spyder.

Please let me know if this works

Hi @dkinneyBU,
You'll need to create a new conda environment to install the dependencies that the installer doesn't bring. So in this new environment please install spyder-terminal. Then you can go to Preferences > Python Interpreter > Python Interpreter and choose the option Use the following python interpreter and in the combobox you should be able to see your new environment to connect to Spyder.

Please let me know if this works

That worked perfectly. As always, you've come to my rescue. :-)
thanks...

That worked perfectly. As always, you've come to my rescue. :-)
thanks...


Appstudio update test-pac-component

Pipelines as Code configuration proposal

Pipelines as Code CI/test-pac-component-on-pull-request has failed.

StatusDurationName

✅ Succeeded
8 seconds
appstudio-init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
10 seconds
sast-snyk-check

✅ Succeeded
10 seconds
appstudio-configure-build

✅ Succeeded
30 seconds
build-container

🏃 Running
---
sanity-inspect-image

🏃 Running
---
get-clair-results

✅ Succeeded
14 seconds
sast-java-sec-check

✅ Succeeded
13 seconds
sast-go

🏃 Running
---
clamav-scan

---
---
conftest-clair

Pipelines as Code CI/test-pac-component-on-pull-request has failed.

StatusDurationName

✅ Succeeded
8 seconds
appstudio-init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
10 seconds
appstudio-configure-build

✅ Succeeded
10 seconds
sast-snyk-check

✅ Succeeded
30 seconds
build-container

🏃 Running
---
sanity-inspect-image

🏃 Running
---
get-clair-results

✅ Succeeded
14 seconds
sast-java-sec-check

✅ Succeeded
13 seconds
sast-go

🏃 Running
---
clamav-scan

---
---
conftest-clair

Pipelines as Code CI/test-pac-component-on-pull-request has failed.

StatusDurationName

✅ Succeeded
21 seconds
appstudio-init

✅ Succeeded
9 seconds
clone-repository

✅ Succeeded
8 seconds
appstudio-configure-build

✅ Succeeded
11 seconds
sast-snyk-check

✅ Succeeded
30 seconds
build-container

🏃 Running
---
sast-go

🏃 Running
---
get-clair-results

🏃 Running
---
sanity-inspect-image

🏃 Running
---
sast-java-sec-check

🏃 Running
---
clamav-scan

---
---
conftest-clair

Pipelines as Code CI/test-pac-component-on-pull-request has failed.

StatusDurationName

✅ Succeeded
8 seconds
appstudio-init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
10 seconds
sast-snyk-check

✅ Succeeded
10 seconds
appstudio-configure-build

✅ Succeeded
30 seconds
build-container

✅ Succeeded
23 seconds
get-clair-results

✅ Succeeded
24 seconds
sanity-inspect-image

✅ Succeeded
13 seconds
sast-go

✅ Succeeded
14 seconds
sast-java-sec-check

🏃 Running
---
clamav-scan

🏃 Running
---
conftest-clair

Pipelines as Code CI/test-pac-component-on-pull-request has failed.

StatusDurationName

✅ Succeeded
21 seconds
appstudio-init

✅ Succeeded
9 seconds
clone-repository

✅ Succeeded
8 seconds
appstudio-configure-build

✅ Succeeded
11 seconds
sast-snyk-check

✅ Succeeded
30 seconds
build-container

🏃 Running
---
get-clair-results

🏃 Running
---
sast-go

🏃 Running
---
sanity-inspect-image

🏃 Running
---
sast-java-sec-check

🏃 Running
---
clamav-scan

---
---
conftest-clair

Pipelines as Code CI/test-pac-component-on-pull-request has failed.

StatusDurationName

✅ Succeeded
21 seconds
appstudio-init

✅ Succeeded
9 seconds
clone-repository

✅ Succeeded
11 seconds
sast-snyk-check

✅ Succeeded
8 seconds
appstudio-configure-build

✅ Succeeded
30 seconds
build-container

✅ Succeeded
15 seconds
get-clair-results

✅ Succeeded
15 seconds
sanity-inspect-image

✅ Succeeded
13 seconds
sast-go

✅ Succeeded
14 seconds
sast-java-sec-check

🏃 Running
---
clamav-scan

🏃 Running
---
conftest-clair

Pipelines as Code CI/test-pac-component-on-pull-request has failed.

StatusDurationName

✅ Succeeded
21 seconds
appstudio-init

✅ Succeeded
9 seconds
clone-repository

✅ Succeeded
8 seconds
appstudio-configure-build

✅ Succeeded
11 seconds
sast-snyk-check

✅ Succeeded
30 seconds
build-container

✅ Succeeded
13 seconds
sast-go

✅ Succeeded
15 seconds
get-clair-results

✅ Succeeded
15 seconds
sanity-inspect-image

✅ Succeeded
14 seconds
sast-java-sec-check

🏃 Running
---
clamav-scan

🏃 Running
---
conftest-clair

Pipelines as Code CI/test-pac-component-on-pull-request has failed.

StatusDurationName

✅ Succeeded
21 seconds
appstudio-init

✅ Succeeded
9 seconds
clone-repository

✅ Succeeded
11 seconds
sast-snyk-check

✅ Succeeded
8 seconds
appstudio-configure-build

✅ Succeeded
30 seconds
build-container

✅ Succeeded
15 seconds
get-clair-results

✅ Succeeded
15 seconds
sanity-inspect-image

✅ Succeeded
13 seconds
sast-go

✅ Succeeded
14 seconds
sast-java-sec-check

🏃 Running
---
clamav-scan

🏃 Running
---
conftest-clair


chore: Bump MSRV to 1.81

This is prep for using #[expect].
Its not clear why RenovateBot didn't do this.

Thanks!
@bors r+

:pushpin: Commit 1b94fb23262a7ff30452c88079d196d8b85af02f has been approved by ehuss
It is now in the queue for this repository.

:hourglass: Testing commit 1b94fb23262a7ff30452c88079d196d8b85af02f with merge e94fde952c37bc510144230a4446b431099b1906...

Its not clear why RenovateBot didn't do this.

It's not clear to me from the docs, but I get the impression that packageRules.schedule can't really override the global schedule (https://github.com/renovatebot/renovate/discussions/25829 is related). Maybe it can only make it less frequent? So it is only running once a month on the first day.

Except it worked as of #14303 and we haven't changed our config since then unless you are saying something at the org level and there was a change there.


suggestion: Ibis for Python + SQL

cool project! I'm not sure to what extent you might already be aware of Ibis and it's Python + SQL capabilities: https://ibis-project.org/why#python-sql-better-together -- it could be a good fit on a few points from the README:

For the moment yato only supports DuckDB as backend/dialect. The main reason is that DuckDB offers features that would be hard to implement with a client/server database. I do not exclude to add Postgres or cloud warehouses, but it would require to think how to do it, especially when mixing SQL and Python transformations.

Ibis supports 20+ backends and would allow you to easily mix SQL and Python transformations across them

I does not. I'm not sure it should. I think that when you're adding Jinja templating to your SQL queries you're already too far.

as a former dbt Labs employee, agreed :) in Ibis it's just Python code -- use for loops and string formatting all you'd like

I haven't looked through the code yet but assume you're doing pandas dataframes back and forth to DuckDB. if your Python transformations returned an Ibis expression, you could do ibis.to_sql(expr) -- you could even do something dbt-esque here where that code is "compiled" into SQL, then the rest of yato just runs
this would also give you expr.to_pandas(), expr.to_pandas_batches(), expr.to_polars(), and a bunch of other output methods

hey @lostmygithubaccount, sorry for getting back to you so late. Thank you so much for your message and your suggestions.
I knew Ibis but I did not consider it as a "proxy" to be able to connect to more backends. By reading the documentation I imagine easily implement it with Ibis.
Still there are philosophical questions for yato, if I add Ibis as a proxy, it's a proxy so it adds directly a layer with so many capabilities that yato will lose in readability and simplicity, second I think having a tooling that's dedicated to DuckDB only (or in-memory single file databases) helps for adoption because you do only one thing and you try to do it well.
I started to think how I can decouple yato and DuckDB to have a layer that is agnostic, I'll try to push this version next week.
Actually I don't do that much pandas dataframes back and forth to DuckDB, if the transformations are only in SQL yato never uses dataframes, everything lies in DuckDB, if it's Python it's different for sure. I'd love to improve the Python abstractclass to have something smoother (and to be able to detect sources even more deeper in the code).


Uncaught SyntaxError: Unexpected string

I have a react app with the lib mode activated in my vite config.
I use import.meta.env.mode then build the app.
When I use the library in an other app I get :
Uncaught SyntaxError: Unexpected string
reproducible example repo
steps to reproduce:
1- install dependencies
pnpm install

2- build the library :
pnpm build

3-  In main.jsx, uncomment line 4 and comment line 3
4- start the dev server
pnpm dev

5- check the console in Chrome
Uncaught SyntaxError: Unexpected string and the webpage shows nothing
Expected behaviour :
The web page displays "yay production" in prod and "not production" in dev mode.
I reproduced it with a Firefox and Chrome.

@BenjaminLesne it looks like this is related to #41. The error is happening because the built library has an object called something like ie.process.env.MODE, and that object is then partially replaced by the plugin as something like ie."development" which is invalid. I still need to come up with a solution, but I have a better idea of the cause now.

I think this static replacement might be the cause.
https://vitejs.dev/guide/env-and-mode.html#production-replacement

@davidmyersdev yeh I saw the Production Replacement thing from vite then I found this
It looks like this related to rollup (see this answer)


build(examples): bump react-scripts to fix CVE

Summary
bump react-scripts to fix CVE
Related Issues or PRs
fixes #400
How To Test
N/A

Thanks for always making an issue for the bugs you work on. I see you and it is appreciated!


Error initializing shadcn-ui on nextjs

I'm using this command to initialize shadcn-ui "bunx --bun shadcn-ui@latest init".
And here's the error that I encounter:

Bun version: 1.0.22+b400b36ca
Nextjs version: 14.0.4

Did you try the command : npx shadcn-ui@latest init?
I have another question: Did you install NextJS with bun?

I am having same issue with initializing NextJs with bun also

Did you try the command : npx shadcn-ui@latest init?
I have another question: Did you install NextJS with bun?

Yes I am using a bun on my nextjs I just copy the command from shadcn-ui

Running bun shadcn-ui@latest init without the --bun flag worked for me.
Not sure if there's any negative side effects to this.

Running bun shadcn-ui@latest init without the --bun flag worked for me.
Not sure if there's any negative side effects to this.

same question

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Build failure: light.src

Steps To Reproduce
nix build nixpkgs/212c9887a784ccab43465260bf1d110f0e68c90f#light.src --rebuild
Build log
error: builder for '/nix/store/86d4irx7npksrmy7g27lhhfxrx9vp4fi-source.drv' failed with exit code 1;
       last 7 log lines:
       >
       > trying https://github.com/haikarainen/light/archive/v1.2.2.tar.gz
       >   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
       >                                  Dload  Upload   Total   Spent    Left  Speed
       >   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
       > curl: (22) The requested URL returned error: 404
       > error: cannot download source from any mirror
       For full logs, run 'nix log /nix/store/86d4irx7npksrmy7g27lhhfxrx9vp4fi-source.drv'.
error:

Additional context
Original repository missing.
Arch uses https://gitlab.com/dpeukert/light
Notify maintainers

@puffnfresh @dtzWill
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.51, NixOS, 24.05 (Uakari), 24.05.20240916.086b448`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.24.6`
 - nixpkgs: `/nix/store/qkzr5clkyq7j6n43qs5ff73in6c3wacv-source`

Add a :+1: reaction to issues you find important.

@Uthar Upstream's repo is deleted. Could you provide a new source?


Make doc tests compile

CI for nightly channel fails because rustfmt is not available. Seems irrelevant. The other channel builds succeed.


Use new recovery function in client module

Using the new recovery function in the client module
Increasing the rpc polling interval so we don't spam our RPC

@blakecduncan Are we going to do a similar update in the extension as well?

@jacque006  Yeah, Quill is already using the new recovery methods.  I just figured out the RPC thing yesterday when I was playing with the instant wallet.  I'll make this update in Quill next


IsEqualTo / isSameAs fails on mocked Stream

Summary
As AssertJ converts Stream to List, if one uses a mocked Stream the conversion will return a null  List that would make assertions to fail.
Example
Stream s = mock(Stream.class);

// fails
assertThat(s).IsEqualTo(s);
assertThat(s).IsSameAs(s);

Java 8 specific ?

YES : create Pull Request from the master branch

Fixed in 623621f0c600f14bdbfcf7571fa9f33bd5e10506.


Print logs after session failed.

Is your feature request related to a problem? Please describe.
Session has a param called show_log which controls whether print backend logs to stdout / stderr, which is great. But when we run a session with show_log=False since we don't want to be distracted by logs, and occasionally session launched failed, we can't get much useful information about why it failed and how to fix it, usually we need to rerun the session with show_log=True.
Describe the solution you'd like
Print logs when session failed, even when show_log=False.

As we discussed, stderr should stream to client when the session launching failed. This should be fixed when addressing the issue #7 .


Import fails with "Failed to execute 'ImportScripts' on 'WorkerGlobalScope'"

I've just started a new react-native project (using the normal cli, not expo).
I have installed this package using yarn:
yarn add react-native-vector-icons
react-native link

...which seemed to be successful, but when I import the package (just the import, not even using it yet), I get the following error.
e.g.:
import Icon from 'react-native-vector-icons/FontAwesome'

I've tried uninstalling, unlinking, re-doing the install etc... no luck.
I then also tried to add that gradle line to apply the plugin, as per the README, but that didn't change anything.
Any ideas?

I have same problem. For latest react-native version (0.53) , you need delete this
rm node_modules/react-native/local-cli/core/__fixtures__/files/package.json
This solved for me.

 rm node_modules/react-native/local-cli/core/__fixtures__/files/package.json

yes,this success for me

just close debugging tab in your browser and turn off Debug JS Remotely

For future people that come across this.
This issue occurs because you are trying to serve an offline bundle and debug it in the browser. If you used npm run build:ios to generate your bundle it probs is offline.  But as soon as you debug js remotely it fails because it's an offline-only bundle. So make sure you do npm start -- --reset-cache to serve a dynamically generated bundle.

expo 40, same issue.

Hello guys, I started getting this problem after updating Expo to version 40, resetting the cache didn't work.

expo 40 same problem

My problem was a wrong import, I did it right and restarted the app, then it ran.

This error was caused by port 19001 busy for my pc. You can find out which application is occupying the required port from Resource Monitor. This method may work for you too

Stop your bundler and restart it. It worked for me.

worked for me !

Stop your bundler and restart it. It worked for me.

Works for me!

just close debugging tab in your browser and turn off Debug JS Remotely

worked for me, just close debugger on browser .

hei guys，i think i found the reason. when open Debug Js Remotely, your jsbundle will executed in worker of chrome. any jsbundle runtime error will cause the  'importscripts' error.  the only debugger method is check 'pause on caughted exceptions'  on chrome.

if it works for you when turn off Debug JS Remotely, like me. the reason may be calling synchronous methods on native modules.
good luck.

For future people that come across this.
This issue occurs because you are trying to serve an offline bundle and debug it in the browser. If you used npm run build:ios to generate your bundle it probs is offline.  But as soon as you debug js remotely it fails because it's an offline-only bundle. So make sure you do npm start -- --reset-cache to serve a dynamically generated bundle.

I accidentally set this field to offline, and your solution reminded me of it. thank you

I realized i had 'react-native-maps' imported somewhere in my app, after i uninstalled it. 🥲

See thread for fixes in particular https://github.com/oblador/react-native-vector-icons/issues/662#issuecomment-568942068


nightly build failed: faster tests failed resource busy (Address already in use)

Tonight the nightly build failed.
The faster tests failed: resource busy (Address already in use)
Examples.All
  Adder
    Basic
    State Automation
    Parallel Adders
    Replay Adder Trace
    Purp1 - 4 goals
    Purp2 - Operand constraints
==> sqatt-logs/test-2018-03-13_23-23-29.714726_UTC/Adder/Purp3_-_Always_+2/txsserver.out.log
> Found configuration file at `.torxakis.yaml`.
> txsserver: Network.Socket.bind: resource busy (Address already in use)
--- EOF ---

==> sqatt-logs/test-2018-03-13_23-23-29.714726_UTC/Adder/Purp3_-_Always_+2/txsui.out.log
>
> TXS >>  TorXakis :: Model-Based Testing
>
--- EOF ---

==> sqatt-logs/test-2018-03-13_23-23-29.714726_UTC/Adder/Purp3_-_Always_+2/SUT.out.log
> Starting 1 adders.
> Starting an adder listening on port 7890
--- EOF ---

    Purp3 - Always +2 FAILED [1]
    Purp4 - Overlapping Steps
    Test 'Stop' command
  Control Loop
    Stepper
    Spec Produce
    Spec Measure
    Spec Correct
  Customers and Orders
    Customers & Orders Test
  Dispatch Process
    Process 01
    Process 02
    Process 03
    Process 04
    Process 05
    Process 05a
    Process 06
    Process 06a
    Process 07
    Process 08
    Process 09
    Process 10 - Data
    Process 12 - Unique ID (Wrong)
    Process 12 - Unique ID (Right)
  Echo
    Stepper
    Simulator
  Lucky People
    Examples Purpose Test
    Lucky By Gender Test
    Random Lucky Test
  Moving Arms
    Stepper Test
    Single Axis Purpose Test

See https://semaphoreci.com/capitanbatata/torxakis/branches/develop/builds/275

Issue does not always occur:
see e.g. https://semaphoreci.com/capitanbatata/torxakis/branches/develop/builds/276

txsserver: Network.Socket.bind: resource busy (Address already in use)
once more  ...
See
https://semaphoreci.com/capitanbatata/torxakis/branches/develop/builds/284

See #578 which is a duplicate for similar info.

And again
https://semaphoreci.com/capitanbatata/torxakis/branches/develop/builds/313

And again https://semaphoreci.com/capitanbatata/torxakis/branches/develop/builds/321

Etc. https://semaphoreci.com/capitanbatata/torxakis/branches/develop/builds/324


[Maint] Update build_napari.yml to drop doctrees

In https://github.com/napari/docs/pull/348 the build of docs was changed slightly to not put the doctrees in the html output.
In this PR I alter the workflow that builds napari docs using the theme to account for this change.
We don't need to delete the doctrees anymore, we can just upload the html folder instead.

So the artifact build and uploaded correctly here. I checked the action page and clicked workflow file:
https://github.com/napari/napari-sphinx-theme/actions/runs/8460246134/workflow?pr=160
it is the one from here.
So I think all the paths and everything are correct for the change in the Makefile.
In https://github.com/napari/docs/pull/380 I use the same paths, with .../html/


Feature/iron component page

Added an index with <iron-component-page> to show component API docs
Allow to run tests in the browser
Prevent copying template files which can be confusing

Iron-component-page (or more accurately Hydrolisis) doesn't support ES6 yet, so won't work here.

FYI pulled several of the other (non component-page) tweaks from this PR into mainline manually

Ok. Thank you. I'll take a look.

This PR is diverged pretty heavily from v1 now, so closing  for now. Thanks though.


Clarify control flow in LockedFile constructor

Eliminate the unnecessary local field "opener" and ensure that
self._opener is assigned exactly once along every trajectory through
the constructor body.
I don't believe the not opener on line 349 of the old implementation could ever have been false, and if _Win32Opener and _FcntlOpener were both non-None then for calls with use_native_locking true the old implementation would have constructed an object just to throw it away. Right?

@nathanielmanistaatgoogle Careful not to step on #478 toes. Assigned to @jonparrott since he is in the process of sorting this out.

This slipped under my radar and I apologize for that. @nathanielmanistaatgoogle if you want to resolve the conflicts then this LGTM. Otherwise, we can just hold off on this in favor of #504's eventual outcome.

Yeah, I'm perfectly happy to just sit on this for a while.

Closing because #470's outcome will be the removal of this module.


Cargo.toml package.version not incrementing

The version of the GitHub tags is incrementing, currently at 0.7.1, while the Cargo.toml package.version is still 0.6.0

Issue has been fixed. python commitzen doesn't use regular expressions - it just replaces old version with new one (version in the file was outdated).


Support source-removal logic in libbs

Signed-off-by: Sambhav Kothari<EMAIL_ADDRESS>
Summary
Adds support for source-removal logic in libbs. Instead of completely empty-ing out the application directory, only relevant files are removed BEFORE copying back the interesting files to the application directory.
Fixes #103
Use Cases

Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Looks good to me. This is a fairly core change though, so I'm going to let @pivotal-david-osullivan take a look as well. If you think it's good @pivotal-david-osullivan feel free to merge.

@pivotal-david-osullivan should be good to go :)


Convert to static array sizes for string parameters - name[static SIZE] and scrub string handling

Is your feature request related to a problem? Please describe.
Many APIs and even internal functions require specific minimum string sizes to avoid a potential buffer overrun, yet take simple pointers, i.e. char *.  Some tools complain about strcpy/sprintf/etc, but the sized versions aren't any safer if the passed in string is too short.
Describe the solution you'd like
Leverage C99 checks on sized parameter arrays, which also will protect at compile time from NULL inputs.  Once minimum sizes are guaranteed, then revisit string comparison/manipulation calls to avoid buffer overruns based on these enforced minimums.
Note not backwards compatible where pointers are passed in vs the array.  Also would REQUIRE the removal of all NULL pointer test cases and checks (would cause compile errors or dead code respectively).
Describe alternatives you've considered
None
Additional context
Code review, related to #1465
Requester Info
Jacob Hageman - NASA/GSFC

Should actually confirm all our supported compilers are compliant (check NULL and undersized arrays) before advancing this change.  At minimum the one used for CI should confirm the core software as configured is good.


Enhanced chat

chat Colors for player groups
prettyColors
gameStatus + nice preview
gameStatus symbol still needs to be fixed in regards to chatUserControl

updated with code review fixes


Create identification fields

Description of feature
Taxonomic identifications may be provided by multiple different tools within MikroKondo and may not resolve to the species identification level ie. Genus or full resolution to subspecies. So MikroKondo should create fields called "predicted_identification_name", "predicted_identification_method" where the name field is populated with the final identification for the sample, and the method should contain the name of the software tool used for the final identification. In the case of a metagenomic sample with multiple taxa this needs some clarification as to whether it would be populated. For version 1, I suggest that it only is populated for single taxon samples and only contains the identification information. ie. s___Salmonella enterica should be Salmonella enterica and the method used for identification ie. mash.

Fixed in PR https://github.com/phac-nml/mikrokondo/pull/90


Invoking the Endpoint - Kerberos OAuth2 Grant

Description:
Document: Kerberos OAuth2 Grant
Section: Configuring Kerberos Grant with API Manager
Step 8
curl -v -X POST -H "Authorization: Basic :" -k -d "grant_type=kerberos&kerberos_realm=&kerberos_token=&scope=" -H "Content-Type:application/x-www-form-urlencoded" https://localhost:8243/token
Explain how to get these values:

kerberos-realm
kerberos-token
scope

Suggested Labels:
Type/Docs
Severity/Major
Priority/High
Suggested Assignees:
N/A
Affected Product Version:
AM2xx docs

We will not be fixing this as this issue is based on an older version of the product


sass stylesheet and colors as variables

This is what I have in mind with #114.
It is quite neat to see what colors there are in the palette.

So I haven't done anything creative, just a lot of typing so far:

collect the colors as named colors
assign the color names to their semantic function
use those semantic variable names in epub.sass instead of direct color hex values

And so 1) all the colors are in one place, and 2) all their function are in one place, no need to hunt them down from everywhere in epub.css.
What do you think about this? I know I like it and will use it myself.
Nonetheless I understand that the stylesheet is a delicate matter -- maybe you're not so interested in this idea, and I don't mind if you would rather cancel this PR for that reason.

I collected all the colors now and updated the screenshot above.
Some of these grays are really close and could be combined.

What do you think about this? I know I like it and will use it myself.

:+1:

Some of these grays are really close and could be combined.

Agreed. If you think the human eye can't really tell the difference, and it doesn't add anything to the design, then I say let's simplify.


UI: Scientific notation 'ts' events are 1e15us in the past, compared to Decimal notation 'ts' events

The following Google Trace Event JSON input describes two different parent-child pairs of events, where the parent events start 6us apart.  The first parent-child 'ts' fields are formatted as scientific notation, and the second parent-child 'ts' fields are formatted as decimal notation.  Perfetto shows these events as starting 1e15us (one quadrillion microseconds) apart.
[
    {
        "cat":"perf",
        "dur":4.75,
        "name":"parentEvent",
        "ph":"X",
        "pid":3933029,
        "tid":1,
        "ts":1.6921085481321545e+15
    },
    {
        "cat":"perf",
        "dur":2.0,
        "name":"childEvent",
        "ph":"X",
        "pid":3933029,
        "tid":1,
        "ts":1.6921085481321572e+15
    },
    {
        "cat":"perf",
        "dur":4.5,
        "name":"parentEvent",
        "ph":"X",
        "pid":3933029,
        "tid":1,
        "ts":1692108548132160.5
    },
    {
        "cat":"perf",
        "dur":2.5,
        "name":"childEvent",
        "ph":"X",
        "pid":3933029,
        "tid":1,
        "ts":1692108548132162.5
    }
]

(Image produced with version v35.0-8019b8809, 'Stable'.  'Canary' produced similar results.)
(Speculation: This may relate to Issue #426)

Our response to this issue falls squarely under our published policy for supporting features in JSON traces: policy here.
TLDR: scientific notation in the JSON trace format is quite an obscure feature and we will not go out of our way to add support for this. Patches for fixing this our welcome.

To clarify:

the Chrome Trace Event Format implies that a 'ts' field is a JSON Number
a JSON Number is "a sequence of decimal digits with no superfluous leading zero. It may have a preceding minus
sign (U+002D). It may have a fractional part prefixed by a decimal point (U+002E). It may have an exponent,
prefixed by e (U+0065) or E (U+0045) and optionally + (U+002B) or – (U+002D).  The digits are the code
points U+0030 through U+0039."
you said "scientific notation in the JSON trace format is quite an obscure feature"
JSON.parse works correctly on above input on the browser I'm using to run Perfetto.

So, does Perfetto use a different JSON parser than the JSON.parse one available to all JS/TS programs, and that different parser is feature incomplete?

All trace parsing (including JSON parsing) is performed in C++ so does not use JSON.parse at all. Specifically the timestamp coercion logic is here.

Should be fixed by https://r.android.com/2718613. Thanks for the contribution here @mw-cwilcox!