Gap at the start of the journey

Your Environment

Plugin version: 4.11.4
Platform: iOS
Device OS version: iOS 16.4.1
Device manufacturer / model: Apple iPhone 8 Plus
XCode version: 14.3
Capacitor info:

💊   Capacitor Doctor  💊
Latest Dependencies:
@capacitor/cli: 5.0.3
@capacitor/core: 5.0.3
@capacitor/android: 5.0.3
@capacitor/ios: 5.0.3
Installed Dependencies:
@capacitor/core: 4.8.0
@capacitor/android: 4.8.0
@capacitor/ios: 4.8.0
@capacitor/cli: 4.8.0
[success] iOS looking great! 👌
[success] Android looking great! 👌

Plugin config provided to #ready:

await BackgroundGeolocation.ready({
  // Geolocation Config
  desiredAccuracy: BackgroundGeolocation.DESIRED_ACCURACY_HIGH,
  distanceFilter: 10,
  stopTimeout: 5,
  logLevel: BackgroundGeolocation.LOG_LEVEL_OFF,
  stopOnTerminate: true,
  startOnBoot: false,
  showsBackgroundLocationIndicator: true,
  logMaxDays: 1,
  locationAuthorizationRequest: 'Always',
  locationAuthorizationAlert: {
    titleWhenNotEnabled: "Location Services not enabled",
    titleWhenOff: "Location Services OFF",
    instructions: "You should enable 'Always' in Location Services",
    cancelButton: "Cancel",
    settingsButton: "Settings"
  },
  backgroundPermissionRationale: {
    title: "Allow {applicationName} to access this device's location in the background?",
    message: "In order to track your activity in the background, please enable {backgroundPermissionOptionLabel} location permission",
    positiveAction: "Change to {backgroundPermissionOptionLabel}",
    negativeAction: "Cancel"
  },
  notification: {
    title: "Location Tracking",
    text: "Background location services activated",
    sticky: true,
  },
});

Expected Behavior
Background Geolocation plugin to capture locations that are close together and that follow the travelled route.
Actual Behavior
Background Geolocation plugin has gaps at the start of the journey.
Context
I seem to be able to consistently get gaps at the start of the journey. I start location tracking at about 16:54 and immediately lock the device. You can see the gap between 16:54 and 16:56 where the location jumps around 800m. The rest of the journey however captures lots of locations so it is really only the start of the journey that I'm having issues with. Coincidentally (or not), all my tests that have this large gap between locations also equate to about 2 minutes. Thank you for your assistance :)
Debug logs

