useDialogs Make Payload a Node Rather than a String

Summary
The dialog payload specifies that payload is expected to be of type string, which feels restrictive. For example, my use case involves passing a complex form as the payload. Expanding support to allow any type of payload would be highly beneficial.
Examples
No response
Motivation
Extending the usefulness of useDialogs.
Search keywords: payload useDialogs

You should be able to pass anything as a payload. e.g. you can pass an object as we do internally for the system dialogs. The docs just use a string as an example. Perhaps we need to clarify this in the documentation?

I can confirm that it's possible to pass any payload, but the PropType in the example was somewhat unclear. It might be beneficial to include a note in the documentation to clearly state that any payload type is acceptable for the component, and to update the PropType from string to any.


Feature request: Multiple JMX ports

Is it possible to specify the JMX ports for every broker and add multiple ones?
We have multiple active JMX ports on our broker, but kafka-manager only shows the highest portnumber (9999 and ignores 9996, 9997, 9998)
And we have other broker with completely different ports...
Would be awesome.
BTW: really nice tool ;)

This feature was supported, you can close this issue.

How can i configure it?
Is there a documentation about it?

@MaxPower87

Start kafka 0.8 server with your JMX port (e.g. 9996)
nohup env JMX_PORT=9996 bin/kafka-server-start.sh config/server.properties >>/dev/null 2>&1 &

Download the newest kafka-manager, start it, add your cluster.

Hey this is not what i want..maybe i expressed my self wrong ;)
we have one server with multiple kafka processes and each process has its own JMX port.
kafka manager only sees one of these processes.
is it possible to see all the process or one special process (special JMX port) in kafka manager?

Of course, check this
10101 and 10103 is on the same machine.

Did you use this command to start kafka server rather than export JMX_PORT=${JMX_PORT:-9999}
nohup env JMX_PORT=9996 bin/kafka-server-start.sh config/server.properties >>/dev/null 2>&1 &

@zheolong Why does your broker with Id 10100 displays -1 in the JMX Port column.
I'm having this issue where even after setting the JMX_PORT and KAFKA_JMX_OPTS and then restarting the kafka-server, I see -1 against that broker in the kafka-manager.


Treemacs open error

Observed behavior
When trying to open Treemacs (M-x treemacs or M-x +treemacs/toggle), it outputs
Opening output file: No such file or directory, /home/username/.emacs.d/.cache/treemacs-persist-at-last-error
I'm using the default .emacs.d directory, and there actually is no .emacs.d/.cache directory, but there is a .emacs.d/.local/cache directory which contains treemacs-persist file, and in .emacs.d/modules/ui/treemacs/config.el there are lines
treemacs-persist-file (concat doom-cache-dir "treemacs-persist")
treemacs--last-error-persist-file (concat doom-cache-dir "treemacs-persist-at-last-error")),
so maybe there is an issue with doom-cache-dir.
Expected behavior
Open Treemacs window.
Steps to reproduce
M-x treemacs
System information

