RFC: discoverability of linter options

Proposal
Once a linter emits a warning the message of that warning should contain the name of the option that disables the linter. Currently this option can basically only be found "by accident" through the auto completion of set_option.
Rust does something similar to this:
fn test(x : u32) -> u32 {
    0
}

yields:
warning: unused variable: `x`
 --> src/lib.rs:1:9
  |
1 | fn test(x : u32) -> u32 {
  |         ^ help: if this is intentional, prefix it with an underscore: `_x`
  |
  = note: `#[warn(unused_variables)]` on by default

(the corresponding incantation to disable it would be #[allow(unused_variables)])
I propose that the equivalent code in Lean:
def test (x : UInt32) : UInt32 := 0

which currently yields:
unused variable `x` [linter.unusedVariables]

should yield something along the lines of
unused variable `x`
note: this lint can be disabled with `set_option linter.unusedVariables false`

I'm not familiar with the linter framework right now but ideally this should be generically possible across all linters.

User Experience: Users will easily discover the options to disable linters as opposed to discovering the feature by accident or in the future by reading that specific part of the docs.

Beneficiaries: All Lean users that aren't aware of these options but want to fiddle with linter settings.

Maintainability: I don't think this is much of a change in terms of maintainability. The only thing to keep in mind is if we end up changing the option names we must reflect that in the messages.

Community Feedback
I know of at least one user that would like this feature to exist and it does seem reasonable to me.
Impact
Add :+1: to issues you consider important. If others benefit from the changes in this proposal being added, please ask them to add :+1: to it.

Once a linter emits a warning the message of that warning should contain the name of the option that disables the linter.

It does though? It's written right there in the message. But I agree that using a set_option line specifically would be more copy-pasteable.

That's great! I wasn't sure if this is actually guaranteed to be the name of the option or just a coincidence, will send a PR!

It is, that's why it's printed in the message :smile:


Better ci.pipeline.run.cause attribute

Better ci.pipeline.run.cause attribute, finish the work started on

https://github.com/jenkinsci/opentelemetry-plugin/pull/187  .

Replace ci.pipeline.run.cause.description and ci.pipeline.run.cause.xxx to just keep ci.pipeline.run.causewith a machine readable value like UserIdCause:anonymous or BranchIndexingCause

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[x] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

@v1v I finished the work started on

https://github.com/jenkinsci/opentelemetry-plugin/pull/187

Can you please review?

Merge so I can move forward on

https://github.com/jenkinsci/opentelemetry-plugin/issues/204
https://github.com/jenkinsci/opentelemetry-plugin/discussions/98


[requires.io] dependency update on master branch

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

Comments from the review on Reviewable.io


always on top

📝 Always on top
Always on top option for each window in title context menu

Tracked in issue #13


llvmPackages_git: 19.0.0-git-2024-06-12 -> 19.0.0-git-2024-06-23

Description of changes
Part of the weekly LLVM git updates, best tested via building llvmPackages_git.clang.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.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.

Add a :+1: reaction to pull requests you find important.

@ofborg build llvmPackages_git.clang


"Exception in thread" error when modifying and saving README.md

Using a fresh install via pip3 install grip, but when I make a change to my README.md, I get this exception:
 * Change detected in README.md, refreshing
<IP_ADDRESS> - - [17/Feb/2017 17:06:42] "GET /__/grip/refresh/ HTTP/1.1" 200 -
Exception in thread Thread-8:
Traceback (most recent call last):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/threading.py", line 810, in __bootstrap_inner
    self.run()
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/threading.py", line 763, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/SocketServer.py", line 602, in process_request_thread
    self.handle_error(request, client_address)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/SocketServer.py", line 599, in process_request_thread
    self.finish_request(request, client_address)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/SocketServer.py", line 334, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/SocketServer.py", line 657, in __init__
    self.finish()
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/SocketServer.py", line 716, in finish
    self.wfile.close()
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/socket.py", line 283, in close
    self.flush()
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/socket.py", line 307, in flush
    self._sock.sendall(view[write_offset:write_offset+buffer_size])
error: [Errno 32] Broken pipe

Seems like it is looking in Python 2 for stuff though.
This error is reproducible by making any change whatsoever to README.md and saving it.

Hmm. Seems there's some incompatibility with the Python 3 executable and and Python 2 builtin libraries. That isn't surprising. I'm curious how a pip3 installation is being mixed with Python 2. (Grip doesn't seek out Python 2 explicitly.)
Are your PYTHONPATHs being crossed somewhere? What happens if you pip install grip instead?

Are your PYTHONPATHs being crossed somewhere?

They appear separate. Here's 2.7:
$ python -c "import sys; print(sys.path)"
['', '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python27.zip', '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7', '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-darwin', '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-mac', '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-mac/lib-scriptpackages', '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-tk', '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-old', '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-dynload', '/Library/Python/2.7/site-packages', '/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python', '/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/PyObjC']

and here is 3.6:
$ python3 -c "import sys; print(sys.path)"
['', '/usr/local/Cellar/python3/3.6.0/Frameworks/Python.framework/Versions/3.6/lib/python36.zip', '/usr/local/Cellar/python3/3.6.0/Frameworks/Python.framework/Versions/3.6/lib/python3.6', '/usr/local/Cellar/python3/3.6.0/Frameworks/Python.framework/Versions/3.6/lib/python3.6/lib-dynload', '/usr/local/lib/python3.6/site-packages']

What happens if you pip install grip instead?

I only have pip3 on my machine, not pip. Trying quite hard to do everything in Python 3.

Ok, those seem normal.
Python 2.7 does come prepackaged with pip now. I wonder if something happened during your transition to 3.
Try running Grip this way to be sure it's running in Python 3.
$ python3 -m grip

That is just the command I was looking for! No more error. Thank you for being so prompt too.
I thought #54 changed grip to run on Python 3 by default though?

Happy to help 😃
grip will be run on whichever pip version installs it. I'll bet there's some wires crossed somewhere since running pip3 install grip then running grip is importing Python 2 packages. I'd recommend re-installing Python. You'll know it's fixed when grip runs fully in the environment you installed it in.

Good suggestion. I removed all traces of pip and pip3, reinstalled Python 3 via Homebrew, and reinstalled grip via pip3 install grip. Now it works as intended! 👍

Awesome! Glad that worked for you 😃


Feature request: Check for cyclic dependencies

Hi,
I have a feature request for you: At the moment the loader accepts a compose file with cyclic dependencies, even when the validation step is enabled.
With cyclic dependencies, I mean something like this:
services:
   service1:
      depends_on:
         - service2
   service2:
      depends_on:
         - service3
   service3
      depends_on:
         - service1

Any updates on this? Do you find this idea useful?
I can work on the feature and contribute it myself if you wish ...

/CC @ndeloof @ulyssessouza


Invalid syntax while trying to start the training

Hi. Thanks for such a great effort. But a little issue is keeping me from being able to train the model. I am getting syntax error while trying to run the Train.py from the command line like this
$: python D:\deep-text-similarity\train.py --is_char_based=False --word2vec_model=D:\wiki.en\wiki.en.vec
Then i get en error-
File "D:\deep-text-similarity\input_helpers.py", line 213
print len(vocab_processor.vocabulary_)
^
SyntaxError: invalid syntax
Kindly help me so that i am able to train the model

Are you using python3?
print len(vocab_processor.vocabulary_) is how you print with python2.


Error 503 on secondary bridge

Hello,
I have 2 bridges in my setup. The first one worked just fine but I’m not able to configure the second bridge. If I push the button nothing happens and after some time (about 30 minutes) your plug-in is starts to “spam” 503 messages on the second bridge (it asks to press the button but immediately report error 503). Do you know why this happens?

Oh and one more thing. If I launch the terminal command “nb auth” it asks to press the button, if i press it it gives me the code but still in “home” app is not appearing

Make sure the HTTP API is enabled on the second bridge.  If nb auth obtains a token, I suppose it is.  I don't know why Homebridge NB wouldn't get a token when nb auth does.  Both share the same code to communicate with the Nuki bridge.
Please list the full error message next time.  I think it's HTTP status code 503, Service unavailable.  In that case, the Nuki bridge reports that it cannot handle the request.  It overloads rather easily, especially if multiple clients connect to it simultaneously.  I'd restart or power-cycle the bridge, and disable other clients while Homebridge NB is starting.

Hi @ebaauw,
It looks like it was a misconfigured item in my LAN setup (I have a mesh wi-fi). The RIP was enabled, and this caused the communication error between your plug-in and the bridge. I was able to get the command line to work because when i tested it I was on the same wi-fi satellite of the bridge (my Pi with homebridge is connected on the main router).
Anyway, disabled the RIP and solved the issue.
Thanks for your help.


Added NextCloud Conference 2017 to the live site [WIP]

Partially fixes the issue #1456
Changes:

Adds NextCloud Conference 2017 to the live site

Screenshots for the change:

Closing due to some problem


Introduced BASE_URL environment variable

Hi,
I've been doing some looking for Open Source privacy-first analytics services for a client project I am working on. I love the look of Ackee and in order to integrate it I've had to modify it a little.
Because our service runs on DigitalOcean's App Platform, we deploy in using a node droplet rather and a docker droplet. This is mostly to do with a desire to avoid using a reverse proxy. However, we encountered an issue when deploying on a URL other than root e.g.
www.mydomain.com/ackee
Ackee will deploy fine on www.mydomain.com however, because the HTML is hard coded to import index.js style.css etc. if these files are being served on a route other than the index route, the system falls over. More than that, I wasn't really in the mood to deploy an Nginx instance to wrap around this for the sake of rewriting the retrieval of 3 files.
For this reason, I've introduced a new environment variable BASE_URL which can essentially be expressed as
BASE_URL=/ackee

which will rewrite imports of index.js to point to /ackee/index.js instead. Similarly it also repoints /api requests to /ackee/api.
This change does not modify the serverless functions notion of it's route (e.g. src/server.js:96), on the basis that DO masks incoming requests e.g.
if I make a request to www.mydomain.com/ackee/api/getViews (I realise this isn't necessarily an actual endpoint, I'm just demonstrating), Ackee will get /api/getViews.
Depending on demand and use cases, it may be appropriate to introduce a SERVER_BASE_URL variable which similarly prepends the serverless instance's notion of its URL.

Hello @nergmada ,
Nice one, I am also looking to integrate Ackee in an existing node app. Did it work for you as a separate node instance or is it running within your existing node instance (same starting point: index.js, app.js ...).
If it is the later case, It would really be very helpful, I don't know why this is not possible from the beginning, I'm expecting such an analytics app to be easily integrated, regardless of separation of concerns or whatever ...
thanks for your additions... I will wait for your confirmation to proceed with testing.

@bacloud14 so DO's App Platform is like an series of microservices all built on one domain but on different endpoints.
Not to get into the finer points of what I did but essentially we have lots of small node instances running on a domain such as www.example.com

client app, being served on /
a content management system, being served on /cms
an analytics service (Ackee), being served on /analytics

Each of these is an individual "server" running it's own node instance. It's kind of like kubernetes, but it's more online and gui based config.
If you can clarify your question a bit more about how you'd want to run it as a single a service?
Off the top of my head if you're running say an express server, on a single machine. I'd put your express server on say port 2000, your Ackee server on say port 3000 and then if you want express to forward all requests for /analytics on to Ackee do something like
app.use('/analytics/*', functionToForwardReqToLocalhost3000)
Note that your function would need to rewrite the request path to work with my code e.g.
/analytics/api/getViews would need to be rewritten as /api/getViews, not that it would be very hard.
Then set BASE_URL in Ackee to www.yourdomain.com/analytics or whatever path you sent for "/analytics/*"
-Adam

I thought it would be possible for Ackee somehow to be a middleware inside an existing app. Without http messaging between the app being monitored and Ackee. That would probably ease the process of integration. Check this simple middleware for instance.
Thank you so much for giving me insights on how you are doing things.
Good luck

Another example for analytics is express-keenio but they offer the service (DB and dashboard I imagine) on the cloud, so...
Now I see Tic seem to be self hosted and a middleware so let's see...


Remove incompatible balancing_mode

balancing_mode review.
net-lb-int

Fix error on apply of example:

Error creating RegionBackendService: googleapi: Error 400: Invalid value for field 'resource.backends[0].balancingMode': 'UTILIZATION'. Balancing mode must be CONNECTION for an INTERNAL backend service., invalid

remove unused balancing_mode variable, as only one value is possible anyhow

net-lb-ext

update in the backends description

net-lb-proxy-int

update in the backends description

net-lb-app-int

added validation of balancing_mode
fixed other validations

net-lb-app-ext

added validation of balancing_mode
fixed other validations
removed validation for locality_lb_policy as this variable is not used in this module

Closes: #1767
Checklist

I applicable, I acknowledge that I have:

[x] Read the contributing guide
[x] Ran terraform fmt on all modified files
[x] Regenerated the relevant README.md files using tools/tfdoc.py
[x] Made sure all relevant tests pass

Great catch. As you were saying in chat, we might want to not even expose balancing mode anymore.

This went a bit bigger as I followed the white rabbit. Please re-check.


Implement noarch python

This PR updates this recipe to be noarch: python, which reduces the CI/packaging burden.
Checklist

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

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Rename CommandType to Name on resource command types

Description
Feedback from https://github.com/dotnet/aspire/pull/6295#discussion_r1803777740.
Note that this is a source breaking change for people who have implemented the proto when they update the proto the identifer will change name. It's not a wire breaking change because the field id (1 in this case) is the same.
@joperezr Where should the commandType -> name breaking change be communicated? The model API is new in 9.0 so I believe can change at this point without breaking change notification.
Checklist

Is this feature complete?

[x] Yes. Ready to ship.
[ ] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[ ] Yes
[x] No

Did you add public API?

[x] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[ ] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[x] Yes

Link to aspire-docs issue:

[ ] No

I think we need to do regular breaking change notifications as we changed this after the rc.

I prefer name because resources have a name, and endpoints have a name. Command names would be consistent.
And I think name makes more sense because the value is being provided by the user. Id feels more appropriate if the value is auto-generated.

/azp run

It turns out we shipped commands in the resource server protocol so this change breaks ACA and we need to either revert all of it, or deprecate the old fields and add the new fields


Support resource name generation for gRPC in Java

Adds a new command line argument to accept the gapic yaml file
Updates the generated build.gradle file with the gapic yaml and resource name plugin

Depends on https://github.com/googleapis/artman/pull/113

LGTM


Open Source Platforms 2019

https://www.tecmint.com/best-free-open-source-softwares-of-2018/

https://opencollective.com/


Put it at car.ipfs.io?

I will remember that domain.

in the meantime we have https://car.on.fleek.co/

It is done.


python310Packages.openstacksdk: 0.99.0 -> 0.100.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.openstacksdk is: "An SDK for building applications to work with OpenStack"
meta.homepage for python310Packages.openstacksdk is: "https://github.com/openstack/openstacksdk"
meta.changelog for python310Packages.openstacksdk is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.100.0 with grep in /nix/store/lk76dx41da15cvwyrm8mrg0i1qy0sy3m-python3.10-openstacksdk-0.100.0
found 0.100.0 in filename of file in /nix/store/lk76dx41da15cvwyrm8mrg0i1qy0sy3m-python3.10-openstacksdk-0.100.0

Rebuild report (if merged into master) (click to expand)

32 total rebuild path(s)

16 package rebuild(s)

16 x86_64-linux rebuild(s)
16 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
openstackclient
python310Packages.openstacksdk
python310Packages.osc-lib
python310Packages.python-heatclient
python310Packages.python-ironicclient
python310Packages.python-manilaclient
python310Packages.python-openstackclient
python310Packages.python-swiftclient
python39Packages.openstacksdk
python39Packages.osc-lib
python39Packages.python-heatclient
python39Packages.python-ironicclient
python39Packages.python-manilaclient
python39Packages.python-openstackclient
python39Packages.python-swiftclient
swiftclient

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lk76dx41da15cvwyrm8mrg0i1qy0sy3m-python3.10-openstacksdk-0.100.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.openstacksdk https://github.com/r-ryantm/nixpkgs/archive/3ea3f3b5571d03404f18bfe9e13c77f43a155add.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lk76dx41da15cvwyrm8mrg0i1qy0sy3m-python3.10-openstacksdk-0.100.0
ls -la /nix/store/lk76dx41da15cvwyrm8mrg0i1qy0sy3m-python3.10-openstacksdk-0.100.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages failed to build:

python310Packages.python-heatclient
python39Packages.python-heatclient

12 packages built:

openstackclient (python310Packages.python-openstackclient)
swiftclient (python-swiftclient)
python310Packages.openstacksdk
python310Packages.osc-lib
python310Packages.python-ironicclient
python310Packages.python-manilaclient
python39Packages.openstacksdk
python39Packages.osc-lib
python39Packages.python-ironicclient
python39Packages.python-manilaclient
python39Packages.python-openstackclient
python39Packages.python-swiftclient

Maintainer pings
cc @emilytrau @SuperSandro2000 for testing.

@ofborg eval


Add React book in free-programming-books-ko.md

What does this PR do?
Add resource
For resources
Description
Korean React Book
Why is this valuable (or not)?
We can learn React with this book
How do we know it's really free?
Publisher provides this book for free in linked site
For book lists, is it a book? For course lists, is it a course? etc.
It's book
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


issue with semantic highlighting - dimming

Does pylance dim code that it thinks is unreachable?  I have some code that modifies itself during runtime.  Pylance significantly dims all of the code after a function call that it may think ends the program, but really doesn't.  Is there a way to disable this behavior?
Thanks.

Can you provide more details about the code immediately before the dimmed section? I'm guessing that there is a call expression to a function or method that has no return type annotation and always raises an exception on all code paths. When there is no return type annotation, Pylance will analyze the code within the function's implementation to infer the return type. If there is no possible way for it to return, it infers a NoReturn type.
If you don't like this behavior, you can work around it by adding an explicit return type annotation to the function or method in question.

Note that this isn't tied to semantic highlighting; if code is marked as unreachable, it gets dimmed via a special diagnostic that VS Code chooses to render by dimming the code.
Feasibly, you could disable dimming in VS Code by setting something like:
{
    "[python]": {
        "editor.showUnused": false
    }
}

But the code flow will still regard the code to be unreachable (which has significant impact for the overall analysis), hence Eric's questioning here.

The function has an os._exit(1) but it is only executed after the subsequent code is read by doing some modifications on the fly.
Thanks very much for your helpful explanation and solution.  Your suggestion took care of it. I'll close this now.


updated query examples

Updated the examples in Myria-Web for the demo tomorrow.
We couldn't use SeaFlow data or Sigma Clipping query examples because they're not in S3 (and production is down). I also removed the profiling example. I could not get that to work.
If you would like, I'd be happy to add more examples or datasets. Perhaps some of the queries you'll be running during the demo tomorrow? Let me know.

I'm not sure what these failing tests are about. I saw there are some recent PRs about this. Is there anything else I can do or fix?

Just did a rebase, thanks for merging that PR!

Looks good, thanks for working on this. I might add more examples later from @ryanmaas's synthetic random point data, but let's merge this for now.


added lateral ganglionic eminence derived neuron

Fixes #2255
I followed CL:4023079 for logical definition - not sure if we need to add the precursor part (I think it is slightly redundant too and will probably involve more complexity with blank nodes and all than actually needed)
#gogoeditdiff

does #gogoeditdiff not work here anymore? :(

ok ignore me, it does, I see its running in actions, just thought it showed in the pr too heh

Thanks @dosumis & @aleixpuigb - soz for the silly mistakes, was focusing more on the setting up everything in the VM and all so super distracted lol, anyway fixed :)

