Cargo concatenates rustflags from .cargo/configss in reversed order

Consider this setup:
outer
├── .cargo
│   └── config
└── inner
    ├── .cargo
    │   └── config
    ├── Cargo.toml
    └── src
        └── main.rs

Less specific outer/.cargo/config contains
[build]
rustflags = ["-Ccodegen-units=1"]

More specific outer/inner/.cargo/config contains
[build]
rustflags = ["-Ccodegen-units=2"]

The compiler's command line in this case is
rustc --crate-name inner --edition=2018 'src\main.rs' ... -Ccodegen-units=2 -Ccodegen-units=1

rustc like many other tools prefers the last specified value for the given option to support hierarchical build systems.
With Cargo's flag concatenation behavior this means that less specific flags override more specific flags, which is not a correct behavior.
cc https://github.com/rust-lang/rust/pull/70729#issuecomment-610034489

@petrochenkov (or anyone following this issue), we are looking to change the order of how arrays get merged in #12515. Do you have a sense of the risk that might pose in breaking existing environments?


[USERSNAP] test from dev

Taken at: https://dev-sanchonet.govtool.byron.network/dashboard
Fields:

Add a comment: test from dev

Browser: Chrome 126 (macOS Catalina)
Screen size: 1440x900
Open #109 in Usersnap
Powered by Usersnap.

😲


Add an insertion point inside validation code

When generating validation code, we should add an insertion point within the generated code, so that users can, with their own renderers, insert some custom code, which they wouldn't be able to do with just the standard validation rule protocol.

Resolved.


Render Dashboard as one Arbre context

Currently, Active Admin allows you to create dashboard sections in a grid. This works, but is not very flexible.
The other issue is that the DSL for defining sections can be confusing. It's currently difficult to know what gets executed at render time and what gets executed at boot time.
The solution would look something more like this:
ActiveAdmin::Dashboards.build :namespace => :admin do

  # Arbre context for the dashboard...

  columns do
    column do
      panel "My Dashboard Section" do
        span "Hello World"
      end

      if current_admin_user.super_admin?
        panel "My Super Admin" do
          span "Super Admin Hello!"
        end
      end
    end
  end

  # etc...
end

+1. I would like to see improvements made to control the dashboard layout. for our dashboard, you have to have a large display in order to view the data in our dashboard without a lot of wrapping or data being off-screen (vertically or horizontally).

that looks great, and very flexible too.
looking forward getting this featured. :)
cheers


class="umb-cell-/App_Plugins/DocTypeGridEditor/Views/doctypegrideditor.html"> rendered in the back-office.

src/Umbraco.Web.UI.Client/src/views/propertyeditors/grid/grid.html

Means that we get funky HTML in the class attribute......

 ...

This is not a DTGE problem, but a core problem.


Update install-software.md

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

#signoff

#sign-off


instalation on windows

i could not compile moleking in windows os.
cmake version: 3.26.0
python version: 3.10.9
em anexo, log da instalação
windows.log

I can build C++ with this cli:
cmake -DCMAKE_CXX_COMPILER=C:/MinGW/bin/g++.exe -DCMAKE_C_COMPILER=C:/MinGW/bin/gcc.exe -S ../ -B Build -G "MinGW Makefiles" -DBuild_Python=OFF
when running pip3 install ./MoleKing i got this:
log.txt

Now, MoleKing can be installed on windows. Just type
pip3 install MoleKing


How to reproduce the zero-shot transfer results of LayoutXLM on XFUN ?

Thanks for your excellent work - LayoutXLM.
And I wonder ask that how to reproduce the zero-shot transfer results of LayoutXLM on XFUN.
I have converted FUNSD into XFUN format and trained LayoutXLM-base following  https://github.com/microsoft/unilm/tree/master/layoutxlm#fine-tuning-for-semantic-entity-recognition.
Then, I test the fine-tuned model on Chinese, but  the result is:

Thanks again for your time and patience.

And a further question is that how to reproduce the multitask fine-tuning results on XFUN (fine-tuning on 8 languages all, testing on X) ?
I trained LayoutXLM-base for 8000 steps, and the result on Chinese is:

Hi
I've the same issues for both tasks and especially  when training with all languages : #440
I've tried with various numbers of steps and LR, but still fail to reproduce

@jpWang : did you succeed in reproducing the results?

No hurry on my side. thanks!

@jpWang How did you convert FUNSD into XFUN format? I tried to convert but didn't succeed.

@jpWang How did you convert FUNSD into XFUN format? I tried to convert but didn't succeed.

You can access the data from https://github.com/jpWang/LiLT#datasets.

@jpWang : did you succeed in reproducing the results?

I have reproduced the results on SER task, but not on RE task so far. So I think I will try more optimization strategies on RE and closed this issue.
The reason why I didn't reproduce it before is that there are some bugs in the dataset construction part of my code. Finally I changed the following lines in xfun.py:
                tokenized_inputs = self.tokenizer(
                    line["text"],
                    add_special_tokens=False,
                    return_offsets_mapping=True,
                    return_attention_mask=False,
                )

into :
                if '/en' in filepath[0]:
                    tokenized_inputs = self.tokenizer(
                        ' '.join([q['text'] for q in line['words']]),
                        add_special_tokens=False,
                        return_offsets_mapping=True,
                        return_attention_mask=False,
                    )
                else:
                    tokenized_inputs = self.tokenizer(
                        line["text"],
                        add_special_tokens=False,
                        return_offsets_mapping=True,
                        return_attention_mask=False,
                    )

after converting FUNSD into XFUN format, since there are some missing words in line['words'] compared with line["text"]. I just follow the official optimization strategy on SER task to reproduce the zero-shot transfer results, and change the max steps to 8000 to reproduce the multitask fine-tuning results.

@jpWang  Same here. I still can't reproduce on RE task so far (I make it on SER). Do you fine any more optimization strategies on RE later? Thanks in advance.


Add windows_file.template operation

Mostly copy/pasted from file.template. One tweak is making the IO buffer-type bytes since
winrm connector _put_file expects a bytes input. Therefore output_file was changed from:
output_file = six.StringIO(output)

to
output_file = six.BytesIO(six.ensure_binary(output))

References https://github.com/Fizzadar/pyinfra/issues/645
I have not added tests yet but may be able to add them later.

v3 will remove Windows support (into a separate package, https://github.com/pyinfra-dev/pyinfra-windows. Once that's released I shall move this over and close this PR.


[FR] -   CLI - Hot key update without restarting node (e.g. using SIGHUP)?

There’s a proper scheme that should be implemented...

+1

Why is it low prio? How can we make sure block-producing node runs uninterrupted without this feature?

Any chance of an update on this Feature Request front? It's been 84 years...

Closing this. If this is still relevant please re-open.


500 and 520 errors when using Short Code for authentication

Short code authentication works once, and fails so many times that it's basically impractical to use it. Any idea why?
EDIT: To be specific, the code is thrown at ChatClient.Connect() method. Other than that it works as intended

This is a Mixer-side issue unfortunately, it's been happening for the last month or two. It's also not specific to short code authentication, it'll occur using standard OAuth as well. You'll see either a 500, 520, or 525 error.
I've reported it to the Mixer devs multiple times, but they haven't seem to come up with a solution to it. Our current guess is this is related to the Spectre & Meltdown vulnerability updates that were deployed out to Azure, which could be causing some CPU over usage that the Mixer web services are not accounting for.
While they still attempt to figure out what's going on, my simple work around has been to disconnect, delay 1000 milliseconds, then re-connect. Sometimes it works on the first try, sometimes it takes a few. I've rarely seen it take more than 5 at most though.

Which part to retry exactly? Just the connecting to chat part or authorization as well?

Just the Chat Client connection. Of all the web sockets connections, only Chat has these issues.

Thank you. That should do it


deps: bump celestia-app to v0.13.0

This bumps celestia-app to v0.13.0. I will cut a new release when this is merged.

hm, @jrmanes do you know how we can work around this and this error? I think the docker publish workflow not being backported to 0.13.x may be the culprit here

Hey @jcstein
I've checked the issue and as you said, the problem is that the image doesn't exist for arm64 (and the pipeline is trying to pull that image), because the version v0.13.0 doesn't contain the fix yet...
There's nothing we can do until the fix will be added... 😓

Node v0.9.2 is still using app v0.12.0. It would be compatible with v0.13.0 but to avoid confusion let's wait for a release from node that uses app v0.13.x.

Sounds good, thanks team!


Ability to see what recipes an item is consumed / produced by

Potentially show this in the recipes tab in step details, or somewhere else?
Requested by kellbot in Discord

Superceded by #929


react native 0.56 not working in ios

before i was using react native 0.55 and it was working perfect but today i upgraded to 0.56 and this is no more working in ios only, android is still working as usual but it seem ios has broken :)

@kunaldodiya which version of 0.55 did you use? It seems that i'm facing the same problem...
Everything works wel... after I kicked back from the browser to the app nothing happend...
the then function doesn't fired.
My specs:
react-native-cli: 2.0.1
react-native: 0.55.4

I am using 0.56.* react native, and issue has been resolved by updating gradle to 4.4.1, closing issue, feel free to reopen if you face any problem

Grandle? Where can i find that?

@dennislaupman
you can find gradle version in file :- android/gradle/wrapper/gradle-wrapper.properties
change
distributionUrl=https://services.gradle.org/distributions/gradle-3.5.1-all.zip
to
distributionUrl=https://services.gradle.org/distributions/gradle-4.4.1-all.zip

Agggrrr.... :P adding this to the AppDelegate.m:
- (BOOL)application:(UIApplication *)application             openURL:(NSURL *)url             options:(NSDictionary<UIApplicationOpenURLOptionsKey,id> *)options {   return [RCTLinkingManager application:application openURL:url options:options]; }
did the trick ;)
The upgrade to 0.56 replaced the AppDelegate.m with a newer version,
A lot of thanks for your help!


chore: setup CI for test framework

This sets up the initial trigger and folder for bpt CI. Will add the build steps for running tests in a follow up.

This is applied

