Is "write a program" accurate?

We have a lot of descriptions that say "write a program". As https://github.com/exercism/exercism.io/issues/2746 explains, this may set up a wrong expectation from students that they are writing e.g. something with a main function (or equivalent entry point in language of choice).
But it seems that most tracks I've seen simply ask the student to write a function. Most languages then verify the behavior with unit tests against that function.
In the interest of setting up expectations correctly, shall we replace "write a program" with "write a function"?
Things I would like to check before doing that:

Are there any tracks that do, in fact, expect the student to write a full-fledged program?
Are there any tracks in which the equivalent concept is not called a function? (Perhaps it's called  a procedure instead? Or something completely different?)

Even if there aren't, what if we add such a track in the future?
If there currently are or could be in the future, should we do anything about it? In other words, do we decide that we'll simply use a "function" as a generic terminology, regardless of how this concept maps onto the target language?

Anything else I'm missing?
Appendix:
$ grep -li "write a program" *.md
change.md
crypto-square.md
etl.md
food-chain.md
grains.md
luhn.md
protein-translation.md
scale-generator.md
sum-of-multiples.md
transpose.md
two-bucket.md

$ grep -li "write a program" *.yml
allergies.yml
all-your-base.yml
anagram.yml
beer-song.yml
binary-search-tree.yml
binary-search.yml
binary.yml
change.yml
flatten-array.yml
food-chain.yml
gigasecond.yml
grains.yml
hamming.yml
hello-world.yml
hexadecimal.yml
house.yml
kindergarten-garden.yml
largest-series-product.yml
leap.yml
luhn.yml
matrix.yml
minesweeper.yml
nth-prime.yml
ocr-numbers.yml
octal.yml
paasio.yml
palindrome-products.yml
parallel-letter-frequency.yml
pascals-triangle.yml
phone-number.yml
point-mutations.yml
protein-translation.yml
proverb.yml
queen-attack.yml
raindrops.yml
rna-transcription.yml
robot-name.yml
saddle-points.yml
say.yml
scale-generator.yml
scrabble-score.yml
secret-handshake.yml
series.yml
sieve.yml
space-age.yml
sum-of-multiples.yml
transpose.yml
triangle.yml
trinary.yml
twelve-days.yml
two-bucket.yml
word-count.yml
wordy.yml
zebra-puzzle.yml

Perhaps "write a program" is the most generic way to address this?  In the case of Ruby, I don't think I have yet to see a "Write a method" request, on its own, the final result is usually a class or a method, and many solutions don't require either, which is perfectly fine with me.  It leaves the implementations open.
Once you ask for a function or procedure or method, you are probably being too specific, even the tests in many cases may be considered too specific, driving a certain way to solve something, which may be impossible to avoid, or just may be hard to avoid.

What were if we ask for writing a library? This leaves everything open
while not suggesting the io stuff.
Victor Goff<EMAIL_ADDRESS>schrieb am Sa., 6. Aug. 2016 12:20:

Perhaps "write a program" is the most generic way to address this? In the
case of Ruby, I don't think I have yet to see a "Write a method" request,
on its own, the final result is usually a class or a method, and many
solutions don't require either, which is perfectly fine with me. It leaves
the implementations open.
Once you ask for a function or procedure or method, you are probably being
too specific, even the tests in many cases may be considered too specific,
driving a certain way to solve something, which may be impossible to avoid,
or just may be hard to avoid.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/exercism/x-common/issues/321#issuecomment-238016688,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADmR_PGOgLAAQOLuc91LxC8NiewpcKhks5qdF_XgaJpZM4JeKmp
.

"write some code"?
(Library sounds big and scary.)

"Write code" sounds cryptic... just kidding.  But yeah, the same "generic" flavor.

I agree that library seemed potentially too large in scope. It's admitted to me that there are some tiny libraries, but maybe such a thing would not be as apparent to students.
"Write code" seems suffciently generic. I think I would prefer it to "write a function".
Here is another possible suggestion I have: Just remove the words completely. As an example, binary-search, before:

Write a program that implements a binary search algorithm.

Binary search, after:

Implement a binary search algorithm.

Seems to avoid duplication if every exercise has "write code" in common

I agree with this, the removal allows the "obvious" interpretation regardless of terms the individual user should choose to replace with the implied statement.  We don't care if they write it or craft it or conjure it, whatever it is, as long as the tests pass and they get the learning experience from it.  (And even the tests passing are only a guide and a common reference)

I like the idea of removing the words where possible, and if not, go with a super generic "some code" to avoid the but this is not an X argument.

Here's a PR for this: #528

Looks like it was merged in, so closing.

Thanks @kotp I forgot to close this.

No problem.  Don't forget that if there is a "fixes #000" in the commit, it will do so automatically when they are brought in.

Thanks for the explanation @kotp. I had "addresses #000" in the commit, so it didn't pick that up as a keyword.


bug: Composite escape keys not working when recording a macro

Check the following:

[X] I have checked the behavior after setting "vscode-neovim.neovimClean" in VSCode settings.
[X] I have read the vscode-neovim docs.
[X] I have searched existing issues.

Neovim version (nvim -v)
1.18.0
Operating system/version
Windows 10
Describe the bug
When trying to record a macro that inserts something, I can't use my composite escape keys "jk" and "kj". I have checked and setting only one of the options (either jk or kj) does not fix the issue.
An interesting detail is that the editor somehow still recognizes that a composite escape key sequence is starting, as it lags behing if I type those keys. But right after I press the second one, it types it normally instead of escaping.
Steps To Reproduce

Start a macro with q
Enter insert mode
Attempt to escape with the composite escape key

Expected Behavior
The expected behavior is escaping the same way both when recording a macro and when not.

Duplicate of #1959


[bug] Fix for signMessage via DeepLink to MetaMask mobile app

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
1.3.0
wagmi Version
1.4.7
Current Behavior
Sign Message button doesn't respond to MetaMask mobile app on iOS, it looks issue of DeepLink.
Expected Behavior
Expected to move MetaMask mobile app, and user can sign.
Steps To Reproduce
No response
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
I found that<EMAIL_ADDRESS>doesn't work, but wagmi examples page below works with MetaMask mobile app.
https://wagmi.sh/examples/sign-message
And I confirmed that<EMAIL_ADDRESS>with<EMAIL_ADDRESS>works.
This is diff of wagmi between 1.4.7 and 1.4.9.
https://github.com/wevm/wagmi/compare/wagmi<EMAIL_ADDRESS>Could you please add minor release to just upgrade<EMAIL_ADDRESS>?

@rnakashiro We've updated the latest wagmi and viem version in this PR: https://github.com/rainbow-me/rainbowkit/pull/1653
Can you let me know if it works now ?

Hi, thank you for the updating, I confirmed the expected behavior 😄

@rnakashiro Ok let me know if there is any other questions you have. If not i'll close this ticket.


RangeCalendar should change month to show range when range is set outside of currently displayed month.

Issue type
I'm submitting a ...  (check one with "x")

[x] bug report
[] feature request
[] question about the decisions made in the repository

Issue description
When using the RangeCalendar component, I have 'shortcut' buttons below the calendar that allow the user to quickly select a date range. (e.g. "Last Week, a month ago, two months ago, etc.")
When they press one of the buttons it updates the selected range of the calendar to the specified range.
If I choose a range that is outside of the currently displayed month, the calendar view doesn't change to show the new selection.
This makes it seem as though the button isn't working, even though it is. (The range indicator is displayed correctly when I am already looking at the date range before pressing the button.)
I can't seem to find a way to make the calendar display anything but the current month. Sorry if I'm missing something obvious here.
Current behavior:

Load RangeCalendar component on screen. (
Set range prop programatically with start/end dates outside of currently displayed month.
Nothing happens.

Expected behavior:

Load RangeCalendar component.
Set range prop programatically with start/end dates outside of currently displayed month.
Calendar changes view to new selected date.

This could be enabled/disabled if needed with a "alwaysShowSelectedRange" prop.
Steps to reproduce:

See sample app.
Related code:
import React, { useState } from 'react'
import { RangeCalendar, Button, Layout } from '@ui-kitten/components';

const MyControlledCalendar = ()=> {

    const [calendarRange, setCalendarRange] = useState(
        {
            startDate: new Date(),
            endDate: new Date(),
        });

    return (<Layout>
        <RangeCalendar
            range={calendarRange}
            min={new Date(0)}
            max={new Date()}
            onSelect={(range) => {
                setCalendarRange(range);
            }}
    />
        <Button
            status='primary'
            onPress={()=>setCalendarRange({
                startDate: new Date('Jan 01 2020'),
                endDate: new Date('Jan 05 2020')
            })}>
            Set date in the past.
        </Button>
        </Layout>)
}

Other information:
Node: 10.19.0
Expo  app running on iPhone 7
<!--
Node, npm: `node --version` and `npm --version`
OS: Windows (7/8/10). Linux (incl. distribution). macOS (El Capitan? Sierra?)
Browser: Chrome/Safari/Firefox/etc?
-->

<!--
Check your `package-lock.json` or locate a `package.json` in the `node_modules` folder.
-->

Sorry-- wrong tab.


Shorten the file names and avoid using windows incompatible symbols in file names

Many file names have the ':' inside which is not allowed in Windows and hence the repository is not getting cloned.
Also many file names are really big which is why Windows is throwing an error while saving them, which is causing unecessary problems. Please shorten the file names and remove the semicolon from the file names.

Had the same issues

Illegal file names for windows systems haa been fixed.


feat: sessions in db

#36
gestion des sessions en db

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

GitHub release
v1.12.0

Your semantic-release bot :package::rocket:


Add support for old package.json license format

I am not sure we should support this here, because it's a deprecated format as we can see here: https://docs.npmjs.com/files/package.json#license so libraries should just adapt to it.
However, since there is no harm on having it, it could be nice to support that old format.
Reference: https://github.com/CodeSeven/toastr/pull/604

As you say, there's no harm. Thank you for your contribution!

Ah nice, I just bumped into this problem as well and was about to open a PR to fix this 😇
Is a bugfix release planned anytime soon?
Thanks! 🙏

@lneves12 @emmenko just published 0.0.8 with these changes!

Oh wait, I think this is not entirely fixed.
In my case, a package has this format

So it's like singular license as an object. But this PR checks for package.licenses only. I can spin up a new PR to make the check more robust.

What a big mess it was before they implement the new rule to follow SPDX format :P I didn't notice that was possible too. Indeed, it would be nice to have it more bullet proof ;)

Sure, no worries 😉 I just opened a PR and added some tests. Have a look please 😇


Signature file definitions don't effect type inference

When a function's types are specified in a signature file and the parameters for that function in the corresponding implementation file do not have their types specified the parameters are inferred to be generic.
Repro steps

create impl and sig files

Example01.fsi
module FCS.Example01

val fn001 : int -> string -> unit

Example01.fs
module FCS.Example01

let fn001 num txt = ()

see error &

Expected behavior
num inferred to be int and txt inferred to be string
Actual behavior
num inferred to be 'a and txt inferred to be 'b
Known workarounds
specify the types in the implementation file
Related information
Provide any related information

F# 4.0
Operating system win10
.NET Runtime, CoreCLR or Mono Version .net 4.5

This is by design. Please submit to fslang-suggestions?


Ansible operator available at quay.io/operator-framework/ansible-operator are having vulnerabilities

Bug Report
Recent images of ansible operator are showing the multiple vulnerabilities.
Details of Vulnerabilities --> here
We are using quay.io/operator-framework/ansible-operator as base image to build image for operator

What did you do?
We are using this as base image to build image for operator
What did you expect to see?
Image with fixes for vulnerabilities.
What did you see instead? Under which circumstances?

Environment
Operator type:

/language ansible

Thanks

Seems like the vulnerability are propagated from ansible-operator-base

@Jainbrt Are you using the latest version available for ansible?
Could you please let us know what is tag where are you checking it?
c/c @fabianvf @jmrodri

@camilamacedo86 quay.io/operator-framework/ansible-operator:v1.11.0 base image only shows 4 high severity vulnerabilities only.

@camilamacedo86 @fabianvf @jmrodri Could you please help us understand for the fix update, if any progress is there ?

@jberkhahn @asmacdo thanks for your attention. When do we expect v1.13.0 to be released, as we are dependent for this fix to be in for our next product release in next 10 days?

@Jainbrt the base-base image is rebuilt, the user-facing base image will be rebuilt when I cut the v1.13.0 release today.
Apologies for the slow response, we get a lot of GH pings, so they are easy to miss. If it is important for us to get something looked at quickly, please feel free to chat with us on Kubernetes slack #operator-sdk-dev. Also feel free to join our community triage meetings. https://github.com/operator-framework/community/

Thanks @asmacdo for your kind response and help on this issue. Sure I will chat with you if anything urgent comes up.
BTW I still see 3 medium vulnerabilities with master image, do we have any plan to fix them also in coming releases ?

@Jainbrt yes, I ran another build and the security scan showed no vulnerabilities. This image will be included in the v1.13.0 release.

We had an undocumented manual step that was required in the release process. This gap has been identified and documented, so these images should not be this far out of date again. https://github.com/operator-framework/operator-sdk/pull/5261

Thanks @asmacdo for the help here.


Notifications endpoint

currently cs3 has no notification mechanisms, but we should plan and collect ideas so we can collaborate on this at the conference, eg:

[ ] long polling?

can be done efficiently with golang
http vs grpc

[ ] do we need a CS3 endpoint?
[ ] needs to aggregate changes from various resources

seperate service with it's own cache
storage providers the user has access to
sharing events
actvities app

[ ] document how to disable the polling using capabilities

Can we have at least a dummy page for this so web browser console logs doesn't complain?

https://github.com/cs3org/reva/pull/417

@madsi1m in theory notifications can be disabled on your configuration:
https://github.com/cs3org/reva/blob/9db3db0b635220af892be62b55207176aa88159e/internal/http/services/owncloud/ocs/capabilities.go#L209-L214

Is that a different thing? The requests i see come from Phoenix

to be entirely clear @madsi1m, the error you're seeing in the console is

yup, the same, the pull request sends a 200 to that endpoint

Needs an ADR


Wide characters in scoreboard have huge font size

Wide characters in the scoreboard (example: öäüßõ) have a huge font size (but the characters actually show correct!).

I actually like that bigger font, maybe just have it for all nicknames, even those without wide/special characters. It is just a minor annoyance that there are two different fonts and sizes.

The font could be customized in "\steamapps\common\Sven Co-op\svencoop\captionmod\CaptionScheme.res"
SmallScoreboardFont is for scoreboard playernames, LargeScoreboardFont is for scoreboard server name.


ci: migrate to GH actions

Summary of PR
Migrate CCI workflow to GH actions. I split it into 2 workflows one for CI which runs tests and listing and CD for building and deploying to NPM
For Maintainers

[x] Setup secret NPM_BOT_TOKEN

Time spent on PR
30 mins
Demo
For some reason npm run publish-coverage is not working. I don't know if this is something I need add a secret or something for coveralls.
https://github.com/saihaj/ShabadOSgurmukhi-utils/actions/runs/324357425
Linked issues
Ongoing migration process
Reviewers
@Harjot1Singh @bhajneet

I rebased mine with dev. I think it probably has something to do with readme 🤷🏼♂ ?


Fix negative matching/filtering

Previously, the highlight function returned true immediately upon finding any match, without checking whether any negative matches should be preventing the match.

Can you give an example of cases where the previous functionality didn't work correctly, but do with the new code?

Any case where a positive match is a substring of a negative match, and the
negative matches are specified last.
"myAltNick -stringContainingMyAltNickIDoNotWishToSee"
On 18 Feb 2016 6:13 AM, "John Reese"<EMAIL_ADDRESS>wrote:

Can you give an example of cases where the previous functionality didn't
work correctly, but do with the new code?
—
Reply to this email directly or view it on GitHub
https://github.com/jreese/znc-push/pull/190#issuecomment-185355966.

Ok, given that example, the fix makes sense to me.  Although in that context, the push variable seems to no longer be named appropriately.  Can you update this fix to rename it to something like ignore or negate_match or similar, so that it's more clear going forward?

Yeah, I agree that makes sense. I've pushed another commit.

Cheers!


[HOW-TO] Upload an excel or a CSV file

Question
**Hello guys,
I want to ask  how do easyrepro upload files?  Hoping for your immediate response**

hello, did you find any solution for this please? @renzo305

renzo305, haidykhaled
Try this https://stackoverflow.com/a/65929899/8951897


T-Shirts on Store page need real photographs

These t-shirts need real photos. The descriptions say: (Real-life photos coming soon!)
I just had the black terminal shirt shipped to me, and when I get it in I'm going to take a few photos for your consideration. While we're at it, they could possibly be retouched too (blue elementary shirt has a yellow background).

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11387110-t-shirts-on-store-page-need-real-photographs?utm_campaign=plugin&utm_content=tracker%2F10548672&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F10548672&utm_medium=issues&utm_source=github).

Sources: https://drive.google.com/file/d/0BwfNEizpnybJN0VYVkRmVTdoNUE/view?usp=sharing
GPL

@alexgleason We just moved to MIT licensing, would you mind us using that to avoid doubling up?

@lewisgoddard Sure, MIT works. I was trying to make it compatible with the elementary logo since it's on my sleeve which I believe is GPL?
Btw when you say "we moved to MIT" do you mean just the website? Thanks.

Yes, the website license is now MIT, I believe it's more compatible with most of the libraries used.

Okay I updated the license on the link above to match the site's license. Use it if you'd like, but it looks like the shop is managed externally so someone with access should do it.
Also, @danrabbit Is the tan background on the blue elementary tee intentional? If not it can be cropped out so the background is handled by CSS. Of course the trade-off is a ~1 MB image vs ~250 kB.

I think a slightly off-white, away from the color of the shirt is technically preferable. For instance, a blue shirt would have a slightly orange-tinged background, and orange would have blue tint. That decreases the artificial brightness around the product and makes the colors appear more natural and full.

TBH, I don't think it's a good idea to use this image. It's pretty apparently fake.

@danrabbit I understand. If you think it's worth fixing I linked the source images above.
Out of curiosity how was the first photo taken?


Rename Framer Motion to Motion

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (5cae9ed) to head (4d85bdd).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #462   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           30        30
  Lines          298       298
  Branches        70        70
=========================================
  Hits           298       298

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Sysl release workflow is not working

Description
The sysl release machinery is not working. When building the release, go.sum is getting modified so the workspace is identified as dirty, which breaks the release. This is why we have not had a release since v0.29.0.
Steps to Reproduce

merge PR to master

Debug&Crash Output

   ⨯ release failed after 0.29s error=git is currently in a dirty state, please check in your pipeline what can be changing the following files:
 M go.sum

First issue:

go sum file is dirty fixed in https://github.com/anz-bank/sysl/pull/754

Next issue:
The makefile isn't getting copied over correctly when the docker image is built using goreleaser
Step 6/11 : COPY . .
---> 64f086f7a424
Step 7/11 : RUN make build
---> Running in 747c75dbccc6
make: *** No rule to make target 'build'.  Stop.
The command '/bin/sh -c make build' returned a non-zero code: 2
: exit status 2

Steps

[ ] Merge workflows into one
[ ] Add check-tidy so that branches with dirty go.mod and go.sum can't be merged in
[ ] Build and upload the docker image using a github action


observation about intersection-multi-agent-v0

Thank you for your excellent environment. I can't understand how its coordinates and speed are defined in this scene. I can't seem to find the relevant documentation. In addition, it seems that I can't get satisfactory results with your configuration as follows.
python experiments.py evaluate configs/IntersectionEnv/env_multi_agent.json
configs/IntersectionEnv/agents/DQNAgent/ego_attention_2h.json
--train --episodes=3000
To be precise, I got the result that one car would always stay in place.
Is it because of the update of the version?
According to my understanding of this code, I think you want all agents to use the same network and the same reward value for training. This setting makes me confused, which means that the training environment is unstable. Am I wrong？

I can't understand how its coordinates and speed are defined in this scene. I can't seem to find the relevant documentation.

See the documentation.

In addition, it seems that I can't get satisfactory results with your configuration as follows.
python experiments.py evaluate configs/IntersectionEnv/env_multi_agent.json
configs/IntersectionEnv/agents/DQNAgent/ego_attention_2h.json
--train --episodes=3000.  To be precise, I got the result that one car would always stay in place.

I got okay results when I tried it, which was a long time ago. But this project is mostly about defining the environment, not the agent. Feel free to experiment more, and try other algorithms / observations types / network architectures.

According to my understanding of this code, I think you want all agents to use the same network and the same reward value for training. This setting makes me confused, which means that the training environment is unstable. Am I wrong？

Yes, it is a cooperative setting where all agents are controlled by the same policy, and they all optimise the same joint reward function. I don't think there is anything particularly wrong with this setting, although others are definitely possible, again feel free to experiment. This is just a simple implementation of a multi-agent task.

I have noticed that you only add the results of each agent's interaction with the environment to the buffer in your code, which means that the environment is unstable for each agent.
This is similar to but somewhat different from vdn. Although each agent uses the same policy network, in the construction of the loss function, in fact, each local q value is added up to approximate the total q value.  Joint rewards are also used in loss. However, in your settings, loss seems to be the q value for each agent, which seems to mean that multiple agents only increase the number of samples taken.  If each agent trains separately, the reward value should be obtained by each agent itself.  Therefore, it is difficult for me to understand your current thinking when constructing the loss function. Can you show more detail?

You have provided an explanation document for the environment, which mainly introduces the coordinate meaning of highway v0, which is indeed easy to understand. But when the environment becomes a intersection, the meaning of speed and location x and y begins to become confusing. Is there a separate document?

You have provided an explanation document for the environment, which mainly introduces the coordinate meaning of highway v0, which is indeed easy to understand. But when the environment becomes a intersection, the meaning of speed and location x and y begins to become confusing. Is there a separate document?

The meaning is the same for the intersection environment: vehicles also have an x/y coordinate and a vx/vy velocity. Different options can be used, e.g. absolute coordinates are used by default for intersection env, while relative coordinates are used for highway. I invite you to take a look at the KinematicsObservation class, and the environment configuration.

I have noticed that you only add the results of each agent's interaction with the environment to the buffer in your code, which means that the environment is unstable for each agent.
This is similar to but somewhat different from vdn. Although each agent uses the same policy network, in the construction of the loss function, in fact, each local q value is added up to approximate the total q value. Joint rewards are also used in loss. However, in your settings, loss seems to be the q value for each agent, which seems to mean that multiple agents only increase the number of samples taken. If each agent trains separately, the reward value should be obtained by each agent itself. Therefore, it is difficult for me to understand your current thinking when constructing the loss function. Can you show more detail?

You are right that each agent could optimise its own per-agent reward, but here I chose another option: each agent receives an "altruistic reward", which is the sum of the per-agent rewards. Thus, each agent optimises not only its own velocity and safety, but also acts in an altruistic manner so as to also increase other agents' safety and speed (e.g. dont cut in before other agents, make them brake too hard, etc.). You can also see it as a specific case of the per-agent rewards, where they are all equal (to a sum of several terms), there is nothing wrong with that. And we can train Q-values for this (joint) reward function as you would for a single-player RL algorithm that assumes that other agents have fixed policies (local approximation).
Again, if this setting is not suitable for you, feel free to change it :) I don't have particularly strong opinions about this, I mostly made this choice so that the environment returns a scalar reward (the same for all agents) instead of a vector reward (different for each agent), so that it can be used directly with single-player RL algorithms and libraries (which expect scalar rewards).