Logs
[log.txt.gz](https://github.com/transistorsoft/capacitor-background-geolocation/files/11449593/log.txt.gz)

Yes, that’s normal.  iOS requires movement of at least 200 meters before tracking initiates.  See wiki “Philosophy of Operation”.
you can manually toggle the plug-in into the “moving” state via method .changePace(true).


Add sizeTerm

This adds a function sizeTerm which is the canonical size of a Plutus Core term. It is used for benchmarking.

This is a nice idea, I can use this to track the sizes of generated PLC programs too.

Good idea! I imagine that matters for costs and whatnot.


Reconcile virtual service

What this PR does / why we need it:
Fixes: #451
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

This change is

/cc @rakelkar @yuzisun @ellis-bigelow


law-edit

implemented the editing use case:

after creating a set, users can edit the sets from the home screen
no drafting feature (yet), so after exiting the edit screen to the home screen, changes are unsaved
after editing the set, the same validations in the create screen are ran. user is notified if there is an error with their input
after editing and pressing the 'done' button, the user is brought back to the home screen and the updated set is shown

code review:

consistent style (e.g., camel case)
design clearly fits in with existing project (clean architecture) (i.e., interactors, presenters, controllers, etc.)
code is easily understandable with well-named functions

Code Review:

Code follows clean architecture
Has a good naming convention and Javadoc to keep track of what going on
Consistent
Clearly has the functions mentioned in the user story


Bluetooth: controller: Address nRF5340 Engineering A Errata 16

Address the nRF5340 Engineering A Errata 16, RADIO: POWER
register is not functional.
This affects the dependency of controller's HAL
implementation that expected correct reset values being
set in the Radio peripheral.
As the SUBSCRIBE_TXEN and SUBSCRIBE_RXEN where not in their
reset value, Radio transmission and reception start using
DPPI was broken.
Fixes #25942.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

@trond-snekvik During debugging, I had switched to using hardware tIFS switching. Now PR updated with fix for software tIFS switching.

Works like a charm now! Tested scan_adv, peripheral_hids and mesh.

Thanks for testing.


fix: use node-modules linker, not pnp

Changes proposed in this pull request
This PR switches from using the pnp linker to using the node-modules linker, because unlike the pnp linker, the node-modules linker allows yarn docker build backend to complete successfully (originally, the plan was to move the docker image build out of yarn. That turned out to require switching back to node modules, since yarn doesn't have any command that packages the dependencies along with the source code. Yarn 1 apparently did this; the docs for yarn 2 remove mention of it. So switching back to node_modules was necessary in order to get sufficient access to the packages to put them in a docker image manually. However, after switching back to the node-modules linker, the existing process started working again, so I didn't have to create a separate docker build script). I'd like to do some further testing to ensure that the image built using this linker will work.
Context

Checklist

[ ] Related issues linked using fixes #number
[ ] Tests added/updated
[ ] Documentation added
[ ] Make sure that all checks pass

We also moved to the node-modules linker internally. So this is probably a good change

@rluckom-coil May be worth adding an issue to optimize the github actions now to cache the node-modules.


I adapted this for use with Python 3.6
had to change

gaussian_filter from skimage.filters is now just called gaussian
some iterators need to be called as next(blabla)  instead of blabla.next()
IntSliderWidget is now just called IntSlider

Thanks for the contribution!
I ended up diffing with nbdime, which worked pretty well, but it means that it's tough to write inline comments. I'll just give you a summary as best I can:

Please update the package requirements listed at the beginning to reflect the changes. (If nothing else, the IPython and skimage versions need to be bumped up.)

Remove unnecessary In [81] cell.

Remove empty cell between In [83] and In [84]

Remove empty cell at the very end of the notebook.

I would prefer
   from IPython.html.widgets import IntSlider

(and updates to the corresponding calls to IntSlider) instead of
   import IPython.html
   #from IPython.html.widgets import IntSliderWidget

Other possible improvements, while you're at it ;)

The image of the convolution widget is broken. (It looks like it's broken on my system for the original version as well, so it probably goes along with Jupyter-version changes.)
Update the skimage.filter import to skimage.filters (with a trailing "s"). That assumes you're running a version similar to the one I'm testing on.

These last three have nothing to do with your changes, so feel free to ignore them.

Good suggestions, thank you.
I added the changes to the notebook and in the process I noticed that IPython.html.widgets should be replaced with ipywidgets.widgets.

Thanks for the updates.
Currently, this doesn't run for me because you import the filters (trailiing s) module from skimage but call filter.gaussian (no trailing s in filter). While you're at it, could you remove the extra convolution_demo.py file.


Is OrderBy escaped?

is OrderBy escaped? If I pass in a value from a client-side app is it safe to do that? I want to open up the orderBy to API calls. Should I just parse it and  make sure it's one of a set of allowable values and if so then allow otherwise throw out?

@happilymarrieddad I think no. you should not expose that to your customer. You have to check if it should be one of your allowable values.


Web repo still links to Vercel

https://github.com/shapeshift/web links to web-shapeshift.vercel.app; it should go to https://app.shapeshift.com and/or https://develop.web-29e.pages.dev

fixed yesterday


Toggle Switches on Map and Player pages

A toggle button for Turbo/Classic and one for weapons.

0c3c2a9efabfce4cd4b30d2d2bc43b54a85226ca


Update Composer dependencies (2024-09-19-00-08)

Visual regression test passed!
View the visual regression test report


Add support for "typedef struct C C". Fixes #459

Close #459
See #11

This change is

Warnings going down from ~45000 to ~17000. :+1:

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

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

a discussion (no related file):
Please fix the merge conflicts.

Comments from Reviewable

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

Comments from Reviewable

Ready for review.

Please resolve conflicts, then I can review.

Done )

Reviewed 2 of 2 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from Reviewable

Reviewed 1 of 1 files at r5.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


"Improve" now edits existing code base

Solves #650.
Improve mode (ie -i flag) now produces code changes and applies those changes to the existing codebase.
Changes:

improve prompt now produces code changes in the git conflict marker format
the LLM output is parsed into code changes (called "Edits")
Edits are applied to existing codebase (either create new file or update existing one)

Note: Unlike Aider (https://github.com/paul-gauthier/aider/blob/main/aider/coders/editblock_coder.py) which uses a lot of code to make sure edits are correctly parsed and applied, I have kept the code very simple. Still, I have found it to work well with an adjusted prompt. Still, I would encourage everyone to test this code.

Did you also run it with 3.5-turbo? Some logic is dependant on LLM output and 3.5 might be worse at following this.
I will try it myself and let you know if I get issues.

Did you also run it with 3.5-turbo? Some logic is dependant on LLM output and 3.5 might be worse at following this.
I will try it myself and let you know if I get issues.

No, I didn't. But I also wouldn't use gpt3.5 for code generation.
I think we should assume people use gpt4 or similarly capable models.
Happy to hear @AntonOsika's thoughts

Wow, excellent work.  super excited to see this in action.

Nice work @UmerHA !
Looking at the code, I think it would make sense to re-use the logic in parse_chat instead of the adding similar logic in parse_all_edits function.
This part of parse_chat would have to be refactored or even removed, as I don't think we should continuously replace the readme.
    # Get all the text before the first ``` block
    readme = chat.split("```")[0]
    files.append(("README.md", readme))

Running b73f793 for improving someone else's program, I get the error:
KeyError: "File 'src/server/util/API.js' could not be found in '/home/axel/Software/Opensource-Contribution-Leaderboard/workspace'" when it is trying to apply the diff. Is the reason that, so far, it only supports improving gpt-engineer generated softwares?
Happy to provide the lengthy stack trace if requested.

(venv) axel@axel-ThinkPad-T470s:~/Software/Opensource-Contribution-Leaderboard$ gpt-engineer -i .
How do you want to select the files?

Use File explorer.
Use Command-Line.

Example: 1,2,3-5,7,9,13-15,18,20 (enter 'all' to select everything)

Select files:54,58,61
The following files will be used in the improvement process:
file_list.txt:
/home/axel/Software/Opensource-Contribution-Leaderboard/src/server/app.js
/home/axel/Software/Opensource-Contribution-Leaderboard/src/server/refresh.js
/home/axel/Software/Opensource-Contribution-Leaderboard/src/server/util/API.js
The inserted prompt is the following:
'make the program get the list of contributors directly from github, instead of reading from the local config'
You can change these files in your project before proceeding.
Press enter to proceed with modifications.
PLANNING:
The current implementation reads the list of contributors from a local configuration file. To fetch the list of contributors directly from GitHub, we need to use the GitHub API.
The GitHub API provides an endpoint to fetch contributors for a specific repository. The endpoint is https://api.github.com/repos/{owner}/{repo}/contributors. We need to make a GET request to this endpoint to fetch the list of contributors.
We will create a new function getContributors in the API.js file. This function will take organization and repository as parameters and will return a list of contributors for the given repository.
Then, we will modify the getAllContributorsInfo function in the refresh.js file to use this new getContributors function instead of reading from the local configuration file.
OUTPUT:
src/server/util/API.js
<<<<<<< HEAD
module.exports = {
    getRepositories,
    getContributorAvatar,
    getOpenPRsNumber,
    getMergedPRsNumber,
    getIssuesNumber,
    getContributorInfo,
    checkRateLimit,
    getStats,
    getRanks,
}
=======
async function getContributors(organization, repository) {
    const res = await get(
        APIHOST + `/repos/${organization}/${repository}/contributors`
    )
    if (res !== undefined) {
        return res.data.map((element) => {
            return element['login']
        })
    } else {
        return []
    }
}

module.exports = {
    getRepositories,
    getContributorAvatar,
    getOpenPRsNumber,
    getMergedPRsNumber,
    getIssuesNumber,
    getContributorInfo,
    checkRateLimit,
    getStats,
    getRanks,
    getContributors,
}
>>>>>>> updated

src/server/refresh.js
<<<<<<< HEAD
let Config = jsonfile.readFileSync(configPath)
let organization = Config.organization
let contributors = Config.contributors
let includedRepositories = Config.includedRepositories
=======
let Config = jsonfile.readFileSync(configPath)
let organization = Config.organization
let includedRepositories = Config.includedRepositories
let contributors = []
for (let repository of includedRepositories) {
    let repoContributors = await API.getContributors(organization, repository)
    contributors = [...contributors, ...repoContributors]
}
contributors = [...new Set(contributors)] // remove duplicates
>>>>>>> updated
```Traceback (most recent call last):

  File "/home/axel/Software/gpt-engineer/venv/bin/gpt-engineer", line 8, in <module>
    sys.exit(app())

  File "/home/axel/Software/gpt-engineer/venv/lib/python3.10/site-packages/gpt_engineer/main.py", line 98, in main
    messages = step(ai, dbs)

  File "/home/axel/Software/gpt-engineer/venv/lib/python3.10/site-packages/gpt_engineer/steps.py", line 366, in improve_existing_code
    overwrite_files_with_edits(messages[-1].content.strip(), dbs)

  File "/home/axel/Software/gpt-engineer/venv/lib/python3.10/site-packages/gpt_engineer/chat_to_files.py", line 150, in overwrite_files_with_edits
    apply_edits(edits, dbs.workspace)

  File "/home/axel/Software/gpt-engineer/venv/lib/python3.10/site-packages/gpt_engineer/chat_to_files.py", line 207, in apply_edits
    workspace[filename] = workspace[filename].replace(

  File "/home/axel/Software/gpt-engineer/venv/lib/python3.10/site-packages/gpt_engineer/db.py", line 64, in __getitem__
    raise KeyError(f"File '{key}' could not be found in '{self.path}'")

KeyError: "File 'src/server/util/API.js' could not be found in '/home/axel/Software/Opensource-Contribution-Leaderboard/workspace'"

I guess my error above is the one that is addressed in #728 @RareMojo ?

@UmerHA Is the idea that it will always improve existing files, not add new ones?
I'm a bit confused with how gpt-engineer is supposed to work in cases like mine. I have a generated project that I want to add more features to, not necessarily changing existing files only but also adding new ones.
If I do gpt-engineer [mydir] with existing files and ask it to add a new feature it will happily wipe my entire workspace.
As for your change I think it's awesome but that it also needs to be able to add new files. In my case I wanted to add more database models but GPT will output everything in the
<<<<<<< HEAD
=======
>>>>>>> updated

format, even new files.
Also if GPT happens to add some other output to the reply, like bash commands, we get a ValueError
    OUTPUT:
    ```bash
    # In your terminal
    npm install package
    ```

I guess my error above is the one that is addressed in #728 @RareMojo ?

Can't dig into it for awhile, but I'm inclined to say no. It looks like the same fix I suggested is applied in apply_edits already here.

@RareMojo , I think you are right, these are different things.

@UmerHA Is the idea that it will always improve existing files, not add new ones?
No, it can also add new files. In this case the before part (between HEAD and =======) will be empty.

I'm a bit confused with how gpt-engineer is supposed to work in cases like mine. I have a generated project that I want to add more features to, not necessarily changing existing files only but also adding new ones. If I do gpt-engineer [mydir] with existing files and ask it to add a new feature it will happily wipe my entire workspace.
When you do gpt-engineer -i [mydir] it should only update the files you named, or create new ones. Can you describe in more detail under which circumstances it wipes out existing files?

Also if GPT happens to add some other output to the reply, like bash commands, we get a ValueError
    OUTPUT:
    ```bash
    # In your terminal
    npm install package
    ```

Could you post the 'ask' you made to gpte? That helps me understand better if we need to add handling of such cases. :)

Here is the project that I'm trying to modify: https://github.com/ATheorell/Opensource-Contribution-Leaderboard/tree/improveCode
The prompt and the file selection are in the stack trace. I tried multiple times and always get the error @UmerHA

When i run out of this branch it is still writing inside the workspace directory inside the project directory

It appears when there are comments above a method or class chatgpt is not returning the match with comment and this code ends up not getting replaced

I am using this for java and but several times it is not actually updating the files and i dont see any error logs. Trying to narrow down but no luck

Here is the project that I'm trying to modify: https://github.com/ATheorell/Opensource-Contribution-Leaderboard/tree/improveCode
The prompt and the file selection are in the stack trace. I tried multiple times and always get the error @UmerHA

Okay, can reproduce now. The issue is that currently gpte expects the code to be in a subfolder name 'workplace'. I'll make the name of the workplace folder editable. In your case, you would then choose 'src'.

Here is the project that I'm trying to modify: https://github.com/ATheorell/Opensource-Contribution-Leaderboard/tree/improveCode
The prompt and the file selection are in the stack trace. I tried multiple times and always get the error @UmerHA

Okay, can reproduce now. The issue is that currently gpte expects the code to be in a subfolder name 'workplace'. I'll make the name of the workplace folder editable. In your case, you would then choose 'src'.

What is the downside of having it work with relative paths from the execution path, rather than an explicit code path? I'm thinking about the case that the improved code files may be scattered over the file tree (and also a bit about the general UX).

Here is the project that I'm trying to modify: https://github.com/ATheorell/Opensource-Contribution-Leaderboard/tree/improveCode
The prompt and the file selection are in the stack trace. I tried multiple times and always get the error @UmerHA

Okay, can reproduce now. The issue is that currently gpte expects the code to be in a subfolder name 'workplace'. I'll make the name of the workplace folder editable. In your case, you would then choose 'src'.

What is the downside of having it work with relative paths from the execution path, rather than an explicit code path? I'm thinking about the case that the improved code files may be scattered over the file tree (and also a bit about the general UX).

Do you project path (not execution path)? I mostly use gpte like this
$ cd path/to/gpte/venv
$ pipenv shell
$ gpt-engineer -i path/to/project

So, the execution path (path/to/gpte/venv) and project path (path/to/project) are different.
In principle, there is no downside to using relative locations to the project path. We would then have make sure gpte-internal files are not editable.
Iirc, there's a proposal to move all of those things into a .gpt-engineer folder, right? We could then just exclude that folder.

I think we need to decide how we want to structure the gpte directory. We have two options:

root dir is for gtpe, a subdir (eg "workspace") is for the code
root dir is for code, a subdir (eg ".gptengineer") is for gpte

We currently do 1, but when we want the ability to edit existing codebases, 2 would be easier. @AntonOsika @ATheorell @pbharrin thoughts?
When that's decided, then I can finish this PR.

I now went ahead and made a PR to use the project path as workplace and move everything else into .gpteng - see #749. That would need to be merged first, to fix & merge this PR.
@AntonOsika @ATheorell

Good job, check my comments and also tests + pre-commit is failing:

The prompt is pretty long, so I'd also like to compare it with sweep.dev's approach:)

@AntonOsika Agree the prompt is long & we should look into making it shorter. I would, however, prefer to separate it from this PR. Let's get this shipped, and then optimize it later.

I added some comments, feel free to merge without addressing, but nice if we at least address after merge!

Addressed all!

tests + pre-commit is failing

works now

Also, reminder we need to merge https://github.com/AntonOsika/gpt-engineer/pull/749 first :)

resolved merge conflicts, all tests pass, Peter has approved; let's merge.

Has anyone tested scenario where the model suggests multiple edits within the same file using the -i|--improve flag?
Tested with 1 file input and many edits within that file. Found that only the first edit is accepted, the rest are ignored (file only has 1 edit after gpt-engineer finishes).

Click to expand console log
Built using Docker on main branch as of Nov 2, 2023 (commit: 1408652)
Command line:
export OPENAI_API_KEY="sk-2kMT........................................AXRK"
export DOCKER_USER="$(id -u):$(id -g)"
export OPENAI_MODEL="gpt-4"

docker run -it --rm -e OPENAI_API_KEY="$OPENAI_API_KEY" -v "/.../my-blazor-pwa:/project:cached" --user "$DOCKER_USER" gpt-engineer -i $OPENAI_MODEL | tee ".gpteng/logs/output_`date +"%Y%m%d_%H%M%S"`.txt"

With output:

Running gpt-engineer in /project
File list detected at /project/.gpteng/file_list.txt. Edit or delete it if you want to select new files.
The following files will be used in the improvement process:
file_list.txt:
/project/Server/Data/MyCloudWebAppDbContext.cs

The inserted prompt is the following:
Can you convert the existing code that depends on Neo4j over to JanusGraph in C#? The client library is Gremlin.Net. The endpoint is available at the environment variable "MY_WEBAPP_JANUSGRAPH_ENDPOINT" with has the value "dev-my-cloud-janusgraph:8182" which should be dynamically parsed and split based on ":" character. Make sure that all the Graph Queries are updated to use the new endpoint.
Make sure that the queries are also translated to Gremlin equivalent. Both for querying as well as for mutations. The queries are already in-line in Neo4j client format and in comments. These need to be converted to Gremlin queries.

You can change these files in your project before proceeding.
Press enter to proceed with modifications.
PLANNING:
The task involves converting the existing code that depends on Neo4j over to JanusGraph in C#. The client library is Gremlin.Net. The endpoint is available at the environment variable "MY_WEBAPP_JANUSGRAPH_ENDPOINT" with has the value "dev-my-cloud-janusgraph:8182" which should be dynamically parsed and split based on ":" character.
The steps involved in this task are:

Replace the Neo4j client initialization with the Gremlin.Net client initialization.
Replace all the Neo4j Cypher queries with equivalent Gremlin queries.
Update the method signatures and implementations to use the Gremlin.Net client.

OUTPUT:
Due to the complexity of the task and the length of the code, it's not feasible to provide a complete solution here. However, I will provide a few examples of how to convert the code.
Server/Data/MyCloudWebAppDbContext.cs
<<<<<<< HEAD
    private BoltGraphClient _neo4jClient { get; }
=======
    private GremlinClient _gremlinClient { get; }
>>>>>>> updated

Server/Data/MyCloudWebAppDbContext.cs
<<<<<<< HEAD
    _neo4jClient = new BoltGraphClient(
        uri: EnvironmentExtensions.GetEnvironmentVariableOrDefault(
            envVarName: "MY_WEBAPP_NEO4J_ENDPOINT",
            defaultValue: "bolt://my-cloud-neo4j:7687"
        ),
        username: neo4j_auth[0],
        password: neo4j_auth[1],
        encryptionLevel: Neo4j.Driver.EncryptionLevel.Encrypted
    );
=======
    var endpoint = EnvironmentExtensions.GetEnvironmentVariableOrDefault(
        envVarName: "MY_WEBAPP_JANUSGRAPH_ENDPOINT",
        defaultValue: "dev-my-cloud-janusgraph:8182"
    ).Split(":");
    _gremlinClient = new GremlinClient(new GremlinServer(endpoint[0], int.Parse(endpoint[1])));
>>>>>>> updated

Server/Data/MyCloudWebAppDbContext.cs
<<<<<<< HEAD
    var user = await _neo4jClient.Cypher
        .WithParam("id", userId)
        .Match("(u:MyCloudUser {id: $id})")
        .Where((MyCloudUser u) => u.Id == userId)
        .Return(u => u.As<MyCloudUser>())
        .Limit(1)
        .ResultsAsync;
=======
    var user = await _gremlinClient.SubmitAsync<MyCloudUser>("g.V().hasLabel('MyCloudUser').has('id', userId).limit(1)");
>>>>>>> updated

Please note that the above examples are just a starting point. The actual conversion might require more changes depending on the complexity of the queries and the specific requirements of the JanusGraph database.Total api cost: $  ...

I've confirmed that only the first edit was made in the target file.

@ATheorell pinging you so @rrmistry's comment doesn't get lost. I don't have time to fix this, but wanted to let the team know.


[Issue] Renewal restores .skel into existing domain

Ubuntu 14.04 LTS
Let's Encrypt extension version: 2.5.3-354
Plesk Onyx version: 17.8.11 Update #8
This issue occurs since one of the updates in 2018.
Let's Encrypt renews the certificates automatically, it restores default index.html, css and img folder from .skel into the existing project root of an existing project.

Please visit Plesk Forum and describe your issue with more details in the Let'sEncrypt extension discussion:
https://talk.plesk.com/threads/lets-encrypt-extension.336954/


update colors.hex after register component not work

var sectionModel = {
    id:'',
    icon:'',
    title :'',
    link:'',
    colors : {
        hex: '#ff00ff'
    }
};

Vue.component('color-picker', VueColor.Chrome);

sectionModel.colors.hex = `#ff0000`;

update colors.hex not trigger the ui's color changed of events
ui's color still is '#ff00ff'

did you set the colors in template? just like this
<color-picker :colors.sync="sectionModel.colors"></color-picker>

<color-picker :colors.sync="colors"></color-picker>

i set it in template

Returning the sectionModel in data might be the problem, I believe Vue needs to know the data beforehand to be reactive, if you want to do dynamically you should use vm.set for each sectionModel properties inside a life cycle hook.

I don't find any problem of my data, and the data change by correct way


feat: Update package versions

Description
Updated npm packages.
Remarks
[Optionally place any follow-up comments, remarks, observations, or notes here for future reference]

Looks like the test and build step is stuck


Dropbox should support range read

From dropbox docs: https://www.dropbox.com/developers/documentation/http/documentation#formats

As with content-upload endpoints, arguments are passed in the Dropbox-API-Arg request header or arg URL parameter. The response body contains file content, so the result will appear as JSON in the Dropbox-API-Result response header. These endpoints are also on the content.dropboxapi.com domain.
These endpoints also support HTTP GET along with ETag-based caching (If-None-Match) and HTTP range requests.
For information on how to properly encode the JSON, see the JSON encoding page.

I'm guessing we can simply add range header for it.

cc @Zheaoli, would you like to add this feature for dropbox?

cc @Zheaoli, would you like to add this feature for dropbox?

Sure

cc @Zheaoli, would you like to add this feature for dropbox?

Sure

I've implemented it in #2848. Sorry for not updating this in the issue in time.


fix: update year in footer from 2022 to 2023

Notes for Reviewers
This PR fixes #97
Signed commits

[x] Yes, I signed my commits.

LGTM @kaushik-rishi 👍


Bugfix/ada trajdescription error

When executing trajectories for ada, the execution occurs, but the following error is thrown:
"AttributeError: 'JointTrajectory' object has no attribute 'GetDescription'"
Seems to be a problem with exec_traj (line 414) not having a description. this error is thrown on the following line in util.py:
294     description = traj.GetDescription()
Returning the other trajectory traj doesn't produce the same error
Note that traj is of class 'openravepy.openravepy.openravepy_int.Trajectory', while exec_traj is class 'trajectory_msgs.msg._JointTrajectory.JointTrajectory'

sigh Please see: https://github.com/personalrobotics/ada/issues/43#issuecomment-150868573

This has come up several times, but no one seems to have resolved it. The underlying issue is in this line:
https://github.com/personalrobotics/ada/blob/master/adapy/src/adapy/adarobot.py#L205
The result of ExecuteTrajectory is supposed to be an OpenRAVE trajectory, but this code is returning a ROS JointTrajectory, which is totally different. Thus, the subsequent code gets messed up, because it can't do any of the usual postprocessing like setting Tags.
Someone needs to go in and convert the return of ExecuteTrajectory from ROS trajectories back to OpenRAVE trajectories. I'm not really sure how this code was originally intended to work in the first place.

@sjavdani @mkoval: I think it's preferable to fix the underlying issue of adapy returning a completely incorrect type instead of removing the working trajectory tags functions from prpy, right?

@psigen is right. The correct place to change this is inside the ExecuteTrajectory function in AdaPy. We convert the OpenRAVE trajectory to a ROS trajectory, execute it, and (optionally) use that return value to wait for the trajectory to finish execution. You should modify that function to return the original OpenRAVE trajectory, instead of the ROS trajectory.

Alright, I see the "quickfix" mike suggested in the other thread. I'll try that next time I'm in lab and submit if it works.
Is there a better fix you might suggest, or is the suggested quickfix the right fix anyway?

Thanks @sjavdani!
@mkoval's quickfix is alright, but the correct thing to do would be to actually convert the returned ROS JointTrajectory into an OpenRAVE trajectory.
Each JointTrajectoryPoint has a list of positions / velocities / accelerations per joint that become the waypoints in an OpenRAVE trajectory.  By diff-ing the time_from_start fields, you can get the delta_time that OpenRAVE uses.
This isn't very hard to do, but it's little bit fiddly.  It's possible that there is already code to do this in a utility module somewhere: if not, there probably should be since OpenRAVE Trajectory <-> ROS Joint Trajectory is a very useful conversion.

The ros_control joint trajectory controller does not currently send feedback via actionlib, which is how we reconstruct the executed trajectory in Python (see https://github.com/ros-controls/ros_controllers/issues/173). It currently only publishes this data on a controller status topic.
I don't think it is worth adding a workaround for this - we'd be much better off modifying the controller to publish the data in the right place. This is non-trivial because it requires adding a realtime-safe method of publishing feedback to realtime_tools.
I suggest you go with the quick fix for now. We can revisit it once we have the controller publishing feedback via actionlib.

The suggested quick fix works. Therefore, I'll close this pull request since it is wrong/unnecessary.

Thanks for spending the time to look into the underlying issue!


Correction to embedded image in document

Please correct the highlighted misspelling on the flow diagram, please.

@ChristopherHolloway . which article are you referring?. Error should be corrected immediately.

@ChristopherHolloway Can you refer to the associated article? A link would be helpful. Thank you.

Yes here is the link: https://docs.microsoft.com/en-us/windows/security/identity-protection/smart-cards/smart-card-certificate-requirements-and-enumeration
Sorry for the delays. If you wouldn't mind giving kudos that would be great, and thank you for all your efforts as well.

Replacement Image, Please swap out and close.

@ChristopherHolloway . Thanks for replacement pic, but there is a misspelling again, if we look faraway some distance, again it looks like clicks, instead of
It should be replaced by press save

@ChristopherHolloway : May I ask you to point out where the misspelling is, since your replacement image seems to be the same as the original one (at least at first glance)?

@illfated The misspelling is at the very bottom of the first diagram highlighted with yellow.

The issue has been noted for investigation by an independent contributor. Thank you for discussion.

@illfated The misspelling is at the very bottom of the first diagram highlighted with yellow.

Thank you. I did not notice earlier, maybe because I did not zoom in on the picture closely enough.
Actually, I think it would be an improvement just to replace the current image with the posted image in https://github.com/MicrosoftDocs/windows-itpro-docs/issues/6777#issuecomment-636032735,

because the highlighted text in the main comment of the issue ticket does not separate the letters c and l well enough, so it looks like a d (which makes the word seem a bit derogatory).

The image has been replaced via PR #7950

https://docs.microsoft.com/en-us/windows/security/identity-protection/smart-cards/smart-card-certificate-requirements-and-enumeration
The image still lives on the site and has not merged via PR at this time please correct at the earliest.

If the ticket had been opened correctly, it would have looked something like this:

[Enter feedback here]

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

ID: 132e8bfa-12ba-2ae0-7745-545bcfd909f1
Version Independent ID: 984c17f4-04a4-bf30-3a9e-3af49209f0bb
Content: Certificate Requirements and Enumeration (Windows 10) - Microsoft 365 Security
Content Source: windows/security/identity-protection/smart-cards/smart-card-certificate-requirements-and-enumeration.md
Product: w10
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Document general date for last update change: 04/19/2017 (https://docs.microsoft.com/en-us/windows/security/identity-protection/smart-cards/smart-card-certificate-requirements-and-enumeration)
Document commit (change log) history on GitHub: https://github.com/MicrosoftDocs/windows-itpro-docs/commits/public/windows/security/identity-protection/smart-cards/smart-card-certificate-requirements-and-enumeration.md
image source link: https://github.com/MicrosoftDocs/windows-itpro-docs/blob/public/windows/security/identity-protection/smart-cards/images/sc-image402.png
image last change date: February 1, 2018, 5:55 PM GMT
non-migrated image update commit here on GitHub: https://github.com/MicrosoftDocs/windows-itpro-docs/commit/69dfb9c3d8cd823c0b5029cdacb6f4665936bd13 (https://github.com/MicrosoftDocs/windows-itpro-docs/commit/69dfb9c3d8cd823c0b5029cdacb6f4665936bd13?short_path=71098f1#diff-71098f1742018b7e9edc1368c60e3f0c49f6b7bc3f5460babf96bee609f8e367)


add in missing TEF logos

What
Added in the TEF provisional outcome logo.
How to review
See that the files are now in the code.

LGTM


Deprecated TLS versions aren't available since the last update

Description
We patch our bundled JRE's security policy to it doesn't restrict the usage of deprecated TLS versions. We do so to make it possible to connect to legacy MSSQL versions: https://github.com/dbeaver/dbeaver/issues/12668. Unfortunately, now it's not the case for Windows.
DBeaver Version
DBeaver CE 23.0.0
Operating System
Windows
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Fixed


Add optional onPlaceholderIndexChange callback function

I'm not sure if you're interested in this or not, but it's something that I found very useful, and I saw at least one other issue where this was asked.
This change adds an optional onPlaceholderIndexChange callback function that triggers whenever the spacer index changes. This is handy if you want to provide some sort of feedback whenever an item shifts. For example a haptic.
Feel free to close this PR if it is not what you want to do.

looks good to me -- thanks!
can you fix the small merge conflict and update the README with the new prop?

Sounds good, I'll sort that out today!

That's done @computerjazz, let me know if you want anything changing.


URI Template support in parameters

@darrelmiller making two (or so) changes then this will be merged

This is more descriptive than 2.0, and it will increase the burden on tooling. For example, matrix parameters are most usually found in Java implementations, and now a project like a documentation generator might have to support matrix parameters in order to be "compliant" or "certified."
That's not a reason to stop the presses, just an observation. Hopefully we'll hear from tooling authors about this if it's a big problem.

Is there a "spec" of the templating features that this modification is intended to provide?
As it stands, for example, it doesn't seem to allow for "simple multi-segment" path parameterization (the cited "Dropbox" URI example cited in Issue 291)

@bassmanitram multi-path segments parameters are not supported.  I attempted to explain why here https://github.com/OAI/OpenAPI-Specification/issues/892#issuecomment-281170254


dom/border.cpp in 3.0.0+1 hangs linux_debian_12-clang_15.0_libc++

It appears Clang 15 with libc++ hangs when compiling dom/border.cpp in 3.0.0. Clang 16 doesn't seem to have this problem nor does 4.0.0 with Clang 15.
We currently don't have a mechanism to blacklist certain build configurations for certain packages (but we will try to add something like this for the next version) so unless this build is excluded in the package manifest, it will keep hanging our build bots. So I am going to create the 3.0 branch off the v3.0.0+1 tag, exclude this build, and release/publish the 3.0.0+2 revision.

Done.


Remote connect to Oracle

I'm trying to connect to the Oracle 12c instance created using the docker image. Command i ran:
docker run -d -p 8080:8080 -p 1521:1521 pengbai/docker-oracle-12c-r1
The image runs on a Digital Ocean VM. When i try to connect using SQLDeveloper i get the following error:

Status: Failure -Test failed: IO-error: The Network Adapter could not establish the connection

From my experience, this usually has to do with the hostname settings. Any way to resolve this?

My bad... Company firewall... Perfect docker image!

@ferdyh I was just about to look into that, thank you for the update


How to properly configure colour schemes...

The documentation is not clear enough on how to configure the theme for neovim with reusing existing colours.
  {
    "rebelot/kanagawa.nvim",
    opts = {
      compile = false, -- enable compiling the colorscheme
      undercurl = true, -- enable undercurls
      commentStyle = { italic = true },
      functionStyle = {},
      keywordStyle = { italic = true },
      statementStyle = { bold = true },
      typeStyle = {},
      transparent = true, -- do not set background color
      dimInactive = false, -- dim inactive window `:h hl-NormalNC`
      terminalColors = true, -- define vim.g.terminal_color_{0,17}
      colors = { -- add/modify theme and palette colors
        palette = {
          matte_cyan = "#0eb8be",
        },
        theme = {
          wave = {
            syn = {
              constant = "matte_cyan", -- I'm aware this is wrong, but I can't figure out the way to reference
                                       -- my own defined colours in the palette
              special3 = "oniViolet", -- I want to use palette.oniViolet here,
            },
          },
          lotus = {},
          dragon = {},
          all = {
            ui = {
              bg_gutter = "none",
            },
          },
        },
      },
      overrides = function(colors) -- add/modify highlights
        local theme = colors.theme
        return {
          NormalFloat = { bg = "none" },
          FloatBorder = { bg = "none" },
          FloatTitle = { bg = "none" },

          -- Save an hlgroup with dark background and dimmed foreground
          -- so that you can use it where your still want darker windows.
          -- E.g.: autocmd TermOpen * setlocal winhighlight=Normal:NormalDark
          NormalDark = { fg = theme.ui.fg_dim, bg = theme.ui.bg_m3 },

          -- Popular plugins that open floats will link to NormalFloat by default;
          -- set their background accordingly if you wish to keep them dark and borderless
          LazyNormal = { bg = theme.ui.bg_m3, fg = theme.ui.fg_dim },
          MasonNormal = { bg = theme.ui.bg_m3, fg = theme.ui.fg_dim },

          TelescopeTitle = { fg = theme.ui.special, bold = true },
          TelescopePromptNormal = { bg = "none" },
          TelescopePromptBorder = { bg = "none", fg = theme.ui.bg_p1 },
          TelescopeResultsNormal = { bg = "none", fg = theme.ui.fg_dim },
          TelescopeResultsBorder = { bg = "none", fg = theme.ui.bg_m1 },
          TelescopePreviewNormal = { bg = "none" },
          TelescopePreviewBorder = { bg = "none", fg = theme.ui.bg_dim },
        }
      end,
      theme = "wave", -- Load "wave" theme when 'background' option is not set
      background = { -- map the value of 'background' option to a theme
        dark = "wave", -- try "dragon" !
        light = "lotus",
      },
    },
  },

I can't seem to pull oniViolet to override. My question becomes, do I need to do the override where I'm doing it, just in a different way, or do I need to do it in the overrides?

I think the easiest way is to just specify the hex code for oniViolet from the README (#957fb8):
wave = {
    syn = {
        special3 = '#957fb8'
    }
}

reference your color explicitly or assign it to a variable, palette colors can be accessed via
local palette = require'kanagawa.colors'.setup().palette
local matte_cyan =  "#0eb8be"
require'kanagawa'.setup{
   ...
}


html2canvas can not get background-image which set by css houdini paint api

using css houdini paint api to set background-image，as following，but the screenshot is blank content, can not get the background-image
@supports ((-webkit-mask: paint(id, type)) or (mask: paint(id, type)))
div {
background-image: paint(paint-worklet);
}
Does html2canvas support this ？

Yes,so i am.Background-imgae attribute used in css by url() function,function argument is from remote server,use h2htmlcanvas  constructer  pass it options contain 'useCores:true',but i  console it get some logs :Access to image at 'https://' from origin 'http://192.1' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.Why use 'useCores:true' cannot resolve cors?


[Epic] Optimize js-waku for the Browser

Problem
js-waku targets the browser. NodeJS user should use nim-waku and JSON RPC API.
There are a number of issues when using js-waku in the browser:

Node/browser switches in the code
Polyfills for using NodeJS APIs (buffer, crypto, stream, process)
Bundle is too big (>1MB, should target 200kb).
Several crypto libraries are used, increasing bundle size.

References

https://www.craft.do/s/dWHJUfi1WSmdKk
This reduces the usability of the library and increases its size: https://discord.com/channels/864066763682218004/865466694554484738/915784971036262402

Tasks

[x] https://github.com/waku-org/js-waku/issues/1415
[ ] https://github.com/waku-org/js-waku/issues/1418
[ ] Review bundler/bundle sizes https://github.com/waku-org/js-waku/issues/579
[ ] Use the Rollup analytics plugin to see what dependencies are in the bundle and what can be removed

Acceptance Criteria
Analysis done in #527. Once done, action plan can be updated.

[x] Remove need of polyfills

[x] crypto
[x] process
[x] stream
[x] buffer

[x] Double check that test_utils is not bundled.
[x] protobuf code (generated) is tree shakeable https://github.com/status-im/js-waku/issues/335
[ ] Review bundler/bundle sizes https://github.com/waku-org/js-waku/issues/579
[ ] Use the Rollup analytics plugin to see what dependencies are in the bundle and what can be removed

Note, if some dependencies do need polyfills, then it is possible to bundle the polyfill so that users do not have to set it up. It is also possible to only bundle the necessary APIs. see https://github.com/ethers-io/ethers.js/blob/4e6d121fb8aa7327290afab7653364be8ddd8d81/packages/shims/src/index.js

For reference, here is my current polyfill instructions for my package:
Crypto-browserify
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "crypto": require.resolve("crypto-browserify") }'
        - install 'crypto-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "crypto": false }

Fix
npm i crypto-browserify --save
Then add to your tsconfig.json:
"compilerOptions": {
    "paths": {
        "crypto": [
            "node_modules/crypto-browserify"
        ]
    }
}

Stream-browserify
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "stream": require.resolve("stream-browserify") }'
        - install 'stream-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "stream": false }

Fix
npm i stream-browserify --save
Then add to your tsconfig.json:
"compilerOptions": {
    "paths": {
        "crypto": [
            "node_modules/stream-browserify"
        ]
    }
}

DUMP_SESSION_KEYS
constants.js:6 Uncaught TypeError: Cannot read properties of undefined (reading 'DUMP_SESSION_KEYS')

Fix
npm i process --save
Add polyfill somewhere:
global.process = require('process');

As well as my commonJS list that I'm using for angular:
"allowedCommonJsDependencies": [
                "it-concat",
                "crypto",
                "debug",
                "libp2p",
                "libp2p-gossipsub/src/utils",
                "@chainsafe/libp2p-noise/dist/src/noise",
                "libp2p-bootstrap",
                "libp2p-websockets",
                "libp2p-websockets/src/filters",
                "multiaddr",
                "peer-id",
                "ecies-geth",
                "secp256k1",
                "libp2p-gossipsub",
                "hashconnect"
            ],

Thanks for that. Regarding DUMP_SESSION_KEYS it looks like I need to do some upstream fix: https://github.com/ChainSafe/js-libp2p-noise/blob/5d2e6ac201e9ba395d9ca465519335040653944c/src/constants.ts#L4
Thanks for the list of deps, I'll try to remove some once migrated to browser only and will probably have to do some upstream fixes :)

Note that I am updating the documentation to cover polyfills with react-scripts: https://github.com/vacp2p/docs.dappconnect.dev/pull/19

@chainsafe/libp2p-noise
This library brings two issues:

Usage of Buffer
Big bundle size: https://bundlephobia.com/package/<EMAIL_ADDRESS>
The bundle size is mostly due to the usage of node-forge.
Need to review whether node-forge could be replace by libp2p-crypto in libp2p-noise.

It is possible to keep the package size small with node-forge: https://mobile.twitter.com/IAmTrySound/status/1476899235723456515 by using source-map-js: https://github.com/vitejs/vite/pull/6556
Not sure if libp2p libraries use source-map-js. If they don't it might be a way to reduce their size.

Moved to icebox. Once #527 is done, we can update this one and move it back to backlog.

updated description to suit better needs of the issue


mask

Hello @pemami4911,
The problem really was with the mask. I've fixed it and the network started to learn. My Decoder now is:
class Decoder(nn.Module):
    def __init__(self, feactures_dim,hidden_size, n_layers=1):
        super(Decoder, self).__init__()

        self.W1 = Var(hidden_size, hidden_size)
        self.W2 = Var(hidden_size, hidden_size)
        self.b2 = Var(hidden_size)
        self.V = Var(hidden_size)

        self.lstm = nn.LSTM(hidden_size, hidden_size, batch_first=True, num_layers=n_layers)

    def forward(self, input, hidden, enc_outputs,mask, prev_idxs):

        w1e = torch.matmul(enc_outputs,self.W1)
        w2h = (torch.matmul(hidden[0][-1], self.W2) + self.b2).unsqueeze(1)

        u = F.tanh(w1e + w2h)
        a = torch.matmul(u, self.V)

        a, mask = self.apply_mask( a, mask, prev_idxs)
        a = F.softmax(a)
        res, hidden = self.lstm(input, hidden)
        return a, hidden, mask

    def apply_mask(self, attentions, mask, prev_idxs):
        if mask is None:
            mask = Variable(torch.ones(attentions.size())).cuda()

        maskk = mask.clone()

        if prev_idxs is not None:

            for i,j in zip(range(attentions.size(0)),prev_idxs.data):
                maskk[i,j[0]] = 0

            masked= maskk*attentions + maskk.log()
        else:
            masked = attentions

        return masked, maskk

For n=10, I'm obtaining the following during training the AC version:

Step 0
Average train model loss:  -81.07959747314453
Average train critic loss:  29.443866729736328
Average train pred-reward:  -0.08028505742549896
Average train reward:  5.2866997718811035
Average loss:  -15.106804847717285
------------------------
Step  1000
Average train model loss:  -0.7814755792869255
Average train critic loss:  0.7740849611759186
Average train pred-reward:  4.219553744537756
Average train reward:  4.272005982398987
Average loss:  -6.201663847446442
------------------------

(...)

Step  19000
Average train model loss:  -0.06441724334075116
Average train critic loss:  0.1361817416474223
Average train pred-reward:  3.0573679950237276
Average train reward:  3.0583059163093567
Average loss:  -1.5689961900115013

I've checked and the returned solutions are all feasible so it seems that it is really converging.
I will clean my code and hope that by the end of the week I will have some training history plots and test set validation.
If you like, I can share my notebook.
Best regards

I think sharing notebooks is always a good idea, so please share it. Would love to check it.

What exactly was the issue with the mask? It looks like you’re taking mask.log()- won’t that return NaN where mask is 0?

It returns -inf, and the softmax function will return 0.
mask test .ipynb.zip

What was the problem with the masking that you fixed?

@pemami4911 Sorry, I've updated the answer above.

Certainly, there must be a more elegant way of doing this :)

Can you tell me what the hyperparameters you are using are? Are you using the exponential moving average critic? Did you try it on TSP_20?

I added the maskk = mask.clone line and it seems to be learning something!! will update soon..

@ricgama  After 1 epoch (10,000 minibatches of size 128), on my held-out validation set of 1000 graphs I got:
Validation overall avg_reward: 3.0088058819770813
Validation overall reward var: 0.1305618987729639

I saw some tours as low as 2.4! It's learning! THANK YOU! haha

After 1 epoch of TSP 20, I got:
Validation overall avg_reward: 4.1758026211261745
Validation overall reward var: 0.14051547629226666

With some tours as low as 3.6. According to the paper, I should be aiming for an average reward of 3.89.
Sweet.

Great!!
For now, I'm just using a simple version with hidden_dim = 128 and no glimpses in the Pointer Net. I'm training with the paper AC network.
PointerNet_TSP.zip
I'm posting here my test sets for n=10 and n=20 for easy results comparison.
For n=10 I do:
tmp = np.load(test_path)
p = list(tmp['p'])
x = list(tmp['x'])
test = [[p[i],x[i]] for i in range(len(x))]

labels_te = np.array([x[0] for x in test],dtype=np.int64)
labels_te = np.lib.pad(labels_te,(0, 1), mode='constant', constant_values=0)
labels_te = np.delete(labels_te,(labels_te.shape[0]-1), axis=0)

inp_enc_te = np.array([x[1] for x in test])

where the labels are the optimal labels. For n=20:
tmp = np.load(test_path)
inp_enc_tr = np.array(tmp['x'])

For n=10 I obtain: true:  2.86695502642 predicted:  2.89031121256 with Supervised Learning.
I will try to post the RL results, for n=10 and 20, by the end of the week.

Cool, I'll update my repo with some results by the end of the week too. For TSP 20 RL, the validation average reward is down to 4.02 and still dropping little by little after 3 hours!
Can't believe it was just a mask.clone() that was breaking everything.. usually small bugs just act as regularizers.. I guess not in Deep RL :(

After 50 epochs for TSP 20, I got 3.95 average val reward! Fairly close to 3.89, it probably would have gotten there eventually if I had let it keep training.
After two epochs on TSP 50, I'm seeing:
Validation overall avg_reward: 6.54308279466629
Validation overall reward var: 0.15608837694146416

not bad!

Do you have the training history plots?

I haven't made any nice plots yet, but these are quick screenshots from Tensorboard for TSP 50. Been running for ~21 hours, looks like the validation avg reward has just about reached 6.05-6.10.

Zoomed in on average training reward (stochastic decoding) first few hours of training

average training reward (stochastic decoding) so far

Validation average reward (greedy decoding)

After 2 epochs for TSP 20, I got
Step  20000
Average train model loss:  -0.4126803118472656
Average train critic loss:  0.23534413764963638
Average train pred-reward:  4.466843187868655
Average train reward:  4.4662888267257435
Average loss:  -6.895590880492309
------------------------

worse than your: Validation overall avg_reward: 4.1758026211261745.
Now I'm trying with one attention glimpse.
Are you using decaying  lr?

Is this SL or RL? And is your train reward with greedy decoding, or stochastic decoding?
I am using the lr schedule from the paper - every 5k steps decrease by a factor of 0.96

It's RL and stochastic decoding. With one attention glimpse it appears a bit better so I will train 2 epochs and do greedy decoding and beam search to compare. My hardware is slower than yours so I what to check n=20  before moving to n=50...

OK- yeah you'll want to compare with greedy decoding, not stochastic. just FYI the beam search in my codebase isn't functional yet- it's only coded to propagate a single decoding hypothesis forward at each time step, which is equivalent to greedy decoding. Fixing that is on my to-do list :D

I've implemented my beam search. It works very well but for now only for batch=1, so it's a bit slower.
I can send it to you if you like...

Meanwhile, I think I will implement the Active Search of the paper. Have you looked into it?

Yeah you can send it to me if you'd like!
I haven't looked into implementing that yet. Not sure when I'll get to that part, got some other things I'm working on in the mean time

@pemami4911 When I was my BS to handle RL trained Pointer Model I found some inconsistencies that I have to look into before I share the code.
After 2 epochs Validation Av reward: 4.262  for n=20. I'm guessing that it's around the same as you within random fluctuations.
@unnir  notebook
If you find any error let me know.

Hello @pemami4911,
During my n=50 training it appeared loss=nan. It's strange because for n=20 it trained perfectly. I'm trying to debug my code to fix this.
While you were training for n=50 the flag print(' [!] resampling due to race condition') appeared often?
Do you have any sugestions?

Yes - You can see in my stochastic decoding function that I check if any cities were sampled that shouldn't have been- and if so, I resample all cities at that decode step. If that occurs (I think it's a race condition..?) I print out [!] resampling due to race condition. You should probably add that check too

So there must be a bug with the .multinomial() function because probability=0 actions should not be sampled.
I'm running a script simulating the masking/sampling loop to have an estimate of the probability of "bad" sampling. It should be very small but different from 0.
I saw your workaround for this problem. As the probability is very small, resampling again does the job. In theory, I think we should have a while condition not satisfied loop to guarantee that we never resample zero prob. actions.
I think that it is worth it to report this issue, " .multinomial() sampling 0 actions"  on Pytorch forum. What do you think?
After a proper masking, we should be able to just sit back and relax while the model is training...

Were you able to replicate the "bad sampling" with your script?

yes. It's just:

def apply_mask( attentions, mask, prev_idxs):
    if mask is None:
        mask = torch.zeros(attentions.size()).byte().cuda()

    maskk = mask.clone()

    if prev_idxs is not None:

        for i,j in zip(range(attentions.size(0)),prev_idxs.data):
            maskk[i,j[0]] = 1

        attentions[maskk] = -np.inf

    return attentions, maskk

def count(n):
    k = 0
    for j in range(n):

        attentions = Variable(torch.Tensor(128,50).uniform_(-10, 10).cuda())
        prev_actions = None
        mask = None
        actions = []
        for di in range(50):

                attentions, mask = apply_mask( attentions, mask, prev_actions)

                probs = F.softmax(attentions).cuda()
                prev_actions = probs.multinomial()
                # due to race conditions, might need to resample here
                for old_idxs in actions:
                    # compare new idxs
                    # elementwise with the previous idxs. If any matches,
                    # then need to resample
                    if old_idxs.eq(prev_actions).data.any():
                        k+=1
                        print(' [!] resampling due to race condition')

                actions.append(prev_actions)
    return k

By the end of the day I will have an estimate.

The relative frequency on a run of 100000 batches, size 128 and n=50, was 0.00043.
Do you mind that I post a question on the Pytorch forum, using the code above?

Yeah, go for it

Done: https://discuss.pytorch.org/t/bad-behavior-of-multinomial-function/10232
Let's see if we get a clarification about this.

Hello @pemami4911 ,
I had a reply: https://discuss.pytorch.org/t/bad-behavior-of-multinomial-function/10232/2
In fact, I've tried with CPU and did not register any bad sampling.
It is possible for you to run my code in you computer just to check if you can replicate the results?
Best regards,
Ricardo

Yeah I'll try running it with GPU and with CPU

Sorry for the late reply.. been busy
I ran it for the CPU and received 0 errors. The script is still running on my GPU (it runs muuuch slower, maybe we should switch to CPU for computing this portion of the code in our implementations..!) and I've already observed multiple resamplings.
Most likely, it is a bug in the low-level data transfer occurring between the CPU and GPU. I imagine they are using multi-threading to accomplish this. We should test this with the new torch.distributions.Categorical in 0.3 as well and then report back.

Hello,
In fact on cpu is much faster then gpu: Wall time: 20.2 s vs Wall time: 1min for 100 realizations.
Yes, I think it is worth it to chank this part in our code.
I've changed the code to Pytorch 0.3 and I will run it for 100000 batches during next days. You can find the notebooks here: https://www.dropbox.com/s/6wxwllae643e673/sampling.zip?dl=0


App Logging, Monitoring & Alerting

The app should replace all print statements with something to syslog, and we should have a plan in place to monitor for specific app failures (Checks failing to load, database connection resets) to allow the scoring team to configure alerts and response to faults in the system

Would be nice to have a config option of the verbosity of logs as well

Replaced all print statements with a logger, and the systemd scripts send it to syslog. I also created a couple configs for rsyslog to drop each (web, worker, engine) into separate log files.


Prepare for v0.5.0

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Release note:
none

/retest


workers_launch_template/arn constantly changing on each apply when using module.eks.aws_autoscaling_group.workers_launch_template

I'm submitting:

[X] bug report

What is the current behavior?
This is odd... with each TF apply when using this module I get this:
  # module.eks.aws_autoscaling_group.workers_launch_template[0] will be updated in-place
  ~ resource "aws_autoscaling_group" "workers_launch_template" {
        arn                       = "arn:aws:autoscaling:eu-west-1:XXX:autoScalingGroup:SOME-UUID:autoScalingGroupName/dev-test-eks-LONGNUMBER"
...
      ~ target_group_arns         = [
          - "arn:aws:elasticloadbalancing:eu-west-1:NUMBER:targetgroup/dev-test-eks-elb/HASH",
        ]

as far as I observed, this shouldn't happen at all, cause as a result that asg gets drained and cycled - it's not a feature right ? :P Let me know how to avoid it pls.
If this is a bug, how to reproduce? Please include a code sample if relevant.
Well basically, try to use this module and set worker_groups_launch_template = local.worker_groups_launch_template to a meaningful and working value :D
What's the expected behavior?
Either hide that field in the TF provider if it is to change every time you apply, or make it somehow ignored by a flag in module. Or explain why it is like that and leave it if this will not harm live traffic.
Are you able to fix this problem and submit a PR? Link here if you have already.
Ain't nobody got time for that now
Environment details

Affected module version: 13.2.1

this looks like it might be similar to: https://github.com/terraform-aws-modules/terraform-aws-eks/issues/737

this looks like it might be similar to: https://github.com/terraform-aws-modules/terraform-aws-eks/issues/737

anyone ?

duh, found a field called target_group_arns and for those who will wander around here trying to get an lb connected to managed worker group, the solution is as follows:
  worker_groups_launch_template = [
    {
      override_instance_types = [ var.eks_config["worker_pool_type"], ]
      asg_desired_capacity    = var.eks_config["nodes_per_az_minimum"] * length(data.aws_availability_zones.available.zone_ids)
      asg_min_size            = var.eks_config["nodes_per_az_minimum"] * length(data.aws_availability_zones.available.zone_ids)
      asg_max_size            = var.eks_config["nodes_per_az_limit"] * length(data.aws_availability_zones.available.zone_ids)
      public_ip               = false
      asg_recreate_on_change = false
      target_group_arns = [ aws_lb_target_group.me_little_alb.arn ]
    },
]


0850b->false is not applied if 0850a is at default true

0850a = browser.urlbar.autocomplete.enabled (default = true)
0850b = browser.urlbar.suggest.history/boomark/openpage (3 prefs, default = true)
we currently have 0850a inactive and 0850b as active
observations

using about:config

If 0850a is toggled to false, then all of 0850b is flipped to false
if 0850a is toggled to true then all of 0850b is flipped to true
if any (or all) of 0850b is set to true it makes sure 0850a is also true
if all of 0850b are set to false it will flip 0850a to false

as long as one of the three are true, then 0850a must be true

using the UI (you can only toggle the three 0850b prefs here)

if any of the three are checked (true) then 0850a -> true
only when all three are unchecked is 0850a -> false

So far so good. Exactly as expected. Works both ways.

user.js

a) if 0850a is set active (and 0850b is inactive) = works as advertised. The three 0850b prefs are also set to false or true as per value of 0850a
b) If 0850a is inactive, then 0850b may not be applied from the user.js depending on the currently set value for 0850a
c) not tested if 0850a AND at least one of 0850b is active

