Support for enabling and disabling Valet mode

Would be awesome to add support for valet mode for many reasons. One of the main ones in my mind is that it can act as a deterrent and/or slow down car thieves trying to disable Bluelink as that's one of the first things they do; I speak from first hand experience when my IONIQ 5 was stolen.
I only have an EU account so cannot speak for other regions and their capabilities.
Turning on valet mode - it seems to take a good few seconds to actually complete the command:
POST /api/v2/spa/vehicles/{id}/control/valet
Post body:
{"action":"activate"}
Turning off valet mode - looks like this can only be done via Bluelink and not via the car:
POST /api/v2/spa/vehicles/{id}/control/valet
POST body:
{"action":"deactivate"}

Since you provided the traffic should be straightforward.
Would you be willing to test or contribute adding this?

Happy to test (I'll have a read of the code to try and make sense of how the different features are implemented but I have a feeling you'll get there a lot faster than me 😅)
Also, if you need me to send proxied requests/response for any other pressing issues, I am more than happy to do so (EU / Hyundai) - feel free to tag me in any issues and I'll help where I can.

I'm not by a computer for a few days.
It would need to be added here and vehicle manager for testing.
https://github.com/Hyundai-Kia-Connect/hyundai_kia_connect_api/blob/master/hyundai_kia_connect_api/ApiImplType1.py
Once that tests out can be added to the integration.

Tested on my vehicle. @emtunc if you can also test it, it would be great!
Thanks for the capture btw

Whoa, that was fast @asychow 🤯
Are you testing via HA or another way? Can you point me to any docs or method I should be using to test unreleased versions? Am I essentially git clone replacing the directory that exists within HA?

from hyundai_kia_connect_api import *

vm = VehicleManager(region=1, brand=2, username="<your email>", password="<your password>", pin="<your pin>")
vm.check_and_refresh_token()
vm.update_all_vehicles_with_cached_state()
print(vm.vehicles)

vm.start_valet_mode(next(iter(vm.vehicles)))

Here is the test script I use. If the command is successful you will see the following in the console
{'retCode': 'S', 'resCode': '0000', 'msgId': ''}
To test in HA, we will also need some changes in kia_uvo repo.

Okay cool - simple enough. Only problem is I've hit the rate limit :(
Will update when it's reset.

Just tested - works 👍


Add packages browser badge to README

Signed-off-by: Ettore Di Giacinto<EMAIL_ADDRESS>

no need to wait for tests @mudler ?

right ! also because workflow won't trigger for README changes..


Implement explicit snapshots

See https://github.com/Level/community/issues/118. Depends on https://github.com/Level/supports/pull/32.

Remaining issues to solve or postpone:

Does not support deferred open (db.open() is required before db.snapshot()). I'm okay with that.
Sublevel behavior is not symmetric. Normally a db.close() closes the snapshot too but this isn't the case on sublevel.close() because the snapshot belongs to the parent database (even if created via sublevel.snapshot()).
Snapshots are protected from a premature close by waiting for inflight read operations. But, this being done in JS, doesn't help the case of a context teardown in classic-level. So perhaps I should remove it, and handle it natively (in the same way that iterators are tracked and closed).

Sublevel behavior is not symmetric.

Fixed.


Can I understand it this way? I only need to modify files seed_idea.json, template.tex to generate research for my research field?

Or do I have to modify the entire project from start to finish?

same question

We have the full list of files you need to modify in: https://github.com/SakanaAI/AI-Scientist?tab=readme-ov-file#making-your-own-template


Non-standard Index size

Graph can only be created w/ the standard DefaultIx size (u32). It would be nice if it was generic over different index sizes so it can support any petgraph stable graph. I can help make this change.

Thanks for the feedback. Sounds good to me. Please file a PR. I am glad to new contributions which expand number of usecases for the project.

https://github.com/blitzarx1/egui_graphs/pull/103

Thank you for the contribution. Waiting for https://github.com/blitzarx1/egui_graphs/issues/100 and packing new release!


Build error this undefined symbol...

Hello!
I have an issue with compile to simulator from main branch:

In SwiftGit/Package.swift, replace the package url of clibgit2 to https://github.com/light-tech/Clibgit2. Our fork uses a arm64 architecture for M1 machines.

Ah many thanks!

Updated Clibgit2 to a universal binary. Building for x86-64 simulator should work now.


[EngSys] Validate Changelog before merge instead of during release

After validating a release I was met with this error:
##[error]The changelog entry has the following sections with no content (Bugs Fixed, Breaking Changes, Other Changes). Please ensure to either remove the empty sections or add content to the section.
We should either move this check as-is into the PR CI gates or if we want to be really fancy, make it happen when we go from Unreleased - > a real date.

@praveenkuttappan This probably affects other languages as well. If so, please move it to the https://github.com/Azure/azure-sdk-tools repo

I think we should do this check when release date is set. Also moving this to azure-sdk-tools repo since this is applicable for all languages and change log check is done by common code.


Conditional order?

Is there a way to allow for queries with a conditional order?
Example (does not compile):
let order = match order_string {
  "id" => id,
  "name" => name,
  "age" => age,
  _ => id
};
let results = users.order(order).load::<User>(&connection);

Do we really have to duplicate like this?
let results = match order_string {
  "id" => users.order(id).load::<User>(&connection),
  "name" => users.order(name).load::<User>(&connection),
  "age" => users.order(age).load::<User>(&connection)
  _ => users.order(id).load::<User>(&connection)
};

You'll need to box either the order or the whole query. See the examples in the docs here and here


What's the difference of LocalCachedMap between open source version and pro version?

As it said only pro version support ultra-fast read/write, what does it mean？

Another Redis commands processing engine is used

In the same usage scenario, how many times is the performance of the Pro version compared to the open source version? The open source version does indeed use local caching to accelerate, right.

how many times is the performance of the Pro version compared to the open source version?
2 or 4. depending on the use case.

The open source version does indeed use local caching to accelerate, right.
Right, but Redis commands processing acceleration isn't available in the open-source version.


feat(frontend): Token grouping visuals

Motivation
Display grouped tokens in the list with new visuals.
Changes

Add token group view
Trigger token grouping
Fix styling based on main changes / merge conflicts

Tests
No additional tests

WORK IN PROGRESS
This PR is being superseded by PRs:

#3030
#3040
#3041
#3043
#3052
#3053
#3054
#3062
#3079
#3086

@darioloetsc as mentioned in comment above, I split it in a few PRs that superseded this one. if you want, you can close this branch


Prioritize clang and GCC in svt_av1_print_version

When building on Windows _MSC_VER is always defined no matter what compiler is used, by swapping a few lines to prioritize clang or GCC, the actual compiler and version will be shown when using on of these compilers.
Realistically this should probably be changed or expanded because whatever Visual Studio version was used isn't very descriptive or helpful in any way, but this now would only happen when compiling with MSVC.

LGTM.


Failed to compile after install eslint locally (propTypes is not defined)

If you are reporting a bug, please fill in below. Otherwise feel free to remove this template entirely.
Can you reproduce the problem with latest npm?
Yes, on OSX Sierra.
Description
create-react-app test
cd test/
yarn add eslint
yarn start

Expected behavior
Should compile successfully.
Actual behavior
yarn start failed to compile, showing error: propTypes is not defined

Environment
Run these commands in the project folder and fill in their results:

npm ls react-scripts (if you haven’t ejected):

<EMAIL_ADDRESS>/Users/dinhquangtrung/Desktop/test
└──<EMAIL_ADDRESS>

node -v:

v8.1.0

npm -v:

5.0.3

Then, specify:

Operating system: OSX Sierra
Browser and version: (any)

Reproducible Demo
Repo: https://github.com/trungdq88/create-react-app-issue
Travis Build: https://travis-ci.org/trungdq88/create-react-app-issue/builds/242429125

Seem to only happen if installed eslint 4.x. I tried with eslint 3.x and in worked. But we still need to fix this since eslint should work "inside the box", isn't it?

@trungdq88 React removed PropTypes since version 15.5, see here
you can use prop-types package for it

@trungdq88 React removed PropTypes since version 15.5, see here
you can use prop-types package for it.

The problem seem to be caused by eslint-loader used the incompatible version of eslint installed outside of react-scripts (which is 4.x in my case).
The solution IMO is to config eslint-loader to use the eslint instance inside react-scripts.
Unfortunately eslint-loader have no such config, so I have submitted this: https://github.com/MoOx/eslint-loader/pull/181

Seem to only happen if installed eslint 4.x.

If you mean that you installed eslint locally in your project, this is not really supported. I know that people do this, but we don't recommend it.
You don't need to install it locally, it will work out of the box. Installing it locally can mess up the versions, which is what probably happened.

@gaearon is this something worth fixing? I am happy to submit a PR.
The plan should be:

Wait for eslint-loader to add eslintPath option in the next release.
Add eslintPath option to eslint-loader in webpack.config.dev.js point to react-scripts/node_modules/eslint/bin/eslint.js

@trungdq88 In general, yes, what you suggest sounds reasonable. I'm a bit surprised it doesn't already work like that. I'd expect eslint-loader to use Node resolution mechanism and find the right eslint.

@gaearon This build says that eslint-loader is using<EMAIL_ADDRESS>instead of the eslint packed with react-scripts, and eslint-loader is not compatible with<EMAIL_ADDRESS>That's the root cause of the bug.

Can you diagnose why eslint-loader locates the wrong copy of eslint despite correct dependency from react-scripts? It's not obvious to me.

I am not sure about that. I tried to install eslint 3.x and then eslint 4.x locally in my project.

The build fail if there is eslint 4.x in package.json.
The build success if there iseslint 3.x in package.json.
I found out that eslint-loader is not working with eslint 4.x.

That's how I come up with the conclusion.

I understand. I am encouraging you to go down the rabbit hole and figure out why it happens. :-)

Thanks, I'll do more investigate.
It's seem weird to me that in both scenarios I had to nuke the node_modules with rm -rf node_module && yarn install && yarn start to reproduce the bug. Or else the results are totally unpredictable.
Anyway, are there test cases in this repo? I can't find them anywhere 😕

It seems like this is a yarn issue as much as a create-react-app one. I am also seeing something similar. Not exactly the same, because if you fix one of these errors and run yarn start or yarn run build again, you most likely get some more errors to fix. It goes on and on, based on some default settings that eslint seems to be incorrectly using. I went from zero warnings/errors, via yarn upgrade-interactive, to 18,700 errors, once it got as far as checking indentation levels. It also does not transpile ES7 features such as class static proptypes, presumably because eslint is falling back to some default configuration, so they are just syntax errors as far as eslint is concerned.
I have tried it with npm and npm5 and although it takes much longer, it does appear to work 100% reliably, whereas yarn fails 100% of the time. So it's something about the way yarn is building the dependency tree and satisfying dependencies that's ending up with a weird inconsistency with eslint 3/4.