@aleixpuigb I checked the merged_import, and it's all from the last Uberon release with many changes in the synonym type.


Suppress Some CMake Warnings

Fixes #1368
Description of the changes:

This commit will suppress CMake warnings CMP0064 and CMP0048 by set default policies.

Before submitting a Pull Request:

[x] I reviewed CONTRIBUTING.md
[x] I built my changes locally
[x] I ran the unit tests
[x] I ran the functional tests with a device
[x] I ran the performance tests with a device

I tested changes on:

[x] Windows
[ ] Linux

/azp run

/azp run

/azp run


Printer fails when encountering RegularExpressionLiteral

TypeScript Version:  2.4.2
Code
import * as ts from "typescript";
let sourceCode = "let a = /regexp/";
let tsSourceFile = ts.createSourceFile(
  "script.js",
  sourceCode,
  ts.ScriptTarget.Latest
);
console.log(
  ts.createPrinter().printFile(tsSourceFile)
);

Expected behavior:
The script should output let a = /regexp/;
Actual behavior:
The script crashes with the following stack trace:
.../node_modules/typescript/lib/typescript.js:3495
            throw e;
            ^
Error: Debug Failure.
    at Object.getLiteralText (/Users/work/GitHub/sample-js-selfparsing/node_modules/typescript/lib/typescript.js:7077:18)
    at getLiteralTextOfNode (/Users/work/GitHub/sample-js-selfparsing/node_modules/typescript/lib/typescript.js:68597:23)
    at emitLiteral (/Users/work/GitHub/sample-js-selfparsing/node_modules/typescript/lib/typescript.js:66918:24)
    at pipelineEmitExpression (/Users/work/GitHub/sample-js-selfparsing/node_modules/typescript/lib/typescript.js:66816:28)
    at pipelineEmitWithHint (/Users/work/GitHub/sample-js-selfparsing/node_modules/typescript/lib/typescript.js:66565:49)
    at pipelineEmitWithSourceMap (/Users/work/GitHub/sample-js-selfparsing/node_modules/typescript/lib/typescript.js:66558:17)
    at emitNodeWithNestedComments (/Users/work/GitHub/sample-js-selfparsing/node_modules/typescript/lib/typescript.js:64324:17)
    at emitNodeWithSynthesizedComments (/Users/work/GitHub/sample-js-selfparsing/node_modules/typescript/lib/typescript.js:64274:13)
    at emitNodeWithComments (/Users/work/GitHub/sample-js-selfparsing/node_modules/typescript/lib/typescript.js:64244:21)
    at pipelineEmitWithComments (/Users/work/GitHub/sample-js-selfparsing/node_modules/typescript/lib/typescript.js:66547:17)
