try to avoid binary content after serial read

See https://github.com/ros-drivers/nmea_navsat_driver/issues/45#issuecomment-420073153
This might help for GPS devices with binary content over serial.
This does not affect any current working case as if $ is the first character it will do nothing.
I put it in serial_driver in order to not affect others drivers.

Looks good to me. Have you verified it by testing with actual hardware?

Finally it does NOT fix the problem in the issue, but the PR actually works and gives a bit more trames to the parser.

I've confirmed that this works with my GPS device, but the code it changes has been moved in another PR. Can you move this change to src/libnmea_navsat_driver/nodes/nmea_serial_driver.py?

Closed due to lack of activity.


Indexing memory sharing discrepancy

Description
MXNet array indexing will only match numpy memory sharing in a few cases.
To Reproduce
Simple slice; mxnet matches numpy
a = mx.np.ones((10,))
b = a[:5]
a[:] = 0
print(a, b)  # [0. 0. 0. 0. 0. 0. 0. 0. 0. 0.] [0. 0. 0. 0. 0.]

Slice with step; mxnet diverges
a = mx.np.ones((10,))
b = a[:5:2]
a[:] = 0
print(a, b)  # [0. 0. 0. 0. 0. 0. 0. 0. 0. 0.] [1. 1. 1.]

as numpy still shares memory between the arrays:
a = np.ones((10,))
b = a[:5:2]
a[:] = 0
print(a, b)  # [0. 0. 0. 0. 0. 0. 0. 0. 0. 0.] [0. 0. 0.]

the new python api standard includes read-only views


remove unnecessary interface

the interface doesn’t do anything, and i can’t find it referenced in
any of the tests. removing the interface and references to it.

This is a so-called marker interface.

i saw that in the comments and was doing some research on them, since I hadn't heard of them previously. while it seems debated whether they are a good practice or not, I could not find actual use of the marker throughout the code. is there a spot it is actually being used?

https://github.com/sebastianbergmann/money/blob/master/src/exceptions/InvalidArgumentException.php#L23
https://github.com/sebastianbergmann/money/blob/master/src/exceptions/OverflowException.php

yah, saw those. still just don't really understand the point. from my (very) brief intro into marker interfaces, my understanding is that it allows you to check that some class implements the marker interface. so something like:
if($someClass instanceof Exception)

but I don't see that anywhere.  would it be better to simply use the built in php exception that all exceptions are extending from?
if($someClass instanceof \Exception)

if you do need to check for that?

In this case the Money's components exception classes are marked, using the marker interface, as "belonging to" or "coming from" the Money component. This can be useful for type declarations used in catch statements.

hmm.. okay. that's interesting and I can see the use there. thanks for clearing that up for me.  the only other thing I'd say about it then is I think it would be much clearer if you make the name more explicit. For example, call the interface MoneyException. this will help avoid confusion and naming collision, because I'm sure the base \Exception is used quite often.


nixos/zabbixServer: move pid file from default (/tmp) to /run/zabbix …

…to avoid issues with PrivateTmp=true

Motivation for this change
sudo -u zabbix /nix/store/c2byq8pqcka13mgzj38cnapvsl8f0byh-zabbix-server-4.2.3/sbin/zabbix_server -R log_level_increase="http poller"
zabbix_server [110889]: cannot open PID file [/tmp/zabbix_server.pid]: [2] No such file or directory

Due to usage of PrivateTmp=true in the systemd service zabbix behavior cannot be modified at runtime.
Things done

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

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

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

Notify maintainers
cc @

Thanks :tada:


add css lint task

As discussed here #556. This is the css lint task implementation. I messed up and created #561 from the master branch, so this includes all of #561's changes. That one should be merged / rejected first.
linters:

stylelint
doiuse
colorguard

LGTM, great job!
Do you think we can make csslint make sure the project is following Google's style guide?

https://github.com/mgechev/angular2-seed/pull/561 seems part of this PR. Lets close it.

@tsm91 what do you think about the Google style guide? Are there any ready-to-use configuration files? If not, we can stick to the current rules we already have.

Do you think we can make csslint make sure the project is following Google's style guide?

I think so. Would make sense to use the google css guidelines. In fact the stylelint-config-standard uses some part of Google's style guides.
Unfortunately i haven't found a stylelint rule implementation of it. I am sure it is possible but it would take a lot of work.

#561 seems part of this PR. Lets close it.

Yeah sorry, i was working in the master branch instead of creating a separate one -,-

1, The postcss-assets should be mentioned somewhere in the readme to avoid confusion and point out the URL resolution possibilities.
2, The Add-custom-Gulp-task
 wiki page should also be updated, to add sass and source maps. Since you only need to drop 3 .pipe to build.html_css.ts it should be another wiki article. And the add custom task article should be changed to add some other rudimentary task.

I'm wondering if postcss-assets will have more negative rather than a positive impact on the seed.
In general I find it very useful and I'd include it in my project. On the other hand, it brings one more level of indirection and may be confusing for the developers who are not familiar with it.
// cc @ludohenin @nareshbhatia @Bigous

Correct, it is certainly possible. In fact it is very handy to just write resolve() to font-face and background urls and it will resolve the file no matter where it is inside of /src/assets.
I think it is very easy to pick it up, also this is an opt-in feature so it is not necessary to use it. A few lines of comment right above background: resolve("more.svg"); would clear the confusion i think.

I'm not familiar with postcss stuffs so difficult to say. If it's an opt-in then that's probably fine.

Postcss takes a bit of mental remapping compared to most of the current tools. That said, it's powerful / fast and pluggable which makes it attractive. I've only tinkered with it & I have thus far been impressed.
IMO - So long as it is opt-in and well documented, I think it has a place here. It's rapidly gained in popularity and the plugin ecosystem seems to double every time I go look.

@tsm91 a rebase will be required. There are a lot of changes because of the latest refactoring which aims bigger modularity of the seed so resolving the merge conflicts will not be trivial...
If you want you can open a new PR. about postcss-assets, how about not including them in the build process but adding a sample integration page in the wiki? How does it sound?

@tsm91 @mgechev - I agree with the sample integration page idea for postcss-assets keeps things nice and pre/post processor agnostic.
@tsm91 - As a side note, having just done this with a pre-refactoring branch. It was significantly faster to create a new branch and recommit the changes than it was to try and work them into a rebase.

@tsm91 any updates?

@d3viant0ne @mgechev hey guys, sorry i am in a middle of a moving process
had to paint the walls of the flat, package up my stuff, but i will do this today and re-submit the PR
i will close this one and open a new one, the re-base process seem to be scary :D

No worries. There were some huge changes in the seed so I agree that a new PR will be an easier option. Looking forward for it! Btw, lets not include postcss-assets initially. I'll be happy if we have a wiki article for it (since I'll be one of the users of it) but the level of indirection it brings could be a bit frustrating.

@mgechev roger that, i wont include postcss-assets only as a wiki tutorial

@tsm91 sounds awesome! Looking forward for it!

css-lint disappeared?


Create code for generating uncertainty label for one simple uncertainty rule

Create code for lighting conditions. Use random or assume some threshold region for each type of material
Use fuzzy rules and piecewise logic functions to generate the uncertainty labels for specific class.
For the remaining classes use uniform distribution. Split the remaining value between the remaining classes.

completed for lighting


Update to React 16

Below is a list of third-party modules that have major version changes which may cause us problems when updating.
rc-slider

5.1.1 current installed version
6.0.0 breaking change, new UI
8.4.0 React 16 support
8.6.0 latest version

react-dates

4.0.2 current installed version
16.3.0 latest version

react-imageloader

2.1.0 current installed version
3.0.0 React 16 support
3.0.0 latest version

react-modal

1.6.5 current installed version
3.2.1 latest version

react-portal

3.2.0 current installed version
4.0.0 React 16 support
4.1.2 latest version

react-router

2.8.1 current installed version
3.2.0 React 16 support
4.2.0 latest version

react-waypoint

7.1.0 current installed version
7.2.0 React 16 support
8.0.0 latest version

react-imageloader doesn't appear to be used in any components, so it will be removed.

rc-slider was updated to the latest version. There were some breaking changes, which I was able to resolve. The new UI changes seemed to have no effect on our range component as we have some style overrides applied with Radium's Style component.
react-dates was updated to 10.2.0 because that major version includes the prop-types package. I have not tried to use anything above version 10. In version 13.0.0, they changed how the apply styles (from SCSS to react-with-styles), so that will likely ruin our style overrides. More work needs to be done to bring react-dates all the way up to the latest version.
react-imageloader has been removed, as mentioned above.
react-modal was updated to the latest version without any problems. The modal component was tested in storybook, but we will want to check it in apps when backpack-ui is updated.
react-portal was not updated beyond version 3 because version 4, despite supporting React 16, broke the Dialog component. Version 3 seems to work fine though. More work will need to be done to assess the breaking changes from version 3 to 4.
react-router was updated to 3.2.0 as that release included support for React 16 and the API from version 2 to 3 didn't really change from what I understand.
react-waypoint was updated to the latest release on version 7. I did not attempt to install version 8.


build(package): update referred packages to latest version

The following PR updates the referred external packages in the okhttp-client-adapter to make sure it is up-to-date and has the latest fixes and improvements.
The following PR also bumps the patch version for the release.
The updated packages are:

org.mockito > mockito-core
org.mockito > mockito-inline
org.jacoco > jacoco-maven-plugin

Closes #41


Distinguish lack of domain from other possible error from libvirt

