[🐛 Bug]: next-on-pages build does not propagate NODE_ENV

Cli version
0.7.0
Next.js related information
 Operating System:       Platform: darwin       Arch: arm64       Version: Darwin Kernel Version 21.6.0: Mon Dec 19 20:46:01 PST 2022; root:xnu-8020.240.18~2/RELEASE_ARM64_T8101     Binaries:       Node: 19.6.0       npm: 9.4.0       Yarn: 1.22.19       pnpm: N/A     Relevant packages:       next: 13.3.0       eslint-config-next: 13.3.0       react: 18.2.0       react-dom: 18.2.0

Vercel version
28.18.3
Description
My project uses .env.[development|test|production] files for managing our non-secret environment variables (including our backend base URL, Auth0 url and client id, etc), as recommended in the Next.js docs
These files work fine and are compiled into the app when I run the default scripts (npm run dev, npm run build && npm run start), and I can even specify the environment I want with NODE_ENV=test npm run dev.  But when running NODE_ENV=test npx @cloudflare/next-on-pages it compiles in variables from the production environment file.
It looks like I can force a correct build by adding an environment to the build command in package.json (eg. "build": "NODE_ENV=test next build"), but that's not ideal.
Note: for a number of reasons we are using our own CI environment to build the project and then are publishing to pages with wrangler, I haven't tried using the default pages pull-from-git approach.
Reproduction
Minimal example after npm create-next-app@latest
https://github.com/dodrian/pages-next-app
Additional Information
No response
Would you like to help?

[ ] Would you like to help fixing this bug?

Thank you so very much for the issue and especially the reproduction that's going to be super helpful 🙂👍

dang, this is a bit annoying 😓
We build the application using the vercel CLI, unfortunately they seem to override/hardcode the NODE_ENV env var to 'production' before building the application 😓
https://github.com/vercel/vercel/blob/f774d5f7a2c72e4b6c8fade5d28d939ab9a63135/packages/next/src/index.ts#L429
Feels like we don't have much choice but to update the build script in the package.json as you mentioned (which definitely is not ideal!) 😓

I've opened a discussion there: https://github.com/orgs/vercel/discussions/2012
Let's see if that yields to anything 😓

Ditto -- running into this right now

my workaround package.json:
{
    "scripts": {
        "pages-dev": "cp .env.development .env.production && next-on-pages",
        "pages-prod": "cp .env.production.real .env.production && next-on-pages",
    }
}

my workaround package.json:
{
    "scripts": {
        "pages-dev": "cp .env.development .env.production && next-on-pages",
        "pages-prod": "cp .env.production.real .env.production && next-on-pages",
    }
}

.gitignore:
.env.production

This works, but it is a mess because we have to manually update the env vars in wrangler.toml to match the current env file.


Enhance README and newman coverage

Signed-off-by: Nana-EC<EMAIL_ADDRESS>Description:

Add useful badges to README

Build actions
Latest Version
RPC API Doc link
Discord link
Made with typescript
License

Enhance newman eth_getBlockByNumber test with lastest tag
Add eth_getBlockTransactionCountByNumber test to newman tests

Related issue(s):
Fixes #421
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

LGTM


Typo in bin/pyenv-sh-activate ?

I get a prompt every time when i activate a virtualenv says 'pyenv-virtualenv: prompt changing not work for fish' . Is it intended to be 'pyenv-virtualenv: prompt changing not working for fish.' ?
A side note: the prompt changing from oh-my-fish actually works.
thanks.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/32253524-typo-in-bin-pyenv-sh-activate?utm_campaign=plugin&utm_content=tracker%2F335155&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F335155&utm_medium=issues&utm_source=github).

haha. I am really bad at English writing.... PRs welcome~

PR submitted. Thanks for your work.


RDK-32275 : CEC Short Audio Descriptor

Reason for change: Add support for sending and receiving  and
Test Procedure: none
Risks: low
Signed-off-by: Bijas Babu<EMAIL_ADDRESS>

Closing this outdated PR. Please submit this to the latest sprint branch if needed.


fix minor typo

fixed some typo at "mkdir build" line so the user can copy and paste the command to their terminal

The change to README.md LGTM, but it's up to @kilograham to decide if the changes to convert.cpp are appropriate.

The change to README.md LGTM, but it's up to @kilograham to decide if the changes to convert.cpp are appropriate.

Some time ago, I was able to compile this program with this modification on Windows using MSYS2, then I recorded some of my screen capture footage for use in testing my converter build under Windows.
https://user-images.githubusercontent.com/42478428/144678089-f35b27a2-6622-43a4-9a84-adc4a60ed4df.mp4


storage: support to get old value through DeltaScanner

Signed-off-by: 5kbpers<EMAIL_ADDRESS>
What problem does this PR solve?
Problem Summary:
Old Value is the previously written value before a delete/update operation.
To support the new collation format and compatible with TiDB Binlog, TiCDC requires TiKV to output the old value for it.
What is changed and how it works?
Separate from https://github.com/tikv/tikv/pull/8071
What's Changed:

Add a flag to DeltaScanner
Look-ahead for seeking old value in DeltaScanner

Related changes

Should be cherry-picked to release 4.0 after all related PRs got merged.

Check List
Tests

Unit test
Integration test

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Release note

N/A

/test

LGTM cancel

LGTM

LGTM

LGTM cancel

lgtm cancel

lgtm cancel

LGTM

LGTM cancel

lgtm cancel

lgtm cancel

lgtm cancel

LGTM

LGTM

lgtm cancel

LGTM

lgtm cancel

LGTM

lgtm

lgtm

LGTM

LGTM

lgtm cancel

lgtm cancel

LGTM

lgtm

lgtm cancel

LGTM

/merge

/merge


fix: package issues

coverage: 100.0%. remained the same
when pulling 33aec56e86c5f73b1e4ca02997977d1b92ea3027 on fix
into 3ff0d20d32b1838ef41ed64de5ac7fce7f4df28e on main.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


feat(meta): a more rational mechanism for managing relation/function/connection/user dependencies in meta

Is your feature request related to a problem? Please describe.
Currently it's really error-prone when introduce some new objects in rw and implementing some management interfaces in meta side. And there have been a few bugs in customer scenarios already and others like #8979 . A more rational mechanism is needed.
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

Will be managed by object_dependency table in sql-backend meta store.


Vulnerability roundup 107: gpac-1.0.1: 16 advisories [7.8]

search, files

[ ] CVE-2021-32136 CVSSv3=7.8 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-32271 CVSSv3=7.8 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-33362 CVSSv3=7.8 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-32132 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-32134 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-32135 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-32137 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-32138 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-32139 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-32269 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-32270 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-33361 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-33363 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-33364 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-33365 CVSSv3=5.5 (nixos-21.05, nixos-unstable)
[ ] CVE-2021-33366 CVSSv3=5.5 (nixos-21.05, nixos-unstable)

CVE details
CVE-2021-32136
Heap buffer overflow in the print_udta function in MP4Box in GPAC 1.0.1 allows attackers to cause a denial of service or execute arbitrary code via a crafted file.
CVE-2021-32271
An issue was discovered in gpac through 20200801. A stack-buffer-overflow exists in the function DumpRawUIConfig located in odf_dump.c. It allows an attacker to cause code Execution.
CVE-2021-33362
Stack buffer overflow in the hevc_parse_vps_extension function in MP4Box in GPAC 1.0.1 allows attackers to cause a denial of service or execute arbitrary code via a crafted file.
CVE-2021-32132
The abst_box_size function in GPAC 1.0.1 allows attackers to cause a denial of service (NULL pointer dereference) via a crafted file in the MP4Box command.
CVE-2021-32134
The gf_odf_desc_copy function in GPAC 1.0.1 allows attackers to cause a denial of service (NULL pointer dereference) via a crafted file in the MP4Box command.
CVE-2021-32135
The trak_box_size function in GPAC 1.0.1 allows attackers to cause a denial of service (NULL pointer dereference) via a crafted file in the MP4Box command.
CVE-2021-32137
Heap buffer overflow in the URL_GetProtocolType function in MP4Box in GPAC 1.0.1 allows attackers to cause a denial of service or execute arbitrary code via a crafted file.
CVE-2021-32138
The DumpTrackInfo function in GPAC 1.0.1 allows attackers to cause a denial of service (NULL pointer dereference) via a crafted file in the MP4Box command.
CVE-2021-32139
The gf_isom_vp_config_get function in GPAC 1.0.1 allows attackers to cause a denial of service (NULL pointer dereference) via a crafted file in the MP4Box command.
CVE-2021-32269
An issue was discovered in gpac through 20200801. A NULL pointer dereference exists in the function ilst_item_box_dump located in box_dump.c. It allows an attacker to cause Denial of Service.
CVE-2021-32270
An issue was discovered in gpac through 20200801. A NULL pointer dereference exists in the function vwid_box_del located in box_code_base.c. It allows an attacker to cause Denial of Service.
CVE-2021-33361
Memory leak in the afra_box_read function in MP4Box in GPAC 1.0.1 allows attackers to read memory via a crafted file.
CVE-2021-33363
Memory leak in the infe_box_read function in MP4Box in GPAC 1.0.1 allows attackers to read memory via a crafted file.
CVE-2021-33364
Memory leak in the def_parent_box_new function in MP4Box in GPAC 1.0.1 allows attackers to read memory via a crafted file.
CVE-2021-33365
Memory leak in the gf_isom_get_root_od function in MP4Box in GPAC 1.0.1 allows attackers to read memory via a crafted file.
CVE-2021-33366
Memory leak in the gf_isom_oinf_read_entry function in MP4Box in GPAC 1.0.1 allows attackers to read memory via a crafted file.

Scanned versions: nixos-21.05: 3b1789322fc; nixos-unstable: 34ad3ffe08a.
Cc @bluescreen303
Cc @mgdelacroix

All of these have been fixed in 2.0.0 (22.05, unstable)


Interpolation search added

** Added the Interpolation_Search Filename as**
** Changes Proposed
The Interpolation Search Algorithm:
Interpolation search is an algorithm for searching for a key in an array that has been ordered by numerical values assigned to the keys. Interpolation Search added in searching directory
** I am assuring that the file is not redundant and making the Pull Request **

[x] Yes

Read {% include Contributing.md %}
Issue:
#2

@codewithdev Can you please review my PR?