Click to expand
- OS: gnu/linux (x86_64-pc-linux-gnu)
- Emacs: 26.1 (Jul 05, 2018)
- Doom: 2.0.9 (develop bd9df22f9951e36ba74b61a3619bcc40906ca1d9)
- Graphic display: t (daemon: nil)
- System features: XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GSETTINGS NOTIFY ACL GNUTLS LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB TOOLKIT_SCROLL_BARS GTK3 X11 MODULES THREADS LIBSYSTEMD LCMS2
- Details:
  ```elisp
  elc count: 0
  uname -a:  Linux username 4.20.6-arch1-1-ARCH #1 SMP PREEMPT Thu Jan 31 08:22:01 UTC 2019 x86_64 GNU/Linux
  modules:   (:feature eval (evil +everywhere) file-templates (lookup +docsets) snippets spellcheck syntax-checker workspaces :completion company ivy :ui deft doom doom-dashboard doom-quit evil-goggles fci hl-todo nav-flash neotree treemacs (popup +all +defaults) unicode vc-gutter vi-tilde-fringe window-select :editor format multiple-cursors rotate-text :emacs dired ediff electric hideshow imenu term vc :tools editorconfig ein gist make magit password-store pdf rgb terraform tmux :lang assembly (cc +irony +rtags) coq data elixir emacs-lisp go (java +meghanada) javascript julia latex markdown (org +attach +babel +capture +export +present) php python scala (sh +fish) :app email twitter write :config default)
  packages:  n/a
  exec-path: (~/.scripts/cron ~/.scripts/pix ~/.scripts/statusbar ~/.scripts/tools ~/.scripts/i3cmds ~/.scripts/.getkeys ~/.scripts/ /usr/local/sbin /usr/local/bin /usr/bin /usr/lib/jvm/default/bin /usr/bin/site_perl /usr/bin/vendor_perl /usr/bin/core_perl /usr/lib/emacs/26.1/x86_64-pc-linux-gnu)

As of 3fd1548 this should be resolved. Please update and let me know if that isn't the case and I'll reopen this issue. Thanks for letting me know!

Yep, it works!


fix: Pace requests to token server for new auth tokens

This PR updates the design for token refresh by decoupling the refresh logic from active requests to be consistent with the other cores.
ref: https://github.ibm.com/arf/planning-sdk-squad/issues/1550

@mkistler @padamstx - I've pushed an amended commit that adds the missing calculation of the refreshTime everytime we get a new token. I've also moved back the property refreshTime to be a part of the iamTokenData and cp4dTokenData structs, respectfully.
As far as the concerns over using locks for network calls, it looks like by default we set a client side timeout of 30 seconds.  I've added two new tests to simulate a client side timeout, TestIamGetTokenTimeoutError and a test to simulate a 504 from the server, TestIamGetTokenServerError.
Note: the client timeout doesn't cover the time spent sending a request rather the time spent reading/receiving.

:tada: This PR is included in version 3.3.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Rename project.prisma to schema.prisma

We have settled on the terminology that the schema file schema.prisma contains data sources, data model definition and generators.
This is described in further detail here: https://github.com/prisma/prisma2/blob/master/docs/prisma-schema-file.md

Updated in all examples. The CLI now supports both schema.prisma and project.prisma to not break existing projects.

Related: https://github.com/prisma/prisma2/issues/126


Bug   不支持新版本paddleocr模型生成文件，inference.json inference.pdiparams inference.yml

问题描述 / Problem Description
不支持新版本paddleocr模型生成文件，inference.json inference.pdiparams inference.yml，请修复一下这个bug，可以兼容json pdmodel版本推理模型
运行环境 / Runtime Environment
复现代码 / Reproduction Code

可能解决方案 / Possible solutions

Hello, we tried to solve the issue.
This is what we did:
Modified the PaddleInferSession class to support both new (inference.json, inference.pdiparams, inference.yml) and old (inference.pdmodel, inference.pdiparams) PaddleOCR model formats. Added checks for file existence and appropriate model loading based on available files.
You can review changes in this commit: https://github.com/mnmnk43434/RapidAI-RapidOCR-290/commit/63345e4f7191f34b937437784990db8a1b43e75d.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Query a subset of fields from a table

Hey, I am trying to use p.prospects.query(fields=[a_list_of_field_names]) to pull only a subset of fields from the prospects table. However, the method only returns 2 columns no matter how long my list is. The returned columns will always be id and whatever field that is the last element in the list. Am I not passing the parameters correctly? Thank you!

Hi @tian-l-park , I think you are misunderstanding the query function and confusing it with one of the read functions.
p.prospects.query will run a query to get a list of prospects matching your query criteria. You can then use the ids returned with p.prospects.read_by_id(id=id) to get the full information of a particular prospect.

Hey @mneedham91 , thank you for your response!
Do you know if there is anyway I can query only a subset of fields? I have a big table with 100+ columns and I noticed it takes more than 6s to complete a call. I am thinking maybe by pulling a select number of fields it will run faster. Do you know if there is a function to do that?
Thank you!

Is it possible to provide a feature that supports fields query either that contains None or with values?


Enable testing with IPv6 on Travis

Commit 64ce7af8ecadd68e225ce04572df76acedcf6be5 disabled it.

See https://github.com/travis-ci/travis-ci/issues/8891.


Sequence in which records are selected in the table is not preserved in the Info panel

Issue observed following these changes: https://github.com/Esri/solutions-components/issues/321
Implemented behavior:

Selecting records in the Table is additive to the current selection set

newly selected records (check the box) are added to the selection set ✅
the last selected record is shown in the Info panel (sequence based on user's selection) ❌

Steps:

Open app http://dev0009449.esri.com/solutions-components/solutions-components/demos/crowdsource-manager.html
Observe descending records in table based on ObjectID
In the Table, select records in this order: 701, 690, 695
Observe

in the Information panel, the latest selected record's popup is displayed (as expected)
the order of selected records is incorrect, the position is 1:695, 2:690, 3:701, but should be 1:701, 2:690, 3:695, where the last selected record is in last position.

https://github.com/Esri/solutions-components/assets/50338557/bcb830d7-d74f-403c-9838-21984ccf7f37
Will need to verify again:

[ ] If the current record is removed from the selection, then fall back to the first record listed in the sequence (e.g. 1 of 10) and do not reorder records in the sequence.
[ ] What happens to the order of selection in the Info panel when the sorting order is changed in the table? (expected: should remain unaffected)

After discussing this with John, we decided to hold and wait for further feedback.
there can be added complexity to this considering switch selection.


Avoid having ecp_sm2p256-armv8.pl in fips.module.sources

I'd argue that SM2 related things in crypto/ec are a bit misplaced...  they would have had a better fit in crypto/sm2, that would have avoided this hack.

@levitte, why?
Germany has custom curves,  What makes SM2 special?

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

@levitte, why?
Germany has custom curves, What makes SM2 special?

Why do we have crypto/sm2?  After all, it's essentially EC...

Why do we have crypto/sm2? After all, it's essentially EC...

The signature (and encryption) algorithm is different from ecdsa but the low-level support used is still the same.
Instead of this hack a different hack could place the assembler .pl file into crypto/sm2/asm but the rest of the sm2p256 sources would still be built from crypto/ec as doing otherwise would be too complicated. @levitte Do you think it would be a better hack?

Merged to the master branch. Thank you for the reviews.
IMO OK for now as is. It can be redone differently later if needed.


Bump e2e tests to Pipelines 0.41, add k8s 1.25.

Changes
We previously were not able to run 1.25 due to a bug in Pipelines not supporting new PodSecurity admission. This was fixed in 0.41.
Removes 1.22 since it is EOL as of 2022-10-28 - https://kubernetes.io/releases/#release-v1-22

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/lgtm

/lgtm

/assign @priyawadhwa

/lgtm

/retest

/retest

Closing in favor of #614


Regarding system setup in circular form from Packmol

I want to align say amino-acid in a circular (like micelle form in which the hydrophobic part in embedded inside and the hydrophilic around outside) fashion and then stack it in cylindrical form.
A rough image of the needed structure is shown:

Any suggestions on how can i make it..??
Any suggestions are appreciated.
Thank you.

This is not a software issue. Responded already by e-mail.


CPP_Demo.a2l with mixed types of STRUCTURE_COMPONENTs

Hey there! I am enjoying reading the ASAM MCD-2 MC v1.7.1. I read in the readme of this repository that it is supposed to work with ASAM MCD-2 MC v1.4. So there might be a problem of used specifications. I could not find the specifications for v.1.4 so I decided to raise a possible issue.
In ASAM MCD-2 MC v1.7.1 for the description of "TYPEDEF_STRUCTURE", there is a note on the "STRUCTURE_COMPONENT". It says "It is not allowed to mix up components which represent measure objects and components which represent adjustable objects inside one structure. That means a structure can contain either measure objects only (defined via TYPEDEF_MEASUREMENT) or adjustable objects only (defined via TYPEDEF_AXIS, TYPEDEF_CHARACTERISTIC or TYPEDEF_BLOB)."
The way I understand the CPP_Demo.a2l, there are measurement objects, this is "value" and adjustable objects, these are "par_ampl", "par_phase", and "par_offset". So there are object types mixed up.
Is it an issue?

Hello,
Thanks for your feedback.
The ASAM Standards involved are MCD-1 MC XCP 1.4 and MCD-2 MC (ASAP2 / A2L) 1.71.
As you pointed out, the A2L specification clearly states that objects with measurement and calibration semantics may not be mixed within a structure.
A calibration tool might want to read, modify and persist the value of a structure instance as a whole, by accessing the memory block at instance address with structure length.
Fundamentally, an object with calibration semantics must never be mutated by the target software, but is allowed to be modified by the tool. When a structure contains components with measurement semantics, a tool might observe unexpected changes of the overall structure instance memory content.
So yes, this is a violation of the standard and might be an issue. We do not recommend this, to achieve maximum compatibility with any tool or future compatibility.
And putting a mixed structure instance in an A2L memory segment would definitely be an issue.
There is no problem with CANape, as long as the structure instance memory area is not inside a memory segment.
The XCPlite CCP_Demo does not create memory segments and CANape does not make any assumptions on the memory area of a structure instance.
Note that CANape does not support the CONSISTENT_EXCHANGE keyword yet.
But this might be subject to change in future versions of CANape:
Possible assumptions made on memory areas of calibration structure instances may be the same that are made on memory segments:

The complete memory range is never mutated by the ECU.
Writing the complete memory range does not cause undefined behavior (even if the A2L description of the memory area does not cover all bytes of the range).
Writing the memory range may be done atomic (the concepts may be implementation specific, are not well defined by the standards, XCP and CANape have a few features to support this).
The checksum of the area may be considered as a hash value to optimize caching.

The XCPlite CCP_Demo simply demonstrates a way how to address calibration objects in dynamically allocated memory in a thread safe way.
To avoid possible issues, you could split measurements and calibrations into different objects.
Regards
RainerZ

I understand the big picture you were explaining. Thank you for the clarification.


Welcome page: updated news links

@rpetrusha I agree with you. Moreover, I don't think those links should be on this page at all.
If it's about news, then the better solution might be to provide the links to the sources of news. Almost all the links now send to the following blogs:
https://blogs.msdn.microsoft.com/dotnet/
https://blogs.msdn.microsoft.com/webdev/
https://blogs.msdn.microsoft.com/visualstudio/
https://blogs.msdn.microsoft.com/vbteam/
So, what do you think about replacing all the links with the following text:

For news about .NET, check the following blogs:

.NET blog
ASP.NET blog
...

I think that's a great idea, @pkulikov. That's especially true since this is an article we're likely to forget to update on a regular basis. The one exception to the news-related links is the link to Microsoft Build 2018, which has gotten a significant number of clicks. That suggests we should add a section for Resources.

@rpetrusha I've replaced the links as discussed, also added the links to the "What's new in..." articles. I've tried to avoid to add the Resources section. Please tell if it worked out; if not, I'll make a correction.

I think it looks really good, @pkulikov. And I really liked the handling of Microsoft Build 2018 and .NET Conf. I'll merge your PR.


foliage on dynamic trees does not move during shaders

Does anyone know how to fix this? The leaves on dynamic trees do not move when I have the shader enabled, unlike the foliage of the vanilla tree.

You need to add the leaves to the shader's foliage file

Вам потрібно додати листя до файлу листя шейдера

Do I need to add a foliage block tag? and to which file in the shader folder should I add this? Sorry, I don't know very well about all this.


Zohosign

When submitting a connector, please check the following conditions for your PR.

[ ] I attest that the connector works and I verified by deploying and testing all the operations.
[ ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a logo.

made corrections/update and pushed here.

@ZohoSign-integ

The files in this PR are incorrect and we have already provided feedback to these files in your previous PR.
You can find your connector files here which were reviewed by us. If you want to make any change to your connector, please get these files to your local repo first, then work on updates.
Let us know the reason why you are adding your connector files to custom-connectors folder again. We have already discussed on this in one-to-one meeting with you. Your connector files should be placed in certified-connectors folder only.
And there are branch conflicts in this PR. You are not following the documentation for open sourcing your connector. Please let us know if you are not able to understand the guidelines provided by Priya.

Hello Srikanth/Priya,
Am not sure why the "custom-connectors/ZohoSign" folder still exist under "dev" branch, but it was removed  in my previous https://github.com/microsoft/PowerPlatformConnectors/pull/843, reviewed and then merged with "dev" branch by you.
I just followed the below,

Fork and then Cloned the "dev" repository to my local machine, freshly

Created a new branch "zohosign"

Added new changes under "certified-connectors/ZohoSign" files and then pushed

Finally, this new https://github.com/microsoft/PowerPlatformConnectors/pull/910

I request you to review all once and share your views on this.
Thanks,
Gopi
---- On Wed, 23 Jun 2021 03:03:09 +0530 Srikanth Yenagandhula @.***> wrote ----
https://github.com/ZohoSign-integ
The files in this PR are incorrect and we have already provided feedback to these files in your previous https://github.com/microsoft/PowerPlatformConnectors/pull/843.
You can find your connector files https://github.com/microsoft/PowerPlatformConnectors/tree/dev/certified-connectors/ZohoSign which were reviewed by us. If you want to make any change to your connector, please get these files to your local repo first, then work on updates.
Let us know the reason why you are adding your connector files to custom-connectors folder again. We have already discussed on this in one-to-one meeting with you. Your connector files should be placed in certified-connectors folder only.
And there are branch conflicts in this PR. You are not following the documentation for open sourcing your connector. Please let us know if you are not able to understand the guidelines provided by Priya.
—
You are receiving this because you were mentioned.
Reply to this email directly, https://github.com/microsoft/PowerPlatformConnectors/pull/910#issuecomment-866349351, or https://github.com/notifications/unsubscribe-auth/AT66YSFH7HQTMJSFELTPTBLTUD6RLANCNFSM47DMJLSQ.


Docker CMD

I made a small change to support this  feature
https://github.com/datawhores/OF-Scraper/issues/157
The only difference is that you now have to pass the
ofscraper with he command instead of just the arguments
+
you also need to mount the volumes at /home/ofscraper
and not /root

If you see this
flag provided but not defined: -log
Usage of fixuid:
-q	quiet mode
Then you need to pass ofscraper with the command
docker run -it --rm --name=ofscraper ghcr.io/datawhores/of-scraper:main ofscraper --log
NOT
docker run -it --rm --name=ofscraper ghcr.io/datawhores/of-scraper:main --log


feat!: add encode/decodeValueType as public callable methods

What kind of change does this PR introduce (bug fix, feature, docs update, ...)?
Feature
What is the current behaviour (you can also link to an open issue here)?
The methods encodeValueType and decodeValueType are currently not available as public method part of the ERC725 class.
These methods can be useful to encode certain LSP2 value type raw. For example, a uin128 (For LSP2 Array length) to obtain an exactly 16 bytes long value.
What is the new behaviour (if this is a feature change)?
Make this method part of the class, so that they can be called and used.
Other information:

we need to update the docs as well, when we do this change

Ready to merge - just waiting for conflict fix and docs update: https://github.com/ERC725Alliance/erc725.js/blob/develop/docs/classes/ERC725.md


Disabling bulk actions for some rows should not check all records in the database table

Package
filament/filament
Package Version
v3.0.53
Laravel Version
10.10
Livewire Version
v3.0.5
PHP Version
8.1.10
Problem description
When I added disable bulk action for some rows, it ran to all the records in the database table. This is okay with small records, but if we deal with thousands, the performance significantly drops.
->checkIfRecordIsSelectableUsing(
            fn (Model $record): bool => $record->status === Status::Enabled,
  );

Expected behavior
It should only apply to the current paginated limit list, in my case, the pagination is limited to 10 records. Querying the entire table record is not very performant.
Steps to reproduce
I added this code in the OrderResource of the Filament Demo. Then I checked using the Clockwork Chrome plugin.
->checkIfRecordIsSelectableUsing(
                fn (Model $record): bool => $record->status === "new",
            )

Reproduction repository
https://github.com/filamentphp/demo
Relevant log output
No response

We cannot pass each $record instance into the function without first fetching them from the database. We get all records so we can render the count for the total number of selectable records.
Try selectCurrentPageOnly() on the table


In ClusterChangeMediator, remove sleep and make it notify based

When we get the Helix cluster change callbacks, we want to process them ASAP
Replace sleep() with wait() and notify() so that the new changes are processed immediately
Testing done:

Check that the queue time matches with the processing time
Run tests without starting the cluster change handling thread
Reduce proactive change check interval to 1 second and check it interacts correctly with Helix callbacks

Codecov Report

Merging #3898 into master will decrease coverage by 0.07%.
The diff coverage is 82.35%.

@@             Coverage Diff              @@
##             master    #3898      +/-   ##
============================================
- Coverage     67.31%   67.23%   -0.08%
  Complexity        4        4
============================================
  Files          1032     1032
  Lines         50865    50877      +12
  Branches       7105     7107       +2
============================================
- Hits          34238    34209      -29
- Misses        14292    14329      +37
- Partials       2335     2339       +4

Impacted Files
Coverage Δ
Complexity Δ

...not/broker/broker/helix/ClusterChangeMediator.java
66.66% <82.35%> (+1.51%)
0 <0> (ø)
:arrow_down:

...ller/validation/OfflineSegmentIntervalChecker.java
26.74% <0%> (-39.54%)
0% <0%> (ø)

...apache/pinot/common/metrics/ValidationMetrics.java
43.47% <0%> (-36.24%)
0% <0%> (ø)

.../org/apache/pinot/client/PinotClientException.java
33.33% <0%> (-33.34%)
0% <0%> (ø)

...egation/function/customobject/MinMaxRangePair.java
75.86% <0%> (-24.14%)
0% <0%> (ø)

...inot/client/JsonAsyncHttpPinotClientTransport.java
63.63% <0%> (-21.22%)
0% <0%> (ø)

...pinot/broker/api/resources/PinotClientRequest.java
23.8% <0%> (-19.05%)
0% <0%> (ø)

...lix/EmptyBrokerOnlineOfflineStateModelFactory.java
86.66% <0%> (-13.34%)
0% <0%> (ø)

...pinot/core/operator/docidsets/OrBlockDocIdSet.java
84.9% <0%> (-9.44%)
0% <0%> (ø)

...e/impl/dictionary/LongOnHeapMutableDictionary.java
88.88% <0%> (-6.67%)
0% <0%> (ø)

... and 24 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 4964f91...c71a67d. Read the comment docs.

@kishoreg Manually tested. Also, if the thread is not notified, all the integration tests will fail due to the 6 minutes wait()


Works on the public schema only.

Hi,
The only table the driver shows are ones in my public schema. Tables from other schemas no matter the access rights of the connecting user are not shown.

Schema support for tables has been added with version 0.3.


Bodyparts compatibility layers aren't registered/working and casuing crash to NPC

species Compatibility layer R3 causes crashes when trying to load parts onto npcs. In my previous post i reported that hypertus no longer sees parts from other mods as hyperable, seems like it causes a crash whenever a dynamic npc tries to equip a hyperable part through species compatibility layer. Seems like  the mod sees that the other mods are installed but for whatever reason cannot load those parts as hyperable which causes a crash whenever an npc generates with said parts. The log file's last entry before the crash is:
addDynamicCharacter(): Adding dynamicnpc0 character No stack available
[RopeBunny, BreedingStock]
ERROR: bodypart with the id avianpenishyperable wasn't found
As in my last post, i'd like to say that R6-1 loaded everything fine.

Please try again with R6-2

The problem i believe is that hypertus R6-2 cannot recognize mod parts and convert them into hyperable versions even though it sees the mods just fine and recognizes that they're installed. i have attached the entire log file this time. R6-1 worked fine.
godot.log

Hyper tree is null???? send save file to the dev -CanInBad

and

addDynamicCharacter(): Adding dynamicnpc0 character No stack available

Please send out a copy of your save file.

Ill try to get a game going where it doesn't crash instantly after pressing the wake up button(no npcs with hyperable parts generate). Btw, i must say one more thing, i did try launching the game with only hypertus, foxlib and compatible parts mods.
Save attached.
new save.zip

@DarkMessiahx I haven't try your save yet, the reason why hyperable parts don't exist because it uses Module's register function and I careless-ly move the function that add bodyparts to registration past the registration phase. https://github.com/CanInBad/HypertusMod/commit/d01f72b58dee6e4a64ad9075c18a09fe0169125c attempts to fix that and when I test it works on my machine™
Please try this HypertusP6-3.zip or the commit mentioned to see if its fixed
if its fixed please close the issue with a tag comment to me, else keep this up.
I will come check again tomorrow evening (8-9pm) my time.

Everything works perfectly, still getting the (Hyper tree is null???? send save file to the dev -CanInBad) thing in the log but now every part shows up as selectable, and npcs can generate just fine, started a new game 10 times just to be safe and didn't crash at all, all npcs generated, no errors, other then the hyper tree is null one, which i have absolutely no idea about what effect it actually has on the game, as my hyper skill tree works just fine, so it must be something related to npcs but i cant for my life figure out what.
Anyways, my main issues (the crash and missing parts) were fixed with the preview build linked above, thank you!
I will close this as the main issue in my op post has been fixed but you may want to look into that hyper tree is null error the log gives and what problems it's causing. Other then that, thx for the swift fix.


refactor: ScopedDeleter doesn't require metadata

Before the change, ScopedDeleter kept ObjectMetadata of all object
to delete. This was unnecessary - object names and generation numbers
are enough, so this change stores them instead of the metadata. This is
useful when the caller has the information but doesn't have the full
metadata.

This change is

Codecov Report

Merging #3380 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3380      +/-   ##
==========================================
+ Coverage   89.34%   89.35%   +<.01%
==========================================
  Files         461      461
  Lines       41929    41944      +15
==========================================
+ Hits        37463    37477      +14
- Misses       4466     4467       +1

Impacted Files
Coverage Δ

google/cloud/storage/parallel_upload.h
96.18% <100%> (-0.03%)
:arrow_down:

google/cloud/storage/client.h
99.77% <100%> (-0.01%)
:arrow_down:

google/cloud/storage/client.cc
94.97% <100%> (+0.12%)
:arrow_up:

google/cloud/storage/well_known_parameters.h
69.56% <0%> (-7.61%)
:arrow_down:

google/cloud/storage/well_known_headers.h
70.37% <0%> (-3.71%)
:arrow_down:

...le/cloud/bigtable/internal/async_retry_unary_rpc.h
78.57% <0%> (-2.92%)
:arrow_down:

google/cloud/bigtable/metadata_update_policy.h
100% <0%> (ø)
:arrow_up:

google/cloud/bigtable/app_profile_config.h
100% <0%> (ø)
:arrow_up:

...gle/cloud/storage/internal/access_control_common.h
100% <0%> (ø)
:arrow_up:

google/cloud/bigtable/mutations.h
100% <0%> (ø)
:arrow_up:

... and 5 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 087b578...9af8101. Read the comment docs.


[2.x] Fix permission issue when creating JWTParser after jjwt upgrade

Description
After the upgrade of JJWT from https://github.com/opensearch-project/security/pull/3092, the JWT Integration Tests in security-dashboards-plugin started failing. See details on: https://github.com/opensearch-project/security-dashboards-plugin/issues/1555
The output from the tests in Github actions was not very revealing, but @RyanL1997 pointed out that when running the test locally after creating a local distro of core + security plugin with 2.x it would output a permissions error:
Caused by: java.security.AccessControlException: access denied ("java.lang.RuntimePermission" "getClassLoader")
	at java.security.AccessControlContext.checkPermission(AccessControlContext.java:485) ~[?:?]
	at java.security.AccessController.checkPermission(AccessController.java:1068) ~[?:?]
	at java.lang.SecurityManager.checkPermission(SecurityManager.java:416) ~[?:?]
	at java.lang.ClassLoader.checkClassLoaderPermission(ClassLoader.java:2060) ~[?:?]
	at java.lang.Thread.getContextClassLoader(Thread.java:1493) ~[?:?]
	at io.jsonwebtoken.impl.lang.Services$1.getClassLoader(Services.java:37) ~[jjwt-impl-0.11.5.jar:0.11.5]
	at io.jsonwebtoken.impl.lang.Services.loadFirst(Services.java:100) ~[jjwt-impl-0.11.5.jar:0.11.5]
	at io.jsonwebtoken.impl.DefaultJwtParserBuilder.build(DefaultJwtParserBuilder.java:191) ~[jjwt-impl-0.11.5.jar:0.11.5]
	at com.amazon.dlic.auth.http.jwt.HTTPJwtAuthenticator.<init>(HTTPJwtAuthenticator.java:135) ~[opensearch-security-<IP_ADDRESS>-SNAPSHOT.jar:<IP_ADDRESS>-SNAPSHOT]
        ...

This PR surrounds the call to build the parser with AccessController.doPrivileged(...) to ensure the call has the privileges needed to create the parser.
This change should also be applied to main. For some reason, the tests are not failing when running on the main branch but are on 2.x.

Category (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)

Bug fix
Issues Resolved
https://github.com/opensearch-project/security-dashboards-plugin/issues/1555
Check List

[ ] New functionality includes testing
[ ] New functionality has been documented
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

This works on main w/o the same fix and I am trying to determine why.


Fix: change wallet type property from camel to snake case

What changed? Why?

changed wallet type event property from camel to snake case. This follows Amplitude convention.

Notes to reviewers
How has it been tested?

locally

Review Error for kirkas @ 2024-07-19 20:36:03 UTC
User failed mfa authentication, see go/mfa-help


allowDropFileTypes issue

Drag and Drop ignores valid files if there are invalid file types in the draggable list when we limit types with allowDropFileTypes option.

GIF

Example (jsfiddle):

Add textarea:

<textarea id="textarea"></textarea>

Create a CodeMirror instance with allowDropFileTypes option:

let options = {
  allowDropFileTypes: ['text/css']
}
CodeMirror.fromTextArea(document.getElementById('textarea'), options);

Prepare three files: image.png (actual image), valid.css and invalid.css (binary file).

Drop the valid.css - you'll see its content in editor.
Drop the valid.css and invalid.css - you'll see the valid.css content.
[issue] Drop the valid.css and image.png - you'll see nothing.

Expected behaviour: we should see the valid.css content in editor.

OS: macOS Catalina 10.15.2
Browser: Chrome 79.0.3945.88
CodeMirror: 5.50.0

Added a pull request for a fix for the issue: https://github.com/codemirror/CodeMirror/pull/6102
The current logic aborts the whole operation if there is any error (expect bad content) in any of the files. The pull request tries to fix this by filtering out any files with errors and then pasting the valid files.


[Backport 2.x] Added release notes for 2.10 release (#277)

Description
Added release notes for 2.10 release (#277)
Issues Resolved
https://github.com/opensearch-project/neural-search/issues/224
Check List

[X] New functionality includes testing.

[X] All tests pass

[X] New functionality has been documented.

[X] New functionality has javadoc added

[X] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

The build is failing due to :
Exception in thread "main" java.lang.IllegalStateException: failed to load plugin opensearch-ml due to jar hell
| 	at org.opensearch.plugins.PluginsService.checkBundleJarHell(PluginsService.java:681)
| 	at org.opensearch.plugins.InstallPluginCommand.jarHellCheck(InstallPluginCommand.java:862)
| 	at org.opensearch.plugins.InstallPluginCommand.loadPluginInfo(InstallPluginCommand.java:830)
| 	at org.opensearch.plugins.InstallPluginCommand.installPlugin(InstallPluginCommand.java:875)
| 	at org.opensearch.plugins.InstallPluginCommand.execute(InstallPluginCommand.java:276)
| 	at org.opensearch.plugins.InstallPluginCommand.execute(InstallPluginCommand.java:250)
| 	at org.opensearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:104)
| 	at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
| 	at org.opensearch.cli.MultiCommand.execute(MultiCommand.java:104)
| 	at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138)
| 	at org.opensearch.cli.Command.main(Command.java:101)
| 	at org.opensearch.plugins.PluginCli.main(PluginCli.java:60)
| Caused by: java.lang.IllegalStateException: jar hell!
| class: org.apache.commons.lang3.AnnotationUtils$1
| jar1: /home/runner/work/neural-search/neural-search/build/testclusters/integTest-0/distro/2.10.0-ARCHIVE/lib/commons-lang3-3.13.0.jar
| jar2: /home/runner/work/neural-search/neural-search/build/testclusters/integTest-0/distro/2.10.0-ARCHIVE/plugins/.installing-40852478789188[98](https://github.com/opensearch-project/neural-search/actions/runs/6093097774/job/16532212834?pr=287#step:4:99)473/commons-lang3-3.10.jar
| 	at org.opensearch.bootstrap.JarHell.checkClass(JarHell.java:316)
| 	at org.opensearch.bootstrap.JarHell.checkJarHell(JarHell.java:215)
| 	at org.opensearch.plugins.PluginsService.checkBundleJarHell(PluginsService.java:679)
| 	... 11 more

Resolved the apache commons dependency which was coming from core.
(base) 22:36 ~/workplace/neural-search 2.x$ ./gradlew :dependencyInsight --configuration resolveableCompileOnly --dependency org.apache.commons:commons-lang3
Picked up JAVA_TOOL_OPTIONS: -Dlog4j2.formatMsgNoLookups=true
=======================================
OpenSearch Build Hamster says Hello!
  Gradle Version        : 8.1.1
  OS Info               : Mac OS X 12.6.8 (x86_64)
  JDK Version           : 11 (Eclipse Temurin JDK)
  JAVA_HOME             : /Users/navneev/.sdkman/candidates/java/11.0.16-tem
  Random Testing Seed   : B3553F4A6A551760
  In FIPS 140 mode      : false
=======================================

> Task :dependencyInsight
Dependency resolution failed because of conflict(s) on the following module(s):
   - org.apache.commons:commons-lang3 between versions 3.13.0 and 3.10

org.apache.commons:commons-lang3:3.13.0
  Variant runtime:
    | Attribute Name             | Provided     | Requested |
    |----------------------------|--------------|-----------|
    | org.gradle.category        | library      |           |
    | org.gradle.libraryelements | jar          |           |
    | org.gradle.status          | release      |           |
    | org.gradle.usage           | java-runtime |           |
   Selection reasons:
      - By conflict resolution: between versions 3.13.0 and 3.10

org.apache.commons:commons-lang3:3.13.0
\--- org.opensearch:opensearch-encryption-sdk:2.10.0-SNAPSHOT:20230906.043946-24
     \--- org.opensearch:opensearch:2.10.0-SNAPSHOT:20230906.043946-240
          \--- resolveableCompileOnly

org.apache.commons:commons-lang3:3.10 -> 3.13.0
\--- resolveableCompileOnly

A web-based, searchable dependency report is available by adding the --scan option.

BUILD SUCCESSFUL in 716ms


project isn't working without code changes

This page of the tutoral isn't working for me
https://codelabs.developers.google.com/codelabs/chrome-es2015/index.html?index=2#1
the following two console errors are logged when I try to make a new note
main.js:149 Uncaught TypeError: Failed to execute 'define' on 'CustomElementRegistry': The callback provided as parameter 2 is not a function.     at main.js:149and main.js:74 Uncaught TypeError: note.setMessage is not a function
at StickyNotesApp.displayNote (main.js:74)
at new StickyNotesApp (main.js:34)
at main.js:88
`