You are right that each agent could optimise its own per-agent reward, but here I chose another option: each agent receives an "altruistic reward", which is the sum of the per-agent rewards. Thus, each agent optimises not only its own velocity and safety, but also acts in an altruistic manner so as to also increase other agents' safety and speed (e.g. dont cut in before other agents, make them brake too hard, etc.). You can also see it as a specific case of the per-agent rewards, where they are all equal (to a sum of several terms), there is nothing wrong with that. And we can train Q-values for this (joint) reward function as you would for a single-player RL algorithm that assumes that other agents have fixed policies (local approximation).
Again, if this setting is not suitable for you, feel free to change it :) I don't have particularly strong opinions about this, I mostly made this choice so that the environment returns a scalar reward (the same for all agents) instead of a vector reward (different for each agent), so that it can be used directly with single-player RL algorithms and libraries (which expect scalar rewards).
I see. Thank you very much for your help. I am learning the reward value settings for multi-agent systems, and your explanation has provided great help.

The meaning is the same for the intersection environment: vehicles also have an x/y coordinate and a vx/vy velocity. Different options can be used, e.g. absolute coordinates are used by default for intersection env, while relative coordinates are used for highway. I invite you to take a look at the KinematicsObservation class, and the environment configuration.

I seem to understand. In fact, there are positive and negative speeds for cars on the east-west direction because some cars move from west to east （positive) and some cars move from east to west (negative).