error Command failed with exit code 1.

Proposed fix:
https://github.com/fwouts/TypeScript/commit/884cadf0a3b5682e8f3b03a7c0ab11f4504957d0
Where would be a good place to add tests for this?

https://github.com/microsoft/TypeScript/blob/master/src/harness/unittests/printer.ts

Thanks @ikatyang, I've added a test in https://github.com/fwouts/TypeScript/commit/170bc6f519ca7e44b522cdf6ce8d7ccc95231736.
Happy to send a PR once the Community milestone is added to this issue.

Send! Thanks for looking into this @fwouts


Fix carriage return issue on Windows

CI failed: https://github.com/alexpovel/srgn/actions/runs/6605779360/job/17941307575
On Windows, using tree-sitter-python, its comments parsing eats into \r\n and, e.g. if --delete is used, will mistake \r as part of a "comment" and delete it, leaving a naked \n. That's an error for files for CRLF style line endings.
Idea: for build in ScopedView, check if \n are generally in or out of scope. If all \n are detected as out of scope, shuffle all \r out as well, as some might have been put In scope. This will require copying.
This might also qualify as a bug upstream, but fixing every single tree-sitter parser/grammar, if at all possible, is much harder than fixing it in our application here, for every single use case.

Done in 496337c5d664db3df4a884f0050ee0d1357d8c2b .


[R] Use Rf_ prefix in C-level R functions

ref https://github.com/dmlc/xgboost/issues/10461
This PR changes the calls to R C-level functions to use the function versions that have names starting with Rf_, as it will be required at CRAN in the future. Along the way, it forcibly adds the macro R_NO_REMAP so that introduction of further non-Rf calls would turn into errors.
Note that the CRAN issue is for the current released version there, so if they request changes, it will be necessary to create a similar PR specifically for the CRAN release. At this point the branches differ too much for a simple backport of this PR, but the required changes will become evident once macro R_NO_REMAP gets added.

Out of curiosity, are there any prefixed functions for STRING_ELT, CHAR, etc?

I don't see any such Rf_ versions in the header:
https://github.com/wch/r-source/blob/0196c9d7cef6c21cad02eed747f11acef036743e/src/include/Rinternals.h

This looks like the right fix to me, thanks for doing it here.
It looks like changing PROTECT() and UNPROTECT() isn't strictly necessary, as those look like they'd always resolve to the Rf_* equivalents regardless of the use of R_NO_REMAP.
/* Pointer Protection and Unprotection */
#define PROTECT(s)	Rf_protect(s)
#define UNPROTECT(n)	Rf_unprotect(n)
#define UNPROTECT_PTR(s)	Rf_unprotect_ptr(s)

https://github.com/wch/r-source/blob/0196c9d7cef6c21cad02eed747f11acef036743e/src/include/Rinternals.h#L382-L384
But doing that type of change does reduce the risk of conflicts from other, not-related-to-R macros named PROTECT and UNPROTECT (which are fairly generic names).
If you have time/interest, I'd welcome a PR in LightGBM also making the change PROTECT -> Rf_protect and UNPROTECT -> Rf_unprotect.


Switches to JUnit 5

I misread earlier: JUnit 5 is no longer alpha. This pull request changes from JUnit 4 to 5 and updates the single test we have.

Test run ok with changes


Possible to copy from one VM to another

Hi
I was wondering if it's possible to copy files from one VM directly to another. I seem to get this error when I try
vagrant scp vm1:/home/vagrant/projects/project-build/project.rpm vm2:~
Warning: Permanently added '[<IP_ADDRESS>]:2200' (RSA) to the list of known hosts.
/home/vagrant: No such file or directory

Do you know why this could be?

You cannot copy directly. It should be done by downloading from one VM to a temp file and then uploading to target VM but this is currently not supported

Is this something you'd be able to implement?
A possible way to implement it could be:
if param1.isVM() and param2.isVM()
    copy param1 temp
    copy temp param2
    del temp

(just logic not actual code)


Updated OS system reqs on hardware-and-software-requirements-0.md

https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/1913

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@TroyStarr can you review?

@TroyStarr done, thanks!

@TroyStarr done, thanks!

Hi @scanum, wasn't sure why you assigned this to me since I signed off on Dario's pull request yesterday.  Is there anything else you need from me for this pull request?

@TroyStarr @scanum I think @MikePlumleyMSFT should merge it now. Thanks to everyone!


grpc: Add agent details to GetGuestDetails gRPC function

Create a new AgentDetails message and return it inside the message returned by GetGuestDetails(). The agent details provided are similar to those logged by the agent at startup in it's "announce" message.
Fixes #381.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

ah - need to update the mock tests...

Branch updated.
/test.

Can we combine this into GetGuestDetails and then runtime can query once and get all the info it wants.

@bergwolf - I was waiting for someone to say that :) Yes, let's try to minimise the overall number of calls we provide and the number of calls the runtime needs to make.
I'll rework and post a comment once done...

Hi @bergwolf - branch updated and re-tested.
Note that I didn't add a bool to the GuestDetailsRequest message to conditionally return information on the agent when the caller calls GetGuestDetails(). I'm not really sure we even need the existing GuestDetailsRequest.mem_block_size bool really - it won't be that expensive as although it's opening a file, it's in /sys/. Maybe we could remove that and only add such bool's if we know a particular check is going to be slow?

One more ack... :)

lgtm!


Debug console doesn't always handle color escape sequences

