Can not restart 0.3.0 under Windows

Description
Once exited out of Zap 0.3.0, you can not restart it under Windows.

Expected Behavior

After exiting out of Zap, once should be able to restarting it.
Actual Behavior

After exiting out of Zap, it  does not restart.
Possible Fix

Not a fix, but a work-aroud.  Try starting Zap again.
Steps to Reproduce

Check Task Manager and make sure no Zap processes are running

Start Zap

Check Task Manager and you'll see 3 Zap processes are running

Exit Zap from the menu

Check Task Manager and you'll see 2 Zap processes are still running

Try starting Zap, a 3rd process will pop up and disaprear quickly but Zap won't start

End the 2 running Zap processes

Try starting Zap and it should start

Context

Stop me using it until I reboot or kill the running processes.
Your Environment

Zap Desktop version: 0.3.0
Operating System and version: Windows 10

@tooraj-enayati thanks for reporting - we just released 0.3.1 which resolves this issue.


Feature/test decision maker

Status
DEVELOPMENT
Description
Added tests for functions  in decision_maker
Related PRs
#1609

@s-azumi Would you look into 1 failure in GitLab CI?
$ colcon test-result
build/decision_maker/test_results/decision_maker/rosunit-test-decision_maker.xml: 4 tests, 0 errors, 1 failure, 0 skipped

#1964 Fixes the CI problem

@sgermanserrano Yes, I'll add some tests, but it takes time to add everything.
and I'll add License notices to the files.

@s-azumi I have approved the PR in case you want to merge and spawn another branch to add more tests, let me know if I can help with anything


COMP: Update to new ITK_DISALLOW_COPY_AND_MOVE macro

Replace the deprecated ITK_DISALLOW_COPY_AND_ASSIGN with
new better descriptive name ITK_DISALLOW_COPY_AND_MOVE.

Hey @hjmjohnson , correct me if I'm wrong but the failed builds don't seem to reflect issue with the minor changes.  Would you like me to go ahead and merge?

@ntustison  It builds fine on my machine.  I think the failure is a broken testing environment.    It should be good to merge now.
@mseng10 Please look into fixing the testing environment for this package.

Great.  Thanks @hjmjohnson


Lazy DataSource Reader

Add the ability to read a data source and create a deferred dataframe with the source's schema.
There should be a common interface for:

[x] CSV
[x] JSON
[x] SQL
[x] Arrow File

SQL is partly implemented, only Postgres is supported for now


Fix letter spacing next line break

There was the artefact that was causing that.
I replayed all examples, everything seems up again.

oops I should have checked better, thanks for the quick fix.
I also fixed a bug I introduced while linting in your example.. didn't notice your where reassigning letterSpacingSpeed..
Just curious, are you working with HIA Technologies ?

I do not recognize @swingingtom, but I work over at HIA @felixmariotto. Is there anything I can do for you? Thanks @swingingtom for this awesome feature addition.

@JoeCoppola-HIA actually I was just wondering, I know you guys use three-mesh-ui for one of your project and I understood that you wanted to contribute at some point. Since @swingingtom is in a row I thought they might work with you. Sheer curiosity !

Hey @felixmariotto, all good! We for sure want to contribute, just rapidly approaching some deadlines at the year end. Was actually looking into the frame shadow casting a little today. Hope to have some stuff for you in the upcoming year!

As @JoeCoppola-HIA said, Im not working in HIA team. At least for now ;)

oops I should have checked better, thanks for the quick fix.

Same applies for myself. I might have been confused by the fact that I've three live examples pages open, official, forked and localhost. Anyways, its a good reminder to be cautious.


More problems reading metadata

In reference to issue #130, this also seems to affect the generation of content URLs. Specifically, segments of the url_format that refer to metadata aren't available when the URL is generated. Here's an example:
class Photo < ActiveRecord::Base

  image_accessor :image
  # details omitted for brevity

end

>> p = Photo.first
  Photo Load (0.2ms)  SELECT "photos".* FROM "photos" LIMIT 1
=> #<Photo id: 2, release_date: "2011-06-21", created_at: "2011-10-10 23:32:17", updated_at: "2011-10-10 23:32:17",  image_uid: "2011/10/10/19_32_17_281_DSC_0171.jpg">
>> p.image.thumb('200x200').url
=> "/media/BAhbB1sHOgZmIikyMDExLzEwLzEwLzE5XzMyXzE3XzI4MV9EU0NfMDE3MS5qcGdbCDoGcDoKdGh1bWIiDDIwMHgyMDA"
>> p.image.apply
=> # long response omitted
>> p.image.thumb('200x200').url
=> "/media/BAhbB1sHOgZmIikyMDExLzEwLzEwLzE5XzMyXzE3XzI4MV9EU0NfMDE3MS5qcGdbCDoGcDoKdGh1bWIiDDIwMHgyMDA/DSC_0171.jpg"

Notice how on the second call to url the :basename and :format segments are properly included?
This ends up creating different URLs for content depending on whether the metadata has been loaded or not, and thus makes it harder for browsers to make a cache-hit...
I'm happy to look at patching this, but I'm not sure where to start. Suggestions?

This is fixed now - the url will be consistent and will NOT include the name at the end unless the .._name column exists
https://github.com/markevans/dragonfly/blob/master/spec/functional/model_urls_spec.rb#L60-65


Cherrypick bump streaming test timeouts into 2.47.0 release branch

Bump streaming WC test timeout

Bump metrics test timeout

Should result in integration tests being more reliable.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python_Runners PreCommit

Looks like ten minutes is not a perfect timeout length for the worcount test but we'll figure it out

Run Python_Integration PreCommit


Elon's Toys exercise - Missing task instructions for two required steps/funcs

https://exercism.org/tracks/go/exercises/elons-toys
On Elon's Toys exercise you are given instructions for four implementation tasks;

Drive the car
Display the distance driven
Display the battery percentage
Check if a remote control car can finish a race

But there are other two implementations that are required in order to pass the tests which are;
// CreateCar creates a new car with given specifications.
func CreateCar(speed, batteryDrain int) *Car {
	panic("Please implement CreateCar() method")
}

// CreateTrack creates a new track with given distance.
func CreateTrack(distance int) Track {
	panic("Please implement CreateTrack() method")
}

IMO, instructions for these two should also be explicit and included in the tasks list.
I can help fixing this issue right away (it would be my first contribution).

@ferueda Thanks for reporting but we already have an issue for this: https://github.com/exercism/go/issues/1782


[W6.2][W09-B2] Charles Goh

Made changes to css theme and added icon to GUI

Good job attempting this task @charlesgoh


RFID Write embeds line feed character

Using Arduino Mega, MIFARE 1KB tag.
The problem I am having is when using the RFID write example code from the library the code successfully writes the correct block, but it adds a line feed character (0A) to the beginning of the block. I do not wish for it to do this, but I am unaware of how to change this. I would prefer it to simply be a space (20) instead.
The written blocks look like this:

Which exact example?

Which exact example?

rfid_write_personal_data

Works as intended.
  Serial.println(F("Type Family name, ending with #"));
  len = Serial.readBytesUntil('#', (char *) buffer, 30) ; // read family name from serial

As you can see, the code has as end the # and not a line ending. So if you send by console # and press the return button you send two signs. The function copies until the # and the return button sign (depending on settings) a line feed character. The line feed character is buffered, so it appears on your next input.
The behavior is indeed not intuitive. Flush/clearing the buffer could solve the problem, but I think there was no function in the arduino api defined https://github.com/arduino/ArduinoCore-API/blob/fc0c12d7faf43782fd44d2e1384abfef6109da21/api/Stream.h . Or as second solution replace the # by line feed char (but maybe incompatible on different clients).
So far example work. Feel free to create a pull request with an tested better solution or improved documentation.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 245 ms

Resolved: ClamAV performance has improved in 0a44265 after 5 minutes.


Slufigy the metric name

Hello,
Thanks for your work.
Have you planned to slugify the metric name ?
xxx_2379[cluster=default,state=StateLeader] xxx_2379-cluster-default-state-stateleader
Can't use the metric through graphite with the first version.
Thanks again.

Closing for inactivity. Please reopen if these changes are still needed.


Add Rice distribution

This PR adds a learnable Rice distribution which is useful in modeling structure factors in X-ray crystallography.

This implementation uses Rician functional forms for all the attributes. However, it is noteworthy that when nu >> sigma, the Rice distribution becomes indistinguishable from a normal distribution with mu=nu,sigma=sigma. There are further details in the wikipedia artical. My own experiments suggest that switching to a normal PDF when nu/sigma > 40 improves numerical stability. Likely this cutoff could be much lower without impacting accuracy. Further study could be beneficial here if we run into numerical issues with this distribution implementation.

Note there were some breaking changes to the setup-python action for MacOS. This PR will also change the test workflow to support python 3.8/3.9 on macos-13 rather than macos-latest.

going ahead and merging this pr to restore CI and add the Rice distribution. as discussed offline with @minhuanli, the tests in this pr do not assess the validity / accuracy of the implicit reparameterization gradients. we should research a method of testing those cheaply and programmatically across the API. i added #8 to remind us of this shortcoming.


Demo broken

Tried the demo today in both Chrome and Safari and all I got was a black screen after I hit the "demo" button.
http://github.hubspot.com/vex/docs/welcome/

Huh, me too. That's not good! Any ideas what's going wrong?

I would wager a guess that the overlay background is popping forward on the z-index too much and also its stacking cause of the multiple modal windows

same here on chromium ubuntu16.04

I made a workaround in ff15cf5. Basically the overlay is now added directly under the body, so using a different appendLocation other than the body will cause the dialogs to appear behind the overlay, unless that appendLocation also has a z-index greater than the overlay.
@timmfin Not sure if you rebuilt the docs from #241 already, but since this changes the demo index.html we'll need to rebuild again, right?

The issue is still present, so I think it needs to be rebuilt ASAP. Trying to recommend this library 😱!

Yup. Docs have been rebuilt now and the demo is working for me in Chrome and Safari. Apologies for the delay!


fix(sed): detect sed on darwin

use sed if available in darwin, useful for darwin-nix users where gnused is installed as sed

Related issues #146  #101 #81 #44


fatal error class DTS/ebaysdk/filetransfer/service/filetransferservice not found

i am not getting any filereferenceid and taskreferenceid....

It looks like you may be connecting to eBay's sandbox. From what I can tell there may be an issue with the sandbox in that it's not returning the FileReferenceID and TaskReferenceID fields.  Sadly the only thing I can recommend is that you may have better luck with using eBay's production server.


