Error after upgrade to Ionic 3.13.0

$ ionic cordova run browser
[INFO] Running app-scripts build: --platform browser --target cordova

[12:41:50]  build dev started ...
[12:41:50]  clean started ...
[12:41:50]  clean finished in 145 ms
[12:41:50]  copy started ...
[12:41:50]  transpile started ...
[12:41:55]  transpile finished in 4.71 s
[12:41:55]  preprocess started ...
[12:41:55]  deeplinks started ...
[12:41:55]  deeplinks finished in 348 ms
[12:41:55]  preprocess finished in 349 ms
[12:41:55]  webpack started ...
[12:41:56]  copy finished in 5.42 s
[12:42:11]  webpack finished in 16.32 s
[12:42:11]  sass started ...
[12:42:13]  sass finished in 1.58 s
[12:42:13]  postprocess started ...
[12:42:13]  removed unused font files
[12:42:13]  postprocess finished in 27 ms
[12:42:13]  lint started ...
[12:42:13]  build dev finished in 23.27 s
> cordova run browser
Unable to load PlatformApi from platform. Error: Cannot find module
D:\Desenvolvimento\xampp\htdocs\app-reciklado-ionic\platforms\browser\cordova\Api.js
Error: Uncaught, unspecified "error" event. (The platform "browser" does not app                                                                                ear to be a valid cordova platform. It is missing API.js. browser not supported.                                                                                )

[ERROR] An error occurred while running cordova run browser (exit code 1).

ionic cordova run android work without error

Hello! Thank you for opening an issue with us!
The Ionic Framework issues list is designated for the reporting of bugs with the Ionic Framework or for requesting new features. As this seems like more of a support question I am going to close this issue.
I urge that you ask this question on our forum or on our slack channel.
This may also be a legitimate issue with the Ionic CLI. If that turns out to be the case, please log an issue in the CLI repo: https://github.com/ionic-team/ionic-cli
Thanks for using Ionic!


Programming exercises: Don't create new submissions for test case changes

Checklist
General

