inlined-as-list-updates

This PR fixes two slots that should have been inlined_as_list (because they were multivalued and has a class in their range what doesn't have an identifier slot) but weren't
It also adds a Python test to detect that situation in the future

Guidelines
Soft Schema Freeze
The nmdc-schema and berkeley-schema-fy24 schemas are under a soft freeze, which means changes should not be made that have any downstream implications. To ensure this, all PRs created during the freeze will be closely reviewed with every component of the NMDC system in mind.
Reviewers
To ensure no changes are made unexpectedly, PR creators will use this PR template to tag and notify all task coordinators. Review should be specifically requested from all Berkeley Schema Roll Out task coordinators that you expect to be affected by this PR.
We expect task coordinators to review PRs and provide feedback/approval within 1 week of when they are identified as reviewers.
PRs will NOT be merged until all task coordinators (or their delegates) have approved it; either here on GitHub (via "Review changes > Approve" or an equivalent comment) or verbally.
Expedition, questions, and discussion can happen at any meeting.
Delays in review & merging should be addressed in meetings or with NMDC leadership.

If you expect the changes toimpact this component...
...request a reviewfrom this person

MetadataSchema
@mslarae13

RuntimeMongo databaseDatabase migrations
@eecavanna,who will pull in@shreddd as needed

PostgresIngest
@naglepuff

Data Portal
@aclum

Workflows: MG & MT
@mbthornton-lbl

Workflows: MetaB & NOM
@corilo

Workflows: LipidO
@kheal

Workflows: MetaP
@SamuelPurvine

ETL code
@sujaypatil96

Jupyter notebooks
@brynnz22

PR Information
What type of PR is this? (check all applicable)

[x] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation
[ ] Schema change: Structure and content

created, updated, or deleted a class, slot, or enum
changed whether a slot is multivalued
changed the way a slot is assigned to a class
changed the permissible_values of an enum
etc.

[ ] Schema change: Cleanup and preparation

updated the description of a class, slot, or enum
updated the mappings of a class, slot, or enum to an ontology
added an enum for future use (it is not in the range of any slot)
etc.

Description

PRs should be small and concise.
Aim to create small, focused pull requests that fulfill a single purpose. Smaller pull requests are easier and faster to review and merge, leave less room to introduce bugs, and provide a clearer history of changes.

Replace this text with a description of what this PR branch contains. Please keep in mind that all reviewers will be reading this description. Example: "In this branch, I..."

Related Issues

All PRs should relate to or fix an issue(s). Please identify the issue(s) below.

Related Issue(s): #
Fixes: #

Did you add/update any tests?

[ ] Yes
[x] No (Add a justification below)
[ ] I need help with writing tests

Could this schema change make it so any valid data becomes invalid?

This is a question about what the schema allows. It is not a question about what happens to exists in the NMDC database right now.
Example: If, in this PR branch, you renamed a slot from foo to foo_bar, the answer to this question would be "yes," even if nothing in the NMDC database currently uses the foo slot.
More examples: slot or class name changes, changes to a slot's multivalued state, changes to a slot's range (e.g. string to integer), changes to slot assignments to classes, changes to an enum's permissible_values

[ ] Yes (A migrator is required)
[ ] No
[ ] I need help determining this

If you answered "Yes", does this PR branch include that migrator?

[ ] Yes
[ ] No, this PR is incomplete and I need help writing the migrator

Does this PR have any downstream implications?

Examples: any change here that requires a change to workflows, workflow automation, the Mongo-to-Postgres ingest process, Jupyter notebooks, the Runtime, etc.

[ ] Yes (Explain below)
[x] No

@mslarae13 brought up the topic of defining some characteristics (i.e. multi-valued-ness) globally and some things locally. @turbomam will designate @mslarae13 as a reviewer after he resolves that.

oops I merged. I will still make the change (uniform multi-valued-ness) and review with @mslarae13


More detail needed to get plugin working

Documentation link

https://github.com/PaloAltoNetworks/docusaurus-openapi-docs#configuring-docusaurusconfigjs-plugin-and-theme-usage
Describe the problem

The guide does not sufficiently describe how to get the plugin working with a docusaurus project. Using the provided code examples does not build any pages.
I have created a new docusaurus project with npx create-docusaurus@latest my-website classic --typescript.
I have installed docusaurus-plugin-openapi-docs and docusaurus-theme-openapi-docs (v1.7.3).
I have downgraded @docusaurus/core and  @docusaurus/preset-classic to v2.2.x as v2.4.0 is not supported by the plugin.
I have used the code recommended for the docusaurus.config.js and copied the petstore.yaml file to the project.
Suggested fix

Include more detail in the installation guide to get the plugin working with a typical docusaurus project.

Hi @jacknevitt , I have also trouble getting the project up and running with latest Docusaurus. From your repository, I cannot see the usage of "docusaurus-theme-openapi-docs" or anything from this repo here. Can you ensure to submit your sample?

I'm having a similar issue (also downgraded docusaurus to 2.2.0 and installed v1.7.3). When I run docusaurus start, everything seems alright, but when I run docusaurus build I get the following  errors:
export 'toString' (imported as 'toString') was not found in 'docusaurus-theme-openapi-docs/lib/markdown/utils
export 'guard' (imported as 'guard') was not found in 'docusaurus-theme-openapi-docs/lib/markdown/utils

My apologies @PeterMerkert, I had accidentally overwritten the repo with redocusaurus.

Will update docs coinciding with next stable releases.


chore(master): release 1.1.122

:robot: I have created a release beep boop
1.1.122 (2023-07-15)
Miscellaneous

deps: update dependency @wayofdev/commitlint-config to v3 (3fc4337)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/keywind/releases/tag/v1.1.122 :sunflower:


Stack token logo

Logo and token adress for stack !

please add Stack logo and token adress

can we get this approved please?

hi your token doesnt currently meet the liquidity requirements to be token listed. please work to get liquidity at 6 figures minimum with sustained volumes and we can consider then.
If you want to submit a PR for just the token logo and remove the listing request we can then process the logo only

Name : Stack Token
Symbol : Stack
Contract : 0x80A62e8A45F6467b499EE7eDAD3Fd7bef0c83026

You can remove this request. PR i got the Logo added


Fix for issue #171

The call to dlImportModelWeights action was passing a parameter that does not exist.  I have restructured the Python code to verify any new parameters exposed by this action to ensure backward compatibility.  Unit tests have been run against the June 2019 SAS Viya release to ensure my changes didn't cause any other issues.  I also verified that the test described in issue #171 runs correctly.

The existing unit tests cover this change.  I ran the unit tests with a Viya server based on the June 2019 release and everything came back clean.

Hello, is this fix included in the latest 1.1.1 version?
I've got the same error when importing InceptionV3 weights.
This is the code:
model_inception = InceptionV3(
conn=s,
model_table='InceptionModel',
n_classes=1000,
n_channels=3,
width=299,
height=299,
scale=1,
offsets=imTrain.channel_means,
pre_trained_weights=True,
pre_trained_weights_file='/home/sasdemo/Inception/inception_v3_weights_tf_dim_ordering_tf_kernels.kerasmodel.h5',
include_top=True)
and this is the log:
NOTE: Model table is attached successfully!
NOTE: Model is named to "inceptionmodel" according to the model name in the table.
NOTE: Cloud Analytic Services made the uploaded file available as table NEW_LABEL_TABLE_BRVDAT in caslib CASUSER(sasdemo).
NOTE: The table NEW_LABEL_TABLE_BRVDAT has been created in caslib CASUSER(sasdemo) from binary data uploaded to Cloud Analytic Services.
NOTE: no dataspec(s) provided - creating image classification model.
NOTE: Model weights attached successfully!
ERROR: Parameter 'gpuModel' is not recognized.
ERROR: Expecting one of the following: nThreads, modelTable, modelWeights, labelTable, formatType, weightFilePath, weightFileCaslib, model, casout, path, caslib.
ERROR: The action stopped due to errors.

This fix should be included in version 1.1.1.


UI. issues with CIS Scan

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):
Issue#1

Enable schedule scan on a cluster.
The schedule scan runs a scan on the cluster.
Run scan manually after Step 2

Expected Result:

The scan should be sorted according to time in the page (cluster --> Tools --> CIS Scan)
A sort on the scan runs on "Date" column, should sort the scans

Actual Result

The schedule scan run is always. on the top in the page
A sort on the scan runs on "Date" column, sorts scheduled scan runs and manual scan runs separately.

Issue#2
The scan run and reports are not displayed on the UI correctly

Note:
On a bigger window size - a monitor, this seems okay, but in a laptop, the issue is seen.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head latest - ui tag: latest2
Installation option (single install/HA): single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): rke DO
Kubernetes version (use kubectl version):

1.17

Verified on latest master-head ui Tag: latest2

Issues are seen fixed

Issue#1

Enable schedule scan on a cluster.
The schedule scan runs a scan on the cluster.
Run scan manually after Step 2
The scan is sorted according to time in the page (cluster --> Tools --> CIS Scan)
A sort on the scan runs on "Date" column, sorts the scans

Issue#2

The scan run and reports are displayed on the UI correctly


No syntax highlighting in notebook

About the half the time I load a torch notebook, I get syntax highlighting; the other half I do not (all text in code cells is just in black).
This is in chrome 39.0.2171.99 on OS X 10.10.1.
For reference, syntax highlighting with the normal IPython profile (eg from using ipython notebook) reliably gives syntax highlighting.

we actually dont use anything different for syntax highlighting than what ipython ships with (CodeMirror). We give a language hint: https://github.com/facebook/iTorch/blob/master/custom.js#L8
and ipython takes care of the rest.

In that case, this is probably the same race condition affecting us at IJulia: https://github.com/JuliaLang/IJulia.jl/issues/168.

I had the same problem. Syntax highlighting appeared when manually changed the "Cell Type" on the top of a notebook to "Code".

Having the same issue. For some reason, it only happend in Firefox, and only for certain notebooks

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

This may have been fixed in IPython 3.2. See the issue at https://github.com/EntropyOrg/p5-Devel-IPerl/issues/40 for more details.