[Bug]执行CLEAN LABEL报错 Encountered: CLEAN

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
V1.1.1
What's Wrong?
执行CLEAN LABEL FROM test_db;或者CLEAN LABEL labelxxxx FROM test_db;
都报错：
ERROR 1105 (HY000): errCode = 2, detailMessage = Syntax error in line 1:
clean label from test_db_pre
^
Encountered: CLEAN
Expected
What You Expected?
CLEAN LABEL成功不报错
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This feature was not supported until v1.2


Error  Unexpected token '.' on init docusaurus

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
I get Unexpected token '.' error when running docusaurus project creation and get the same error when using new.docusaurus or manual installation
Reproducible demo
No response
Steps to reproduce

Run
npx create-docusaurus@latest docusaurus-test classic
OR
npm init docusaurus@latest docusaurus-test classic

Logs display error

Expected behavior
docusaurus installation success
Actual behavior
Logs display :
npx : 42 installé(s) en 2.756s
file:///Users/lallouegeoffrey/.npm/_npx/34580/lib/node_modules/create-docusaurus/lib/index.js:36
return PackageManagersList.find((packageManager) => process.env.npm_config_user_agent?.startsWith(packageManager));
^
SyntaxError: Unexpected token '.'
at Loader.moduleStrategy (internal/modules/esm/translators.js:140:18)
Your environment
Node : v12.22.10
Npm : 6.14.16
MacOS 12.0.1
Self-service

[ ] I'd be willing to fix this bug myself.

@glalloue you need Node 14 to run Docusaurus
I don't know why but it seems this declaration in create-docusaurus doesn't print you a proper error message 🤷♂
  "engines": {
    "node": ">=14"
  }

Indeed, it seems npx / yarn create and friends don't respect the engines field...


Upper pool size limit?

Is there an upper limit for a single pool size?
if not, is it bound by something?
I just tried to simplify a test by merging a couple pools and increasing the pool size. But one pool only seems to give me 12500.
Might be doing something wrong, of course.
Thanks :)

It is limited by number of processes on Erlang VM, I have started pools with millions of workers, you could change this limit by setting vm_args=+P 1000000, for example. By default, process number is not 12k, it is too low, what error message do you get when you try to start more than 12k?

How does it feel to start millions of workers?!
(I'll know soon :)
OK, have a laugh at me: Don't wait for a signal that never arrives. Problem solved.
Thanks again, gentlemen.


[bug] raise when real dataset has not enough examples in evaluation

This PR fixes a bug in the evaluation. If the real dataset has not enough samples in the test set, an exception is raised.

Thanks a lot.
Can we use tf.errors.OutOfRangeError instead of defining a new exception?

tf.errors.OutOfRangeError needs 'node_def', 'op', and 'message' to be raised, so I decided to raise a custom Exception which was more explicit. But if you prefer, I can raise the original exception.


build failed on Ubuntu

Error message is like following.
c++: error: /FImsvc/alg_vs.h: 没有那个文件或目录
CMakeFiles/astar_demo.dir/build.make:62: recipe for target 'CMakeFiles/astar_demo.dir/src/astar_demo.cpp.o' failed
make[3]: *** [CMakeFiles/astar_demo.dir/src/astar_demo.cpp.o] Error 1
CMakeFiles/Makefile2:992: recipe for target 'CMakeFiles/astar_demo.dir/all' failed
make[2]: *** [CMakeFiles/astar_demo.dir/all] Error 2
CMakeFiles/Makefile2:1004: recipe for target 'CMakeFiles/astar_demo.dir/rule' failed
make[1]: *** [CMakeFiles/astar_demo.dir/rule] Error 2
Makefile:443: recipe for target 'astar_demo' failed
make: *** [astar_demo] Error 2

Having looked through the CmakeList.txt, it seams that the default msvc building option is not open.
So how to build on Ubuntu/Linux?
I am new to Cmake, could you tell me how to fix this?

SET( CMAKE_CXX_FLAGS  "${CMAKE_CXX_FLAGS} /FI"msvc/alg_vs.h"" )
message( ${CMAKE_CXX_FLAGS} )
把CMakeLists.txt 里面的这两句话注释了就可以了, 我也不知道是怎么回事.


AD Authentication and Group Security case sensitivity

Hi, I have been trying to find out what was wrong with the autorizations for more than two days (not easy to find out, as our ldap communications are all encrypted). But, finally, I observed that the groups are not all lowercase on our AD tree.
So I changed, in my ldap/AD configuration the groups definitions to look EXACTLY like what I see in the AD (mix of lowercase and uppercase) and it worked !!!
So I guess that everything received should be lowercased before the comparison to avoid a false positive.

Hello,
Thanks for the feedback. The team should discuss this to decide if it is relevant or not.
Regards


chore(buildpacks): update heroku-buildpack-ruby to v145

See https://github.com/heroku/heroku-buildpack-ruby/compare/v144...v145 and deis/deis#4974.

I tested this with the workflow-beta1 chart and example-ruby-sinatra.


Enable running CodeQL task daily

Enable running CodeQL task daily.
Resolves: #137
Signed-off-by: Alan Jowett<EMAIL_ADDRESS>

Would it be worth adding a Windows run as well?  In the new C++ Windows test framework there are different code paths and uses that would be good to scan over.  (CodeQL only scans code that is compiled).

CI/CD change only.


Can the algorithm be applied on single-cell RNA seq data?

Hello team,
Thanks a lot for developing the great tool and look forward to the paper publication!
For transcriptomic data, may I ask if the algorithm is applicable to single-cell RNA seq data?
Compared to the bulk one, the gene expression matrix of the single-cell one would have a much higher sparsity.
So I wonder if this could affect the accuracy of the algorithm.
Thank you very much!

Hi,
we have not tried to use it for single cell RNASeq data. We did, however, observe that scarce data does usually not yield high prediction accuracy. That said we had good predictions with a proteomics dataset covering 150 enzymes in the network. But I would expect single cell RNASeq data to have even less coverage. At the end it is dependent on whether or not there are strong changes in particular enzymes and whether or not these are captured in the RNASeq.
Best regards,
Ines


arrows: data formatting

Right now, the data format of "arrows.json" expects the following inputs:

x coordinate
y coordinate
time
name

Right now, I need to calculate the x and y coordinates upfront, which can be a little tricky, because it requires some input from the visualization (see below). Thus, I would like to move the calculation of the x and y coordinates to this App.
Basically, I use the following input for calculating of the x and y coordinates:

x and y slope parameters
the standardized length of the arrow

The standardized length of each arrow would be in [0,1], such that the final length of the arrow would be arrow.length * SCALE. This SCALE would typically be equal to the maximum distance any object has from the origin (let's call it max_dist). Therefore, if an arrow has length = 1 it would point across the whole map (up until its borders), whereas a shorter arrow (say, .5) would only reach across "one half" from the origin to the borders of the map. So ultimately, you would need to draw an arrow through (x,y) and rescale it such that it has length arrow.length * max_dist. To do so, it should suffice to divide the vector (x,y) by its norm and multiply the result by (arrow.length * SCALE).
Right now, I make these computations on the input side and then provide final x, y coordinates to you. However, it would be much cleaner for me to only them in the format described above.
Please let me know if that makes sense to you. I can also provide further details or some exemplary files / calculation, if that makes things clearer.

Definitely doable
To clarify something first about arrows requiring "some input from the visualization".
I should have been more explicit in the doc, but the arrow coordinates are currently on the same scale as the circle coordinates. For example, if
{"x":2.6571,"y":-0.0351,"time":"2019-06","name":"google.com","continuous_feature_1":693.1472,"discrete_feature_1":"1","discrete_feature_2":"0","discrete_feature_3":"0","discrete_feature_4":"0","discrete_feature_5":"0","discrete_feature_6":"1","discrete_feature_7":"0","discrete_feature_8":"0"}
and
{
    "name": "FACTOR 1",
    "x":2.6571,
    "y":-0.0351,
    "time":"2019-06"
  },

then the head of the arrow "FACTOR 1" will be right where the "google.com" circle is at time  "2019-06".
So the arrow position is related to the position of the circles as provided in inputs (these positions are on the same scale).
But you don't have to anticipate how these are going to be cast into the D3 SVG generated by the App (which has it's own coordinate system to which transforms are applied when zooming and panning).
To clarify how you'd like this to be implemented
If I understand correctly, an arrow with input length = 1 should appear in the visualization to have a length corresponding to the largest distance from any circle to the (0,0) origin across the entire dataset.
Something like :
$$ viz.length(input.length) = input.length * \left[max \sqrt{x^2 + y^2} \right],$$
where $x$ and $y$ are circle coordinates in inputs.

[ ] One question: Should it be the max over all time periods, or should it be a period-specific max?

That is, should visualisation length be a time dependent function of $input.length$:
$$ viz.length_t(input.length) = input.length * \left[max_{i \in N} \sqrt{x_{i,t}^2 + y_{i,t}^2} \right]$$
where $N$ is the set of circle indices, or a constant function of $input.length$ like
$$ viz.length(input.length) = input.length * \left[max_{i \in N, t \in T} \sqrt{x_{i,t}^2 + y_{i,t}^2} \right],$$
where $T$ is the set of time indices?
Let me know what you'd prefer. Regardless, it will slow down booting time a little but not much, and I am happy to build it in if it makes your life easier.

Hi Martin,
If I understand correctly, an arrow with input length = 1 should appear in the visualization to have a length corresponding to the largest distance from any circle to the (0,0) origin across the entire dataset.
Exactly.
Should it be the max over all time periods, or should it be a period-specific max?
If the arrow length stays = 1, the arrow should not change its size in the visualization. This way, the user can (correctly) infer, that the underlying factor did not lose / gain any explanatory power. So, please use the max across all time periods!

Thanks for the clarification. Will work on it.


Messages should not be added in FormHXRequest post method

In the current implementation of the FormHxRequest the post method adds a success or error message depending on whether the form is valid and then calls form_valid or form_invalid.
https://github.com/yaakovLowenstein/hx-requests/blob/447ea2f3ad769de33991cd714c7d462d45ccc0f2/hx_requests/hx_requests.py#L353
The messages should be moved into their respective form_valid/invalid methods. This would prevent unwanted messages if something goes wrong in the form_valid/invalid methods.

What could go wrong in form_valid? it means that the form is alredy valid?

The form itself may be valid but something else may occur within the form_valid/invalid methods that would prompt us to not save the form and return a message that isn't the success/error message added in the post. With the current implementation I have no control (other than clearing the messages) over what message(s) should be sent to the user. For example, I try to save a field in the form valid but there was an error of some sort and now the user will get both a success message and an error message.

Makes sense that it should be in form_valid.
I think the reason i put it in the post was so that whenever form_valid is overridden which is very common, you wouldnt need to also set the message. But I guess Django anyway does not have default messages for a reason and this can just be a breaking change and you would need to set them yourself if you do override form_valid :thinking:


Issue with select applet command in dart_pcsc_example.dart on windows

Hello,
I am trying to run the example program in dart_pcsc_example.dart with the command [0x00, 0xa4, 0x04, 0x00] which i believe should select the default applet, but the response is always a random 2 bytes:
Readers: [Alcor Micro USB Smart Card Reader 0, Microsoft UICC ISO Reader 4fff2ad5 0]
Waiting for card
Connecting to card in Alcor Micro USB Smart Card Reader 0
Transmitting
Status: 0x8000
PS D:\codelab\dart_pcsc_test> dart run .\main.dart
Readers: [Alcor Micro USB Smart Card Reader 0, Microsoft UICC ISO Reader 4fff2ad5 0]
Waiting for card
Connecting to card in Alcor Micro USB Smart Card Reader 0
Transmitting
Status: 0xd030

I run the program  on 2 machines on  windows 10 and windows 11 with different card readers but still the same random results.
I also tried multiple cards.

Hi @T-eli , thanks for the report! I'm sorry for taking so long to reply. The issue should hopefully be fixed now and the patch is available in the latest release (v2.0.2). If the bug persists, please reopen this issue.

Hello @jjanku , I can confirm the issue is resolved, thank you for the fix.

Chiming in to say that I too confirm that the issue is resolved. A week ago when I tried this library, random results were returned on every run. The present version, on the other hand, returns correct results on every run. Thank you for the fix.

Chiming in to say that I too confirm that the issue is resolved. A week ago when I tried this library, random results were returned on every run. The present version, on the other hand, returns correct results on every run. Thank you for the fix.

Glad to hear that, thanks for the confirmation!


Watchman crawl failed.

I'm running into this error on master with a project created with npm run create-react-app ../test

jest-haste-map: Watchman crawl failed. Retrying once with node crawler.
Error: Watchman error: resolve_projpath: none of the files listed in global config root_files are present in path /Users/vjeux/random/test or any of its parent directories. Make sure watchman is running for this project. See https://facebook.github.io/watchman/docs/troubleshooting.html.

I came up with the "Painless unit testing" sentence. It's now a reality :)

None of these words make any sense to our users: jest-haste-map watchman crawl node crawler resolve_projpath config root_files. They shouldn’t be in our messages.
If we recommend installing Watchman we should explain why and what it gives you (faster watching)? The message should not include any references to configs and jargon.

Also the warning links to https://facebook.github.io/watchman/docs/troubleshooting.html which says nothing about how to install Watchman.
https://facebook.github.io/watchman/docs/install.html suggests to use brew or macports which is not at all beginner-friendly.
I think we need a flag to completely disable this warning/suggestions because it’s optional, and instructions are unfortunately confusing. We can still put instructions on “how to speed up watching” in our Howto.

cc @cpojer

@vjeux Did you previously install RN? @cpojer told me this warning only shows up if:

The folder isn't a Git repo
You already have Watchman

I'm a Facebook engineer so watchman is installed on my machine by default ;)

Ah that explains it. 😄

I confirmed that this error is only thrown when using Facebook equipment so I think this isn't actionable and we can close it. I expect FB engineers to figure out how to use watchman, however we might want to improve the error message in Jest to make sure people understand it is no reason to panic.
how to repro a non fb-environment:
create ~/watchman and put {} into the file then run:
watchman shutdown-server; WATCHMAN_CONFIG_FILE=~/watchman watchman --foreground --logfile=/dev/stdout --no-save-state --statefile=/dev/null
then run jest --no-cache and you shouldn't see the error.

Thanks for checking.

For what it's worth, I got this error as well, so it can happen on non-fb equipment. It was because I'd tried jest in the past and ended up with watchman installed on my machine. At least I think that's how I ended up with it. Uninstalling it made the error go away. It would be nice to not ever show this message though. It was cryptic.

I think we should close this one. I understand the concern but I updated the error message to say to create a .watchmanconfig file in Jest 15. If you have watchman installed manually, I feel like it is ok for Jest to try to use it and tell you how to get rid of the message (git init, hg init or touch .watchmanconfig).

See https://github.com/facebook/jest/blob/master/packages/jest-haste-map/src/index.js#L403

Would something like "Watchman is installed but not running. Using standard watcher instead." be less frightening and filled with words I don't understand? Maybe this is all kind of edge case, but I thought it was worth a mention.

The problem is that we have to surface watchman's error and we can't hide how cryptic it is. Maybe create an issue on watchman's bug tracker about improving their error messages?

I'm a new user to react-native, just got it installed. The documentation seems to be missing the react-native start step and then I noticed the same watchman warning as in this thread (I'm not a facebook engineer :trollface: ) so I think the docs are still lacking a section about how to improve performance of the watchers in general. My 2c.

