warning while training

hi there
i am experiencing this warning while training
01/13 09:56:11 d2.engine.train_loop]: Starting training from iteration 0
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
/usr/local/lib/python3.8/dist-packages/torch/nn/functional.py:718: UserWarning: Named tensors and all their associated APIs are an experimental feature and subject to change. Please do not use them for anything important until they are released as stable. (Triggered internally at  /pytorch/c10/core/TensorImpl.h:1156.)
return torch.max_pool2d(input, kernel_size, stride, padding, dilation, ceil_mode)
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)
[01/13 09:56:45 d2.utils.events]:  eta: 20:18:34  iter: 19  total_loss: 14.61  loss_cls_stage0: 4.223  loss_box_reg_stage0: 0.002018  loss_cls_stage1: 4.189  loss_box_reg_stage1: 0.003367  loss_cls_stage2: 4.404  loss_box_reg_stage2: 0.00404  loss_mask: 0.693  loss_rpn_cls: 0.6935  loss_rpn_loc: 0.3204  time: 1.6258  data_time: 0.0255  lr: 9.7405e-07  max_mem: 28114M
/usr/local/lib/python3.8/dist-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
return lib.intersection(a, b, **kwargs)

hello,friend! i meet the same question. except this, 'cuda out of memory' always happend. could u please show me your detect config ? and i wanna know your memory size（mine is 32g ）

you need to change batch size to 1 if you are training on 32GB of GPU

@Quan-Sun is this warning normal? I also get it when trying to train EVA-02 base MIM_to_COCO model on google colab.


Adding Redux to plots

Tests were great to catch bugs, but this is definitely the kind of PR that needs to be tried as well as checked.

I'm seeing a degradation in performance as well:

https://user-images.githubusercontent.com/37993418/174284521-6bd51bd0-aa02-46b2-b444-3446fe7cf0e0.mov
Could be that my laptop can't handle the webpack dev server being run at the same time as experiments and having the webview open but it is chugging.

I'll check on the performance. That quite unexpected.

It's weird, it seems like the performance problem happens after the Redux change (in the component itself).
https://user-images.githubusercontent.com/3683420/174351176-ce93bec0-eff5-4903-ba55-3b4e41807cf4.mov
See how "order changed" comes after a delay while "use effect triggered" is almost instantaneous?
This is the part of the code I used for that test:
export const Ribbon: React.FC = () => {
  const revisions = useSelector(
    (state: RootState) => state.webview.selectedRevisions
  )
  const [order, setOrder] = useState<string[]>(revisions.map(revision => revision.id ||''))
  const reorderId = 'id'

  useEffect(() => {
    setOrder(pastOrder => performOrderedUpdate(pastOrder, revisions, reorderId))
    console.log('TEST: use effect triggered')
  }, [revisions])

  useEffect(() => {
    console.log('TEST: order changed')
  }, [order])
...

Still investigating what could be the problem. My computer does not show the same long delays as yours, so I didn't notice it at first, but after testing main and my branch one after another, it is noticeable.

Found the cause of performance lags and I think I also found how to fix it.

@sroy3 I'm late to the party, but why do we need to store the problematic objects in the Redux state?

We have many levels between the App component where the plots are received and the actual plot. Using Redux allows to store the plots until they are needed. The goal here is to improve the performance by cutting down the number of re-renders. Also, it cleans up a lot of our logic and makes code more readable.

https://user-images.githubusercontent.com/3683420/175325788-1799e04e-c7f4-41b7-9fe3-fa34ce1b60fe.mov
https://user-images.githubusercontent.com/3683420/175328092-223074a8-b8b1-4469-9935-38d5da5d2b4a.mov
Performance looks much better now, to me it seems faster than main:


Support for FunAsSeq

Closes #569. Instead of translating FunAsSeq to SMT, we are simply using its TLA+ definition via MkSeq.

[x] Tests added for any new code
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] Documentation added for any new functionality
[ ] Entry added to UNRELEASED.md for any new functionality

Looks fine, but I wonder if we couldn't change the interface of FunAsSeq(fn, maxSeqLen) to FunAsSeq(fn) instead, and just compute maxSeqLen == FoldSet(MaxOf2, 0, DOMAIN fn) (at the cost of performance)

That would not help us, as MkSeq requires a constant length.

However, we should think further. Maybe there is a way to implement FunAsSeq without passing constant length directly.


Grapher issue: Error: : syntax error in line 3 near ','

I used Guice's grapher to print .dot file (which dump dependency graph from xtext), and attempt to use graphviz to open it.
But the error occured:
Error: : syntax error in line 3 near ','
I installed graphviz-2.38.0, use guice com.google.inject_3.0.0.v201605172100.jar and guice-grapher-3.0.jar (from mvnrepository.com). My OS is windows7.
The .dot file looks good, is it possible because I am using the wrong Graphviz version?
Does anyone have the same problem as me? Or anyone can open this attachment file?
Thanks.
MyDslGraph.zip

Yes, this problem also occurs to me. For me, the problem was in the margin parameters where commas were set incorrectly. At least in small files that could be fixed by hand.


[Rebased] Exposed Spotify Client ID/Secret in config.yml

#416 rebased against master.

I'll merge this in. Thanks @ManveerBasra!


Adding tags to filter through when running

There should be a flag allowed to specify what tags you want for the StackOverflow to filter through, or multiple different custom settings, like when running, something like
wtpython file.py --python --json, and instead of setting tagged to python in settings.py, you could refer to the multiple tags if specified, so like python;json in this case, as StackOverflow uses ; as a separator for "and" with the tags in the API search.

I think a syntax like wtpython file.py --tags python json would make more sense.

Would there be a benefit to having a config file with options as well? Or is that over-complication?

Not really overcomplication if it's left optional.

The CLI usage for wtpython is wtpython [<wtpython options>] <your script> [<your script options>] so the tags would have to go before file.py
The intent (thus far) is to make it as simple as possible to get to some answers by adding wt at the beginning of your command and that's it. I almost feel like manually adding tags or other parameters to the search pushes you to the point where you should just launch your browser. I'm certainly not opposed to considering the option but I'm not sure this fits in with the "simplicity" of how wtpython is intended to work.
As for the config file option, that would not work because whatever you put in there would probably get applied to EVERY search. I highly doubt that you're going to want every thing to have "json" added to each query since you're probably not always working with json.
My goal is to parse out information from the tracebacks to determine which modules and packages were used and automatically add them to the query parameters.

While I understand the move behind simplicity for wtpython, it should also have more customizability with the Stack Overflow search so users can possibly set the parameters for the search, instead of most of the parameters being preset

While I understand the move behind simplicity for wtpython, it should also have more customizability with the Stack Overflow search so users can possibly set the parameters for the search, instead of most of the parameters being preset

Of course, I believe that some of you have mentioned that in the future WTPython will be less reliant on StackOverflow, so that could be a possible negative to implementing this issue

I tagged this to the New Features milestone since it's something worth considering. But at least for now, I think we need to make our code base a little better. So we can revisit this when we get to that milestone for sure.
I agree it makes sense given the current state of the application. But for now, I'm personally pushing for  getting our code structured better, having a good CI/CD pipeline, and then polishing the UI/UX around the current features. Once that's good to go, we'll have a good framework for adding new data sources and know the best ways to add improvements like this.


Place holders are not replacing when hard-code the values are in the bean class

Replace only when the namespace presents in the deployment.yaml.
https://github.com/wso2/carbon-kernel/blob/master/core/src/main/java/org/wso2/carbon/kernel/internal/configprovider/ConfigProviderImpl.java#L87

As a workaround, we can make sys and env variables resolved in bean class itself(bean class constructor). Like repositoryLocation in DeploymentConfiguration class[1]

https://github.com/daneshk/carbon-deployment/blob/config-model/components/org.wso2.carbon.deployment.engine/src/main/java/org/wso2/carbon/deployment/engine/config/DeploymentConfiguration.java#L30-L35

Moved issue to https://github.com/wso2/carbon-config/issues/3


Update "try Cognitive Services for free" link

Purpose

...

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

The current terminology specifies the speech resource as Speech Studio does not work with the all-in-one Cognitive key. The new text gives the impression that a CogSrv all-in-one key would work.

Correct, the text should point to the "Speech" Azure resouce for getting a key, not to the "Cognitive Services" Azure resource.


feat: (flux) add flux bundle

Closes https://github.com/kairos-io/kairos/issues/638
Tested and working on k3s with this config.
It creates a ConfigMap to ensure only one node tries to boot Flux at once.
VSCode had some things to say about formatting for README.md, I can remove if desired.

Fantastic job guys, thanks!


Reference Links: Fixed documentation reference links

This PR fixes the reference links in documentation

This change is

 Thank you!

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


Face Detection, Recognition and Facial Attribute Analysis with state of art algorithms.

This is already developed and popular library in the field of face recognition. thought it would help the community. I would like to create an end to end implementation of detection, recognition and facial atrribute analysis. Assign me this. again this is already developed by @serengil and thw whole credit goes to him.

This issue has been assigned.
@Raghucharan16 Do follow the project guidelines for PR.

@Raghucharan16 I would like to work on this issue , Please assign this task to me


[hardknott] broken link with Xerces-J recipe

I cannot believe this. This not Meta-Java week.
When trying to do a fresh 'hardknott' build, I get failed fetch for: /meta-java/recipes-core/xerces-j/xerces-j_2.11.0.bb
Another broken link.  This must just happen. The broken link is: http://archive.apache.org/dist/xerces/j/Xerces-J-src.2.11.0.tar.gz
thanks..

I have the same issue, temporarily I fixed the recipe as:
-SRC_URI = "http://archive.apache.org/dist/xerces/j/Xerces-J-src.${PV}.tar.gz"
+SRC_URI = "http://ftp.deu.edu.tr/pub/Infosystem/Apache/xerces/j/source/Xerces-J-src.${PV}.tar.gz"

It occurs also on branch kirkstone, so I'd appreciate fixing the issue in other branches as well, thanks.

Apache has moved to a new FQDN. However, only 2.12.2 is available. We should seriously consider upgrading and backporting the upgrade (there may be some vulnerabilities).

I just checked the archives URL and 2.11.0 is still available.

hi,
will dunfell also be updated? it also has this broken link.

hardknott (which is long EOL) merged in https://github.com/meta-java/meta-java/commit/ca397f38fca7d83ae0a4b57980164e2f91d527b8


[question]reward for Humanoid-v2 (using ddpg)does not increase after 300