I got that same error on a state variable defined at class level. I guess it is the same issue, isn't it ?
export default class MyComponent extends Component {
  state = {
    open: true
  }

Failed to compile.

./src/components/MultiSelect/index.js
  Line 7:  'state' is not defined  no-undef

The OP issue looks like this Yarn bug: https://github.com/yarnpkg/yarn/issues/3535. I'll close in favor of it, but if you can still reproduce this after Yarn releases 0.25, please create a new issue with reproducing example.

Tried with Yarn 0.27 (beta) and still yield the same error.

Please report it to Yarn.

(and create a reproducible example)

I had the same problem on a project with eslint installed globally.
I solved this case by installing in eslint project:
npm i eslint --save
bye
jeff

@jfroussel your tip is very helpfull! Thanks!

ok cool

i've the same problem with eslint 3.19 installed in the project
i think that it use the global version that in my case is 4


[in_app_purchase] Android purchases not sent to Google Analytics

I have Google Analytics on my project, and every event is collected normally, included in app purchases coming from iOS. But there is 0 trace of the in app purchases made on Android ; what can I do to understand what's wrong ?

@TheCarpetMerchant
I think it sounds more like an issue with GA rather than with Flutter. You can reach out using below links for further resolution:
GA4 - Filter Events
Analytics Academy
reference: https://stackoverflow.com/a/74763148

@TheCarpetMerchant I am not sure. If the data is being sent to GA but somehow it is not being received there, could be related to server side. Do you already have specific filters in place or could be related to the Google play account you have that you can check ?

Thanks for the update. Can you try the plugin example and check if using it you still get the same behavior or not ?


Read raw data

Hello,
I can send raw data to my Custom VKP80 printer with the command _raw().
Is there a way to read binary data?

That looks like a pretty cool printer.
We don't have a lot in the API for getting information yet, but you can access the connection property directly. It's usually named device, but depends on what kind of connection you made. You can see that property assigned in each __init__ here.
There is a couple lines for reading status commands over at #143.

Hi,
i have similiar wish.
I'm using Custom TG1260H printer and had a issue with CUT. Solution: _raw('\x1c\xc0\x34')
Now i need read paper status.
If you want, i can provide documentation for TG1260H, TG2460H, TG2480H, TL60 and TL80 printers
HAND

This will be closed in favor of #143 (it basically describes the same issue)


Shortend LeSetAdvertisingParametersCommand() by 3 bytes

I also changed at positon 3 the byte 0x12 to 0x0F, as this is what my program returns when I run it.
I even installed nunit to test whether the environment differs, but still there's 0x0F as a proper result.

I'll try to integrate both edits into one pull requests, this somehow didn't work as I expected...


Generic Object Editor

When I try and use an object editor for an object property of my model I get:
Uncaught Missing required 'schema.subSchema' option for Object editor

I would expect to see some kind of key value pair editor where I can edit each property of the object and add or remove properties.

Have you written your schema similar to this?  You need to define the object schema in the subSchema property:
var schema = {
    address: { type: 'Object', subSchema: {
        street: {},
        zip: { type: 'Number' },
        country: { 'Select', options: countries }
    }}
};

I understand that the Object editor requires a subSchema, I just was hoping that it would also work on arbitrary user defined key-value pairs. Perhaps there is a different editor that achieves this?

You could achieve something like this with the List editor.  Each item in the list would be an object of 2 fields (key and value).


release: highlights of v0.0.4

/release v0.0.4

/release v0.0.4

/shipit

/shipit

🎇 🎉 🎆 New version has been shipped! 🚀🚀🚀

🎇 🎉 🎆 New version has been shipped! 🚀🚀🚀


Add RIFT token

Official announcement: https://riftmember.com/
Project team background:
The RIFT team was established in May 2019 and started design and development. The founder, Alex Siow, Ph.D., has research interests in futures payments and financial information processing. He is engaged in large-scale financial information systems and distributed application practices. Design experience. Team members have conducted in-depth research on the scalability, reliability, and elastic optimization of distributed systems. They have a deep understanding of the underlying technology and working principles of the blockchain and have practical experience.
Project basic information:
RIFT is committed to building a blockchain financial ecological payment system that truly has a compound business model and is applied on the ground. Based on the peer-to-peer network technology, a new form of global financial payment is built based on blockchain and smart contract technology.RIFT is a virtual currency used in the digital currency trading market. We effectively solve the problems in the operation of the financial market, and effectively improve our performance in the market, and complete the global payment application scenario, making you who do not know how to create investment profits Participate in our operations and enjoy profitable performance in a rapidly changing market!
Recommended Gas Limit (For ERC-20 project teams) for transaction: 60,000

欢迎在社区中推荐用户使用 imToken 2.0 管理资产。


Update xilinx fpga device plugin location

Describe the bug
in https://github.com/dellhpc/omnia/blob/devel/roles/k8s_start_services/vars/main.yml
update line 80:
fpga_device_plugin_yaml_url: https://raw.githubusercontent.com/Xilinx/FPGA_as_a_Service/master/k8s-fpga-device-plugin/fpga-device-plugin.yml
with updated URL:
fpga_device_plugin_yaml_url: https://raw.githubusercontent.com/Xilinx/FPGA_as_a_Service/master/k8s-device-plugin/k8s-device-plugin.yml

it also appears that we are not detecting if there is an FPGA. all instances install the fpga device plugin, please check the variable that is populated when probing the system for hardware


[Task]: Harmonize extended landing page metrics

Task description
All landing pages which are compatible with Stardust show following extended metrics:
https://www.figma.com/file/j4RLa61iZ18VXwiabeyu9k/IOTA-Explorer-UI?node-id=128%3A53544

Tokens created: "11.2k"
NFTs minted: "34.6k"
Active addresses: "34k"
Locked storage deposit: "549k SMR"
Daily transactions: "10k"

Requirements

On non Stardust compatible networks the extended metrics are hidden and blocks are named messages
Extended metrics are hidden if their endpoints are not available

Acceptance criteria

Metrics load in less than a second.

Visible on alphanet only currently until we wire the API up.


Improve best case complexity for get_work

Yes, we only improve the best case complexity, not the worst case.
In summary, you'll get sublinear in aomunt of tasks if your worker
doesn't do any tasks that use global resources.
get_work has gotten way to complex once we added global resource
constraints, as the resource constraints interaction with priorities is
a hairy beast requiring a global perspective, not allowing for
worker-local optimization. See these related tickets which added the
expressiveness/complicated stuff:

spotify/luigi#445
spotify/luigi#462
spotify/luigi#496
spotify/luigi#513

Note that the optimization once done in spotify/luigi#584 which helped
@daveFNbuck's company a lot doesn't help Spotify's setup. Most of
Spotify's tasks lingering in the scheduler are pending tasks and not
done tasks.
The proposed hacky solution here, which I mostly see as a temporary
solution until we do a bigger rewrite and rethink of the central
scheduler is the following:
Categorize the get_work scenario in two different buckets, "trivial" and
"nontrivial". Trivial is when none of the tasks of the worker require
global resources, in that scenario, just loop over the tasks the worker
have registered for instead of all the tasks. For the non-trivial case,
we do what we've always done previously.
So in that sense this commit contains two parts:

Add an "index" so you can quickly retrieve the tasks for a worker
Do the special-casing of trivial and nontrivial in get_work

At Spotify, we don't use the resources feature that much, so we're
hoping for tremendous speed improvements, no more 30-second get-work
calls! :)

This is awesome, really always wanted to spend more time refactoring this code!
Do you have any performance numbers?

Not really, but here's a start at least: https://github.com/spotify/luigi/pull/986/files#diff-3425f46b36ec1572c55d6448f0ed6957R751

Should we merge this?
And again can you share some performance stats? I guess I can do it also

I think it can be merged if you've reviewed it as well (Dave has as well).
Well, I haven't tried this in a real environment yet. But clearly it's way faster according to the test I've added.

Happy to merge this. Would be great to have some numbers. But I can run later. I wonder if we should have some sort of benchmark tool

Less talking more merging. :p
I'll see if we can deploy this in the Spotify environment anytime soon. :)

nice!


Add Politecnico Grancolombiano

Add Politecnico Grancolombiano Domain

@sallacas I am sorry, the domain poligran.edu.co is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


refactors

Goals

[ ] ignored keys should not allow addition and removal to pass

This should be re-written to use the same comparison mechanism as the elixir versino


Allow text selection in demo for code sample, but avoid text selection flashing while using demo

I realize the demo is a WIP, but I also realized I could make a nicer solution for the text selection problem. It applies a selecting class the body onStart, which applies (vendor-prefixed) user-select: none styles, and removes that class onStop, also clearing all text selections that might’ve occurred while the user was dragging around the page (getSelection().removeAllRanges()).
Note that I didn’t update the code sample in the demo with those changes, because they also requiressome custom CSS, but I’d be happy to do so if that would be desirable. I could just put a comment in onStart:
    onStart(evt) {
        // Applies 'user-select: none' to all elements on the page via a stylesheet
        document.body.classList.add('selecting');
    },

UPDATE I pushed up a commit that adds that comment that I proposed and updates the example section in the demo HTML.

That's basically a good idea, but  I think it would confuse people on the demo-website because it isn't related to selection. How would it be when this would be directly integrated into selection.js?

I've fixed this problem in the core, so you don't need to manually do this.
Thanks for bringing this to my attention!


Some EA code descriptions are missing from the ea_codes.csv/EA variable codebook spreadsheet

 Issue by SimonGreenhill
Thursday May 07, 2015 at 00:45 GMT
Originally opened as https://github.com/NESCent/dplace/issues/206