Version: 1.33.0-insider (user setup)
Commit: c1d001a915fe85cac93492393ea69fdb0c49ec0d
Date: 2019-03-13T06:18:20.237Z
Electron: 3.1.6
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.15063
When coloring debug console output with ANSI escape codes, VSCode will not always properly handle new colors. Instead, it seems to apply a hierarchy of colors where, for example, white will override any other succeeding colors.
For example, printing the following string:
\u001b[31m Red \u001b[32m Green \u001b[34m Blue \u001b[37m White \u001b[32m Green \u001b[31m Red \u001b[0m

Results in this output in VSCode:

Powershell, on the other hand, handles this properly:

Dup of some of these https://github.com/microsoft/vscode/issues?utf8=✓&q=is%3Aissue+is%3Aopen+debug+console+ansi+colors

@isidorn I don't see which one it would be a duplicate of. #67217 is about a different format, #42388 is related to theming and colors, #38834 and #571 relate to other locations in the program (not the Debug Console).

How about this one https://github.com/Microsoft/vscode/issues/67217
And here's a broader list
https://github.com/microsoft/vscode/issues?utf8=✓&q=is%3Aissue+is%3Aopen+debug+console+color

@isidorn #67217 is about support for a different format for color codes. This specific bug notes that the colors I am trying to use are supported and rendered, but not handled properly when multiple colors are used in succession.
I don't see it in that list either.

Ok, let's reopen this one.

We accept PR's that fix this.
Code pointer https://github.com/Microsoft/vscode/blob/a93905ebf8233c9f755e389b348cdc9b91c8f5f7/src/vs/workbench/contrib/debug/browser/debugANSIHandling.ts#L21

I'll work on it, thanks!

Make sure to add tests here https://github.com/Microsoft/vscode/blob/b4d347d9f634283eb122dc58ae518594726a2b9e/src/vs/workbench/contrib/debug/test/browser/debugANSIHandling.test.ts#L1


Add banner.txt with ASCII art saying "Arbitrader"

Cool? :smile:

Related to https://github.com/scionaltera/arbitrader/issues/167


Avoid error when IP address is null for UserDataBag

Think it should also be creatable with a null IP Address?
Used in Symfony getClientIP() can return no IP if no is set: https://github.com/getsentry/sentry-symfony/blob/42787b4f8a8fbff03da56b7291d3de59fb669bfe/src/EventListener/RequestListener.php#L63

I don't think that we should change a method intended to create a valid instance of the value object from an IP address by making it accept an invalid value for it. However, it would be better to avoid this situation in the Symony SDK instead

Created a PR there: https://github.com/getsentry/sentry-symfony/pull/565


Move common details to interface

@matthewkimber @otiai10 @pine613 could you review this PR?

yes, sir

@NN---
Sorry, I commented many points which is not in your commit.
but the original chrome.d.ts is not so correct, and I think it's good time to clean this up.
Please let me comment a bit more :bow:

@NN---
summary of my reviews bellow

[improve] timeStamp can be deleted 1
[improve] parentFrameId can be deleted 1, 2
[trivial] ? suffix on optional field (but it's not critical problem)

that's all. thank you always!

@otiai10 Changed it.
I also renamed CallbackDetails to WebRequestCallbackDetails , since I see webNavigation details also share a common interface, when I have time I'll add WebNavigationCallbackDetails.

@NN--- thanks mate!
@otiai10 thanks!


Removal of Slack button from homepage

She was a beautiful button.

This could have been one commit with the message "Remove Slack links"


[feat]: More Maintainers

Feature description
This library just seems unmaintained. There are over 500 unresolved issues and over 540 pull requests.
Affected component/components
No response
Additional Context
Additional details here...
Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues and PRs

@shadcn No offense btw but you could use some help

Yes, u are right. and i have been following from long time. I noticed this thing as well. And library is so cool i am using it in my two projects. Those two looks so good.
@shadcn please take an action.

I was really excited to start using shadcn-ui for my project because it looks great, but as others have mentioned, the amount of issues and PRs that are ready to merge from last year are really concerning. It doesn't seem dead given there are still recent commits. Could you please provide an update on the status and plans for the library? @shadcn  Thanks!

Although I am not using this project personally, I would like to give my best show to start contributing as the way of expanding my knowledge. @shadcn count me in in case you need help!

@shadcn?

Hi everyone. Just a quick update: I'm looking into it (and also working on a plan for handling all the open source projects) - I'll come up with a plan and update. Thanks for starting the conversation. Appreciate it!

Hi everyone. Just a quick update: I'm looking into it (and also working on a plan for handling all the open source projects) - I'll come up with a plan and update. Thanks for starting the conversation. Appreciate it!

I'm open if you need people to help maintain.

@shadcn any updates on the plans?


fix nc capabilities re to allow for namespaced tags

Junos uses namespaced tags in their hellos. this caused no matches for capabilities in scrapligo
the following re allows namespaced tags as shown here https://regex101.com/r/w1M6Lp/1

LGTM!


i cannot get the bearer token from the web

PLease help !

Why? Describe your problem.

it is always make an error on the script
i tried about 5 tokens in the same acc


feat: Add cwd option to use relative paths for compiler args

Resolves #13
Proposal

Add cwd as loader option. If specified, change paths that are passed to compilers to relative ones.
Update README to follow the above change.

Note
I add some conversion codes before executions of utility.execute(), but originally paths are explicitly converted to be absolute at process(), so it may be more simple to make paths to be handled as relative internally.
My implementation intends not to add breaking changes to the original behavior (yes, it should make no changes as long as you don't use the cwd option), but if requested, I'll try another way.

Since this PR is based on my personal motivation, I want to provide a working sample for it, but I cannot because my use case is based on separated proposal #9. When #13 (my PR for #9) is merged I'll try to provide some working sample here.
I don't mind if it is merged by maintainers' decisions before I do it.


File corruption

I have compiled the new changes you made , which are not on the releases page (it still shows from 2023) a few files start to give read errors and won't run. I'm also getting this issue with popopo's version. I am using an original PCB. when reverting to the original compiled file on here (compiled in 2023 , so cannot have the changes made 4 months ago, no issues.

blank.mdv.zip
MicroPicoDrive.uf2.zip
Here is one of the offending microdrive images, with the compiled image on github, this works fine (pics to come) but when I compile the newer code from github, this mdv fails. I loaded the newer firmware on mdv2_ and the older on mdv1_  both with the same MDV file from above, and mdv2_ failed, mdv1_ had no issues

the bad name error on mdv1_ is not related, it's because it's trying to call TK2_EXT and it's not installed

Jobd0ne, thank you for reporting it.
Your image has a different size.
The firmware tries to recognize the format by its size. Thus your image is not an standard one and is ignore by my firmware.
Anyway that is a blank file as I have tried it out in QEmulator.
From my firmware, it ignores directly non standard size (as I said). In order to keep same philosophy than the original one. So I guess that happen after the second change of the Firmware you mean  from this repo, so hereby mine too that I cloned as base for my modifications.
But thinking about to change it.
Bad  or changed medium... what a pain :) I have been working around it last 3 weeks and solved partially.
I need to go deeper in other part of the software that I didn't yet.
So I would apreciate if you contact me directly and helps to try some changes that I Am working on.
Ah! also same image that you shows in the photos but not a blank cartridge.

Apologies, wrong image uploaded, now edited the post

Hi,
It loads fine :) in both. QEmulator and in my QL with MDPs.
I have noticed the QDOS header, but it doesn't bother at all :) it loads fine. Also I see that now you have use the MDV size compatible.
So, is all clear and OK for you?
Please do not hesitate to contact me if you have another issue. Thanks to your reports I am enjoying and learning more and more.
So you may contact me directly in Facebook group or Telegram.
Happy to help


How to delete a Specific row ?

Hi I am working with this Library working Great But i need delete a particular row like after long press for that I am using Gestures below is my Gesture method
CGPoint p = [gestureRecognizer locationInView:self.tableView];
NSIndexPath *indexPath = [self.tableView  indexPathForRowAtPoint:p];
id message = self.conversation[indexPath.section][indexPath.row];
int index = (int)[[self messages] indexOfObject:message];
it is showing currenct index value but getting error like beyond the bounds exception. I thing it is problem with the section and rows in soMessaging but I am unable to delete that row can any body help me please

you extract your message from self.conversation and get index from self message
i can't understand this logic.

Beacause of the sections. In that table not showing exact indexpth.row value so i am using above code it is showing exact indexpath. I am able to delete in sqlite.But  getting error like beyond the bounds exception.

The exception beyond the bounds exception is occur when you try to access the index of array which is not present in array. For example an array contain 5 objects and you try to access the 5th index.
And i think when you delete from sqlite then you are removing it from your message array too. That is why this error occur.


Support Gen 2 GCP functions with higher memory

Currently the maximum memory for GCP functions is 4096MB while GCP functions do support 8192MB and 16Go / 32Go if it is a gen 2

Are you open to a pull request ?

Pr is welcome!

https://github.com/TriPSs/nx-extend/pull/339


How to specify native dll and custom file to remote?

In your paper, it says there can be non-assembly files uploaded, but I cannot find any example how to do that? any help? sorry, new to Prajna, but it is cool :)

An example of include 3rd party assembly file is:
In which you can call unmanaged DLLs at remotely.
https://github.com/MSRCCS/Prajna/blob/master/samples/DistributedSort/DistributedSort.fs
Generally, the remoting of files use the pattern:
let curJob = JobDependencies.setCurrentJob remoteJobName
curJob.AddDataDirectory( localDepsDirToDistribute ) |> ignore
the Directory entry can include both data files and unmanaged dlls, and you can
include multiple directory entries.
E.g.,
curJob.AddDataDirectoryWithPrefix( null, rootdir, "root", "*", SearchOption.AllDirectories )
will add a local directory pointed by rootdir to remote. At remote, the directory will be at:
CurrentDirection"root"[Local Direcotory]
You can also use,
curJob.EnvVars.Add(DeploymentSettings.EnvStringSetJobDirectory, depDirName )
to add Environment Variable entry to remote. We find it sometime necessary to properly remote the CUDA environment.
Let me know if you have additional questions.
Best,
Jin
From: Xiang Zhang<EMAIL_ADDRESS>Sent: Sunday, January 31, 2016 11:17 PM
To: MSRCCS/Prajna<EMAIL_ADDRESS>Subject: [Prajna] How to specify native dll and custom file to remote? (#148)
In your paper, it says there can be non-assembly files uploaded, but I cannot find any example how to do that? any help? sorry, new to Prajna, but it is cool :)

Reply to this email directly or view it on GitHubhttps://github.com/MSRCCS/Prajna/issues/148.


Update script stripping regex to account for attributes

Apparently script stripping breaks in Ember-CLI 2.7 apps when the script tags have been updated to look like the following:
<script src="{{rootURL}}assets/vendor.js"></script>

I'm happy to send in a PR that matches that with the regex that is there.  Or what do you think about having an optional override for the regex so that add-on users can tailor things to their needs?

Hi @acorncom, thanks for pointing this out. A PR would be awesome!
As far as an override goes, I think it may be overkill as users could just install ember-cli-replace if they want to do anything beyond stripping out the script tags. Happy to discuss more though, maybe I'm overlooking a use case.
FYI anything that happens here will probably be ported over to ember-cli-staticboot which I've been working on off the back of this.


Version number and nuget package name

Update version number to 1.1 and nuget package name to 'Tudormobile.QIFLibrary'

See the WPFApp repo for guidence.


Simplest query

The following code does not work
%load_ext pyprql.magic

%prql duckdb://

!wget https://raw.githubusercontent.com/graphql-compose/graphql-compose-examples/master/examples/northwind/data/csv/products.csv

%%prql
from 'products.csv'

and it probably should

Yes agree, the docs need an update, I'll try and get to that today.

FYI I didn't get to this today, but this & I'll prioritize this & https://github.com/PRQL/prql/issues/1531, hopefully tomorrow


@octokit/request was removed (?)

Check Action error, somehow the package was removed from the npm registry, any ideas on how to fix this?

The problem seems to have been temporary. The @octokit/request dependency is still available: https://www.npmjs.com/package/@octokit/request

Oh yeah I see! Nice!


images diaplay #1031

Description
In Lite Api 'image_front_url' was written as 'imageFrontUrl' in #909 . so the image url was always null.
On home pressed I finished the productAcivity.
Related issues and discussion
closes #1031
Checklist
Make sure you've done all the following (You can delete the checklist before submitting)

[x] Apply the AndroidStyle.xml style template to your code in Android Studio.
[x] Code is well documented
[x] All user-visible strings are made translatable
[x] Code passes Travis builds in your branch
[x] If you have multiple commits please combine them into one commit by squashing them.
[x] Read and understood the contribution guidelines .

Merged, thanks @jaztriumph!


updates SI collection

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md

I am:

[x] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[ ] Unaffiliated with the project (not self-promoting as e.g. a maintainer, creator, contractor or employee)

EDITING AN EXISTING TOPIC OR COLLECTION
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[x] Content (and my changes are in index.md)

Please explain why these changes are necessary:
CURATING A NEW TOPIC

[ ] I've formatted my changes as a new folder directory, named for the topic as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME])
[ ] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please explain why you think this topic should be curated:
CURATING A NEW COLLECTION

[ ] I've formatted my changes as a new folder directory, named for the collection as it appears in the URL on GitHub (e.g. https://github.com/collections/[NAME])
[ ] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please explain why you think this collection should be curated:
Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.
cc: @damaneice @brandonrosage

Hey @github/site-engineering could one of you review/merge these changes in? I do expect to make more changes to the Social Impact Collection but I expect it will be a few weeks before I need to do that and wanted to push these updates through now, if possible. Thanks!!!

Giving this another bump as I haven't heard back from anyone. Please let me know if there's anything on my end that's keeping this from being merged. Thanks!


FE: folder-item highlighted when droppable content is dragged over

Requirements

Folder items should be highlighted when any droppable content is dragged on top


Use flask app.get and app.post

Much cleaner, most of the time.
But does this work for blueprints?

No, this does not seem possible for blueprints.


Here's a new site I'd like you to consider blocking

Here's a new site I'd like you to consider blocking:
www.nytimes.com
(please include any other relevant details)

Unfortunately, their content is locked behind a paywall and I don't have an account, so I can't see any comments. I tried creating a free account but that allowed me to read a single article that didn't seem to have comments anyway.
If you can provide a class name or id for the section of the page that shows the comments (you'll need to know a bit about css selectors), I'll add them to the list.

Nevermind. I discovered that I can simply decline cookie access and then all comments disappear. The nytimes restyles their comments section frequently so I doubt the juice is worth the squeeze here


Removing all instances of a Message does not remove the Message Element XML: Only way to remove Messages is by editing the XML...

When creating messages on Receive Tasks and Message Events, you can use the + button to create a new message.  But once created, there is no way to delete the message.
You have to edit the XML directly and find the Message Element in question and delete it.
The User should have the ability to delete the message without having to exit the XML.  Leaving unused Message Element XML is bad hygiene

This relates to https://github.com/camunda/camunda-modeler/issues/136. It's not a bug, but a missing feature.


Propose a change for floating number format for mean climate metrics …

Current compute_metrics.py used either ".5f" or ".3f" to form the floating numbers for the output from the mean climate driver. As the magnitude of the model variable can range from 1e5 (e.g. pressure ) to 1e-3 (e.g. humidity), using the ".5f" and ".3f" can sometimes truncate the small values to zero. It seems that using a standard scientific notation can be beneficial if the user processes the data without converting the unit data ahead. In this way, it also provides possibilities for users to convert the units within the diagnostic datasets.
I submit this pull request to trigger some discussions and see if a better solution can be made to the PMP master code.

An example can be seen in the PMP metrics data for zonal wind stress (tauv) below:

https://raw.githubusercontent.com/PCMDI/pcmdi_metrics_results_archive/refs/heads/main/metrics_results/mean_climate/cmip6/historical/v20220928/tauu.cmip6.historical.regrid2.2p5x2p5.v20220928.json

You can see that the biases in many metrics results are zeros ,while the mean values of this variable are around 10^{-2}. I think for this scenario, 3 digt decimal points for biases are not enough to  provide useful information.

@zhangshixuan1987 thank you for the PR. I am merging it now.


Unable to Pack when dll mapping is specified

I have a VSTS task to pack the solution files using a mapping files to include dll's and webresources.
When i run the command locally using the solution packager and the same mapping xml file it packs with no errors.
However when i pack on VSTS the solution packager build task cannot find a mapping matching my dll.
I must be doing something wrong in my mapping file but cannot tell what. The packager works fine if i remove the dll file to file mapping:
<?xml version="1.0" encoding="utf-8"?>
<Mapping>
  <FileToPath map="*.jpg" to="..\WebResources\img\" />
  <FileToPath map="*.png" to="..\WebResources\img\" />
  <FileToPath map="*.gif" to="..\WebResources\img\" />
  <FileToPath map="*.js" to="..\WebResources\js\" />
  <FileToPath map="*.html" to="..\WebResources\html\" />
  <FileToPath map="*.css" to="..\WebResources\css\" />
  <FileToFile map="xxxxCMCRM2016.dll" to="..\..\xxxx.CM.CRM2016\bin\**\xxxx.CM.CRM2016.dll" />
</Mapping>

Output from the build on VSTS:

2017-05-16T13:45:49.9621209Z Processing Component: SolutionPluginAssemblies
2017-05-16T13:45:49.9621209Z  - xxxx.CM.CRM2016, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=4139be68fb23a728
2017-05-16T13:45:49.9621209Z Matched file 'd:\a\1\s\xxxx.CharityManagement\SolutionFiles\PluginAssemblies/xxxxCMCRM2016-23E903C6-6AAC-4739-A4DE-FA2D3C51503F/CHORUSCMCRM2016.dll' was not found in any mapping. Pack operation is aborted.
2017-05-16T13:45:49.9631206Z Cannot find required file 'd:\a\1\s\xxxx.CharityManagement\SolutionFiles\PluginAssemblies\xxxxCMCRM2016-23E903C6-6AAC-4739-A4DE-FA2D3C51503F\xxxxCMCRM2016.dll'.
2017-05-16T13:45:50.1251207Z ##[error]Solution Pack operation failed with exit code: 1
2017-05-16T13:45:50.1391198Z ##[section]Finishing: MSCRM Pack Solution

Also there are no errors reporting when unpacking using the same script and the dll is not extracted from the customizations zip file.

Did you try to map directly to the release folder?

Thanks @WaelHamze, total NOOB reason!
I had not put a build task into by build definition and so the binaries were never created on the build server.
Problem solved!


incompatible with weaponstab

I saw this in log
System.TypeInitializationException: The type initializer for 'Numbers.StaticConstructorOnGameStart' threw an exception. ---> System.ArgumentNullException: Value cannot be null.
Parameter name: type
  at System.Activator.CreateInstance (System.Type type, System.Boolean nonPublic) [0x00003] in <567df3e0919241ba98db88bec4c6696f>:0
  at System.Activator.CreateInstance (System.Type type) [0x00000] in <567df3e0919241ba98db88bec4c6696f>:0
  at RimWorld.PawnColumnDef.get_Worker () [0x00008] in <c36f9493c9844ddaa7fb5c788416098f>:0
  at Numbers.StaticConstructorOnGameStart+<>c.<AddTrainablesToAnimalTable>b__21_0 (RimWorld.PawnColumnDef x) [0x00000] in <87052afb44fa40d281208d2dc8848ab6>:0
  at System.Linq.Enumerable+WhereListIterator`1[TSource].MoveNext () [0x00037] in <351e49e2a5bf4fd6beabb458ce2255f3>:0
  at Numbers.StaticConstructorOnGameStart.AddTrainablesToAnimalTable () [0x00076] in <87052afb44fa40d281208d2dc8848ab6>:0
  at Numbers.StaticConstructorOnGameStart..cctor () [0x00102] in <87052afb44fa40d281208d2dc8848ab6>:0
   --- End of inner exception stack trace ---
  at (wrapper managed-to-native) System.Object.__icall_wrapper_mono_generic_class_init(intptr)
  at Numbers.WorldComponent_Numbers.NotifySettingsChanged () [0x0000b] in <87052afb44fa40d281208d2dc8848ab6>:0
  at Numbers.WorldComponent_Numbers.FinalizeInit () [0x000c4] in <87052afb44fa40d281208d2dc8848ab6>:0
  at RimWorld.Planet.WorldComponentUtility.FinalizeInit (RimWorld.Planet.World world) [0x00013] in <c36f9493c9844ddaa7fb5c788416098f>:0

https://gist.github.com/idkman2021/4e1ecc3a5701859e7b137ec67203a729

also presets are not loading.
I didn't have numbers in my save and now I added it and it doesnt work also I added another mod WeaponsTab and it also doesn't work the tab doesn't open
EDIT I disabled weaponstab and this numbers works now and no error so it is incompatible with weaponstab

That's an impressive error. Seems like weaponstab triggers it though. From your log:
Exception loading from System.Xml.XmlElement: System.MissingMethodException: Default constructor not found for type Verse.AI.ThinkNode
  at System.RuntimeType.CreateInstanceMono (System.Boolean nonPublic) [0x0007b] in <567df3e0919241ba98db88bec4c6696f>:0
  at System.RuntimeType.CreateInstanceSlow (System.Boolean publicOnly, System.Boolean skipCheckThis, System.Boolean fillCache, System.Threading.StackCrawlMark& stackMark) [0x00009] in <567df3e0919241ba98db88bec4c6696f>:0
  at System.RuntimeType.CreateInstanceDefaultCtor (System.Boolean publicOnly, System.Boolean skipCheckThis, System.Boolean fillCache, System.Threading.StackCrawlMark& stackMark) [0x00027] in <567df3e0919241ba98db88bec4c6696f>:0
  at System.Activator.CreateInstance (System.Type type, System.Boolean nonPublic) [0x00020] in <567df3e0919241ba98db88bec4c6696f>:0
  at System.Activator.CreateInstance (System.Type type) [0x00000] in <567df3e0919241ba98db88bec4c6696f>:0
  at Verse.DirectXmlToObject.ObjectFromXml[T] (System.Xml.XmlNode xmlRoot, System.Boolean doPostLoad) [0x0051d] in <c36f9493c9844ddaa7fb5c788416098f>:0
  at Verse.DirectXmlToObject.ObjectFromXmlReflection[T] (System.Xml.XmlNode xmlRoot, System.Boolean doPostLoad) [0x00000] in <c36f9493c9844ddaa7fb5c788416098f>:0
  at Verse.DirectXmlToObject.ObjectFromXml[T] (System.Xml.XmlNode xmlRoot, System.Boolean doPostLoad) [0x0091e] in <c36f9493c9844ddaa7fb5c788416098f>:0

WeaponsTab has an issue with some thinknode, and the resulting error puts the reflection stack in an unstable state or something.
at RimWorld.PawnColumnDef.get_Worker
=>

public class PawnColumnDef : Def
{
	public Type workerClass = typeof(PawnColumnWorker);

	public PawnColumnWorker Worker
	{
		get
		{
			if (workerInt == null)
			{
				workerInt = (PawnColumnWorker)Activator.CreateInstance(workerClass);
				workerInt.def = this;
			}
			return workerInt;
		}
	}
}

There's no way for workerClass to be null, unless something is really borked. I can't fix that, and I don't really want to write a bunch of error handling code for such a wonky error.
I'm grateful for the report, and now I'm going to close this as won't fix.


Replace templating and custom deployment with Helm

This PR replaces the custom OFC installation scripts (shell script
and golang) with the OFC Helm chart which is pulled from the OFC release
specified in the init.yaml.
This PR requires https://github.com/openfaas/openfaas-cloud/pull/680 merged, and ofc-version master selected in ofc-bootstrap, or waiting for a new ofc release
Fixes #223
Fixes #154
Signed-off-by: Alistair Hey<EMAIL_ADDRESS>Description
How Has This Been Tested?

This has been tested using my init.yaml taken from a working cluster and
applied to a new cluster (same init.yaml) with the new deployment method
using the chart
This was using Route53 DNS on a Civo k3s cluster
Checklist:
I have:

[x] checked my changes follow the style of the existing code / OpenFaaS repos
[ ] updated the documentation and/or roadmap in README.md
[x] read the CONTRIBUTION guide
[x] signed-off my commits with git commit -s
[ ] added unit tests

I want to get the upstream chart changes merged and a few deploys over a few sets of options (customer URL, customers secret, DO DNS, etc) before im willing to remove the WIP status.
I also need to do a full feature-comparison with whats been removed vs added.

@Waterdrips please could you see my comments?

Yeh there's going to be a load of re work now we have multi arch.
I'll address comments while doing the updates


Update authors.md

I forgot to add author in the previous PR.
I will continue to contribute my insights as well. :)

Please continue!


sorbet-runtime fails to consume sig for respond_to_missing

Input
→ View on sorbet.run
# typed: strict

require_relative './gems/sorbet-runtime/lib/sorbet-runtime'

class A
  extend T::Sig

  sig {params(m: Symbol, include_private: T::Boolean).returns(T::Boolean)}
  def respond_to_missing?(m, include_private=false)
    true
  end
end

A.new.respond_to?(:foo)

Observed output
This is a bug in the runtime.
Traceback (most recent call last):
        12: from foo.rb:13:in `<main>'
        11: from foo.rb:13:in `respond_to?'
        10: from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/_methods.rb:212:in `block in _on_method_added'
         9: from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/_methods.rb:373:in `maybe_run_sig_block_for_method'
         8: from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/_methods.rb:377:in `maybe_run_sig_block_for_key'
         7: from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/_methods.rb:397:in `run_sig_block_for_key'
         6: from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/_methods.rb:203:in `block in _on_method_added'
         5: from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/_methods.rb:298:in `run_sig'
         4: from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/_methods.rb:341:in `build_sig'
         3: from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/signature_validation.rb:39:in `validate'
         2: from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/signature.rb:18:in `new_untyped'
         1: from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/signature.rb:18:in `new'
/Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/signature.rb:72:in `initialize': The declaration for `respond_to_missing?` has arguments with duplicate names (RuntimeError)
Reflection: [[:req], [:req]]
From sig:   {nil=>#<T::Private::Types::NotTyped:0x00007fcec78b4438>}

Note: those last two lines are two extra lines of debugging information I added. They are not printed on master at time of report.

Expected behavior

The sig provides types for all the arguments of the method, so there should be no error.

(repro courtesy of @damolina-stripe)

This happens because we attempt to find a signature for A#respond_to_missing?'s super method:
https://github.com/sorbet/sorbet/blob/49e90fca13518a24f82de9e879f7a8e6cafea271/gems/sorbet-runtime/lib/types/private/methods/signature_validation.rb#L22-L44
We don't find one, so we attempt to construct an untyped signature.  To construct that untyped signature, we take what Kernel.method(:respond_to_missing?).parameters tells us:
irb(main):001:0> Kernel.method(:respond_to_missing?).parameters
=> [[:req], [:req]]

and then we turn it into a hash for the arguments:
https://github.com/sorbet/sorbet/blob/49e90fca13518a24f82de9e879f7a8e6cafea271/gems/sorbet-runtime/lib/types/private/methods/signature.rb#L14-L16
but this process loses information about how many arguments we had, since #parameters indicated that we have two arguments with the same "name"...and then we hit the error from the original report.
I think this will happen when attempting to override any method that's written in C with more than one parameter, since we don't have appropriate parameter information for such methods.  But I would have suspected such problems to show up before now?

This happens because we attempt to find a signature for A#respond_to_missing?'s super method:
https://github.com/sorbet/sorbet/blob/49e90fca13518a24f82de9e879f7a8e6cafea271/gems/sorbet-runtime/lib/types/private/methods/signature_validation.rb#L22-L44
We don't find one, so we attempt to construct an untyped signature.  To construct that untyped signature, we take what Kernel.method(:respond_to_missing?).parameters tells us:
irb(main):001:0> Kernel.method(:respond_to_missing?).parameters
=> [[:req], [:req]]

and then we turn it into a hash for the arguments:
https://github.com/sorbet/sorbet/blob/49e90fca13518a24f82de9e879f7a8e6cafea271/gems/sorbet-runtime/lib/types/private/methods/signature.rb#L14-L16
but this process loses information about how many arguments we had, since #parameters indicated that we have two arguments with the same "name"...and then we hit the error from the original report.
I think this will happen when attempting to override any method that's written in C with more than one parameter, since we don't have appropriate parameter information for such methods.  But I would have suspected such problems to show up before now?

Wow thanks for the investigation @froydnj! Nice find.

Wow thanks for the investigation @froydnj! Nice find.


Please allow for a metric-based clock

Currently, the clock panel does not allow the use of any metrics. This could be useful, however; both to have a clock running since an event in the past (e.g., the boot of a device) or an event in the future (e.g., expected expiration of a TLS certificate).
Please allow for using metrics to select the countdown timer or clock start point.

would still be nice, but I'm not going to fight bots :shrug:


No transparency in Mac OS Big Sur

No transparency in Mac OS Big Sur. Can you please help?

@hovnatan Same bug with me .. If you could solve it please share the solution and me too

@hovnatan If you want to make app windows full screen add this script to the game object "UniWindowController"
using UnityEngine;
using System.Threading.Tasks;
using Kirurobo;
public class FullScreen : MonoBehaviour
{
private UniWindowController uniwinc;
async void Start()
{
    // set application window resolution to screen resolution
    Screen.SetResolution(Screen.currentResolution.width, Screen.currentResolution.height, false);

    // must wait to apply window size change (which disables window transparency)
    await Task.Delay(500);

    // re-enable window transparency
    uniwinc = GameObject.FindObjectOfType<UniWindowController>();
    uniwinc.isTransparent = true;
    uniwinc.isTopmost = true;
    uniwinc.isHitTestEnabled = false;
}

void Update()
{

}

}

At least, the current version of LibUniWinC.bundle is not built for Apple silicon, so you need to select Intel 64-bit to run it for Unity 2020.3.

But that's not all. The window behavior is not stable in Unity 2020, e.g. it maximizes when transparency is enabled in my environment.
I haven't tried UniWinC in Unity 2020. I'll look into a solution...

@kirurobo I could make it work on Unity 2019 as explained in my comments above. But still when I choose full screen or maximized widow from player settings, transparency and other properties do not work. I tried adding script to game object and set the properties again after some delay but didn't work.
Also how can I make the app appear on top of dock or top bar of macOS (but without full screen mode)?


Update README.md

I tried it on a Debian buster with a recent kernel (5.4.14).
Debian kernel configuration does not have all required options by default, so I added a command to check the 3 options I have to add.
Also on recent debian DRI devices are created with render group not video group (video is still need for /dev/fkd).
ls -la /dev/dri/renderD129
crw-rw---- 1 root render 226, 129 Jan 24 09:45 /dev/dri/renderD129

This may fix this following issues that have same kind of error I got:
https://github.com/RadeonOpenCompute/ROCm/issues/930
https://github.com/RadeonOpenCompute/ROCm/issues/923
https://github.com/RadeonOpenCompute/ROCm/issues/889

stale PR


Add Support for Sovereign clouds like AzureUSGovernment

It would be great if this action supported AzureUSGovernment clouds.
There are two changes that need to be made to make this happen.

vault.azure.net needs to be changed to vault.usgovcloudapi.net
the --resource tokenArgs needs to be changed to support vault.usgovcloudapi.net as well.

I will submit a PR with an attempt at adding this capability
-Aaron

We certainly could use this for our AzureUSGovernment customers.

@N-Usha any updates here?

Thanks for following up. We have it on our backlog and would be addressing this over next few months. Hope that helps. Thanks!

Confirmed with @ds-ms , since this is fixed, closing the issue.


Add shared credentials support

I've added INIParser as a dependency for reading the config files.
I also removed the SharedCredential.default property since I was unsure what it was intended for. Instead when AWSClient is created, it will attempt to create a SharedCredentials object.

Sweet! Thank you for getting this added!
On first glance, this looks like a pretty clean/simple dependency to add.
Any chance you could add a test for the newly implemented code you added?

No problem. Just committed new tests.

Thank you @oliveroneill!
@noppoMan I think it's alright to bring it in. @jonnymacs mentioned that since we have the SwiftJSON parser from Kitura, bringing in small-scoped dependencies for targeted usecases is reasonable.
If it turns out that we need to fork that code and bring it over, we can move it under the swift-aws org. What do y'all think?

I'm going to go ahead and merge this in 🎉
@noppoMan if I misread you, please let me know and I'm happy to put in the effort to either fork this inside the swift-aws organization or help with other tweaks.
@oliveroneill thank you again for contributing this!! 🙇


Syntax highlighting breaks after using function with string containing colon

When utilizing any function and inputting a string containing a colon (":"), the syntax highlighting for the remainder of the code breaks.
The syntax highlighting should remain intact for the entirety of the code after utilizing the function, irrespective of the presence of a colon in the string.
Steps to Reproduce:

Use the function with a string containing a colon, like so: EventHandler.SendNetworkEvent("giveitem:BFG9000");
Observe the syntax highlighting behavior in the subsequent code.

Thanks for the issue notice. I'll fix it once I can.
I've probably missed something in the strings handling.

This will be a funny response: I can't re-create the issue. I tried copy-pasting your piece of code in a few places to see if there was some kind of edge-case issue, but I can't get the syntax highlighting to break.

Thanks for checking, but it happens to me every time, here is what it looks like:

From the picture you've sent me it seems that the highlighting was broken way before the first double quote.
So, I'll ask you:

May you send me the same screenshot, but with the default vscode theme?
Are you using any string escaping in the code (there's an issue open about this at time of writing)
Is it possible to receive a bigger snippet of code?


[Task] Support of session-based model definition with masking techniques for TF

Problem:
The session-based blocks defined in #451 are implemented as standalone tasks and we need to link them together to provide known session-based models.
Goal:

Implement the high-level blocks (inputs block, next item prediction task) needed for defining sequential models.

Constraints:

#451 should be finalized before starting this work
Update examples and unit tests with the new definition of InputBlock, Masking and Next Item Prediction task.

Starting Point:

[x] Simplify the InputBlock
https://github.com/NVIDIA-Merlin/Merlin/issues/404
Notes:

[x] Simplify the definition of input block by exposing the embedding argument to the user and remove the current nested structure to support 2-D and 3-D inputs.
https://github.com/NVIDIA-Merlin/Merlin/issues/404

[x] Create a SequenceBasedInputBlock that accepts context and sparse schemas. Dropped as the new InputBlockV2 accepts multiple branches and also automatically broadcast context features to 3-D scores.

[x] Add to-2D and to-3D transformations as a post block in the InputBlockV2.
-https://github.com/NVIDIA-Merlin/models/pull/737

[x] Support advanced masking tasks: CLM, MLM (PLM is excluded based on our research paper)
https://github.com/NVIDIA-Merlin/models/pull/780

[x] Implementation of Transformer block  based in the configs defined in HuggingFace - transformers-based library.
Note: This is an example of how Transformer-based architectures are implemented in HF (as keras layers).
https://github.com/NVIDIA-Merlin/models/pull/771

[x] Refactor base model to include a new design of masking/target generation for loss calculation and metrics.
https://github.com/NVIDIA-Merlin/models/pull/780

[x] Update weight-tying strategy  based on the new refactor of the InputBlock (embeddings are independent tables and model context would be removed)
https://github.com/NVIDIA-Merlin/models/pull/715

[x] Support setting the masking task for fit and evaluate
https://github.com/NVIDIA-Merlin/models/pull/780

[x] Example of transformer session-based recommendation for training and deployment
https://github.com/NVIDIA-Merlin/models/pull/1045

Closing this ticket after linking the tasks to the related merged PRs


Allow DefineFunc to define funcs without receiver

Coverage decreased (-20.3%) to 3.954% when pulling 0068ade9398bb2a1dd0422abad421be6a419b315 on matthchr:feature/codegen-helper-improvement into bde77d3101aa89eb9c68d29b7a24cb1b06f19364 on Azure:master.

I've got a nicer version of this in my Test Cases branch, should get the PR out this week.

@theunrepentantgeek - Take a look at https://github.com/Azure/k8s-infra/pull/312/files#r516265457 which I left on your PR. I think we can probably fold this PR into yours, main question I have is around merge timelines and if it'd be better to split some of the astbuilder stuff out of your PR into its own PR.

I've incorporated the extra check into #312

Abandoning this in favor of #313


Getting this error on compressing video obtained after trimming using the flutter package video_trimmer.

E/Transcoder( 9380): Fatal error while transcoding, this might be invalid format or bug in engine or Android.
E/Transcoder( 9380): java.lang.IllegalArgumentException
E/Transcoder( 9380): 	at android.media.MediaMetadataRetriever.setDataSource(MediaMetadataRetriever.java:214)
E/Transcoder( 9380): 	at com.otaliastudios.transcoder.source.UriDataSource.applyRetriever(UriDataSource.java:33)
E/Transcoder( 9380): 	at com.otaliastudios.transcoder.source.DefaultDataSource.ensureMetadata(DefaultDataSource.java:41)
E/Transcoder( 9380): 	at com.otaliastudios.transcoder.source.DefaultDataSource.getLocation(DefaultDataSource.java:138)
E/Transcoder( 9380): 	at com.otaliastudios.transcoder.engine.Engine.transcode(Engine.java:316)
E/Transcoder( 9380): 	at com.otaliastudios.transcoder.Transcoder$1.call(Transcoder.java:134)
E/Transcoder( 9380): 	at com.otaliastudios.transcoder.Transcoder$1.call(Transcoder.java:124)
E/Transcoder( 9380): 	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
E/Transcoder( 9380): 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
E/Transcoder( 9380): 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
E/Transcoder( 9380): 	at java.lang.Thread.run(Thread.java:919)
The thumbnail generation method works fine on the same file.

I'm facing same issue Any update?

I'm facing same issue Any update?

@BilalPervez not yet.

@BilalPervez not yet.

Same issue here

Same issue here

same here.

same here.

Guy you need to compress the video first then trim it,
this is only solution that get right now to get rid of this problem.

Guy you need to compress the video first then trim it,
this is only solution that get right now to get rid of this problem.

same issue

@ajosecueto did you try to compress it first and then apply trimming on it and still facing the same issue?

@BilalPervez Really i'm able to compress without problem using a real device, that happens only in android emulator... i think is because the path where video trimming is not full accessible... idk

Guy you need to compress the video first then trim it, this is only solution that get right now to get rid of this problem.
I was finding the solution since last 2 days but this worked for me..
Thank man!

tempDir = await getTemporaryDirectory();
file = await File(tempDir.path+'Vid${DateTime.now().millisecondsSinceEpoch}.mp4').writeAsBytes(File(updatedPath).readAsBytesSync());
mediaInfo = await VideoCompress.compressVideo(
file.path,
quality: VideoQuality.Res960x540Quality,
deleteOrigin: false,
);
just make a temporary file in cache directory and then give its path in compress.
it will work!!


update docs, changelog, add contributing.md

@jpthiele , @pjaap  I think this can be released now as 1.9.
RGB type piracy has been removed, but pure tuples for rgb colors in the moment are not deprecated. E.g. Matplotlib happily lives with them, and I fear removing this possibility would  break too much existing user code for almost no gain in functionality. I agree though to use the RGB(r,g,b) pattern in all examples and new code.

As for the format check, see https://github.com/fredrikekre/Runic.jl/issues/107


Added actions/action creators/reducers that update redux state when feathers service is changed in real time.

Summary
Added the following combination of actions/reducers that update the redux state  accordingly when a particular feathers service data is changed in real time.
  // ACTION TYPES
const ON_CREATED = `${SERVICE_NAME}ON_CREATED`;
const ON_UPDATED = `${SERVICE_NAME}ON_UPDATED`;
const ON_PATCHED = `${SERVICE_NAME}ON_PATCHED`;
const ON_REMOVED = `${SERVICE_NAME}ON_REMOVED`;

  // ACTION CREATORS
  onCreated: createAction(ON_CREATED, (payload) => ({ data: payload })),
  onUpdated: createAction(ON_UPDATED, (payload) => ({ data: payload })),
  onPatched: createAction(ON_PATCHED, (payload) => ({ data: payload })),
  onRemoved: createAction(ON_REMOVED, (payload) => ({ data: payload })),

The actions above essentially update the queryResult object of the respective reducer/service. It works by dispatching the respective action within the service.on() method, for example:
  const task = feathersApp.service('/task');
    task.on('created', (data) => {
      dispatch(services.task.onCreated(data))
    })
    task.on('updated', (data) => {
      dispatch(services.task.onUpdated(data))
    })
    task.on('patched', (data) => {
      dispatch(services.task.onPatched(data))
    })
    task.on('removed', (data) => {
      dispatch(services.task.onRemoved(data))
    })

Looking forward to use it 💯

I'm a but busy today. I'll review tommorrow (wed).
On Tue, Nov 14, 2017 at 8:02 AM, Russian Rebouças<EMAIL_ADDRESS>wrote:

Looking forward to use it 💯
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/feathers-plus/feathers-redux/pull/37#issuecomment-344252142,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABeznxBsX18Rxgwp8Kl2t3gsRpQRqGTgks5s2Y9RgaJpZM4Qclwj
.

Should I wait on your next PR before publishing this one?

https://github.com/feathers-plus/feathers-redux/pull/39

Could you fix the tests?


liquibase: 3.5.5 -> 3.6.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/liquibase/versions.
These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 3.6.0 with grep in /nix/store/q0h0dm35qda3vls6axin875ky93n0idg-liquibase-3.6.0
directory tree listing: https://gist.github.com/d7568f9f2cd317086e4b519f4749b8d8

cc @nequissimus for review

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: liquibase
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

Failure on x86_64-darwin (full log)
Attempted: liquibase
Partial log (click to expand)
cannot build derivation '/nix/store/by9y7a223z5h9jzzf0c5bvcapv64wclv-pkg-config-0.29.2.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/g4afb6cj74h1vf2lqqbsmw1y3prc1cxf-freetype-2.7.1.drv': 8 dependencies couldn't be built
cannot build derivation '/nix/store/kmmsmmchkb71kvnajlx7qvsm8c4rpcdg-libssh2-1.8.0.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/1j4agb9v3ip2dll3xzii77qgf0058v1v-mysql-connector-java-5.1.46.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/kfm1fc118va0gx7r547kjm48rcw7pkvv-nghttp2-1.24.0.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/wpgpzki7mfwnjx298416p51p89395mwy-curl-7.58.0.drv': 8 dependencies couldn't be built
cannot build derivation '/nix/store/9kv22svnfkycaxvivdyb3d3y0d80aqy3-zulu1.8.0_121-<IP_ADDRESS>.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/mabl8q209fqk2v54v0a1v1gg84i32dnp-liquibase-3.6.0-bin.tar.gz.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/0fc2vhyy8nxcicz55dspdvd4gbqa0ih6-liquibase-3.6.0.drv': 6 dependencies couldn't be built
error: build of '/nix/store/0fc2vhyy8nxcicz55dspdvd4gbqa0ih6-liquibase-3.6.0.drv' failed

Success on x86_64-linux (full log)
Attempted: liquibase
Partial log (click to expand)
no Makefile, doing nothing
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/v0ln491na8p8r2lssfxah1gyhs0wcmsc-liquibase-3.6.0
strip is /nix/store/sfh1pwkz226kj57q8q9g63p9hwv53340-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/v0ln491na8p8r2lssfxah1gyhs0wcmsc-liquibase-3.6.0/lib  /nix/store/v0ln491na8p8r2lssfxah1gyhs0wcmsc-liquibase-3.6.0/bin
patching script interpreter paths in /nix/store/v0ln491na8p8r2lssfxah1gyhs0wcmsc-liquibase-3.6.0
/nix/store/v0ln491na8p8r2lssfxah1gyhs0wcmsc-liquibase-3.6.0/bin/liquibase: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/fg1s5amakn3w817xig75mmxgziahnva2-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/v0ln491na8p8r2lssfxah1gyhs0wcmsc-liquibase-3.6.0...
/nix/store/v0ln491na8p8r2lssfxah1gyhs0wcmsc-liquibase-3.6.0


It is not opening after i launched it once

Hello , i have onced opened chris titus tool and now it is not opening again even tho i entered all different commands to open it . what should i do? when i type the command , nothing is happening

I have the same problem. Power Shell and CMD is just blank when trying both .exe and official shortcut both with and without administrator privileges. Thank you in advance!

Works for me now running "irm christitus.com/win | iex" in PowerShell. Maybe a temporary issue with the link or your DNS?

irm christitus.com/win | iex

You are 100% correct in my case. Tried with a VPN and it worked. So most probably a DNS issue on my side and it also makes sense since I have a new router. Thank you! I consider it solved!


AsyncMongoMockClient dosen't suport AsyncIOMotorGridFSBucket

This simple test:

import pytest
from mongomock_motor import AsyncMongoMockClient
from motor.motor_asyncio import AsyncIOMotorGridFSBucket

@pytest.mark.asyncio
async def test_dummy():
    mongo_client = AsyncMongoMockClient()
    mongo_db = mongo_client["test_database"]
    grid_fs = AsyncIOMotorGridFSBucket(mongo_db)

Provide this error:
self = <motor.motor_asyncio.AsyncIOMotorGridFSBucket object at 0x107e0d710>
database = <motor.motor_asyncio.AsyncIOMotorDatabase object at 0x107b091d0>
bucket_name = 'fs', chunk_size_bytes = 261120, write_concern = None
read_preference = None, collection = None
def __init__(
    self,
    database,
    bucket_name="fs",
    chunk_size_bytes=DEFAULT_CHUNK_SIZE,
    write_concern=None,
    read_preference=None,
    collection=None,
):
    """Create a handle to a GridFS bucket.

    Raises :exc:`~pymongo.errors.ConfigurationError` if `write_concern`
    is not acknowledged.

    This class conforms to the `GridFS API Spec
    <https://github.com/mongodb/specifications/blob/master/source/gridfs/gridfs-spec.rst>`_
    for MongoDB drivers.

    :Parameters:
      - `database`: database to use.
      - `bucket_name` (optional): The name of the bucket. Defaults to 'fs'.
      - `chunk_size_bytes` (optional): The chunk size in bytes. Defaults
        to 255KB.
      - `write_concern` (optional): The
        :class:`~pymongo.write_concern.WriteConcern` to use. If ``None``
        (the default) db.write_concern is used.
      - `read_preference` (optional): The read preference to use. If
        ``None`` (the default) db.read_preference is used.
      - `collection` (optional): Deprecated, an alias for `bucket_name`
        that exists solely to provide backwards compatibility.

    .. versionchanged:: 3.0
       Removed support for the `disable_md5` parameter (to match the
       GridFSBucket class in PyMongo).
    .. versionchanged:: 2.1
       Added support for the `bucket_name`, `chunk_size_bytes`,
       `write_concern`, and `read_preference` parameters.
       Deprecated the `collection` parameter which is now an alias to
       `bucket_name` (to match the GridFSBucket class in PyMongo).
    .. versionadded:: 1.0

    .. mongodoc:: gridfs
    """
    # Preserve backwards compatibility of "collection" parameter
    if collection is not None:
        warnings.warn(
            'the "collection" parameter is deprecated, use "bucket_name" instead',
            DeprecationWarning,
            stacklevel=2,
        )
        bucket_name = collection

    db_class = create_class_with_framework(AgnosticDatabase, self._framework, self.__module__)

    if not isinstance(database, db_class):
        raise TypeError(
            "First argument to %s must be  MotorDatabase, not %r" % (self.__class__, database)
        )

  self.io_loop = database.get_io_loop()

E       TypeError: 'AsyncIOMotorCollection' object is not callable

I've created PR with changes that should enable working with GridFS (see added test for the example). Can you check if your use case now works?


Fixed fallthrough warning

Fixed fallthrough warning

Thanks.  I am working on a new epoll implementation for linux that also fixed these.  Coming soon


Use location instead of organization

I would like to use Synthea to generate data that references Location resources (for facilities) rather than Organization resources. Thoughts on what a PR would require to support it and if the maintainers would accept it?

Hmm... well, a straight-forward PR would require changing four different exporters: fhir (STU3), fhir_dstu2 (DSTU2), hospital.fhir (STU3), and hospital.fhir_dstu2 (DSTU2).
Those exporters can be found in src/main/java/org/mitre/synthea/export.
However, I don't think we'd accept a PR that just made a one-for-one swap... well, maybe we would (I'm open to persuasion), but that would might just need to be changed again. Perhaps where we want to go is to have both Organization and Location together once we add clinicians as Practitioner resources and we can generate synthetic data suitable for a Provider Directory, e.g. http://www.fhir.org/guides/argonaut/pd/index.html
@citizenrich what is your use-case for Location? Provider Directory? What advantage do you see Location having over Organization?
I'd be interested in having others comment on this issue. Adding clinicians/practitioners, and locations could be a big chunk of work.

Indeed, thanks for the discussion. It would seem like quite a bit of work. I'm happy to have it discussed and learn what you all think.
The use case is to try to match what we (a global health NGO) see in resource-constrained countries and Location resources are more accurate for such contexts. So, imagine creating a synthetic dataset that includes immunizations for vaccine-preventable diseases in Liberia where most facilities are government-led and small. Hospitals as Organizations aren't a terribly accurate representation. But, I could be wrong.

Just updating this issue to note that there's now an international locations repository to support Synthea:
https://github.com/synthetichealth/synthea-international


[OSDEV-1384] SPIKE: Add opensearch-query-service for OpenSearch testing (partial impl)

Helper service in addition to OSDEV-1384

This tool is intended to run locally via CLI
When run containers, go to opensearch-query-service container using this commnad:
dokcer exec -it opensearch-query-service /bin/bash
Inside container run:
python main.py climate-trace-data-min.csv --name-search --address-search

As a result, there should be created OpenSearch query object based on src/opensearch-query/query.json and send it along with POST http://opensearch-single-node:9200/production-locations/_search to the local OpenSearch, get result and populate it in the console.
This service uses limited data from the ClimateTrace for test purpose. It require futher adjustments.

@VadimKovalenkoSNF Tested tool locally, looks good as a result of current state in the spike.

Hi @VadimKovalenkoSNF,
Do you think we can close this PR to avoid having it in the list of open PRs?

Hi @VadimKovalenkoSNF, Do you think we can close this PR for now to avoid having it in the list of open PRs?

cc @VadimKovalenkoSNF


Various accessibility API updates.

Objective
bevy_a11y was impossible to integrate into some third-party projects in part because it insisted on managing the accessibility tree on its own.
Solution
The changes in this PR were necessary to get bevy_egui working with Bevy's AccessKit integration. They were tested on a fork of 0.11, developed against bevy_egui, then ported to main and tested against the ui example.
Changelog
Changed

Add bevy_a11y::ManageAccessibilityUpdates to indicate whether the ECS should manage accessibility tree updates.
Add getter/setter to bevy_a11y::AccessibilityRequested.
Add bevy_a11y::AccessibilitySystem SystemSet for ordering relative to accessibility tree updates.
Upgrade accesskit to v0.12.0.

Fixed

Correctly set initial accessibility focus to new windows on creation.

Migration Guide
Change direct accesses of AccessibilityRequested to use AccessibilityRequested.::get()/AccessibilityRequested::set()
Before
use std::sync::atomic::Ordering;

// To access
accessibility_requested.load(Ordering::SeqCst)
// To update
accessibility_requested.store(true, Ordering::SeqCst);

After
// To access
accessibility_requested.get()
// To update
accessibility_requested.set(true);

One of the checks failed, looks like a problem with the Cargo.lock file, has a duplicate entry. Since the PR doesn't include Cargo.lock you be able to re-run the tests or rebase.

Interesting, thanks, any idea how a duplicate entry got in there? Didn't even know that was possible.

I'm not clear where the conflict is, I merged main and everything went fine. Is this a github bug or am I genuinely missing something?

OK, maybe that got it, just spotted the link to resolve.


Unhelpful selection color for blue backgrounds

This is a Bug Report

Problem: In a few places, we use a blue background. For example, part of https://kubernetes.io/training/
In that case the blue selection color from https://github.com/kubernetes/website/pull/40796 is problematic.
Example Solution:
Use background-color: initial; and possibly color: initial; on the selection to let the browser pick something appropriate.

Additional Information:
/area web-development

/retitle Unhelpful selection color for blue backgrounds

/assign

I don't think #40802 works, as far as I can tell it removes selection completely

/triage accepted

/unassign
/assign @mostafahanafi

Fixes are welcome.

I think I can help, Should I take this?

Anyone is welcome to open a pull request to propose a fix.
@Gauravpadam you might like to try to liaise with people who've previously said they intend to start work; it's polite to do that, but not mandatory.

Anyone is welcome to open a pull request to propose a fix.
@Gauravpadam you might like to try to liaise with people who've previously said they intend to start work; it's polite to do that, but not mandatory.

Absolutely; If the assignee wants to collaborate, I can be of help

/remove-lifecycle stale

@mostafahanafi Are you still working on this issue?


fix: support globbing from dependencies

Fixes #2390
The condition source.includes('import.meta.glob') won't catch source code like import.meta['glob'] or new lines between meta and glob, but the subsequent transform() code won't either anyways.
        const { s: start, e: end, ss: expStart, d: dynamicIndex } = imports[
          index
        ]

        // This as well doesn't catch `import.meta['glob']`
        const isGlob =
          source.slice(start, end) === 'import.meta' &&
          source.slice(end, end + 5) === '.glob'

The advantage of source.includes('import.meta.glob') is that we skip es-module-lexer parsing.

(Forced pushed to change the commit message from fix: to feat:.)

It's high prio for vite-plugin-ssr. I wrote a patch that simulates this PR (https://github.com/brillout/vite-fix-2390) but it's painful to make sure that the patch works against each release.

This looks good to me, but I think @yyx990803 explained in an issue why globbing from dependencies was not a good idea (but I can not find it). We'll need to wait for his input about this one.

There was a previous request for adding support for globbing from dependencies at https://github.com/vitejs/vite/issues/1875 which got fixed by Evan You. So I'm assuming this PR would get accepted as well.

@matias-capeletto Maybe we should label this "p3-downstream-blocker" since it's blocking vite-plugin-ssr.

@brillout could we add a test for this feature?

@matias-capeletto Yes! That would actually be great to solidify that use case. I'll make a stab at it (I'm bit woried about how I can use a non-symlinked dependency with the current test architecture, but I'll dig into this). Sorry for the late reply, I was heads down implementing Client-side Routing for vite-plugin-ssr.

@patak-js I'm almost done with the test. I expect the test to get some back-and-forth with the reviewers so I'm a bit worried that it may delay the merging of this PR. If it's ok with you I will open a second PR for the test so that this PR can get merged quicklier. Please let me know if you object.

I am fine with that, but this counts as a new feature so we need @yyx990803 approval to merge it.

I just changed the commit prefix from fix: to feat:. So I guess Shinigami92's approval still holds.

I'm almost done with the test

Done: https://github.com/vitejs/vite/pull/2740.

I hit this too building the Solid Starter. For now patching it but look forward to a solution being merged.


fpr_table_cv_summary_memo fails.  waterfall culvert summaries break build as they are not in the pscis_phase2 object and do not have photo directories under PSCIS ID name

we can beat this by setting fpr_table_cv_summary_memo params to
dat = pscis_phase1, site = {my_crossing_reference}, site_photo_id = {my_crossing_reference}, col_filter = my_crossing_reference

I changed the my_crossing_reference for both sites to their psci_ids in the phase2 sheet but forgot to commit the xlsm. Wouldn't that be easier? As opposed to altering the fpr_table_cv_summary_memo params

Actually ws able to address by taking care of #41 because then we get a pscis id in the object and a caption that reads correctly