Same here.

Same

Change createdCallback to connectedCallback, it will work.

@nkurapati : not working for me
I had to check the version of @JuanMaRuiz : https://github.com/JuanMaRuiz/chrome-es2015 (opened pull request) to get the project to work.

This problem still persist. JuanMaRuiz's repo have same problem too. ; (


Type error on PolygonExtrusionLayerOptions

I think there is an error in the type definition of the PolygonExtrusionLayerOptions: fillPattern?: string; should be fillPattern?: string | DataDrivenPropertyValueSpecification<string>; as in the type definition of PolygonLayerOptions.
With the current definition, you cannot define the fillPattern as an expression.

According to the docs, expressions are not supported on the fill pattern option of extruded polygon layers: https://learn.microsoft.com/en-us/javascript/api/azure-maps-control/atlas.polygonextrusionlayeroptions?view=azure-maps-typescript-latest#azure-maps-control-atlas-polygonextrusionlayeroptions-fillpattern
Interestingly I just tested an expression with an extruded polygon layer recently and was surprised it worked. I remember when this was first added to Azure Maps 5 or 6 years ago, and the underlying map render engine (used to be Mapbox gl js, now MapLibre), had a lot of bugs with expressions for fill-patterns so there was a decision not to document that at the time. If you passed one in (either JavaScript or ignored the Typescript type error), it would still work, but in the past had a lot of issues. It looks like this is more stable than it issued to be, so it may be safe to document this now. I'll reach out to the Azure Maps team to let them know.