Follow up for #162
We need a method to distinguish different types of errors (by subtype?) from libvirt, having access not only to string error value, but also to some enum/numerical value of it.
First place where this would be useful - ListContainers, but probably this will help also in other places in future.

Already done long time ago during change to use golang libvirt bindings.


[PSM Interop] Legacy test builds always pull the driver from master (v1.56.x backport)

Backport of #33699 to v1.56.x.
Similar to https://github.com/grpc/grpc-java/pull/8982.
All other languages already to this.
Needed for easier deployment of https://github.com/grpc/grpc/pull/33685.
ref b/274944592

[x] grpc/core/v1.56.x/branch/linux/grpc_xds_v3
[x] grpc/core/v1.56.x/branch/linux/grpc_xds_v3_python


Add support for llama-cpp GPU acceleration

Add n_gpu_layers param to llama-cpp that allows some processing on the GPU
This is in preparation for added GPU support in llama-cpp
https://github.com/abetlen/llama-cpp-python/pull/203/  for when is approved and merged
This is the llama-cpp commit that this PR adds support for
Before submitting

No New integration

Who can review?

@hwchase17
@agola11

Thanks all for the wonderful project!

Is there an easy way to test all of the functionality in one go or there is quite a lot of chained merges below?

Is there an easy way to test all of the functionality in one go or there is quite a lot of chained merges below?

