fix: Change nonce to height.

Fixes #104
Technically this might not even be necessary from a functional point of view; it's simply adding a constructor field for the initial height, which will be overwritten on the first update anyways. The nonce check was already <= instead of == +1.
Documentation was updated though to make it clearer.

@evan-forbes @SweeXordious do we care about maintaining the Go code with wrapper updates in this repo, or no?

https://github.com/celestiaorg/quantum-gravity-bridge/issues/104#issuecomment-1087303226
Ah so @SweeXordious made me realize something: data roots should verify the start of the range, not only the end. Issue with only verifying the end is that if the relayer skips relaying an attestation, then relays the next, suddenly the validators have signed off on an invalid attestation. Orchestrators should be completely stateless, so they should be asked to sign over a range, and that range should be what's verified by the contract.

Should be ready for review @SweeXordious @evan-forbes.

Will wait for @evan-forbes to confirm

We should be able to delete peggo very soon, along with all of these failing CI tests


Docs nav enhancements

Some steps to make the docs navigation more user-friendly:

Add aliases to menu items and adjust styling and order
Shrink the search 'input' into a single icon (less deceptive, less space)
Reorganize tutorials and move first tutorial to that section
Experiment with reorganization of extra

WIP and I'm not entirely happy with all of it yet, but I feel like its going somewhere. Especially the organization of extras is challenging.

Thanks for looking at organising the extras sidebar!
I like what you have done with the collapsible categories. Here is my suggestion for how to organise them. It's sort of halfway between the current layout and your new version. I think it's neater to keep hooks etc separate and then add subcategories to the components.
https://github.com/threlte/threlte/assets/14927523/e379a413-229c-43c8-a7ba-4f4d02225cf1

@edsunman nice!  seems pretty organized for the extras sidebar 👍
It can be a tricky challenge. To show the breadth of options, I made a starlight docs variation but extras still feels a bit un-organized in it.
still not sure where we're going with the docs just yet 🤷♂
The ideas are marinating😄

I really like @edsunman's organization suggestion for extras (I'm also a bit biased, I really like how drei organizes extras, which is similar).

Omgosh! I just noticed! GitHub has the same UI problem as us, they've gone with making a collapsed down ... UI component

Might be confusing if we end up with a burger icon and a triple dot icon, but it works 😅

Thanks @edsunman, this is exactly the input I was hoping for since I had no idea on how to organize this! When I get around to it, I'm gonna try and pick up @DefinitelyMaybe's starlight branch and integrate this stuff with it.

I've opened a new pull request for the sidebar categories: https://github.com/threlte/threlte/pull/912
It can either be merged on it's own or added along with other docs changes - or just ignored completely 😂


Typo and Grammar Fix

This PR is to fix typos and grammatical errors.
Pls let me know with any changes .

Check now @thumbimurali


request: Markov changes

Custom emojis aren't showing
Should ignore URLs as those produce buggy output
Should get full message content, i.e embed content included

This should be re-opened considering only 1 of the 3 suggestions were implemented.


Make the status bar ready state date more precise

"last year" that this extension reports is too vague

The other extension gives better approximation

Last deployment was indeed 6 months ago.

This is now available on the marketplace. Please update to 2.3.1.


How can i format numbers with writeFile options?

I'm trying to format number values to currency, I would like to know if is it possible to do that with writeFile() ?
My code:

What I need:

Loop and set the z property to "#,##0.00" (notice it uses the en-US form -- Excel will automatically use the local settings):
const range = xlsx.utils.decode_range(newWorkSheet["!ref"]);
for(var R = 1; R <= range.e.r; ++R) {
  const cell = newWorkSheet["I" + (R+1)];
  if(cell) cell.z = "#,##0.00";
}

@SheetJSDev should i pass range in writeFile?


Could you suggest what is the reason to result like this ?

MOT17-02-DPM.txt

kindly ignore  MOT17-02-DPM.txt, please check the result in the bellow attached file and suggest
MOT17-02-FRCNN.txt


(PUP-6043) Change syntax error "at end of file" to "end of input"

This changes the error message produced when there is a syntax error
because the parser hits the end of its input. When this happens the
parser does not know if the text it is parsing is from a file or from a
string, or from the command line.
Since users found it confusing that the error speaks of "end of file"
when there is no file involved, the best is thought to be naming it "end
of input" as that works for all of the various ways input may be given.

CLA signed by all contributors.


Too many values to unpack when including sample_weights

If sample_weights are included, I think that data has a len() of 3, so I think it should be x, y, sample_weight = data.
My question is how do I properly include the sample_weight in the training step? I tested not using the sample_weights and it gives wrong initial loss.
I followed the tensorflow guide and included the sample weight as such
      loss = self.compiled_loss(y, y_pred, sample_weight=sample_weight, regularization_losses=self.losses)
for the 2 calls to self.compiled_loss in the 2 separate
    with tf.GradientTape() as tape:
blocks. Is this the correct implementation for both class_weights and sample_weights?

It seems to be the correct implementation if you also add the sample_weights to self.compiled_metrics.
I just added support for sample weights to the implemetation but have not tested it exhaustively yet.

cool, thanks.


Write Golang tests for repositories and other files

The bus repository has a test example. Other files can be tested too. Repository example
You don't need write all tests, only one test will help a lot.
Please use the BDD method for the tests.

Can i help with this @rodrigo-brito ?

Yes! Have fun @ibtehalahmed


[Feature] Wandb sweeper for hydra

Is your feature request related to a problem? Please describe.
Currently wandb sweeps can be used alongside Hydra but it's not very convenient.
Most of the times I would like to overwrite only certain parameters in hydra config, by downloading them from wandb sweep server. This is possible, but requires me to manage the overriding logic by myself.
Describe the solution you'd like
It would be great if wandb provided a custom sweeper plugin for hydra, similar to the one that's available there for optuna: https://hydra.cc/docs/next/plugins/optuna_sweeper
This way doing sweeps alongside hydra could be as easy as:
# initialize sweep and automatically override some of the config parameters
python train.py --multirun hparams_search=wandb_sweep.yaml

Additional context
I think supporting sweeps in hydra has been mentioned here recently: https://github.com/wandb/client/issues/1233

Hey @hobogalaxy
Thanks for the feature request
We will look into this!

hey @hobogalaxy, I am currently trying to use hydra sweeps with wandb, but I am facing an issue where when a new run in the sweep starts and the id of the wandb run changes (I do that using the hydra:job.num) , and a new wandb object gets created with this new id, the version attribute  of wandb get set  to the same id as the first run in the sweep and does not change. Thus making it impossible to track separate runs of the sweep in wandb. Wondering if you have some ideas on this problem.

Hi @MohammedAljahdali, I don't really see a reason for setting id of the wandb run to hydra job number? Why not just stay with random ids?
If you want to keep it organized, I usually just come up with different name for each hydra sweep and set it to wandb "group" parameter (so it's easier filter it in UI).
Something like: python train.py  +experiment=experiment_name --multirun model.lr=0.01,0.005 logger.wandb.group="mnist_conv_net_sweep1"

I found a solution to my issue which is calling wandb.finish. As for naming the run, I actually use meaningful name, but also concat the job number at the end to separate the sweep runs from each other. Also, using the group name is great idea. Thank you a lot 🙏🏻

Commenting to express interest in this feature as well. :)

Seeing as others are commenting, I may as well do the same. This would be a useful feature.

Yes, I follow the other comments to express interest.
Right now when using sweeps

I config my sweep yaml flags like "encoder.dim" (fields/sub-fields in my OmegaConf config.yaml)
I then "manually" add arguments for the ArgumentParser in my model using a default OmegaConf config.yaml. Such that the ArgumentParser recognize my custom flags from the sweep and adds new fields automatically if I add new fields in the config file.
when sweeps then calls "python train.py --flags_to_sweep --my_custom_layer.custom_feature I load the default OmegaConf and update any values according to ArgumentParser args and use the updated config object (must treat it as a dict because I now have "." in my argparser arguments) to build the model and train.

This works for any particular model but gets more complicated when I need to compose models using different default config.yaml files.... It may be a bad solution but it works for sweep and to use a OmegaConf yaml file to define my models (omitting composability).
Could there be a way to define that the sweep uses "+" (as done with hydra/OmegaConf) instead of the normal "--" argparse flags? That could perhaps solve the issue as I use it currently... but I guess it could break everything as well... anywhoo
Thanks for a great product.

@ErikEkstedt This feature actually exist, it's hidden in the documentation here
Add this to sweep config:
command:
  - ${env}
  - ${interpreter}
  - ${program}
  - ${args_no_hyphens}

I'd also love to see this happening.An easy way to launch Wandb sweeps from a hydra project would be fantastic.
I'm especially missing the early-stopping feature that wandb sweeps have. Currently supported sweepers in Hydra don't allow that yet (e.g. pruning in Optuna). Wandb sweeps can also use pre-existing knowledge of previous runs to optimize the next sweep.

For posteriority, here are few resources I prepared for users trying to build Hydra + wandb projects.

Report describing basics, known pitfalls, and solutions: https://wandb.ai/adrishd/hydra-example/reports/Configuring-W-B-Projects-with-Hydra--VmlldzoxNTA2MzQw

minimal example of using W&B with hydra: https://github.com/wandb/examples/tree/master/examples/minimal-hydra-example

A hydra plugin integrating W&B sweeper with Hydra sweeper is also in the works. 😄 https://github.com/captain-pool/hydra-wandb-sweeper


Catalog tests failing with astropy 4.1

Using the last version of astropy (4.1, released 3 days ago) we got the catalog tests all failing:
FAILED test_catalogs.py::test_gbm_catalog - AttributeError: 'str' object has no attribute 'decode'
FAILED test_catalogs.py::test_LAT_catalog - AttributeError: 'str' object has no attribute 'decode'
FAILED test_catalogs.py::test_LLE_catalog - AttributeError: 'str' object has no attribute 'decode'
FAILED test_catalogs.py::test_swift_catalog - AttributeError: 'str' object has no attribute 'decode'

The problem seems to be here:
>   new_index = [x.decode("utf-8") for x in pandas_df.index]
E   AttributeError: 'str' object has no attribute 'decode'

../catalogs/VirtualObservatoryCatalog.py:111: AttributeError

I am working on it.

This should be related:
https://github.com/astropy/astropy/pull/9505

