Change port for the dns

Hey guys, I'm trying to migrate from rest to nessie and my nessie server is not approachable by port 80 but it is approachable with port 443, how can I change the migrator to address the nessie server through port 443?
The error I get:
Unable to register the table Forbidden: Connecting to nessie.dns:80 connecting to XXX.XX.XX.XX:80 dial tcp XXX.XX.XX.XX:80: connextex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
**My connected host is only approachable at port 443 hence the error, can someone please help me fix the problem so I also can use the migrator with my host?
Thank you so much

Does using httpS://... in the Nessie Catalog URL work?

I'm getting this error when I'm using the nessie catalog url with https.
My Nessie server stands behind an openshift server and the migration is trying to connect to the openshift server with port 80
the IP in the error is the IP of openshift

@tamarinkeisari can you please move this to our Zulip chat?
GH issues are for rather for bug reports, we keep questions/discussions in the chat.


v0.8.0-beta Exception in Application start method

After installing dfc_windows_0_8_0-beta.exe I get error screen right away, a previous 0.7 version did work (not sure but probably v<IP_ADDRESS>-alpha)
After reading some other issues I tried :

deleting C:\ProgramData\FAForever\cache
starting as admin
deleting C:\ProgramData\FAForever
reinstalling several times
running at one screen at 1080p@100% scaling after cleaning window Size/Location cache with ccleaner

But problem remained. Running on Windows 10 Pro, version 1709
My laptop with Windows 10 Home, also 1709, runs this client fine.
It might have to do with my screen settings, I have 1 4k@150% screen and 2 1080p@100% screens.
I noticed before the java client gets confused when having screens at different scaling, but step 5 didn't fix this either.
error.log:
Exception in Application start method java.lang.RuntimeException: Exception in Application start method 	at com.sun.javafx.application.LauncherImpl.launchApplication1(LauncherImpl.java:917) 	at com.sun.javafx.application.LauncherImpl.lambda$launchApplication$154(LauncherImpl.java:182) 	at java.lang.Thread.run(Unknown Source) Caused by: java.lang.IndexOutOfBoundsException: Index: 0, Size: 0 	at java.util.ArrayList.rangeCheck(Unknown Source) 	at java.util.ArrayList.get(Unknown Source) 	at com.sun.javafx.collections.ObservableListWrapper.get(ObservableListWrapper.java:89) 	at com.faforever.client.fx.WindowController.getVisualBounds(WindowController.java:68) 	at com.faforever.client.fx.WindowController.maximize(WindowController.java:115) 	at com.faforever.client.fx.WindowController.maximize(WindowController.java:72) 	at com.faforever.client.main.MainController.display(MainController.java:337) 	at com.faforever.client.FafClientApplication.showMainWindow(FafClientApplication.java:97) 	at com.faforever.client.FafClientApplication.start(FafClientApplication.java:83) 	at com.sun.javafx.application.LauncherImpl.lambda$launchApplication1$161(LauncherImpl.java:863) 	at com.sun.javafx.application.PlatformImpl.lambda$runAndWait$174(PlatformImpl.java:326) 	at com.sun.javafx.application.PlatformImpl.lambda$null$172(PlatformImpl.java:295) 	at java.security.AccessController.doPrivileged(Native Method) 	at com.sun.javafx.application.PlatformImpl.lambda$runLater$173(PlatformImpl.java:294) 	at com.sun.glass.ui.InvokeLaterDispatcher$Future.run(InvokeLaterDispatcher.java:95) 	at com.sun.glass.ui.win.WinApplication._runLoop(Native Method) 	at com.sun.glass.ui.win.WinApplication.lambda$null$147(WinApplication.java:177) 	... 1 more

Seems like JavaFx can't find a screen for the current center of the application to maximize on.
https://github.com/FAForever/downlords-faf-client/blob/bfb5e289ac8d5840a30338f1f22dcd3d47a1a033/src/main/java/com/faforever/client/fx/WindowController.java#L68
As a quickfix, you could try having a look at the properties file of the client (.faforever in home directory on linux, currently trying the find out the windows location, I'd guess it's in Documents\My Games) and unset the maximized option, and then, don't maximize it.

Why do we look for the center of the current window? Why not check for all intersecting with the current window borders and take the one with the greatest overlap? (easy to calculate) We should also definetly add a fallback to Screen.getPrimary().
@EmielWitteman If you still need it, the config file for the client it located %APPDATA%/Forged Alliance/client.prefs. Probably the center of you window is located outside of all screens at the moment of maximizing. Without more debug information I can't say a lot about why this happening.
You could help us with finding the problem by running this adapted version of the client (prints more debug information) and telling us the parts of the log that contain "WindowController".
Just replace the jar in "C:/Program Files (x86)/Downlord's FAF Client/lib/downlords-faf-client-0.8.0-beta.jar" with the one in the zip file (make a backup first).
adaptedVersion.zip

Because the center is on the screen with the greatest overlap :)

No, not if it is offscreen.

Ok for off-screen we definitely need a fallback ;) But I wonder how that can happen.

I hope we can see this in the logs of the adapted version. It should print current stage pos and all screen bounds.