[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I documented the Java code using JavaDoc style.

Motivation and Context

We noticed a bug where the template was not rebuilt after the tests were changed. This lead to some debugging and the decision that in the case of a test case change, there should not be a new submission. Instead the results of the template and solution will be added to the latest submission. Only if no corresponding submission exists, a new one is created.
Description

No longer create new submissions if the test case changed. Additionally the build for the template is now triggered immediately when the REST endpoint for a test case change is called and not after receiving the result for the solution.
Steps for Testing

Prerequisites:

1 Instructor
1 Programming Exercise

Log in to Artemis
Create the exercise
Submit to the Template repository
Check that Artemis correctly displays the building animation, e.g. at the bottom of the programming exercise details view
Check that a new submission was created for the Template
Submit to the Solution repository
Check that Artemis correctly displays the building animation, e.g. at the bottom of the programming exercise details view
Check that a new submission was created for the Solution
Submit to the Test repository
Check that Artemis correctly displays the building animation for both repositories, e.g. at the bottom of the programming exercise details view
Check that no new submission was created but the results are associated with the submission before

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

We will try to solve this differently

I added a simpler bugfix for now in #7005

@JohannesStoehr What's exactly to be done here? Can we close this in favour of Stephan's PR?
Otherwise, consider converting the PR into a draft PR.

Fixed by another approach


Add Range to GetBlobOptions and fix return response

I haven't written tests for this yet but I'm happy to if you think this approach is OK.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Hello @ncw !
The operation you are looking for is GetRange, and it is already supported.

@ncw, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

@mcardosos thank you - I've tested that and it works fine :-)

@ncw Awesome!
In any case, thanks for submitting a PR :D


fix(anta): Workaround to bypass resource on none posix system

Description
WIP
for now detecting when using non posix system and bypassing resource call
Fixes #917
TODO

[x] Add windows runner in CI
[ ] Add test (faking system)
[ ] Find a solution to do the same check on Windows (if achievable)

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have run pre-commit for code linting and typing (pre-commit run)
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes (tox -e testenv)

psutil was a deadend for windows rlimit: https://github.com/giampaolo/psutil/issues/1149


Should we remove commit: true as an option?

Currently the commit option is the reason we separate config by command - which we would like to stop doing (it would greatly simplify the config).
As part of this, I want to talk about where changesets intersects with git - and should it?
Currently, we intersect with git in:

Committing changesets + committing bumps
writing changelogs (we default grab and hand back commit info)
status has --since-master option, which will make it care about git

We've said in the past that we don't want changesets to be tied to git, so what happens if we don't tie it to git?

Committing bumps is still easy enough to do in CI when you want to commit them
Committing changesets become more likely to be missed - you can add a changeset, push changes, and have forgotten to add the files - this seems a weak argument to me.
Writing changelogs having access to the git hash helps out a lot with writing changelog messages. Could the logic that fetches it be split into an optional library, similar to how fetching info from github is split out?
The status command doing this still seems fine.

We've said in the past that we don't want changesets to be tied to git

Where was this said before? I vaguely agree that not tying changesets to git is a good thing but I'd be interested to hear some reasons behind it.
More things for the list of places where we use git:

Getting the changed packages when creating a changeset
Creating tags in release (this might be hidden behind bolt?)

Committing bumps is still easy enough to do in CI when you want to commit them
Committing changesets become more likely to be missed - you can add a changeset, push changes, and have forgotten to add the files - this seems a weak argument to me.

👍

Writing changelogs having access to the git hash helps out a lot with writing changelog messages. Could the logic that fetches it be split into an optional library, similar to how fetching info from github is split out?

I think this is okay on the condition that we have the changelog packages stuff we've been talking about exists because people having to write even more complex changelog entry generator things would be annoying. One thing that we'd have to make sure of if we did this is only deleting the changesets after the changelog entries are generated.

Where was this said before?

Good callout. This has come up in some internal brainstorming when presenting changesets. When we went to move it into Keystone, it always committed, and not committing was added as a 'feature' for keystone.
Since then, it's been sort-of an ideology, and sort-of not. There are some aspects of changesets that clearly want info from git, or don't work nearly as well, but at a conceptual level, changesets are interesting because they are just writing and consuming files.
As such, changesets baking itself on top of git is more an accident than a decision, and getting clear separations around it would be nice. That or this exploration leads us to more clearly scope where changesets touch git, and refuse to budge on it touching in other places.

Getting the changed packages when creating a changeset

This, to me, is the most compelling use of git. It's the same as the status usage, effectively (get something since master). We can probably include this, but not fail the command if we can't find this info, which I believe it currently does.
I think the standard here should be 'works well with git', but not 'breaks without git'

I think the standard here should be 'works well with git', but not 'breaks without git'

I like this a lot.
So the only places that changesets itself will care about git are:

add for getting the changed packages
status for --since-master

And there will be a helper for getting the commit from a changeset. (I assume we'll pass the name of the changeset in then?) The one thing that slightly worries me is the release line generators caring about things that exist in the file system but that's probably okay.
If that's correct then I think I'm happy with this.

Just a little thing on this: I just ran changeset add in a directory without a git repo and it failed
❯ yarn changeset
yarn run v1.16.0
$ /Users/mitchellhamilton/projects/changesets-test-thing/node_modules/.bin/changeset
Error: fatal: not a git repository (or any of the parent directories): .git

    at ChildProcess.child.on.code (/Users/mitchellhamilton/projects/changesets-test-thing/node_modules/projector-spawn/index.js:53:16)
    at ChildProcess.emit (events.js:189:13)
    at maybeClose (internal/child_process.js:970:16)
    at Socket.stream.socket.on (internal/child_process.js:389:11)
    at Socket.emit (events.js:189:13)
    at Pipe._handle.close (net.js:597:12)
error Command failed with exit code 1.

Gross. This is probably a separate issue. We're probably not going to remove git from 'add' - issue should probably be 'catch errors from git being absent and continue on as best possible'

👍 I'll open another issue.

@mitchellhamilton I think removing this is going to be more pain than it's worth - probably going to close this as discussion ended, but work on isolating the code that does the git adding etc into its own very clearly distinct function, not spread through the main workflow, unless you have strong opinions against.

👍 to that, my main opinion would be do the thing that is easiest/won't slow down getting v2 done(whether that's isolating the logic or not) for now and any refactoring we want to do can come later.


Allow switching contract address quickly

I'm now using quite a lot of code to constantly switch between which contract to use. Rob mentioned something was available in the module, though not exported.
It would be great if switching between target contracts would be super easy.

Would you mind sharing your code?

I refactored my code today using a Promise, and such that I can have a static factory address (from epm.json deployed).
function getContract(account, issueName) {
    return new Promise(function(resolve, reject) {
        factory.getContractByName(erisC.utils.padRight(erisC.utils.asciiToHex(issueName), 64), function(error, result) {
            contractData = require(epmPath);
            contractName = "Issue";
            rfcAbi = JSON.parse(fs.readFileSync(expandHomeDir("~/.eris/apps/simplechain/abi/" + contractName)));
            contractsManager = erisC.newContractManagerDev(erisdbURL, accountData[account]);
            resolve(contractsManager.newContractFactory(rfcAbi).at(result))
        });
    });
}

For every call I want to make on a contract, I first use the above code, followed by the code below:
getContract(req.user.alias, issueName)
        .then((x) => x.getStatus()   // where getStatus is a function on a contract

I guess it works fine for now, though it would be nice if it would be more straightforward.
Node is not my main language, so perhaps you have suggestions.

@NodeGuy this is basically what I was talking about in our call the other day

Agreed; this will be super easy with the new API.


Race condition when opening files leads to data loss

I have had this problem happen in other circumstances (disconnected windows into the same file), but I seem to be able to reproduce it reliably this way.
Open a terminal, type atom file.txt & ; atom file.txt  (make sure file.txt does not current exist)
Two windows into file.txt will open. In the top one, type some text, save and close.
The text is not reproduced in the second. Closing this window will prompt to save the file even though you have not typed anything. This will save a blank file, overwriting your text.

What version of atom were you using?

1.0.11, on Mac OS X.
However, I did just try running with --safe, and the bug seems to disappear, so it could be a package I'm running (or just with --safe atom starts faster and avoids the problem)

Is reproducible with --safe, just only seems to happen about 40% of the time on my machine.

Just a quick poke on this, I'm not sure what more information you want, and a friend of mine can reproduce it (also on Mac)

Can reproduce. Expected behaviour second window notice that file changed on the disk and loads updated version (assuming you have no unsaved changes for that file in that tab)


Feature Request: Save buffer to file

I am using ov as the pager of psql, and sometimes I would like to save the result of the query to a file.
Is it possible to add "Save to file" like less's -O and s?

Thank you for the issue.
I'm thinking of implementing a save function.

Even in the current version you can also save using a redirect.
\setenv PSQL_PAGER 'ov -F -C -d "|" -H1 --column-rainbow --column-mode >address.txt'
SELECT * FROM address;

It's not that I want to save every result. Usually, after seeing the result, I decide if I should save it.
I believe this is a common workflow when using the pager for output of a command.
For example, suppose you want to use ov for git and you set GIT_PAGER as you described. Then you would get the output file every time you invoke git. This would be so annoying.

yes. I'm trying to implement another.
However, psql's \setenv can be used to temporarily set things.

Implemented in v0.32.0. thank you.


Simplify barrier handling to make code easier to understand and debug

I need to investigate the following two CI failures which might be related to these changes:
https://circleci.com/gh/WallarooLabs/wallaroo/18389?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
https://circleci.com/gh/WallarooLabs/wallaroo/18383?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link


reading existing xml sitemap

Hi i am facing the following
i have dynamic website that adds post everyday and i want to create sitemap index and add post to the sitemap.
this can be achieved by this library  but as i am still developing more features i want a way to read the xml
maybe like this
var sitemap = sm.readSitemap({xmlPath:'route/to/xml/file });
then do something like :- sitemap.add({url: '/page-1/'});

is this possible to achieve because currently am stuck
PS:- i know there is no issue while the server is up and running meaning
var sitemap = sm.createSitemap ({       hostname: 'http://example.com',       cacheTime: 600000     });
sitemap here will always be defined and accessed but what happens when i restart the server for adding new features
please help me
thanks

Same concern,

I am just dynamically deploying it so it is 100% up to date.

Implemented. Feedback welcome.


AMD integration tracker

Collecting all the requests for better AMD support for AO here:

Reddit feedback
https://github.com/pytorch/ao/issues/958
https://github.com/pytorch/ao/issues/1066
https://github.com/pytorch-labs/gpt-fast/issues/12, https://github.com/pytorch-labs/gpt-fast/issues/78, https://github.com/pytorch-labs/gpt-fast/issues/6 in gpt-fast.

Current work items:

https://github.com/pytorch/ao/pull/1201
[] Run AO baseline benchmarks on AMD hardware

2:4 sparse hipSPARSELt integration

https://github.com/pytorch/ao/pull/1206
https://github.com/pytorch/pytorch/pull/124320

@jcaip  thanks we will look into this.


Remove registry from Azure job store

The AWS job store maintains a registry due to eventual consistency issues. @joelarmstrong do you recall why Azure also maintains a registry? If there does not exist a compelling reason it may be worth forgoing the cost of writing to a reading from an extra table in Azure.
Instead the existence of a job store can be determined by probing for some component of it that is eventually consistent.

No compelling reason at all, I kept the registry only for symmetry with the AWS jobStore, in case there was a use case for querying the registry to see what jobStores existed. The same functionality could be achieved by checking for the existence of the config pickle or something.

Thanks for the info @joelarmstrong. @hannes-ucsc with this in mind would you feel more comfortable removing the registry in #916 while I'm already touching that part of the code? Or shall we contend with it in a different PR?

I'm ok with removing it in #916.
Just make sure that the last storage component you remove is the one that you detect job store existence with. This applies to all job stores.


Refactor model creation using functions

What and Why
The function create_model is too long - and it's hard to profile. We should use functions to make the model creation modular.
Possible Drawbacks
No response
Related Issues
#489

Blocked by #255

I believe this is not blocked anymore. @datejada, confirm?

@abelsiqueira is correct; the changes in the model have been made, and the remaining issues in Epic #255 are related to documentation

Great idea, I think this is a nice one to pick because you can nibble at it, i.e., you can move one constraint to a function at a time. That way you can make sure that you don't have food poisoning - the tests are passing before moving to the next constraint - and you decide when to stop eating - create a pull request.
I hope all these eating metaphors don't make you throw up 😉

@abelsiqueira First of all, did I mention I had a food reaction/poisoning a week ago and threw up my lunch? 😅

😨 oops, you hadn't mentioned. I just have a natural ability to faux pas.

Also, can you give an overview of the changes I should do? Do you mean to separate each constraint into a function or something else? Should I be looking for patterns where the same function can do multiple constraints?

What I would try to do is move constraints by "theme". Like all balance constraints inside a function, possibly in a separate file. This should keep the number of files to a manageable amount. I would not look into patterns at first, just moving things around.
I would do it like this to move the balance constraints, for instance:

Create a new file (e.g., src/constraints/balance.jl)
Create a function (e.g., add_balance_constraints!(model, ...) (arguments are still unknown)
Select one of the balance constraints in src/create-model.jl, and cut-and-paste to the new function
VSCode should warn you of undefined variables and functions, which you can add as parameters to the function

You should be able to define x = model[:x] for JuMP variables and expressions, so you don't need them in the function arguments. But the dataframe, for instance, will need to be passed to the function.

Add a call to add_balance_constraints!(model, ...) in src/create-model.jl,
Run the tests, and fix stuff until it passed
Create a commit "move constraint ... to separate file/function"
Repeat until the function is done

That's what I'm thinking for now

@abelsiqueira First of all, did I mention I had a food reaction/poisoning a week ago and threw up my lunch? 😅

😨 oops, you hadn't mentioned. I just have a natural ability to faux pas.

Also, can you give an overview of the changes I should do? Do you mean to separate each constraint into a function or something else? Should I be looking for patterns where the same function can do multiple constraints?

What I would try to do is move constraints by "theme". Like all balance constraints inside a function, possibly in a separate file. This should keep the number of files to a manageable amount. I would not look into patterns at first, just moving things around.
I would do it like this to move the balance constraints, for instance:

Create a new file (e.g., src/constraints/balance.jl)

Create a function (e.g., add_balance_constraints!(model, ...) (arguments are still unknown)

Select one of the balance constraints in src/create-model.jl, and cut-and-paste to the new function

VSCode should warn you of undefined variables and functions, which you can add as parameters to the function

You should be able to define x = model[:x] for JuMP variables and expressions, so you don't need them in the function arguments. But the dataframe, for instance, will need to be passed to the function.

Add a call to add_balance_constraints!(model, ...) in src/create-model.jl,

Run the tests, and fix stuff until it passed

Create a commit "move constraint ... to separate file/function"

Repeat until the function is done

That's what I'm thinking for now

I agree with the procedure, but regarding the classification of how to split, I prefer to discuss it with the whole team before continuing. balance seems natural, but we have two structures for time (inter- and -intra) and maybe splitting by that criteria will make more sense (or not).

After some discussion we've decided to group primarily by asset type, to match the documentation and be clear to users.


Nukeops is broken again

Probably https://github.com/space-wizards/space-station-14/pull/10005

what a descriptive issue

The 3 other times it broke you could repro by just trying to load it but I don't have time to confirm it this time.


[DO NOT MERGE YET] Fixed z̨͖̦͐͂̔̋͆̃a̟͔̼̪̙ͪ̌l̞̤̱͚̺̮͈̒̓ͮg͛ͤ̉̋̈̆o̰̱̟̜͊͆͐̐̓͊͌̀ text

** DO NOT MERGE THIS YET, THIS CAUSES AN ISSUE WITH THE MENTIONS **
This PR:

Fixes #1970
Manually solves all the trouble caused by GitHub desktop (and other clients that do not support hooks)

can you reset the changes that don't have to do with zalgo text

can you reset the changes that don't have to do with zalgo text

I'll close this PR (for now) and re-add it once I'm done overhauling the chat parts


WIP: Docs

This branch/PR will track documentation work. I figured I'd toss it here so that we can start using it and tweak it as we hit issues.

@robhudson ^^^ That adds "Migrate from Tower" documentation. What do you think so far? Does that look usable?

This looks really good and is clear to me what I should do. :+1:

^^^ That adds an installation/configuration chapter. The installation is a lie. The configuration is hopefully correct, but isn't really tested until we have tests for extract and merge commands.

Closing this out because I landed it already.


Document additional entries in the CV_CFL_LANG enum

Several items were missing from the CV_CFL_LANG enum. This documents the latest set of values from cvconst.h

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

@arlosi Thanks! Sorry for the slow response here.

@Mikejo5000 thanks for the review! Let me know if there's anything else I need to do to get this merged.

@arlosi No, it will get synced and then published this week. Thanks!


Getter throws error about name format

Describe the bug
Getter methods for accessing planning entities are not usable. I am unsure if this is an intended behavior of Timefold or not, but this was the recommended method for OptaPy. If not a bug then increased documentation would be helpful.
Expected behavior
I expect to be able to annotate either atttributes or methods in a class and still construct a valid PlanningSolution.
Actual behavior
Error is thrown
To Reproduce
I was able to reproduce by modifying tests/test_solver_factory.py. The Solution class now reads:
@planning_solution
@dataclass
class Solution:
    entities: List[Entity]
    value_range: Annotated[List[int], ValueRangeProvider]
    score: Annotated[SimpleScore, PlanningScore] = field(default=None)

    def get_entities(self) -> Annotated[List[Entity], PlanningEntityCollectionProperty]:
        return self.entities

    def __str__(self) -> str:
        return str(self.entities)

Note the approach here of Annotating the method similar to the original Annotated attribute.
I received:
java.lang.java.lang.IllegalStateException: java.lang.IllegalStateException: The getterMethod (public ai.timefold.jpyinterpreter.types.collections.PythonLikeList org.jpyinterpreter.user.__main__.Solution.$method$get_entities()) with a PlanningEntityCollectionProperty annotation has a methodName ($method$get_entities) that does not start with a valid prefix ([get, is]).

Environment
Timefold Solver Version or Git ref:
1.11.0b0
Python version used:
3.12.4
Output of uname -a or ver:
Darwin MacBook-Pro-4.local 23.5.0 Darwin Kernel Version 23.5.0: Wed May  1 20:14:38 PDT 2024; root:xnu-10063.121.3~5/RELEASE_ARM64_T6020 arm64
Additional information
Occurred during migration from OptaPy.

timefold has a significantly different API than optapy. In particular, instead of trying to mimic Java API, it provides a pythonic interface. Currently, using annotations that require setters (namely, PlanningEntityCollectionProperty, PlanningEntityProperty, and PlanningVariable) with Annotated is not supported. Read-only annotations on methods and Attributes are supported though:
@planning_solution
@dataclass
class Solution:
    entities: Annotated[List[Entity], PlanningEntityCollectionProperty]
    score: Annotated[SimpleScore, PlanningScore] = field(default=None)

    def value_range(self) -> Annotated[List[int], ValueRangeProvider]:
        return [i for i in range(10)]

    def __str__(self) -> str:
        return str(self.entities)

Created https://github.com/TimefoldAI/timefold-solver-python/issues/87 for tracking.


Extension doesn't seem to be working/enabled after install

I've tried to install the extension and use it on one of my Markdown files inside my Hugo site directory. However, typing 'h-' doesn't bring up any prompts and typing some of the examples does nothing either. Is there something I must do/enable before using the extension?
VS Code: Version: 1.38.1
Extensions Enabled: [Markdown All in One, Markdown Preview Enhanced, Markdown Lint, Hugo Helper, Hugo Snippets, Hugo Language and Syntax Support]

You can trigger suggest the snippets with ^Space https://code.visualstudio.com/docs/languages/markdown#_snippets-for-markdown

I have added this to the readme


react native of expo using mobx can't re-render when change observable variable

When I change the observed variable, the observable value  in the method with /@action  changes but doesn't re-render.

My package.json file configuration

Please help me check whether it is due to the mobx-react version or the incompatibility expo.

Please provide a reproducible reproduction when filing a bug report.

oh I found my fault.I Set the staticrendering to true which caused the error


New Features

Expanded on -c option to allow for a comma delimited list of checks to run

You can now have either a single check or a list of checks

Added a NEW option (-x pattern1,pattern2) to exclude containers from the check if the match any of the patterns specified. This can also be a single pattern or a comma delimited list.

Specifically this feature will be useful when we need to exclude certain containers where compliance measures haven't or can't be taken and therefore eliminating them from the test results.

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:mpritter76/docker-bench-security.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Awesome and thanks @mpritter76!
Could you run shellcheck on docker-bench-security.sh and fix the minor stuff like double quotes?
The only thing I noticed was that -x didn't catch containers in check_5_29.

diff --git a/docker-bench-security.sh b/docker-bench-security.sh
index 6a0a4ba..588fb60 100755
--- a/docker-bench-security.sh
+++ b/docker-bench-security.sh
@@ -92,7 +92,7 @@ main () {
     containers=$(docker ps | sed '1d' | awk '{print $NF}')
   else
     pattern=$(echo "$exclude" | sed 's/,/|/g')
-    containers=$(docker ps | sed '1d' | grep -Ev '$pattern' | awk '{print $NF}')
+    containers=$(docker ps | sed '1d' | grep -Ev "$pattern" | awk '{print $NF}')
   fi
   # If there is a container with label docker_bench_security, memorize it:
   benchcont="nil"
diff --git a/tests/5_container_runtime.sh b/tests/5_container_runtime.sh
index 1eec675..4a8023f 100644
--- a/tests/5_container_runtime.sh
+++ b/tests/5_container_runtime.sh
@@ -1001,10 +1001,12 @@ check_5_29() {
             cName=$(docker inspect --format '{{.Name}}' "$c" 2>/dev/null | sed 's/\///g')
           else
             pattern=$(echo "$exclude" | sed 's/,/|/g')
-            cName=$(docker inspect --format '{{.Name}}' "$c" 2>/dev/null | sed 's/\///g' | grep -Ev '$pattern' )
+            cName=$(docker inspect --format '{{.Name}}' "$c" 2>/dev/null | sed 's/\///g' | grep -Ev "$pattern" )
+          fi
+         if ! [ -z "$cName" ]; then
+            info "     * Container in docker0 network: $cName"
+            logjson "5.29" "INFO: $c"
           fi
-          info "     * Container in docker0 network: $cName"
-          logjson "5.29" "INFO: $c"
         done
       fi
       currentScore=$((currentScore + 0))

Refer to https://github.com/docker/docker-bench-security/pull/292

Why close?


Multiple Charts with same .on('draw', function(data)... ) ?

I am assigning new Pie charts to different variables. I want all of the pie charts to have the same animation settings etc... Is there an easy way to do this with out doing repetitive code, chart.on('draw', function(data){...} and chart.on('created', function() {..] for every chart?

Like this?:
function drawHandler() {}

chart1.on('draw', drawHandler);
chart2.on('draw', drawHandler);

Or if you want, like this:
function addDrawHandler(chart) {
  chart.on('draw', function() {});
}

addDrawHandler(chart1);
addDrawHandler(chart1);


New ghc-parser release?

Currently ghc-parser-<IP_ADDRESS> on Hackage has an upper bound of ghc < 9.1. This prevents IHaskell from being built on the latest nixpkgs-unstable branch against GHC 9.2.
It looks like the current in-tree ghc-parser has been updated to support 9.2. Could you tag a new ghc-parser release and put it on Hackage? Thanks!

I've uploaded new versions of ghc-parser, ipython-kernel, and ihaskell.

Awesome, thanks!


Automated deployment

Trigger a build of the deployment repository in CircleCI with build parameters that specify what to deploy.
See also the changes to the deployment repo here:
https://github.com/ad2games/deployment/compare/ecf956...e0094b
For now, we only trigger a deployment build for the staging branch.

The possibility of two deployments happening simultaneously was brought up during the discussion.
One easy check we could implement would be to only deploy if the current build is the most recent one.  We can get recent builds via the API:
https://circleci.com/docs/api#recent-builds-project


Define expected datatypes for datasets

I've noticed most the columns in the interconnection data, are object types which causes problems when I try to write the data to parquet files because there are mixed types. To resolve this I do a lot of type conversions.
It would be nice to have gridstatus' methods return dataframes with columns that do not have mixed datatypes and use data types that support pd.NA.
I'm only familiar with the interconnection queue data so I'm not sure if this issue exists in the other gridstatus datasets. Something like Pandera or pydantic could be used to organize and correct dtypes.

I don't think many other datasets have mixed datatypes, at least I haven't noticed it be a problem.
That being said, I would very much welcome a contribution doing it for at least one of the interconnection datasets to give me a better sense if it would be more broadly helpful.
I'm wondering it could replace/improve a good chunk of our testing code.


HasSuffix not working during the key scans

Hi,
I've stumbled upon this strange behaviour while using the library. There's the example in the documentation, that shows how to iterate all the keys in the db and find ones with a certain prefix.
db.View(func(tx *bolt.Tx) error {
    c := tx.Bucket([]byte("MyBucket")).Cursor()

    prefix := []byte("1234")
    for k, v := c.Seek(prefix); bytes.HasPrefix(k, prefix); k, v = c.Next() {
        fmt.Printf("key=%s, value=%s\n", k, v)
    }

    return nil
})

I've tried to search by suffix based on this example. And it just won't work.
db.View(func(tx *bolt.Tx) error {
    c := tx.Bucket([]byte("MyBucket")).Cursor()

    suffix := []byte("1234")
    for k, v := c.Seek(suffix); bytes.HasSuffix(k, suffix); k, v = c.Next() {
        fmt.Printf("key=%s, value=%s\n", k, v)
    }

    return nil
})

So i've tried to debug this and did a search by all the keys, and checked all the results. And there were indeed keys that had this suffix. I wonder if anyone can reproduce this, and what may cause this.
db.View(func(tx *bolt.Tx) error {
    c := tx.Bucket([]byte("MyBucket")).Cursor()

    suffix := []byte("1234")
    for k, v := c.Seek(suffix); k != nil; k, v = c.Next() {
        fmt.Printf("Found : %x | Searched For Suffix : %x | Is it a suffix? : %t", k, p, bytes.HasSuffix(k, suffix))
    }

    return nil
})

Thanks

Keys are sorted, and Seek() goes to the first key matching the prefix. The loop exits on the first result not matching the suffix. Unless all your keys with that suffix are contiguous in the sorted list, and having the same suffix as a prefix, you can't expect that for() loop to work.
Finding all the matching suffixes would require a slower full scan (or changing your keys, or using another bucket as an index, etc.):
for k, v := c.First(); k != nil; k, v = c.Next() {
    if bytes.HasSuffix(k, suffix) {
        fmt.Printf("key=%s, value=%s\n", k, v)
    }
}

@dtfinch is correct, if you use stringified numbers then they'll be sorted in the bucket like this:
1
10
11
12
2
3
4
5
6
7
8
9

If you want to store a numeric value as a key and filter it then I'd recommend using binary.BigEndian.PutUint64():
myNumericKey := 123456
key := make([]byte, 8)
binary.BigEndian.PutUint64(key, myValue)

tx.Bucket([]byte("MyBucket")).Put(key, []byte("some value"))

Then the number will always be in order in your cursor. Also, it's important to note that little endian doesn't work as the byte order is inverted and Bolt sorts on simple byte order.

Cool, thank you.
I'm not yet using numeric values. Will keep this in mind. For now will do as suggested and go through all the keys each time and find the ones I need. Will try to think of something better if there will be performance issues.


BottomSheetBehavior

I create a bottomsheetbehavior and add inside a recyclerview contains theme list.
But i click item recyclerview to change theme ThemeManager animated theme on over bottomsheet and bottomsheet has Gone.

Please help


new packages doesn't get indexed

New packages doesn't get indexed, example:
I tried to use it on this package : https://www.npmjs.com/package/reponse
I used : https://api.npms.io/v2/package/reponse
but i get this error {"code":"NOT_FOUND","message":"Module not found"}
@satazor

agree

+1. getting {"code":"INTERNAL","message":"Internal server error"} for any package I tried

+1, I'm getting {} for my new package even after 2 weeks.

+1 same issue 🥲

+1, Trying to get https://api.npms.io/v2/package/react-international-phone (since 2 months from package publish)

Same issue.
Response is {"code":"NOT_FOUND","message":"Module not found"} on request to https://api.npms.io/v2/package/itertools-ts
Package is active: https://www.npmjs.com/package/itertools-ts

Same issue here.
getting error: {"code":"NOT_FOUND","message":"Module not found"} on request https://api.npms.io/v2/package/react-native-international-phone-number
Package on NPM: https://www.npmjs.com/package/react-native-international-phone-number

Same, what's going on?

I'm having the same issue.
My Package1: https://api.npms.io/v2/package/fe-theme (Working)
My Package2: https://api.npms.io/v2/package/fe-pilot (Not working)

Hi,
I am experiencing the same issue with my package not being found via the npms.io API. The package is published on npm and has been actively maintained with regular updates every two weeks. The package name is packetjs-di, and here is the link to it on npm:
https://www.npmjs.com/package/packetjs-di
When I query the API using https://api.npms.io/v2/package/packetjs-di, I receive the following error response:
{
  "code": "NOT_FOUND",
  "message": "Module not found"
}

I have tried:

Republishing the package multiple times.
Checking the package.json for any issues (it is correctly formatted).
Testing from different networks to rule out caching issues.

It seems to be an indexing problem on npms.io's side, as the package has been live on npm for months, and everything works fine there.
I would appreciate it if this issue could be looked into. Please let me know if further information is needed to assist in resolving this.
Thank you for your help!
Best regards,


Is it possible for Gene_maps values to use the same scale instead of min-max per gene?

If I understand correctly; the gene maps are minimum and maximum expression for each gene.
However, genes that are hardly expressed in your biological sample will look like they have a strong expression pattern which is overblown (imho).
Would it be possible to have an output where the gene_maps of all genes have the same scale? In this way, genes that are hardly expressed would also show dramatic expression. For example in my tissue, CD1E expression is really low, but it appears enormous in XFuse (with a distribution that doesnt make much sense, its more noise that gets amplified...)

Maybe I did sth wrong and this should not be? I expect in this case that the entire tissue would be blackish colour...

Yes, that's correct. There is currently no option to give the gene maps a common scale. The reason for this is that they are produced one gene at a time, so we don't know the max until all genes have been processed. What you could do possibly is to save the underlying data that the maps are based on by setting writer = "tensor" in the gene_maps section of the config toml file. Then you could create the figures manually. Keep in mind, however, that the tensor file gets very large, so you probably don't want to do this for all genes in the dataset and instead limit the gene set using the gene_regex option.
As an aside, I agree that this gene map looks quite weird. Usually, microstructure (e.g. nuclei) are typically visually distinct. What is the resolution of the H&E after scaling? Usually, I aim for something like 1500 - 2000 px in height/width.

Is that after masking? I mean, when cutting it out, it is around 12.000x12000 pixels, scale factor 0.15, so 1800 pixels after scaling
With the whole image (including Visium dotted frame its about 20.000x20.000 pixels.
I dont know what the measures are based upon. Maybe I just need to scale less
OR: this data is trained on 5 samples and that is not enough. However, in another issue you said that for superresolution, you dont need many samples. So I don't know. Maybe it's just my tissue (liver) where hepatocytes are less clearly demarcated?

I tried what you said, and the genes I selected are now in the output folder as a .pt format.

How do I turn these .pt files into tiff gene expression maps?

Are these gene maps then trained on the entire gene expression data, or only on the genes that I put in the gene_regex?

Can you just elaborate a bit on what writer ="tensor" does (and maybe where in the code it is located - would like to understand)

my toml file was something like this;
[xfuse]
network_depth = 6
network_width = 16
min_counts = 50

[expansion_strategy]
type = "DropAndSplit"
[expansion_strategy.DropAndSplit]
max_metagenes = 50

[optimization]
batch_size = 3
epochs = 100000
learning_rate = 0.0003
patch_size = 768

[analyses]
[analyses.metagenes]
type = "metagenes"
[analyses.metagenes.options]
method = "pca"

[analyses.gene_maps]
type = "gene_maps"
[analyses.gene_maps.options]
gene_regex = "^(GENE1|GENE2|GENE3| (and so on until GENE 34))$"
writer = "tensor"

[slides]
[slides.section1]
data = "sections/sample1/data.h5"
[slides.section1.covariates]
section = 1

[slides.section2]
data = "sections/sample2/data.h5"
[slides.section2.covariates]
section = 2

[slides.section3]
data = "sections/sample3/data.h5"
[slides.section3.covariates]
section = 3

[slides.section4]
data = "sections/sample4/data.h5"
[slides.section4.covariates]
section = 4

Awesome!

How do I turn these .pt files into tiff gene expression maps?

You will have to do this manually. To give you an idea, something like the below script should work (it's untested so it will likely need some slight modifications to work). You need to run it in the same directory as the *.pt files. It first computes the max expression over all maps and then normalizes the gene maps based on that. Note that it could be that the max will be too high and all maps will then be quite dark. In that case, it could make more sense to clip the maps on the 99th percentile or similar instead of the max.
import os
from glob import glob

import matplotlib.cm as cm
import tifffile

def load_gene_map(path):
    gene_map = torch.load(path, map_location="cpu")
    gene_map = gene_map.mean(0)
    return gene_map

max_expression_all = 0
gene_map_files = glob("*.pt")
for path in gene_map_files:
    gene_map = load_gene_map(path)
    max_expression = gene_map.max()
    if max_expression > max_expression_all:
        max_expression_all = max_expression

for path in gene_map_files:
    gene_name,_ = os.path.splitext(os.path.basename(path))
    gene_map = load_gene_map(path)
    gene_map = gene_map / max_expression_all
    gene_map = gene_map.numpy()
    gene_map = cm.inferno(gene_map)
    tifffile.imsave(f"{gene_name}.tif", gene_map)

Are these gene maps then trained on the entire gene expression data, or only on the genes that I put in the gene_regex?

Good question! As long as the model is trained on the entire gene expression data, the latent state and therefore maps will also be informed by other genes.

Can you just elaborate a bit on what writer ="tensor" does (and maybe where in the code it is located - would like to understand)

It sets the gene map writer to the "tensor" writer, which saves the sampled gene maps as pytorch tensors (as opposed to image files saved by the "image" writer). The tensor writer is defined here:
https://github.com/ludvb/xfuse/blob/c420abb013c02f44120205ac184c393c14dcd14d/xfuse/analyze/gene_maps.py#L157-L160


Shorten integration tests' names

Our GitHub Workflow failed in #1653 because the automatically generated
name for the object file was too long:
Error: could not write output to /home/runner/work/newsboat/newsboat/target/debug/deps/configpaths_create_dirs_returns_false_if_xdg_config_dir_exists_but_data_dir_doesnt_and_couldnt_be_created-9170954e0544fc50.configpaths_create_dirs_returns_false_if_xdg_config_dir_exists_but_data_dir_doesnt_and_couldnt_be_created.2a36eb5b13484269-cgu.0.rcgu.o: File name too long

This is caused by us using pretty long names for integration tests.
These descriptive names come from the C++ tests, and I'd prefer to keep
them, since they aid understanding. So I made a compromise: the names
are just numbers now, but the test function itself still has a long
descriptive name. That way, cargo test prints out long, useful names,
which one can git grep for if one needs to look at the code.
An alternative approach would be to put all of module's tests into
a single file (e.g. configpaths.rs). But that will require us to be
careful about each tests' environment: chdir-ing into the initial
directory once the test finishes, clearing any environment variables,
resetting the locale and so on. In addition to that, we would also have
to introduce some global lock, to force Rust to run the test serially.
That's pretty inconvenient, so I opted for an easier, but hackier
solution.

Coverage remained the same at 59.031% when pulling f9352c8e5ae35c5077e450394428e85d450e387c on bugfix/integration-tests-filename-too-long into b739c58d536f29419b1bbaf5560ee37ab89914bc on master.


Gardening: Migrate test suite to GH issues p. 15

Sorry I am behind with this, I will try to review at least 3 PRs tomorrow.

Thanks, Pavel! Please don't feel obliged to keep up with the pace at which I'm spawning these PRs though.

@swift-ci please test


Test if chart needs a version bump on pushing to branch

Chart (with changes) will be released on being merged to master branch. Releasing fails if the chart version has not been increased.
See https://github.com/grafana/helm-charts/blob/main/.github/workflows/lint-test.yaml

to be solved by https://github.com/PharmaLedger-IMI/helmchart-ethadapter/pull/37

to be solved by https://github.com/PharmaLedger-IMI/helmchart-ethadapter/pull/37


Convert E2E tests to Typescript

Part of PECO-1390

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

Project coverage is 92.91%. Comparing base (c239fca) to head (f85172f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #257   +/-   ##
=======================================
  Coverage   92.91%   92.91%
=======================================
  Files          64       64
  Lines        1553     1553
  Branches      275      275
=======================================
  Hits         1443     1443
  Misses         46       46
  Partials       64       64

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


Notary Fix Origin State

Description
Make the notary use states that originate from the Notary's domain.

Adding executor test fix via backend bump. This fixes flake of TestSendManagerMessage


Should hangs Karma (when using RequireJS?)

For a long time I thought this was a defect with Karma, which was newer to me than Should.js, but the issue finally was resolved when I removed the latter from my project.
If needed I can revert to older revision to troubleshoot, but I don't have much time for this at moment. The observable behavior was that I would kick off a karma process with file watching, and tests would run fine first time, but when I'd touch files, most of the time but not all the time, the tests would not re-run. This seemed to be an issue with karma and OS file-watching, but turned out was immediately resolved by removing Should.js. When this issue occurred, even refreshing the test run browser would refuse to execute, would hang with no output anywhere.
Sounds like a race condition (not 100% of the time, but maybe 90% of the time tests would fail like this), and though I never confirmed this, I highly suspect is specific to using RequireJS as well. Maybe Should.js is being re-initialized in some way, and doesn't play well as it modifies Object.prototype. I then read something that (I guess wrongly?) led me to believe Should.js is suitable for Node code but not expected to work well in browser.
As I recall, I had tried including 'should' as a framework in karma.conf, and tried giving it a stub: exports: clause. That doesn't seem to be supported. I tried explicitly importing into requirejs test spec modules, which seems to trigger the problem, and then not explicitly, resulting in more obvious race condition where on some runs the tests would fail as should.js had not initialized when they are loaded.
Hope that helps, I may have more time later to troubleshoot or provide a working example, but my code is not open source so I'd have to spend some time on this that I don't have at the moment.

I create examples repository. One of them how requirejs and should work together.
I need to say that all problems i had in configuration was from requirejs and karma adapter.
So i close this repo, as there is no any bug.


UI 作る

[ ] Request form
[ ] Fix title
[ ] Ask in Slack
[ ] JIRA issued
[ ] Prepare with the organizer
[ ] Setting the goal
[ ] Slides
[ ] Movies preparation
[ ] The day!
[ ] PDF release in SlideShare
[ ] Publish the movie
[ ] Subtitle to the movide
[ ] Publish the blog

Details of request

https://magiceden.io/marketplace/genopets_genotype_crystals


cannot flash automatically (IDFGH-822)

Hi,
I use DevkitC v4.
I cannot flash automatically.
I must set "menuconfig -> Serial flasherconfig -> Before flashing -> No reset" and control EN & Boot  buttons by manual.
How can I flash automatically?

$ python /home/foo/esp/esp-idf/components/esptool_py/esptool/esptool.py --chip esp32 --port COM21 --baud 115200 --before default_reset --after hard_reset write_flash -z --flash_mode dio --flash_freq 40m --flash_size detect 0x1000 /home/foo/esp/blink/build/bootloader/bootloader.bin 0x10000 /home/foo/esp/blink/build/blink.bin 0x8000 /home/foo/esp/blink/build/partitions_singleapp.bin
esptool.py v2.6-beta1
Serial port COM21
Connecting......................................_____
A fatal error occurred: Failed to connect to ESP32: Timed out waiting for packet header

There should either be a button labeled either 0 or boot. Holding it while the "Connecting........." dialog is happening should start the flashing process. I've had issues with the auto-flash abilities of some dev boards, so you just have to manually hold the button while you are flashing to start it. The SparkFun ESP32 Thing is one that I can consistently get to auto-flash, but it's kind of expensive compared to other ESP32 dev boards. I believe it has to do with the reset/boot circuit on the board.

@tatsutaigu Hi, does your issue still exist? Thanks.

Thank you very much for your feedback. Please try to remove the capacitance C15, which is connected in parallel on IO0. Thank you!

Hi,
C15 was already removed when i bought.
did you update flash sequence?

Hi,
I tried latest setup tool with idf.py -p COM20 flash.
https://docs.espressif.com/projects/esp-idf/en/latest/get-started/windows-setup.html#esp-idf-tools-installer
However, it cannot flash.
I guess DevKitC v4 has hardware bug.
C9 must be 1uF.
I replace C9 0.1uF to 1uF, it could flash automatically.
And, esptool.py may have bug.
idf.py menuconfig -> serial flasher config -> Before flashing (No reset)
idf.py -p COM20 flash, then it flash DevkitC v4 with C9 1uF without pressing control SW1 and SW2.

Also seeing this on a DevKitC V4 purchased from Digikey this month. C15 is not loaded which I believe is causing the issue.
Here's a zoomed out capture of the boot sequence.

with more detail on the rising edge of EN. Without C15, IO0 rises before EN and therefore program mode is not entered.

Hi @mill1000,
Thanks for providing the traces. C15 was removed from the DevKitC in 2018 in order to keep the output signal from IO0 clean, in particular this pin can be used a clock output for an ethernet MAC.
There is a limitation with the auto reset circuit because there is a transition period where RTS & DTR are both asserted, and the "auto reset" logic will drive both EN & IO0 high, before RTS is de-asserted and IO0 goes low. The first event corresponds with "AX" (roughly) and the second event with "BX" (roughly). If the chip starts up during this period as EN passes the high threshold, it straps into the wrong boot mode.
Are you using Windows? The traces seem to show 15ms between these two transitions, normally even on Windows this period is normally <1ms (meaning that IO0's second falling edge comes before EN has finished rising.) This event is determined by the OS USB stack and driver timing.

Hi @mill1000,
Thanks for providing the traces. C15 was removed from the DevKitC in 2018 in order to keep the output signal from IO0 clean, in particular this pin can be used a clock output for an ethernet MAC.
There is a limitation with the auto reset circuit because there is a transition period where RTS & DTR are both asserted, and the "auto reset" logic will drive both EN & IO0 high, before RTS is de-asserted and IO0 goes low. The first event corresponds with "AX" (roughly) and the second event with "BX" (roughly). If the chip starts up during this period as EN passes the high threshold, it straps into the wrong boot mode.
Are you using Windows? The traces seem to show 15ms between these two transitions, normally even on Windows this period is normally <1ms (meaning that IO0's second falling edge comes before EN has finished rising.) This event is determined by the OS USB stack and driver timing.

Hi @projectgus
Thanks for that information. I am using WSL to flash these devkits and I am beginning to suspect this may be part of the problem. I have occasionally encountered failures with esptool as well due to serial timeouts. So I was beginning to think whatever "mapping" WSL has to use to access the USB devices could be part of the problem.

Hi @projectgus
Thanks for that information. I am using WSL to flash these devkits and I am beginning to suspect this may be part of the problem. I have occasionally encountered failures with esptool as well due to serial timeouts. So I was beginning to think whatever "mapping" WSL has to use to access the USB devices could be part of the problem.

Hi @mill1000, Thansk for getting back to me. That seems likely that maybe WSL is adding latency between the serial port operations. I'm glad you found the workaround by adding C15 back, although note that you may find the board now occasionally boots into Download Mode when you don't want it to. I am reminded that this is the other reason C15 is no longer populated.

Hi @mill1000, Thansk for getting back to me. That seems likely that maybe WSL is adding latency between the serial port operations. I'm glad you found the workaround by adding C15 back, although note that you may find the board now occasionally boots into Download Mode when you don't want it to. I am reminded that this is the other reason C15 is no longer populated.

Ha well I've actually opt'd to forgo my own workaround and instead just rely on pressing the flash button.

Ha well I've actually opt'd to forgo my own workaround and instead just rely on pressing the flash button.

Something possibly related; I have been flashing an ESP32-WROVER-based device successfully with an older HP i7-4770 system, but a new Lenovo Ryzen 7 PRO 4750G system was not successfully flashing it. Both with Win 10 Pro, both with same SiLabs driver. After a lot of fiddling-around, I realized that the ports on the Lenovo are USB3.x and the HP ports are USB2. I have a USB2 external hub. Plugged the ESP32 into the external USB2 hub on the Lenovo and am again successfully flashing the ESP32.

Something possibly related; I have been flashing an ESP32-WROVER-based device successfully with an older HP i7-4770 system, but a new Lenovo Ryzen 7 PRO 4750G system was not successfully flashing it. Both with Win 10 Pro, both with same SiLabs driver. After a lot of fiddling-around, I realized that the ports on the Lenovo are USB3.x and the HP ports are USB2. I have a USB2 external hub. Plugged the ESP32 into the external USB2 hub on the Lenovo and am again successfully flashing the ESP32.

Hi @danak6jq,
Thanks for noting this! We had a similar report regarding esptool on some Ryzen chipsets, it seems like the USB host driver is different. I'm not sure if this is the same root cause, but if it is then it's good to hear you found an effective workaround.

Hi @danak6jq,
Thanks for noting this! We had a similar report regarding esptool on some Ryzen chipsets, it seems like the USB host driver is different. I'm not sure if this is the same root cause, but if it is then it's good to hear you found an effective workaround.

Closing this as there's nothing we can generally do to fix these issues in ESP-IDF.
If you think you've identified a flashing issue that can befixed in esptool then please report it there, however note that a lot of flashing issues are hardware issues (either with the development board reset circuit, or with USB host behaviour) that can't realistically be fixed in ESP-IDF.


Appveyor may not have been compiling beta.

Please help keep the CHANGELOG up to date by providing a one sentence summary of your change:
N/A - fix a Windows build issue.
I'm pretty sure after the move to the Makefile we missed overriding what version of Rust to use, so Appveyor has always been compiling with stable, even on the beta channel.

This makes me more confident. 😄 I'd appreciate someone else looking at it to verify.

LGTM :)


UI changes for small box in user home page

Color changes for small box

Thank you for making the changes.


name conflicts with /usr/bin/sq -- some alternatives

Has anyone else run into the naming conflict with the /usr/bin/sq that's shipped with ispell?  (For lurkers -- 'ispell' has been around since the early 70's and tends to be installed today as a backend for many text editors, including both vim and emacs, and is available for all major operating systems, including UNIX, Linux, MacOS, and even DOS/Windows.)
In my own case, I wound up deploying this package as 'seeq' to avoid the conflict.  A slightly shorter name that I don't like as much is 'siq'.  Neither of those come up in the output of apt-file search bin/siq, apt-file search bin/seeq, or googling for man siq or man seeq.

@stevegt Apologies for having neglected to respond to this issue. I hear what you're saying re the sq name conflicting... and I honestly didn't know about the ispell binary from back in the day. But at this stage there's too much stuff out there already depending on the sq name.

@neilotoole Thanks for answering.  I'm not sure that's the right conclusion.  What stuff depends on your version of sq?  Is it primarily things you own or control personally, such as the sq.io domain name?
There are millions of installs of ispell out there.  It's reasonable to expect that either you're going to break a lot of machines or your package will not enjoy much adoption.  The latter would be sad, because it really is a good package.
As of this writing, you've got 419 stars and 9 forks -- it will be easier to fix this before those numbers tick higher.

...thinking.  If you do decide to stick with sq, it might be best to include a mention in the README that it conflicts with an internal command in the ispell package, and that installing as e.g. seeq will avoid conflicts.  Because sq is an internal command, not usually called directly by users, the breakage users will see will be hard to track down otherwise.

@stevegt That's a very good idea.

the breakage users will see will be hard to track down otherwise.

Do you recall what error/state you experienced? E.g. "if when using ispell, you see X error, this is probably because of a conflict with the ispell's internal sq command.". What's the "X error" that you saw, and I'll include it in the readme.
Thank you for your thoughtfulness and insight about all this. It's much appreciated.


How to convert to tf-serving ?

How do I convert your model to tensorflow serving format?

I've not done it myself, but you could easily find something on the internet. Check this:
https://github.com/bendangnuksung/mrcnn_serving_ready


perhaps include the system auth log and messages log on the server detail?

maybe in the future, yes


[NEW CRYPTO] SLP, AXS, DPET

Add some common NFT Games. SLP, AXS, DPET, SKILL, PVU, etc.

@rgvillanueva28 These all already exist. If you join the support discord there is a bot that will help you get the tickers you need. Click on the discord logo in the readme for an invite.


FIX: Desktop touchscreens and allow edits on mobileView

This PR:

fixes an issue where the ✏ Edit Table button is not visible on desktops with touch screens.
allows editing tables on mobile

Thanks @davidtaylorhq ! :tada:


Pretrained weights

Hi
Thanks for sharing the code. Is it possible to obtain the pre-trained weights for the model on NYUD-v2 for academic purposes?
Thanks.

Hi,
Thanks for your attention. Sorry, we couldn't supply the pre-trained models in the paper because I didn't bring them out when I left the company where I was an intern. However, you could easily re-produce them with this repo and only 4 GPUs (>= 11GB each) are needed.

Hi.
Thanks for the quick reply. We will train the model ourselves then.
No worries.

Hi.
Thanks for the quick reply. We will train the model ourselves then.
No worries.

Hi, have you trained the model by yourself from scratch which achieves comparable performances?


[Site Request] Bilibili Playlist

Checklist

[X] I'm reporting a site feature request
[X] I've verified that I'm running yt-dlp version 2021.02.24
[X] I've searched the bugtracker for similar site feature requests including closed ones

Description

I see you've implemented channel downloads, something not even annie can do. However i'd like to request for a playlist download like annie. for example this link https://www.bilibili.com/video/BV1bK411W797 has multiple videos, suffixed with https://www.bilibili.com/video/BV1bK411W797?p=1
Possible for a switch to download all of them at once?

something not even annie can do

annie?

I should be able to start working on this next weekend if no one else wants to.

something not even annie can do

annie?
another video downloader, https://github.com/iawia002/annie

I am not very familiar with bilibili. Can you explain to me how this multiple videos thing works?
Does https://www.bilibili.com/video/BV1bK411W797 point to both the video and the playlist? Is there any separate URL for one or the other?

Possible for a switch to download all of them at once?

Adding a switch for this seems inelegant since no other extractor would make use of it. Any other ideas how to implement this? Maybe --no-playlist and --yes-playlist can be used?

I should be able to start working on this next weekend if no one else wants to.

That would be awesome

I should be able to start working on this next weekend if no one else wants to.

something not even annie can do

annie?

another video downloader, https://github.com/iawia002/annie
Also: are you archiving from bilibili? If you don't mind, pm me on discord: Otoman#6721

I'm not quite archiving, just that some channels have content that i really like and downloading 1 by 1 is quite tedious. Are you running a project to archive bilibili?

I am not very familiar with bilibili. Can you explain to me how this multiple videos thing works?
Does https://www.bilibili.com/video/BV1bK411W797 point to both the video and the playlist? Is there any separate URL for one or the other?

Possible for a switch to download all of them at once?

Adding a switch for this seems inelegant since no other extractor would make use of it. Any other ideas how to implement this? Maybe --no-playlist and --yes-playlist can be used?

I should be able to start working on this next weekend if no one else wants to.

That would be awesome

The site doesn't seem to distinguish which videos are part of a playlist or not, at least not from the url. The base link https://www.bilibili.com/video/BV1bK411W797 points to the video and the playlist linked to it, there isn't a direct link to the playlist, except that adding ?p=1 ?p=2 ?p=3 etc to the end will link to the 1st, 2nd, 3rd videos of the playlist


[Bug] Failed to execute 'postMessage' on 'DOMWindow'

🐞 Describe the Bug
Failed to execute 'postMessage' on 'DOMWindow': The target origin provided does not match the recipient window's origin ('https://www.sandbox.paypal.com').
Same is working on localhost.
🔬 Minimal Reproduction

add product to cart
add to cart and go to cart page
Paypal button is not loading correctly. Errors logged in console.

😕 Actual Behavior
PayPal script not working on our website but works on localhost
🤔 Expected Behavior
It should load correctly
🌍 Environment

Software
Version(s)

react-paypal-js
4.1.1

Browser
chrome Version 89.0.4389.82

Operating System
Windows 7

Hi @bhavikagrawal it's great to hear you are using react-paypal-js!
I took a look at your dev site and noticed your web server is setting the http response header X-Frame-Options: SAMEORIGIN. I think this header will block the paypal button from working since we depend on being able to serve up code from www.paypal.com/www.sandbox.paypal.com using iframes.
Can you try removing this header to see if it fixes the problem?

@gregjopa Can you please delete the issue permanently from repository. Or remove the hyperlink to our dev site.

@gregjopa, a basic question here: Why paypal wants to iFrame client application? I think, it should be the other way. correct?

Hi @bhavikagrawal, I removed the hyperlink to your dev site.
@RishikeshDarandale we use iframes for a number of reasons. It's a guaranteed way to sandbox our 3rd party code. The PayPal JS SDK use Zoid for managing the iframe abstraction with components and this react library is just making it easier to use the JS SDK in react land. You can read more about the benefits of iframes here: https://bluepnume.medium.com/introducing-xcomponent-seamless-cross-domain-web-components-from-paypal-c0144f3e82bf

@gregjopa I have tried removing the header, but it did not worked.
When i disabled adobedtm script from running on website, it started working. What could be a possible conflict because of other external script.

Hi @bhavikagrawal, I'm not sure why Adobe DTM is conflicting with the PayPal JS SDK. Adobe DTM typically loads a bunch of 3rd party tags on websites so it could be one of those 3rd party scripts that's conflicting. I doubt this issue is related to this react library. Can you try creating a minimal reproduction that does not use React and instead uses the PayPal JS SDK and Adobe DTM directly? Please create an issue over in the paypal-checkout-components repo with the minimal reproduction and then the sdk team can take a look: https://github.com/paypal/paypal-checkout-components/issues


[bug] File drag and drop configuration seems broken in v2

Describe the bug
I can't find a way to disable file drop events on the window with tauri v2 beta. This is to handle file drop events within the web view.
When upgrading my app from v1 to v2 (beta.14) I get the following issue:
unknown field `fileDropEnabled`, expected one of `label`, `url`, `user-agent`, `userAgent`, `drag-drop-enabled`, `dragDropEnabled`, `center`, `x`, `y`, `width`, `height`, `min-width`, `minWidth`, `min-height`, `minHeight`, `max-width`, `maxWidth`, `max-height`, `maxHeight`, `resizable`, `maximizable`, `minimizable`, `closable`, `title`, `fullscreen`, `focus`, `transparent`, `maximized`, `visible`, `decorations`, `always-on-bottom`, `alwaysOnBottom`, `always-on-top`, `alwaysOnTop`, `visible-on-all-workspaces`, `visibleOnAllWorkspaces`, `content-protected`, `contentProtected`, `skip-taskbar`, `skipTaskbar`, `theme`, `title-bar-style`, `titleBarStyle`, `hidden-title`, `hiddenTitle`, `accept-first-mouse`, `acceptFirstMouse`, `tabbing-identifier`, `tabbingIdentifier`, `additional-browser-args`, `additionalBrowserArgs`, `shadow`, `window-effects`, `windowEffects`, `incognito`, `parent`, `proxyUrl`

After some searching, I found the hint in discord (nowhere in the changelogs), that it had been renamed to dragDropEnabled, but this results in:
Error `tauri.conf.json` error on `app > windows > 0`: Additional properties are not allowed ('dragDropEnabled' was unexpected)

Reproduction
Removing the fileDropEnabled (and respective dragDropEnabled) makes the app compile again, but file drop events do not reach the webview.
You can test it with https://github.com/noxan/katalog (not a minimal example, but where I encounter the issue)
Expected behavior
No response
Full tauri info output
Error `tauri.conf.json` error on `app > windows > 0`: Additional properties are not allowed ('dragDropEnabled' was unexpected)
ELIFECYCLE  Command failed with exit code 1.```

Stack trace
No response
Additional context
No response

Try updating @tauri-apps/cli to beta.12 (it's on beta.4 in your project)


Is there repos for China? I can not visit the default repo if it's without vnp.

Or can I create one giving service to ?

You are free to create a mirror for china. I don't really know what works and what doesn't for china.
That said, most of this effort has been moved as part of conda-forge itself, and now, many packages, including numpy, scipy, and others are built for aarch64 as part of conda-forge.

You are free to create a mirror for china. I don't really know what works and what doesn't for china.
That said, most of this effort has been moved as part of conda-forge itself, and now, many packages, including numpy, scipy, and others are built for aarch64 as part of conda-forge.

thanks


Rust API on Windows

I've managed to get rust working on Windows, with the exception that compiling against the debug version doesn't work very well, so I've set it to link against the release version in both debug and release mode on Windows (it only works if you override the CFLAGS/CXXFLAGS environment variables, but it's not sufficient to do that in build.rs since it has to also affect one of the dependencies).

I ran into the same issue with arrow and openssl that I did with the Linux tests. It seems that the arrow_bundled_dependencies library will use openssl if it finds it on your system, regardless of whether or not ARROW_USE_OPENSSL is enabled (I'm not sure why; as far as I can tell parquet is compiling the nossl version of their encryption_internal code).
I've set it up to link against openssl when running the tests on windows, but it does have more difficulty finding it than on Linux. Adding the openssl-sys crate as a dependency on Windows helps with finding the library, but you either need vcpkg to detect it (something the openssl-sys error message mentions), or to manually set OPENSSL_DIR (which I've included in the github workflow). I might have to look into how using vcpkg works, but if we're relying on OPENSSL_DIR it might make more sense to just add ${OPENSSL_DIR}/lib to the linker search paths ourselves since the openssl rust bindings from openssl-sys aren't needed.


Dart embedder API returns isolate pointer, which can't be validated later

The Dart embedder API (dart_api.h) defines a Dart_Isolate handle, which is used by several public-facing functions. Dart_Isolate is implemented as a cast of the memory address of the Isolate instance. Since the VM does not maintain an isolate list, it is possible to pass a handle to a deleted isolate as an unverifiable parameter to those functions.

Stale.


Report about tests which never failed

User Story
As a Stryker user I would like to get a report about tests which never failed so that I can identify the useless test.
Details
Let's assume this is my test code:
var assert = require('assert');
var oMathHelper = require("../src/MathHelper");

describe('MathHelper', function() {
    describe('#sumUp()', function() {
        it('adds up two possitive numbers', function() {
            assert.equal(oMathHelper.sumUp(2, 2), 4);
        });

        it('adds up two float numbers', function() {
            assert.equal(oMathHelper.sumUp(0.5, 0.5), 1);
        });

        it('adds up two negative numbers', function() {
            oMathHelper.sumUp(-10, -10); // note: no assertion
        });
    });
});

I would like to have a hint by the framework, that the last test ('adds up two negative numbers') never failed (as the developer forgot to put an assert).
As far as I understand Stryker, the test run is always stopped as soon as the mutant has been killed. However, for the purposed feature, the test run should continue until the end and it should count as often every test failed. This should be a separate Stryker option or mode because it will be much slower then the original stop-when-killed mode.

Thanks for the detailed issue! This was something we were thinking about as well :)


Webdrivers fails to download the latest Gecko WebDriver.

Summary
It seems the Mozilla team deviated from its default naming of releases for the Gecko WebDriver. This causes the automatic download of the WebDriver to fail. I've also opened up an issue at their repository because I'm not sure who's "mistake/problem" this is.
See https://github.com/mozilla/geckodriver/issues/1866
Debug info
Webdrivers::NetworkError: Net::HTTPServerException: 404 "Not Found" with https://github.com/mozilla/geckodriver/releases/download/v0.29.1/geckodriver-v0.29.1-macos.tar.gz
/Users/stefan/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/webdrivers-4.6.0/lib/webdrivers/network.rb:19:in `get'
/Users/stefan/.rbenv/versions/2.7.2/lib/ruby/gems/2.7.0/gems/webdrivers-4.6.0/lib/webdrivers/system.rb:74:in `block (2 levels) in download_file'
/Users/stefan/.rbenv/versions/2.7.2/lib/ruby/2.7.0/tempfile.rb:291:in `open'

Webdrivers version: v4.6.0
Ruby version: v2.7.2
Operating system / CI Environment: macOS

Expected Behavior
To download the latest version of the Gecko driver.
Actual Behavior
It fails to download the latest version of the Gecko driver.

Thank you for reporting this both here and with geckodriver. Looks like they were able to quickly fix things as a result.


Save user sent metadata

Fixes #163.
I needed this functionality more or less, so I've given a crack at it.
Screenshot below is from Symfony dump for file upload completion event, which previously did not contain any user sent metadata.

There are no tests yet added, wanted to have your opinion on this approach first.

There are a couple tests that keep failing, so I'll have to take a look at them.

@alucic I've taken a look at failing tests and most were actually issues with mocks or expected cache changes (additional fields added). Overall I've got a feeling that some tests are redundant and check internal behavior, but that's outside the scope of this PR.
I've taken the liberty to:

Switch from a specific count of calls to ->atLeast()->once() when getRequest was supposed to be called in tests related to this PR.
Add file paths to exception message when File fails to copy.

I just tried this. It is working fine here. Looking forward to the merge.


Have dan read titles of imgur images

When a link like http://i.imgur.com/heooq.jpg gets posted, it should remove the .jpg and fetch the title of the page that you get from removing the file extension.
Also have it read the titles of the pages if something like http://imgur.com/gallery/heooq is posted, it currently doesn't do it for imgur for some reason.

Is possible by utilizing their API.
Docs here: http://api.imgur.com/
Example JSON: https://api.imgur.com/3/gallery.json (without Auth header will return access denied)

So let's assign uclcommander to this? :P


New image failover taking a second but appears ready in rio

In v0.6.0, manually edit a deployed service and changed the docker image to use a different version. It looks like the endpoint is immediately ready, but executing curl a few times on it has the first couple of responses being from the previous image version.
This happens because the deployment was updated, and we have to wait for the pod to be terminated and new pod to be running.

What's the issue in here? From what i see that's the normal behaviour.

Would be nice for service to show “not ready” if pods are all terminated and none of new ones are ready yet.

Test again on alpha3

v0.6.0 works much better, but could still be enhanced so I'm going to leave this open. Result after editing service and running rio ps in v0.6.0-alpha.3 (see PodInitializing) help text:
NAME              IMAGE                                                                                ENDPOINT                                                PORTS     SCALE      WEIGHT    CREATED             DETAIL
demo              ibuildthecloud/demo:v3                                                                   http://demo-v0-default.7i5cu7.on-rio.io:32655           80        1/1 100%   100%      29 seconds ago      demo-z52bw(PodInitializing)

The only enhancement to be made here at this point would be to show not ready while the old pod is still terminating.


Drop events do not register when sourceNode.parent === targetId

https://github.com/minop1205/react-dnd-treeview/blob/bae03af59b02698d211549ba7966c213b60faeed/src/hooks.ts#L73
Hi,
I understand why this has been implemented in this way, however, it would be great if there were a way to disable this behaviour. I can't share the true reason why I'm flagging up this behaviour as it's todo with a trade secret of my company. However, this could be useful to people for a variety of reasons e.g. If you'd like to display an warning/error (or play a sound) to the user when they try to carry out this action.
Current Behaviour
Dropping a child node onto it's parent does not result in the onDrop callback being called.
Desired Behaviour
Dropping a child node onto it's parent results in the onDrop callback being called.
Many thanks,
Ben

I've started work on a PR for this by the way - I think we can do this in a way which preserves the current behaviour unless the user specified an alternative behaviour 👍

@bcheidemann
ありがとうございます！
現在の動作をデフォルトとして、次のようなAPIを追加して、兄弟間（または自分自身）へのドロップを許可するのはどうでしょうか？
<Tree
  {...props}
  acceptDropToSiblings={true} // default is false
  )}
/>

@minop1205
This one one way I thought to implement it and I would be happy to do it like this.
Alternatively, we could do something like this:
<Tree shouldDrop={(...args) => boolean} ... />
Default would be existing behaviour.
This would be more flexible but may be overkill.
Let me know which option you prefer! :)

Passing the shouldDrop callback as you suggested is more flexible, so let's implement it this way.
But It's a very powerful extension, but depending on how you use it, it could potentially break the tree.
For example, dropping from parent to child as shown in the figure below may cause inconsistencies in the tree, but I think that is the user's own responsibility.

The property name could be shouldDrop, or it could be something like canDrop. Also, I think it is sufficient to include source and target in the args, but you can include anything else if you want.
I would like this property to default to undefined, and if undefined, to do the current behavior.

Yeah I agree - presumably if the user is defining behaviour like this they know what they're doing. But none the less, I will include a warning in the docs 👍 I like canDrop - that's seems quite intuitive to me.

@bcheidemann I just released this feature as v1.3.0.
I have made some commits for the release. Please check this PR for details.
https://github.com/minop1205/react-dnd-treeview/pull/27
Thank you very much!

No problem at all, thanks for publishing it! :)


feat: initial application layout

Description

Titlebar
Sidebar
Text editor
container for visualization not functionality just visual

Related issue(s)
progresses #88

@jonaslagoni should i split it in a smaller commit?

i will remove eslints


Cannot set properties of undefined (setting 'sentryEventId')

Issue workflow progress
Progress of the issue based on the
Contributor Workflow

[ ] 1. The issue provides a
minimal reproduction available on
Stackblitz.

Please install the latest @envelop/* packages that you are using.
Please make sure the reproduction is as small as possible.

[ ] 2. A failing test has been provided
[ ] 3. A local solution has been provided
[ ] 4. A pull request is pending review

Describe the bug
sometimes this error appears
Cannot set properties of undefined (setting 'sentryEventId')

with no stack or whatsoever.
To Reproduce Steps to reproduce the behavior:
:shrug:
Expected behavior
No error.
Environment:

OS:
NodeJS:
@envelop/ versions:

@envelop/core: 5.0.0
@envelop/sentry: 8.0.0

Additional context

I suspect its coming from here: https://github.com/n1ru4l/envelop/blob/main/packages/plugins/sentry/src/index.ts#L107C20-L107C20

Hi @judastheo  and thank you for the report!
Sorry but I'm not adding a lot here but just labeling it according to our new Contribution Guide and issue flow.
One way of pushing this forward would be to add an example of the input and the output you expect to get.
Maybe even adding a failing test so we could get this into stage 2!
Thank you and sorry that this comment is not a complete solution (yet).


localhost:1313 keeps appearing even when configuring baseURL on config.toml

Hello. I keep seeing //localhost:1313 popping up all over my head > meta tags and it messes with the way search results card image get rendered: Here is an example

I'm running hugo behind traefik proxy and serving it with hugo server command

Thank you a lot for the fast response!


Dupe glitch

Uncrafting iron bars or any craftable item with multiple outputs gets you more resources by just crafting more of the same item.
For example, crafting iron bars which takes 6 ingots for 16 bars gets you 16 ingots in the uncrafter and 48 iron ingots in the advanced uncrafter.
Unfortunately this happens with every item that produces more than 1 item like ladders, doors etc.
Is there a way the mod can detect to not allow uncrafting if the output of the recipe is more than 1?
or possibly convert the iron ingots to nuggets?
The issue with the blacklist is that I would have to manually go over every item in game to see if the output is more than 1 and then add it to the blacklist which would be a pain. If there was such a list somewhere, I would gladly accept the list, but my modpack also might be different.
Also with lightman's currency it allows the coins to be uncrafted into the pile of coins duplicating X9.

I updated the list with all Vanilla items, but I can't find the blacklist file.

#13


fix: upload file endless loop

🤔 What is the nature of this change? / 这个变动的性质是？

[ ] New feature / 新特性提交
[ ] Fix bug / bug 修复
[ ] Style optimization / 样式优化
[ ] Code style optimization / 代码风格优化
[ ] Performance optimization / 性能优化
[ ] Build optimization / 构建优化
[ ] Website, documentation, demo improvements / 网站、文档、Demo 改进
[ ] Refactor code or style / 重构代码或样式
[ ] Test related / 测试相关
[ ] Other / 其他

🔗 Related Issue / 相关 Issue
(Describe the source of related requirements, such as the related issue discussion link. Example: close #123, close #456. /描述相关需求的来源，如相关的 issue 讨论链接。 例如：close #123, close #456. )
💡 Background or solution / 需求背景和解决方案
(The specific problem solved. / 解决的具体问题。)
📝 Changelog / 更新日志
(Describe changes from the user side, and list all potential break changes or other risks. / 从用户角度描述具体变化，以及可能的 breaking change 和其他风险。)

1111

222

@CodiumAI-Agent /ask 今天星期几？

Ask❓
今天星期几？
Answer:
The question "今天星期几？" translates to "What day is it today?" in English. This question is unrelated to the PR and its contents.
However, focusing on the PR itself, the changes made in the FileUpload.vue file are aimed at fixing an issue where the file upload process could potentially enter an endless loop. The new code introduces a conditional check to ensure that the fileList.value is only updated if the allowList length is different from the current fileList.value length. This prevents unnecessary updates and likely resolves the endless loop issue.
If you have any specific questions about the code changes or need further clarification on the PR, please let me know!

Ask❓
2222
Answer:
It seems like the question number "2222" is not clear or specific. Could you please provide more context or clarify what you would like to know or discuss about the changes in this PR? I'm here to help with any questions or feedback you might need regarding the code changes.

@CodiumAI-Agent /ask 333

Ask❓
333
Answer:
It seems like there was an issue with the question you intended to ask. Could you please provide more details or clarify your question regarding the PR? I'm here to help with any inquiries you have about the code changes.


Cleanup

This PR cleans up the code base a little bit. No major changes were made to the code.
The primary edits were:

Minor changes to documentation formatting
Changes to log levels for better verbosity level control (to be implemented in next PR)
A change in how the BaSiCPy class is initialized for better type hints when coding. We originally overrode __init__ and used **kwargs to handle input arguments. This makes for messier type hints in code editors.

This closes #102.


Slide height calculated incorrectly in vertical mode

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 3.4.1

Platform/Target and Browser Versions: Chrome 56.0.2924.87

What you did
I initialized a vertical Swiper like this:
var swiper = new Swiper($sliderContainer, {
   direction: 'vertical'
});

Expected Behavior
I expect the swiper to be as high as the highest single slide. I do not want do set a fixed height, as the content should define its height.
Actual Behavior
The swiper and each swipe gets an inline-height set, which equals the height of all slides together.
i.e. if I have 2 slides of a natural height of 500px they get a height set to 1000px, width 3 slides it's 1500px, etc.
If I set the direction to horizontal, it works fine.
You can see the behaviour in this fiddle: https://jsfiddle.net/y1f5z4uo/1/

I used a workaround for now, where I get the curernt slide, calculate its height and apply it to the wrapper.

Hi @janein Try to set max height for swiper-container

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


mysides list command Doesn't List all the Favorites

Hello Expert,
mysides list command Doesn't List all the Favorites that is added on the favorites, Is there any other tool to list all the favorites add with its file path.
Thanks in advance.

It should list favourites
doesn’t list iCloud, Locations or Tags
Can you attach screen shoots of output and finder sidebar?

No, it actually lists all the favorites, but as there are more favorites in my side bar, the command mysides list lists only top 20 to 22 favorites and it stops there.
For security reasons, i couldn't share a screenshot, but you could give it a try by adding multiple favorites , > 40 favorites and then you can try.
Thanks in advance.

The api I used is heavily deprecated so not sure if it will catch all types of sidebar items. I'll give it a shot

Is there a way to list items inserted under iCloud instead of under Favorites?

@huyz
nope raise a feature request issue
But it may not be possible.


Support Apache syntax highlight

I see Nginx syntax highlight is supported, is there a chance to add Apache as well. Thanks.

No. We're not adding more modes to the main distribution—the current way to get support for a language is to write the mode as a separate project, and distribute it on npm as its own package.


Remove IP filtering code for prometheus metrics

Closes #2657

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS against e64a59aecc1c3318bc3fdb4a15ee51088f39ef8e


[AutoPR sql/resource-manager] Update swagger documetation to mark Collation property as writable.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4314

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3680


Broken on default version of Vite

After @vitejs/plugin-react-swc changed their version to 3.6.0 (which, I assume, contains an swc upgrade), every compilation fails with:
0: failed to invoke<EMAIL_ADDRESS>    as js transform plugin at<EMAIL_ADDRESS>    1: RuntimeError: unreachable
           at <unnamed> (<module>[1290]:0x71005)
           .....

Might be related to #22.

Experiencing the same issue. I'm also looking for a project with working Vite+React+@vitejs/plugin-react-swc so that I can copy the version numbers. I have tried overwriting plugin-react-swc to use @swc/core 1.3.105 (using package.json overrides), but the error remained, so I guess the error is coming from somewhere else.

@vitejs/plugin-react-swc v3.5.0 still works with current version of Vite (v5.1.1) , so pinning that version is a workaround, for now.

This is currently working for me:
{
  "type": "module",
  "devDependencies": {
    "@swc-jotai/react-refresh": "0.1.0",
    "@tsconfig/vite-react": "3.0.0",
    "@types/node": "20",
    "@types/react": "18.2",
    "@types/react-dom": "18.2",
    "@vitejs/plugin-react-swc": "3.5.0",
    "typescript": "5.3.3",
    "vite": "5.1.2"
  },
  "overrides": {
    "@swc/core": "1.3.105"
  }
}

I expect this to be a @vitejs/plugin-react-swc issue, but feel free to reopen if it persists with latest versions.

Hey guys, no solution for this yet?


Add virtuoso

This is a request to add my new plugin virtuoso. This is a quite simple Fish plugin extracted from my config.fish; I've found it to be quite useful myself, and am hoping it may be of interest to others too.
Basically, it's a convenience plugin for people that use Vi-like editors, which does the following:

Depending on what's in your $PATH, it sets your $EDITOR to either nvr, nvim, vim, or vi. This is useful for people who prefer using Neovim, but want to automatically fall back to vim or vi when they ssh into a host that does not have Neovim available (but does have fish).
If you start Fish inside Neovim (using :term), or start it inside tmux, then it automatically switches $EDITOR over to nvr if available. In practice, this means that opening files in $EDITOR reuses an existing Neovim instance that is currently visible on the screen. (Either a "surrounding" nvim instance if you run Fish in :term, or one instance per tmux window if you use tmux.)
The commands edit (e), vsplit (vs), split (sp), and tabedit (tabe) should behave roughly the same way in Fish as when you run them in the (Neo)Vim command mode. This reduces the context switching between opening files from within an editor or from inside Fish. All of these commands use the autoselected $EDITOR for portability between hosts/contexts.

Thank you, @jabirali. The documentation looks great. Now consider sharing your prompt with the community, e.g., r/fishshell and if there's community interest I'll merge here.


add new "reason" to electron-alternative

Hi,
I added new "reason" to electron-alternative.

Thank you


Small tweaks to README

thanks, however, the better module for working with leveldb ranges is: https://github.com/dominictarr/ltgt

also o/ long time no see @aredridel !

Oh nice. I was mostly just fixing things in passing as I evaluated modules.

btw, you should pop into ssb when you get the chance, lots has happened since you last visited!

I will have to do that then!


Failed to create sample pipeline (edges pipeline and iris pipeline)

What happened?:
I tried to create new pipelines from

edges pipeline: edges.json
python_train iris pipeline: python_train.json

Command: pachctl create-pipeline -f https://raw.githubusercontent.com/pachyderm/pachyderm/master/doc/examples/opencv/edges.json
I observe user container is having CrashLoopBackoff. And here's the output of kubectl log
mbp-tvlk:iris arinto$ kubectl logs -f pipeline-model-v1-q5ws5 -c user
panic: could not copy /pach-bin/certs to /etc/ssl/certs: could not create /etc/ssl/certs/ca-certificates.crt: open /etc/ssl/certs/ca-certificates.crt: file exists

goroutine 1 [running]:
main.main()
    /go/src/github.com/pachyderm/pachyderm/src/server/cmd/worker/main.go:95 +0x1a9

What you expected to happen?:
New pipeline is created and no error in the Kubernetes pods.
How to reproduce it (as minimally and precisely as possible)?:

Setup pachyderm in Minikube.
Version the data into pachyderm. (patchctl put-file images master liberty.png -c -f http://imgur.com/46Q8nDz.png)
Create pipeline: pachctl create-pipeline -f https://raw.githubusercontent.com/pachyderm/pachyderm/master/doc/examples/opencv/edges.json

Anything else we need to know?:
Environment?:

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.6", GitCommit:"6260bb08c46c31eea6cb538b34a9ceb3e406689c", GitTreeState:"clean", BuildDate:"2017-12-21T06:34:11Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.0", GitCommit:"fc32d2f3698e36b93322a3465f63a14e9f0eaead", GitTreeState:"clean", BuildDate:"2018-03-26T16:44:10Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64

Pachyderm CLI version (use pachctl version):

mbp-tvlk:iris arinto$ pachctl version
COMPONENT           VERSION
pachctl             1.7.2
pachd               1.7.2

Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s): minikube
OS (e.g. from /etc/os-release): macOS Sierra 10.12.6
Others:

Face to the same issue with Pachyderm deployment in GKE
Environment?:

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.7", GitCommit:"dd5e1a2978fd0b97d9b78e1564398aeea7e7fe92", GitTreeState:"clean", BuildDate:"2018-04-19T00:05:56Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"8+", GitVersion:"v1.8.10-gke.0", GitCommit:"16ebd0de8e0ab2d1ef86d5b16ab1899b624a77cd", GitTreeState:"clean", BuildDate:"2018-03-20T20:21:01Z", GoVersion:"go1.8.3b4", Compiler:"gc", Platform:"linux/amd64"}

Pachyderm CLI version (use pachctl version):

COMPONENT           VERSION
pachctl             1.7.2
pachd               1.7.2

Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s): gkt
OS (e.g. from /etc/os-release): macOS Sierra 10.12.6
Others:

After upgrading the cluster and client to v1.7.3, I can run the sample pipelines. Here's my current version:
mbp-tvlk:opencv arinto$ pachctl version
COMPONENT           VERSION
pachctl             1.7.3
pachd               1.7.3


Fixing DataType Issues Giving 400 Bad Request

While fuzzing any endpoint, there were many 400 Bad Requests given as response from the endpoint.
One of the Major reasons being Type conversion errors for three different Data Types:
Edm.Int64: At the end of the Int64 Digit there was a Character 'L' appended which was giving the 400 errors, removed the concatenation of the character at the end of the digit and no more 400 Bad Requests were given for this particular Data Type.
Example:
Earliar:<PHONE_NUMBER>854775807L
Now:<PHONE_NUMBER>854775807
These changes have only been made for the body, the structure for Int64 remains same for the URI
Edm.Double: There were two changes to this Data Type.
Character Removal: A character 'd' was appended to the end of the digit which was giving the error, removed the character from the end and the 400 Bad Requests were resolved.
Example:
Earliar: 9.2237623764e%2b39d
Now: 9.2237623764e%2b39
URL Encoding: The Character '+'/'-' in the range (2.23e-40, 1.19e+40) for decimal was URL encoded when sent to the endpoint which was giving the 400 Bad request, the digit was encoded before creating the URL Payload, resolved the issue by sending the digit without encoding while creating the URL Payload.
Example:
Earliar: 9.2237623764e%2b39
Now: 9.2237623764e+39
These changes have only been made for the body, the structure for Double remains same for the URI

Codecov Report
Base: 62.79% // Head: 62.72% // Decreases project coverage by -0.07% :warning:

Coverage data is based on head (1c5b48e) compared to base (1b8cab6).
Patch coverage: 33.33% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #116      +/-   ##
==========================================
- Coverage   62.79%   62.72%   -0.08%
==========================================
  Files          19       18       -1
  Lines        3693     3694       +1
==========================================
- Hits         2319     2317       -2
- Misses       1374     1377       +3

Impacted Files
Coverage Δ

odfuzz/generators.py
64.46% <33.33%> (-1.22%)
:arrow_down:

odfuzz/__init__.py

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

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


Sentiment Scale in Figma

Full description in https://github.com/trimble-oss/website-modus.trimble.com/issues/449

@soniakaukonen and @egunther39 We need new icons for this


Remove form from /delete endpoint

This was never supposed to work according to the Flask documentation.
Fixes: #317
Signed-off-by: Patrick Uiterwijk<EMAIL_ADDRESS>

Well, I guess the question is why does httpbin consider this behaviour so important? I guess we should look at why the test was added in the first place, what is the reason that httpbin specifically expects this behaviour from Flask/werkzeug.
@puiterwijk ,  what about this, which you said on IRC but didn't mention here:
"Which makes sense, given that DELETE requests don't even provide a Content-Type application/x-form, and unless that exists, parsing it as a form is at best not according to standards, at worst dangerous"

OK, so httpbin was explicitly changed to do this in 2593a8ab -
"Add form and files to DELETE Endpoint
The DELETE endpoint was missing form and files, so if you hit the endpoint with data in the envelope and a Content-Type header set to application/x-www-form-urlencoded, the data will disappear.  Since RFC 2731 doesn't disallow content in the body of a DELETE request, let's allow that data to pass through
Closes #143"
@puiterwijk , what do you think of that?