Access Copy Page function for a role that only has view only for 'All' but can create their own.

Describe the bug
If a user only has view access on a book page, the user should still be able to copy that page to a book that they do have create/update access for.
Steps To Reproduce
Steps to reproduce the behavior:

Create a role with Create/Update for own books/pages etc
Add View access to other books to the same role
Login as user with that role
Go to view only book. The 'More' menu on the right hand side is not available.

Expected behavior
For a view only page a user should be able to 'copy' that page to a book provided they have create/update access on at least one book.
Screenshots
Copy Functionality

Additional context
Use Case: I have a bunch of pages that are templates. I don't want users to be able to edit them. But I do want them to be able to take a copy and then edit that copy.

Now implemented as of merge in #1202. To be in the next patch release.


do not handle PR closed event

Current orchestrator behaviour attempts to handle PR closed, this should be ignored

Closing since I think that we do need to run unlock on pr closing. A future improvement could be that we unlock on the orchestrator itself after performing cloud based locks


Add json_dump method and use that for converting to json #3

As there are 2 PRs being stalled and the issue is unsolved since a long time. This is a third try to patch it.
References are

https://github.com/graylog-labs/gelf-rb/pull/64 (original PR)
https://github.com/graylog-labs/gelf-rb/pull/73 (another approach but revoked)
https://github.com/fluent/fluentd/issues/2404 (effected parties)

 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.Rene Wagner seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

The build issue doesn't seem related to my change.
Gem::GemNotFoundException: can't find gem bundler (>= 0.a) with executable bundle
only for Ruby: jruby-19mode


Add link to the Demo repository so people can use it as example

The Readme has a link to the Demo, however there is no link to the source code of that actual demo. Adding the source code can be helpful for people to have another example on how to use this great library

Not sure what do you mean here

@khmyznikov sory that was not clear indead. I mean the demo from the readme https://khmyznikov.com/pwa-install/
Any examples of implementations are useful. I ended up searching though your repo to find the source code of it, but it wouldve been helpful if it would just be in the reafdme as well

@khmyznikov sory that was not clear indead. I mean the demo from the readme https://khmyznikov.com/pwa-install/ Any examples of implementations are useful. I ended up searching though your repo to find the source code of it, but it wouldve been helpful if it would just be in the reafdme as well

Can you share where you found the code for the demo ?

https://github.com/khmyznikov/khmyznikov.github.io

@MbBrainz  @abdulrahimiliasu  this is the source for demo https://github.com/khmyznikov/pwa-install/tree/main/docs


[ID-335] Remove account from the system - all BBs

Description
This PR adds a Deleted flag to OrgAppMembership to mark app memberships as deleted as requested by a user. These deleted memberships will have all but their identifying data (ID and AppOrgID) removed and they will be retained until all interested building blocks have been informed of the update. The memberships will be deleted permanently only after this occurs.
Edit (from the comments below): The above statement relies on the assumption that all of the interested building blocks are working as expected. The interested building blocks are expected to make one request to the Core BB to get deleted account/membership information within the period between runs of the timer on the Core BB which finally deletes the marked account/membership data from the DB. The Core BB does not actually verify that all building blocks interested in deleted accounts/memberships have received the data before the timer performs the final deletion.
Depends on #319
Resolves #335
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[ ] Within a week
[x] When possible

Type of changes
Please select a relevant option:

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[x] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[x] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My change requires updating the documentation.
[x] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[x] My changes generate no new warnings.
[x] New and existing unit tests pass locally with my changes.
[x] Any dependent changes have been merged and published in downstream modules.

Hi @shurwit and @roberlander2, is this still valid from today point of view? Thanks

Hi @shurwit and @roberlander2, is this still valid from today point of view? Thanks

Hi @petyos, yes this will still be needed, thanks for checking. We will work on updating the PR when we get a chance, then re-request your review.

Hi @shurwit and @petyos, this PR is ready for review again.

Hi @roberlander2 , thanks for your PR.
Some common comments: As I see you have taken the approach to keep the deleted accounts within the original accounts structure together with the active accounts. This has disadvantages as this mixes them instead of keeping the deleted ones uncoupled from the active accounts in a separate collection which would serve only the deletion mechanism. As you have implemented it then I am ok with it, let's @shurwit also confirm this is good in his opinion.

Thanks @petyos, sorry for the delay in responding to you on this. @shurwit and I talked about your suggested separate deleted accounts and active accounts collections, and we agreed that it would be better than what I currently have implemented, so I will make this change and re-request your review.

More specific questions: As I understand when the user deletes account/membership it is only marked as "for deletion". After that a timer removes all data for the marked accounts except the one needed for the deleted-membeships API. Is this the process as a whole?

When a user deletes his account/membership, all data associated with that account/membership is deleted except for information that is required to specify to other building blocks which account/membership was deleted. The timer performs the final deletion of the account document from the accounts collection or the membership sub-document from the account, depending on the situation.

You also say ".. removed and they will be retained until all interested building blocks have been informed of the update. The memberships will be deleted permanently only after this occurs." How Does Core BB know when to fully remove all removed accounts and verify all interested BBs have removed all needed?

You are right, I should have written something different here. This statement relies on the assumption that all of the interested building blocks are working as expected. The interested building blocks are expected to make one request to the Core BB to get deleted account/membership information within the period between runs of the timer on the Core BB which finally deletes the marked account/membership data from the DB. The Core BB does not actually verify that all building blocks interested in deleted accounts/memberships have received the data before the timer performs the final deletion.

Ok, Thanks!

Hi @petyos, this PR is ready for review again. Please let me know if you have any additional questions or concerns. Thanks!

Hi @roberlander2,
Thanks for reworking this. Keeping the deleted ones outside the accounts is much more clear.
How often have you designed the deleted ones to be removed from core BB? In my opinion we should not remove them or if we remove them this should happen very rarely - maybe once a week or if we want shorter then min once a day during the night. In this way we will be sure that there is no BB which has missed a removed account. In general the deleted ones' records will not be a huge number and we can allow them to live for a long time in the deleted memberships collection. As I see the default time you have set is 2 hours. What do you think about it?

Hi @roberlander2 I did small tweaks and I am merging it. Let me know if you have any comments or questions.


Sandbox environment cannot be identified for each user

Environment information
System:
  OS: macOS 14.5
  CPU: (8) arm64 Apple M2
  Memory: 173.20 MB / 24.00 GB
  Shell: /bin/zsh
Binaries:
  Node: 20.9.0 - ~/.nvm/versions/node/v20.9.0/bin/node
  Yarn: 1.22.19 - ~/.nvm/versions/node/v20.9.0/bin/yarn
  npm: 10.1.0 - ~/.nvm/versions/node/v20.9.0/bin/npm
  pnpm: 8.11.0 - ~/.nvm/versions/node/v20.9.0/bin/pnpm
NPM Packages:
  @aws-amplify/backend: 1.0.2
  @aws-amplify/backend-cli: 1.0.3
  aws-amplify: 6.3.4
  aws-cdk: 2.143.0
  aws-cdk-lib: 2.143.0
  typescript: 5.4.5
AWS environment variables:
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
No CDK environment variables

Description
All table names created have a NONE suffix, so if you create a sandbox for each user in the same AWS account, they will not be identifiable.
A table named TODO-<appsync-api-id>-NONE would be created as follows.

Here is the schema I created.
const schema = a.schema({
  Todo: a
    .model({
      id: a.string(),
      content: a.string(),
    })
    .authorization((allow) => [allow.guest()]),
  EchoResponse: a.customType({
    content: a.string(),
    executionDuration: a.float(),
  }),
});

Hey👋 thanks for raising this! I'm going to transfer this over to our API repository for better assistance 🙂


Support of parallel launch?

Is it possible to add parallel invocation of commands? Something like GNU Parallel:
ls | parallel md5sum

Of course, using parallel directly is a solution... but some simple pooling in this package might also be helpful.

Sounds interesting! Can you suggest a suitable design?

Well, I can think of a few ways. Maybe have an option to construct multiple pipelines manually and have them run in parallel. Or take an input, split based on some criteria (such as lines), then for each subpart run a pipeline in parallel. Optionally use a thread pool to limit the simultaneous processes. Then there might be a need to join their outputs. Maybe it can be done by lines, in-order or out-of-order.
There are quite a few possibilities, but I guess something like a parallel xargs might be a good start.

Great! Let's take a concrete example to help us think about what this might look like as code. The particular use case you mentioned can be done with:
script.ListFiles(".").SHA256Sums().Stdout()

Can you think of another reason you might want to run a bunch of parallel Exec commands?

ExecParallel here would probably be the simplest API.
The second API I mention is if you want to do ExecParallel("a | b"), and in this case it might be nice to be able to do Parallel(script.Exec("a").Exec("b")) in case you want to use a script pipeline. Of course, this would make the API look like a graph description language...
Reordering is probably a niche use case, and it does require finite, small output for the command to work well. I would probably not consider this a frequently used function.


Use _setjmp instead of setjmp

Apparently on macOS setjmp manipulates the process-wide signal mask
which adds a good deal of overhead. We don't actually need this
functionality so this commit switches to using the underscore-prefixed
version of setjmp/longjmp which is defined to not maniuplate the signal
mask. This came out of poking around on #2644 and on macOS locally thi
sdropped the overhead from 721ns to 55ns.
I'll note that I'm not sure if the underscore-versions are present in other platforms, but I figure that CI can figure it out here.