I came across this issue this morning when running Jest on a different machine. I had no idea I had watchman installed. Looking in my usr/local/bin I found out I installed it back in April 2005 along with React-Native when I was experimenting. I will post an issue with watchman in regards to their error message in hopes they will make it less cryptic.

I found out I installed it back in April 2005 along with React-Native when I was experimenting

Are you a time traveler? 😄

Don't tell anyone! Though 2015 sounds better.

This fixed for me.
npm r -g watchman
brew update && brew upgrade
brew install watchman

Hi @cpojer ,
I tried your instructions:

create ~/watchman and put {} into the file then run:
watchman shutdown-server; WATCHMAN_CONFIG_FILE=~/watchman watchman --foreground --logfile=/dev/stdout --no-save-state --statefile=/dev/null
then run jest --no-cache and you shouldn't see the error

And it does not work, btw, 'WATCHMAN_CONFIG_FILE=~/watchman' does not look like a command.
On the other hand, my situation is:

Windows 7 Pro 64-bit,
I previously created a RN app by following the official instructions, and it works in Genymotion, so I have watchman installed globally already.
I checked out a repo from Github, which is a RN boilerplate, and when running it, I got this error, it is not exactly the same as reported by vjeux, in my case it is:

jest-haste-map: Watchman crawl failed. Retrying once with node crawler.
Usually this happens when watchman isn't running. Create an empty .watchmanco nfig file in your project's root folder or initialize a git or hg repository in
your project.
Error: Watchman was not found in PATH.  See https://facebook.github.io/watchma
n/docs/install.html for installation instructions
Watchman:  Watchman was not found in PATH.  See https://facebook.github.io/watch
man/docs/install.html for installation instructions
ERROR  Watchman was not found in PATH.  See https://facebook.github.io/watchman
/docs/install.html for installation instructions
{"code":"ENOENT","errno":"ENOENT","syscall":"spawn watchman","path":"watchman","
spawnargs":["--no-pretty","get-sockname"]}
Error: Watchman was not found in PATH.  See https://facebook.github.io/watchman/
docs/install.html for installation instructions
at exports._errnoException (util.js:1018:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
at onErrorNT (internal/child_process.js:367:16)
at _combinedTickCallback (internal/process/next_tick.js:80:11)
at process._tickCallback (internal/process/next_tick.js:104:9)

I tried uninstalling watchman and re-installing it, not work.
I tried your approach, not work either

I would really appreciate if you can help me out. Thank you.

My error is slightly different:
jest-haste-map: Watchman crawl failed. Retrying once with node crawler.
Usually this happens when watchman isn't running. Create an empty .watchmanconfig file in your project's root folder or initialize a git or hg repository in your project.
Error: Watchman error: fileData.mtime_ms.toNumber is not a function. Make sure watchman is running for this project. See https://facebook.github.io/watchman/docs/troubleshooting.html.
Loading dependency graph, done.
Any ideas on what this means and/or how I can fix it?

@shireen I am having the exact same error

Any updates on this? I'm coming back to react-native after a year and I'm still stuck at the setup phase. The version 0.38 felt like a breeze compared to the new "optimizations".  Sorry about the rant but setting up is supposed to be the easy part, if its the tool that gets a bad rep.

@shireen @valetarton I'm running into this issue now. Did ya'll figure out a solution or workaround?
My specific error is Error: Watchman error: query failed: synchronization failed: Unknown error.
I'm on Windows 7. Thanks

it fix in my solution:
clear root dir
$ brew uninstall watchman
$ brew install watchman
Warning: watchman 4.9.0 is already installed, it's just not linked.
$ brew unlink watchman
$ brew link --overwrite watchman

re init react-native project，it will work.

After reading all comments and implementing them again getting same error
expo: 27.0.1
react: 16.3.1
react-native: 0.55.2

watchman shutdown-server; worked for me.

i simply copied the code from my project, created a new project and pasted there...
and it works:)

@theapache64 worked for me too.


vCenter-Content Enrichment

Required items, please complete
Change(s):

Workbook, Analytical rules added, Parser -updated

Reason for Change(s):

Content enrichment of vCenter solution based on customer feedback

Version Updated:

Yes

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Hi @rushriva
Can you please resolve the failed checks.
Thanks

@aprakash13 Can you please review and approve analytic rules?
Thanks


Reverse Lookup of D3D11 texture fails

Working on updating the oculus rift code to one of these branches.
I have a little problem in that when I use GFXD3D11TextureTarget with textures generated by the oculus sdk, the texture format is being erroneously reported as GFXFormatDXT3. It seems that the reverse lookup table for textures is not able to resolve the correct format for DXGI_FORMAT_R8G8B8A8_TYPELESS which would be something like GFXFormatR8G8B8A8.
This causes problems since I cant use these textures properly in render targets. I can work around it by explicitly looking for the type but it might be prudent to use a better lookup table.

Yeah just any more needed formats here https://github.com/rextimmy/Torque3D/blob/dx11_deferred/Engine/source/gfx/D3D11/gfxD3D11EnumTranslate.cpp#L43 . Let me know what you add and i'll chuck in

From what I understand though you can have normal variants of that AND *_TYPELESS variants  of essentially the same format (at least in torque-land) which introduces ambiguity. I guess I could add in GFXFormatR8G8B8A8_TYPELESS etc though.
Also while I opted not to use an SRGB format, it's also a valid option with the SDK to use an SRGB variant (DXGI_FORMAT_R8G8B8A8_UNORM_SRGB) - though then again i'm not sure what plans if any are in place to support non-linear rendering in stock T3D (I know doing it in D3D9 is a PITA).

Yeah DX9 is what stopped me implementing sRGB for now in GL/DX11. Yep i agree GFXFormatR8G8B8A8_TYPELESS or something like that is a nice way of doing it. Talking of sRGB this current deferred branch does by default convert to a linear format, it can be disabled though. https://github.com/rextimmy/Torque3D/blob/dx11_deferred/Templates/Full/game/shaders/common/torque.hlsl#L289-L312  just watch out for that, this was all done before dx11 was in place as part of the deferred change over.

I should also point out i will be moving everything in dx11_deferred branch over to dx11_clean and sending a PR to the official repo very soon. Straight after that i will create a dx11_phase2 branch which will be performance fixes and the like, will be adding some new texture formats too, that might be a good time to add in anything you need for your oculus work


bug: Rack configuration options ignored due to ActionPack instrumentation

Summary
When thinning out our instrumentations, solely using Rack, I found that the middleware wasn't being installed.
I then stumbled across this: https://cloud-native.slack.com/archives/C01NWKKMKMY/p1629852592034300?thread_ts=1629850975.033500&cid=C01NWKKMKMY
After adding ActievJob, as follows:
OpenTelemetry::SDK.configure do |config|
  config.service_name = "foo"
  config.use "OpenTelemetry::Instrumentation::ActionPack"
  config.use "OpenTelemetry::Instrumentation::Rack", { untraced_endpoints: ["/health"] }
end

I noticed the following INFO logs:
[09:47] INFO Instrumentation: OpenTelemetry::Instrumentation::ActionPack was successfully installed with the following options {:enable_recognize_route=>false} Context: { schema: rails }
[09:47] INFO Instrumentation: OpenTelemetry::Instrumentation::Rack was successfully installed with the following options {:allowed_request_headers=>[], :allowed_response_headers=>[], :application=>nil, :record_frontend_span=>false, :retain_middleware_names=>false, :untraced_endpoints=>[], :url_quantization=>nil, :untraced_requests=>nil} Context: { schema: rails }

Seeing that untraced_endpoints was an empty Array and not taking in my config (see ☝ logs), I then switch the ordering of ActionPack & Rack and got this (👇):
[09:50] INFO Instrumentation: OpenTelemetry::Instrumentation::Rack was successfully installed with the following options {:allowed_request_headers=>[], :allowed_response_headers=>[], :application=>nil, :record_frontend_span=>false, :retain_middleware_names=>false, :untraced_endpoints=>["/health", "/ping"], :url_quantization=>nil, :untraced_requests=>nil} Context: { schema: rails }
[09:50] INFO Instrumentation: OpenTelemetry::Instrumentation::ActionPack was successfully installed with the following options {:enable_recognize_route=>false} Context: { schema: rails }

Now :untraced_endpoints is correctly populated.
I believe this is happening due to the ActiveJob instrumentation installing Rack with an empty config {}, here: https://github.com/open-telemetry/opentelemetry-ruby-contrib/blob/main/instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/railtie.rb#L13
Share details about your runtime
Operating system details: macOS, Catalina 10.15.7
RUBY_ENGINE: "ruby"
RUBY_VERSION: "2.7.1" & "3.0.3"
RAILS_VERSION: "<IP_ADDRESS>" & "6.1.5"

As suggested by @arielvalentin, here, using ENV variables works as expected as this pulls this is pulled in separately from the user-supplied configuration:
export OTEL_RUBY_INSTRUMENTATION_RACK_CONFIG_OPTS="untraced_endpoints=/health"

https://github.com/open-telemetry/opentelemetry-ruby-contrib/blob/main/instrumentation/base/lib/opentelemetry/instrumentation/base.rb#L272

I believe this is something of a known issue (although I do not recall where we have discussed prior). It’s probably worth figuring out how to make this work correctly regardless of the ordering in the config block. Or, we could at least document this behavior (and the workarounds) somewhere more prominently.
If I remember correctly, we had presumed that folks wouldn’t typically be installing Rack instrumentation in addition to something that builds on it. However, I think that presumption originated in a time when we didn’t have as many interesting Rack options for one to set. :)