@rbrundritt Yes, I created this ticket because I also tested it and it worked. :)

Hi, @Jxlle , thanks for reporting this. We will update the API spec to reflect the feasibility.


Making ECR docs more declarative for Grype

A little confusing how to configure ECR vs Docker, making it a little more obvious
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-465


[Peek][Monaco Preview] Open Monaco URI in default browser

Summary of the Pull Request
Open Monaco URI in default browser
Aligned behavior in Monaco Preview Pane

PR Checklist

[x] Closes: #27649
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed
Tested on Peek and PreviewPane:

CTRL+Click on a URI in a file previewed in Monaco
Link is opened in the default browser

/azp run

Haven't thought about security since we are already doing this with PDF and MD links but that's a valid point.

I have checked how things are working now:

PT
PDF
MD
Monaco

Peek
default browser (not working https://github.com/microsoft/PowerToys/issues/27403)
default browser (not working https://github.com/microsoft/PowerToys/issues/27403)
not default browser

PreviewPane
doesn't open URLs
default browser
not default browser

Do we want to have a confirmation dialog?
Do we want this to be configurable?

My issues were not because of the URL that open browsers. More on the fact that it wants to support general URIs, which might open other apps.
I don't know if we can do dialogs on Preview Pane, more thinking of those for Peek, especially since it can run as admin. 🤔

Thanks for the investigation, by the way 😄

We could look to only handle http: and https: protocols. Opening other protocols from a text file is something that would be unexpected anyway.

Looks like Monaco provides URI navigation for http:// and https:// but for MD and PDF you can have any kind of URIs (for example mailto://).
I am fine with add a dialog for Peek. Should we make this optional or let's wait for user feedbacks?

Let's wait for user feedback :)
But I think a confirmation dialog would be a good default for Peek.

Ready to review!
I was inspired by VSCode but Peek have a much better looking dialog! 😃

I have tested this on Monaco, Markdown and PDF files with https and mailto URIs in order to test browser and another app.
There was a problem on MD and PDF. I made a change and left a TODO that would be nice to revert once WebView2 is fixed. The related issue is https://github.com/microsoft/PowerToys/issues/27403 (let's keep it opened even if fixed by the PR).
The workaround shouldn't have any side effects.
Note that WebView2 is not raising events when clicking a link on a PDF with CTRL+Click and the dialog is bypassed 😞 Probably related to https://github.com/MicrosoftEdge/WebView2Feedback/issues/2004

@davidegiacometti Feel free to merge after resolving merge conflicts :)


reuse of existing ws connection

hi
would it be possible to use an existing ws connection?
the ws server could be the rest client... and it would adapt to my use case :)

you could have that connection as a variable stored somewhere and just use inside the ware
in case you need server options, you can pass them to middleware args
so please be more concise