not that I can think of (This is dependent on one PR though) so maybe it's not too bad to test
you need the llama-cpp-python binding to be updated, and for that
you could pip install from https://github.com/moejay/llama-cpp-python (That's my branch with this updated)
I can add some more detailed testing instructions in the PR description when I come back later this evening if that helps.
I did notice that a bunch of the examples were broken (There are probably issues for those that exist at the moment, maybe I'll be able to contribute some fixes there later on)

Looks like the same change was merged a bit later here
So, this should now work with the latest llama-cpp-bindings

duplicate of #4739


where is java 8 installed

As you said, i have to put a different java version than the bundled, but if i put the one in c:/program files (x86)/Java/jre8thing
it crashes with exit code 1. What java version should i use?

As you said, i have to put a different java version than the bundled, but if i put the one in c:/program files (x86)/Java/jre8thing it crashes with exit code 1. What java version should i use?

Make sure to set the runtime to javaw.exe from that jre folder. And make sure to check that little box to skip the java runtime version check.

As you said, i have to put a different java version than the bundled, but if i put the one in c:/program files (x86)/Java/jre8thing it crashes with exit code 1. What java version should i use?

Make sure to set the runtime to javaw.exe from that jre folder. And make sure to check that little box to skip the java runtime version check.

the thing is i dont have the java folder where it should be

As you said, i have to put a different java version than the bundled, but if i put the one in c:/program files (x86)/Java/jre8thing it crashes with exit code 1. What java version should i use?

Make sure to set the runtime to javaw.exe from that jre folder. And make sure to check that little box to skip the java runtime version check.

the thing is i dont have the java folder where it should be

You have to download java 8 then. Make sure you download JRE version not SDK.

As you said, i have to put a different java version than the bundled, but if i put the one in c:/program files (x86)/Java/jre8thing it crashes with exit code 1. What java version should i use?

Make sure to set the runtime to javaw.exe from that jre folder. And make sure to check that little box to skip the java runtime version check.

the thing is i dont have the java folder where it should be

You have to download java 8 then. Make sure you download JRE version not SDK.

ok
https://tenor.com/view/sad-cement-seal-java-programming-gif-24524699

As you said, i have to put a different java version than the bundled, but if i put the one in c:/program files (x86)/Java/jre8thing it crashes with exit code 1. What java version should i use?

Make sure to set the runtime to javaw.exe from that jre folder. And make sure to check that little box to skip the java runtime version check.

the thing is i dont have the java folder where it should be

You have to download java 8 then. Make sure you download JRE version not SDK.

ok

As you said, i have to put a different java version than the bundled, but if i put the one in c:/program files (x86)/Java/jre8thing it crashes with exit code 1. What java version should i use?

Make sure to set the runtime to javaw.exe from that jre folder. And make sure to check that little box to skip the java runtime version check.

the thing is i dont have the java folder where it should be

You have to download java 8 then. Make sure you download JRE version not SDK.

also i see you didn't add the new autoblock 😭

also i see you didn't add the new autoblock 😭
What autoblock? Also we have a policy where we don't add bypasses on the remap repo as it's just a remapping project to just make the code readable so we can later make a rebase which we will maintain and make bypasses on.

What autoblock? Also we have a policy where we don't add bypasses on the remap repo as it's just a remapping project to just make the code readable so we can later make a rebase which we will maintain and make bypasses on.

the autoblock is still not blocking and the hypixel autoblock is what i'm talking about

What autoblock? Also we have a policy where we don't add bypasses on the remap repo as it's just a remapping project to just make the code readable so we can later make a rebase which we will maintain and make bypasses on.

the autoblock is still not blocking and the hypixel autoblock is what i'm talking about

Enable oldhitting, but be aware chances are it won't bypass.


Question: Refined Storage Systems... Exploding?

Question
Randomly Random Refined Storage Systems are just disappearing when we aren't looking leaving a giant hole in the base I've seen it 4 times on our server... is this refined storage or is it something else happening?
Currently, It COULD be a griefer using a type of explosive that is placeable through FTB utilities protections but that's unlikely since 1 case was in an isolated area no one, (That I know of) had access to.
Steps to reproduce:

Unknown

Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: <IP_ADDRESS>4
Refined Storage: 1.2.22

Does this issue occur on a server? yes
If a (crash)log is relevant for this issue, link it here:

can't find the other bases currently Sorry :(
This occurs on: Skyfactory 3 Version: 3.0.7 Relase

Impossible


Destroy method doesn't remove all the styles

CodePen demo
https://codepen.io/dsasko/pen/LQpyyr
Steps to reproduce the problem

Initiate a popper instance
Popper applies styles to the dropdown/tooltip/whatever
Destroy popper instance

What is the expected behavior?
The destroy method should remove all applied styles.
What went wrong?
The destroy method removes all applied styles except will-change.
Other comments
This seemed like a silly thing to open up a PR for...
/packages/popper/src/methods/destroy.js - line 18
add: this.popper.style.willChange = '';

Please feel free to send a PR


meta: how should the community contribute?

As a very happy and enthusiastic user of github.com/rogpeppe/go-internal/testscript, I'm very happy to see a semi-official release of the fantastically useful functionality in this module!
The obvious problem raised in https://github.com/rsc/script/pull/1 was silently fixed in https://github.com/rsc/script/commit/334f6c18cff37d66fb02d957d28daf992c18fb9f without any acknowledgment or announcement, and fixing https://github.com/rsc/script/issues/3 would require only pushing a new tag.
The description of this module says:

Posting it here makes it available for others to try without us committing to officially supporting it.

Would you like contributions from the community? If so, would you like issues, PRs, etc.? Or is this a code dump of "as-is" software?

With no response here, I asked on golang-nuts: https://groups.google.com/g/golang-nuts/c/leGd1uOXpIA

I apologize for not pushing the v0.0.2 tag. I thought I had done that long ago - I created it locally, just forgot to push it, apparently.
This is mostly a code dump of "as-is" software. My focus since posting it has been on other Go work. Also, I do not want this package to get ahead of Go's cmd/go/internal/script, because then more work would have to be done to un-diverge them if we do eventually pick a standard place for it and have cmd/go import it directly.


ctrl + clicking links with use:route does not open in new target

Opening links in a new tab is an expected behavior of links. The action provided by use:route overrides this. I'd consider this a bug.
This other svelte5-router library does a great job at dealing with links and binding them. This is the only thing that library does better than this one, so I will just be taking some of the logic into my own app for now but I would highly recommend doing similar logic, esp in regards to the meta/alt/ctrl/shift key, and target checks.
https://github.com/jpcutshall/svelte5-router/blob/master/src/lib/actions.ts

Also a top level bind:links like in that other lib would be awesome in this one as well, so that every link doesn't need to use the bind directly every time.

https://github.com/mateothegreat/svelte5-router/pull/22 fixes this behavior and pulls in the code that the other library used for this logic. Also introduced a top level action


Sunset/Sunrise calculator based on location

We need to write/find a library in javascript that does this calculation based on location.
One of the example implementation is https://gist.github.com/Tafkas/4742250. See #24 for more details.
Definition of done

[ ] we have a set of unit-tests that are run through some library and they test if that library does proper sunset/sunrise calculation based on Lat/Long and maybe TimeZone (idk why it's needed).
[ ] input and output data for the tests are validated using 3rd party calculators (find reliable one)


Add Support to Act Like JS Confirm()

This should have similar functionality to confirm() in JS
A Boolean, indicating whether "OK" or "Cancel" was clicked in the dialog box:
true - the user clicked "OK"
false - the user clicked "Cancel" (or the "x" (close) button in the top right corner that is available in all major browsers, except Firefox)
Example:
if (document.getElementById("back-button").disabled ||
$confirm({title:'',text: "The 'Back' button will be disabled if a field " +
"name is changed. Press 'OK' to confirm.", ok: 'OK', cancel: 'Cancel'}) == true)
I have read your documentation and do not see an example like this.

Unfortunately that would have to be a blocking operation in order for it to function like you describe. I'm certain that would be an undesirable change for the vast majority of users.


How can I make the drawer have its own vertical scroll bar?

Hi there
The demo drawer spans items across, how can I make it have its own scroll bar. If I use x-clip, its using the document scrollbar, which is not favourable.
https://play.tailwindcss.com/XCGtFKSDYf
Thank you in advance

Remove that overflow-y-auto from menu: https://play.tailwindcss.com/Nsol0HBFTY

Thank you, is it possible for the scroll bar to be "inside" the drawer and not on the body?

Sure. Here's an example: https://play.tailwindcss.com/lfIx166zzt


Has the dataset been cleansed of complex substances?

The main focus of cryo2struct is on the protein's map2pdb task. Then, the Cryo2StructData dataset is a dataset for complexes, which contains a large amount of information on small molecules and nucleic acids. May I ask if there has been any processing done in terms of dataset usage? Thank you 🙏

Hi @ZishanShu , Thank you for your interest in our work.
We pre-processed by resampling and normalizing the dataset. The location of these codes are:
Resampling : preprocess/get_resample_map.py
Normalization: preprocess/get_normalize_map.py
The Cryo2StructData paper explains what these two resampling and normalization does. Paper: https://doi.org/10.1038/s41597-024-03299-9
Let us know if this answers your question.


Fixes #25 - added warning to documentation

Fixes #25

Maybe @sebmarkbage (or someone else that has more context on SSR+hydration) can review this PR to make sure it's sufficient.

Thanks! This warning is what I had in mind :)

It's a bit confusing wording because the missing nodes won't get the wrong attributes, it's the other nodes that might get the wrong attributes if one sibling is missing.

Thanks for weighing in @sebmarkbage. Would this be an improvement?

However, it's dangerous to have missing nodes on the server render as this might cause sibling nodes to be created with incorrect attributes.

Sounds good

Cool beans. Changed in 8acec80. Thanks again!


💡 Add support for NOT operator

For example:
WHERE NOT (State = 'IL' OR State = 'NY')

I think it can be implemented via whereNot and andWhereNot methods.
@msmakouz WDYT?


socat: <IP_ADDRESS> -> <IP_ADDRESS>

Motivation for this change
Update to latest upstream release <IP_ADDRESS>
Changelog :https://repo.or.cz/socat.git/blob/HEAD:/CHANGES
Also, follow the new pattern for meta.
Things done

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

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

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

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109124 run on x86_64-linux 1

2 packages marked as broken and skipped:

libsForQt512.discover
libsForQt514.discover

2 packages failed to build and are new build failures:

pantheon.appcenter: log https://termbin.com/mpnx
python39Packages.powerline: log was empty

23 packages built:

acme-sh
dasht
disnixos
flatpak
flatpak-builder
gnome-builder
gnome3.gnome-software
k3s
plasma5.discover (libsForQt5.discover)
plasma5.kwallet-pam (libsForQt5.kwallet-pam)
libsForQt512.kwallet-pam
libsForQt514.kwallet-pam
malcontent-ui
mpvc
pantheon.sideload
powerline (python38Packages.powerline)
python37Packages.powerline
rabbitmq-server
socat
vala-language-server
vimPlugins.vim-dasht
xdg-desktop-portal
xdg-desktop-portal-gtk

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109124 run on x86_64-linux 1

2 packages marked as broken and skipped:

libsForQt512.discover
libsForQt514.discover

2 packages failed to build and are new build failures:

pantheon.appcenter: log https://termbin.com/mpnx
python39Packages.powerline: log was empty

23 packages built:

acme-sh
dasht
disnixos
flatpak
flatpak-builder
gnome-builder
gnome3.gnome-software
k3s
plasma5.discover (libsForQt5.discover)
plasma5.kwallet-pam (libsForQt5.kwallet-pam)
libsForQt512.kwallet-pam
libsForQt514.kwallet-pam
malcontent-ui
mpvc
pantheon.sideload
powerline (python38Packages.powerline)
python37Packages.powerline
rabbitmq-server
socat
vala-language-server
vimPlugins.vim-dasht
xdg-desktop-portal
xdg-desktop-portal-gtk


Datepicker: Unable to enter date manually

As seen from demo page: https://react.lightningdesignsystem.com/components/date-pickers/
Click on the date picker component and start entering a date using the keyboard
Expected:  the characters should be displayed as typed in, date picker component should close, user should be able to continue to enter numeric and date format characters based on locale ( 0-9 / - etc).
Actual:  As soon as first numeric character is entered, the date is auto-completed to the first day of the first month of this year.  Enter "1" and it fills in "1/1/2021".  Enter "2" and it fills in "2/1/2021".  You cannot modify this date without it continuing to change it on you.
This was previously reported 2 years ago but has not been properly resolved.
https://github.com/salesforce/design-system-react/issues/2285

The previous bug removed focus from the input, but great call out on the example. Would changing state on focus blur of the input fix the issue instead of the onChange event?

That may solve it.  As long as the user can finish typing in the date before it attempts to format it should be fine.

@chuyler If you look at the examples on storybook, you should be able to see an example that uses a custom input. In the documentation they point out to the examples as well to get more familiar with the library.

@AbuHafsa Your example suffers from the same bug.  Regardless of what is used for an input, when the user types in a single character into the text box, it tries to format the date.  It needs to wait until all characters have been entered before triggering the formatter.  I tried using "onBlur" instead of "onChange" to update the state but that doesn't work either.

I have made an attempt at fixing this so our customers can enter dates manually.  Please review the fix and suggest improvements.

Hello, anyone has resolved this feature ?

@brahimkraiem see my pull request above that has not been merged.  If you can convince them to merge it, we'll have this working.

@interactivellama
I hope you are doing well.
Could you please take this issue on priority? Actually, we are facing this issue and the user is not able to edit the date manually.
Please let me know if I can take this issue

Is this package still under development? How has no one else prioritized the ability to change dates?  I provided a solution but no one is willing to review and merge it.


custom config file can't replace all options

though I'm concerned about sql_mode , some others option are affected too
according to docs:

The Percona startup configuration is specified in the file /etc/mysql/my.cnf, and that file in turn includes any files found in the /etc/mysql/conf.d directory that end with .cnf. Settings in files in this directory will augment and/or override settings in /etc/mysql/my.cnf

but really

cat /etc/mysql/my.cnf

!includedir /etc/mysql/conf.d/
!includedir /etc/mysql/percona-server.conf.d/

cat /etc/mysql/percona-server.conf.d/mysqld.cnf

[mysqld]
skip-host-cache
skip-name-resolve
user   = mysql
pid-file = /var/run/mysqld/mysqld.pid
socket   = /var/run/mysqld/mysqld.sock
port   = 3306
basedir    = /usr
datadir    = /var/lib/mysql
tmpdir   = /tmp
lc-messages-dir  = /usr/share/mysql
explicit_defaults_for_timestamp

local-infile = 0

# Recommended in standard MySQL setup
sql_mode=NO_ENGINE_SUBSTITUTION,STRICT_ALL_TABLES

# Disabling symbolic-links is recommended to prevent assorted security risks
symbolic-links=0

i.e. /etc/mysql/percona-server.conf.d/mysqld.cnf has higher priority over anything in  /etc/mysql/conf.d/ and make harder to provide own setting (by example sql_mode)

I believe 5.7 is when percona switched to use the percona-server.conf.d directory. If you are building your image in top of percona, then you could add an rm on the default file or COPY into the percona directory. If you are bind mounting the config file at run time then you could do the whole percona directory and mask the file that is there: -v /my-conf-dir/:/etc/mysql/percona-server.conf.d/.

yes , I know how to manage this (though is slightly different with openshift origin)
but probably docs should be fixed.

the image isn't allow to customize all settings by providing own /etc/mysql/conf.d/config-file.cnf
the image isn't "drop-in replacement for MySQL"

I've created testing image https://hub.docker.com/r/oprudkyi/percona-latest/
https://github.com/oprudkyi/docker-percona
with fixed priority of dirs (updated my.cnf)
it works correctly for me
may I updated this repository as well ?

Documentation is over in https://github.com/docker-library/docs/blob/39c5a9db418362f775de2cf7936a0937a878946e/percona/content.md -- a PR to update them to reflect upstream's change in behavior would be welcome!

Can we have this fixed?
This is a bug since you cannot change sql_mode from /etc/mysql/conf.d/ since it is redefined in /etc/mysql/percona-server.conf.d/mysqld.cnf
It really just is a matter of updating the image to have /etc/mysql/my.cnf contain
#
# The Percona Server 5.7 configuration file.
#
#
# * IMPORTANT: Additional settings that can override those from this file!
#   The files must end with '.cnf', otherwise they'll be ignored.
#   Please make any edits and changes to the appropriate sectional files
#   included below.
#
!includedir /etc/mysql/percona-server.conf.d/
!includedir /etc/mysql/conf.d/

Correct, in order to change sql_mode with upstream's configuration, you'll need to either modify /etc/mysql/percona-server.conf.d/mysqld.cnf or put a new file in /etc/mysql/percona-server.conf.d which loads sooner than mysqld.cnf.

I got around this for now by naming my custom file zcustom.cnf and placing it in /etc/mysql/percona-server.conf.d/
Obviously not the best solution, but according to the mysql docs includedir does not guarantee the order of the files anyways, so I guess its OS related.

@dtunes I was doing the same, but it stopped working since the 5.7.21 image tag. I've no idea why.

Just started running into the same error, adding a file to the image:
~> cat /etc/mysql/percona-server.conf.d/zz_mysqld.cnf
[mysqld]

max_allowed_packet=1024M
innodb_log_file_size=256M

[mysqldump]
max_allowed_packet  = 1024M

still results in:
mysql> show global variables like '%allowed_packet%';
| max_allowed_packet       | 16777216   |
...

Image: percona:5.7                 03dba90d5b6b
regards,
Roman

@uuf6429 @strowi, I've upgraded my container to the latest image 5.7.22-22.
5.6 if I remember correctly you had to copy custom configuration into /etc/mysql/conf.d/zcustom.cnf
When I first upgraded to 5.7 quite a while ago copying custom config into /etc/mysql/conf.d wound't work, after some testing it was clear that I needed to copy them into /etc/mysql/percona-server.conf.d/zcustom.cnf and this worked. Following the documentation in this case was no good, as the documentation pointed to /etc/mysql/conf.d.
After upgrading to 5.7.22-22 (and possibly 5.7.21-20) it looks like they have reverted it back to /etc/mysql/conf.d, I've just copied my zcustom.cnf into /etc/mysql/conf.d instead of the /etc/mysql/percona-server.conf.d and it worked as expected.
Hope this helps.

sorry for the delay and THX! Additionally i had some too wide file permissions, so mysql would reject the content.

As another alternative, you should be able to supply arguments to mysqld which will take the most precedence for configuration of MySQL:
$ docker run ... percona:5.7 --max_allowed_packet=1024M
...

$ docker exec ... mysql -uroot ...
...
mysql> show global variables like 'max_allowed_packet';
+--------------------+------------+
| Variable_name      | Value      |
+--------------------+------------+
| max_allowed_packet |<PHONE_NUMBER> |
+--------------------+------------+
1 row in set (0.01 sec)

(Also closing this issue, given that the original question is now covered in quite some detail.)


Spacing issue involving single '=' in fractions in mrow (with mathmlSpacing=true)

Issue Summary
This is related to #2391, but this is with mathmlSpacing=true. In this case, the problem is the spacing around the '=' in the denominator of the third fraction:

In this example, for some reason the spacing around the '=' influences the width of the fraction in the last fraction, but not in the second fraction. The second fraction uses MathML's embellished operator rules and so the spacing around the second '=' is (appropriately) increased.
The header is
<script src="https://polyfill.io/v3/polyfill.min.js?features=es6"></script>
<script type="text/javascript" id="MathJax-script" async
<EMAIL_ADDRESS></script>
 <script>
    MathJax = {
  chtml: {
    scale: 1,
    mathmlSpacing: true
  }
};
</script>

and the MathML is
<math xmlns="http://www.w3.org/1998/Math/MathML" display="block">
  <mn>1</mn>
  <mfrac>
	<mo lspace="0" rspace="0">=</mo>
	<mn>2</mn>
  </mfrac>
  <mn>1</mn>
  <mfrac>
	<mo>=</mo>
	<mn>2</mn>
  </mfrac>
  <mn>1</mn>
  <mfrac>
	<mn>2</mn>
	<mo>=</mo>
  </mfrac>
  <mn>1</mn>
</math>

A codepen showing this at https://codepen.io/nms/pen/PoqLOpG

Right, the first two fractions are embellished operators, and so the spacing for the equal sign is used around the fraction.  The last one is not, so the spacing for the equal sign is used around the equal sign itself.  According to section <IP_ADDRESS>.2 because the <mo>=</mo> is not in an mrow, its default form attribute is infix.  According to the operstor dictionary, the infix entry for = has the default for lspace and rspace both equal to 5.  So this equal sign should have thickmathspace spacing around it.  That is where the extra spacing in the fraction is coming from.
If there is somewhere in the specification that says you aren't to use the default spacing under certain circumstances, I don't recall seeing that.  I see that Firefox does not use this spacing, but does show an explicit lspace or rspace value (which seems contrary to the default nature of those attributes in the operator dictionary).  WebKit does include the space within the fraction.
If I am mistaken about how the operator dictionary defaults apply (a distinct possibility), can you cite the section where that is described?

This is a case of where the MathML spec is both not specific and probably misleading. So don't feel bad about interpreting them incorrectly; blame me or someone else for writing them poorly :-)  I wrote those tests to see if my feeling about what should be done for the MathML Core spec respects what current implementations (mostly) do.
My interpretation (and what most others did) is that the spacing rules apply inside of an mrow. The lspace/rspace of the first/last operator are essentially inherited by the mrow so that if there is an outer mrow, they are used by the outer mrow for spacing. The other way of looking at it (and what MathPlayer implemented) is that you look (recursively) at the first/last child for the lspace/rspace of that child. Either way gets you the same result, but one is top-down and the other is bottom-up.
MathPlayer has one additional wrinkle that I'm going to recommend for MathML Core: when computing the spacing: instead of rspace(before) + lspace(after), do max(rspace(before), lspace(after)). Most of the time it won't matter, but there are a few prefix (maybe postfix) operators that have spacing on their "outside". In particular, differential d comes to mind. This rule means that not only will ∫f(x) dx look right, so will dy = dx because the rspace of '=' won't add to the lspace of 'd'.

I have finally gotten the chance to fix this.  It turns out not to be as hard as I thought it was going to be.  I also implemented your Max(rspace,lspace) idea.  So I think that should resolve the issues here.

PS, currently this is only fixed in v3.  I'm not sure if I will back-port it to v2, where the fix will be a bit harder.

I don't think you should fix in v2; it is good to have incentives to move to v3 :-)

Good point!


[Bug] http://<IP_ADDRESS>:8000 Web showing {"detail":"Not Found"}

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I am sure the issue hasn't been already addressed by searching through https://github.com/agiresearch/AIOS/issues.
[X] The usage issue is not resolved by updating to the latest stable version in the main branch.

Describe your current environment
Mac OS
Python 3.10
Describe the bug
After the command:
bash runtime/launch_kernel.sh

INFO:     Started server process [169]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)