Thanks for the quick response. The client.prefs file caused the problem, changing the x to positive fixes the problem:
{   "mainWindow": {     "width": 1280,     "height": 693,     "maximized": true,     "lastView": "VAULT",     "lastChildViews": {},     "x": -1279.3333740234375,     "y": 720.0   },
the adapterVersion doesn't seem to print more debug

Oh, ok, didn't realize the old position gets stored in there aswell. So if the window is created of screen, that explains the error. Nonetheless we should add a default fallback (e.g. primary screen, or add a more complicated computation) mechanism.
Thanks for testing, the adapted version should just print about 4 - 8 lines of debug information to the warning output, which should be enabled on the default log level of the application. But I don't think there is a need to investigate this further. Thanks for your help/report, gonna fix that now.


Cant override product variation title

There is an issue in the ProductVariation class on the method getTitle(). The title cannot be extended unless there is no AttributeValues but I need to update the title  after the default title is created.
Can I suggest instead of  return implode(self::config()->title_glue, $labelvalues); use  $title = implode(self::config()->title_glue, $labelvalues);
This way the code can exit the if statement and we can extend the title regardless of if one is present or not. Thoughts??

Sounds like a good change. Keen to submit a PR?

Done. Fixed by https://github.com/silvershop/silvershop-core/pull/576

Take 2. Fixed with https://github.com/silvershop/silvershop-core/pull/577


refresh to v0.21.3

Checklist before submitting

[x] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Fixes https://github.com/open-ce/horovod-feedstock/issues/21.
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

@bethune-bryant, do you think we can merge this one?


I put test explorer in debug... and now I can't put it back

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.60.2
OS Version: WINDOWS_NT x64 10.0.19043

Steps to Reproduce:

Do not make any tests
Drag the tests icon in the sidebar to the run and debug sidebar
Fiddle with these:

Changing those settings will cause in weird bugs of some dropdowns disappearing and reappearing
You also can't put the test explorer back to the sidebar

@jeanravenclaw you have various options for resetting the position of the test explorer.
it sounds like you may have hidden it with the first dropdown shown above. make sure it is checked to show it in run and debug. after, that you can return it to its own activity bar icon by right clicking the title and "Reset[ting] location".
alternatively, you can run the command Reset View Locations but this will reset all view locations.

I closed the app yesterday to close my computer.... and now I've opened it I can see the dropdown

Am I correct that this resolves the issue?


Missing line numbers in convert and finalizer

Seems to be caused by those functions being in the system image. When the same code is jitted, the line numbers are present.

Not a Gallium issue. Should be tracked in julia base if discovered again.


Extension is not available in tasks list

TFS 2015 update 2 instance.  Extension says it has been installed to the project collection (confirm screen says "Already Installed".  Uninstall/reinstall does not work per previous closed issue...

Same issue on our On-Premise TFS Server.

So I got it to appear!  It turns out the task.json file has a minimum agent version of 1.95.1.  All of my agents were version 1.95.0.  I changed the task.json file to 1.95.0 and repackaged the vsix file and the task appeared.  I'm not sure if this extension truly needs agent version 1.95.1 or what the difference is between the two, but at least it shows up in the task list now.  Hope this helps.

@eeinworks8 do you use VS Online or TFS on premise?

On premise with Update 2

@eeinworks8  Does it works correctly after changing to 1.95.0?

@sergeyzwezdin Yes it works great!


LnD: Update Alpine and ruby of base image

What
Update Alpine to 3.21 and ruby to 3.3.6
This should reduce the noise from the ruby parser about using a 3.3.6 parser with 3.3.5
It may also reduce security warnings if the go update in alpine addresses some of the issues
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
The standards in the Git Workflow document on Confluence should be followed
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.


consider aligning with github.com/containers

This is just a first quick glance at your code.

It looks like you have a reimplementation of a subset of our oci_spec crate here https://github.com/anuvu/puzzlefs/blob/master/oci/src/lib.rs
I have some oci directory code that I've been meaning to make into a crate and would be happy to consider sharing as a crate for the non-spec bits of that code

Then bigger picture, we also have https://github.com/containers/composefs - per the kernel discussion thread  we could also try to build a comparison doc and see if there's any common ideas or code we could share.  (I actually want to try to argue to do some composefs tooling in Rust)

@cgwalters do you want to setup a separate repo with your oci directory code? I think that would be a good starting point

Eventually yeah...

@cgwalters I see the oci directory code is now in its own crate, I'll look into it and see if I can use it in PuzzleFS.

Yep, though also interested in that bigger picture scoped discussion about composefs vs puzzlefs at some point

@cgwalters I'll be at LPC next week, any chance you're also coming?

With the integration of the PuzzleFS image into the OCI spec, my latest PR and my contributions to ocidir-rs, PuzzleFS now uses ocidir-rs extensively, so I'm quite happy we manged to align and work towards our common goals, @cgwalters !
Of course, the work doesn't end here, but I'm closing this issue and I'll open another issue/discussion for the bigger picture composefs vs puzzlefs conversation.


Misaligned checksum and file cards with translated strings

Describe the bug
Misaligned checksum and file cards when translated strings are more than one line long at the default window size.
To Reproduce
Steps to reproduce the behavior:

Test Collision with translations where "Checksum" or "FIle" are two lines long.

Expected behavior
That the two cards were aligned.
Screenshots

Environment (please complete the following information):

OS: Fedora Silverblue 36
Install method: GNOME Builder

Additional context
It is solved when the window size is increased to a size large enough for the string to fit on one line.

Thanks for raising this issue!
This is caused from the use of StatusPage for each tool.
Quick brainstorming: replace them with a grid with the top row being labels (title class) and bottom row the tools. This will also solve #62


Add Mjolnir

Type

[ ] Bugfix
[ ] Enhancement
[x] New feature (I guess?)
[ ] Documentation

Description of the changes
Mjolnir, and don't you dare make fun of my branch name

Nope nevermind, that has my termino stuff on it... heck


Web NFC support

implements #72

feel free to test and give some feedback :)

Any idea if this will be merged? I need to use NFC in a PWA app.

not sure you can fork my branch and try if it works for your use-case though.

Any idea if this will be merged? I need to use NFC in a PWA app.

not sure
you can fork my branch and try if it works for your use-case though.

Are you testing on web or native?
Have a look at this file, where I used the WebNFC plugin:
Landing_viewmodel.dart
Specifically look at function activateNFCScan().
You need to use FlutterNfcWeb.instance.startNFCRead() with its callback function onTagDiscovered().
you will get a List records, where you should be able to read the record[id].
Also make sure to use my fork on branch web-nfc, instead of master.

Ah, I see.
I think for this to work you have to create your own fork and edit my branch. Currently it seems like the serial number is not passed from JS-side to Dart side.
Have a look at the example of web-nfc:
https://googlechrome.github.io/samples/web-nfc/
You can see that JS ndef is able to get the serialnumber in event "reading":
ndef.addEventListener("reading", ({ message, serialNumber }) => {
      log(`> Serial Number: ${serialNumber}`);
      log(`> Records: (${message.records.length})`);
    });

So you would have to change the js file of my fork:
https://github.com/v1lling/flutter-nfc-manager/blob/web-nfc/assets/flutter_nfc.js
In function startNDEFReaderJS(), the plugin starts JS Web NFC and reads the NDEF records and translates them into Dart Object.
Specifically in line 265, I define the onreading event mentioned above. In line 267 I get the NDEF records via "event.message.records;". Looking at the Web nfc example above, this event should also have an attribute "event. serialNumber".  Theoretically you should be able to retrieve it there.
If you retrieve it succesfully, you have to add it to nativeNDEFMessage in line 275 by changing function getNativefromWebNDEFMessage() and adding the serialNumber into this object. Afterwards you should be able to retrieve it on Dart side.
After changing the flutter_nfc.js File, make sure to minify it. The plugin uses flutter_nfc_min.js instead of flutter_nfc.js for performance reasons. In the comment header at the top of the file flutter_nfc.js you can see how to do it.
Sorry, due to a setup change I'm currently not able to implement and test it myself.
Have a try and let me know if it works.

I am testing on WEB
Did as you suggested but I see that the id field within the records is blank. I assume the return records are only from the user memory pages of the tag.

I'm trying to get at the UID (serial number) which is stored in memory page 0h to first byte of 02h. User memory (where the Ndef records are kept is from 04h onwards)

On the android version of the package the callBack to the .startSession() function returned an NfcTag object which contained the UID (aka serial)
If we are successfully reading the Ndef records it feels like we are close to getting the tag UID

Ah, I see.
I think for this to work you have to create your own fork and edit my branch. Currently it seems like the serial number is not passed from JS-side to Dart side.
Have a look at the example of web-nfc:
https://googlechrome.github.io/samples/web-nfc/
You can see that JS ndef is able to get the serialnumber in event "reading":
ndef.addEventListener("reading", ({ message, serialNumber }) => {
log(> Serial Number: ${serialNumber});
log(> Records: (${message.records.length}));
});
So you would have to change the js file of my fork:
https://github.com/v1lling/flutter-nfc-manager/blob/web-nfc/assets/flutter_nfc.js
In function startNDEFReaderJS(), the plugin starts JS Web NFC and reads the NDEF records and translates them into Dart Object.
Specifically in line 265, I define the onreading event mentioned above. In line 267 I get the NDEF records via "event.message.records;". Looking at the Web nfc example above, this event should also have an attribute "event. serialNumber". Theoretically you should be able to retrieve it there.
If you retrieve it succesfully, you have to add it to nativeNDEFMessage in line 275 by changing function getNativefromWebNDEFMessage() and adding the serialNumber into this object. Afterwards you should be able to retrieve it on Dart side.
After changing the flutter_nfc.js File, make sure to minify it. The plugin uses flutter_nfc_min.js instead of flutter_nfc.js for performance reasons. In the comment header at the top of the file flutter_nfc.js you can see how to do it.
Sorry, due to a setup change I'm currently not able to implement and test it myself.
Have a try and let me know if it works.

This works perfectly. Thank you so much for the help. I have butchered the code so badly I don't want to create a pull request. But I just implemented your above suggestion and it worked.

Nice! I'm glad I could help.

Any possibility to get this merged? Any help needed with testing or fixes to the code?


Added intro blurb to jazzy generated documention

Closes #1498

https://sdk.ably.com/builds/ably/ably-cocoa/pull/1499/jazzydoc/


Regarding Tip Loss Model in OpenFAST's FVWM Calculations

Hi，
I am writing to inquire about the FVWM (Free Vortex Wake Model) in OpenFAST. Specifically, I would like to know if the tip loss model is still considered during the calculations using FVWM.
Additionally, from a theoretical perspective, when using FVWM to calculate rotor aerodynamics, should we still consider tip loss correction model?
Thank you for your attention to these questions. I look forward to your response.
Wang Suyu

Dear @WANGSUYU0809,
The FVWM of OpenFAST (that is, OLAF, within AeroDyn) intrinsically considers the effect of tip and hub loss.  The engineering tip and hub loss models implemented in the BEM theory model of AeroDyn are not used with OLAF.
Best regards,

I'm not sure why there is such a erratic behavior with the aero loads in the second half of the blade span. During the OC6 Phase III project, we looked at the loads along the blade span for the BEM and FVW approaches and observed similar trends between them:

Do you observe that erratic behavior in the force normal to the rotor plane? Does this odd behavior disappear if you make the blade rigid or reduce the time step?
I hope that helps!


Does anyone know how to get the Hadmap.xml?

the data of Hadmap is stored in the file named Hadmap.xml,but i do not know the meaning of the data in it and how to get those data.If you know about it,help me please! And I'll appreciate it if you leave your contact information.

We can provide the HDMap specification to you, which describe the meaning of data in hdmap.xml, you can leave your email here, and we will email to you as soon as possible.
You can get more infomation from here: https://github.com/ApolloAuto/apollo/issues/3005, Please feel free to let me know if you have other problems, thanks!

Closing this issue as there has been no responses in a while.

the data of Hadmap is stored in the file named Hadmap.xml,but i do not know the meaning of the data in it and how to get those data.If you know about it,help me please! And I'll appreciate it if you leave your contact information.

can you mail me the hadmap.xml? I have the spec of it<EMAIL_ADDRESS>

Hello, everyone.
Could you please email me the Apollo OpenDrive specification please ?
My email is<EMAIL_ADDRESS>I really appreciate it.

Hello, everyone.
Could you please email me the Apollo OpenDrive specification please ?
My email is<EMAIL_ADDRESS>I really appreciate it.

Hello, everyone.
Could you please email me the Apollo OpenDrive specification please ? I really appreciate it.


Is there a way that I can install it by bower?

That would be the default :P bower install --save hammerjs

thanks!!!
On Wed, Oct 28, 2015 at 2:06 PM, Chris Thoburn<EMAIL_ADDRESS>wrote:

That would be the default :P bower install --save hammerjs
—
Reply to this email directly or view it on GitHub
https://github.com/hammerjs/hammer.js/issues/862#issuecomment-151935640.


Beware if your host is in more than one group

My inventory defines hosts and groups.
Each group has a firewall_v4_group_rules dictionary defined.
I was expecting to get a merge of all the firewall_v4_group_rules from each group a host belongs to, but instead, the host only got the rules defined from one group.
Example:
[all]
myhost

[web]
myhost

[database]
myhost

Then group_vars/web.yaml and group_vars/database.yaml define each  firewall_v4_group_rules
As a result, myhost only receives rules from one of the 2 groups.
I can work around this issue by setting globally for ansible "hash_behaviour = merge" but is there a way to get this in the role (without changing ansible config) ? Maybe at the template level (I'm not an expert)?

Hi,

I was expecting to get a merge of all the firewall_v4_group_rules from each group a host belongs to, but instead, the host only got the rules defined from one group.

That's not how Ansible works, see https://docs.ansible.com/ansible/2.5/user_guide/playbooks_variables.html?highlight=variables#variable-scopes. "Child groups override parent groups, and hosts always override their groups." Also see this question on stack overflow which covers your exact same case: https://stackoverflow.com/questions/23525546/ansible-host-in-multiple-groups. Also https://github.com/ansible/ansible/issues/6538. "This is a common misconception. [...] When variables are all at the same level, there is no defined order."
The firewall_v4_default_rules, firewall_v4_group_rules and firewall_v4_host_rules could be also something like firewall_v4_level_1, firewall_v4_level_2 and firewall_v4_level_3. They are not related to groups/hosts in Ansible. They are just pain variables in a role. They have a relation to them in the sense that they are merged together by the role in a specific order. But the content of those 3 variables for a host are specific for this particular host at any point in time. The group variable is not shared among all the groups.
Hope this helps and thank you for your question and interest in the role!

A little late to the party here, but we use merge_vars to accomplish that. It works quite well.


HA Database bloat after introduction of Bermuda_Global sensors

Configuration
Not applicable (I think?)

Describe the bug
Since v0.7 and the addition of the Global Bermuda device, my HA database has increased in size +15%.  This seems to all be in the State table.
My database is MariaDB so using phpMyAdmin I checked what the highest state counts are, grouped by entity_id:
select states_meta.entity_id,count(*) from states left join states_meta on states.metadata_id = states_meta.metadata_id group by entity_id order by count(*) desc;

entity_id					count(*)
sensor.bermuda_global_visible_device_count	151901
sensor.bermuda_global_total_device_count	 89860
sensor.deck_sensor_humidity			 31100
sensor.atc_c280_humidity			 30346
sensor.deck_sensor_temperature			 28670
sensor.lywsd03mmc_7d73_humidity			 26242
sensor.atc_c280_temperature			 24805
sensor.lywsd03mmc_7d73_temperature		 24085
sensor.zm_memory_used				 22791
...

The sensor.bermuda_global_visible_device_count	 has a state change count that is an order of magnitude higher than most sensors.
I suspect a lot of the bloat is coming from the lack of state_class attributes for these sensors, which causes every state change to be recorded as a discrete value rather than being aggregated:

I think the following attribute should be added to these sensors:
state_class: measurement

I have now manually added these attributes using the customize.yaml file and will update on success of this.
Alternatively, these sensors should perhaps be disabled by default?
Diagnostics
Not applicable (I think?)

Update; My edits to customize.yaml did what I was hoping from the point of view that the states are now seen as numbers and can be aggregated and fed into the statistics engine.  Whether this causes the state changes to be stored more efficiently I am unsure (but I have some other very high-change stats that do not bloat the database like these were, so I am quietly confident...)

Hi Jeremy!
v0.7.1 introduced rate-limits on the global sensor updates, can you check how it works for you on v0.7.2?
You're right about the state_class though, I'll add that to fix the treatment of the sensor values - I don't think it directly affects the state recording, at least until it goes into long-term stats where the aggregations happen - but it will fix how they're displayed.

Ha I am always late to the party!  I'm running 0.7.2 so the rate limit (once per minute?) is already in place, so my "testing" will be guaranteed successful :)
My recorder purge is 7 days so by this coming weekend it will be a fair test to see the difference there.

About 25 hours since I purged the state data, now looking much better:
select states_meta.entity_id,count(*) from states left join states_meta on states.metadata_id = states_meta.metadata_id
 where states_meta.entity_id LIKE 'sensor.bermuda_global_%'
 group by entity_id
 order by count(*) desc;

entity_id					count(*)
sensor.bermuda_global_total_device_count	1566
sensor.bermuda_global_visible_device_count	1315
sensor.bermuda_global_active_proxy_count	 412
sensor.bermuda_global_total_proxy_count		  25

The ~152k in the OP was 7 days so approx 21k per day.
With state data being written every 1 minute (1,440 per day) and it being very likely that a change is observed at every interval, it is now trending towards 22k for the 7 days (which is still high; it'll be in the top 10 noisiest sensors but a vast improvement).

Great, thanks for the follow-up!
I think it's probably worth winding it back further - these sensors are more about gathering a wide-viewed picture of health (and there are more to come!) so perhaps every 5 minutes would be sufficient - possibly with an extra mechanism to force a refresh for any realtime diagnostic needs. Given their purpose it doesn't make a lot of sense for them to be near the top of that list! :-)


feat: add debug_executePayload RPC to capture arbitrary execution witnesses

This PR adds a new RPC endpoint:
debug_executePayload(parentBlockHash: BlockHash, payloadAttributes: PayloadAttributes, transactions: Vec<Bytes>) -> ExecutionWitness

This endpoint allows executing arbitrary transactions on top of parent block hash and returns the necessary state keys and trie preimages to verify the execution with only a state root.
To allow executing partial blocks, if a transaction fails, the storage loads and stores will still result in witnesses being included to be able to prove that the execution failed.
Return Value
{
  "keys": { ... },
  "state": { ... },
  "codes": { ... },
}

Currently, I have it returning the same data as debug_executionWitness, but I think we'll also want to include transaction results (receipts, which transactions failed/succeeded, etc).
I'm not sure what the interface should look like for the transaction results, but we could go with something like this to keep things simple:
{
  "result": {
    "state": "success",
    "receipts": [...],
    "successfulTransactions": ["tx1"],
    "failedTransactions": [],
  },
  "witness": {
    "keys": { ... },
    "state": { ... },
    "codes": { ... },
  }
}

Let me know if adding a transaction result makes sense and what
Open Questions

Should we add a flag allowFailure: boolean that enables the behavior of not failing when a transaction fails?
Should we include transaction status in the output? (See Return Value section)

Update 10/31 2:50 PT: Added payload attributes field after clarifying with Optimism team.

Waiting on some new block builder improvements before adding this RPC endpoint. This will allow us to execute system transactions before other transactions in a block.

this has landed with https://github.com/paradigmxyz/reth/pull/12622
although not properly tested, if you notice anything odd, please flag

for the additional tx info in the response we can easily change the result object
if you have a suggestion/request please open an issue for that

awesome, thank you so much for your work on this!


Updated Documentation URLs + Added invite URL to log upon launch.

@1tzemerald Got a new PR for you to merge 👍


Not able to reconnect to stream

I am streaming to our streaming server setup, The streaming works fine
When the stream is connected and I disconnect from my wifi (can also be case when publisher has slow or intermittent internet), the app receives callback onConnectionFailedRtmp and stream disconnects.
When I connect back to internet, and try to reconnect it fails to connect with following reason "Error configure stream, publish permitted failed"
there are a lot of discarded frames in Logcat. Maybe the stream buffer is not getting cleared. Is there any way to flush the stream buffer?
Help me out

Buffer is cleared when you call stopStream method:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/rtmp/src/main/java/net/ossrs/rtmp/SrsFlvMuxer.java#L322
Can you tell me server used to reproduce it?

It is a ngnix media server.
Whenever I reconnect after network has failed, error logged is "RtmpConnection: handleRxInvoke(): onStatus NetStream.Publish.BadName"
Can you help me knowing when this occurs?

Try add this to file configuration:
https://github.com/arut/nginx-rtmp-module/wiki/Directives#drop_idle_publisher

Thanks drop_idle_publisher 15s; worked for me like a charm.


tx init got 404 error in the last step.

Run the rx init command in the terminal.
after selecting the organization, I got an error like these:
[?] Select the organization to use [1-4]: 4
tx ERROR: 404 Client Error: Not Found for url: https://api.transifex.com/organizations/roscn/projects/

Hello @tolerious!
Based on your output, it looks like you are using the transifex-client library which is deprecated.
Migrating to the CLI of this repo will solve this issue.


Retry attaching to a process with ptrace::attach() if the call is interrupted, bail out otherwise

This fixes issue #39

lgtm

It feels like we should setup Github auto merge or mergify or something, especially now that we are actually running tests in CI.


[NOREF] Remove branch naming check

NOREF
Changes and Description

Remove branch naming check in favor of using GitHub rules

How to test this change

Try to create a branch in GitHub that doesn't match our expected naming patterns!

PR Author Review Checklist

[ ] Met the ticket's acceptance criteria, or will meet them in a subsequent PR.
[ ] Added or updated tests for backend resolvers or other functions as needed.
[ ] Added or updated client tests for new components, parent components, functions, or e2e tests as necessary.
[ ] Updated the Postman Collection if necessary.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.
Always make comments, even if it's minor, or if you don't understand why something was done.

THANK YOU FOR THIS!!!! ✨🌟⭐✨🌟🌠💫🌠🌟🌠✨💫🌟✨
This check almost always broke my pre-commit locally 😂😂😂


In “Blocks”, help document logo failed to change to unselected status when clicking blank area.

Repro steps:
Navigate to https://makecode.microbit.org/beta
Click “?”->Blocks to show the help document in right. “>” is selected
Click blank area.
Expected Result:
“>” on the help document is unselected after clicking blank area
Actual Result:
“>” is always selected after clicking blank area

Issue here is that the Blockly canvas is not focusable. This will be resolved when we eventually get to making Blockly fully accessible. Closing for now. You can click anywhere but the Blockly canvas, to lose focus from the Sidebar link.


Cannot use devices behind I2C mux (TCA9548A)

Our device is an nRF9160 attached to a TCA9548A I2C multiplexer with eight LSM6DSL IMUs attached, one per mux channel.  Following the tca954x test overlay, here is our overlay:
&i2c1 {
    compatible = "nordic,nrf-twim";
    status = "okay";
    sda-pin = <26>;
    scl-pin = <27>;

    mux: tca9548a@70 {
        compatible = "ti,tca9548a";
        reg = <0x70>;
        status = "okay";
        label = "i2c_mux";
        #address-cells = <0x1>;
        #size-cells = <0x0>;

        ch0: mux_i2c@0 {
            compatible = "ti,tca9546a-channel";
            label = "i2c1_mux_0";
            reg = <0x0>;
            status = "okay";
            #address-cells = <0x1>;
            #size-cells = <0x0>;

            lsm6dsl0: lsm6dsl@6a {
                compatible = "st,lsm6dsl";
                label = "LSM6DSL32_0";
                reg = <0x6a>;
            };
        };

        ch1: mux_i2c@1 {
            compatible = "ti,tca9546a-channel";
            label = "i2c1_mux_1";
            reg = <0x1>;
            status = "okay";
            #address-cells = <0x1>;
            #size-cells = <0x0>;

            lsm6dsl1: lsm6dsl@6a {
                compatible = "st,lsm6dsl";
                label = "LSM6DSL32_1";
                reg = <0x6a>;
            };
        };

        ... for six more channels/sensors
    };
};

From this issue, I also include the following in our prj.conf
CONFIG_I2C=y
CONFIG_SENSOR=y
CONFIG_LSM6DSL=y
CONFIG_I2C_TCA954X=y
CONFIG_I2C_TCA954X_ROOT_INIT_PRIO=61
CONFIG_I2C_TCA954X_CHANNEL_INIT_PRIO=62

Then in our init code, again following the tca954x test code, we have (just trying to test getting things up and running)
#if DT_NODE_HAS_STATUS(DT_NODELABEL(ch0), okay)
#define I2C_0_CTRL_NODE_ID DT_NODELABEL(ch0)
#define I2C_0_CTRL_DEV_NAME DT_LABEL(I2C_0_CTRL_NODE_ID)
#else
#error "I2C 0 controller device not found"
#endif

...

#if DT_NODE_HAS_STATUS(DT_NODELABEL(lsm6dsl0), okay)
#define SENSOR_0_DEV_NAME DT_LABEL(DT_NODELABEL(lsm6dsl0))
#else
#error "Sensor 0 device not found"
#endif

...

const struct device *sensor[8];
const struct device *ch[8];

ch[0] = device_get_binding(I2C_0_CTRL_DEV_NAME);
ch[1] = device_get_binding(I2C_1_CTRL_DEV_NAME);
ch[2] = device_get_binding(I2C_2_CTRL_DEV_NAME);
ch[3] = device_get_binding(I2C_3_CTRL_DEV_NAME);
ch[4] = device_get_binding(I2C_4_CTRL_DEV_NAME);
ch[5] = device_get_binding(I2C_5_CTRL_DEV_NAME);
ch[6] = device_get_binding(I2C_6_CTRL_DEV_NAME);
ch[7] = device_get_binding(I2C_7_CTRL_DEV_NAME);

for (int i = 0; i < 8; i++)
{
    if (ch[i] == NULL)
    {
        LOG_WRN("Could not get I2C mux channel %d", i);
    }
}

sensor[0] = device_get_binding(SENSOR_0_DEV_NAME);
sensor[1] = device_get_binding(SENSOR_1_DEV_NAME);
sensor[2] = device_get_binding(SENSOR_2_DEV_NAME);
sensor[3] = device_get_binding(SENSOR_3_DEV_NAME);
sensor[4] = device_get_binding(SENSOR_4_DEV_NAME);
sensor[5] = device_get_binding(SENSOR_5_DEV_NAME);
sensor[6] = device_get_binding(SENSOR_6_DEV_NAME);
sensor[7] = device_get_binding(SENSOR_7_DEV_NAME);

for (int i = 0; i < 8; i++)
{
    if (sensor[i] == NULL)
    {
        LOG_WRN("Could not get LSM6 device %d", i);
    }

    uint8_t buffer[1];
    i2c_reg_read_byte(ch[i], 0x6a, 0x0f, buffer);
    if (buffer[0] != 0x6a)
    {
        LOG_WRN("Bad WHOAMI from sensor %d", i);
    }
}

Here, the calls to i2c_reg_read_byte and the device_get_binding of each of the mux channels works great.  However, the calls to device_get_binding for the sensors all return NULL, and worse, on the oscilloscope we can see that they don't try to change the channel on the mux before calling to the sensors (which is why they fail).  Luckily the mux starts with the channels all disabled, because obviously if it initialized with one of them enabled, it would see an LSM6DSL at the end of it, even though it might be the wrong one, but I digress.
Is there some other secret to getting the fw to switch the mux channel appropriately for the corresponding device that we are missing?
EDIT: I meant to add that this is build v3.0.99-ncs1-1

cc @teburd

I think this is more of a device tree issue than an i2c issue if I'm reading this right cc @mbolivar

@jacobw56 Where you able to fix your issue? I'm experiencing the same problems


Question: Would you mind if I publish this additionally on nest.land for optimal persistence?

https://nest.land/
Or would you rather do it yourself?

@michael-spengler this is a great idea. I just published djwt on https://nest.land/package/djwt. I am not very familar with this service yet, so please let me know if I did something wrong. Thank you!
https://github.com/timonson/djwt/commit/eaeecdc33f52d4ecae666d9e26ccfa51c9d04a21

cool @timonson - you could add an entry in the egg.json saying:
repository: "https://github.com/timonson/djwt"
this would help people to give feedback / improvement proposals over time.

Should I use the same versions like on Github @michael-spengler  ?

This is also a really good question - I do not provide versions in the egg.json typically - rather going with
eggs publish --bump minor  as an example.
I created a related issue yesterday. A solution of this issue could perhaps automatically take the release / version which is also used for deno.land: https://github.com/denoland/deno_registry2/issues/128
Feel free to give it a thumbs up if you like it.

in case you're not already in the following links might be valuable for you:
Deno: https://discord.gg/HyeKmx
Arweave: https://discord.gg/xJhCUf


Doesn't build on riscv32i-unknown-none-elf without default features (regression 0.7.10)

When adding heapless as a dependency:
[dependencies]
heapless = { version = "=0.7.10", default-features = false, features = [] }

it does not build:
   Compiling heapless v0.7.10
error[E0432]: unresolved import `atomic_polyfill`
  --> /home/kriskras99/.cargo/registry/src/github.com-1ecc6299db9ec823/heapless-0.7.10/src/spsc.rs:90:5
   |
90 | use atomic_polyfill::{AtomicUsize, Ordering};
   |     ^^^^^^^^^^^^^^^ use of undeclared crate or module `atomic_polyfill`

For more information about this error, try `rustc --explain E0432`.
error: could not compile `heapless` due to previous error

This regression was introduced with version 0.7.10 (aka it works with version = "=0.7.9")
Also see the Rust Crypto issue.

I've bisected the change to commit 63c0370d690ba7a9121754c3ecd361db6088b694


feat: add windicss

This PR add support for Windicss.
Related to https://github.com/konstaui/konsta/issues/21
I'm currently looking to see if it can be done with Unocss as well.
I'm not certain how to test what I have done.
So i made the logic without testing.
it's a fix of this PR: https://github.com/konstaui/konsta/pull/130

https://windicss.org/posts/sunsetting.html


Some parser issues

Having self as part of a name doesn't work:
pattern foo = self-foo

Results in:
loading policies from <internal>:default.dog
Simple { span: 304..305, reason: Unexpected, expected: {Some('/'), None, Some('p')}, found: Some('-'), label: None }
Error:
   ╭─[<internal>:default.dog:9:34]
   │
 9 │ pattern certificate-bundle = self-signed || root-signed
   ·                                  ┬
   ·                                  ╰── unexpected character found -
───╯
Error: build error: [Parser(SourceLocation { name: "<internal>:default.dog" }, Simple { span: 304..305, reason: Unexpected, expected: {Some('/'), None, Some('p')}, found: Some('-'), label: None })]

Using && no longer works
loading policies from <internal>:default.dog
Simple { span: 1422..1423, reason: Unexpected, expected: {None, Some('p'), Some('/')}, found: Some('&'), label: None }
Error:
    ╭─[<internal>:default.dog:57:49]
    │
 57 │ pattern root = pem::AsCertificate(trusted-root) && x509::DER(root-certificate)
    ·                                                 ┬
    ·                                                 ╰── unexpected character found &
────╯
Error: build error: [Parser(SourceLocation { name: "<internal>:default.dog" }, Simple { span: 1422..1423, reason: Unexpected, expected: {None, Some('p'), Some('/')}, found: Some('&'), label: None })]

Well crap. Need more tests!


heap-profiler tests fail

I've got the 2.4 release from https://launchpad.net/ubuntu/+archive/primary/+files/google-perftools_2.4.orig.tar.gz (tried with a version from repo but wasn't able to run autogen.sh without errors), then run ./configure && make check.
Result:
============================================================================
Testsuite summary for gperftools 2.4
============================================================================
# TOTAL: 47
# PASS:  45
# SKIP:  0
# XFAIL: 0
# FAIL:  2
# XPASS: 0
# ERROR: 0
============================================================================
See ./test-suite.log
Please report to<EMAIL_ADDRESS>============================================================================

From another issue I've got the idea to retry with --enable-frame-pointer but this option is not known i the configure script.
test-suite.zip
config.log.txt

Please re-test with 2.5. Bug against old version is not relevant.

Rechecked with master (2016-09-25T03:56:22).
The results are the same:
FAIL: heap-profiler_unittest.sh
FAIL: heap-profiler_debug_unittest.sh

test-suite-screen-capture.log.txt
config.log.txt

Something is weird with your awk or shell. Looking at specific test that fails, I see that pprof outputs correct text. Also feeding this output to same command on my box I get correct behavior (exit 1 is executed by awk).
Do you by any chance have shell that is not bash or weird awk? What sh --version and awk --version give?

BTW I just tested gperftools-2.5.tar.gz on centos 5 docker container and it passed those tests.

I see these two tests failing for slightly different reasons.  My failure output from test-suite.log is
--- Test failed for Allocate: didn't account for 90% of executable memory
--- Program output:
[...snip...]
--- pprof output:
Using local file [...]/gperftools-2.7.90/.libs/lt-heap-profiler_unittest.
Using local file /tmp/heap-profiler_unittest.a5nhc2/test_17027.1548.heap.
Total: 36.6 MB
    36.6 100.0% 100.0%     36.6 100.0% _ZL8Allocateiii.constprop.23
     0.0   0.0% 100.0%     36.6 100.0% __libc_start_main
     0.0   0.0% 100.0%     36.6 100.0% _start
     0.0   0.0% 100.0%     36.6 100.0% main
---

So the awk script in heap-profiler_unittest.sh/VerifyMemFunction doesn't find ^Allocate$.  My gcc is 7.3.0.  When I hack heap-profiler_unittest.sh to search for "_ZL8Allocate.*" instead of just 'Allocate' the test passes, but I don't think that's really the right fix.

What sh --version and awk --version give?

I'm quite sure that was whatever shipped with SLES9. the configure script found ac_cv_prog_AWK=gawk so I hope that the testsuite also uses that by AWK... But it does neither output the version of it nor of SHELL='/bin/sh', it might be useful to do so.

This is odd. Either c++filt isn't working right on your box or isn't available or pprof isn't doing right thing.
So what OS we're talking about here? SLES9 if this is not mistake is from 2006 according to wikipedia.

So what OS we're talking about here? SLES9 if this is not mistake is from 2006 according to wikipedia.

According to GitHub, this issue is from 2016. I don't have access to that system (which may still runs).
The last question of @alk was, what the outputs of $AWK --version and $SHELL --version are - and I answered that I don't know but would suggest to present that either in config.log or testsuite.log, so if that question should ever arise it would be easy to check that.


INDOOR_HUMIDITY notification

I just implemented the INDOOR_TEMPERATURE notification in one of my modules, and I discover that there is no INDOOR_HUMIDITY notification available. Would that be something interesting to implement or maybe is already too much information on the currentweather module? Temperature and Humidity usually go hand by hand, but, again, maybe that belongs in another module..
I already have a PR ready, but I just wanted to check first if that's something we want.
Cheers,
Rick

I think a good work can be create a feature to implement these things by configuration instead of the into the code.


[Applications Cell Models] Style the titles and block margins

Current state: https://franklin.moleculardevices.com/applications/3d-cell-models/organoids
https://franklin.moleculardevices.com/applications/cell-imaging
Original: https://www.moleculardevices.com/applications/3d-cell-models/organoids

@jose-correia what is left here? Is that overlapping with #862?

@mhaack this is an issue that in the meantime was fixed by Duy i think, will close


doc: FederatedHPA docs cleanup

What type of PR is this?
/kind cleanup
/kind documentation

What this PR does / why we need it:
It's unclear in FederatedHPA docs.
Which issue(s) this PR fixes:
Fixes #none
Special notes for your reviewer:
none

/cc @XiShanYongYe-Chang

/cc @XiShanYongYe-Chang @samzong

This file appears to have no matching directory:

Already fixed

/lgtm


Failed to execute 'attachShadow' on 'Element'

This library seems really useful but I'm having some issues getting it running.
I'm getting the following error while trying to run the basic editor:
Uncaught DOMException: Failed to execute 'attachShadow' on 'Element': This element does not support attachShadow
    at minimalEditor (https://icrann.irish/float/prism-code-editor/setups/index.js:22:38)
    at basicEditor (https://icrann.irish/float/prism-code-editor/setups/index.js:39:18)
    at https://icrann.irish/float/index.js:5:16

My code is below:
<!DOCTYPE html>
<html lang="en">
    <head>
        <meta charset="UTF-8">
        <meta name="viewport" content="width=device-width, initial-scale=1.0">
        <title>Float</title>

        <script type="module" src="index.js"></script>
    </head>
    <body>

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

    </body>
</html>

import { minimalEditor, basicEditor, fullEditor, readonlyEditor } from "./prism-code-editor/setups/index.js"
// Importing Prism grammars
import "./prism-code-editor/prism/languages/markup"

const editor = basicEditor(
  "#editor",
  {
    language: "html",
    theme: "github-dark",
  },
  () => console.log("ready"),
)

Thanks for any help.

The element to add the element to has to be a container you can attach a shadowroot to. You can find a list on MDN. Using a <textarea> as a container makes so sense since textarea elements can't have anything else than plain text as children.
I'm sure you might've figured this out since you closed the issue.


Adds handlebars dependency explicitly

handlebars-loader needs handlebars to be added to the dependencies list as it won't handle the dependency itself.
Additions

Adds handlebars 4.0.2 to package.json.

Testing

gulp build && gulp test should run.

Review

@jimmynotjim
@sebworks
@KimberlyMunoz

:+1:


Fuel.upload with form data not sending form data

Fuel Upload supports optional form data parameter.i passed this parameter and multipart data to upload , but only file upload works form data not associated with request

 val formData = listOf(
                "title" to "test",
                "desc" to "test")

        FuelRequest = Fuel.upload(URL, method = Method.POST, parameters = formData)
                .dataParts { request, url -> listOf(DataPart(File(FILE_URL), "file")) }

i logged request object but it only contains.
BTW if we pass empty data part array formdata will be submitted
this is what request obj look like

"Body : --163773469cc
Content-Disposition: form-data; name="file"; filename="ydu.mp4"
Content-Type:
������ftypmp42��������is

and without file (empty datapart array)

-"Body : --1637736aeab
Content-Disposition: form-data; name="title"
Content-Type: text/plain

test

--1637736aeab
Content-Disposition: form-data; name="desc"
Content-Type: text/plain

test
--1637736aeab--
"
"Headers : (3)"
Accept-Encoding : compress;q=0.5, gzip;q=1.0
Content-Type : multipart/form-data; boundary=1637736aeab

It's because you're not passing the formData value properly. It's param not parameters :)
It should be Fuel.upload(... param= formData)

@ParkerK Actually, you're wrong. Check this

yeah @jahirfiquitiva is correct . i tried multiple time but nothing works

@jahirfiquitiva  please change
.dataParts { request, url -> listOf(DataPart(File(FILE_URL), "file")) }
to
.dataParts { request, url -> listOf(DataPart(File(FILE_URL), type = "file")) }


Fail allocation if cant do scheduler IP scheduling

Fail allocation if the external scheduler is unavailable and a
user has configured the scheduler IP feature and an allocation
request is created that has a port request that would need the feature
to operate correctly.
Addresses https://github.com/rancher/rancher/issues/10187

Tested by:

Create an enviroment with scheudler IPs configures
create a service (svcFoo) with a port mapping with no host IP defined (to trigger scheduler IP logic) with a healthcheck
forcing scheduler to go unhealthy by changing the image to something that wont pass the healtcheck
once scheduler is unhealthy, cause the service from step 2 to fail its healthcheck
observe that cattle attempts to resolve svcFoo's failing healthcheck by deploying a new container
new container fails to schedule with error message Cannot perform IP scheduling because external scheduler is unavailable
restore the scheduler to fully functional and observe that the svcFoo is successfully redeployed

LGTM


99-Editable-Fields-in-the-Details-View

Related to US #99
Main changes
Details fields are now editable
When looking at the detail view, the fields that are not primary keys are editable and can be saved. After a refresh, the updated fields will be displayed.
What it looks like on the detail page with no edits yet.

What it looks like when you edit the fields that can be edited.

Once you save what the results look like after a refresh and reselect of the table.

New column in Columns View
There is now a new column called "is_editable" which checks for each row whether the column_name is a primary key, if it is then the field will be set to false, otherwise, it's set to true.
Minor changes
-Added to the column class object in the VMD by adding an is_editable as a new value and a new update row Accessor specifically for the details view.
-Added tests to the tablelistview for the sliding panel
-Added a new SlidingPanelComponent folder with LookUpTableDetails in it.

Changed have been made, and for the ordering, it should be fixed with US #129


Can't build freeimage.

I've been having trouble building freeimage with vcpkg.
Running Windows 10 (latest patch) and pt-br locale.
-- Configuring x86-windows-rel
-- Configuring x86-windows-rel done
-- Configuring x86-windows-dbg
-- Configuring x86-windows-dbg done
-- Build x86-windows-rel
CMake Error at C:/vcpkg/scripts/cmake/vcpkg_build_cmake.cmake:129 (message):
Command failed: C:/vcpkg/downloads/cmake-3.10.2-win32-x86/bin/cmake.exe;--build;.;--config;Release;--target;install;--;-v;-j1
Working Directory: C:/vcpkg/buildtrees/freeimage/x86-windows-rel
See logs for more information:
C:\vcpkg\buildtrees\freeimage\install-x86-windows-rel-out.log
Call Stack (most recent call first):
C:/vcpkg/scripts/cmake/vcpkg_install_cmake.cmake:24 (vcpkg_build_cmake)
C:/vcpkg/ports/freeimage/portfile.cmake:51 (vcpkg_install_cmake)
C:/vcpkg/scripts/ports.cmake:72 (include)
Error: Building package freeimage:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: freeimage:x86-windows
Vcpkg version: 0.0.103-2018-02-01-47265676398778e03ee16bfeb33ef357e8658676
Here's also the log file, it seems to be a linker issue:
install-x86-windows-rel-out.log

Thanks! I believe this is fixed in master; please pull and run vcpkg upgrade, then vcpkg install freeimage (the upgrade is needed to update libwebp).


Profile page for lockup contract should probably have the information about the locked balance

Problem
Here is just a random lockup account: https://wallet.near.org/profile/7f86ec378296b21bb616cb2ee9cf0dcdd7b08ca7.lockup.near

Wallet profile page only displays the "native" balance of 35 tokens while there are 500+N delegated through the lockup contract (Explorer pulls that information directly from the staking pool by querying the selected staking pool for the lockup contract: https://explorer.mainnet.near.org/accounts/7f86ec378296b21bb616cb2ee9cf0dcdd7b08ca7.lockup.near)
The owner profile displays the locked balance just fine: https://wallet.near.org/profile/nexus.near (though it lacks the link to the lockup contract informing users about its existance :thinking:)
Expected Behavior
Display the locked balance on the lockup profile page, include it in the total balance, and inform users about the delegation

Closing as this issue is now outdated.


Build, publish and host the web5 CLI

Originally built here https://github.com/TBD54566975/web5-go/pull/63
We should automate the building and hosting of the CLI tool such that other projects could add the dependency to their hermit env and use it!

From here

I can highly recommend goreleaser for building and releasing Go binaries. It will automatically build and upload to GH releases. FTL uses it, for example.

Somewhat relevant, but this was with respect to the SDK https://github.com/TBD54566975/web5-go/issues/83


Documentation issue about radio-group

Seems that documentation is bit wrong about radio-group
Correct syntax seems to be:
                <radio-group :value.sync="chartType" type="primary">                   <radio-btn value="left" checked>Left</radio-btn>                   <radio-btn value="right" checked>Right</radio-btn>                 </radio-group>
According to code, seems that it nees buttons to be  elements, not
Is there any point for me to update doc and create pull request.
Or those pull request wont be merged anyway ?

Do you mean it should be <radio-btn> instead of <radio>?

Yes, you are right, my bad.


Don't print EOF in diagnostics

Currently, if I attempt to parser JSON input text like [2 or fals (without trailing newline), I get the following error string:
unexpected byte '�'

Or:
expected 'e' instead of byte '�'

Not sure if this considered acceptable. It wasn't for me, so I've fixed it in my fork. If interested, you can pull the fix from here: https://github.com/boris-kolpackov/pdjson/commit/5b0500c1f3868c3fb731374de257eab1aaa0829d

Thanks, Boris! I've added you as a collaborator, which I should have done long ago, and I'll leave it to your discretion. Alternatively I could archive my pdjson and point to your fork as the "official" one. For me personally, I've outgrown this library, so I'm unlikely to ever use it again, let alone spend time maintaining it.

Thanks! I am happy to continue applying fixes in your repository. I need JSON5 support so I may also outgrow this pretty soon.

Fixed in https://github.com/skeeto/pdjson/commit/78fe04b820dc8817f540bdd87fb22887e0ef3981

Thanks!


Unable to restart some GaMD simulations

Hi,
I am currently testing the GaMD protocol on "big" systems (GPCR with lipid membrane) and I have issues restarting some MD simulations that reached the time limit. It seems that for an unknown reason the system becomes unstable after only a few steps. FYI, I am running duplicates of the same system and I am able to successfully restart some of them (after trying several times). Do you have any solution to fix that issue?
Here attached the input XML file and the error that I am getting.
Thanks!
Best,
Jerome.

Hi Jerome,
That is actually not an error, and is an expected warning that arises from an older version of ParmEd, and is harmless. It is likely that your simulations are continuing to run just fine after the warning is raised. Do your simulations continue running or do they halt after this warning?

Hi Lane,
It looks like ParmEd was the culprit here, updated it to the latest version and it works. Thanks!
To answer to your question, yes the calculations were stopped right after the warning.

Hi Lane,
Sorry about that, I don't know where I had my head at that day, I just realized that I didn't put the correct error message...
So to be sure that the problem was not related to just a warning being considered as an error on our local cluster, I restarted the MD simulation within an interactive job.
Here's the output:
(mm) srun --nodes=1 --cpus-per-task=4 --mem-per-cpu=4G --time=0:30:00 --qos=30min --partition=pascal --gres=gpu:1 --pty bash
srun: job 61994332 queued and waiting for resources
srun: job 61994332 has been allocated resources
(mm) ls
system.inpcrd  system.pdb  system.prmtop  date.txt  lower-dual  lower-dual.xml  md_4294967294.err  md_4294967294.out  merge.inp  run_jobs.inp
(mm) gamdRunner -p OpenCL -r xml lower-dual.xml
Warning: importing 'simtk.openmm' is deprecated.  Import 'openmm' instead.
restarting from saved checkpoint: lower-dual/gamd_restart.checkpoint at step: 116340000
Running: 	  163660000  steps
Failure on step 500
Particle coordinate is nan
stepCount:  53.0
windowCount:  1.5799006179572742e+59
stage:  1.0
stageOneIfValueIsZeroOrNegative:  -26497191.0
stageTwoIfValueIsZeroOrNegative:<PHONE_NUMBER>756.0
stageThreeIfValueIsZeroOrNegative:<PHONE_NUMBER>2756.0
stageFourIfValueIsZeroOrNegative:<PHONE_NUMBER>02756.0
stageFiveIfValueIsZeroOrNegative:<PHONE_NUMBER>002756.0
Vmax_Dihedral:  2.383039914797042e+66
Vmax_Total:  8.832317274826064e+244
Vmin_Dihedral:  3.5687844108716943e+129
Vmin_Total:  -3.392748492226618e+106
Vavg_Dihedral:  -4.545932152020869e-26
Vavg_Total:  1.4060231413362273e+210
oldVavg_Dihedral:  2.883805351158775e-263
oldVavg_Total:  3.9366612489152784e-42
sigmaV_Dihedral:  -4.081211932408778e-304
sigmaV_Total:  6.518644230977538e+295
M2_Dihedral:  -8.557944129478806e+282
M2_Total:  -124503035.16378178
wVavg_Dihedral:  -3.557412228062907e+47
wVavg_Total:  -3.052445310908564e+39
k_Dihedral:  -3.8556344013299853e-240
k_Total:  -5.165762982841169e-208
k0prime_Dihedral:  -2.666305870615468e+74
k0prime_Total:  6.741552816581078e+211
k0doubleprime_Dihedral:  -0.00012973204321248944
k0doubleprime_Total:  -1.1148705641481027e+83
k0doubleprime_window_Dihedral:  -2.104236648750214e-120
k0doubleprime_window_Total:  -8.623580592758635e-267
boosted_energy_Dihedral:  -1.9095338710386873e-11
boosted_energy_Total:  -1.7873488232394516e-200
check_boost_Dihedral:  -3.895616360512721e+158
check_boost_Total:  9.627556564818428e-08
threshold_energy_Dihedral:  3119736.4695280353
threshold_energy_Total:  7.84039081015344e-286
thermal_energy:  1.7604647507373824e+109
collision_rate:  -1.0729571471930571e-90
vscale:  1.0
fscale:  -0.0
noisescale:  0.0
StartingPotentialEnergy_Dihedral:  nan
StartingPotentialEnergy_Total:  nan
ForceScalingFactor_Dihedral:  4.3550441268881773e+136
ForceScalingFactor_Total:  1.980489975163592e+28
BoostPotential_Dihedral:  1.8425597917388823e-49
BoostPotential_Total:  4.845944560916026e-293
k0_Dihedral:  -9.233254446365938e-282
k0_Total:  -1.2558769805332703e+249
sigma0_Total:  -1.6112857510977502e-233
sigma0_Dihedral:  6.004057998728309e-61

After that, the execution just stop. Sometimes, it takes several retries/restarts to finally overcome that error. This error only happens when I am trying to restart a MD simulation, never during.
Best,


AKS Checklist clarifications

Recommendation item:

If using Azure Disks and AZs, consider dedicating nodes per AZ and WaitForFirstConsumer

The details provided doesn't have adequate clarification on how to dedicate nodes per AZ, and how setting WaitForFirstConsumer will behave (Eg: Will the disk created be pinned in same zone as the VM, and the subsequent pods sharing the PVC scheduled in same zone?) More clarity would be helpful.

Make sure no changes are performed by operators in the node RG (aka 'infra RG')

More details on how to achieve this would be helpful. Applying locks doesn't help and it restricts cluster functionality. If any alternates are there, it would be good to document.

If not using ephemeral disks, use larger OS disks for the nodes, especially if many pods/node

More details on how would large disk help in multi pods/node scenarios would be helpful.

Hey @infosatheesh2020 thanks for the feedback! The checklist is not supposed to replace Azure Docs, just provide some hints, plus an additional link to Azure Docs where more information is available.
Taking that into account, it would be great if you could send a PR on suggested modifications to the list :-)
Thanks!

@erjosito Have submitted PR to resolve this issue. Please review.

Hey @infosatheesh2020 thanks a lot for the PR, really appreciate it! I added a few comments to #110  , could you please check? Thanks!

@erjosito Comments are missing. Did you forgot submitting it?

oops. Now they are in :-)

Including @seenu433 FYI

@erjosito Updated as per comments. Thanks!


Update Chromium versions for api.HTMLScriptElement.crossOrigin

17382

 Quality gate: NA
Details

 Quality gate: NA
Details


add confirmation step in "done" button

Would be nice if "done" button has a confirmation step before remove the task, it could be a modal or just change the button text to "remove" or something like this

modals are a big no-no for this project, since the idea is to keep motion/distraction at a minimum.
about the text change onClick => task done onSecondClick, any suggestions to what that text could be? interested to hear your ideas.
q: is the intention of this to avoid losing a task on an accidental click? if so, i might have a feature in mind that solves this issue in a different manner.

modals are a big no-no for this project, since the idea is to keep motion/distraction at a minimum.
about the text change onClick => task done onSecondClick, any suggestions to what that text could be? interested to hear your ideas.
q: is the intention of this to avoid losing a task on an accidental click? if so, i might have a feature in mind that solves this issue in a different manner.

Every action that cannot be undone needs a confirmation step to avoid frustrating accidents, it's a well know design pattern. If you could elaborate more about this feature, we could implement it

the intention is to briefly enable the possibility of "undoing" the "done" on a task in the future, as in to bring it back to the list. i will open an issue on this soon™, and we can develop this idea further over there.
closing this for now.


After installing sample data, create marketplace results in Operation Failed.

Version Info
current master with OSCM_Pull_and_start
Describe the bug
Creation of Marketplace with name: MP1 results in Operation Failed message, and this even though marketplace MP1 does not exist yet.
How to Reproduce
Steps to reproduce the behavior:

Go to the "Create Marketplace" page.
Create Marketplace with name "MP1".

Observed behavior
Operation failed message.
Expected behavior
Success message for this particular case.
In case marketplace cannot be saved in general, show Error message and not Operation failed.
Screenshots

Log file
system.log
Additional context
N/A.

This is related with #1112.


[Question] Create empty object (without map)

First of all, great library! I've been using it in a lot of projects.
Besides using it for mapping, I also want to use it the other way around: create a new instance, fill out the properties and then convert it to a JSON string to send to the server.
Required init? requires a map, but I can't seem to create an empty one, and it's not nullable.
As a workaround, I currently do the following:
let comment = Mapper<Comment>().map("{}")
comment?.body = "Hello world!"

let json = Mapper().toJSONString(comment!)

Is there an easier/cleaner way to do this?
I've tried searching for this, but I couldn't find anything.
Thanks in advance and keep up the good work! 😄

You can create a Map object as follows:
let map = Map(mappingType: .FromJSON, JSONDictionary: [:])
Otherwise, you can just add the init(){} function to your model object and initialize it the standard way.
Hope this helps.

Thanks, @tristanhimmelman! I didn't figure it was that simple as just adding init(){}.


[Downloader] Print underlying error in HTTPError

Closes #5577

Will update tests and fix spacing if this looks good

👍

@colby-swandale r?

@bundlerbot r+


GeoLib v3.0.3 is having to install with npm

When I try to install the library using command npm install geolib I get the following errors:
npm ERR! Linux 4.18.0-20-generic
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "install"
npm ERR! node v8.10.0
npm ERR! npm  v3.5.2
npm ERR! code EMISSINGARG

npm ERR! typeerror Error: Missing required argument #1
npm ERR! typeerror     at andLogAndFinish (/usr/share/npm/lib/fetch-package-metadata.js:31:3)
npm ERR! typeerror     at fetchPackageMetadata (/usr/share/npm/lib/fetch-package-metadata.js:51:22)
npm ERR! typeerror     at resolveWithNewModule (/usr/share/npm/lib/install/deps.js:456:12)
npm ERR! typeerror     at /usr/share/npm/lib/install/deps.js:457:7
npm ERR! typeerror     at /usr/share/npm/node_modules/iferr/index.js:13:50
npm ERR! typeerror     at /usr/share/npm/lib/fetch-package-metadata.js:37:12
npm ERR! typeerror     at addRequestedAndFinish (/usr/share/npm/lib/fetch-package-metadata.js:82:5)
npm ERR! typeerror     at returnAndAddMetadata (/usr/share/npm/lib/fetch-package-metadata.js:117:7)
npm ERR! typeerror     at pickVersionFromRegistryDocument (/usr/share/npm/lib/fetch-package-metadata.js:134:20)
npm ERR! typeerror     at /usr/share/npm/node_modules/iferr/index.js:13:50
npm ERR! typeerror This is an error with npm itself. Please report this error at:
npm ERR! typeerror     <http://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /home/haseb/test-proj/npm-debug.log

@ansari-haseb This is an issue with npm. Have you tried to use the latest npm version?
npm ERR! typeerror This is an error with npm itself. Please report this error at: npm ERR! typeerror     <http://github.com/npm/npm/issues>


Remove one pasqal target and rename another

Free target should be removed
Rename the other from "_" to "-", since pasqal has not updated the target name on their end, will omit the test and resume once the change is finished on pasqal end.

/azp run


FQDN issues on load jobs

Steps to reproduce the behavior (Required)
We are testing FQDN with Starrocks. We setup 1 FE 1 BE in the new cluster. Updated the /etc/hosts files in both servers as shown below:
<IP_ADDRESS>	localhost

<IP_ADDRESS>	starrocks-front-de-0.acme.internal
<IP_ADDRESS>	        starrocks-back-de-0.acme.internal

Launched cluster without problem. We can confirm that FE connects to BE. However, when we tried to load a parquet file from Azure, we are getting an error below.
2024-02-09 12:03:48,281 WARN (heartbeat-mgr-pool-3|153) [HeartbeatMgr$BackendHeartbeatHandler.call():315] backend heartbeat got exception, addr: starrocks-back-de-0.acme.internal:9050
org.apache.thrift.transport.TTransportException: Socket is closed by peer.
	at org.apache.thrift.transport.TIOStreamTransport.read(TIOStreamTransport.java:130) ~[libthrift-0.13.0.jar:0.13.0]
	at org.apache.thrift.transport.TTransport.readAll(TTransport.java:86) ~[libthrift-0.13.0.jar:0.13.0]
	at org.apache.thrift.protocol.TBinaryProtocol.readAll(TBinaryProtocol.java:455) ~[libthrift-0.13.0.jar:0.13.0]
	at org.apache.thrift.protocol.TBinaryProtocol.readI32(TBinaryProtocol.java:354) ~[libthrift-0.13.0.jar:0.13.0]
	at org.apache.thrift.protocol.TBinaryProtocol.readMessageBegin(TBinaryProtocol.java:243) ~[libthrift-0.13.0.jar:0.13.0]
	at org.apache.thrift.TServiceClient.receiveBase(TServiceClient.java:77) ~[libthrift-0.13.0.jar:0.13.0]
	at com.starrocks.thrift.HeartbeatService$Client.recv_heartbeat(HeartbeatService.java:61) ~[starrocks-fe.jar:?]
	at com.starrocks.thrift.HeartbeatService$Client.heartbeat(HeartbeatService.java:48) ~[starrocks-fe.jar:?]
	at com.starrocks.system.HeartbeatMgr$BackendHeartbeatHandler.call(HeartbeatMgr.java:273) ~[starrocks-fe.jar:?]
	at com.starrocks.system.HeartbeatMgr$BackendHeartbeatHandler.call(HeartbeatMgr.java:251) ~[starrocks-fe.jar:?]
	at java.util.concurrent.FutureTask.run(FutureTask.java:264) ~[?:?]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128) ~[?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628) ~[?:?]
	at java.lang.Thread.run(Thread.java:829) ~[?:?]
2024-02-09 12:03:48,281 WARN (heartbeat mgr|14) [HeartbeatMgr.runAfterCatalogReady():164] get bad heartbeat response: type: BACKEND, status: BAD, msg: Socket is closed by peer.
2024-02-09 12:03:52,700 WARN (loading_load_task_scheduler_priority_pool-1|9715) [QueryRuntimeProfile.waitForProfileFinished():236] failed to get profile within 5 seconds
2024-02-09 12:03:53,290 WARN (Thread-96|140) [CoordinatorMonitor$DeadBackendAndComputeNodeChecker.run():91] Cancel query [499fa339-4ddf-4e83-b313-d1244e4e037f], because some related backend is not alive
2024-02-09 12:03:53,290 WARN (Thread-96|140) [DefaultCoordinator.cancel():779] cancel execState of query, this is outside invoke
2024-02-09 12:03:53,291 WARN (loading_load_task_scheduler_priority_pool-1|9715) [LoadTask.exec():79] LOAD_JOB=11068, error_msg={Failed to execute load task}
com.starrocks.common.LoadException: Backend node not found. Check if any backend node is down.

Backend node is definitely alive, when we submit this job. And from the FE machine I can telnet to the BE on port 9050.
root@starrocks-front-de-0:~$ telnet starrocks-back-de-0.acme.internal 9050
Trying <IP_ADDRESS>...
Connected to starrocks-back-de-0.acme.internal.
Escape character is '^]'.

StarRocks version (Required)

3.2.2

This would be better as an issue discussed on foru.starrocks.io.   Github issues are for code problems, not configure issues.


Add current-directory as an option

I'm surprised that this wasn't already an option, so I added it. Useful if you've already created the directory and set up git etc.

You can create an app in the current folder with
create-react-native-app .


Add support for Long Decimal Type in Native Parquet Reader

@Yuhta, all tests are passing. Can you take another look? Thanks.

@majetideepak the linux-adapters tests are failing, can you take a look?

@Yuhta fixed. I missed a change during rebase. Thanks!


[Trivial] Typo on Franklin Github Bot success page

"Your" is misspelled on the Github Bot success page. https://www.hlx.live/tools/bot/setup

Fixed. Thanks for yor bug report :)


misc: add CI actions/dependency-review-action for security

Motivation
Scan PR for potentially malicious deps upgrades
See https://github.blog/2022-04-06-prevent-introduction-known-vulnerabilities-into-your-code/
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
ci
Related PRs
https://github.com/facebook/docusaurus/pull/6984

Cool!


[Feature Request] OPF Metadata Update Mode for EPUB files

Light Novel support has been added since v0.19.0 and stabilized in v0.20.0, but currently it's only target API to update the metadata. One of file format usually used for light novel is EPUB, which store metadata in OPF file inside the EPUB archive.
It would be nice if it's also possible to write metadata into the OPF file directly similar to how we can write metadata into ComicInfo file in CBZ files.
Something to keep in mind is that EPUB 2 and EPUB 3 have a bit different way to store certain metadata^1.

This has been discussed on discord but I'll also write in this issue.
So far the OPF mode seems working fine with some notes:

Eventhough orderBooks is set to true, currently it's not being used to set series index in OPF mode.
OPF mode option in komf-userscript has not been added.


Release 0.2.0

Merge release 0.2.0

Pull Request Test Coverage Report for Build 410

997 of 1339 (74.46%)  changed or added relevant lines in 65 files are covered.
20 unchanged lines in 6 files lost coverage.
Overall coverage increased (+2.7%) to 65.416%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

connectors/gridgo-file/src/main/java/io/gridgo/connector/file/support/limit/NoLimitStrategy.java
25
26
96.15%

connectors/gridgo-http/src/main/java/io/gridgo/connector/http/HttpConstants.java
0
1
0.0%

connectors/gridgo-jetty-http/src/main/java/io/gridgo/connector/jetty/server/DelegateServlet.java
8
9
88.89%

connectors/gridgo-jetty-http/src/main/java/io/gridgo/connector/jetty/support/HttpEntityHelper.java
21
22
95.45%

connectors/gridgo-netty4/src/main/java/io/gridgo/socket/netty4/impl/AbstractNetty4SocketServer.java
20
21
95.24%

connectors/gridgo-netty4/src/main/java/io/gridgo/socket/netty4/ws/Netty4WebsocketServer.java
5
6
83.33%

connectors/gridgo-netty4/src/main/java/io/gridgo/socket/netty4/ws/Netty4WebsocketUtils.java
0
1
0.0%

connectors/gridgo-socket-zmq/src/main/java/io/gridgo/socket/zmq/ZMQSocket.java
9
10
90.0%

connectors/gridgo-socket/src/main/java/io/gridgo/socket/impl/DefaultSocketReceiver.java
0
1
0.0%

connectors/gridgo-vertx-http/src/main/java/io/gridgo/connector/vertx/VertxHttpConnector.java
11
12
91.67%

Files with Coverage Reduction
New Missed Lines
%

core/src/main/java/io/gridgo/connector/impl/resolvers/UriConnectorResolver.java
1
83.09%

connectors/gridgo-socket/src/main/java/io/gridgo/socket/impl/DefaultSocketConsumer.java
1
77.03%

connectors/gridgo-vertx-http/src/main/java/io/gridgo/connector/vertx/VertxHttpConsumer.java
1
81.75%

connectors/gridgo-socket/src/main/java/io/gridgo/socket/impl/DefaultSocketProducer.java
1
63.93%

connectors/gridgo-netty4/src/main/java/io/gridgo/socket/netty4/impl/AbstractNetty4Socket.java
6
63.79%

connectors/gridgo-netty4/src/main/java/io/gridgo/socket/netty4/Netty4SocketOptionsUtils.java
10
18.18%

Totals

Change from base Build 255:
2.7%

Covered Lines:
3085

Relevant Lines:
4716

💛  - Coveralls


Add support for expressions

This ticket is here to start discussions around expressions. See https://blog.mapbox.com/announcing-expressions-in-gl-js-a72b55d0a6af for a good introduction to the subject.
Questions we need to answer

How are users currently using expressions, is the JSON editor sufficient?
If not how would you like to see expressions implemented?

Currently unsupported features, due to lack of expression support

line-gradient
*-sort-key

I think it could be nice to have smaller JSON fields for authoring individual expressions, but that doesn't add a ton of value and could be a decent amount of effort. The mini-language could be interesting to try if people are interested, but in general I think the editor covers it

Thanks @pjsier I expect you're right on this one. I'll leave the ticket open for others to comment

Here's a quick UX comp for expressions I've been working on.
Expressions UI ideas:

use color to separate types of expressions into categories
use of lines and nodes to demonstrate relationships between expressions and their children
values returned by expressions float right to make them stand out
expressions are expandable and collapsable (click arrow to left of expression)
use icons on values to define their type (" in example for strings)

Looking to open discussion about this UX

So if I've got it right it's kind of a scratch like interface. I'd love to see a prototype, see how it feels to edit an expression. I think anything we develop in this area we'd need to see a prototyped out version first, outside the editor to see if it works (something like https://orangemug.github.io/simple-expr/)
Would you aim to support the entire syntax or a partial subset. Also things to consider/questions I have

"values returned by expressions float right to make them stand out" - I don't think this is possible. Expressions are based on the context of the map feature so couldn't be displayed in the UI.
The nesting could go pretty deep in an expression, how would that UI cope with that.
Why a language like UI rather than a mini language? I'm considering the popularity of spreadsheet formulas for example

@orangemug, yeah I'll build a prototype to demonstrate the functionality.
Addressing your points:

I would be aiming to support all expression functionality.
As for the values returned by the expression, I'm referring to potential values that will be returned if the given conditions are met. For example, in the comp above, the color values that might be returned by the expression are "#FFFFFF" and "#FF0000". The expression will evaluate to one or the other depending on the layer features. We can align the potential values to the right to give the designer an at a glance idea of which colors could be returned by the expression.
The design above does make good use of the horizontal space to allow for a lot of nesting, however, we do have to consider that nesting might go really deep. I've got some ideas on how the interface might handle deep trees, but will need to prototype them to see if they could work.
As for UI vs mini-language - it comes down to usability. I do believe the expression syntax out of the box is difficult to use. Therefore, there is some opportunity to create an interface which makes expressions easier to understand and write. I'm striving to do just that with this interface, but until I have a working prototype I cannot prove that it will be easier in practice.

As for the values returned by the expression, I'm referring to potential values that will be returned if the given conditions are met. For example, in the comp above, the color values that might be returned by the expression are "#FFFFFF" and "#FF0000". The expression will evaluate to one or the other depending on the layer features. We can align the potential values to the right to give the designer an at a glance idea of which colors could be returned by the expression.

Yeah but you couldn't do that for an expression like the following. Or anything with values from the feature. This is the major usecase for mgl expressions (in my opinion)
[
    "rgb",
    ["get", "score"],
    0,
    0
  ]

I'm striving to do just that with this interface, but until I have a working prototype I cannot prove that it will be easier in practice.

Can you point towards another applications where this type of approach has been successful? It might be useful to research products where this type interface exists currently.

Yeah but you couldn't do that for an expression like the following. Or anything with values from the feature. This is the major usecase for mgl expressions (in my opinion)

Good point. I didn't consider this use case in my comp. I'll take this into consideration when putting together future comps and the prototype.

Can you point towards another applications where this type of approach has been successful? It might be useful to research products where this type interface exists currently.

I actually haven't found much in other applications that I can point to for this type of functionality. It is a complex tree of relationships which will require an innovative approach to make accessible. I do have extensive experience in the private sector designing and building applications which manage high levels of complexity and I look forward to applying my skills to tackle this challenge.

@orangemug, ok I've got the first cut of an expression editor. You can see it and interact with it here. Keep in mind this is still pretty rough and is missing some critical functionality (like JSON parse error notifications).

Whats the status on this? How can i help?

Hey @sanjin-saric we basically just need to work out what we want from this feature. There are multiple ways we can do this, my suggestion is a tiny language see

https://orangemug.github.io/simple-expr/
https://orangemug.github.io/simple-expr/fmt.html

Others prefer a UI or simple code editor to enter JSON.
Basically what I'd like is some feedback from people who use the editor as to what they want. Since creating this ticket mapbox-gl expressions are are requirement for some fields. I believe line-gradient is not very useful without an interpolate expression for example. So maybe more important to get sorted than it was previously.
So to answer how you can help.

Get involved with discussion in this ticket to come up with a proposed solution of how we'd support expressions
Do research into other types of editor that have a similar requirements

Hmmm. I am also torn. On one hand I like the UI that the @justenPalmer came up with, I believe it can be really helpful. On the other, I'd like to see a raw text editor to begin with.
Although I like the notion of a more semantic approach to adding expressions, I believe it will introduce issues and I would be more work to maintain when mapbox updates its expressions.
I am going to do a UX research on filters like this and try to add my two cents on the UI provided above.

Another option:
https://github.com/mapbox/expression-jamsession/
"Write Mapbox GL expressions in a more familiar, handwritable, spreadsheet-like, programming-like syntax. This library translates these handwritten formulas into valid spec-compliant Mapbox GL expressions that you can use in a Mapbox style."

That looks very familiar to https://orangemug.github.io/simple-expr/ 😄
Given that it's on the mapbox organisation account, it'll probably be the one that sticks. Let's monitor the repo for a couple of weeks see how it progresses.

As I worked a lot with nested (nasty) expressions the last days  I would like to add some words here. I think for expressions the mini language solution @orangemug promoted and the guys from mapbox propose is the more feasable one. This aproach is also the one that's used in common GIS software like QGIS to create rule based styling. Looking at one of my expressions pasted into @justenPalmer's prototype still gives me a light headache, even though it's a remarkable improvement compared to looking at it in a text editor. :-)

None the less @justenPalmer's solution could be a great solution to visualize nested filters on the UI as the syntax is less complex here with only a few comparision operators.

This issue appears in the answers to question “What annoys you about Maputnik?” in the survey (original spelling):

No UI tool to generate expressions

Let's monitor the repo for a couple of weeks see how it progresses.

No progress so far: https://github.com/mapbox/expression-jamsession/
I'm :+1: for https://orangemug.github.io/simple-expr/

For me the biggest benefit with Maputnik is the ability to adjust colors interactively with a color palette.  The biggest downside for me with expressions has been losing this ability.  Editing expressions for other properties in code is pretty straight forward.  I vote for any solution that preserves the ability to change colors interactively.

Hi! Thanks for keeping Maputnik awesome! I've been able to use some of the new filter syntax ( "get" )in the maputnik editor, but running into issues with on a large project trying to implement an expression.
[
          ">=",
          [
            "length",
            [
              "get",
              "ref"
            ]
          ],
            5
        ]

this expression allows me to filter based on the number of characters in the "ref" field for shield labeling. This is currently getting stripped out of my style when I load into Maputnik with the error string expected, array found . Judging by this issue, im guessing it's not supported yet. Just wanted to throw my use case in the mix and hoping it will be supported soon 🤞 .

There's a new build to try out https://2095-84182601-gh.circle-artifacts.com/0/artifacts/build/index.html
Most of the bugs are fixed, still a few styling issues to fix and undo/redo is a little odd at the moment (I'll explain that later). Bugs listed from before

The expression textarea should be a codemirror editor (and probably full width)
The inline errors currently also display in the map UI
Various bugs with entering expressions

Any feedback is welcome and encouraged

Thanks for starting this @orangemug, some feedback

for filters there is no possibility to use expressions

I find the Fx button intuitive for switching to "expression mode", but would favor something else than the trash can for switching back. This could also mean you want to delete the whole value.

for filters there is no possibility to use expressions

Yeah that's probably quite important :). Let's get the rest of the UI solid and something we're happy with, then I'll add this.

I find the Fx button intuitive for switching to "expression mode", but would favor something else than the trash can for switching back. This could also mean you want to delete the whole value.

How about 2 buttons a 'revert' button and a 'trash' button. Because we can only 'revert' if the value is a plain ['literal', VALUE], that however is really useful if you've just pressed the Fx button because you can have a 'revert' button near where the Fx button just was in the UI. Once the value differs from ["literal", VALUE] we can disable the revert button in the UI. Thoughts?

How about 2 buttons a 'revert' button and a 'trash' button.

Sounds good

I've added a 'reset/undo' alongside the 'trash' button.

Demo https://2118-84182601-gh.circle-artifacts.com/0/artifacts/build/index.html#0.36/0/0
Does that UI make sense?

Added fx button into layer filter.

Updated build https://2139-84182601-gh.circle-artifacts.com/0/artifacts/build/index.html#0.36/0/0

Thanks @orangemug, currently playing around with it.
If you hover directly over the "disabled" reset/undo button, the mouser pointer changes and the disabled symbol is not visible anymore.

Hey @pathmapper, so I think I've landed on a solution that feels more 'correct'. The editor now has one 'map style' which it uses for all the editor UI (except for the map surface), this 'map style' can valid or invalid. An invalid 'map style' shows error messages in the UI as before.
The change is that the map surface, in the case of an invalid style tries to render what it can. So with the attached broken_osm_liberty.json the map will render everything but the invalid property showing an error both in the layers/layer panel and the error bar at the bottom of the map. Now the user can fix map style issues one by one and the map surface will update as these are fixed.
The undo/redo functionality has been fixed with these changes because we're now using a single authoritative 'map style' as we were previously. This also means undo/redo works on an invalid style, so if you're fixing a style with multiple issues, you can undo during the process of fixing the style.
Also, fixed the disabled button styling issue raised previously
Demo https://2152-84182601-gh.circle-artifacts.com/0/artifacts/build/index.html#0.25/0/0

I've also just added inline errors for data/zoom functions

Note, that I'm aware the CSS is a bit off for data functions, that's next on my list
Demo https://2179-84182601-gh.circle-artifacts.com/0/artifacts/build/index.html#0.25/0/0


Multiple keys remapings - How to contribute

Hi, I would like to help with the feature to allow multiple keys remappings like <super>c to <ctrl>c or <alt>Left to <ctrl>Left, those are impossible to remap under Wayland with xkb. Do you think this is possible with evscript and what is the approach for this?

evscript does not remap (unless you want to proxy all keys through it and disable the original keyboard from the compositor). If you're okay with the original key combo also being pressed, that's easy.

Oh I see. Is there a way to disable the event generated by the original keys and only emit the key events that we want? If we can export the function to disable a key event, that should be fairly easy to implement key remap in dyon script

Only if you want to proxy all events and exclude the original keyboard in the compositor.
You can't disable events on an existing keyboard device. evscript (and everything, really) has read only access to actual keyboards' /dev/input/* entries.

@myfreeweb OK, thank you. I will try a different way since this seen to be a big task


Repo sync for protected CLA branch

The pull request is created from main637909969149388806 to main to fix git push error for protected CLA branch

@corob-msft , fyi
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


docs(accordion): Upon selecting 'Accordion Panel Props', the scroll bar is moving down and automatically scrolls back up

Describe the bug
Scrollbar moves down the page and automatically scrolls back up
To Reproduce
Steps to reproduce the behavior:

Go to https://docs-preview-972--upbeat-sinoussi-f675aa.netlify.app/api/accordion/
Click on 'AccordionPanel Props link'
Scrollbar scrolls down the page and automatically scrolls back up

Expected behavior
The scroll bar should not scroll back up unless the action is performed
Screenshots
https://somup.com/c0e3qMy7a3
Desktop (please complete the following information):
All OS/Combinations
Note
When the option 'AppBar' is selected , the page is displayed but the scroll bar scrolls to the middle of the page automatically

Confirmed that this is behaving as expected now 🌟


Test on OpenJDK 12

AppVeyor support not yet available: https://github.com/appveyor/ci/issues/2894

Codecov Report

Merging #1554 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1554   +/-   ##
=========================================
  Coverage     74.72%   74.72%
  Complexity     1737     1737
=========================================
  Files           328      328
  Lines          5281     5281
  Branches        962      962
=========================================
  Hits           3946     3946
  Misses          810      810
  Partials        525      525

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 956fb89...66f9c51. Read the comment docs.

Codecov Report

Merging #1554 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1554   +/-   ##
=========================================
  Coverage     74.72%   74.72%
  Complexity     1737     1737
=========================================
  Files           328      328
  Lines          5281     5281
  Branches        962      962
=========================================
  Hits           3946     3946
  Misses          810      810
  Partials        525      525

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 956fb89...66f9c51. Read the comment docs.


Geofancy Zone support with style fixes

This adds support for the zone feature in geofancy & fixes the style & rename problems.
Sorry about second pull requests - I misunderstood some of your instructions.
You can use
Set location to the GPS coordinates of a zone
/api/geofancy?device=your_device_id&zone=zone_name
Set location to away (unset GPS)
/api/geofancy?device=your_device_id&away=1
Set location to coorrdinates
/api/geofancy?device=your_device_id&latitude=114.169&longitude=22.27&id=your_location_description

Actually I would like to propose one more change to this regarding the id= parameter.
/api/geofancy?device=your_device_id&latitude=114.169&longitude=22.27&id=your_location_description
When using the component with GPS coordinates, you are supposed to submit the id parameter, which is used for the location_name parameter of SEE.
I think "id" doesn't describe well what this parameter does and would prefer to rename it to
"location_name".
My only concern is backwards compatibility to the original version of this component, but since it has not been released to the master branch, do you think this change would be ok?

Looking at the Geofancy documentation, they do not support zones so I am a bit confused now myself about this PR

Hey, sorry for the late reply. Completely forgot about this.
You are right, I didn't even realize geofancy was sending coordinates as I have always just setup a custom URL with my own parameters, which is how I would like to continue using it with the zone parameter.
Should we still integrate it into the geofency version or should we create another component, which is almost the same code and does almost the same thing?

I am confused. You are not using geofancy right? Aren't you using this other app? The platform should be called after the other app and should support whatever that app uses to send over it's data


qt5everywheredemo segmentation fault

I'm getting the following error when running qt5everywheredemo under qemux86-64 with Xfce:
# /usr/share/qt5everywheredemo-1.0/QtDemo
libGl error: pci id for fd 8: 1234:1111, driver (null)
libGl error: No driver found
libGl error: failed to load driver: (null)
libGl error: pci id for fd 8: 1234:1111, driver (null)
QFontDatabase: Cannot find font directory /usr/lib64/fonts.
Note that Qt no longer ships fonts. Deploy some (from https://dejavu-fonts.github.io/ for example) or switch to fontconfig.
Segmentation fault

fonts should be unrelated to this issue, as I mkdir /usr/lib64/fonts it still segfaults with QFontDatabase: Cannot find font directory /usr/lib64/fonts. message gone

I found that passing -vga virtio to qemu sucessfully got rid of Seg Fault. However experiencing "libGl error failed to load driver virtio_gpu" error.
This also happens on glxgears, so not a meta-qt5 issue. Closing.


Add support warnings to DependencyDetection

In some cases, we have instrumentation that will install on library versions that are listed as unsupported in Ruby agent requirements and supported frameworks.
There aren't any known incompatibilities with these versions, but they are no longer being tested against the agent.
Emit a message to the New Relic agent logs whenever one of these versions is detected to be installed, warning about the potential incompatibility.

This is no longer the approach we want to take, closing.


Coursier not resilient to changing local ivy cache directory

For resolving local cached dependencies, Coursier assumes they're in the $user.home/.ivy directory, even though that is absolutely not guaranteed.
See the following lines in Cache.scala:

  private lazy val ivy2HomeUri = {

    // a bit touchy on Windows... - don't try to manually write down the URI with s"file://..."

    val str = new File(sys.props("user.home") + "/.ivy2/").toURI.toString

    if (str.endsWith("/"))

      str

    else

      str + "/"

  }

It's very likely that a user may change their ivy home location via the SBT options:

-Dsbt.ivy.home=D:/mocut/.ivy2/

-Divy.home=D:/mocut/.ivy2/

Thus, Coursier should try sys.props("ivy.home") or sys.props("sbt.ivy.home") before deferring to sys.props("user.home")

It shouldn't be an issue when using coursier from sbt, as the ivy2 local repository comes from sbt itself, not from coursier.Cache.ivy2Local. The right Java properties should already be handled fine thanks to that.
It can only be a problem when using coursier via its API (and using directly coursier.Cache.ivy2Local), or from the CLI of coursier.
I'd be happy to merge a PR taking into account the ivy.home property for coursier.Cache.ivy2*.

Fixed by https://github.com/coursier/coursier/pull/705.


Update RELEASING.md

Adding a step, as suggested by @pellared

Codecov Report
Patch and project coverage have no change.

Comparison is base (d69e103) 82.32% compared to head (758ee1d) 82.32%.

:exclamation: Current head 758ee1d differs from pull request most recent head 381d2bb. Consider uploading reports for the commit 381d2bb to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1951   +/-   ##
=======================================
  Coverage   82.32%   82.32%
=======================================
  Files          62       62
  Lines        2699     2699
=======================================
  Hits         2222     2222
  Misses        414      414
  Partials       63       63

Flag
Coverage Δ

Linux
82.32% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.