@ndilalla This, looks like some changes I made to deal with all these weird UTF-8 strings back when we were first changing to py3. It seems satrapy keeps going back and forth on these things...

I pushed a fix in the fix_test_catalog branch. Now I am going to fix also the installation script.


[WIP] Adding support for nix buildpack in repo2docker

This pull request is a work in progress. I would like feedback from the nix community that it is implemented correctly and has the necessary features.
Demo repo2docker repository: https://gitlab.com/costrouc/nix-binder-example
Why add nix as a buildpack?
Nix would be a great addition to reproducible data science. It is a unique package manager. Some notable features:

100% reproducible environments (pin to exact commit in repository)
both a source and binary package repository
allows customized compilation and version of every package
can run identical environment outside of docker (all linux distros + dawin)
as of now 45,000+ packages
fully declarative environments
packages: python, javascript, julia, R, haskell, perl, and many other languages (some better than others).

Questions needing feedback:
repo2docker questions

Should I use a custom Dockerfile not taking advantage of base?

The reason for this is that nix does not need apt or other package managers. So the smaller the base image the better. Nix handles all dependencies.

Where should NixBuildPack be in ordered list of buildpacks?

As of now NixBuildPack only depends on a default.nix file being present.
nix specific questions

Any shorter way to initialize nix in a docker container and launch with a nix shell?

The definition as of now is quite short but I would like to make it shorter. In general things that I was not happy including nixshell command and nix-channel initialization. This adds about 25 Mb to the image.

Should nix-shell be launched with the --pure option?

Launching with a pure option would make the environment more reproducible by removing all non included packages. Has the downside that some expected tools will not be available.

Anyway to ensure that jupyter is in installed dependencies?

Right now I do not check that jupyter is present.

How to prevent nix-shell from evaluating shellHook twice?

Right now shellHook is evaluated in lines:

nix-shell default.nix --command "command -v jupyter"
ENTRYPOINT ["/usr/bin/nixshell"] when the docker container is run

Should I include the unstable channel in the NIX_PATH?

I have included it for ease of use. However, like in the nix binder example I pin the exact nixpkgs version https://gitlab.com/costrouc/nix-binder-example/blob/master/default.nix#L4-7

Nice work!
re: New base image or not? I would keep the shared base image. We try and have our build packs be composable (you can use the R and conda build pack at the same time) which will be hard to do if they don't share a base image. It also means a lot of the standard things (creating the right user, copying the repo, etc0 are taken care of and we only need to maintain one setup.

Definitely. Yeah I see from a maintainer side that sharing the same dockerfile would be a better approach. From the tests I did the difference is about 100 MB in the final uncompressed docker image. I will make the changes.

In my experience the images that you end up with are measured in GBs, so I think we don't have to feel too bad about inflating them by 100MB.
For maintenance this is a big win. Also I'd imagine that people who use nix might want to also pull in things from other package managers (or vice versa). Say 90% of my packages comes from nix and the rare ones I directly install with pip. Or something like that. This means that maintaining composability is a big thing for users as well.

@betatim I have modified the commit to make it composable with other buildpacks. Looking at the size of the builds the difference with 900 MB with alpine as a base and 1.7 GB for buildpack as the base. That is with installing numpy, scipy, and jupyterlab. The 1.7 GB may not be such a big deal especially since the early layers will match other images.
I have checked that it worked using the following command locally. Please let me know what additionally needs to be done for testing. I have added a nix test that hopefully is done right.
repo2docker https://gitlab.com/costrouc/nix-binder-example

Any thoughts from @minrk or @yuvipanda on adding nix support in general and specifics when it comes to composing it with other ways of specifying dependencies? Unfortunately nix isn't a tool I use or know anyone who uses a lot so trying to gently feel my way around for what is best.

Re: alpine, I would avoid it here. Lots of things don't really behave as expected (many Python packages don't work right on alpine, and Python wheels can't be installed). It's a great base for well-defined service images, but I wouldn't recommend it for anything interactive.

I think it definitely should use the same base image, though I'm not sure about inheriting the default Python env or not. It depends a bit on what is to be installed (i.e. is it the kernel environment). One of the reasons composing works well with the other setups is that the installation mechanisms (conda/pip/julia/r) play nice together (at least if they are done in the right order). Since nix owns its whole install, it's less clear what the right thing to do is:

give nix control over the whole env, or
inherit the notebook server env, and give nix control of the kernel env only

Nix installs all of the dependencies for a package (including things like libc etc.) so technically the base image does not matter to nix. I think that sharing a base image for now is the best path because as of now nix does not have every python package and conda does handle some packages better. In the future I certainly see that a smaller base image could be used.
I have spent some time trying to figure out why the travis test is failing? Is it because of appendix? The issue is that in order to expose all of the nix package I need to create a "virtualenv" by setting the ENTRYPOINT=nix-shell. It is similar to running pipenv shell or source activate myenv.

Ok to summarize the changes that have been made with feedback. The NixBuildPath now passes (for some reason julia keeps failing? Looks like this happens in other PRs). I have intentionally kept this PR as the bare minimum to have nix fully working with repo2docker. This bare minimum should be more than enough for any nix based repository.

based on base BuildPack. I found that when basing off of PythonBuildPack there was no advantage since nix can install conda if it needs it. Using PythonBuildPack increased the docker image size by around 700 MB.
a non pure nix-shell is used for now. This means that the programs installed by the base image are still available.
 With the first 700 MB cachable.
Documentation has been added for how to use nix

I believe that with these changes it is ready for merging pending additional questions.

Fixed merge conflict and small documentation typo. Ready for merge pending any questions that you may have. I've described in the previous comments exactly what was included in this commit.

woooo! thanks for all the work @costrouc

You could deploy it tomorrow if you want to. Deploys are made via a PR to https://github.com/jupyterhub/mybinder.org-deploy/ with a guide here: https://mybinder-sre.readthedocs.io/en/latest/deployment/how.html#updating-repo2docker
We usually batch deploys on Tuesdays and Thursdays so that people can plan to be around for a few hours after the deploy they made to revert/fix side effects.
While I have your attention maybe you have some thoughts on #448.

Thank you! I looked in the PRs and saw how easy it is to update.
I commented here with my thoughts earlier https://github.com/jupyter/repo2docker/pull/448#issuecomment-435901093. I don't expect any changes to the base image to have any effect on nix so I think the change is perfectly fine. It will only affect the resulting docker image size from my perspective.


MurckoScaffold.MakeScaffoldGeneric() has issues with isotopes

Describe the bug
When using MakeScaffoldGeneric(), if the input molecule has explicit isotopes, this is retained in the generic scaffold. Even when the atom is not carbon. This is a less obscure problem than it seems, because e.g. ChEMBL has dozens of molecules in there with explicit isotopes (which is how I discovered this) within the CSK.
To Reproduce
from rdkit import Chem
from rdkit.Chem.Scaffolds import MurckoScaffold

smi1="[14CH2]1CCC1"
smi2="[15NH]1CCC1"
smi3="[235U]1CCC1"
mols=[Chem.MolFromSmiles(smi) for smi in [smi1,smi2,smi3]]

for m in mols:
    csk=MurckoScaffold.MakeScaffoldGeneric(MurckoScaffold.GetScaffoldForMol(m))
    print(Chem.MolToSmiles(csk))

this outputs:
C1C[14CH2]C1
C1C[15CH2]C1
C1C[235CH2]C1

Expected behavior
No isotope flags. Expected output:
C1CCC1
C1CCC1
C1CCC1

Configuration (please complete the following information):

RDKit version:  2023.03.3
OS: Ubuntu 20.04
Python version (if relevant): not relevant
Are you using conda? yes
If you are using conda, which channel did you install the rdkit from? conda-forge
If you are not using conda: how did you install the RDKit? n/a

Additional context
The fix is probably to add atom.SetIsotope(0) after line 40 in the murcko script. https://github.com/rdkit/rdkit/blob/master/rdkit/Chem/Scaffolds/MurckoScaffold.py#L40
Just posting the issue here to make sure this makes sense before doing a pull request.

Agreed that this looks like a bug.
This is something which should be fixed at the C++ level, not in Python. I'm happy to do that if you aren't a c++ developer.


Fix table edit history

Save the correct table state to the history upon edit. Whenever an edit to a
cell is begun, the pre-edit state is now saved to the history so that a
restore is possible (via undo button).

@myliang : what are your thoughts on this PR? Thanks!

@myliang : please let me know if there's anything I can do to assist the review of my active PRs.

I have a simpler fix for this, will update the PR soon accordingly


Built Header and Made it responsive

Libraries installed
Framer motion
React Icons

Fixed #11

i think you forgot the login button

Nice job @DanielAgbeni.

I will see to dat ASAP


64-bit Module

Is there a 64-Bit version of this module?

I tried running the build by updating the Xcode project to arm 64 by following these steps:
http://docs.appcelerator.com/titanium/release-notes/?version=3.5.0.RC#modules
1- Updated Xcode to arm 64 & updated Deploy iOS: 7.0