@bharathkkb - I think this should have triggered for ( I don't see it) https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/pull/1629?

@bharathkkb - I think this should have triggered for ( I don't see it for push a few minutes ago ) #1629?

I saw the trigger was disabled, which a fresh apply changed.


Extended number formatting

Fixes #264.
TODO:

[x] Add format specifiers for strings (and possibly other types?)
[x] Fix todos
[x] Check that format specifiers are valid (currently anything invalid will crash Numbat)
[x] Fix units not being printed in quantity interpolations with format specifiers
[ ] Research possible choices other than strfmt
[x] Add tests
[x] Document
[ ] Improve error messages

Thanks!

Changes are live on https://numbat.dev/


IntervalUnit cannot handle microseconds

Describe the bug
In PostgreSQL, the interval type supports microsecond precision. However, our IntervalUnit implementation only stores milliseconds.
To Reproduce
Steps to reproduce the behavior.
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

Maybe we can consolidate our IntervalUnit implementation with pg_interval::Interval, which is introduced by #5646


Ownership change events after setup within same tx not being picked up

We are experiencing issues deploying Safe instances through our factory contract. Here is an example:
https://gnosisscan.io/tx/0x66425f06902a0ffc50d041599e60830d94d666c433e603095d03fca5897673b7#eventlog
The problem is that the tx service endpoint /owners/MY_ACCOUNT/safes won't return the newly created Safe. We tracked down the issue further and believe the problem lies in the switch of ownership during the deployment. This does not seem to be picked up by the indexer and it continues to "believe" the Safe is owned by our factory contract.

To give a little bit more context: When a safe proxy is created with createProxyWithNonce, the sole owner is set to the msg.sender which is a factory contract (StakeFactory). Within the same transaction, after a few other internal transactions, the owner of the safe proxy is swapped to a different EOA address.
Given the owner of the safe proxy is a contract, the signature gneration follows EIP-1271, i.e. approveHash then execTransaction. Payload of the swapOwner safe transaction is thus encoded in execTransaction.
What we've observed is that, for the said proxy creation call, in the safe-transaction-service, only setup and approveHash internal calls got indexed.

Maybe is related with the contract version, we are currently working adding support for 1.4.1 version https://github.com/safe-global/safe-transaction-service/issues/1433

As @moisses89 states, it would be important also to know what versions are you using of the contracts, as service should not have any problem detecting owner swap

Could you provide more feedback about this issue?

Hi there. Sorry that the thread got lost. We use contracts of v1.4.1

Sorry for the delay guys. I see that the issue is resolved now so I will go ahead and close the ticket. Please let us know if you are still experiencing any issues 🙏


Incorrect shortcut styling for dark-only navbar

The shortcut feature added by https://github.com/easyops-cn/docusaurus-search-local/pull/111 styles the search hint (CTRL+K) incorrectly when the navbar style is "dark":

Fixed in v0.21.2.

Awesome, thank you very much!


xgboost with 'rank:pairwise' crashes R session

Operating System:
Windows 8.1 64bit
Package used:
R
xgboost version used:
0.4.4
The R sessionInfo()
R version 3.3.1 (2016-06-21)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows >= 8 x64 (build 9200)

locale:
[1] LC_COLLATE=Polish_Poland.1250  LC_CTYPE=Polish_Poland.1250    LC_MONETARY=Polish_Poland.1250 LC_NUMERIC=C
[5] LC_TIME=Polish_Poland.1250

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] xgboost_0.4-4       RevoUtilsMath_8.0.3

loaded via a namespace (and not attached):
 [1] magrittr_1.5     Matrix_1.2-6     RevoUtils_10.0.1 tools_3.3.1      stringi_1.1.1    grid_3.3.1       data.table_1.9.6
 [8] stringr_1.0.0    chron_2.3-47     lattice_0.20-33

The command to install xgboost if you are not installing from source:
install.packages("xgboost")
Steps to reproduce:
library(xgboost)

dmat <- xgb.DMatrix(data = as.matrix(iris[, -5]),
                    label = as.numeric(iris$Species),
                    group = 150)

model <- xgboost(data = dmat, nrounds = 10,
                 params = list(objective = "rank:pairwise"))

What have you tried?
Nothing really helps.
Worth noting:
Currently I am using Revolution R Open version of R, but the same problem exists in GNU R. There's no additional info before the crash.

I've also checked that the code above works in R on Linux, with the following sessionInfo():
` R version 3.3.2 (2016-10-31)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 14.04.4 LTS
locale:
[1] LC_CTYPE=pl_PL.UTF-8       LC_NUMERIC=C               LC_TIME=pl_PL.UTF-8
[4] LC_COLLATE=pl_PL.UTF-8     LC_MONETARY=pl_PL.UTF-8    LC_MESSAGES=pl_PL.UTF-8
[7] LC_PAPER=pl_PL.UTF-8       LC_NAME=C                  LC_ADDRESS=C
[10] LC_TELEPHONE=C             LC_MEASUREMENT=pl_PL.UTF-8 LC_IDENTIFICATION=C
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] xgboost_0.4-4
`
This is not the different version issue (checked).

The version 0.4-4 in cran is very outdated.

Indeed, the xgboost version was the problem. Taking this into account I believe the issue can be now closed.
I have additional question, though. Is there any CRAN-like reliable source of newer, already-compiled versions of xgboost? I'm asking because I will most likely be unable to compile it on my corporate laptop.


bug? lr command line argument is ignored for YF and instead 1.0 is used

In the line https://github.com/JianGoForIt/YellowFin/blob/master/char-rnn-tensorflow/model.py#L92
the lr is set to 1 and not to the command line argument value.
Later in https://github.com/JianGoForIt/YellowFin/blob/master/char-rnn-tensorflow/train_YF.py#L138
the learning is set to the command line argument value but for YF this has no effect because the connection between the variable model.lr and YF was never made (for Adam and SGD this will work because model.lr is passed as the learning rate)

Hi @udibr Thanks for checking this out, but it is not really a bug. In the experiments for our paper, we uniformly use lr = 1.0 as the initial learning rate, i.e. we compared our no-tuning results to grid-searched adam and momentum SGD.
Could you please issue an pull request to wire it up?
Thanks,
Jian

Hi @udibr, we have updated the command line interface for char rnn experiment, please check them out.


No issue, just thank you!

I've been missing a causal inf. estimation package in Julia for a while -- I was so excited to find your package! Thank you 🥳

Thank you for the positive feedback, much appreciated!


fix(ffe-datepicker-react): Do not overwrite aria-describedby if given…

I have a use-case where it would be helpful to be able to set the aria-describedby on the input field wrapped by Datepicker, but the current implementation overwrites this field without checking the inputProps. This commit adds a check, making sure the Datepicker won't overwrite  aria-describedby if it was passed down as a prop.

Coverage increased (+0.02%) to 81.943% when pulling 419b0061ce417a605f88c65383ad3972e664e592 on Snikanes:do-not-overwrite-aria-describedby into 1015b95d3023bac68dd66c0c58fd3e3e966a0802 on SpareBank1:develop.


Expose a way to read an object given an id

Exposes a method on ApolloClient that allows you to read an entire object given its id without worrying about how that object has been normalized within the store. See #474 for some reasoning behind this.
TODO:

[x] Update CHANGELOG.md with your change
[x] Make sure all of the significant new logic is covered by tests
[x] Rebase your changes on master so that they can be merged easily
[x] Make sure all tests and linter rules pass

I feel like this method should take a graphql fragment as an argument, so that you can read nested data originating at this location in the store, and you can get a result in the result format rather than serialized arguments, ID references, etc.

@stubailo After thinking about using fragments vs. this approach, I agree with going w/ the fragment-based approach.
For the sake of recording the decision, I'll describe why the current approach doesn't make sense. Say you fire queries that look like this:
query($friendId: Int) {
   user {
     name
     friend(id: $friendId) {
       name
     }
   }
}

If this query is fired for multiple friendIds and then we do a readObjectById on the id given to a particular user, it is totally non-obvious how we would go about combining the different "friend" values that exist if we want the object that's read to look like a GraphQL result. You can imagine putting them all into a list in this case, but this gets more complicated if there's an argument to friend like type (e.g. CLOSE or DISTANT as enum values). This means that constructing a result object out of these is pretty application-specific.
On the other hand, if we use a fragment, we can avoid all of this pain. For the original example (mentioned in #474 ), we can still do something that looks like:
client.readObjectFromId(id, gql`
fragment {
  name
  friends {
    name
  }
}`);

This will reach into object with the id of id and resolve all of the fields that we want.
I'll fix up this PR accordingly.

If I'm reading the patch correctly, it doesn't appear like this PR reads anything beyond the data (NormalizedCache) portion of Apollo's state?
Which, I think, means this method won't reflect any optimistic updates? (it probably should!)

I assume that optimistic updates probably don't have the same ids as the values returned from the server in most cases - given this, it probably doesn't make sense to roll both of them into the same method.

I assume that optimistic updates probably don't have the same ids as the values returned from the server in most cases - given this, it probably doesn't make sense to roll both of them into the same method.

We often do. For example, updates to existing/known entities (the current user's preferences, optimistically changing the state of some already retrieved resource, etc)

This could be obviated by #617

I'm going to close this for now, we should take care of it in the refactor.


Attribute formatting breaks when not wrapped with "

These are expected to be formatted the same way, but the formatting is broken when the attribute is not wrapped with ". The HTML specs allow bare attributes (without " wrapping), so the first example is valid, and should be formatted correctly.

@mjbvz I tried to see if I could find out what's going on here, but my lack of experience build syntax definitions is slowing me down massively. I've been able to find the relevant parts of the HTML and TypeScript syntaxes that cover unquoted attribute values and template substitutions, but I can't figure out why the HTML definition is taking precedence, and can't find a way to introspect the scopes or tokens. I also can't find great documentation on controlling precedence and injections.
Any guidance? Thanks!

I think this part tracks just the formatting. Highlighting is tracked by https://github.com/mjbvz/vscode-lit-html/issues/46
You can use the inspect scope command to help debug highlighting issues

oops. Thanks for the tip!


Statsd-go now returns any errors

This adds a variable for that and panics on any errors

thanks!


Massive amount of memory leaks in Framework

I'm seeing anywhere from 30 to 200+ memory leaks from Parse IOS framework, there's too many to list so here's a screenshot. Let me know if you're seeing the same sort of thing. I just starting seeing this after IOS 10. Before IOS10, I never saw leaks like this, not sure if it's my doing something wrong or the framework itself. Let me know if you've seen the same thing. I can link more screenshots of the guts if need, but here' s a couple of shots.

Seeing the same thing and noticing a lot more low memory situations in my apps' logs.


Run octoscan as part of Flowzone workflows

Change-type: minor
See: https://balena.fibery.io/Work/Improvement/Run-octoscan-as-part-of-Flowzone-workflows-2384

230 warnings... most of them seem actually quite unhelpful in that they are core functionality and there is no other way to solve the problem.
https://github.com/product-os/flowzone/security/code-scanning?query=pr%3A1262+is%3Aopen


bug(nvidia-container-toolkit): incorrect containerd config

What happened:
A bug in nvidia-container-toolkit 1.17.0 results in a containerd config that omits some EKS defaults. Some symptoms we're aware of:

The pause container image used by containerd will be the default (registry.k8s.io/pause:3.5) instead of the regional ECR image that EKS provides. If your nodes have network access to registry.k8s.io, this will work fine. If your nodes can't reach registry.k8s.io, your nodes will not be able to create pods.
The cgroup driver used by containerd will be incorrect, resulting in your pods being placed in the wrong part of the cgroup tree. This may impact your telemetry.

The bug has been fixed in 1.17.1 and EKS will release an AMI with this version of the nvidia-container-toolkit as soon as possible.

Bug: https://github.com/NVIDIA/nvidia-container-toolkit/commit/a06d838b1cd296547fa9446314aed0eca98152fd
Fix: https://github.com/NVIDIA/nvidia-container-toolkit/commit/1995925a7df644ead7afb767608841d9a08bcbc4

not sure if it's related. but since we updated from AL2 v20241106 tov20241024 we found that the  /etc/containerd/config.toml is totally changed, and it removes the
 [plugins."io.containerd.grpc.v1.cri".registry]
      config_path = "/etc/containerd/certs.d:/etc/docker/certs.d"

which causes the new nodes to not be able to pull containers from private registries we have set up.

The release for this bug fix was completed
https://github.com/awslabs/amazon-eks-ami/releases/tag/v20241109


Big file upload (and download) runs into 10 sec timeout

Reproduce:

Run ocis server
Login to webui
Create big file fallocate -l 6G bigfile.bin
Click +New => Upload File => bigfile.bin
Upload Fails :boom:

File with 1 GB works.
Relevant Loglines
6:00PM ERR home/ilja/go/pkg/mod/github.com/cs3org/reva@v0.0.2-0.20200115110931-4c7513415ec5/internal/http/services/owncloud/ocdav/put.go:219 > error doing http request error="Put http://localhost:9156/data/albert/6g.bin: net/http: request canceled (Client.Timeout exceeded while awaiting headers)" pid=26242 pkg=rhttp traceid=c37834485d7339d24ae135da647d08ce
6:00PM ERR home/ilja/go/pkg/mod/github.com/cs3org/reva@v0.0.2-0.20200115110931-4c7513415ec5/internal/http/interceptors/log/log.go:112 > http end="24/Feb/2020:18:00:25 +0100" host=::1 method=PUT pid=26242 pkg=rhttp proto=HTTP/1.1 size=0 start="24/Feb/2020:18:00:15 +0100" status=500 time_ns=10020856613 traceid=c37834485d7339d24ae135da647d08ce uri=/remote.php/webdav/6g.bin url=/remote.php/webdav/6g.bin
6:00PM ERR home/ilja/go/pkg/mod/github.com/cs3org/reva@v0.0.2-0.20200115110931-4c7513415ec5/internal/http/services/dataprovider/put.go:39 > error uploading file error="ocFS: error writing to tmp file /var/tmp/reva/data/albert/files/._reva_atomic_upload556955768: unexpected EOF" pid=26365 pkg=rhttp traceid=c37834485d7339d24ae135da647d08ce

@IljaN we need to configure by config the HTTP client timeout

reva currently uses an http client that has the timeout set to 10 sec: https://github.com/cs3org/reva/blob/0dbce24f7e8b508c1ecab2d145979562888a7a76/pkg/rhttp/client.go#L37
We need to increase that or better yet, make it configurable for the ocdav up and download.

Is the issue resolved? Cause I got this while trying with 10GB file upload:

P.S.: Sync with desktop client is also working well with such large files.
Server Configurations

latest ocis master


Infinite recursion in ModelUtil.emptyMeet

Run the IssuesTests_0500_0999.testBug796 test to reproduce
	at com.redhat.ceylon.model.typechecker.model.ModelUtil.emptyMeet(ModelUtil.java:1653)
	at com.redhat.ceylon.model.typechecker.model.ModelUtil.emptyMeet(ModelUtil.java:1624)
	at com.redhat.ceylon.model.typechecker.model.ModelUtil.emptyMeet(ModelUtil.java:1653)
	at com.redhat.ceylon.model.typechecker.model.ModelUtil.emptyMeet(ModelUtil.java:1624)

The issue here is that the family type E declared by Bug796_Graph<N,E> is not marked as a self type, where line 1624 expects that. I'll have to dig a little further to figure out whether it should be or not.

@FroMage I believe this is a model loader bug: Edge gets its selfType set to E, but E doesn't get its selfTypedDeclaration set to Edge.


Run ModelChain from POA irradiance

[x] Closes #536
[x] I am familiar with the contributing guidelines
[x] Tests added
[x] Updates entries to docs/sphinx/source/api.rst for API changes.
[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

Provides methods to run a ModelChain starting with broadband plane-of-array irradiance or effective irradiance, and with cell temperature or module temperature. Creates a number of helper functions for these two methods.

Does this close #536 ?

Does this close #536 ?

Not for the use case of a broadband instrument in the plane of array (e.g. pyranometer). Yes for the use case of a plane-of-array reference cell (with similar spectrum and surface reflectance).

Not for the use case of a broadband instrument in the plane of array (e.g. pyranometer). Yes for the use case of a plane-of-array reference cell (with similar spectrum and surface reflectance).

For the case of POA measured by pyranometer should we recommend using GTI-DIRINT to back-estimate the DNI & DHI components before using ModelChain or should there be a model chain that calls GTI-DIRINT under the hood given pyranometer-POA input?
Either way, I believe the documentation for ModelChain should clarify the point you make regarding POA from reference cell vs. pyranometer.

Thoughts on 1. the current PR's approach of adding e.g. run_model_from_poa methods vs. 2. subclassing ModelChain into e.g. ModelChainPOA and overriding run_model ?

Not for the use case of a broadband instrument in the plane of array (e.g. pyranometer). Yes for the use case of a plane-of-array reference cell (with similar spectrum and surface reflectance).
For the case of POA measured by pyranometer should we recommend using GTI-DIRINT to back-estimate the DNI & DHI components before using ModelChain or should there be a model chain that calls GTI-DIRINT under the hood given pyranometer-POA input?

I'm hesitant to build that POA to DNI etc. to POA components translation into the method. We can inform the user that the GTI-DIRINT function is available, but they may have other information (e.g. DHI from a nearby weather station) and can find their own path to POA components.

Either way, I believe the documentation for ModelChain should clarify the point you make regarding POA from reference cell vs. pyranometer.

Absolutely. I think the docstrings for these methods should have an Example section.

Good points. ModelChain is really a mess.
The primary ModelChain entry points (run_model, prepare_inputs, and complete_irradiance) all expect a single DataFrame with all relevant information. These methods don't require any other time series information from ModelChain attributes. I think we should keep this design pattern. Some historical context may help: ModelChain.run_model originally included three keyword arguments: times, irradiance, and weather. Over several releases, we've reduced it to just weather. Let's not go in the opposite direction. Let's also not start to rely on users setting explicitly attributes such as weather - no other pvlib methods require this and it's generally not a pattern I want to encourage.
Keeping that single input DataFrame design pattern while allowing for the input of were previously intermediate modeling results certainly raises issues with how data is grouped into attributes and how those attributes are named. My proposed solution for that is to generalize the DataFrame argument name to something like data, and then assign parts of data to the relevant ModelChain attributes. I'm open to a broader rethink, but it's difficult to see how that could be done cleanly without breaking a lot of user code.

@wholmgren where would we retain the intermediate outputs such as effective_irradiance? As new columns in data and as ModelChain attributes?  I'm not sure I see the value in adding columns to a dataframe that is just passed as input to other functions, maybe collect all the attributes into a data attribute which is the Dataframe? We would reserve weather for external data that is passed into ModelChain methods, separate from quantities generated by ModelChain methods.
weather would contain ghi, dni, dhi, temp_air, wind_speed, precipitable_water, relative_humidity.
ModelChain.data would be a dataframe with everything else?

Sorry for being unclear. I am not in favor of modifying the input DataFrame. I'll try to be more concrete:

run_model(weather) --> No changes. We could, for consistency with new methods, rename weather to data.

run_model_from_poa(data) --> ModelChain selects known weather data (air temperature, wind speed, ghi, dni, dhi, precipitable_water, relative_humidity, more?) from data, stores on self.weather. Selects required POA columns from data and saves on self.total_irrad. Compute effective irradiance, module temperature, power as normal.

run_model_from_effective_irradiance(data) --> ModelChain selects known weather data stores on self.weather. Extracts known poa data and stores on self.total_irrad. Selects required effective_irradiance from data, stores on effective_irradiance. Compute module temperature, power as normal.

We could also assign a self.data but it's not required and I'm not sure if it's a good or bad idea.

run_model_from_poa(data) --> ModelChain selects known weather data (air temperature, wind speed, ghi, dni, dhi, precipitable_water, relative_humidity, more?) from data, stores on self.weather. Selects required POA columns from data and saves on self.total_irrad. Compute effective irradiance, module temperature, power as normal.

Working on this again. "selects known weather data" implies that we hardcode a list of keys that will comprise the weather dataframe attribute. This strikes me as difficult to maintain; when a new function is added somewhere (e.g. spectral model) which requires a new weather column, we'll have to add a key to this this list.
What about weather always, and only, containing weather-related quantities external to pvlib functions? Assign to ModelChain.weather to preserve the input. Then run_model_from_poa would have two arguments: data, from which poa_global etc. is pulled, and weather which gets stored on ModelChain for downstream functions.
To follow the rule "don't modify input dataframes" we shouldn't be filling in temp_air and wind_speed with default values if they aren't found in weather.

"selects known weather data" implies that we hardcode a list of keys that will comprise the weather dataframe attribute. This strikes me as difficult to maintain; when a new function is added somewhere (e.g. spectral model) which requires a new weather column, we'll have to add a key to this this list.

This is already the case. For example, air pressure in solar position
https://github.com/pvlib/pvlib-python/blob/49da0318256d8b46f90d4b29a7023de680f8410b/pvlib/modelchain.py#L884-L888
precipitable water in the first solar correction
https://github.com/pvlib/pvlib-python/blob/49da0318256d8b46f90d4b29a7023de680f8410b/pvlib/modelchain.py#L646-L650
I don't see how two DataFrame inputs improves the situation. I think it makes it worse since now I have to worry about if I am putting the data in the right place too.

To follow the rule "don't modify input dataframes" we shouldn't be filling in temp_air and wind_speed with default values if they aren't found in weather. Not sure I see how to work around this yet, looking for feedback about weather first.

Correct. I think we can fix this with a weather.copy()

Would you favor a new constant listing the permissible keys for weather?

Would you favor a new constant listing the permissible keys for weather? Similar to pvlib.pvsystem._DC_MODEL_PARAMS etc.

Yes, good idea.

@wholmgren see if this is closer to what you had in mind. Since the SPI use case also anticipates entering ModelChain with different temperature data (air, back-of-module, or cell), I've added a prepare_temperature function to deliver cell_temperature to a ModelChain instance. Feels rather hacky with the various try's and if's.

Test is unrelated. To do, perhaps in follow-on PRs:

[ ]  docstrings for new ModelChain constants
[ ]  implement fallback to effective irradiance for cell temperature models, if ModelChain.total_irrad['poa_global'] is absent

@cwhanse can you make issues for the follow up work?

@srlightfoote FYI if you are still watching.

thanks all!  I think this feature will help ease the burden for folks wanting to use pvlib to model expected performance characteristics of operational plants where pyranometer sensors mounted at plane of array are the norm.

thanks all! I think this feature will help ease the burden for folks wanting to use pvlib to model expected performance characteristics of operational plants where pyranometer sensors mounted at plane of array are the norm.

@slightfoote that's the motivating use case. You may be interested in one of our current projects to build a modeling service for O&M use cases. There's some information at solarperformanceinsight.github.io

Thanks @cwhanse!


Fix H5FD_lock error when writing to hdf with distributed client

When writing a single HDF with multiple datasets, it is possible to get a concurrency error even when locking is enabled (lock=True), if the cluster is using processes. When we have a distributed client with processes, it feels logical that a lock from multiprocessing should be used. I don't have enough experience in this area of code, so I may be missing something here.

[x] Closes https://github.com/dask/dask/issues/9972
[x] Closes https://github.com/dask/dask/issues/10315
[x] Tests added / passed
[x] Passes pre-commit run --all-files

@hendrikmakait @graingert To my knowledge, this is ready to go, if you can take a look.

cc @hendrikmakait

Failing test is unrelated.
            with warnings.catch_warnings():
                # Suppress RuntimeWarning about All-NaN slice
                warnings.filterwarnings(
                    "ignore", "invalid value encountered in cast", RuntimeWarning
                )
>               base = data.astype(np.int64)
E               RuntimeWarning: invalid value encountered in cast


javaagent bean creation exception

hello! I'm trying to attatch pinpoint-agent into my Spring Boot Application .
java -jar -javaagent:/pinpoint-agent-2.3.3/pinpoint-bootstrap-2.3.3.jar -Dpinpoint.agentId=test -Dpinpoint.applicationName=toyou -Dpinpoint.config=/pinpoint-agent-2.3.3/pinpoint-root.config -Dspring.profiles.active=prod toYou-0.0.1-SNAPSHOT.jar
But, when I add -javaagent option, a bean creation exception occurs. (pinpoint agent was normally started)
If I only use ,  java - jar toYou-0.0.1-SNAPSHOT.jar, application normally start
spring -boot 2.5.6
openjdk:11

There's no prod profile provided by us. Did you create it on your own?
Please share full exception logs.

Thank you for your reply. The problem has been resolved now and it was my mistake. There was no prod in the profile but I gave the prod option…

Thank you for your reply. The problem has been resolved now and it was my mistake. There was no prod in the profile but I gave the prod option…


exception while running lexvec

I'm experimenting with building lexvec and running it.

go get github.com/alexandres/lexvec
go build

./lexvec -threads 64 -corpus 59m.txt -output lexvectors
.....
cds total: 67097075.384447
calculating ppmi matrix
create vectors
running lexvec
panic: nan
goroutine 102 [running]:
main.learn(0xc420210a20, 0xc447a48b40, 0xc424326d70, 0x5f7720, 0xc42132e570, 0xc424648000, 0x0, 0xc4a465a000, 0x12c, 0x12c, ...)
/var/lib/jenkins/workspace/lexvec/lexvec.go:319 +0x4a0
main.main.func3(0xc4200ce330, 0xc4200ce370, 0xc4200ce310, 0xc4200ce2a0, 0xc6294e9980, 0xc449fb4200, 0x40, 0x40, 0xc449fb4000, 0x40, ...)
/var/lib/jenkins/workspace/lexvec/lexvec.go:1022 +0x1929
created by main.main
/var/lib/jenkins/workspace/lexvec/lexvec.go:1056 +0x2f22

same thing happens with pre built binary
also note that this used to work 2016-08-10 with the same input file.
lexvec_v1.0.2_linux_amd64/lexvec -threads 64 -corpus 59m.txt -output lexvectors
build vocab
2370000
creating positional vocab words
vocab size: 103011
context vocab size: 412040
corpus size: 996025910
raw corpus size (only valid when constructing vocab):<PHONE_NUMBER>
initializing cooc storage
identify coocurrence
1009822000
cds total: 67097075.384447
calculating ppmi matrix
create vectors
running lexvec
panic: nan
goroutine 107 [running]:
panic(0x571a60, 0xc8a30fbf50)
/home/alex/go/src/runtime/panic.go:464 +0x3e6
main.learn(0xc82020db30, 0xc847f229f0, 0xc831a191d0, 0x7f7df5bff0b8, 0xc82604fb30, 0xc84c450000, 0x0, 0xc84c454000, 0x12c, 0x12c, ...)
/home/alex/goworkspace/src/bitbucket.org/atsalle/lexvec_release2/lexvec.go:319 +0x6fd
main.main.func3(0xc8200c0660, 0xc8200c06f0, 0xc8200c05e8, 0xc8200c0520, 0xc870678e60, 0xc87b67c200, 0x40, 0x40, 0xc87b67c000, 0x40, ...)
/home/alex/goworkspace/src/bitbucket.org/atsalle/lexvec_release2/lexvec.go:1035 +0x1e2c
created by main.main
/home/alex/goworkspace/src/bitbucket.org/atsalle/lexvec_release2/lexvec.go:1056 +0x3d8b

So using https://github.com/alexandres/lexvec/releases/tag/v1.0.1 you don't get the nan?

@jave : I just identified a bug that could be causing your issue. Can you clone master and build or download release 1.0.3 and let me know if it corrects your nan?

The issue appears fixed now thanks!


ensuring that tags does not show in branches

Fixes issue #63 tags shows as a branch when issuing
git name-rev --name-only HEAD

On a tagged commit

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

This is still an issue when using git 2.17.1. (from a fresh install of Ubuntu 18.04)
Is there a way to move this forward?


Check binary files that might be wrongly commited

You should grep 'application/[^/]*$' to avoid false positives (application-server.nit: text/plain…). But, even with this, you may have problems with the fact that the application namespace is not only for binary files: it is an “everything else” category. It includes things like programming languages (a subject of debate), mail formats, etc. Furthermore, your script does not look for binary image files, video files, audio files, etc.

Heuristic of Git:

builtin_diff() calls diff_filespec_is_binary() which calls buffer_is_binary() which checks for any occurrence of a zero byte (NUL “character”) in the first 8000 bytes (or the entire length if shorter).
— http://stackoverflow.com/a/6134127

$ grep -Paq '\x00' -- bin/nitc; printf '%d\n' "$?"
0
$ grep -Paq '\x00' -- src/nitc.nit; printf '%d\n' "$?"
1

Works with GNU Grep. I don’t know how to do the same with POSIX Grep, though.

+1

test this please


Afront: new adapter doc

🏷 Type of documentation

[x] new bid adapter
[ ] update bid adapter
[ ] new feature
[ ] text edit only (wording, typos)
[ ] bugfix (code examples)
[ ] new examples

📋 Checklist

[x] Related pull requests in prebid.js or server are linked -> https://github.com/prebid/Prebid.js/pull/10264
[ ] For new adapters check submitting your adapter docs

Closed as related PR was closed


Look for .env files in the parents of cwd, up to root of current git repo

Component
Forge
Describe the feature you would like
Feature description
Mimic the default behavior of the --root CLI argument for loading a .env file and consider up to the root directory of the current git repository in addition to cwd and project root.
Context
Since #2604, the .env file is only read from 2 places (cwd & project root).
When a foundry project is used as a dependency, invocations that need to use a local .env file will do e.g. forge script --root path/to/dependency. But the .env file may not be in the cwd! It may be located in the root directory of the current git repository.
Example:
.
|- .env
|- .git
|- dir
   |- script.js # invokes forge script

./dir$ script.js # .env is one level above

Here, looking one level above would load the current repo's .env file.
Additional context
No response

this is implemented here, which loads from cwd + (dir(foundry.toml) | gitroot))
https://github.com/foundry-rs/foundry/blob/32200ac58f579f8bf2fa022a2b45fc3c5a13297b/cli/src/utils.rs#L178-L187
perhaps this should do dir(foundry.tml) + gitroot instead?

Thank you for pointing me to the code, I was mistaken about what forge does.
For my use case, the currentb behaviour is enough (the important part is cwd + gitroot). It may be good to do cwd + dir(foundry.toml) + gitroot anyway though -- I can imagine needing that in the future.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I have experience with JavaScript and Python, along with a solid understanding of Solidity. My background in open-source projects equips me to effectively implement features and contribute to documentation. I can leverage my skills to enhance the Foundry toolset, ensuring clear communication and functionality for developers.
How I plan on tackling this issue
To approach the problem, I would first locate the root directory of the current git repository by checking for the presence of the .git folder. Then, I would traverse the parent directories starting from the current working directory to search for a .env file up to the git root. If found, I would load its contents into the environment. Additionally, I would write tests to ensure that this functionality works as expected and update the documentation to clarify the new behavior. This implementation would enhance the Forge tool's ability to load .env files in various project structures.


add setup.py command to create git tag

and easy publish on pypi:
first generate packages and sent to pypi:
python setup.py publish
after, create a gittag and push to github:
python setup.py tag
bdist_wheel require wheels package was installed. You probably should not worry about it because pip> = 7.0 install the wheels package by default
The zip_safe=False flag can be used to force or prevent zip Archive creation. In general you probably don’t want your packages to be installed as zip files because some tools do not support them and they make debugging a lot harder.

Very useful, thanks.

@aykut thanks for this project, saved my life.
This package has officially support Python 3?
if yes, it is good to include the trove classifiers:
https://github.com/django/django/blob/master/setup.py#L61-L66

It is supposed to support Python 3, I've included Python 3.4 in test environments. I'll add 3 and 3.3 to travis.yml

include in the tests is good, but I was talking to include python3 trove classifiers in setup.py


Fixes #4199

Requirejs optimize no longer uglifies
Now using uglify-es and grunt-contrib-uglify-es instead of requirejs

we still have
'fileExclusionRegExp': /^(\.|<%= target %>|tests|tools|grunt|optimist|properties-parser|readdirp|underscore$|shelljs$|oae-release-tools|benchmark|robots\.txt)/,

where I'd try and remove these grunt|optimist|properties-parser|readdirp|underscore$|shelljs$|oae-release-tools|benchmark but I'm not sure that's a good idea. I just removed the ones that have been added during 2017 ±, I think I'll leave the others for another time, do you agree?

Yeah, agreed. Grunt, optimist, oae-release-tools etc won't be used while the app is running anyway.


feat: Implement Shell Sort Algorithm in TypeScript

Pull Request: Implement Shell Sort Algorithm in TypeScript (Ascending Order)
Description
This pull request introduces the implementation of the Shell Sort algorithm in TypeScript #63 , focusing on sorting data in ascending order.
Changes Made

Added the Shell Sort algorithm implementation in TypeScript.
Ensured that the algorithm correctly sorts data in ascending order.

Attac

hments
Additional Notes
This contribution aims to enhance the project's capabilities by adding a valuable sorting algorithm. Your feedback and suggestions for further improvements are highly appreciated.
Closes #63

https://user-images.githubusercontent.com/43691873/135965000-a3019b12-c089-4a61-bae3-81f915ec39a5.png
Pls read the readme file again. Your PR will be excluded by system. Pls submit genuine PR.

Oh thanks, @extinctsion, I'll keep this in mind from now onwards.


Adjust project's style

Adjust project's style

Great work ajusting the style 🎉 now the UI looks amazing


maxPending

closes https://github.com/rollkit/rollkit/issues/1524

Summary by CodeRabbit

New Features

Introduced a limit on the number of pending blocks to enhance system stability and performance.

Closing as we are addressing this differently.


mrc-4200 wire up cli for scheduling, status and cancel

Wires up the cli to trigger scheduling. Also adds status and cancel commands.

Looks great. Took me a minute to work out why you were writing the config back into the container! Bit confused about Serializable.

I can add a comment for that too!


Categories in vss-extension needs updating

Reading https://docs.microsoft.com/en-us/azure/devops/extend/develop/manifest?view=vsts#required-attributes I believe the categories in the vss-extension.json needs updating to ...  "categories": ["Azure Pipelines"],

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

ID: 44581f4a-93df-7507-3844-eadedf9a6cc4
Version Independent ID: 33c7bcf0-1512-975f-5ef8-9f2a192a84ad
Content: Add a Build or Release Task
Content Source: docs/extend/develop/add-build-task.md
Product: devops
GitHub Login: @elbatk
Microsoft Alias: elbatk

Hi @steveculshaw, I've reached out to the dev who helped me write this article, and we'll get it fixed!

We got this change handled and is on the way to live today - thank you!


rna-transcription: correct abbreviations in user-facing descriptions

Because RNA & DNA are biochemically defined abbreviations, they should not be in lower-case, unless there is a technical reason (like in property fields or in slugs).

Thank you @katrinleinweber and good eye!


NODE-798 Driver hangs on count command in replica set with one member

count should not hangs on when replicat have only one primary but have secondaryPreferred preferred.

This change is

can you please add a test case

@christkv test added


Exclude pre-releases from latest-tag-button

Fixes #2239
Tests
I can't find a repo that has pre-releases right now, but I also included some automated tests to make sure the algorithm works.

https://github.com/desktop/desktop Has some recent pre-releases

And?

Nope, I just wanted to let you know a example repo where the feature can be tested

The point of this feature is to tell visitors “the version you’re using is newer than the one shown here”
Usually people don’t use pre-releases, especially as dependencies, so it would make sense to exclude them

Ah you answered my request. Thank you ☺

My bad, I totally misread that comment 🤦♂


Bulk actions in the log filter are misleading

I had a server that was sending errors into a log like fashion, we updated the server to report these warnings and errors to exceptionless. We found pretty quickly that a certain error was being spammed into the log with the same message everytime.
I thought that if I used the filter (message:[message]) and then did a bulk action on the remaining of the Most Recent then it would only delete the messages I had filtered since the UI was only reporting the number of log messages that matched my filter. However when I did the bulk action it deleted all my log items.
I think that expected behaviour for a situation like this would be to only delete the items that were presented to you in the returned filter results or inform the users about the number of items that they are about to delete so they can check the filtered number to see if these numbers match. (Which they wouldn't in my case)

Thanks for your feedback.
Did you delete items from the Most Frequent? If you delete items from the Most Recent (events) it will only delete those events. But if you delete from the Most Frequent (stacks) you are removing the whole stack and all of it's event occurrences (which we don't know the count of without doing a query). What do you think the message should be when trying to delete a stack or events (You are trying to delete (4) stacks and all of its events??)?

Just so I am clear on the logging dashboard I see: http://i.imgur.com/8ZB246Q.png
as you can see I have a lot of unhandled rpc messages (most of the log is tem actually)
You can see in the next screenshot I've filtered only the type of message that I want to delete:
http://i.imgur.com/zn43Bw3.png
Now, in the second image if I click the checkbox on the (global) stack item and choose the option delete I am expecting 62,822 messages to be deleted. however increase I get all 153,142 log messages deleted.
I think that the message should be something that says like "This action (delete) will not take your filter into consideration, are you sure?" Or if the functionality is updated to account for the filter I'm not sure if a message change would be expected at that point? Does the behaviour I am explaining sound expected for deleting stacks in a filter result?

 When you delete a stack you are deleting everything. I'll have a discussion with the team about clarifying the delete message.
If you set the source of the log message you create (Ex. the type of the class that's sending it), they will be stacked by that instead of falling into a global stack.

Can that be arbitrary? So could I use that to create a stack of "rpc 10" for example?

You can set it to anything :). It just takes a string. You can even put extended data on the log message. We have a overload that sets the log level too (Trace, Info, Debug, Warn, Error). We only special case those log levels in the ui but you can set that to anything as well. I think there is a limit on the string length (but it would be hard to hit it unless you put a massive string as the source).

@ocbaker Would you mind submitting a pull request for this change

@ocbaker do you think maybe the best solution would say Are you sure you want to delete these stacks and all related events? instead of Are you sure you want to delete these stacks?

Issue moved to exceptionless/Exceptionless.UI #84 via ZenHub.io


GDPR Export

Adds an endpoint to get all data related to a user

E2E tests failed: https://drone.owncloud.com/owncloud/ocis/20947/68/1
:boom: To see the trace, please open the link in the console ...

npx playwright show-trace https://cache.owncloud.com/public/owncloud/ocis/20947/tracing/space-members-can-be-invited-with-an-expiration-date-alice-2023-3-30-08-23-40.zip
npx playwright show-trace https://cache.owncloud.com/public/owncloud/ocis/20947/tracing/space-members-can-be-invited-with-an-expiration-date-brian-2023-3-30-08-23-47.zip

Looks good in general.
open items:
[ ] Add Request to the libregraph spec


Add noise/bump mapping for small wave ripples

It would be a waste to create a very detailed mesh for the Gestner wave, when small details can be done in the fragment shader with normal/bump mapping. Perhaps we don't even need to use Gerstner waves in this case, and can use a pre-defined normal map to create the ripples.

Implemented in the latest branch in master.


Bump version to v0.15.1

Purpose
Bumps ClimaAtmos to v0.15.1 with the latest radiation dependency fixes and comms_ctx fixes for the coupler.
Review checklist
I have:

followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
checked that this PR does not duplicate an open PR.

In the Content, I have included

relevant unit tests, and integration tests,
appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

[x] I have read and checked the items on the review checklist.

bors r+

bors r+


Security/features

#24
Add new config for compiling platforms

@terreng
Managed to compile and get the sig working. Just run npm run makeWin (You must have the Windows 10 SDK)
Compiled app:
https://drive.google.com/file/d/16COUWdowyiME96rextY7Ve8hzsq4xVoC/view?usp=sharing
To install:

Right click > Properties > Digital Signatures
Click the signature and click Details
Click View Certificate > Install certificate
Click Local Machine
Select Place all certificates in the following store > Browse
Select Trusted People
Click Next > Finish
Double click the appx and click install

@terreng I turned this into a Security/features pr, and it is ready to merge. It resolves many vulnerabilities I found

I think the logging should happen in real time instead of being stored in memory and getting saved to disk every 10 minutes.

@terreng With rapid logging requests, this could cause a disk overload or loss of data / corruption of data. It would be smartest to store it in memory and save this every little while

@terreng I'm not going to save logs realtime, if you would like I can change the setTimeout to 10 seconds

Saving every 10 seconds would be better than 10 minutes, but it's probably still not a great way of doing it. (That should be setInterval by the way.)
I know that this isn't going to capture crash logs either way, but it would still log some things leading up to the crash.
There are node packages out there made specifically to handle logging. I also think we could use fs.createWriteStream to create a write stream to our log file, but I haven't tried it.

@terreng

There are node packages out there made specifically to handle logging. I also think we could use fs.createWriteStream to create a write stream to our log file, but I haven't tried it.

I implemented a createWriteStream function for the WSC.ileSystem. I will take a look into this later
Could we go ahead and merge and have a more advanced 'logging update' in the future?

Take a look at this simple implementation: https://github.com/quirkey/node-logger.
I really don't think this is going to cause any disk issues or corruption.

Merge when you're ready. The new logging should be very simple to implement, so it would be great if you could add it sooner rather than later.
Apologies that I haven't had a lot of time to work on this over the last few weeks or so. I appreciate the work you've done. There isn't a lot left to do with the UI. Once that's done, I look forward to releasing this.


same error across samples - phplog file unwritable

Warning: file_put_contents(phplog): failed to open stream: Permission denied in /var/www/vhosts/mydomain.com/httpdocs/library/anetsdk/lib/net/authorize/util/Log.php on line 269
I have the define("AUTHORIZENET_LOG_FILE","phplog"); line ahead of the 'use' statements for including the namespaces for authorize.net for contract and controller.
I still get the error outputting lots of times ahead of the executeWithApiResposne() function call.
WHERE do I assign the phplog value so this function has something to use rather than generating warnings?

Do not define the constant ("AUTHORIZENET_LOG_FILE","phplog"), if you do not want to see the logs.
Otherwise, if you want the logs, try changing the permissions of the root folder (sample-code-php) .

@netpassprod1 were you able to fix this issue?
I am closing this issue for now. If you have any questions, feel free to reopen this issue.

In the absence of any further questions, I'll close this issue for now. Please let us know if you need any more information or help.


Adapt with inherited Class of List loses additional properties

Good morning,
I have this code:
public class PagedList<T> : List<T>
{
public int PageNumber{get;set;}
public int PageSize{get;set;}
public int TotalCount{get;set;}
}

public PagedList(Ienumerable<T> collection) : base(Collection)
{
}

public class Source
{
public int Val1{get;set;}
}

public class Dest
{
public int Val1{get;set;}
}

If I do:
var lFrom = new PagedList<Source>{new Source{Val1=6} } ;
lFrom.PageNumber=5;
lFrom.PageSize=10;
lFrom.TotalCount=1;

var lTo = lFrom.Adapt<PagedList<Dest>>();

lTo contains source items but properties PageNumber, PageSize and TotalCount are equal to 0 and not valorized.

Nobody?


Adding missing dependencies for OpenLDAP headers (etc.)

Adds OpenLDAP headers to resolve issue #1

@cosmocracy Thanks for the PR. I will check it out.
I'm puzzled about what could be different in our configurations, since I have been working on this repo today and it's building without errors for me.

Yep, build fails on Docker Hub, fixed by your PR. Merging. Thank you.

Thanks for taking the time to create this image. It's let me get a head start learning Pybossa!
.


confusion in the net.test function ,line 87

In the /model/net.py file, line 87, when the model do inference from [predict_start ,predict_start+ predict_step] , it can use the true information of them, which is not properly for model test. In the test stage, the model only input the true value of x[predict_start], and the  following step should input the last predict mu instead.

Thank you for your suggestion. If you look at line 93, we have replaced x[self.params.predict_start + t] with the last predicted mu, so it indeed does what you described.

Thanks for your reply!
I have another confusion about the evaluate function. I noticed that in the function 'train_and_evaluate,' after the training of each epoch, there is a evaluation without sampling. But during each epoch's training  in the 'train' function,  the evalution is with sampling but the test_metrics have not been used. What's the purpose of this evaluation and the difference about sampling or not (I tried to use sampling in the 'train_and_evaluation' function but get poorer result) ? I'd like to also make sure that is there anywhere else used the sampling including but not limited to evaluation function.

Sampling is not done during training and is only used for evaluation (inference). If you set sample_times in /experiments/base_model/params.json to a number that is big enough, the result should be similar to just parsing mu without sampling. The reason sampling is included is to calculate the confidence interval during inference.


Adding Issue templates

Hello there,
I would like to add an issue template for the repository. This template would have issues namely bug, feature. Kindly assign me this issue as a part of DWOC.
THANK YOU

Not Required.
Whenever you see someone else raise this issue, kindly refer to these🙏
#375 #331 #378

Not Required.
Whenever you see someone else raise this issue, kindly refer to these🙏
#375 #331 #378


Run our internal functional test job against PR deployments

This change has our dev Docker deploys run our internally defined functional test job against the newly deployed stack and report back to PRs.
This should give us the benefit of running the functional test suite against PRs again.
Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)

@higs4281 I think really the only thing here is that the functional test run showed back up in the GitHub checks. It failed, but @anselmbradford is working on that in #6512.


[BUG - Release] Unable to hook Blueprint Library functions

Hopefully not a duplicate, but from a quick look I couldn't find the issue being mentioned anywhere.
Branch or Release
3.0.1 Release
Game and Engine Version
Palworld (UE 5.1.1), Stray (UE 4.27) and most likely every other game.
Describe the bug
When using Lua or LiveView to hook to any blueprint library functions, it freezes the game and returns an error:
[04:29:32] [RegisterHook] Registered native hook (5, 6) for Function /Script/Engine.KismetMathLibrary:Add_IntInt
[04:29:32] Tried to execute UFunction::FuncPtr hook but there was no function map entry for UFunction:
<PHONE_NUMBER>000000 None.
	No original function to execute.
	Executing ProcessInternal instead.

To Reproduce
Steps to reproduce the behavior:

Launch game
Hook any blueprint library function
Wait for the blueprint library function to run or call a blueprint library function manually
Game is now frozen with an error in the console until you restart the game

Expected behavior
Same as other normally hooked functions, no crashing.

I'm unable to reproduce this in latest Palworld.
RegisterHook("/Script/Engine.KismetMathLibrary:Add_IntInt", function()
	print("/Script/Engine.KismetMathLibrary:Add_IntInt\n")
end)
RegisterKeyBind(Key.I, function()
	local Math = StaticFindObject("/Script/Engine.Default__KismetMathLibrary")
	local Val = Math:Add_IntInt(2, 3)
	print(string.format("Val: %s\n", Val))
end)

Result:

For some reason, my own call doesn't trigger this problem but using the menu does.

I'm unable to reproduce this in latest Palworld.

Try hooking Function /Script/Pal.PalUtility:AdjustActorToFloor and load into a World, should immediately give an error once the load is finished.

This happens when hooked native UFunction is called from blueprint via EX_CallMath kismet expression.
UE4SS uses Stack.CurrentNativeFunction to find out what function was called. But in EX_CallMath realization in UE source the line setting Stack.CurrentNativeFunction to called function is commented out, so hook can't find original function.
Also, when native function is called from any kismet bytecodes, it doesn’t seem to put params in Stack.Locals, which ue4ss uses for parameters values, and they becomes random values.

Yeah this is a known long-standing problem.
I'm unsure how to fix it.

@Okaetsu If I make a draft PR, can you, when you have time, hook a bunch of different functions and see if you can manage to cause any problems ?

@Okaetsu If I make a draft PR, can you, when you have time, hook a bunch of different functions and see if you can manage to cause any problems ?

Yeah

Actually it involves changing both UE4SS and the Unreal module.
I'll put the patches here and you'll need to apply them locally instead, that'll probably be the easiest way to do this.

These are based on main, not your bp_hooks, hopefully that won't cause too much of a headache for you.
UE4SS:
Index: UE4SS/src/Mod/LuaMod.cpp
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/UE4SS/src/Mod/LuaMod.cpp b/UE4SS/src/Mod/LuaMod.cpp
--- a/UE4SS/src/Mod/LuaMod.cpp	(revision 7d6f790dc0a027935cb14da67d12ab42c345015e)
+++ b/UE4SS/src/Mod/LuaMod.cpp	(date<PHONE_NUMBER>290)
@@ -127,12 +127,13 @@
         LuaType::RemoteUnrealParam::construct(lua_data.lua, &context.Context, s_object_property_name);

         // Attempt at dynamically fetching the params
-        uint16_t return_value_offset = context.TheStack.CurrentNativeFunction()->GetReturnValueOffset();
+        const auto FunctionBeingExecuted = lua_data.unreal_function;
+        uint16_t return_value_offset = FunctionBeingExecuted->GetReturnValueOffset();

         // 'ReturnValueOffset' is 0xFFFF if the UFunction return type is void
         lua_data.has_return_value = return_value_offset != 0xFFFF;

-        uint8_t num_unreal_params = context.TheStack.CurrentNativeFunction()->GetNumParms();
+        uint8_t num_unreal_params = FunctionBeingExecuted->GetNumParms();
         if (lua_data.has_return_value)
         {
             // Subtract one from the number of params if there's a return value
@@ -145,7 +146,7 @@
         {
             // int32_t current_param_offset{};

-            for (Unreal::FProperty* func_prop : context.TheStack.CurrentNativeFunction()->ForEachProperty())
+            for (Unreal::FProperty* func_prop : FunctionBeingExecuted->ForEachProperty())
             {
                 // Skip this property if it's not a parameter
                 if (!func_prop->HasAnyPropertyFlags(Unreal::EPropertyFlags::CPF_Parm))
@@ -276,12 +277,13 @@
             LuaType::RemoteUnrealParam::construct(lua_data.lua, &context.Context, s_object_property_name);

             // Attempt at dynamically fetching the params
-            uint16_t return_value_offset = context.TheStack.CurrentNativeFunction()->GetReturnValueOffset();
+            const auto FunctionBeingExecuted = lua_data.unreal_function;
+            uint16_t return_value_offset = FunctionBeingExecuted->GetReturnValueOffset();

             // 'ReturnValueOffset' is 0xFFFF if the UFunction return type is void
             lua_data.has_return_value = return_value_offset != 0xFFFF;

-            uint8_t num_unreal_params = context.TheStack.CurrentNativeFunction()->GetNumParms();
+            uint8_t num_unreal_params = FunctionBeingExecuted->GetNumParms();
             if (lua_data.has_return_value)
             {
                 // Subtract one from the number of params if there's a return value
@@ -289,7 +291,7 @@
                 --num_unreal_params;

                 // Set up the return value param so that Lua can access the original return value
-                auto return_property = context.TheStack.CurrentNativeFunction()->GetReturnProperty();
+                auto return_property = FunctionBeingExecuted->GetReturnProperty();
                 auto return_property_type = return_property->GetClass().GetFName();
                 int32_t name_comparison_index = return_property_type.GetComparisonIndex();
                 if (LuaType::StaticState::m_property_value_pushers.contains(name_comparison_index))
@@ -306,7 +308,7 @@
             bool has_properties_to_process = lua_data.has_return_value || num_unreal_params > 0;
             if (has_properties_to_process && context.TheStack.Locals())
             {
-                for (Unreal::FProperty* func_prop : context.TheStack.CurrentNativeFunction()->ForEachProperty())
+                for (Unreal::FProperty* func_prop : FunctionBeingExecuted->ForEachProperty())
                 {
                     // Skip this property if it's not a parameter
                     if (!func_prop->HasAnyPropertyFlags(Unreal::EPropertyFlags::CPF_Parm))

Unreal:
Index: src/UFunctionStructs.cpp
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/src/UFunctionStructs.cpp b/src/UFunctionStructs.cpp
--- a/src/UFunctionStructs.cpp	(revision d3d482dc8d55bcccca0426b1e231562a4256f0b4)
+++ b/src/UFunctionStructs.cpp	(date<PHONE_NUMBER>096)
@@ -137,7 +137,8 @@
         try
         {
             HookedUFunctionMap& FunctionMap = GetHookedFunctionsMap();
-            const auto Iterator = FunctionMap.find(TheStack.CurrentNativeFunction());
+            const auto FunctionBeingExecuted = TheStack.Code() ? *std::bit_cast<UFunction**>(&TheStack.Code()[0-sizeof(uint64)]) : TheStack.CurrentNativeFunction();
+            const auto Iterator = FunctionMap.find(FunctionBeingExecuted);

             if (Iterator == FunctionMap.end())
             {
@@ -145,18 +146,18 @@
                 // This could be due to a bug.
                 // Either way, if CurrentNativeFunction is nullptr, it might be safe to call ProcessInternal.
                 // If CurrentNativeFunction is non-nullptr, then we call whatever is stored in CurrentNativeFunction.Func.
-                if (TheStack.CurrentNativeFunction())
+                if (FunctionBeingExecuted)
                 {
                     Output::send<LogLevel::Warning>(STR("Tried to execute UFunction::FuncPtr hook but there was no function map entry for UFunction:\n\t{:016X} {}.\n\tExecuting original function instead.\n"),
-                                                    std::bit_cast<uintptr_t>(TheStack.CurrentNativeFunction()),
-                                                    TheStack.CurrentNativeFunction() ? TheStack.CurrentNativeFunction()->GetFullName() : STR("None"));
-                    TheStack.CurrentNativeFunction()->GetFuncPtr()(Context, TheStack, RESULT_DECL);
+                                                    std::bit_cast<uintptr_t>(FunctionBeingExecuted),
+                                                    FunctionBeingExecuted ? FunctionBeingExecuted->GetFullName() : STR("None"));
+                    FunctionBeingExecuted->GetFuncPtr()(Context, TheStack, RESULT_DECL);
                 }
                 else
                 {
                     Output::send<LogLevel::Warning>(STR("Tried to execute UFunction::FuncPtr hook but there was no function map entry for UFunction:\n\t{:016X} {}.\n\tNo original function to execute.\n\tExecuting ProcessInternal instead.\n"),
-                                                    std::bit_cast<uintptr_t>(TheStack.CurrentNativeFunction()),
-                                                    TheStack.CurrentNativeFunction() ? TheStack.CurrentNativeFunction()->GetFullName() : STR("None"));
+                                                    std::bit_cast<uintptr_t>(FunctionBeingExecuted),
+                                                    FunctionBeingExecuted ? FunctionBeingExecuted->GetFullName() : STR("None"));
                     static auto Object_ExecuteUbergraph = UObjectGlobals::StaticFindObject<UFunction*>(nullptr, nullptr, STR("/Script/CoreUObject.Object:ExecuteUbergraph"));
                     static auto ProcessInternal = Object_ExecuteUbergraph->GetFuncPtr();
                     Output::send(STR("Object_ExecuteUbergraph: {}\n"), (void*)Object_ExecuteUbergraph);
@@ -176,7 +177,7 @@
                 }
                 catch (std::exception& e)
                 {
-                    Output::send(STR("Error executing hook pre-callback {}: {}\n"), TheStack.CurrentNativeFunction()->GetPathName(), to_wstring(e.what()));
+                    Output::send(STR("Error executing hook pre-callback {}: {}\n"), FunctionBeingExecuted->GetPathName(), to_wstring(e.what()));
                 }

                 Iterator->second.GetOriginalFuncPtr()(Context, TheStack, RESULT_DECL);
@@ -187,7 +188,7 @@
                 }
                 catch (std::exception& e)
                 {
-                    Output::send(STR("Error executing hook post-callback {}: {}\n"), TheStack.CurrentNativeFunction()->GetPathName(), to_wstring(e.what()));
+                    Output::send(STR("Error executing hook post-callback {}: {}\n"), FunctionBeingExecuted->GetPathName(), to_wstring(e.what()));
                 }

                 Iterator->second.bIsMidExecution = false;
@@ -195,7 +196,7 @@
         }
         catch (std::exception& e)
         {
-            Output::send(STR("Error executing hooked function {}: {}\n"), TheStack.CurrentNativeFunction()->GetPathName(), to_wstring(e.what()));
+            Output::send(STR("Error executing hooked function: {}\n"), to_wstring(e.what()));
         }
     }
 }

As i see all other bytecodes calls UFunction::Invoke (at least when function isn't remote) which sets Stack.CurrentNativeFunction, so maybe we don't need Stack.Code for other bytecodes
EX_VirtualFunction and EX_FinalFunction -> UObject::CallFunction -> UFunction::Invoke (when not remote)
EX_LocalVirtualFunction and EX_LocalFinalFunction -> ProcessLocalFunction -> UFunction::Invoke
Also, EX_FinalFunction and EX_LocalFinalFunction stores function pointer, but EX_VirtualFunction and EX_LocalVirtualFunction stores function name.

Did some quick testing and while the problematic hooked functions don't freeze the game anymore, the values provided within the callback seem to be filled with junk except for Context which works normally.
LiveView doesn't display any data whenever those functions are called and as for Lua, when I tried accessing int values from /Script/Engine.KismetMathLibrary:Add_IntInt, the second parameter seemed to work fine while the others were filled with random data. This happens with other blueprint library functions aswell.

Did some quick testing and while the problematic hooked functions don't freeze the game anymore, the values provided within the callback seem to be filled with junk except for Context which works normally.
LiveView doesn't display any data whenever those functions are called and as for Lua, when I tried accessing int values from /Script/Engine.KismetMathLibrary:Add_IntInt, the second parameter seemed to work fine while the others were filled with random data. This happens with other blueprint library functions aswell.

As far as I can tell, BP function params are either inlined into the byte-code or in Stack.Locals.
This would explain why the second parameter works but not the first one.
The second one is probably a local, which UE4SS handles, while the first one is probably inlined which UE4SS doesn't handle.
Not sure yet how to fix this.
The byte-code in this screenshot hopefully explains the problem:

We would probably need to be able to actually to step backwards into the byte-code until the param type is found (i.e. EX_LocalVariable or Ex_IntConst) and once determined, we'd step forward and retrieve the value appropriately.
There's maybe a possibility of reusing some of the code in the KismetDebugger mod for this.

If the original crash is going to be fixed at any point soon, we'll have to delay the param fix for later.
A new issue can be created for it.


Get errors when performing sc.pp.highly_variable_genes!

I am following workflow of 'Best-practices in single-cell RNA-seq: a tutorial' to analyze my single-cell sequencing data sets.
I have calculated the size factor using the scran package and did not perform the batch correction step as I have only one sample. Then, I intended to extract highly variable genes by using the function sc.pp.highly_variable_genes. Unfortunately, I got an error:
'LinAlgError: Last 2 dimensions of the array must be square'
LinAlgError Traceback (most recent call last)
in
----> 1 sc.pp.highly_variable_genes(adata)
~/miniconda3/lib/python3.6/site-packages/scanpy/preprocessing/highly_variable_genes.py in highly_variable_genes(adata, min_disp, max_disp, min_mean, max_mean, n_top_genes, n_bins, flavor, subset, inplace)
94 X = np.expm1(adata.X) if flavor == 'seurat' else adata.X
95
---> 96 mean, var = materialize_as_ndarray(_get_mean_var(X))
97 # now actually compute the dispersion
98 mean[mean == 0] = 1e-12 # set entries equal to zero to small value
~/miniconda3/lib/python3.6/site-packages/scanpy/preprocessing/utils.py in _get_mean_var(X)
16 mean_sq = np.multiply(X, X).mean(axis=0)
17 # enforece R convention (unbiased estimator) for variance
---> 18 var = (mean_sq - mean**2) * (X.shape[0]/(X.shape[0]-1))
19 else:
20 from sklearn.preprocessing import StandardScaler
~/miniconda3/lib/python3.6/site-packages/numpy/matrixlib/defmatrix.py in pow(self, other)
226
227 def pow(self, other):
--> 228 return matrix_power(self, other)
229
230 def ipow(self, other):
~/miniconda3/lib/python3.6/site-packages/numpy/linalg/linalg.py in matrix_power(a, n)
600 a = asanyarray(a)
601 _assertRankAtLeast2(a)
--> 602 _assertNdSquareness(a)
603
604 try:
~/miniconda3/lib/python3.6/site-packages/numpy/linalg/linalg.py in _assertNdSquareness(*arrays)
213 m, n = a.shape[-2:]
214 if m != n:
--> 215 raise LinAlgError('Last 2 dimensions of the array must be square')
216
217 def _assertFinite(*arrays):
Versions of my modules:
scanpy==1.3.7 anndata==0.6.17 numpy==1.15.4 scipy==1.2.0 pandas==0.24.0 scikit-learn==0.20.2 statsmodels==0.9.0 python-igraph==0.7.1 louvain==0.6.1
I have downgraded pandas to 0.23.4, however, it not works. But I figured out where the problem lies in.
adata.X /= adata.obs['size_factors'].values[:,None]
This step transform the adata.X to a structure of matrix.
Before the adata.X is
<6242x15065 sparse matrix of type '<class 'numpy.float32'>'
with 19234986 stored elements in Compressed Sparse Row format>
But after performing this step, the adata.X is
This is my adata.X looks like right now:
matrix([[0. , 0. , 0. , ..., 0. , 0. , 0. ],
[0. , 0. , 1.203, ..., 0. , 0. , 0. ],
[0. , 1.096, 0. , ..., 0. , 0. , 0. ],
...,
[0. , 0. , 2.042, ..., 0. , 0. , 0. ],
[0. , 0. , 0. , ..., 0.926, 0. , 0. ],
[0. , 0. , 2.951, ..., 0. , 0. , 0. ]]),
And this format of adata.X caused error of sc.pp.highly_variable_genes. But I don't know how to fix it.
Looking forward your response!
Thank you !

Hi, I have fixed the issue.
It appears that adding, subtracting or dividing numpy.ndarrays with scipy.sparse matrices returns a numpy.matrix. numpy_array /= scipy_sparse_matrix, This command changed the type of numpy_array to numpy.matrix which caused downstream problems. So, you have to transfer the matrix to sparse format again for downstream analysis.
I used the command 'adata.X = scipy.sparse.csr_matrix(adata.X) ' after dividing the measured counts by the size factor.
So, I paste it here as a note of warning when performing this type of operation.

Thank you for sharing this! We should indeed print warnings if numpy.matrix causes problems. I wasn't aware of it.


Handle feature not licensed errors

Gallagher API licenses features by the piece, if a feature isn't available the server responds with a 403 with the following json:
HTTP/1.1 403 Forbidden
Cache-Control: no-cache
Content-Length: 34
Content-Type: application/json; charset=utf-8
Date: Sat, 10 Jun 2023 05:08:12 GMT

{
    "message": "Feature not licensed"
}

the API client should have the ability to handle these responses and throw an exception

While experimenting with responses to implement #5 I found that the discovery endpoint does not return an endpoint for features not licensed by the instance.
We should depend on the responses being set to None for the particular endpoints and an exception being thrown by the client.

Partially handled by #7
We still need to ensure that we handle this in user interfaces to provide meaningful information to the users.
At an API level the developer is expected to catch and handle this.
Leaving this open until documentation is completed.


Update tidyread---streamline-your-daily-reading extension

Description

add search rss page
add redirect route to improve onboarding experience
improve gen digest panel
add some feedback iamges in degest
fix default maxItemsPerFeed 10 => 5

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

It's done, Most of the code come from svg, so don't worry about it! @pernielsentikaer 🙏

No more commits now🫠

No more commits now🫠

Here is a bug in search list when scroll fast, let me fix in 1 hour.

It's done, Most of the code come from svg, so don't worry about it! @pernielsentikaer 🙏

It's done, Most of the code come from svg, so don't worry about it! @pernielsentikaer  🙏
With this change, users can add RSS more easily

Add generate digest in background, cause sometimes long task(generate digest) in user interface will cause Raycast back to root

No more commits, if you are ok for that, looking forward to release 🥰 @pernielsentikaer


Unable to add dependency in KTOR project

I am trying to add firestore support in my KTOR project but getting this error on building the project.
Tried

apply plugin: "org.jetbrains.kotlin.jvm"

and

apply plugin: 'kotlin'

still getting the same error
Execution failed for task ':compileKotlin'.
> Could not resolve all files for configuration ':compileClasspath'.
   > Could not resolve dev.gitlive:firebase-firestore:1.2.0.
     Required by:
         project :
      > No matching variant of dev.gitlive:firebase-firestore:1.2.0 was found. The consumer was configured to find an API of a library compatible with Java 14, preferably in the form of class files, and its dependencies declared externally, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm' but:
          - Variant 'debugApiElements-published' capability dev.gitlive:firebase-firestore:1.2.0 declares an API of a component:
              - Incompatible because this component declares a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'androidJvm' and the consumer needed a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm'
              - Other compatible attributes:
                  - Doesn't say anything about its component category (required a library)
                  - Doesn't say anything about how its dependencies are found (required its dependencies declared externally)
                  - Doesn't say anything about its target Java version (required compatibility with Java 14)
                  - Doesn't say anything about its elements (required them preferably in the form of class files)
          - Variant 'debugRuntimeElements-published' capability dev.gitlive:firebase-firestore:1.2.0 declares a runtime of a component:
              - Incompatible because this component declares a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'androidJvm' and the consumer needed a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm'
              - Other compatible attributes:
                  - Doesn't say anything about its component category (required a library)
                  - Doesn't say anything about how its dependencies are found (required its dependencies declared externally)
                  - Doesn't say anything about its target Java version (required compatibility with Java 14)
                  - Doesn't say anything about its elements (required them preferably in the form of class files)
          - Variant 'iosApiElements-published' capability dev.gitlive:firebase-firestore:1.2.0:
              - Incompatible because this component declares a usage of 'kotlin-api' of a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'native' and the consumer needed an API of a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm'
              - Other compatible attributes:
                  - Doesn't say anything about its component category (required a library)
                  - Doesn't say anything about how its dependencies are found (required its dependencies declared externally)
                  - Doesn't say anything about its target Java version (required compatibility with Java 14)
                  - Doesn't say anything about its elements (required them preferably in the form of class files)
          - Variant 'iosArm64ApiElements-published' capability dev.gitlive:firebase-firestore:1.2.0:
              - Incompatible because this component declares a usage of 'kotlin-api' of a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'native' and the consumer needed an API of a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm'
              - Other compatible attributes:
                  - Doesn't say anything about its component category (required a library)
                  - Doesn't say anything about how its dependencies are found (required its dependencies declared externally)
                  - Doesn't say anything about its target Java version (required compatibility with Java 14)
                  - Doesn't say anything about its elements (required them preferably in the form of class files)
          - Variant 'jsApiElements-published' capability dev.gitlive:firebase-firestore:1.2.0:
              - Incompatible because this component declares a usage of 'kotlin-api' of a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'js' and the consumer needed an API of a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm'
              - Other compatible attributes:
                  - Doesn't say anything about its component category (required a library)
                  - Doesn't say anything about how its dependencies are found (required its dependencies declared externally)
                  - Doesn't say anything about its target Java version (required compatibility with Java 14)
                  - Doesn't say anything about its elements (required them preferably in the form of class files)
          - Variant 'jsRuntimeElements-published' capability dev.gitlive:firebase-firestore:1.2.0:
              - Incompatible because this component declares a usage of 'kotlin-runtime' of a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'js' and the consumer needed an API of a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm'
              - Other compatible attributes:
                  - Doesn't say anything about its component category (required a library)
                  - Doesn't say anything about how its dependencies are found (required its dependencies declared externally)
                  - Doesn't say anything about its target Java version (required compatibility with Java 14)
                  - Doesn't say anything about its elements (required them preferably in the form of class files)
          - Variant 'metadataApiElements-published' capability dev.gitlive:firebase-firestore:1.2.0:
              - Incompatible because this component declares a usage of 'kotlin-api' of a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'common' and the consumer needed an API of a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm'
              - Other compatible attributes:
                  - Doesn't say anything about its component category (required a library)
                  - Doesn't say anything about how its dependencies are found (required its dependencies declared externally)
                  - Doesn't say anything about its target Java version (required compatibility with Java 14)
                  - Doesn't say anything about its elements (required them preferably in the form of class files)
          - Variant 'releaseApiElements-published' capability dev.gitlive:firebase-firestore:1.2.0 declares an API of a component:
              - Incompatible because this component declares a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'androidJvm' and the consumer needed a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm'
              - Other compatible attributes:
                  - Doesn't say anything about its component category (required a library)
                  - Doesn't say anything about how its dependencies are found (required its dependencies declared externally)
                  - Doesn't say anything about its target Java version (required compatibility with Java 14)
                  - Doesn't say anything about its elements (required them preferably in the form of class files)
          - Variant 'releaseRuntimeElements-published' capability dev.gitlive:firebase-firestore:1.2.0 declares a runtime of a component:
              - Incompatible because this component declares a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'androidJvm' and the consumer needed a component, as well as attribute 'org.jetbrains.kotlin.platform.type' with value 'jvm'
              - Other compatible attributes:
                  - Doesn't say anything about its component category (required a library)
                  - Doesn't say anything about how its dependencies are found (required its dependencies declared externally)
                  - Doesn't say anything about its target Java version (required compatibility with Java 14)
                  - Doesn't say anything about its elements (required them preferably in the form of class files)

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

With KTOR you should be using https://github.com/googleapis/java-firestore


add **kwargs param to _format_output

_generate() and _agenerate() both accept **kwargs, then pass them on to _format_output; but _format_output doesn't accept **kwargs.  Attempting to pass, e.g.,
 timeout=50

to _generate (or invoke()) results in a TypeError.
Thank you for contributing to LangChain!

[ ] PR title: "package: description"

Where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..."  for template changes, "infra: ..." for CI changes.
Example: "community: add foobar LLM"

[ ] PR message: Delete this entire checklist and replace with

Description: a description of the change
Issue: the issue # it fixes, if applicable
Dependencies: any dependencies required for this change
Twitter handle: if your PR gets announced, and you'd like a mention, we'll gladly shout you out!

[ ] Add tests and docs: If you're adding a new integration, please include

a test for the integration, preferably unit tests that do not rely on network access,
an example notebook showing its use. It lives in docs/docs/integrations directory.

[ ] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Additional guidelines:

Make sure optional dependencies are imported within a function.
Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests.
Most PRs should not touch more than one package.
Changes should be backwards compatible.
If you are adding something to community, do not re-import it in langchain.

If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, hwchase17.

 Your solution is necessary to support the experimental tool calling!


fix: changing masking policy on a column requires FORCE

Attempting to change masking policy without FORCE would cause the following error:
003529 (42601): Specified column already attached to another masking policy. A column cannot be attached to multiple masking policies. Please drop the current association in order to attach a new masking policy.
This PR adds FORCE to the generated ALTER statement and improves error information if masking policy change fails to apply.
Test Plan
Added TestAcc_TableWithMaskingPolicy that

creates 2 masking policies (pol1 and pol1)
creates single column table using masking policy pol1
changes masking policy from pol1 to pol2

References

Updating masking policy requires force: https://docs.snowflake.com/en/user-guide/security-column-intro#replace-a-masking-policy-on-a-column

Hey @malduarte. Thanks for the contribution.
Unfortunately, we are currently changing the implementation used in resources as a part of our ongoing SDK rewrite. This means we will eliminate table implementation residing in the snowflake package.
However, we will add this behavior as a part of the process.

@sfc-gh-asawicki Is there an active branch? I'm happy to submit the MR there

Yes, there is an open PR: #2042.


github instructions on tutorials template

Need instructions for creators to upload their tutorials to this repository. See empty section at bottom of tutorials template.

Will be making a Github tutorial


Escape a single quote in a warning message  in PeepholeFoldConstants

This change is

Maybe someday we can switch to a more intuitive message formatter.