nice. thanks for letting me know @zmughal


Not an issue but FYI for Wiki potentially

I gave you the info about getting this working on Android using Lunix on a mobile phone. I recently put it on a Synology NAS and got it working also, this info might be of help for some people as it wasn't clear for me as being new to NAS.

Install Java on NAS from package centre app
Put jar file onto NAS in a folder of your choice
Run JAR file from terminal using command line: nohup java -jar ha-bridge.jar &
Make sure your in the folder with the ha-bridge.jar file when running the command as you may think you are but type dir to check as you may be in a folder with the same name that is virtual.

My Syno start/stop script
#!/bin/sh
PATH=$PATH:/var/packages/Java8/target/j2sdk-image/bin:/var/packages/Java8/target/j2sdk-image/jre/bin
BASEPATH=$(dirname $0)
mkdir -p $BASEPATH/data
JARFILE="ha-bridge.jar"
LOGFILE="$BASEPATH/${JARFILE}.log"
#CFGFILE="$BASEPATH/data/habridge.config"
JAVA="ava8"
PIDFILE="/var/run/ha-bridge.pid"
cd $BASEPATH

start(){
	if [ "$JAVA" != "$(which java|grep -oP $JAVA)" ]; then
		echo "Java8 not found - please install first"
		exit 1
	fi
	if [ -f $JARFILE ]; then
		java -jar $JARFILE &> $LOGFILE &
		echo $! > $PIDFILE
	else
		echo "HA Bridge JAR file not found. Download from https://github.com/bwssytems/ha-bridge/releases"
		echo "Rename or link it to '$JARFILE'"
		exit 1
	fi
}
stop(){
	kill $(cat $PIDFILE)
}

case $1 in
	start)
		start
		;;
	stop)
		stop
		;;
	*)
		echo "usage: $0 {start|stop}"
		;;
esac

Added to the wiki https://github.com/bwssytems/ha-bridge/wiki/Run-on-Synology-NAS


Translation Delivery

Translation Delivery

This pull request is for translation delivery. Please review and approve them,  @D067796

Closed as we got a newer delivery without merge conflicts which also includes all changes.


sf cli is not available when installing sfdx using npm

Summary
I thought that installing sfdx using npm would also install sf, the new cli. But this didn't happen.
As you can see in the following image, sf is not available

Steps To Reproduce:
1 - run npm<EMAIL_ADDRESS>-g --unsafe-perm=true --allow-root
2 - execut sf --version and verify it wasn't installed

This problem is not related to my node version. I tried with node 16 and I had the same result

When we install sfdx using the .pkg or .exe we also get sf. We can also have sf when running sfdx update. That is why I thought that if I intalled sfdx I would also get sf, but this does not happen. To install sf cli using npm we have to add this other package https://www.npmjs.com/package/@salesforce/cli


Request AS923 and US915 support for Seeed WM1302 module

Currently there are only EU868 preset for Seeed WM1302 on Concentratord. I Already tried to changed channel plan and test on Dragino LDDS75 sensors but doesn't work.
I'm using on SenseCAP M1, It would be nice for running dual mode Helium network and Chirpstack.

Bot regions are now included for the WM1302 region support:
https://github.com/chirpstack/chirpstack-concentratord/blob/master/chirpstack-concentratord-sx1302/src/config/vendor/seeed/wm1302.rs


Exploding Machines

Issue Description:
Whenever I put any machine that accepts any energy, even the ones that puts in capacitors the machine doesn't explode but it does make the sound that it is
Steps to reproduce:

Make any machine that has an energy input.
Then try to make the machine accept any energy
When the machine charges up it makes the explosion sounds and it doesn't stop and if you stand near the machine it does hurt you.

Affected Versions (Do not use "latest"):

EnderIO: 1.12.2-5.0.43
EnderCore: 1.12.2-0.5.57
Minecraft: 1.12.2
Forge: <IP_ADDRESS>25
SpongeForge? no
Optifine? yes
Single Player or/ Server? Both

Your most recent log file where the issue was present:
Never get a crash log.

What wire are you trying to connect to your machine?  The explosion noise is from EIO code telling you that something is trying to insert energy multiple times per tick (which some of the wires from other mods do).  It CAN be disabled via the config file.

What wire are you trying to connect to your machine? The explosion noise is from EIO code telling you that something is trying to insert energy multiple times per tick (which some of the wires from other mods do). It CAN be disabled via the config file.

Where is the config for it and i'm was using Mekanism's universal cables

In enderio off and yes Mekanism ducts are bad for attempting to bypass machine limits.

In enderio cfg and yes Mekanism ducts are bad for attempting to bypass machine limits.

Can you show me where the config is?

In the config directory where ever your Minecraft is saves to

Since no one could actually be bothered to properly explain how to disable this absolutely asinine feature that was added:
Go into the config folder of your instance, find EnderIO.cfg, locate the following line:
    # Should TEs protected their maximum energy input against multiple inserts? [default: HARD]
    S:protectAgainstEnergyOverflow=HARD

Change HARD to NONE
Than make sure to comment to the devs about how absolutely ignorant it was to add this feature and enable it by default.

The whole point of capacitors is to provide an upgradable limit to the amount of energy a machine stores and accepts.  If Mod B decides to play fast and loose with the "n energy per tick" limit by doing multiple inserts, then I rather believe Mod B needs to rethink their design.

Why not ask Mod B to change it then instead of annoying people who just want to play the game.

Feel free to tell them

"We'll force others who don't follow our way of thinking!" - I... I don't even. I really don't. Always Opt-In not Opt-Out.
It's their mod, they can do as they see fit. But the total lack of proper responds (other than @Yamza) astounds me.

I really feel I shouldn't reply to this, but I'm going to.  Sorry all.
Surely "It's their mod, they can do as they see fit" applies both ways, no?  Suggesting Mod B should have free reign on account of it being their mod and their code, but Mod A not having equivalent freedom with their mod and code seems disingenuous, at best.

@Ommina Isn't that basically what I said though? It's why Opt-in should always be the default not the other way around. That does not mean I don't understand the reasoning behind it. But going out of their way to deny people to play how they want without changing settings to allow other mods to function as intended seems like a bad idea and not winning anyone over.

The only reason this was set to hard was to see how many mods were bypassing energy limits once we got a good lust thise mod devs were informed and it was turned to soft by default in the next update. Also the explosion should only happen once and only be strong enough to blow up enderios machine.

Also the explosion should only happen once and only be strong enough to blow up enderios machine.

Did you mean that the other way around? I've only seen the charging block blow up. Booted up Minecraft just to test and indeed- When I plop down a Solar flux panel the panel is blow up. The EnderIO machine still sitting there.

nope the explosion is suppost to be only strong enough to blow up enderio's block that means that solar flux must have a 0 explosion resistance

Correction: The explosion has minimum possible strength unless some mod is sending hundreds of energy push operation per tick.
And expecting other mods to use an API correctly has nothing to do with "opt-in". If you cell phone charger was outputting 400 Volts instead of 5 Volts, you wouldn't just say "it blew up my iPhone in a fiery explosion, but hey, the manufacturer just didn't opt-in to the USB standard".


Eliminate a frameback navigation initialization race

During the first frameback navigation forward with reuseFrame set, if the
client controller within the frame completes before the frame's load event
fires we were previously failing to wire up our listener to handle subsequent
navigations.

lgtm 👍


Different behaviour inside a Grid than Xamarin.Forms

Hi Daniel,
just stumbeld upon a strange effect. I have here a reproproject.
https://github.com/escamoteur/ImageGridTest
If you change CachedImage to Image the Images fill the whole screen. When using CachedImage there is a GAP inbetween.
FYI: The with of the Columns is set at runtime in the Code behind file.
Thomas

I experienced this problem in the 1st moment that I tried FFImageLoading, exactly the same issue.
I resolved using WithRequest and HeightRequest inside ffimageloadin:CachedImage.  I hate to use absolute values, but this used grid exceptionally had 3 lines 25 dpi and 1 column 75 dpi. The Image uses 3 rows (rowspan 3), so the available space was 75x75. I put widht and height request to 75x75 and the used space comes equal as Image tag. Resolved, bad, but resolved.

It's probably because we're calling InvalidateMeasute method after image loading... and Image uses InvalidateMeasureInternal method which also calls some internal stuff and it's internal as name suggests. https://github.com/xamarin/Xamarin.Forms/blob/74cb5c4a97dcb123eb471f6b1dffa1267d0305aa/Xamarin.Forms.Core/VisualElement.cs#L651-L655
The only way to fix this is to use reflection.

@escamoteur  Could you test the newest prerelease? There were some changes to ImageViewAsync.

Hi @escamoteur,
I tested your sample and the behaviour of FFImageLoading is correct. Xamarin.Forms Image behaviour is incorrect though. Please notice that you've set AspectFit which means that whole image should be shown and that's exactly what CachedImage is doing. See screenshots:
CachedImage with AspectFill

CachedImage with AspectFit

Image shows the same result for both AspectFill / AspectFit


Put files into VM

How can I copy a file from my server to the VM?
I tried to run commands like '/bin/touch' to create a file, however I can't put contents into the file, because the CheerpX run command with echo prints everything in the terminal:
Javascript Code:
await cheerpx.run('/bin/echo', ['"Hello World"',  '>', '/var/www/index.html'], {});
Output:
"Hello World" > /var/www/index.html
The commands wget and curl are not available either. Is there a way to put a file into the VM using the API?

Hi and thanks for your curiosity!
You can create files in this way, but you'll need to execute the commands using bash. You're probably looking for something more in the lines of this:
await cheerpx.run('/bin/bash', ["-c", "/bin/echo Hello World > /var/www/index.html"], env)


Negative spike in output file on Windows

There is a short negative spike in the output file produced by Windows x64 binary if -q is enabled but regardless of -p value:
stftpitchshift.bat -p 1 -q 1

Not observed in both Mac and Linux ARM builds, and also not in Python on all platforms.
E.g. use the Dump class to debug the spectral envelope calculation.

Finally fixed in 428932a