hi
My reward stays at 300 after 100w steps whereas yours increases almost linearly.  My actor loss is between -4.5 and -0.5 while my critic loss is between 0 and 0.06, which is way much smaller than yours. The man cannot walk at all.
We have similar settings with yours (mlpPolicy.env with action noise)  in hyper-parameter, model structure, activation function, buffer size, 100 steps of interaction after training  50 steps.
The only difference is the optimizer (mine is adam).
Would you be able to provide some suggestions? Are there any other things I can change to make it work? Is the small loss and the optimizer the key issue?
ps: my codes work perfectly fine for Pendulum-v0.
Thanks~~

Did you try the parameters from rl-zoo? I am also slightly confused: When you say "my code works on Pendulum", do you mean the same code using stable-baselines works on Pendulum but does not work on Humanoid?

We have similar settings with yours

Please fill the issue template completely. It sounds like your are using a custom DDPG implementation?

@araffin    appologies for the format . It's my first time asking question and adding code ....l will upload the complete version and send out the link tomorrow (it's now nearly midnight here in China.)

Did you try the parameters from rl-zoo? There seems to be no parameters for Humanoid env specifically, but you could take the closest match to see how that works.
by  saying "my code works on Pendulum", i mean my code, not the stable baselines.
I am also slightly confused: When you say "my code works on Pendulum", do you mean the same code using stable-baselines works on Pendulum but does not work on Humanoid?
Edit: Possibly related discussion/points #837

by

@hill-a  sorry for the delay. I got some progress! my reward has become nearly 800. The changes are:

increasing gamma in calculating q value from 0.9 to 0.99

adding larger noises on actons.

my questions are:

in which situation should we add more emphasis on instant reward (larger gamma)? 0.9 works fine in Pendulum.
why are noises so important ? how much should it be compared to the original action
it looks like we should never expect critic loss to converge very quickly, should we ?

Closing this as it is apparently unrelated to SB.


Closure test crashes nora

The following test crashes nora:
;; RUN: norac %s | FileCheck %s
;; CHECK: 12
(linklet () ()
  (define-values (fn) (values 0))
  (let-values (((x) (values 2)))
     (set! fn (lambda (y) (+ x y))))
  (let-values (((x) (values 3)))
     (fn 10)))

There's a warning from ubsan as well:
Cannot set undefined identifier.
Expected lambda expression in Application.
/home/pmatos/dev/nora/src/include/interpreter.h:43:52: runtime error: member call on null pointer of type 'ast::ValueNode'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /home/pmatos/dev/nora/src/include/interpreter.h:43:52 in
UndefinedBehaviorSanitizer:DEADLYSIGNAL
==23846==ERROR: UndefinedBehaviorSanitizer: SEGV on unknown address 0x000000000000 (pc 0x56310e36d3d7 bp 0x7ffebfb63690 sp 0x7ffebfb63650 T23846)
==23846==The signal is caused by a READ memory access.
==23846==Hint: address points to the zero page.
    #0 0x56310e36d3d7 in Interpreter::getResult() const /home/pmatos/dev/nora/src/include/interpreter.h:43:52
    #1 0x56310e36cae7 in main /home/pmatos/dev/nora/src/main.cpp:53:26
    #2 0x7f2ef4c3c78f  (/usr/lib/libc.so.6+0x2378f) (BuildId: 4a4bec3d95a1804443e852958fe59ed461135ce9)
    #3 0x7f2ef4c3c849 in __libc_start_main (/usr/lib/libc.so.6+0x23849) (BuildId: 4a4bec3d95a1804443e852958fe59ed461135ce9)
    #4 0x56310e33aa74 in _start (/home/pmatos/dev/nora/build/bin/norac+0x164a74) (BuildId: ff1360b6f90819159bf3d99f7e8a9ddb2f700ae2)

UndefinedBehaviorSanitizer can not provide additional info.
SUMMARY: UndefinedBehaviorSanitizer: SEGV /home/pmatos/dev/nora/src/include/interpreter.h:43:52 in Interpreter::getResult() const
==23846==ABORTING

I also think that even once this is fixed nora will print 13, rather than 12.

So we need to implement closures. We can define a function isClosure on a lambda that returns true if the lambda body has free variables. If so, it self-transforms the Lambda instance to a Closure instance that has an attached environment. When the Closure instance is interpreted, the environment containing the free variables in the Lambda body are captured.


youtube live video, quality, speedo meter support

this library does not support video, only audio.


first commit for branch-2 to include combined.txt!

Ben's first pull request. I will be doing changes to my-new-branch-2

merge pull request for my-new-branch-2 ! Ke Nna!


Specify size of matrices in rotatedrect

Description
Fixes an issue with MakieTeX layouting.
There was a method ambiguity when constructing Mat(args...), which I fixed by explicitly providing sizes (Mat{n, m}(...)).  I guess this method wasn't used so the error was never caught.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist

[x] Added an entry in NEWS.md (for new features and breaking changes)

Thanks :)


Winston HP incorrect during primal range

Due to custom health on winston, his primal HP is slightly above the expected 1000hp. Likely no clean solution exists to fix this issue.

mentioned in commit 8dacc362e27c34fe61588ed0c1287df33ac2dddb


Squad Leader 3.0.0 Product Owner Review

I used a Samsung Galaxy S4 - Model Number SCH-1545 on Android version 5.0.1 for the review. The Squad Leader version was downloaded from: https://esri.app.box.com/s/9hv7urwcvzisn67lyvfps855l8lediic

[x] I would place Setup before Usage at https://github.com/Esri/squad-leader-android/blob/master/documentation/UserGuide.md Usage firstseems like more of a natural flow.
[ ] The app crashes when I attempt to send a spot report.
[x] I have seen the same issues as #115.

@jrweakland, at what point in the spot report creation process does the app crash? Is it when you tap the map, when you change field values in the form, when you submit the form, or some other time? This is going to be tricky because spot reports work fine on all three of my devices of different models.
The other items you mentioned make sense and I can fix them soon.

Setup and Usage sections swapped in fd279411043845fe142cba3f1a57858c4dd51a92

#115 is fixed in 68f0df537d855cc38ff7b00a6b121d7ee223d273
Now I just need to work with @jrweakland on debugging the spot report crash, since I cannot reproduce it on my three devices.

The spot report crash is fixed and documented in 610f5cefc5a3248eaccb936061d05e48827e8d99. Even if the user does not make the change we suggest (i.e. uncheck "Don't Keep Activities"), the app should not crash, though it will reset its layers when the user goes to the spot report or settings form and back to the map.
@jrweakland, I will get you another build to test soon.

@jrweakland, can this issue be closed now?

All of my original issues have been addressed by @garys-esri


make dec bench functional again and much more robust

It took a really large sample size for the numbers to stabilize, but on my M1 mac:
Dec/F64:
addition:       0.65
subtraction:    0.78
multiplication: 4.07
division:       31.80

There is definitely more variance in the dec results compared to fp.

Similar results on my x86 machine:
Dec/F64:
addition:       0.71
subtraction:    0.71
multiplication: 6.48
division:       33.06


Can't get environment variables when prebuilding by Github worflow

This is what I wrote in utils.ts:
export const supabase = createClient<Database>(
  Deno.env.get("SUPABASE_URL") as string,
  Deno.env.get("SUPABASE_KEY") as string,
);

