[TS-4444] Prevent segfault dealocating a NULL buffer_reader.

Signed-off-by: David Calavera<EMAIL_ADDRESS>

--none--

--none--

Can you provide a TS ticket or some explanation of what triggers this? Have you seen a crash associated with this code path, if so, can you share the backtrace?

@bgaff https://issues.apache.org/jira/browse/TS-4444

[approve ci]

[approve ci]

Testing one more manual comment post.

[approve ci]

Can one of the admins verify this patch? Only approve PRs which have been reviewed.

Can one of the admins verify this patch? Only approve PRs which have been reviewed.

Can one of the admins verify this patch? Only approve PRs which have been reviewed.

Can one of the admins verify this patch? Only approve PRs which have been reviewed.

Passed CI build tests.
https://ci.trafficserver.apache.org/view/github/job/Github-FreeBSD/164/
https://ci.trafficserver.apache.org/view/github/job/Github-Linux/72/

@shinrich Can  you review this please?

I'm ok with this change.  It does solve the immediate problem, but I fear that this may be a symptom of a broader issue.  I spent some time this winter tracking down read buffers that were going NULL in the midst of a transaction.  That problem ultimately was the result of a race condition that was freeing the underlying iobuffer.

Yeah, I totally agree with @shinrich. I don't think the NULL reference is the real issue either. Unfortunately, my expertise with ATS code base and internal behavior is still quite limited to find a better immediate solution.

Does this only happen on SSL connections? We're seeing something strange related to IOBuffers and SSL connections.

No, we've seen it without SSL enabled too.

@shinrich , according to @calavera 's git bisect it's this commit: https://github.com/apache/trafficserver/commit/af76977adb9f3c0296a232688bbcb5a1421a6768, we're seeing a similar issue related to POSTS w/ HTTP/2 where read complete events aren't being fired on a VIO. I'm suspecting it's also related to this commit. Our situation seems to happen w/ small posts where the entire payload is sent in a single chunk. It definitely sounds like these two issues could be related.

The TS-3612 commit resulted into significant restructuring resulting in new issues or adjusting timing to reveal latent issues.  I'll take a look at the small post scenario.
For the disappearing buffer_reader problem, sounds like that is a broader issue covering http1.x and probably http2.  I think this may be a latent issue since @oknet noted in the bug comments seeing a similar crash in 6.0.x (which is before the TS-3612 commit).
I'm going to go ahead and merge this commit, but file two more issues.  One to track the short post problem and another to track the disappearing buffer reader problem.


go to function definition for Jupyter Notebooks in external files

I have read this issues
https://github.com/microsoft/vscode-jupyter/issues/1499
https://github.com/microsoft/vscode-jupyter/issues/2341
Is it possible to extend "go to definition (f12)" functionality by opening the external file holding definitions?
I am trying to do it on a function in a notebook but nothing happens.
My function is in .py file a folder close to the notebook and visible in the workspace
Does it only work for definition within the notebook? (so asking for new feature)
Should it open the .py and go to the definition? (so a bug in my side, help!)
Thanks

This should work today, however this depends on your folder structure.
If the notebook and *.py files are in the root directory of your workspace folder then this will work without any changes.
However if either the notebook or *.py file are in different sub directories, then you will need to update the *.env file to set the env variable PYTHONPATH.
I'm assuming you are already doing something similar in code today to get the notebook to run (e.g. sys.path.append or the like)
Please let me know if that works.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


6144f / Sanaa allaw / https://github.com/sanaaAllaw

https://github.com/orgs/projets2017cl/teams/vc/members


Translation not applied in dropdown selection (card search)

This is caused by my inappropriate practice of using i18next
https://github.com/MalitsPlus/HoshimiStage/blob/83ede0dcc80ef84f979f914b2001814519a5bcbe/components/search/SearchOptions.tsx#L125-L126
I placed t() in the field of useState, this causes preset strings cannot be updated during switching languages (rerendering) since the states was never been changed.
I will come up with a solution in few days.


Do not rely on the existence of config.py while detecting modules

Throughout the existence of Godot, the build system relied on whether the module has config.py created as a way to detect modules.
The existence of SCsub is checked instead now. This file is required for all modules, and prevents the build system to leave modules without config.py undetected, leading to silently ignoring the module during compilation.
The build system still expects both SCsub and config.py for a module, but the odds of missing SCsub is far less than config.py (now, if config.py does not exist, scons would still be able to detect the module but also emit an error).
In fact the existence of config.py can be made optional, but this is outside of scope for this PR.
Worth cherry-picking to 3.2.

Thanks!

Cherry-picked for 3.2.2.


Interesting architecture

[Enter feedback here]
Interesting architecture. But it will be more resourceful or helpful if you can provide backing solution for the same. Blog probably will be a good idea.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8f6e327d-b280-a35c-2588-87a19ac2a4ba
Version Independent ID: fa9b0713-771f-4792-979a-2f7a4f4445a2
Content: Microservices with AKS - Azure Solution Ideas
Content Source: docs/solution-ideas/articles/microservices-with-aks.md
Service: architecture-center
GitHub Login: @adamboeglin
Microsoft Alias: pnp

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#179296

@rahulsahay19 Have you seen this article? It covers this type of architecture in more depth.

Previous comment answers this issue.  Closing.


Sort part parameter creation modal (#4251)

Resolves #4251

It seems httpstat.us just had some issues, returning 503 on every request. So thats prbably why the QC Check is failing in unrelated places

@ldet nice work thanks :) Clean solution!


Restart -- warn

Hi,
any way to send the update messages in restart like, arkmanager restart --warn?
Luis

You should be able to use exactly that command to warn players before restarting.

Ok,
let me try, docs are not clear.

You right, it works, txs


Vehicle License Number Plate Detection using YOLOv7 model

Problem: Traditational CNN models are much slower than YOLOv7 due their complex multi-stage pipeline
Solution: YOLOv7 is 509% faster and has 2% higher accuracy than Mask-R CNN Model
It requires several times cheaper hardware than other neural networks and can be trained much faster on small datasets without any pre-trained weights.
Approach: We will perform Transfer Learning and fine-tune our CNN (convolutional neural network)/ deep learning model using the yolov7 architecture. Then after performing OCR text detection and extraction, the model will be able to predict vehicle number by taking video and image input.
Further, we can make this take real time video input since YOLO models because of their high speed can be easily used for real-time object detection which can be implemented in practical scenarios such as traffic surveillance systems  in case of non-adherence to traffic rules on roads and highways by the Government, whereas CNN models would lag in such a case.

Please sir,
I'd love to work on this issue since I have prior knowledge of working on YOLO models with Face Detection algorithms, can you please label this issue for 'gssoc' and the 'level.'
This would be practically helpful for the Government too for traffic surveillance in case of non-adherence to traffic rules on roads and highways.
I'd be really grateful if you would assign me this issue. I promise to work diligently on the project.

The issue has been assigned.
Do follow the project guidelines for PR.

@invigorzz313 Sure ma'am! Thankyou so much!
Additionally, could you please add the 'level' label to the issue as well..?

Level labels are put on the PRs after reviewing it.

I want to contribute to this. Please assign it to me.


Unexpected Behavor at implementing Per User Authorization Feature.

Issue Explanation:
While doing the Quickstart guide for the Amplify Gen-2 I found a confuse behavor at implementing the per user authorization feature.
Here:

Due to the initial code of the amplify-vite-react-template at App.tsx the Per User Authorization feature won't work correctly, because if the user changes between to accounts to validate that the Per User Authorization works, the data got stock to the previous account:
https://github.com/aws-samples/amplify-vite-react-template/assets/54689488/21367b71-4e44-4d29-bbd6-b9039b126d48
In the example above, the account with email<EMAIL_ADDRESS>has 2 todos items written in Spanish, while the other account<EMAIL_ADDRESS>has 2 todos items written in English. As you can see when changing the account, the data got stuck and i need to reload the page to update the data.
I think this is not expected to happened because in the quickstart guide you are meant to use subscription method to query your data.
Request of changes:
I think one of the easiest ways of solving this is moving the TodosList to a separated component.
Something like this:

This way when the user implements the Authenticator component, the TodosList will be unmounted and mounted correctly, and updating subscription query data, retrieving the expected data for each user.
Maybe you can upadate the quickstart documentation to handle this properly, or maybe updating the template also would work!

I added user as a dependency to the  query  effect like this:
  const { user, signOut } = useAuthenticator();
  useEffect(() => {
    client.models.Todo.observeQuery().subscribe({
      next: (data) => setTodos([...data.items]),
    });
  }, [user]);


2db -t new fails

Running the command 2db -t new fails with the message Unknown type: none and I was unable to find any workaround to this.
Conversely, db2 -t new on the same path prints Convert complete but there are no changes to the original file or any new files.

Perhaps the command you need to execute is 2db -t spine.

Sorry, 2db -t new is a typo.


Centralize deployment configurations

Currently users have to modify the iframe_peer.html, debugger.html, and intent_chooser.html when modifying backend root url, api configurations, ect.
Rather than modify each of these files, include an iwc.conf.js in the dist files that each reference.  Settings in this config file are not required, rather just overrides for default values in the IWC bus construction. This will effect current ozone build scripts, so updates will need to made to them as well.
This issue tracking includes documentation on the conf parameters.
@arw180 heads up about this. Not using OzoneConfig.js for this configuration file as it is purely IWC configurations and I don't want this to create deployment confusions as to the varied behavior among config files.

Current implementation on feature-deploy-config.
Documentation/Testing underway. will be updating to version 1.1.0 with this change.

Came here from #267. :+1: I like it.

Makes sense, I like it. We'll just have to make sure we update our dev-tools and Jenkins scripts to sed that config file accordingly.

Merged to master, will be releasing 1.10.0 after addressing #327. Can be built and ran locally with normal IWC grunt use: grunt serve initial doc updates can be found here. Not on gitbook yet because it only updates on releases.
Additional config params are open for request

Documentation updated to the gitbook here
Closing as tagging 1.1.0


Doesn't work for uploading on Teensy MicroMod

I have a special project that includes a Teensy MicroMod. I have the MM in Dual Serial mode. There are two basic problems here it looks like:

It detects the second serial port first. I'm not sure if this matters or not but it seems to try to use the second port preferentially whereas it seems the first port would be more appropriate? At least for me, I'd expect and want to connect to the first port.
All your programs seem to fail with: "I/O error while writing to '/dev/hidraw4': Broken pipe" any time I try to do an upload. The Teensy goes into bootloader / Halfkay mode by this point but no matter how many times I try it always says it had an I/O error on hidraw4 and that there is a broken pipe. Both the normal Teensy GUI upload window and the Teensy CLI tool can flash the board.