Rename _Roulette ABC to _Trochoid

Is your feature request related to a problem? Please describe.
Right now the ABC for Hypotrochoid and Epitrochoid is called _Roulette as the "general" case but I find this name misleading - will rename to _Trochoid which is more accurate as this library is only concerned with trochoid's and not more general roulette's

Solved during #23


2022: lambda expression draft

Should we add a new (may be too common) type for lambda so we can use it later or leave it anonymous? Will the concept of closure be introduced?

Maybe just leave it anonymous, since it only occurs in semantic phase.
Closure will be introduced in Bonus.


add ipeiko resolver

close #39

Current coverage is 86.15%

Merging #113 into master will increase coverage by +1.40% as of 0aacc99

@@            master    #113   diff @@
======================================
  Files           45      45
  Stmts         1043    1062    +19
  Branches       259     264     +5
  Methods          0       0
======================================
+ Hit            884     915    +31
- Partial         68      72     +4
+ Missed          91      75    -16

Review entire Coverage Diff as of 0aacc99

Powered by Codecov. Updated on successful CI builds.


”Vulturești”, ”Buhăiești”, ”Podeni”, ”Voinești” are listed in the drop-down menu

Description:
When user writes ”Vulturești”, ”Buhăiești”, ”Podeni”, ”Voinești” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 3
Write in the search bar "Buhăiești"

Expected results
The location appears in the drop-down menu as "Sat Buhăiești, VASLUI (Vulturești)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Podeni"

Expected results
The location appears in the drop-down menu as "Sat Podeni, VASLUI (Vulturești)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Voinești"

Expected results
The location appears in the drop-down menu as "Sat Voinești, VASLUI (Vulturești)".


erlang / utf / bullet

When using https://raw.github.com/extend/bullet/master/src/bullet_handler.erl
I get an error of:
src/bullet_handler.erl:5: cannot parse file, giving up
src/bullet_handler.erl:5: cannot translate from UTF-8
(I reordered the lines so that it looked like:
-module(bullet_handler).
-behaviour(cowboy_http_handler).
-behaviour(cowboy_websocket_handler).
%% Copyright (c) 2011-2012, Loic Hoguin<EMAIL_ADDRESS>%%
%% Permission to use, copy, modify, and/or distribute this software for any
%% purpose with or without fee is hereby granted, provided that the above
%% copyright notice and this permission notice appear in all copies.
)
Anyway, the problem is that erlang (r17-rc2 on Ubuntu 64) seems to
dislike the "   ï   " in Loic, and changing it to "   i    " fixed it.

still occurs. in 18.0. I've submitted https://github.com/ninenines/bullet/pull/83 as a resolution


Add "ETH Staking by P2P.org" App

Revision checks

[x] Used smart contracts were audited.
[x] Your Safe App include a manifest.json file at the root with the required data.
[x] The app can be loaded as a custom Safe App in the Apps section of https://app.safe.global.
[x] The app auto-connects to the Safe.
[x] You are able to trigger and execute one transaction with a Safe.
[x] RPC requests are optimised (not triggering many requests in a very short time period).

Name/Description
Name: ETH Staking by P2P.org
Description: P2P.org provides secure non-custodial staking services for professional investors
Those are in the manifest.json. The App allows Safe users to make direct Ethereum staking as a service by P2P.org.
Type

New addition

Compatible Networks
- Mainnet

Audit document
https://github.com/mixbytes/audits_public/tree/master/P2P.org/ETH2 Depositor %26 ETH Staking Fee Distributor (v.2)
Code for review
Code is stored in private repositories, private access should be requested first.
IPFS hash/App URL
Custom Safe App link
Team information

- Project logo: https://eth.p2p.org/p2p-logo.svg
- Project landing page: p2p.org
- Team or discord contact: https://t.me/P2Pstaking
- Twitter link: https://twitter.com/p2pvalidator
- Github link: https://github.com/orgs/p2p-org/

@rgb2hsl Could you add me to the repo?
Thank you

@dasanra sure
I've added you to our dApp and microservice repos with read permissions, pls check the invitations on email
Maybe it'll be more suitable to also reach out in the telegram or Discord to have a conversation about the code. Please feel free to add me:

https://t.me/rgb2hsl
rgb2hsl#0056 (Discord)

@dasanra FYI currently we have some major changes in QA stage and they'll be merged soon. We'll switch from ethers to https://viem.sh/ You can find more details in corresponding PR that you can find in staking dApp repo.

App should be available now!


roll engine hotfix

roll engine hotfix from https://github.com/flutter/engine/pull/22678

Landing TBR. This is not part of a regular flutter release.


Is there authentication guide to Trains

Hi
i used the AMI to deplot to AWS. but i can see Everyone can login .
is there a guide for authentication ?
also - in the AMI deployment , where exactly is the trains-init script , could not find it , not even in the docker instances (webserver/api)
thanks
Shlomi

See matching thread here: https://github.com/allegroai/trains-server/issues/3
Feel free to vote with reaction :+1:
TL;DL
TRAINS-server is for internal use only, the main goal is full transparency, so everyone in the company can login (which is mostly for read access).
If you are using the AMI distribution, I highly recommend configuring the security-group of the instance to limit the access to the server for internal company IP group...

in the AMI deployment , where exactly is the trains-init script , could not find it , not even in the docker instances (webserver/api)

The trains-init script is just an easier way to configure credentials for the SDK (i.e. the TRAINS python package). The TRAINS-server does not need any credentials configured.
Once you login to the web-app (with any username, as currently there is not limitation)
You can generate your key/secret credentials pair (which the TRAINS package is using, in a similar way to AWS credentials etc) in the profile page (which is currently called 'admin' but will be changed in the next release) at http://localhost:8080/admin

thanks for the explanations.
regarding limiting IP addresses - this is a problem , as we use sagemaker to train models.
and i cannot tell which IP address sagemaker will use. (in order to report the metrics you collect)

@bmartinn - i have put the instance in AWS behind a LB , so it could be available to our Sagemaker server
but i get 405 method not allowed on POST - i believe this is because of there headers sent to my browser
if i go directly to the instance i get these headers which suggest that the local IP address is used as allowed origin
Access-Control-Allow-Credentials: true
Access-Control-Allow-Origin: http://<IP_ADDRESS>:8080
meaning this cannot be put behind ELB ?
thanks
Shlomi

The demoapp.trainsai.io is an almost exact replica of the pre-installed trains-server AMI and it works fine behind an ELB.
@shlomiken Can you please share the trains-server logs? (default location /opt/trains/logs)

trains.tar.gz
attached
i believe this is CORS issue - as when i request directly from the instance i see OPTIONS is being sent to server.
i might be missing something in configuration that say hey this is my domain name for this server.

The webserver's CORS configuration uses origin "*", which means any request origin is allowed, and the response headers will simply echo the origin - this is the reason you see Access-Control-Allow-Origin: https://demoapp.trainsai.io in our demo server's response, and Access-Control-Allow-Origin: http://<IP_ADDRESS>:8080 in yours (when you do succeed reaching the server).
Just to make sure, the responses we're talking about are those received by your browser when trying to access the webserver in port 8080, right?
This issue might be related to a HTTPS redirect issue, can you please describe how you configured the ELB?

Hi @bmartinn - the ELB is configured with both 8008, 80 and 443 listeners. that forward to the instance on port 8080 or 8008.
this is an Application LB .
Is there special rule i need to add ? can you share you configuration ?
thanks
Shlomi

I've just set up an AWS EC2 instance using the trains-server AMI to try and replicate your use-case. It works as expected, and my browser can successfully access the webserver with the URL http://<my-elb>.us-east-1.elb.amazonaws.com:8080.
I've used the following ELB configuration:

ELB Type: HTTP/HTTPS load balancer.
Listeners:

HTTP:8008 listener, forwarded to a target group:

Target type: Instance
Protocol: HTTP
Port 8008

HTTP:8080 listener, forwarded to a target group:

Target type: Instance
Protocol: HTTP
Port 8080

the difference is i use port 80 to mask port 8080 cause our sec groups do not allow 8080 traffic ( just like your demo server)
can you please share the configuration of the demo server ?
maybe its conf files without tokens?

You're correct, I've reproduced the issue right now and the reason is the different port used for accessing the webserver. The webapp assumes the webserver uses port 8080. Since in your case the webserver uses port 80, the webapp fails to access the apiserver.
I'll add this issue as a configuration feature request.
The demo server uses subdomains (demoapp, demoapi) instead of port numbers to redirect traffic to the webserver and the apiserver (the webapp can detect these names and adjust accordingly).
Since in your case, sagemaker only requires access to port 8008 (which is the port used by the trains package), you can:

Use ELB with a single 8008 listener and configure sagemaker/trains to use the external ELB address with port 8008
Use the EC2 instance private IP port 8080 in your browser, like you currently do

@bmartinn - how can i use subdomains as well ?

You can configure ELB to direct the app subdomain to port 8080 and the api subdomain to port 8008. Please make sure to use HTTP and not HTTPS as changing this will require reconfiguration of the webserver (which is part of the pre-built AMI).

@bmartinn sure , but how the UI knows to which api server to address.
my problem is the the UI is also not functioning cause of this issue - we have no problem with sage maker calls.

The UI can detect the app subdomain and use the api subdomain to access the server.

i have created 2 separated domains in AWS.

i have tried that (although sounds weird) - but the api requests still try to get to trainsapp

Use app and api, we in app.trains.kenshoo-lab.comand api.trains.kenshoo-lab.com.

Apologies, it seems the code related to custom subdomains redirection is not part of the docker build. I'll include it in the next release (soon). For now, you can use the ports approach discussed earlier.


fix: typo; Github --> GitHub

I stumbled across a typo for GitHub.

 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.


Disable llvm-loop-vectorization default option

The llvm compilation pass relies on llvm-loop-vectorization to emit SIMD instructions, instead we want all generated pointwise-ops to operate on vectors and not scalar.

So far this can be done by :

[ ] Extending the LoadStoreVectorization to handle pointwise-ops with broadcast semantics.