more to follow

the user.js behaviour is kinda hit and miss. for example, in a) above:
if the 3prefs are already false in prefs.js, and from the user.js I ONLY set 0850a as true, then all 3prefs are flipped. BUT if only one or two of the 3prefs are already set as false (i.e at least one pref is already true) then none of the 3prefs change, because no rule was broken.
It makes sense, that if it needs to flip the 3prefs, then rather than decide which one as a minimum. it does all three, otherwise if at least one is true then nothing needs to be done.
Just need to work out how to combine this/word it. But for sure, 0850a is more of a master than the 0850b's when it comes to the user.js

this is about the best I came come up with for now

clarified the rules listed in 0850b
made them all active false to be consistent

/* 0850a: disable location bar autocomplete [controls 0850b] ***/
user_pref("browser.urlbar.autocomplete.enabled", false);
/* 0850b: disable location bar suggestion types [controls 0850a]
 * [SETTING-56+] Options>Privacy & Security>Address Bar>When using the address bar, suggest
 * [SETTING-ESR] Options>Privacy>Location Bar>When using the location bar, suggest
 * [NOTE] If any of these are set true from the UI or about:config, 0850a will be FORCED to true
 * and if all three are set as false from the UI or about:config, 0850a will be FORCED to false
 * [NOTE] Setting all three as false from the user.js does not change 0850a
 * [WARNING] If all three are false, search engine keywords are disabled ***/