I am experiencing this too. Tried the following:
  config.use "OpenTelemetry::Instrumentation::Rack", ...
  config.use_all

But it raises: OpenTelemetry error: unexpected configuration error due to Use either use_alloruse, but not both
Would make sense if we could combine these two.

Yes, they are exclusive (although we may be able to make it more permissive).
When I think about the reasons someone would wish to do this, it sounds like you wish to use all instrumentation but pass specific options to one of the libraries. That is possible with use_all:
OpenTelemetry::SDK.configure do |c|
  c.use_all { 'OpenTelemetry::Instrumentation::Rack' => { untraced_endpoints: ['probes/ready'] } }
end

...however, I don't know that this is documented well enough. I'll make a note on our documentation issue that this is a thing that could be more clear.

Yes, but as noted above, this currently doesn't work b/c ActionPack loads before Rack.

@robertlaurin et.al. I looked into this a bit today and a proper fix for this issue is going to be a bit difficult due to state management of the configuration options and the interactions between the SDK Configurator, Registry, and the Instrumentations.
The instrumentation options are passed as arguments to the registry at configuration time:
https://github.com/open-telemetry/opentelemetry-ruby/blob/03e36cec755df97e08139494b85473a7ad547001/sdk/lib/opentelemetry/sdk/configurator.rb#L161
The registry is unaware of any dependent installation order and installs them in the order they were registered:
https://github.com/open-telemetry/opentelemetry-ruby/blob/03e36cec755df97e08139494b85473a7ad547001/registry/lib/opentelemetry/instrumentation/registry.rb#L78
The config parameter for in install is specific to the instrumentation and does not contain any programmatic options for dependent instrumentations.
As @johnnyshields pointed out the ActionPack Railtie installs the Rack instrumentation without any configuration options, which means the only way to override these options is via environment variables or to manually register the instrumentations knowing the dependent order.
https://github.com/open-telemetry/opentelemetry-ruby-contrib/blob/main/instrumentation/action_pack/lib/opentelemetry/instrumentation/action_pack/railtie.rb#L13
I think we need to review the Configurator's design and consider an approach where either...
... add configuration state to the individual instrumentations during registration time or
... the Registry takes on the responsibility of holding the state of programatic configurations or
... introduce a dependency graph to ensure instrumentations are installed in the appropriate order

introduce a dependency graph to ensure instrumentations are installed in the appropriate order

This is perhaps a bold take, but I suspect this will be useful. I also don't think it would be terribly hard to implement; I think Rails has some examples we could ~steal~borrow. ^1 ^2
Other approaches could work, though - collecting all installation options and deferring the actual installation until the end of the configure block (and merging them somehow) maybe could work, although that idea is not fully formed in my mind.
Since this is only stuff run at SDK initialization, we can probably do something friendly and backwards-compatible with the registry (something something something metaprogramming) to make it a non-breaking enhancement, if we're careful?

@johnnyshields That's true, but if you squint at it that feels (to me) like it's a shortcut to a dependency graph.
Or more specifically, making our users construct the appropriate dependency graph 😆 😆


so what does this do exactly

it just adds like a cut button and mod badges or its incomplete

its code for xgdps libcurl.dll

any way to compile this tho?

any way to compile this tho?

idk

any way to compile this tho?

idk man. maybe visual studio?


add SEAICE_eccen to the documentation

What changes does this PR introduce?
Docs update.
What is the current behaviour?
The runtime parameter SEAICE_eccen is missing from the documentation.
What is the new behaviour
The runtime parameter SEAICE_eccen is described in the documentation.
Does this PR introduce a breaking change?
No.
Other information:
Suggested addition to tag-index
(To avoid unnecessary merge conflicts, please don't update tag-index. One of the admins will do that when merging your pull request.)

LGTM

LGTM


Fix osx arm64 build

The ARM OSX build was enabled by https://github.com/conda-forge/pymcubes-feedstock/pull/4, but it never succeeded in building. This PR aims to fix the build error.

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.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found some lint.
Here's what I've got...
For recipe/meta.yaml:

❌ This recipe is using a compiler, which now requires adding a build dependence on {{ stdlib("c") }} as well. Note that this rule applies to each output of the recipe using a compiler. For further details, please see https://github.com/conda-forge/conda-forge.github.io/issues/2102.

For recipe/meta.yaml:

ℹ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

@conda-forge-admin, please rerender

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/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

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/meta.yaml) and found it was in an excellent condition.

Hi @willGraham01, could you please review this PR when you have a chance? Thanks!

Not quite sure what the changes to the hidden files are doing, but I assume this is the conda-forge admin re-rendering.

Yup, exactly. Thanks for local testing and merging this PR!


html_raw missing meta utf-8

Describe the bug
Postal version 2.1.2 exist meta declaration and not bad őúéáűí characters..
url: /org/xxx/servers/xxx/messages/15/html_raw
v. 2.1.2:

v.2.1.4:

Result after postal upgrade:

temporary solution?

Same issue here. Please make a fix

We need feedback on this issue here!

Same issue after upgrading to 2.1.4 from 2.1.1

Same issue after upgrading to 2.1.4!!!!

Yes, I understand that the encoding display is incorrect. How can I fix this error?

Can't reproduce. Please re-open a new issue with full reproduction steps if this persists.

After the update the bug was fixed . Thanks!


Issue with Directory.rename() and Directory.renameSync()

According to the Directory.rename() and Directory.renameSync() documentation

If newPath identifies an existing directory, that directory is replaced.

An existing directory is replaced only when it's empty. If if is not empty an error occurs. I believe, it's expected, but documentation should be updated to reflect it
Dart VM version: 2.1.0-dev.9.4 (Unknown timestamp) on "linux_x64"

As @sortie hinted, the behavior on Windows is very dangerous:
https://github.com/dart-lang/sdk/issues/47633
https://github.com/dart-lang/sdk/issues/47634
I'd like to fix those two bugs and then bring the documentation in line with the actual semantics.

I fixed the documentation in 0e5f3f49c3917c838b13b79afdaf262d4b1b5d3d


Update fs2-core to 1.0.0-M1

Thanks, the dust around cats-effect/FS2 seems to be settling, so I suspect the latest version will be something that we can start working on.


Farm ID legitimacy check

:warning: Needs extra testing: someone other than me should test this as well.