This is done already.


Seals of the Dragon do not modify the correct stats, or derived stats

Seals of the dragon should modify all physical stats, and power. Currently, it does not seem to be recalculating all derived stats.

Max health should increase, as should fatigue, and so on.

The test is being done with 4 seals, and 2 Doors of the dragon open.

Thank you for reporting this issue,
Dragon seals now properly affect derived data in v1.1.0
If there are no other issues, please close this issue


need fetch "from" state not just the "to" state

i think it's necessary to add get from state info like beforeRouteEnter (from,to,next) now we can fetch the to state by store.route but some times to a  view  we must decide wether or not load data by the from state's path.
i.e. go to a product-details view from product-list view need loading details info and in that product-details view go to anther order view and back ,some contents managed by vuex had changed,so in details view i don't want load data because not necessary)

#36 Is a proposed changed to support storing the from transition in state.route

done by merging #36, now out in 4.0


help

There was an issue building your app. This can mean your app.json's project is not a valid Heroku application. Please ensure your app is deployable to Heroku and try again.

I have the same problom


Add HashMap and HashSet to intro

Two more data types that have roughly similar equivalents in Python/Rust can be added to the intro piece.

Python
Rust

dict
HashMap

set
HashSet

This is a good chance to highlight the differences too: things like how HashMap requires that the data types are homogenous, whereas dict allows key/value pairs of arbitrary types.

@sanders41 I'm letting you take this one whenever you have the time. Also, I may have missed some common concepts/ideas that have common ground between Python/Rust, so feel free to add anything more as you see fit. The goal of the intro piece is to showcase rough counterparts in Python/Rust as much as possible, and the reader would come away with it thinking that Rust and Python have similar analogs (in many cases), and it's not infeasible to move between the languages.

Sounds good, I'll work on some examples.


sync data from mysql how to filter columns

How to sync data from mysql use custom sql? For example I have user table 4 columns (id,username,nicknane,passowd) but I dont sync password to es. Any ideas? thanks

Supply a filter column configs and don't sync may be ok. but I have no time to add it now. Could you send me a PR?

I'm really sorry,I dont know about golang.


chore: make getting Patreons async

After hooking up with the backend, getting Patreons will need to be asynchronous.
This MR makes it asynchronous.

Closes #20


Global-fix

fixes #66

cc @ndelangen

Any update to this. It would be great to get this in and fix the global/window issue first and then tackle the overall improvement. thx.

Great job, I am facing this issue since the new 7.0 release. My question is, once this is merged into master, should we just wait until having a new storybook-addon-console npm release? Thank you!

Thanks for merging. Will the new version release anytime soon? thx

I assume this project is deprecated because we have no new release. Thanks anyway for your efforts.

Sorry for the late reply, I was on vacation.
I just released v.2.0.0, making it compatible with storybook 7.0.0!


Reduce method count in reduks core

After adding support for modules, method count in reduks core has increased to significantly (the module package has now 524 method). Need to find a way to reduce it. Using data classes for module def was a bad idea probably, because it adds tons of methods I don't actually need. See attached full list of methods
core_method_list.txt

One alternative to solve this is to again split the module out from core. This would make core as bare-minimum as possible.

Yes, I would probably do it

Update method list of core, after optimizations with jvmfield and data class removal reduks_core_method_list_b3.txt

After all optimizations, now method count in reduks-core is acceptable (around 430 methods)


Browse tables Rownum max not working

Describe the issue
When using Browse Tables and setting the Rownums, the Rownum max value does not seem to be considered and seems to stay fixed at the default 100. Both 7.6(.1) and 7.7-20211123.120619 have this issue.
Reporter
Ricardo/Fop

To Reproduce
Steps to reproduce the behavior:

Go to 'Browse tables'
Fill in the fields Rownum min with a value > 100 and a Rownum max > Rownum min
Press send
See error Rownum max must be greater than or equal to Rownum min
Fill in Rownum min = 80 and Rownum max = 120
Only records 80-100 are shown

Screenshots

Configuration
Please provide the configuration of the Pipe or Receiver with the problem.

Input
Please provide an example of the input message.
Alternatively, provide a Ladybug report.

Environment
FF! 7.6.1: ibis4adn 20211112_1330-7c5ba44a-master-20
Running on srtzzapp0442 using IBM WebSphere Application Server/<IP_ADDRESS>
Java Version: Java(TM) SE Runtime Environment (<IP_ADDRESS> - pxa6480sr6fp26-20210216_01(SR6 FP26))
Heap size: 725M, total JVM memory: 2048M
Free disk space: 2GB, total disk space: 2GB
Up since: 2021-11-22 13:26:49 (8d)
===============================================================
FF! 7.7-20211123.120619: Ibis4Test
Running on srdzzapp0455 using IBM WebSphere Application Server/<IP_ADDRESS>
Java Version: Java(TM) SE Runtime Environment (<IP_ADDRESS> - pxa6480sr6fp26-20210216_01(SR6 FP26))
Heap size: 452M, total JVM memory: 512M
Free disk space: 1GB, total disk space: 2GB
Up since: 2021-11-24 00:22:52 (6d)

Additional Environment

DBMS: [e.g. Oracle, MSSQL, MariaDB, MySql, PostgreSQL]
Browser: [e.g. Chrome, Safari, Edge, Firefox]

Additional Context
Add any other context about the problem here. (f.e. ladybug report / test adapter with larva test)

I suspect the issue is https://github.com/ibissource/iaf/blob/master/core/src/main/java/nl/nn/adapterframework/webcontrol/api/BrowseJdbcTable.java#L98:
maxRow = Math.min(Math.max(maxRow, 1), 100);
I think the Math.min should not be there.


Seems like MOVR announcement was also delayed

Just for reference:
My bot picked up MOVR at 2021-8-11 2.59.45
This is consistent with the timestamp on binance.com:

However, in terms of trading activity, the wave already at its peak by then

Seems consistent with previous findings
https://www.youtube.com/watch?v=H2ylujgbDzw

Here you find an Screenshot made of the 10s Candels from gate.io API. Looks like the Pump started at ~ 02:59:10

My Bot catched the announcement at 02:59:25

What's this issue for?


containerd systemd unit started before containerd is fully ready

Description
This was already reported in https://github.com/containerd/containerd/discussions/5109
I'm running kubeadm in a systemd unit with After=containerd.service, and sometimes get the following error:
level=fatal msg="getting status of runtime: rpc error: code = Unknown desc = server is not initialized yet"

containerd logs
containerd[2853]: time="2023-05-03T09:41:09.370125143Z" level=info msg=serving... address=/run/containerd/containerd.sock.ttrpc
containerd[2853]: time="2023-05-03T09:41:09.370188618Z" level=info msg=serving... address=/run/containerd/containerd.sock
systemd[1]: Started containerd container runtime.
containerd[2853]: time="2023-05-03T09:41:09.430108908Z" level=info msg="containerd successfully booted in 0.345311s"
containerd[2853]: time="2023-05-03T09:41:09.430325341Z" level=info msg="Start subscribing containerd event"
containerd[2853]: time="2023-05-03T09:41:09.450471617Z" level=info msg="Start recovering state"
containerd[2853]: time="2023-05-03T09:41:10.419322678Z" level=info msg="Start event monitor"
containerd[2853]: time="2023-05-03T09:41:10.420005372Z" level=info msg="Start snapshots syncer"
containerd[2853]: time="2023-05-03T09:41:10.420028803Z" level=info msg="Start cni network conf syncer for default"
containerd[2853]: time="2023-05-03T09:41:10.420039435Z" level=info msg="Start streaming server"

Describe the results you received and expected
containerd should tell systemd it's ready to handle every commands without error
What version of containerd are you using?
1.7.0
Any other relevant information
No response
Show configuration if it is related to CRI plugin.
No response

interesting.. should we wait for restart to complete in CRI before containerd can be considered to be ready.. might have a recursion issue here..

My workaround right now is to run crictl info in a loop and wait for RuntimeReady and NetworkReady

Looks like the plugins (e.g. cri) are initialized in a separate goroutine: https://github.com/containerd/containerd/blob/5dda3d8d5539b141589efbb44014efe7bbbda86d/cmd/containerd/command/main.go#L194
Should we wait for server.New() to return before notifying systemd?

We hit a similar issue in Bottlerocket, where kubelet tries to interact with CRI before it's initialized:
May 19 07:49:42 ip-10-40-52-4.eu-west-1.compute.internal kubelet[1417]: Error: failed to run Kubelet: unable to determine runtime API version: rpc error: code = Unknown desc = server is not initialized yet
May 19 07:49:42 ip-10-40-52-4.eu-west-1.compute.internal systemd[1]: kubelet.service: Main process exited, code=exited, status=1/FAILURE
May 19 07:49:42 ip-10-40-52-4.eu-west-1.compute.internal systemd[1]: kubelet.service: Failed with result 'exit-code'.
May 19 07:49:42 ip-10-40-52-4.eu-west-1.compute.internal systemd[1]: Failed to start Kubelet.

This can be worked around by configuring the kubelet service unit to always restart, but this isn't very elegant especially when containerd is configured as a Type=notify service and has indicated that it's ready.

Does it make sense to wait here for some kind of "ready" notification from the s.Run()?
I am thinking in the lines of:
func (c *criService) Run(ready chan<- bool) error {
  ...
  // Set the server as initialized. GRPC services could start serving traffic.
  c.initialized.Store(true)
  ready <- true
  ...
}