These are either the NA states (with should be coded as the missing value in the EA, but aren't):
EA001 - NA
EA002 - NA
EA003 - NA
EA004 - NA
EA005 - NA
EA094 - NA

(i.e. EA v1 state = NA)
... or are things that the EA has values for but doesn't describe what the value is (?!).
EA032 - 4
EA039 - 0
EA042 - 4
EA046 - 9
EA053 - 4
EA056 - 4
EA066 - 1
EA069 - 9
EA072 - 2
EA074 - 9
EA083 - 0

e.g. v32 has:

.... but is missing a state 4 (which is found in e.g. Ai37 Koalib or Ai40 Moro, Ag14 Baga etc)

Has the new import code fixed this? @stefelisabeth

@stefelisabeth this should no longer be an issue. can you close it if you are satisfied that the new EA variable list is complete?


changed Enumerable.Empty to Array.Empty.

also changed string[]to IEnumerable<string> for *Criteria Indices properties

resolved #60


python3Packages.env-canada: 0.5.12 -> 0.5.13

Motivation for this change
Update to latest upstream release 0.5.13
Change log: https://github.com/michaeldavie/env_canada/releases/tag/v0.5.13
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 138939 run on x86_64-linux 1

2 packages built:

python38Packages.env-canada
python39Packages.env-canada

Result of nixpkgs-review pr 138939 at 2d5323d9 run on aarch64-linux 1

2 packages built successfully:

python38Packages.env-canada
python39Packages.env-canada


pythonPackages.django_3: init at 3

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

there's a lot of packages with similar breakages:
  ERROR: Could not find a version that satisfies the requirement Django<3.1,>=1.11 (from django-mailman3==1.3.4) (from versions: none)
  ERROR: No matching distribution found for Django<3.1,>=1.11 (from django-mailman3==1.3.4)

please rebase on the latest master. I sorted python-packages.nix

@FRidh I am back from my break. Have you had time to rework this PR, or do you want me to take over it?

@lsix feel free to take it over!

I have updated the work done here in #98463
I close this PR in favor of the new one.


Fix the test coverage for implementation enhancements of Rust's channels

Description
There are impl blocks for extending a Rust channel in the code base,
that must be excluded from coverage if a test is too trivial (because it is not needed to test the std library behavior of Rust)
or tests must be written for them to satisfy the coverage.
Example where unit tests are written to test the trivial Rust channel behavior the satisfy coverage:
common/src/to_server_interface.rs
Example where no tests are written:
common/src/from_server_interface.rs
There are potentially more code paths with the same issue or mixture.
Goals
Have a single solution for dealing with such scenario. Document the decision in the developer guide.
Final result
Summary
To be filled when the final solution is sketched.
Tasks

[ ] Search through the code base where trivial implementations like Rust channels are tested or not tested
[ ] Discuss solution with Architect @krucod3 or @christoph-hamm to decide whether to exclude them from coverage or to write unit tests to satisfy test coverage.
[ ] Add or remove tests according to the result of the discussion and document it for future use in developer guide or contribution guide.

Will be done with #259


fix: Remove non essential assertions from ln_collab_close test

Somehow we've introduced a regression that mines blocks before we expect them.
For the time being, fix the test by commenting out the assertions - it's a bug
in the test code, not in the production code.
It's more important to keep testing that the collab close works.

edit: @luckysori @holzeis I reckon this PR should be closed in favour of #722 (which turned out to be the culprit).


Request: Send conversion to clipboard

Any chance this plugin could send the output to clipboard instead of or as well as an external document? Would be super helpful for me :).

Hi,
Would you like a toggle switch in the settings that, when enabled, copies the file to the clipboard whenever you export?
If so, that should be an easy feature using this library

That would be great! Honestly for my purposes I don’t need all the style css on the front, just the basic text formatting, but I can take that off easy enough.
Unless there could be toggle option to remove that too?
Thanks!
Thanks,
Jared.
On 7 Sep 2021, 3:28 PM +1000, OliverBalfour @.***>, wrote:

Hi,
Would you like a toggle switch in the settings that, when enabled, copies the file to the clipboard whenever you export?
If so, that should be an easy feature using this library
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

So just to be super clear on my above comment re the basic formatting, what I'd love would be an option to just export/capture what's between the body tags. In terms of copying to another platform, eg Wordpress, I'd mostly be wanting to use the look and formatting that's on the other platform, rather than what's in Obsidian. This is probably a whole other request really :).

I think I have the same requirement. What I'm looking for is the ability to copy my note and paste it into GMail (which is rich text format, I think), rather than having to tidy up all my nested bullet points.


Feature: Pin the Status Center Open

What feature or improvement do you think would benefit Files?
Allowing users to have the option to pin the status center (active file transfers, etc) open so it stays open even if we click elsewhere in the app or click out of the app.
Requirements

Allows users to keep the status center open, maintaining constant view of all active tasks

Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3593
Comments
No response

Thanks for the feedback, I've added this to the project board.

it would also be nice to be able to popout dialogue boxes with the file operation information like windows does.
eg:

@0xicl33n Isn't it annoying that you have to bring the dialog to the foreground when it went to behind?
This is why we implement it in static and fixed place.

@0xicl33n Isn't it annoying that you have to bring the dialog to the foreground when it went to behind? This is why we implement it in static and fixed place.

i have multiple monitors and like to use my right side monitor for those dialog boxes so i can keep an eye on the file transfer progress. im not saying they should pop out by default, but an option would be nice

Hello. My wife HATES with passion the progress ring, both in apps like Google Chrome but also in files. It would be great that the progress could be kept open, so she can look at the copying progress at any time, without having to click in the ring to see progress.
While I do not have such strong feeling about this UI idea, I all for giving more power and control to the user to see progress permanently.
Thanks for your understanding, I hope this feature is implemented :)


Command to get the file size

This PR implements the std::fs::GetFileSize (aliased get_file_size) command mentioned in #221.

Codecov Report

Merging #222 (2fe17ba) into master (237123c) will decrease coverage by 3.24%.
The diff coverage is 97.18%.

@@            Coverage Diff             @@
##           master     #222      +/-   ##
==========================================
- Coverage   93.62%   90.37%   -3.25%
==========================================
  Files         452      454       +2
  Lines       16156    16225      +69
==========================================
- Hits        15126    14664     -462
- Misses       1030     1561     +531

Impacted Files
Coverage Δ

...cript_sdk/src/sdk/std/fs/get_file_size/mod_test.rs
95.83% <95.83%> (ø)

duckscript_sdk/src/sdk/std/fs/get_file_size/mod.rs
96.15% <96.15%> (ø)

duckscript_sdk/src/sdk/std/fs/mod.rs
100.00% <100.00%> (ø)

duckscript_sdk/src/utils/io.rs
97.56% <100.00%> (+0.50%)
:arrow_up:

duckscript_sdk/src/utils/io_test.rs
100.00% <100.00%> (ø)

duckscript_sdk/src/utils/annotation.rs
0.00% <0.00%> (-90.00%)
:arrow_down:

.../src/sdk/std/env/set_current_directory/mod_test.rs
10.34% <0.00%> (-86.21%)
:arrow_down:

duckscript_sdk/src/sdk/std/json/encode/mod.rs
9.02% <0.00%> (-70.84%)
:arrow_down:

duckscript_sdk/src/sdk/std/test/test_file/mod.rs
16.66% <0.00%> (-61.54%)
:arrow_down:

...kscript_sdk/src/sdk/std/test/test_directory/mod.rs
25.00% <0.00%> (-61.54%)
:arrow_down:

... and 29 more

Continue to review full report at Codecov.

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

@Red-Teapot  This is great, thanks a lot.
I'll wait for the CI to finish and then merge it.


:seedling: Bootstrap governance

This PR adds a GOVERNANCE.md file inspired by CNCF's Project Governance template https://github.com/carolynvs/cncf-project-template/blob/79bce741b54f33f1e81a313b0b077e2dd79c0096/GOVERNANCE-maintainer.md.
The main changes are:

a manifesto of goals to be a good Kube citizen is added
a set of values to be an open and inclusive project are added
a bootstrap set of maintainers is defined which are going to be replaced by maintainers following "the rules".

This PR also removes former Red Hat maintainers from the OWNERs file, and replaces them with volunteers from the community of interested parties who want to see kcp to continue under its own governance.
This PR must be lgmt'ed and approved by Red Hat's former maintainers.

LGTM

/verify-owners

/verify-owners

Waiting for @clubanderson to agree with putting his name into OWNERS for now.

agreed
/lgtm

/approve

/hold cancel


feat(rest-api-client): support  guest space

[x] Record Client
[x] App Client
[ ] File Client
[ ] BulkRequest Client
[ ] Add unit tests for buildPath

Jest's each function might be helpful to write unit tests for buildPath.
https://jestjs.io/docs/en/api#testeachtablename-fn-timeout


Update version of github actions on the template project

Hello
Description
I propose an update to date, some actions in template ci.yml
Checklist:

[ ] I've made sure that tests are updated accordingly (especially if adding or updating a template option)
[ ] I've updated the documentation or confirm that my change doesn't require any updates

Rationale
When i push my django's project to github , i have some alert on my pipeline

That seems good now

@DelphineLemire Good catch. Thanks 🚀 🚢


rocAL - Warnings

rocAL not tested on OCL Backend

Multiple Warnings

In file included from ../../../../rocAL/rocAL/include/loaders/image/image_read_and_decode.h:28:
../../../../rocAL/rocAL/include/decoders/image/turbo_jpeg_decoder.h:60:10: warning: 'is_partial_decoder' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]
    bool is_partial_decoder() { return _is_partial_decoder; };
         ^
../../../../rocAL/rocAL/include/decoders/image/decoder.h:114:18: note: overridden virtual function is here
    virtual bool is_partial_decoder() = 0;
                 ^

Run Tests

HIP Backend

/home/kiriti/develop/mivisionx-kiriti/MIVisionX/rocAL/rocAL/source/decoders/image/fused_crop_decoder.cpp: In member function ‘virtual Decoder::Status FusedCropTJDecoder::decode(unsigned char*, size_t, unsigned char*, size_t, size_t, size_t, size_t, size_t&, size_t&, Decoder::ColorFormat, DecoderConfig, bool)’:
/home/kiriti/develop/mivisionx-kiriti/MIVisionX/rocAL/rocAL/source/decoders/image/fused_crop_decoder.cpp:144:30: warning: ‘y1’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  144 |     if( tjDecompress2_partial(m_jpegDecompressor,
      |         ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~
  145 |                       input_buffer,
      |                       ~~~~~~~~~~~~~
  146 |                       input_size,
      |                       ~~~~~~~~~~~
  147 |                       output_buffer,
      |                       ~~~~~~~~~~~~~~
  148 |                       max_decoded_width,
      |                       ~~~~~~~~~~~~~~~~~~
  149 |                       max_decoded_width * planes,
      |                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~
  150 |                       max_decoded_height,
      |                       ~~~~~~~~~~~~~~~~~~~
  151 |                       tjpf,
      |                       ~~~~~
  152 |                       TJFLAG_FASTDCT, &x1_diff, &crop_width_diff,
      |                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  153 |                     x1, y1, crop_width, crop_height) != 0)
      |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/kiriti/develop/mivisionx-kiriti/MIVisionX/rocAL/rocAL/source/decoders/image/fused_crop_decoder.cpp:166:33: warning: ‘x1’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  166 |         unsigned int xoffs = (x1-x1_diff) * planes;   // in case x1 gets adjusted by tjpeg decoder
      |                              ~~~^~~~~~~~~
[ 80%] Building CXX object rocAL/rocAL/CMakeFiles/rocal.dir/source/decoders/image/open_cv_decoder.cpp.o

