Assign being treated as named argument

TG contains this code: if(isnull((smoothing_groups = GLOB.bitflag_lists[txt_signature])))
It causes the following runtime: [ERRO] world.log: Exception occurred: /proc/isnull has no argument named smoothing_groups
According to wixoa, the compiler is treating that as if the parentheses aren't there and it's a named argument.

Small test case:
/proc/foo()
	var/list/test_list_one= null
	var/list/test_list_two= list("bar")
	if(isnull((test_list_one= test_list_two[1])))
		world.log << "was null"
	else
		world.log << "wasnt null"

Call foo() and you'll see the runtime happen.

Decided to taken on this myself. My investigation has shown that the Expression does call ExpressionPrimary() with the allowparenthesis still set to true. So the parenthesis are consumed once the expression has been parsed, making it harder to discern whether the assignment was wrapped or not.

I was able to fix this, PR pending. However I did discover that the current implementation does not support parenthesis wrapping of expressions where you should be able to do in BYOND. See #1276


Fixes broken icon links (#18)

msapplication-icon-144 was created from android-chrome-512x512 icon, resized in Photoshop and compressed using pngquant, final size is 1.14 KB

👍


chore: Update the link for eslintignore doc

What Changed
Since the .eslintignore documentation has been moved from https://eslint.org/docs/latest/user-guide/configuring/ignoring-code to https://eslint.org/docs/latest/use/configure/ignore-deprecated, I have updated README and no-uninstalled-addons.md accordingly.
Checklist
Check the ones applicable to your change:

[ ] Ran pnpm run update-all
[ ] Tests are updated
[x] Documentation is updated

Change Type
Indicate the type of change your pull request is:

[ ] maintenance
[x] documentation
[ ] patch
[ ] minor
[ ] major

Thank you!


Fix concat in previews

closes #2055
What are you trying to accomplish?
fix the buf described in #2055

Anything you want to highlight for special attention from reviewers?

legend 💪


[Bug] Copy-DbaLogin example should be added for LoginRenameHashtable

Report
An example is needed in Copy-DbaLogin to show how you can use the parameter -LoginRenameHashtable

Hey @wsmelton
There is an example for the rename, but just one login? Did you want this expanded?
https://github.com/sqlcollaborative/dbatools/blob/development/functions/Copy-DbaLogin.ps1#L116

Yeah, didn't catch seeing it there. Be good to show that with splatting to draw attention to it as well :wink:

Added some splatting ;)


[design] - Paypal

login URL: https://www.paypal.com/en/signin

can I take this, sir?

Yah, this is cool 👍

Note: please do not use your master branch to create a PR, create another branch first before start working

Terima kasih 🙌


PR: Fix eol for missed files

It seems that some files were missed in bdfe0b59 (#18691).
If I'm in error, please advise.

Hello @mrclary! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file rope_profiling/test.py:

Line 28:10: E122 continuation line missing indentation or outdented
Line 29:10: E122 continuation line missing indentation or outdented
Line 30:10: E122 continuation line missing indentation or outdented
Line 31:10: E122 continuation line missing indentation or outdented
Line 32:10: E122 continuation line missing indentation or outdented
Line 33:10: E122 continuation line missing indentation or outdented
Line 34:10: E122 continuation line missing indentation or outdented
Line 37:1: E302 expected 2 blank lines, found 1
Line 56:1: E305 expected 2 blank lines after class or function definition, found 1
Line 58:1: E302 expected 2 blank lines, found 1
Line 60:41: E128 continuation line under-indented for visual indent
Line 66:1: E302 expected 2 blank lines, found 1
Line 70:1: E302 expected 2 blank lines, found 1
Line 74:1: E402 module level import not at top of file
Line 77:1: E302 expected 2 blank lines, found 1

In the file scripts/spyder_win_post_install.py:

Line 40:29: E261 at least two spaces before inline comment
Line 48:5: E301 expected 1 blank line, found 0
Line 50:5: E301 expected 1 blank line, found 0
Line 134:31: E261 at least two spaces before inline comment
Line 135:80: E501 line too long (84 > 79 characters)
Line 158:32: W605 invalid escape sequence '\S'
Line 158:40: W605 invalid escape sequence '\s'
Line 161:32: W605 invalid escape sequence '\S'
Line 161:40: W605 invalid escape sequence '\s'
Line 184:76: E502 the backslash is redundant between brackets
Line 194:75: E502 the backslash is redundant between brackets
Line 197:80: E501 line too long (81 > 79 characters)
Line 203:80: E501 line too long (83 > 79 characters)
Line 207:50: E231 missing whitespace after ','
Line 209:79: E502 the backslash is redundant between brackets
Line 212:77: E502 the backslash is redundant between brackets
Line 217:75: E502 the backslash is redundant between brackets
Line 220:79: E502 the backslash is redundant between brackets
Line 224:12: E225 missing whitespace around operator
Line 237:75: E502 the backslash is redundant between brackets

I only addressed py files in the spyder folder. Somes files here should not be changed, for example: external-deps/qdarkstyle/docs/make.bat

I agree with @impact27: batch files should be left out of this. The rest looks good to me.

You should probably add the commit to [.git-blame-ignore-revs]. I wonder why the bat files are showing here, they should be excluded with .gitattributes file

@mrclary, please rebase to use the changes to our test suite done in PR #18802.

Did you update the commit hash in .git-blame-ignore-revs after the rebase?

@impact27, all good?

Looks good to me!


subgit: 3.3.17 -> 3.3.18

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/subgit/versions.
meta.description for subgit is: Tool for a smooth, stress-free SVN to Git migration
meta.homepage for subgit is: https://subgit.com
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.3.18 with grep in /nix/store/mv72simxs672z9l6vwmzkiq9nzpalm3i-subgit-3.3.18
found 3.3.18 in filename of file in /nix/store/mv72simxs672z9l6vwmzkiq9nzpalm3i-subgit-3.3.18

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

subgit

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mv72simxs672z9l6vwmzkiq9nzpalm3i-subgit-3.3.18 \
  --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 subgit https://github.com/r-ryantm/nixpkgs/archive/d21929eb39bbf45b4999d98f55f68203db2446f7.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d21929eb39bbf45b4999d98f55f68203db2446f7#subgit

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mv72simxs672z9l6vwmzkiq9nzpalm3i-subgit-3.3.18
ls -la /nix/store/mv72simxs672z9l6vwmzkiq9nzpalm3i-subgit-3.3.18/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.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

subgit

Maintainer pings

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 361073

aarch64-darwin

:white_check_mark: 1 package built:

subgit


Feature request

Is there a location picker here?

We have a separate repo for this, though it's not as refined as I'd like to have it.

Can be close this issue?

Due to lack of response and an old error, I will close this issue.
If the bug still exists, feel free to open a new issue


Allow running if installed as project specific

closes https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/4468
replaces https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/4469

I thought we already closed #446. what happen we need to fix it again ?

I believe @SpacePossum meant #4468 (which replaced #4469 was fixing).
BTW what was wrong with #4469?

wrongly copy paste indeed, I tried to merge/squash the other one, didn't work out, opened this one but run out of time of merging, will merge later

@SpacePossum any news on this? I had a similar issue like in #4468 and it looks like your PR would fix that.

Thank you @ticktackk.


Remove unused import.

_virtualenv.py doesn't need to import __future__.annotations, as it has none.
Removing the import:

Restores the action of the VIRTUALENV_PATCH on Python 3.6

Eliminates 24 lines of error messages displayed by Python 3.6 when it starts in an environment created by uv:

Error processing line 1 of /tmp/tmp.ENwqZ0oeyb/lib/python3.6/site-packages/_virtualenv.pth:

  Traceback (most recent call last):
    File "~/.pyenv/versions/3.6.15/lib/python3.6/site.py", line 168, in addpackage
      exec(line)
    File "<string>", line 1, in <module>
    File "/tmp/tmp.ENwqZ0oeyb/lib/python3.6/site-packages/_virtualenv.py", line 3
      from __future__ import annotations
                                       ^
  SyntaxError: future feature annotations is not defined

Remainder of file ignored

(Python displays the errors above twice.)
I appreciate the Python team no longer support Python 3.6, but RedHat-style Linux distributions will support Python 3.6 in their /usr/libexec/platform-python until releasever 8 expires in 2029. I'm happy for the community to move on, in general, but don't see the harm in helping those who can't.
I'm not yet sure what in the “remainder of file ignored” is necessary for my project's build, as I haven't yet finished digging that from under Hatch. I'll follow up on #6426 when I do, so we can concentrate on getting to the happy cow.
Test Plan
( set -eu
  export VIRTUAL_ENV="$(mktemp -d)"
  ./target/release/uv venv "$VIRTUAL_ENV" --python=python3.6
  ./target/release/uv pip install cowsay
  $VIRTUAL_ENV/bin/python -m cowsay --text 'Look, a talking cow!' )

Happy output:
Using Python 3.6.15 interpreter at: ~/.local/bin/python3.6
Creating virtualenv at: /tmp/tmp.VHl4XNi3oI
Activate with: source /tmp//tmp.VHl4XNi3oI/bin/activate
Resolved 1 package in 929ms
Installed 1 package in 17ms
 + cowsay==6.0
  ____________________
| Look, a talking cow! |
  ====================
                    \
                     \
                       ^__^
                       (oo)\_______
                       (__)\       )\/\
                           ||----w |
                           ||     ||

Thanks, @charliermarsh, and thanks @zanieb for fixing the snapshot.


Cursor Preview Trixel is wrong

The position of the cursor-preview-trixel is now always correct. This is especially true when we're zoomed in and offset our grid.

This one can be solved by:

Transforming the cursor into a center-position of a triangle T
If the cursor is not within T, we concider T's left/up neighbour N iff it exist
If the cursor is not within N or N does not exist, we consider T's right/down neighbour M, iff it exist
A solution can't be found if cursor is not within M or M does not exist

This issue is almost solved. We still need to fix the cursor Preview for all situations where scale > 1.


pysobatools not supported on Windows

hi Steve! thank you for this repo as I'm really interested in it. I just wonder whether there is any workaround for pysobatools to be installing on Windows?

error: command 'C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Tools\MSVC\14.36.32532\bin\HostX86\x64\cl.exe' failed with exit code 2

Hi Johan, Thank you for your interest! I'm sorry to hear about the issue. Unfortunately, I don't have a Windows machine to debug it for you. However, since this repository is built on pycocotools, you might find a workaround in their repository.


I want to add Linked List program in C++