Something is not right?
Compiled for T_MM Dual Serial and it added second device icon with RED button.
Then recompiled same sketch for MTP and, even if power cycled, both device ports are shown and the SerMon data - from Seremu used with MTP - can appear on either device?
This same behavior now appears on T_4.1. Dual Serial added a second device, and it survives going to MTP w/Seremu and output - only sent to 'Serial' in code may appear on either instance.
There is something odd - may be the sketch I've been using on T_4.1 doesn't like T_MM ...
I closed TyCommander and reopening with MTP w/SEREMU T_MM and T_4.1 work properly, but once it sees Dual Serial changing to MTP w/SEREMU doesn't delete the second device.
I can't say I've done this before to rule out recent code change?
@collin80 - can you try a prior release of TYQT?

can confirm similar issue when uploading to a 4.1
[upload@11058150-Teensy] I/O error while writing to '\.\HID#VID_16C0&PID_0478#6&31027823&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}'

Thanks for the report.
This is a regression I introduced in the latest release (0.9.6).
It is fixed in release 0.9.7: https://github.com/Koromix/tytools/releases/tag/v0.9.7
If not, feel free to comment and I will reopen the bug ;)


Please provide information and required permissions for SQL CEIP Service account

[Enter feedback here]
The document contains the information/permission set for all other services except SQL CEIP. Please provide information and required permissions for SQL CEIP Service account.

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

ID: 9b3831a3-6c86-243c-4b07-8960e4d06d7b
Version Independent ID: dfd7599d-88eb-99f2-c227-0c9dc82bfd87
Content: Configure Windows Service Accounts and Permissions - SQL Server
Content Source: docs/database-engine/configure-windows/configure-windows-service-accounts-and-permissions.md
Product: sql
Technology: configuration
GitHub Login: @markingmyname
Microsoft Alias: maghan

@ram2ybabu -- Rambabu, thank you for your feedback.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@markingmyname -- Mark, please look into this issue.

Please provide an update on this request at the earliest.

@ram2ybabu
Thank you for your patience.
submitted a PR to reflect this change.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion, and we will reopen the issue. To see what the community is wanting, please visit the SQL Server User Feedback site.


Update logging docs to indicate that IsLoggingContentEnabled requires EventLevel.Verbose

see https://github.com/Azure/azure-sdk-for-net/blob/89ebe48fe7037f9b8eb13f7c2e142ff40959fb8e/sdk/core/Azure.Core/src/Diagnostics/AzureCoreEventSource.cs#L69

API change check
API changes are not detected in this pull request.


Merge Release 1.1.0 into master

Coverage decreased (-0.006%) to 55.309% when pulling 2b7d9fcb701271b3cb55e12a61e6b57ed0ca1fc9 on wknet123:release-1.1.0 into 13d9b05dcb837ff8db299637542ff7e6057882bf on vmware:master.


Pin multiple columns

Describe the solution you'd like
We would love to pin multiple columns (to the left), think of firstname and lastname as two dedicated columns (for sorting) and we would love for both of them to remain sticky on the left (horizontal scrolling).
Describe alternatives you've considered
Was thinking of merging the two columns, but that would break sorting. Alternative would be grouping them, but the docs says that does not work either.
Additional context
While looking for other people who might have reported something similar, I stumbled across https://v2.mantine-react-table.com/docs/guides/column-pinning which supports pinning multiple columns - I prefer the styling and UX of this project, so would prefer not migrating - just as inspiration :)

This is indeed a highly desired feature, that has been required and discussed before, see https://github.com/icflorescu/mantine-datatable/discussions/229 and https://github.com/icflorescu/mantine-datatable/issues/477 or https://github.com/icflorescu/mantine-datatable/issues/347.
At the moment I don't have enough resources to dedicate to this, but, as I mentioned before, I'm 100% open to a PR:

it would be fantastic if someone would provide some help with scaling this to something like pinLeftColumns: number and pinRightColumns: number.

I think it could be implemented by dynamically computing the size of the pinned columns at render time.
Here are some code starting points:
https://github.com/icflorescu/mantine-datatable/blob/f8a40149f4a12b2e023cc81e2572ad493edc3342/package/DataTable.tsx#L149
And the CSS to render the scroll shadows as needed, which is a bit convoluted and tricky:
https://github.com/icflorescu/mantine-datatable/blob/f8a40149f4a12b2e023cc81e2572ad493edc3342/package/DataTable.css#L101-L242

Just to clarify things for anyone bumping into this: it's not trivial but it can be done.
I'll address it at some point in the future, I just don't have the time/resources to do so now.
You can help speed up the process by:

either giving it a try in a PR
or, if your company is using Mantine DataTable and can afford it, sponsoring the development of the feature (drop me a message at the email address listed in my GitHub profile before) or hiring my services


My data got added

merge this pull request.
Everthing is done by following all steps of contributing.md

Good job @tussharpaul13
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.
Note that this repo is excluded from Hacktoberfest but this PR is still eligible for ClueLess Swags.

Star this repository. ⭐

SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.

@Rajdip019 https://www.clueless.tech/profile/96624900


Publicize some functions in PreviewingController file

Make some functions available to other files to be able to use the peekpop view in MKAnnotations since annotations cannot detect long tap gestures using BAPeekPop
Changed text color of actions buttons

Hi @BramYeh, this PR is ready for Code Review now. Thanks for looking on this!

Hi, @kimmanuel
I found Travis cannot build successfully because BAPreviewingViewController.h isn't public header.
I would merge this PR first and correct it.
Thanks for your assistance to refine color and make it supports MKAnnotations


No antemeridian support?

Describe the bug
What says on the tin.
To Reproduce
https://graphhopper.com/maps/?point=68.95933%2C179.993481&point=68.954544%2C-179.954919&profile=car&layer=Omniscale
Expected behavior
Drive from point A to point B, just a few kilometers away connected by a single road.

The web UI fails to properly center as well, so here's a quick screenshot for reference:

https://wiki.openstreetmap.org/wiki/180th_meridian

In theory, it should be a meridian like any other. In OSM, there's two copies at ±180° longitude because the data model doesn't wrap around, so they mark the east-west ends of the world, with line tagged with closure_segment=yes. Hence features crossing the meridian are generally being separated into two parts in OSM.

Looks like we need to handle points with latitude -180/+180 as equals.


Error: SQLITE_BUSY: database is locked errno: 5, code: 'SQLITE_BUSY'

Note: this is the technical bug tracker, please use other platforms for getting support and starting a (non technical) discussion. See the getting help page for details.
I'm submitting a ...
[x] bug report
[ ] question about the decisions made in the repository
Action taken (what you did)
i set up the bot nd push start
Expected result (what you hoped would happen)
waiting for the first trade to be made
Actual result (unexpected outcome)
poup message saying Error: SQLITE_BUSY: database is locked
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
how to make sure the api key is working?

You probably have two instances of processes working on the same sqlite database.
Try to stop one.

error: sqlite_busy: database is locked
received error in gekko
help me


Remove logging dependencies from compile-scope

Currently (v 1.1.3) the logging deps (slf4j-log4j12, jcl-over-slf4j, log4j) are in compile scope which means that everybody has them in his project. But often users want to use a different logging impl (e.g. logback).
Only slf4j-api should be kept in compile scope, all others should go to optional, provided or better test.
Best regards
Lars

Fixed in the current build.


ui settings

it's easier than having to change the settings in settings.json

Transferring this VS Code since the settings UI is handled by VS Code.
This is for the following settings that have to be set via JSON:

These are contributed via package json like this:
                "python.pylintSeverity": {
                    "default": {
                        "convention": "Information",
                        "error": "Error",
                        "fatal": "Error",
                        "refactor": "Hint",
                        "warning": "Warning",
                        "info": "Information"
                    },
                    "additionalProperties": {
                        "enum": [
                            "Error",
                            "Hint",
                            "Information",
                            "Warning"
                        ]
                    },
                    "description": "Mapping from severity of `pylint` message type to severity shown in problem window.",
                    "scope": "resource",
                    "type": "object"
                },
                "python.pylintPath": {
                    "default": [],
                    "description": "When set to a path to pylint binary, extension will use that for linting. NOTE: Using this option may slowdown linting.",
                    "scope": "resource",
                    "items": {
                        "type": "string"
                    },
                    "type": [
                        "array"
                    ]
                }


⚠ ClamAV has degraded performance

In 69adb17, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 178 ms

Resolved: ClamAV performance has improved in 69156d5.


Decrypting File Encrypted To Myself On Another Machine

A couple days back I encrypted a file on my mac using:
keybase encrypt mcshauno -i secret.txt -o secret.txt.x
I then put that file in my Dropbox.  I then attempted to decrypt that file on another computer (that had keybase installed with the same account) using:
keybase decrypt -i secret.txt.x -o secret.txt
And it fails and tells me I should try on other devices.  I imagine I am missing something simple here but would appreciate some help.  Thanks!

Did you provision the second device after you performed the encryption?
If so, the original encryption wasn't encrypted for your new device.

@maxtaco is it possible to decrypt a file on a new device that was provisioned after the file was originally encrypted if you use a paper key? In that scenario, would the paper key need to have been created pre-encryption?

cc: @mlsteele and @oconnor663 who have thought a bunch about this.

@brandonblack yes, and yes. Take a look at the keybase decrypt --paperkey flag.
Right this very second we're working on a new version of saltpack to solve this problem, by using shared encryption keys that later devices can get a copy of (the same keys we use to encrypt files in KBFS). Hopefully the "use a different device" scenario will be a thing of the past. Sorry for all the trouble in the meantime.


Docs site CSS not run through autoprefixer

Describe the bug
Design-system has three build steps for CSS:

It has the gulp task styles.js used by cfgov-refresh and elsewhere.
It has a shell script, compile-css for directly compiling the less.
It has another shell script watch-css for compiling the less on changes (also run if the compiled file is missing).