@LakshmiKumar23 let me know if OCL build is tested with TOT

@LakshmiKumar23 can you please check these warnings
Ubuntu 22.04
-- The C compiler identification is GNU 11.2.0
-- The CXX compiler identification is GNU 11.2.0
-- rocAL -- Using HIP -- Path:/opt/rocm	Version:5.3.22374	Compiler:clang

/home/kiriti/develop/mivisionx-kiriti/MIVisionX/rocAL/rocAL/source/meta_data/bounding_box_graph.cpp: In member function ‘virtual void BoundingBoxGraph::update_box_encoder_meta_data(std::vector<float>*, pMetaDataBatch, float, bool, float, std::vector<float>&, std::vector<float>&)’:
/home/kiriti/develop/mivisionx-kiriti/MIVisionX/rocAL/rocAL/source/meta_data/bounding_box_graph.cpp:145:15: warning: ‘void* memcpy(void*, const void*, size_t)’ writing to an object of non-trivially copyable type ‘BoundingBoxCord’ {aka ‘struct BoundingBoxCord_’}; use copy-assignment or copy-initialization instead [-Wclass-memaccess]
  145 |         memcpy(bb_coords, full_batch_meta_data->get_bb_cords_batch()[i].data(), full_batch_meta_data->get_bb_cords_batch()[i].size() * sizeof(BoundingBoxCord));
      |         ~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /home/kiriti/develop/mivisionx-kiriti/MIVisionX/rocAL/rocAL/./include/meta_data/meta_data_graph.h:26,
                 from /home/kiriti/develop/mivisionx-kiriti/MIVisionX/rocAL/rocAL/./include/meta_data/bounding_box_graph.h:25,
                 from /home/kiriti/develop/mivisionx-kiriti/MIVisionX/rocAL/rocAL/source/meta_data/bounding_box_graph.cpp:22:
/home/kiriti/develop/mivisionx-kiriti/MIVisionX/rocAL/rocAL/./include/meta_data/meta_data.h:37:16: note: ‘BoundingBoxCord’ {aka ‘struct BoundingBoxCord_’} declared here
   37 | typedef struct BoundingBoxCord_
      |                ^~~~~~~~~~~~~~~~

#1013 fixes these warnings


DataError: Failed to read large IndexedDB value