The page (http://<IP_ADDRESS>:8000) showing:

Do you attempt to open the aios kernel in the browser by clicking the http://<IP_ADDRESS>:8000 url? AIOS kernel itself does not intend to show anything in the web browser. It is required to use AIOS SDK(Cerebrum) to interact with the AIOS kernel.

Yes I thought that would be a valid UI, good to know that.
I'll close it.


🐛 Bug Fix: Quantile test cases for lower & upper bounds

✅ Fixes: #146
This pull request includes several changes to improve the quantile functionality and its associated tests. The main changes involve moving the quantile tests to a new file, adding more comprehensive test cases, and enhancing the quantile function to handle additional edge cases and input validation.
Enhancements to quantile functionality:

pysnippets/stats/quantile.py: Added input validation to ensure all elements in the data list are numeric and improved error handling for invalid quantile values.

Improvements to tests:

Tests/stats/test_quantile.py: Created a new test file for quantile tests, added more comprehensive test cases including edge cases, repeated elements, negative numbers, floating-point numbers, and non-numeric values.
Tests/stats/test_data_distribution.py: Removed the quantile tests to keep the file focused on z-score normalization tests.

Screenshot

@UTSAVS26 can you review this PR and assign me on this, with proper labels.


add zh-cn translate for "Space Exploration" mod.

I doesn't test it.
How I generate this file:

Run factorio.exe --dump-prototype-locale dump the localized json file. (learnt from your factorio-dump.bat file)
Extract all ids in the data/sxp/data.json
Try very hard to find the localized string. There are so many special case.

Some strings I didn't translate them because I doesn't install some mod(like "grappling-gun"). These mod is not required by SE.

You maybe generate the wrong id: se-antimatter-canister-burn, your English string is "Antimatter canister : Magnetic canister". I think the id should be "se-antimatter-canister-se-magnetic-canister-burn".
You have some undefined names in the ids that start with "se-rocket-launch-pad-silo-".

If it doesn't bother, I hope you can build these localized json by yourself. There are many you custom generated ids, it is not easy to translate it. If you can do this in your side, it will be best.
You can modify the factorio's config.ini , before you dump the json. Then it will generated localized files.

Unless you are specifically requesting Chinese translations for SE, I think I'm just going to close this PR. If you would like to open this as a feature request, feel free to do so and I can look into it.


Make "all" the default target in distro/pods/drake-distro.

This contributes to https://github.com/RobotLocomotion/drake/issues/2136.
I'm sending you this PR because I can't think of a reason you would want VERBOSE=1 make to fail.  If there is a reason, we can instead change the Drake build to build the all target explicitly.

I think this is a standard pod makefile, are they all broken this way?
In the future we can have drake build director as a standard CMake project and retire this makefile.

I guess they are!  That's fun.
Agreed, the day we build director as CMake will be a happy one.  I think @mwoehlke-kitware is working towards that.

Looking at other pod makefiles as an example, adding this line to the top of the makefile would make it match the others:
default_target: all

I'm not sure why it is missing from the director makefiles.  It appears that your patch has the same effect though.  The fix should be applied to both makefiles:
distro/pods/drc/Makefile
distro/pods/drake-distro/Makefile


[stable/fluentd] Prefix configuration files with numbers to clarify o…

…rdering (#9708)
Signed-off-by: Matthew Shin<EMAIL_ADDRESS>
What this PR does / why we need it:
Prefixes configmap keys used for fluentd configuration with numbers.
Fluentd loads configuration alphabetically from the configmap, this will ensure that consumers of the chart have full control over where their configuration goes (this matters due to how fluentd does matching).
This commit also allows users to disable default values by providing the yaml null value for an existing key.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #9708

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x ] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@rendhalver
@Miouge1
@hectorj2f

/assign @rendhalver


Command line open a same file twice, generate two windows

VSCode Version:  0.10.11
OS Version: OS X 10.11.4

Steps to Reproduce:

In Terminal, type "code ~/.profile" will open the window with .profile.
Type "code ~/.profile" again, a new window will open with the same .profile file. It should resume the window in step 1.

Two options:

you configure VS Code to open files in the same window: window.openFilesInNewWindow = true
you start with "-r" option to reuse last active instance

closing as designed, given the workarounds.


swagger-ui multiple API versions

Hi,
I'm currently developing a multi version WebAPI.
I've choose the following package to do so: https://github.com/Microsoft/aspnet-api-versioning
My question is, which is the best way to have separated swagger-UI pages for each version?
I would like to have something like:

http://api.site.com/v1/docs

http://api.site.com/v2/docs

Now I am able to have a 'mixed' UI doc page, with all the controllers, but it is a little confusing for my costumers, I think it would be better to display all the endpoints.
Thanks,
Xavi

@Xavina did you know that there is an API Explorer for API Versioning to support this now?  You can see the Web API sample here. Although it's RC, it's stable. I'm tightening up the last few bits this week and I think things will be ready for the first, official release.


Fixed check for installed XD roles

The check now works even if the key named Products is not present

Great news! Thanks for your help :)