Issue
In the current implementation, the farm ID textfield does not check the given ID. This makes it easy to enter the wrong one without knowing any better. This is bad practice in general, and can cause cryptic issues down the line with ZOS. I have personally used a wrong farm ID by accident and  thus had issues with my node, and the cause was not instantly clear.
This is an extra step in ensuring no node is created with wacky farm ID's

Fix
To combat this issue, I've implemented a simple fetch from the Threefold GraphQL database to verify a farm exists with the given ID.
As a QOL change I made the right textfield show the name for the farm, as an extra measure to ensure correct ID.
To fix the odd choice of retrieving the ID from that field, I introduced a variable that holds the ID, and the URL gets generated from that.
https://github.com/threefoldtech/0-bootstrap/assets/73200952/3cb91076-6ac9-4365-abc8-104f2a4178f1
(reload page if giving error)
Sanitizing input
issue #4  was not implemented fully, making it possible to still input dots (even though these are removed before use). This is now also taken care of.

fixed minor spelling mistakes

@xmonader

For performance reasons, the declaration of the endpoint and the query can be extracted, so they are not re-initialized for every trigger.

Thanks, I quickly checked your pull request and it seems clean. I'll double read it monday and will probably merge it :)

Tested, overall workflow works as expected.
Some remarks, to be able to run the code, locally I have a missing dependency for Markup (from flask import Markup). After a double check, Markup is not used on the code so the import can be dropped. There are also 1 or 2 extra space on the code but nothing important, I'm gonna fix that later :)

It's in production. Thanks !
I just thinked about something, now you use graphql to validate the farmer but there are different graphql endpoint depending on the network (test and dev have their own graphql and own farm ids). That would be interesting to support theses case :)


Different Shapes for Mouse Utilities

Description of the new feature / enhancement
@jaimecbernardo I was wondering that there might be different ways to add to mouse utilities options as something similar to the rectangle besides the classic circle.
There is something on Android made by BlackBerry to get an idea about my message, the name of the application is Privacy Share.
Scenario when this would be used?
Detailed reading, smart focus and also in audience environments and you don't want others to see the full content consumed by you
Supporting information
I tweeted about this case: https://twitter.com/GuigaG1/status/1455000999362453516?s=20

Looking at that, it feels like a different use case outside find my mouse. To do what they did is surprisingly hard due to all the UX frameworks.
There is a focus my app style ask where we dim out all but the in focus app
or is the actual ask just let me dim all but a sized rectangle for me?


City Council

City Council - Kempsvile added
Mayors Added.

Hi @neortls007idev thanks for the help!
could you follow the pattern of naming the images the last name of the candidate the same as here
https://github.com/Code4HR/okcandidate/tree/develop/assets/img/candidates

OK i'll rename them

Btw any special folder you want it in?

hi @neortls007idev you can drop them in the /assets/img/candidates folder linked here
https://github.com/Code4HR/okcandidate/tree/develop/assets/img/candidates

ok all done with appropriate name and path.


Accept HTTP_PROXY and HTTPS_PROXY on launch

For configure PHP FPM, please create a file bin/install/php-fpm.sh :
#!/bin/bash

echo "Configure PHP FPM..."

if [ $HTTP_PROXY ]; then
    	echo "env[HTTP_PROXY] = $HTTP_PROXY" >> /etc/php5/fpm/pool.d/www.conf
fi

if [ $HTTPS_PROXY ]; then
	echo "env[HTTPS_PROXY] = $HTTPS_PROXY" >> /etc/php5/fpm/pool.d/www.conf
fi

Next load it in bin/launch.sh :
source $BIN_DIRECTORY/install/toran.sh
source $BIN_DIRECTORY/install/nginx.sh
source $BIN_DIRECTORY/install/ssh.sh
source $BIN_DIRECTORY/install/php-fpm.sh

You need help ?


[Issue]PScredential error running Test-ProxyLogon.ps1

Describe the issue
I am running Test_proxyLogon.ps1 against Exchange 2010 and receive the error shown in the script output below. I have tried it for the local Exchange server and against all servers with the same results.
Expected behavior
I am expecting the script to run and provide the output report
Script Output
Unable to find type [pscredential]: make sure that the assembly containing this type is loaded
At E:\Test-ProxyLogon.ps1:84 char:27

        [pscredential] <<<<

CategoryInfo          : InvalidOperation: (pscredential:String) [], RuntimeException
FullyQualifiedErrorId : TypeNotFound

Additional context
It is running under an account with domain admin and organizational management rights for Exchange. The powershell version is version 2.

same error on Exchange 2010, powershell 2.0
Unable to find type [pscredential]: make sure that the assembly containing this type is loaded.
At D:\Downloads\Test-ProxyLogon.ps1:79 char:27

        [pscredential] <<<<

CategoryInfo          : InvalidOperation: (pscredential:String) [], RuntimeException
FullyQualifiedErrorId : TypeNotFound

Same as issue #108

Yeah, duplicate of #108 . We'll continue to use that issue for discussion and any updates around this.


Fill arg and _apply_grid_transform improvements

Few years ago we introduced non-const fill value handling in _apply_grid_transform using mask approach:
https://github.com/pytorch/vision/blob/0d69e35c4e951109dbaa8b42b0a8416d199aee0b/torchvision/transforms/functional_tensor.py#L550-L568
There are few minor problems with this approach:

if we pass fill = [0.0, ], we would expect to have a similar result as fill=None. This is not exactly true for bilinear interpolation mode where we do linear interpolation:
https://github.com/pytorch/vision/blob/0d69e35c4e951109dbaa8b42b0a8416d199aee0b/torchvision/transforms/functional_tensor.py#L567-L568

Most probably, we would like to skip fill_img creation for all fill values that has sum(fill) == 0 as grid_sample pads with zeros.
- if fill is not None:
+ if fill is not None and sum(fill) > 0:

Linear fill_img and img interpolation may be replaced by directly applying a mask:

         mask = mask < 0.9999
         img[mask] = fill_img[mask]

That would match better PIL Image behaviour.
https://github.com/pytorch/vision/blob/0d69e35c4e951109dbaa8b42b0a8416d199aee0b/torchvision/transforms/functional_tensor.py#L567-L568

cc @datumbox

Since we have another report in #8083, do we want to tackle this? IMO, we should just align the two branches
https://github.com/pytorch/vision/blob/f69eee6108cd047ac8b62a2992244e9ab3c105e1/torchvision/transforms/v2/functional/_geometry.py#L588-L594
with something like
bool_mask = mask < 1
float_img[bool_mask] = fill_img.expand_as(float_img)[bool_mask]

This removes the blending and in turn the "shadow" for bilinear interpolation. Plus, this is equivalent for nearest interpolation, since the mask in that case only contains 0.0 and 1.0 entries.

@pmeier the value 0.9999 for mask was sort of on purpose. In the description example affine rotation by 50 degrees with bilinear mode creates a rotated mask with unique values:
tensor([0.00000000, 0.02883029, 0.02883148, 0.10955429, 0.10955477, 0.11125469,
         0.11125565, 0.19197845, 0.19197917, 0.19367909, 0.19367981, 0.27440262,
         0.27440357, 0.35512805, 0.35512924, 0.35682678, 0.35682797, 0.43755341,
         0.43755519, 0.43925095, 0.43925512, 0.51997960, 0.51998138, 0.60240537,
         0.60240555, 0.68312985, 0.68313217, 0.68482971, 0.68482977, 0.76555562,
         0.76555634, 0.76725388, 0.76725554, 0.84798002, 0.84798050, 0.92870331,
         0.92870587, 0.93040466, 0.93040580, 0.99999994, 1.00000000]))

and 0.99999994 can appear inside the mask:
plt.imshow(((mask > 0.999) & (mask < 1.0))[0, 0, ...], interpolation="none")

so, using mask < 1 gives:

this hasn't been fixed


Nonexistent commands throw an error on all running bots

Just kind of annoying for the system admin testing stuff. Meh.

Changed to slash commands, so this is no longer an issue.


appgate-sdp: fix substitution bug

Motivation for this change
Hacky fix for the situation when nix-store hash contains mv in it. @etu
Things done

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

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

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

Result of nixpkgs-review pr 113917 at 8e220056 run on aarch64-linux 1

1 package built:

appgate-sdp

2 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/networking/appgate-sdp/default.nix:124:3:
    |
124 |   installPhase = ''
    |   ^

warning: no-uri-literals
URI literals are deprecated.
Near pkgs/applications/networking/appgate-sdp/default.nix:177:16:
    |
177 |     homepage = https://www.appgate.com/support/software-defined-perimeter-support;
    |                ^

@SuperSandro2000 done. Thanks for the heads up!


Disk Drive Textures Appear Purple

mmm missing texures....
(64k Storage Disk)

Version 1.2.17 of Refined Storage
<IP_ADDRESS>1 of Forge.

weird. any logs about missing textures in the console?

i need to check when i get back home...

https://gist.github.com/Joshwoo70/079713c42322a9b4cadca103d3acc9df/raw/0ccbfadd9e9afa0f79f7373c9faf5752cf1ebacf/fml-client-latest.log

i can see this issue too. seemed to have no functional consequences, though. The drive is working.

i can see this issue too. seemed to have no functional consequences, though. The drive is working.

yeah that's the case it works but not aesthetically pleasing...

Don't see anything about missing textures in the logs.

I can't reproduce this.

I too have this issue. It started when I updated to forge 2221.  The issue was not present with forge version 2185 which I was previously running.

I too have this issue with latest Forge.  Thought it may have been Optifine or FoamFix, but remains the same when removing them.

not sure if it is important. But found this in forge changelog: "Build 1.11.2-<IP_ADDRESS>0:
LexManos:
New client config option to force all chunk rendering off the main
thread. This may help FPS issues, but may also result in world
holes/rendering issues. Also fix Forge configs not saving properly.
". Related? Perhaps different Options lead to not reproducable results

I can confirm this also.
Minecraft 1.11.2
Forge 2223
refinedstorage-1.3.5
http://imgur.com/pdcZyxk

This is due to a change in Forge, I'll be investigating this.

Awesome, thank you very much!


Update query from /derive to the new  /query endpoint

New feature description
Library support for the new /query endpoint instead of continuing to use the /derive endpoint
Checklist

[ ] All acceptance criteria are met.
[ ] Relevant documentation, if any, has been written/updated.
[ ] The changelog has been updated, if applicable.
[ ] New functions/types have been exported in index.ts, if applicable.
[ ] New modules (i.e. new .ts files) are listed in the exports field in package.json, if applicable.
[ ] New modules (i.e. new .ts files) are listed in the typedocOptions.entryPoints field in tsconfig.json, if applicable.
[ ] Commits in this PR are minimal and have descriptive commit messages.