And in the caller:
ready := make(chan bool)
go func() {
if err := s.Run(ready); err != nil {
  log.G(ctx).WithError(err).Fatal("Failed to run CRI service")
  ready <- false
}

if ok := <-ready; ok {
  log.G(ctx).Info("CRI service is ready")
}

@mikebrow

interesting.. should we wait for restart to complete in CRI before containerd can be considered to be ready.. might have a recursion issue here..

What's the recursion issue you're worried about?

https://github.com/awslabs/soci-snapshotter/blob/main/integration/util_test.go#L674-L675 also uses an approach with a loop of dummy requests to confirm containerd (and soci) are running before proceeding.


Adds storm.zookeeper.retry.times and storm.zookeeper.retry.interval

This patch adds two new options for zookeeper:
storm.zookeeper.retry.times
storm.zookeeper.retry.interval

They define zookeeper clients retry policy.Please check the difference.

Merged in with some modifications (to unify the code with the usage of zookeeper in transactional topologies). Thanks!


[Feature]:Identify sensitive content based on user comments

I would like to propose adding a new feature to our web scraping tool that identifies sensitive content based on user comments scraped from IMDb. This feature will enhance the tool's functionality by providing users with a way to filter and classify movies based on their potentially sensitive content like 18+ content.
Proposed Solution:
The feature will involve:
Collecting user comments from IMDb for a variety of movies.
Training a machine learning model using NLP libraries(I will be using nltk) to classify movies as containing sensitive content or not based on the content of the comments.
Integrating the trained model into the web scraping tool to classify new comments in real-time.
Please assign this issue to me under the label GSSoC'24
@sanjay-kv

Thankyou for assigning me this issue


Res/layout

Hello everybody. First I do not speak English well . A really interesting application, but I find the XML file layout in res / layout . I newbie android programming. A limit can someone explain to me how the view is first performed ?
Thank you in advance for your answers.

Hi, I don't don't know if I understand your question correctly. The views (authentication/home) are decided by the authentication state of the user. And the state is preserved by redux.

Hello,
Thank you for responding so quickly. I just wanted to see the res / layout of the login activity since the design is really cool ( EditText and animations ) . I was just looking to replicate the same good but I know nothing at all redux .
But thank you for letting me answer.


Generate model for '201' response

Closed it in favor of
https://github.com/epam-cross-platform-lab/swagger-dart-code-generator/pull/154


number of jobs is increasing

I am running FALCON on an ~1.6GB genome. When using the script recommended for checking job progress during the daligner stage I got 747 total jobs. After a few days of running, the number of completed jobs got to 747, but then the total just bumped up to 1510 and it kept running more jobs. Is this normal? Is there any way to tell how long it will really take?
The command used for total jobs is:
ls 0-rawreads/daligner-chunks/ | wc -l
The command for completed jobs is:
find 0-rawreads/daligner-runs/j_*/uow-00 -name "daligner.done" | wc -l
based on the page: https://github.com/PacificBiosciences/pb-assembly
Thanks,
Michael

UPDATE: It finished the 0-rawreads/daligner tasks after the 1510 jobs. Now working on the 1-preads_ovl/daligner. There seems to be no method of determining how long the rest of this will take, but at least it is continuing to do things.

Yes, that's normal. The task-graph is dynamic, which is good and bad. Easy to work with, but not predictable.
The number daligner runs will be N*N, where N is the number of blocks in build/raw_reads.db. If you watch a single daligner run, and divide its runtime by twice the number of blocks it compared (since it compares A to B and B to A), you can divide that into N*N to estimate the total runtime of daligner.


sync_and_async_middleware throws exception

Which SDK and version?
sentry_sdk==0.20.0
uvicorn==0.13.3
gunicorn==20.0.4
django==3.1.6
Steps to Reproduce
copy the middleware example from django docs:
import asyncio
from django.utils.decorators import sync_and_async_middleware

@sync_and_async_middleware
def simple_middleware(get_response):
    # One-time configuration and initialization goes here.
    if asyncio.iscoroutinefunction(get_response):
        async def middleware(request):
            # Do something here!
            response = await get_response(request)
            return response

    else:
        def middleware(request):
            # Do something here!
            response = get_response(request)
            return response

    return middleware

Run django async
uvicorn.run(
        "apps.asgi:application",
        host="<IP_ADDRESS>", port=8000, lifespan='off', loop='asyncio', reload=True,
        ...
    )

Actual Result
...
File "/usr/local/lib/python3.8/dist-packages/sentry_sdk/integrations/django/asgi.py
Exception Value: 'function' object has no attribute '__acall__'

The problem is, that the decorator style has no __acall__ and the middleware is a function (not a class)
No problems in 0.19.x
This bug was introduced with this PR https://github.com/getsentry/sentry-python/pull/992

Hey @Speedy1991 Thanks for opening this issue
Looking into it

thanks for the quick fix!
Any idea when this will be released?

Anytime, It is already released :)


Docs for child processes

Add a README for Child_processes.
Notes:

there's a mention of the process kind for Libp2p_helper, which was added to master-qa, but not compatible, yet. This mention will become valid in the near future, then.
there are some changes of coda to mina, but not for environment variables, which still in use in the codebase
the exit 99 when the prover or verifier dies is changed to a failwith. I ran a demo node locally, killed the prover, and verified the daemon still exits. Reviewer: Coda_run.handle_crash gets called in this case, is that OK?

In general, I'd prefer if we don't make code changes along with the documentation PRs

I'll move those changes to a separate PR, if after reading my justifications, you're still not convinced.

Sorry for the delayed response. It's ok to leave them here, the justifications work for me.


[FEATURE] Cevbreaker

add a cev breaker

This is already a module

This is already a module

what? in which version?

This is already a module

what? in which version?

its called crystalbomber its in combat

i am pretty sure speedmine has a cev setting idk i havent use 3arthh4ack in a while

its a speedmine setting in 1.3.1

This is already a module

what? in which version?

its a speedmine setting in 1.3.1


Add code to count the number of redundant copies in Spandex

This changeset adds functionality to the Spandex log analysis script to report the number of datasets that have more than one copy.

Build failed.

retest this please

This build looks good.


Add supertrait to schema

Following discussion at https://github.com/obi1kenobi/cargo-semver-checks/issues/232, this PR adds a supertrait field to trait in order to enable the new lint on cargo-semver-checks.

Not sure what the workflow is to push the changes to all rustdoc branches, so I just worked on v24, which I had on my machine. Let me know if I should move or cherry-pick these changes to other branches.

I also cherry-picked this PR onto the other format versions we currently support — v23 and v26. I'll make new releases next.

I have queued up new releases for v23, v24, and v26: https://github.com/obi1kenobi/trustfall-rustdoc-adapter/actions
Once they get published, running cargo update in cargo-semver-checks will pick them up automatically and you'll be able to use the new edge to write lints!

I've updated the Trustfall rustdoc playground with the new edge.
Here's a query showing all traits with supertraits + where they are defined in the given crate: link


Protecting ENV

Hey @jorgemanrubia,
I've been trying out a way to prevent direct access to certain ENV values in the console. The gem already takes care of Rails Credentials, so I thought it'd be a nice addition to the gem doing something similar for ENV. If you're up for it, I'd be happy to open a PR so we could work together on this feature.

That's a great idea. Please go for it 🙏


Translation request, we should translate the Asciidoc contributors/Edit this page section of the website

Adding "Table of Contents" to the request as well


CLI: Error when migrating large project

The error:
[SEVERE] Suggestor.generatePatches() threw unexpectedly.
type 'DynamicTypeImpl' is not a subtype of type 'InterfaceType' in type cast

The code:
class ActiveSpeechDisplay extends HookWidget {
  const ActiveSpeechDisplay({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return IgnorePointer(
      child: AnimatedList(
        key: useProvider(speechList),
        shrinkWrap: true,
        reverse: true,
        physics: NeverScrollableScrollPhysics(),
        // Error caused by migrating this line (adding .notifier)
        initialItemCount: useProvider(SpeechItems.provider.notifier).visible.length,
        padding: const EdgeInsets.symmetric(horizontal: 100, vertical: 4),
        itemBuilder: (_, index, animation) => index < 4
            ? SpeechBubble(
                key: context.read(SpeechItems.provider.state)[index].key,
                animation: animation,
                index: index,
              )
            : Container(),
      ),
    );
  }
}

This then ends the migration process, and when I manually upgrade to riverpod 14.x, there is no error - the update the migration tool is making is correct. I could probably get through the migration by temporarily removing this part of my code, but wanted to raise this issue in case it's not a one-off bug.
Even after commenting out the line that causes the issue, even the provider declaration is throwing the same error.
The StateNotifier & Provider (post-migration):
class SpeechItems extends StateNotifier<List<SpeechBubbleModel>> {
  SpeechItems(this._read) : super([]);

  final Reader _read;