Fix scatter issues

Tests were updated to include instances of the requirements.

Thanks!
On Mon, Oct 15, 2018 at 8:24 PM Alex Wolf<EMAIL_ADDRESS>wrote:

Merged #304 https://github.com/theislab/scanpy/pull/304 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/theislab/scanpy/pull/304#event-1904871033, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEu_1X1kzdMFubtvfmXdm3bfv6i_PGZeks5ulNLfgaJpZM4XcU1H
.


VanillaTemporalModule.forward() missing 1 required positional argument: 'encoder_hidden_states'

When trying to create an animation I encounter the following error:
Error occurred when executing KSampler:

VanillaTemporalModule.forward() missing 1 required positional argument: 'encoder_hidden_states'

File "/home/xxxxxxxx/ComfyUI/execution.py", line 152, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "/home/xxxxxxxx/ComfyUI/execution.py", line 82, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "/home/xxxxxxxx/ComfyUI/execution.py", line 75, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "/home/xxxxxxxx/ComfyUI/nodes.py", line 1236, in sample
return common_ksampler(model, seed, steps, cfg, sampler_name, scheduler, positive, negative, latent_image, denoise=denoise)
File "/home/xxxxxxxx/ComfyUI/nodes.py", line 1206, in common_ksampler
samples = comfy.sample.sample(model, noise, steps, cfg, sampler_name, scheduler, positive, negative, latent_image,
File "/home/xxxxxxxx/ComfyUI/custom_nodes/ComfyUI-Impact-Pack/modules/impact/sample_error_enhancer.py", line 9, in informative_sample
return original_sample(*args, **kwargs)
File "/home/xxxxxxxx/ComfyUI/custom_nodes/ComfyUI-AnimateDiff-Evolved/animatediff/sampling.py", line 161, in animatediff_sample
return wrap_function_to_inject_xformers_bug_info(orig_comfy_sample)(model, *args, **kwargs)
File "/home/xxxxxxxx/ComfyUI/custom_nodes/ComfyUI-AnimateDiff-Evolved/animatediff/model_utils.py", line 162, in wrapped_function
return function_to_wrap(*args, **kwargs)
File "/home/xxxxxxxx/ComfyUI/comfy/sample.py", line 97, in sample
samples = sampler.sample(noise, positive_copy, negative_copy, cfg=cfg, latent_image=latent_image, start_step=start_step, last_step=last_step, force_full_denoise=force_full_denoise, denoise_mask=noise_mask, sigmas=sigmas, callback=callback, disable_pbar=disable_pbar, seed=seed)
File "/home/xxxxxxxx/ComfyUI/comfy/samplers.py", line 785, in sample
return sample(self.model, noise, positive, negative, cfg, self.device, sampler(), sigmas, self.model_options, latent_image=latent_image, denoise_mask=denoise_mask, callback=callback, disable_pbar=disable_pbar, seed=seed)
File "/home/xxxxxxxx/ComfyUI/comfy/samplers.py", line 690, in sample
samples = sampler.sample(model_wrap, sigmas, extra_args, callback, noise, latent_image, denoise_mask, disable_pbar)
File "/home/xxxxxxxx/ComfyUI/comfy/samplers.py", line 630, in sample
samples = getattr(k_diffusion_sampling, "sample_{}".format(sampler_name))(model_k, noise, sigmas, extra_args=extra_args, callback=k_callback, disable=disable_pbar, **extra_options)
File "/home/xxxxxxxx/.local/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/xxxxxxxx/ComfyUI/comfy/k_diffusion/sampling.py", line 137, in sample_euler
denoised = model(x, sigma_hat * s_in, **extra_args)
File "/home/xxxxxxxx/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/xxxxxxxx/ComfyUI/comfy/samplers.py", line 323, in forward
out = self.inner_model(x, sigma, cond=cond, uncond=uncond, cond_scale=cond_scale, cond_concat=cond_concat, model_options=model_options, seed=seed)
File "/home/xxxxxxxx/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/xxxxxxxx/ComfyUI/comfy/k_diffusion/external.py", line 125, in forward
eps = self.get_eps(input * c_in, self.sigma_to_t(sigma), **kwargs)
File "/home/xxxxxxxx/ComfyUI/comfy/k_diffusion/external.py", line 151, in get_eps
return self.inner_model.apply_model(*args, **kwargs)
File "/home/xxxxxxxx/ComfyUI/comfy/samplers.py", line 311, in apply_model
out = sampling_function(self.inner_model.apply_model, x, timestep, uncond, cond, cond_scale, cond_concat, model_options=model_options, seed=seed)
File "/home/xxxxxxxx/ComfyUI/custom_nodes/ComfyUI-AnimateDiff-Evolved/animatediff/sampling.py", line 537, in sliding_sampling_function
cond, uncond = sliding_calc_cond_uncond_batch(model_function, cond, uncond, x, timestep, max_total_area, cond_concat, model_options)
File "/home/xxxxxxxx/ComfyUI/custom_nodes/ComfyUI-AnimateDiff-Evolved/animatediff/sampling.py", line 519, in sliding_calc_cond_uncond_batch
sub_cond_out, sub_uncond_out = calc_cond_uncond_batch(model_function, sub_cond, sub_uncond, sub_x, sub_timestep, max_total_area, sub_cond_concat, model_options)
File "/home/xxxxxxxx/ComfyUI/custom_nodes/ComfyUI-AnimateDiff-Evolved/animatediff/sampling.py", line 431, in calc_cond_uncond_batch
output = model_function(input_x, timestep_, **c).chunk(batch_chunks)
File "/home/xxxxxxxx/ComfyUI/comfy/model_base.py", line 63, in apply_model
return self.diffusion_model(xc, t, context=context, y=c_adm, control=control, transformer_options=transformer_options).float()
File "/home/xxxxxxxx/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/home/xxxxxxxx/ComfyUI/custom_nodes/SeargeSDXL/modules/custom_sdxl_ksampler.py", line 70, in new_unet_forward
x0 = old_unet_forward(self, x, timesteps, context, y, control, transformer_options, **kwargs)
File "/home/xxxxxxxx/ComfyUI/custom_nodes/FreeU_Advanced/nodes.py", line 173, in __temp__forward
h = forward_timestep_embed(module, h, emb, context, transformer_options)
File "/home/xxxxxxxx/ComfyUI/comfy/ldm/modules/diffusionmodules/openaimodel.py", line 61, in forward_timestep_embed
x = layer(x)
File "/home/xxxxxxxx/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)

It occurs with this workflow which is known to be working. Once I delete FreeU_Advanced it all works fine.
Youtube_ControlNet_AnimateDiff.json.txt

Had the same issue using animatediff as until the new one came in there was no problems. I had to sit and remove the FreeU advanced from all my workflows and there were so many to be edited. The issue never comes up with certain workflows but with AD it is more of a hassle.

It's not compatible with AnimateDiff models. There has been no chance to ComfyUI UnetModel.forward thus nothing changed in FreeU code. Just a limitation of AnimateDiff or ComfyUI. AnimateDifd are not normal SD models.

It's not compatible with AnimateDiff models. There has been no chance to ComfyUI UnetModel.forward thus nothing changed in FreeU code. Just a limitation of AnimateDiff or ComfyUI. AnimateDiff are not normal SD models. They are editing ComfyUI with patches just like FreeU does.

Just in case it wasn't clear, I was not using your advanced node, just the normal built in FreeU node. FreeU Advanced simply being installed was conflicting with the AnimateDiff setup. Upon deleting it, all was well again.

They are editing ComfyUI with patches just like FreeU does.

Understand and will convey same to animate diff and raise the issue there so they can attend to it for same. Awesome and thanks for your feedback.

Understand and will convey same to animate diff and raise the issue there so they can attend to it for same. Awesome and thanks for your feedback.

Looks like Comfy may add the Input/Middle patching, so at that point, FreeU advanced will not longer need to patch the UnetModel.forward


Add exclusion for gatsby-plugin-catch-links

Summary of changes

Add exclusion for gatsby-plugin-catch-links
Remove unnecessary title from section component

Frontend
List all significant changes to Gatsby code
Backend
List all significant changes to the Drupal site (e.g. new fields, content types, modules, etc)
Test Plan
Insert steps. Include URLs of Gatsby Cloud test site and Drupal multidev.

I just tested this and UG events pulled from WordPress worked normally. Since events are the only WordPress content being rendered by our Gatsby frontend (and gatsby-plugin-catch-links is only a dependency of the WordPress plugin), I'd say this is fine to merge.


$checkcondition in $alwaysexecute

All my $alwaysexecute commands had the $checkcondition code. But if something like $onlyif[$message==abc;] happens and someone sends "" the botbot is spamming the error.

No changes.


Plugin isolation

I think plugins (really anything with a separate Gemfile) should be isolated so the gems in it don't conflict. For example, adding slim to a plugin and to the main Rails application causes bundler warnings.

I am thinking in a way to manage plugins like a rails gem where the people only will need add the gem like this:
gem "camaleon-ecommerce"

Yeah, camaleon-ecommerce would be a Rails engine. That's how refinery plugins work too.
It's pretty neat how plugins work right now but the lack of isolation is also a disadvantage.

correct, please let me know if you can create a way to manage that disadvantage.

Ok, I'll think about some possible solutions

I added the support for plugins in gem version!


For a any event adhoc data that is sent in the request, must be received in the webhook callback.

We are unable to identify the internal resource from the Webhook data provided by codechat. We want certain data that will be part of the request, to be part of the Webhook data.

Format the data to base64 and add in the URL query in the webhook record.

Let me elaborate on the requirement.
When I send a text message, I receive message.update event with DELIVERY_ACK post delivery. When I make a request to send a message, I will send some additional data. I would need that data to be returned to me in the webhook callback.
This would help me in identifying the resource that triggered the message. The data I send is dynamic that varies for each message wrt resource that triggered this event.


DNS lookup triggers local configuration backup

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
Performing a DNS lookup with the built-in diagnostics tool creates a local backup of the OPNsense configuration.
This didn't happen with the previous DNS lookup tool.

Environment
OPNsense 22.7.r1_8 (amd64, OpenSSL)

Nice catch, related to #5843 as the new "memory" model defies the current config expectations. Handing it over to @AdSchellevis

fixed in https://github.com/opnsense/core/commit/40a64dc9f535254057d1c25778a7c6054f959b2e, @maurice-w thanks for reporting!


Enhancement - Enroll to multiple udemy accounts

Please add the ability to enroll into multiple Udemy accounts. It would be great if the account cookie list can be provided through a text file.

We might add it up in the list for future updates


tx: Increase estimateGas constant by four

Workaround for https://github.com/oasislabs/private-parcel/issues/986

Superseeded by https://github.com/oasislabs/private-parcel/pull/1000


Opening download directory selection browser crashes on Android 6.0

Opening the download directory chooser crashes on Android 6.0, @stuxo mentions this may be due to needing to request permissions to browse the directories now.
The browser also missed out on getting theme'd and should be restyled to match as well
Crash report:
Error Report on: Tue Mar 29 23:00:24 MDT 2016
what.whatandroid version name: 2.1.6.b, version code: 27
Android Version: 6.0.1
Device: shamu
Brand: google
Model: Nexus 6

Stack Trace:
java.lang.NullPointerException: Attempt to invoke interface method 'int java.util.List.size()' on a null object reference
at android.widget.ArrayAdapter.getCount(ArrayAdapter.java:337)
at android.widget.ListView.setAdapter(ListView.java:491)
at what.whatandroid.settings.FolderPickerDialog.onCreateDialog(FolderPickerDialog.java:119)
at android.app.DialogFragment.getLayoutInflater(DialogFragment.java:407)
at android.app.FragmentManagerImpl.moveToState(FragmentManager.java:973)
at android.app.FragmentManagerImpl.moveToState(FragmentManager.java:1148)
at android.app.BackStackRecord.run(BackStackRecord.java:793)
at android.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1535)
at android.app.FragmentManagerImpl$1.run(FragmentManager.java:482)
at android.os.Handler.handleCallback(Handler.java:739)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:148)
at android.app.ActivityThread.main(ActivityThread.java:5417)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)