MFA Authentication : error (?) in exemple and posible amelioration

Hello,
Two remarks for this article :

In all the exemples for the MFA authentication, in the section of code concerning Skype for Business Online you use the argument UserName in the CmdLet New-CsOnlineSession, This argument is not existing for the CmdLet out of MicrosoftTeams module.
In the exemple for "basic" authentication, you show a way to connect to all services by saving the credential in a variable. Is it also possible in the MFA authentication. If yes, how ?

Thanks a lot in advance

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

ID: 6adeef66-5e54-ba2f-37ac-c1898ef942c6
Version Independent ID: f8a6eda6-b300-dc76-4392-9aeb6c532822
Content: Connect to all Microsoft 365 services in a single PowerShell window - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/connect-to-all-microsoft-365-services-in-a-single-windows-powershell-window.md
Service: o365-administration
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

@Lerritier Thank you for your feedback.
1-  We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/3905
2-  It is currently not possible to save the credentials if the account is enabled MFA. However, you can use an App password instead. App passwords allow replacing your regular account password and will enable an app to bypass multi-factor authentication.  -https://docs.microsoft.com/en-us/azure/active-directory/user-help/multi-factor-authentication-end-user-app-passwords
Hope this helps!
Thanks
Sri

@Lerritier Thank you for your feedback.
1-  We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/3905
2-  It is currently not possible to save the credentials if the account is enabled MFA. However, you can use an App password instead. App passwords allow replacing your regular account password and will enable an app to bypass multi-factor authentication.  -https://docs.microsoft.com/en-us/azure/active-directory/user-help/multi-factor-authentication-end-user-app-passwords
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @Lerritier Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@msbemba Thank you very much for the contribution and updating the documentation with PR. @Lerritier Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[Fix #5973] Add Style/IpAddresses cop

Added a cop for #5973. It registers an offense if a string is found that is a valid IP address. It allows for strings to be whitelisted if desired (it only includes :: by default, which is found a whole bunch of times in the rubocop code). It does not match strings that contain an IP address as a substring.
I tried to implement this in a way that made sense to me, but please let me know if there are any suggestions, particularly re naming or namespace I chose.
I also have this set to disabled by default, but am happy to change it if desired.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

Thank you @dvandersluis !!! 🥇


Use alpine:latest instead of edge

#1219  👈
feature/proof-of-stake

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lazynina and the rest of your teammates on  Graphite


nlp = spacy.load('en'), error appeared

for nlp = spacy.load('en') when I run on jupyter i faced following error.

OSError                                   Traceback (most recent call last)
 in ()
1 #python -m spacy download en as Administrator
----> 2 nlp = spacy.load('en')
C:\Users\Public\Anaconda3\lib\site-packages\spacy_init_.py in load(name, **overrides)
13     if depr_path not in (True, False, None):
14         deprecation_warning(Warnings.W001.format(path=depr_path))
---> 15     return util.load_model(name, **overrides)
16
17
C:\Users\Public\Anaconda3\lib\site-packages\spacy\util.py in load_model(name, **overrides)
117     elif hasattr(name, 'exists'):  # Path or Path-like to model data
118         return load_model_from_path(name, **overrides)
--> 119     raise IOError(Errors.E050.format(name=name))
120
121
OSError: [E050] Can't find model 'en'. It doesn't seem to be a shortcut link, a Python package or a valid path to a data directory.
To which you suggested to apply  "python -m spacy download en as Administrator" . I was given following error
(base) C:\Users\Public>python -m spacy download en as Administrator
Collecting as
Could not find a version that satisfies the requirement as (from versions: )
No matching distribution found for as
(base) C:\Users\Public>python
Python 3.6.5 |Anaconda, Inc.| (default, Mar 29 2018, 13:32:41) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

-m spacy download en as Administrator
File "", line 1
-m spacy download en as Administrator
^
SyntaxError: invalid syntax
exit()

Do review and advice

Please start an Anaconda Prompt as administrator and run this:
python -m spacy download en
This is the commented line of code just before loading the model.

Thanks for trying this @mithundeshmukh8 !
Closing this for now unless you have more questions

It worked successfully. Thanks. My mistake was I was putting as administrator in command.


Close ActionSheet on click off on iOS

I'm trying to have it so that when I click off an ActionSheet, it will close.
I was able to get it working on Android only with cancelButtonIndex: -1, but on iOS it does not work.
Is there a way to do it? Can't seem to find it in the documentation.
Native Base: 2.13.8

Hi @Aryk , Please update to latest Native-Base v2.13.14 and let us know if the issue still persists.

@Aryk Did you get this to work? I had the same problem and upgrading to 2.13.14 did not help.

@skrcode  - 2.13.14 fixes it for me, but it completely screwed up my headers as already documented here (https://github.com/GeekyAnts/NativeBase/issues/3095), so I have to wait until that is fixed.

Hi @Aryk , can you provide minimal code reproduction with more description?


dependency lines and ntesting

Can we create nested data/grouping in react-scheduler?
Sorta like this:
- layer 1
    - resource 1
    - resource 2
        - sub resource 1
        -sub resource 2
    - resource3

And then be able to create dependency lines?

Plus one on this request please.

Teams. For example, I want to nest all resources under a certain heading and expand/collapse:

Developers

Person 1
Person 2
Person 3

Testers

Person 1
Person 2
Person 3

Could be similar if scheduling rooms in different buildings, etc.


Custom Java compiler selection

Basically I would like to compile my mod for Java 6 instead of 8.
Is there a way to change the used compiler?

You can try Options > Recompile > Set JAVA_HOME used for compiling


[Backport release-23.05] {ungoogled-,}chromium,chromedriver: 120.0.6099.71/109 -> 120.0.6099.109/129

Description of changes
Manual backport of #275940
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

20 packages built:

chromedriver
chromium
chromium.sandbox
fast-cli
google-chrome
netflix
playwright-test
puppeteer-cli
python310Packages.pytest-playwright
python310Packages.pytest-playwright.dist
python311Packages.pytest-playwright
python311Packages.pytest-playwright.dist
reveal-md
selendroid
selenium-server-standalone
sharedown
ungoogled-chromium
ungoogled-chromium.sandbox
vhs
wayback

# git describe --abbrev=40
23.05-4851-gd554402150285ec5567922e60d78c3c5e5c964aa


move core_intrinsics attribute to crate root

the attribute must be in the crate root, otherwise it is ignored

Thank you! Any help with completing WASM support is much appreciated.


Additional fix for using private Google Cloud Storage Buckets

If you have private, non-public Cloud Storage buckets and use enableSigning = true, "predefinedAcl" => "private" will be used instead of "publicRead".

Looking at the history of that setting again, I think we should have a separate setting for the ACL visibility as our original use case was a two bucket setup with storage and target, and we still used public ACLs for that. I think "objectVisibility" with a default of "PublicRead" could be fine, then you can overwrite that to Private. What do you think?

Yes your solution would be better. I actually started it that way when I was implementing it, but then I changed it to the "simpler" adaptation.

@kitsunet I added the targetOption objectVisibility. By default it is publicRead


Setup and Deployment of Entity Framework Project

Hi EF Team ,
Thanks for making a EF an Open source project , so now we can contribute to it and make it much better.
I have a small clarification I won't say it as an issue.
How to move a project which used EF as a ORM into a production or to a client machine.
Let me explain you the scenario.
I have created a project in WPF which uses entity framework as an ORM, Now I need to pass the setup i.e exe/msi to the client machine. How to achieve it ?
Currently I am creating a setup of the project using install shield of visual studio and then changing the connection string and all is done manually , Is there a process to avoid the same.
Cheers
Sathish

@kumarsathishD it is unclear if this is question is about EF6 (for future reference, maintained at https://github.com/aspnet/entityframework6) or EF Core (maintained here). I know the repository name of the latter is confusing. That is why we are planning to rename this one very soon.
Regardless, you would need to include all the assemblies from the product alongside the application in the installer package.
For something like the connection string, I have seen several approaches:

Have it hardcoded (e.g. if the scope of the application is very small and this is acceptable)
Having in the configuration file or a separate file and modifying manually
Include a step in the setup process that creates the right connection string
Include a configuration screen/page in the application that allows configuring and persisting the connection string, possibly invoked automatically on first usage.

Regarding how to do it as part of the setup process, or even getting other, possibly better suggestions on how to do this, I would suggest you ask this question somewhere else, e.g. in https://stackoverflow.com, where you have more chances of finding experts on that area.


[AMBARI-24410] - Remove conf-select Tool From Ambari Framework

What changes were proposed in this pull request?
Mpack do not provide a replacement for conf-select, a utility which was previously used to provide parallel configurations so that 2 components (such as NameNode and DataNode) could use different configurations while in an upgrade. It ensured that breaking configuration changes stay isolated in their respective directories:
/usr/hdp/<IP_ADDRESS>/zookeeper/conf -> /etc/zookeeper/<IP_ADDRESS>/0
/usr/hdp/<IP_ADDRESS>/zookeeper/conf -> /etc/zookeeper/<IP_ADDRESS>/0
/usr/hdp/current/zookeeper-server -> /usr/hdp/<IP_ADDRESS>/zookeeper

When hdp-select was used to change the current symlink, the conf directories would “automatically” switch over. This allowed a complete separation of configurations and the ability to have parallel configurations on disk.
If Ambari is managing your configurations, then we know what to write and when to write it
Even in cases where breaking configuration changes were made, since Ambari kept both old and new configurations in our database, a downgrade would always write out the correct configurations for the version being downgraded to
Let’s fast-forward to Ambari 3.0 and mpacks … The current structure does not allow for multiple configurations for a given service inside of a service group:
instances
└── hdpcore
    ├── default -> /usr/hwx/instances/hdpcore/HDPCORE
    └── HDPCORE
        └── default
            ├── zookeeper
            │   └── zookeeper_server
            │       └── ZOOKEEPER
            │           ├── conf
            │           │   ├── configuration.xsl
            │           │   ├── log4j.properties
            │           │   ├── zoo.cfg
            │           │   ├── zookeeper-env.sh
            │           │   └── zoo_sample.cfg

Instead of symlinks scoped by a version, each service instance has a regular conf directory. A few points here:

Since configurations are now at the component instance level, DataNode and NameNode won't share configs during an upgrade.
For Ambari managed configurations, this should be fine since we keep track of old and new versions. So, even on a downgrade, we’d know to write the correct values in here
If Ambari is not managing a configuration file, say foo-site.xml, then:
-- We don’t have to worry about copying it from one versioned directory to another. This seeding process is necessary in Ambari 2.x, but wouldn’t be here
-- If there was a change to the structure of a file which Ambari does not manage, then we have a problem on downgrade as Ambari wouldn’t know to replace anything. I suppose it’s the same issue on upgrade too since Ambari wouldn’t know to change the file either.

How was this patch tested?
Simple mpack deploy to make sure nothing unrelated broke.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3477/
Test FAILed.
Test FAILured.


Commands are interpreted when modals are active

If you start VaultEd and enter a filename like 'test2' the 2 will toggle revealing the terrain.  Similarly a '1' will turn lights on/off.  Keypresses should only be interpreted by the active modal when one is active.

fixed by making these Function keys -- only keys that can't ever be used by other things should get intercepted for special processing in this way


Nothing seems to happen in this repo?

As far as I can see, you only define an interface? am I missing something obvious?

@taddison92 I believe this library is only for parsing an RRULE string into a struct for easier interaction. I'm sure you'll need to combine this with something else to actually run tasks at those times.

Where is the interface implementation?


[BUG] cmake needs to be installed

Description

If cmake is not installed, conda env create -f reco_base.yaml will fail.
In which platform does it happen?

How do we replicate the issue?

Expected behavior (i.e. solution)

Other Comments

CMake is required by XLearn as noted here: https://github.com/microsoft/recommenders/blob/master/SETUP.md#dependencies-setup
This will make it a bit more difficult for windows installation, but should still be possible.
While you can install cmake through conda it doesn't seem to include or always pick up the c / c++ compilers needed, so installing cmake at the os level is probably the simplest.
I'm checking with @yueguoguo in #1121 if using apt-get install -y build-essential cmake is sufficient or if we need to pull from source to build cmake. Depending on that we can update the instructions to make it a bit easier for linux.


Missing haar landmark file

xvfb-run -s "-screen 0 140
0x900x24" python ex_dlib_pnp_head_pose_estimation_video.py ../../data/1.mp4 ./output.avi
Estimated camera matrix:
[[0. 0. 0.]
 [0. 0. 0.]
 [0. 0. 1.]]

Traceback (most recent call last):
  File "ex_dlib_pnp_head_pose_estimation_video.py", line 213, in <module>
    main()
  File "ex_dlib_pnp_head_pose_estimation_video.py", line 139, in main
    my_detector = faceLandmarkDetection('./etc/shape_predictor_68_face_landmarks.dat')
  File "/home/ben/anaconda3/envs/deepgaze/lib/python2.7/site-packages/deepgaze/face_landmark_detection.py", line
 35, in __init__
    raise ValueError('haarCascade: the files specified do not exist.')
ValueError: haarCascade: the files specified do not exist.

Looks like the correct paths are:
my_cascade = HaarFaceDetector("../../etc/xml/haarcascade_frontalface_alt.xml", "../../etc/xml/haarcascade_profileface.xml")
and .dat file in the next line from
wget dlib.net/files/shape_predictor_68_face_landmarks.dat.bz2

Thank you for pointing that out. I added a warning if the file dat is missing.
The path of the XML files has also been fixed.


Do not use the deprecated confirmation annotation

confirmation.garden.sapcloud.io is deprecated with https://github.com/gardener/gardener/pull/1833 and will be removed in a future release of gardener/gardener.

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

/ping @ib-steffen @iberryful @JoergBastian


Move Groups above Plants in location dropdowns

Groups are more important than individual plants and thus should be located higher up in location dropdowns.

Bumping into the TODO column now that groups are even more useful/important

This seems to have been completed and deployed at some point


[auto-sync] 2021-03-12 14:27 UTC

This PR is created via a script automation and CI.
It use the owner account to execute it, please ignore it.

Trumped by #23


sys: support /dev/random and /dev/net/tun

This commit adds  "/dev/random, /dev/urandom and /dev/net/tun" support.

Thanks for working on this!
You also need to add copyright holder name to AUTHORS file. If you contribute as an individual, then just add your name to AUTHORS as well.

@sploving you also need to sign in Google CLA at https://cla.developers.google.com/

Otherwise looks good to me.

Yes. I have signed it.

Thanks!


Can export this graphs like Bitmap?

I tried to use buildDrawringCache() method, and getDrawingCache() but not working. Thanks. It's for paste in a PDF.

Hi, I've managed to save chart as bitmap using the following piece of code. Make sure you are using copy of the drawing cache bitmap.
//chartView.setBackgroundColor(Color.WHITE);//by default chart have transparent background
chartView.setDrawingCacheEnabled(true);
chartView.buildDrawingCache();

final String filename = "chart-image.png";
final File file = new File(Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_PICTURES), filename);
FileOutputStream outputStream = null;
try {
    outputStream = new FileOutputStream(file);
    final Bitmap bitmap = Bitmap.createBitmap(chartView.getDrawingCache());//important to make copy of that bitmap.
    bitmap.compress(Bitmap.CompressFormat.PNG, 100, outputStream);
} catch (Exception e) {
    e.printStackTrace();
}finally {
    try {
        if(outputStream != null) {
            outputStream.close();
        }
    } catch (IOException e) {
        e.printStackTrace();
    }
}


Update docs site for 0.17.0

resolves #2284
Description

Use .sources for sources instead of .nodes
Fixed some build issues around changing highight.js versions
Lots of code cleanup

Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

fyi @beckjake - merging this one


[FEATURE] Link the netlify prod on learn.webxdao.xyz

Detailed description
We should link the netlify server into one of our subdomain. Let's discuss about the name!
Context
Much better for visibility and link sharing
Possible implementation

I propose learn.webxdao.xyz

Code of Conduct

[X] I agree to follow this project's Code of Conduct

sure, ritvik suggested devpath.webxdao.xyz

Closing the issue for now as the discussion is complete


[BUG] Issues with TensorDictPrimer

Without the primer, the collector does not feed any hidden state to the policy
in the RNN tutorial  it is stated that the primer is optional and it is used just to store the hidden states in the buffer.
This is not true in practice. Not adding the primer will result in the collector not feeding the hidden states to the policy during execution. Which will silently cause the rnn to loose any recurrency.
To reproduce, comment out this line
https://github.com/pytorch/rl/blob/0063741839a3e5e1a527947945494d54f91bc629/tutorials/sphinx-tutorials/dqn_with_rnn.py#L269
and print the policy input at this line
https://github.com/pytorch/rl/blob/0063741839a3e5e1a527947945494d54f91bc629/torchrl/collectors/collectors.py#L733
you will see that no hidden state is fed to the rnn during execution and no  errors or warnings are thrown
The primer overwrites any nested spec
Consider an env with nested specs
 env = VmasEnv(
        scenario="balance,
        num_envs=5,
    )

add to it a primer for a nested hidden state
    env = TransformedEnv(
        env,
        TensorDictPrimer(
            {
                "agents": CompositeSpec(
                    {
                        "h": UnboundedContinuousTensorSpec(
                            shape=(*env.shape, env.n_agents, 2, 128)
                        )
                    },
                    shape=(*env.shape, env.n_agents),
                )
            }
        ),
    )

the primer code in https://github.com/pytorch/rl/blob/0063741839a3e5e1a527947945494d54f91bc629/torchrl/envs/transforms/transforms.py#L4649 will overwirite the observation spec instead of updating it, resulting in the loss of all the spec keys that previoulsy were in the "agents" spec
The same result is obtained with
    env = TransformedEnv(
        env,
        TensorDictPrimer(
            {
               ("agents","h"): UnboundedContinuousTensorSpec(
                            shape=(*env.shape, env.n_agents, 2, 128)
                )
            }
        ),
    )

here, updating the spec instead of overwriting it should do the job
The order of the primer in the transforms seems to have an impact
In the same vmas environemnt as above, if i put the primer and then the reward sum
 env = TransformedEnv(
        env,
        Compose(
            TensorDictPrimer(
                {
                    "agents": CompositeSpec(
                        {
                            "h": UnboundedContinuousTensorSpec(
                                shape=(*env.shape, env.n_agents, 2, 128)
                            )
                        },
                        shape=(*env.shape, env.n_agents),
                    )
                }
            ),
           RewardSum(
                        in_keys=[env.reward_key],
                        out_keys=[("agents", "episode_reward")],
                    ),
        ),
    )

all works well
but the opposite
 env = TransformedEnv(
        env,
        Compose(
            RewardSum(
                in_keys=[env.reward_key],
                out_keys=[("agents", "episode_reward")],
            ),
            TensorDictPrimer(
                {
                    "agents": CompositeSpec(
                        {
                            "h": UnboundedContinuousTensorSpec(
                                shape=(*env.shape, env.n_agents, 2, 128)
                            )
                        },
                        shape=(*env.shape, env.n_agents),
                    )
                }
            ),
        ),
    )

causes
Traceback (most recent call last):
  File "/Users/Matteo/PycharmProjects/torchrl/sota-implementations/multiagent/mappo_ippo.py", line 302, in train
    collector = SyncDataCollector(
                ^^^^^^^^^^^^^^^^^^
  File "/Users/Matteo/PycharmProjects/torchrl/torchrl/collectors/collectors.py", line 644, in __init__
    self._make_shuttle()
  File "/Users/Matteo/PycharmProjects/torchrl/torchrl/collectors/collectors.py", line 661, in _make_shuttle
    self._shuttle = self.env.reset()
                    ^^^^^^^^^^^^^^^^
  File "/Users/Matteo/PycharmProjects/torchrl/torchrl/envs/common.py", line 2143, in reset
    tensordict_reset = self._reset(tensordict, **kwargs)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/Matteo/PycharmProjects/torchrl/torchrl/envs/transforms/transforms.py", line 814, in _reset
    tensordict_reset = self.transform._reset(tensordict, tensordict_reset)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/Matteo/PycharmProjects/torchrl/torchrl/envs/transforms/transforms.py", line 1129, in _reset
    tensordict_reset = t._reset(tensordict, tensordict_reset)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/Matteo/PycharmProjects/torchrl/torchrl/envs/transforms/transforms.py", line 4722, in _reset
    value = self.default_value[key]
            ~~~~~~~~~~~~~~~~~~^^^^^
KeyError: ('agents', 'episode_reward')

For the first issue, I think we should go for a solution where the Primer becomes optional and needed only if users want the hidden states in the collection buffer.
But without the primer users should still be able to use rnns in collectors with the logic that anything coming out of step_mdp is refed to the policy

Since there are multiple issues I'd suggest to open a tracker.
I'll comment on the first here: it's optional in the sense that you can make the env run without primer if no other module is involved. If a ParallelEnv or a collector is used things will indeed break.

Yeah I'll eventually spread them into separate issues.
If that is the case regarding the first, I suggest we make it extra clear that in the tutorial the Primer is not optional, as we are using a collector.
In general, do we really have no way to make the collector work without the primer? it would be nice to have it optional in collectors for users that do not want the hidden states as part of the output buffer

Adding to my previous comment, I think to solve the first issue we could add the output of the policy (looking at the next key) to the shuttle
https://github.com/pytorch/rl/blob/371181cb067a4cd0456b2016d9035140bbb2adae/torchrl/collectors/collectors.py#L733
I would really like to not use the Primer as it is a huge pain in large projects

I don't see a way of not having a primer, we need to let the env know about extra entries in the tensordict.
Is that such a "huge" pain though? We've worked hard with @albertbou92 to provide all the tooling to make this work as seamlessly as possible.

In the collector, we could automatically check if any primer is missing and add it. Or a warning. We can extract the expected primers from the actor.
I do find it that inconvenient to use the primer. I simply do:
if primers := get_primers_from_module(actor):
            env.append_transform(primers)

However, is some user does not think about adding the primer,  silently not using recurrency is definitely problematic and can cause headache.

What is the technical limitation that is preventing us from reading the hidden state from the policy output? It seems to me that since we are running the policy at collector init time, its outputs in the "next" tensordict could be captured and accounted for during the collector rollout (aka move them to the policy input at every step_mdp).
With this you would maybe loose the possibility of obtaining zeroed states after resets, but the absence of the hidden state (i.e. it being none) should be possible to handle in the rnn.
The pain I am referring to is that the Primer in the multiagent setting will require to know: the number of agents, the group names, the hidden  sizes. All this information is not immediately available. Plus having it optional would make approach from new users easier IMO and less bug prone

Sorry to ask again, can we split this issue?
I'd like to close the pieces that are solved

Yeah we solved everything apart the first one, I ll make one for that


Missing directory

Hello,
When running Docker I run into this error:

[12/12] RUN cd /monodepth/python/src/test && python3 ./test_depthmapAlign_module.py --task 1:
#16 0.278 /bin/sh: 1: cd: can't cd to /monodepth/python/src/test

I'm not well versed in Docker, but seems like it's referencing a directory that's not there. What can I do?

Hi,
You can get rid of that line on the Dockerfile. This was from an old version of the code and I forgot to update it.


Update pdf.js to version 4

PDF.js 4.0 is a major release that contains a number of API changes, features and bugfixes. The following changes are highlighted:

    Conversion to JavaScript modules for the builds
    Conversion to Fluent for localization
    Optimizations for CPU and memory usage
    Improvements and bugfixes for the annotation editor and annotation rendering
    Removal of the deprecated SVG back-end
    Removal of other deprecated functionality and options

One of the limitation of this release is that node should be >= 18.

https://github.com/lfoppiano/streamlit-pdf-viewer/security/dependabot/1

https://github.com/lfoppiano/streamlit-pdf-viewer/commit/5f09846fbc5b4cd71ab910f98b343d681e5808ed

I've modified the method for loading pdfjs-dist and fixed transpilation settings.
With these changes, the text is now displayed on top of the PDF in my environment as shown in the attached file.
The position is still misaligned, though. 🤔
Since I have updated several library versions, you will need to delete node_modules and package-lock.json once to run in your environment.

Thanks great! Thanks!


Add support for socket asynchronous api for recv & recvfrom

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).
SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Is this PR a good opportunity for supporting both active and passive ssl sockets (and also when using socket as backend for gen_tcp and gen_udp)?
If so I suggest adding additional commits inside of this PR instead having just one: I rather prefer having a layered PR that is more comfortable to review step by step rather a monolithic one.


Issues when using with Bullet Threading Plugin

Hey Gavin,
It's Anthony Baker. I know you're not responsible for the Bullet Threading plugin, but you and I have chatted on Twitter earlier about it. I'm still seeing issues.
Am on Luma 0.2.1 (latest) and seeing that the threading tied to your bullets is off — I think you mentioned this might have to do with sizing/positioning you've adjusted? Are you seeing this too?

I think #22  will fix this. I could only intermittently reproduce this. If you refresh Logseq, does the issue go away?

@gavinmn No, but let me disable my custom.css and see if it's anything I've done.

@gavinmn And it's 100% repro for me.

Still can't consistently reproduce this. Are you at 100% zoom level and using the theme completely as is? No other plugins enabled?

I can reproduce this now by disabling and enabling the plugin, but a refresh of Logseq fixes it every time. My thought is that Logseq is having trouble deciding which CSS to use — the bullet threading plugin's or my overrides to it, but I'm not sure how to verify that or fix it.

@gavinmn When you say "refresh" what do you specifically mean? A reload? Forced reload?
And yeah, one of the things that irks me about their custom.css is that it sometimes has issues where it seems like they're not honoring it fully or there's been a hiccup.

@gavinmn Okay, maybe it's just me. I have done the following, and in all cases, I still see the issue afterward.

Reload Logseq
Force reload Logseq
Disable and re-enable Bullet Threading
Uninstall and re-install Bullet Threading
Delete my custom.css

No idea what's going on, but in all the above cases (which were done independently of one another), the issue remained. Agh. I wish I didn't care. LOL.

@gavinmn Maybe I need to add am !important on your CSS attributes.

When you say "refresh" what do you specifically mean? A reload? Forced reload?

⌘+r or documents -> refresh
@StirlingBaker can you try nothing in custom.css and just using the theme from the marketplace?

@gavinmn Did it. Here's what I see:

This was after me deleting all copy from my custom.css and saving it, then refreshing Logseq from the Documents option menu.

Ok this time I think I was able to fix it. Could you update to the latest and tell me if it's working for you now?

@gavinmn Huzzah! It worked! Thanks a ton. 😁


fix(sdk): updated token for npm publishing

Codecov Report
Patch and project coverage have no change.

Comparison is base (86a07c6) 89.41% compared to head (da82d79) 89.41%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #498   +/-   ##
=======================================
  Coverage   89.41%   89.41%
=======================================
  Files          93       93
  Lines        3609     3609
=======================================
  Hits         3227     3227
  Misses        251      251
  Partials      131      131

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


fix: "Stores" field to not show loading spinner (#4669)

What
"simple input text field " was showing loading spinner, even if there were no autocomplete suggestions available.
Any input field, without autocomplete enabled, will now not show loading spinner.
Screenshot
untitled.webm
Fixes bug(s)
Fixes #4669

Codecov Report

Merging #4678 (acebdd8) into develop (da78240) will increase coverage by 0.00%.
Report is 1 commits behind head on develop.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           develop   #4678   +/-   ##
=======================================
  Coverage     9.91%   9.91%
=======================================
  Files          310     310
  Lines        15798   15792    -6
=======================================
  Hits          1566    1566
+ Misses       14232   14226    -6

Files
Coverage Δ

...app/lib/pages/product/simple_input_text_field.dart
0.00% <0.00%> (ø)

... and 10 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


BRC8 Smarthome Fernbedienung support

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
In the code i found the battery powered BRC8-remote-control with 8 buttons.
But in HA there is only 1 generic button. It does not reacts on any button-press.
Would be great :-)
Describe the solution you'd like
8 events in HA, each for one pressed button.
additional long-press events would be great, too.
Describe alternatives you've considered
Hints to do it myselfe (not a very good py-coder!)
Additional context
Can i help with some logs of the 8 button-pressed from SHC or HA? or something like that?
Where to catch the right logs?

BRC8 can only report a pressed Button. But can not report a specific Button.

The event entities in HA do not yet support modifiers, so there would need to be 16 different events… This feature is quite new in HA and I‘m unsure how it is supposed to be used.
You need to use the device triggers. Just open the device and click + besides Automations. There you can add automations for all buttons (and long presses)

Found it. Thank you for your advice.


feat(fullstack): Main Page

What core or extra feature does this satisfy?
This is a core feature for Main Page.
Describe the feature

[ ] User can sign up to our app.

Figure out authentication tokens/cookies between backend + frontend.
[ ] Data function (Aero)
[ ] Route (?)
[ ] Frontend (Leek)

[ ] User can sign in to our app.

Figure out authentication tokens/cookies between backend + frontend.
[ ] Data function (Aero)
[ ] Route (?)
[ ] Frontend (Leek)

[ ] User to enter some input where if it is empty, it would query the user's animelist, else it would take in a MyAnimeList username.

[ ] Data function (Aero)
[ ] Route (Mudkip)
[ ] Frontend (Mudkip)
User should receive the same frontend as #6.

[ ] Frontend to link myanimelist to user profile (Owen)
[ ] Express to make a request to jikan api to get a specific user's animelist (Owen)

Additional context
Recommended/suggested endpoints(please modify as you see fit when working on the issue):

Main HTML - localhost:3000/animelist
Sign in to app - localhost:3000/accounts/signin
Sign out to app - localhost:3000/accounts/signout
Recommendation List HTML from user input - localhost:3000/animelist/recommendations

Blocked by: #6

Issue has been redistributed in workload


Centered element non centered on small screen

From @webmasterlpo on December 29, 2015 9:50
Why the centered elements aren't centered on small screen ?
The code I'm using:
 <row>
    <columns large='12' small='12' class="center">

centered text

    </columns>
  </row>

Copied from original issue: zurb/foundation-emails#25

From @tdhartwick on January 4, 2016 17:46
Looks like there's an issue with the paragraph respecting it's 100% width when it reaches a small screen. Thanks for pointing this out, we'll take a look at this!

From @jordanbommelje on January 11, 2016 20:7
I seem to be having a similar issue with an image inside.
      <row>
        <columns large='12' small='12' class="center" >

        </columns>
      </row>


Make cache path canonical

Thanks for this PR! I've submitted a different fix and added some test coverage, but LMK if you think we should actually be canonicalizing the path early. We can continue discussing in #9.


Capability system causes issues with __dispatch_kernel() for slang-torch use-cases

We have two ways to use slang-torch: [AutoPyBindCUDA] which generates the kernel launch and a manual appraoch that allows the user to directly launch kernels through __dispatch_kernel()
Currently, the capability system is causing all the manual-mode tests to fail in the slang-torch repository.
Here is a smoke test:
float computeOutputValue(TensorView<float> A, uint2 loc)
{
    return A[loc] * 2;
}

[CudaKernel]
void mul_kernel(TensorView<float> A, TensorView<float> result)
{
    uint2 location = (cudaBlockDim() * cudaBlockIdx() + cudaThreadIdx()).xy;
    result[location] = computeOutputValue(A, location);
}

[TorchEntryPoint]
TorchTensor<float> multiply(TorchTensor<float> A)
{
    var result = TorchTensor<float>.zerosLike(A);
    let blockCount = uint3(1);
    let groupSize = uint3(A.size(0), A.size(1), 1);

    __dispatch_kernel(mul_kernel, blockCount, groupSize)(A, result);
    return result;
}

which causes the following error:
error 36100: 'mul_kernel' requires capability 'textualTarget + cuda + vertex | textualTarget + cuda + fragment | textualTarget + cuda + compute | textualTarget + cuda + hull | textualTarget + cuda + domain | textualTarget + cuda + geometry | textualTarget + cuda + raygen | textualTarget + cuda + intersection | textualTarget + cuda + anyhit | textualTarget + cuda + closesthit | textualTarget + cuda + miss | textualTarget + cuda + mesh | textualTarget + cuda + amplification | textualTarget + cuda + callable' that is conflicting with the 'multiply''s current capability requirement 'textualTarget + cpp + vertex | textualTarget + cpp + fragment | textualTarget + cpp + compute | textualTarget + cpp + hull | textualTarget + cpp + domain | textualTarget + cpp + geometry | textualTarget + cpp + raygen | textualTarget + cpp + intersection | textualTarget + cpp + anyhit | textualTarget + cpp + closesthit | textualTarget + cpp + miss | textualTarget + cpp + mesh | textualTarget + cpp + amplification | textualTarget + cpp + callable'.
    __dispatch_kernel(mul_kernel, blockCount, groupSize)(A, result);
                      ^~~~~~~~~~

Moving out to Q4 because there is currently a workaround in place.


[hevce] Only VDEnc mode is available on JSL & EHL

The SDK always use VDEnc mode for AVC & VP9 encoding on JSL & EHL by
default. To keep the hehavior consistent across codecs, use VDEnc mode
by default for HEVC encoding as well

@thomasli21801  This repo is for MediaSDK, not oneVPL. According to https://github.com/Intel-Media-SDK/MediaSDK#media-sdk-support-matrix, oneVPL runtime is not available on JSL/EHL.

BTW, this change is for VPL and you might need to create another PR for 1_x branch, right?

It's handled semi automatically.


Circle's animation jitter

Recently we have updated OpenLayers from version 4.6.5 to 6.5.
In OpenLayers 4.6.5 we used snapToPixel option set to false for ol.style.Circle for "accurate" rendering mode during the animation to prevent noticable jitter. This phenomenon was also described in the documentation for the version 4.6.5:

Using true allows for "sharp" rendering (no blur), while using false allows for "accurate" rendering. Note that accuracy is important if the circle's position is animated. Without it, the circle may jitter noticeably. Default value is true.

Unfortunately, this parameter has been removed in version 5.2.0, because it should be no longer needed according to the upgrade notes:

The renderer now snaps to integer pixels when no interaction or animation is running to get crisp rendering. During interaction or animation, it does not snap to integer pixels to avoid jitter.

However, when we compare the same animation from version 4.6.5 to the animation from version 6.5 we can see the difference in rendering circle's animation which leads to the visible jitter (it is most visible at the end of the animation when the circle's diameter is the greatest). We prepared also a sample video with animation comparison:
https://www.youtube.com/watch?v=XGZdVtwdHMA
We are wondering if this change is related to some kind of a change in how snapping to pixels work right now? Is there anything we can do to make this animation smoother?

During each step of your animation, you could set frameState.animate = true to indicate that you're running an animation, and then pixel snapping will be turned off.

Thank you for your answer @ahocevar!
I tried experimenting with setting frameState.animate value to true based on the Custom Animation example available on the project's page. However, I don't really think that this parameter affects the animation in any way (at least in the case of a circle style). What's more, I am pretty sure that the circle's jitter is also visible on the page with already mentioned Custom Animation example (the issue may be slightly less pronounced due to the smaller circle's width).
The second thing is that in our application we are animating features by using StyleFunction directly. It is most convenient for us since we have one place for all the styles, we can synchronize multiple animations for features and we have a direct access to the currently animated Feature object. Assuming frameState.animate property is working properly - is there any possibility to access current render frame state while using StyleFunction?

@piotr-przebieracz I was wrong about frameState.animate, so please scratch that. @MoonE is working on a fix, see #12467.


Update "Create Plugin" doc

Please verify the following:

[x] Everything works on iOS/Android
[x] ignite-base ava tests pass
[x] fireDrill.sh passed

Describe your PR
This finishes the doc for creating an Ignite plugin from a 3rd party plugin

What's LGTM mean?
Nice work, @robinheinze

Looks good to me :)


Image / Assets

Hi Ben, Thanks for setting this up. Could you extend example to use an image from Elm? I can't get it to work myself.

I think this is just how Parcel works, and the only workaround is to externalize the images. :(


Upgrade all packages

This is the result of running flutter update-packages --force-upgrade. I can't find a way to run all tests locally so I'm opening the PR in the first instance to get a full CI run.
I don't know if timing of updating packages is important or if we're confident enough in the tests that this is generally at any time. I'm trying to get an updated test package in that fixes an issue with being able to navigate to tests from a test runner (the test framework previously reported the location of the test call inside the testWidgets function; now we get both locations).
If we should re-do this at a specific time, let me know and I'll comeback to it.

LGTM

@xster Tests all look good - is it safe to just merge this? I'm not sure whether we try to merge at any particular times (for ex. avoiding upgrading right before a release in case it breaks anything a test didn't catch).

I'm told it's fine to land these if all the tests are passing; so I'll land when I see the build green (it's been broken for a little while; though the reason doesn't seem to match the code in the first broken changeset, so I'm not really sure what's happened there..)

@Hixie @devoncarew  After I merged this, flutter_gallery_instrumentation_test has failed twice with this message:
stderr: FAILURE: Build failed with an exception.
stderr:
stderr: * What went wrong:
stderr: Execution failed for task ':app:preDebugAndroidTestBuild'.
stderr: > Conflict with dependency 'com.android.support:support-annotations' in project ':app'. Resolved versions for app (27.0.0) and test app (27.1.1) differ. See https://d.android.com/r/tools/test-apk-dependency-conflicts.html for details.
stderr:
stderr: * Try:
stderr: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
stderr:
stderr: * Get more help at https://help.gradle.org
stderr:
stderr: BUILD FAILED in 16s
stdout: 108 actionable tasks: 98 executed, 10 up-to-date

I'm unsure whether this is related - the error seems Android-y and none of these packages are v27 but my upgrades a Dart packages; yet since that test is otherwise all green (except for one yellow flake - coincidentally on my previous change) it seems like to be the cause?
I'm not sure how best to determine with it's the cause? Should I revert this? (it's getting late here now, so feel free to have someone else revert while we investigate if I'm not responsive).

We should probably revert while we investigate. To try to reproduce, you can run the devicelab test locally.

This change was reverted and the devicelab test has gone back green.

The error is plugin related. Flutter plugins are pub packages, but bring in Android (and iOS) dependencies which may introduce their own conflicts. Should be solved by #18501 and once that lands, I'll try and re-land the package upgrade.

Flutter plugins are pub packages, but bring in Android (and iOS) dependencies

Aha, of course! It seems upgrading packages is riskier than I thought (I actually only really wanted to upgrade test but couldn't find any way to only do the one). Thanks for the info 👍; please let me know if you re-land the upgrade :-)

@zoechi This PR was re-done in https://github.com/flutter/flutter/pull/18506 after the plugin dependency was fixed; so all is good on master.
I'm not sure if it's normal for end users should be running upgrade-packages --force-upgrade in their beta branches; seems kinda risky. I just checked out the beta branch and ran flutter update-packages (without --force-upgrade) and it runs fine:
dantup-macbookpro:flutter dantup$ cd examples/catalog/
dantup-macbookpro:catalog dantup$ flutter run
Launching lib/main.dart on iPhone X in debug mode...
Starting Xcode build...
 ├─Assembling Flutter resources...                    1.4s
 └─Compiling, linking and signing...                  6.7s
Xcode build done.                                           10.0s
Syncing files to device iPhone X...                          2.8s

🔥  To hot reload changes while running, press "r". To hot restart (and rebuild state), press "R".
An Observatory debugger and profiler on iPhone X is available at: http://<IP_ADDRESS>:8100/
For a more detailed help message, press "h". To quit, press "q".

I'll add some notes to that issue.


always set the number on pagination param when fetching chat threads CORE-9096

Otherwise it would send 0 over on the subsequent pages and get 0 results.
Also, do we really need the entire thread? Can we check in this loop to see if we have enough data to reasonably render the widget? It seems like we can abort out of this loop and save a bunch of traffic.

Oh wait nvm, I see how it works, it does break out early.


prestige-frontend never stops

When using the make stop command, or from the supervisor interface, the frontend server is never stopped. The corresponding Node server is still running and is usable on port 3040. No errors are reported.
Initial process tree:
/sbin/init
 \_ npm exec parcel serve src/index.html --dist-dir dist-serve --port 3040
     \_ sh -c parcel "serve" "src\index.html" ""--dist-dir" "dist-serve" "--port" "3040"
         \_ node ~/prestige/frontend/node_modules/.bin/parcel serve src/index.html --dist-dir dist-serve --port 3040
             \_ ...

Same process tree after make stop :
/sbin/init
  \_ node ~/prestige/frontend/node_modules/.bin/parcel serve src/index.html --dist-dir dist-serve --port 3040
      \_ ...

The Node server lose its parent process and stay has a detached standalone process.
To restore the initial state, it must be killed manually. If it is not done, then on subsequent restart attempts:

The front end startup fail on : Error: Port "3040" could not be used.
It is reported as stopped on the supervisor interface.
But, Prestige works perfectly : the old process is still serving on port 3040...

Environment
Node : 16.6.0
Python: 3.8.10
OS: Kubuntu 20.10
Prestige:  d7be7e0

Interesting! I'm running macOS and I can't reproduce this. For me, make stop or stopping it from the supervisor interface stops the frontend process and releases the port almost immediately. I'm wondering if this is a Linux-only problem.
Taking a second look though, I can imagine why this might be happening, and I'm surprised this isn't happening on macOS. Would it be possible for you to change the command for prestige-frontend in the supervisord.conf (line 63) to the following and see if that resolves it? If not, I'll spend some time testing this on a Kubuntu VM this weekend to investigate further.
command=%(here)s/frontend/node_modules/.bin/parcel serve src/index.html --dist-dir dist-serve --port 3040

The change we are making is instead of having npx invoke a child process with parcel, we invoke parcel directly, so there shouldn't be a child process. Supervisor cannot stop child processes, its a documented limitation unfortunately.

Perfect, now it works as expected.


Handle expired challenge responses in acmechallenges controller

What this PR does / why we need it:
Updates the acmechallenges controller to handle 'challenge expired' responses from the acme server.
This will in turn trigger a retry of the order eventually.
Which issue this PR fixes: fixes #TODO
Release note:
Automatically retry expired Challenge resources

/retest


fix: Remove target=_blank from internal docs

No strong feeling about those. I personally don't mind _blank for truly external links 🤷♂ How do other dev tools do it?

From my experience docs with external links typically mark them as such while including target=_blank, but that's my anecdotal observation.


Add new URL to white-listed URL section

A URL was omitted from the list of URLs that are required to be white-listed in order for the Readiness Toolkit to function.

Thanks for submitting this change.


Visible whitespace disappears when editor.renderIndentGuides is true

VSCode Version: Version 1.7.0-insider (1.7.0-insider) c038219f8d137b4972a15346b81ea03751d669d4
OS Version: macOS Sierra 10.12 (16A323)

Steps to Reproduce:

Set "editor.renderWhitespace": "boundary" and "editor.renderIndentGuides": true
Open a TypeScript file (haven't tried others
Only the indent guides are visible

Although if you toggle those settings while the file is opened it will render properly.

@eamodio I tried with the following user-level settings.json:
"editor.renderWhitespace": "boundary"
, "editor.renderIndentGuides": true

in a folder without any other settings and without any extensions and I cannot reproduce.
Is there something more to it? Can you repro with code --disable-extensions?

@alexandrudima I can no longer reproduce with on 1.7 or 1.8


MAINT: Fixed several unused imports and unused assignments in scipy/interpolate

Reference issue
Further work on gh-11529
What does this implement/fix?
To reduce Pyflakes warnings, remove five(5) unused imports; eleven (11) unused variable assignments; commented on why one apparently unused import needs to be kept;

Review comments addressed, no further activity, so let's merge it in. Thanks Paul


Fix Proxy internal class and Object.prototype.toString() handling

Object.prototype.toString() is the only place in E5.1 where the object internal class is exposed to user code. When called with a Proxy object as the "this" argument the returned string should reflect the class of the target, not the Proxy. For example, if the target is an Array, the result should be [object Array]. For Duktape (at least up to 1.5.0) the result is [object Object] instead.
There are several ways to fix this - for example, since the class is immutable the Proxy could just get the target's class on creation. This may not work directly right now because the class number is also used for making decisions about the object's struct layout for example (see duk_hobject.h). The target lookup could also be made on-the-fly in Object.prototype.toString().
As a related issue, it'd be nice for Proxy objects to have their own internal class. While the class is not easily accessible from Ecmascript code, it would be visible to the debugger object inspection for example. Right now a debug client can only reliably detect a Proxy object by looking at the artificial properties returned by GetHeapObjInfo and checking for the exotic_proxyobj flag. It would be nicer if the Proxy could be identified e.g. from the class string of the object.

Hmm, reading through duk_hobject.h, the internal classes look like a fixed list and not something I can ever put arbitrary strings in.  I guess I can always do like not and add a .inspect method that is like .toString, but for pretty-printers.

If you need a class just for cosmetic purposes, it'd be quite easy to add support for Object.prototype.toString() to check for an explicit class name property and use that if present and otherwise fall back to the internal class number.

@creationix I'm a bit late to the party, but the kind of thing you describe is exactly what ES6 @@ToStringTag is meant for:
function MyClass() {}
MyClass.prototype[Symbol.toStringTag] = "MyClass";
myObj = new MyClass();
console.log(myObj.toString());  // [object MyClass]

Duktape is getting Symbol support very soon: #982

neat.


support endpointslices.discovery.k8s.io for prometheus.operator.servicemonitors

Request
The prometheus.operator.servicemonitors component currently only supports the legacy endpoints discovery resource.
Recently support was added to the prometheus operator for the new endpointslices.discovery.k8s.io resources: https://github.com/prometheus-operator/prometheus-operator/pull/6672
Following the pattern they implemented, would suggest we need a role = "endpoints|endpointslices" argument on the component to decide which resource to use for discovery, as they did not opt for a field on the CRD to denote the role.
endpointslices are already supported by discovery.kubernetes.
Use case
This is important as endpointslices are far more flexible, supporting FQDNs as well as IP addresses, and offers tenants a way to define static_configs / discovery.relabel targets via a resource in kubernetes rather than in the agent config itself.

Also interested in this, I'd thought that Grafana would have supported endpoint slices at this point. We  actually just hit a bug with endpoints that isn't present in EndpointSlices.


bug(build): minimum the change for the binary owner to unblock build

Issue #, if available:
Description of changes:
try to minimum the change for the binary owner. and also unblock the build
Per my verification, AL2 kubelet already owner by root.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Testing Done

See this guide for recommended testing for PRs. Some tests may not apply. Completing tests and providing additional validation steps are not required, but it is recommended and may reduce review time and time to merge.

/ci


Python script returns error on Ubuntu 18.04

File: engine/install/ubuntu.md
$ sudo add-apt-repository \

"deb [arch=amd64] https://download.docker.com/linux/ubuntu
$(lsb_release -cs)
stable"
Traceback (most recent call last):
File "/usr/bin/add-apt-repository", line 12, in
from softwareproperties.SoftwareProperties import SoftwareProperties, shortcut_handler
File "/usr/lib/python3/dist-packages/softwareproperties/SoftwareProperties.py", line 28, in
import apt_pkg
ModuleNotFoundError: No module named 'apt_pkg'

Thanks for sharing the issue. The topic has been updated. If the issue still persists, please let us know.


pull request for e2e test at 11/27/2016 02:26:04

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


function publishing issues

Hey there, Issue encountered while working on a function.
I cannot seem to update a function.  I thought that simply calling hypar publish again would result in an update, however I got an error saying I was not authorized.  I got around this by first deleting the function, then publishing again.
If I can delete, it seems I have the correct permissions after all.  am I wrong that hypar publish should update the function?

This seems to work after changes that were made to the running of functions.  I suspect this issue also had to do with a "team" feature mentioned on twitter.  Thanks!


Correction for raw affiliation MoISA, Univ Montpellier, CIRAD, CIHEAM-IAMM, INRAE, Institut Agro, IRD, Montpellier, France.

Correction needed for raw affiliation MoISA, Univ Montpellier, CIRAD, CIHEAM-IAMM, INRAE, Institut Agro, IRD, Montpellier, France.
raw_affiliation_name: MoISA, Univ Montpellier, CIRAD, CIHEAM-IAMM, INRAE, Institut Agro, IRD, Montpellier, France.
new_rors: 05kpkpg04;04abkkz03;051escj72;05q3vnk25;03rnk6m14;0005r2j17;003vg9w96;011xg1225
previous_rors: 05kpkpg04;04abkkz03;051escj72;05q3vnk25;03rnk6m14;0005r2j17;003vg9w96
works_examples: W4396746567
contact: a2dd9c2285bced1df6bec0b9a410a5c1:1f6a9787ccc0d3675e7b36854762e5f83282 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Updated Build-images pipeline and refactor build scripts

@irfanhabib I get:
/home/nwm/dev/v2backend/deploy/docker-compose/../..//deploy/common-build.sh: line 31: TAG_LATEST: unbound variable


Fix problem with missing .gitignore file

Add .gitignore with the same rules as in .hgignore but adopted to be valid for git
Untrack all files that match rules in .gitignore

Thanks! I accepted the pull request. I must have made the mistake when importing from hg.  :)


add KeyParseFunc for times when parser needs input val

This feature adds a new builder method .fetcher(KeyParseFunc<Key,Raw,Parsed>) which will allow us to have parsers with access to the input param. This work is needed for SectionFrontStore

lgtm


Please add Under Investigation and Under Consideration to Suggestion Status section

https://docs.microsoft.com/en-us/visualstudio/ide/suggest-a-feature?view=vs-2019#suggestion-status
Here are some screenshots:

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

ID: 785f2716-1297-f343-24fa-705a42c2ff64
Version Independent ID: 732cf4ab-6056-31a6-abda-c63a37f90e45
Content: Suggest a feature - Visual Studio
Content Source: docs/ide/suggest-a-feature.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @madskristensen
Microsoft Alias: madsk

@mrlife Those two states only apply to problem reports. This is by design, though we have a work item to look at aligning all states across the ticket types.

@madskristensen Thanks for clarifying that. I was trying to understand the status of some of my submitted issues but can't find definitions for these 2 statuses. Would you be able to provide a brief description here?

The translation from problem to suggestion states are not perfect, but can be thought of as:
Under Consideration = Under Review
'Under Investigation' = 'On Roadmap'

@madskristensen Thank you, this really helps.

@mrlife -- thank you for your suggestion and followup.
@madskristensen -- thank you for clarifying.
I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


perf(linter): use CompactStr in no-console

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6402
#6401
#6400
#6399  👈
#6398
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @camchenry and the rest of your teammates on  Graphite


Move information about old GSoC projects to CC Open Source

We'd like to move old information about CC's participation in Google Summer of Code from the public wiki to the Internships > History section of CC Open Source.
https://wiki.creativecommons.org/wiki/Summer_of_Code_2006
https://wiki.creativecommons.org/wiki/Summer_of_Code_2007
https://wiki.creativecommons.org/wiki/Summer_of_Code_2008
https://wiki.creativecommons.org/wiki/Summer_of_Code_Selection_Criteria
There may be other pages as well, this needs to be researched.
We shouldn't be copying this information blindly; part of this task involves figuring out which information is useful/interesting and how to organize it.

@kgodey @dhruvkb I would like to work on this issue? Is the content should be present in Intership>contents.lr in history section?

@madhhuurrii this needs some discussion, please feel free to review the content and make a proposal for where the information should go.

Noting that we have the Archives page, which could be an appropriate place to link this material, in addition to other places.
https://opensource.creativecommons.org/archives/

@zackkrida Since no one is working on this issue so shall I take over this issue.

@zackkrida  Actually I am planning to update the Technical Blog starting from 2006 and updating the required files as well.

@Cronus1007 there hasn't been much activity on this PR recently so you're welcome to take it up. Can you elaborate on

update the Technical Blog starting from 2006 and updating the required files as well

as I'm not able to infer what you meant there?

-@dhruvkb Actually Archives page has one of the list element CC Tech Blog(07-14) we can update it to CC Tech Blog(06-14). Because as per my observations the activity of 2007-2014 has been thoroughly updated on the Blog but it lacks for 2006 year.

@Cronus1007 where will you get the blogs for 2006 from?

@dhruvkb Actually I am also having a thought about it but I think https://creativecommons.org/blog/ this blog can help a lot.

No @Cronus1007, that is the CC blog, which is completely separate from the CC Open Source blog. We should not move posts from there.

@dhruvkb Rather we can use this https://wiki.creativecommons.org/wiki/Summer_of_Code_2006.

It would be better if you suggest some resources
.

Noting that we have the Archives page, which could be an appropriate place to link this material, in addition to other places.
https://opensource.creativecommons.org/archives/

But @dhruvkb above refers to the Tech Blogs rather than CC Open Source Blogs.


July 2023 updates

Codecov Report
Patch coverage: 97.10% and project coverage change: +0.03 :tada:

Comparison is base (5fda2cf) 96.10% compared to head (cdc809c) 96.13%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #244      +/-   ##
==========================================
+ Coverage   96.10%   96.13%   +0.03%
==========================================
  Files          74       76       +2
  Lines        6317     6655     +338
==========================================
+ Hits         6071     6398     +327
- Misses        246      257      +11

Impacted Files
Coverage Δ

Cecilifier.Core/Cecilifier.cs
94.87% <ø> (ø)

Cecilifier.Core/Misc/TypeResolverImpl.cs
100.00% <ø> (ø)

...e/TypeDependency/TypeDependencyCollectorVisitor.cs
100.00% <ø> (ø)

Cecilifier.Core/Extensions/ISymbolExtensions.cs
87.23% <71.42%> (-1.41%)
:arrow_down:

Cecilifier.Core/Extensions/SyntaxNodeExtensions.cs
96.82% <92.30%> (+0.74%)
:arrow_up:

Cecilifier.Core/AST/StatementVisitor.ForEach.cs
94.85% <94.85%> (ø)

Cecilifier.Runtime/TypeHelpers.cs
93.02% <96.00%> (+0.48%)
:arrow_up:

Cecilifier.Core/AST/ExpressionVisitor.cs
95.82% <98.09%> (+0.06%)
:arrow_up:

Cecilifier.Core/AST/CompilationUnitVisitor.cs
98.27% <100.00%> (+0.12%)
:arrow_up:

Cecilifier.Core/AST/MethodDeclarationVisitor.cs
99.33% <100.00%> (+<0.01%)
:arrow_up:

... and 10 more

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


ValueError:

[ValueError](valueerror: too many values to unpack (expected 2) fix)
any fix?

Umm provide me more details, as this is insufficient information to fix the issue.

sure, here:
Traceback (most recent call last):
File "C:\Users\User\Desktop\Crunchyroll-Account-Checkemaster\main.py", line 33, in
main()
File "C:\Users\User\Desktop\Crunchyroll-Account-Checkemaster\main.py", line 26, in main
src.checker.CrunchyrollChecker.create(filename)
File "C:\Users\User\Desktop\Crunchyroll-Account-Checkemaster\src\checker.py", line 41, in create
self._checker()
File "C:\Users\User\Desktop\Crunchyroll-Account-Checkemaster\src\checker.py", line 46, in _checker
loginDetail = self._filterEmailPass(line)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\User\Desktop\Crunchyroll-Account-Checkemaster\src\checker.py", line 122, in _filterEmailPass
self.email, self.password = loginDetail[0].split(':')
^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: too many values to unpack (expected 2)


Enable Configuring read regions and auto failover in CosmosDB

Hi, this pr enables configuring read regions and auto failover in CosmosDB in following services:

azure-cosmosdb-sql
azure-cosmosdb-sql-account
azure-cosmosdb-mongo-account
azure-cosmosdb-table-account
azure-cosmosdb-graph-account

In the provision and update step in these services, I add two parameter: readRegions and autoFailoverEnabled, customers can use these two parameters to manage geo-replication of the cosmos account.
Document has been added. Life cycle test case has been added. Feel free to contact me if you have any question.

A summary of today's commit:

In pollingUntilReadLocationsReady, change the timeout for each read region from 15 mintues to 7 minutes. It is observed that the creation of an empty read region takes about 5 minutes.
In pollingUntilReadLocationsReady, disable timeout in updating step, it is because updating may cause copying data from existing region to new region and it's unpredictable how long the copy process will take.
Truncate the id of read regions to 50 characters.

Hi @krancour and @jeremyrickard , could you please have a look?

Something I'm not quite understanding about this PR is why there's so much complexity in the update process.
It seems that with provisioning, the steps are deploy the ARM template, then wait for read regions to be ready, but with updating, it seems you're not initially using ARM (using a Cosmos management client instead), waiting for read regions to be ready, and then updating the ARM template.
This seems quite odd to me because typically (OSBA or otherwise) you shouldn't mutate Azure resources that were created using ARM through other means. You should just update the deployment. That's how all OSBA updates work, up until now. They just update the existing ARM deployment.
So what I'm trying to understand is the rationale behind this more complex update process.

Hi @krancour , the process of update is: update read regions, waiting for read regions to be ready and then update other properties. Both of the first step and the third step are done using ARM. The reason is mentioned here. The cosmosDB has a really strange contraint: it cannot update properties and
add/remove regions at the same time. So we have to update read regions firstly, keeping other properties the same as in provision, waiting for read regions ready, and then update other properties.

Any update on this PR?

Hi, I have finished this feature(ideally), could you please have a look on it?
And I think there are two items worth considering:

Due to the reason claimed here, there is no time limit for polling in update step. Can we set a very long time limit for it?
It is designed that the user is not allowed to fill provision parameter location in the readRegions array as claimed here. But the validation requires both location and readRegions so that it can't be done using property schema. In updating step, it can be done using ValidateUpdatingParameters. But currently OSBA don't have ValidateProvisioningParameters method, so validating it in provisioning step requires an extra provisioning step. As far as I'm concerned, it's neither well-designed nor user-friendly to add an extra provisioning step. So my current implementation is ignoring location in readRegions array. That is, if the user fill provision parameter location in the readRegions array by mistake, the result is the same as removing location from the readRegions array.

How do you think about these two items?

What is the latest on this? I am very interested in COSMOSDB hitting preview. But want to make sure it is performant and interops well with OSBA.

Hi @campbelldgunn , this feature is almost finished and my concern is the two items mentioned above. And it should work well if users follow the document and no unexpected exception happens in updating.
Hey @jeremyrickard and @krancour , do you have any suggestion on these two items? Also, please let me know if you have any concern blocking the merge.

There is a constraint in CosmosDB: it can't update readRegions and other properties at the same time. And my previous implementation is using complex logic to enable updating readRegions and other properties at the same time. After consideration, I decide to remove this logic. I think it's a good practice to leave as it is rather than using complex logic to enable it in the broker

This LGTM with all the clarification. Will merge it pending lifecycle tests.


Error in Load Model

Two issues found:
(1) A Restart Runtime occurs on !pip install requirements.txt . This, in turn, resets the
current directory to /current. But even after manually updating the current directory....
(2) Under Load Model: ImportError: /usr/local/lib/python3.7/dist-packages/torchtext/_torchtext.so: undefined symbol: _ZN3c106ivalue6Future15extractDataPtrsERKNS_6IValueE

I have tried to run the notebook and also failed.
ImportError                               Traceback (most recent call last)
<ipython-input-10-9274c6da60ac> in <module>()
      3 import clip
      4 get_ipython().magic('cd feed_forward_vqgan_clip')
----> 5 from main import load_vqgan_model, CLIP_DIM, clamp_with_grad, synth
      6 import torchvision
      7

10 frames
/usr/local/lib/python3.7/dist-packages/torchtext/vocab.py in <module>()
     11 from typing import Dict, List, Optional, Iterable
     12 from collections import Counter, OrderedDict
---> 13 from torchtext._torchtext import (
     14     Vocab as VocabPybind,
     15 )

ImportError: /usr/local/lib/python3.7/dist-packages/torchtext/_torchtext.so: undefined symbol: _ZN3c106ivalue6Future15extractDataPtrsERKNS_6IValueE

Hi, thanks for reporting this, it seems to be an error with the new PyTorch (1.10), I will fix it

is fixed

I just tried it, the notebook works now.
This is so much faster than training each input. Thanks a lot!

Cool! thanks for the update

I just ran the colab notebook on the site. Had to restart runtime in an early cell.
Then, I get the following. Am I missing something?
ModuleNotFoundError                       Traceback (most recent call last)
 in ()
2 import torch
3 import clip
----> 4 from main import load_vqgan_model, CLIP_DIM, clamp_with_grad, synth
5 import torchvision
6
ModuleNotFoundError: No module named 'main'

Hi @metaphorz, t's probably because you needed to run the cell cd feed_forward_vqgan_clip again after restarting runtime

@mehdidc I'm getting the "ImportError: /usr/local/lib/python3.7/dist-packages/torchtext/_torchtext.so: undefined symbol: _ZTVN5torch3jit6MethodE" at the moment, might be the same cause as last time?

@amirham Yes I was fixing it indeed :), works now


Fix Ubuntu installation instructions

Summary
Fix Ubuntu install instructions
Reason
I tried to install Kong based on the current instructions and it didn't work.
Testing
Follow the instructions on Ubuntu

There is a small dependency issue when installing kong with the latest version of Ubuntu Jammy.
dpkg: dependency problems prevent configuration of kong-enterprise-edition:
 kong-enterprise-edition depends on zlib1g-dev; however:
  Package zlib1g-dev is not installed.

after installing zlib1g-dev it installs just fine. If you broke the package manager while trying to install Kong though you have to fix it before you can install that dependency.

@Guaris Interesting, I didn't hit that issue running Jammy on AWS, but I have seen it before on other versions. It's not a Jammy specific issue.
How would you recommend proceeding? Add to docs or leave it out and let people follow the error message?

I have a gut feeling that if someone has a working Ubuntu installation with other packages, they wont run into this issue. I added a note to address dependency issues if someone is approaching this doc with a brand new ubuntu install.


JS error when closing toast

Bug description
Auto disappearing toasts set a timer (usually 5s) to clean it up automatically. This code throws an error if you had already manually dismissed the toast using the close icon.
Uncaught TypeError: Cannot read properties of null (reading 'removeChild')
    at app.js?id=da854d5812ee2964b8dfea0d4e19f93a:2:145600

Steps to Reproduce
Reproducible with any scenario that uses an auto-closing dismissable toast, eg.:

Open a published publication
Open DevTools
Click the lock or unlock button
Dimiss the toast with the close icon
Wait 5s until the timeout expires
See error on debug console

Automatic testing scenario

Execute toast triggering action
Dismiss the toast
Wait 5000ms
Test will automatically fail if the JS error still occurs

Risk classification
Likelihood
1: Frequent
Consequences
3: Marginal (no problem for end users but tests often fail because of this)
Expected behavior
No JS error
Screenshots

System information
Verified in Chrome, Firefox, Safari and Edge

Reported an issue for this in the htmx repo: https://github.com/bigskysoftware/htmx/issues/2497


[Bug]: Incorrect accent color

Requirements

[X] I checked the troubleshooting section in the README to verify that I have the latest Mushroom version.
[X] I did a search to see if there is a similar issue or if a pull request is open.
[X] I have read the state of card mod support (in pin issues) and understand card mod support is not provided in this repository.

Current Behavior
It displays incorrect color when changing icon_color or badge_color to accent color

Expected Behavior
It should be the same as the default card
Steps To Reproduce

Create a template or any card that supports icon_color or badge_color attributes
Set its color to accent

Context
YAML state

Environment
- Browser: Edge, native iOS app
- HA Version: 2024.9.3
- Mushroom: 4.0.7

Anything else?
No response

Mushroom is using rgb-accent-color because it only supports rgb color while while Home Assistant is using accent-color. I thought these values were sync by Home Assistant but apparently not. You can set rgb-accent-color in your theme to fix your issue.

Mushroom is using rgb-accent-color because it only supports rgb color while Home Assistant is using accent-color. I thought these values were sync by Home Assistant but apparently not. You can set rgb-accent-color in your theme to fix your issue.

Thanks a lot for your help.  I tried that setting as you suggested but it didn't work. I tried more and saw that when accent-color directly set to hex code or rbga then it works (Ex: accent-color: #ff9900), but when accent-color is referred by other variable then it doesn't work (Ex: accent-color: var(--light-primary-color) )
Is it a bug? Thank you so much.

It's not really a bug but a limitation, if you are using a hex code, it's easier to convert it to rgb code.
But if you are using variables, it's impossible as we don't know if the variable is hex or not.


More consistent instantiation from a .stan file in various languages

The changes I'm proposing in #172 do not expose a separate constructor for .stan files compared to .so files, but rather just branch on the extension of the file passed as the first argument.
So, in R, it would look like:
StanModel$new("foo_model.so", data="foo.data.json") # loads a model already compiled
StanModel$new("bar.stan", data="bar.data.json") # compiles, *then* loads

In Python this looks different for the two cases:
StanModel("foo_model.so", data="foo.data.json") # loads a model already compiled
StanModel.from_stan_file("bar.stan", data="bar.data.json") # compiles, *then* loads

In Julia it looks more like R, but is technically doing type dispatching rather than checking the .stan extension:
StanModel("foo_model.so", data="foo.data.json") # loads a model already compiled
StanModel(stan_file="bar.stan", data="bar.data.json") # compiles, *then* loads

I think the real outlier here is Python, which deserves to be unified IMO. We can leave the existing function, but just update some logic in the constructor to handle .stan files. Whether Julia should also be updated or left using this disbatching I am less sure.

Agreed, Python should be unified so that StanModel("bar.stan", data = "foo.data.json") works.  This also means adding a keyword argument named data, since in Python StanModel(...) currently has a keyword argument named model_data, different from R and Julia (which both use data).  I like data better.
In Julia, I think allowing StanModel("foo_model.stan", ...) would be a good thing.
These changes would take some care to ensure backward compatibility, ya?

The easiest way to ensure backwards compatibility is just leave the existing functions but re-direct their logic to the new suped-up constructor. Renaming a keyword argument in Python is a bit more annoying to do in a backwards-compatible way, but doable.


Hide to-address in wallet transaction history

Hey Izumi - we're getting feedback that hiding the To-address is going to be seen as a bug. For now let's truncate the address for display and not allow user to copy the full address. Does that make sense?

@markelrod Updated the code to do so.
https://github.com/user-attachments/assets/e19ee9d2-fb1c-4bb7-ac58-3f2e5787c7d7


Support Pull mode

From nodejs side we need

Export basic profile collection data (cpu, heap), for people that want to add
Maybe export a middleware for popular frameworks? Not entire sure how much work would be to support most popular frameworks, but worst case scenario people can implement themselves (using 1)
Maybe follow go's pprof (https://pkg.go.dev/net/http/pprof) example and create a server ourselves? We could use a simple http server https://nodejs.org/en/knowledge/HTTP/servers/how-to-create-a-HTTP-server/

I think this is done yeah?

I'm currently trying to implement Pyroscope into our nodeJs pods in production, and so far part of it seems to work according to the documentation. As we have a scrapper that collects this data, I'm setting it up in Pull mode, with the following setup:

Pyroscope.init({})
Pyroscope.startHeapProfiling()

router.get("/pprof/profile", Permissions.public, async (req, res) => {
  try {
    const p = await Pyroscope.collectCpu(0.01);
    res.send(p);
  } catch (e) {
    console.error('Error collecting cpu', e);
    res.sendStatus(500);
  }

});

router.get("/pprof/heap", Permissions.public, async (req, res) => {
  try {
    const p = await Pyroscope.collectHeap();
    res.send(p);
  } catch (e) {
    console.error('Error collecting memory', e);
    res.sendStatus(500);
  }
});

According to the docs this should be correct. The problem now is that collectHeap errors out:
2023-05-31 13:56:58 Error collecting memory Error: Heap profiler is not enabled.
2023-05-31 13:56:58     at v8Profile (/usr/src/app/node_modules/pprof/out/src/heap-profiler.js:32:15)

So, of course I thought "ah well, maybe I need to manually turn it on", which is why there is a line with  Pyroscope.startHeapProfiling() in it. Apparently this triggers a specific check, and then requires there to be a server address:
throw 'Please set the server address in the init()';

If I set it, I can then just pull the data, but I suspect that it will try launching it's own server in the meantime.
The culprit seems to be this codebit:
https://github.com/grafana/pyroscope-nodejs/blob/f0f5a777536fa010920e76c8112d3c62b5e04295/src/index.ts#L258-L270
It does not seem to respect if we are trying to use it in pure pull mode or not, and will blindly require a serverAddress.

bump?

i'v changed default path for memory heaps cause of scraping from grafana-agent without changing grafana-agent configuration (related to https://github.com/grafana/pyroscope-nodejs/issues/36)
and starting HeapProfiling inside handler, init was just for pass the errors about appName and serverAddress
in DEBUG=pyroscope i dont see any heap profiles sended to server, and they are scraped
may be it's may be usefull for somebody :)
Pyroscope.init({
    serverAddress: 'localhost',
    appName: 'appName',
});
app.get('/debug/pprof/allocs', async function handler(req, res) {
    Pyroscope.startHeapProfiling();
    logger.info('Collecting memory');
    try {
        const p = await Pyroscope.collectHeap();
        res.send(p);
        Pyroscope.stopHeapProfiling();
    } catch (e) {
        logger.error('Error collecting memory', e);
        res.sendStatus(500);
    }
});


The Jail option reports "occambsd: created" but is not created

After "occambsd: created", the jail is not visible with jls(8)
Any ideas?

I have done the following

clone occambsd
clone freebsd 13.0 into src
./occambsd.sh -j
sh boot-jail.sh

This is what I got in the end.
root@mbp0:/tmp/occambsd # cat boot-jail.sh
jail -c -f /tmp/occambsd/jail.conf
jls
root@mbp0:/tmp/occambsd # cat jail.conf
exec.start = "/bin/sh /etc/rc";
exec.stop = "/bin/sh /etc/rc.shutdown";
exec.clean;
mount.devfs;
occambsd {
        path = "/tmp/occambsd/jail-mnt";
        mount.devfs;
        host.hostname = "occambsd";
#       ip4.addr = <IP_ADDRESS>;
        exec.start = "/bin/sh /etc/rc";
        exec.stop = "/bin/sh /etc/rc.shutdown jail";
        }
root@mbp0:/tmp/occambsd # sh boot-jail.sh
occambsd: created
ELF ldconfig path: /lib /usr/lib /usr/lib/compat
32-bit compatibility ldconfig path:
Setting hostname: occambsd-jail.
Updating motd:.
Updating /var/run/os-release done.
Creating and/or trimming log files.
Clearing /tmp (X related).
Starting syslogd.
/etc/rc: WARNING: failed to start syslogd
Starting cron.

Fri Mar 11 19:01:21 UTC 2022
   JID  IP Address      Hostname                      Path
     2                  occambsd-jail                 /tmp/occambsd/jail-mnt
root@mbp0:/tmp/occambsd # jls
   JID  IP Address      Hostname                      Path
     2                  occambsd-jail                 /tmp/occambsd/jail-mnt
root@mbp0:/tmp/occambsd #

Can you share the output of dmesg and /var/log/messages ?

I simplified the jail code, added a stop scrip, and it now works.