  // Error also caused by migrating this line (<SpeechItems> -> <SpeechItems, List<SpeechBubbleModel>>)
  static final provider =
      StateNotifierProvider.autoDispose<SpeechItems, List<SpeechBubbleModel>>((ref) => SpeechItems(ref.read));
  ...
}

I would be happy to provide the full StateNotifier if that could help but omitted that for now as I'm not sure it's relevant (and quite large).

I can take a look later to see if I can reproduce, I doubt I'll need the full StateNotifier.

Edited my original post to show that the provider declaration itself will also throw the same error when given the opportunity. I am also having this same error pop up throughout my project on my StateNotifiers. They are all following a similar pattern, so it makes sense that it would occur for all of them.

Did you make sure that you had all of the version constraints at ^0.13.0? (you should not upgrade the version constraints before running the migration tool). The DynamicTypeImpl error would indicate that the analyzer wasn't able to statically analyze your code to determine all of the types (i.e. there were types that were unable to be inferred etc). I did find an issue related where: context.read(SpeechItems.provider.state)[index].key would not be converted to context.read(SpeechItems.provider)[index].key which I fixed via #441

:facepalm: I had changed to 0.14.0+1, then reverted to 0.13.0+1 (doesn't exist) and pub didn't tell me I was wrong. So, after running it with 0.13.0, it works, although there are many pieces I have to manually migrate. When I have a minute to sit down and go through them I'll share if there's a pattern in what it didn't fix. Thanks for looking into it!

I was able to get everything cleaned up and working. It missed removing .state and adding .notifier in several places, but the weirdest was the following case:
class BuiltTabs extends StateNotifier<List<bool>> {
  BuiltTabs(int numTabs) : super(List<bool>.filled(numTabs, false));

  static final provider =
      StateNotifierProvider.family<BuiltTabs, List<bool>, int>((_, numTabs) => BuiltTabs(numTabs));
}

Where the provider is read, the migration tool wrote:
final shouldBuildTab = context.read(BuiltTabs.provider<BuiltTabs, List<bool>>(navItems.length));

instead of the correct:
final shouldBuildTab = context.read(BuiltTabs.provide(navItems.length));

So it seems it didn't handle the StateNotifierFamily case correctly. My providers are all statically declared members of their respective StateNotifier as well (as can be seen above), so not sure if that could be impacting the tool as well.

Hmm, I have a fix pending for the problems with transitioning from .state -> nothing when using a static provider, and the weird error when it adds type parameters to family accesses.
However, I'm unable to find any errors when it does not add .notifier. Any specific patterns there?

It this still an issue?

It's not blocking me anymore, so I'll close it.


Discrepancies between icefall and sherpa

Following : https://github.com/k2-fsa/icefall/issues/1119

For the same wav file, I got different transcripts between ./build/binsherpa-onnx in sherpa-onnx and ./build/bin/sherpa-online in sherpa. I found that for the streaming flow, sherpa-onnx explicitly sets snip_edges to false.
https://github.com/k2-fsa/sherpa-onnx/blob/bebc1f1398e77b5b1be00893cd1e0f058924d7bd/sherpa-onnx/csrc/features.cc#L44
However, sherpa simply takes the default snip_edges value, which is true.
https://github.com/csukuangfj/kaldifeat/blob/17861dce6bfe0863f47273283c210248e3b38243/kaldifeat/csrc/feature-window.h#L44
I changed sherpa-onnx to use snip_edges = true. After that, sherpa and sherpa-onnx gave me the same transcripts. My questions are

What is the correct values for snip_edges in streaming and offline flows?
Should we make things consistent between sherpa and sherpa-onnx?

I appreciate your help!

Once snip_edges is set to the same, sherpa and sherpa-onnx give the same output for the streaming flow. The transcripts are the same. The joiner outputs are the same as well (except for some 10^-6 rounding errors).

What is the correct values for snip_edges in streaming and offline flows?

Kaldi uses snip_edges=true by default.
In next-gen Kaldi, we are using lhotse to compute features, which hardcodes snip_edges == false.
I think it won't make a big difference about whether to set it to false or true.

Should we make things consistent between sherpa and sherpa-onnx?

Yes, I agree that we should make it consistent so that it makes debugging easier. Could you make a  PR to fix sherpa-onnx so that it uses snip_edges=false? We use false since the model is trained using features computed with snip_edges=false.

@csukuangfj  thanks a lot for sharing the background. More than happy to contribute. sherpa-onnx already sets snip_edges = false. Do you want me to create a PR and fix sherpa (not sherpa-onnx) to use snip_edges = false?

"I think it won't make a big difference about whether to set it to false or true.": @csukuangfj  for the same model, we sometimes observed absolute WER differences of 1 %to 2% between sherpa and sherpa-onnx, which is most probably due to the different snip_edges settings.

Do you want me to create a PR and fix sherpa (not sherpa-onnx) to use snip_edges = false?

Yes, please.

@csukuangfj Awesome, I will create a PR on sherpa then. Thanks a lot!


add section to split datasets from dataset repositories

some items are single datasets, other are repositories containing multiple datasets.
what about having two sections?

"Single dataset"
"Dataset collection" or "Dataset repository" (eg Zenodo, see #10)

Good idea! I assume we can always also deep link into dataset collections, in order to keep pointers to all data sets in a single place. Please issue a pull request.


HBASE-24631 Loosen Dockerfile pinned package versions of the "debian-revision"

After a bit of research into 0 and 1, and a bit of experimentation, it seems we can use a partial wild-card expression for these version strings. Let's try this for now. If it works out, we should expand this usage to all the version package numbers, pinning them to their epic:upstream-version components.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1965/1/console in case of problems.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1965/1/console in case of problems.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1965/1/console in case of problems.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
3m 47s
Docker mode activated.

-0 :warning:
yetus
0m  4s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 24s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  7s
Maven dependency ordering for patch

_ Other Tests _

5m 11s

Subsystem
Report/Notes

Docker
Client=19.03.12 Server=19.03.12 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1965/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1965

Optional Tests

uname
Linux a8e1b080eef7 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 8cdb2cca44

Max. process+thread count
56 (vs. ulimit of 12500)

modules
C:  U:

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1965/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
3m 59s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  0s
Shelldocs was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 23s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  8s
Maven dependency ordering for patch

+1 :green_heart:
hadolint
0m  5s
There were no new hadolint issues.

+1 :green_heart:
shellcheck
0m  0s
There were no new shellcheck issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

Let's try it on master branch.

Thanks @Apache9


hparam_pb2 missing

There is no hparam_pb2 under proto.
Here is the full trace:
ImportError                               Traceback (most recent call last)
<ipython-input-8-14ac9b22f6ac> in <module>
      1 import model_search
      2 from model_search import constants
----> 3 from model_search import single_trainer
      4 from model_search.data import csv_data
      5

~/Projects/model_search/model_search/single_trainer.py in <module>
     15
     16 import kerastuner
---> 17 from model_search import oss_trainer_lib
     18 from model_search import phoenix
     19 from model_search.proto import phoenix_spec_pb2

~/Projects/model_search/model_search/oss_trainer_lib.py in <module>
     26 import kerastuner
     27
---> 28 from model_search import hparam as hp
     29 from model_search import phoenix
     30 from model_search import registry

~/Projects/model_search/model_search/hparam.py in <module>
     24 import re
     25
---> 26 from model_search.proto import hparam_pb2
     27 import six
     28

ImportError: cannot import name 'hparam_pb2' from 'model_search.proto' (unknown location)```

I tried compiling the proto files using command protoc --python_out=./ model_search/proto/phoenix_spec.proto in root directory of the project. It worked.

Building on what away-sreeram is saying, what fixed the issue for me was running the following from the root directory of the project:
protoc --python_out=./ model_search/proto/phoenix_spec.proto protoc --python_out=./ model_search/proto/hparam.proto protoc --python_out=./ model_search/proto/distillation_spec.proto protoc --python_out=./ model_search/proto/ensembling_spec.proto protoc --python_out=./ model_search/proto/transfer_learning_spec.proto

Works well.  Thank you both.

Building on what ajay-sreeram is saying, what fixed the issue for me was running the following from the root directory of the project:
protoc --python_out=./ model_search/proto/phoenix_spec.proto
protoc --python_out=./ model_search/proto/hparam.proto
protoc --python_out=./ model_search/proto/distillation_spec.proto
protoc --python_out=./ model_search/proto/ensembling_spec.proto
protoc --python_out=./ model_search/proto/transfer_learning_spec.proto
I meet the same problem too, and I do understand where to running the the command you mention in your answer, I use the Visual Studio Code IDE and my computer system is windows, can you tell me how to running the command?

Building on what ajay-sreeram is saying, what fixed the issue for me was running the following from the root directory of the project:
protoc --python_out=./ model_search/proto/phoenix_spec.proto
protoc --python_out=./ model_search/proto/hparam.proto
protoc --python_out=./ model_search/proto/distillation_spec.proto
protoc --python_out=./ model_search/proto/ensembling_spec.proto
protoc --python_out=./ model_search/proto/transfer_learning_spec.proto
I meet the same problem too, and I do understand where to running the the command you mention in your answer, I use the Visual Studio Code IDE and my computer system is windows, can you tell me how to running the command?

@dghlnvyps simple solution could be, run these protoc commands in google colab notebook and download the generated .py files and paste into your local directory.

@ajay-sreeram OK，thanks for your help!~ I install protoc compilation tool in my computer and it can compiling the proto files now.


for help

I am very confused if I want to calculate ET using Landsat datas and  your SEBAL codes,because you miss some neccessary guiding files such as README and test_data or a demo_test.I hope I could get your help.Thanks a lot!

README file will be added as soon as possible. The whole SEBAL packages is still under development. To calculate ET based on Landsat you only need the SEBAL/SEBAL folder. Here you will find 3 documents (1 excel and 2 python files). Start filling in the excel. (here you already see some lines filled in as example). If you want to perform a Landsat run you do not need to fill in the VIIRS_PROBAV and additional data sheet in the excel document. In the Run_SEBAL_V.... .py you need to change the path to the excel file in line 10, and the row number of the excel file that you want to determine. in line 12.


Release 1.x (1.4.4)?

Hello,
When will the next 1.x version be released?
Thank you.

We are actively working on 1.4.4 maintenance release that should hopefully be out within the  next month or so. Is there a specific bug fix that you were looking for in this release ?
Also, for future, please use our developer forums for similar questions for clarifications . The GitHub repo is mainly for tracking issues.

Waiting for #994
Thank you.

Thanks for the info on the bug of interest (CC @pasin ).


fix cancelButton doesn't show on 2nd tap

@evollu @michaelhelvey Take a look. This will fix the bug but doesn't need hideCancelButton in #72

I like this. I think it would be good to expose a resignFirstResponder type of method, since it has so much broader applicability, also, but I think this makes the showsCancelButton prop much more intuitive.

@michaelhelvey I don't have much experience with native development. If you want to "expose a resignFirstResponder type of method", you are welcome to create a pull request.

by reading this code
if _jsShowsCancelButton is true
then it will show cancel button on begin editing and won't hide it when end editing (which is my issue in #34 )
if _jsShowsCancelButton is false
then the cancel button will never show
Am I missing anything?

2.15 is pushed to npm


Store combinations that are available into the AvailableCombinationResult

1. Why is this change necessary?
When you use the combination loader you might want to have the combinations that are available -- like the name lets you expect -- you can get them out of there.
2. What does this change do, exactly?
Store every possible options set that is added as well as the original value to read via a getter.
3. Describe each step to reproduce the issue or behaviour.
Try to read the possible combinations without questioning every option set and check whether the hash is in the result.
4. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

A test would be awesome :)

This PR was closed due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://docs.shopware.com/en/shopware-platform-dev-en/contribution/contribution-guideline


How to access the hierarchy and then Three.js changes each object.

Discussed in https://github.com/google/model-viewer/discussions/4712

Originally posted by bigbea1r March 13, 2024
I have a task. Change the model imported into modelviewer and editor it in Three.js. How to do it? I need this in order to make an editor based on modelviewer. I read all the documentation, but found only a few items:
You are also welcome to hack/fork us. You can use mvElement[Object.getOwnPropertySymbols(mvElement).find(e => e.description === 'scene')]; to get access to our internal three.js scene and do whatever you want. And what you link to Post Process Effects
The code currently looks like this:

Editor Table

            View in your space

            Variant:

Текстуры
Высота
Размер

Please don't cross-post.


docker file breaks binder

Thinking about this our ultimate envt will be launched from jupyter hub (i think) to support classes so it may be OK that this build currently breaks binder (still testing it).
The EA envt is really only powerful when attached to some data to work with unless there's a sneaky way to read stuff into memory. probably not ideal for our class.
I believe that a dockerfile pre-empts a yaml envt so that could be why the build is breaking.

actually it's building FURTHER than it has today so it could be the new PR fixed binder @mbjoseph  more to come...

this is working now!! so i will close this issue.


Add an Identity Head weights

Thanks for the good paper, the open code and the models weights! In the paper you present the Identity Head, but I didn't find weights for it, can you release them?

Hi! We decided not to release the weights for the identity head because it deals with human faces and has the potential for misuse. However, you can adapt the codebase for this use case with the following steps:

The training code for the identity head is identical to the appearance similarity head; you just need to replace the DAVIS frames with images and SDEdits from CelebA-HQ in the config. You also need to make sure that you update the folder structure of CelebA-HQ such that the images are grouped by identity (similar to how the DAVIS frames are grouped by video).

The guidance code for the identity head is also the same as the appearance similarity head, but you need to add an additional argument to the edit such that it includes a bounding box for the face, which is used here.


Suggest class name if class name not found

fixes #4475

Nice! Can you show a python code snippet that uses this?

for example sg.create_machine("liblinear"), such exception will be thrown.
---------------------------------------------------------------------------
SystemError                               Traceback (most recent call last)
<ipython-input-2-a1dac361590b> in <module>
----> 1 sg.create_machine("liblinear")

~/shogun/build/src/interfaces/python/shogun.py in _internal_factory(name, *args, **kwargs)
    107     def _internal_factory(name, *args, **kwargs):
    108
--> 109         new_obj = _obj(name, *args)
    110         for k,v in kwargs.items():
    111             new_obj.put(k, v)

SystemError: Class liblinear does not exist. Did you mean LibLinear ?

for example sg.create_machine("liblinear"), such exception will be thrown.
---------------------------------------------------------------------------
SystemError                               Traceback (most recent call last)
<ipython-input-2-a1dac361590b> in <module>
----> 1 sg.create_machine("liblinear")

~/shogun/build/src/interfaces/python/shogun.py in _internal_factory(name, *args, **kwargs)
    107     def _internal_factory(name, *args, **kwargs):
    108
--> 109         new_obj = _obj(name, *args)
    110         for k,v in kwargs.items():
    111             new_obj.put(k, v)

SystemError: Class liblinear does not exist. Did you mean LibLinear ?

Nice! @karlnapf could you check if that error message is sufficient, please?

for example sg.create_machine("liblinear"), such exception will be thrown.
---------------------------------------------------------------------------
SystemError                               Traceback (most recent call last)
<ipython-input-2-a1dac361590b> in <module>
----> 1 sg.create_machine("liblinear")

~/shogun/build/src/interfaces/python/shogun.py in _internal_factory(name, *args, **kwargs)
    107     def _internal_factory(name, *args, **kwargs):
    108
--> 109         new_obj = _obj(name, *args)
    110         for k,v in kwargs.items():
    111             new_obj.put(k, v)

SystemError: Class liblinear does not exist. Did you mean LibLinear ?

Nice! @karlnapf could you check if that error message is sufficient, please?

Love it. I think it is decent. I guess doing something like `"Machine "liblinear\ does not exist. Did you mean "LibLinear""? is hard (adding the type?).

Love it. I think it is decent. I guess doing something like "Machine "liblinear\ does not exist. Did you mean "LibLinear""? is hard (adding the type?). Yes, that should be possible, we just need to overload a struct with a string_viewmember for the name and increate_objectuseT` to lookup the name. Unfortunately I cannot think of a way to automate that part.

template <typename T>
struct name_lookup {
  static constexpr std::string_view name = "SGObject";
};

template <>
struct name_lookup<Machine> {
  static constexpr std::string_view name = "Machine";
};

// get the name
name_lookup<Machine>::name;

@LiuYuHui can I merge this?

@LiuYuHui can I merge this?

yes :D

Thanks, this is great :)


Feature request: Please make the Playground editor embeddable in other web applications

Hi,
I think TypeScript is a great language and I really like the web-based playground editor editor due to its auto-completion/Intellisense features (which also shows method documentation, etc).
Unfortunately the editor doesn't seem to have a license so that it could be used for other web applications, as discussed here: http://typescript.codeplex.com/discussions/401549
Our use case: We are developing a web application where users can type their own Javascripts/Typescripts which extend the server's functionality. When the server runs these scripts, it provides an API object with defined properties and methods.
So, TypeScript would be ideal here because we can write a .d.ts definition file containing the API definition and then present a typescript editor which uses this definition file.
Then, when the users edit their scripts they would get auto-completion with IntelliSense etc.
Currently we are trying to use one of the alternatives mentioned in the above Codeplex discussion that is based on the ACE editor (e.g. the one from basarat), but there seem to be some bugs which don't allow a smooth editor experiene, and its auto-complete feature doesn't show property types, documentation etc.
Would it be possible to release the Playground editor under a license (which wouldn't need to be opensource) that allows to embed it in other application?
Thanks!

The playground is a component that we, the TypeScript team, do not own. The VSCode team would be the authority here. We use it in the playground in the website, but can not licence it for repackaging. We have gotten this request multiple times, and the playground editor with all the intellisense support is an awesome product, and personally I would love to see it in more places.
//CC: @egamma, he might have more comments here.

Hi @mhegazy,
thanks for your reply!
As an alternative if the repacking won't be possible, I could image a variant using an iframe to embed the editor that would also work for us (but I don't know if this also requires a license or something like that).
For example: The typescript editor currently allows to submit userdefined content in the querystring, e.g. http://www.typescriptlang.org/Playground#src=var s%3A string %3D "Hello World!"%3B

Now, if there were a page similar like the one above that would accept parameters submitted in a POST body containing the script content and .d.ts definitions (either the content directly or a CORS-enabled URL reference) and a destination URL, the page could allow to edit the submitted content, and if the user clicks on a Save button, the page would do a POST request to the destination URL, containing the edited typescript and javascript.
That variant would also work for us, because we then could use an iframe in our configuration dialog which redirects to the typescript page. Would something like this be possible?
Thanks!

@T18970237136 it sounds like what you really want isn't the full TS playground, but the editor (the Monaco editor) used by the playground with the TS language service so that you can embed into your site. In addition, you would like to add a .d.ts  for your API. I don't see how this could be covered using the iFrame approach. I don't recommend to go down this path.
My team is currently heads down working on VS Code milestone. We can look into this request in a month or so. The editor actually has API to register an additional .d.ts which isn't exposed in the playground (check out the winjs playground: http://try.buildwinjs.com/playground/)

Hi @egamma
thanks a lot for your reply!

it sounds like what you really want isn't the full TS playground, but the editor (the Monaco editor) used by the playground with the TS language service so that you can embed into your site. In addition, you would like to add a .d.ts for your API.

Yes, that is what we'd want. Sorry for not being clear.

I don't see how this could be covered using the iFrame approach. I don't recommend to go down this path.

Can you elaborate on why you don't see this being covered by an iframe approach?
I thought of using an inline-frame as an alternative if repackaging of the editor code on another (foreign) site won't possible.
In that case, e.g. the typscript page (www.typescriptlang.org) could host a static HTML page similar to the playground, containing the monaco editor. Then, a HTML page from a different host could use that page in an iframe and use window.postMessage() to send commands, e.g. to initialize the editor something like
{
  "command": "ShowContent",
  "editorContent": "if (true) { var s: string = 'Hello World!'; }",
  "tsDefUrls": [
    "https://github.com/Microsoft/TypeScript/blob/master/lib/lib.core.d.ts",
    "https://example.org/myDefinition.d.ts"
  ]
}

The editor HTML page then could listen to MessageEvents to receive the commands, and in case of the above command, load the (CORS-enabled) .d.ts files and show the specified content in the editor.
Then, if the surrounding page wants to get the current editor contents (because it wants to save the state) it could send some "GetState" command, and the iframe page could in turn send a message containing the current typescript and javascript contents of the editor.
This would mean the editor library files wouldn't need to be packaged by our (or some other) site but we still would be able to use the editor in an iframe to let users edit their type-/javascripts on our website.
However, I don't know how licensing would apply here and I would understand if you don't want to host such a page.
If it would be possible to repackage the monaco editor into other sites, I would prefer that approach.
Thanks!

I can see that you can host the Monaco editor inside an iFrame and use postMessage to work (in fact this is his how VSO embeds the Monaco editor). However I don't think this is applicable when embedding the TS Playground, which is a particular application that uses the Monaco editor. In other words the TS Playground does too much for your scenario.
We can look into the repacking or a CDN approach in roughly +1 month

OK, thank you!

@egamma  It would be a huge step in evolving of TS-related public web tools if you'd allow to embed a MonacoEditor (with intellisense and ability to load custom .d.ts) into third-party products.
Eagerly awaiting to hear any progress in this direction!

+1

Voting for this issue to be implemented ASAP.

+1

++

+1

+1

it would be great

+1

@egamma, Any news?

https://github.com/Microsoft/vscode/tree/master/src/vs/editor/browser

By the way, the open sourcing of the whole of VSC is totally awesome!  Hopefully it will help out everyone.

If it is what I'm thinking about - it's awesome!

This is really great! Thanks a lot!
Now, I tried to use the Monaco editor in a simple HTML page just like the playground does, but I'm having problems and don't know how to to it correctly.
I downloaded the vscode repository and invoked tsc in the src directory to compile the TS to JS. Then in that directory I created a HTML page that directory using the TS playground as template:
<!DOCTYPE html>

<meta charset="UTF-8" />

Test Monaco

However, when hosting the directory in IIS Express and opening the HTML page in a browser, I get various JS errors, such as
TypeError: nls is undefined keyCodes.js:432:13
TypeError: nls is undefined severity.js:18:9
TypeError: nls is undefined message.js:7:5
TypeError: nls is undefined actionbar.js:528:9
TypeError: b is undefined css.js:11:5
TypeError: nls is undefined lintRules.js:29:9
TypeError: nls is undefined pluginsRegistry.js:330:17
TypeError: ModesRegistry.Extensions is undefined monarchLexer.js:12:9

I'm a bit lost. Can somebody give me some pointers how to use the editor?
Thank you!

Probably best asked in the VSCode repository, not here @T18970237136.


Automate release process.

For this repository we can conceivably almost entirely automate the release process. We should therefore aim to do that!
This issue tracks thoughts on how we plan to do that.
The script needs to do the following:

Given a changelog, it should place it in the Debian and RPM build files.
It should increment the version number.
It should build source packages and upload to the PPA.
It should build binary debs.
It should build binary RPMs.
It should upload debs and RPMs to release.
It should email notifying completion.

Some other notes:

It should be super defensive: for example, if it sees more than three version numbers in the code it should stop and require manual intervention.
It should be version controlled, ideally in this repository.

Have just completed some improvements here, for the 1.4.2 release.  Improved process is documented at https://docs.google.com/document/d/1YEZLpHsxWEnMxdrYS_8PVENsjTJskDcCH_0Y7EPnmYs/edit#, using some scripts at https://github.com/tigera/process/.
This puts us in a decent state now, so I'm going to close this issue.  Obviously we can still continue to add further increments of automation in the future, as we gain experience.


[Docs] Adds trigger performance examples

Changes
Starts a trigger-customization section and adds prefetch count examples
/kind documentation
Fixes #424

There are some trailing white spaces alert from reviewdog that need to be fixed, apart from that, everything is looking fine

/retest

/retest

/retest

/retest

/needs-rebase

/retest

/lgtm
The only outstanding minor change can be done in a separate PR as well.

/approve
/lgtm


⚠ Bofrost.de has degraded performance

In 678eb21, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2462 ms

Resolved: Bofrost.de performance has improved in 5b92c91 after 16 minutes.


Add CpuTimer to airlift-stats

"stats-" is redundant

"airlift-" is redundant

@martint - @dain  and @electrum  are ok with adding this. But they want a green light from you.

Yeah, go for it.


Uncaught SyntaxError: Unexpected token <

Hey,
After yarn, running yarn start which compile successfully but gives an error in console:
Uncaught SyntaxError: Unexpected token <   (in main.js)
I am using Windows.
Any idea why ?

@BenRoz I use MacBook and this doesn't happen in macOS. But this needs to be fixed in Windows somehow.
Are you able to debug it? Pl see of you can get the logs so we'll be able to debug it further.

I have the same thing. The main.js being bundled by webpack is the same HTML code - that and sometimes it doesn't show up in the sources.

@marwan38 path lib should automatically take care of appending the '/' to 'build'. Also I tried this in my machine. When you replace path: path.resolve(process.cwd(), 'build') with path: path.resolve(process.cwd(), '/build'), npm script start:production is failing.

I just ran everything in a new windows machine and there were no issues. Is someone still seeing this issue or are we good to close this out?


Create rolling release not tied to specific k8s version

I wonder if having releases of cdk8s tied to k8s versioning is the correct way to go for this project. Not much seems to change between versions as the constructs exposed here are mostly on stable resources.
It seems to have the following down sides

manual effort every 6 months to bump, branch, and deprecate
bug / patch changes must be pushed maintained versions
3rd party construct libraries must also branch for cdk8s plus version or re implement these constructs themselves
communicates users can use only if their k8s version is supported

What if there was a "rolling release" of this library that implicitly targeted support of the past 3 releases of k8s and was only concerned about making compatible changes within those?
Since at the end of the day this is just a fancy YAML generator, version specific changes could be implemented by having the constructs have a k8sSyntaxVersion property that defines how the YAML object get written. This could explicitly be set on a per object basis or could be determined via a K8S_SYNTAX_VERSION shell variable that defaults to the most current K8S version.
Having this would decrease project maintenance, reduce versioning confusion, and would make of 3rd party construct libraries more feasible.
My familiarity with the implementation of this project is not incredibly deep so there may be some thing I'm not considering here. Does this seem feasible?

We considered this before coupling the library with a k8s version. The main reason not to have a rolling release were compile time validations.
With a rolling release, there is no way to have compile time error when a user uses a property/resource that doesn't exist in the k8s version they are targeting. Runtime validations are also tricky to implement because we would have to mark every resource/property with the k8s versions it supports. This effectively means that validations are moved to deploy time, which is what we wanted to avoid.
I agree that this poses a burden on project maintenance. We decided to take that "hit" in order to provide a better authoring experience. I also agree that it makes it a little confusing at first to choose which library should one be using, but once that decision is made, and the process is established, we feel the continuous authoring experience is better.

Not much seems to change between versions as the constructs exposed here are mostly on stable resources.

True, but even stable resources add features to it, that might not be available on earlier k8s versions.


CLOUDP-165325: Data Federation and private endpoint Atlas CLI Changes

Proposed changes

Adding generation of Data Federation for kubernetes config generate command.
New flag --dataFederationName has been added to export specified data federation instances, similar to flag --clusterName
Data Federation Private Endpoints will not be included in this due to the inability to identify what resource a private endpoint is connected to.
Jira ticket: CLOUDP-#165325
https://jira.mongodb.org/browse/CLOUDP-165325

Closes #[issue number]
Checklist

[x] I have signed the MongoDB CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added any necessary documentation in document requirements section listed in CONTRIBUTING.md (if appropriate)
[ ] I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
[ ] I have updated test/README.md (if an e2e test has been added)
[ ] I have run make fmt and formatted my code

Further comments

Down to 1 commit, to ease the rebase.

Nope, reverting...

I am not rebasing, rather will apply the exact same changes in this PR to the latest master HEAD.

@gssbzn please take a look when you can:

It is migrated to the new API DSK.
I fixed the test to use the atlas CLI rather than calling the API.
It is rebased (or rather manually rewritten) on a recent change.

evergreen keep-definitions

K8S tests are all green. Merging.


Clean up repeated package import

Signed-off-by: Zechun Chen<EMAIL_ADDRESS>

some job error，reset push re-run job


[feat] Implement the functionality of isMemberOfProfile #5

Added Openzeppelin dependency for role bases access checks
Error struct was not public throwing error for invisible scope
AccessControlComponentEvent is dependent on SRC5Component hence added
Made role as felt256 so that it can expand once unwrapped

Could you please resolve the merge conflicts @Naveenkhasyap


got "undefined" in component

I have got "undefined". I can't find this bug. Please help me.
main.js

import Vue from 'vue'
import App from './App'
import router from './router'

import MultiLanguage from 'vue-multilanguage'
import Language from './lang/language'

Vue.config.productionTip = false

Vue.use(MultiLanguage, Language)

/* eslint-disable no-new */
new Vue({
  el: '#app',
  router,
  components: { App },
  template: '<App/>'
})

router/index.js
import Vue from 'vue'
import Router from 'vue-router'
import Login from '@/components/Login'

Vue.use(Router)

export default new Router({
  routes: [
    {
      path: '/',
      name: 'Login',
      component: Login
    }
  ]
})

compoenents/Login.vue
<template>

</template>

Hi @webexpert1989 ,
You have a demo on the repo. I got it working from there. Try to copy the demo and understand how it works!
Cheers,

Hi @webexpert1989,
Can you post the './lang/language' file? But I see that you use capitalize modifier in v-lang directive, try always in lowercase e.g v-lang.categories.posts , the vue-multilanguage transform all in lowercase for have a simple syntax.

thank you. its working


Query: fix edge cases around Cast/OfType result operator lifting

There are some issues around result operator lifting, specifically Cast and OfType:
1.) cast used as type validation doesn't always throw when it should, e.g.:
from g in ctx.Gears.Cast<Officer>().Cast<Gear>()
select g.Nickname

when run on the client it will throw, because not every gear is an officer. However it works fine on sql sever, even though during QM optimization we turn this to:
from g in ctx.Gears
select ((Gear)((Officer)g)).Nickname

2.) Before we try to lift cast, we handle OfType, by updating the type on EntityQueryable directly, e.g:
from o in (from g in EntityQueryable<Gear> select g).OfType<Officer>()
select o.NumberOfReports

will get converted to:
from o in EntityQueryable<Officer>
select o.NumberOfReports

This however breaks down if Cast is introduced to the mix:
from o in (from g in EntityQueryable<Gear> select g).Cast<Gear>().OfType<Officer>()
select o.NumberOfReports

will fail, because we process OfType first:
from o in (from g in EntityQueryable<Officer> select g).Cast<Gear>()
select o.NumberOfReports

which then gets lifted to:
from o in (from g in EntityQueryable<Officer> select g)
select ((Gear)o).NumberOfReports

which is wrong

2.) can be addressed by removing all Cast operators that come before the given OfType, but that potentially makes 1) worse

@maumar Can you write down some of the ideas we discussed in triage?

We should force client eval on Cast operator to mimic its type asserting functionality. However, some queries inject unnecessary casts, e.g.
from Customer c in ctx.Customers
select c.Name

we should strip casts for those cases (where the cast type is the same as element type of EntityQueryable

would be irrelevant now that we don't have client eval.

2nd case is fixed in 3.1 release.


wipe onboarding server for makers in bad standing

Added hip to remove cn470

also removed the older hip around pricing @abhay

Okay with this being in two separate PRs? But I'm open to bringing both to discussion (both spicy!)