user_pref("browser.urlbar.suggest.history", false);
user_pref("browser.urlbar.suggest.bookmark", false);
user_pref("browser.urlbar.suggest.openpage", false);

looks good to me

A nice choice!

tweaked it, see edited version above, but its still not 100% true. Maybe we could use something simpler
/* 0850: disable location bar autocomplete and suggestion types
 * If you enforce any of the suggestion types, you MUST enforce 'autocomplete'
 * If *ALL* of the suggestion types are false, 'autocomplete' must also be false
 * If *ANY* of the suggestion types are true, 'autocomplete' must also be true
 * [SETTING-56+] Options>Privacy & Security>Address Bar>When using the address bar, suggest
 * [SETTING-ESR] Options>Privacy>Location Bar>When using the location bar, suggest
 * [WARNING] If all three suggestion types are false, search engine keywords are disabled ***/
user_pref("browser.urlbar.autocomplete.enabled", false);
user_pref("browser.urlbar.suggest.history", false);
user_pref("browser.urlbar.suggest.bookmark", false);
user_pref("browser.urlbar.suggest.openpage", false);

This seems simpler (and saved 1 whole line?)

^^ removes all the rubbish about the ui & about:config (which btw conforms to the two rules I left in) which just confuses things. This refers just to the user.js and makes it as basic as I can get it