Superseded by https://github.com/inrupt/solid-client-access-grants-js/pull/1195


Navigate to failed step after clicking resolve in workflow view

The browser will jump to specific step in workflow view.


Omis notification quote cancelled

This sends a notification to the customer and all the advisers on the order when this is cancelled.

Codecov Report

Merging #659 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #659      +/-   ##
===========================================
+ Coverage    96.48%   96.49%   +<.01%
===========================================
  Files          129      129
  Lines         3930     3939       +9
  Branches       337      338       +1
===========================================
+ Hits          3792     3801       +9
  Misses          97       97
  Partials        41       41

Impacted Files
Coverage Δ

datahub/omis/notification/client.py
100% <100%> (ø)
:arrow_up:

datahub/omis/order/signals.py
100% <100%> (ø)
:arrow_up:

datahub/omis/notification/constants.py
100% <100%> (ø)
:arrow_up:

datahub/omis/notification/signal_receivers.py
86.66% <100%> (+0.95%)
:arrow_up:

Continue to review full report at Codecov.

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


SpotifyLocalAPIClass times out when running local example project

DownloadString during SpotifyLocalAPIClass.Connect passing "http://<IP_ADDRESS>:4380/simplecsrf/token.json?&ref=&cors=&_=1423252676" fails with a standard timeout exception
System.Net.WebException occurred
HResult=-2146233079
Message=The operation has timed out
Source=System
StackTrace:
at System.Net.WebClient.DownloadDataInternal(Uri address, WebRequest& request)
at System.Net.WebClient.DownloadString(Uri address)
at System.Net.WebClient.DownloadString(String address)
at SpotifyAPI.SpotifyLocalAPI.RemoteHandler.query(String request, Boolean oauth, Boolean cfid, Int32 wait) in c:\Users\joshe\Dropbox\dev\SpotifyUtility\SpotifyAPI-NET\SpotifyAPI\SpoitfyLocalAPI\RemoteHandler.cs:line 143
InnerException:
Netstat says that port is open, and navigating to that url in my browser just gives me "not found"

Spotify web helper is def running

Hey,
Actually, I can't test it myself at the moment, since the SpotifyWebHelper is crashing everytime I want to make calls to it. (IDK why it happens yet)
The "Not found" message is expected, it pretty much says the WebHelper is running (else Chrome would display an error)

@JohnnyCrazy, based on your response I get the feeling you're seeing what I'm running into as well. I have a simple app using your library that primarily just waits for events whenever a track changes. However, it quickly starts to no longer get events as a result of SpotifyWebHelper no longer running (possibly crashing). I'm assuming this isn't something you've seen happen much previously?

@gusper
I noticed that too, but I'm unable to find any reason why it would crash yet.
@whereismyspaceship
Could you try these Steps:

Close spotify and the API-Testproject
Comment out the RunSpotifyWebhelper method in the API-Testproject
Start spotify normally (SpotifyWebHelper should be running after this)
Start the API-Testproject

This worked for me, maybe you are not allowed to start the WebHelper manually

Did some searching and mostly came up empty. However, I did find that there's a beta desktop client available now and it uses a SpotifyBetaWebHelper.exe instead. If I get some time, I might test adding support for it as well to see if the issue around SpotifyWebHelper.exe disappearing has been fixed.

Just sent you a PR with support for it. Will now test whether or not the beta version of spotifybetawebhelper.exe is more stable.


test_rendering error with firefox

With firefox 51.0.1 on ubuntu, the ol.rendering.layer.VectorTile / vector tile layer / renders correctly with the canvas renderer  rendering test fails with:
Error: Error: expected '11.64' to be below 11.6 (http://localhost:3000/node_modules/expect.js/index.js:96)

same with make test-rendering

We should change all our rendering tests to use larger reference images, and maybe a different image diff tool. For now, many rendering tests only pass on Firefox/Linux.

And only on some Firefox versions there it seems.

didn't we get pointed to https://github.com/mapbox/pixelmatch a while ago?

Yes, I think we should use that, and bigger reference images. See #4288.


SPARK-5492. Thread statistics can break with older Hadoop versions

Test build #26495 has started for   PR 4305 at commit b7d4497.

This patch merges cleanly.

LGTM pending tests. Thanks, Sandy

Test build #26495 has finished for   PR 4305 at commit b7d4497.

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

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


Allow suffix for update scripts

To work around all the merging conflicts.
E.g. 34-foo.php and 34-bar.php could both be checked in.
There would be no guarantee for their execution order.
Suffix could be issue number.
Should the framework remember which files were executed or which version (like now)?

Yii approach;

Interesting! How do they store the last executed migration script resp. the list of all migration scripts that have already been executed?

Not sure..
Here's docu: https://github.com/yiisoft/yii2/blob/master/docs/guide/db-migrations.md

They're tracking a list of executed scripts:

After applying a migration, the migration tool will keep a record in a database table named migration. This allows the tool to identify which migrations have been applied and which have not. If the migration table does not exist, the tool will automatically create it in the database specified by the db application component.

And they allow you to modify the migration history by hand in dev:

Sometimes, we may want to modify the migration history to a specific migration version without actually applying or reverting the relevant migrations. This often happens when developing a new migration. We can use the following command to achieve this goal.

yii migrate/mark 101129_185401

This command is very similar to yii migrate/to command, except that it only modifies the migration history table to the specified version without applying or reverting the migrations.

the migration tool will keep a record in a database table named migration.

We should get this for free once we use "Setup Scripts" for migrations.
See https://github.com/cargomedia/CM/issues/1652

Laravel approach:


Make GUI rows and columns start from 1

It seems more sensible that they would start from 1 because there is no "row 0" of a GUI really, I understand you did this because MC starts at slot 0, but you could just change the math slightly. You could also make this an option in GUI creation if you don't want to force it, though I feel it will increase readibility

This will absolutely be a breaking change however, I think it's a pretty solid suggestion.
Starting at (1, 1) will be much more intuitive and much less effort to count GUI Slots.


Concurrency bug in put-file / finish-commit

This needs to be triaged and reproduced, but based on discussion we think is a valid bug in theory
a) Initiate a long running PutFile() request that is streaming data to an open commit
b) call FinishCommit() while the PutFile() request is still streaming
The commit will be finished, but the diffs that are associated w the commit are in an undefined state. If the PutFile() completes, its probably as the user intended. If the PutFile() fails, the commit is finished, but there is data 'missing'.

Another problem is that FinishCommit computes the size of the commit.  So if there's an ongoing PutFile while FinishCommit runs, the commit can end up with an incorrect size.

PPS only finishes a commit after all pods have finished, which means they have also finished any put-files, so the only way to trigger this is as a user doing 'manual' put-files.
So we think this is an edge case, and we can de-prioritize.

Is this actually still an issue? There have been some big model changes and a lot of refactoring since this was opened

I'm pretty sure this is not still an issue. It was always theoretical but we've added a lot more safe guards to put-file since then. Feel free to reopen if you find a way to reproduce this.


Create Group_Anagrams.cpp

C++ code for the issue #223
Please accept it.
Thank you @avinash201199

They are not considering dsa,cp,algorithm repository :(
all the pr will not be considered in hacktoberfest but your contribution will be for lifetime, this will help many students to practice dsa .
Thank you for your contribution :)
you can contribute to other repo


Updating free videos link

Earlier the videos link redirected to https://home.robusta.dev/learning which gave a 404 updated it to https://home.robusta.dev/videos

Thank You for accepting my PR :)
will look out further for good first issues.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: arikalon1:x: sassukeuchihaYou have signed the CLA already but the status is still pending? Let us recheck it.

Can you please sign the CLA @sassukeuchiha so we can merge it  ?

Done 👍


Fix #97 and #106

PR which allows to fix/close the following issues: #97 and #106
First of all, it should be noted that explainations below assume that the #92 was never merged. That fix was a mistake and it is reverted by this new PR. See #106 for more details about #92 mistakes.
Expected behavior

If a resource is private, client must be authenticated to access it. If it is not, it must be challenged with a 401 response
If a resource is public, client must be allowed to access it, even if not authenticated (case of a guest identity)
If client is authenticated but not authorized to access a resource (case of an authenticated identity with extended permissions system), a 403 response must be returned.

Current behavior
Authentication is needed for public and private resources, even if the developer makes them public. This is due to the fact that at the authentication level (authenticationPost), we are returning the response, leading to short-circuit of the route event on which authorization listeners are also attached (they have a lower priority than authentication listeners).
Thus, returning a response from the authentication layer, in case of an authentication failure, means that authorization rules for a guest identity are never evaluated and therefore, public resources are not because authentication is always required.
New behavior
Authentication layer

In case of an authentication failure, the response is no longer returned. We ensure that the response is set on the MvcEvent instead. This allows to not short-circuit the route event, and thus, evaluate authorization rules in the context of a guest identity prior returning the response.

Authorization layer

If the client is authorized to access the resource, the MVC response which can have been modified by authentication layer is reseted. This avoid challenging the client in case of a guest identity that is allowed to access the resource after all (public resources).
The response, if already an ApiResponseProblem instance, is not overriden by the ZF\Apigility\MvcAuth\UnauthorizedListener listener. This allows sending correct response (401) in case of an authentication failure. (A separated PR for the zf-apigility module will be made according this change).

About 401 status code vs 403 status code
Generally speaking, the 403 status code involves an authenticated user. User is authenticated but is not authorized to access the requested resource. This is generally the case when an authenticated user try to access resource owned by another user.
Currently, Apigility provides a very basic permissions system, based on HTTP methods and static identities. Access to a resource requires either an authenticated identity or a guest identity. The extended permissions system implementation, if it is required, is left to the developer. Of course, a 403 response is still send by the authorization layer in case of an unauthorized client, but only if no authentication is required (case of an already authenticated user). This will be the case if the developer implements extended permissions system (e.g, conditional ACL with assertions).
Regarding the above, it should be noted that even if authentication and authorization layers have differents concerns, this doesn't prevent one layer (e.g. the authorization layer) to be aware of the other. Furthermore, it shouldn't be forgotten that these layers are intimately linked. In other words, we can see the zf-mvc-auth module as a component that provides sub-components (authentication, authorization) which are coworkers. As such, they can share information.
Final note: Sorry for the verbosity here. I wanted to be as clear as possible.

Note: This PR involves merging of another PR related to the zf-apigility module. I'll reference it here soon.