let's say we have this config :
machine A : behind a firewall and unreachable from the internet
machine B : on the internet and reachable by machine A
machine B needs to control machine A
i can use good old tcp connection or socket.io or websocket but i'd like to use http methods (with express middleware)
so the need is the following :
machine B runs a websocket server
machine A runs a http server with express middleware (machine B won't connect to this server)
machine A connects to ws of machine B to create a communication link
machine B sends http methods to machine A through the ws link
machine A receive http methods from the ws link and route them with express

@ichbtm how should that websocket link be sent? through what
with http response?

not sure to understand your question but i imagine that the ws could be passed as an argument to the tinyws constructor.
if this argument is non null then tinyws would reuse this link instead of using/creating a http server
it's king of a reverse connected http server in fact (the http server create the connection with the client)

@ichbtm ok I got you
I will add an optional ws object argument as second argument in the ware


DragSourceOptions type is not aligned with the docs?

Describe the bug
useDrag DragSourceOptions type is not aligned with the docs
Docs on useDrag mention options: "specifying a custom arePropsEqual(props, otherProps) function"
But the actual type contains only dropEffect
https://github.com/react-dnd/react-dnd/blob/80733610ef7fb6104112697686cc7e4bd857105b/packages/react-dnd/src/hooks/types.ts#L40
https://github.com/react-dnd/react-dnd/blob/80733610ef7fb6104112697686cc7e4bd857105b/packages/react-dnd/src/types/options.ts#L1-L8
Reproduction
codepen

have useDrag
populate it with options dict
add arePropsEqual
see that typescript complains about the type

Expected behavior
Please either update the docs or populate the type with valid keys
Screenshots

Desktop (please complete the following information):

OS: Mac
Browser Chrome latest

Tried to fix it myself, but have no branch permisssions
cc @darthtrevino sorry a typescript types guy here, I would be happy to contribute if you can give me permissions, we are using react-dnd extensively in our codebase

Sure, you can write PRs now. This looks like a documentation issue. What you probably want is to use index in the deps array (it acts like useMemo).
e.g. useDrag(() => {...spec}, [id, index])

Smashed a change in main

Thanks @darthtrevino lemme craft a PR


Text layer suggestions

Feature suggestion:

Text font selection
Text size selection
Text rotation angle (very common for funny videos: the text is slightly rotated)
Text background color

thanks!


New version: TuringPatterns v0.0.3

Registering package: TuringPatterns
Repository: https://github.com/yurivish/TuringPatterns.jl
Created by: @juliohm
Version: v0.0.3
Commit: 6356abec77554c6b829977264c698ee1586af632
Reviewed by: @juliohm
Reference: https://github.com/yurivish/TuringPatterns.jl/commit/6356abec77554c6b829977264c698ee1586af632#commitcomment-33970478

Depends on #1423

You merged this one without waiting for #1423

Yes.


evaluate-response should be a single form with options

Right now, the evaluate response page shows the options as different separate forms:
https://github.com/tweaselORG/platform/blob/6f627350058dd1aedbe8e5bc2015274d51f59207/src/pages/p/[token]/evaluate-response/index.astro#L45-L53
However, semantically this should be an option input which submits a single form when clicked/focus is lost. This would require us to explain to users that a context change will occur (https://www.w3.org/TR/2016/NOTE-WCAG20-TECHS-20161007/G13).
But instead, we could also provide a separate submit button altogether (labeled "Continue" or something), which I think is better practice anyway (https://www.w3.org/TR/2016/NOTE-WCAG20-TECHS-20161007/G80). At least was a bit startled after clicking one of the links and immediately starting an analysis.

I also suggest redirecting to the /p/ page instead of erroring, if the user accesses the page with a wrong proceeding status.


Store: Add a few objectives for Store's data touched/fetched amount and sizes

Signed-off-by: Douglas Camata<EMAIL_ADDRESS>

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Add a few objectives for the summaries reporting the quantity and size of items fetched per data type from object storage & cache. They are: 50, 95, and 99 quantiles.

The intent is to provide some metrics that will help inform users when setting values for the flags store.grpc.series-sample-limit and store.grpc.touched-series-limit.
Verification

Ran it locally and confirmed the metrics are there and producing the configured quantiles.

@moadz I would love to simplify and make the metric names shorter, but I don't think it's a good idea to change their names after they have been out since many releases. The reason is that people might be already relying on them for dashboards (i.e. kube-thanos uses them) and alerts.


WCHAR is not the same between Windows and Linux, how about typedef char16_t WCHAR?

https://github.com/microsoft/DirectX-Headers/blob/48f23952bc08a6dce0727339c07cedbc4797356c/include/wsl/stubs/basetsd.h#L55C8-L55C8
That will makes it the same between Windows and Linux

No. The intent is not ABI compatibility a-la WINE. WCHAR should be usable with C library wide char functions.


Add rxjs to the docker suite

Fixes #33125
They're still broken, as they've not opted to update @types/node (plus another error).

cc @benlesh

I'm going to merge this and just update the baselines tomorrow, since they are surely quite different than when this was created.


get tag not updating on changes from set tag

Version: 0.1.8
Details
The <get> tag isn't receiving the changes from the <set> tag whenever the <set> tag has its value updated.
Expected Behavior
The <get> tag should update its returned value upon the <set> tag's context changing.
Actual Behavior
Nothing happens and nothing is updated.
Possible Fix
Firstly, It seems like there's a typo in this line.
        this.sub = this.subscribeTo(this.provider).on("___change", this.sync);

should be:
        this.sub = this.subscribeTo(this.provider).on("___changed", this.sync);

in order to match the name of event that's emitted from the <set> tag..
Additionally, in the <set> tag, because component.___rendered is set to true, the ___changed event is only ever emitted once and not on each render. I don't know if this is intentional and the ___changed event is intended to be fired elsewhere (I could not find any other references in the codebase), but changing:
  if (typeof window === "object" && !component.___rendered) {
    component.___rendered = true;
    component.emit("___changed");
  }

to
  if (typeof window === "object" && typeof component.emit === 'function') {
    component.emit("___changed");
  }

seems to solve the issue and allows the tag to update.
Additional Info
Your Environment

Environment name and version (e.g. Chrome 39, node.js 5.4): Node 12
Operating System and version (desktop or mobile): MacOS
Link to your project:

Steps to Reproduce
https://codesandbox.io/s/twilight-wind-wojol?file=/components/test.marko
Stack Trace

Greatly appreciate you taking the time to look into this. I'll try my best to get a patch out soon! Surprised our test suite missed this case, but guess we must not be testing when a child is not rerendered with its parent.

Scratch that, you made a PR already!


move tutorials folder to csharp folder

cc @BillWagner @richlander moving csharp tutorials

LGTM.
:shipit: when ready.


pipeline example not working.

I have cloned the pipeline example and I have got the following exception:
raise ValueError('Could not find base_task_project={} base_task_name={}'.format(
ValueError: Could not find base_task_project=examples base_task_name=pipeline step 1 dataset artifact

The error is strait forward, maybe the execution of the pipeline on my end is incorrect ?
For context I am using python 3.8.5, and the execution line is: python3 pipeline_controller.py

Hi @AndroZa0812
I'm assuming you are referring to this example.
In order to launch the pipeline, we have to make sure all the pipeline steps are in the system.
In order to do that, someone, somewhere sometime executed a code once and that created the initial experiment (Task).
Specifically in this example, you have to first run steps 1,2,3 , they will each create its own task in the system.
Then you can run the pipeline, that clones those experiments and launches them (enqueues them into an execution queue).
Make sense ?

Oh thanks , it worked !
I have executed all the stages and then executed the pipeline and it worked. Very cool concept I must say .

Very cool concept I must say .

Thank you @AndroZa0812 !
Feel free to tell your friend ;)


Problem with root owned folders and files

dat link seems to not include root owned folders and files if executed as a normal user.

Hey,
Can you possibly show us the exact steps and the ls -l of the folder you're trying to share, and what happens?
Thanks a bunch!

I think @mafintosh just fixed this in master


Blank page in Firefox

Nothing is displayed.
The error in the console shows:
NS_ERROR_UNEXPECTED:  jpg-glitch:1:0

hey @xuv thank you for taking the time to report this bug. unfortunately i'm having a hard time reproducing this bug.
could you help me out and provide some details?

which operating system were you using?
which version of firefox were you using?
does this bug occur if you reload the page?
were you on a mobile connection or a wifi?

thank you!

Hey @snorpey
Sorry. My fault. I must have some weird addon to Firefox that blocked something. It works in Privacy mode. So now worries. Excellent work. Love it. And thanks for your reply.


Storage : create a signed upload policy for uploading objects

Fixes #6657
Did the Implementation as per python Bucket.generate_upload_policy. Removing snippets, As google-cloud-bom is pulling older version of google-cloud-storage. I will raise a separate PR for snippets once google-cloud-bom is updated

@athakor, thanks for creating this PR. I'd say keep PR as an example as we go into Q12020 but we will need to modify it to work with V4 signed URLs as the documentation pointed to right now is specifically for V2.
I'm closing this PR as we should not continue adding support for V2 features.


Add Styleguidist for docs

@joamag this is now (finally) working:

https://ripe-sdk-components-react-git-gc-docs.platforme.vercel.app/
https://ripe-sdk-components-react-git-gc-docs.platforme.vercel.app/storybook

@joamag I believe this one is good to go. The unit tests are being addressed in https://github.com/ripe-tech/ripe-sdk-components-react/pull/18


'shift-d' delete with selection doesn't work as expected

If I select a couple of lines and then I hit 'shift+d' I expect that will delete those lines completely but instead it behaves like normal 'd' in vim deleting till selection.

As stated in the README, this package is no longer maintained and is deprecated. We recommend that people use the vim-mode-plus package instead. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!


Update cloneSourceAppKidToTargetApp.js

turned it into a bookmarklet (but it can still run as a Snippet, etc)
tested a little, but a little more can't hurt

Thanks I'll just need to update my readme when I merge this

Tested and works great as well ty


Feat injector

:tada: This PR is included in version 8.0.0 :tada:
The release is available on:

v8.0.0
GitHub release

Your semantic-release bot :package::rocket:


Couldn't open the code-server, just opening with a black window

Open the code-server in the browser, and it opening with a black window. There is nothing. I find the localStorage also has nothing.
Description
I open the code-server in the browser, and it opening with a black window. There is nothing. I find the localStorage also has nothing. I have changed some browser, such as chrome and IE. It doesn't affect.

I found the similar situation. In log i find this

ERROR SHARED stderr {"data":"Uncaught Exception:  Error: Unexpected end of JSON input\n\nSyntaxError: Unexpected end of JSON input\n    at JSON.parse ()\n    at pfs_1.readFile.then.contents (eval at exports.requireModule (/usr/local/bin/code-server:468:27625), :81639:45)\n    at \n    at Timeout.setTimeout [as _onTimeout] (eval at exports.requireModule (/usr/local/bin/code-server:468:27625), :56921:31)\n    at ontimeout (timers.js:498:11)\n    at tryOnTimeout (timers.js:323:5)\n    at Timer.listOnTimeout (timers.js:290:5)\n"}

In browser there is 404 on request http://host:8443/__webpack_hmr
Run on last available docker image

That error shouldn't matter. What version of chrome are you using @JinGangRed ?

I've struck this as well. From memory I've always seem the 404 on webpack_hmr, but I've noticed this in the JS console now:
 INFO  Loading IDE
webpack-internal:///./packages/logger/out/main.js:1  INFO  Starting socket [1/5]...
/__webpack_hmr:1 Failed to load resource: the server responded with a status of 404 ()
webpack-internal:///./packages/logger/out/main.js:1  INFO  Connected to socket
webpack-internal:///./packages/logger/out/main.js:1  INFO  Start workbench
webpack-internal:///./packages/logger/out/main.js:1  ERROR  Failed "Start workbench" duration=759ms
main.js?f92d:1 duration (i): iexpected: 1000ms:<PHONE_NUMBER>091__proto__: Object
main.js?f92d:1 error (Error): Error: [UriError]: Scheme contains illegal characters.
    at _validateUri (uri.ts?336c:38)
    at _URI.URI (uri.ts?336c:175)
    at new _URI (uri.ts?336c:391)
    at Function.URI.revive (uri.ts?336c:367)
    at Object.reviveWorkspaceIdentifier (workspaces.ts?067d:38)
    at CodeRendererMain.reviveUris (main.ts?bd53:89)
    at CodeRendererMain.init (main.ts?bd53:70)
    at new CodeRendererMain (main.ts?bd53:61)
    at main (main.ts?bd53:331)
    at Workbench.eval (workbench.ts?f527:192)
webpack-internal:///./packages/logger/out/main.js:1  ERROR  [UriError]: Scheme contains illegal characters.

@kylecarbs The version of chrome is Version 70.0.3538.77

There is nothing in the Local Storage and session storage.

The same problem:  (version 1.32.0-310)

@JinGangRed could you try updating chrome? I'm not certain what API we'd be using that could cause the issue.

@kylecarbs Thanks for  your replay.
I can load it before on this version of chrome. Now I couldn't open it, not only opening on the chrome, but the Firefox on another machine.
I think the container of mine is down. This issue has occur, when I created a workspace before. Then I remove the Local Storage and reopen it. It works. But this time, there are nothing in the Local Storage, I have tried clean it. But it doesn't work.
I think it will be better if we create a web admin UI to manage the container.

Same happened to me after adding a folder to the workspace. Black screen after that, even if I restart code-server.

I have the same issue - occurs immediately once I load my usual workspace. This workspace contains three folders - initially with relative paths but it did not work if I changed it to absolute paths either.

Also getting this black screen error; occurs on latest chrome, brave and safari, whenever I add a folder. This seems like a pretty critical bug to me.

I get this when in an office where the firewall blocks websockets.
It's pretty clear from the console that this is the issue in my case as it is retrying and failing to connect to the sockets. I have run separate scripts to demonstrate that the firewall is indeed blocking websockets.
That's not something that can be fixed so perhaps a getting started document could list some pre-requisites for the application.

Same here, code-server is running on Ubuntu 18.04. When i connect from my macbook pro using either safari or chrome it works.  I tried safari, chrome and firefox neither works. I just downloaded the latest version and tested, same thing. If you need any kind of help, documentation of the problem let me know i will do my best to get it for you.

There was an old bug that would show that error if you set a custom workspace. Try going into localstorage and delete the workspace property.

@nol166  yes. I have delete the workspace property once,and it works. However, there were nothing in the localstorage now.

Also experiencing this issue from iPad Pro

@AALMA Apparently only the new iPad OS works well with code-server. The current iOS either doesn't load code-server or lots of keyboard commands don't work correctly.

Experiencing this myself on iPadOS Beta 2 in Safari. Just a blank/black screen. I don't run code-server with auth, but I do use an ngrok.io tunnel to provide my basic auth and https. Don't have any issues using ngrok on the same server for hosting other sites so don't think it's related to that?
I am hosting on AWS lightsail for what it's worth.

I am seeing the same black screen on iPadOS Beta 2 in Chome/Safari.  I have auth w/no tunnel.

I am seeing the same black screen on iPadOS Beta 2 on iPad pro 1 1 in Chome/Safari. I have auth w/no tunnel.

Same issue. Btw I used to deploy on DO using docker and works perfectly

If you're using iPad and seeing a black screen with HTTP authentication through a reverse proxy (not code-server's built-in login screen), this is caused by a webkit bug. The browser won't send the HTTP password with the request to connect to the websocket.

I think this issue #89 is the same.
And it's not related code-server it's an old WebKit browser bug Bug 80362

Just updated to iPadOS beta 3 - same issue

The OP was using Chrome, so I won't close this issue. If you're having issues with iPad/Safari, please discuss or :+1: in #89.

For iPad I fixed it with a cert from lets-encrypt.

For iPad I fixed it with a cert from lets-encrypt.

Would you mind publishing a post on how to do it?

For iPad I fixed it with a cert from lets-encrypt.

Which version of iOS or iPadOS are you using?

For iPad I fixed it with a cert from lets-encrypt.

Which version of iOS or iPadOS are you using?
Are you using HTTP Basic access authentication?

Beta 13.0. I think this is the latest version as I installed it 3 days ago.

I am having the same issue (blank black screen)  trying to access it from the iPad Browsers (Safari and Chrome).
When I try to access is from Windows Desktop (Chrome) it seems to work but doesn't load any of the icons (There was an error on my VPS that says favicon not found).
I was not able to follow the comments along probably. Does this have any fix ?

I have the same issue - I can connect and it works great on mac - chrome but on ipad os 13.1 safari I get the following 500 error and a black screen.

Weird. I updated to the most recent build of iPad OS 13.1 and ran into the blank page issue on Safari and other App Store browsers (chrome, Firefox, etc). Linux desktop is a virtual machine inside a Synology NAS. I used the self-signed cert from the quick setup documentation.
I got around it by exporting my Let’s Encrypt cert and key from my Synology and slotted in directly to the code-server app (no need to convert formats or anything...pem direct) like another user did. Suddenly it worked.

Weird. I updated to the most recent build of iPad OS 13.1 and ran into the blank page issue on Safari and other App Store browsers (chrome, Firefox, etc). Linux desktop is a virtual machine inside a Synology NAS. I used the self-signed cert from the quick setup documentation.
I got around it by exporting my Let’s Encrypt cert and key from my Synology and slotted in directly to the code-server app (no need to convert formats or anything...pem direct) like another user did. Suddenly it worked.

Stable non-beta version still has this issue. Could you publish a post describing how you solved it?

Have the same issue on ipad pro 11". tried both chrome and safari

Hey all, it seems that this has been outdated for a long time ago, and this seems to only affect v1 so I suggest to use v2. iPad users please upgrade your devices to iPadOS since it has been reported to have better compatibility with apps like code-server.


feat: start implementing OpTransaction

Closes #12473

Gm @emhane, a few points on which I'm unsure of the path to head on to, sorry in advance:

Compact implementation:

from_compact: not really if I'm missing something, I can't seem to find an implementation of that for any related type

TransactionExt: Deposit variant, not really sure if tx_hash() is the appropriate function to call here
I hope this isn't too much of a hassle, thanks in advance!

need a hand here @edisontim ?

I'm getting build error for the revm-interpreter since the merge
error[E0004]: non-exhaustive patterns: `HaltReason::FailedDeposit` not covered
   --> /Users/timothyedison/.cargo/registry/src/index.crates.io-6f17d22bba15001f/revm-interpreter-14.0.0/src/instruction_result.rs:110:15
    |
110 |         match value {
    |               ^^^^^ pattern `HaltReason::FailedDeposit` not covered
    |
note: `HaltReason` defined here
   --> /Users/timothyedison/.cargo/registry/src/index.crates.io-6f17d22bba15001f/revm-primitives-14.0.0/src/result.rs:441:1
    |
441 | pub enum HaltReason {
    | ^^^^^^^^^^^^^^^^^^^
...
478 |     FailedDeposit,
    |     ------------- not covered
    = note: the matched value is of type `HaltReason`
help: ensure that all possible cases are being handled by adding a match arm with a wildcard pattern or an explicit pattern as shown
    |
139 ~             HaltReason::InvalidEXTCALLTarget => Self::InvalidEXTCALLTarget,
140 ~             HaltReason::FailedDeposit => todo!(),
    |

error: could not compile `revm-interpreter` (lib) due to 1 previous error

hm maybe remove the lock file and compile from scratch again? if you push your code I can have a look


add my profile pratish shrivastava

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

@Panquesito7  When will my PR will be merged


Re: Migrate from os.system to subprocess.Popen

Closes #33: Migration to "subprocess" for executing commands
I closed #38 due to unavoidable conflicts. It can't be opened again due to forced pushes to the fork. Hence, I PR'd once again.
Improvements:

Uses subprocess.Popen instead of os.system for compilation and execution statements.
In each of the child processes created, the flow of stdin and stdout are controlled i. e. Both are set in subprocess.PIPE and manually controlled using obj.stdin and obj.communicate().
Converted all file paths into OS-agnostic paths using os.path.join. The use of '/' as a separator might cause unexpected issues in several systems; os.path takes care of that.

The problem mentioned in #31 caused by whitespaces in the directory/file names is no longer a problem after the use of subprocess. The arguments are passed as an array to subprocess.Popen, which takes care of escaping the characters in an OS-agnostic way to avoid any possible error:
...
obj = subprocess.Popen(['g++', full_filename, '-o', full_output_filename],
                stdin=subprocess.PIPE, stdout=subprocess.PIPE)
...

The changes have been tested and Codemon successfully works in both GNU/UNIX-like (MSYS2, bash, WSL2, xterm) Windows (cmd.exe, PowerShell) environments.

Got it! Let me do the changes ASAP. I'll let you know once done.

@ankingcodes Done with the changes! Please review.

@ankingcodes Done! Please review.

@paramsiddharth - LGTM ! Thanks for contributing.


Add support for global_state in dryruns

These changes add a global_state field to DryRunTransactionPrams.
Example code showing a possible usage:
from graviton.blackbox import DryRunExecutor, ExecutionMode, DryRunTransactionParams
from tests.clients import get_algod
teal = """
#pragma version 8
byte "value"
app_global_get
btoi
return
"""
algod = get_algod()
executor = DryRunExecutor(algod, ExecutionMode.Application, teal)
txn_params = DryRunTransactionParams.for_app(global_state={"value": b"\xff"})
run = executor.run_one((), txn_params=txn_params)
print(run.tabulate(500))
print("Cost:", run.cost())

Output:
   step |   PC# |   L# | Teal                              | Scratch   | Stack
--------+-------+------+-----------------------------------+-----------+----------------
      1 |     1 |    1 | #pragma version 8                 |           | []
      2 |     8 |    2 | pushbytes 0x76616c7565 // "value" |           | [0x76616c7565]
      3 |     9 |    3 | app_global_get                    |           | [0xff]
      4 |    10 |    4 | btoi                              |           | [255]
      5 |    11 |    5 | return                            |           | [255]
Cost: 4

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Office 2019 on Server 2019

Hi
According to this article you can't use Active Directory-based activation for Office 2019 (with GVLK) that's installed on Server 2019?
Kind regards
Frederik.

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

ID: 88987141-eb61-186f-ada4-6b8765764737
Version Independent ID: 9264a57c-554a-c69e-b408-4c3196788ec5
Content: Activate volume licensed versions of Office by using Active Directory - Deploy Office
Content Source: DeployOffice/vlactivation/activate-office-by-using-active-directory.md
Product: office-perpetual-itpro
GitHub Login: @DHB-MSFT
Microsoft Alias: danbrown

@FredVH That might just be an oversight on our part. We'll check with the product team responsible for activation to see if Windows Server 2019 is supported.

@FredVH - According to the product team, no changes were made to Active Directory activation for Windows Server 2019. Based on that info, it should work. So we've added Windows Server 2019 to the list in the requirements section in this article. Thanks for bringing this to our attention.


US91774 Improve performance measures

Improves performance measuring with the addition of lazy loading.
TODO:

[x] Update performance-timings.md
[x] Handle case where widget isn't initially visible
[x] Better way of detecting last visible tile?

Not ready to merge yet, but wanted to get up a graph similar to the one posted in #428, but with improved marks/measures:

(Both this graph and the one in the other PR were created using local measurements, over an average of 40 page loads.)
The good news is that this does indeed improve load time, by an average of about 100ms, over not lazy loading! For the graph above, the No lazy load and Lazy (all tiles) data was collected using the d2l.my-courses measure; Lazy (visible tiles) used the d2l.my-courses.visible-images introduced in this PR.

Finished implementing the TODO items I had from Friday, so this is ostensibly ready for review. Things of note:

using the new d2l-course-image which lazy-loads images only once they're in the viewport
pretty significantly changed how we're tracking important points (see performance-timing.md for more)
the method to determine when we've finished "visible" stuff makes two assumptions

we know how many tiles are visible before the first /organizations request finishes
we get responses in order (not a great assumption to make, but I think should be okay here?)


Repo-digest fixes

Use local timezone for date logging output
Output value of --since as RFC 3339 and Unix Date "prettysince"
Properly parse --since at command run time, not during init()
Add gendoc command to generate markdown documentation

When adding new documentation, run:
./repo-digest gendoc > README.md

LGTM


Isolate is very slow when running a function that returns a very long list

In my Flutter project for Windows, I use the compute function to run some code in an Isolate, in order to avoid freezing the user interface.
But I'm facing a problem: the code I run in the Isolate returns a very long list of double (a few millions of items), and because of that, the overall execution takes a ridiculously big amount of time, compare to if I don't use Isolate at all.
Here is the function that makes many calculations and returns a big list:
static Future<List<double>> extractWavSamples(File fileWav) async
{
    // read the audio file:
    Uint8List bytes = await fileWAV.readAsBytes();
    bytes = bytes.sublist(44);

    // convert to a list of doubles:
    List<double> samples = [];
    Uint8List samplesBytes = Uint8List(2);
    int sample = 0;
    for (int a = 0 ; a < bytes.length - 2 ; a += 2)
    {
      samplesBytes[0] = bytes.sublist(a + 1, a + 2)[0];
      samplesBytes[1] = bytes.sublist(a, a + 1)[0];
      sample = ByteData.view(samplesBytes.buffer).getInt16(0);
      samples.add(sample / 32768.0);
    }
    return samples;
}

Here is how I run that function without compute (it takes 5-6 seconds to run, but it freezes the user interface):
List<double> samples = await extractWavSamples(myFileWav);

Here is how I run that function with compute (it takes about 20 seconds to run):
List<double> samples = await compute(extractWavSamples, myFileWav);

And I found out that, because the extractWavSamples() function returns a big list, that's why the code takes a way too long time to run. If I make it return an empty list, it's much faster.
Tha thing is: I need that list in the main thread, so I can display it (it allows me to display the waveform of the audio file).
Did I do something wrong, or could it be improved?
Thanks.
Logs

Logs
$ flutter doctor -v
[√] Flutter (Channel master, 2.6.0-12.0.pre.22, on Microsoft Windows [version 10.0.19043.1237], locale fr-FR)
    • Flutter version 2.6.0-12.0.pre.22 at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 83dfb2237a (9 days ago), 2021-09-17 23:48:04 -0400
    • Engine revision 31792e0340
    • Dart version 2.15.0 (build 2.15.0-120.0.dev)

[!] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\AndroidSDK
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.0)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.31605.320
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 4.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[√] IntelliJ IDEA Community Edition (version 2021.2)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2021.2
    • Flutter plugin version 59.0.4
    • Dart plugin version 212.4746.57

[√] Connected device (4 available)
    • SM M515F (mobile) • RF8NC0PSMMV • android-arm64  • Android 11 (API 30)
    • Windows (desktop) • windows     • windows-x64    • Microsoft Windows [version 10.0.19043.1237]
    • Chrome (web)      • chrome      • web-javascript • Google Chrome 94.0.4606.61
    • Edge (web)        • edge        • web-javascript • Microsoft Edge 93.0.961.52

! Doctor found issues in 1 category.

Hi @mregnauld
Is the issue only on Windows? Did you try Android or web?
Also to better address the issue, Can you please provide a minimal complete reproducible code sample including a sample file (fileWav) and flutter run -v
Would be even better if you cad ad timestamps in the process function, perhaps print(DateTime.now()) in the beginning and at the end.
Thank you


Accept points as well as polygons as input

In some cases, you may have OD data between a zone and a point. E.g. travel to school starts somewhere in the zones but all trips finish at a defined point.
Currently inputting a point into the zones_d argument returns
Error in st_cast.POINT(X[[i]], ...) :
  cannot create MULTIPOLYGON from POINT

Semi-realted If you use subpoints for origins then you must use subpoints for destinations. In my case, there is no sub-point just a single point.

Emma's hit this problem as well. In the short term, the workaround is to make a zone for each school point and make the polygon be a small circle around the point. odjitter itself could be more flexible here though.


Payment service

Add simple web service for simulating the payment step. When a user clicks pay, the backend sends an authenticated call to set the paid flag of the basket.

I think this is out of scope for this prototype and it won't add much value to the evaluation of the protoype


Is there a way to repeat a test in case it is unstable until it fails?

I have a test that fails sometimes. --retries for flaky tests is designed to try until it succeeds and then the outcome is a success. In my case, I'd like to retry until it fails. Is there a way to do that?

It would be also useful for checking tests stability.
Consider scenario where you have many tests with tight timing and would like to know whether they are stable. Ofc it can be achieved with shell loop that runs test for defined amount of tries but having that available in tests runner has more possibilities like printing summary of which tests failed and how many times.


bugfix/CAST-31316-csm-1.2 - cray-dhcp-kea increase timeout on readiness check to from default value.

Summary and Scope
Summarize what has changed. Explain why this PR is necessary. What is impacted? Is this a new feature, critical bug fix, etc?

cray-dhcp-kea increase timeout on readiness check to from default value.

Is this change backwards incompatible, backwards compatible, or a backwards compatible bugfix?
yes
Issues and Related PRs
List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.
CAST-31316
Testing
List the environments in which these changes were tested.
Tested on:
NERSC Perlmutter
mug
Test description:
How were the changes tested and success verified? If schema changes were part of this change, how were those handled in your upgrade/downgrade testing?

Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)?n
Were continuous integration tests run? If not, why?n
Was upgrade tested? If not, why?y
Was downgrade tested? If not, why?y
Were new tests (or test issues/Jiras) created for this change?n

Risks and Mitigations
Are there known issues with these changes? Any other special considerations?

none

Pull Request Checklist

[x] Version number(s) incremented, if applicable
[ ] Copyrights updated
[x] License file intact
[x] Target branch correct
[ ] CHANGELOG.md updated
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

closing given we're pruning release/1.2


feat: Adding custom rule tool

Cloud One Service
[X] Conformity
Proposed Changes

Adding custom rule tool

Hey @JustinDPerkins can you provide some feedback here?
You have a lot more experience in 🐍  than me 🙃


[0.8.x] Moving to Typescript

Target version 0.8.x
📣 Summary
This PR introduces a reworked code based which embeds Typescript in every file and component. Vue components have been rewritten to use <script setup lang="ts"> and Pinia store was improved with better static analysis and types.
This PR follows the spirit of building a package that should provide a strongly typed codebase to ensure its resilience to user induced issues due to potential poor type checking. Typescript will give the development team a more robust source code to work with. This should also facilitate new feature development and bug fixes.
🚩 Impact and breaking changes

TL;DR: No major breaking changes as this only affects Vue components, which are only supposed to be used internally

As this PR only modifies the underlying code base to Typescript, no changes should affect the end users of the package. If you are a developer working on the package, please note that no fixes or new features will be accepted if they're not built with Typescript.
We will be thoroughly testing the package before the upcoming release.
And as always, the package is still in its 0.x, and no guarantees are to be made as the core functionality of the package. However, we always aim to avoid BCs, yet sometimes changes are necessary to move forward.

ready to merge


Export builtin pytest fixtures

I thought this should work but I encounter ERROR test_builtin_fixture_aliases.py - KeyError: 'reexport_capfd' when running the added test...
Fixes #1

The problem is because the fixtures in that module are not loaded by the fixture manager at the time the pytest_pycollect_makeitem method is run for our AnnotatedFixturePlugin.
This was also a problem for fixtures using the @pytest_annotated.fixture that I worked around using importlib in AnnotatedFixturePlugin._resolve_from_type to import the module and pass it to the fixture manager to parse.
Here is a similar work around for this problem that works.
diff --git a/src/pytest_annotated/__init__.py b/src/pytest_annotated/__init__.py
index 234ab3b..e5d5225 100644
--- a/src/pytest_annotated/__init__.py
+++ b/src/pytest_annotated/__init__.py
@@ -100,6 +100,11 @@ class AnnotatedFixturePlugin:
         if self.fm is None:
             self.fm = self.pm.get_plugin("funcmanage")
             assert self.fm is not None
+            builtins_mod = importlib.import_module("pytest_annotated.builtins")
+            self.fm.parsefactories(
+                builtins_mod,
+                None,
+            )

I expect there is probably a better way to deal with these issues, but I haven't had the time to figure it out yet, so would be fine accepting this as a solution for now.


Method searchpos()->charcol() the position of the combined character is not returned

Steps to reproduce

Run gVim -clean
Paste on first line string абажу́р
To check:
Set the carriage to the symbol ж and enter the command :echo col('.'). The result will be the value 7.
Now enter the command echo charcol('.'). The result will be the value 4.
Now execute the same commands for the characters у́ and р:
col('.') for у́ — 9
charcol('.') for у́ — 5
col('.') for р — 13
charcol('.') for р — 6
Well
Now run the following command for each of the above characters (for the purity of the experiment, we will return the carriage to the beginning of the line with the Shift+^ command): :echo searchpos(<char>, 'cW', line('.'))->charcol()
The result
:echo searchpos("ж", 'cW', line('.'))->charcol() — 7 (the carriage is placed on this symbol, as expected)
:echo searchpos("у́", 'cW', line('.'))->charcol() — 0 (but the carriage is placed on this symbol, as expected)
:echo searchpos("р", 'cW', line('.'))->charcol() — 0 (but the carriage is placed on this symbol, as expected)

Hmm, not only is the position of the symbol for the combined symbol and the symbol after it not returned, but also the position of the symbol before the combined symbol is not returned correctly.
5. Okay. Let's try to execute the following command for these characters: :echo searchpos(<char>, 'cW', line('.'))->col()
The result
:echo searchpos("ж", 'cW', line('.'))->col() — 7 (the carriage is placed on this symbol, as expected)
:echo searchpos("у́", 'cW', line('.'))->col() — 9 (the carriage is placed on this symbol, as expected)
:echo searchpos("р", 'cW', line('.'))->col() — 13 (the carriage is placed on this symbol, as expected)
It is possible that this somehow intersects with this issue #12579.
Or I don't know how to cook such methods and didn't understand something in the documentation.
For English speakers:
абажу́р — lampshade
ж — <ж> 1078, Hex 0436, Oct 2066, Digr z%
у́ — <у> 1091, Hex 0443, Oct 2103, Digr u= < ́> 769, Hex 0301, Octal 1401
р — <р> 1088, Hex 0440, Oct 2100, Digr r=
Expected behaviour
The correct value of the character index of the column position is returned
Version of Vim
Vim 9.0.1677
Environment
Windows NT 6.1
Logs and stack traces
No response

opps, wrongly closed. Sorry.

It's okay, the main thing is to solve it somehow.
If, of course, it is solvable at all.

@chrisbra  @RestorerZ  Correct me if I'm wrong, but I think everything is working as intended here and calling searchpos()->charcol() is a mistake.
As we can see in :h searchpos():
searchpos({pattern} [, {flags} [, {stopline} [, {timeout} [, {skip}]]]])
     Same as |search()|, but returns a |List| with the line and
     column position of the match. The first element of the |List|
     is the line number and the second element is the byte index of
     the column position of the match.

So when you do searchpos()->charcol() you are passing [lnum, byte_index] to charcol(). This is why case (4) is failing. Argument of charcol() should be a list of [lnum, column_position] instead. Maybe screenpos() is a better choice for case (4)?

@julio-b
Yes, you may be right.
I'll run your script again.
Thanks!


Expose original request URL

We currently expose fullPath as a logging method, but this is constructed from the HTTPure request parts object and while it should be equivalent to the original request URL from a routing perspective, it isn't identical.  We should expose a way of getting the full original request URL from the request.

I've just dropped fullPath and added original URL to the Request.Request. If you agree with this changes I can create a PR:
https://github.com/cprussin/purescript-httpure/compare/master...paluh:fullPath

P.S.
Oh, sorry I've commit there also shell.nix changes so if you agree with this direction I'm going to cleanup this branch before PR ;-)

So, I like the idea of adding access to HTTP.requestURL, but I'm not sure I like dropping fullPath entirely... it isn't completely equivalent to HTTP.requestURL and I would find the original fullPath also valuable for logging since it indicates what HTTPure matched (although maybe that can be solved batter by instead just having a log formatter that logs the components of the Request in a clearer format).
Regarding the shell.nix changes, I'd prefer to revert those, for the following reasons:

I'd prefer to keep bower, jshint, etc managed by package.json for now--that way is more accessible to folks who aren't on nix.
It's important to me to keep the package versions, as well as the nixjs and nixpkgs paths, as arguments to the nix module--I use those to test different package versions and packaging tools (e.g. nix-shell --arg nixjs path/to/local/nixjs --arg purescript 1.12.5).
Since HTTPure has simple package dependencies (and probably should remain that way easily work with a regular package set), there's no reason to introduce spago--psc-package serves our needs just fine (anyways it looks like you're configuring nixjs to export an overlay that has spago but you're not actually using the exported spago package).

What do you think about just adding url into the Request (requestURL name sounds a bit redundant in this context :-)) type:
https://github.com/cprussin/purescript-httpure/compare/master...paluh:add-request-url

LGTM... open up a PR and I'll merge!

Cool! Done: https://github.com/cprussin/purescript-httpure/pull/155 ;-)


Adding Developer Portal//Documentation

https://howto.xinfin.org/
Would we add something like this https://howto.xinfin.org/ or a gitbook. Will be so much easier write here than write on HTML code.

Our main focus was a seamless user experience for the developer portals with expansion to allow for features beyond documentation.  A Jekyll template on GitHub pages provides the balance we were looking for.  With that said, using Gitbooks for documentation will admittedly make it easier for the community to contribute, and therefore we are going to restructure to continue to use GitHub pages for the static xdc.community, but will link to Gitbooks for documentation.
Great recommendation!
Thank you.

@xcantera I am re-opening this issue so that you can close it out once you feel it has been properly addressed.


Limit Collection

Hi I was wondering if there is some way I can limit what elements of a collection each user can see like the built in Meteor.publish function allows?
Can I pass a cursor to Restivus.addCollection()?

Right now, the only way to limit or alter what's returned is to completely override the collection endpoint:
Restivus.addCollection Meteor.users
  endpoints:
    get:
      action: ->
        # Do all your work here as you would in a regular (non-collection) endpoint
    getAll:
      action: ->
        # Do all your work here as you would in a regular (non-collection) endpoint

If you're going to go through all the work of passing me a filtered cursor, as you suggested, all you would need to do is to just call cursor.fetch() to return those documents (well, I guess Restivus could at least wrap them in the JSend-formatted response).  I think there may be a chance to abstract things away a bit more via the endpoint options. I just need to give it a little thought. Of course, I'll gladly accept any suggestions.
One thing that I intend to do, regardless, is to add hooks to all the endpoints to allow you to modify the response data immediately before it's returned. That would certainly be more convenient than having to manually override the get and getAll endpoints, but still less so than providing some filtering options on collections endpoints. I also need to add this.collection to the collection endpoint context so that it's accessible in those hooks and any override methods.
I'm in the process of adding automated testing, which is long overdue. Once I have a decent test suite setup I'll make this feature my top priority. I expect the initial test suite to be complete this weekend, so I should be able to get this feature published by next weekend. At the absolute least I will add the onResponse endpoint hooks, but I'm sure we can do something cleaner for the collections.

add hooks to all the endpoints to allow you to modify the response data immediately before it's returned

This. A transform-style hook. It would work with findOne as well as find, and maintains the Meteor/Mongo Collection convention.

You're right, that would be perfect. It's funny, I was just reading the Meteor Docs on Mongo Collections in an unrelated matter, and when I came across the transform option I thought, "Why didn't I already think to allow users to configure this on collection endpoints?" This just shot up to the top of my list since it's so easy, and it's a critical issue for collection support. Thanks for suggesting this! It's just what I needed.
I'll update this once I decide how to add it to the API, in case you want to give some feedback on that. It may just be an extra option on the get and getAll collection endpoints.

@mvryan @kahmali Hello!
Did you find the solution how to filter returned documents only for current user?
Code like
Api.addCollection('books', {
    routeOptions: {
      authRequired: true
    },
    endpoints: {
      getAll: {
        action: function() {
            return Books.find({writtenBy: this.userId});
        }
      }
    }
  });

doesn't work for me

This works for me. But I'm not quite sure if it is the correct way.
getAll: {
        action: function() {
            return { status: 'success', data: Photos.find({ userId: this.userId }).fetch() }
        }
},


joint collisions

joint collision callbacks are missing and need to be added. (probably at nb.prototype.C)

After analyzing the code, I concluded that no collision is happening with joints; only related disc positions and velocities are updated.


Debug is not precise when checking errors after running preLaunchTask

Refs: https://github.com/Microsoft/vscode/issues/61678

Use the VSCode sample ts-extension
Make an unrelated error in some .json file, for example package.json
Start debugging -> notice how debugging complains that an error was found in the preLauncTask, even though the error is completely unralted

For this to work properly we would need some corellation between errors and markers, since what I do from the debugService is ask for markers after a preLaunchTask run and if there are some markers I show the prompt. And this can be very inprecise as shown in the example above.
@alexr00 @dbaeumer @sandy081 is there any way the marker service would have more information and that it would at least know that some task is responsible for generating those markes. Every marker has an 'owner' could this be used?
fyi @weinand @dantup

This needs https://github.com/Microsoft/vscode/issues/31968 because everything else will be guessing and might be equally wrong.

I see you already moved that to october, but the october one year ago 😁

Agree. Never got enough traction. PR welcome :-)