Update oneDAL to 2023.0.1; add patch for setup; update tests

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


Bug - undefined pose using teleportation.

pose is not defined

Thanks, yeah we still need to hook up the VR in engineless.

👍

Fixed, thanks!


Fixed javadoc for processBundle

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

The 2nd li element for processBundle(InputT... inputElements) was missing a proper closing tag.

I spared a Jira ticket since it's a very small change, hope it's ok.
R: @kennknowles

LGTM, thanks,


Allow trailing comma in tuples

We had a bug in the formatter that was adding a trailing comma to multi line tuple but this isn't something we actually support.
From bug https://github.com/microsoft/cadl/issues/787, bug fix the formatter to remove the trailing comma but question is should we allow the other way around and have the formatter add it automatically.
TS/JS support that and prettier (at least our config) will automatically add one. This make it nicer to work with multi line arrays, reducing the number of line changed in diff when adding a new entry at the end and make it easier to move items around.

+1 we should allow it.
I am less opinionated on what the formatter should do, but I think following the prettier defaults makes sense. Having the trailing one makes nicer diffs when you add something to the end.

I'm thinking we should maybe consider allowing trailing in all lists and leave it to the formatter if we have opinions about when not to use it (like in a single line).
When I refactored everything to go throuhg parseList helper, I preserved the cases where trailing delimiter is not allowed, but there's an explicit trailingDelimeterIsValid: false for that. We could consider removing that and saying it's also valid. We should still add formatting tests for all the lists this impacts, and update the grammar if we go down this route.

I made example of all the places:
Playground Link

Proposal

Allow trailing comma in all lists. (Make everything NOT OK here OK.)
Allow formatter to do whatever comes easiest and is most consistent with JS/TS prettier defaults
Don't implement any formatter knobs yet. If that is desired, consider it as a separate, lower priority issue.

Implementation issue notes:

Delete trailingDelimiterIsValid, and behave as though it was always true.
Add parser tests for all cases.
Add formatter tests for all cases.
Update language grammar

Approved on 8/3 design meeting. #809 tracks implementation.


implement toc content

I think we need endpoint information in TOC.
Because, the URI is very important thing, and the user will search by it.
So, I add endpoint information and link in TOC.
And, I add default option and write readme, so we can release this feature.

I modified the default option and href :)

Thanks!


Unable to bindstream from firestore .snapshot().listen

Discussed in https://github.com/jonataslaw/getx/discussions/1712

Originally posted by CSharper63 July 30, 2021
Hi guys,
The app : Manage movie and who is currently watching the movie.
I'm facing of a problem. I try since 2 days to bind a stream from firestore query using a Rx<List> but I m unable to populate my list. I already test every lines of the following requests everything is working fine. Something goes wrong when try .bindstream().
My code :
My database.dart (file to interact with firestore)
Future<Stream<List<UserModel>>> viewersStream(String movieId) async {
    List<String> viewersIds = [];
    List<UserModel> viewers = [];

    await _firestore
        .collection(MOVIES_COLLECTION)
        .doc(movieId)
        .collection(VIEWERS_COLLECTION)
        .get()
        .then((value) {
      value.docs.forEach((element) {
        viewersIds.add(element.id);
      });

      return _firestore
          .collection(USERS_COLLECTION)
          .where(FieldPath.documentId, whereIn: viewersIds)
          .snapshots()
          .listen((event) {
        event.docChanges.forEach((element) {
          print("try adding to return list user: " + element.doc.id); // works
          viewers.add(UserModel.fromDocumentSnapshot(
              documentSnapshot: element.doc));
          print("user add to return list user: " + element.doc.id); // works
        });
        return viewers;
      });
    });
  }

then my movie_controller.dart :
void fetchingViewers(String movieId) async {
    if (movieId != null || movieId != "") {
      print("fetching viewers for movie " + movieId);
      _participants.bindStream(await Database().viewersStream(movieId));
      print("fetched viewers for movie " + movieId);
    }
  }

call stack error:

I hope someone could help me.
I created an issue with this problem because I can't find any information about this problem anywhere. I already tried in async/sync mode, nothing works. So hope any getx developer could help me to understand how bindstream method works.

hi,
I am a first time in this in github. however i can recommend you use only Stream to update the UserModel every time it is modified in the database, or only Future to update it only once, when the application is started for example, but not both. I have many Streams in my app that i'm developing, so i will write you a example to use bindStream correctly,I hope it helps you and everything works.
`
class MovieController extends GetxController {
// this is the controller of the user that you can update in the pages of the app
var movieId =
''.obs; //wait the user update the movie that want, i guess. From the UI
List viewersIds = [];
RxList viewers =
RxList(); // here it would have been the first error of your coude
// when the controller start
void onInit() {
ever(movieId, getvierwersIds()); //ever change in movieId, the callback gonna be getvierwersIds
viewers.bindStream(viewerStream());
super.onInit();

}
getvierwersIds() async {
await FirebaseFirestore.instance
.collection(MOVIES_COLLECTION)
.doc(movieId.value) //from get the real value of the observer
.collection(VIEWERS_COLLECTION)
.get()
.then((value) {
return value.docs.forEach((element) {
viewersIds.add(element.id);
});
});
}
Stream<List> viewerStream() {
// Stream that feach UserModel
if (movieId.value.isNotEmpty) {
//check null value
return FirebaseFirestore.instance
.collection(USERS_COLLECTION)
.where(FieldPath.documentId, whereIn: viewersIds)
.snapshots()
.map((event) =>
event.docs.map((e) => UserModel.fromFireStore(e)).toList());
} else {
return Stream.empty();
}
}
}
//i guess this is when the user tap or pressed the movie that want or something like that, so onTap() => movieId.value = 'value'
`
its an example that you can adapt. In UserModel one example to this code is:
`
class UserModel {
String? userId;
String? email;
UserModel({this.userId, this.email});
factory UserModel.fromFireStore(DocumentSnapshot data) {
return UserModel(userId: data.get('userId'), email: data.get('email'));
}
}
`
i do it in this way, sorry if i have some mistakes, I am Spanish.
Any question or doubt I will try to help so everyone learns.

Thank you very much for you anwser I will test this code and get back to you asap. 🙏🙏🙏

Hello, I tried what you provide me. It seems to work. I just have several questions. How to create a setter with RxString like movieId. I tried to set the value but always throw an error :
On tap event I try the following code to fetch my viewers stream :
Get.find<MovieController >().setCurrentMovieId = THE ID I PROVIDE;

 ════════ Exception caught by gesture ═══════════════════════════════════════════