Please describe where this occurred and what may have provoked the crash to help debug the issue:

One user tested manually granting the "Storage permission" to the app and opening this no longer crashed, so it's definitely that we're not requesting the right permissions.

I thought that might have been the case. Its an annoying addition to android 6.0


[BUG] Missing Attributes

As of the latest version 1.3.1 code generation now seems to be valid typescript but some attributes seem to be missing when using the generated types.
Example WSDL: https://community.workday.com/sites/default/files/file-hosting/productionapi/Human_Resources/v36.1/Human_Resources.wsdl
Assuming the client has been auto generated it should be possible to initiate a change work contact information which you can see an example below, but some attributes seem to be missing such as the Primary field which exists in the WSDL as an attribute on Type_Data but does not seem to exist in any of the types.
Change_Work_Contact_InformationAsync({
    Change_Work_Contact_Information_Data: {
        Universal_ID_Reference: {ID: ['123']},
        Person_Contact_Information_Data: {
            Person_Email_Information_Data: {
                Email_Information_Data: [{
                    Email_Data: [{
                        Email_Address<EMAIL_ADDRESS>                    }],
                    Usage_Data: [{
                        Type_Data: [{
                            // Primary Field should be here
                        }]
                    }]
                }]
            }
        }
    }
});

Well, wsdl-tsclient is working correctly. Probably there's problem with node-soap not parsing AttributeElement in ComplexType, I even found out not-merged PR for that https://github.com/vpulim/node-soap/pull/1013
I have to do more investigation.... but right now, there are 3 options that comes to my mind