@weierophinney
Works done as promissed on IRC. Please review and see also related PR made on zf-apigility which is available at https://github.com/zfcampus/zf-apigility/pull/146
Thanks you for your time.

Thanks for the work on this! I look forward to testing on my end. Very much appreciated!

@jackdpeterson
Thanks. ;)
I'll now check behavior of Oauth authentication adapter. For me, current behavior is wrong (403 instead of 401) when authentication is expected... Behavior should be consistent over all adpaters.

After more thinking, I think that the solution provided here is bad because this involve too much kaking in existents listener (eg, resetting response and so on). Right now, close this PR. I'll provide a new one soon.


Falco Driver Loader missing from docs

What to document
The falco driver loader is not really mentioned in the falco docs, but enough of an example to set it up from the helm chart is available on that repo's README.md file to make a working install.
It would be nice to see that documented more properly on falco.org.

@tspearconquest: There is not a label identifying the kind of this issue.
Please specify it either using /kind <group> or manually from the side menu.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/kind documentation

Following on from https://github.com/falcosecurity/charts/issues/150#issuecomment-865049495:

The falco.org mostly includes documentation regarding artifacts that are considered official (as per these definitions. Unfortunately, our helm chart is formally still "incubating". For this reason, there's no such documentation regarding the chart on the website.

Hello, I am so sorry I was not more clear here. Please allow me to clarify.
My original goal from the thread above was to highlight the fact that the documentation currently provides an example for using falco-no-driver and falco-driver-loader in least privileged mode, but does not provide the same for fully privileged mode. However, the split image setup is desirable even for fully privileged mode when working in environments where storage or resource constraints are carefully managed and smaller images are preferred.
My use case is that we prefer 2 or 3 smaller images over 1 large image, and we also prefer eBPF probes over kernel modules, even though we do not run Kernel 5.8 yet. Therefore, we want to use the eBPF probe in Fully Privileged mode, but also use the falco-driver-loader and falco-no-driver images together.
So, as a short synopsis...

The eBPF probe must be run in fully privileged mode unless you have Kernel 5.8 or higher. Conversely, the documentation only provides an example for how to load the eBPF probe with falco in least privileged mode, but no examples are provided for doing the same with falco-driver-loader and falco-no-driver in fully privileged mode, and that is a configuration I think could prove to be popular if provided as an example for users to use.


OnMount function is being called before elements are mounted

Describe the bug
Hi Everyone
I am struggling with an issue that need your help to solve it:
I see that OnMount function is being called while the ref element is still not initialized, this happened when I run the solid app using double click on index.html generated by "npm run build" using vite
The issue isn't reproduced if I run npm start
Here is a snippet code showing the issue:
const Slider: Component  = (sliderProps) => {
let sliderContainerRef: HTMLDivElement | any;
onMount(() => {
console.log("onmount is called, sliderContainerRef:" + sliderContainerRef);
}
return (<div class={"slider-line"} ref={sliderContainerRef}>);
Results in console when running npm start:
[vite] connecting...
client.ts:53 [vite] connected.
slider.tsx:153 onmount is called, sliderContainerRef:[object HTMLDivElement]
Results in console when double clicking on index.html:
onmount is called, sliderContainerRef:undefined
Sleeping for a while solved the issue:
onMount(async () => {
while (sliderContainerRef == undefined)
{
await sleep(50);
}
console.log("onmount is called, sliderContainerRef:" + sliderContainerRef);
});
Results in console when double clicking on index.html with sleep:
onmount is called, sliderContainerRef:[object HTMLDivElement]
Your Example Website or App
Dont have
Steps to Reproduce the Bug or Issue
Please see description.
Expected behavior
I am expecting that the ref element should be initialized before OnMount is being called since OnMount: "runs after initial render and elements have been mounted. Ideal for using refs and managing other one time side effects."
Screenshots or Videos
No response
Platform
Windows 10
Chrome
Additional context
No response

Have a see at #1199. Solid 1.5 change the effect execute order.

I want to clarify #1199 is an initial misunderstanding. Effects execution order did not change in 1.5.

@ryansolid Thanks for your reply.

Is it guaranteed that onMount is called only after all html elements including ref elements is mounted/rendered?
Can you tell me why running index.html by double clicking and running "npm start" would give different results in solid-js as explained in the description?

Not if there are things that are conditional rendered. But in general onMount/createEffect run after all internal effects run which means both all initial DOM elements are created and attached.
No idea, this makes no sense. I guess let me see you are using a vite template I gather? I wonder if something is weird at the mount point. npm start in the template looks like it aliases dev. Try running npm run serve. After build that might help us narrow it down to a dev vs non-dev thing.

@ryansolid
Issue is reproduced if running: "npm run serve" after "npm run build" while it isn't reproduced when running "npm start" meaning it is dev vs production thing, any idea what makes this different behavior?
Here is my package.json:
"scripts": {
"start": "vite --open",
"dev": "vite --open",
"build": "vite build",
"serve": "vite preview"
},

Offhand no. I think I need a reproduction. I've never heard of this being reported before.

Can you tell me why running index.html by double clicking and running "npm start" would give different results as explained in the description?

Dumb question from me sorry.  Won't double-clicking on the html file just open it as a file from your file system as opposed to serving it through a url such as http://localhost:3000?
I'd expect you need to start the server and browse to the relevant port instead of double-clicking the file.  Or perhaps there is some other magic in play here?

Double-clicking the html file wouldn't work.

I mean it sounds like this issue is independent of that. Vite creates a dist folder. If you open that locally in the filesystem it will work without making a server. Chrome will open it up locally and open relative resources.
But it's bedside the point. The issue here is with the prod build. But need a reproduction.

Maybe it works on Mac but on Windows I get network tab errors about JS files being blocked.  It may also depend on whether the vite config has some other "/" based public folder configured - meaning all the resource links are not relative to the html file.

Hi
Thank you all for your response.
@ryansolid :
I am working to reproduce the issue with minimal code as possible, should I attach the source code here via github?
@Brendan-csel
It is reproduced on windows 10, I will need to check it on mac to see if it is mac vs windows thing,
@lxsmnsyc
double clicking the html file would give issue with CORS (JS files being blocked) since you need to start a server but this can be solved by three ways:

You can make inline html.index by vite-plugin-singlefile package: https://www.npmjs.com/package/vite-plugin-singlefile
You can run "npm run serve" which will start a server and run the html.index
You can fetch a public folder via express server by: app.use(express.static('public')) where public is the folder name located inside your express project.

Please note I was able to reproduce the issue with the three methods above.
Thanks

Stackblitz might be the easiest. But I can work from GitHub repo.

Hi @ryansolid
Here is the github repo with main branch which reproduce the issue by the following steps:

git clone to main branch https://github.com/Nizarazo/solid-js-issue
open vs code and then open terminal
lerna bootstrap
cd solid-js-issue\packages\blender\client
npm run build
npm run serve
Type the link http://<IP_ADDRESS>:4173/ in browser
inspect element you see error due reference element still not being initialized:

ObjectsliderContainerRef: undefined[[Prototype]]: Object
index.4f5de954.js:5 Uncaught TypeError: Cannot read properties of undefined (reading 'getBoundingClientRect')
at E (index.4f5de954.js:5:1045)
at index.4f5de954.js:5:1309
at de (index.4f5de954.js:1:10546)
at Object.fn (index.4f5de954.js:1:10578)
at Yt (index.4f5de954.js:1:11509)
at ge (index.4f5de954.js:1:11452)
at rt (index.4f5de954.js:1:10499)
at Xt (index.4f5de954.js:1:10571)
at xn (index.4f5de954.js:5:1263)
at index.4f5de954.js:1:5491
With branch solid-js-fix there is a fix which is moving the folder solidjs-infra from root to blender/client/src with the fix you should see sliders.
Don't know why moving the folder to the src directory solves the issue, but I need the folder in root folder in order to have a reusable code.
Please let me know if you need any more info.
Thanks

Hmm.. moving it making a difference suggests an interesting problem. Maybe we have 2 versions of Solid. The one for user effects is different than what is being imported by the JSX transform. Let me see if I can confirm that.

@ryansolid any update about the fix?

There are definitely 2 versions of Solid in your project and that is causing the problem. Just haven't figured out where the 2nd copy is coming from.

@ryansolid To check the version of solid-js I run the command npm list it looks same version 1.5.6 on both location:

blender/client
solidjs-infra

Please see the results below of npm list:
PS C:\p4client\ProAudio\dev_main\ProAudio\XPlatform\Common_Sources\HTML_Packages\packages\blender\client> npm list
<EMAIL_ADDRESS>C:\p4client\ProAudio\dev_main\ProAudio\XPlatform\Common_Sources\HTML_Packages\packages\blender\client
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>-> .....\infra
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>-> .....\solidjs-infra
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>PS C:\p4client\ProAudio\dev_main\ProAudio\XPlatform\Common_Sources\HTML_Packages\packages\blender\client> cd ..
PS C:\p4client\ProAudio\dev_main\ProAudio\XPlatform\Common_Sources\HTML_Packages\packages\blender> cd ..
PS C:\p4client\ProAudio\dev_main\ProAudio\XPlatform\Common_Sources\HTML_Packages\packages> cd .\solidjs-infra
PS C:\p4client\ProAudio\dev_main\ProAudio\XPlatform\Common_Sources\HTML_Packages\packages\solidjs-infra> npm list
<EMAIL_ADDRESS>C:\p4client\ProAudio\dev_main\ProAudio\XPlatform\Common_Sources\HTML_Packages\packages\solidjs-infra
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>-> ...\infra
└──<EMAIL_ADDRESS>So it is the same version used can you advice?
Thanks

I'm more interested in this:

<EMAIL_ADDRESS>

Yeah i tried matching versions and it didn't help either. But I see 2 copies of every function in the build output (the second set has a $1 attached to the names). This is a mono-repo/bundling issue, but I am not certain what a good solution looks like.

export default defineConfig({
  plugins: [solidPlugin(), solidSvg()],
  build: {
    target: 'esnext',
    polyfillDynamicImport: false,
  },
  resolve: {
    dedupe: ['solid-js'], <----------------------
  },
});

https://stackblitz.com/edit/github-scrvmy?file=packages%2Fblender%2Fclient%2Fvite.config.ts

I added the code change in vite.config.ts given by @LiQuidProQuo and it fixed the issue.
@ryansolid The question if a good solution should be in vite.config.ts or in solid-js framework?

Its definitely outside scope of the framework. Outside of the warning we give in dev there is nothing we can do. We didn't see warning in dev I think? Since Vite is bundling stuff differently. Lerna has the ability to hoist dev deps which is how I usually solve this. Have them install at the root. Otherwise Vite config seems fine I guess. Although I wonder if dedupe works well with different versions. Which is probably the main reason we wouldn't just put this config in tte plugin.

If going with vite config solution is ok then this issue can be closed.
Thanks


Repo sync for protected CLA branch

The pull request is created from master637362397176015815 to master to fix git push error for protected CLA branch

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


IDA* unable to find path when using Chebyshev or Euclidean distance

Need to inspect IterativeDeepeningAStar<TResult, TStep> class.

Issue is resolved. Further test cases are to be added.


Fix failing AppVeyor CI

Check CONTRIBUTING guideline first and here is the list to help us investigate the problem.
Is your feature request related to a problem? Please describe.
Currently, AppVeyor CI is always failing.
We should pay attention to AppVeyor CI.
Describe the solution you'd like
Fix AppVeyor CI with several PRs.
Describe alternatives you've considered
We should mark omittable testcases for hard to fix.
Additional context
This makes obscure whether Fluentd on Windows works well or not.

https://ci.appveyor.com/project/fluent/fluentd/builds/31359133/job/9j76r04jtjfhclj5#L152 @i.retry is sometimes failing.
We should investigate it.

It sometimes happens in not windows env https://travis-ci.org/fluent/fluentd/jobs/655185903

Hmm..., @i.retry assertion seems to be fragile....

Fragile testcases should cause failure in other platforms.
So, we can close this issue ticket due to fixing Windows related failures.


Add duration and a icon for videos in the bottom left corner

I use librephotos mainly for videos and the previews are way too resource intensive to the point it's almost impossible to watch a video on firefox. Videos kinda play on chrome but the experience still isn't great with the computer fan going crazy.
I wish there was a setting to disable the previews or only play them on hover.
Not related to this issue but I would also like if the duration of the videos would be presented on the thumbnails.
Overall this is a nice alternative to google photos, just needs some polishing.
Keep up the good work!

It happens with me as well.
This new update make it almost impossible to use

Play video previews only on hove is now implemented. I will rename the issue to better reflect the next step.

Beautiful! Massive performance increase for firefox users, the page is much more fluent now and videos play fine. Good job! 😉

Video duration is now added together with an video icon


docs: Step installation method by Git

CodeIgniter4 can be installed in a number of different ways: manually, using Composer, or using Git.

Those are the opening lines of the CodeIgniter installation method.
But on the documentation does not found how to install using GIT.
https://codeigniter4.github.io/CodeIgniter4/installation/index.html

Hmm interesting. I guess this would be a method that used a locally cloned repo for the System path so we t would be easy to keep updated. A nice middle ground between download and Composer, I do think we should include instructions.

I would like to remove the Git installation from the user guide as no one has addressed this issue.

I mean, it's just a git clone so would be an easy writeup, but honestly, I'd prefer to encourage Composer installation so I'm not concerned with removing the git reference, personally.

Another way using GitHub directly just by activate repo as template .
Then can run git clone.
ref: https://docs.github.com/en/repositories/creating-and-managing-repositories/creating-a-repository-from-a-template

Which repository do you install with git clone?
Does anyone actually use a git install?

Another way using GitHub directly just by activate repo as template .

If there would be a template repo, that would be the appstarter. Not this development repo.

Does anyone actually use a git install?

We have a user in the forums who is openly using git clone.

You would need to do one of two things:

Clone the framework repo and then modify files in-place (basically never pull again)
Clone the AppStarter then run composer install

But 1) is strongly discouraged (framework repo should be for development only) and 2) still require Composer. So I would be in favor of dropping this from the UG. There is no harm in people doing it but I don't think it adds anything useful.

I think if a user do git clone appstarter, they can update project files with git merge.
But the same thing can be done with https://github.com/tattersoftware/codeigniter4-patches

I have no problem with someone using git clone appstarter but I don't think it deserves a dedicated "installation method" section in the docs because it still requires Composer and it introduces a level of complexity that I wouldn't want to support for developers who are unfamiliar with resolving merge conflicts. It also makes the project mostly unusable as a separate Git repo, which seems like a bad practice to encourage in 2022.


Azk init doesn't recognise the correct rails version

I created a new rails app as shown below. Note the rails version is 4.2.
$ azk shell --image rails --shell /bin/bash
root@c111ccd0c87e:/azk/azk-labs# rails -v
Rails 4.2.0
root@c111ccd0c87e:/azk/azk-labs# rails new rails42
      create
      create  README.rdoc
      create  Rakefile
      create  config.ru
...
$ exit

Now I used the azk init command to generate the Azkfile.js.
$ cd rails42
$ azk init

The problem is that Azkfile.js was generated using rails 4.1.
systems({
  rails42: {
    // Dependent systems
    depends: [],
    // More images:  http://images.azk.io
    image: "rails:4.1",
...

So, when I tried to run azk start, it started to download the rails:4.1 image...
Anyway, the azk init needs to be improved to recognise the correct rails version.

My reccomendation:
$ azk shell --image azukiapp/ruby --shell /bin/bash
# gem install rails -v 4.2.0
# rails -v
Rails 4.2.0
# rails new rails42
# exit
$ cd rails42

I do not particularly like the concept of using an image with rails already installed. In rails community, update framework versions during the project development is something trivial and common.
The rails is just another dependence and not a runtime. When provisioning runs, the rails are installed as defined in Gemfile.
If used an image with the rails already installed, the image will have two versions of rails which may end up providing some problems. Mainly around the deployment, it will be only installed the version that really matters.
Another motivation for this approach is that the user can choose the version that will use to project generation, because each version has its peculiarities.

I agree with you, but even doing your suggested steps, once the rails app was created, executing azk init it will put the wrong image in the Azkfile.js. The app is rails 4.2 and azk recognise it as a 4.1 app.
1. we can fix that; or
2. we can change azk init behaviour to suggest a ruby image instead a rails image like you suggested.
Anyway, in the second approach, it will need use some evidences to suggest the correct ruby version. If the ruby version is locked in Gemfile its easy, but if not, one possibility is to look at the rails version and suggest a ruby version that is compatible with that rails version.
We need to be careful here because the azk init job will become a little more complex.

I am inclined the second solution for the same reason noted above.
Create a image rails and suggestion for each version would be very complex and something unnecessary since the rails is installed during the 'bundle install`, whether or not a version of rails already installed.


100% test coverage

Codecov Report

Merging #5 (f99b64d) into master (ac31b06) will increase coverage by 19.29%.
The diff coverage is 99.41%.

@@             Coverage Diff             @@
##           master       #5       +/-   ##
===========================================
+ Coverage   79.77%   99.07%   +19.29%
===========================================
  Files           9        9
  Lines         801      973      +172
===========================================
+ Hits          639      964      +325
+ Misses        162        9      -153

Impacted Files
Coverage Δ

pons/_abi_types.py
98.61% <98.61%> (ø)

pons/__init__.py
100.00% <100.00%> (ø)

pons/_client.py
100.00% <100.00%> (+11.30%)
:arrow_up:

pons/_contract.py
100.00% <100.00%> (+1.29%)
:arrow_up:

pons/_contract_abi.py
99.16% <100.00%> (+8.66%)
:arrow_up:

pons/_entities.py
100.00% <100.00%> (+23.72%)
:arrow_up:

pons/_provider.py
97.61% <100.00%> (+46.19%)
:arrow_up:

pons/_signer.py
90.00% <100.00%> (+0.52%)
:arrow_up:

pons/abi.py
100.00% <100.00%> (+16.66%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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


Schedule Reset

Version of the custom_component

Latest
Configuration

Add your logs here.

Describe the bug
A clear and concise description of what the bug is.
I have a schedule to run charging at the cheapest times. 00:30 - 06:30. I noticed it I toggle the charging allowed switch to ON the schedule is disabled in the pod point app. When the switch is OFF the schedule is enabled however my custom times are also reset to 00:00 which means charging doesn't happen at all.
Is there a way to restore my settings and just toggle the schedule on/off?
Debug log

Add your logs here.

This is how the custom component works.
Charging enabled: turns off any scheduling on the Podpoint which means charge always
Charging turned off: adds a schedule for each day between 00:00:00 and 00:00:01 which means basically it's off all the time.
You then don't use the schedules in the Podpoint app as everything is done with the custom component instead. The behaviour you are seeing is exactly as expected.

Ah makes sense. Thank for clarifying.


babel 7 support

I'm currently modernising an angular 1.5 app
I would like to use babel 7 but this plugin has not been updated, are there any plans for this update?

@p3x-robot This repo is maintained. Please feel free to open a PR if you'd like to fix this.

I use this plugin together with babel 7 without any problems.

I created a PR https://github.com/schmod/babel-plugin-angularjs-annotate/pull/43. I am not very sure of my changes since I don't follow closely the evolution of Babel but tests are passing 🙏

Thanks for arousing my interest @StabbarN, I've since discovered this indeed works with Babel 7 - you just need to require it rather than using the string literal reference as instructed in the README.
module.exports = {
  presets: [
    '@babel/preset-env'
  ],
  plugins: [
    require('babel-plugin-angularjs-annotate'),
    'proposal-optional-chaining'
  ]

Though now that I think about it, this should also work just fine:
module.exports = {
  presets: [
    '@babel/preset-env'
  ],
  plugins: [
    'angularjs-annotate',
    'proposal-optional-chaining'
  ]

I think all the package naming changes just threw me off.


add usable filter for network

这个 PR 实现什么功能/修复什么问题:
network添加 usbale filter 参数
是否需要 backport 到之前的 release 分支:

release/2.11.0
release/2.10.0
release/2.9.0
release/2.8.0

/area region
/cc @swordqiu @yousong

Codecov Report

Merging #1573 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1573      +/-   ##
=========================================
- Coverage    7.69%   7.68%   -0.01%
=========================================
  Files         411     411
  Lines       62934   62949      +15
=========================================
- Hits         4840    4839       -1
- Misses      57685   57701      +16
  Partials      409     409

Flag
Coverage Δ

#aFlag
7.68% <0%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/compute/models/networks.go
0.93% <0%> (-0.02%)
:arrow_down:

pkg/util/rbacutils/rbac.go
73.83% <0%> (-0.94%)
:arrow_down:

pkg/appsrv/response.go
69.23% <0%> (+3.07%)
:arrow_up:

Continue to review full report at Codecov.

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

/lgtm
/approve