After I push the commit, I got this error message on GIthub workflow page:
Build step >
Run deno task build
Task build deno run -A dev.ts build
.
.
.
.
error: Uncaught (in promise) Error: supabaseUrl is required.
    at new p (https://esm.sh/v132/@supabase/supabase-js@2.4.0/esnext/supabase-js.mjs:2:2168)
    at nt (https://esm.sh/v132/@supabase/supabase-js@2.4.0/esnext/supabase-js.mjs:2:5286)
    at file:///home/runner/work/a-pp-greeting-cards-platform/a-pp-greeting-cards-platform/utils/utils.ts:20:25
Error: Process completed with exit code 1.

And I've setup the env var on deno deploy dashboard and this works on local .env file.

Did you set up variables or secrets?

@ieiekk Do you happen to use a setup with a fresh.config.ts or without one? Asking because before we introduced fresh.config.ts the dev.ts would always load main.ts which in turn imports every file of the application. In doing so it will encounter the supabase call and complain that the environment variables are not set. That's the main reason we introduced the fresh.config.ts which allows us to properly split the development mode internally so that you can create a build without loading main.ts

Did you set up variables or secrets?

I setup the secrets in Github workflow, but still got the same error.

Did you set up variables or secrets?

This works! I add SUPABASE_URL and SUPABASE_KEY as secrets:

And add these lines in my deploy.yml:
name: Deploy
on:
  push:
    branches: [master, develope]
  pull_request:
    branches: master

jobs:
  deploy:
    name: Deploy
    runs-on: ubuntu-latest

    permissions:
      id-token: write
      contents: read

    steps:
      - name: Clone repository
        uses: actions/checkout@v3

      - name: Install Deno
        uses: denoland/setup-deno@v1
        with:
          deno-version: v1.x

      - name: Build step
+        env:
+            SUPABASE_URL: ${{ secrets.SUPABASE_URL }}
+            SUPABASE_KEY: ${{ secrets.SUPABASE_KEY }}
        run: "deno task build"

      - name: Upload to Deno Deploy
        uses: denoland/deployctl@v1
        with:
          project: "pp-greeting-cards-platform"
          entrypoint: "./main.ts"

Hope this helps anyone encountered the same issue.

@ieiekk Do you happen to use a setup with a fresh.config.ts or without one? Asking because before we introduced fresh.config.ts the dev.ts would always load main.ts which in turn imports every file of the application. In doing so it will encounter the supabase call and complain that the environment variables are not set. That's the main reason we introduced the fresh.config.ts which allows us to properly split the development mode internally so that you can create a build without loading main.ts

I leave that file as default:
import { defineConfig } from "$fresh/server.ts";
import twindPlugin from "$fresh/plugins/twind.ts";
import twindConfig from "./twind.config.ts";
export default defineConfig({
  plugins: [twindPlugin(twindConfig)],
});

So is there a way to avoid import some files during build time?  Thanks in advance 🙏

I think there needs to be a way to know which variables will be needed. The only way I could verify which variables are needed for the build to not crash was by renaming my local .env and trying a local build. It appears the secrets are overwritten once the server is running and the environment variables are loaded in anyway.


Include explicit licensing information in the RoadEventFeedInfo

Reference Issue #19
This PR assigns a Creative Commons - Public Domain License (CC0) to a feed.  A license property will be added to the RoadEventFeedInfo object.  Its value will be "CC0"
Defining an explicit public domain license to the WZ Data frees consumers to publish WZ information.

@DeraldDudley these license-related changes should be made in a new branch specific to those changes, branches off of spring-2021-release, rather than to spring-2021-release as you did here. Then, the PR should target spring-2021-release as the base branch, that it, it proposes merging your new branch with the version changes into spring-2021-release.
Thus we need to:

Close this PR
Roll back commit 52c4cae on spring-2021-release
Create a new branch from spring-2021-release for use for these changes, called something like add-license
Make the changes originally made in commit 52c4cae to add-license
Make a new PR proposing merging add-license into spring-2021-release

I will do 1 and 2.

Closed as the changes should be made in a branch specific to those changes, not the base spring-2021-release branch.

Thanks Jacob.
Derald Dudley
NGDA Transportationhttps://communities.geoplatform.gov/ngda-transportation/ Theme Lead
FGDC Transportation Subcommitteehttps://communities.geoplatform.gov/ngda-transportation/transportation-subcommittee/ Chair
Phone:<PHONE_NUMBER>
Pronouns: he/him/his
The Office of Spatial Analysis and Visualizationhttp://maps.bts.dot.gov/
Bureau of Transportation Statisticshttp://www.rita.dot.gov/bts/
Office of the Assistant Secretary of Transportation for Research and Technologyhttp://www.rita.dot.gov/
United States Department of Transportationhttp://www.transportation.gov/
From: Jacob Brady<EMAIL_ADDRESS>Sent: Thursday, December 17, 2020 2:23 PM
To: usdot-jpo-ode/jpo-wzdx<EMAIL_ADDRESS>Cc: Dudley, Derald (OST<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [usdot-jpo-ode/jpo-wzdx] Include explicit licensing information in the RoadEventFeedInfo (#151)
CAUTION: This email originated from outside of the Department of Transportation (DOT). Do not click on links or open attachments unless you recognize the sender and know the content is safe.
@DeraldDudleyhttps://gcc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FDeraldDudley&data=04|01|derald.dudley%40dot.gov|fcb89d23fbeb4209ad8608d8a2c13dde|c4cd245b44f04395a1aa3848d258f78b|0|0|637438298135863707|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=se9jwUArp2OxOyjc%2Be9udZlMXrxvB6F80SKgMftf0Cc%3D&reserved=0 these license-related changes should be made in a new branch specific to those changes, branches off of spring-2021-release, rather than to spring-2021-release as you did here. Then, the PR should target spring-2021-release as the base branch, that it, it proposes merging your new branch with the version changes into spring-2021-release.
Thus we need to:

Close this PR
Roll back commit 52c4caehttps://gcc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fusdot-jpo-ode%2Fjpo-wzdx%2Fcommit%2F52c4cae776857df86c7ed8c010d8afc471f3134b&data=04|01|derald.dudley%40dot.gov|fcb89d23fbeb4209ad8608d8a2c13dde|c4cd245b44f04395a1aa3848d258f78b|0|0|637438298135873660|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=J5fvhHRsY7BktP2RGdbSQIiXrZBtYpk68kHRZcX90oQ%3D&reserved=0 on spring-2021-release
Create a new branch from spring-2021-release for use for these changes, called something like add-license
Make the changes originally made in commit 52c4caehttps://gcc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fusdot-jpo-ode%2Fjpo-wzdx%2Fcommit%2F52c4cae776857df86c7ed8c010d8afc471f3134b&data=04|01|derald.dudley%40dot.gov|fcb89d23fbeb4209ad8608d8a2c13dde|c4cd245b44f04395a1aa3848d258f78b|0|0|637438298135873660|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=J5fvhHRsY7BktP2RGdbSQIiXrZBtYpk68kHRZcX90oQ%3D&reserved=0 to add-license
Make a new PR proposing merging add-license into spring-2021-release

I will do 1 and 2.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://gcc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fusdot-jpo-ode%2Fjpo-wzdx%2Fpull%2F151%23issuecomment-747647407&data=04|01|derald.dudley%40dot.gov|fcb89d23fbeb4209ad8608d8a2c13dde|c4cd245b44f04395a1aa3848d258f78b|0|0|637438298135873660|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=lP4fTacPROw%2FRXz7NuP8cv4VDgSlPGfWJqFIB3T0SoA%3D&reserved=0, or unsubscribehttps://gcc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAB2LFGWC5QY5HLYGLP46WX3SVJLDDANCNFSM4U76HGSA&data=04|01|derald.dudley%40dot.gov|fcb89d23fbeb4209ad8608d8a2c13dde|c4cd245b44f04395a1aa3848d258f78b|0|0|637438298135883615|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=OdbSFpcPVmKhEJlSDQtIsHGKK5OIkVdPsSxBP8%2BVAXc%3D&reserved=0.


FbDirect : Converting circular structure to JSON

When running fbDirect project in emulatorBrowser im getting issue as 'TypeError: Converting circular structure to JSON' in my CMD.
Seems like /src/emulator/browser/index.js is failing on line 121 while stringify an array of object.
'convos' is as Array class 'Convo' which is not able to stringify

This issue has been reported already. Please refer to https://github.com/codeforequity-at/testmybot/issues/74


sar_autojump doesn't work at all

After getting sar recommended for autojump command I tried it and it just doesn't work.
Here's a video of it (sorry if the video lags or stutters): https://youtu.be/8On45GN-BEE

Well, no more autojump while progressing and doing achievements


NVS: Overwriting an item with a shorter matching item fails

Describe the bug
Calling nvs_write to overwrite a data item with another data item that is a truncated version of the existing data will not overwrite the old item.  The "is this equal" check in nvs_write ignores differing lengths and then compares the old/new data up to the length of the incoming item, but it should consider differing lengths as non-matching.
To Reproduce
Steps to reproduce the behavior:

Call nvs_write and store a string like "ABCD"
Call nvs_write again to overwrite the item with "ABC"
Read back the NVS item and get "ABCD"

Expected behavior
In step 3 above, you should read back "ABC"
Impact
Could be causing subtle issues for any application that uses NVS.  Workarounds are possible if you're aware of the problem.
Additional context
A simple patch to fix this is attached.
nvs_patch.diff.txt

@jbrzozoski, thanks for reporting this. Your analysis is correct as well as the proposed fix.

@jbrzozoski thanks for the patch, could you actually open a Pull Request with that code?


Bugfix/v2.11.4

Whats Changed?

Added more Analytics test points
Phrase crash reminder
Converted requiredActivity to getActivity handling the crash more gracefully

Closing ... added improved PR currently open


Adding gradient bandeau and remove right secondary side bar

Cool!!
It seems that the white on the background of the schematic is back.
In addition, the contrasts of the font in the banner is a bit small in black background because the font is white. Maybe we could change the definition of colors of the background using transparency, so that  when the background is white they match the color for scikit-learn, but when the background is black they are darker (thanks to transparency)

Also, we have a failure in the build. Looks like a failure in the examples. I'm not sure how we got there.

Looks great!
It seems an error slipped in with #514, I corrected it #591. Just merge with main @Vincent-Maladiere and it will become green

Closing for https://github.com/skrub-data/skrub/pull/589

Reopened: my PR was a PR to your PR (yes, very confusing, sorry)

Will merge this PR once it is green. However, should we not remove the index.rst right now?

Darn removing index.rst messed up our navbar.
We should add it back, but mostly empty, as it is unused.

LGTM (looks gorgious to me). Merging :heart: :heart: :heart:


Adjust parameters in generate.py as opposed to train.py

Is there a way to use a varying style weight (and other parameters) as part of the generate.py script? It takes so long to train the models that it's hard to experiment with different parameters. If we could adjust weights after the model has already been trained we could experiment much faster. Might there be a way to do this?

That wowuld be cool, yeah. Have you read the paper ?
Trick : extend the image, apply the filtrer, reduce the image.  You can also do the other way around but you'll lose quality.

I haven't read the paper, do you think that could help me understand ways to accomplish my goal?
That's an interesting idea. Unfortunately I'm already maxing out my gpu's ram with the current image sizes, so I definitely can't run the style on any larger images.


Add KeyVault pde documentation.

Fix link to sharing pde documentation.

Please do in dev branch
where did the code snippets come from?


error standardizing ionization

Using the following SMILES string, i have encountered the following error:
USED SMILES:
standardize_smiles('[Na].[Na].OSe=O')
OUTPUT:
log.info('Ionizing %s to balance previous charge corrections', self.acid_base_pairs[ppos].name)
TypeError: tuple indices must be integers or slices, not NoneType
The command works with this SMILES
O=Se[O-].[Na+].[Na+]
Thank you in advance

Maybe it's part of the issue of rdkit, but I actually think the problem is that your input SMILES is non-sense and it can't be the issue of MolVS. Do you now which version of rdkit you use? I can't read both cases through rdkit.


Archi error on startup (MacOs) - Cannot invoke "org.eclipse.core.runtime.IExtensionPoint.getConfigurationElements()" because "point" is null

Archi Version
5.4
Operating System
14.5
Archi Plug-ins
No response
Description
This is related to a corporate desktop, therefore there maybe some constraint or permissions issue. Works fine on my personal laptop.

Archi really slow to start.
Once started this error continually appears
Steps to reproduce
Archi startup

also appears each time when you switch view.
you can press OK and the continue to edit.

Please post the contents of the error log file. This can be accessed in Archi by selecting the "About Archi" menu item. In the dialog box press the "Installation Details" button, then select the "Configuration" tab and press the "View Error Log" button.
Please ensure that Archi has write access to its data and configuration folders at user.home/Library/Application Support/Archi/ and also at user.home/Documents/Archi
You could also try deleting the folder at user.home/Library/Application Support/Archi/config and re-starting.

archi error log
logarchi5.4.txt

Please post the contents of the error log file. This can be accessed in Archi by selecting the "About Archi" menu item. In the dialog box press the "Installation Details" button, then select the "Configuration" tab and press the "View Error Log" button.
Please ensure that Archi has write access to its data and configuration folders at user.home/Library/Application Support/Archi/ and also at user.home/Documents/Archi
You could also try deleting the folder at user.home/Library/Application Support/Archi/config and re-starting.

Unfortunately permissions are good and deleting the config directory did not fix. Thanks

As I can't reproduce this and we've not had this error reported before now I'm afraid you're going to have to try different things such as re-installing, deleting data folders, etc.

I presume there was nothing in the log that indicated anything?

Only that a tabbed property sheet extension was null which indicates something weird like a corrupt installation, config area or conflict to do with an Archi plug-in or conflicting Archi versions perhaps running at the same time.


multiple endpoints for xor query parameters

with Spring @RequestMapping params it is possible to split the endpoint into multiple controller methods for mutually exclusive query parameters.
Would be possible by inventing a mapping configuration.

mapping ... because it looks like there is no simple way of defining it in openapi.


Is this library still maintained? [import Adapter from 'enzyme-adapter-react-15';]

I am upgrading large project to React 16. I have installed following dependencies:
"chai": "^4.1.2",
"chai-enzyme": "^1.0.0-beta.1",
"enzyme": "^3.0.0",
"react": "^16.0.0-0",
"react-dom": "^16.0.0-0"

This is my testConfig.js file:
var chai = require('chai');
var sinonChai = require('sinon-chai');
var chaiAsPromised = require('chai-as-promised');
var chaiEnzyme = require('chai-enzyme');

chai.use(sinonChai);
chai.use(chaiAsPromised);
chai.use(chaiEnzyme());
global.expect = chai.expect;

window.matchMedia = window.matchMedia || function() {
  return {
    matches : false,
    addListener : function() {},
    removeListener: function() {}
  };
};

But when I run my tests I am getting missing enzyme adapter error:
Enzyme Internal Error: Enzyme expects an adapter to be configured, but found none. To
configure an adapter, you should call `Enzyme.configure({ adapter: new Adapter() })`
before using any of Enzyme's top level APIs, where `Adapter` is the adapter
corresponding to the library currently being tested. For example:

import Adapter from 'enzyme-adapter-react-15';

To find out more about this, see http://airbnb.io/enzyme/docs/installation/index.html

When I looked at your source code you don't use custom adapter anywhere except in test/support/helper?
https://github.com/producthunt/chai-enzyme/search?q=isEnzyme3&unscoped_q=isEnzyme3
What am I doing wrong? Thanks

I believe I managed to fix the issue by instantiating my own adapter:
const Adapter = require('enzyme-adapter-react-15')
enzyme.configure({ adapter: new Adapter() })

I thought this should be done by chai-enzyme? Why use chai-enzyme when I still have to do all the setup manually?

It's absolutely not done by chai-enzyme; just like enzyme itself, nobody but YOU can possibly know what react version you're using.

That makes sense. Updating 300+ tests was no fun though 😄

@developer239 you should be able to do the configure in a single file, without having to touch all your other test files.

@ljharb I meant in general 🙂 I had to update enzyme to version 3.x


Roadmap 0.3 - Refactor Journal interfaces

pub trait Journal {
    type StreamReader;
    type StreamWriter;

    async fn create_stream(&self, name: &str) -> Result<()>;
    async fn delete_stream(&self, name: &str) -> Result<()>;
    async fn new_stream_reader(&self, name: &str) -> Result<StreamReader>;
    async fn new_stream_writer(&self, name: &str) -> Result<StreamWriter>;
}

Introduce stream reader/writer to separate concerns. Most use cases will not read and write the same stream at the same time. Separating stream reader/writer has another benefit, allowing different compositions of reader and writer.

allowing different compositions of reader and writer

If you use associated type, I'm afraid that each instance of Journal has fixed composition of reader and writer. For example:
impl crate::Journal for MemJournal {
    type StreamReader = MemStreamReader;
    type StreamWriter = MemStreamWriter;
    ...
}

@huachaohuang The StreamReader in Journal interfaces in master branch seems that missing the watch method?

At the same time, the StreamWriter interface cannot express the semantics of a leader based journal(which client is responsible for acting as the leader) and provide a mechanism which detecting leader failures .

@w41ter-l We need to further refactor the Journal interfaces. Can you foresee what kind of methods do we need in a leader-based journal implementation?

@huachaohuang I thought about it for a while, maybe we don't need to add an interface for StreamWriter, but block when calling new_stream_writer until it win the campaign. And the leader failure detector is hidden in new_stream_writer.

I am considering some thing like this:
enum Event {
    Update(...),
    ChangeRole(Role),
}

enum Role {
    Leader,
    Follower,
}

trait Stream {
    type Watcher: Stream<Item = Event>;
    async fn watch(&self) -> Self::Watcher;
}

Multiple instances can watch the same stream at the same time. One of them will receive an event Event::ChangeRole::Leader and others will receive Event::ChangeRole::Follower. If an instance receives the former, it will act as a leader.

There are two question about the former proposal. First the watch might take a parameter as the cursor, otherwise Journal couldn't find the right position to publish update events. Second Event::ChangeRole introduce the complexity to implement, imagine that a user receives a ChangeRole::Follower event but invoke new_stream_writer immediately, the Journal need to handle those situitions. I think blocking on new_stream_writer is enough to handle role changes.

@w41ter-l since you are working on the Journal, feel free to prototype the interfaces as you see fit. We can discuss it when we have a more detailed design.

We can also further divide the interfaces into two levels. The first level provides some basic interfaces. The second level extends the basic interfaces with leader-based features.

We can remove the timestamp from the Journal too. Users can encode their timestamps in the data when necessary. Journal can maintain an increasing sequence number internally and returns the sequence number to users on every write. Then users can use the sequence number to archive or release data from Journal.

We can remove the timestamp from the Journal too. Users can encode their timestamps in the data when necessary. Journal can maintain an increasing sequence number internally and returns the sequence number to users on every write. Then users can use the sequence number to archive or release data from Journal.

I will start refactoring some interfaces for this so that I can continue working on the upper Kernel and Engine. But I will leave the leader-based semantics to future works.

@huachaohuang I suggest you guys create subtasks for dedicated work so that a ref #229 replaced by ref #xxx and gather related information. Also, you can even duplicate previous discussion in PR description to help reviewers catch up the motivation and major changes quickly.
Some good cases:

https://github.com/engula/engula/pull/238
https://github.com/engula/engula/pull/224
https://github.com/engula/engula/pull/252
https://github.com/engula/engula/pull/193

@tisonkun Thanks for the suggestions. Makes sense to me.

Some of the jobs are done here. For the leader-based API, please open a new issue for it when necessary. Also check #203 for more information.


sending funds from on-chain wallet

I am trying to transfer SATS from my on-chain wallet to a Joinmarket wallet, and it is giving me an error that there are insufficient funds, I have lowered the value to a significant value that should be able to be transferred and it still errors out. I remember having the error before, is there a minimum balance that raspiblitz wants to retain in the on-chain wallet?

@openoms can you give details on sending funds to joinmarket here?

@openoms can you give details on sending funds to joinmarket here?

Not sure what is the problem. When you send to JM you just send a simple bitcoin transaction to a new address.
Make sure your funds in the LND wallet are comfirmed and consider using the onchain functions of RTL for a GUI.

Not sure what is the problem. When you send to JM you just send a simple bitcoin transaction to a new address.
Make sure your funds in the LND wallet are comfirmed and consider using the onchain functions of RTL for a GUI.

The problem isn't Joinmarket related, I was just trying to move as much out of the LND onchain wallet as possible. I was calculating fees and making a judgement on what value I could send, however I was never able to get the remaining value as low as I possible. In the end I was able to move all but 617 SATS out of my wallet. Is there an easier way to send the full value out of the onchain wallet? I didn't want to  close all channels and shutdown my node. I just wanted to move 100% (minus fees) of the value of my onchain wallet.

The problem isn't Joinmarket related, I was just trying to move as much out of the LND onchain wallet as possible. I was calculating fees and making a judgement on what value I could send, however I was never able to get the remaining value as low as I possible. In the end I was able to move all but 617 SATS out of my wallet. Is there an easier way to send the full value out of the onchain wallet? I didn't want to  close all channels and shutdown my node. I just wanted to move 100% (minus fees) of the value of my onchain wallet.

Yes, RTL has a sweep function which is available in the CLI as well: https://api.lightning.community/#sendcoins

Yes, RTL has a sweep function which is available in the CLI as well: https://api.lightning.community/#sendcoins


Fix missing tokenizer attribute in test

This should hopefully fix the error in test_causal_lm_training_mutli_gpu_4bit (see e.g. here). I haven't tested it, as it's multi GPU.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


PyCon UK 2016 Spacy Notebooks

PR to add the notebooks demonstrated by Cytora at PyCon UK 2016.
Includes an introduction to spaCy, followed by examples where they used spaCy to analyse the text of Pride and Prejudice and the RAND dataset.
Link to their repo, and the corresponding GitHub issue regarding using the notebooks.

Ah, this is amazing! Good idea and thanks for asking them to let us use it 👍
Btw, since you're added as repo collaborator, so feel free to push straight to master next time! I think this is totally fine for the notebooks and this way, updates won't block on me or @honnibal. Also, if others ever make pull requests to this repo, I totally trust your judgement – so if it looks good to you, just go ahead and merge the PR.

Sure! I just make the PRs so everything is better documented. :)


Allow colgroups to not be rendered

This is a small change that parameterises the colgroup rendering - the reason in my case is that I use CSS to set widths, and the colgroups can cause issues in some browsers which get confused, like Firefox. Disabling them from being rendered at all lets the CSS work properly.

How do you define the column width? Which browsers you are confused? The colgroup works even in old browsers....

Before jump onto this, I would suggest a conference call to analyze this case together. I have an impression we can do something about that without messing in the colgroup which will have a huge impact in some future releases.
My skype: paul.hodel


failed to execute command "sealer run"

What happened:
Background:
sealer: v0.1.4
Command:
sealer run kubernetes:v1.19.9 --masters <IP_ADDRESS> --nodes <IP_ADDRESS>,<IP_ADDRESS> --passwd xxxx
Result:
2021-05-26 16:35:46 [INFO] join masters: [<IP_ADDRESS>]
2021-05-26 16:35:52 [EROR] exec remote cmd failed, kubeadm init phase upload-certs --upload-certs -v 0 [ssh][<IP_ADDRESS>]run command failed [kubeadm init phase upload-certs --upload-certs -v 0], Process exited with status 1
2021-05-26 16:35:52 [EROR] get certifacate key failed []
What you expected to happen:
expect successful operation
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

sealer version (use sealer version):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

see #187

/invite


About data persistence on Bigtable emulator

Hello,
I use Bigtable emulator on my local docker environment, but I cannot save data on it somehow.
Do you have any ideas to fix this issue?

@TatsuyaKITABORI, can you please elaborate on the nature of the problem?  How are you connecting to the emulator?  Do you have any stacktraces?

I think OP thinks the emulator writes to persistent storage, but it doesn't.

@enocom @sduskis @jba
Thank you for your quick response.
Yes, emulator works well.
But I want to perpetuate data on my local HDD or SSD.
Are there options something to fix this?

The emulator is in-memory only and for now we don't have any plans to change that. So it's probably best to explore alternative approaches for what you're trying to accomplish.


Move custom sass utils to julep repo

https://github.com/meetup/julep
This will allow us to open source useful fragments of Sassquatch2, such as our declarative (class-based) flexbox abstractions

merged to release/1.0.0


Disable extension for specific boards

Can be implemented as a button next to "Redraw", i.e. "Disable", but then it would still require some code to run. If done in popup.js for current board - extension can be stopped much earlier.


create oversimplistic template for button widget

text rendering is difficult so this widget is presumably text-less

so far the development is not going to incorporate widget system


Cannot run opt 125m examples with latest energonai docker images

Run with own build docker images with colossalai 0.2.0 as base image, no response for a very long time.
Run with latest energonai docker images, code version mismatch, raise following error:
[root@eb3f1650fdbf opt]# python3 opt_fastapi.py opt-125m
Traceback (most recent call last):
File "/workspace/EnergonAI/examples/opt/opt_fastapi.py", line 7, in
from energonai import QueueFullError, launch_engine
ImportError: cannot import name 'QueueFullError' from 'energonai' (/opt/conda/lib/python3.9/site-packages/energonai/init.py)

Hi, here is a dockerfile I've tested:
FROM hpcaitech/colossalai:0.1.10-torch1.11-cu11.3

COPY opt-125m /data/opt-125m

# install energonai
RUN git clone https://github.com/hpcaitech/EnergonAI.git && \
    cd EnergonAI && \
    pip install -r requirements.txt && \
    pip install . && \
    cd examples/opt && \
    pip install -r requirements.txt

WORKDIR /workspace/EnergonAI/examples/opt

Put your opt-125m pretrained weights folder to ./opt-125m before building image.

I also encountered the same problem.
Using the latest mirror, no response for a long time
python opt_fastapi.py opt-125m --max_batch_size=1
And also found that the system log is not output, it is difficult to locate the problem
Below is my client code：
`import requests
import json as js
url = "http://<IP_ADDRESS>:7070/generation"
headers = {
'Content-Type': 'application/json; charset=utf-8'
}
data = {
'max_tokens': 64,
'prompt': 'Question: Where were the 2004 Olympics held? Answer:',
'top_k': 1,
'top_p': 0.7,
'temperature': 0.99
}
result = requests.post(url, data=js.dumps(data))
#result = requests.post(url, data=data, headers=headers)
print(result.json())`


feat: switch to npm

can't set version to "latest" unfortunately.

Damn there's no like version: ^1.0.0 or anything? This makes me sad.


sanitized integration name on Windows contains whole path

Release Note
NONE

Fixes #1462

Locally all the tests passes. I guess those are glitches.

Let's merge this. Tests are fixed with #1485.


Update docs and its build action

[x] This PR updates the docs action to rebuild the website only if any files under the website directory are modified;
[x] Scafold the docs using content from the README and the internally used onboarding document;
[x] Extend the instructions on building and testing the website for local development.

@epiercehoffman, thanks for the review; I made the changes you suggested.

Home page:
The landing/home page seems like it may not give enough information about what GATK-SV actually does, and the three highlights on the landing page are somewhat sweeping claims that I am not sure we can substantiate. What if we turned the home page into a short summary of the pipeline, with some relevant citations, and a quick guide to navigating the website?

@hab45 and @RCollins13 are working on that.

Content:
I noticed some sections that differ from the README in terms of content. I would prefer at this stage to directly port the entire unaltered README over to the website format and hold off on any further changes to the content for now. Reasoning:

It's difficult to review content changes when every file is brand new, and I do think we want to review any changes carefully
It will be complicated to maintain two different versions of the documentation while we are getting the website set up, and more so if they have different information
Some of the content on this branch is not ready to be public-facing. For example: one section under troubleshooting & the entire cohort mode Terra workspace
Some information, such as citations and gCNV training, seems to be missing from this branch of the website, and we don't want to lose track of that
We want to be working with the User Ed team as we expand our documentation

We tried to start this with content from README and onboarding. Following your suggestions in the comments, I removed all the content from the onboarding doc (if I have not missed any), which should include non-public content.

It's difficult to review content changes when every file is brand new, and I do think we want to review any changes carefully

True. Though I am not sure what could be a good alternative to splitting the content into multiple files. Organization in multiple files simplifies the navigation and keeps each page focused on one particular topic.

Some information, such as citations and gCNV training, seems to be missing from this branch of the website, and we don't want to lose track of that

Agree, this is a starting point, and we will incrementally extend and improve.

It will be complicated to maintain two different versions of the documentation while we are getting the website set up, and more so if they have different information
We want to be working with the User Ed team as we expand our documentation

I am under the impression that this will largely replace the README so we will not be maintaining two docs. Also, these should be technical docs that we should incrementally improve and extend in faster interactions. I would defer to @mwalker174

Thanks for your quick response and updates @VJalili! Quick clarification:

I am under the impression that this will largely replace the README so we will not be maintaining two docs.

Agreed, and that is why I want to make sure that the starting website matches the content that's currently in the README, so that we can move over to the website without loss of information, and then once the website is established we can improve the documentation further in one place.
There are still a number of differences between the README and the website in the current version of your branch so I wanted to clarify this point. What do you think about starting with a direct port of the README over to the website without content changes, so that future content updates are easier to review?

There are still a number of differences between the README and the website in the current version of your branch so I wanted to clarify this point. What do you think about starting with a direct port of the README over to the website without content changes, so that future content updates are easier to review?

I am not so sure about a direct port. Are you suggesting undoing all the websites, creating a single .md file that is a copy-paste of README, and continuing from there?

Are you suggesting undoing all the websites, creating a single .md file that is a copy-paste of README, and continuing from there?

No, I'm suggesting keeping all the text the same but just reorganizing it into the files needed for the website. So it would involve changes to the organization but save any content changes for later so that we can focus on one thing at a time.

After implementing your suggestions, I think they should be mostly the same. There could be some sentences added/modified, but most detailed descriptions, in particular for the modules, should be the same.

@mwalker174 Thank you for the review. I implemented your suggested changes.

Thank you, @mwalker174 and @epiercehoffman!

Thanks, @epiercehoffman! I implemented your suggested changes.


Docs Revamp: new "Repositories" page for hub-docs

This is part of the process of revamping the documentation (#62) + Docs Proposal
This PR contains the first version of the "Repositories" documentation for the Hugging Face Hub, which aims to give readers a introduction to how repos are used in the Hub (for models, datasets, and Spaces) and shows how to get started with cloning, committing, pushing, and more. In-depth git tutorials are offloaded to third-party content. Information in this doc is split into "Getting started", "Best practices", and "Next steps", each covering increasingly complex or niche features.
Note that at the moment everything is contained within a single page, but I think this is a prime candidate for nested pages. Since (as far as I know) Moon Landing doesn't support nested pages for these docs yet, I was thinking that this could start as a single page, then nesting pages could be implemented in Moon Landing, and then this page could be split (probably with each Heading 2 being a separate page). Or the Heading-level split could actually be handled by Moon Landing!
Also note that links to non-existing sections are marked as TODO, and will be patched in a future PR before the revamp release.

Note that at the moment everything is contained within a single page, but I think this is a prime candidate for nested pages. Since (as far as I know) Moon Landing doesn't support nested pages for these docs yet, I was thinking that this could start as a single page, then nesting pages could be implemented in Moon Landing, and then this page could be split (probably with each Heading 2 being a separate page). Or the Heading-level split could actually be handled by Moon Landing!

What we could do for now while developing is:

in /repositories.md, just have something like a Table of Contents. In a follow up PR, from moon landing most likely, we can do some fancy design for it.
and then have a different .md for each of the pages. This won't be in the sections.yml but it will be linked from repositories, so it should be fine. They would still render.

cc @gary149 about this since we'll have to change some UX code for the docs as we were discussing

in my opinion you can already split into nested pages.
the changes required in moon-landing to support this should be super simple, i can do it early next week at the same time (or before) we merge this

To handle nested pages for the moment I did it with @osanseviero's suggestion of having a table of contents and linking to the pages. I also put the pages into a repositories directory for the moment, but I can adjust it to whichever format would work best for moon-landing, @julien-c ! Any thoughts?
Also, I've applied almost all the feedback (except for moving the images, WIP), so I think this is ready for another round of reviews 😄


Segfault in some pointer to pointer schenerio

all cester cester_assert_ptr_not_equal, cester_assert_str_equal
is causing segfault in a pointer to pointer scenerio, fix in libcester
char **ptr;

Debugged the issue down to cester_concat_str, the issue is trying to free the *out variable that was never malloced, I guess. in line 991

the previous comment was wrong, this bug is from hell.

The fix for now is overriding the macro cester_assert_str_equals
CESTER_BODY(
#define cester_assert_str_equal(x,y) cester_assert_true(xstring_cstr_equals(x,y))
)

Lol.


Remove binary blobs from .git when generating the zip archive  (IDFGH-10410)

Is your feature request related to a problem?
When downloading the esp-idf following the link in the archive it contains all the contents of .git directory. This directory alone contains ~800 MB of data, this represents ~60% of the size of the zip file.
Describe the solution you'd like.
When generating archive files remove useless .git contents. I guess that this directory is included to grab the current commit of esp-idf, but then at least I think .git/modules (contains 757.2 MB of data) could be removed?
Describe alternatives you've considered.
No response
Additional context.
No response

We do include the entire .git directory so that it is possible to switch between IDF versions, for instance to compare the behavior between releases or to do git bisect.
We could remove it, but then the instructions for switching between IDF versions in https://docs.espressif.com/projects/esp-idf/en/latest/esp32/versions.html#updating-esp-idf would have to cover the case if the .git directory doesn't exist.
Could you please check if doing a shallow clone (git clone --depth=1 --recurse-submodules --shallow-submodules ...) would work for you instead of downloading the .zip file?

In some cases a git clone may work although there are different use cases for which I use he zip file:

In CI/CD scripts to build the firmware and run some tests
On my system I install esp-idf from the package manager (which downloads the zip file, in this case a git clone won't work)
On some development machines

In some cases a git clone and a checkout to a tagged version may work, but in the cases it won't would be nice to reduce the size of the zip file (maybe by moving the .git/modules directory to a different zip file for debugging?). Although I don't know if there is much interest in that.

hello Ivan

We do include the entire .git directory so that it is possible to switch between IDF versions, for instance to compare the behavior between releases or to do git bisect.

i am not insisting that it's a single best way of handling this issue, however in general it's common that software releases are size/format optimized for typical usage, what you are describing is related to espressif/esp-idf troubleshooting/development?

We could remove it, but then the instructions for switching between IDF versions in https://docs.espressif.com/projects/esp-idf/en/latest/esp32/versions.html#updating-esp-idf would have to cover the case if the .git directory doesn't exist.

perhaps easiest way to handle this issue, is to point user to GitHub releases, and just ask them to download and unpack version they want or latest stable release?
i feel like manipulating Git repo is probably more useful to espressif/esp-idf developers, than to folks using it to put their own code on Espressif SoC-powered device.

Could you please check if doing a shallow clone (git clone --depth=1 --recurse-submodules --shallow-submodules ...) would work for you instead of downloading the .zip file?

that of course works very well:

name/version
clone type
size
clone time

esp-idf-v5.3.2
full
2.2G
3m8.277s

esp-idf-v5.3.2
shallow
578M
1m25.328s


python312Packages.cirq-ionq: 1.4.0 -> 1.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.cirq-ionq is: Framework for creating, editing, and invoking Noisy Intermediate Scale Quantum (NISQ) circuits
meta.homepage for python312Packages.cirq-ionq is: https://github.com/quantumlib/cirq
meta.changelog for python312Packages.cirq-ionq is: https://github.com/quantumlib/Cirq/releases/tag/v1.4.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.4.1 with grep in /nix/store/w4nknjhyrl2s5br8fxmqcwy7ldahhxh0-python3.12-cirq-ionq-1.4.1
found 1.4.1 in filename of file in /nix/store/w4nknjhyrl2s5br8fxmqcwy7ldahhxh0-python3.12-cirq-ionq-1.4.1

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

21 total rebuild path(s)

21 package rebuild(s)

First fifty rebuilds by attrpath

python311Packages.cirq
python311Packages.cirq-aqt
python311Packages.cirq-core
python311Packages.cirq-ft
python311Packages.cirq-google
python311Packages.cirq-ionq
python311Packages.cirq-pasqal
python311Packages.cirq-rigetti
python311Packages.cirq-web
python311Packages.stim
python312Packages.cirq
python312Packages.cirq-aqt
python312Packages.cirq-core
python312Packages.cirq-ft
python312Packages.cirq-google
python312Packages.cirq-ionq
python312Packages.cirq-pasqal
python312Packages.cirq-rigetti
python312Packages.cirq-web
python312Packages.stim

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w4nknjhyrl2s5br8fxmqcwy7ldahhxh0-python3.12-cirq-ionq-1.4.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/48d660b87468b6c530fa291ce75cc507acb46276#python312Packages.cirq-ionq

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w4nknjhyrl2s5br8fxmqcwy7ldahhxh0-python3.12-cirq-ionq-1.4.1
ls -la /nix/store/w4nknjhyrl2s5br8fxmqcwy7ldahhxh0-python3.12-cirq-ionq-1.4.1/bin

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

8 packages marked as broken and skipped:

python311Packages.cirq
python311Packages.cirq-rigetti
python311Packages.cirq-rigetti.dist
python311Packages.cirq.dist
python312Packages.cirq
python312Packages.cirq-rigetti
python312Packages.cirq-rigetti.dist
python312Packages.cirq.dist

4 packages failed to build:

python311Packages.cirq-ft
python311Packages.cirq-ft.dist
python312Packages.cirq-ft
python312Packages.cirq-ft.dist

28 packages built:

python311Packages.cirq-aqt
python311Packages.cirq-aqt.dist
python311Packages.cirq-core
python311Packages.cirq-core.dist
python311Packages.cirq-google
python311Packages.cirq-google.dist
python311Packages.cirq-ionq
python311Packages.cirq-ionq.dist
python311Packages.cirq-pasqal
python311Packages.cirq-pasqal.dist
python311Packages.cirq-web
python311Packages.cirq-web.dist
python311Packages.stim
python311Packages.stim.dist
python312Packages.cirq-aqt
python312Packages.cirq-aqt.dist
python312Packages.cirq-core
python312Packages.cirq-core.dist
python312Packages.cirq-google
python312Packages.cirq-google.dist
python312Packages.cirq-ionq
python312Packages.cirq-ionq.dist
python312Packages.cirq-pasqal
python312Packages.cirq-pasqal.dist
python312Packages.cirq-web
python312Packages.cirq-web.dist
python312Packages.stim
python312Packages.stim.dist

Maintainer pings
cc @drewrisinger @fabaff for testing.

Superseded by https://github.com/NixOS/nixpkgs/pull/338568


Loop between  Minitest/AssertTruthy and  Minitest/AssertWithExpectedArgument when passing msg option to assert

desc
I think there is an issue when passing the optional argument msg that is allowed for all assert* methods. When using that optional argument i think the gem gets confused and thinks you want to test equality between the first argument (the test) and the second optional argument (msg). i've added more context to the expected behavior section with code examples.
ref: https://docs.ruby-lang.org/en/2.1.0/MiniTest/Assertions.html

Expected behavior
I expect Minitest/AssertWithExpectedArgument not to be raised when passing optional msg argument to assert
Actual behavior
When passing msg rubocop complains about using assert_equal(test, msg)
Steps to reproduce the problem
given:
  def assert_routes_get(path, msg = nil)
    msg = message(msg) { "Expected GET #{path} to be routable" }

    assert(routable?(method: :get, path:), msg)
  end

when:
$ rubocop
then:
C: Minitest/AssertWithExpectedArgument: Did you mean to use assert_equal(routable?(method: :get, path:), msg)?
    assert(routable?(method: :get, path:), msg)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

given:
  def assert_routes_get(path, msg = nil)
    msg = message(msg) { "Expected GET #{path} to be routable" }

    assert_equal(true, routable?(method: :get, path:), msg)
  end

when:
$ rubocop -a
then:
C: [Corrected] Minitest/AssertTruthy: Prefer using assert(routable?(method: :get, path:), msg).
    assert_equal(true, routable?(method: :get, path:), msg)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

and we are back to the first example...
RuboCop version
$ bundle exec rubocop -V
1.41.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.24.0, running on ruby 3.1.2) [arm64-darwin21]
  - rubocop-minitest 0.25.0
  - rubocop-performance 1.15.1
  - rubocop-rails 2.17.3

https://github.com/rubocop/rubocop-minitest/blob/b97ae78e0f0f9f3d71131eae1bc18b3e7b3a6913/lib/rubocop/cop/minitest/assert_with_expected_argument.rb#L9-L11
So I think, the best solution is to just disable this cop for that line.

I've opened #206. The second argument to the assert method named message and msg is allowed. Because their names are inferred as message arguments.


Merge main into dev

Since I had some merge conflicts I open this PR to merge the main branch into develop

I'm not sure why the CI can't build this... Locally everything works.
"/Users/runner/work/1/s/src/CommunityToolkit/Xamarin.CommunityToolkit/Xamarin.CommunityToolkit.csproj" (Build target) (1:54) ->
  /Library/Frameworks/Mono.framework/Versions/6.4.0/lib/mono/msbuild/Current/bin/Roslyn/Microsoft.CSharp.Core.targets(59,5): error MSB3105: The item "ObjectModel/Internals/BaseCommand.netstandard.cs" was specified more than once in the "Sources" parameter.  Duplicate items are not supported by the "Sources" parameter. [/Users/runner/work/1/s/src/CommunityToolkit/Xamarin.CommunityToolkit/Xamarin.CommunityToolkit.csproj]

@jfversluis pinging you since you're our devops hero!


Bip39 mnemonic phrase not correctly normalized / sanitized

The bip39 mnemonic phrase not correctly normalized / sanitized in the EIP2333 web tool https://iancoleman.io/eip2333/ .
For example, adding a trailing space (or new-line) after the last word on the "BIP39 Mnemonic" field causes the tool to generate a completely different BIP39 seed, therefore completely different keys.
Same issue when adding extra spaces between the words, etc.
Note that this bug does NOT exist in the classic bip39 web tool https://iancoleman.io/bip39/

Thanks for this @loupiote

bug is not fully fixed:
adding an extra space (or new-line) between the seed words (e.g. 2 or more spaces or new line between words) causes the seed to change. it should not (no such issue with the ian coleman bip39 tool).
any word separator(s) should all be internally converted into 1 space.
the current fix only applies to leading / trailing space (or new-line).
example of equivalent seed phrases:
"all    all all"
"all
all  all"
etc


[enhancement] Changeable USB VID & PID

Implement the Changeable USB VID / PID feature in order to let user decide its favorite VID/PID, in order to bypass endpoint restrictions.

This is easy enough to do by the end user.
Add this to the end of boards.txt and select Cactus WHID under boards instead of LilyPadUSB when you upload the Arduino_32u4_Code sketch.
`##############################################################
CactusWHID.name=Cactus WHID
CactusWHID.vid.0=0x1B4F
CactusWHID.pid.0=0x9207
CactusWHID.vid.1=0x1B4F
CactusWHID.pid.1=0x9208
CactusWHID.upload.tool=avrdude
CactusWHID.upload.protocol=avr109
CactusWHID.upload.maximum_size=28672
CactusWHID.upload.maximum_data_size=2560
CactusWHID.upload.speed=57600
CactusWHID.upload.disable_flushing=true
CactusWHID.upload.use_1200bps_touch=true
CactusWHID.upload.wait_for_upload_port=true
CactusWHID.bootloader.tool=avrdude
CactusWHID.bootloader.low_fuses=0xff
CactusWHID.bootloader.high_fuses=0xd8
CactusWHID.bootloader.extended_fuses=0xce
CactusWHID.bootloader.file=caterina-LilyPadUSB/Caterina-LilyPadUSB.hex
CactusWHID.bootloader.unlock_bits=0x3F
CactusWHID.bootloader.lock_bits=0x2F
CactusWHID.build.mcu=atmega32u4
CactusWHID.build.f_cpu=8000000L
CactusWHID.build.vid=0x0000
CactusWHID.build.pid=0xFFFF
CactusWHID.build.usb_product="Cactus WHID"
CactusWHID.build.usb_manufacturer="April Brother"
CactusWHID.build.board=AVR_LILYPAD_USB
CactusWHID.build.core=arduino
CactusWHID.build.variant=leonardo
CactusWHID.build.extra_flags={build.usb_flags}`

Replace this portion with your spoofed VID/PID.
CactusWHID.build.vid=0x0000 CactusWHID.build.pid=0xFFFF CactusWHID.build.usb_product="Cactus WHID" CactusWHID.build.usb_manufacturer="April Brother"

Cool solution!
I never had time (or need yet) to dig into this, and didn't want it to get lost in the issues. (reason why I opened here, since ESPloit V2 will be delivered with Cactus WHID by default).
Glad I opened the issue and you immediately proposed a solution! Nice job man!

Speaking of which ask April Brother to ship with the latest firmware as I set it up to connect to my server when you go to Update Firmware and check for updated releases on Github, display if an update is available, then download the update and apply it if the user chooses to.
This will make it simple for an end user to keep in the loop without needing any knowledge about Arduino or Github etc... all the old features are retained, this is just a bonus.

This feature is very cool and I like it! However, might be better to be configurable on the UI (i.e. enable/disable autoupdate check)
Imagine the scenario where a Cactus WHID is deployed during an engagement and is using the client wifi network as Client and the attacker is controlling it within the same LAN... With your auto-update feature on... it would start to generate outbound traffic to the internet (i.e. your server) that would be logged and likely traverse target's IPS/IDS/FW.
Ideally, is always better to keep these devices as silent and stealthy as possible from a network activity perspective.

It will only be triggered if someone goes to the config page and clicks on update firmware.

Now I got it! So is triggered only under user's action. Cool then!
Yeah, is definitely the best solution indeed!
Well actually the dealing-with-Arduino stuff... was mainly to not keep things that simple, that everyone will be able to use it immediately (the so called plug-n-hack) =P

Forgot to close the ticket :)

I will actually leave this one open so people can find it easier until I get around to adding it to the readme along with changing keyboard layouts.


Auto import not following the format rules

When accepting an auto-import, I get the following format:
import {Something} from 'somewhere'
while I have this option setup:
  "typescript.format.insertSpaceAfterOpeningAndBeforeClosingNonemptyBrackets": true
This config works on save, not on auto import

I have the same issue.


[i2c, rtl] Loopback test and other changes

Enabled line loopback test
Implemented target waiting for host to pull SCL low before stretching the clock
Increased depth from 32 to 64 in all four FIFOs

Signed-off-by: Igor Kouznetsov<EMAIL_ADDRESS>

Can someone please review this PR? Thanks

Sorry about that, @igk8. I think we can move this in, but just note that depending on the outcome of the discussion in https://github.com/lowRISC/opentitan/issues/5112 we will have to change the FIFO size again.

Thank you @msfschaffner ! Changing the FIFO size in RTL would be easy enough.


fix BASE lookup in cargo-check-benches

Noticed another bug in the old code (the fetch respecs were broken) and refactored the whole thing.

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2536880

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2536881

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2536936

The CI pipeline was cancelled due to failure one of the required jobs. Job name: test-linux-stable Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2536936

^ j

The CI pipeline was cancelled due to failure one of the required jobs. Job name: test-linux-stable Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2536936

^ just a rusty-cachier network fluke


Update mission-critical-application-design.md

updating the user flow description

@heoelri : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

Can be closed. Adressed in https://github.com/MicrosoftDocs/well-architected-pr/pull/180


Fix checking /_app

Fixes: #35
[ ] Documentation update
[+] Bug fix
[ ] New feature
[ ] Other, please explain:
What changes did you make? (Give an overview)
Which issue (if any) does this pull request address?
Is there anything you'd like reviewers to focus on?

Okay, done, @aralroca

@TheMatrixan thanks for your contribution. Your code is already available in 2.5.3-canary.2) prerelase


Moving capture area to another monitor records the default capture area on the primary monitor

When running multiple monitors and trying to record something not on the primary screen I am seeing Kap instead record the main screen at the default position/dimensions.

Hey, thanks for the report. This as a duplicate of https://github.com/wulkano/kap/issues/33 so I'm going to close it. 👍

You beat me to it @doot0. I was just about to update. Thanks!


fix: typo in KVB regionalverkehr id

I dug a bit deeper and fixed a typo (reversed digits in the above ID) that seems to have fixed issue #34 .
fixes #34
In case this gets merged, I would appreciate a minor release so I can proceed with my depending application. :-)

Any chance for this fix to be merged and released soon?

Thanks for merging this, @leona-ya ! Could you also create a new release? I guess with #33 , it will be 0.5.0?

Hey, I wanted to check back and ask again, if you could publish a new version to PyPI...

Hey, I wanted to check back and ask again, if you could publish a new version to PyPI...

Hey @akloeckner I already forwarded your request to the matrix room: https://matrix.to/#/#pyhafas:matrix.org but did not yet get an answer.
In the meantime you can either build a package of the current state on your own by running python -m build in the cloned repository or install it directly from git: pip install git+https://github.com/FahrplanDatenGarten/pyhafas.git

0.5.0 should now be published...

🎉 Thanks to both of you!


No way to set rendering order (sorting) of Map repeaters without transforming the expression to an array

Since Maps don't have an inherent way to set the iteration order like Arrays (unless by recreating the map by re-adding all the elements in order - not really reliable), this presents a problem in rendering them in a sorted way without resorting to hacks, like transforming them into Arrays first. This is hacky and can introduce performance problems, because every time the expression is evaluated, the whole Map has to be reiterated and converted to an Array.
So the current workaround is:
<template repeat.for="entry of news | getArrayFromMapValues | sortByField:'date'>

Perhaps there a better solution to this problem that I'm not aware of?

you could replace the MapRepeatStrategy.  Maybe you could enhance it with a convention that looks for a sort function on the map and use it to control the order.


Version in git does not match with the version with Metis.version

in Git the Version is 0.1.1

but when I call in Python 'Metis. __ version__', I get 0.0.0
Please update the version in Metis/init.py

ok, thank your for the explanation. I close this issue.


Added pipe (|) to terminator characters

https://github.com/markdown-it/markdown-it/blob/master/lib/rules_inline/text.js
It seems like the comment  above the defined characters implies the existence or intent of adding the pipe character to the terminator characters, it would be useful to add that character or any of the character defined there if that was the intent, thanks.

We do not add changes "just for fun". Do you have a concrete example, where lack of | is a problem?

Trying to implement a Discord-style spoiler block. Not being able to even extend said list is unfortunate. I was mostly just wondering why the character was on the comment despite not being in the list itself.

I was mostly just wondering why the character was on the comment despite not being in the list itself.

I don't remember, if it was placed in comment by mistake or missed in code.
Right now you can replace text rule by your custom one. Then, when your plugin ready, just kick me and I will add the desired char into mainstream.


Support for hash nested under an array

I expected to get an array from a form like this, but got an object instead.
<form>
  <input name='people[0][name]' value='fred' />
  <input name='people[0][age]' value='12' />

  <input name='people[1][name]' value='bob' />
  <input name='people[1][age]' value='14' />

  <input name='people[2][name]' value='bubba' />
  <input name='people[2][age]' value='15' />
</form>

Desired output:
{
  "people": [
    {"name": "fred", "age": "12"},
    {"name": "bob", "age": "14"},
    {"name": "bubba", "age": "15"}
  ]
}

Actual output:
{
  "people": { // <-- people is not an array. darn.
    "0": {"name": "fred", "age": "12"},
    "1": {"name": "bob", "age": "14"},
    "2": {"name": "bubba", "age": "15"}
  }
}

I think that this expectation is consistent with what other libraries do, since I've used this form structure in the past. Any chance this form structure could be supported?

Have you tried removing the 0, 1, 2 from between the [] and just leaving []. I think the lib detects that and will make the array.

Yes, I tried that, but here's the result...
<form>
  <input name="people[][name]" value="fred">
  <input name="people[][age]" value="12">

  <input name="people[][name]" value="bob">
  <input name="people[][age]" value="14">

  <input name="people[][name]" value="bubba">
  <input name="people[][age]" value="15">
</form>

{
  "people": {
    "][name": ["fred", "bob", "bubba"],
    "][age": ["12", "14", "15"]
  }
}

For anyone else who may need this, I've temporarily worked around my issue by using this module to parse the form into a querystring and the qs module  to parse the querystring into an object.

So the issue is with using [] followed by another [] it seems.
What if you omit the first []? Does the library figure out you have
multiple?
On Monday, May 11, 2015, Stephen John Sorensen<EMAIL_ADDRESS>wrote:

Yes, I tried that, but here's the result...

{
"people": {
"][name": ["fred", "bob", "bubba"],
"][age": ["12", "14", "15"]
}
}
—
Reply to this email directly or view it on GitHub
https://github.com/defunctzombie/form-serialize/issues/18#issuecomment-100917555
.

I assume you mean like this:
<form>
  <input name="people[name]" value="fred">
  <input name="people[age]" value="12">

  <input name="people[name]" value="bob">
  <input name="people[age]" value="14">

  <input name="people[name]" value="bubba">
  <input name="people[age]" value="15">
</form>

That gives the following output:

By the way, I created a test for this on runnable.com to make it easier to test out different variations: http://runnable.com/VVDyUCD_d_hq0H8h/form-serialize-issue-18

@jxson you worked on nesting support most recently. Thoughts about this?

hmm, yeah. The nesting support I added was because I was doing the same exact thing @spudly was with the qs module. I think the expectation for name attributes with numbered indexes should do what @spudly initially pointed out.
I am not sure something like this be serialized to though...
<form>
  <input name="people[][name]" value="fred">
  <input name="people[][age]" value="12">

  <input name="people[][name]" value="bob">
  <input name="people[][age]" value="14">

  <input name="people[][name]" value="bubba">
  <input name="people[][age]" value="15">
</form>

ping on this issue. Would entertain a PR to look into this if still desired.

I can take a crack at it, might take a bit since I am traveling...

Just a heads up on this: I have a solid fix for this issue and #23 just cleaning up code and adding more tests before sending a PR.


[SEO] Don't let bot crawl front-edge/front-qa (duplicate content bad for SEO)

Description
Fixes https://github.com/dust-tt/tasks/issues/1280 (go there for context)
Risks
Not that I can see
Deploy
Front

LGTM
make sure you apply infra before you deploy front

:pray:  useful reminder :+1:


Use ENGLISH Locale while writing out the Infinispan subsystem namespace

A user has reported an bug in the forum https://developer.jboss.org/message/960348#960348 where, while running the server under a non-English Locale and using the CLI to create resources, leads to Infinispan writing out a subsystem configuration which doesn't load on server reload.
The commit here fixes that issue by using the ENGLISH Locale for writing out to the configuration file. My minimal look at the code suggests that this is very specific to the Infinispan subsystem because of the way it keeps track of the namespace URIs. Someone might have to run tests or check the code extensively, to make sure there are no other subsystem doing something similar.

Thanks, Jaikiran. I've opened https://issues.jboss.org/browse/WFLY-6857 for this. Do you mind updating the commit message to reference that?

Done!

@dmlloyd just now saw your comment, after merging. Probably better, yes, although I don't think it should matter for this case. We can correct this before .Final if you think it's worth it.

Nah it's just something that has been in the back of my mind.  Now it is also in the back of your mind :)

@jaikiran @bstansberry @dmlloyd This is a bigger story though, the server does not even boot on e.g. Hindi systems, opened https://github.com/wildfly/wildfly/pull/9056 to fix this everywhere and using ROOT locale.


Add wiki page with faq, basic/advanced setup

Can you add wiki page/extend readme with faq, project setup.
Description of tools for convert iso's and usage usage of them, etc...?

https://bitbucket.org/ShaolinAssassin/open-ps2-loader-0.9.3-documentation-project/wiki/Home

@ElPatas1
This appears to be resolved, are we able to close this?

Closed.
Best regards.


Handle .fireAndForget scheduling internally

Just an idea following a discussion about .fireAndForget. Not too sure of the implications but thought I'd check if it's worth exploring further.

I'm aborting mission... for now :P Just realised this wouldn't do what I was thinking it would.


Upload progress bar

Google Chrome 53
function addXhrProgressEvent() {
    if (!$.trumbowyg && !$.trumbowyg.addedXhrProgressEvent) {

addXhrProgressEvent would not setup the event listener because !$.trumbowyg would always return false. I mended this by just removing it from the if statment.
Then the progress bar it self wouldn't move, I found that
//original
progressUpload: function (e) {
    $('.' + prefix + 'progress-bar').stop().animate({
        width: Math.round(e.loaded * 100 / e.total) + '%'
    }, 200);
},

this animate function was doing some weird things so I removed it and just set the width, there is already a transition style on the progress bar so it will still animate.
//modified
progressUpload: function (e) {
    $('.' + prefix + 'progress-bar').width(Math.round(e.loaded * 100 / e.total) + '%');
},

Fixed thanks for you :)


Yapf

Added yapf changes to make the code look a bit nicer. All changes are entirely cosmetic, nothing else.

I'm sorry for whoever has to code review this.

also, people should merge right away or this'll be a nightmare later

Yeah that’s the logic. A strict style guide that we can enforce with one line of code will be useful as we grow. Not sure I love every aspect of it either; we can create a style file for yapf if you have preferences.


Docker images with Debian

We need base-debian-12.dockerfile, debian-12.dockerfile and  debian-12-pg.dockerfile recipes (see Ubuntu recipes here https://github.com/userver-framework/userver/tree/develop/scripts/docker). With those recipes we would be able to run CI on Debian and make sure that the build dependencies are OK.
Internal ticket: EDUPRJ-280

I'm working on this one.


Character controller collides with sensor colliders (Typescript)

Built in Character controller collides with sensor colliders
package: @dimforge/rapier3d
version: 0.11.2

I have the same issue with a sensor collider in Bevy.

This defeats the purpose of sensor, issue still there :/

And the issue is still there, cannot use sensors at all

The character controller takes an optional filterFlags?: QueryFilterFlags argument. To exclude sensors, simply specify QueryFilterFlags.EXCLUDE_SENSORS:
characterController.computeColliderMovement(collider, desiredMovement, QueryFilterFlags.EXCLUDE_SENSORS);

Closing this now since this is a usage issue.
Please reopen if excluding the sensors this way doesn’t work as expected.

I will say I would have expected that to be the default (as it is how sensors interact with other types of colliders), but thanks for pointing us in the right direction. Maybe it is worth mentioning on the characther controller guides that they collide with sensors by default?


Volar-ized Glint Roadmap

Overview
This roadmap absorbs the contents of the now-closed Language Tooling RFC.
Now that the initial Volar Refactor PR has been merged, there a number of next steps to pursue before Glint 2.0 is ready for primetime.
Roadmap

[x] Fix test suite, all green tests, via test.skip() (and add to this issue for any remaining feature TODOs)

[ ] - [x] Fix glint --build --clean specs (broken since Volar -- cleans everything but .gts file outputs). Johnson thinks it can be supported, needs upstream fix.

[x] Support for .gjs (and passing tests)

[ ] Support for .hbs and companion files

Reinstate disabled CI:

[ ] eslint
[ ] windows (blocked on volar 2.4.0 breaking stuff, discussion here)
[ ] type-tests
[ ] test-floating-deps
[ ] test-ts-nightly

[ ] Figure out how to resolve conflicts/overlapping with vsc-ember-syntax https://github.com/lifeart/vsc-ember-syntax/issues/72

I need to review how/why I inlined portions of that addon into this branch; I believe it's because otherwise Glint/Volar can't determine a language ID for .gts files

[ ] get all {{ @glint-expect-error }} and related directives working

[ ] constrain data objects on nested embedded codes https://discord.com/channels/1192759067815464990/1252021970443440128/1252298644695158874

CI is green; now to unskip and fix a number of specs where the underlying behavior is actually correct, but the test is broken.

Closing this in favor of a soon to be published GH project. All of the items contained within have been extracted/converted into GH issues.

@machty has the GH project been published? and if so, could you provide its URL plz?


multi-program mapping

Mapping logical to physical qubits for multiple programs at the same time.

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

@googlebot I signed it!

Fix merge conflicts. I wonder why this is not running the checks.

I don't have this conflict in my fork. I didn't add ant thing to this file. I don't know where it comes from. Do you have any idea?

How can I solve this error?
TypeError: Tuple[t0, t1, ...]: each t must be a type. Got cirq.GridQubit(1, 0).

How Can I define node type from networkx? I wrote networkx.NodeView and networkx.Graph.NodeView but didn't work

I get this error when I run autoformatter:
Comparing against revision 'upstream/master'.
check/format-incremental: line 116: black: command not found
Any idea to solve that?

I get this error when I run autoformatter:
Comparing against revision 'upstream/master'.
check/format-incremental: line 116: black: command not found
Any idea to solve that?

It's a missing tool - black.
To ensure I have all tools and dependencies, I usually run this from the Cirq folder:
pip install -r requirements.txt -r dev_tools/conf/pip-list-dev-tools.txt -r cirq/contrib/contrib-requirements.txt

pip install -r requirements.txt -r dev_tools/conf/pip-list-dev-tools.txt -r cirq/contrib/contrib-requirements.txt

Thanks! I tried it but I got "bad interpreter: Permission denied". I don't know why?

pip install -r requirements.txt -r dev_tools/conf/pip-list-dev-tools.txt -r cirq/contrib/contrib-requirements.txt

Thanks! I tried it but I got "bad interpreter: Permission denied". I don't know why?

That seems odd - do you have a virtual python environment correctly setup? We have the description for linux environments here: https://github.com/quantumlib/Cirq/blob/master/docs/dev/development.md#setting-up-an-environment. Are you on Linux or another platform? I'd love to provide a good experience that works for our contributors, so this conversation can help fixing it if it's lacking!

pip install -r requirements.txt -r dev_tools/conf/pip-list-dev-tools.txt -r cirq/contrib/contrib-requirements.txt

Thanks! I tried it but I got "bad interpreter: Permission denied". I don't know why?

That seems odd - do you have a virtual python environment correctly setup? We have the description for linux environments here: https://github.com/quantumlib/Cirq/blob/master/docs/dev/development.md#setting-up-an-environment. Are you on Linux or another platform? I'd love to provide a good experience that works for our contributors, so this conversation can help fixing it if it's lacking!

Anyway, I installed 'black' separately by 'python -m pip install black' and again tried './check/format-incremental --apply' but I got this error:
./check/format-incremental: /Users/mozafari/Fereshte_files/tools/#google/Cirq/newcirq/bin/black: /Users/mozafari/Fereshte_files/tools/#google/Cirq/newcirq/bin/python: bad interpreter: Permission denied

From the error you're reporting, it's possible that your virtualenv install is broken or out of date. You can try force-reinstalling it as described in the answer to this SO question:
pip install -U --force-reinstall virtualenv

From the error you're reporting, it's possible that your virtualenv install is broken or out of date. You can try force-reinstalling it as described in the answer to this SO question:
pip install -U --force-reinstall virtualenv

When I am inside my virtual environment? I did that and it finished successfully but again I have the same error

When I am inside my virtual environment? I did that and it finished successfully but again I have the same error

My understanding is that you would need to run the command outside the virtual environment, then create a new virtual environment with your fresh install of virtualenv.
If that fails, you can also view the necessary formatting changes by clicking the Details links for each of the failing checks, but you'll need to apply those changes manually. This can be tedious, but it should at least allow you to move forward while we get the core issue sorted out.

Anyway, I installed 'black' separately by 'python -m pip install black' and again tried './check/format-incremental --apply' but I got this error:
./check/format-incremental: /Users/mozafari/Fereshte_files/tools/#google/Cirq/newcirq/bin/black: /Users/mozafari/Fereshte_files/tools/#google/Cirq/newcirq/bin/python: bad interpreter: Permission denied

I had problem in my path: /Users/mozafari/Fereshte_files/tools/#google/Cirq/
By removing '#' from /#google/Cirq, solved!

Hi @fmozafari
Is this still stuck due to python environment problems?

OK, this looks to have gone stale.  I will close, but if you want to reopen, please don't hesitate.


OCW-14: Adjust setting to enable elasticsearch 2 on staging.

Coverage decreased (-0.01%) to 65.117% when pulling 4f5aac77d85bf964bd9615e18e970402cd6d7d9f on OCW-14 into 891c3d50f7f4e40fbf963518db12abf9e0edab27 on master.