removes all the rubbish about the ui & about:config which just confuses things.

I didn't dare to say that myself, but I totally agree :)
Doesn't really need the 4th line (5th if you count the title) IMO.

* If you do not enforce the above, the settings may not be applied by the user.js
I added it as a bit of an afterthought - cuz people will ask .. but why? what happens if... ? It shoudl be kinda obvious. Maybe I should stop pandering...


Use explicit constructor in Draggable to fix IE10 null reference

There's a problem in IE10 with ES2015 classes and the super class.
Essentially the prototype isn't set up correctly which leads to
this.props being null when setting up position in the construction.
However, props are passed to the class constructor explicitly by React,
so we can use them from there.
Tested by running the examples in IE 10 on Win 7.
This is basically the same problem as addressed in #145, but fixed to work rather than just not blow up (and for version 2 props).

So it appears class property assignments are broken in Babel with IE < 11?

No, it's more that accessing properties placed on the object by the super constructor is broken.
this.props works no better in the explicit constructor than in a class property.
For example, writing this for Draggable:
 constructor(props) {
  super(props);
  console.log(this.props);
 }

On chrome: Object { zIndex: 100, children: Object ... }
On IE10: undefined
As I understand it there's a basic incompatibility between IE <= 10 and babel ES2015 classes because you can't set the prototype properly. There's a bunch of work arounds but they all have edge cases and side effects.
Using the explicit constructor just gives us independent access to the props as part of the published React API.