Program like: merge two sorted linked list, Bubble sort in linked list,Reverse linked list,Search in Linked list
Assign me!!

Please assign this problem to me. Can I do these problems in java.


Fix local annotate_sex() patches

This updates PR #1, in which initialisation of variables used later was skipped when {tmp_prefix}/ploidy.ht exists and is used: Initialise the first bunch of variables unconditionally. Lift them out of the ploidy.ht checkpoint conditional. In particular, x_contigs is used unconditionally later (in the compute_fstat stanza).
Context: https://centrepopgen.slack.com/archives/C030X7WGFCL/p1702013181867999?thread_ts=1701994397.101339&cid=C030X7WGFCL
(You will need to hit the Settings cog / Hide whitespace button to review this.)
To be honest, I am dubious about PR #1 in general. At present, annotate_sex() is only called once, in cpg_workflows/large_cohort/sample_qc.py, with the default compute_x_frac_variants_hom_alt (False), compute_fstat	(True), infer_karyotype (True). If it was called multiple times with different compute_* requests, it would not necessarily return the expected results. e.g. earlier stanzas' annotations will (?) be overwritten/removed by if infer_karyotype's ploidy_ht = hl.read_table(path) — note that all stanzas annotate into and reread checkpoints into the same ploidy_ht object (but with different filenames)… 🤔
Note also that if we were to activate compute_x_frac_variants_hom_alt, then filtered_mt is currently uninitialised and would need to be recomputed or probably also checkpointed and reread.
However with the way it is called in production-pipelines at present, the previous PR's rereading from checkpointing does not appear to cause any issues for now.

Thanks. Yes, probably one to bring up in the new year — I'll schedule myself a note to do so.


Perf nit:  Check if anyone is hooked into events before doing math to…

… determine whether said events should fire
Overview
This is very much a nit, just something that I came across and found odd.

/azp run


numba cov() does not support aweights parameter.

To calculate numpy version of exponential weighted covariance, aweights  is needed in numpy.cov() function.
sample code showed as below:
def numpy_ewcov(values):
    length = values.shape[0]
    alpha = 2/(1+length)
    values = values[::-1]
    weight = (np.ones(length) * (1-alpha))**np.arange(0,length,1)
    ew_cov = np.cov(values.T,aweights=weight)

    return ew_cov

Hope aweights parameter for cov() func could be included in next version of Numba update.

Thanks for the report @adiyaDalat. The docs mention that Numba only support the first five arguments to np.cov. I've marked this issue as a feature request.


Fix Unittest for alternative agegroups in getVaccinationData

In getVaccinationData the unittest for alternative agegroups prints 'Error in transformation...' for every county. See also Issue #194

Closed by #252


401 Client Error: Unauthorized for url:

Describe bug
An issue occurs when attempting to fetch stock information using the yfinance library. The request results in a 401 Client Error: Unauthorized.
import yfinance as yf
msft = yf.Ticker("MSFT")
get all stock info
msft.info
401 Client Error: Unauthorized for url: https://query2.finance.yahoo.com/v10/finance/quoteSummary/MSFT?modules=financialData%2CquoteType%2CdefaultKeyStatistics%2CassetProfile%2CsummaryDetail&corsDomain=finance.yahoo.com&formatted=false&symbol=MSFT&crumb=2abgHhN2kUC
Environment:
Operating System: Windows 10, Windows 11
Region: Europe
both current yfinance version
Simple code that reproduces your problem
import yfinance as yf
msft = yf.Ticker("MSFT")
get all stock info
msft.info
Debug log
...
Bad data proof
No response
yfinance version
0.2.40
Python version
Python 3.11.4
Operating system
W10/11

Having same issues.
Operating system
Linux
Region: USA
yfinance version
0.2.40

Same issue also.

Same issue - worked fine a few days ago. Upgrading yfinance does not solve:
OS: Mac 13.2.1
Region: USA
Python: 3.11.4
yfinance: 0.2.40

Same issue - worked earlier today.
OS: Mac 14.5
Python: 3.12.3
yfinance: latest 0.2.40

Same issue - worked fine few hours ago
OS: Windows 11
Region: Italy
Python: 3.12.3
yfinance: 0.2.40

Same issue - worked last week
OS: Windows 11
Region: Brazil
Python: 3.12
yfinance: 0.2.40

The issue ties back to the crumb. It's likely Yahoo changed something with their cookies or how they expect to receive the crumb.

I had the same issue yesterday, but now it works again.

its randomly back working for me too


Adapt to django 2.0/2.1

Hi there, @noviluni  thanks for your effort. The feature you wanted has been included in the new version. I recently got in charge of the repo and I wasn't aware of your PR. Sorry. Kind regards.


Steps for running

Run a local ganache instance

ganache-cli -f http://kendrick-nuc.local:8544 -i 1 -d  -e 1000 -l 9000000

Run truffle migrate

Run the bot

Run the test

Test might fail because of bot disputing.

Fixed by #2


Switch Pro Controller Stops

After a certain amount of playing on my PS1, the game stops responding to my controller, but connection is maintained between my Switch Pro controller and the BlueRetro Dongle. If I hit the reset button on my controller, the dongle and the controller disconnect and resync, but the game remains unresponsive, as if the controller was unplugged.
Hitting the reset button the BlueRetro dongle and waiting for the controller + dongle to pair up allows me to play again. Sometimes I get a couple of hours of gameplay before this happens, sometimes 20 minutes. I haven't noticed it on my Dual Shock 4 but I need to test more.

This issue has been fixed for me with the 1.8 firmware update that's been released a few days ago. At least with my aliexpress gamecube dongle and switch online snes controller.

1.8 still has this problem

My Switch Pro controller is constantly disconnecting from my GC internal adapter. Most times it will reconnect after a few seconds, but sometimes it requires a full power cycle of the console to get it to reconnect. Running the latest firmware (1.8.3).

Fix is now in v1.9.1


Reduce Permissions on cluster-role for policy propagator

Related to : https://github.com/stolostron/backlog/issues/26574
Signed-off-by: Chaitanya Kandagatla<EMAIL_ADDRESS>

/lgtm


Extract platform tab rendering.

The new method is a bit bigger, but (a) it is needed for the landing pages too (b) with the new URL structure proposed in the document it will get shorter.

Check travis...


Update version.py from autover for various fixes