The method 'call' was called on null.
Receiver: null
Tried calling: call("tzovkhW6IOgLp337Ujtv")
════════════════════════════════════════════════════════════════════════════════

My second question is can you explain me why in my first code version where I got my problem I can't return a list from stream in .then() from the first query ? I can't understand how to use bindStream properly.
Thanks you for your support !!!!!! @rus0n

What must I do to fetch my stream on document change ? Like using snapshot().listen ? I have trouble understanding how to use the future and store data in real time.

Hello,
It's two ways to set a variable, that depends how you initialize. For example:
Firstly, If you have to initialize with a value it's like:
var movieId = 'value'.obs //the last term means that it is a variable that is in observed mode

If you want that the variable doesn't need initialize, it's like:
var movieId = Rx<String>() ;

To set a value in the observer variable you only have to put:
movieId. value='value'; //you can put it in the UI or in the MovieController, don't need a setter.

Stream means that the value updates when it's a new value , in this case a new value in the database. Firstly, don't need listen, snapshot() shot when this happen. In my code you can see that I only put snapshot that after I put a map to convert it, in this map I get the documents of the databases that the values updated or new values, then I get another map of this documents to fetch the Model.
Future is only to call it onces and fetch one time, don't shot when the values were updated or new values from database.
If you want I can show you an example of my app or a video that how the data update in realtime synchronized with the database. It's works really good.
See you,  i hope it was helpfully @CSharper63

Hello,
It's two ways to set a variable, that depends how you initialize. For example:
Firstly, If you have to initialize with a value it's like:
var movieId = 'value'.obs //the last term means that it is a variable that is in observed mode

If you want that the variable doesn't need initialize, it's like:
var movieId = Rx<String>() ;

To set a value in the observer variable you only have to put:
movieId. value='value'; //you can put it in the UI or in the MovieController, don't need a setter.

Stream means that the value updates when it's a new value , in this case a new value in the database. Firstly, don't need listen, snapshot() shot when this happen. In my code you can see that I only put snapshot that after I put a map to convert it, in this map I get the documents of the databases that the values updated or new values, then I get another map of this documents to fetch the Model.
Future is only to call it onces and fetch one time, don't shot when the values were updated or new values from database.
If you want I can show you an example of my app or a video that how the data update in realtime synchronized with the database. It's works really good.
See you,  i hope it was helpfully @CSharper63

Thank you I think an example video could be good to understand concept.
For my obs I think I understand how does it work. But I made my project in MVC so I want to use the controller in view to set the value of movie Id each time in to. My problem is each call in my setter to change my RxString is call on null. I can't understand why
My call in my view :
Get.find<MovieController>().setCurrentMovieId = movie.id;

In my controller :
RxString partyId = RxString('no_movie'); // already try to use like var movieId = 'no_movie'.obs;
set setCurrentMovieId(String value) {
    try {
      this.movieId.value = value;
    } catch (e) {
      print(e);
    }
  }

But everytime I call the setter it's null so I can't use the ever function to update automatically the stream.
error :
I/flutter (19261): NoSuchMethodError: The method 'call' was called on null.
I/flutter (19261): Receiver: null
I/flutter (19261): Tried calling: call("29bzrYUImjIeyGn45WSZ")

Thank you for your time !

To set or get the value of movieId in the statelessWidget first after the build(context). (There are many options) but I will put the classic option, you have to put
final controller = Get.put(MovieController());

When I have to use the movieId, the setter is
controller.movieId.value = 'value';

The getter
controller.movieId.value

For you I think the best option in the MovieController is
var movieId = Rx<String>();

When I stay at home at night I will try to send you a video.

Thank you if you want I have telegram : @CSharper63 !


AttributeError when displaying dask-gateway cluster object in a notebook in latest pangeo-notebook mage

This came up in https://github.com/pangeo-data/pangeo-eosc/issues/22#issuecomment-1267403760.
With a newly deployed Daskhub, using pangeo/pangeo-notebook:2022.09.21 Docker image, when creating and displaying a dask-gateway cluster using:
from dask_gateway import Gateway
gateway = Gateway()
cluster = gateway.new_cluster(worker_memory=4)
cluster.scale(6)
cluster

The following error is displayed:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
File /srv/conda/envs/notebook/lib/python3.9/site-packages/IPython/core/formatters.py:921, in IPythonDisplayFormatter.__call__(self, obj)
    919 method = get_real_method(obj, self.print_method)
    920 if method is not None:
--> 921     method()
    922     return True

File /srv/conda/envs/notebook/lib/python3.9/site-packages/dask_gateway/client.py:1225, in GatewayCluster._ipython_display_(self, **kwargs)
   1223 widget = self._widget()
   1224 if widget is not None:
-> 1225     return widget._ipython_display_(**kwargs)
   1226 else:
   1227     from IPython.display import display

AttributeError: 'VBox' object has no attribute '_ipython_display_'

This does not prevent using the cluster.
This was not present with pangeo/pangeo-notebook:2022.09.06.
I did not dig further, just opening this issue if someone has a guess.

That was fixed in https://github.com/dask/dask-gateway/pull/609 (incompatibility with ipywidgets >= 0.8). There isn't a dask-gateway release with it yet, so you'll want to print(cluster) in the meantime.

Thanks @TomAugspurger! I could have search a bit more :wink:.
I'll let you close this issue if you feel like it.

is there a dask-gateway release now? we should bump here.