:+1: Thanks for the further information. It's a shame this bug exists, and I appreciate you taking the time to fix it.

Published in v2.0.1.


Remove credscan stage from our build pipeline. it is blocking releases

Further discussion in #30683

The fact that this stage can invoke, means that the yml change is a safe one. Overriding and merging. No reason to wait.

/check-enforcer override


Meaningful unit test coverage - Pagination

Problem
As part of issue #59, all snapshots tests where removed. We would now like to replace these snapshots with a more meaningful unit test.
Acceptance Criteria
Using @testing-library/react create a test for Pagination.js found in src/components/Pagination/

I'd like to add the tests for this if nobody else is working on it.

Awesome! Thanks for picking it up.


Point to the right version of core-js 😇

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: trailing slash in request url & remove whitespace when parsing external config properties

ref: https://github.ibm.com/arf/planning-sdk-squad/issues/1738, https://github.ibm.com/arf/planning-sdk-squad/issues/1742
Changes/fixes:

Empty path segments "" are no longer added to the request url to prevent the java request url builder library from adding a / in place of the empty path segment
When parsing a credentials file (vcap or environment file), the credential's key/value pairs are trimmed of whitespace

:tada: This PR is included in version 8.1.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Merge pull request #1 from ktaranov/master

Merge changes to my fork
Fixes # .
Changes proposed in this pull request:

How to test this code:

Has been tested on (remove any that don't apply):

Case-sensitive SQL Server instance
SQL Server 2008
SQL Server 2008 R2
SQL Server 2012
SQL Server 2014
SQL Server 2016
Amazon RDS
Azure SQL DB

merging to my repro


hlsjs: Video not properly encoded

I have seen reported this problem many times and seems like you fixed them all. However, with latest version of hlsjs + using Wowza Streaming Engine 4.5 the same problem appears.
When I try to seek to a point which has not been yet downloaded (beyond buffer indicator) I get this:
"hlsjs: Video not properly encoded"
Although, seeking to a point that is already downloaded works fine. I tried two fixes, none of them worked:

add codec and resolution to the m3u8 playlist
converted the mp4 file to move the moov atom to the begining of the file

My m3u8 looks like this:
#EXTM3U #EXT-X-VERSION:3 #EXT-X-STREAM-INF:BANDWIDTH=634832,CODECS="avc1.66.21,mp4a.40.2",RESOLUTION=484x272 chunklist.m3u8
I would be happy to send you my stream URL, but do now want to disclose here... let me know what to do.

@akovacs123 - does the stream work in the hls.js sample player with the currently used version 0.6.14 in the plugin: http://streambox.fr/mse/hls.js-0.6.14/demo/ ?

@blacktrash: wow, it works. I used "strict: true" that made the error pop-up. Without this option, it is okay now. Thanks for pointing to the sample player.

@akovacs123 - depending on what exactly you want, playing with the (not so new) recoverMediaError and recoverNetworkError options may be useful for testing. With obviously greater care in production: https://flowplayer.org/docs/plugins.html#hlsjs-options


[question] spatial audio for discord

hello bep! I came across your plugin in this video and even though you mentioned it's broken, seemed to work just fine for me (my friends only heard me speaking from the left side but still, seems like it worked)
I was taking a look at your plugin's code and couldn't help but wonder: would it be possible to create this, but "reversed"?
What I mean by that is, instead of stereo-izing your input, doing so for the output. I imagine something like automatically (or manually) assigning everyone to some kind of "seat" in a virtual 3d space and every voice coming from different directions. This would specially be useful for when there's a lot happening (music, multiple people speaking, etc)
I want to take a little look at this, but from the context you've already acquired by doing this, maybe you know if it's impossible to do so straight from the get go? 😄 thanks!

hello,
its easier to do this with the built in discord capability of the RPC server, like is done for that among us spatial voice chat mod.
the capability is here https://discord.com/developers/docs/topics/rpc#setuservoicesettings

also the left side audio thing is the reason why its broken. discord does some internal c++ audio processing built for left channel only thats too much of a pain to fix + everyone who keeps demanding i fix it is really annoying to me about it


Add my new Tidbyt app for Port Imperial Bus Times

My app displays NJ Transit Bus Times for the stop # 21923 in Port Imperial, New Jersey. I want to modify this in the future so users can input any bus route and bus stop number!

I have read the CLA Document and I hereby sign the CLA

recheck

recheck

I have read the CLA Document and I hereby sign the CLA

Canceling PR


[SPARK-10770][SQL] SparkPlan.executeCollect/executeTake should return InternalRow rather than external Row - WIP

Almost certain tests would fail.

Merged build triggered.

Merged build started.

Test build #42892 has started for   PR 8878 at commit bfd0ffe.

Test build #42892 timed out     for PR 8878 at commit bfd0ffe     after a configured wait of 250m.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42892/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #42928 has started for   PR 8878 at commit 564fe6c.

Test build #42928 has finished for   PR 8878 at commit 564fe6c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42928/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #42944 has started for   PR 8878 at commit 621df6e.

Test build #42944 has finished for   PR 8878 at commit 621df6e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42944/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #42955 has started for   PR 8878 at commit f91733a.

Test build #42955 has finished for   PR 8878 at commit f91733a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42955/
Test PASSed.

Merged build finished. Test PASSed.

I rebased and created a new PR: https://github.com/apache/spark/pull/8900


Is there any way we could get the translations to update automatically?

The problem with translations is that each time a resource is added to the main repo, the translation becomes out of date.
Is there a simple way to keep the translations but to automatically add any new links?
The only way I can think of would be a single repo with translated snippets for each section and resources as objects, and pull it all together in different languages... but I feel that's too much :joy:
Otherwise maintainers of each translation will just have to watch the main repo and manually update their translation.

Well actually the best-case scenario would be to only list articles in French (for instance) on the French translation repo: if somebody can read an article in English, they'll head over to the English language repo :slightly_smiling_face:


Font Customization for Placeholder Label

I can set the font on the UITextField I'm using, but how do I specify the font for the label above the text field?

The "titleFont" property should allow you to set the title font - however, there are two caveats:

The font you set will be used only in non-placeholder mode. In place-holder mode, the control will use the standard font since the control is using the built-in placeholder display functionality.

The title does not adjust for the font size changes currently. This is a bug that I'm in the process of fixing right now.


VsCode prettier setup

I created a project with npm init adonis-ts-app and selected the option Setup eslint? (y/N) · true. This setup gives me the following .eslintrc.json
{
  "extends": [
    "plugin:adonis/typescriptApp"
  ]
}

I also have configured my .vscode/settings.json to this
{
  "editor.formatOnSave": true,
  "editor.codeActionsOnSave": {
    // For ESLint
    "source.fixAll.eslint": true,
    // For TSLint
    "source.fixAll.tslint": true,
    // For Stylelint
    "source.fixAll.stylelint": true
  }
}

But when I save the .ts files, prettier seems to ignore eslint settings.
What should I do to make eslint correctly format my .ts files on save?

Found a solution https://github.com/adonisjs/core/discussions/1590


Fix metadata, update base image version and add prompt.

This PR updates the container image metadata to include the version value in the org.label-schema.docker.cmd label and also correctly render the ${HOME} variable in that label.
Existing:
PS> docker image inspect azuresdk/azure-cli-python:2.0.20 --format "{{json .Config.Labels}}" | ConvertFrom-Json | ConvertTo-Json
{
    "org.label-schema.build-date":  "2017-10-19T18:26:52Z",
    "org.label-schema.description":  "The Azure CLI 2.0 is the new Azure CLI and is applicable when you use the Resource Manager deployment model.",
    "org.label-schema.docker.cmd":  "docker run -v :/root -it azuresdk/azure-cli-python:\u003cversion\u003e",
    "org.label-schema.license":  "MIT",
    "org.label-schema.name":  "Azure CLI 2.0",
    "org.label-schema.schema-version":  "1.0",
    "org.label-schema.url":  "https://docs.microsoft.com/en-us/cli/azure/overview",
    "org.label-schema.usage":  "https://docs.microsoft.com/en-us/cli/azure/install-az-cli2#docker",
    "org.label-schema.vcs-url":  "https://github.com/Azure/azure-cli.git",
    "org.label-schema.vendor":  "Microsoft",
    "org.label-schema.version":  "2.0.20"
}

Updated:
PS> docker image inspect azuresdk/azure-cli-python:2.0.20-test --format "{{json .Config.Labels}}" | ConvertFrom-Json | ConvertTo-Json
{
    "maintainer":  "Microsoft",
    "org.label-schema.build-date":  "2017-10-26T00:38:26Z",
    "org.label-schema.description":  "The Azure CLI 2.0 is the new Azure CLI and is applicable when you use the Resource Manager deployment model.",
    "org.label-schema.docker.cmd":  "docker run -v ${HOME}/.azure:/root/.azure -it azuresdk/azure-cli-python:2.0.20-test",
    "org.label-schema.license":  "MIT",
    "org.label-schema.name":  "Azure CLI 2.0",
    "org.label-schema.schema-version":  "1.0",
    "org.label-schema.url":  "https://docs.microsoft.com/en-us/cli/azure/overview",
    "org.label-schema.usage":  "https://docs.microsoft.com/en-us/cli/azure/install-az-cli2#docker",
    "org.label-schema.vcs-url":  "https://github.com/Azure/azure-cli.git",
    "org.label-schema.vendor":  "Microsoft",
    "org.label-schema.version":  "2.0.20-test"
}

This PR also changes the prompt in the container to azure-cli.
PS> docker run -v ${HOME}/.azure:/root/.azure -it --rm azuresdk/azure-cli-python:2.0.20-test
azure-cli# az vm
--help                  convert                 encryption              list                    nic                     run-command             user
-h                      create                  extension               list-ip-addresses       open-port               show                    wait
assign-identity         deallocate              format-secret           list-sizes              perform-maintenance     start
availability-set        delete                  generalize              list-skus               redeploy                stop
boot-diagnostics        diagnostics             get-instance-view       list-usage              resize                  unmanaged-disk
capture                 disk                    image                   list-vm-resize-options  restart                 update
azure-cli# az vm

This PR also updates the container base image version to python:3.6.3-alpine.

@derekbekoe  - I messed up the commits trying to squash. I will submit a new PR with your requested changes.


Update index.js

Resolve 'all' event set response header twice in expressjs

It's my first time to fork and pull request :-)


Cakephp 3.6 compatibility

Hello,
Do you have any plans on upgrading this plugin to work with Cakephp 3.6?
Thanks!

@stripthis https://github.com/burzum/cakephp-imagine-plugin/tree/3.x should be 3.6 compatible. But what, except the deprecation warnings, is stopping you from using it with an older version?

@burzum yes I was talking about the deprecation warnings :)

@stripthis give this branch a try. https://github.com/burzum/cakephp-imagine-plugin/tree/3.x


Keystroke sent to address bar after URL redirection on macOS Safari