The manual page at least on Linux suggests that _setjmp() is deprecated but pointed to sigsetjmp(), which seems to offer the same capability: if savesigs (the second arg) is zero, it won't save the signal mask. Maybe that's what we want (and maybe it'll resolve the CI issue on Windows)?

Ah indeed good point! Looks like Windows also doesn't have sigsetjmp but I think having unices use sigsetjmp makes more sense, so I've updated the conditional code to do that.


Safe: Cannot connect wallet

Describe the bug
Cannot connect wallet. When clicking "connect wallet", nothing happens.

Chrome console shows WalletConnect errors.

To Reproduce
Steps to reproduce the behavior:

open safe.celo.org
click "connect wallet" (top right)

Expected behavior
Wallet modal
Desktop (please complete the following information):

OS: MacOS
Browser Chrome
Version 121.0.6167.184

Additional context
Add any other context about the problem here.

Looks like the browser is tried to establish a connection to wss://8.bridge.walletconnect.org/?env=browser&host=safe.celo.org&protocol=wc&version=1 but failling to do so.

safe.celo.org still has a bug related to "connect wallet"


Not getting the NTP.now()

So i am using   DateTime tanzaniaLocalTimeFromDateTimeNow = await NTP.now(); in my app and it works fine sometimes, but then sometimes it doesnt give me the value. i tried keeping it in a try and catch but still i dont know what is wrong and why am i not getting the dateTime. My whole app depends on this. I am using version ntp: 1.0.7. Please help

Hey, how often are you calling this?
It might be best to call it once and same it locally. This can be done by saving ntpOffset and adding that yourself as Duration on DateTime.now().

Hey, how often are you calling this?
It might be best to call it once and same it locally. This can be done by saving ntpOffset and adding that yourself as Duration on DateTime.now().

Hi, I'm just calling this once. I did the ntpOffset as well but still sometimes i won't be able to get the time from it. So now I have shifted to version ntp:^1.0.8, so far so good.
This error was occurring in the version of 1.0.7.
Let me share that part of my code

Find below the function i use to call the ntp.now()
static Future<List<int>> loadDefaultData() async {
    print("inside loadDefaultData");
    var byteData = await rootBundle.load('packages/timezone/data/2020a.tzf');
    return byteData.buffer.asUint8List();
  }

  static Future<String> getTimeStamp() async {
    print("inside getTimeStamp");

    await loadDefaultData().then((rawData) async {
      print("inside loadDefaultData.then before initializeDatabase");

      initializeDatabase(rawData);
      print("After initializeDatabase");

      // DateTime tanzaniaLocalTimeFromDateTimeNow = DateTime.now();
      DateTime tanzaniaLocalTimeFromDateTimeNow = await NTP.now();

      print(
          "MY tanzaniaLocalTimeFromDateTimeNow is $tanzaniaLocalTimeFromDateTimeNow which is after initializeDatabase");

      final tanzaniaTimeFromTimeZone = TZDateTime.from(
          tanzaniaLocalTimeFromDateTimeNow, getLocation('Africa/Nairobi'));
      final torontoTime = TZDateTime.from(
          tanzaniaLocalTimeFromDateTimeNow, getLocation('America/Toronto'));
      final indiaTime = TZDateTime.from(
          tanzaniaLocalTimeFromDateTimeNow, getLocation('Asia/Kolkata'));

      var formatter = new DateFormat('yyyy-MM-dd HH:mm:ss');
      String formatted = formatter.format(tanzaniaTimeFromTimeZone);

      print("tanzaniatimefromtimezone is ====" +
          tanzaniaTimeFromTimeZone.toString());
      print("Timestamp : " + formatted);

      globalTimeStamp = formatted;
      return formatted;
    });
  }

So as you see, i have sprinkled some prints to show me where the issue rises because i am not getting any error. So it always stops in the  print("After initializeDatabase"); which is before  DateTime tanzaniaLocalTimeFromDateTimeNow = await NTP.now();

Ok, I am facing this issue with version ^1.0.8 as well

Hey sorry for late reply. Are you still facing this issue with 2.0.0? Should work fine now :) If not, changing lookupAddress might help. (pool.ntp.org, time.apple.com)

Hey sorry for late reply. Are you still facing this issue with 2.0.0? Should work fine now :) If not, changing lookupAddress might help. (pool.ntp.org, time.apple.com)

Yes I am facing this with 2.0.0 as well.

I had the same issue and indeed, changing the lookupAddress fixed it for some reason.

I had the same issue and indeed, changing the lookupAddress fixed it for some reason.

Which lookupAdress do you use?

I had the same issue and indeed, changing the lookupAddress fixed it for some reason.

Which lookupAdress do you use?

I used this one: pool.ntp.org and it worked.
When not specifying the lookupAddress
await NTP.now()
wasn't returning anything and my code was stuck.