(If this is going to get some attention, it'd be really nice if it wasn't specific to preLaunchTasks... It's strange that with/without a preLaunchTask behaves differently given that some project types don't have them but still generate diagnostics in the background https://github.com/Microsoft/vscode/issues/51341)


Compare auto gradient values agains explicit gradients

Compared autograd values with @dsteinberg-grad values for ELBO w.r.t. hyperparameters variance, regulariser and RBF lenscale, and accurate to 1e-10.

@dsteinberg can probably be deprecated in next major release :p

:-(
On Thu, 18 Feb 2016 12:14 Louis Tiao<EMAIL_ADDRESS>wrote:

@dsteinberg https://github.com/dsteinberg can probably be deprecated in
next major release :p
—
Reply to this email directly or view it on GitHub
https://github.com/NICTA/revrand/issues/47#issuecomment-185535204.


v-intro-disable-interaction doesn't work

As title, the directive doesn't work wether you set the value to true or false, before or after the intro.

fixed


(maint) Remove Bolt::Inventory::Inventory#validate now that it's un…

…used
As part of #2993, Bolt now resolves plugins and validates the resolved
inventory when the inventory is needed, instead of when instantiating
the Inventory class required to bring up the Bolt application. As part
of this, inventory groups will validate themselves when loaded, which
makes Bolt::Inventory::Inventory#validate not useful, and should mean
it's no longer called. This removes the function and it's tests.
!no-release-note

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fixed the usage of transitionend in motion util

The ES6 + Webpack usage of foundation.util.motion.js was broken because of a missing import. The previous version will only work if Foundation is exposed not if it's encapsulated.

Thanks @swey! And congrats on first merged PR

@kball Thank your work and the fast merging!


feat: new ems_json_walk twig filter

Q
A

Bug fix?
n

New feature?
y

BC breaks?
n

Deprecations?
n

Fixed tickets
n

new ems_json_walk twig filter

I have doubts:

no sure about the name. ems_tree_walk, ems_array_walk, ems_json_menu_walk, ems_tree_recursive_walk
maybe add the parent in the yield?
does the yield should return an object? EMS\CommonBundle\Json\Node ?

the JsonMenuNested is already smart. I think we should just make it smarter and immutable.
https://github.com/ems-project/EMSCommonBundle/blob/master/src/Json/JsonMenuNested.php


Drop io support

The build is failing on io right now

This may require some complex travis matrices, since we will need to use the most recent version of jsdom to support io. It may not be worth the effort.

Updated, we should just remove io from the travis file 'cuz nobody uses that thing anyway.

Completed in #34