After URL redirection, the address bar is activated by default on macOS Safari. It means any keystroke after URL redirection is sent to the address bar rather than the webpage.
This is counter-intuitive because users normally work on the webpage after typing a URL. For example, people may press space key to browse/scroll the webpage after typing bbc.co.uk in the address bar. This problem does not exist on ipadOS Safari though.
A simple fix could be using Javascript to send a escape keystroke after URL redirection if the browser is macOS Safari.

Hi @m-zheng, sorry for the late reply, and sorry, I'm not sure if I understand you correctly.
For example, this is how the redirection works with https://github.com/mshibanami/redirect-web/discussions/5, but the address bar is not activated after that.:
https://github.com/mshibanami/redirect-web/assets/1333214/45c8b605-062d-4692-9b8d-8ba74831e809
Would you please send me a video and the redirect rule you tried?

Oh, I found your email reporting the same issue with a video. I understand the issue now.
I see. This seems to be a special case of startpage.com you want to jump to. Safari on macOS doesn't seem to properly handle the pre-loading websites for the search results of the text typed on the address bar, and the focus remains on the address bar as a result.
I don't think the escape key event triggered by Javascript can manipulate the browser behaviour. But anyways, this is Safari's issue, I guess.
I'll investigate a bit more and submit feedback to Apple so that they will address it.
Thank you for reporting it!

Thanks for the update. I will close the ticket for now because the culprit is Safari.


Trying to get the project running

Hello! 👋
I'm an engineer on the documentation team at GitHub. My colleague @sarahs and I have been watching this project for a while, and now I'm finally trying it out.
I've got this set up:

[x] App is running on Node.js 10, macOS
[x] Created a Google API project with Drive and Cloud Store access
[x] Created a service account
[x] Created server/.auth.json with credentials file created by Google
[x] Created OAuth 2 client id
[x] Created .env with all values
[x] Created a new folder in Drive using my personal Google account and updated related values in .env
[x] Created a new Google Doc in that folder with some filler content

Unfortunately, I'm still seeing a lot of errors in my console. I added them to a gist here: https://gist.github.com/zeke/c2572650c5211985b33aa42106f1ec02
Some highlights from that output:

'Cannot find module '/Users/z/git/forks/library/custom/cache/store'
warn: GOOGLE_APPLICATION_CREDENTIALS was undefined, using default ./.auth.json credentials file...
Cannot find module \'/Users/z/git/forks/library/custom/userAuth\''
Cannot find module \'/Users/z/git/forks/library/custom/csp\

Context (Environment)
$ node --version
v10.16.2

$ uname -a
Darwin calvisitor-10-105-181-69.calvisitor.1918.berkeley.edu 18.7.0 Darwin Kernel Version 18.7.0: Sat Oct 12 00:02:19 PDT 2019; root:xnu-4903.278.12~1/RELEASE_X86_64 x86_64

Expected Behavior
When running npm run watch, I expect to see a local running server.
Actual Behavior
I see a bunch of errors: https://gist.github.com/zeke/c2572650c5211985b33aa42106f1ec02
To Reproduce
Follow the checklist above. :)
Possible Solution
It looks like there are some undocumented expectations in the code about certain local files that need to exist.
Maybe the Google App doesn't have access to the Google Drive folder I created?
I'd be happy to jump on a video call with someone to pair on this. Once I've got a successfully running installation of library I'd be happy to follow up with updates to the documentation to make the setup process more clear for newcomers.
Thanks for reading!

Hey @zeke! Thanks for providing such a detailed description of the steps you tried. Many of the traces you mentioned are noncritical, and probably unrelated to the problem you are encountering. (They're often expected, and we should make them less prominent.)
One of the later traces displayed in your console is most likely the cause of the issue:
[0] debug: searching for files > 0
[0] warn: failed updating tree
[0] { message: 'File not found: .',
...
 data:
[0]       { error:
[0]          { errors:
[0]             [ { domain: 'global',
[0]                 reason: 'notFound',
[0]                 message: 'File not found: .',
[0]                 locationType: 'parameter',
[0]                 location: 'fileId' } ],
[0]            code: 404,
[0]            message: 'File not found: .' } },

That trace comes from the part of the code where we make a request to the Drive API to list the documents, and could result from a lack of permissions. Did you share the folder you created with the email of your service account? That's the final piece that grants the service account access.

Thanks for the quick response!

Did you share the folder you created with the email of your service account?

☝ I had not! Now I have:

Still seeing the same (or similar) errors in the console after restarting the app. Does that invitation need to be accepted somehow by the service account?

Normally adding the service account is all that should be needed, though in some cases with shared folders (vs a Shared/Team drive) we've noticed permissions can take a little while to update on Google's side.
Are you still seeing that File not found error in your application logs since you added the account a few hours ago? In particular, the continued presence of this log line would suggest something else could also be wrong:

warn: failed updating tree

Just tried again this morning. Still seeing warn: failed updating tree.

Hi @zeke ! Sorry for the delayed response. Your setup process looks correct, but based on errors I've seen in the past, this seems like it might be a configuration issue. A few things to check/try:

Put the Google doc with filler content into a folder nested within your shared folder. Do you still see the failed updating tree error message?
In your .env file, is DRIVE_TYPE set to folder or shared? There's conflicting documentation on this (working on updating now!), but it should be set to folder.
Check that DRIVE_ID in .env includes the entire hash in the Google Drive folder's url, that .auth.json is in the server/ directory, and that GCP_PROJECT_ID in .env is the same as "project_id" in .auth.json

I'll reach out and we can find some time to pair if nothing mentioned above resolves the problem.

I am having the same exact problem as @zeke.  I was wondering if anyone has made any traction on this issue?

Hey @maxine thank you for this info. I haven't yet made time to try out your suggestions above, but I intend to, and will report my findings here. Perhaps over the holiday break. :) 🎄

Hi @maxine!  FYI, I'm getting some errors on startup similar to @zeke.  Your suggestion to put the filler content in a nested folder took care of the tree updating errors, thank you very much.  But I do still see the MODULE_NOT_FOUND errors:
"Cannot find module '/Users/scottblumenthal/workspace/blumysden/library/custom/cache/store'",
...
"Cannot find module '/Users/scottblumenthal/workspace/blumysden/library/custom/userAuth'",
...
"Cannot find module '/Users/scottblumenthal/workspace/blumysden/library/custom/csp'"

Pretty sure all the auth stuff you listed above checks out.  And the app is running and displaying my files, so that's good!
(I'm excited to finally get to play around with my own Library install!)

Due to the topic of this issue, it is being converted into a discussion! This issue will be closed, but discussion can continue in the newly created discussion.


#MindSpore2023 Your Voice Matter — MindSpore Resolution 2023

Dear valued developers,
MindSpore is our versatile deep learning framework, and we're constantly looking at new ways to make AI modeling and execution more efficient, and as well as cover more scenarios. With 2023 just around the corner, the MindSpore community would like to wish you a happy holiday – we hope you'll give us some feedback, so that we can create a community that best serves your needs.
We'd appreciate it if you could take a few minutes to answer the three questions below – please write your comments on this page. Thank you for your support!

What changes would you like to see to MindSpore in 2023? Feel free to list a few different suggestions.

Are you interested in MindSpore meetups, courses, tutorials, and conferences? If so, which interests you the most? What do you hope to gain from these activities?

Would you consider using MindSpore in your Google Summer of Code (GSoC) projects in 2023? If yes, please give us more details.

The MindSpore community highly values your feedback. We have prepared special Christmas prizes for a few lucky winners. Leave a comment below for the chance to be picked; all winners will be contacted via GitHub. Merry Christmas and a happy new year!
For more info, follow our social media accounts.
Twitter: https://twitter.com/MindSporeOSS
LinkedIn: https://www.linkedin.com/company/80968760/
YouTube: https://www.youtube.com/@MindsporeOSS
Reddit: https://new.reddit.com/r/MindSporeOSS/

I expect to see MindSpore incorporating more operators of mainstream AI frameworks.
I'm most interested in courses about computer vision subfields such as object identification and classification, so that I can try MindSpore on my previous CV projects.
I'd be happy to use MindSpore for my planned project of GSoC of a smart camera that can recognize and identify different pets in a house.


[update] go modules

内部仕様の変更点
インフラ系のライブラリに変更があったので更新

Codecov Report

Merging #580 (5e0b20d) into master (6f46935) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #580   +/-   ##
=======================================
  Coverage   82.05%   82.05%
=======================================
  Files           5        5
  Lines          78       78
=======================================
  Hits           64       64
  Misses          7        7
  Partials        7        7

Flag
Coverage Δ

unittests
82.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Include  to avoid "error: call to 'abs' is ambiguous"

Only integer versions of abs() in <cstdlib> seem to be available without explicit inclusion of <cmath>, and it generates the following errors in compilation of some tests:
~/git/clipp/test/mixed_params_test.cpp:31:12: error: call to 'abs' is ambiguous
        if(abs(x.cv - y.cv) > 1e-4f || abs(x.dv - y.dv) > 1e-4f ||
           ^~~
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/stdlib.h:137:6: note: candidate function
int      abs(int) __pure2;
         ^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/stdlib.h:111:44: note: candidate function
inline _LIBCPP_INLINE_VISIBILITY long      abs(     long __x) _NOEXCEPT {return  labs(__x);}
                                           ^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/stdlib.h:113:44: note: candidate function
inline _LIBCPP_INLINE_VISIBILITY long long abs(long long __x) _NOEXCEPT {return llabs(__x);}

macOS 10.13.6
Apple LLVM version 9.1.0 (clang-9<IP_ADDRESS>)

Thanks for fixing this!


Use mention on next turn

After a player has made a move, use an @mention for the current person in the embed so they're notified

This was closed with the prior commit that added PGN support


How to add include paths using clangd server?

I have tried adding the clangd arguments with --enable-config with .clangd in my root project.
.vimrc's content
call LspAddServer([
	{
		filetype: ['c', 'cpp'],
		path: '/usr/bin/clangd',
		args: [
			'--enable-config'
		],
		initializationOptions: {
			fallbackFlags: [
				'-std=c++17',
			]
		}
	},
	{
		filetype: ['php'],
		path: '/usr/local/bin/intelephense',
		args: ['--stdio'],
		syncInit: v:true,
	}
])

.clangd's  content
CompileFlags:
  Add:
    - "-std=c++17"
    - "-I."
    - "-I./src"

The LSP diagnostic always says "file not found"

Should you add a compile_commands.json file in the source or build/ directory?

It turns out .clangd is the configuration for specific folder.
The thing I need is compile_flags.txt to replace all flags to the whole projects.
compile_flags.txt's content
-I.
-I./src