We're having this issue as well with await NTP.getNtpOffset. Some of our users complained about this, and today I had the issue, too. The future just doesn't complete. The only workaround is to add a .timeout(const Duration(seconds: 2) and load from another resource the time.
We are using 2.0.0

I have the same issue with version  2.0.0 ,  change address to pool.ntp.org look fine yet .
Is this a problem from time.google.com ?

I have same problem my app freeze because of NTP.now() in ios
I use 2.0.0

I have same problem my app freeze because of NTP.now() in ios
I use 2.0.0

I have same problem my app freeze because of NTP.now() in ios I use 2.0.0

some my user report it

It does look like the lookUpAddress is no longer working. I was able to find a bunch of addresses on https://tf.nist.gov/tf-cgi/servers.cgi since all of the servers are based in the USA, I am unsure how bad the latency can be overseas. However, this does solve the issue.

I'm also getting this error in 2024.. Any solution? Already tried all mentioned above.


[ORGANIZATION] Center for Biological Diversity

Resource title
Center for Biological Diversity
URL
https://www.biologicaldiversity.org/about/
Relevant tags (1 per line)
Organizations
Politics/US
Resource Summary
A nonprofit focused on legal protection of endangered species and wildlife, plus some energy justice work. Founded in 1989 by Kieran Suckling, Peter Galvin, Todd Schulke and Robin Silver.
Relevance
A nonprofit with good coverage of endangered species and conservation efforts and decades of experience
Other comments
https://www.biologicaldiversity.org/programs/energy-justice/pdfs/Policy-Brief-for-Positive-Vision.pdf for archive purposes

Sorry about the long response time on this.

Their plan for renewable energy justice is recommended policy inspiration.

I think with the resource summaries it would be good to stray from wording that is a personal recommendation. Would you mind rewording that? (I'm not familiar with their work, so I don't think I'd do it justice). Also the "Resource Summary" section can't have links in it as the site doesn't support it (but feel free to bump that into Other comments so its still visible on this issue).

Would you mind rewording that? (I'm not familiar with their work, so I don't think I'd do it justice). Also the "Resource Summary" section can't have links in it as the site doesn't support it (but feel free to bump that into Other comments so it's still visible on this issue).

Done.

Thanks @JarrodBaniqued !

Resource added in 85d1c68


BUG: Typo in Examples Page?

In the examples on http://keras.io/examples/, I think the lines that read
from keras.layers import Dense, Dropout, Activation

should be
from keras.layers.core import Dense, Dropout, Activation

If not, there is a problem with my installation.;P

Try the latest version of Keras. You can import all core layers from keras.layers.

Thanks, that worked.
For reference, I cloned the repo with this commad:
pip install --upgrade --no-deps git+git://github.com/fchollet/keras.git


Can't pipe stdout/stderr to the same pipe for a child process

 Issue by alexcrichton
Monday Nov 04, 2013 at 23:24 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/10269
This issue was labelled with: A-io in the Rust repository

This is more of a problem with the rt::io::process bindings rather than the std::run bindings (the latter needs a larger overhaul in general). The rt::io::process bindings currently don't expose the ability to have something like 2>&1 and then you capture the output of &1 (redirecting stderr to stdout and then reading stdout).
A possible solution is to add another enum variant to StdioContainer named along the lines of PipeTo(int) which means that the output of this stdio slot should get redirected to the pipe which was defined at slot int. Perhaps an IoError should be raised if it doesn't reference a CreatePipe slot, perhaps a failure.

Is there a way to do this? Is this going to get addressed ever?

This is possible through FromRawFd and FromRawHandle for std::process, so closing.

The docs for FromRawFd claim:

This function consumes ownership of the specified file descriptor. The returned object will take responsibility for closing it when the object goes out of scope.

If that's true, how can you create two different Stdio objects to redirect the child process stdout and stderr using from_raw_fd if both own the descriptor?
That is to say:
Command::new("ls")
    .arg("-l")
    .stdout(Stdio::from_raw_fd(x))
    .stderr(Stdio::from_raw_fd(x))
    .spawn()

Will that double close file descriptor x?

That does seem reasonable, though we should update the docs on Stdio to note that fact.
If it did close the file descriptors, you could use the libc dup function to keep things from breaking.

A Stdio object does close its file descriptor when it drops. That's because it contains a FileDesc object, and because FileDesc implements Drop. @sfackler is right that if you want to use the same descriptor in two places, you need to use dup. (The standard library actually exposes dup in a roundabout way, via File::from_raw_fd and the File::try_clone method.)

@oconnor663 Thanks for the correction! I failed to see the Drop for FileDesc which is where the close of the descriptor appears to occur (for some reason I thought a RawFd was being stored and not a FileDesc).
Explicitly duplicating the descriptor seems like an easy, if not platform-specific, approach to making this work properly.  However, it would be nice if we could do something like .stderr(Stdio::redirect_to_stdout()) and have the stdlib handle the platform-specific details behind the scenes.

@peterhuene for what it's worth there is a stderr_to_stdout method in my duct crate :-D

Hey, maybe reopen this? As @peterhuene said, .stderr(Stdio::redirect_to_stdout()) would be very useful. The hacky way of duplicating file descriptors is not particularly obvious.
For example, Python has this - https://docs.python.org/3/library/subprocess.html#subprocess.STDOUT - and it's pretty neat.

@WGH- if you want to set up pipes yourself, another option is to use the os_pipe crate. That's what duct uses internally, and the pipes implement Into<std::process::Stdio>.

@oconnor663 that's a good suggestion, but it still doesn't solve the main problem.
You cannot use child.stdout.take() because child.stdout is initialized only when you use Stdio::piped().
You can't wrap the other end of the pipe with tokio::fs::File, because it doesn't work well with non-blocking fds, returning WouldBlock error instead of waiting properly.
You can't construct tokio::process::ChildStdout from a raw fd, either.
Unless I'm missing something, your only option seems to be to basically duplicate code from here, which is pretty low-level and far from portable:
https://github.com/tokio-rs/tokio/blob/4c4699be00bf4aee6218f84ec606e043286a8c2a/tokio/src/process/unix/mod.rs#L140-L231

Err, sorry, I kind of conflated this with tokio problem. Right, in "vanilla" Rust it's indeed relatively easy to just wrap the other end of the pipe and go from there.

Is this possible yet?


fix: #814 update dynamic icon import to fix pipeline slow issue

update frontend dynamic import icons to only import the icons we used refs: #814

I checked if Carbon also provides a "@Type" package.
Looks like it does have types for @types/carbon__icons-react  but it does not have<EMAIL_ADDRESS>Seems Vue is still not fully supported by Carbon with typing after years...

The PR looks good. Thanks Catherine!
I have researched into vite option to optimize a little on our frontend.
Could you also add this option in "vite.config.ts" (vite is a compile/packaging tool for Vue for production), it does not impact your change but it will do a better bundling in this case for dynamic import for async components.
If you are not sure where to add, I can quickly show you tomorrow or I can add it in your branch.
        build: {
            chunkSizeWarningLimit: 1600,
            outDir: "dist",
            sourcemap: true,
            rollupOptions: {
                output: {
                  inlineDynamicImports: true,
                },
            }
        },


New Mailing List - Mainframe Open Education

Please complete all the sections below to request a new mailing list. Requests can take a day or two, so please be patient :-).
Project name

Mainframe Open Education

Would you like a combined -user and -dev list, or seperate

Not necessary at this point

Moderators for the mailing list

Guilherme Cartier<EMAIL_ADDRESS>Viviane Sanches<EMAIL_ADDRESS>Deborah Carbo<EMAIL_ADDRESS>Lauren Valenti<EMAIL_ADDRESS>Yvette Lamar<EMAIL_ADDRESS>

Sorry for the delay @gcartier94 - this is done now. You should have both a -discussion and -private list provisioned. Let me know of any issues or concerns.


Ensure that we can lookup the antora provided paths

fixes #416
This is an attempt to (spot) fix #416. in the issue it is mentioned that the !include directive requires some refactoring, but i'm not intimate enough with the solution to figure that out.
for more information please see the issue.

Fixed in https://github.com/ggrossetie/asciidoctor-kroki/pull/422


Heatmap fixes

I really like the changes you made, it looks "clearer", more organised, if it makes any sense. I like it a lot!


ValueError: invalid literal for int() with base 10: '0.69327074'

When the input has floating values i.e the bounding boxes I am getting the following error
ValueError: invalid literal for int() with base 10: '0.69327074'
I changed the line 263 of the ssd_batch_generator.py file to
 obj.append(float(i[self.input_format.index(item)].strip())) but in vain

A few things here:

As the documentation of the method in question (parse_csv()) clearly states, the bounding box coordinates are expected to be absolute coordinates, not relative coordinates. In particular, they need to be integer values.
The transformations in the generate() method rely on absolute coordinates, so even if you rewrote parse_csv() to work with relative coordinates, you'd still also have to change code in other places to work with relative coordinates.
The easiest solution to this is to just convert your labels from relative to absolute bounding box coordinates.

Ok but my coordinate values are between 0 and 1 so what changes should I make in the parse_csv() ?

As I said above, you should just change your coordinate values from relative to absolute. Doing that is way easier than changing the batch generator.
If you want to change the code to work with relative coordinates, that's your own adventure. I don't know of the top of my head all places in the code you'd have to change and I don't intend to invest time in it. If I were you I would just write a simple preprocessing function that takes your CSV with relative coordinates and outputs either another CSV with absolute coordinates or a Python list with the labels in the format as the batch generator constructor needs them.

ok thank you


Remove get_ prefixes from all functions

There are a number of functions, public or otherwise, which still have the get_ prefix. This violates the suggested naming conventions laid out by the Rust API Guidelines, and as such they should be removed.
@MabezDev would you like me to tackle this before the next release, or wait until after we've published 0.22.0?

Let's get this out of the way and do it now, I hope it should be fairly straight forward.


feat/ add multicall contract

This PR adds support for the MakerDAO Multicall2 contract.

Codecov Report
Base: 89.76% // Head: 88.99% // Decreases project coverage by -0.78% :warning:

Coverage data is based on head (8e1749d) compared to base (3078463).
Patch coverage: 53.07% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #176      +/-   ##
==========================================
- Coverage   89.76%   88.99%   -0.78%
==========================================
  Files          69       73       +4
  Lines        5190     5326     +136
==========================================
+ Hits         4659     4740      +81
- Misses        531      586      +55

Flag
Coverage Δ

unittests
88.99% <53.07%> (-0.78%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...rr/tests/test_token_vault_factory/test_contract.py
98.62% <ø> (ø)

...ectooorr/contracts/token_vault_factory/contract.py
86.33% <ø> (ø)

...lectooorr/skills/elcollectooorr_abci/behaviours.py
99.82% <ø> (ø)

...ges/elcollectooorr/contracts/artblocks/contract.py
30.35% <6.66%> (-7.54%)
:arrow_down:

...ectooorr/contracts/artblocks_periphery/contract.py
34.86% <14.28%> (-5.14%)
:arrow_down:

...oorr/contracts/artblocks_minter_filter/contract.py
33.33% <14.70%> (-12.67%)
:arrow_down:

...es/elcollectooorr/contracts/multicall2/contract.py
93.02% <93.02%> (ø)

...r/agents/elcollectooorr/tests/helpers/constants.py
100.00% <100.00%> (ø)

...lectooorr/tests/test_agents/base_elcollectooorr.py
68.05% <100.00%> (+0.44%)
:arrow_up:

...es/elcollectooorr/contracts/multicall2/__init__.py
100.00% <100.00%> (ø)

... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Removal of CzmlDataSource from Viewer breaks the Viewer

I am using Cesium to display a subset of the Iridium Next constellation, and I am loading sats via the following:
// satData is the valid CZML sent from the backend over the socket
viewer.dataSources.add(Cesium.CzmlDataSource.load(satData)).then(function (datasource) {
                    ds = datasource;
                });

and then I am attempting to remove it when the socket that sent the satData dies as such:
if (viewer.dataSources.contains(ds)) {
            // satData represent whichever data was past to the frontend from the socket that just disconnected
            var removalPromise = viewer.dataSources.remove(ds);
            Cesium.when(removalPromise, function() { console.log('The promise has resolved') });
}

When I get here, ONLY WHEN THERE ARE MULTIPLE SATS IN THE VIEWER (ala satData1, satData2, etc) I get the following error:
DeveloperError: This object was destroyed, i.e., destroy() was called. Error at new DeveloperError
 (webpack:///./node_modules/cesium/Source/Core/DeveloperError.js?:43:19) at
 PolylineCollection.throwOnDestroyed (webpack:///./node_modules/cesium/Source/Core/destroyObject.js?:45:19) at
 PrimitiveCollection.update

(webpack:///./node_modules/cesium/Source/Scene/PrimitiveCollection.js?:368:27) at PrimitiveCollection.update

(webpack:///./node_modules/cesium/Source/Scene/PrimitiveCollection.js?:368:27) at PrimitiveCollection.update

(webpack:///./node_modules/cesium/Source/Scene/PrimitiveCollection.js?:368:27) at

updateAndRenderPrimitives (webpack:///./node_modules/cesium/Source/Scene/Scene.js?:2936:27) at

executeCommandsInViewport (webpack:///./node_modules/cesium/Source/Scene/Scene.js?:2775:13) at
 updateAndExecuteCommands (webpack:///./node_modules/cesium/Source/Scene/Scene.js?:2583:13)

at render (webpack:///./node_modules/cesium/Source/Scene/Scene.js?:3223:9) at tryAndCatchError
 (webpack:///./node_modules/cesium/Source/Scene/Scene.js?:3243:13)

Apologies for the weird line breaks. Thanks for the help!

Thanks for the bug report @BenjaminWinks!  Can you share sample data that reproduces this error?  It would be really helpful for us to have that and a short Sandcastle Example to reproduce the crash.

I can't reproduce it in the sandcaste because of the way the data comes in - I can't get websockets working with the sandcastle, so it seems like an exercise in futility at the moment - if I come across more time I will. I just used the satellite czml in the sandcastle as a starting point and renamed them.

Could be a typo, but you should be passing ds to viewer.dataSources.contains() instead of satData.  And likewise with the call to viewer.dataSources.remove().

Can you reproduce by just inlining several valid data structures then adding them dynamically with a setTimeout?  If adding dynamically via timeout doesn't break things but adding dynamically based on send-via-websocket, that would be pretty weird, right?

Since we weren't able to reproduce this, I'm going to close this issue.  If you're still having trouble feel free to re-open it.  Thanks!

I'm running into this exact same problem as well. Trying to remove multiple dataSources causes Cesium to crash. It works fine for just one, but not when I attempt to remove the second element.

I would suggest to both the original author and Isaac that you make sure you're logging every call to viewer.dataSources.remove with the argument, to be absolutely sure that you're not trying to remove anything twice.  If you can guarantee that remove is never being called with the same argument twice, reproducing this behavior in Sandcastle would be extremely valuable toward getting it fixed.

I have verified I'm calling viewer.dataSources.remove with the correct argument and it's not trying to be removed twice. I'll try to get an example put together at some point.

Any update on this issue? I've updated to Cesium 1.74 and still experiencing this issue. As previously stated, I'm unable to provide the CZML files.

Please look at this example error. It is loading 2 different czmls. A setTimeout is set and will run removeAll() and error. If only one czml is added (the other is commented out) removeAll works fine, as does the individual remove(). I've been trying to fix/circumvent this for hours since I'm handcuffed by it. For my application, I'm trying to load and at times remove multiple, much larger czmls and it is necessary I free up memory instead of just hiding them. I believe it has something to do with the time ranges/clock. Because, when I add another czml without time ranges/clock, removeAll() works with that and just one of these timed czmls. I've tried removing various parts of the 2nd czml to pursue that theory and still can't get it to work.
Sandcastle Example

@hpinkos can you reopen this ticket since @Trylor has provided an example via Sandcastle of the issue at hand. Thanks.

@Trylor check out #9154 where I created another ticket related to this issue that has some test cases I listed out. Thanks.

@Trylor check out #9154 where I created another ticket related to this issue that has some test cases I listed out. Thanks.

@icarter09 I did confirm that when I remove the polylines from the 2nd CZML, the remove() and removeAll() methods work correctly.


After minimizing, app window won't resurface

App appears to work great, until i minimize it.
When i minimize the app, it's icon remains in systray, but when i click on it (no matter which mouse button), just the context menu pops up.
When i click on Open or Minimize, nothing happens.
When i click on Close, the app closes.
When i open the exe file again i get the window back.
If the previous instance is still running at that time, i get a second icon in systray.
What gives?
I remember i had the same problem with the clockify app, i had to resort to using keyboard shortcuts to reopen the window.
I'm running on Windows 10 Enterprise 22H2 - Build 19045.4529

Ok, i guess i ran into false assumptions here, as in, the app doesn't minimize to systray.
The app still remains on the normal windows taskbar, which i didn't notice because the window was on a second screen and the taskbar is hidden.


M-: C-r doesn't work immediately

You have to type a character, then delete it, in order to get results.

See also #209.

I unbound the key, so this is not too relevant anymore.


Pass right data to filepond preview from own image editor.

Hi!
Summary
I have opened an issue on pqina/filepond-plugin-image-edit#1 because there are many things unclear to me how to use an own editor with filepond.
How to reproduce
I use the latest versions of filepond and its plugins.
I have followed this instructions plugins/image-edit to setup filepond. Assigned also the imageEditEditor property. It is unclear what type of data I have to pass to output for  onconfirm: (output) => {}.
What I want to do
I want to use cropperjs as editor.
data: {
  crop: {
    center: { // center relative to what? Pixel, percentage?
      x: .5,
      y: .5
    },
    flip: { // this is clear and works for me
      horizontal: false,
      vertical: false
    },
    zoom: 1,
    rotation: 0, // data range? -180 to 180 or 0 to 360?
    aspectRatio: null
  }
}

Any help is appreciated! Thx

Please don't open multiple issues. I've read your question and have simply had no time to respond yet.


Remove use of removed !avatar syntax from inte

I presume this is a copy of #632 that was posted accidentally.


proposal: x/sys/unix: Adding GetsockoptLinger() and GetsockoptTimeval()

Currently, in the x/sys/unix package, there are SetsockoptLinger() and SetsockoptTimeval() for setting the Linger and Timeout options, however, there is no corresponding getter for them.
Due to the lack of getters, there's no viable way to get those socket options (i.e. linger and timeout), unless doing syscalls myself, which is not preferable.

I don't think this needs to be a proposal, especially since it is concerning x/sys/unix. Feel free to send a CL adding these two functions.


Introducing logging system

Summary
This PR introduces logging system which makes possible for library users being aware of IS Core modules activity and to have a visible and comprehensible error output.

Only feedback is to add more context for the log.error (the class / func )


bug: Collection Repeat Broken On RC V1

Type: bug
Platform: mobile webview
I Just upgraded to v1.0.0-rc.1. I was on beta 14 previously before that. I have a collection repeat which was working fine. After the upgrade, it seems the collection repeat is now broken because after I scroll around 10 items, I start seeing items that were already there before and then empty entires before
I start scrolling into a blank space.

Greetings @saniyusuf!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Is it possible to see this in a CodePen (or similar)?
I have working collection-repeats (with 100s of items) in RC-0, so it would be good to spot what could be up.

Yes, please post a codepen. I'm not seeing any problems.

A code pen is going to be thought because the collection repeat is working with a custom component of mine. I am trying to upload a video to demonstrate the behaviour.

Here we go for a video @ajoslin  @toddhalfpenny  https://www.youtube.com/watch?v=s8oZnzKFuko&feature=youtu.be

the problem you might find is that you need to recompile your sass files in order to get the css needed for the collection repeat to work.

stellasoft-will
http://workservis.ru/pribyl
24.03.2015, 19:24, "stellasoft-will" notifications@github.com:the problem you might find is that you need to recompile your sass files in order to get the css needed for the collection repeat to work.
—Reply to this email directly or .

That's definitely a problem. Is collection-repeat necessary? Perhaps native scrolling will work better and not have the updating/watch cycle issues we're seeing.  Is anyone else seeing this?  It'll be difficult to fix without being able to reproduce it.

I am currently investigating to ensure that this error has nothing to do with my setup 100% and also trying some of your feedback. After this is done, I will try and work on a demo to reproduce this on a code pen

Sorry guys, I have found the problem. School boy error on my side. I did not recompile my SASS so the new collection repeat styles were not present. All solved now


Archive and lambda functions

I try to add arrows to my plots (see #34). The archive cannot handle lambda functions:
hv.archive.auto()

def add_arrow(plot, element, xy, xytext, arrowprops=dict(arrowstyle="->", connectionstyle="arc3")):
    axis = plot.handles['axis']
    axis.annotate("", xy=xy, xycoords='data',
          xytext=xytext, textcoords='data',
          arrowprops=arrowprops)

%%opts Curve [final_hooks=[lambda *args: add_arrow(xy=(0,0), xytext=(1,0), *args)]]
hv.Curve([1,1])

PicklingError: Can't pickle <function <lambda> at 0x7f757e56c938>: it's not found as __main__.<lambda>

Without special magic, lambda functions cannot be pickled in Python , so you'd need to (a) make a named function in a well-defined location (an importable module) for your hook, and (b) you'd need to make sure that this module is available for import later when you read the archive.
We've made hacks to pickle the bytecode for lambdas in some projects, but that's not a robust approach in general...

Is there a way to not even try to pickle lambda functions while archiving the notebook? Most of the time, I use the archive function to have:

A fast loading overview of the notebook contents
SVG export of all images

It might be handy to provide a parameter to only warn on unpicklable objects, rather than failing?

I agree! I'll assign myself this issue in order to make sure warnings are issued instead of failures.
In addition, I also wrote the following reply during the brief GitHub outage:
Your example sets a lambda in a plot option - we could consider trying to strip out lambdas when pickling but then the plot would change when unpickled.
Are you using the pickles at all? From the sound of it, you might just want the SVG export. This might be an argument for removing the pickler by default:
class NotebookArchive(FileArchive):
    """
    FileArchive that can automatically capture notebook data via the
    display hooks and automatically adds a notebook HTML snapshot to
    the archive upon export.
    """
    exporters = param.List(default=[Pickler])

If you only want the SVG output, you can try removing the Pickler from hv.archive.exporters.

Sounds good. I guess for my own purposes I'd want the "pickle_failures_as_warnings" flag to be on by default, since I'd rather have a run succeed but possibly have some un-unpicklable state than to have the whole thing fail for a reason that usually doesn't matter.  But different people might have different needs, and so might want to re-enable the current behavior (e.g. if they have set up an automated document-building workflow crucially relying on the full objects to be unpicklable).

And of course those users might want dill or cloudpickle support, as Philipp suggests, because the pickles would then be for enabling a specific workflow rather than for long-time archiving.  But as I don't think any of us are in that category, maybe best to wait on such super-pickle support until it's clear that someone actually needs such potentially problematic functionality.


Is it safe to open multiple stores?

This is a question
Currently I have an app with multiple repositories

each repository handles its local data source
previously I had a custom HiveLocalDataSource base class that was implemented by all local data sources
I'd like to replace this current system with stash

For example my new local data source for my account_repository will look like this.
final accountStore = await newObjectboxLocalCacheStore(
    fromEncodable: (json) => Account.fromJson(json),
  );

  // Creates a cache with a capacity of 10 from the previously created store
  final cache = await accountStore.cache<Account>(
    name: 'cache1',
    maxEntries: 10,
    eventListenerMode: EventListenerMode.synchronous,
  );

Is it safe to do this in all my repositories or is it more efficient to have one store shared through the entire app?

In my honest opinion it’s probably more performant and memory efficient to use just one store However it all depends on your app constraints as it could make sense to separate. I wouldn’t do it though without a strong reason. This is valid mostly for persistent backends something like memory is very light
For example I have one app where I use a vault just to store assets ( images / binaries etc ) and a custom database for the metadata.
It all depends on your scenario but I don’t see any technical constraints to do not do it

If I did use one store wouldn't i loose the automatic deserialization?

Ha I see it’s different types ? Meaning you need different fromEncodable functions per store ?
If thats the case yes you will need different stores. I have to say that this is a limitation of the library itself as in theory I should be able to allow the configuration of multiple fromEncodable functions per store name ( partitions ). I will consider the implications of doing that, seems a nice feature to add.
Answering directly with the current implementation, you need multiple stores if you have different object types. Bear in mind that even on a scenario where I hypothetically  support  multiple fromEncodable functions you would always loose the generic option as you will not be able to specify a type (as you can have multiple underneath )

Yes let met look at this as it seems fairly easy to do, however something tells me that at some point in time I had already looked at this but for some reason I didn't implemented it.
Let me look at it on this weekend to see if there's any blocker for this work

Hi from what I was able to investigate this is possible to do. I will add this feature at vault / cache creation. It still needed a couple of days to do this due to the number places that this changes

That's awesome news, I'll be ready to test anytime.

Thanks, currently on holiday, will test tomorrow

Seems to work just fine, want me to leave the issue open until you find time to add those tests?

Thanks you can close it. I will try to add it in a future release


APOLLO_SV count vs STATO count?

Hi folks, I was perusing your ontology and attracted to a number of APOLLO_SV terms - population census etc - that I'd import into GenEpiO.  One question though, I saw that you had a "count" term very similar to STATO's, and was wondering if this term was deliberate or an opportunity (MIEROT) for reuse.
Which raises another question - are there plans to make APOLLO_SV part of OBOFoundry / OntoFox  (I just checked and couldn't see it there)?
Regards,
Damion

Hi Damion, I believe the main difference between our class and theirs is that, by asserting that a 'count' is the specified output of 'counting'

Hi Damion, I believe the main difference between our class and STATO's is that we assert that a 'count' is the specified output of 'counting'. In doing so, we specify that the count is about a finite set of entities (this is subsumed from the definition of 'counting'), rather than it being an exhaustive count of every individual of a particular class.

Good to tackle that fundamental definition of counting. I spotted the "count of simulated population" comment that result could be fractional.  Makes me also think of other things reported as counts, e.g. cups of flour left in this bag, or cups of coffee consumed by Fred on Tuesday, which might be reported as fractional, e.g. 1.5 cups, but we'd call those counts too.  So do you want a separate definition for counting where the input to the process is known to be a set of whole things, and another definition where the units of measure in the counting process are of a scale that allows for rounding?  (There'd be no fuss if the subatomic weight of everything was known.)
FYI, in GenEpiO, I'm also trying to anticipate the need to describe census counts at points in time and averaged or simulated datums of various counts involved in epidemiological outbreaks.

Hi Damion, I apologize for the late reply here. This is something that I'll have to take a look at in more detail. I believe that count data are generally considered to be non-negative integers, and therefore that counts of simulated populations also take the form of non-negative integers.
The fractional "counts" that the comment is referring to, I'm assuming, would be the output of some calculation that takes as input the simulated population counts (e.g., the counts of both the simulated susceptible population and the simulated infected population). Therefore, these fractional "counts," I believe, are in fact a different type of measurement data. Wikipedia describes this pretty well, as does this slide set that I found online.
As for the examples that you mentioned, I believe they would be the outputs of some sort of measurement process other than counting, and therefore would also not be considered counts.

Ok, I get how certain things like 'degrees of freedom' are intrinsically indivisible, as are raw counts of countable entities. Thanks for the links, I'll have a look. Am pondering edge cases where model/simulator might benefit from having fractional numbers rather than rounding issues presented by counts over time. Sounds though like that forces stats to be separated out to a separate layer that works only on rounded numbers.  Oy!


Fix segwit redeem scripts

For segwit we need to use the WitScript property instead of ScriptSig

@miladsoft I am gonna push this so you wont have conflicts


Evaluation request: Phind/Phind-CodeLlama-34B-v2

I'm curious how well https://huggingface.co/Phind/Phind-CodeLlama-34B-v2 scores on your leaderboard, as it's currently one of the best offline code generation models.

Thanks for bringing this up. Kindly find the evaluation results for Phind-CodeLlama & Mistral-7b on https://evalplus.github.io/leaderboard.html.
Interestingly, if we evaluate Phind-CodeLlama on HumanEval, it is ranked as No 4 (worse than ChatGPT & WizardCoder). But on our rigorous HumanEval+, it is ranked as No 2! This might imply that Phind-CodeLlama is good at producing higher quality code which better deals with edge cases.

How to do an evaluation test on a 34b model? I tried on a single A100 with 80GB VRAM but got an out-of-memory (OOM) error. I saw information that this test must be run on a single GPU, so how can I do it for a large model?

@ganler I did today own evalplus test with 0.7073 score !
(base) root@C.7286656:~/kamil/evalplus$ evalplus.evaluate --dataset humaneval --samples ../workdir/humaneval/phind-code-llama-34b-v2_temp_0.0-sanitized/
Load from ground-truth from /root/.cache/evalplus/2670b22d020cab45ede5e8285730b6ee.pkl
Reading samples...
164it [00:00, 726.60it/s]
100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 164/164 [00:03<00:00, 53.42it/s]Base
{'pass@1': 0.7073170731707317}
Base + Extra
{'pass@1': 0.6707317073170732}


[Bugfix] Fix unexpected failure of dev1.1.2

Proposed changes
Issue Number:#11681
Problem summary
Add some codes which was omitted when cherry-pick commit to dev1.1.2
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[x] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

@yiguolei @morrySnow


Checkpoint files missing

Can you please provide the checkpoints files required for test.py?

I am also looking for the same can you please provide the checkpoints files?


vscode python can't jump to some packet function impleament of sklearn ,etc, sklearn.model_selection.train_test_split. but pycharm could

Environment data
VS Code version: Code 1.22.2 (3aeede7, 2018-04-12T16:32:53.389Z)
OS version: Darwin x64 15.5.0
Actual behavior
vscode python can't jump to some packet function impleament of sklearn ,etc, sklearn.model_selection.train_test_split. but pycharm could
Expected  behavior
I hope that i could jump some function implement of sklearn freely
Steps to reproduce:

it can go to the init.py of this packet Folders, but cannot from declare in init.py to function implement in other file in the same Folders.

2、but ,,, can't go to the function impleation in _split file, below as folows: (ps : this is my manul open _split file)

I hope your team could solve this BUG ，Thanks, I reality love this extension, so i hope it will better
😄 @brettcannon @smortaz @DonJayamanne

I just jump implement function by push   Ctrl + Click  down

Just to help clarify what the post is saying: they want to jump to sklearn.model_selection.train_test_split which is exposed from sklearn.model_selection._split as a direct import into sklearn.model_selection.__init__.
IOW, to get back to train_test_split it would go through personal code -> sklearn.model_selection.__init__ -> sklearn.model_selection._split due to the import in the __init__.

Duplicate of #1224


extent example or documentation on how to create multiple actions

The documentation describes the webPubSub binding with an example, by context.bindings.actions = { "actionName": "sendToAll", ...
The name of the webPubSub binding is "actions" - it's plural - this implies, it is able to hold multiple actions? (e.g. sending a message to a group, sending another message to a user, aso.)
Could you please extent this example or the documentation how to do this? Should "actions" then be an array of action definitions?
Would this work?
context.bindings.actions = [{
    "actionName": "sendToAll",
    "data": "hello",
    "dataType": "text"
  }, {
    "actionName": "sendToAll",
    "data": "hello again",
    "dataType": "text"
  }
];

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

ID: 0c4c897d-50cd-34e9-c2e7-7ca235090b8c
Version Independent ID: 65d9edff-959a-e5d7-1164-947261bb592a
Content: Reference - Azure Web PubSub trigger and bindings for Azure Functions
Content Source: articles/azure-web-pubsub/reference-functions-bindings.md
Service: azure-web-pubsub
GitHub Login: @vicancy
Microsoft Alias: lianwei

Thanks for the feedback! We are currently investigating and will update you shortly.

Multiple actions are supported. See this sample
module.exports = function (context, request) {
  context.bindings.actions = [];
  context.bindings.actions.push({
    "actionName": "sendToAll",
    "data": JSON.stringify({
        from: '[System]',
        content: `${context.bindingData.connectionContext.userId} connected.`
      }),
    "dataType" : "json"
  });

  context.bindings.actions.push({
    "actionName": "addUserToGroup",
    "userId": `${context.bindingData.connectionContext.userId}`,
    "group": "group1"
  });

  context.bindings.actions.push({
    "actionName": "sendToAll",
    "data": JSON.stringify({
          from: '[System]',
          content: `${context.bindingData.connectionContext.userId} joined group: group1.`
      }),
    "dataType": "json"
  });
  context.done();
};


Give users permissions to create their own backup plans

User Story
We create backup plans for everyone here - https://github.com/ministryofjustice/modernisation-platform-terraform-baselines/blob/main/modules/backup/main.tf
This issue is to give users permissions to create their own backup plans, so if they need more flexible plans than what we offer they can create them.
User Type(s)
MP user
Value
Better user flexibility and experience
Definition of done

[ ] member pipeline permissions updated to allow plan creation and vault access
[ ] user docs have been updated
[ ] another team member has reviewed
[ ] tested in a member like account

Reference
How to write good user stories

https://github.com/ministryofjustice/modernisation-platform/pull/2843


fix date fail

Summary by CodeRabbit

Bug Fix: Corrected the date format in SQL queries from 'yyyy-MM-dd' to 'yy-MM-dd' for better compatibility.
Refactor: Updated the REGEXP_COUNT function's second argument from a date type to a timestamp type, enhancing precision.
Refactor: Changed the third argument in REGEXP_COUNT from an integer to a decimal number, improving accuracy.
Bug Fix: Fixed errors in SQL insert statements, ensuring data integrity and consistency.
Refactor: Altered date and timestamp formats in SQL queries for improved data handling.
Refactor: Modified condition in formatting.c, comparing id field instead of len field, optimizing code logic.

#506


HTTPS/SSL Blocked causing invalid API error

Getting two errors, believe they are related, though not certain.

"Service workers can only be used over HTTPS, or on localhost for development"
"{code: "auth/invalid-api-key", message: "Your API key is invalid, please check you have copied it correctly.", a: null}"

Deploying with Docker template from that SelfhostedPro at https://github.com/SelfhostedPro/selfhosted_templates, which defines this Docker config:
  {
    "type": 1,
    "title": "Reactive-Resume",
    "name": "reactive-resume",
    "description": "A one-of-a-kind resume builder that's not out to get your data. Completely secure, customizable, portable, open-source and free forever.",
    "logo": "https://raw.githubusercontent.com/SelfhostedPro/selfhosted_templates/master/Images/reactiveresume.png",
    "image": "amruthpillai/reactive-resume:latest",
    "categories": [
      "Other",
      "Tools"
    ],
    "platform": "linux",
    "restart_policy": "unless-stopped",
    "ports": [
      "80/tcp"
    ],
    "volumes": [
      {
        "container": "/usr/src/app",
        "bind": "/portainer/Files/AppData/Config/ReactiveResume"
      }
    ]
  },

To fix, seemingly need to enable SSL on the Docker image, ie, latest version, which might then fix the blocked API call, or need to provide a valid API key.
Appreciate any pointers and suggestions?

Also having these errors

SSL isn't really necessary in the docker image, you could just put it behind a reverse proxy that performs SSL termination - separation of concerns.
Re. the invalid API key, see discussion in #436

Closing this in favor of Reactive Resume v3 release.
If you continue to face this issue, please raise a new one and I'll have a look at it as soon as I can.
In the meantime, if you are able to, please consider donating to keep supporting the project: https://www.buymeacoffee.com/AmruthPillai :)
Thank you!


force pull

As a user, I want to be able to pull commits from a remote dot, even though I have uncommitted changes or local snapshots since I cloned/last pulled from the remote dot, so that I can discard unwanted local changes and re-synchronise with a remote dot.
This can be worked around by stopping local containers, dm reset --hard HEAD, then dm pull, but that's multiple steps.
Therefore, we should allow a dm pull -f or similar that does a pull, overriding the checks against local divergence. It can seamlessly do the the Docker stop-and-restart containers thing just like dm checkout does when switching branches, for extra ease of use.

TO be turned into product roadmap


Add phpsgi format

Related example: https://github.com/phpsgi/Funk/blob/master/app.phpsgi

I just fixed the tests. could you take a look? thanks!

I'm afraid the example you link to is literally the only file on GitHub with that extension and so doesn't meet our contributing guidelines for adding a new language.
If you'd like to override the language detection of files like these in your repos then please consider using the Linguist overrides


⬐ smarter wget

If for some reason make will be interrupted, wget will try to download .zip file and cause the name is taken it will save it to .zip.1 which wont be recognized. This fixes it + you save some internet bandwidth by continuing the download.

neat! Thanks.


Package Repository Information

Feature Request 🛍
Suggested by @steelbrain, add repository information to package.json, see npm's documentation
Use Case
It'll point people discovering you packages on NPM to you Github repository
Required Repositories

[ ] ergo
[ ] cicero
[ ] concerto
[ ] markdown-transform
[ ] web-components

Hello @irmerk I'm interested in this 🙂

I had seen this repositories:

ergo
cicero
concerto
markdown-transform
web-components

and found that these repositories ( ergo,  cicero, markdown-transform) had their repository information to package.json
So I will make PR in  concerto & web-components soon.

Given that the linked repositories contain mono-repos, I think the directory attribute is missing

Should this be re-opened?

May I work on this?

@irmerk, I think we can close this now...


Patches are already applied when running make generate-release

See https://github.com/openshift-knative/eventing-kafka-broker/blob/bfbcd5a2d87e6614ea2903142daa7c34572c271f/openshift/release/generate-release.sh#L7-L8
Signed-off-by: Pierangelo Di Pilato<EMAIL_ADDRESS>

/cc @aliok


A question about testing 3DGS on human dataset

Hello, thank you for your contributions to the open-source community. I recently wanted to test human body data on 3DGS, but COLMAP is not working properly.
I noticed the code you shared at this link. This code has been very helpful to me.2
I have a small question: if the internal and external parameters are known but there is no sparse point cloud（from sfm）, is the initialization of 3DGS random?

In such a case, you can initialize from randomly sampled points from SMPL. However, there is no significant difference between the initializations from SMPL or at random.


Rancher 2.X Automatic Rebuild after node is unavailable past timeout

When using a node driver it is easy to rebuild a node using the UI or API by just deleting it from the pool which causes it to automatically to start provisioning a replacement. It would be awesome if I could have Rancher automatically delete/rebuild a node if it is unavailable for too long (a configurable timeout)

Close in favor of https://github.com/rancher/rancher/issues/15737


 I want more traffic to head to head to Gateways on one particular cluster for a particular amount of time, based on a schedule. This could be because I have extra resources for a period of time that I wish to utilise, or a particular event may be occurring where I want to ferry more traffic to a larger cluster.
This proposal introduces a schedule field to the DNSPolicy spec, which allows users to define time periods during which the policy should be applied.
Goals

Provide users with the ability to specify when a DNSPolicy should be active
Offer flexibility in defining schedules, e.g., specific days of the week, times of the day, or even specific dates.

Proposed Changes
Add a schedule field to the DNSPolicy spec:
apiVersion: kuadrant.io/v1alpha1
kind: DNSPolicy
metadata:
  name: prod-web
  namespace: multi-cluster-gateways
spec:
  ...
  schedule:
    - days: [Monday, Wednesday, Friday]
      times:
        - start: "08:00"
          end: "17:00"
    - dates: ["2023-12-24", "2023-12-31"]

In this example:

The policy is active on Monday, Wednesday, and Friday from 08:00 to 17:00 UTC.
Additionally, the policy is active all day on the 24th and 31st of December 2023.

Fields

days: A list of days in the week when the policy should be active.
times: A list of time intervals during which the policy should be active. Each interval has a start and end time. These should be UTC.
dates: Specific dates on which the policy should be active.

Behaviour

If no schedule is provided, the DNSPolicy is always active (current behaviour).
If a schedule is provided, the DNSPolicy is only active during the specified days/times/dates.
If the current time does not match any schedule, the DNSPolicy should be considered inactive.

Validation

The days field should only accept valid day names (e.g., Monday, Tuesday).
The times field should only accept valid time formats (HH:MM) and the end time should be after the start time.
The dates field should only accept valid date formats (YYYY-MM-DD).

Alternatives
An alternative approach to applying scheduled policies would be to leverage Kubernetes CronJobs. A CronJob creates Jobs on a time-based schedule, as specified by the standard cron format. In the context of our use case, a CronJob could be set up to activate or deactivate the DNSPolicy by modifying the relevant Kubernetes resources at the specified times.
The downside here is that a user would have to manage two resources, with an intersection between the two.

@maleck13 idea I was kicking around after that demo a few weeks back.

Interesting. One immediate question that comes up, is if I don't have a DNSPolicy that is active what is happening with my DNS?

Thought about this a bit. In the simplest case, this would be the equivalent to having no policy at all.
I think in most use-cases, users would probably define a "default" DNSPolicy, whereas scheduled ones would be additive. When ones with schedules are inactive, the behaviour would revert to that of a default DNSPolicy. By default here, I think we'd look at that as part of the same CR (rather than users having to create 2x policies). That does make the spec balloon in complexity a bit, though.
One other consideration is users would probably want some kind of alerting or metrics to be aware of transitions between policies, or when a policy is about to become inactive.

Some sort of example of what I'd been thinking of:
apiVersion: kuadrant.io/v1alpha1
kind: DNSPolicy
metadata:
  name: prod-web
  namespace: multi-cluster-gateways
spec:
  targetRef:
    name: prod-web
    group: gateway.networking.k8s.io
    kind: Gateway

  loadBalancingPolicies:
    - name: geoUS
      geo:
        defaultGeo: US
    - name: weightedAWS
      weighted:
        defaultWeight: 120
        custom:
          - weight: 10
            selector:
              matchLabels:
                kuadrant.io/lb-attribute-custom-weight: AWS
    - name: weightedGCP
      weighted:
        defaultWeight: 255
        custom:
          - weight: 255
            selector:
              matchLabels:
                kuadrant.io/lb-attribute-custom-weight: GCP

  schedule:
    - policyName: geoUS
      days: [Monday, Wednesday, Friday]
      times:
        - start: "08:00"
          end: "17:00"
    - policyName: weightedGCP
      dates: ["2023-12-24", "2023-12-31"]

  defaults:
    policyName: weightedAWS

This is based on one of the example policies:
apiVersion: kuadrant.io/v1alpha1
kind: DNSPolicy
metadata:
  name: prod-web
  namespace: multi-cluster-gateways
spec:
  targetRef:
    name: prod-web
    group: gateway.networking.k8s.io
    kind: Gateway
  loadBalancing:
    geo:
      defaultGeo: US
    weighted:
      defaultWeight: 120
      custom:
        - weight: 10
          selector:
            matchLabels:
              kuadrant.io/lb-attribute-custom-weight: AWS
        - weight: 255
          selector:
            matchLabels:
              kuadrant.io/lb-attribute-custom-weight: GCP

of course open to other suggestions

right I see, it is a policy that allows multiple different LB policies to be embedded and then selected based on criteria. It is an interesting idea, I worry that it is making the policy quite complex.
What other options are available to users. I guess they could have multiple policies defined and create a new one and delete the old one based on the schedule. Its almost as though what you are describing is a form of meta policy that decides when a certain policy of a set of policy will be applied.

That's an interesting idea - I think I prefer the idea of a separate resource to dictate what is applied when. Keeps DNSPolicy focused on, well, DNSPolicy.

This issue is stale because it has been open for 60 days with no activity.

This issue was closed because it has been inactive for 30 days since being marked as stale.


queryflux kapacitor node.

It would be great if people could write Flux scripts for batch queries.
A QueryFluxNode that could handle these queries would be great.

@docmerlin Please link PR and close


Move from JCenter to Maven Central

Fixes #319
TODO:

[x] Wait for Sonatype to approve my ticket: https://issues.sonatype.org/browse/OSSRH-67754
[ ] Re-publish 3.0.0

fallowing this change. have issues with my migration..

See https://github.com/googlesamples/easypermissions/issues/324 for the test failures.


Reports_fetch_all

Updated GET reports to support pagination

Requests fail if anything else than nextToken is in payload
This change allows to paginate with from sp_api.util.load_all_pages
Changing default logic for base Client
init has default argument for marketplace if not provided
previously the argument would be overwritten by env variable
even when provided by the user on function call
Now only defaulting to env variable when marketplace given is None
assuming the environment variable is set

Hello @saleweaver ,
The issues should be resolved, can you review and merge? Thanks!


Helmlint supports required values

Ran into another edge case with helm lint. See the README changes for an explanation.

Merging and releasing!


Upload the data used

This material appears to have outlived the data sources used. For example https://www.transtats.bts.gov/DownLoad_Table.asp doesn't work, and the weather data isn't working for me either. Perhaps you (or anyone else) might upload copies of the data to this repo if you still have them?

I think I have fixed the airline data, you can check my fork:
https://github.com/jrybicki-jsc/effective-pandas/blob/master/modern_1_intro.ipynb
(the old pandas functions like ix won't work)


dYdX V4 orders aren't received by dydx.trade

Hi,
I've been using your dydx-tradingview-integration github repo to make trades on dydx V3. I created a new webservice and used tradingview-alert-connector to trade on dydx V4 and but the trades aren't making it to the dydx4 exchange. I'm not seeing any error messages in my webservice logs. I've provided my TV alert message and what I see in my web service logs below. Not sure what i'm missing. I'm hoping you can provide some guidance.
TradingView Alert Message:
{
"exchange": "dydxv4",
"strategy":"SOL_USD",
"market":"SOL_USD",
"size":"10",
"reverse":false,
"order":"{{strategy.order.action}}",
"position":"{{strategy.market_position}}",
"price":"{{strategy.order.price}}"
}
Webservice Log
Recieved Tradingview strategy alert: {
exchange: 'dydxv4',
strategy: 'SOL_USD',
market: 'SOL_USD',
size: '10',
reverse: false,
order: 'buy',
position: 'long',
price: '115.931'
}
strategyData { reverse: false, isFirstOrder: 'true' }
latestPrice 115.931
orderParams for dydx { market: 'SOL-USD', side: 'BUY', size: 10, price: 115.931 }

Hi,
I've made some changes yesterday,
so, could you please check if your deployed version is the latest one?
https://tv-connector.gitbook.io/docs/setup/faq#how-to-deploy-the-latest-version-of-the-tool
also, did you export your mnemonic phrase on dydx website?
https://tv-connector.gitbook.io/docs/setup/dydx-v4-setup#export-your-secret-phrase-a-set-of-24-words-from-dxdx-website-then-put-it-in-.env-file

Hi,
I can confirm I deployed the latest version. I did export my mnemonic phrase to an .env file, here is an example of what my .env file looks like, it's a fake phrase for demonstration purposes.
Required variables for dydx4
DYDX_V4_MNEMONIC=orange papaya fig kiwi cherry mango banana raspberry honeydew xigua grape zucchini elderberry cherry date papaya vanilla banana xigua raspberry fig orange tangerine honeydew

Hi,
I just deployed the latest commit and i'm getting an error now when an alert is received. I have provided it below. I also I did export my mnemonic phrase to an .env file
TypeError: Cannot read properties of undefined (reading 'address')
at get address [as address] (/app/node_modules/@dydxprotocol/v4-client-js/build/src/clients/subaccount.js:13:37)
at CompositeClient.placeOrderMessage (/app/node_modules/@dydxprotocol/v4-client-js/build/src/clients/composite-client.js:283:83)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)


Ansible galaxy node exporter download fails intermittantly

This issue crops up quite regularly on larger build out. The work around is to rerun. We should consider some internal retries.
TASK [geerlingguy.node_exporter : Download and unarchive node_exporter into temporary location.] ***********************************************************************************************************************
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "msg": "Failure downloading https://github.com/prometheus/node_exporter/releases/download/v1.6.0/node_exporter-1.6.0.linux-amd64.tar.gz, Request failed: <urlopen error [Errno 104] Connection reset by peer>"}
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "msg": "Failure downloading https://github.com/prometheus/node_exporter/releases/download/v1.6.0/node_exporter-1.6.0.linux-amd64.tar.gz, Request failed: <urlopen error [Errno 104] Connection reset by peer>"}
changed: [<IP_ADDRESS>]
changed: [<IP_ADDRESS>]
changed: [<IP_ADDRESS>]
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "msg": "Failure downloading https://github.com/prometheus/node_exporter/releases/download/v1.6.0/node_exporter-1.6.0.linux-amd64.tar.gz, Request failed: <urlopen error [Errno 104] Connection reset by peer>"}
changed: [<IP_ADDRESS>]
changed: [<IP_ADDRESS>]
changed: [<IP_ADDRESS>]
changed: [<IP_ADDRESS>]
changed: [<IP_ADDRESS>]

This is more likely related to github rate limits or connection issues between the node(s) running the download and github. we've run into the rate limit issue before when running repeated updates frequently. github download and api operations are pretty low for unauthenticated requests.
Not sure there's anything we can do to get a third party module to retry. If it happens again, I'd recommend trying to curl that file manually with some debug enabled and grab the output for additional review.


Abortion Policy (Independent Publisher)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[x] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[x] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello Partners,
Thank you very much and congratulations on your new submission. Please allow us some time while we are making sure that this connector is not violating any rights in the community. Our Certification Engineers will review your submission within 1-2 weeks.
Please let us know if you have any questions.
Thank you very much again for working with us.

@thatapiguy  Apologies for the delay in response.
We are preparing your connector for production deployment and will queue up for the next deployment schedule. Once your connector onboards the next deployment schedule, it starts to deploy your connector in our production environments that typically takes 3-4 weeks. Please note that the documentation for your connector will not be generated until the deployment process is in the final steps. Thanks.

[[certify-connector]]

Hello @thatapiguy,
I hope you are doing well.
Your connector is in production deployment and queued up for the next deployment schedule. This process typically takes 3-4 weeks. Please note that the documentation for your connector will not be generated until the deployment process is in the final steps.
Please let us know if you have any questions.
Thank you very much for working with us.


fix: when the right clicking in selection, it will not be reselected

fix: when the right clicking in selection, it will not be reselected

LGTM

LGTM


cargo test in CI is crashing on Windows with no information

I noticed the same failure on PRs #652, #654, #655.
Something is silently crashing the test harness after running only 23 of the 31 tests in tests/test.rs.
running 31 tests
test gnu_debug_fp ... ok
test gnu_compile_assembly ... ok
test gnu_debug_fp_auto ... ok
test gnu_define ... ok
test gnu_debug_nofp ... ok
test gnu_extra_warnings0 ... ok
test gnu_flag_if_supported ... ok
test gnu_flag_if_supported_cpp ... ok
test gnu_extra_warnings1 ... ok
test gnu_i686 ... ok
test gnu_i686_pic ... ok
test gnu_include ... ok
test gnu_opt_level_1 ... ok
test gnu_opt_level_s ... ok
test gnu_set_stdlib ... ok
test gnu_shared ... ok
test gnu_smoke ... ok
test gnu_static ... ok
test gnu_warnings ... ok
test gnu_warnings_into_errors ... ok
test gnu_warnings_overridable ... ok
test gnu_x86_64 ... ok
test gnu_x86_64_no_pic ... ok
error: test failed, to rerun pass '--test test'
Error: Process completed with exit code 1.

This was resolved and can be closed.


Added try except to check if file exists

Does this look good?

Does this look good?

@AthulJoshi I think @twentyse7en will let you know his suggestions.
But I was curious, why did you close this PR? :-)

It's already good!

I closed it as i hadnt used a branch. I pushed to the master branch itself... :D. So thought it was bad.. lol


Pass network errors to server instead of crashing

When erros from socket are not handled, nodejs 8 crashes with
events.js:183
      throw er; // Unhandled 'error' event
      ^

Error: getaddrinfo ENOTFOUND stun.l.google.com
    at errnoException (dns.js:50:10)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:92:26)

With this patch, network errors are passed to StunServer

Thanks, @webmaster128!


Incompatibility with tokio 0.3.0

The following code produces this error:
panicked at 'must be called from the context of Tokio runtime configured with either `basic_scheduler` or `threaded_scheduler`', /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/tokio-0.3.0/src/task/spawn.rs:131:10 PanicInfo { payload: Any, message: Some(must be called from the context of Tokio runtime configured with either `basic_scheduler` or `threaded_scheduler`), location: Location { file: "/usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/tokio-0.3.0/src/task/spawn.rs", line: 131, col: 10 } }

if let Ok((conn, mut ldap)) = ldap3::LdapConnAsync::new("ldap://<IP_ADDRESS>:389").await {
        ldap3::drive!(conn);
}

Right, the library requires Tokio 0.2. I don't think you can use a 0.2 future directly on a 0.3 runtime, you need to go through a compatibility layer. I'll push a 0.3 port on master soon.

Try 0.8.0, just published.

@eutampieri Does 0.8.0 work for you? If so, I'd like to close the issue.

If it’s not a problem I’ll try later today

So, my use case is to bind to an LDAP server inside an Actix-web route, but it doesn't work. The error now is ```
thread 'actix-rt:worker:0' panicked at 'there is no reactor running, must be called from the context of Tokio runtime', /Users/eugeniotampieri/.cargo/registry/src/github.com-1ecc6299db9ec823/tokio-0.3.1/src/io/driver/mod.rs:232:18
The code is
```rust
#[post("/login")]
pub async fn login(params: web::Form<LoginDetails>, sessions: SessionsRW) -> impl Responder {
    let username = &params.username;
    let password = &params.password;
    if let Ok((conn, mut ldap)) = ldap3::LdapConnAsync::new(LDAP_ADDRESS).await {
        // do stuff
    }
}

Well that's something else; did you integrate Actix-web and Tokio by way of actix_rt, as seen, e.g., in actix/actix-web#1283? Note that the current version of actix_rt requires Tokio 0.2, so you'll have to use ldap3 0.7 anyway.

Sorry, my bad


[BEAM-3344] Changes for serialize/deserialize PipelineOptions via jackson in DirectRunner

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

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @lukecwik

retest this please

Run Portable_Python PreCommit

Run Java PreCommit

retest this please

@lukecwik Could you please review my pull request.

Run Java PreCommi

Run Java PreCommit

Run Portable_Python PreCommit

All minor suggestions, please take a look and accept changes that you agree with.

Run Java PreCommit

Run Portable_Python PreCommit

retest this please

@lukecwik Thanks for the suggestions and time. I agree with all the changes you suggested and accepted them.

Run Java PreCommit

@lukecwik Yes, your changes will still address BEAM-3344 issue and also it fixed the failing tests of DataStoreV1Test . Thanks for helping me in this.

Thanks, everything looks good and the tests passed.


Add local cache for the data (Room)

The data is being fetched from the network every time. Cache it locally using room persistence library.
Follow the repository pattern.
Add instrumentation and unit tests

Resolved


Trapped Lootr chests aren't an option.

As per:
https://discord.com/channels/524907814603456522/675675241493692450/800872686678835210

This was fixed in a recent commit and should be in the next release.

This was fixed in a recent commit and should be in the next release.


Fix subroutines calling subroutines

Previously the inner subroutine would not be discovered

@ribrdb Done. I've added tests for compiling & assembling, though compiling was already correct it seems like a useful test case regardless.

Thanks!


Proposed 0.33.0-b5

Includes PRs: #1796, #1800, #1803, #1805, #1808, #1811 and #1812.

Current coverage is 71.29% (diff: 95.92%)

Merging #1814 into develop will increase coverage by 0.30%

@@            develop      #1814   diff @@
==========================================
  Files           869        871     +2
  Lines         69535      69944   +409
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          49363      49866   +503
+ Misses        20172      20078    -94
  Partials          0          0

Powered by Codecov. Last update d51a278...d8b85c0


feat: update "binaryTargets" reference for Linux Alpine arm64

Describe this PR

Changes

update binaryTargets's reference to reflect the newest Linux Alpine arm64 support
specify that Prisma supports x86_64 architectures by default

What issue does this fix?
Contributes to https://github.com/prisma/prisma/issues/8478

Is this still relevant? Should we resolve the conflict and merge?

Ready to merge @janpio?


clean the destination navigator or pop every previous destinations

How can we pop every previous destination from the destination navigator?
For example:
I have 3 screens/destinations
1- Login Destination
2- Home Destination
3- Detail Destination
When the user logs in, I want him to be unable to go back the login destination using the back button.
I tried this method, but unfortunately did not work:
navigator.navigate(home_destination) {
    popUpTo(home_destination.route) {
        inclusive = true
    }
}

The aforementioned code did not working. But using the navController works.
So basically when the user logs in, how can we prevent the destinationNavigator to go back to login screen using the back button?
Any help would be appreciated.
Thanks.

navigator.navigate(HomeDestination) {
    popUpTo(LoginDestination) {
        inclusive = true
    }
}

This should work. No need to use route either, there is a popUpTo extension function that takes the actual Destination 🙂
Let me know if this helps!

Thanks, it worked flawlessly.
But it did not produced the desired effect for this similar scenario:
HomeDestination -> A_Destination -> B-Destination -> C-Destination
Basically, when the user reaches the C-Destination, I want to clear the whole stack and make the user stay on the C-Destination
navigator.navigate(C-Destination) {
clearAllPreviousScreens = true
}
Is that possible?

You can use the same option but popUpTo(NavGraphs.root).
From memory, this should do what you want. 🙂

Thanks. That's what I was looking for.
 fun navigateAndClearPreviousDestinations(destination: Direction) {
            destinationsNavigator.navigate(destination) {
                popUpTo(NavGraphs.root) {
                    inclusive = true
                }
            }
    }


Kerberos support

Currently, djangoldapdb only supports anonymous and non-anonymous simple binds. The underlying python-ldap does support Kerberos (SASL GSSAPI) so we should do so, too.
There are at least two scenarios:

The webserver hvaing a static Kerberos ticket
The browser passing a Kerberos ticket through the webserver to Django

Both cases should be supported

I agree. The first case should be relatively easy; but the second would be much more complex.
However, that second approach might make sense for other DBs as well — one might want to have a connection to a PostgreSQL DB authenticated with the actual user's credentials.


[FLINK-2639] Add repository for hdp specific jetty to 'vendor-repos' profile

This adds a second hortonworks repository to the vendor-repos profile.
Certain HDP-specific Hadoop releases depend on
[INFO] |  +- org.mortbay.jetty:jetty:jar:6.1.26.hwx:compile
[INFO] |  +- org.mortbay.jetty:jetty-util:jar:6.1.26.hwx:compile

Which is only available in a special Hortonworks repository.
The dependency comes from org.apache.hadoop:hadoop-common:jar:<IP_ADDRESS>.2.6.0-2800:compile.
Usually, we exclude jetty and other libraries which are only needed for Hadoop's web interfaces.
In this case, the dependency comes from the flink-shaded-include-yarn-tests: The module builds a full shaded Hadoop package. For the flink-yarn-tests we need to also have jetty in the dependencies, because the YarnMiniCluster is also starting web servers (and we actually use these to test that Flink's web interface proxy is properly working).
This change does not affect the regular builds, because the additional repository is "hidden" by the maven build profile.

Looks good, +1 to merge

+1
Will merge it in the next batch


Contributing instructions for changing flutter_tools

Would be nice to write down somewhere what are the proper steps for making changes to flutter_tools.  Including things like running w/o a snapshot, running tests, etc.
See https://github.com/flutter/flutter/blob/master/CONTRIBUTING.md for inspiration (honestly could probably use an update too).  This came up due to the recent addition of several contributors to the flutter_tools section of our code (internal and external).
Either a README.md in the flutter_tools package or a wiki article on flutter/flutter seem fine.  Just a place to put this stuff would be good.  I don't personally know the right steps for running flutter_tools w/o the snapshot for instance. :)
@devoncarew @danrubel @johnmccutchan

Added more info in https://github.com/flutter/flutter/pull/36560


Testing reservations

This Branch Implements

Tests for Total Reservation Counts.
Test for Adding Counts in Reservations.
Test for Adding null.

Cheers guys 🍾

**Hi @shuja-shah **
Amazing Work! 🎉

Your project is nice!  There is nothing else to say other than... it's time to merge it 🚀 :
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


React-native: Async function swallowing console error

Describe the bug
When an exception occurs in a component function with the async keyword in the signature, the error is not being logged to the console.
Steps to reproduce

Pull down this repo's master branch and run (linked below).  This is a clone of the rn063example folder with the changes in App.js to reproduce what I am experiencing.
When the app is running, click the red text button.
Doing this is when I did not see the console error.

Environment

Bugsnag version: 7.3.0-alpha.12 (originally experienced on 7.5.2)
Browser framework version (if any):

React: 0.63.0 (originally experienced on 0.63.2)

Device (e.g. iphonex): iPhone 11 (13.2.2) and Pixel 3 XL (API 29)

Example Repo
https://github.com/ShawnCholeva/bugsnag-async-func-error
Example code snippet

Error messages:

Just to add more context, I pulled the repo down again and noticed that the "Send Handled JS Exception" and "Trigger Promise Rejection" button from the rn063example

Hey @ShawnCholeva, thanks for raising providing the repro. Confirmed the behaviour you're seeing. We'll take a look into this.
When running without Bugsnag enabled, we'd expect to see a warning when a possible unhandled rejection occurs:

With Bugsnag enabled, this unhandled rejection error does get reported to Bugsnag, but it's hidden from the console.

@xander-jones  Also just ran into this. I'm on<EMAIL_ADDRESS>This bug cost me a bunch of time. My async code was throwing an exception during development and bugsnag was swallowing it. It was sent to BugSnag but async exceptions no longer get logged to console. It took me awhile to figure out because of all things I except an uncaught exception to never just disappear!
IMO this needs to be a high priority bug. It's going to seriously trip up a lot of people's development workflows. What's the last known working version so we can downgrade until there's a fix?

Hi @AndrewMorsillo,
This behaviour will be happening for all versions of @bugsnag/react-native so unfortunately there is no known working version until we can resolve this.
What you could do, is switch off the unhandledRejections error type when in development which may help somewhat in the meantime.
https://docs.bugsnag.com/platforms/android/configuration-options/#enablederrortypes
https://docs.bugsnag.com/platforms/ios/configuration-options/#enablederrortypes

Any update on this @johnkiely1 ? This continues to be a pretty big irritation.

@AndrewMorsillo, this is next up on our to-do list. We'll keep you posted in this thread when an updated release is ready.

Fixed in v7.6.0

Fixed in v7.6.0

@mattdyoung this seems like it was improved in v7.6.0 but not really fixed. There is now a console log for Possible Unhandled Promise Rejection (id: 1): and a stack but it still swallows the error message. That means you need to log into bugsnag to read any async error messages which isn't ideal. Can this be reopened until error messages are no longer being swallowed in development?


Find or create Invenio Json Schema

Please link the resources you used to create the current state of this repo @HLWeil , i'd guess there are some kind of docs on the json they expect at least?

https://inveniordm.docs.cern.ch/reference/metadata/#overview
Here you can find the documentation about the specification. I took this as basis for the current state of the converter

looks like this is the correct schema to validate against: https://github.com/inveniosoftware/invenio-rdm-records/blob/master/invenio_rdm_records/records/jsonschemas/records/record-v5.0.0.json

The json schemas there use a weird way of referencing local files, which are not valid according to json schema. i opened an issue here: https://github.com/inveniosoftware/invenio-rdm-records/issues/1288


MMB: Add expand button with icon under sector info

Add animation for icon on click

(left iOS simulator, right figma design)

Arrow is not is not vertically centered on iOS

Arrow is not is not vertically centered on iOS

Yes, I forgot that, thanks

@jvaclavik @JoseDuSV 🚀


White Vegetable Oil Ink should be made with Saltpeter

Nether Quartz seems a bit too expensive for white dye, as you need to go to the Nether for it. I think saltpeter would fit, what do you think?

Fair enough. Nether quartz was kind of just a "it's white-ish so I guess that works" sort of thing because vanilla Minecraft doesn't have a source of white pigment (titanium, lead, zinc, lime/chalk).

Saltpeter IRL cannot be used to make dye. Why not some other pigment in the overworld?

https://en.wikipedia.org/wiki/List_of_inorganic_pigments#White_pigments

@Sunconure11, any suggestions for which white pigment should be added and how? I couldn't figure out how to do it without adding something to worldgen.

Why not have it obtained in a fashion similar to mining sulfur and saltpeter, except from stone? To compensate, it should be fairly rare.
Or even a process something like this: https://en.wikipedia.org/wiki/Zinc_oxide#Production

While I think adding drops to other blocks is an okay solution, I don't want to overdo it, and stone already has a rare drop of bones. I also would like to avoid adding something that is only used for a single recipe and has little use outside of that.
Would it be acceptable to keep the current nether quartz recipe as the only recipe when VO is installed on its own, and then rely on other mods to supply other, easier, forms of white pigment (titanium/zinc/lead/etc)?

That could work. You could try maybe oxeyes or some other white flower.

Okay, maybe I'm late to the party here, but Titanium/Zinc could be a drop from Iron Ore... now to find another use in it. Maybe it could be used in the crafting of Encrusted Obsidian?

That might be a good idea.

Correct me if I'm wrong, but isn't white dye technically bleach?  Most real-world whitening materials tend to be things that remove color, rather than add white.  Perhaps add a recipe for hydrogen peroxide?

Instead of coming up with a recipe to make white dye, would it be possible to just come up with some kind of bleach recipe, which can then be used on existing dyes to make them white?

I like idea of Titanium or Zinc being drops from ores rarely.

White ink has been removed in the latest version, commit 344a1db4511ed3a2b0c0d0060fe5477f9edba551, due to vanilla adding 3 methods of getting white dye besides killing skeletons - using their composter to make bone meal, mining fossils in the nether, and the lily of the valley flower.