2- Updated the 1manifestfile to 0.4 (bumped the version of the module) 3- Updatedtitanium.xcconfig` to compile with SDK 3.5.0.GA
However, while running the build I got:
** BUILD FAILED **

The following build commands failed:
	CompileC build/TiPaperFold.build/Release-iphoneos/TiPaperFold.build/Objects-normal/armv7/FoldView.o PaperFold-for-iOS/PaperFold/PaperFold/PaperFold/FoldView.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
	CompileC build/TiPaperFold.build/Release-iphoneos/TiPaperFold.build/Objects-normal/armv7/TouchThroughUIView.o PaperFold-for-iOS/PaperFold/PaperFold/PaperFold/TouchThroughUIView.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
	CompileC build/TiPaperFold.build/Release-iphoneos/TiPaperFold.build/Objects-normal/armv7/JpMsmcTipaperfoldModule.o Classes/JpMsmcTipaperfoldModule.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
	CompileC build/TiPaperFold.build/Release-iphoneos/TiPaperFold.build/Objects-normal/armv7/ShadowView.o PaperFold-for-iOS/PaperFold/PaperFold/PaperFold/ShadowView.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
	CompileC build/TiPaperFold.build/Release-iphoneos/TiPaperFold.build/Objects-normal/armv7/PaperFoldNavigationController.o PaperFold-for-iOS/PaperFold/PaperFold/PaperFold/PaperFoldNavigationController.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
	CompileC build/TiPaperFold.build/Release-iphoneos/TiPaperFold.build/Objects-normal/armv7/MultiFoldView.o PaperFold-for-iOS/PaperFold/PaperFold/PaperFold/MultiFoldView.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
	CompileC build/TiPaperFold.build/Release-iphoneos/TiPaperFold.build/Objects-normal/armv7/PaperFoldView.o PaperFold-for-iOS/PaperFold/PaperFold/PaperFold/PaperFoldView.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler
	CompileC build/TiPaperFold.build/Release-iphoneos/TiPaperFold.build/Objects-normal/armv7/FacingView.o PaperFold-for-iOS/PaperFold/PaperFold/PaperFold/FacingView.m normal armv7 objective-c com.apple.compilers.llvm.clang.1_0.compiler


Remove very long log line from get_examples()

Perhaps isn't the most useful log line now.

Ignoring examples files: ['/home/ubuntu/modal/examples/01_getting_started/pycache/generators.cpython-311.pyc', '/home/ubuntu/modal/examples/01_getting_started/pycache/hello_world.cpython-311.pyc', '/home/ubuntu/modal/examples/02_building_containers/pycache/import_sklearn.cpython-311.pyc', '/home/ubuntu/modal/examples/02_building_containers/urls.txt', '/home/ubuntu/modal/examples/06_gpu_and_ml/pycache/mini_dalle_slackbot.cpython-311.pyc', '/home/ubuntu/modal/examples/06_gpu_and_ml/pycache/mlc_inference.cpython-311.pyc', '/home/ubuntu/modal/examples/06_gpu_and_ml/blender/blender_dockerfile', '/home/ubuntu/modal/examples/06_gpu_and_ml/controlnet/demo_images', '/home/ubuntu/modal/examples/06_gpu_and_ml/dreambooth/pycache', '/home/ubuntu/modal/examples/06_gpu_and_ml/dreambooth/assets', '/home/ubuntu/modal/examples/06_gpu_and_ml/dreambooth/instance_example_urls.txt', '/home/ubuntu/modal/examples/06_gpu_and_ml/embeddings/pycache', '/home/ubuntu/modal/examples/06_gpu_and_ml/embeddings/wikipedia', '/home/ubuntu/modal/examples/06_gpu_and_ml/llm-frontend/index.html', '/home/ubuntu/modal/examples/06_gpu_and_ml/obj_detection_webcam/webcam', '/home/ubuntu/modal/examples/06_gpu_and_ml/openai_whisper/finetuning', '/home/ubuntu/modal/examples/06_gpu_and_ml/openai_whisper/pod_transcriber', '/home/ubuntu/modal/examples/06_gpu_and_ml/openai_whisper/streaming', '/home/ubuntu/modal/examples/06_gpu_and_ml/spam-detect/readme.md', '/home/ubuntu/modal/examples/06_gpu_and_ml/spam-detect/spam_detect', '/home/ubuntu/modal/examples/06_gpu_and_ml/spam-detect/tests', '/home/ubuntu/modal/examples/06_gpu_and_ml/stable_diffusion/demo_images', '/home/ubuntu/modal/examples/06_gpu_and_ml/stable_diffusion/frontend', '/home/ubuntu/modal/examples/06_gpu_and_ml/stable_lm/requirements.txt', '/home/ubuntu/modal/examples/06_gpu_and_ml/text-to-pokemon/README.md', '/home/ubuntu/modal/examples/06_gpu_and_ml/text-to-pokemon/tests', '/home/ubuntu/modal/examples/06_gpu_and_ml/text-to-pokemon/text_to_pokemon', '/home/ubuntu/modal/examples/06_gpu_and_ml/vision_model_training/assets', '/home/ubuntu/modal/examples/07_web_endpoints/pycache/badges.cpython-311.pyc', '/home/ubuntu/modal/examples/07_web_endpoints/chatbot_spa/app.js', '/home/ubuntu/modal/examples/07_web_endpoints/chatbot_spa/index.html', '/home/ubuntu/modal/examples/09_job_queues/doc_ocr_frontend/app.jsx', '/home/ubuntu/modal/examples/09_job_queues/doc_ocr_frontend/index.html', '/home/ubuntu/modal/examples/10_integrations/pycache/algolia_indexer.cpython-311.pyc', '/home/ubuntu/modal/examples/10_integrations/pycache/pyjulia.cpython-311.pyc', '/home/ubuntu/modal/examples/10_integrations/dbt/.gitignore', '/home/ubuntu/modal/examples/10_integrations/dbt/sample_proj_duckdb_s3', '/home/ubuntu/modal/examples/10_integrations/kedro_modal/README.md', '/home/ubuntu/modal/examples/10_integrations/kedro_modal/kedro_modal', '/home/ubuntu/modal/examples/10_integrations/kedro_modal/poetry.lock', '/home/ubuntu/modal/examples/10_integrations/kedro_modal/pyproject.toml', '/home/ubuntu/modal/examples/11_notebooks/basic.ipynb', '/home/ubuntu/modal/examples/misc/pycache/webscraper.cpython-311.pyc']

It's used in synthetic_monitoring in our monorepo, so I would have to remove it from there first 🫠


Add support for service-specific service accounts in place of default in generated kustomize manifests

Is your feature request related to a problem?
Yes - I ran into an issue when deploying multiple service controllers into the same ack-system namespace in an Operator Lifecycle Manager based cluster.
OLM establishes an owner reference on service accounts that are used for installed controllers. When two controllers use the same service account, the original owner reference is removed which breaks OLM's ability to upgrade the controller.
NOTE - this is being patched on the OLM side! https://github.com/operator-framework/operator-lifecycle-manager/pull/2128
However - Kubebuilder has enabled support for non-default service accounts as of v3.0.0:
https://github.com/kubernetes-sigs/kubebuilder/releases/tag/v3.0.0
and, more specifically
https://github.com/kubernetes-sigs/kubebuilder/pull/2070
With that in mind, I think it may still be beneficial to add support for service-specific non-default service accounts.
Describe the solution you'd like
Existing kustomize configuration files that are generated for a controller will configure RBAC in association with the default service account. That is, (Cluster)RoleBindings will, by default, target the default service account, and the controller deployment manifest will render with no serviceAccountName definition, which means that it will get the default service account when applied to a cluster.
Consider scaffolding these with a service account reference relative to the controller that's being generated. E.g. ack-sagemaker-service-account for the SageMaker controller.
To implement, I think the changes would be fairly minimal:

Add a serviceAccountName key with a templated value ack-{{ .ServiceIDClean }}-service-account to https://github.com/aws-controllers-k8s/code-generator/blob/main/templates/config/controller/deployment.yaml.tpl#L55
Replace the subject name from default to ack-{{ .ServiceIDClean }}-service-account in https://github.com/aws-controllers-k8s/code-generator/blob/main/templates/config/rbac/cluster-role-binding.yaml.tpl#L11
Create a net-new template for the service account to create in https://github.com/aws-controllers-k8s/code-generator/tree/main/templates/config/rbac
Add the net-new service-account.yaml file to the https://github.com/aws-controllers-k8s/code-generator/blob/main/templates/config/rbac/kustomization.yaml.tpl.

A note that, as described, there would not be backwards compatibility with the previous default service account. I would assume that backwards compatibility would be a matter of adding a flag for the service account to the relevant generator (which I believe is the controller generator) and then handling the user input or default values. If no backwards compatibility is required, then you should just be able to make the above changes.
Describe alternatives you've considered
I've considered making adjustments to the service account in post-generation scripts, but it overall seems like it would be a bit too flimsy.

chmod-ing this issue number is dangerous!

/remove-lifecycle stale

Double check the below commit, test and verify:
https://github.com/komish/code-generator/commit/75cbebed04a3ade87b955923d9e88fc0d30b07d2

/assign

/lifecycle frozen


[FEAT] An npm package with raw icons for any kind of project

Is your feature request related to a problem? Please describe.
Currently there is no package to use those beautiful icons, the only is to either put them inside the project by hand (annoying) or use the css link (huge)
Describe the solution you'd like
Same as the React package but with much less work to do. Just create a package with all the svg-s like the heroicons package does
Describe alternatives you've considered

either put them inside the project by hand (annoying) or use the css link (huge)

Additional context
None

@paescuj Yes sure! Thank you Pascal!
@gawlk Could you help us to better understand the need? I see that heroicons has 2 npm libraries, react and vue. But I can't see anything like you described.

@lucaburgio Sure ! Here it is https://www.npmjs.com/package/heroicons


Video - Invalid file is downloaded when downloading video

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.43-12
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
**Issue reported by:**Applause - Internal Team
Slack conversation:
Action Performed:

Go to staging.new.expensify.com
Go to any chat
Send a video
Right click on the video > Download

Expected Result:
The video is downloaded
Actual Result:
On web, mweb and desktop app, it downloads invalid file
On Android and iOS app, it shows error when downloading the video
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/8be83132-a19d-4ec2-bdbb-e580d1179ff3
View all open jobs on GitHub

We think that this bug might be related to #vip-vsp
CC @quinthar

finding offending PR

@BhuvaneshPatil Thanks for the proposal. Your RCA is correct. The solution looks good to me.
🎀 👀 🎀 C+ reviewed
Link to proposal

@Krishna2323 Thanks for the proposal. Your RCA is correct. As for the solution it makes sense to extract the file name but I think it's better to just use the one available in attachmentInfo field.

@Krishna2323 Thanks for brining this case up. I think we should cover that one too but I'd prefer to avoid such extraction. Can we instead pass the attachmentInfo through the context values?

@Krishna2323 Thanks for the update. I was checking again and I think we should just update fileDownload util so that we supply the source url without the auth token to FileUtils.getFileName.  Can you look into that approach?

@s77rt I have updated proposal that is easy tom implement without making lot of changes

We should also look into any service changes that have occurred.
Staging vs Prod different html for the video:
Staging:

Prod:

You only download the corrupt file on the context menu picker. The attachment download works fine.
Prod does not have the context menu picker, is the check that renders the download there does not pass based on "isAttachment"
Prod html
   "html": "<video data-expensify-source=\"https:\/\/www.expensify.com\/chat-attachments\/6973500176401041544\/w_4f1f4047c9f81aaeec4c636b644dfae469e0c558.mov\" data-expensify-width=640 data-expensify-height=368 data-expensify-duration=53 data-expensify-thumbnail-url=\"https:\/\/www.expensify.com\/chat-attachments\/6973500176401041544\/w_4f1f4047c9f81aaeec4c636b644dfae469e0c558.jpg\">Before_additional_changes-small.mov<\/video>",                             "text": "Before_additional_changes-small.mov",
staging html
     "html": "<video data-expensify-source=\"https:\/\/staging.expensify.com\/chat-attachments\/5593190361139373290\/w_0bcbf0160801b56e75512a67d0e52a182c4e7d0d.mov\" data-expensify-width=1280 data-expensify-height=732 data-expensify-duration=53 data-expensify-thumbnail-url=\"https:\/\/staging.expensify.com\/chat-attachments\/5593190361139373290\/w_0bcbf0160801b56e75512a67d0e52a182c4e7d0d.jpg\">Before_additional_changes2.mov<\/video>",                             "text": "[Attachment]",

@BhuvaneshPatil Thanks for the update. I still think we should update fileDownload lib or the FileUtils lib since it will also prevent similar bugs from occurring and it should be an easier fix.

@jeremy-croff What we have in staging is a new feature https://github.com/Expensify/App/pull/30829

Proposal
Please re-state the problem that we are trying to solve in this issue.
Video file names are not being handled in multiple areas of web app for this new video support

In context menu picker <- corrupted
in attachment modal <- wrong file name
in video player controls <- corrupted

What is the root cause of that problem?
Our current attachment logic  is only solving for images, and for example uses regex to extract the file name for download on anchor tags. It's giving no fileName back for videos corrupting the download of a video.
What changes do you think we should make in order to solve the problem?

Our report action already has the fileName property on the report actions. We should use this and pass it along for our download action in the context menu picker instead of running the image regex that fails.
We currently pass source from the file to the Attachment modal, We can start passing name for videos instead of the source so that the attachment can grab the name from the modal.
In video player controls, which we need to pass the file name to the VideoPopoverMenuContextProvider

What alternative solutions did you explore? (Optional)

@jeremy-croff Thanks for the proposal. Your RCA is correct. The solution looks same as @Krishna2323's.

@jeremy-croff

In context menu picker <- corrupted
in attachment modal <- wrong file name
in video player controls <- corrupted

Seeing this is a deploy blocker I think it's fine to have videos downloaded with the alternative name (the generated one) as that would probably be an easy fix to deploy.  - Then we can work on the better naming as a follow up

@Krishna2323 Thanks for the update. That's closely what I'm looking for (alternative solution 2).
🎀 👀 🎀 C+ reviewed
Link to proposal

@Krishna2323 Can you raise a PR now?

Proposal
Please re-state the problem that we are trying to solve in this issue.
Invalid video file name
What is the root cause of that problem?
getFileName function is not able to parse the correct filename from the URL
What changes do you think we should make in order to solve the problem?
To fix line number 90 to ensure that the proper file name is obtained from the URL
https://github.com/Expensify/App/blob/539e269ef024f691591904f85ab4072bf4d4c166/src/libs/fileDownload/FileUtils.ts#L89-L90
What alternative solutions did you explore? (Optional)
Video file issue.webm

@askavyblr Thanks for the proposal. Your RCA is correct. The suggested solution is not complete as you are only pointing where the changes should be but not clarifying the needed changes.

@s77rt
I have created a PR please take a look, testing is pending.

@Krishna2323 Thanks for the update. Can we use the fileName in attachmentInfo and pass that instead? Similar to what you proposed in first alternative solution.

@jeremy-croff I think your suggested solution is closest here in achieving the correct file name. Can you elaborate how will we pass fileName to the other components?

@askavyblr Please raise PRs only when assigned

@Krishna2323 Your solution requires extracting the filename from the html. I would prefer if we can relay on one source of truth only that is fileName in attachmentInfo.

@s77rt  I already did this, it requires extracting the filename from the URL

After some internal discussion, we'er going to go with @Krishna2323's Alternative 2 that @s77rt approved earlier for now and come back around and make the back end changes necessary to complete it in full to return the original file name.

Adding the Bug label so we get a BugZero agent.
A wild @dylanexpensify appears…

HELLO!

Let's get this movin!

Sounds like we're just waiting on payments.

Apologies, I've been OOO sick, but getting to this today!

Apologies, paying now!

Ahh no no not bothering me! @Krishna2323 paying literally right now!

Payment summary:

Contributor: @Krishna2323 $500
Contributor+: @s77rt $500

Please apply here asap or request!

@Krishna2323 offer sent!

thank YOU!

@dylanexpensify Applied! Thanks!

woooot!

@s77rt sent offer!!

Accepted!


[Figma] CTA Section/Block

Should this be split into two with a dependency on one?
Add the CTA section component to Figma kit. Remember we are only building variants for xlg, md, and sm.
Design spec
Website
Pre-work: Do we need a separate CTA block story for Figma? Plan was to combine these in code.
Must haves:

[ ] Content should be flexible - the designer should be able to condense or stretch the component and the content should respond. Removing text for example should maintain the right padding automatically through auto layout.
[ ] This component uses a fluid type token for the heading. If this is not created already, create a base to help swap the header type based on breakpoint size. See Content group in the library. This should be tested with for Cards.
[ ] Variant for single button and button group. Use the Expressive button and Button group existing Figma components.
[ ] Variant toggle for border at the bottom
[ ] Variants for Default, with Content item, and with Link list
[ ] Variants to toggle Heading on and off, Copy on and off, and Expressive button on and off.

Resources

C4IBM.com Component checklist is available in the Read Me 👀 page of the Figma library
Bases demo from the Figma Fridays (Episode 23)
Reference the best practices from Figma guild
Info on auto layout

See Figma board for details


Option "Hashtags" doesn't work well with special characters

With the option to "Add hashtags in the content as native tags and replace the #tag with the tag-link." activated, write a blog post containing hashtags with special characters. I tried "#Schlagwörter" and "#Fließtext".
Expected: the complete words should appear as links in the resulting blog post.
Instead, only the prefix up to the special character appears as a link ("#Schlagw", "#Flie"), linking to tags that obviously don't exist.

Can you share an example?
As far as I know is, that most plattforms does not support umlauts (twitter, mastodon, ...)?!?

Sorry, I should have clarified that this bug appears in the Wordpress blog itself after activating the plugin option, not in any resulting posts on other platforms. See image attached.

I assumed that, I wanted to argue that hashtags does not support Umlauts in general. Maybe that might be the problem that it does not show up on mastodon?

The issue described in #182 appears for tags that do not use Umlaute or other special characters. In my various test articles, I used common words such as Hund, Katze, Maus etc., but also random phrases like TestWortEins, TestWortZwei etc. Those tags were not created as hashtags in the content, but separately ("Tags"/"Schlagwörter" functionality of Wordpress).
Here, the issue just exists within a single Wordpress installation and does not extend to any shares of the article. Still, I believe that this plugin causes it by trying to format hashtag-looking strings into links. If this is not possible because the string in question contains special characters, then that specific string should probably not be rewritten at all.

Here, the issue just exists within a single Wordpress installation and does not extend to any shares of the article. Still, I believe that this plugin causes it by trying to format hashtag-looking strings into links. If this is not possible because the string in question contains special characters, then that specific string should probably not be rewritten at all.

That is valid point!

Having a look at the code, I think it might be enough to just change the regex in activitypub.php#L22 to only match full words (or nothing at all), like so:
(?:(?<=\s)|^)#(\w*[A-Za-z_]+\w*$)
I don't know enough about this repo yet, so I'm not sending a PR for this at the time. Let me know if I should.

sure, please send a PR! I am not very good at writing regexes, so I'm happy about any help!


New warnings in zcvt.c and zsync.c

New warning in zcvt.c:
zcvt.c: In function ‘NCZ_stringconvert’:
zcvt.c:337:44: warning: pointer targets in passing argument 2 of ‘NCZ_stringconvert1’ differ in signedness [-Wpointer-sign]
  337 |      if((stat = NCZ_stringconvert1(typeid, src, &str)))
      |                                            ^~~
      |                                            |
      |                                            char *
zcvt.c:238:52: note: expected ‘unsigned char *’ but argument is of type ‘char *’
  238 | NCZ_stringconvert1(nc_type srctype, unsigned char* src, char** strp)
      |                                     ~~~~~~~~~~~~~~~^~~

Also:
zsync.c: In function ‘decodeints’:
zsync.c:1885:38: warning: pointer targets in passing argument 3 of ‘NCZ_convert1’ differ in signedness [-Wpointer-sign]
 1885 |  if((stat = NCZ_convert1(jv,NC_INT64,(char*)&v))) goto done;
      |                                      ^~~~~~~~~
      |                                      |
      |                                      char *
In file included from zincludes.h:57,
                 from zsync.c:6:
zarr.h:33:63: note: expected ‘unsigned char *’ but argument is of type ‘char *’
   33 | extern int NCZ_convert1(NCjson* jsrc, nc_type, unsigned char* memory0);
      |                                                ~~~~~~~~~~~~~~~^~~~~~~

These got fixed at some point.


suffixMeta should be defined from opts/commandline

Same as any other suffix, suffixAcl, etc..

I can work on this if no one else has already started working on it.

I had a draft, but I think I have to start that again!
I remembered this was either not trivial or it had to carry options all over those functions.
The idea is to not create global variable that would make ldnode single instance only.
Let me know (the proxy issue is needed if you want!)

Ok. I'll let you do it because you are more familiar with how the options work after the refactoring.

Perfect!
On Fri, Jul 17, 2015 at 9:01 PM -0700, "Martin Martinez Rivera"<EMAIL_ADDRESS>wrote:
Ok. I'll let you do it because you are more familiar with how the options work after the refactoring.
—
Reply to this email directly or view it on GitHub.

doing in #97


[CONTROLLER/PROMETHEUS] fixes prometheus app index bug

This PR is for:

Server

fixes prometheus app index bug
Steps to reproduce the bug
Changes to fix the bug

init index pool after creating

Affected branches

main

Checklist

[ ] Added unit test to verify the fix.
[ ] Verified eBPF program runs successfully on linux 4.14.x.
[ ] Verified eBPF program runs successfully on linux 4.19.x.
[ ] Verified eBPF program runs successfully on linux 5.2.x.

 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.zhengya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Added Kik's bot store.

Please Merge this on October 1st.
Thanks :)

Noted :smile:

@MatejMecka Can you rebase the pull request?


why gdpp loss always (almost) negative， it can't compute BCE loss

Hi, I was trying your gdpp loss with pytorch version, but when I add gdpp loss on G_loss, the training process crash, because the G_loss(~1.0) + gdpp_loss is negative. So, it can't compute BCE loss .
My question is why is gdpp loss always negative, does it make sense??

Hi,
Sorry for the delay I was on holidays.
If I'm correct you're using the DCGAN mode ? But the BCE isn't computed on G_loss + gdpp_loss, but on the score given by D.
To sum up you have:
G_loss = BCE(D(G), True) + gdpp_loss (+others depending on your config)
D_loss = BCE(D(G), False) + BCE(D(Image ground truth), True) (+others depending on your config)
So having negative values shouldn't make your model crash. Can you show me your configuration file ?

Yes，I'm using DCGAN for check it work, but I'm  failed, I will check my configuration  and reply to you

@Molugan I just following ReadMe step-to-step。

python datasets.py cifar10 $PATH_TO_CIFAR10 -o $OUTPUT_DATASET
python train.py PGAN -c config_cifar10.json --restart -n cifar10 --GDPP true

and I also set _C.GDPP = True in models/trainer/standard_configurations/pggan_config.py
But， when I enable --GDPP configuration, it has runtime error: No compute graph , please using retain_graph=True when you first backward .
I think the error occur in [models/base_GAN.py](https://github.com/facebookresearch/pytorch_GAN_zoo/blob/1a0c76496792d4aa0904eb0afc80ea42c5f73b08/models/base_GAN.py#L261)   because  when G_loss backward the computer graph have been released, so when GDPP config is True, and GDPP backward  the tensor phiGFake in the graph but gradient have been release!!

In https://github.com/facebookresearch/pytorch_GAN_zoo/blob/1a0c76496792d4aa0904eb0afc80ea42c5f73b08/models/base_GAN.py#L257
Can you add the option retain_graph=True  ?

No， the source code have not

retain_graph=True

I just modified GDPP configuration set True for testing GDPP work or not

@Molugan  how to use gdpp-loss???
if I want to use gdpp-loss in DCGAN :
step 1:  compute G_loss
step 2 : G_loss.backward(retain_graph=True)
step 3:  compute gdpp-loss
step 4: gdpp_loss.backward()
Is right?

Why did you update the two of G_loss and gdpp_loss separately??
Iin paper and original tensorflow version code,  the G_loss = G_loss + gdpp_loss and only backward onece G_loss

Doing the backward in two changes does not affect the loss, though it can affect the execution time. In this case it allows a more modular architecture.
Please do not post the same message in different issues.

@Molugan  oh，sorry， but when I try to gdpp loss train there are many problems,  did train with gdpp loss sucessfully and improve performance？

Sorry for the delay, busy weeks with many deadlines.
I had several successful training with gdpp: it should improve the SWD score.

can you show some log for gdpp loss in training , I can not use it train any model.
Error in epoch 19:

RuntimeError: reduce failed to synchronize: cudaErrorAssert: device-side assert triggered

I only train DCGAN with mnist , but it failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [81,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [82,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [83,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [84,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [85,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [86,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [87,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [88,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [89,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [90,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [91,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [92,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [93,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [94,0,0] Assertion `input >= 0. && input <= 1.` failed.
/pytorch/aten/src/THCUNN/BCECriterion.cu:42: Acctype bce_functor<Dtype, Acctype>::operator()(Tuple) [with Tuple = thrust::detail::tuple_of_iterator_references<thrust::device_reference<float>, thrust::device_reference<float>, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type, thrust::null_type>, Dtype = float, Acctype = float]: block: [0,0,0], thread: [95,0,0] Assertion `input >= 0. && input <= 1.` failed.
Traceback (most recent call last):
  File "main.py", line 166, in <module>
    main()
  File "main.py", line 158, in main
    gan.train()
  File "/media/yue/Backup_Data/home_DeepLearning/Zi2Zi/pytorch-generative-model-collections/GAN_GDPP.py", line 198, in train
    D_fake_loss = self.BCE_loss(D_fake, self.y_fake_)
  File "/home/yue/anaconda3/envs/py37/lib/python3.7/site-packages/torch/nn/modules/module.py", line 541, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/yue/anaconda3/envs/py37/lib/python3.7/site-packages/torch/nn/modules/loss.py", line 498, in forward
    return F.binary_cross_entropy(input, target, weight=self.weight, reduction=self.reduction)
  File "/home/yue/anaconda3/envs/py37/lib/python3.7/site-packages/torch/nn/functional.py", line 2065, in binary_cross_entropy
    input, target, weight, reduction_enum)
RuntimeError: reduce failed to synchronize: cudaErrorAssert: device-side assert triggered

Then ,I code is :


[feature request] Add a "DDEV_PROVIDER" env variable that is available to custom commands

Is there an existing issue for this?

[x] I have searched the existing issues

Is your feature request related to a problem?
DDEV allows multiple providers (platform, acquia, local, rsync) and custom ones can also be generated easily.
It would be great to be able to add the upstream provider as an environment variable, so that pulling a DB can be scripted further.
Describe your solution
As per the documentation under https://ddev.readthedocs.io/en/stable/users/extend/custom-commands/#environment-variables-provided I am thinking something like:
config.yaml
upstream_provider=platform

custom "pull" command (.ddev/commands/host/fetchdb):
#!/usr/bin/env bash

## Description: Get latest DB from live site
## Usage: fetchdb
## Example: "ddev fetchdb"

ddev pull $DDEV_UPSTREAM_PROVIDER --skip-files -y

With that, all our projects need one command (fetchdb), changing to another host only needs the env var to be updated and there is no need to re-train your muscle memory. Also, when working in a team with a vast number of projects you don't really have to care about where something is hosted to remember the right command, the project has it described and knows it for you.
Describe alternatives
No response
Additional context
We often run into scenarios where we need scripts to run differently depending on the host. Adding the variable described above would also solve such issues too.
Example:

Pantheon does not allow slashes / in git branch names. Thus, all our Pantheon hosted projects need different scripts when it comes to git branching.

Hi @bserem.
We generally don't want to have too many configs, your use case can be solved with the '.env' file:
$ cat .env | grep DDEV_UPSTREAM_PROVIDER
DDEV_UPSTREAM_PROVIDER=platform

$ cat .ddev/commands/host/fetchdb
#!/usr/bin/env bash

## Description: Get latest DB from live site
## Usage: fetchdb
## Example: "ddev fetchdb"

set -a
source "${DDEV_APPROOT}/.env"
set +a

ddev pull $DDEV_UPSTREAM_PROVIDER --skip-files -y

You are free to choose a different .env location.

@stasadev that's a solid approach! Thanks for the suggestion and the detailed response.
Do you mind if I update the docs page (https://ddev.readthedocs.io/en/stable/users/extend/custom-commands/#environment-variables-provided) with information from here?

@bserem it's not a provided environment variable, so I'm not quite understanding what you would do in the docs.
I think this might be a great candidate for a Stack Overflow question-and-answer or a ddev.com blog though, when you get it going the way you want. Info on how to do a guest blog is at https://github.com/ddev/ddev.com

@rfay I was thinking something like:
If the environment variables provided by default are not enough, you create custom ones and pass them to your scripts:

Create a `.env` file:

   DDEV_UPSTREAM_PROVIDER=platform

And load it where needed. Example below is a custom command:

   #!/usr/bin/env bash

   ## Description: Get latest DB from live site
   ## Usage: fetchdb
   ## Example: "ddev fetchdb"

   set -a
   source "${DDEV_APPROOT}/.env"
   set +a

   dev pull $DDEV_UPSTREAM_PROVIDER --skip-files -y

I don't think it belongs to https://ddev.readthedocs.io/en/stable/users/extend/custom-commands/#environment-variables-provided
Ideally, custom commands should have access to the env variables in .ddev/.env and .ddev/.env.*, this can be done in ddevapp. DockerEnv() if I got it right.
Everything else is simple bash:
set -a
source "${DDEV_APPROOT}/.env"
set +a

Closing, I think it's solved. Happy to continue the conversation here or elsewhere.


Won't build on FreeBSD

I tried to build this project on FreeBSD and after making what I thought were the necessary changes, it still won't link due to the lack of a bulkwalk function in net-snmp. I'm using net-snmp 5.7.3. I can provide the detailed changes and error messages if needed. Have you guys tried building with this version of net-snmp? Just thought I'd drop a note here instead of sending a patch first, based on what I read here: https://github.com/google/cups-connector/wiki/Build-from-source#get-ready

I'm happy that you're trying the connector on FreeBSD. I recently made some changes to build tags that would give you some trouble, but you must have discovered those already.
I regularly test with Ubuntu 14.04, which uses NetSNMP 5.7.2, and the Debian package builds on testing and unstable without incident, those use NetSNMP 5.7.3.
I don't have a FreeBSD machine, and don't have bandwidth to set one up. Feel free to send a PR.

 I'm going to remove the SNMP stuff right now.

SNMP is gone now. Looking forward to your patch; happy to help with any other concerns.

Thanks! I'm working on it now, but ran into an issue with the logging using systemd journal logging. Systemd doesn't exist on FreeBSD but as far as I can tell, the logging relies on the systemd journal logging and at the same time compiles on darwin (Mac OS X), so I'm a bit confused. Suggestions?

Fixed in #199


Return a defined constant rule

start = expression

spaceRuleDef = " "

someExpression = "SOME RANDOM TEXT" { return spaceRuleDef;}

I want to do something like this? Is it possible ? I have one of the rules defined which returns a constant. If there is some rule that evaluates, I want to return this rule defined with constant. How can I achieve this?
One way is creating a javascript constant.

{
  var SPACE_CONSTANT = "";
}

start = expression

spaceRuleDef = " "

someExpression = "SOME RANDOM TEXT" { return SPACE_CONSTANT ;}

But this means I am maintaining two copies of my constant which is not ideal. So is there a way to achieve this ?

Issues are intended for bug reports and feature requests, please ask questions at the Google Group. Thanks!


[🚀esl-traversing-query]: add ability to find closest element

As an ESL consumer, I want to be able to use the 'closest' traversing operation.
Operation definition:
::closet(sel) = ::is(sel) or ::parent(sel)  - find the closest element parent starting and including the current element
Implementation details
esl-utils/dom/traversing should be extended with the findClosest function or ability to include the current element for "sequence finder"-based utils.
##Example

TraversingQuery.first($el, '::closest(div)')
findClosest($el, 'div') or findPrent($el, 'div', /*Include initial*/ true)

Priority: Normal

CC: @NastaLeo

:tada: This issue has been resolved in version 4.0.0-beta.10 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


RTL support

plz add RTL to snackebar :+1:

This library is no longer under development

wwwwwhyyyy?

Read the first line of the README. Use the snack bar included with the design support library.


Add Dockerfile for a fips server that will be used by e2e tests

What does this PR do?
It adds a Docker image that will help us testing FIPS encryption on the Agent
Which scenarios this will impact?
Motivation
Additional Notes

/merge


feat adding async_kfold_validation notebook

first notebook added to official folder

Hi @stevewalker-de - great notebook - I'm happy to say it's passing the tests! A few things:

I notice this isn't following the template, and doesn't contain auth instructions, a license header, the nice buttons for opening in Colab, etc. It might be helpful to go back to the original template and add some things back. (I would edit your notebook in Colab.)
For comments that are more like instructions I would create markdown cells instead of writing longer comments in Python. The template is also pretty helpful as a model for providing a balance of md-to-code blocks.


chore: kubernetes_service_account_v1 secret always show diff

issue: https://github.com/hashicorp/terraform-provider-kubernetes/issues/1043#issuecomment-1134245891
What does this PR do?
🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

Motivation

More

[x] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[ ] Yes, I have added a new example under examples to support my PR
[ ] Yes, I have created another PR for add-ons under add-ons repo (if applicable)
[ ] Yes, I have updated the docs for this feature
[x] Yes, I ran pre-commit run -a with this PR

Note: Not all the PRs required examples and docs except a new pattern or add-on added.
For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

@kahirokunn thanks for the PR. Can you please open an issue for tracking and add steps to reproduce as well as results of your testing after applying this fix?

I forgot. I will do it.

Late

closing for now since we need to understand if there are any consequences to ignoring these changes long term


added heroku-client

Codecov Report

Merging #4 into master will increase coverage by 9.52%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   88.88%   98.41%   +9.52%
==========================================
  Files           1        3       +2
  Lines           9       63      +54
  Branches        5       23      +18
==========================================
+ Hits            8       62      +54
  Misses          1        1

Impacted Files
Coverage Δ

src/command.js
100% <100%> (ø)

src/api_client.js
100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7f0630...8dcecc2. Read the comment docs.


Crash when opening big text file (800MB)

I use VS code to read logs. When I try to open big log file (~800MB text file) VS code crashes. I use few extensions to work with log (highlight-words, filter line, Bookmarks). I trie to set colorizing from .log to Plain Text, but didn't help.
Split file to multiple 500MB works
Let me know, if you need more information.

CPUs
Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz (8 x 2904)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.86GB (5.80GB free)

Extension
Author (truncated)
Version

Bookmarks
ale
10.5.0

code-gnu-global
aus
0.2.2

filter-line
eve
1.3.2

cpptools
ms-
0.25.1

highlight-words
rsb
0.1.4

@michalzemko
What's your value for max memory?

Do you get any notification to restart VS Code with your configured max memory?

I open files of 2GB without any issue.
What happens when you disable your extensions?

/needsMoreInfo

When I've disabled all extensions, VS crashes again. Only "The window has crashed"dialog is shown

No aditional dialgo about seting max memory is shown.
When I checked "editor.largeFileOptimizations", VS didn't crash.
When I set "files.maxMemoryForLargeFilesMB" to 8192 or to  16384 VS crashed.

When I checked "editor.largeFileOptimizations", VS didn't crash.

@michalzemko If you plan to open huge files, then please configure "editor.largeFileOptimizations": true (the default).
That is the purpose of the setting, if you configure "editor.largeFileOptimizations": false, then the editor will burn memory like crazy to give you features such as word wrapping and folding for any file size... That will lead to an out-of-memory crash.


config-reloader throttling of CPU in vmagent

`Status: FIRING 🔥
Annotations:

summary: Processes experience elevated CPU throttling.
description: 33.33% throttling of CPU in namespace monitoring for container config-reloader in pod vmagent-***.
Details:
container: config-reloader
`

When deploying VMAgent using vm-operator, it is currently not possible to specify the resource limits for the config-reloader, which often leads to CPU throttling alerts for VMAgent. Is there a plan to improve and add support for this in the future?

Hello,
It's possible to globally rise limit per CRD object with env variables for operator:
VM_VMALERTDEFAULT_CONFIGRELOADERCPU  100m
VM_VMAGENTDEFAULT_CONFIGRELOADERCPU 100m
VM_VMAUTHDEFAULT_CONFIGRELOADERCPU 100m
VM_VMALERTMANAGER_CONFIGRELOADERCPU 100m
https://docs.victoriametrics.com/operator/vars/
Or patch value directly per CRD object:
kind: VMAgent
spec:
  containers:
  - name: config-reloader
     resources:
       limits:
          cpu: 200m

Hello,
It's possible to globally rise limit per CRD object with env variables for operator: VM_VMALERTDEFAULT_CONFIGRELOADERCPU 100m VM_VMAGENTDEFAULT_CONFIGRELOADERCPU 100m VM_VMAUTHDEFAULT_CONFIGRELOADERCPU 100m VM_VMALERTMANAGER_CONFIGRELOADERCPU 100m https://docs.victoriametrics.com/operator/vars/
Or patch value directly per CRD object:
kind: VMAgent
spec:
  containers:
  - name: config-reloader
     resources:
       limits:
          cpu: 200m

got it, thank you.


Get commit delta from HEAD

This should make @politas a little happier. I thought we'd updated this build script at the same time as netkan's. Apparently not!
Credit to @pjf for his git-fu.

If I'm understanding this right, the command as it was is comparing the current master of the repo to ... nothing? and thus our list of "changes to check" is the entire repo. And this will change it so that it compares the current master branch to Jenkin's previously synced HEAD, and will thus get only the most recently changed files. Have I got that right? I'm still finding git commands to be quite opaque.
What is the difference between these two commands?
git diff --diff-filter=AM --name-only --stat origin/master...HEAD
git diff --diff-filter=AM --name-only --stat origin/master HEAD

@pjf explains it best over in KSP-CKAN/NetKAN#2430

It's a safe well tested change, that will save lots of hair pulling and jenkins a lot of unnecessary work.


[BUG] Bugs that really hinder my experience

Describe the bug
1- Excessive use of processing power even with the program closed, having to close it through the task manager to be able to finish it.
2- The application freezes several times during use (I use a ryzen processor).
To Reproduce
1- Close the app and see it on the task manager
2- Just using it
Expected behavior
1- Have a button in the settings to disable background use
2- Don't Freeze LOL
Screenshots
1-
https://github.com/hydralauncher/hydra/assets/168583247/bcae4e70-b48b-4e78-b688-5d53da9aa671
2-
https://github.com/hydralauncher/hydra/assets/168583247/2cb5607d-6744-4105-a1e7-b2025445c3d2
Additional context

OS: Windows 11 Enterprise 😔
Hydra Version: 1.1.0

20% of my processor don't help me 😅😅

Knowns issues. Fix on next update

Please, see releases page for new update

Thanks👍👍


Incorrect Formula for the Derivative

In class 8, "Finding the Max/Min of a Function" there's an incorrect formulation of the derivative.
In the left-hand side of the equation, the change in x is given as the numerator, yet after the equals sign it is then in the denominator.
I think that's a mistake and it should be flipped. It looks like this currently:

@UPstartDeveloper - thanks for catching this mistake! We've updated it in our latest commit


Sharing - can share with yourself.

Modifying your own permissions on a narrative that's been shared with you is probably a good use case, but you can lock yourself out of a narrative you own (or, more likely, make a strange error happen).

you can lock yourself out of a narrative you own

This shouldn't be possible at the workspace level - are you sure? You can lock yourself out of a narrative you can admin, write to, or read, but removing your own ownership shouldn't work.

I didn't check too hard. That calls set_permissions, so you can probably still cause problems. Either way, it shouldn't do that.

It looks like trying to modify your own permissions when you're the owner is a noop. Maybe it should throw an error?
ipython
Python 3.6.9 (default, Apr 18 2020, 01:56:04)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.4.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: from biokbase.workspace.client import Workspace

In [2]: with open('/home/crushingismybusiness/.kbase_token_ci') as f:
   ...:     token = f.read().strip()
   ...:

In [3]: ws = Workspace('https://ci.kbase.us/services/ws', token=token)

In [4]: ws.list_workspace_info({'owners': ['gaprice']})
Out[4]:
*snip*

[50118,
  'gaprice:narrative_1588989318474',
  'gaprice',
  '2020-05-11T01:36:10+0000',
  8,
  'a',
  'n',
  'unlocked',
  {'cell_count': '1',
   'narrative_nice_name': '[redacted]',
   'searchtags': 'narrative',
   'is_temporary': 'false',
   'narrative': '1'}]]

In [5]: ws.set_permissions({'new_permission': 'n', 'id': 50118, 'users': ['gapri
   ...: ce']})

In [6]: ws.get_workspace_info({'id': 50118})
Out[6]:
[50118,
 'gaprice:narrative_1588989318474',
 'gaprice',
 '2020-05-11T01:36:10+0000',
 8,
 'a',
 'n',
 'unlocked',
 {'cell_count': '1',
  'narrative_nice_name': 'Because an actual narrative is too complicated for Boris :P',
  'searchtags': 'narrative',
  'is_temporary': 'false',
  'narrative': '1'}]

In [9]: ws.get_permissions_mass({'workspaces': [{'id': 50118}]})
Out[9]: {'perms': [{'boris': 'r', 'gaprice': 'a', 'bsadkhin': 'r'}]}

It looks like trying to modify your own permissions when you're the owner of a workspace is a noop. Maybe it should throw an error?
ipython
Python 3.6.9 (default, Apr 18 2020, 01:56:04)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.4.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: from biokbase.workspace.client import Workspace

In [2]: with open('/home/crushingismybusiness/.kbase_token_ci') as f:
   ...:     token = f.read().strip()
   ...:

In [3]: ws = Workspace('https://ci.kbase.us/services/ws', token=token)

In [4]: ws.list_workspace_info({'owners': ['gaprice']})
Out[4]:
*snip*

[50118,
  'gaprice:narrative_1588989318474',
  'gaprice',
  '2020-05-11T01:36:10+0000',
  8,
  'a',
  'n',
  'unlocked',
  {'cell_count': '1',
   'narrative_nice_name': '[redacted]',
   'searchtags': 'narrative',
   'is_temporary': 'false',
   'narrative': '1'}]]

In [5]: ws.set_permissions({'new_permission': 'n', 'id': 50118, 'users': ['gapri
   ...: ce']})

In [6]: ws.get_workspace_info({'id': 50118})
Out[6]:
[50118,
 'gaprice:narrative_1588989318474',
 'gaprice',
 '2020-05-11T01:36:10+0000',
 8,
 'a',
 'n',
 'unlocked',
 {'cell_count': '1',
  'narrative_nice_name': '[redacted]',
  'searchtags': 'narrative',
  'is_temporary': 'false',
  'narrative': '1'}]

In [9]: ws.get_permissions_mass({'workspaces': [{'id': 50118}]})
Out[9]: {'perms': [{'boris': 'r', 'gaprice': 'a', 'bsadkhin': 'r'}]}


I was trying to call a function from cpp, without using cmake, can you please help

Uploading JNITest.zip…

the code is still uploading for 3 hours, might be something wrong?

JNITest.zip

the difference is the FindClass(), the change "." with "/" for class name.  other changes are minor.     https://gist.github.com/ggfan/6ccef9ac609d400e36504134ce0257a7
get GlobalReference portion is commented out in this case since it is still valid; creating a globalReference before using it would be safer

thanks it is working


Add debugging log

We need to be able to see the Asciidoc that the plugin generates, for debugging purposes: it is very difficult otherwise to work out why Lutaml is generating the output it does.
There shall be a log of all Asciidoc generated by the plugin in a compilation, {filename}.lutaml.log.txt . I am hoping you can retrieve the file name from the plugin: you do see the Asciidoc document and Asciidoc nodes in the plugin for preprocessing, and you will get the parent document filename from node.document.attr("docfile")
Each instance of Asciidoc generated in the current execution shall be output to the log, and prefixed with a header.
That header should have some information about where the plugin was invoked, although the plugin may not know much.
Metanorma::Utils::Log is in my opinion appropriate scaffolding for this, although it is set up to output HTML: https://github.com/metanorma/metanorma-utils/blob/main/lib/utils/log.rb#L40 . I don't think you should use it as is: HTML (which we've recently introduced) will make a mess of Asciidoc sourcecode.
But the function Metanorma::Utils::Log.new.current_location(node), where node is an Asciidoctor node, will return either an Asciidoctor line number, or the nearest clause heading. (Lutaml is a block, so I expect the former.) I think, given that we break Asciidoctor into include files, the nearest clause heading is preferable: https://github.com/metanorma/metanorma-utils/blob/2e4d54318efb20367c78442b7a58c94d81e62415/lib/utils/log.rb#L59C9-L66C14 .
And I think that is the only location guaranteed to work: I'm not getting a random paragraph in Asciidoctor giving useful data for node.lineno or node.source_location or node.xreftext : https://www.rubydoc.info/gems/asciidoctor/Asciidoctor/Block

Not waiting, this is become a little too pressing...

This is a preprocessor, so the per-node log advice given above won't work anyway, need to dump the result as one big Asciidoc file.

Logfile (= populated Asciidoc after Lutaml Preprocessor runs, but only if it finds lutaml or lutaml-express blocks in the document) is output to {filename.adoc}.lutaml.log.txt

Successful, and quite revealing: I can immediately see errors...


FederatedOperationHandler Retains information from previous operation.

Describe the bug
The FederatedOperationHandler has retains a field called graphs and context which persists for the life of the Handler within the FederatedStore, retaining information from the previous execution.
Expected behaviour
Handler should not retain state from the previous execution.

Same issue happens with the context field within this class. But that was introduced earlier.

A potential solution is to just pass the graphs in to getAllGraphResults and mergeResults as parameters, and not keep them around at all between runs.


validation: check genesis predeploys against standard config

Predeploy spec: https://specs.optimism.io/protocol/predeploys.html
Standard config spec states

[Genesis state should have] Only standard predeploys and preinstalls, no additional state.

https://github.com/ethereum-optimism/superchain-registry/pull/373 is a sketch at a solution for this.

Q: with interoperable OP is it problematic if a chain enables the governance token on their chain?
A1: This OP token thing — there should be nothing deployed I think
A2: (Possibly) EAS predeploys — idk if chains other than OP Mainnet and Base have them. if they do, great, if not, I don’t think we need to require,

Actually here is the source, it is also in the specs https://github.com/ethereum-optimism/optimism/blob/d510910381d99660295f99e4eccd553df5829df6/op-e2e/actions/fjord_fork_test.go#L24-L29

Here are some results investigating the distribution of codehashes across the superchain registry. (For each predeploy implementation address, it shows a number of codehashes, each with a list of chains which use that implementation).
https://github.com/ethereum-optimism/superchain-registry/blob/3ff4a2d6e69d248c36a9a1fee8a9b7df461d7710/validation/results.toml

We should check with @blmalone about his approach for this check.

We are planning to leverage @blmalone script. The work for this ticket will be to make sure the script is fully automated and integrate it into the superchain-registry CI

Latest plan here https://www.notion.so/oplabs/L2-Genesis-Predeploys-Verification-c1c6f0b5c38d406bbfe09854653dade4?pvs=4#fbe8b779b04d46b1b7d5b80f9c6f506e leaning on runbook here https://www.notion.so/oplabs/BHIC-L2-Genesis-Predeploy-Verification-Metal-Mode-Zora-f42ecf4dd0164b4d9303b933d9e98bf6#0a94fd37ca37474d998ab1668e655264

We should take a look at the genesis tooling, and “save” any important metadata into a file which (along with the deploy-config.json) gets tracked in the SCR. The principle is we want to be able to run the tool in the future to produce the identical result.

just to increase visibility, I'm sharing a document with the current document for this implementation: https://www.notion.so/oplabs/Genesis-validation-update-f453b1b676d440efb0ac43adc05cab2f - we are thinking that approach 2 is what we will do to get this check done.


IntTypes.h unknown type name

I just installed the latest (0.2 161130 (November 30, 2016) ) version of the bridge.
After I try import our open source Groestlwallet iOS project (https://github.com/Groestlcoin/groestlwallet) to UWP app using vsimporter, everything looks ok. But when I try build and run the UWP project, there are 26 errors show in my window:

Attached the build log for the context of what is happening:
breadwallet.zip

(Option 1) Fix: Rename Groestlwallet's IntTypes.h file to something else. (Recommended)
inttypes.h is a system header that is included from WinObjC's CoreFoundation.h header. In this case, CoreFoundation.h seems to be including Groestlwallet's IntTypes.h instead of the system header.
(Option 2) Conversely, a potential SDK change would be to modify the sbclang.targets file to use -iquote instead of -I for project header maps. However, this produces breaking changes for other projects.

**The change below for sbclang.targets is not needed if you have renamed IntTypes.h**
<ItemGroup>
  <ClangCompile>
    <HeaderMapInclude Condition="'%(ClangCompile.HeaderMap)' == 'Project'">-iquote"$(ProjectHMapFilename).hmap"</HeaderMapInclude>
    <HeaderMapInclude Condition="'%(ClangCompile.HeaderMap)' == 'Combined'">-I"$(CombinedHMapFilename).hmap"</HeaderMapInclude>
  </ClangCompile>
</ItemGroup>

Thank you for your kind reply.
I went for option 1 and renamed IntTypes.h to something else. I am getting now other errors:

Are you able to build the application by renaming IntTypes.h to something else?

Any updates on this?

@gruve-p I apologize for the long wait.
I was not able to reproduce the exact error you are seeing, but I did hit something similar while it compiled BRAppDelegate.m. It said that "cannot find protocol declaration for 'PKPushRegistryDelegate'" followed by and error saying "expected a type".
There have been quite a few changes since Feb 17th. It looks like the project you are trying out still has some errors during build, you can re-import the project again to see where it's at.
Since the initial question in this issue has been answered, I'm going to close this one out. You are welcome to open a new issue with further questions.


Include an HTML minification option for the HTML loader

It would be nice to have htmlmin on the html loader so imported templates could have whitespace removed, etc.

FuseBox v4 Major Upgrade
As of today, the latest major version of FuseBox v4 is available in fuse-box@next. FuseBox has undergone a full re-write from scratch, learning from its mistakes in the previous versions.
There are a lot of improvements, speed and config wise.
Get started here
We are still working on the documentation and polishing before the major release. We are getting very close. And we need help testing.
npm install fuse-box@next

Most of the issues that I am closing have been resolved, however, if you find that your issue is still relevant in v4 don't hesitate to re-open or (better) create a new one with v4 configuration.


Results not show all gists

Extension Github Gist – Title
Author: @koinzhang
When I search for a gist show results for the newer gists, older gist are not showed. Maybe there are a limit?
If I do the same search with https://github.com/Gisto/Gisto I get all results.

Hi @koinzhang,
Thank you for the replay.
The problem it is not about the limit of displayed results. The problem is that seems the search is not performed over all the gists. For exemple:
Search: laravel
Restult: 2 gists
But I know that I have older gists that are not shown in the results.

Hi @6ui11em. I understand what you mean.
The mechanism of how this command works is to show you 30 gist and then search through those 30 gist.
If it is not shown, it will not be searched.
Next version I'll add a new setting item that lets you choose the maximum number(30-100)  of gist you can show at a time.
And allow you to load the next page of Gist when the last Gitst is selected.

Hi @koinzhang ,
This will be a great upgrade!
Thank you!

Hi @6ui11em
Github Gist update has been released with the following new features:

Update Preference options: you can choose the number(30-100) of Gist loaded per page.
Add “Load More”: When you select the last item in the list, the next page of Gist will be loaded automatically.
Add “Open in Ray.so” action: Create beautiful images of your gist code using ray.so.


Fixed a broken link to another library page.

This should be fixed by this, thanks for the fix though!


Router does not trigger loader or beforeLoad when navigating to same url

Describe the bug
Hi
I have a search form view which contains some search options as html inputs and a search button. It works like this:
I press "Search" button, which does nothing but programmatically navigates to the current path + also incudes some query params. This causes the load and beforeLoad functions of the current route to trigger, looking for the updated search parameters and performing the search.
If I now press the "Search" button again, without making any changes to the  search parameters, it programmatically navigates to the current url again. This time, load and beforeLoad functions are not executed. It seems that these functions are not executed if the url of the page has not changed.
There should be away to make these function trigger even if the navigation occurs to the same url. Otherwise, I cannot perform the same search again.
I tried including shouldReload: true option for the route, but it did not help.
This worked well in 0.0.1-beta.185 but not in 1.19.4

please provide a minimal complete example, e.g. by forking one of the existing examples on stackblitz

I investigated my situtation even further and I think that previously loader and beforeLoad were called due to the fact that I was using router.invalidate() as a part of the "Search" button click. If I remove it, the functions are not triggered if url is not changed after the button press.
I agree that giving a minimal working example would be beneficial in any situtation when trying to resolve bugs. Unfortunately, I am quite busy right know as I have already spent a whole work week trying to upgrade the router library. :D

I was able to reproduce this.
Bug was introduced in 1.18.4.
Reproducer:

click on "open preview in new tab"
open developer tools to see console logs
click on "search"
click on "click me".

shouldReload, loaderand beforeLoad should log each time "click me" is clicked.
working in 1.18.3: https://stackblitz.com/edit/tanstack-router-acssx4?file=src%2Froutes%2Fsearch.tsx
not working in 1.18.4: https://stackblitz.com/edit/tanstack-router-hfpfdi?file=src%2Froutes%2Fsearch.tsx%3AL24

Great to hear you were able to reproduce the bug! Hopefully you will be able to fix it. :)

any known workarounds ? or an update for a fix?

@mta-trackunit
Not sure if this is at all related but I kept getting the white screen due to what I think is some race condition when attempting to run a beforeLoad function while updating the query params. This is my solution:
export const Route = createFileRoute('/_protected')({
  component: ProtectedLayout,
  beforeLoad: async ({ context, navigate, location }) => {
    const validated = await context.auth.validate();
    console.log('validated', validated);
    if (!validated) {
      console.log('redirecting');
      throw redirect({
        to: '/',
        search: {
          // Use the current location to power a redirect after login
          // (Do not use `router.state.resolvedLocation` as it can
          // potentially lag behind the actual current location)
          redirect: location.href
        }
      });
    } else {
      // Work around a bug in the router where the location is not updated
      navigate({
        to: location.href
      });
    }
  }
});

Also, not even sure how to access router within beforeLoad and nothing in docs tells us where to import it from.

shouldReload, loaderand beforeLoad should log each time "click me" is clicked.

Retesting this with the 1.32.12 release, the console logs get logged out on every click.
Retest: https://stackblitz.com/edit/tanstack-router-mvbt6c?file=src%2Froutes%2Fsearch.tsx

@SeanCassiere No, the issue that I have is that I have some tabs that update the URL with a search parameter to persist the view but it seems like some exception is thrown or there is a race condition when beforeLoad runs executing my async validation function. It causes a white screen which requires a manual refresh to see the page. The only way I got around it was by force navigating but I thought I wouldn't have to do that.

@SeanCassiere No, the issue that I have is that I have some tabs that update the URL with a search parameter to persist the view but it seems like some exception is thrown or there is a race condition when beforeLoad runs executing my async validation function. It causes a white screen which requires a manual refresh to see the page. The only way I got around it was by force navigating but I thought I wouldn't have to do that.

@rajbirjohar will need a reproduction for  this.

as noted by @SeanCassiere  this was already fixed


Install using angular CLI failing

Having created a new angular project using ng new clarity-test, upon running ng add @clr/angular as in the instructions npm generates the message

Cannot read property 'build' of undefined

Using angular v6.1.0, clarity v0.13, node v10.5.0 and npm v6.1.0. for the moment I've therefore dropped back to manually adding clarity.

I just tested without any issues, can you give me the exact commands you used and the resulting output? It helps to use the bug report issue to capture things in an easy format.

Same error for me
Command:
ng add @clr/angular
Installing packages for tooling via npm.
Cannot read property 'build' of undefined

Version:
ng -v

Angular CLI: 6.2.1
Node: 8.11.3
OS: darwin x64
Angular: 6.1.7
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.8.1
@angular-devkit/build-angular     0.8.1
@angular-devkit/build-optimizer   0.8.1
@angular-devkit/build-webpack     0.8.1
@angular-devkit/core              0.8.1
@angular-devkit/schematics        0.8.1
@angular/cli                      6.2.1
@ngtools/webpack                  6.2.1
@schematics/angular               0.8.1
@schematics/update                0.8.1
rxjs                              6.2.2
typescript                        2.9.2
webpack                           4.18.0

Ok confirmed when I use the CLI 6.2.0+, it appears Angular CLI 6.2.x has a breaking change in the angular.json file format and I have been testing on CLI 6.1.x.
The workaround is to follow the installation guide on the website for now, and I'll get a fix asap.

This looks related, but says its for 7.0 https://github.com/angular/angular-cli/commit/fd7bcd25fe1af58bf25c22968852aa2cbdb4dde1

Probably a bit late now but here's my report for ng -v too
Angular CLI: 6.2.1
Node: 10.5.0
OS: win32 x64
Angular: 6.1.7
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.8.1
@angular-devkit/build-angular     0.8.1
@angular-devkit/build-optimizer   0.8.1
@angular-devkit/build-webpack     0.8.1
@angular-devkit/core              0.8.1
@angular-devkit/schematics        0.8.1
@angular/cli                      6.2.1
@ngtools/webpack                  6.2.1
@schematics/angular               0.8.1
@schematics/update                0.8.1
rxjs                              6.3.2
typescript                        2.9.2
webpack                           4.18.0

Thanks @rtb21 and @webmaxru for the report, I'll have to ensure we test on both 6.1.x and 6.2.x + in the future. This will be fixed after today's release of 0.13.1


pyproject.toml contains unused dependencies. (PACMAN-630)

The Component Manager version
main
ESP-IDF Version
NA
python Version
3.10
Operating System
macOS
Browser (for https://components.espressif.com  Issues)
NA
Description
It seems the pyproject.toml file contains some unused dependencies.
deptry . --extend-exclude 'integration_tests'
Scanning 82 files...

docs/en/conf.py:8:0: DEP001 'conf_common' imported but missing from the dependency definitions
idf_component_manager/utils.py:6:0: DEP003 'packaging' imported but it is a transitive dependency
idf_component_tools/file_cache.py:79:12: DEP001 '_winreg' imported but missing from the dependency definitions
idf_component_tools/file_cache.py:91:8: DEP001 'win32com' imported but missing from the dependency definitions
idf_component_tools/file_cache.py:108:20: DEP001 'win32api' imported but missing from the dependency definitions
idf_component_tools/file_cache.py:142:8: DEP001 'com' imported but missing from the dependency definitions
idf_component_tools/file_cache.py:143:8: DEP001 'com' imported but missing from the dependency definitions
idf_component_tools/file_cache.py:173:12: DEP001 'win32com' imported but missing from the dependency definitions
idf_component_tools/file_cache.py:183:20: DEP001 'com' imported but missing from the dependency definitions
idf_component_tools/manifest/constants.py:39:8: DEP001 'idf_py_actions' imported but missing from the dependency definitions
idf_component_tools/sources/git.py:21:4: DEP001 'urlparse' imported but missing from the dependency definitions
idf_component_tools/sources/web_service.py:29:4: DEP001 'urlparse' imported but missing from the dependency definitions
pyproject.toml: DEP002 'colorama' defined as a dependency but not used in the codebase
pyproject.toml: DEP002 'future' defined as a dependency but not used in the codebase
Found 14 dependency issues.

For more information, see the documentation: https://fpgmaas.github.io/deptry/

To Reproduce
poetry add --group dev deptry
deptry . --extend-exclude 'integration_tests'

Expected behaviour
The pyproject.toml should not contain any unused dependencies and there should be no modules imported that are not in the project's dependencies.
Additional info
I believe many of the issues can be resolved by:

replacing urlparse with urllib.parse
Adding pywin32 as a dependency.
Removing the dependency colorama
Removing the dependency 'future'

Disclaimer: I am the author of deptry. I see this as nice way to contribute to other projects while simultaneously testing deptry on a variety of projects and gather some feedback.
I have checked existing issues and online Documentation

[X] I confirm I have checked existing issues and online Documentation.

Hello @fpgmaas
Thank you so much for making deptry and running it on our codebase!
We forgot to remove future and add packaging during migration to poetry, so the report is useful. To avoid it in the future we decided to add it to our pre-commit hooks and CI workflow.
While we use poetry for our development environment, we still need to support py2.7 for some time, so for package building, we still need setup.py. As a result, dependencies are listed twice, and it's easy to make a mistake.

Implemented in 6d0039dc9c8cb3ec146035bfee678366ab9db07d


[Help] Return value of GSField.spinner

I am trying to get the value of the spinner widget. Unfortunately, I will only get the Instance of 'SpinnerDataModel' in my json.
GSField.spinner(
          title: 'What do you want to contact us about?',
          hint: 'Select a reason',
          tag: 'feedbackType',
          required: true,
          weight: 12,
          items: FeedbackType.values // this is a normal enum
              .map((e) => SpinnerDataModel(
                    data: e.name, // I am guessing this is used to transfer the data?
                    name: e.name.capitalizeFirstOfEach,
                    id: 1,
                  ))
              .toList(),
        ),

Maybe I am using the widget wrongly, but as I can not use the value parameter I don't really know why I am only getting the instance of the class....Any help is appreciated.

Sorry for my mistake, I can obviously extract the data from SpinnerDataModel after receiving the JSON.

Hi Paul, yes you can get dynamic data from SpinnerDataModel instance.

Hello Paul. yes you can get dynamic data from SpinnerDataModel.