our pangeo-bot dispatcher to catch package updates needs a manual kickstart every 60 days :( not sure how to fix that but here is the issue https://github.com/pangeo-bot/dispatcher/issues/2

pangeo/pangeo-notebook:2022.10.18 now has dask-gateway=2022.10.0 I did a quick test on the staging hub and things appear to work but print(cluster) doesn't return anything, just displaying the client connected to the cluster:

also, you have to put the full URL into the dask dashboard to get it to work (so something like https://staging.us-central1-b.gcp.pangeo.io/services/dask-gateway/clusters/staging.0b841d0d64b9473d9864b3c9a150aac9/status)
Not sure who is responsible for bumping the image version on the production 2i2c pangeo hub?

@rabernat there's progress being made on https://github.com/2i2c-org/infrastructure/issues/1253 (https://github.com/2i2c-org/infrastructure/issues/1825 was just merged today).
In the meantime, you don't actually need to make PRs to the repo each time, any jupyterhub admin can manually modify the image with the configurator: https://docs.2i2c.org/en/latest/admin/howto/configurator.html


IGNITE-10075 .NET: Make a correct return of a custom type object from the Java service.

This PR is associated with a resolved ticket. Closing the PR, please reopen it if you still need it.


Issue In Stremio

Whenever I configure media Fusion with real debrid and watch anything,it works for sometime after which it just stop scraping links from debrid and doesn't even show in stremio

Hi, it might due to rate limit which set from elfhosted. 3 days back, we reduce it again.


 Thanks - really awesome lib.

Is there a way for me to check if the file of the FTPserver is different from the ones i have locally ?
i am always downloading file. Never uploading.
thanks in advance.

The best you can do is Stat() the file and check if the remote file size matches your local file. Of course, the files could be different but just happen to be the same size.

old issue, I know...
some servers(and a few public ones too) support some kind of checksumming
for example - proftpd has mod_digest which adds

XCRC (requests CRC32 digest/checksum)
MD5/XMD5 (requests MD5 digest/checksum)
XSHA/XSHA1 (requests SHA1 digest/checksum)
XSHA256 (requests SHA256 digest/checksum)
XSHA512 (requests SHA512 digest/checksum)

you will have to implement this yourself though

Thanks I didn't know about that. I'm updating this issue to track support for the checksumming in goftp.


wire can not recognize the alias any of interface{} in go 1.18.2

You can use go bug to have a cool, automatically filled out bug template, or
fill out the template below.
Describe the bug
cannot use (*sql.DB)(db) (value of type *sql.DB) as sqlc.DBTX value in argument to sqlc.New: wrong type for method ExecContext (have func(ctx context.Context, query string, args ...invalid type) (database/sql.Result, error), want func(context.Context, string, ...interface{}) (database/sql.Result, error))
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior.
Expected behavior
A clear and concise description of what you expected to happen.
Version
go1.18.2
Which version of Wire are you seeing the bug with?
Additional context
Add any other context about the problem here.

go 1.18.2
func (db *DB) ExecContext(ctx context.Context, query string, args ...any) (Result, error) {
lower version
func (db *DB) ExecContext(ctx context.Context, query string, args ...interface{}) (Result, error) {

Hey @xiazemin, I tried reproing your failure with the following two file, but wire ran successfully for me. Do you have a repro that I can follow?
==> main.go <==
package main

import "fmt"

func NewAny() any {
        return 0
}

func NewTest(any) int {
        return 0
}

func main() {
        fmt.Printf("got %d\n", InitializeTest())
}

==> wire.go <==
//go:build wireinject
// +build wireinject

package main

import "github.com/google/wire"

func InitializeTest() int {
        wire.Build(NewTest, NewAny)
        return 0
}

I believe I have the same or similar error.
First the error message I see (note the file in question is not a wire.go file):
wire: /home/blah/blah/blah.go:290:31: cannot use params (variable of type []interface{}) as []invalid type value in argument to db.Query
The line in question:
       params := make([]interface{}, 0)
...
< do stuff like adding items to params and query>
...
rows, err := db.Query(query, params...)`
db.Query is defined with '...any':
func (db *DB) Query(query string, args ...any) (*Rows, error) {
I FIXED it by uninstalling wire and reinstalling wire using go 1.18.4. I think the issue is that wire on my system was compiled long ago before Go 1.18 came out. 'any' was introduced in Go 1.18 so that old existing binary didn't know what 'any' was. It works just fine after cleaning out the old wire binary and reinstalling the same version, but compiled with Go 1.18

I believe I have the same or similar error.
First the error message I see (note the file in question is not a wire.go file): wire: /home/blah/blah/blah.go:290:31: cannot use params (variable of type []interface{}) as []invalid type value in argument to db.Query
The line in question:  params := make([]interface{}, 0) ... < do stuff like adding items to params and query> ... rows, err := db.Query(query, params...)`
db.Query is defined with '...any': func (db *DB) Query(query string, args ...any) (*Rows, error) {
I FIXED it by uninstalling wire and reinstalling wire using go 1.18.4. I think the issue is that wire on my system was compiled long ago before Go 1.18 came out. 'any' was introduced in Go 1.18 so that old existing binary didn't know what 'any' was. It works just fine after cleaning out the old wire binary and reinstalling the same version, but compiled with Go 1.18

I agree and verify.


chore(sidecar): use alloy dependency instead of individual packages

fixes #143

Great, thank you! Gonna merge this ignoring the CI as it makes sense to fix it in the base  branch


Update description

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (1dd19ae) compared to base (0b1e85d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #50   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines          346       346
  Branches        97        97
=========================================
  Hits           346       346

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

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


[SPARK-47030][TESTS] Add WebBrowserTest

What changes were proposed in this pull request?
This PR aims to add a new test tag, WebBrowserTest.
Why are the changes needed?
Currently, several browser-based tests exist in multiple modules like the following. It's difficult to find and run them.
common/tags/src/test/java/org/apache/spark/tags/WebBrowserTest.java
core/src/test/scala/org/apache/spark/deploy/history/ChromeUIHistoryServerSuite.scala
core/src/test/scala/org/apache/spark/deploy/history/HistoryServerSuite.scala
core/src/test/scala/org/apache/spark/ui/ChromeUISeleniumSuite.scala
core/src/test/scala/org/apache/spark/ui/UISeleniumSuite.scala
sql/core/src/test/scala/org/apache/spark/sql/execution/ui/UISeleniumSuite.scala
sql/core/src/test/scala/org/apache/spark/sql/streaming/ui/UISeleniumSuite.scala
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/UISeleniumSuite.scala
streaming/src/test/scala/org/apache/spark/streaming/UISeleniumSuite.scala

In addition, the previous ChromeUITest is designed to disable ChromeUI* suite and doesn't cover all WebBroser based test suite.
Does this PR introduce any user-facing change?
No, this is a new test tag.
How was this patch tested?
Pass the CIs.
Was this patch authored or co-authored using generative AI tooling?
No.

Thank you, @HyukjinKwon and @LuciferYang !


How to deploy wordpress with now-php ?

How to deploy wordpress with now-php ?

Hi @fydy
We don't maintain now-php here.
It is maintained by the community in this repository: https://github.com/juicyfx/now-php
Closing as a duplicate of https://github.com/juicyfx/now-php/issues/33


AddSVGObject doesn't work with 21.9.0

The SVG is not embedded/displayed in the output PDF.

@dirq
Please use the latest version of Aspose.PDF for .NET 22.5. If you still face problem, please share your input SVG and code example here for testing. We will investigate the issue and provide you more information on it.

Since there have been no updates or additional questions for more than half a year, the issue is considered resolved.


Use with ES6 imports or CommonJS requires

I want to use Dimple in my react app, which is using ES6 imports:
First, I did npm install --save dimple-js.  This adds the dimple-js folder to my project's /node_modules/.
Second, I try to import it from my component:
import * from 'dimple-js';

...

However, trying to import anything from 'dimple-js' throws a syntax error.
How should DimpleJS be imported?

hi! i can't remember where i found this but you actually have to use
npm install -S git+https://github.com/PMSI-AlignAlytics/dimple.git
in order to import (then you should import from dimple rather than dimple-js)
good luck! if you get dimple working in react, i would loooove to see how you did it

Dimple denpends on the D3. So we need install D3 too. Following is the way I make the ES6 import work:
// install the libraries first
npm i<EMAIL_ADDRESS>npm install -S git+https://github.com/PMSI-AlignAlytics/dimple.git

// import the libraries
import 'd3';
import dimple from 'dimple';


refactor(Algebra/Homology): use the new homology API

This PR refactors the construction of left derived functors using the new homology API: this also affects the dependencies (Ext functors, group cohomology, local cohomology). As a result, the old homology API is no longer used in any significant way in mathlib. Then, with this PR, the homology refactor is essentially complete.
The organization of the files was made more coherent: the definition of a projective resolution is in Preadditive.ProjectiveResolution, the existence of resolutions when there are enough projectives is shown in Abelian.ProjectiveResolution, and the left derived functor is constructed in Abelian.RightDerived; the dual results are in Preadditive.InjectiveResolution, Abelian.InjectiveResolution and Abelian.LeftDerived.

The organization of the files was made more coherent: the definition of a projective resolution is in Preadditive.ProjectiveResolution, the existence of resolutions when there are enough projectives is shown in Abelian.ProjectiveResolution, and the left derived functor is constructed in Abelian.RightDerived; the dual results are in Preadditive.InjectiveResolution, Abelian.InjectiveResolution and Abelian.LeftDerived.

This part of you message seems to have swapped some lefts and rights.

Thanks very much @jcommelin for all these reviews!
bors merge


RTCConfiguration.stunCandidateKeepaliveIntervalMs causes a  SIGSEGV crash

Setting a value to the stunCandidateKeepaliveIntervalMs field in RTCConfiguration object cause a SIGSEGV crash when creating peer connection.
val rtcConfig = PeerConnection.RTCConfiguration(iceServers);
rtcConfig.stunCandidateKeepaliveIntervalMs = 60000

After setting the value of stunCandidateKeepaliveIntervalMs to any integer, the crash  happens when I call
factory.createPeerConnection(rtcConfig, peerConnObserver)

Hi @sourish115, the new release, version 1.3.0, is now available and includes the m125 patches. Could you check if this issue still occurs with the updated release?

Thanks for your reply. Yes this issue still persists in version 1.3.0


AP Stream adapter: Update docs, add 'front matter'

Related to: prebid/Prebid.js#5508

adapter released with 4.2


Further issues barring entrance to the Mozilla Add-ons gallery

Your version was rejected because of the following problems:

Dynamic creation of script nodes can be unsafe if contents are not static or are otherwise unsafe, or if src is remote.
ng-inspector/data/inject.js
 5
The "mouseover", "mouseout" events are dispatched with high frequency and can cause performance issues. Use mutation observers instead https://developer.mozilla.org/en-US/docs/Web/API/MutationObserver

Tested on Ubuntu 14.04 with Firefox v38

is debatable, although we do create a script node, the contents are static and are not remote. What they consider unsafe exactly might be the bigger issue.

could be solved by either the suggested method, or better yet (imo) using delegate events, which is a long overdue optimization either way.

Well, that's fun. Was this the first version you pushed a few weeks ago, or is this an update that they're rejecting?

It's the second attempt. Took a while for them to review this time.
Getting in the extension gallery would be nice for the automatic updates, right now in the ng-inspector.org users can only download the .xpi directly :(

Damn -- sounds awfully similar to submitting iOS apps for Apple's approval.

Yep! This exact thought popped up in my mind haha

Have these issues been fixed yet? I am no longer able to use the xpi from your websites due to further restrictions to the addon signing policy :(
Firefo 41.0a2 (2015-08-06) - i.e. newest Developer Edition

Hey @esphen,
Thanks for the heads-up. For the time being (work-around), it looks like you have the option in the stable and nightly builds to disable the requirement for addons to be signed.
We're hoping to dig deeper into the Firefox issues soon (both the rejection from the gallery, as well as the new code-signing requirements).

Okay great, and thanks for the link! Please be advised that this workaround will only work until FF 41 hits stable, which is scheduled for 2015-09-22. Godspeed you guys!

Let the race begin :)
As a side-note, we have recently discussed having a separate build for either a bookmarklet, or just a built JS file that you could include in a project during development to expose ng-inspector functionality, without worrying about browser extension restrictions (this would also allow us to support IE).
I'm pretty sure @rev087 and I are on the same page about desiring that functionality. Having said that, the "when" is still up in the air.


[SPARK-48075] [SS] Add type checking for PySpark avro functions

What changes were proposed in this pull request?
Add type checking for to_avro and from_avro for PySpark.
Why are the changes needed?
If we perform type checking for arguments and output sensible errors when the type of arguments passed into the functions don’t match, we can give the user a better user experience
Does this PR introduce any user-facing change?
No
How was this patch tested?
Unit test
Was this patch authored or co-authored using generative AI tooling?
No

Merged to master.


Update README to reflect renaming of OS X to macOS

Thank you. It seems El Capitan and all previous versions will still be called OS X so I will use both names. That is also better for searchability.


import in typescript

In webstorm I have error:
TS7016: Could not find a declaration file for module 'any-date-parser'. '/media/nvme/pro/cm/poc_fb_date_parse/node_modules/any-date-parser/index.js' implicitly has an 'any' type.   Try `npm i --save-dev @types/any-date-parser` if it exists or add a new declaration (.d.ts) file containing `declare module 'any-date-parser';`

To fix add option
"noImplicitAny": false

to tsconfig.json. Option explaination

/* Raise error on expressions and declarations with an implied 'any' type. */

Source:
https://stackoverflow.com/a/62681335/6398044
I propose to describe it in docs.

There was other problem with import:

Cannot use import statement outside a module

https://github.com/mochajs/mocha/issues/4531
Better fix:
replace import from by require.
const parser = require('any-date-parser')

@gustawdaniel I'm not so familiar with Typescript but I'm working on generating an index.d.ts file for this repo so that using it with Typescript is painless.

@gustawdaniel Version 1.4.5 now includes an index.d.ts file. Please let me know if there are any problems.

Hi. In my case, the package worked only if I use require: const DateParser = require('any-date-parser'). After I switched the project to 'type':'module', I can no longer use require, and import DateParser from 'any-date-parser' gives me a constant error: Error:(4, 24) TS2306: File '.../node_modules/any-date-parser/index.d.ts' is not a module. Can anyone advise any solution?

As a temporary solution, I added the line declare module 'any-date-parser' to the types.d.ts file in the root of my project. After that the error is gone, but of course there is no typing either.


'General Opacity' option does not affect colors in Gradient mode.

The fix for this is in version 1.1.1 which has been published now.


Don't use structlog versions newer than 16.0.0 for python 2.6

Got round to looking at what changed to make the Python 2.6 builds fail - if it's still considered important to support this version, of course. :-)

Thanks!


Debug for non-exhaustive patterns: Patch.

I didn't aware of your pull request, sorry... The problem was fixed now.

It's ok. Thanks for fixed this problem.


Further RNPM integration

This commit removes rnpm folder that we left during initial merge to
keep the diff cleaner. There's now:

core part - it will be further integrated with the utils folder
(esp. the config part that I am working on concurrently)
link
install

The 2 and 3 reproduce the same structure as in any other command.
From most notable differences, the src folder is now gone. The new
structure should make it easier for people to work with the stuff and
also move us closer to 100% rnpm integration.

CC: @Kureev

134 files, man 😵
Is it possible to deliver it by chunks? Probably 10 PR by 13 files or so? ^^

I'll try to take a look tomorrow tho

Is it possible to deliver it by chunks? Probably 10 PR by 13 files or so? ^^

I wouldn't consider path changes a big deal. What's the point of delivering it in 10 PRs?

@Kureev, you should look at the diff first. There are no changes to code except moved files (counts as a file in a diff, but it's empty) and path changes (src/ chunk removed from every path since it's now gone).
There's nothing to review code-wise here in my opinion, just to check the new structure.

@facebook-github-bot shipit

I think CircleCI issue is not related to this change, so we can safely ship it

Thanks for importing.If you are an FB employee go to Phabricator to review internal test results.


sort person edit people list by face count

Should fix #765.
Also added a cursor: pointer style to improve UX.

I might have misread the original issue 🤦
This improves the ModalPersonEdit modal when tagging photos from the lightbox sidebar.

Excellent! But still, I don't like the design where it's sorted by the number of faces a person has. I prefer it how it was before, sorted alphabetically. It would be better if there was a section in the settings to decide whether to sort it alphabetically or by the number of faces a person has.

Please keep alphabetic order by default but add sorting by number of faces as additional capability ! ;-)


Migrate article: Accounts, Teams, Members and Roles

Migrate the content from this article to fit the documentation portal format https://help.aiven.io/en/articles/4206498
(Assign this issue to yourself before you start, and remember to mention the issue number when you open your pull request)

@lornajane solved this one: https://developer.aiven.io/docs/platform/concepts/projects_accounts_access.html

Fixed in #186


MSVC RTTI created VFT's not typed as a named type

When MSVC RTTI creates a virtual function table it will be typed to the structure not the named type, this was originally the case (it being typed to a NTR) but at some point it changed to allow for PDB information.
You can repro this by opening a binary with RTTI (any windows binary basically) and then:

Navigating to a VFT
Hitting the Make Pointer key bind (default to o)
Notice that the type is now not typed with a structure name
Notice the user type with the name of the data var

Fixed in builds >= 4.2.6451.


JS preprocessors

I'm quite accustomed to CoffeeScript, so I like to use it here and there. It would be nice if I can use a custom preprocessor with nearley (e.g. Babel, CoffeeScript, 5to6, PromisedLand...).

I don't know how that would be implemented. Is there a way to instruct a preprocessor to only process code in {% %}'s?

@Hardmath123 we can simply pass the code between the {% %} to the preprocessor.
Also for coffeescript, we can wrap the non-coffee code in ``, and it won't be parsed. I'm not sure about the other preprocessors. (BTW, promisedLand shoud already work, as it is a superset of JS)

Hmm, doesn't that require you to know which preprocessor to use ahead of time?

@Hardmath123 we can pass it as cmd line option to the compiler. Another option will be to place it on top of the file, e.g.:
``
@preprocessor coffee
@{% combine=(d)->d.join("") %}
w -> "1" "10" "asdf":+ {% combine %}
``

The latter sounds better. Are you open to implementing this yourself? It'll take me a while because I'm busy with school stuff—but the actual code should be really simple.

@Hardmath123 well, I might try to do something this weekend, but don't expect too much.
Hint: I'm currently making an atom grammar for nearley :smile:


Update toc.yml

Changed "Share point one drive options" to "SharePoint OneDrive options"
Instructions: Add any supporting information, such as a description of the PR changes, here.

[!NOTE]
The following guidance is for Microsoft employees only. Community contributors can ignore this message; our content team will manage the status.

After you've created your PR, expand this section for tips and additional instructions.

do not merge is the default PR status and is automatically added to all open PRs that don't have the ready to merge label.
Add the ready for content review label to start a review. Your PR won't be reviewed until you add this label.
If your content reviewer requests changes, review the feedback and address accordingly as soon as possible to keep your pull request moving forward. After you address the feedback, remove the changes requested label, add the review feedback addressed label, and select the Re-request review icon next to the content reviewer's alias. If you can't add labels, add a comment with #feedback-addressed to the pull request.
After the content review is complete, your reviewer will add the content review complete label. When the updates in this PR are ready for external customers to use, replace the do not merge label with ready to merge and the PR will be merged within 24 working hours.
Pull requests that are inactive for more than 6 weeks will be automatically closed. Before that, you receive reminders at 2 weeks, 4 weeks, and 6 weeks. If you still need the PR, you can reopen or recreate the request.

For more information, see the Content review process summary.

@loryanstrant , thanks for your PR. I've fixed this internally so I'll go ahead and close this PR here. Thanks again.


[CameraRoll] Image Orientation Bug

This is blocking bug for us. I have seen a couple of issues and followed the suggest tips but I am still unable to get this image to render with proper orientation. This issue #1567 make reference to the bug and memory issues large camera rolls.
Here is an example of what I am talking about as you can see top image is rotated incorrectly. Doesn't happen with all images though oddly enough.

@a2 ping

@brentvatne ping

@myusuf3 - I'm not familiar with this part of the codebase, it sucks that it's a blocking issue for you :( perhaps @nicklockwood has some insight

That would be awesome @nicklockwood if you could provide some insight!

I don't have time to investigate this fully right now, however I have a theory about what the issue is.
UIImage has a concept of orientation which may be being ignored when the image is drawn.
The orientation metadata means that images in the camera roll aren't necessarily stored "right-way-up" on disk, but are instead rotated to the correct angle when drawn by iOS. If we're not taking the orientation into account, it would explain why some images are rotated and others aren't.

@myusuf3 does the orientation appear differently on your thumbnails than it does on the Apple Photos app ?

Some reading on orientation for those interested: http://www.daveperrett.com/articles/2012/07/28/exif-orientation-handling-is-a-ghetto/

@ide thanks for the article. nice read!

@VonD yes it does appear differently in both places. correct in the photos app and broken in our react application.

@myusuf3  OK, here's a little experiment : a photo of my keyboard taken in the 4 orientations. Below are two screenshots : on the top the Photos app, on the bottom my RN app. The orientations seem consistent. I'm using RN 0.6.0 + the patch provided here : #1567 , and the screenshots are from a device with ios 8.3. How does this test work for you ?

Is this the patch exactly?
  if ([imageTag hasPrefix:@"assets-library"]) {
    [[RCTImageLoader assetsLibrary] assetForURL:[NSURL URLWithString:imageTag] resultBlock:^(ALAsset *asset) {
      if (asset) {
        // ALAssetLibrary API is async and will be multi-threaded. Loading a few full
        // resolution images at once will spike the memory up to store the image data,
        // and might trigger memory warnings and/or OOM crashes.
        // To improve this, process the loaded asset in a serial queue.
        dispatch_async(RCTImageLoaderQueue(), ^{
          // Also make sure the image is released immediately after it's used so it
          // doesn't spike the memory up during the process.
          @autoreleasepool {
            ALAssetRepresentation *representation = [asset defaultRepresentation];
            ALAssetOrientation orientation = [representation orientation];
            UIImage *image = [UIImage imageWithCGImage:[asset thumbnail] scale:1.0f orientation:UIImageOrientationUp];
            RCTDispatchCallbackOnMainQueue(callback, nil, image);
          }
        });

This to me doesn't make sense since you are forcing all the images in the up orientation which should result in everything that wasn't take in up orientation to be broken.

@VonD  ^^^

@myusuf3 I mean I can't say for certain that will work. It was just a thought. My thought was that the -thumbnail method is documented as performing the rotation for you and having an Up image orientation.

So if I apply thumbnail [asset thumbnail] it will put everything in the up position once its done? @a2 Anyways I got that piece working and things in the application aren't flipped anymore. I was using a custom Image component in the camera roll screen. I will fix my PR #1842 to reflect it.

@myusuf3 What's the status for orientation issue? Do you have a fork which works for you?

This should be fixed now. If not, please re-open.

@nicklockwood Orientation doesn't work for me on current master. When rnplay.org will update to 0.9.0 I'll post example there, maybe there is some misunderstanging.

@unknownexception I am still patching the file for it to work. I hoping to see if this  #2008 was merged into 0.8.0 so we could ditch this patch we currently have in our code which works well.  @nicklockwood can you confirm that PR was merged in?

@myusuf3 Changes from #2008 were committed to master on 7/21 (https://github.com/facebook/react-native/commit/9c73e2ff7aeac8c65f25472135e4c0410f797eea). @nicklockwood committed additional enhancements shortly thereafter (https://github.com/facebook/react-native/commit/85cb35c51439a17c0961cc80518d2e979d7279c0).
Merged into 0.9.0-rc.

@myusuf3 @nicklockwood I've created PR #2214 to resolve this.  The original fix was overwritten by other recent camera roll enhancements.

@aroth @myusuf3 can you provide some sample photos that show at the wrong orientation in the camera roll example? I've tried taking photos in all orientations with the camera, but they always show the right way up even without @aroth's fix, and I don't want to merge it unless I can verify that it works.

Without fix:

With fix:

@aroth can confirm, that fixed some cases. Thank you for your fix.

I've got the same problem here.
My images get from CameraRoll has a bad orientation.

When I read message above seems to be fixed...
Do you have an idea when it will be in a released version?
Best regards,
Tom
ps: You are making an awesome job guys! Thank you!

@thomashermant The fix for this has been included in 0.11-rc.
https://github.com/facebook/react-native/releases

its not in 0.10.0? ugh.

@brentvatne this is fixed already in .11. Close.

Reopen... I have the same problem in RN 0.43.

Same problem, did you manage to resolve yours after the month or so?

I return to RN 0.42 ...

@pawellewandowski Have you resolved this issue? I have this with RN 0.48.3 and 16.0.0-alpha.12

I'm getting this issue on iOS using React Native v0.52.0

Getting this issue on Android, React Native 0.55

Hey @jose920405 I see you used react-native-exif to fix your issue (in the stackoverflow article). I gave it a try, but the thing is, is it's telling me to rotate images 90 degrees correctly, but in other instances it's also telling me to rotate images that are already correct.
Below is a screenshot of two images, both of which are actually portrait images (and display as portrait in default gallery app). But as you can see, the Exif value says rotate 90 for both :S.


Rename Models

As of today, the default model is now text-davinci-002-render-sha while the legacy model is now text-davinci-002-render-paid.

@slavakurilyak thank you! I will check this out

A change is rolled out to update the model names. Thank you @slavakurilyak!


Analytics 10.17.0 RC1

Analytics 10.17.0 RC1 #no-changelog

We may have another fix. Please hold the merge.

All patches are in. The PR is ready to merge, thanks.


Create Mesop app that in the same process runs Autogen

Managed to get 2 way communication between Mesop app and Autogen getting through, while enabling streaming of Autogen responses.
One notable snag is that in Mesop state variable it is not possible to have Queue objects and similar, since the state gets serialized and deserialized around. So some kind of registry is needed, and only Id gets stored in Mesop state.
What needs to be done next is to orhanize conversation in Mesop app into smaller chunks of [(User provided input, Stream of autogen responses)]


Parse Exception Error Message Update

ParseException: Line 27:0: Expected a statement here but got Dedent instead (was there an unbalanced if statement earlier?)
^ would it be possible to get this to display the name of the node where the parse exception occurred?
That'd be a big help in tracking down these types of errors. :) (right now I'm getting one of these in a massive file)

Ok, try out the DLL in this package. It should report errors a little better. If it works out, I'll merge it into master.
YarnSpinner-0.9.6-8-g01cfae5.unitypackage.zip

(The DialogueRunner.cs file has also been updated, so import that too!)

Cool! That seems to work. :)
This code seems to fail on line 27. I'm not really seeing why? Could be a parser error or I could be missing something obvious. :P
<<if $mom > 0>>     <<if $mom_church is 0>>         Mom: How's the headache?         Mae: Still there. Being out and about helps.         Mom: I bet!         <<set $mom_church += 1>>     <<elseif $mom_church is 1>>         Mom: Thanks for visiting me here, btw.         Mom: I like seeing you! It brightens my day!         Mae: Feels weird being here today.         Mae: Like... there's so much empty space.         Mom: Come on sunday! It's full then!         Mae: I bet it is!         <<if $money_trouble is 1>>         	Mae: sorry you have to work b/c money problems         	Mom: mae I've always had a job         	Mom: this is just another one         <<endif>>         <<set $mom_church += 1>>     <<elseif $mom_church is 2>>         [[MomChurch_A3D3_Loop]]     <<endif>> <<else>>      <<if $mom_church is 0>>         <<set $mom_church += 1>>         [[MomHome_A3D3_00]]     <<elseif $mom_church is 1>>         <<set $mom_church += 1>>         [[MomHome_A3D3_01]]     <<elseif $mom_church is 2>>         [[MomChurch_A3D3_Loop]]     <<endif>> <<endif>>

Weird. I ran that through the latest parser, and it worked fine. I also double-checked the if-else-end balancing, and it also seemed fine.
Hmm.

Yeah, weird still get 'ParseException: In file : Error parsing node MomChurch_A3D3: Line 27:0: Expected a statement here but got Dedent instead (was there an unbalanced if statement earlier?)'
Maybe I'll try copy-pasting the rebalanced code O.o

Summarising a conversation I had with @InfiniteAmmoInc: This turned out to be because an if statement was one space further indented than its endif, causing the parser to get confused about why there was a Dedent token when it expected an EndIf token.
Fixed in 52a3ae392d by making if statements ignore any Dedent tokens before their <<else>>, <<else if and <<endif>>.

Wait, this particular bug is not yet fixed. Will re-open and continue looking.

Alrighty, try this one.
YarnSpinner-0.9.7-5-ge2f1cfb-source-minimal.unitypackage.zip

This hopefully fixes the issue by making the lexer only track indentations that occur either on the line that contains a ->, or on the line following. Dedents are only tracked if they cross the threshold established by a previously tracked indent. Whew.

Now properly fixed in v<IP_ADDRESS>.


router-0  pod is in Init:CrashLoopBackOff

Describe the bug
While deploying kubecf on AWS kubeadm cluster (no EKS) the pod router-0 always in CrashLoopBackOff state.
router-0                                 0/7     Init:CrashLoopBackOff
To Reproduce
Direct deployment of kubecf 2.7.12.

helm install cf-operator --namespace cf-operator --set "global.singleNamespace.name=kubecf" cf-operator.tgz
helm install kubecf --namespace kubecf --values values.yaml kubecf_release.tgz
ingress set to false.

Expected behavior
it should deploy kubecf  on AWS kubernetes cluster
Environment
kubernetes version : 1.18.4
cf version 6.53.0+8e2b70a4a.2020-10-01
Additional context
no logs available for router-0 pod.  As I said above, I'm deploying kubecf on AWS.
k get pods -n kubecf
NAME                                     READY   STATUS                  RESTARTS   AGE
api-0                                    17/17   Running                 1          4h4m
auctioneer-0                             6/6     Running                 1          4h4m
cc-worker-0                              6/6     Running                 0          4h4m
cf-apps-dns-747456bc6-s7dx5              1/1     Running                 0          4h7m
coredns-quarks-866dff75df-cbhn7          1/1     Running                 0          4h6m
coredns-quarks-866dff75df-kp2cb          1/1     Running                 0          4h6m
credhub-0                                8/8     Running                 0          4h4m
database-0                               2/2     Running                 0          4h5m
database-seeder-d77d9a8451cb30da-k9pvr   0/2     Completed               0          4h6m
diego-api-0                              9/9     Running                 2          4h4m
diego-cell-0                             12/12   Running                 2          4h4m
diego-cell-1                             12/12   Running                 2          3h55m
doppler-0                                6/6     Running                 0          4h4m
log-api-0                                9/9     Running                 0          4h4m
log-cache-0                              10/10   Running                 0          4h4m
nats-0                                   7/7     Running                 0          4h4m
router-0                                 0/7     Init:CrashLoopBackOff   51         4h4m
routing-api-0                            6/6     Running                 0          4h4m
scheduler-0                              12/12   Running                 1          4h4m
singleton-blobstore-0                    8/8     Running                 0          4h4m
tcp-router-0                             7/7     Running                 0          4h4m
uaa-0                                    9/9     Running                 0          4h4m

@naveenspen14  did you get it working,? I have the same issue.


Support CDATA gracefully

Hello,i'm from China and developing a trade system. This system needs to use wechat pay (wechat is the most popular IM App in China).
The pay API using xml and json to connect third party development system. And CDATA was used heavily. So i forked node-xml2js and add a new way to support CDATA.
If you think this PR can be useful. Please accept.
Thanks!

Coverage decreased (-1.1%) to 96.512% when pulling 1522ba41824e13dcc28ad5c79247db722b44dd84 on qixin1991:master into af83e313790d28e65ad0f99324a6fbdef9da54fd on Leonidas-from-XIV:master.

Coverage decreased (-1.1%) to 96.512% when pulling 1522ba41824e13dcc28ad5c79247db722b44dd84 on qixin1991:master into af83e313790d28e65ad0f99324a6fbdef9da54fd on Leonidas-from-XIV:master.

Coverage decreased (-1.1%) to 96.512% when pulling 1522ba41824e13dcc28ad5c79247db722b44dd84 on qixin1991:master into af83e313790d28e65ad0f99324a6fbdef9da54fd on Leonidas-from-XIV:master.

Except for removing all kinds of CI information, contributor info, what does this PR improve?

@Leonidas-from-XIV Sorry for that,it's my mistake. Few days ago,before PR was accepted,i want to publish into npm website,however it failed. So i try to delete something.
This PR improve CDATA usage. For example:
Screenshot1

Screenshot2

Can you hear me? 😅

Well, the real PR content is  src/builder.coffee at the bottom

This PR is unusable in this form, closing.


mido-PixelExperience-twelve-plus-NRanjan-17

Dont do 2 build in one time

Dont do 2 build in one time
https://t.me/ROM_builders/80858
Please check all my messages after this


Links not working in documentation

2 of the 3 links at the bottom of the page are not working:

Create Early-Bound Entity Classes with the Code Generation Tool (CrmSvcUtil.exe)
Use the Early Bound Entity Classes for Create, Update and Delete

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

ID: f0949367-dc3e-ba70-b494-8945530a6fed
Version Independent ID: 93472f86-1b45-0421-2e1e-7e2857e64534
Content: Create extensions for the code generation tool (Common Data Service) - Power Apps
Content Source: powerapps-docs/developer/common-data-service/org-service/extend-code-generation-tool.md
Service: powerapps
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@StevenRasmussen Thanks for reporting this.
@phecke
There should be valid targets within the Power Apps CDS documentation for all three links at the bottom of this page.
Can you update these?

@StevenRasmussen @JimDaly
This has been fixed and will be published by EOD today. Thanks for notifying us.
I removed the broken links from the extend topic and added new links to the related  generate topic.


[5.8] Add runtime for each migration to output

This adds the runtime for each individual migration to the output of the artisan migrate and artisan migrate:rollback commands. I find this useful to get an idea of how long each migration will take to run in various environments (dev, staging) before deploying to production.
Example output:

It would be nice to have the same with seeders


4/10/2019 Publish

Docs Build status updates of commit 59ecc0b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-autogen/microsoft-cognitiveservices-speech-sdk/index.yml
:warning:Warning
View (azure-node-latest)
Details

preview-packages/docs-ref-mapping/reference.yml
:white_check_mark:Succeeded

docs-ref-autogen/microsoft-cognitiveservices-speech-sdk/index.yml

[Warning] Uid(microsoft-cognitiveservices-speech-sdk) has already been defined in docs-ref-autogen/microsoft-cognitiveservices-speech-sdk/index.yml.

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


GetFEN() and restore from FEN does not include the move histories...

Whenever I display the game board I highlight the previous move on the board. After de-serialization it's not populating the Moves property so I'm not able to tell what the last move was.

Ah yes, of course... I totally forgot about the Moves property in our previous conversation. FEN doesn't contain any moves by design.
Here's my plan: a GetSerializableObject() method on ChessGame that returns an object with the FEN, the move list, and the three other properties that aren't in the FEN. Then a way to go back to a ChessGame from that object. I won't be able to implement that before next week though, so if you'd want to work on it already, go ahead; if not, I'll do it next week.

What if we add the missing properties to GameCreationData and make that serializable? To make it serializable we would have to add parameterless constructors to each Piece and Position and remove the protected set from its public properties, but I'd think that would be a small price to pay to re-use the existing constructor around game creation and keeping backwards compatibility.
We could then create a new function GetGameCreationData if it doesn't already exist and spit out everything needed to re-create the existing game in its current state.

Yes, you're right, that works too.

Alright, I'll get started on it sometime tonight probably.


"Error: Overlapping ranges are not allowed" when regex replacing after 10000 items

VSCode Version: 1.30.2
OS Version: Windows 10 x64 1809

Steps to Reproduce:

Open a big project like this one
Search and Replace in files Ctrl+Shift+F
Turn regex on
Search ^.*import.*$ or any query that captures from beginning or to the end of the line
About 10000 matches
Click Replace All
Wait until it's finished
All replaced perfectly fine
Now hit enter and click "Replace All" again
ERROR:

  ERR Error: Overlapping ranges are not allowed!
    at e.applyEdits (pieceTreeTextBuffer.ts:216)
    at o._applyEdits (textModel.ts:1314)
    at o.applyEdits (textModel.ts:1271)
    at e.pushEditOperation (editStack.ts:168)
    at o._pushEditOperations (textModel.ts:1264)
    at o.pushEditOperations (textModel.ts:1173)
    at e.apply (bulkEditService.ts:96)
    at e.apply (bulkEditService.ts:213)
    at e.<anonymous> (bulkEditService.ts:355)
    at errors.ts:237
    at Object.s.__generator.a.label [as next] (errors.ts:237)
    at s (errors.ts:237)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188)

Some files now have multiple copies of the same result on the left
17548 results instead of 10000
Files are not being changed after clicking Replace All

Queries I've tried with restarting each time (all affected):
^.*var.*\n
^.*var.*$
.*\(\).*
\{.*\n
It's irrelevant whether there are multiple matches on the same line or not.
Screenshot:

Clearly visible:

Multiple search results on the same line that shouldn't be there
More than 10000 results overall instead of roughly 10000

Does this issue occur when all extensions are disabled?: Yes
NOT a duplicate of https://github.com/Microsoft/vscode/issues/51624
NOT a duplicate of https://github.com/Microsoft/vscode/issues/55606

Now I'm not able to repro it

I'm still able to repro.
It seems like it became even worse with the new update 1.31.0.
Now it gets stuck in an infinite loop. I'm seeing Windows hourglass cursor appearing and disappearing 10 times per second and the vscode process maxing out the CPU core. Main window is still responsive though.

Still unable to regex replace anything over 10000 results.

@roblourens You can repro it using steps here

Thanks for the steps @sandy081! This turned out to be a bug that I don't know why it doesn't come up more often. It could have resulted in missing search results and other issues.


How to center ContributionGraph?

Hi. I tried using style prop to center the graph but didn't work.
const graphStyle ={
    justifyContent: 'center',
    alignItems: 'center',
    flex:1,
  };

Then, I found out paddingLeft value was hardcoded.  Can I add a pull request to change this value customizable via props? Or do let me know if I made any mistake in the above code.
src/contribution-graph/index.js
line 8: const paddingLeft = 32;
inside the renderSquare function
<Rect
        key={index}
        width={squareSize}
        height={squareSize}
        x={x + paddingLeft}
				...
/>

inside the renderMonth function
<Text
          key={weekIndex}
          x={x + paddingLeft}
          y={y + 8}
          {....getPropsForLabels()}
 >
          {this.props.getMonthLabel
            ? this.props.getMonthLabel(endOfWeek.getMonth())
            : MONTH_LABELS[endOfWeek.getMonth()]}
</Text>

and also, I think it would be nice to make padding between month label and graph customizable.

+1 Found the same issue, it'd be an easy way great to make the graph center

The same issue

@ji-1 How did you set the colors of the text (Jan, Feb, Mar, ...?)


Importing @iconify/tools fails

When importing @iconify/tools into a TypeScript project it fails:
Could not find a declaration file for module '@iconify/tools'. './node_modules/.pnpm/@iconify+tools@2.1.1_@iconify+types<EMAIL_ADDRESS>implicitly has an 'any' type.

The simple fix is to add:
"types": "./lib/index.d.ts"

to package.json:
"exports": {
		"./*": "./*",
		".": {
			"require": "./lib/index.cjs",
			"import": "./lib/index.mjs",
			"types": "./lib/index.d.ts"
		},

What node and typescript versions are you using?

nodejs: v16.18.1
tsc: 4.8.4

Some relevant documentation:
https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/#ecmascript-module-support-in-node-js
https://www.typescriptlang.org/docs/handbook/esm-node.html#packagejson-exports-imports-and-self-referencing

Thanks. I've published update that fixes it.
Weird thing is, I'm not getting such errors in any of my dev environments, including same versions of node and typescript as you are using. Can you please tell me if latest version fixes error message?

Maybe it's related to the fact I'm using pnpm? There's a couple of oddities with pnpm and Typescript still.
It works now, I just realize that so far this fix didn't seem to be enough for VSCode to pickup the type definitions. Restarting VSCode fixed that.

Could be.
Thanks a lot for checking and for reporting issue!


ci-e2e workflow configuration

Current configuration table of ci-e2e:

Name
Kernel
Routing
EP routes
IPv6
Encryption
kube-proxy
LB mode
MASQ
Egress GW
Host FW

1
4.19
vxlan
-
+
-
iptables
-
iptables
-
-

2
5.4
direct
-
+
-
iptables
-
iptables
-
-

3
5.10
direct
+
+
-
iptables
-
iptables
-
-

4
5.10
vxlan
+
+
-
mixed
SNAT
BPF
+
-

5
5.15
direct
+
+
-
mixed
DSR
BPF
+
+

6
6.0
vxlan
-
+
-
KPR
SNAT
BPF
+
+

7
bpf-next
direct
-
+
-
KPR
SNAT
BPF
+
-

8
bpf-next
geneve
+
+
-
iptables
-
iptables
-
-

9
4.19
vxlan
-
+
IPsec
iptables
-
iptables
-
-

10
5.4
direct
-
+
IPsec
iptables
-
iptables
-
-

11
5.10
direct
+
+
IPsec
iptables
-
iptables
-
-

12
5.10
vxlan
+
+
WireGuard
mixed
SNAT
BPF
+
-

13
5.15
direct
+
+
WireGuard
mixed
DSR
BPF
+
-

14
6.0
vxlan
-
+
WireGuard host-to-host
KPR
SNAT
BPF
+
-

15
bpf-next
direct
-
+
WireGuard host-to-host
KPR
SNAT
BPF
+
-

16
bpf-next
geneve
+
+
IPsec
iptables
-
iptables
-
-

Longer term, we may want to cover IPsec on bpf-next. We don't in any CI job today. We should cover that because we currently rely on an internal characteristic of the kernel's IPsec implementation (that is, cb[4] is preserved) and we have no guarantee that it won't change: https://github.com/cilium/cilium/blob/c46eb0bd0590ae9dac3b3f02fc6fc558ec48fa44/bpf/lib/encap.h#L27-L30

Longer term, we may want to cover IPsec on bpf-next

What other configurations would be relevant for IPsec + bpf-next?

Longer term, we may want to cover IPsec on bpf-next

What other configurations would be relevant for IPsec + bpf-next?

Any one of the configurations that use skb->cb[4]. Currently that's only IPAM={EKS,Azure,AlibabaCloud}, but with the upcoming IPsec changes, all configurations will use skb->cb[4] so testing should become possible on ConformanceDatapath.

We are no longer updating the issue to reflect the ci-e2e configurations.


Question: Why do you apply the shim to a fakeGlobal first?

Looking at shim.js it seems that it already checks for availability of existing features. Applying the shim to a fakeGlobal makes these extra checks redundant at best.
Or is it because of the integration into esX-shim?
Since the integration is tightly coupled with esX-shim (index.js), would it not be best to remove the now redundant checks from shim.js in order to speed things up?

It's not about "tightly coupled". It's that things should not modify the global environment by default.
At the moment, this module's code is mostly scraped from https://github.com/johnmccutchan/ecmascript_simd, and I'm trying to have as loose coupling with that as I can.
What checks are you referring to? I don't see any in this repo (this repo's issues should only about the simd module itself)

Well, calling the module with a fakeGlobal and then in that module you
line 35ff
if (typeof global.SIMD === "undefined") {
  // SIMD module.
  global.SIMD = {};
}

and later on you will
e.g. lines 263ff
if (typeof SIMD.Bool64x2 === "undefined") {

This is what I meant that the checks are basically moot since it will always be the fake global. Yet, and since you incorporate the code from a different source, I see no reason in changing its original behavior.
Thanks for the input!

ah, that makes sense. Yes, it's just because the original source operates on a "global" object, so "SIMD" must exist in order for its checks to work.


Add --disable_carvers flag to disable Bulk Extractor file carving

Testing locally, it seems like JPEG files are still being carved and written to the output. This might be a bug with Bulk Extractor 2.1.1 but needs a bit more testing.

Bulk Extractor issue opened: https://github.com/simsong/bulk_extractor/issues/480


sdk-core, node: add file system abstractions

This PR moves file system abstraction to sdk-core, allowing other clients to use this abstraction to achieve, for example, database support, without reimplementing the business logic.
Database file support has also been moved to sdk-core, which works with FileSystem interface now.

Can we please merge changes from the dev/your previous branch?

Merged.


how can i  use async api in b-thread?

in your example, all calls in b-threads are sync functions.
but in a real case, I will use  some async functions (eg. HTTP request)
how can I do that?

Sorry for the late reply @stormslowly
Essentially you can use promises in the function generators:
function *() {
  yield { wait: 'addHot' }
  callSomePromise()
    .then(() => this.request('promiseResolved'))
}
// another b-thread can wait for 'promiseResolved'
function *() {
  yield { wait: 'promiseResolved' }
  // do something else
}


test fdo servers

This PR does:

install fdo packages and start fdo servers
includes the changes in https://github.com/virt-s1/rhel-edge/pull/2124 , the user information has been added into data/fdo/serviceinfo-api-server.yml

/test-rhel-9-2-simplified

/test-rhel-9-2-simplified

Should be ready for test, this pr has passed in my local rhel9.2

/test-rhel-9-2-simplified

/test-cs9-simplified

/test-rhel-8-8-simplified

/test-rhel-9-1-simplified

/test-rhel-9-2-simplified

/test-rhel-8-8-simplified

/test-rhel-9-1-simplified

/test-cs9-simplified

/test-cs8-simplified

I ignored a fact that onboarding user cannot happen on os rather than rhel9.2 and centos9, hence the new configuration files will not work for them,
Will fix it and push again.

/test-rhel-9-1-simplified

/test-rhel-9-2-simplified

/test-cs9-simplified

/test-cs9-simplified

/test-rhel-9-2-simplified

/test-rhel-9-1-simplified

/test-cs8-simplified

/test-rhel-8-8-simplified

/test-rhel-8-8-simplified

As we used fdo aio server on both upstream and downstream testing, and it works well, closing this pr for now, may need to add test for all fdo servers later.