Error
at z (https://cdn.jsdelivr.net/npm/dexie:1:4631)
at new t (https://cdn.jsdelivr.net/npm/dexie:1:6191)
at Fe.rejectionMapper (https://cdn.jsdelivr.net/npm/dexie:1:71437)
at Ne (https://cdn.jsdelivr.net/npm/dexie:1:10384)
at IDBRequest. (https://cdn.jsdelivr.net/npm/dexie:1:39572)
at IDBRequest. (https://cdn.jsdelivr.net/npm/dexie:1:11929)

Could you describe with more context?

I am executing this query with dexie:
 db.campaigns.where({
        status: "Running"
    }).count();

And then get above-mentioned error
here is the schema
campaigns: 'id, module, created, lastRunDate, name, status, autoResume',
Devtool says there are 189 entries in total. But can access only 149 entries
if I do db.campaigns.each(console.log) then it logs some and then exits with the same error
I tried to take backup and restore, but due to this error, can't take backup too


Operation pre/post hooks

It doesn't seem to be possible to set functions to fire before or after a mutation/query.
Some examples of that are

graphcool: https://www.graph.cool/docs/reference/functions/hooks-pa6guruhaf/
mongoose: https://mongoosejs.com/docs/middleware.html

What I would want is something close to these 2 libraries. Some use cases would be:

Before creating a user, set a property based on sent by the user (I can add this to my createUser resolver, but a User can also be created through a relation)
Validate an update mutation sent by the user before saving it to the database (same problem as previous)

It's possible to do a "after operation" hook using subscriptions, but I can't see any way to do the "before operation" logic for Update, Create, Delete
Thanks!

This would bring Prisma closer to a full ORM replacement. This is a fantastic project and I hope to continue seeing it evolve. Hooks into the CRUD actions for create, update, etc. are key to an ORM's functionality!


[Ember-Migration] [Tests] Footer

Issue

Write tests for the footer component - For reference, refer other files of tests

Note

PR should be raised to develop-ember not develop
Create a separate branch test/footer from develop-ember and write the tests in footer-test.js

Yes, I want to work on this issue.

I would like to work on it!

@rohan09-raj Assign this issue to me, as it seems work is not started yet.

Please assign this issue to me.


Scoreboard not positioned

Scoreboard does not order with positions

Needs testing on real game data.

Tested on real game data. Cannot reproduce. Probably fixed with some other fixes.


Press cancel but page did not close

In iOS, while clicking on the cancel button, page did not close while it does throw exception :
PlatformException(USER_CANCELED, User has canceled the operation., null)

Platforms affected
☑  iOS
☐  Android

Also, when you tap cancel and search it doesn't select the selected address, it just comes out empty.


OS Script templates improvements

What this PR does / why we need it:
Refactoring of OS scripts:

unify the installation and configuration across different OSes
flatcar improvements
flatcar containerd config generation (a-la we do it in machine-controller)

Special notes for your reviewer:
Does this PR introduce a user-facing change?:
OS Script templates improvements

/retest

/retest


Commas in GET URIs being transformed into %2C

When you pass in a request that uses , in the query params (which is how X handles user.fields etc) it is being transformed into %2C which causes those custom fields to fail to be fetched from X.
I was able to fix this locally by changing the following
  def escape_query_params(uri)
      URI(uri).tap do |u|
        u.query = URI.encode_www_form(URI.decode_www_form(u.query)).gsub("%2C", ",") if u.query
      end
    end

I would push the fix myself for a PR but I'm getting a 403. I changed the code above and also added this test
    def test_escape_query_params_with_commas
      uri = "https://api.twitter.com/2/tweets/search/recent?query=%23ruby&expansions=author_id&user.fields=id,name,username"
      request = @request_builder.build(http_method: :post, uri:, authenticator: @authenticator)

      assert_equal "query=%23ruby&expansions=author_id&user.fields=id,name,username", request.uri.query
    end


gui-wm/river: add 0.3.2

I totally missed https://github.com/bsd-ac/wayland-desktop/pull/89. Closing this because that PR is looking better than mine.


[Feature Request] Snip hotkey to automatically send the OCR request

Is your feature request related to a problem? Please describe.
Creating the screenshot, switching to the image2Latex app, pressing enter and switching back to the document takes a bit of time. It would be great if a snip hotkey was added, just like in the main Mathpix app.
Describe the solution you'd like
It would be great if I could press a hotkey (e.g. alt+shift+s) and a screenshot creator would open, letting me drag the screenshot area just like the default operaing system screenshot taker. Then, the OCR request gets automatically sent, so that after the response I can directly paste the formula in my document, without having to interact with the app directly.
Additional context
This already happens in the main mathpix application, and I find it extremely userful and productive.

I have thought about implementing this feature in the past. After some research, I found that Java programs do not support global monitoring of key events. So, this feature won't be implemented unless the whole app was rewritten in a different language.

Maybe a keypress listener writes in C/C++ can send the signal to this app. But I am not familiar with these languages.

After a bit of research I found https://github.com/kwhat/jnativehook . Could this be useful?

Last time I saw this library, it wasn't updated for quite a long time. But I am glad to know this package is maintaining by someone. I will try this one if I have some time.


[WIP] Show due date and task number in Kanban card

First draft design:

@MattGaiaver, your thoughts?

Why even have the document no? Also, let's try to make this upstream as well.

Preferably the Post It will have the 'Due in' phrasing and the numerical date in the top right corner.

Why even have the document no? Also, let's try to make this upstream as well.

Helps in quick lookup for a specific task.

Preferably the Post It will have the 'Due in' phrasing and the numerical date in the top right corner

Yeah, I'll make that change.

They can just...click on the card to open the task!

NO not by clicking on it… on the Kanban card!

On Nov 27, 2018, at 10:05 PM, Valmik<EMAIL_ADDRESS>wrote:
They can just...click on the card to open the task!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/DigiThinkIT/frappe/pull/35#issuecomment-442331008, or mute the thread https://github.com/notifications/unsubscribe-auth/An9d-AN6B_nNadgwFcgN_5JlI7HsfEFbks5uzifHgaJpZM4Y0-R5.

@MattGaiaver different discussion lol

Well, ideally there would be a search bar on the board, but if I ask you to refer to task #8, how are you going to find it on the board without the number? 😛

@Alchez My main issue is that fugly as hell split that won't work if the naming series changes, therefore cannot go upstream

That's a fair point, but I don't know a quick way to fix that, outside of having document numbers being saved in the doc natively.

With the new changes requested by D9.

Pull Request Test Coverage Report for Build 272

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 44.899%

Totals

Change from base Build 271:
0.0%

Covered Lines:
17249

Relevant Lines:
38417

💛  - Coveralls

PR created upstream: https://github.com/frappe/frappe/pull/6555


Upgraded to 0.6 :)

Oh FML! Actix hasn't upgraded to http 1 yet. Great

I HATE the hacky workaround I'm doing, but whatever. Let me know if this fine

@maccesch that testing problem we discussed about is popping up here. And actix isn't willing to upgrade to http1 until 0.5 (https://github.com/actix/actix-web/pull/3208#issuecomment-1837204631) so we're gonna have to do a workaround till then

Thanks! I didn't even know that you can specify dependencies like that. Looks good to me. Once Actix updates we can simplify but until then I think this is a good solution.

Wait, so how are we gonna make a new release with the tests failing?

The tests are failing because if you enable both features you're defining the headers variable twice. If you think about it, when you call cargo test --all-features which version of the use_cookie function are you expecting it to use?
There are two ways I've seen things like this handled:
Option A: Prevent enabling both features
#[cfg(all(feature = "foo", feature = "bar"))]
compile_error!("Only one feature \"foo\" or \"bar\" may be enabled for this crate.");

#[cfg(all(feature = "actix", not(feature="axum")))]
const COOKIE: http0_2::HeaderName = http0_2::header::COOKIE;
#[cfg(all(feature = "axum", not(feature="actix"))]
const COOKIE: http1::HeaderName = http1::header::COOKIE;

#[cfg(any(feature = "axum", feature="actix")] // <-- important so people using custom integrations can still use leptos-use
pub fn use_cookie(cookie_name: &str) -> Option<Cookie<'static>> {

Update the github action to call cargo test --axum and cargo test --actix separately.
Cons

No single command to run all tests.
Not really idiomatic as Rust features should be additive and not mutually exclusive according to the docs:

A consequence of this is that features should be additive. That is, enabling a feature should not disable functionality, and it should usually be safe to enable any combination of features.

Option B (my preference): Separate modules for platform specific implementations
You can have two different files/modules for the two integrations that have the same/similar function signatures in them. Users would then import from the appropriate module for their integration like so:
// Axum
use leptos_use::axum::use_context;

// Actix
use leptos_use::actix::use_context:

Cons

Some code duplication (can mitigate somewhat with private functions that are shared between them for common parts of the implementation. use_cookie is small enough already though).

Thanks for that write up!
Yes, this has already been discussed with @maccesch and I in a private DM. These were the approaches we had in mind, the question was more of "what's the plan to make a release with a failing build" rather than "how do we solve this". We have a few solutions in mind that would mitigate this in the future, so this is just for a temporary workaround we need to do till actix 5 releases


ISPN-5817 Add cache blacklisting support to the endpoints

https://issues.jboss.org/browse/ISPN-5817
Added a couple of operations to ignore and un-ignore caches for all endpoints:
Ex:
/subsystem=datagrid-infinispan-endpoint/:ignore-cache-all-endpoints(cache-names=["cache1","cache2"])
/subsystem=datagrid-infinispan-endpoint/:unignore-cache-all-endpoints(cache-names=["cache1","cache2"])

I was not sure about the common practice regarding version bump. I suppose it must follow the project version?

Yes, that's the general rule we've used up to now.
On Mon, Nov 30, 2015 at 9:24 PM, Gustavo Fernandes<EMAIL_ADDRESS>
wrote:

I was not sure about the common practice regarding version bump. I suppose
it must follow the project version?
—
Reply to this email directly or view it on GitHub
https://github.com/infinispan/infinispan/pull/3862#issuecomment-160750967
.


Add tests for nested namespace

https://github.com/personalrobotics/chimera/pull/259#issuecomment-465843414

Added by https://github.com/personalrobotics/chimera/pull/283


Run shellcheck on bash file

Ref #64

shellcheck doesn't support zsh unfortunately

shellcheck doesn't support zsh unfortunately

that's a shame!
Thank you for this one :rocket:


Link is not working for Hitnet

Issue Type
Documentation Bug
OS
Ubuntu
OS architecture
x86_64
Programming Language
Other
Framework
PyTorch
Model name and Weights/Checkpoints URL
NA
Description
142_HITNET/download.sh is not working due to invalid link.
Relevant Log Output
No response
URL or source code for simple inference testing code
https://s3.ap-northeast-2.wasabisys.com/pinto-model-zoo/142_HITNET/resources.tar.gz


[CSV Reader] Support disabling double quote escapes

Our CSV reader currently interprets double quotes as escapes; we should allow this behavior to be disabled.
Arrow's CSV reader already supports this behavior via it's reader builder: https://docs.rs/arrow2/latest/arrow2/io/csv/read/struct.ReaderBuilder.html#method.double_quote

I can pick this up!


Suspected memory corruption bug in PB2NC on WCOSS.

Describe the Problem
Perry Shafran at NOAA/EMC found odd behavior using METplus-3.0/met-9.0 on the WCOSS machine venus.  PB2NC creates ASCII output files for each message type (e.g. *_ADPUPA.txt) even though the "-dump" command line option has not been used.  John HG tested and confirmed this behavior on venus, but the problem does not appear on NCAR machines.  I also confirmed that recompiling with code changes either makes the behavior the same... or disappear, depending on the location of the change.  I strongly suspect this is a memory corruption bug where some other code is stomping on the memory of the boolean "dump_flag" variable.
RT Ticket:
https://rt.rap.ucar.edu/rt/Ticket/Display.html?id=94683
Expected Behavior
Those ASCII output files should only be written when the "-dump" command line option is used.
Environment
Describe your runtime environment:

Machine: WCOSS Venus.
Version: met-9.0

To Reproduce
Describe the steps to reproduce the behavior:

Log on to venus.
Navigate to /u/John.H.Gotway/MET/MET_Help/shafran_data_20200323.
Run the script test_pb2nc.sh
Check for the ASCII output files being written and the following log message being printed:
DEBUG 1: Dumping to ASCII output directory:     .

Relevant Deadlines
4/1/2020 prior to the WCOSS production switch while Venus is still available.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee

[X] Select engineer(s) or no engineer required (John HG)
[X] Select scientist(s) or no scientist required (Perry Shafran)

Labels

[X] Select component(s)
[X] Select priority
[X] Select requestor(s)

Projects and Milestone

[X] Review projects and select relevant Repository and Organization ones
[X] Select milestone

Testing to see if @jprestop receives this notification

Testing on Venus in:
/u/John.H.Gotway/MET/MET_Help/shafran_data_20200323/pb2nc
This same behavior also exists with met-8.1.  So this is likely an old bug, not newly introduced in met-9.0.  It does not exist for met-8.0, for which D_PBL and D_CAPE cannot be derived.

I did not see these files introduced in older versions of MET.  The first time I noticed it was the beta4 version of METv9.  Is it possible that they were produced but then deleted by the code?  I ran plenty interactive METplus runs with the older 8.1 version but never saw the code.
I also want to add that I have not updated to METplus v3.  I am working on the beta1 version of METplus v3 from last fall.  Could that be a source of error?
Thanks!
Perry

@PerryShafran-NOAA, I'm not all that surprised you didn't see this behavior in early versions.  Memory corruption is difficult to track down and dependent on the exact configuration you're running.  I count us lucky to have a pattern that's so repeatable.
More testing with: /u/John.H.Gotway/MET/MET_Help/shafran_data_20200323/pb2nc/run_pb2nc_8.1.sh
Trying to narrow down the behavior.  The problem persists with:
export OBS_BUFR_VAR_LIST="["XXX","D_CAPE"]";
Where XXX is TOB, QOB, UOB, or VOB, but PMO, TDO, PWO, TOCC, D_RH, HOVI, CEILING, or D_PBL.

Good news, I found the offending lines and it's very easy to fix:
https://github.com/NCAR/MET/blob/master_v9.0/met/src/tools/other/pb2nc/pb2nc.cc
Starting on line 147.  There are now 8 variables to derive.   So I'll change the 6 to an 8 and add the codes for CAPE and PBL.
// Number of variable types which may be derived
static const int n_derive_gc = 6;
// Listing of grib codes for variable types which may be derived
static const int derive_gc[n_derive_gc] = {
dpt_grib_code,  wdir_grib_code,
wind_grib_code, rh_grib_code,
mixr_grib_code, prmsl_grib_code
};
The array overflow occurred on line 2365:
bufr_derive_code[vIdx] = bufr_var_index;

Adding @hsoh-u as an assignee so he can fix this bug in the develop branch.

Still need to delete the development branch for this issue:
bugfix_1298_master_v9.0_point2grid


[DOC][Testing Passive STS]Provide a reference how to test this via Admin service

Document: https://docs.wso2.com/display/IS540/Testing+Passive+STS
Please provide a reference/sample on how to do the same thing, which is done through Management Console, via Admin Services.

Refere "Configuring STS WebApp as the Service Provider through Admin Services" [1]
[1] - http://wso2-oxygen-tank.10903.n7.nabble.com/Configuring-STS-WebApp-as-the-Service-Provider-through-Admin-Services-td153933.html

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


Add pause button

resolve #137

@mthenw Please see this PR

@mthenw ping

Hey, I'm sorry about super late answer. What I actually see in this PR is a "ignore new lines" functionality. Could you elaborate more about what's the use case and are there any reported issues that this PR solves?

We have a large stream of logs and use filters.
First, it is difficult to keep a part of logs in view when new entries are added.
Second, the new data quickly displaces the necessary data from the buffer. We can't endlessly increase the number of rows that are stored in the browser.
Third, there is often a situation when you need to quickly stop the log in order not to search for the necessary information for a long time.

As a frequent frontail user I'm in support of this functionality, but I'd like to see it tell you how many new log lines you've missed since it's been paused. and leave that in the list when it's unpaused so it's clear that there is a gap in the data you're seeing.

@RickeyWard maybe need to add a line with counter after a pause if it's possible.
Line like ==> SKIPED: 999 <==.

@mthenw What prevents you from accepting my contribution?

@levonet sorry for being so terrible at answering. I'm basically not sure if this is the type of functionality that I want to include in frontail. I don't have any strong arguments, tho :(
Let's do this. I'm testing this functionality right now and will give you some feedback shortly.

@levonet let me know if you are still willing to contribute (you already did a lot and I totally get it if you're no longer interested).
If not I will fix those few issues by myself?

In the current implementation favicon badge updates as if there're new lines. But when the streaming is paused we shouldn't probably update it. Can you fix that?

This problem occurs when the filter input loses focus.
Apparently, this problem was before my changes. I'll try to fix it.

@mthenw done

@levonet thank you for your contribution and sorry that it took so long.


Use <input type="color" /> on color type input fields (new label / edit label)

Description
Using the inspector to change only the input type to color, at least under firefox the resulting values are directly compatible with the semantics of the field for saving the color. So at the very least there's not that much fighting to do with the current github.
- <input required="" aria-describedby="label--color-error" value="#5319e7" type="text" name="label[color]" id="label-color-" class="form-control input-monospace pb-1 mr-0 js-new-label-color-input width-full" maxlength="7" autocomplete="off" spellcheck="off" pattern="#?([a-fA-F0-9]{6}|[a-fA-F0-9]{3})" title="Hex colors should only contain numbers and letters from a-f">
+ <input required="" aria-describedby="label--color-error" value="#5319e7" type="color" name="label[color]" id="label-color-" class="form-control input-monospace pb-1 mr-0 js-new-label-color-input width-full" maxlength="7" autocomplete="off" spellcheck="off" pattern="#?([a-fA-F0-9]{6}|[a-fA-F0-9]{3})" title="Hex colors should only contain numbers and letters from a-f">

Challenges(?) to handle after would be:

I suppose, fixing whatever onchange handlers that update the color in the related DOM elements
Removing the default suggestions on activation

Example URLs

https://github.com/<org>/<repo>/labels
https://github.com/refined-github/refined-github/labels

Duplicate of https://github.com/refined-github/refined-github/issues/2224


reusable run/debug buttons without need to find main file(solution:  add history of runs to run part)

Problem
we are bothering to run or rerun a main file of python/go/js ...  when working in other files
but run file is fixed so we want a UI/UX to be on our looks to see it for click
not finding main file inside of other files then going to menu to run
solution
add history of runs to run part

In this case, the best solution for you is probably to create a launch.json file with your preferred run configurations in it, then you can see your configs and run them from anywhere.


Advice on random ellipsoid packings

Dear Vasili,
I am trying to build an analogous code for building random close packing of differently sized ellipsoids.
This would help me dramatically in synthetic label generation for training a Unet segmentation algorithm.
Since you have a lot of experience with building this algorithm for spheres, I was curious about your intuitions about how to approach this problem with ellipsoids.
My goal is as follows: I have a volume of size N x N x N, and I would like M ellipsoids in there that vary in their orientation and the sizes of their major axes (both sampled from a reasonable range, s.t. the largest ellipsoid has at most 8 times the volume of the smallest ellipsoid). The close packing is essential, because otherwise the problem becomes trivial for the Unet.
Currently, I am using Python in combination with numba, but my code is still quite slow (40 seconds per generation of synthetic image). The most expensive step is solving a polynomial in order to determine if two ellipsoids are colliding (and numpy is already optimized here). I am aware that I can model an ellipsoid as multiple spheres stuck together on the longest axis to make collision detection easier, but I was wondering if you had other ideas as well. For example, is there a specific algorithm I can use to facilitate this process? At this point I use poisson disc sampling in order to obtain the seeds for growing ellipsoids à la lubachevsky-stillinger. My goal is to be about 500-1000 times faster (so 0.04 - 0.08 seconds per image generation).
Any advice would be greatly appreciated.
Kind regards

If you are say, looping over all the ellipsoids and performing the collision checks, this could be parallelized potentially. I'm assuming that you already perform a sort of accept/reject test on which ellipsoids need to be checked for collision with regards to other ellipsoids that are too far away from the candidate to ever intersect.

Hi @wandadars, I vectorized the the "rough" collision check using spherical ideals for all pairs of ellipsoids like so:
@njit
def vectorized_gross_collision(positions, semi_axes, tri_indices):
    # Compute pairwise squared distances for the specific indices in tri_indices (upper half)
    pairwise_distances_squared = np.sum((positions[tri_indices[0]] - positions[tri_indices[1]]) ** 2, axis=-1)
    max_radii_0 = np.zeros(len(semi_axes))

    for i in range(len(semi_axes)):
        max_radii_0[i] = np.max(semi_axes[i])  # Max semi-axis for ellipsoid i

    # Compute the sum of the max semi-axes for each pair
    radii_sums = max_radii_0[tri_indices[0]] + max_radii_0[tri_indices[1]]
    collision_mask = pairwise_distances_squared < radii_sums ** 2
    colliding_pairs = np.column_stack((tri_indices[0][collision_mask], tri_indices[1][collision_mask]))
    return colliding_pairs

This takes about 0.0005 second on my machine, which is fast, but could potentially be improved.
I am not sure how though. One idea is to divide my volume into subgrids to avoid checking the gross collisions for all pairs of ellipsoids. Another may be to use cupy instead of numpy. Using @njit(parellel = True) actually slowed things down...
After the gross collision detection, I run an exact detection on the flagged pairs. This takes a bit longer because of np.roots (the whole function calculating the collision takes up to 0.001 seconds). Adjusting the collisions actually works relatively quickly.
I hope that clarifies my bottlenecks.

Hi @postnubilaphoebus and @wandadars ,
@postnubilaphoebus, thanks for the question! @wandadars , thanks for your inputs!
I think it's indeed a tricky endeavor that you've started and i can provide only rough brainstorming ideas.
First, there is this paper Neighbor list collision-driven molecular dynamics simulation for nonspherical hard particles.: II. Applications to ellipses and ellipsoids. I don't remember the contents that well, but maybe you might find something there. Also, maybe you can just search for the code of the authors or just contact them. I was not following papers in this area for some time, so maybe you can check its citations or in general search for some new developments.
Also, i think Python is a tricky choice for your goal. It's obviously a popular and good language with ubiquitous support. But some years ago my rule of thumb was that Python is ~100 times slower than C/C++, and ~50 times slower than C#/Java. Numpy is obviously fast, but any non-numpy code is slow. You do use numba, and my understanding is that numba also tries to compile python into machine code and vectorize it, and it shall be fast in theory, but i don't have experience with it. My guess is that it won't be able to vectorize/compile all of the code well (otherwise everyone would be using numba and forget about C, but all core deep learning code is still written in C). I am not sure if it is an option for you to switch, and maybe it's not.
Anyway, for optimizations, you can consider the following (if not yet using them):

using Lubachevscky-Stillinger collision tracking algorithm from their original paper (in the readme). Maybe you use it already, but just to make sure: their algorithm is not just running event-driven simulations while compressing the packing, but they also track and update collisions in a very smart way and make the problem linear by the number of particles, not quadratic. It's really a game-changer in performance (i already forgot the numbers when i implemented it, but it was like x400 times faster for 10 000 particles)
searching for approximate neighbors at first. My code uses so-called cell lists (splitting the space into NxNxN cells and then for a particle in a cell i, j, k, you check nearby cells), and also Verlet lists. There are actually some more advanced algorithms, like KD-trees or Octtrees ("binary" trees in three dimensions). Scipy supports them out of the box, maybe you can just use it. Cell lists, for example, make the problem of checking collision times with neighbors linear vs. N_particles instead of quadratic. But they do pose a problem for the LS algorithm at small densities: particles from far cells can actually fly fast and hit your target particle earlier than any neighbor from the nearby cells. That's why I ask in the readme to use the FBA algorithm at first (the time step is fixed and small there, so particles don't move much inside an update step). Maybe one can introduce some heuristics to the LS algorithm on how to select the radius for nearest neighbors to overcome this problem completely

For colliding ellipsoids, and it is highly probably that you did this research, there is for sure some literature on how to optimize the search. Maybe you can create a triangular mesh from the surface and use algorithms from computer graphics for finding collisions, they shall be highly optimized.
I hope this is somewhat helpful.
Best Regards,
Vasili

I don’t think algorithms like NN and KDTree of SciPy or ... are the best options in terms of performance based on my little experience, as their efficiency highly depends on the case type, size, and other factors, and also there are better ways to do that. You might be able to achieve comparable speeds to C++ or C with Numba itself, but optimized effectively. And also you must take care of some probable reduced precisions during using Numba.
In general, I’ve previously asked related questions on Stack Overflow, and the answers from some experts there might help you better understand various aspects and nuances. You can find some of those relevant questions on my profile.
Regarding the code structure you shared here, the less you rely on NumPy modules and write just in pure Python, the better your performance will be when using Numba.


Lambda to EventBus construct

Hi.
I would love to have a feature that adds lambda the functionality to write to an event bus.
Even though it is super simple I think it should be done as it is similar to LambdaToSQS construct in its idea.
What do you think?
This is a :rocket: Feature Request

Thanks, we'll take a look

I may have misunderstood you because as far as I understand you can interact with non-default event bus.
    const eventbus1 = new EventBus(this, 'eventbus1', {eventBusName: "eventbus1"})
Then, you can use this eventbus to grant permission for publishing events:
eventbus1.grantPutEventsTo(my_lambda)
And then, you would also want to send the event bus name as a variable to that lambda:
  my_lambda.addEnvironment('EVENTBUS_NAME', eventbus1.eventBusName)
That could all be done with a constract such as "lambdaToEventBus(this, my_lambda, eventbus1)`

I agree that it can be done in the CDK, but the construct interface for EventBridge currently does not support the concept of non-default event busses.  One of our tenets is that services work consistently across all constructs - e.g. once someone understands how the SQS-Lambda construct defines and implements a Lambda function, then they also understand how Lambda functions are defined for S3-Lambda, SNS-Lambda, etc.
So the interface defined in DESIGN_GUIDELINES.md will need to be updated to include EventBusProps, ExistingEventBus; plus exposing the EventBus as a property. Then all the EventBridge constructs will need to be updated to support this functionality.
We agree it is a worthy goal, but our limited resources are committed elsewhere right now so it's on our backlog until we can tackle it.

Since leaving the above comment, we have an internal volunteer to work on this. Depending upon the bandwidth they can devote to the task, it may be coming sooner than originally thought. We're going to start with updating the existing constructs to use non-default event buses, as the interaction with targets and rules will make them more complicated that this construct as we figure out the interface design.

We started with adding custom event bus functionality to the existing EventBridge constructs - the last release included custom event bus features on aws-eventbridge-lambda. Our thought was that because they deal with rules and targets they would be more complex than lambda-eventbridge when defining the changes to the interface. The others are in progress and we should be implementing aws-eventbridge-lambda after these.

@biffgaut please close this

Closed with 368

(this is in the GitHub repo and will be pushed out to NPM, PyPi, etc. this weekend.


FR: Add support for Vimeo videos with domain restrictions

Description
It appears that Embedded Assets doesn't cater for Vimeo videos with domain restrictions.
Steps to reproduce

Restrict a Vimeo video to a domain

Note: I used a client's domain for this example, and their CMS for this test.

Try and embed it in Assets

Other information

Embedded Assets version: 2.2.7
Craft CMS version: Craft Pro <IP_ADDRESS>

Talking to Vimeo support, they offered the following advice:

To provide the whitelisted domain in the oEmbed request, the plugin must set a "Referer" header with the value set to your video's whitelisted domain. Here's an example oEmbed request for a video with domain-level embed privacy expressed as a curl:
curl -X GET 'https://vimeo.com/api/oembed.json?url=https://vimeo.com/307074610' -H 'Referer: https://example.com'
For more information, please take a look at our oEmbed documentation

Hi @Kashkin,
Could you please try this with the Embed library's version 3 demo site and see if it happens there?  Embedded Assets' Vimeo support is provided using Embed v3, so if it's happening with the demo site I'd recommend opening an issue over there.

Sure.
It doesn't happen on the demo site, but given that the requests to Vimeo happen server-side, it's hard for me to determine why.
My speculation is that the Embed demo uses the oEmbed library and regular requests to build the page. If you just use the oEmbed library to query a restricted video, it returns almost no details, not even the title.
{
  "type": "video",
  "version": "1.0",
  "provider_name": "Vimeo",
  "provider_url": "https://vimeo.com/",
  "html": "<iframe src=\"https://player.vimeo.com/video/307074610?app_id=122963\" width=\"640\" height=\"360\" frameborder=\"0\" allow=\"autoplay; fullscreen\" allowfullscreen></iframe>",
  "width": 640,
  "height": 360,
  "domain_status_code": 403,
  "video_id": 307074610,
  "uri": "/videos/307074610"
}

I have made a hotfix that resolves the problem. It's not a good solution, but it should demonstrate the required changes to the embedded asset plugin.
Embed's create function allows for a custom DispatcherInterface to be passed in:
public static function create($url, array $config = null, DispatcherInterface $dispatcher = null)

It is possible to provide custom headers as part of a DispatcherInterface.
CURLOPT_HTTPHEADER => array("Referer: https://example.com")

I've edited this file within vendor/embed on the server to include the domain, and Vimeo returns the video's title and thumbnail.
EmbeddedAssets currently doesn't allow for a custom DispatcherInterface to be passed in. I think the solution should be updating the plugin to provide this header for all Vimeo requests.

+1 for this...
I wonder if it's necessary to only set the CURLOPT_HTTPHEADER for Vimeo requests, as it may do no harm to set it for any request sent. This would require minimal code changes.
I'd be happy to make a PR @ttempleton ?

@johndwells If you'd like to work on a PR for this, we'd be happy to merge it.

Embedded Assets 2.4.1 is now out with the new referer setting as added in #146.  Thanks again @johndwells!


AKSM decreased BUT KSM NOT increased ！

I had 47.9894 KSMs and 50.8890  AKSMs before I  redeemed. And after I redeemed 20 AKSMs to KSMs. I  got 30.8579 AKSMs (  with 20 redeemed and 0.0311 fee) and still 47.9894 KSMs (which exactly should be 67.9894).Just see:

and also the Publish Record and Redeem Record do NOT show any records yet .
PCX mainnet address:  5VJ1T6URYRh6zfEG3PDb9AeSLUMJuPHGfEj2Vot3tyLqPcaq
ATP recipient address:  atp1ulu6e258693y825u73qn2yw9le5aukldwsqnl2

We are very sorry that the server was shut down early this morning, and we did not find out in time. Now that we have fixed the problem, you can continue to test it. Thank you so much for your feedback!


Output Type PostgreSQL database not supported

VS Code Version:  1.74.2
Extension Version: 1.4.3
OS Version: Windows_NT x64 10.0.22621

Steps to Reproduce:

Open ASA Job in Azure Portal which contains "PostgreSQL" Outputs

Select "Query" --> "Open in VS Code"

Check Output Folder (is emtpy)

Also AzureExplorer shows that the output is not supported

Hi @KonstantinPfeiffer , thank you for using our VS Code extension. Sorry that the current version didn't support PostgreSQL output yet. But we have plans to include it in our upcoming semester. We will notify you as soon as the feature becomes available.


Docker build fails due to large js chunks

Describe the bug
Docker started failling building image with the following warning:
(!) Some chunks are larger than 500 kBs after minification. Consider:
- Using dynamic import() to code-split the application
- Use build.rollupOptions.output.manualChunks to improve chunking: https://rollupjs.org/configuration-options/#output-manualchunks
- Adjust chunk size limit for this warning via build.chunkSizeWarningLimit.
✓ built in 11.44s

To Reproduce
Steps to reproduce the behavior:

Go to frontend/
Run npm run build
See warning

Expected behavior
Currently we have custon vite setting chunkSizeWarningLimit, but we should find an optimal solution.
Screenshots
N/A
Desktop (please complete the following information):
N/A
Smartphone (please complete the following information):
N/A
Additional context
N/A

Here is how to do this in the Vue Router Docs:

https://router.vuejs.org/guide/advanced/lazy-loading.html#Lazy-Loading-Routes

// replace
// import UserDetails from './views/UserDetails'
// with
const UserDetails = () => import('./views/UserDetails.vue')

const router = createRouter({
  // ...
  routes: [
    { path: '/users/:id', component: UserDetails }
    // or use it directly in the route definition
    { path: '/users/:id', component: () => import('./views/UserDetails.vue') },
  ],
})


"st2 key delete" does not obey interface and populate user query params

st2 --debug  key delete  -s user -u stanley netdev_servicewrapper_address
2016-09-23 17:13:48,545  DEBUG - Using cached token from file "/home/vagrant/.st2/token-st2admin"
curl -X GET -H  'Connection: keep-alive' -H  'Accept-Encoding: gzip, deflate' -H  'Accept: /' -H  'User-Agent: python-requests/2.11.1' -H  'X-Auth-Token: 235ca028e8e545efbd28806090ca3bd6' 'http://<IP_ADDRESS>:9101/v1/keys/netdev_servicewrapper_address?scope=user'
{
"faultstring": "KeyValuePair with name: st2admin:netdev_servicewrapper_address and scope: user not found in db."
}

Thanks for opening this issue, will look into it.
On a side note - do you use RBAC?

Good catch.
Fix is in #2925 and will be included in v2.0.1
Thanks again!


Dart analyser on cli doesn't use analysis_options.yaml excludes files

I have the following analysis_options.yaml:
include: package:flutter/analysis_options_user.yaml

analyzer:
  exclude:
    - 'bin/**'
    - 'lib/l10n/messages/**'
    - 'lib/**/*.*.dart'
    - 'ios/**'
    - 'android/**'
  errors:
    no_duplicate_case_values: error
    avoid_relative_lib_imports: error
    always_put_required_named_parameters_first: error
    always_declare_return_types: error
    invariant_booleans: error
    prefer_single_quotes: error
    avoid_init_to_null: error
    mixin_inherits_from_not_object: ignore

linter:
  rules:
    - always_declare_return_types
    - always_put_control_body_on_new_line
    - always_put_required_named_parameters_first
    - always_require_non_null_named_parameters
    - annotate_overrides
    - avoid_annotating_with_dynamic
    - avoid_as
    - avoid_bool_literals_in_conditional_expressions
    - avoid_catches_without_on_clauses
    - avoid_catching_errors
    - avoid_classes_with_only_static_members
    - avoid_double_and_int_checks
    - avoid_empty_else
    - avoid_field_initializers_in_const_classes
    - avoid_function_literals_in_foreach_calls
    - avoid_init_to_null
    - avoid_js_rounded_ints
    - avoid_null_checks_in_equality_operators
    - avoid_relative_lib_imports
    - avoid_return_types_on_setters
    - avoid_returning_null
    - avoid_returning_this
    - avoid_setters_without_getters
    - avoid_single_cascade_in_expression_statements
    - avoid_slow_async_io
    - avoid_types_as_parameter_names
    - avoid_types_on_closure_parameters
    - avoid_unused_constructor_parameters
    - camel_case_types
    - cancel_subscriptions
    - cascade_invocations
    - close_sinks
    - constant_identifier_names
    - control_flow_in_finally
    - directives_ordering
    - empty_catches
    - empty_constructor_bodies
    - empty_statements
    - hash_and_equals
    - implementation_imports
    - invariant_booleans
    - iterable_contains_unrelated_type
    - join_return_with_assignment
    - library_names
    - library_prefixes
    - list_remove_unrelated_type
    - literal_only_boolean_expressions
    - no_adjacent_strings_in_list
    - no_duplicate_case_values
    - non_constant_identifier_names
    - omit_local_variable_types
    - one_member_abstracts
    - only_throw_errors
    - overridden_fields
    - package_names
    - package_prefixed_library_names
    - parameter_assignments
    - prefer_adjacent_string_concatenation
    - prefer_asserts_in_initializer_lists
    - prefer_bool_in_asserts
    - prefer_collection_literals
    - prefer_conditional_assignment
    - prefer_const_declarations
    - prefer_constructors_over_static_methods
    - prefer_contains
    - prefer_equal_for_default_values
    - prefer_final_fields
    - prefer_final_locals
    - prefer_foreach
    - prefer_function_declarations_over_variables
    - prefer_generic_function_type_aliases
    - prefer_initializing_formals
    - prefer_interpolation_to_compose_strings
    - prefer_iterable_whereType
    - prefer_single_quotes
    - prefer_typing_uninitialized_variables
    - recursive_getters
    - slash_for_doc_comments
    - super_goes_last
    - test_types_in_equals
    - throw_in_finally
    - type_init_formals
    - unawaited_futures
    - unnecessary_brace_in_string_interps
    - unnecessary_getters_setters
    - unnecessary_lambdas
    - unnecessary_null_aware_assignments
    - unnecessary_new
    - unnecessary_null_in_if_null_operators
    - unnecessary_overrides
    - unnecessary_parenthesis
    - unnecessary_statements
    - unnecessary_this
    - unrelated_type_equality_checks
    - use_rethrow_when_possible
    - use_setters_to_change_properties
    - use_string_buffers
    - use_to_and_as_if_applicable
    - valid_regexps
    - void_checks

Everything is good on AS or IntelliJ (except when the file is open like already reported in https://github.com/dart-lang/sdk/issues/35475) but once I do it manually as command line:
dartanalyzer lib
or
dartanalyzer --options analysis_options.yaml lib

And both give me 1109 errors as it take into account my excludes files.
It's anoying as I can't use any CI except by removing my rules...
Env: Dart VM version: 2.1.0-dev.9.4.flutter-f9ebf21297 (Thu Nov 8 23:00:07 2018 +0100) on "macos_x64"

I can confirm exactly this behavior. You have a lot of errors in the exluded files. After doing this: https://github.com/dart-lang/sdk/issues/33193#issuecomment-391030810 . Everything seems to work unless you open this file again in VS Code. Then you have do this: https://github.com/dart-lang/sdk/issues/33193#issuecomment-391030810 again.
Also I can confirm, that by calling analyzer by command line like this:

dartanalyzer lib
everything works as expected.

You don't have to open a file - it's enough to just call a method/class defined in the file to trigger analysis. It's especially annoying when dealing with generated JSON or Protocol Buffers. If a generated file relies on another generated file (e.g. classes and enums in protobuf), you have to explicitly open both files to get rid of errors (e.g. in the one that references enums).

Same issue here :
# analysis_options.yaml
analyzer:
  exclude:
    - lib/**.g.dart
    - lib/src/version.dart

No error reported by the analysis server (i.e. via an editor like VSCode or WebStorm). But if I run dartanalyzer from the command line:
$ dartanalyzer --options=analysis_options.yaml bin lib test

... some errors are reported on the excluded files (even after deleting the .dartServer cache folder):
Analyzing bin, lib, test...
  lint - Avoid using `as`. - lib\src\cli.g.dart:10:28 - avoid_as
  lint - Avoid using `as`. - lib\src\cli.g.dart:11:30 - avoid_as
  lint - Avoid using `as`. - lib\src\cli.g.dart:13:34 - avoid_as
  lint - Avoid using `as`. - lib\src\cli.g.dart:14:36 - avoid_as
  lint - Document all public members. - lib\src\cli.g.dart:34:9 - public_member_api_docs
5 lints found.

Dart VM version: 2.8.1 (stable) (Thu Apr 30 09:25:21 2020 +0200) on "windows_x64"

Thanks for reporting this issue! This was previously reported at https://github.com/dart-lang/sdk/issues/25551; please follow along there.

Duplicate of https://github.com/dart-lang/sdk/issues/25551


how to tunnel the whole system (windows) like tun mode in clash

Hi
I'm wondering how I can tunnel the whole windows 10 network with v2ray?
as you know clash premium has an ability for that (tun mode)
how can I do the same with v2fly core in windows 10?
thanks

This is certainly on the table, but no one can predict when such a feature will be introduced.

V2Ray has no TUN mode for now. You can use Proxifier as the front proxy of V2Ray.

V2Ray has no TUN mode for now. You can use Proxifier as the front proxy of V2Ray.

You have complete training How to do this?

V2Ray has no TUN mode for now. You can use Proxifier as the front proxy of V2Ray.

You have complete training How to do this?

Hi Mehdi, did you find any way how to use Proxifire?
If you did, is it possible to share us?

in version 6.23 its have "tun mode". just run it as administrator and active it and see it in blew sidbar


Wrong paths when generating actions and reducers in Windows

When generating actions in Windows SO the path generated in containers/App.js are in windows' path format: "..\\actions\\name.js".
Same error occure when generating reducers, in the reducers/index.js

I am using the path module to generate those paths. I have currently no access to a Windows machine, do you think you could provide a PR fixing this issue? If not I will try to get hold of a Windows machine some time next week.

Thanks for the response. I will try to fix it later today and make the PR.

Thank you, that would really be awesome :-)

Alright, merged your additions and released a new version, thank you for your contribution.

Happy to help :)


How could I use stitch to analyze my antibody sequence produced by PEAKS ?

I run the the code .\stitch.exe batchfiles\monoclonal.txt ,before that I put my file in dataset, named my file as the input of file name mentioned by monoclonal, and I maintained format and  headers just same as your original csv file. But it reported errors like not a valid number.

It could be that you use a different version of Peaks compared to what we normally use, what version did you use? Can you show an excerpt of the peaks file you want to run using Stitch? Just the top 10 lines for example, the full file is also fine if you, but if you do not want that to be public just send it to my email d.schulte<AT>uu.nl.

Thanks for your reply! Firstly, I used PEAKS AB, and the exported file is like this

after that I found my format and content of my file is different from yours so I edit my file like this

finally, I downloaded PEAKS Studio 11, but the exported file is also different and only contain sequence of one certain enzyme, like this

I tried all these files on the basis that I could run your original files successfully.

That explains why it did not work, I have never had Peaks 11 files before, we are still using 10 here. For the edited data you showed in the second picture you are missing the 'Predict RT' column after 'RT' column L, this column can be filled with cells that are all '-' as is the case for the example dataset. If for the other two formats you showed all columns I can get you an update to stitch shortly that can handle these formats. But for that it would be really helpful the have the full files so I can test them on my side.

Thank you so much! I'll send my data to you!

I just pushed an update that supports peaks Ab. To use type 'Ab' in place of the 'X+' for the format field of the peaks data in your batchfile. Note that it normally takes about an hour before all tests are run and the new nightly version can be downloaded.
If you also have some peaks 11 data you want to run feel free to send me an example dataset, then I will build support for that as well. Otherwise I will add support when we get our hands on our own install of peaks 11 in a couple of weeks.
Thanks for opening the issue, I am always happy to support more input formats to make it possible to run Stitch for everyone. And feel free to reach out again if this does not work for you.

Hi I downloaded the new version, and tried again on the base of that I could run your original data successfully, but it still report errors like this, maybe I used the wrong way to run my own data? So could you please describe the detailed steps about how could I use the software to analyze my own data? By the way, I use the Windows system.

Ok thank you so much, I have run it successfully, I did not change the batchfile to Ab last time.

Hi, I have used the files from Peaks 10 and everything works fine, but there is a problem when processing a file from PeaksAB.
Also I didn't understand if it is possible to use files from peaks 11? And where can I download the latest "nightly" [versions?]()

You seem to have used an older version of Stitch from before I build in the support for peaks 11 and peaks Ab. The nightly version you can find here: https://github.com/snijderlab/stitch/actions. Then go to the last run, click it, scroll down, and there you can find the nightly-deploy-{system} zips files.

Setting Format: 11 is the way to go, but make sure you use a very recent version of Stitch as the support for Peaks 11 was only added very recently. Use a nightly version or v1.5.0-rc.1 (which you can find in the releases tab).

Thank you so much, I have run it successfully,


Markdown rendering not working in hero cards

I have some markdown in my

content.text

property of the hero card, but the rendering doesn't seem to work... I don't know if this is an issue with the WebChat or AdaptiveCards as a whole

Which version of WebChat are you using? I had similar issues with v0.12.1 but it's working fine again with the latest version (see https://cdn.botframework.com/botframework-webchat/latest/botchat.js)

I'm using the current version built from code. I also tried botchattest

@Alaiven same issue
What version is used in the latest CDN?
Adaptive Cards with Markdown Url not rendering too.
I have build a fresh v0.12.1 version and with no luck.
I am working on a customized version, so the CDN version is useless for me.
Do I need to include the markdown-it to my web page?

Can anyone confirm that this is a bug and maybe propose a solution?
I'm attaching a screenshot from the current botchattest, where you can see one of my bots displaying a Hero Card with some buttons. Markdown is not being rendered.

Hi there,
I do have the same issue.  I downloaded the 0.13.1 version and compiled it and use the botchat.js in a HTML-Page for the "Web Chat"
Any idea how can I fix the issue?
-->When I use the CDN  Markdown is OK

Here is the same page with CDN botchat.js

@alavien:
Me too. I do have the problem when I build the code for the custom edition. Do you find any solution?

No :( It seems that AdaptiveCards library stopped accepting Markdowns. I hope it is a bug and not a feature...

found a solution here. It was deep in some posts.  You need to add the markdown-it.js before your custom  botchat.js
related also to 972
I don't know how to enable it in the emulator. In the Web Chat the rendering is now correct.

Note to fixer: in order to use Markdown in Adaptive Cards, user need to load markdown-it.js.

Will fix in PR #1034.

Web Chat 0.14.1 is out with the fix #1034.


Ensure field "spriteMap" is updated on redraw()

An exception is currently thrown on the first call to spriteMap.Texture in Show() if spriteMap is null, and the ImageMessage will fail to display. Why might spriteMap be null? Well, if redraw() was called prior to this, the SpriteMap component will have been destroyed as part of the call to Object.Destroy(), and while a new SpriteMap is created in redraw(), the field member 'spriteMap' is not being set to this new component. Hence, any call to redraw() will lead to the exception since redraw() calls Show().
The solution I've presented here is the smallest change to fix the issue, but I think the better solution would be to forgo the local versions of "spriteMap" (which hide the member of the same name, triggering a warning) and replace lines 118 and 138 with "spriteMap = gameObject.AddComponent();" so there doesn't have to be an extra step of setting the field to the local - a step that is easy to miss, as in the case of this redraw() method.
Long-term, you might also consider decoupling the creation of the GameObject from the creation of the ImageMessage object, so the user can have the option to display the object sometime after the event that triggered it. At present, a scene change taking place between the event and the call to Show() will destroy the object without a publicly-defined way to get it back. I discovered the issue addressed above while experimenting with making redraw() public so I could display the ImageMessage in such a case.

Also: There's a good bit of overlap between redraw() and the constructor. You might want to separate out the GameObject and Component creation bits into their own method to avoid these kinds of discrepancies.

Thanks for the PR, @JayArrrgh is this still an issue for you? I have raised it on our internal ticketing system, as this repo is just a mirror.

Thanks for the PR, @JayArrrgh is this still an issue for you? I have raised it on our internal ticketing system, as this repo is just a mirror.

Thanks for the PR, @JayArrrgh is this still an issue for you?

I'm not currently using the SDK, but good luck!

Thanks for the PR, @JayArrrgh is this still an issue for you?

I'm not currently using the SDK, but good luck!


Add support for coverage

As discussed in https://github.com/newtriks/generator-react-webpack/issues/158

Coverage is now included and will be available in the next release.


introduce a micro stdlib for testing

Pull Request Description
This introduces a tiny alternative to our stdlib, that can be used for testing the interpreter. There are 2 main advantages of such a solution:

Performance: on my machine, runtime-with-intstruments/test drops from 146s to 65s, while runtime/test drops from 165s to 51s. >6 mins total becoming <2 mins total is awesome. This alone means I'll drink less coffee in these breaks and will be healthier.
Better separation of concepts – currently working on a feature that breaks all enso code. The dependency of interpreter tests on the stdlib means I have no means of incremental testing – ALL of stdlib must compile. This is horrible, rendered my work impossible, and resulted in this PR.

Important Notes
Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide dist and ./run ide watch.

Clever! Reminds me of my efforts to define minimal Java.
Once I suggested that runtime.jar shall carry a minimalistic library inside of itself to be usable as a standalone JAR. If we packaged this micro stdlib into the runtime.jar, I would have what I wanted!


[Feature] Integrate TRL with Flash Attention 1/2?

Would like to reap the benefits of Flash Attention that has been integrated in Hugging Face in TRL as well.
Seeing that model.to_bettertransformer() and with torch.backends.cuda.sdp_kernel(enable_flash=True, enable_math=False, enable_mem_efficient=False): both work with regular Hugging Face trainers, it should not be a stretch for them to also work with TRL trainers. My current need is to use the PPOTrainer.
PR https://github.com/huggingface/transformers/pull/25598 is working on integrating Flash Attention 2, but I don't know if the same API will be recycled and used.
I lack in-depth understanding of both techniques, so I will not be able to contribute. Hopefully someone can take up this feature.

I think you can directly import bettertransformer to convert model before deliver it to trainer

Silly me! Somehow I did not think of this solution 😅 Thanks @LuJunru!
I am still met with an error, but it does not seem like a TRL issue, so I'll close this for now.


Documentation for the spreadsheet

We should have some documentation for what the spreadsheet format is.  This should include the improvements made in #159.

In general, I like the idea of using github's resources for knowledge management.  So far, that's mostly been README.md files and using the issue tracker (e.g. things marked 'question').  One thing we haven't been using is the wiki.  I'm somewhat disinclined to have too many tools for such a purpose, so howabout a SPREADSHEET.md file inside the src directory?

Sounds good to me.
On Tue, Feb 24, 2015 at 5:04 PM, Berkeley Churchill <
<EMAIL_ADDRESS>wrote:

In general, I like the idea of using github's resources for knowledge
management. So far, that's mostly been README.md files and using the issue
tracker (e.g. things marked 'question'). One thing we haven't been using is
the wiki. I'm somewhat disinclined to have too many tools for such a
purpose, so howabout a SPREADSHEET.md file inside the src directory?
—
Reply to this email directly or view it on GitHub
https://github.com/eschkufz/x64asm/issues/160#issuecomment-75885504.

Yeah, I like that much better than a wiki, too.  This way we have all the information in one place (the repository)!


npm audit low-severity warning

I think it's actually not a vulnerability but I don't like this warning. Solution is about to upgrade stylelint  to proper version. Seems like this PR refers to this problem: https://github.com/stylelint/stylelint/pull/3660

Thanks for the heads up @trszdev!

Looks like primer/css is using an upgraded version of stylelint, which should win out in dependency hoisting! Closing for now.