Allow develop_install to work (https://github.com/ioam/autover/pull/41)

Failed to report 'dirty' for commit count of 0 (https://github.com/ioam/autover/pull/44)

The tests for this are in autover, so going to merge without waiting for the ioam travis queue to clear...


Implement Pagination on /search/stories?

Dependent on #8819

Actually this was done by Mel as part of #8819 so I'm closing this ticket.


Fix output inconsistent result (default_integrations/monitoring)

Fixes #840
Add default: False to default_integrations and default_monitoring.
This was not found by the tests because both fields are set in each test.

Sure, I'll get it later tonight
On Sun, Oct 13, 2024, 5:49 PM Toby Brain @.***> wrote:

@.**** commented on this pull request.
Can we get a changelog entry for this one please?
—
Reply to this email directly, view it on GitHub
https://github.com/elastic/terraform-provider-elasticstack/pull/841#pullrequestreview-2364999877,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFMTBBDOHEDK3O77JMQ2JLZ3LTHJAVCNFSM6AAAAABP2WPJ46VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRUHE4TSOBXG4
.
You are receiving this because you authored the thread.Message ID:
@.***
.com>


allow command + click to homepage cards

if i click, open target="_self", if i command click, allow open in new tab (target="_blank")
something like:
// util funcs
export const cmdPlusClicked = (e: any) => e.metaKey || e.ctrlKey;
export const openInNewTab = (url: string) => window.open(url, `_blank`);

 {
    if (cmdPlusClicked(e)) {
        openInNewTab(`${path}`);
        return;
    }
    navigate(`${path}`);
}}>

@cdt-eth I think you can just wrap an anchor around Link to fix this.
<Link>

</Link>

ah yes, was referencing how we had to do it with prop house (bootstrap), but nextjs has 's, youre right @ryeshrimp


Add Conn() support to all storage drivers.

This PR adds support for Conn() to all drivers. The main purpose is allow access to the underlying database client from the interface.
Fixes #440

thanks for that big PR -> will check it soon

Wouldn't Conn() be more logical than DB() 🤔🤔

@efectn Agree, I can change it

I'm still not sure what to do with the memory driver, given that it returns a Map of Entry.

Done, all functions have been renamed from DB() to Conn()

@efectn @ReneWerner87 Who can approve the workflows to be run? Thanks

I'm not sure which field to use for ArangoDB.

@efectn @ReneWerner87 I replaced the return with driver.Client since from the client the user can call Databases() and get all the databases, collections, etc.


Templatize Github release jobs per platform & Tar the .deb output

Cherry-picked from #89

I think we can release v0.8.1 with the same internal component versions, with a short note saying this is an improvement in how artifacts are packaged. Any reason not to do this?

Why not just re-release 0.8.0? The release is already a little confusing with .deb copies

In reply to: 937386347


Introducing new Event class that can be send via analytics.send(Event event)

Second attempt at separating events from main analytics.dart file through an additional class. Handling the events this way will make it easy for future developers to audit what events are being sent by this package by viewing Event.dart
Previously, we used the analytics.sendEvent(DashEvent eventName, Map eventData) API to send events. While the eventName parameter was limited to what was in enums.dart, the eventData was not standardized. This map was left flexible so that clients using this package could add data for each event as needed.
After discussing this design with @hixie, we are pivoting to also standardize the eventData we will be sending as well. This is to be transparent in what we are collecting. And if anyone would like to audit what events we are collecting, they can refer to event.dart that contains all of the events along with their accompanying data.
Previously using the .sendEvent method – notice how the eventData is variable
final Analytics analytics = Analytics.development(
  tool: DashTool.flutterTool,
  flutterChannel: 'ey-test-channel',
  flutterVersion: 'Flutter 3.6.0-7.0.pre.47',
  dartVersion: 'Dart 2.19.0',
);

// To send data for hot reload time
final eventName = DashEvent.hotReloadTime;
final eventData = {
  'timeMs': 50
};

analytics.sendEvent(
  eventName: eventName,
  eventData: eventData,
);

New approach using the Event class
final Analytics analytics = Analytics.development(
  tool: DashTool.flutterTool,
  flutterChannel: 'ey-test-channel',
  flutterVersion: 'Flutter 3.6.0-7.0.pre.47',
  dartVersion: 'Dart 2.19.0',
);

// Creation of [Event] class with data which is exported
// from `package:unified_analytics`
//
// This design doesn't include a variable `eventData` map, instead
// the Map is created within the package itself using named parameters
// on the `Event` named constructors
final hotReloadEvent = Event.hotReloadTime(timeMs: runTime);

// Use new API to send [Event]
analytics.send(hotReloadEvent);

[x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

@bwilkerson I have applied the required keyword for the events that don't need refactoring and added TODOs for the events we will need to think about more

This PR proposes this API:
final hotReloadEvent = Event.hotReloadTime(timeMs: runTime);
analytics.send(hotReloadEvent);

...which I expect will usually be used like this:
analytics.send(Event.hotReloadTime(timeMs: runTime));

Have we considered going even simpler?:
analytics.sendHotReloadTime(timeMs: runTime);

This PR proposes this API:
final hotReloadEvent = Event.hotReloadTime(timeMs: runTime);
analytics.send(hotReloadEvent);

...which I expect will usually be used like this:
analytics.send(Event.hotReloadTime(timeMs: runTime));

Have we considered going even simpler?:
analytics.sendHotReloadTime(timeMs: runTime);

I did consider this option but I decided against it in favor of having a separate event.dart file that focuses only on the events we send. In terms of transparency, this solution seemed cleaner because anyone that wants to see what we collect can view this one file and look at analytics.dart to see how other available APIs on the Analytics class.
Additionally, if we wanted to use dart code to query against the BigQuery database, it would be easy to import this package into the dart code and use the existing Event classes to query records in the database.

Done and done. See https://github.com/dart-lang/tools/pull/115.

FYI that the next time this rolls into the SDK the uses there will need to be updated (https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket/8777753770184906449/+/u/build_dart/stdout).

FYI that the next time this rolls into the SDK the uses there will need to be updated (https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket/8777753770184906449/+/u/build_dart/stdout).

Good call, @bwilkerson is this something you will be able to handle on the SDK side?

I can handle the analysis server uses. I don't know what other uses there are in the SDK, so I don't know how much domain knowledge I'd need. I'll guess I'll find out when I try making the server changes.
This probably warrants some thought as to the workflow for future updates (though hopefully there won't be too many more big changes like this one).


Disable 1ES codeql due to unreliability

From Azure sdk for js repo >

All the other languages have disabled codeql and we are now hitting blocking issues when running codeql on macos (https://dev.azure.com/twcdot/Data/_workitems/edit/138580) and even on the other OS's the configuration causes the builds to run for a long time.

https://github.com/Azure/azure-sdk-for-js/pull/30277

No changes needing a change description found.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/3770/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/3770/


Crash when getting refresh token

Environment details

OS type and version: MacOS Monterey 12.2
Python version: 3.9.10
pip version: 21.3.1

google-api-core                   2.5.0
google-api-python-client          2.37.0
google-auth                       2.6.0
google-auth-httplib2              0.1.0
google-auth-oauthlib              0.4.6
googleapis-common-protos          1.54.0

Code example

from googleapiclient.discovery import build
from googleapiclient.errors import HttpError
from oauth2client.service_account import ServiceAccountCredentials

scope = ['https://spreadsheets.google.com/feeds',
             'https://www.googleapis.com/auth/drive']

KEYFILE = 'google_api_keyfile.json'
creds = ServiceAccountCredentials.from_json_keyfile_name(KEYFILE, scope)
service = build('sheets', 'v4', credentials=creds)

Console output
Attempting refresh to obtain initial access_token
Refreshing access_token

Then the console hangs.
Not only that, though - an osx task call kernel_task starts using the maximum
disc on my machine, thrashing my ssd and making my machine unusuable until
I force the console to terminate ??!?

Hi @kewp,
It looks like you're using oauth2client for auth, which is now deprecated. Could you try this instead (using google-auth?
requirements.txt
google-auth
google-api-python-client

Code
from googleapiclient.discovery import build
from google.oauth2 import service_account

scope = ['https://spreadsheets.google.com/feeds',
             'https://www.googleapis.com/auth/drive']

KEYFILE = 'google_api_keyfile.json'
credentials = service_account.Credentials.from_service_account_file(
    KEYFILE, scope)
service = build('sheets', 'v4', credentials=creds)

In order to use a service account to access the API you must also perform domain-wide delegation. (https://support.google.com/a/answer/162106?hl=en#zippy=%2Cset-up-domain-wide-delegation-for-a-client)
Otherwise, the typical way to access this API is through a user's oauth2 credentials. https://developers.google.com/sheets/api/quickstart/python has a full example of this.

Hmm ok so the format of the call is a bit different:
creds = service_account.Credentials.from_service_account_file(KEYFILE, scopes=scope)

but I still get the same problem as before when I try read values: kernel_task
starts hammering my hard drive (like 100MB per second read and write),
my computer slows down to a crawl and the python script just hangs:
service = build('sheets', 'v4', credentials=creds)
result = service.spreadsheets().values().get(
        spreadsheetId=sheet_id, range=sheet_name).execute()

Ok, nevermind. Seems my code is the one that is freezing.
Still not sure why it froze before ... thank you for the help @busunkim96 . Sorry for the trouble!


add daily wallpaper

I tested this and I really like it. Though I think it should have its own separate settings. I'm thinking of a setting like enabling/disabling the dynamic background and we should include this there. So please don't close this yet. I'll make the settings for the background first then we'll merge this. Thank you!


New version: JetBrains.PhpStorm.EAP version 242.19890.20

Created by 🥟 Dumplings in workflow run #6343.

Log
Updated: 242.19533.60 → 242.19890.20
Submitting WinGet manifests
Creating manifests
WinGet exits with exitcode -1978335231
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run

/azp run

https://github.com/microsoft/winget-pkgs/issues/142854


Termination condition for simulation

Hello! I've been finding this simulation tool extremely useful, and think a feature that would enhance this is to be able to have a rule type which will terminate a simulation when a specified condition is met. What I have in mind I think would be a heaviside function that would end the simulation when a species, or the sum of a subset of the species (kept track of by an additive rule) exceeds or goes below some threshold. Ideally it would return the truncated simulation. Thanks for your help!
Best,
Rory

Thanks for this suggestion! In the future can you please leave comments on the biocircuits fork of bioscrape which we have officially moved to?


HAXM is about 10x slower than kvm in Nested virtualization environment

I have 2 vm running in a linux host accel by kvm。One is windows10 and the other is ubuntu19.1 ，both installed android studio。The android emulator is pretty slower in the windows system accel by haxm , it   takes about 5min to boot up。And in the ubuntu，the emulator take about 30s to boot up accel by kvm。
I noticed that , there is a pull request "Optimizing VMCS reads/writes with a new cache implementation" try to fix this problem ，is there any progress？or is there some patch i can test ?

Thanks for your report. KVM supports nested virtualization while HAXM has not yet. That is why the emulator within the Ubuntu guest can boot up fast while Windows guest cannot.
The pull request #130 will not resolve this issue completely. This pull request is under review and rebasing. You may retrieve the current patches through below commands. Thanks.
$ git checkout -b kryptoslogic-vmcs-refactor master
$ git pull https://github.com/kryptoslogic/haxm.git vmcs-refactor

Android studio can understand that it runs in a virtual machine and don't uses haxm in this case? And in ubuntu android studio uses kvm, not haxm?


Feat/sidecar lints

Codecov Report
Base: 17.84% // Head: 18.14% // Increases project coverage by +0.29% :tada:

Coverage data is based on head (d6e9df8) compared to base (f3082c4).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
+ Coverage   17.84%   18.14%   +0.29%
==========================================
  Files         115      113       -2
  Lines        5356     5247     -109
==========================================
- Hits          956      952       -4
+ Misses       4400     4295     -105

Impacted Files
Coverage Δ

...ecar/lib/src/utils/syntactix_entity_utilities.dart
0.00% <ø> (ø)

...sidecar/lib/src/test/resources/resource_mixin.dart
42.85% <0.00%> (-7.15%)
:arrow_down:

packages/sidecar/lib/src/client/cli_client.dart
72.54% <0.00%> (-1.53%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


⚠ Bofrost.de has degraded performance

In 55b1f5a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1263 ms

Resolved: Bofrost.de performance has improved in a96f933.


fix: boilr cli requires the tag flag

Oh another docs derp! Sorry again. I struggle so hard with docs haha. It's much appreciated! <3


Set fields to only appear if content is non-null, tweak notes button …

…to appear in-line with content.
This PR fixes #94 #93.

General comment: We should probably use colours or just indicate field names for these alternative fields, since with this PR it's no longer clear what fields means what. E.g. while I could infer that @asdasd is Bernice's telegram handle, at a glance I can't tell what asd represents for Alex

Hey Kok Rui, thanks for the comments. Have removed dead code & fixed indentation accordingly.


#Failed to load the client on the blockchain network

Unable to initialize the client failed to initialize configuration: unable to load endpoint config: failed to initialize endpoint config from config backend: network configuration load failed: failed to load channel configs: failed to load network TLSConfig: failed to load orderer/peer TLSConfig : failed to load orderer cert: failed to load pem bytes from path /home/blockchain/projects/cartrack/network/crypto-config/ordererOrganizations/carscm.net/tlsca/tlsca.carscm.net-cert.pem: open /home/blockchain/projects/cartrack/network/crypto-config/ordererOrganizations/carscm.net/tlsca/tlsca.carscm.net-cert.pem: no such file or directory
after launchservices and docker-compose up in the log files this issue is there?? Any idea how to remove it?

Got the mistake. directory path was not correct. just changed it and the network worked


Exception in task execution graph ready listener does not fail build

Trying to implement check that will prevent dangerous task execution without explicitly set parameter, let's say -Pforce. However when throwing an exception in task execution graph listener / when graph ready, the exception is not causing a build failure.
Expected Behavior

Exception is thrown and build ends with failure.
Current Behavior

Exception is thrown, however build ends with success.
Context

The task is configured lazily using new API task.register. Regardless that the task could be unintentionally eagerly loaded in plugin user buildscript by using tasks.getByName() instead tasks.named() , I need to hook to the moment just after configuration phase to perform such check. During task execution is too late, because my build could be running for some time (even few minutes) then failing at last task requiring that -Pforce set could be very annoying. And that's the reason I tried with:

Steps to Reproduce (for bugs)

gradle-should-fail.zip
Your Environment

$ sh gradlew --version

------------------------------------------------------------
Gradle 4.10.2
------------------------------------------------------------

Build time:   2018-09-19 18:10:15 UTC
Revision:     b4d8d5d170bb4ba516e88d7fe5647e2323d791dd

Kotlin DSL:   1.0-rc-6
Kotlin:       1.2.61
Groovy:       2.4.15
Ant:          Apache Ant(TM) version 1.9.11 compiled on March 23 2018
JVM:          1.8.0_121 (Oracle Corporation 25.121-b13)
OS:           Windows 10 10.0 amd64

heh, upgraded to 5.0 and seems that you have fixed that in the meantime :) I am curious, what was the root cause?

I suspect it's related to #7329 if this worked in 4.9.
Since this is a really common pattern, we really should have something like:
<T extends Task> void whenTaskInGraph(TaskProvider<T> task, Action<T> action);

in TaskExecutionGraph that does something similar to what you're doing with whenReady.
That way, you could just do:
val a = tasks.register("a", A::class.java)
project.gradle.taskGraph.whenTaskInGraph(a) {
     this.willBeExecuted()
}

Is this something you'd be interested in contributing?

Hard to say if it is common pattern. Sure, I will consider contributing after completing my plugin development. Thanks for your response.


Using Markup.OnScreenSize in Code Behind with .net maui?

I wanted to use  Markup.OnScreenSize in my Code behind with .net maui but it isnt working.
I guess this feature isnt implemented in .net maui NuGet Package?
Or is the imlementation different than shown in the example?

Hi,
Yes. There is a nuget for maui please see https://www.nuget.org/packages/OnScreenSizeMarkup.Maui
😜

Hi,
The Problem is I installed the maui nuget but the feature doesnt work.
I added a Screenshot. In my xamarin project (with xamarin nuget package) the code is working fine.
https://paste.pics/d915508bc37a8fd78f4741a43773f738

Hi again,
Using the Maui Nuget I sent before, try this:
            using OnScreenSizeMarkup.Maui.Helpers;
...
            Grid grid = new Grid
            {
                RowDefinitions =
                {
                    //first row
                    OnScreenSizeHelpers.OnScreenSize(defaultSize: new RowDefinition { Height = new GridLength(100) },
                        large: new RowDefinition { Height = new GridLength(200) },
                        extraLarge: new RowDefinition { Height = new GridLength(300) }
                    ),
                    //second row
                    new RowDefinition(),
                    //third row
                    OnScreenSizeHelpers.OnScreenSize(defaultSize: new RowDefinition { Height = new GridLength(100) },
                        large: new RowDefinition { Height = new GridLength(200) },
                        extraLarge: new RowDefinition { Height = new GridLength(300) }
                    ),
                }
            };

HI again,
Just complimenting the previous answer, the latest of the nuget for MAUI had some minor adjustments due to the move to MAUI Community Toolkit that soon might occur.
While that move doesn't happen, I need to find a free spot for creating documentation a specific for MAUI.


The automated release is failing 🚨

:rotating_light: The automated release from the master branch failed. :rotating_light:
I recommend you give this issue a high priority, so other packages depending on you could benefit from your bug fixes and new features.
You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can resolve this 💪.
Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.
 You can also manually restart the failed CI job that runs semantic-release.
If you are not sure how to resolve this, here is some links that can help you:

Usage documentation
Frequently Asked Questions
Support channels

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.

No vsce personal access token specified. (set env "VSCE_TOKEN")
Unfortunatly this error doesn't have any additionnal information. Feel free to kindly ask the author of the semantic-release-vsce plugin to add more helpful informations.

Good luck with your project ✨
Your semantic-release bot :package::rocket:

Fixed.


Bug: Application error: Cannot read properties of undefined (reading 'schema')

Error message: Cannot read properties of undefined (reading 'schema')
Error stack:
TypeError: Cannot read properties of undefined (reading 'schema')
    at F._next (altair://-/main.js:1:2114040)
    at F.__tryOrUnsub (altair://-/main.js:1:11460)
    at F.next (altair://-/main.js:1:10705)
    at C._next (altair://-/main.js:1:9904)
    at C.next (altair://-/main.js:1:9678)
    at H0e._next (altair://-/main.js:1:178107)
    at H0e.next (altair://-/main.js:1:9678)
    at Ne._next (altair://-/main.js:1:16066)
    at Ne.next (altair://-/main.js:1:9678)
    at sCe._next (altair://-/main.js:1:9904)

When did this error occur? And does it re occur, or it happened only once?

Please provide some helpful screenshots of the bug
Version 5.0.22

Dunno, the prompt turned up and it asked me if I wanted to file a report.Maybe related to the fact that the schema disappears if the schema window sticks around long enough?-- Sent from Rob McMillin’s iPadNESR data janitorDramatis personae: ONB Libby, NESR Jed, and various NESR fostersLocation: Woofwood, Little Rock, AROn Apr 19, 2023, at 12:48, Vishal Yadav @.***> wrote:
I would like to contribute on this issue, so can u elaborate it more clearly
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Please provide me more helpful screenshots of the bug

When does this error occur? And does it re occur, or it happened only once?
Please respond to the other questions in the issue template else this issue is not useful. Thanks.


Jennaramdenee/webdev 113 allow url params without get - WIP, DO NOT MERGE

Parliament::Request::UrlRequest

New method set_url_params that produces a string representing the query part of a GET request
Amendment to method_missing and query_url methods to build a URL using params

specs

New test for set_url_params method
Amendment to existing tests for new data structure

Coverage remained the same at 100.0% when pulling 97b8f19c4e7b6e6016acb2c961e5c547b3fc5c1b on jennaramdenee:jennaramdenee/webdev-113_allow-url-params-without-get into d1c1afcd563b25da57be22d44fe16dc76d389520 on ukparliament:master.


Generated types has an array type for a database Function that returns a table

Bug report
Generated types for functions that return a table are erroneously an array type.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Create PG Function that returns a table:

create or replace function blog_posts(post_author_username text)
    returns table
            (
                id                  bigint,
                created_by          uuid,
                created_by_username text
            )
as
$body$
select np.id,
       np.created_by,
       p.username created_by_username
FROM public.news_posts np
         inner join public.profiles p on np.created_by = p.id
where p.username = post_author_username;
$body$ language sql;

generate types: supabase gen types typescript --local > ./src/types/database.types.ts
query db:

type ArrayElement<ArrayType extends readonly unknown[]> = ArrayType extends readonly (infer ElementType)[]
	? ElementType
	: never

export type NewsPost = ArrayElement<Database["public"]["Functions"]["blog_posts"]["Returns"]>

...

const [posts, setPosts] = useState<NewsPost[]>([])
useEffect(() => {

useEffect(() => {
	const fetchData = async () => {
		const { data, error } = await supabase.rpc("blog_posts", { post_author_username: username })
		setPosts(data || [])
	}

	// call the function
	fetchData().catch(console.error)
}, [])

Breaks because generated type is

blog_posts: {
   Args: { post_author_username: string }
      Returns: {
          id: number
          created_by: string
          created_by_username: string
        }[]
      }
   }

Expected behavior
Generated type should not be an array type. Works fine if [] is deleted.
System information

supabase --version: 1.34.5
Version of Node: v18.13.0

Hey @AlexDunmow, I think this is related to https://github.com/supabase/postgrest-js/pull/394. Can you try npm update @supabase/postgrest-js and see if that fixes it?

Same issue here. I dont have a reference to @supabase/postgrest-js in my project. I guess we need to wait for a new version of the supabase npm package to be published?

Sorry, I should've been clearer: @supabase/supabase-js has @supabase/postgrest-js as a dependency - presumably you'd have an older version of @supabase/postgrest-js in your package-lock.json, and updating the package should fix it.

Resolved in https://github.com/supabase/postgrest-js/pull/394


bump httpx to 0.26.0

I am the creator on Ollama Conversation, I am wanting to get the integration listed on Home Assistant itself but in order to do that this condition needs to be met communication-with-devicesservices.
So I need to switch from making direct API calls to using this library, however the version of httpx required by this library has conflicts with Home Assistant requirements.
This PR just bumps httpx version to 0.26.0 in order to meet the requirements.

Closing, already done https://github.com/ollama/ollama-python/blob/main/pyproject.toml#L13


Hope to provide Mysql Metadata Service Module

Describe the solution you'd like
Implement the interface of querying Sql metadata
Describe alternatives you've considered
no.
Additional context
no.

thanks it is closed in #405


Feature/ledger error reporting

Created the new branch fix/guard-error-handling containing a cleaner version of this PRs content


Use anemone for memory pool

Need this before the next snapshot

this should happen at the same time as switching to keeping track of alloc'd memory per entity?

you mean anemone should keep track of allocation?

yup. I was planning on overestimating - counting total number of blocks allocated rather than actual used bits. which means it should only be at most 1MB larger than it really is

ie keep total size of allocated blocks in top-level mempool struct, so fast case allocs are still fast, when it runs out of a block it adds that to the total on mempool since it has to touch it then anyway

Worth noting that this will break icicle-make (pulling in anemone in generated code), but I don't think that's a priority for the moment.


feat(quicklog): Add quicklog

The prize for the first 'real commit' goes to @nhzaci ! Thanks for all the hard work!


Fix broken links to chapters within NCEA curriculum guides

Description
There are many broken links to chapters within the NCEA curriculum guides, for example:
[2.1 - What’s the bigger picture?](algorithms.html#whats-the-big-picture)

which should be corrected to:
[2.1 - What’s the bigger picture?](chapters/algorithms.html#whats-the-big-picture)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the issue.

[x] I have linked any relevant existing issues/suggestions in the description above (include #??? in your description to reference an issue, where ??? is the issue number)
[x] I have applied relevant labels

This is still an issue. In the NCEA curiculum guide page some links are broken - Algorithms - Searching (Merit/Excellence) for example leads to a 404 page.

Thanks @microlith57, I fixed the broken links but then broke the working ones by merging in some old redesign work. I'll get this fixed in the next 12 hours (hopefully fairly quickly).

@microlith57 Creating a new issue (#483) for the bug you found as it's a slightly different error to this one.

I've closed the new issue #438 as I've deployed version 2.10.1. Should all be fixed now!


Convenience library

Use this to disable skip directives in the tests so we don't need to
remember to replace or delete code while we are evaluating.

This still needs the update in the readme, if we can use this.  Around here it says to define the skip method to disable the skips in the test.

This is handy, thanks!


As a user, I want to toggle between stage views

I want to be able to show all applicants per stage or all applicants in the opening. This would be a scrollable beautiful dnd component that loads 25 applicant a a time like a trell o

No longer relevant at this time


Article display doesn't account for ' or " in article

The TechCrunch articles being displayed render funky characters in place of ' and " in the articles' text. Not a big deal, but something to improve.

@dunnapey. Thanks for suggestion !  I will fix it soon 👍🏻


Add Ecumenopolis from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of Vortinium, to add Ecumenopolis to CKAN.
Please direct questions about this pull request to Vortinium.
Mod details:
name = /mod/2272/Ecumenopolis
author = Vortinium
abstract = A planet-wide city.
license = CC BY-NC-SA 4.0
Homepage =
description =
This is your mod listing! You can edit it as much as you like before you make it public.
To edit this text, you can click on the "Edit this Mod" button up there.
By the way, you have a lot of flexibility here. You can embed YouTube videos or screenshots. Be creative.
You can check out the SpaceDock markdown documentation for tips.
Thanks for hosting your mod on SpaceDock!

#rebuild


Prevent redirect to previous page - Modal signup / Signin

Hi,
I am using a modal signUp / signIn workflow and so when a user signs up, all that needs to happen is close the signUp modal on that page and not redirect. Currently, after the signup, the user is redirected to one page back.
This is my onSubmitHook:
var submitFunc = function(error, state){
  if (!error) {
    if (state === "signIn") {
      $('#modalSignIn').modal('hide');
    }
    if (state === "signUp") {
      $('#modalSignUp').modal('hide');
    }
    // Router.go(Router.current().url); -- This does not work either
  }
};

Hello @niranjans!
Lets have a look at this thread by @Maxhodges. He also provided a condensed example here in the wiki.
See also this other solution proposed by @BradRyan
We'll than have to converge to a common solution so tu put it inside v2.0 which is under development...
Let me know!

Hi @splendido , Thanks for your reply and for the resources. It really helped. I also found out about AccountsTemplates.avoidRedirect = true;  which stopped the redirection.

:+1:


Add Quilt Mappings support

There are still some things left I want to do, but this should work as the last PR (#4)

[ ] Add intermediateMappings configuration to allow users to use Fabric mods as dependencies by remapping with any of the dependencies in the configuration
[ ] Multiple access wideners support - Not necessary
[x] Support for yarn and other mappings with the mapping file in mappings/mappings.tiny instead of hashed/mappings.tiny
[ ] Update tests

What's the status on this?


fix(dash): live to vod transition seek start

Fixes https://github.com/shaka-project/shaka-player/issues/7346
To be honest i'm not 100% sure of my PR, it's working but i may break something so happy to be corrected and if you see better approach let me know
Also i will need more device to test it as i'm limited at home

I think the change could be good, but I need to test it with several streams from different providers.

I have tested it with livesim2 and with ATEME streams and it works correctly


Rename hooks file and updated README

I made this PR because the Pre-commit API changed and the hooks file was not found.
Also Removed a previous conflict marker in the README.MD file

@shihoonhootsuite please review


Consider precomputing a DAFSA or perfect hash for the list

A common strategy used by many PSL implementations is to optimize the PSL for both size and speed, given that it contains a complete set of the IANA Root Zone Database, in addition to public and private registerable domains.
Firefox currently uses a simple hash, Chromium previously used a gperf-generated hash tree, and presently uses a DAFSA, while Guava uses a trie
These solutions each make different trade-offs with respect to size (in memory) and speed, but all offer better performance than the current implementation, and are using relatively friendly licensing.

The built-in data uses a sorted array, the lookup is a binary search, lookup performance is O(log n).
The good thing is startup time - no startup code has to be executed, no pointers have to be resolved.
For short-running tools like curl or wget, library startup time matters - the common use case either doesn't use cookies / libpsl at all or need just a handful of lookups. So introducing a library startup (building a hashmap) would contradict my assumptions when I designed libpsl (builtin data).
But just have a look at current speed: on a Intel i3 using 1 core, I can do 5-10 million lookups per second (taken the PSL data as input). The libpsl code is reentrant, so you could use any number of cores in parallel - if that matters to you. But of course, a hashmap could boost the lookups by a factor of ~10 (roughly).
This in mind, what kind of bottleneck do you have resp. what is your use case where speed matters that much ?
For dynamic loaded PSL data I keep this in mind - initialization is roughly the same for a hashmap and for a vector. Code complexity is bit higher, but nothing to worry.

The good thing is startup time - no startup code has to be executed, no pointers have to be resolved.

Well, both the gperf and DAFSA code of Chromium share this property, so I don't think it is unique to your solution.
The naive solution (which is similar to what you're using now) was roughly an order of magnitude larger than the gperf solution (which traded a one-time cost at build for faster load times), which itself was about 8X larger than what we currently use for the DAFSA.
In Chrome, we found that library size (as in, the size of the .so/dll) to understandably contribute to load times and page faults, which is why we reoptimized things. Opera contributed the DAFSA code to make our PSL implementation smaller for mobile - 200K vs 35K is a BIG difference for small apps.
My hope is that libpsl will optimize to use the smallest amount of memory possible (that is compatible with your complexity and performance goals), to make it more suitable for use in smaller devices. I filed the issue since it seemed like an easy win, low-hanging fruit, and friendly licensing of existing solutions :)

I read about DAFSA - very nice concept, indeed. I'll implement that definitely, i am just pretty busy right now. Hope, I can do that within the next weeks.

DAFSA is in branch 'develop' now, thanks again for your request.
The dictionary size (and thus the library size) has been dramatically reduced by DAFSA.
Some measurements:
Average DAFSA lookup (see tests/test-is-public-all.log after a 'make check') is ~3 times slower than binary search in static array (as before DAFSA). I don't care much, this still means ~1.5 million lookups per second on a 3.1GHz i3 (on a single core).
Size of the library (stripped) here (amd64) is now 46816 vs 435352 before, a factor of 9.3 !!!

Released in libpsl-0.12.0
Thanks for pointing out DAFSA !


fix issue #2124 and rephrase the challenge a bit to avoid confusion

Spotted an issue in the challenge and also had an idea to improve the wording a bit to avoid confusion, since the challenge requests matching characters that are not spaces.
Thanks for your time to review the pull request!

Also fixes #2035


Simple tvpaint publish

Changes

review instance is always created even if review is not in instances of workfile metadata

subset name is hardcoded to "reviewMain"

instances without "uuid" in data are skipped on pyblish plugin toggle callback
removed review creator

we should maybe keep this creator to be able change subset name of review?

Closes https://github.com/pypeclub/client/issues/67

Sorry, dont quite get this. Can the user disable/toggle the review still in the Pyblish window?
Just wanna make sure users can publish faster by only outputting what is needed.

Can the user disable/toggle the review
Yes of course.

The point of this is to make it as easy a quick as possible for artists that maybe don't have full training, to open a scene, do some work and publish with simple review without having to use creator. Production request to make life easier for freelancers.

The point of this is to make it as easy a quick as possible for artists that maybe don't have full training, to open a scene, do some work and publish with simple review without having to use creator. Production request to make life easier for freelancers.

Sorry, yeah I get that and agree. Just wanna make sure advanced users still have options.

@tokejepsen just noticed, that my "Yes, of course" got lost in the quote in the previous post. So just to make sure. Yes, it should be possible to disable the review just as any other instance ;)


Build fails due to unused static variable "LOG_TAG" in esp-idf/components/esp_adc_cal/esp_adc_cal_esp32.c (IDFGH-5941)

Hello everyone,
I am trying to compile my project using idf.py build on a Debian machine with latest ESP-IDF (commit 3e370c4296247b349aa3b9a0076c05b9946d47dc).
But I get the following error:

/home/user-xy/.espressif/tools/xtensa-esp32-elf/esp-2021r1-8.4.0/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc -DESP_IDF_LWIP_HOOK_FILENAME="global_lwip_hooks.h" -I/home/user-xy/esp/projects/project-xy/webinterface/main/global-include -Iconfig -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_adc_cal/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/newlib/platform_include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/freertos/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/freertos/include/esp_additions/freertos -I/home/user-xy/esp/projects/project-xy/esp-idf/components/freertos/port/xtensa/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/freertos/include/esp_additions -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_hw_support/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_hw_support/include/soc -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_hw_support/include/soc/esp32 -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_hw_support/port/esp32/. -I/home/user-xy/esp/projects/project-xy/esp-idf/components/heap/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/log/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/lwip/include/apps -I/home/user-xy/esp/projects/project-xy/esp-idf/components/lwip/include/apps/sntp -I/home/user-xy/esp/projects/project-xy/esp-idf/components/lwip/lwip/src/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/lwip/port/esp32/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/lwip/port/esp32/include/arch -I/home/user-xy/esp/projects/project-xy/esp-idf/components/soc/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/soc/esp32/. -I/home/user-xy/esp/projects/project-xy/esp-idf/components/soc/esp32/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/hal/esp32/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/hal/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/hal/platform_port/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_rom/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_rom/include/esp32 -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_rom/esp32 -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_common/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_system/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_system/port/soc -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_system/port/public_compat -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp32/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/xtensa/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/xtensa/esp32/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/driver/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/driver/esp32/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_pm/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_ringbuf/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/efuse/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/efuse/esp32/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/vfs/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_wifi/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_event/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_netif/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_eth/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/tcpip_adapter/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_phy/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_phy/esp32/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_ipc/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/app_trace/include -I/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_timer/include -mlongcalls -Wno-frame-address   -ffunction-sections -fdata-sections -Wall -Werror=all -Wno-error=unused-function -Wno-error=unused-variable -Wno-error=deprecated-declarations -Wextra -Wno-unused-parameter -Wno-sign-compare -ggdb -mfix-esp32-psram-cache-issue -mfix-esp32-psram-cache-strategy=memw -O2 -fmacro-prefix-map=/home/user-xy/esp/projects/project-xy/webinterface=. -fmacro-prefix-map=/home/user-xy/esp/projects/project-xy/esp-idf=IDF -fstrict-volatile-bitfields -Wno-error=unused-but-set-variable -fno-jump-tables -fno-tree-switch-conversion -std=gnu99 -Wno-old-style-declaration -D_GNU_SOURCE -DIDF_VER="v4.4-dev-3235-g3e370c4296" -DESP_PLATFORM -DNDEBUG -MD -MT esp-idf/esp_adc_cal/CMakeFiles/__idf_esp_adc_cal.dir/esp_adc_cal_esp32.c.obj -MF esp-idf/esp_adc_cal/CMakeFiles/__idf_esp_adc_cal.dir/esp_adc_cal_esp32.c.obj.d -o esp-idf/esp_adc_cal/CMakeFiles/__idf_esp_adc_cal.dir/esp_adc_cal_esp32.c.obj   -c /home/user-xy/esp/projects/project-xy/esp-idf/components/esp_adc_cal/esp_adc_cal_esp32.c
/home/user-xy/esp/projects/project-xy/esp-idf/components/esp_adc_cal/esp_adc_cal_esp32.c:86:19: error: 'LOG_TAG' defined but not used [-Werror=unused-const-variable=]
const static char LOG_TAG[] = "ADC_CALI";
^~~~~~~
cc1: some warnings being treated as errors
[747/1085] Building C object esp-idf/console/CMakeFiles/__idf_console.dir/esp_console_repl.c.obj
[748/1085] Building C object esp-idf/esp_hid/CMakeFiles/__idf_esp_hid.dir/src/esp_hidd.c.obj
[749/1085] Building C object esp-idf/console/CMakeFiles/__idf_console.dir/linenoise/linenoise.c.obj
[750/1085] Building C object esp-idf/console/CMakeFiles/__idf_console.dir/argtable3/argtable3.c.obj
[751/1085] Building CXX object esp-idf/asio/CMakeFiles/__idf_asio.dir/asio/asio/src/asio.cpp.obj
ninja: build stopped: subcommand failed.
ninja failed with exit code 1
Executing action: all (aliases: build)
Running cmake in directory /home/user-xy/esp/projects/project-xy/webinterface/build
Executing "cmake -G Ninja -DPYTHON_DEPS_CHECKED=1 -DESP_PLATFORM=1 -DIDF_TARGET=esp32 -DCCACHE_ENABLE=0 /home/user-xy/esp/projects/project-xy/webinterface"...
Running ninja in directory /home/user-xy/esp/projects/project-xy/webinterface/build
Executing "ninja all"...

I noticed during compilation that this warning (here treated as an error) also occurs in other components of the ESP-IDF but there it is not treated as an error.
Very interesting is that the error does NOT occur with the same project, same ESP-IDF commit on my windows machine.
Any ideas why this error pops up on linux (and interestingly not on windows) ?
Don't know if it's important, but I have customized the CMakeLists.txt in the project folder (not in the main folder), because I need a global LwIP hook:

cmake_minimum_required(VERSION 3.5)
include($ENV{IDF_PATH}/tools/cmake/project.cmake)
add_compile_definitions(ESP_IDF_LWIP_HOOK_FILENAME="global_lwip_hooks.h")
include_directories("main/global-include")
project(My-Project)

Thank you!

Hi @zinke-ct-video,
are the sdkconfig files in your project directory the same between the two computers?
It looks like this warning may occur if the log level is set to "None" in sdkconfig.

 The issue does not appear anymore. Seems as if someone has solved this. Thanks a lot!


Adjust "ember-core" package name

:+1:

@homu r+

:pushpin: Commit 2535ebd has been approved by stefanpenner

:zap: Test exempted - status


Firewizard updates rogy

firewizards PR with edits too:

re-adding magento version under feature flag
sku_code required, keeping product ID the same as firewizards recommendation (sku code can end up as a duplicate of ID, this is fine)
updated some old branding (QuBit is now Qubit)

@firewizard made a few edits that I have to stick too my side, look ok?

Also, I had started implementing the 'configurable product' switch on the product pages (but on the old style implementation before your updates)
If you see: https://github.com/QubitProducts/UniversalVariable-Magento-Extension/blob/updates-rogan-2/app/design/frontend/base/default/template/qubit/uv_configurable.phtml
You can see the way I was going for, do you think this is a smart way to go forward?

@rogy, here's my feedback:

you forgot a print($data); in app/code/community/QuBit/UniversalVariable/Model/Uv.php on line 61
not sure if you needed to move the getSku line outside the if. if I understood the specs correctly, sku and stock info should be displayed together: http://docs.qubitproducts.com/uv/specification/#additionalpropertiesforproductsrequiringstockkeeping - there's not required to any of these
voucher field is a string, not an array: http://docs.qubitproducts.com/uv/specification/#additionalpropertiesforproductswithpromotions
the show_magento_version label and comment are missing from the csv
i see you changed all texts to Qubit, shouldn't the namespace change as well? that's a camel-case too
why did you remove the modman file?

regarding configurable products, i can't really say for sure as I don't know what getCustomOptions is supposed to do. The approach might work, but then again, i'm not sure if it has to be done this way. i'll think about it over the next days. Personally, I'd merge everything without the configurable part and work on that with the new coding style.

Hi @firewizard

good spot, thank you
Yes, understood, just the way we do it just now requires to always have access to the SKU, so we don't want to have a situation where they use the product ID and not stock, so we cannot access SKU
See http://docs.qubitproducts.com/uv/specification/#basket and transaction pages, vouchers should be an array of strings, i know in the Magento case by default it only allows one, but just to keep it standard doing this works (this was requested by a colleague to change this to an array like the spec)
thank you
namespace isn't too important as it won't be seen by anyone but developers really, easier than updating the whole fire structure
accidental - I'd never actually used modman and now I can't stop!

regarding the configurable, absolutely will put that in another PR.

sorry, my bad, i was looking at the product specs, where voucher is not an array, but a string.


[HUDI-6902] Try container for Azure tests

Change Logs
As title.
Impact
Isolate the jobs.
Risk level (write none, low medium or high below)
None.
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@hudi-bot run azure

@hudi-bot run azure

duplicating the efforts.


Crash when launching in Ubuntu

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.90.0 89de5a8d4d6205e5b11647eb6a74844ca23d2573 x64
OS Version: Ubuntu 22.04.4 LTS

crashreport.zip
Hi,
This morning when I tried to start vscode, it shows a window for a few seconds and then shuts down again without a warning. I generated a crash report and attached dmp files which were located in the pending subfolder.
Thanks for your help,
Thomas

Crash occurs with and without extensions enabled.

Also tried reinstalling:
sudo snap remove code
rm ./.vscode
rm -rf ~/.config/Code/
sudo snap install --classic code
Did not help!

@joyceerhl I am also facing the same issue .Kindly help

Workaround: https://askubuntu.com/a/1373424/955461
snap revert code

Workaround: https://askubuntu.com/a/1373424/955461
snap revert code

Thanks PoLaKoSz! I did not help, though:
error: cannot revert "code": no revision to revert to

How can we specify a working version as a snap channel if we unfortunately uninstall code for trying to re-install it?

which snap revision? dc96b... is too long for a revision.

Reverting to
1.89.1
dc96b837cf6bb4af9cd736aa3af08cf8279f7685
x64

fixed it for me

Thank you Perondas, but can u please explain how you exactly executed the snap revert command?

Reverting to
1.89.1
dc96b837cf6bb4af9cd736aa3af08cf8279f7685
x64

fixed it for me

Thank you Perondas, but can u please explain how you exactly executed the snap revert command?

The revision is dc96b837
Id doesn't seem to be published on any channel tho, so unless snap revert works for you, I don't know how you could get it.

Never mind
snap install code --revision 159

should do it

For those interested, this issue doesn't seem completely new and is investigated in this other issue

Never mind
snap install code --revision 159

should do it

thx you, it helped

/duplicate
Pointing dups to https://github.com/microsoft/vscode/issues/204159#issuecomment-2151226947


Implement some sort of RootStore

We need a RootStore that stores all working instances.
RootStore should be able to:

[ ] Add an instance
[ ] Get an instance
[ ] Update an instance
[ ] Destroy an instance
[ ] Link related instances together (is Inversify involved here?)
[ ] Provide a public API as an alternative to hooks, primarily for non-React applications until we can provide first-class support for other frameworks

Let's keep local instantiations in mind. I should be able to create an instance on the fly, store it in the RootStore, perform a mutation that transports the data to the server and updates that same instance in response. Though already without thinking too much about it, this seems impossible as there's no identifier to connect the dots. Perhaps the solution there is just to move the data from the response, including the ID, into the local instance.

Implemented, seefb39c38


Fix command-line syntax

Currently, if user executes command notation sign --help, the print-out shows
Usage:
  notation sign [reference] [flags]

The reference  parameter is mandatory, and the flags parameters are optional.
We should improve command-line syntax to give users a clear indication. Strongly suggest fixing the syntax for rc.1 release.
Suggest using the following syntax:

Notation
Description

Text without brackets or braces
Items you must type as shown.

<Text inside angle brackets>
Placeholder for which you must supply a value.

[Text inside square brackets]
Optional items.

{Text inside braces}
Set of required items. You must choose one.

Vertical bar ( | )
Separator for mutually exclusive items. You must choose one.

Ellipsis (…)
Items that can be repeated and used multiple times.

Reference: https://learn.microsoft.com/en-us/windows-server/administration/windows-commands/command-line-syntax-key
By applying this syntax, the notation sign --help print-out will look like below:
Usage:
  notation sign <reference> [flags]

/cc: @dtzar @FeynmanZhou

This issue will be fixed by the updates on CLI specs.

Close it as this issue is addressed by CLI specs.


Adding "content-available" property

Hi, is it possible to add content-available property inside aps json? I need this for background notification with phonegap-plugin-push.
{
    "aps": {
        "data": "Test silent background push",
        "moredata": "Do more stuff",
        "content-available": 1
    }
}

Thank you.

Hi,
Yes, just call apn_payload_set_content_available(payload, 1)

oops.. I posted issue on the wrong repository. Actually I wanted to ask about this feature in php-apn.
I could not find apn_payload_set_content_available() function from php-apn dll library downloaded from : http://windows.php.net/downloads/pecl/releases/apn/1.0.3/php_apn-1.0.3-5.4-nts-vc9-x86.zip


Add missing license field in opam file

This is in the dune-project file already. Maybe I forgot to build the updated opam file? Too bad we are not currently able to catch that in CI!

This is in the dune-project file already. Maybe I forgot to build the updated opam file? Too bad we are not currently able to catch that in CI!

Yep. I noted that in my commit message too. :)


i got error when i have use  in sfc vue file

if i wrote nothing such as this

then i will get some error

and when i have wrote something such as space key or some enter key， then it would work

i think you have not create the file for style when it has nothing, but the  *.vue still import the style file

I try to reporduction for this, but it always work for me.Can you give a minimize reproducction for this?

I try to reporduction for this, but it always work for me.Can you give a minimize reproducction for this?

https://github.com/Jeromy-L/test-vite/tree/test2
this branch
and http://localhost:9876/dev    dev page , you can see the error
and could you help me to find the sourcemap breakpoints error?
https://github.com/vitejs/vite/issues/1156
this issue and this branch can also reporduction
thanks

In here, we just discuss  i got error when i have use <style scoped></style> in sfc vue file #1163 this issue.
I clone your reproduction and use test2 branch, but also not enconter some error.... Can you ensure that?

do you open the page?
 http://localhost:9876/dev
this page, and then open devtools to the network panel, then will find that the css request fail
------------------ 原始邮件 ------------------
发件人:                                                                                "vitejs/vite"<EMAIL_ADDRESS>发送时间: 2020年11月29日(星期天) 下午3:36
收件人<EMAIL_ADDRESS>抄送: "✨Jeromy💫"<843755624<EMAIL_ADDRESS>主题: Re: [vitejs/vite] i got error when i have use <style scoped></style> in sfc vue file  (#1163)
In here, we just discuss  i got error when i have use <style scoped></style> in sfc vue file #1163 this issue.
I clone your reproduction and use test2 branch, but also not enconter some error.... Can you ensure that?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

In here, we just discuss i got error when i have use <style scoped></style> in sfc vue file #1163 this issue.
I clone your reproduction and use test2 branch, but also not enconter some error.... Can you ensure that?

do you open the page?
http://localhost:9876/dev
this page, and then open devtools to the network panel, then will find that the css request fail

Have you reproduction？
------------------ Original ------------------
From: underfin<EMAIL_ADDRESS>Date: Mon,Nov 30,2020 6:07 PM
To: vitejs/vite<EMAIL_ADDRESS>Cc: Jeromy<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [vitejs/vite] i got error when i have use <style scoped></style> in sfc vue file  (#1163)
Closed #1163 via underfin/vite-plugin-vue2@0dd5e0a.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

In here, we just discuss i got error when i have use <style scoped></style> in sfc vue file #1163 this issue.
I clone your reproduction and use test2 branch, but also not enconter some error.... Can you ensure that?

have you open the page? the pic that you upload just only run the dev:vite, no with open the page. and the bug was only happen when you open the page which has import

Yeah. That already fixed, please waite vite-plugin-vue2 next version publish.


Node doesn't run at all now

I think I had a few versions of Node installed on my system in various ways, but they were all outdated so I tried this nvm-windows.
It detected the 3 or 4 other versions I had on my system and asked if it wanted me to manage them. I said yes.
Install completed fine, but when cmd.exe loaded up I got this error, but nvm seems to run fine so I don't think much of it. I try nvm install latest. It downloads 0.12.7. And now...
C:\Users\Mark>nvm list

    1.1.0
    0.8.9
    0.12.7
    0.10.35

C:\Users\Mark>nvm install latest
Version 0.12.7 is already installed.

C:\Users\Mark>node --version
The system cannot find the path specified.

C:\Users\Mark>nvm use 1.1.0
Now using node v1.1.0 (64-bit)

C:\Users\Mark>node --version
The system cannot find the path specified.

Even though 0.12.7 is supposedly installed, it's not listed, and node doesn't even exist anymore.

Is it really listing 1.1.0? A 1.x.x version of node hasn't been released yet. The list of available distributions NVM for Windows pulls from doesn't have that either... so I'm not sure where that's coming from.
As for 0.12.7, run nvm use 0.12.7, then node --version. Remember that NVM for Windows creates a symlink to to the node executable, so if you set it to use 1.1.0, it is looking for a non-existant version, thus why your system doesn't recognize node anymore. What's perplexing me is that you should not have received  Now using node v1.1.0 (64-bit)... it simply doesn't exist yet.
Do you have io.js installed? The only thing I can think of right now is a potential conflict with it.

Yes. I think I'm pretty sure I did install iojs at some point.
I just uninstalled nvm-windows, then searched for all my copies of node and uninstalled/deleted those too.
Then reinstalled nvm-windows. nvm list comes up clean. I tried installing and using 0.12.7 again, but it still didn't work. I checked my path, and see a reference to C:/Program Files/nodejs. That folder doesn't exist on my system. The variable says "symlink" so I tried getting the link manually. Then node.exe started working, but not node. Same error as above: "The system cannot find the path specified."
Which I realize now is actually different than the typical program not found error:
C:\Users\Mark>aergearg
'aergearg' is not recognized as an internal or external command,
operable program or batch file.

I replaced the value of NVM_SYMLINK with the direct path instead: C:\Users\Mark\AppData\Roaming\nvm\v0.12.7 and now everything works fine. I'm not sure what will happen if I try nvm using something else.
What does nvm use do exactly? Does it update the C:/Program Files/nodejs symlink to point to the new folder, or does it edit the NVM_SYMLINK variable?

As you guessed, nvm use updates the symlink. The NVM_SYMLINK is used in the PATH so the OS has a consistent directory to search for the node executable. You may want to update the settings.txt file as well so the tool knows where to map the symlink, but you shouldn't have a problem with nvm use for other versions.
Overall, I'm glad you got it working, but I'm truly sorry for the hassle. You managed to hit all the hot spots at once :) Most of this surrounds the installation process, which has been a bit convoluted since the release of this project... io.js didn't exist, then it did, then their was the announcement it was merging back with node... meanwhile I didn't have time to keep up (see updates on gitter). There has been alot of feedback about installation of NVM for Windows in general. There is a completely new process for this in the next version, using a commercial tool graciously donated. It's not ready for release yet, so if you have any feedback, feel free to leave a note on gitter or another issue I can mark as an enhancement request. I'll do my best to support as many use cases as possible. For now, I'm going to close this issue... I can reopen it if there is something unique to keep an eye on.

I ran into the same issue. Changing the NVM_SYMLINK path like what was mentioned above worked for me. One weird thing is I feel like these 2 numbers should equal each other:

I just had the same problem. I solved it by deleting the empty folder "c:\program files\nodejs". I then created the folder again. I guess there was a rights issue because the folder got created by an older node installer.
After that I started a console (cmd) with admin rights (right click -> run as administrator). Now I just used the "nvm use x.x.x xx" and everything is fine now . If the symbolic link has been created succssfully, it will be shown as a link symbol in "c:\program files".
Hope this helps.

@herrfrank  it's userful, just delete the folder, and create again.

Initially "C:\Program Files\nodejs" was not created, I just used 'nvm use VX.X.X' after installing "nvm install latest" then "C:\Program Files\nodejs" was created . it is working fine now.

I faced the same issue when nvm-windows installed from windows setup.
if we select path in setup that is obvious exists but actually for symlink it shouldn't so need to delete nodejs folder and nvm use again. I have made troubleshooting steps.
http://techbrij.com/nvm-windows-node-npm-not-recognized
Hope it helps.


feat(theme): HTML mode integration

closes gravitee-io/issues#8124
:memo: Test scenarios
For this PR, only forms have been managed, not the email templates.

Choose a form you want to update from the Theme section of your security domain
Switch to HTML mode, do your change and press PUBLISH
Refresh the page, your HTML content must have been saved

:computer: Add screenshots for UI

There is a bug, here is the steps to reroduce:

Select Login from the drop down and publish a simple html such as
login

Select WebAuthn Login from dropdown and publish another simple html such as
webauthn

Now we have 2 published page as expected.
Select the desktop window icon in between page and HTML to view the rendered output.
Now change in between the two saved pages.
The output window won't refresh with the correct html content. It Login is already selected then selecting WebAuthn Login won't change the content as expected.

@tcompiegne , found one more issue. Here is the steps to reproduce:

Create a new domain [not an existing one].
Go to Theme. Provide  a valid value for log width and select Publish. All good until this point and theme will be published as expected.
Now change the logo width with a valid value and try to Publish again.
ThemeAlreadyExistsException is thrown instead of publishing the theme.

The reason is this.theme.id remains undefined after first create call as  this.theme is initialised in ngOnInit method. As a result in the subsequent calls this.themeService.create is executed instead of update method.


Welcome page Connect to: Name the command to run in remoteExtensionTips

Testing #177702

new profile with not remote extensions installed
click on Connect to
command show up, click on one

This will run the first command found in the `'statusBar/remoteIndicator' menu
I suggest adding a new property connectCommand to the remoteExtensionTips. That is less fragile and allows the remote extension to use a tailored command

Moving this to next milestone as part of issue tracking and cleaning for the current milestone as we are close to the release.


fix(deps): Update package versions

fix(deps): update dependency googleapis to v140.0.1
fix(deps): update opentelemetry-js monorepo
chore(deps): update dependency mocha to v10.5.2
chore(deps): update dependency typescript to v5.5.2
chore(deps): update terraform google to ~> 5.35.0
chore(deps): update terraform kubernetes to ~> 2.31.0
fix(deps): update dependency @google-cloud/firestore to v7.9.0
fix(deps): update dependency @google-cloud/pubsub to v4.5.0
fix(deps): update dependency @google-cloud/spanner to v7.9.1

This is a single PR that groups all the update PRs from renovate to have one single test run.


Turtle overhaul for alpha

This is a placeholder for Kate's in-progress revisitation of the turtle visualizer, in #402.
I am pretty sure Kate is aware of the following, but this should address
#76, #223, #364 (maybe duplicate of #76), #412
And the overhaul could consider also addressing #381.

The overhaul is merged and all of the above-mentioned issues are resolved, so closing.


Make withRouter() update inside static containers

Fixes #3439.

I think on net this is a bit messier than the earlier implementation that just updated the router object in-place.

Yea. On the other hand, this keeps the desired property of Context* being generic. If they aren’t updating context to new objects, I’m not sure how much useful they would be as an outside module.

I mean, it's an unfortunate caveat to have, but I'm guessing it's probably slightly more performant to not recreate the object every time. Having to keep the same object identity is a caveat that I can live with.

it's probably slightly more performant to not recreate the object every time

I think this would only be relevant if navigation was something that’s performed hundreds of time per second. I wouldn’t expect this to have any measurable performance in this particular scenario.
Happy to change it though. You sure?

What I like about this is while it’s a net messier, the messy bits are all in the “polyfill”. As opposed to spreading through the legit code.

I’m fairly content with what I posted before but I added 7b3abdb39e5a3c74eb114f1bc52a35daecf2cfce as a minimally invasive alternative approach. It adds object-assign as a dependency but React already uses it anyway.

cc @jquense who's done some stuff to work around this issue. At this point I'm really torn.

Ultimately we can go either way and switch if we find problems later. I’d probably use the last commit because it doesn’t touch this.context so if React (unlikely) adds a warning on setting to it, this wouldn’t affect us.

[Current coverage][cc-pull] is 95.70%

Merging [#3443][cc-pull] into [next][cc-base-branch] will increase coverage by +0.08%

@@               next      #3443   diff @@
==========================================
  Files            28         28
  Lines           835        837     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            798        801     +3
+ Misses           37         36     -1
  Partials          0          0

Powered by Codecov. Last updated by [72ed6e7...6aeba93][cc-compare]
[cc-base-branch]: https://codecov.io/gh/reactjs/react-router/branch/next?src=pr
[cc-compare]: https://codecov.io/gh/reactjs/react-router/compare/72ed6e7e817db25f4f9099e3e6f9981735e8878b...6aeba9381c9e767c4e6fc6ad0e51e4797755c0f0
[cc-pull]: https://codecov.io/gh/reactjs/react-router/pull/3443?src=pr

Yeah, I'm fine with imposing a bit of extra work on the users of this polyfill I think. Probably I'd be happier mutating my context objects in place rather than worrying that setting this.context will break. It's all a workaround anyway.

In case we want to change the mind later, the test case I added should catch any mistakes.

I'm happy with this for now.

@taion One last problem is this still doesn’t update containers with shallow equality check. We can either revert the last commit or add a bogus prop just to force an update.

Alternatively, we can inject params and location as props to useRouter(), in addition to router itself. Then it wouldn’t matter.

You mean in case someone does withRouter(connect(whatever)(MyComponent))?

Yeah.

I like the idea of withRouter injecting the additional props – makes withRouter-ed components work identically to route components, which seems like a nice symmetry.


Swap calendar names for container IDs in kopia

Issue
In order to protect against name collisions from shared or like-named calendars, we need to stop persisting calendars by their display names and use their underlying m365 ID instead.
Acceptance Criteria

[x] Swap out calendar display names for IDs.
[x] Add a path string with display names into the item details. https://github.com/alcionai/corso/issues/2412
[x] When running selectors.Reduce, replace the path with the display name path string (when available).
[x] ~Update the details display output to reference display names instead of IDs.~
[x] ~Check selector usage in GC to ensure path comparisons follow the new format.~
[x] Remove the "other calendar" middle folder.

Remaining changes to close: manual release testing and merger to main branch.


Calendar download icon is messed up

Originally created by Tom Smyth at 2018-01-19 11:35, migrated from redmine-#7862

Version set to 5.1

By Tom Smyth at 2018-02-07 03:35

Status changed from New to Closed

By Tom Smyth at 2018-02-09 04:03


Fix issues related to docs after changes from fkm_nonlinear

There is an error in the docs build ̀ AttributeError: module 'nbconvert.preprocessors.execute' has no attribute 'CellExecutionError'` which I don't know how to handle

There is an error in the docs build ̀ AttributeError: module 'nbconvert.preprocessors.execute' has no attribute 'CellExecutionError'` which I don't know how to handle

Actually there was an error in a notebook cell which was the cause. This seems like an error in exception handling of the nbconvert package. Now that the notebooks are all working, the docs build succeeds.


Test DRS on CCAM data.

Inputs:
/datastore/raf018/CCAM/WINE/ACCESS1-0/50km/cordex
Outputs:
/datastore/raf018/CCAM/WINE/ACCESS1-0/50km/DRS/CORDEX/

Hi @bschroeter , naïve question, which branch will I need to be on to pick up this fix? I was on i15 from the previous one but I think that's gone now, just not sure what I should be using? Looking at the commit history I think dev?
Thanks!

Hi @hot007 , yes you are correct. Dev will have the most recent changes in all merged branches. It is still considered volatile until we get to a release candidate (hence why I haven't incremented any version numbers yet) so use at own risk!
If you discover any bugs please don't hesitate to raise issues.

Goodness me I'm having a journey here!
I had to re-clone the repo, git pull was messing with what pip install was looking at (or rather, git was up to date but the axiom executable was still the original one I'd built for ??reasons.
Anyway, re-cloned the directory (as an aside, I can open a new issue for this if you want, but I think the doco is out of date, I rebuilt docs in the new repo but it still points to CRE github). I must still be doing something daft because while it's doing better, I'm now getting schema errors. I have installed axiom-schemas alongside axiom (ie repos/axiom; repos/axiom-schemas), is that wrong, is axiom-schemas meant to be inside axiom??
Here's my error. And so sorry for all the questions, I thought I was going to be able to do this trivially months ago but I keep hitting silly problems :(
(env) [$USER@host axiom]$ axiom drs --input_files /g/data/e53/users/$USER/3hr_ccam_delwp_noresm1-m_rcp45_50km.202002.nc --output_directory /g/data/e53/users/$USER/ -s 2020 -e 2020 -f 1M -p ACS -m NorESM1-M -d AUS-50 -v tas
2022-03-24 17:32:08,343 - axiom.drs - DEBUG - No input resolution supplied, auto-detecting
2022-03-24 17:32:08,344 - axiom.drs - DEBUG - Input resolution detected as 50.0 km
2022-03-24 17:32:08,344 - axiom.drs - DEBUG - Loading project (ACS) and model (NorESM1-M) metadata.
2022-03-24 17:32:08,344 - axiom.drs - DEBUG - User has supplied variables (['tas']).
2022-03-24 17:32:08,344 - axiom.drs - DEBUG - Creating /g/data/e53/users/$USER/
2022-03-24 17:32:08,348 - axiom.drs - DEBUG - Constructing a list of years to process
2022-03-24 17:32:08,348 - axiom.drs - DEBUG - Assembling interpolation context.
2022-03-24 17:32:08,349 - axiom.drs - DEBUG - Loading files into distributed memory, this may take some time.
2022-03-24 17:32:10,513 - axiom.drs - DEBUG - Standardising units
2022-03-24 17:32:10,594 - axiom.drs - DEBUG - Extracting variables ['tas']
2022-03-24 17:32:10,599 - axiom.drs - DEBUG - Subsetting levels
2022-03-24 17:32:10,600 - axiom.drs - DEBUG - Sorting data
2022-03-24 17:32:10,606 - axiom.drs - DEBUG - Applying metadata schema
Traceback (most recent call last):
  File "/home/599/$USER/.local/bin/axiom", line 230, in <module>
    parse_and_dispatch(parser)
  File "/home/599/$USER/.local/bin/axiom", line 25, in parse_and_dispatch
    func(**args)
  File "/home/599/$USER/.local/bin/axiom", line 158, in _drs
    ad.main(**kwargs)
  File "/home/599/$USER/.local/lib/python3.9/site-packages/axiom/drs/__init__.py", line 159, in main
    dss = au.apply_schema(dss, schema)
  File "/home/599/$USER/.local/lib/python3.9/site-packages/axiom/utilities.py", line 331, in apply_schema
    for key, _schema in schema['_global'].items():
TypeError: string indices must be integers

Any idea what I'm doing wrong?
Thanks!

Also how do I reopen this issue? Or do I need to make a new one referring to it?

It occurs to me probably I needed to rerun pip install . after doing git pull, is that right? That would explain the first problem at least...


Adds --skip-verify to dataconverter

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 46.341%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
3888

Relevant Lines:
8390

💛  - Coveralls


Properties that are a reference are not expandable

When a property of a model is a reference to another model, the property is not expandable.

Example spec:
openapi: "3.0.0"
info:
  version: 1.0.0
  title: Example API
  description: Hello
paths:
  /examples/{id}:
    parameters:
      - in: path
        name: id
        schema:
          type: string
        description: The ID of the customer.
        required: true
    put:
      tags: [Examples]
      summary: Get an Example
      requestBody:
        content:
            application/json:
              schema:
                $ref: '#/components/schemas/Example'
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/Example'
            application/xml:
              schema:
                $ref: '#/components/schemas/Example'

components:
  schemas:
    Metadata:
      description: About the entity.
      readOnly: true
      properties:
        createdBy:
          type: string
          description: Who made it.
        createdAt:
          type: string
          description: When they made it.
          format: date-time
    Example:
      description: An example model.
      required: [test, meta]
      properties:
        test:
          type: string
          description: A test field.
        meta:
          $ref: '#/components/schemas/Metadata'
tags:
- name: Examples
  description: |
    A small example demonstrating the issue.

See the result here: https://rebilly.github.io/ReDoc/?url=https://gist.githubusercontent.com/niknetniko/ebdd032f6e4e132c0864b32181bfaef1/raw/61fe049675904795b9b45adaacffa7acbaa3800a/sample.yaml#tag/Examples
As you can, the meta field of the Example model is not expandable in the documentation.

I think I have found where the bug is:
https://github.com/Rebilly/ReDoc/blob/597688e7208159864c0d5964237df6fb3445e2b7/src/services/models/Schema.ts#L96
At this line, the schema's type is used or type detection happens. However, for references, the type of the schema is undefined. This is no problem; the type detections correctly determines that it is in fact an object. The problem lies later: multiple functions rely on the schema's type, which is undefined.
I believe this could be fixed by:

Finding out why (if this is a problem, not sure) a property with referenced model has an undefined type
Set the type of the schema to the one returned by detectType (I'm not sure if this breaks things)

Thanks for the quick fix and the workaround 👍


wasm: accept cluster name in grpcCall()

Signed-off-by: Shikugawa<EMAIL_ADDRESS>
For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Commit Message: Close #15170. Add capability to accept yaml style GrpcService message for grprCall(). It is required to support gRPC related operation on SDK, such as Rust which doesn't have gRPC support. It is required to ease use of grpcCall()
Additional Description:
Risk Level: Low
Testing: Unit
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

cc @mathetake

@PiotrSikora I think that this is not so bad (even if this is just a workaround) until ABI v1.0.0 released. (It can be also tech debt since we should introduce runtime guard flag of this bahavior)
Do you have any thoughts?

@PiotrSikora How is going?

@Shikugawa friendly ping to get this in before next release. Also, it looks like it needs a merge with main branch.


Point MuTalk repo to "pharo-contributions/mutalk"

Fix #110

Fixes the issue, you can try with
Metacello new
  baseline: 'SmallAmp';
  repository: 'github://mabdi/small-amp/src';
  load.


Some changes to the WinPCAP network interface

Description
Replaced ipLOCAL_MAC_ADDRESS with pxEndPoint->xMACAddress
Removed earlier debug code around FreeRTOS_MatchingEndpoint()
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Merged as part of PR #738