Neither of the shell scripts include autoprefixer and run less compilation directly on the source files without further processing, leading to missing vendor prefixes, such as the issue of appearance not being parsed in the drop-downs.
The directions say to run yarn and yarn start. yarn start calls yarn run watch-css, which calls the watch-css script, which calls the less-watch-compiler compiler.
yarn run build and yarn run build-netlify both call the compile-css script, which calls the lessc compiler.
I could not find where yarn run build would ever be run (I've noted it in the list of undocumented scripts in the repo). There's an identically named command in the root directory package.json that is called by yarn run release, but that calls the script in that package.json, which calls gulp build.
yarn run build-netlify appears to be called by the netlify build command, though I'm unclear what triggers that as it is undocumented in the repo, but I presume this runs when building the test URLs in the PR previews.

PR for running doc site CSS through autoprefixer --> https://github.com/cfpb/design-system/pull/1030

This is done


fix: missing Content-Type header for HTML in express server example

Summary
Basically, the following two code snippets works differently when dealing with the HTTP header Content-Type:
const { html, status, statusText, headers } = await renderPage(url, {
    manifest,
    preload: true,
    // Anything passed here will be available in the main hook
    request,
    response,
    // initialState: { ... } // <- This would also be available
  })

  response.writeHead(status || 200, statusText || headers, headers)
  response.end(html)

server.use(
    '/' + asset,
    express.static(path.join(__dirname, `${dist}/client/` + asset))
  )

Express won't auto detect and set Content-Type for the actual content html in the first code snippet, where express.static(...) will handle the Content-Type header for each individual assets correctly in the second code snippet.
And as for another security issue involved scenario, where we will need the X-Content-Type-Options header set to nosniff present for each request in order to prevent MIME type attacks, reference:

These vulnerabilities can occur when a website allows users to upload content to a website however the user disguises a particular file type as something else. This can give them the opportunity to perform cross-site scripting and compromise the website.

However, this security header helps prevent these types of attacks by disabling the MIME sniffing functionality of IE and Chrome browsers so that the browser is required to use the MIME type sent via the origin server. Consider the following example of how X-Content-Type-Options works for a particular web request.

Whether or not the X-Content-Type-Options header present, the server should always set the Content-Type header correctly. Therefore we need to set the Content-Type explicitly, which is this PR's meaning here.
References
X-Content-Type-Options header - MDN
X-Content-Type-Options HTTP Header - KeyCDN Support

Thanks! I'm thinking that maybe we should provide these headers automatically in the returned headers object 🤔


Adds content for datalad catalog

Hey @jsheunis, thank you for the submission!
Please ask your co-authors to fill in the form as well, possibly before tomorrow at noon UTC: https://forms.gle/fLnjWLk4dPu1TJjQ7
Also, please check if the papers that you are citing are already cited in the document - if so please use those citations instead. I'm fairly sure at least one paper is cited already.


Create Ingredient Categories

It would be nice to be able to create ingredient categories.
Useful Examples:

Amaro is a rather large category of liqueur a user might want to create if they own many different Amaro
A user has 30 different Jamaican Rums. It would be a good if the user could create a Jamaican Rum category.

Yeah, this is missing currently in web app (alongside glass types). It will be added in the stable release.
In the meantime you can manually create and edit categories through API calls.

Implemented in new settings page in v0.6.0.
Thanks!


Method PROPFIND is not allowed by Access-Control-Allow-Methods in preflight response

Methods option is skipped from cors lib: https://github.com/expressjs/cors#configuration-options
I have Access-Control-Allow-Origin: * in server responce but its not enough..
Access to XMLHttpRequest at 'http://localhost:8010/proxy/webdav/server/.ts/' from origin 'http://<IP_ADDRESS>:8000' has been blocked by CORS policy: Method PROPFIND is not allowed by Access-Control-Allow-Methods in preflight response.

default Access-Control-Allow-Methods is GET,HEAD,PUT,PATCH,POST,DELETE. Might extensions overwrite it.

Hey, I forked this repository for the same reason.
You may be interested in this package: proxy-cors-cli It's support WebDAV / CalDAV HTTP methods.


No support below kingdom level for known E. coli contigs

Hi, I am running CAT to annotate contigs from metagenomes that are known to be E. coli by several other approaches.  For example, greater than 99% average nucleotide identity with reference E. coli genomes and BLASTx also identifies the vast majority of ORFs as E. coli. For me, CAT identifies several other contigs with species support in the metagenome, and the db and taxonomy folders from CAT prepare seem to be correct as no errors are encountered.
Despite this, across many samples, of 239 contigs expected to be E. coli, 9 are classified at the family level (Enterobacteriaceae) and the rest are classified as Bacteria.
When I look at the ORFs themselves following add_names, I see something similar as the vast majority of individual ORFs receive no support below Bacteria.
I tried this with a lab E. coli reference genome as a single "contig" and it is classified as Bacteria.

I am using CAT v5.2.3 and the original version of the DB from 2021-01-07 as described in the repo. I do see several strains and species of E. coli in names.dmp and the taxid's are present in nodes.dmp.

Dear @mpgriesh,
Thanks for this report! It's something that we have noticed ourselves as well, and I think it stems from the fact that many E. coli are misannotated in NCBI nr, as human for example. It may also be that foreign vectors within lab coli's are annotated as coli... There is currently not an easy solution for this that we can implement except cleaning nr ourselves (we're thinking on how to do this automatically).
For now, could you try one of the latest databases (see https://tbb.bio.uu.nl/tina/CAT_prepare/)? NCBI is removing misannotations so newer databases may be better, then again they may also contain more misannotations. A more viable alternative may be to use the GTDB databse instead of NCBI nr (we have implemented it in the latest version of CAT). GTDB does not have the nr misannotations, but of course does have a smaller search space. You can also find the latest GTDB database formatted for CAT on https://tbb.bio.uu.nl/tina/CAT_prepare/.
Do keep me updated!
Best wishes,
Bastiaan

Hi, first of all, thanks for the awesome too. I am using it in my metagenome/transcriptome pipeline, but I am having similar problems with this. In one example, there was one viral contig whose proteins have hundreds of hits against the correct virus sequences in NR, but there was also one artificial construct whose fastaid2LCAtaxid is 1. This ruins the taxonomy assignment of that entire contig. This also happens for other contigs. I imagine similar entries in the NR database like artificial sequences or metagenome sequences would ruin taxonomy assignment similarly.
I know GTDB is curated and a potential solution, but it doesn't have the virus and eukaryotes that I need. I think there may be two potential solution.

Create a blacklist of tax IDs that find_LCA_for_ORF function would just ignore. This is easier to do, maybe that's something you are considering. In addition to tax ID of 1, is there anything else you think that can be ignored?
Use a voting approach for find_LCA_for_ORF. Like how CAT assign taxonomy based on voting of taxonomy from all ORFs of the contig, you can assign taxonomy to ORFs based on the consensus of all hits based on a similar fraction option like -f. IN this case, the smaller number of "poisonous" proteins in the database wouldn't matter. This could make the program a lot slower though.


Added new fild - how to remove?

How can I remove just added new field?

just make its value blank


Make parameter names consistent with Generations + JumpBoard template

This will allow for easy loading of data from one game and saving to another, when such a feature is implemented.

Wow - really sorry for not seeing this until now!
Thanks for the PR!

What does the m mean in m_MonkeyTarget?

it's kind of a prefix used in C++, m means Member, used for class members


Last utterance is filling slot in new form

Rasa version: 2.1.2
Rasa SDK version (if used & relevant): 2.1.2
Python version: 3.8.6
Operating system (windows, osx, ...): osx
Issue:
I have a form that finishes by asking for a size of shoe, filled with a number entity extracted using Duckling. After I submit this form, I kick off another form that asks a survey. The survey asks for a 1-5 rating and for open text feedback. What happens is that when the survey form is kicked off after the user gave their shoe size, the number extracted for shoe size also fills the rating slot, skipping this question. Example:
Your input ->  check inventory
I can help you find some shoes!
What color are you looking for?
Your input ->  blue
What size shoe would you like?
Your input ->  9
You're in luck! We have those in stock.
Is there any other feedback you can provide?

The last question is the second question in my form, showing that the first question (rating question) was skipped.
Bot is located here: https://github.com/RasaHQ/retail-demo
Enable squad definition of done

[ ] Determine if this is expected behaviour given configuration
[ ] If not, then fix the bug
[ ] Else, open an enhancement issue

@ArjaanBuijk didn't you fix this in https://github.com/RasaHQ/rasa/pull/7771 ?

@desmarchris ,
Can you post the full debug log?
It sounds similar to #7771, but not exactly.

Hi @desmarchris , I've just had a go at a bot conversation using rasa shell --debug after going through all the steps in your README. I was not able to replicate the issue in my first conversation: after the questions about colour and shoe size I got this bot reply which seems to be expected behaviour:
You're in luck! We have those in stock.
? On a scale of 1-5, how would you rate your conversation today?

Could you please let me know what else I can try to replicate this issue?

@ancalita yes sorry that's because I changed the order of the questions to get around it :) I have it on my to do list to rewrite how to replicate this.

@ancalita try using this commit: https://github.com/RasaHQ/retail-demo/tree/05aeb3486dd995262799e3995570da2cefb27398
I just changed the order of the slot filling after this. Same steps apply

@desmarchris Thanks for sending this - I was able to replicate using this commit.
I noticed one thing in the domain.yml of this commit that I have a hunch might be causing this behaviour: there is only one number slot that gets used by both forms: product_stock_form and survey_form.
When I introduced a new slot rating and a new entity feedback_rating, the behaviour you flagged disappears.
Instead I have a new issue: although the survey_form gets activated after the product_stock_form, it reverts to action_listen because There was no action found to ask for slot 'rating' name to be filled.
Here's an extract of the debug log:
2021-04-29 11:24:50 DEBUG    rasa.core.policies.rule_policy  - There is a rule for the next action 'action_survey_start'.
2021-04-29 11:24:50 DEBUG    rasa.core.policies.ensemble  - Predicted next action using policy_2_RulePolicy.
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Predicted next action 'action_survey_start' with confidence 1.00.
2021-04-29 11:24:50 DEBUG    rasa.core.actions.action  - Calling action endpoint to run action 'action_survey_start'.
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Policy prediction ended with events '[]'.
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Action 'action_survey_start' ended with events '[<rasa.shared.core.events.FollowupAction object at 0x14b744c40>, <rasa.shared.core.events.SlotSet object at 0x14f841430>]'.
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Current slot values:
	color: None
	email: None
	number: None
	rating: None
	open_feedback: None
	requested_slot: None
	verified_email: None
	survey_complete: None
	session_started_metadata: None
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Predicted next action 'survey_form' with confidence 1.00.
2021-04-29 11:24:50 DEBUG    rasa.core.actions.forms  - Activated the form 'survey_form'.
2021-04-29 11:24:50 DEBUG    rasa.core.actions.forms  - No pre-filled required slots to validate.
2021-04-29 11:24:50 DEBUG    rasa.core.actions.forms  - Validating user input 'UserUttered(text: 9, intent: inform, entities: 9 (Type: number, Role: None, Group: None), use_text_for_featurization: False)'.
2021-04-29 11:24:50 DEBUG    rasa.core.actions.forms  - Validating extracted slots: {}
2021-04-29 11:24:50 DEBUG    rasa.core.actions.forms  - Request next slot 'rating'
2021-04-29 11:24:50 DEBUG    rasa.core.actions.forms  - There was no action found to ask for slot 'rating' name to be filled.
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Policy prediction ended with events '[]'.
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Action 'survey_form' ended with events '[<rasa.shared.core.events.ActiveLoop object at 0x14b683be0>, <rasa.shared.core.events.SlotSet object at 0x14f841730>]'.
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Current slot values:
	color: None
	email: None
	number: None
	rating: None
	open_feedback: None
	requested_slot: rating
	verified_email: None
	survey_complete: None
	session_started_metadata: None
2021-04-29 11:24:50 DEBUG    rasa.core.policies.memoization  - Current tracker state:
2021-04-29 11:24:50 DEBUG    rasa.core.policies.memoization  - There is a memorised next action 'action_listen'
2021-04-29 11:24:50 DEBUG    rasa.core.policies.ted_policy  - TED predicted 'action_listen' based on user intent.
2021-04-29 11:24:50 DEBUG    rasa.core.policies.rule_policy  - Predicted 'action_listen' after loop 'survey_form'.
2021-04-29 11:24:50 DEBUG    rasa.core.policies.ensemble  - Predicted next action using policy_2_RulePolicy.
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Predicted next action 'action_listen' with confidence 1.00.
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Policy prediction ended with events '[]'.
2021-04-29 11:24:50 DEBUG    rasa.core.processor  - Action 'action_listen' ended with events '[]'.
2021-04-29 11:24:50 DEBUG    rasa.core.lock_store  - Deleted lock for conversation 'd67271b0207e463e9b8bb62534243b22'.

I've also edited the custom actions SurveyStart and SurveySubmit to return SlotSet("rating", None) instead of SlotSet("number", None). Is there anything I'm missing?

I've managed to trigger the survey form immediately after the product_stock_form by modifying the action code:
class SurveyStart(Action):

    def name(self) -> Text:
        return "action_survey_start"

    async def run(self, dispatcher: CollectingDispatcher,
            tracker: Tracker,
            domain: Dict[Text, Any]) -> List[Dict[Text, Any]]:

        if tracker.get_slot("survey_complete")==True:
            return[]
        else:
            dispatcher.utter_message(template="utter_ask_survey_form_number")
            return [FollowupAction("survey_form"), ActiveLoop("survey_form")]

However it identifies the entity number instead of feedback_rating 😒 and then it predicts utter_start_over - still investigating.

Hi @desmarchris sorry for the multiple posts; I've managed to sort the issue you were having which leads me to think this was a usage error.
Here were the steps I took:

deduplicate number slot used in both forms & redefine the slot in survey_form as rating
rating slot is defined by entity number like so:

  survey_form:
    rating:
    - entity: number
      type: from_entity
    open_feedback:
    - type: from_text

add the code in bold in SurveyStart Action in the else branch:

        if tracker.get_slot("survey_complete")==True:
            return[]
        else:
            **dispatcher.utter_message(template="utter_ask_survey_form_number")**
            return [FollowupAction("survey_form"), **ActiveLoop("survey_form")**]

@wochinge do you agree with my conclusion?

oh sweet! thanks for exploring @ancalita!


xstrided_view split

This can be merged once all the light are green. I'll implement the dynamic view in a dedicated PR to avoid many rebase and merge due to other developments changing the xstrided_view.


Updated cs (Czech) translation

Commit made via Stringlate

Hi, I created Czech translation. Please add it to your awesome app.


Refactor UX calls to be blocking; simplify dispatcher

To be reviewed and merged after #61.

Codecov Report

Merging #62 (fca6f3b) into experimental (ee99834) will increase coverage by 8.59%.
The diff coverage is 100.00%.

@@               Coverage Diff                @@
##           experimental      #62      +/-   ##
================================================
+ Coverage         75.17%   83.76%   +8.59%
================================================
  Files                15       17       +2
  Lines               882     2076    +1194
================================================
+ Hits                663     1739    +1076
- Misses              219      337     +118

Flag
Coverage Δ

unittests
83.76% <100.00%> (+8.59%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/common/buffer.h
100.00% <ø> (ø)

src/common/wallet.c
84.68% <ø> (+32.08%)
:arrow_up:

src/common/buffer.c
89.34% <100.00%> (+0.17%)
:arrow_up:

src/common/script.c
98.63% <100.00%> (+0.26%)
:arrow_up:

src/common/wallet.h
100.00% <100.00%> (ø)

src/debug-helpers/debug.h
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Fixes grid selection to be aware of nested tables

Why
As pointed in #4647 Table selection behaved with issue on moving selection between nested tables. What happened is that we expect nodes anchor/focus (from/to) to be in the same grid by default, but it isn't true for nested tables.
What
Here after finding focus and anchor cells we check if grids for both of them are equal - that would mean we are on the same level, if not we are moving one selection up. This change makes selection behaviour as next:

If we move focus from higher level to lower - same higher cell to be selected.
If we move focus from nested level to level above (any cell) that would mean we make whole original cell to be focused.
If we move focus from higher to same level including some of nested tables - they will be skipped to focus higher level cells.

Before:
https://github.com/facebook/lexical/assets/5208565/59dcb11c-c267-4b29-947f-2ef1840ea61f
After:
https://github.com/facebook/lexical/assets/5208565/2f526ffa-4cec-4ba0-bfbb-5d7b53690ca8

A bit on the nit picking side, but if I do 3-level table in table nesting, selection still breaks.

A bit on the nit picking side, but if I do 3-level table in table nesting, selection still breaks.

yeah, wasn't sure if that's something lexical want to support or not, let me see if could generalize it to cover it.

Great, however, when I use the / menu and select '5x5 table' and then in any cell I do it again '/' and then '5x5 table' it crashes the selection directly even when using 2 tables nested within each other.

Great, however, when I use the / menu and select '5x5 table' and then in any cell I do it again '/' and then '5x5 table' it crashes the selection directly even when using 2 tables nested within each other.

I cannot reproduce it locally on this branch, could you please share video steps?

https://github.com/facebook/lexical/assets/7893468/f37b5364-7bfe-4f88-810c-15fdc04d2662
So I managed to replicate the issue, it appears when I use the keyboard arrows to navigate, it happens 50% of the time. If I do point and click, it works fine, but if I navigate with the arrows upon inserting the second table, selection freezes.

broken_tables.mp4
So I managed to replicate the issue, it appears when I use the keyboard arrows to navigate, it happens 50% of the time. If I do point and click, it works fine, but if I navigate with the arrows upon inserting the second table, selection freezes.

Thanks for the details, was able to replicate now, it seems that selection and grid actions are located in multiple places and not the same for mouse and keyboard events, I'm putting down all possible use cases with keyboard and going to update PR with fixes for all edge cases I could find there.

Hey @icrosil , is this good for review? I see it working well in the playground preview, but the tests you've added fail often, so unsure if you are planning on more change for this PR.

Hey @icrosil , is this good for review? I see it working well in the playground preview, but the tests you've added fail often, so unsure if you are planning on more change for this PR.

Hey I wanted to fix tests first, to make sure I'm not breaking anything, but if that would take too long I'd probably slim them down as functionality now is in better state.
I wouldn't claim that it fixed all possible scenarios because of the fragmented selection control I think that would be extremely difficult, but at least we fixed highlighted use cases. Let me try update fixes first, will ping you here on thread when ready for a review. Thanks!

Hey @icrosil , is this good for review? I see it working well in the playground preview, but the tests you've added fail often, so unsure if you are planning on more change for this PR.

I've decided to remove flaky tests from diff to see if CI will pass on this, doing more manual checks locally please feel free to review code, especially if CI is green :)


Packet-in rate limiting with OF Meter

Add OF meter to implement rate-limiting for packet-in.

Add meter entry while initialization.
While building the flow that will trigger packet-in, except traceflow, apply meter entry to it.
Update libOpenflow and ofnet version, because OF meter related commit has been merged into them.

Since windows OVS doesn't support OF meter, we skip OF meter related operations for now. We contacted OVS team to ask for this support and will remove skip after windows OVS support OF meter.
The benchmarks are as below:

Use hping3 <IP_ADDRESS> -p 80 --flood -2 and OF meter pktps bands=type=drop rate=100

top - 22:28:03 up 5 days,  5:34,  1 user,  load average: 0.03, 0.38, 0.55
Tasks: 157 total,   1 running, 156 sleeping,   0 stopped,   0 zombie
%Cpu(s):  4.0 us,  5.8 sy,  0.0 ni, 60.2 id,  0.0 wa,  0.0 hi, 30.0 si,  0.0 st
MiB Mem :   1987.6 total,    587.1 free,    472.5 used,    927.9 buff/cache
MiB Swap:      0.0 total,      0.0 free,      0.0 used.   1333.6 avail Mem

    PID USER      PR  NI    VIRT    RES    SHR S  %CPU  %MEM     TIME+ COMMAND
2022714 root      10 -10  235600  39464  12156 S   7.3   1.9   0:02.00 ovs-vswitchd
     10 root      20   0       0      0      0 S   5.7   0.0   0:42.08 ksoftirqd/0
2022555 root      20   0 1267928  51976  34572 S   5.0   2.6   0:02.49 antrea-agent
  14079 root      20   0 1941628  72840  30968 S   2.3   3.6 164:48.40 kubelet

Use hping3 <IP_ADDRESS> -p 80 --flood -2 without OF meter

top - 22:19:06 up 5 days,  5:25,  1 user,  load average: 1.48, 1.02, 0.75
Tasks: 147 total,   3 running, 144 sleeping,   0 stopped,   0 zombie
%Cpu(s): 33.3 us, 19.1 sy,  0.0 ni, 11.1 id,  0.0 wa,  0.0 hi, 36.5 si,  0.0 st
MiB Mem :   1987.6 total,    566.6 free,    480.7 used,    940.3 buff/cache
MiB Swap:      0.0 total,      0.0 free,      0.0 used.   1326.7 avail Mem

    PID USER      PR  NI    VIRT    RES    SHR S  %CPU  %MEM     TIME+ COMMAND
2010190 root      10 -10  235704  46428  12156 R  98.0   2.3   1:40.03 ovs-vswitchd
2009999 root      20   0 1341660  56128  34732 R  55.4   2.8   1:02.89 antrea-agent
     10 root      20   0       0      0      0 S   7.3   0.0   0:40.94 ksoftirqd/0
  14079 root      20   0 1941628  73520  31496 S   3.6   3.6 164:32.39 kubelet

Fixes #2069

Codecov Report

Merging #2121 (b53b2c7) into main (474fd45) will increase coverage by 0.07%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##             main    #2121      +/-   ##
==========================================
+ Coverage   41.35%   41.43%   +0.07%
==========================================
  Files         131      131
  Lines       16434    16509      +75
==========================================
+ Hits         6797     6841      +44
- Misses       9059     9087      +28
- Partials      578      581       +3

Flag
Coverage Δ

unit-tests
41.43% <14.28%> (+0.07%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/agent/openflow/client.go
23.28% <0.00%> (-0.13%)
:arrow_down:

pkg/ovs/openflow/ofctrl_action.go
11.34% <0.00%> (-0.20%)
:arrow_down:

pkg/agent/openflow/pipeline.go
23.19% <100.00%> (-0.32%)
:arrow_down:

pkg/antctl/raw/traceflow/command.go
23.82% <0.00%> (-1.92%)
:arrow_down:

...ntroller/networkpolicy/networkpolicy_controller.go
53.87% <0.00%> (-1.65%)
:arrow_down:

pkg/agent/flowexporter/connections/connections.go
59.85% <0.00%> (-0.29%)
:arrow_down:

pkg/agent/flowexporter/exporter/exporter.go
47.19% <0.00%> (-0.08%)
:arrow_down:

pkg/antctl/antctl.go
100.00% <0.00%> (ø)

...agent/controller/traceflow/traceflow_controller.go
0.00% <0.00%> (ø)

... and 8 more

@GraysonWu as a follow up PR, we can look into exposing meter stats through anctl and Prometheus metrics

@GraysonWu as a follow up PR, we can look into exposing meter stats through anctl and Prometheus metrics

NP.

/test-all

/test-conformance

/test-networkpolicy

/test-e2e

/test-all

/test-networkpolicy  /test-windows-networkpolicy

/test-e2e

/test-e2e

@GraysonWu is having Internet issues today, so I am updating his PR. I opened https://github.com/antrea-io/antrea/pull/2121 to replace this one. We had to take care of an issue with meter support in the OVS kernel datapath. Turns out they require a kernel version >= 4.18.

Thanks. @antoninbas
Feel sorry about my internet issue at this critical time.


Using Background: in Cucumber scenarios with sauce_ruby starts two SauceLabs sessions per scenario

Hi
I noticed that when my Cucumber scenarios have Background: section then sauce_ruby/sauce-cucumber starts two Selenium browser sessions per scenario.
This might be related to the way how sauce-cucumber integrates with the Cucumber - using Before hook, and looks like this hook is called twice for every scenario when there is Background tag.
Also I see that Cucumber for runs Background step before starting scenario - it could be related to my problem:
https://github.com/cucumber/cucumber/issues/52
PS I'm not using "@selenium" tag in my scenarios but register Suace hooks manually in the features/support/hooks . I'm doing this so there will be no need to modify my features while I'm running them locally.
I'm thinking about
Before do |scenario|

  if ENV['SELENIUM_DRIVER'].nil?
    @browser = Selenium::WebDriver.for :firefox
    @local_browser = true
  else
    Sauce::Capybara::Cucumber.before_hook
    @browser = Capybara.current_session.driver.browser.driver
  end

end

if ENV['SELENIUM_DRIVER']
  Around do |scenario, block|
      Sauce::Capybara::Cucumber.around_hook(scenario, block)
  end
end

Just came across this issue while working on sauce labs.  Currently the background scenarios triggers a sauce session and the normal scenario triggers a new one which seems to fail. Is there a way to workaround the background scenarios?

Not currently;  The bug in Cucumber was patched for Cuke 2+ but not Cuke 1.  There might be a way I'm not sure of, and if so PRs are welcome!


Cato memes

I think I fixed the background on the catjam gif

Looks like it has a ghosting effect where the previous frames linger in the view now, can you try a different frame interpolation mode?


[Help] Auto-extract short descriptions from SDK methods

@JasonRShaver I think this applies across the board to all services, not just network.

This is a duplicate of item #365


Version of Tensorflow?

Hello,
Can you please clarify which version of Tensorflow is expected by Deepmedic? I was not able to find that noted in the documentation anywhere.
Thanks,
Chris

Hello,
Good question, sorry for missing this. I can't remember currently what's the minimum required. I ve been using 1.8 the last few months. I will try to find time to go backwards and check what's the minimum and update the docs.

Thanks, even just having 1.8 as a recommended version is helpful - appreciate it.

Added a note in readme under (Sec 1.1 Required libraries) about latest versions tested.
As of Febr 2019, v0.7.1 of DM was tested using:
Python 3.5.2
Tensorflow 1.12.0
nibabel 2.3.3
numpy 1.16.1
Thanks a lot, closing.


Please drop libcurl 7 builds

Same as https://github.com/conda-forge/tiledb-feedstock/issues/240, please remove the conda_build_config.yaml overrides here.
CC @jdblischak

This feedstock has no conda_build_config.yaml. We removed the extra openssl/libcurl variants back in February (#16).
I searched the repo for "libcurl", and I couldn't find any remaining trace of libcurl 7. All the YAML files in .ci_support/ have libcurl 8.
I apologize if I am missing something obvious. If there are additional steps I still need to take, please re-open this Issue.

Hm, apologies for the false alarm. I was convinced yesterday that I had seen this in this repo here; serves me right for not adding a reference to the commit or lines in question.


Background Color Changes after Final Install

What's the problem?
I have restarted, redownloaded, and followed the exact steps for installation. After it is completed, I am left with this screen. After seeing other people's installation, their background is normal. I downloaded Windows 10 using the link provided in the install instructions but still nothing has changed. I'm not sure what to do in this case.
My questions about possible issues:
Is it something regarding the windows product key? (although I have seen others do without and still not get this problem, I don't think that's it)
I am using VirtualBox so may it be an issue with that?
Is it a problem with the script or configuration for install?

Steps to Reproduce
I followed the instructions exactly:

Download Windows 10
Disable windows updates and virus and threat protection
Used the scrips provided for install
Everything finishes installing
Then the screen ends up like this

Environment
I am using the most updated version of virtual box for hypervisor
I am using the Windows 10 2022 Updated version
Additional Information
No response

This is from a tool to warn you that the internet is reachable. This may not be desirable in most malware analysis cases so is enabled by default now.
For details on disabling it if you don't want to be informed on internet connectivity I refer to one of the devs here.

@mr-tz I initially installed with internet so that it can install the tools needed from Google but plan on disconnecting once I set up my lab. Could you give me the steps to disable it if you know?

@mr-tz I was able to figure it out and your right it is just a tool to inform you if you have internet connection on. I had it installed before but it may have been an added feature recently. Thanks for the help!


Can implement the functions of ETH eip712, for the project dao management probability of use is very high?

Also to facilitate the migration of ETH contracts to polka

Can you please come up with a better description of what you want to do.


Add additional TFM do dependencies can be optimised

Is your feature request related to a problem? Please describe.
I want to minimise dependencies in my project by utilising framework dependencies wherever possible
Describe the solution you'd like
I want the package to have additional TFM'S which contains less dependencies. I.e. net core 3 & net 5, this will reduce maintenance effort.
Describe alternatives you've considered
Accept the additional dependency
Additional context
n/a

Which TFMs do you want to support?
I'm thinking to support new TFMs and it looks reasonable to support only net46 (or above), net6, net8 , netstandard2.0, and netstandard2.1. Are they accetable for  you?

What I would propose at a min is:

drop net standard 1.3
drop uap
Drop windows phone
add net 6
Add net standard 2.1


Can't bundle install when 'jekyll-related-posts' is listed in Gemfile

I'm trying to use the jekyll-related-posts plugin, but when I run bundle install with it listed in the Gemfile, I get this readout:
Bundler could not find compatible versions for gem "liquid":
  In snapshot (Gemfile.lock):
    liquid (= 4.0.0)

  In Gemfile:
    jekyll (= 3.8.3) was resolved to 3.8.3, which depends on
      liquid (~> 4.0)

    jekyll-related-posts was resolved to 0.1.2, which depends on
      liquid (~> 3.0)

Running `bundle update` will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

When I take it out, everything else installs fine. I'm on Jekyll 3.8.3.

exec bundle update It will be fine


node-sass added for sass compiling

Related Issue or bug
Fixes: #36
Describe the changes you've made

node-sass package added.

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots

Original
Updated

original screenshot
updated screenshot

@SankalpaFernando Are you working on this?


sqlplus does not exit after running build

The readme suggests making a task to run sqlplus to build/compile code.
Unless the script/file contains an "exit" statement, sqlplus will hang in the background and never complete, as described here: http://serverfault.com/questions/87035/run-oracle-sql-script-and-exit-from-sqlplus-exe-via-command-prompt
A solution that works better is to create a bat file, such as run_sqlplus.bat and place the following command inside it:
echo exit | echo show errors | sqlplus %1 %2
The build task definition then becomes:
 {
        "version": "0.1.0",

        // The command is a shell script
        "isShellCommand": true,

        // Run sqlplus via a batch file
        "command": "run_sqlplus.bat",

        "args": ["username/password@sid", "@\"${file}\""]
    }

This will run sqlplus, output any errors, and then exit cleanly back to VS Code.
It would be great if you could update the readme with this workaround/improvement.

Thank you for sharing this information. I'll include it in the readme...

fix in version 0.0.9

Another approach can be to add an "echo" command with the "sqlplus" command as a command argument as shown:
"tasks":[
        //Run sqlplus in args
        {
        "taskName":"sqlplus",
        "command":"echo",
        "args":["exit","|","echo","show","errors","|","sqlplus","username/password@sid","@\"${file}\""],
        "isShellCommand": true,
        "showOutput": "always"
        }
    ]


[zh-cn]  update containerd tab

at windows tab, it show linux config-path.

The preview looks good.
/lgtm


Adding sample, fixing minor issues and expanding Readme.

Adding a minimum sample file to actually run and test the code without having to download the entire dataset.
Removing generated polyiou.py file from the repo and added it to ignore list, because it would be changed by any user running a different version of SWIG.
Adding SWIG install instructions for MAC.
Fixing two issue in obb_anns.py:

Assuming a list of strings, but actually getting a string instead
Concatenating the subdirectory of the segmentation images twice, creating a path like sample/sample/segmentation/xyz_seg.png instead of sample/segmentation/xyz_seg.png

Thanks!


test: add anaconda-iso build tests with signed containers

Add anaconda-iso iso build tests with signed containers.
The rest of the images can be also added to the test once
[1] and [2] are merged
[1] https://github.com/osbuild/images/pull/990
[2] https://github.com/osbuild/osbuild/pull/1906
Signed-off-by: Miguel Martín<EMAIL_ADDRESS>

From user perspective, can we assume bib can now build out one anaconda-iso with signed bootc container image ?

From user perspective, can we assume bib can now build out one anaconda-iso with signed bootc container image ?

Yes, since #676 was merged

In order to validate feature related to this, assuming I had one signed container image,e.g:registry.redhat.io/rhel9/rhel-bootc:9.4. So do I have to execute sudo skopeo copy --remove-signatures registry.redhat.io/rhel9/rhel-bootc:9.4 registry.redhat.io/rhel9/rhel-bootc:9.4 before I call sudo podman run --rm -i --privileged --pull=newer --security-opt label=type:unconfined_t -v /var/lib/containers/storage:/var/lib/containers/storage quay.io/centos-bootc/bootc-image-builder:latest --type anaconda-iso --local registry.redhat.io/rhel9/rhel-bootc:9.4 ?

No, you don't need to copy anything, using the signed container directly should work as is. Not sure if the change is already available in quay.io/centos-bootc/bootc-image-builder:latest though.


ModuleNotFoundError: No module named 'speechbrain.pretrained'

Describe the bug
Have tested on pypi, git@main, and git@develop.
>>> import speechbrain.pretrained
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'speechbrain.pretrained'
>>>

Interestingly, this issue also arises when installing and running https://github.com/cvqluu/simple_diarizer, which uses speechbrain.
this is the whole speechbrain object when imported (minus builtins):
__cached__ :: /home/mlbot/.local/lib/python3.10/site-packages/speechbrain/__pycache__/__init__.cpython-310.pyc
__doc__ ::  Comprehensive speech processing toolkit

__file__ :: /home/mlbot/.local/lib/python3.10/site-packages/speechbrain/__init__.py
__loader__ :: <_frozen_importlib_external.SourceFileLoader object at 0x7a0b872e4fd0>
__name__ :: speechbrain
__package__ :: speechbrain
__path__ :: ['/home/mlbot/.local/lib/python3.10/site-packages/speechbrain']
__spec__ :: ModuleSpec(name='speechbrain', loader=<_frozen_importlib_external.SourceFileLoader object at 0x7a0b872e4fd0>, origin='/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/__init__.py', submodule_search_locations=['/home/mlbot/.local/lib/python3.10/site-packages/speechbrain'])
__version__ :: 1.0.0
alignment :: <module 'speechbrain.alignment' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/alignment/__init__.py'>
augment :: <module 'speechbrain.augment' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/augment/__init__.py'>
core :: <module 'speechbrain.core' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/core.py'>
create_experiment_directory :: <function create_experiment_directory at 0x7a0b872edbd0>
dataio :: <module 'speechbrain.dataio' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/dataio/__init__.py'>
decoders :: <module 'speechbrain.decoders' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/decoders/__init__.py'>
f :: <_io.TextIOWrapper name='/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/version.txt' mode='r' encoding='UTF-8'>
lm :: <module 'speechbrain.lm' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/lm/__init__.py'>
lobes :: <module 'speechbrain.lobes' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/lobes/__init__.py'>
nnet :: <module 'speechbrain.nnet' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/nnet/__init__.py'>
os :: <module 'os' from '/usr/lib/python3.10/os.py'>
parse_arguments :: <function parse_arguments at 0x7a0a29cf1630>
processing :: <module 'speechbrain.processing' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/processing/__init__.py'>
tokenizers :: <module 'speechbrain.tokenizers' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/tokenizers/__init__.py'>
utils :: <module 'speechbrain.utils' from '/home/mlbot/.local/lib/python3.10/site-packages/speechbrain/utils/__init__.py'>
version :: 1.0.0

Expected behaviour
speechbrain.pretrained ought to exist as per the intro docs:
from speechbrain.pretrained import EncoderDecoderASR

asr_model = EncoderDecoderASR.from_hparams(source="speechbrain/asr-conformer-transformerlm-librispeech", savedir="pretrained_models/asr-transformer-transformerlm-librispeech")
asr_model.transcribe_file("speechbrain/asr-conformer-transformerlm-librispeech/example.wav")

To Reproduce
No response
Environment Details
on Python 3.10.12
Relevant Log Output
No response
Additional Context
No response

maybe the version problem, the usage can refer to :
from speechbrain.inference.interfaces import foreign_class
asr_model = foreign_class(source="speechbrain/asr-wav2vec2-ctc-aishell",  pymodule_file="custom_interface.py", classname="CustomEncoderDecoderASR")
asr_model.transcribe_file("speechbrain/asr-wav2vec2-ctc-aishell/example.wav")

in version 1.0.0
here does not exist "pretained" package

This is expected due to the move to SpeechBrain 1.0 which introduced a breaking change on the inference API. See the update notes: https://colab.research.google.com/drive/1IEPfKRuvJRSjoxu22GZhb3czfVHsAy0s?usp=sharing
This should be a relatively straightforward change to update to the new API. Alternatively, if you must, you can pin your SB dependency to 0.5.16 in your environment, which should be the last version to have the pretrained module.

Maybe time to update the README. As someone trying to use this project for the first time, it's not straightforward to make the update as suggested. Thank you.

Yeah, the README is what got me here as well.  Having that out-of-step with the current version caused me a few hours of heisenbug-chasing, and I can imagine that's going to be a common experience.


Event listener with jrjc?

Hello,
I wanted to ask, is there any way we can integrate an event listener with jrjc? So that when a new issue is created in jira, it's logged/printed in application?
Please help.
Thankyou!

Hi @acyuta1,
I'm not sure if you're referring to any of our article.
Anyway, these kinds of questions are better to be asked on sites like stackoverflow.com or community.atlassian.com.
Here, we want to keep only the issues related to our codebase.


Case insensitive src blocks

This is a fix for https://github.com/eigenhombre/clj-org/issues/3#issue-487584476
It just makes source block tags case insensitive so that this works
#+begin_src
#+end_src

@garlic0x1 Thanks for the PR!  I reverse-PRed your branch with some unit tests for your change.  If you could accept and merge that, I'll merge this PR.  Thanks again.

@garlic0x1 Merging this now per my comments in your repo.  Thanks again!


Getting numbered list in PromptDialog.choice

hi
how can I have numbered list in PromptDialog.Choice in C# chat bot.
PromptStyle.PerLine, shows bulleted list like
•	Item 1
•	Item 2
•	Item 3
•	Item 4
I need  as

Item 1
Item 2
Item 3
Item 4

please suggest.

Hi @gouravbeyondBGCM,
The ability to have a numbered list is not a supported feature at this time. However, as the BotBuilder SDK is open source, you are free to make changes to the code to meet your specific needs.
The following code is an example of a custom PromptStyler that generates ordered lists. This addition would live in the PromptDialog.cs file of the SDK.
As I believe this should solve your question I am going to close this issue. If you have further related questions, you can reopen.
Hope of help!
Steve.
[Serializable]
        public class OrderedListPromptStyler : PromptStyler
        {
            public override void Apply<T>(ref Message message, string prompt, IList<T> options)
            {
                var builder = new StringBuilder();
                var pos = 0;
                foreach (var elt in options)
                {
                    builder.Append($"{++pos}. {elt.ToString()}\n");
                }
                message.Text = $"{prompt}\n{builder.ToString()}";
            }
        }

hi
I can see the file , but since its open source , how can I download the complete library code here to change the same , as you mentioned, please suggest .


Allow an easy enabling/disabling of synchronizers

Description
According to the demo, and having had a look at the code, I think there is no easy way to disable a Synchronizer without manually removing viewers.
In my case, I handle viewers on the go, removing them, adding them to the view, etc. The trouble is we have a "Synchronize" button that toggles the synchronization between viewers.
Instead of removing all viewers and adding them all back, and registering new viewer only if the Synchronized mode is active, it would be way easier to add them systematically to the synchronizers and toggling their status from disabled to enabled:
panZoomSynchronizer.enabled = false;
If you think that's a valid idea and/or that there are no alternative, I'd be happy to submit a PR.

The solution for this was merged, so I will close the issue.


Collapse/expand log if too much lines

Hello,
Thanks for creating logtrail, is great and useful.
Just an enhancement if possible: sometime we have logs with lot of lines in a single message and it just occupy whole viewport. So would be good if we can have a feature to show max of 3~5 lines per message, then can expand it if needed

this would be useful for me too.
I config filebeat to join multi-line for java stacktrace, or some api resonse
sometimes there are so many multi-line log, and make my chrome relly stuck, even worse the page just crash

It is possible to use css temporarily like below.
`
.message {
width: 1500px;
white-space: nowrap;
text-overflow: ellipsis;
overflow: hidden;
}
.message:hover {
overflow: visible;
white-space: normal;
}
`


File watching issues

Fast file watching is hard, a file change event just means a write has started, node based tools like to stream in chunks with great delays between writes. Like watchify will write an empty file and idle for anywhere between 25-100ms (or greater but that's the average write times I've been seeing from file change to compilation being done) before writing new chunks.
Using the API's of the supported compilers is not a solution, because the issue is the same with the --watch feature which is meant to watch external files of any type, so any compilation pipeline will see the same problem. You'll see this in the wild with for example the beefy browserify server, where it will serve empty bundles sometimes, experienced the same with live reload in the past.
The clean solution is locking, but with unix locks being advisory and not even in node core or libuv, It's not an easy one.

For now, just use<EMAIL_ADDRESS>as it did atomic writes compared to the multiple write behaviour seen in 3.x
Upstream issue https://github.com/substack/watchify/issues/239

Have to close this as non actionable, the best I can do is write a ranty blog post about the topic, which I will do but at a later date.
The fundamental problem is basically nodejs tools like to stream things, you can't have a watch mode and stream writes, It just wont work when there is no file acquisition. Anyone watching the file will see multiple writes.
I've also left the watchify issue hanging for quite some time since 2.x works fine. Writing amokify helped me understand the compilation pipeline and i know how to address it.


Create jira_transition parameter for JiraAlerter

To allow for customizing in which status the new issues created by JiraAlerter will be in we should add jira_transition parameter to the alerter configuration options.
It will allow to specify which transition the issue will go through after being created.
It could be either a single entry or a list of transitions in a proper order.
Here are some hints how to do it: https://answers.atlassian.com/questions/232622/is-it-possible-to-change-jira-issue-status-with-python-jira

Hi, I have added this on my own fork and would like to contribute it.  Shall I just PR it?

Yes

Sorry for the delay, submitted #1668 today.


"Press Enter to Reconnect" is not appropriate when using --once

Describe the bug
In previous versions, the message that appears on exit made more sense. It used to be "Connection Closed" - which was more appropriate for my use cases.
I think it should be reverted to this message, if not for all cases, at least when running with --once.
To Reproduce

Run ttyd --once --port 3000 --readonly  echo PONG
Open the browser at http://localhost:3000
See PONG followed by "Press Enter to Reconnect"

Expected behavior
See "Connection Closed" as the exit message.
Environment:

OS: Ubuntu 20.04 LTS
Browser: Chrome

Thanks, will change the message for --once for the next release.


Fixes motivated by extensive linting

This PR applies fixes of linting issues which become visible by the introduction of more extensive linting in PR #684.
Most issues are reported by pylint.

Attached the issues reported by pylint.
pylint.txt

I am merging these relatively small changes into PR #684. We can discuss additional changes required there.


Admin - Logout

Endpoint: /api/admin/logout
Method: POST
Description: Logs out admin users by invalidating their authentication token.
Authorization Required: Yes
Feedback:
Success: "You have been logged out successfully."
Failure: "Logout failed. Please try again."

I will work on the logout

#99 https://www.loom.com/share/3cf0318a1ac2486fb78946271f45f4a4?sid=c98f7580-c8bd-4c7a-8f7e-e9d1fa4c2024

@ejebulley Where is a PR link and add the review label if ready to be reviewed


You should support Windows

You have already supported Linux and Mac OS already...
now it's time to support windows...

No windows, linux only.

Dup to #2532 We're working on feature/windows


Disabling Auth

I have pulled the latest version of Caddy and the file manager plugin. Is there a way to disable the new auth features via Caddy directives? I have no desire for them (I front Caddy with an authenticating proxy already).

I'll think about creating an option for that. Right now it is not possible in any way.

Hello @martinbaillie! Now available on caddyserver.com. Just add the option [no_auth] to the Caddyfile 😄

Works perfectly @hacdias. Appreciate your quick turnaround on this and thanks for the great plugin!👍

NP 😄 I'm the one who should thank you.


Remove GOV.UK PaaS

As PaaS is being decommissioned we shouldn't be recommending it any more.
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

Change looks good to me. Are you part of the team on CDDO that owns the content? If not I assume we should get a quick tick from them that they're happy with this.

there is no owner on the in CDDO and this change was give green light by Lisa Scott, PaaS Product owner

Ok cool - thanks. I'll get it merged.


Fatal error: Unable to parse version string

Fatal error: Unable to parse version string when packaging app for platform win32 ia32 using electron v1.2.3 on Windows 8.1 Pro.

Which version of electron-packager are you using?

7.0.4

What CLI arguments are you passing? Alternatively, if you are using the API, what parameters are
you passing to the packager() function?

electron-packager . --out=dist --app-version=$npm_package_version --prune --asar --overwrite --all

What version of Electron are you building with?

1.2.3

What is the host platform are you running electron-packager on?

Windows 8.1 Pro x64

What target platform(s)/architecture(s) are you building for?

--all

Is there a stack trace in the error message you're seeing?

PS C:\Users\simon.roth\Downloads\electron-boilerplate-master\boilerplate> npm run build

><EMAIL_ADDRESS>build C:\Users\simon.roth\Downloads\electron-boilerplate-master\boilerplate
> electron-packager . --out=dist --app-version=$npm_package_version --prune --asar --overwrite --all

Packaging app for platform linux ia32 using electron v1.2.3
Packaging app for platform win32 ia32 using electron v1.2.3
Fatal error: Unable to parse version string
 undefined

npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run
" "build"
npm ERR! node v4.4.5
npm ERR! npm  v2.15.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: `electron-packager . --out=dist --app-version=$npm_package_version --prune --asar --overwrite
--all`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'electron-packager . --out=dist --app-version=$npm_package_version --prune
 --asar --overwrite --all'.
npm ERR! This is most likely a problem with the app package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     electron-packager . --out=dist --app-version=$npm_package_version --prune --asar --overwrite --all
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs app
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!
npm ERR!     npm owner ls app
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\simon.roth\Downloads\electron-boilerplate-master\boilerplate\npm-debug.log

Please provide either a failing testcase or detailed steps to reproduce your problem.

Can be reproduced by building https://github.com/sindresorhus/electron-boilerplate.

Possibly related to https://github.com/npm/npm/issues/8574.

Try changing the version in package.json from 0.0.0 to 0.0.1.

I am getting the same error when building for win32 on a mac.
electron-packager build/ gifbar --platform=win32 --arch=x64 --overwrite --tmpdir=false --out=releases/ --app-version=0.0.2
I do not get the error if I remove the --app-version param.

I can't reproduce either error.

Host platform/arch: Linux x86_64
Target platform/arch: Windows x64
electron-packager version: 7.0.4
Electron version: 1.2.3
Wine version: 1.8.2 (32-bit)
App versions tried: 0.0.0, 0.0.1, 0.0.2

This error also doesn't occur in our CI tests, which we have for all three platforms.
The error message comes from rcedit. The definition of the version string parser indicates that it should work.
@simonroth does the error occur when you build for Windows x64? Are you sure $npm_package_version has a value in it?
@adamweeks Perhaps make sure that your version only contains ASCII? I'm at a loss as to what else to recommend trying here. (I'm assuming you're getting the exact same Unable to parse version string error.)

Good morning guys. It turned out that the error occures due to different variable substitution on Windows (see https://github.com/npm/npm/issues/8574 and http://blog.keithcirkel.co.uk/how-to-use-npm-as-a-build-tool/#thewindowsproblem). When changing
$npm_package_version to %npm_package_version%
the build passes without errors. It seems that it is not possible to run a build script using variable substitution cross plattform at the moment.

I'm closing this issue then, as the original problem has to do with NPM and not electron-packager.

Okay, thanks.


chore: remove deprecated set-output command.

As per documentation the set-output command has been deprecated and will be removed sooner than later.

This is the action run against this branch: https://github.com/replicatedhq/kURL/actions/runs/4978270474


[go_router] routes parameter could accept an Iterable<GoRoute> instead of forcing a List<GoRoute>

Use case
Right now, the GoRouter and GoRoute classes have a parameter List<GoRoute> routes. The order of the routes in the list doesn't matter :
From the doc

Only a single route can be matched, specifically the one whose path matches the entire location (and so it doesn't matter in which order you list your routes).

So it would make more sense to use a Set<GoRoute> in my opinion. We could still support the List<GoRoute> by only requesting an Iterable<GoRoute>

Proposal
The routes parameter could be an Iterable<GoRoute> instead of List<GoRoute>.

Hello @ValentinVignal. Your proposal is that it would make sense for routes to be a Set since only one location would match. But to be backward-compatible, you're suggesting to use an Iterable. But using an Interable wouldn't convey the information that a Set would and, I think most, users would still opt for a List.

But using an Interable wouldn't convey the information that a Set would and

I'm not sure what you mean by that. Do you mean that users won't be able to know they can use a Set?

I think most, users would still opt for a List.

I don't know about that, maybe you are right. But with Iterable, at least the few users that want to use a Set will be able to do so.

Do you mean that users won't be able to know they can use a Set?
No I meant the semantics of having a Set. With a Set, it's clear that this parameter's order doesn't matter and that duplicates aren't allowed. While Iterable doesn't have the same semantics.

Anyways, I will keep this feature request open for the team to bring some insights to. Thanks for filing it :)

I think using of Iterable is useless because it brings nothing except disadvantages. For example if someone need to do some List related actions he will need to convert the Iterable to the List (why if we have this already?). Using Set make sense because this defends progremmer from route duplicates.

For example if someone need to do some List related actions he will need to convert the Iterable to the List

If the order of the routes doesn't matter, why would someone need to do some List related actions?

Using Set make sense because this defends progremmer from route duplicates.

The issue I have with using a Set is that it is a breaking change.

The order does matter if you have multiple matches, it will always return the first match.

The order does matter if you have multiple matches, it will always return the first match.

Then, shouldn't the doc be updated  Because "Only a single route can be matched, specifically the one whose path matches the entire location (and so it doesn't matter in which order you list your routes)." says the opposite

It appears we forgot to update the doc

closing this as the documentation update will be part of the gorouter.dev migration


foregroundPresentationOptions having no effect

I'm trying to prevent notifications from showing when my app is in the foreground, it seems like foregroundPresentationOptions is the right thing for this but it doesn't seem to have an effect.
My code:
notifee.displayNotification({
      title: 'Hello',
      subtitle: `Notification`,
      ios: {
        foregroundPresentationOptions: {
          badge: false,
          banner: false,
          list: false,
          sound: false,
        },
      },
    });

I've done some investigation and adding alert: false to foregroundPresentationOptions prevents this. I'm on @notifee/react-native version 5.6.0. The docs suggest badge and banner should suffice to do this and I shouldn't need alert. Maybe something has been missed in https://github.com/invertase/notifee/commit/6bf2765392406b8b35d2778907193955664fb51d

Further to this I have noticed that the options passed to requestPermission also seem to be totally ignored.

Hello Sir, Can you assign this issue to me ?

Hey @Krish5g - no need to do assignments or anything, if you'd like to work on anything in the repo just check the repo out, and post PRs, we are always happy to collaborate with people


Re-order resource types

Reference:
https://github.com/OutpostUniverse/OPHD/pull/983#discussion_r654735980
ResourceNamesRefined[0],
ResourceNamesRefined[2],
ResourceNamesRefined[1],
ResourceNamesRefined[3],

The index order 0, 2, 1, 3, is strange, confusing, and unexpected. I believe I've noticed this in a few places now.
If we typically need to use resources out of natural index order, perhaps we should re-order what the indexes mean?

I would agree if it's a typical use case. I've been using it as Common Metals/Common Minerals/Rare Metals/Rare Minerals but display may show Common/Rare Metals, Common/Rare Minerals.
I'm not opposed to changing it if that would make more sense.


New scrollableTimeRangeForDate function taking long time to execute

Hello, Thanks for your great Calendar library. In the one of the latest release I notices that there is lag when moving between dates. Using profile I came to know that this function is taking long time to return answer. The old way use  Date for the calculation while the new function use the Calendar to do the job. But the Calendar way involve many other time consuming tasks. Can you check please?
- (MGCDateRange*)scrollableTimeRangeForDate:(NSDate*)date

N/A


pip3.exe install -r requirements_pt2.txt error

when i run this command in windows 11
$ pip3.exe install -r requirements_pt2.txt
Obtaining taming-transformers from git+https://github.com/CompVis/taming-transformers.git@master#egg=taming-transformers (from -r requirements_pt2.txt (line 38))
......
ERROR: Ignored the following versions that require a different python version: 0.55.2 Requires-Python <3.5; 1.6.2 Requires-Python >=3.7,<3.10; 1.6.3 Requires-Python >=3.7,<3.10; 1.7.0 Requires-Python >=3.7,<3.10; 1.7.1 Requires-Python >=3.7,<3.10
ERROR: Could not find a version that satisfies the requirement triton==2.0.0 (from versions: none)
ERROR: No matching distribution found for triton==2.0.0
(.pt2)
Tierney@BDM /d/Github/generative-models (main)
$ python -V
Python 3.10.6
(.pt2)
Tierney@BDM /d/Github/generative-models (main)
$ pip -V
pip 23.2 from D:\Github\generative-models.pt2\lib\site-packages\pip (python 3.10)
(.pt2)
Tierney@BDM /d/Github/generative-models (main)
$ ^C
(.pt2)

Remove the triton==2.0.0 line and try again. It's not strictly necessary.
The requirements files contain lots of extraneous fluff anyway that's not necessary; if some of my currently open PRs get merged, you could use just a handful of requirements to get inference going.


deleteWhere method modifies the model to be instance of Query builder and not instance of Eloquent

The applyConditions method called in deleteWhere does this.
Type error: Argument 2 passed to Prettus\Repository\Events\RepositoryEventBase::__construct() must be an instance of Illuminate\Database\Eloquent\Model, instance of Illuminate\Database\Eloquent\Builder given, called in /var/www/nhanhviec_admin/vendor/prettus/l5-repository/src/Prettus/Repository/Eloquent/BaseRepository.php on line 587

Please update composer package: prettus/l5-repository

Sorry, I fixed it by running the command
composer update


Removes extra braket from README example

There was an extra closing bracket in the code example for the options example for the runit_sevice block.  Wasn't sure styleguide-wise which bracket you preferred to keep, so I went with the one I thought looked best.
Not sure if I am following the contributing guidelines correctly for this project, but it also seems like it might be copy-pasta from another "Chef CORP Proper" cookbook, so I just went with a simple github PR.  Feel free to close and implement this minor fix in a different release as you see fit.
Thanks for the great cookbook!

Thanks Nick. :+1: on the contributing process too.


Axonaraw

[ ] depends on https://github.com/catalystneuro/nwb-conversion-tools/pull/227

Here we add the following:

a hussaininwbconverter module with various nwb converters for combinations of raw, unit, eeg/egf and position data.
a tutorial notebook on how to use them: Axona_to_nwb_conversion.ipynb.

Missing: Intanconverter. In principle, an intandatainterface already exists, but it currently does not pass the metadata validation: https://github.com/catalystneuro/nwb-conversion-tools/blob/master/nwb_conversion_tools/datainterfaces/ecephys/intan/intandatainterface.py

You can just create one or two NWBConverter objects. They can omit fields by just not entering the source location for that data


Validate create requests against tenants available configurations

@Mats-SX thanks for your review! I take the closest matching region now. You are right, we dont need to let the user decide, we can make a good decision here.


Minor Python cleanup

Nonfunctional change. Rename reportError.js into util.js and move simpleRunPython there. Also run formatter.

Thanks for the review @fhanau!


Support for System property of HADOOP_HOME in windows

I tried to run mini yarn cluster but while starting i got followed exception:
java.lang.UnsatisfiedLinkError:
Can't load library: C:\develop\myfolder\..\.\windows_libs\null\lib\hadoop.dll

although that i have my HADOOP_HOME setup correctly.

The "null" part of the path would indicate that you don't have a profile version set. What is HADOOP_HOME set to?

Created pull request for this issue

Thanks @NoRights - the fix makes sense. I'll keep an eye on the CI runs and will merge it when those wrap up.

Thanks for the PR @NoRights ! It has been merged.


(Minor typo) An address can control 255 Sub-Accounts instead of 256

Was great fun getting to understand this repo. Was wondering how can I contribute.
Thanks

it should be 256, because the term subAccount here actually includes the caller's address (if that make sense)

it should be 256, because the term subAccount here actually includes the caller's address (if that make sense)

Got it,makes sense then.
I thought the emphasis was on the "control" because a user doesn't actually owns the other 255 addresses.

How would you rephrase the sentence so it makes more sense that covers both the total accounts under control, and the "255 others"?
Would love your suggestion here!

Sorry, I think I am overthinking here :)  What's written works 👍👍

Closing now ;) thanks for the feedback anyway


New inverted monochrome menubar icon not white

Prerequisites

[x] I'm using latest version: v1.2.0
[x] I've checked Known issues
[x] I've checked Open issues

Description
Since the logo change, the inverted monochrome menubar icon is no longer white and thus does not match the system design (at least in Gnome 3 and Windows 10 dark theme).  Here are the old and new icons for both systems:

OS
Old icon (white)
New icon (grey)

Ubuntu

Windows

Operating system: Ubuntu 20.04 and Windows 10
Steps to Reproduce

Install stretchly
Change settings: “Theme” → “Menubar icon” to “Monochrome” and tick “Inverted Monochrome”
(On Windows: change to dark theme)

Expected behavior: white icon that “blends in” with system icons (like before)
Actual behavior: grey icon, distinct from system icons
Reproduces how often: always

Hi @csemken and thanks for feedback, but I don't plan to change that.

Actually, I'll keep this opened for a while and see what other people think.

Thanks, I appreciate it.  Any particular reason why you were not planning to change this?  If it is about the extra work, would you accept a PR that makes graphics/appicon-mono-dark-mode{,paused}.svg white?

Hi @csemken ,
it was done by designer and I like it ;) But other reasons as well:

in my experience with these things is once I change it, people start asking to bring it back. It's hard to please everybody and not every OS has same colors.
I never really liked the white one pre 1.0, seems to be a bit off
so it looks to me that it should not be white, but white-ish, if changed. I would like to do some research how different OSes/Distros/Apps deal with such a theme. Then we also need to look at dark one. MacOS is changing color automagically (from dark to light), so I'll try to check how it looks there once I have a chance.

So yeah, if we can get some data, I am willing to change my mind :)

Hi, I will close this as not likely to happen.


Updated RevNum.java in Interface section

@KUSHAGRA-JAISWAL make sure your main branch is up to date with the base branch, then open another PR.


Validation error: : Error loading /root/.fhir/packages/hl7.fhir.r4b.core#4.3.0/package/StructureDefinition-Appointment.json: Unknown FHIRVersion code '4.3.0' on line 1. Jump to error.

.

I've been having the same issue as of today.

Can't get past this.  Unable to validate resources in unrelated packages.

Just tested and am no longer getting this error.
Per this zulip thread, restarting the validator resolved the issue.
From that same thread, this appears to be caused by the following:
US Core Test Kit shares Inferno validator service with Inferno validator app. So if someone upload new FHIR R4B IG, then Inferno validator service could not identifies that version. We are considering spin off a separate service for US Core test kit. Also we will update Inferno validator service to recognized FHIR version 4.3.0.


Fix DataDecodedParameter property name

DataDecodedParameter.paramType is now DataDecodedParameter.type as it should be according to the specification

Coverage: 85.599% (+0.1%) from 85.483% when pulling 46b48bc0bc9db681e7be013007f9c04f23f27cc4 on data-decoded-parameter-type into 36b109db04e3cba2565096a68c7598b428474337 on main.


Heroku/Self-Hosted version?

Is there a way to run this self-hosted or through a free service like Heroku?

Yes, the readme describes how to set this up through Google Cloud Functions. That's free in practice given that the number of invocations for these functions will be way below the billing threshold.


Bird profiles to csv

@stijnvanhoey @plieper @adokter rather than to stumble my way around R, I'd rather ask first: have you already done a data transformation similar to this:
Input vp data
radar_1 / time_1 / dens (can be retrieved from attributes I assume):
0      200      400      ...      4800
NA 35.59053 41.27831 ...    NA

radar_1 / time_1 / u:
0        200        400        ...        4800
NA  -9.421702  -8.746634  ...

This is what readvp function does in bioRad:
library(bioRad)
myvp=readvp(yourfile.h5)

To access the table above:
myvp$data

However I try to avoid people accessing the data structure directly, and have them use the fetch function to extract particular fields, e.g. for the bird density 'dens':
fetch(vp, quantity = "dens")

This is because of various thresholds applied to the bird density column, in particular the radar cross section and radial velocity standard deviation thresholds that are applied in a 'posthoc' way. Users may change these in bioRad, and then the bird density column 'dens' in this table gets recomputed from column 'eta'.

Thanks @adokter
I was indeed using fetch[^1] before, but that way I only access one variable at a time, with altitude bins as columns (and the variable as the row). With myvp$data I indeed get it all at once in a data.frame, with the altitude bins as rows and the variables as columns, which is perfect. Since I'm just transforming (i.e. restructuring) data, I think it's fine to use that method rather than fetch.
Next step is to search in myvp$attributes to get the radar name and timestamp, cause I'll need that in my table too.
[^1]: which might have a bug for vplist #4


New package: MUMPS v1.0.0

Registering package: MUMPS
Repository: https://github.com/JuliaSmoothOptimizers/MUMPS.jl
Created by: @dpo
Version: v1.0.0
Commit: e1aead08bca12d92c82615a9b57c5ed8f2d92bcf
Reviewed by: @dpo
Reference: https://github.com/JuliaSmoothOptimizers/MUMPS.jl/commit/e1aead08bca12d92c82615a9b57c5ed8f2d92bcf#commitcomment-34201290

This is a different package from the one that was just renamed to MUMPSjInv?

It is.

Is this supposed to merge with JuliaSparse/MUMPS.jl?

No, it's a different package.

Is this supposed to merge with JuliaSparse/MUMPS.jl?

JuliaSparse/MUMPS.jl has moved (#1798), discussion regarding the change is here (https://github.com/JuliaInv/MUMPSjInv.jl/issues/21) and here (#958).

Ok, here's our first name reclamation in the post-Pkg3 world. Let's see how this goes...


Possibility to provide custom layout of a camera scanner

Hi there! Ralf can you please add some opportunity for user to fill barcode in edittext ? It can be some cases when zxing can't read the barcode. So if we will have option to add barcode and confirm it manually it will be perfect.
Thanks in advance!

@Antonkog Adding direct support for EditText is outside the scope of this library. However, from version 3.0 you can create your own Activity with custom layout and logic, which should cover your use case.
I sent you an email with more details.

@rkistner How can I custom my own layout and logic inside a fragment ?

@tuandd6585 This example is probably the best starting point: https://github.com/journeyapps/zxing-android-embedded/blob/master/sample/src/main/java/example/zxing/ContinuousCaptureActivity.java
It doesn't use fragments, but that shouldn't make any significant difference.