Creating Issue for node-soap project (they don't have GH Issue)

Updating closed PR 1013

Forking node-soap (which I already did) and implement it there

I think it maybe has to do with this part but not sure.
Currently, when supplying JSON args, elements may not contain both child elements and a text value, even though that is allowed in the XML specification.

As for the 3 options it seems to be like the best Option is 2 & 3. Perhaps forking for now and using the fork until the PR is reviewed and hopefully merged at which point it could be switched back.

@dderevjanik I posted on the closed PR and it seems like they are open to merging it if it gets fixed and updated. So if you already have a branch/fork that is updated and working I think you should make a pull request and they will likely accept it.

Does anyone have a working fork?

Here's a smaller repro:
<?xml version="1.0" encoding="utf-8"?>
<wsdl:definitions xmlns:s="http://www.w3.org/2001/XMLSchema" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:http="http://schemas.xmlsoap.org/wsdl/http/" xmlns:mime="http://schemas.xmlsoap.org/wsdl/mime/" xmlns:tns="http://example.com/" xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:tm="http://microsoft.com/wsdl/mime/textMatching/" xmlns:soapenc="http://schemas.xmlsoap.org/soap/encoding/" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" targetNamespace="http://example.com/">
  <wsdl:types>
    <s:schema elementFormDefault="qualified" targetNamespace="http://example.com/">
      <s:element name="RequestData">
        <s:complexType>
          <s:attribute name="name" type="s:string"/>
        </s:complexType>
      </s:element>
      <s:element name="ResponseData">
      </s:element>
    </s:schema>
  </wsdl:types>
  <wsdl:message name="RequestMessage">
    <wsdl:part name="parameters" element="tns:RequestData"/>
  </wsdl:message>
  <wsdl:message name="ResponseMessage">
    <wsdl:part name="parameters" element="tns:ResponseData"/>
  </wsdl:message>
  <wsdl:portType name="MyServiceSoap">
    <wsdl:operation name="Submit">
      <wsdl:input message="tns:RequestMessage"/>
      <wsdl:output message="tns:ResponseMessage"/>
    </wsdl:operation>
  </wsdl:portType>
  <wsdl:binding name="MyServiceSoap" type="tns:MyServiceSoap">
    <soap:binding transport="http://schemas.xmlsoap.org/soap/http"/>
  </wsdl:binding>
  <wsdl:service name="MyService">
    <wsdl:port name="MyServiceSoap" binding="tns:MyServiceSoap">
    </wsdl:port>
  </wsdl:service>
</wsdl:definitions>

I'm using the following utility type to add an attributes to the generated types.
/**
  * Add an attributes property to all nested object.
  * See: https://github.com/dderevjanik/wsdl-tsclient/issues/20
  */
 export type WithAttr<T> = T extends object
         ? {
                         [P in keyof T]: WithAttr<T[P]>;
           } & { attributes?: Record<string, any> }
         : T;

@icholy how do you use this WithAttr? because in some sense even something that would have been a string could have attributes too?


CI times out after 50 minutes

With all the new scripted policies, tests take longer than ever and builds are failing because Travis times out after 50 minutes.

Can you get under the limit by reducing the number of trials you run per scripted policy? 1000 is likely overkill.

Yes, but with fewer trials, there's a higher chance of having an unlucky test. I'll have to decrease the success rates slightly in order to get things to pass consistently.

Yes. That is always the trade-off with statistical testing. Realistically, the difference between 100 and 1000 trials is not super large. The sampling error grows at approximately O(1/sqrt(n)) so 100 to 1000 is not as large a jump as it seems.
If you want to keep around the 1000-trial tests for local usage, your test suite can detect that it's in the CI using the CI environment variable and react accordingly, or you can create separate tests with more trials, mark them as huge (or something else), and then exclude that marker when running in the CI. I think approach #2 is preferable and you can see it used throughout garage.


getMediaType: Report an error for any encoding property not defined in the schema.

Fixes issue #1711

Thanks!


Babylon 4.2: can Cardboard VR be supported without Google VR Services?

Hi! Great 4.2 release, guys!
I have a question / feature request: can Babylon.js support Cardboard VR without Google VR Services?
Here's how it looks when I try the first VR example from Getting Started:
https://doc.babylonjs.com/start/chap8/vr

When I click that goggles button, it redirects me to Google VR Services app that is not supported on my device:

But the problem is: I can run Cardboard VR just fine without Google VR Services installed:

Is it possible to enable Cardboard VR mode without Google VR Services?
The google code for their Cardboard VR app that works without Google VR Services is open source, by the way: https://github.com/googlevr/cardboard
Thanks!

Hello! do you mind asking the question on the forum?
Thanks a lot
@RaananW  FYI

@deltakosh sure! Though it's sort of a feature request, if Cardboard VR can't be activated without the Google VR Services app.

anyway. we recommend forum discussion for everything (as stated in the github issue template)


Reduce the PBIRS main page session timeout

Hi
I have a site that has an iframe that connect in PBIRS using CustomAuthentication. My Custom Authentication uses the UserInformation stored in Database to do Custom Authentication.
When I am authenticated on the site, I can open the 'http://.../Reports/Browse' in another tab. The problem is that after I do log out/close my site, I want to block the 'http://.../Reports/Browse' from the user. This access remains open to the user for more than 30 minutes after the ReportServer block the session.
---Before login in my site----
http://...../Reports/browser: access blocked
------After login in my site and custom authentication in PBIRS----
http://...../Reports/browser: access open for MySiteUser
------After logout in my site/close the page that call ReportServer----
http://...../Reports/browser: access remains open for MySiteUser
------More than 30 minutes after close the page----
http://...../Reports/browser: access blocked again
There is an entrance in web.config or rsreportserver.config that can help me to reduce this session time to close 1 minute at least after I close my site? I already tried the sessionState tag in the Web.config, but not resolved.
Thank You

I tried to resolve this problem reducing the 'timeout' value in the web.config section 'loginurl="logon.aspx" name="sqlAuthCookie", timeout="1"'....but now I have another problem, always the user has his access blocked in ReportPage/browse and in the iframe too after 1 minute, even when the user is manipulating the report in the Iframe.
This is not what I want, I want that my access in the http://.../Reports/browse be blocked after 1 minute of inactivity, only inactivity.

Have you tried using sliding expiration? https://docs.microsoft.com/en-us/dotnet/api/system.web.security.formsauthentication.slidingexpiration
sliding expiration should reissue the cookie after half of the time passed, but the customer is still actively using it, you probably want to set it longer than a minute, maybe 10 minutes?

@gboreki . Thank You!!!
This solved my problem!!


CAMEL-10723: Improved camel-aws kinesis documentation

@oscerd looks good to merge

I'll merge soon :-) Thanks @ullgren

Thanks @ullgren . The PR has been merged. Can you close this?

@oscerd do you mind checking if 2.18.x branch can have this PR as well, as that default value in the component doc schema would be good to have backported

Sure, I'll check

Backported on 2.18.x too

Thanks for merging.


BLD: update to h5py 2.8.0

This will let us use numpy 1.14 on the floor this cycle

This is slightly different that the conda-forge recipe as 1.12 is the oldest version of numpy that the anaconda channel has numpy built for 3.6.


style(deletetask , tasks/page): redesign Delete Task button

Pull Request Template
Description
Please include a summary of the changes and the motivation behind them. Link to any related issues or pull requests.

Related Issues: Closes #[issue number] or Connects to #[issue number] (if applicable)
Type of Change: Bug fix / New feature / Refactoring / Documentation

Changes Made
List out the specific changes or updates made in this PR.

[ ] Update 1
[ ] Update 2
[ ] Any additional notes on modifications

How to Test
Provide step-by-step instructions on how to verify the changes. Include any setup details or special conditions that need to be met.

Step 1
Step 2
Step 3

Screenshots (if applicable)
Attach any screenshots or screen recordings that demonstrate the new feature, UI changes, or bug fix.
Checklist
Please ensure that all boxes are checked (replace [ ] with [x]). This checklist will help in maintaining a high standard for each PR.

[ ] I have tested the changes and they work as expected.
[ ] I have added relevant documentation (if applicable).
[ ] I have updated relevant tests (if applicable).
[ ] This PR has a descriptive title and description.
[ ] This PR follows the code style and conventions of the project.

Additional Notes
Add any other context or information that reviewers should be aware of.

Thank you for your contribution! We appreciate your effort to make this project better.

#56


fix(UtilityProvider): fixing console warning

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[SignalR] Avoid over-negotiating in client

There are 2 cases where we might over-negotiate in the Typescript client (should audit the other clients too).

Client specifies a transport and server sends a larger list of supported transports, the client will still evaluate transports higher in the list if there are any, and cause a re-negotiate to happen
Client doesn't support WebSockets or EventSource and we evaluate that too late and cause a re-negotiate

Acceptance checklist (check one item)

[ ] We decided not to take this fix.
[ ] The fix is tests-only.
[ ] This is a Docs issue.
[x] The fix contains product changes (check all items below).

[x] Relevant XML documentation comments for new public APIs are present. - Not needed
[x] Narrative docs (docs.microsoft.com) are updated. (check one item below)

[ ] The change requires a new article. An issue with an outline has been filed here: [ISSUE LINK]
[ ] The change requires a change to an existing article. A closed docs PR with these changes is linked here: [PR LINK]
[x] The change requires no docs changes.

[x] Verification has been completed. (check one item below)

[ ] The change is in the shared framework and was verified against the following versions

SDK installer: [VERSION]
ASP.NET Core Runtime: [VERSION]
.NET Core Runtime: [VERSION]

[x] The change is in an OOB NuGet/NPM/JAR package and was verified against the following version of that package: aspnet-signalr-3.0.0-preview7.19325.3


Keyboard in editor unusable in latest code

Cant edit code, can click and drag with the mouse, but the keyboard does nothing.
I'm running Linux.

Godot 3.0 master commit 2e3145de65257ed39a5d01c5592ea0fb90cd00ba
Tested on Linux 64bits and Windows10 64bits, keyboard in editor is not working on either

I think is not keyboard in editor (because you can use it to move nodes on 2D editor) but Controls that ignore input, all except LineEdit ignore keyboard input and middle mouse/wheel, happens on editor and running projects with Controls.

Fixed in 3744d9f.


Similar to RSVP.hash

Is there any way to send parallel requests similar to RSVP.hash

Yes.
These will run in parallel:-
const promise1 = fetch('http://website.com/url1');
const promise2 = fetch('http://website.com/url2');
results = yield Promise.all(promise1, promise2);


Update readme with usage instructions

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
32

Relevant Lines:
39

💛  - Coveralls


Use external LB and DNS for install

We should have a way to disable haproxy and DNS in bastion when using external LB and DNS services.

Tested to use predefined DNS and load balancer on a different host, the installation completed and works fine. Here are the manual steps:

  on ather host to config DNS and haproxy with predefined IP and hostname, note for DNS config, need to use load balancer IP for api, api-int and *.apps.

 run ocp4-helpernode to config the bastion as normal

 update /etc/dhcp/dhcpd.conf, using external DNS IP for domain-name-servers

 restart dhcpd

 stop DNS and haproxy (optinal, not required)

 run ocp4-playbooks to install OCP

There are two tasks need to be done to implement it:

Enable ocp4-upi-powervm to support predefined IP for all cluster nodes ( bastion, bootstrap, masters and workers)
Enable the ocp4-helpernode to support to use external DNS for DHCP config

PR created for ocp4-helpernode: https://github.com/RedHatOfficial/ocp4-helpernode/pull/219


turbo_frame_reload changes the src of the frame?

I tripped across this today, but not sure if this is by design / unavoidable in turbo, or a bug?
I have a turbo frame with a src like:
<turbo-frame id="blah" src="/blah/1">
in my rails controller, on another action (let's say I'm editing blah) I tell it to reload that frame:
#blah_controller

def edit
  #do stuff
  render turbo_stream: turbo_stream.turbo_frame_reload('blah')
end

Then the turbo frame src of the frame I'm reloading changes to:
<turbo-rame id="blah" src="/blah/1/edit">
which is incorrect, the src should stay the same.
Is this intentional, or due to some side effect of Turbo reloading? IMO, the src of a turbo frame should never change. And its causing unexpected issues in my app because of the url its putting there.

This might be some other issue in my app, I'm seeing some REALLY weird turbo behavior so I'm going to clean up some code and verify that the above really is happening the way I think it is.

Sigh, yeah, this is turbo behavior inside a frame where my button to the edit action is replacing the url of the src in the turbo frame.  Not a fault of turbo power. Closing the issue

Hey @DaveSanders, thanks for reaching out. I'm glad you figured it out!
If there's anything I can do to improve this expierience within Turbo Power let me know! I'd love to make this as seamless as possible!


File Word Wrap disabled as soon as you leave the tab

VSCode Version: Code 1.10.2 (8076a19fdcab7e1fc1707952d652f0bb6c6db331, 2017-03-08T14:02:52.799Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

vscode-eslint
dbaeumer
1.2.8

EditorConfig
EditorConfig
0.6.2

csharp
ms-vscode
1.8.1

PowerShell
ms-vscode
0.11.0

quokka-vscode
WallabyJs
1.0.20

Steps to Reproduce:

Open a text document with long lines of text, requiring word wrap
Turn on Word Wrap
Open another document (new or existing) in another tab
Return to the original document

Expected: Word Wrap is still enabled on the original document
Actual: Word Wrap is disabled on the original document
I have also attempted this with all extensions disabled and receive the same result.

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: http://code.visualstudio.com/Download#insiders

Reproduced in 1.10.2 and the latest insiders.

I think I fixed this in the meantime. e.g.:

@alexandrudima @bpasero I still have the behavior described in this issue with the recent 1.13.0 verson (on Windows). Any settings having an impact on this?


Feature/display existing backend

@elnzv please review this pull request.

Looks fine, provide some comments to code and it can be merged.

@apinf/developers please review. I have made suggested changes.


Added SC using imdb tt search

Sorry for the cancellation of the last pull request but I fixed the search to be more accurate by using the imdb tt instead of the search string.
SC is different than SP. Code change starts at line 639. Script name should be fixed.

Did I make a mistake on this one? Is there a reason this pr hasn't been merged yet?

The SC, SP and tMDB changes are great, however there's still an entirely new IMDb Scout.user.js that's being dragged in. Nothing against your current changes, but it will take a little work to merge

There we go. I figured out how to delete the other script. You shouldn't see that file anymore. Sorry for the confusion! Future requests shouldn't have this issue. Thanks for bearing with me as I figure out github lol.


Link to samples? guides? blog post?

Can we post related content links?

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

ID: 54a47d2c-61a3-01b3-d0dc-73a4b7d5117f
Version Independent ID: 4c713fb3-f086-5319-cf44-33346a7a14b7
Content: Windows.Networking.Vpn Namespace - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.networking.vpn.yml&version=GBlive&_a=contents
Product: windows
Git Login: wdg-dev-content
Microsoft Alias: wdg-dev-content

@kwiqniss, my sincere apologies for the delay. I'm still making enquiries internally to see what info I can gather to help resolve this issue, as well as #448. ~Steve

Hi, @kwiqniss, again, apologies for the delay. I've been in touch with internal folks now and so far nothing has sprung to mind that we could link to in terms of related content. If that changes in the future then I'll immediately update the topic with links.

@kwiqniss, I'll go ahead and close this issue now since there's nothing further we can do in the immediate term to entirely resolve it. Please reopen if I made a mistake in closing. ~Steve


Need to send APRS messages more frequently on decend closer to the ground

subj

Is this important? We want to avoid flooding the APRS system and every 30 seconds is already fairly often. Dithering was added in 75a7707 which should help.

Might need to chat to Lee about this. I am not sure what's the protocol here.

This is probably not necessary. Especially with the high signal quality of the PCB and the directional antenna build, increasing the signal frequency will probably just drain the battery faster on the ground.

I think we need this. The last seconds are the most important in transmitting the exact location. 30 second interval is too large.

after talking to everybody we need to send messages with 10-15 seconds delay during last 1km of descend.
please note that base elevation in eastern Washington is quite higher than sea level.


Fixes, Improvements, and Upgrades

Wow. I wasn’t expecting so much feedback. Instead of simply recognizing the faults I have decided to contribute instead. I have a pull request ready that does some cleanup/fixes/improvements/modernization to the project that alleviates some of the issues I mentioned.
Can I get access to the git repo so that I can push a new branch and create the pull request for review?

Please fork and raise a PR? Don't need repo access for that?


Rods import as the wrong type

All the Rods are importing as trinkets.  Dnd5e has an item type for rods, its under Consumable for some reason.  Its actually better for Auto effects too if Rods are imported as Consumable\Rod.  This is sorta tied to that annoying Item sheet change in 2.2.1.  Tim's only adding the feet unit field to certain item sheet types and "Trinkets" are not getting that fix, but also trinkets don't get the midi flags at the bottom of the sheet either.  So you really should want to change rods to Consumable\Rod types.

Fixed in next version.


Property synthesis

I'm using RMStepsController in QULQuestionnaire. However, I'm not able to push a new version to the trunk of CocoaPods, as I'm getting the following warning:
RMStepsController/RMStepsController/RMStepsBar.h:91:52: warning: auto property synthesis will not synthesize property 'delegate'; it will be implemented by its superclass, use @dynamic to acknowledge intention
Side note: How did you manage to push a new version of RMStepsController to the trunk with this warning?

That's because I did not push a new version for a long time. Last time I pushed a version, Xcode did not throw this warning.

I pushed a fix for the warning :)

Great, thanks! Unfortunately, you'd have to tag the new version in order to be accessible in a pod spec.


[Feature]: Fitting Multiple Datasets with a single expression

Feature Request
One way to prevent over-fitting when trying to find an equation for a particular trace of data is to provide multiple traces for a single equation as examples of the kind of results the equation produces. Is this something already implemented,  and if not what would it take to get there?

Great question. This is something I’m really eager to have. Some help would be much appreciated though.
The ongoing effort: the PR https://github.com/SymbolicML/DynamicExpressions.jl/pull/73 adds some necessary utilities to get this, which would let you be much more flexible in terms of how you define an expression — such as constraining functional forms, or such as learning parametric functions with per-‘category’ parameters.
If you are interested in helping, the next step would be to modify SymbolicRegression to use AbstractExpression added in that PR, rather than the current behavior which uses AbstractExpressionNode (a less flexible type). This should be possible to work on right away because the AbstractExpression interface has matured — I’m just adding more tests at the moment.

Another option, if you don’t wish to do it directly with SymbolicRegression.jl, is to do what we did in https://arxiv.org/abs/2202.02306. We learn a single expression (the force law) while also learning per-planet mass parameters. It’s easier to do this with deep learning; you essentially have the per-system parameters be trainable, and simultaneously fit an MLP. Then, finally, use the method in https://arxiv.org/abs/2006.11287 (basically, fit the inputs and outputs of the MLP with PySR) to get the actual parametric form of the equation.
There are some issues from this compared to a regular genetic algorithm, so I think it would be nice to have a proper implementation directly with SR.jl.

Hi @MilesCranmer, I wanted to look into it if thats alright and had a few questions:
How does the algorithm work with parametric equations? Iiuc, we can pass in some parametric expression with the values of parameters for each dataset we have with https://github.com/SymbolicML/DynamicExpressions.jl/pull/73. How does the search algorithm work in that case? Looking at the test for parametric expression in https://github.com/SymbolicML/DynamicExpressions.jl/pull/73/files#diff-22d700493bea715bfef1d81576940fb67942b05bbac15c06b86bf549d6af3407R295, does the expression passed parse_expression it like a starting point for other expressions which are generated and explored?
It would be very helpful on sketching a concrete example on how parametric equations would be potentially used with SymbolicRegression.jl (for my understanding).
It would be great if you could give some pointers on what things to change, specifically where to change AbstractExpressionNode to AbstractExpression as iiuc, for evaluation, it would still be converted to a normal Node like in https://github.com/SymbolicML/DynamicExpressions.jl/pull/73/files#diff-22d700493bea715bfef1d81576940fb67942b05bbac15c06b86bf549d6af3407R219. I can look into it and make a PR. (I am still getting familiarized with the codebase 😅 )

#326 should implement this


Minor typo fix in readme: "advance" -> "advanced"

I noticed this small typo in a readme example.

Thanks again!


Generated classes clash with existing classes.

Hi,
I have 2 classes defined in my app, PointOfInterest and PointOfInterestType, however when using requery for my persistence layer the classes it generates clash with my existing PointOfInterestType.
I'm using AutoValue if that makes a difference, but I was wondering if there was a way to have the generated classes added as inner classes of my existing models, or if I could put them in a model/requery sub-package.
Thanks.

Hi you can override this generated class name using @Entity(name = "PointOfInterestDef") or similar alternate name. For AutoValue requery defaults to suffixing 'Type' onto the class name which collides with your existing class.

Will that also rename my database table or will that still be based on the class name of my entity?

Nope that doesn't effect the table name, just the generated class name


What are the modifications?

Can you please list the modifications that you made here? it would be really helpful to know what they are...
Thanks

Hi, this is actually an uncompleted project. Please go to Jianwei Yang's repo if you are looking for the pytorch version of faster rcnn.


Virtual paging for sources that don't respect the "images per page" setting

On some sources (or some APIs), it is not possible to set the number of images per page. An example is the Zerochan RSS API, which always return 100 images.
If the "images per page" setting is set to 20 for example, Grabber could simply load the first Zerochan page, and use it for the 5 first "virtual pages", each containing 20 images.
⚠ Some sources return exotic images count (37, 42, etc.), so it might not always be divisible by the images per page setting. In those cases, Grabber might need to load two pages to satisfy the setting. This might need to be disable-able.
If the user's "images per page" setting is higher than the number of images returned, Grabber might also need to load more pages, until there are enough images.
Example with 100 images returned and 20 images per page

1: page 1, 1-20
2: page 1, 21-40
3: page 1, 41-60
4: page 1, 61-80
5: page 1, 81-100
6: page 2, 1-20
...

Example with 37 images returned and 20 images per page

1: page 1, 1-20
2: page 1, 21-37 + page 2, 1-3
3: page 2, 4-23
...

Example with 10 images returned and 20 images per page

1: page 1, 1-10 + page 2, 1-10
2: page 3, 1-10 + page 4, 1-10
...

Yeah this would be a good idea. Come across this multipele times when I use certain sources

There can be further optional uses for such a feature. Like a setting can make Grabber enforce the image per page setting after post processing. Which will definitely reduce the amount of clicks involved and make the search experience seamless. Not like now that if one has an extensive blacklist, one can end up with much lower amount of images per page.


ItemStack.getData throws UnimplementedOperationException

Is there an existing issue for this?

[X] I have searched the existing issues

Are you using the latest version of MockBukkit?

[X] I am using the latest version of MockBukkit.

Minecraft Version
1.19
Describe the bug
ItemStack.getData calls MockUnsafeValues.toLegacy which throws an UnimplementedOperationException if the material is not a legacy material.
My plugin calls getData because, as far as I know, it is the only way to get the type of potion or book from an ItemStack.
Reproducible Test
new ItemStack(Material.POTION, 1).getData();

Anything else?
No response

Same root cause as #300.