No module named 'core'

Trying to run the bt.ipynb and get the above error
any ideas?

will try to provide more details

Hi, first all jupyter scripts need to be moved one folder higher to work (still haven't found a solution to that, the classic one is to integrate my tool in pip and let you install it from there). This is what I meant with MOVE_THEM_TO_ROOT_TO_USE_THEM.
For the zoneinfo, you are right with Python 3.8  and below you need the backports, with newer Python zoneinfo works directly.

Okay, I resolved the problem by doing the following
import sys
sys.path.append(".")

from core import strat, indicators, bt

and then modify the settings.py at line 145
from pathlib import Path

# Build paths inside the project like this: BASE_DIR / 'subdir'.
BASE_DIR = Path(__file__).resolve().parent.parent

...

if DEBUG and DEBUG!="False":
    with open(os.path.join(BASE_DIR, 'trading_bot/etc/DJANGO_SECRET')) as f:
        SECRET_KEY = f.read().strip()
    with open(os.path.join(BASE_DIR, 'trading_bot/etc/DB_SECRET')) as f:
        DB_SECRET_KEY = f.read().strip()
    with open(os.path.join(BASE_DIR, 'trading_bot/etc/DB_USER')) as f:
        DB_USER = f.read().strip()

The os.path.join(BASE_DIR, is a good idea indeed, I neglected this point.
However with the sys append(".") I am not able to start the jupyter scripts from a subordner, so I don't really see the change in comparison to before. Maybe I misunderstand your actions.

may be try it with two dots like this
import sys
sys.path.append("..")

from core import strat, indicators, bt

It did worked for me.

It made my day, many thx!

 :)


EPS: Sync OBC time to EPS, and sync EPS time to OBC

The EPS has an internal always-on RTC.

Should this just run on the watchdog thread?

Probably run in a different thread so that we can easily modify the resync period by changing the sleep period.
Let's move those two eps threads into a new file


average_backlog_size is flat

Currently, the average_backlog_size metric in the quality-of-service.rb script doesn't take into account the history of repositories. It simply checks the current average number of issues in repositories. Instead, it should return the average backlog size over the entire period of time from f.since until now.

@yegor256 Do I understand correctly that we need to find all the open issues from the date f.since and count them to the average number?

@yegor256 may I try this one?

@Yegorov I think we should enlist all days from f.since until now. Let's say there are 7 days. Then, we go day by day and check how many open issues we had by the end of that day in each repository. If we have 10 repositories, we will get 70 number. Then, we calculate the average of all 70.


[Suggestion!] "Accept Character Theft" option in Miscellaneous!

Is your feature request related to a problem? Please describe.

For role purposes, playing a thief can be a tricky thing. Trade is incredibly important to developing multiple storylines, though it doesn't seem like it's always obviously used. Stealing items, whether through TRP or through emotes can be taboo on occasion - especially if it's interfering with current plotlines.

Describe the solution you'd like
In Miscellaneous, there's a "Accept Character Death", "Accept Character Injury", "Accept Character Romance", I'd like to suggest adding a "Accept Character Theft" or "Accept Character Larceny" to the list. This helps rogue/thief characters continue to develop naturally, and give them clear storylines and abilities to follow.
Describe alternatives you've considered
At the moment this entirely relies on the working of other people -- adding notes to profiles, or adding to IC/OOC notes. This is an overlooked thing for everyone other than in a rogue type of role, and is often not put into use -- which generates a lot of footwork for RPers to ask OOC permissions beforehand.
Additional context
Add any other context or screenshots about the feature request here.

Generally we've preferred not to add more fields for that page since there's a large number of character archetypes that someone could possibly play out, all of which may have varying ideas on what options would be useful to show there.
To me personally it feels a bit like the proper solution here would be to either allow customization of that page - which would be a significant task - or to somehow push information like that into the existing "Additional Information" system on the Characteristics page.

Generally we've preferred not to add more fields for that page since there's a large number of character archetypes that someone could possibly play out, all of which may have varying ideas on what options would be useful to show there.
To me personally it feels a bit like the proper solution here would be to either allow customization of that page - which would be a significant task - or to somehow push information like that into the existing "Additional Information" system on the Characteristics page.

Fair. Though the option to disclose certain information or not seems really helpful with that. The "Do Not Show" really reduces how much is there. Granted, worst case scenario if that list entirely grows too large, you could limit the use of actual factors. Say only 5-6 options can actually show at a time.
I dunno how all that works, but I would think it'd maybe help streamline it a little bit. c:

Fair. Though the option to disclose certain information or not seems really helpful with that. The "Do Not Show" really reduces how much is there. Granted, worst case scenario if that list entirely grows too large, you could limit the use of actual factors. Say only 5-6 options can actually show at a time.
I dunno how all that works, but I would think it'd maybe help streamline it a little bit. c:


Have discord pass back the user data. Use schema to create user within database.

Task

Refactor login with Discord button into a component to improve project structure.
Pass user data from React/Discord to backend.
Create user within database on backend.

I would like to take a stab at this issue.

All yours!


ERROR: Could not GET 'httpmime-4.5.6.pom'. Received status code 502 from server: Bad Gateway Enable Gradle 'offline mode' and sync project

ERROR: Could not GET 'https://jcenter.bintray.com/org/apache/httpcomponents/httpmime/4.5.6/httpmime-4.5.6.pom'. Received status code 502 from server: Bad Gateway
Enable Gradle 'offline mode' and sync project

Enabling offline creates the issue below.
ERROR: No cached version of org.apache.httpcomponents:httpmime:4.5.6 available for offline mode.
Disable Gradle 'offline mode' and sync project

Thanks for reporting this.
I'm closing this issue because I cannot reproduce it.
Please reopen if you still think this is an issue with the sample and you can provide some reproduction steps.


Error handling issue in Timex.to_unix

Steps to reproduce

API function called: Timex.to_unix
Arguments provided: %InvalidTime{}

Description of issue

What are the expected results? - A valid exception should be thrown. Instead the following exception is thrown:
** (UndefinedFunctionError) function :insufficient_date_information.exception/1 is undefined (module :insufficient_date_information is not available)
This is happening because https://github.com/bitwalker/timex/blob/6ea19d1de8e23a789dc645d6d565bb5fa750e758/lib/convert/convert.ex#L39 returns {:error, :insufficient_date_information} to https://github.com/bitwalker/timex/blob/master/lib/datetime/map.ex#L14 which tries to raise :insufficient_date_information

Thanks for the report, I'll get this fixed.


Deserialize textpool for some objects

CI tests have shown that some text pool entries are missing i.e. there are diffs after pulling.
Fixes inserting the textpool for objects that are not handled by RPY_PROGRAM_INSERT. This applies for example to function groups and exit includes (zx...).

You're correct. But the method wasn't working if there's only the report title maintained.


Format the wasm/ directory using prettier

We could do the whole repo, but I figured it's best to restrain it to files that are more frequently edited/read; the actions workflow yaml files' formatting doesn't matter too much.

cc @mireille-raad


bug: CollectionWrapper remove method doesn't work

Hello,
We were trying to use the remove() method available on the CollectionWrapper and it doesn't seem to work.
https://github.com/rdkcentral/Lightning-ui/blob/1d9801d26bdcd1a4a17aec495459812702d1d918/src/helpers/CollectionWrapper.js#L132-L143
From our investigation, the culprit seems to be the that this.wrapper.children doesn't contain the property assignedID.
Using the internal get items() seems to fix the issue, because it returns the component if it is alive.
https://github.com/rdkcentral/Lightning-ui/blob/1d9801d26bdcd1a4a17aec495459812702d1d918/src/helpers/CollectionWrapper.js#L526-L534
If replacing the blocks that use this.wrapper.children with this.items seems good to you, I can provide a PR to be merged.

Thanks for bringing this to my attention. This is fixed in version 1.3.3


Point users to vagrant.log also

The majority of issues today have been vagrant related and require looking in the ~/.nanobox/apps/<app-name>/vagrant.log but there is no instruction to look there.

I'll get this in right away


Add IM/EC3 documentation

Short Description of the issue
We need some documentation on how IM and/or EC3 are used within fedcloud.

IM is covered in https://docs.egi.eu/users/cloud-compute/im/ and EC3 is covered in https://docs.egi.eu/users/cloud-compute/ec3/.
Can we close this task @enolfc ?

Done in #242


[core/ssr] Add Node builds that don't explode

Background
Before this PR, importing Lit with Node causes errors such as window is not defined. The only way to load Lit was to first ensure that the dom-shim.js module provided by the @lit-labs/ssr package was loaded, which defines window plus APIs like HTMLElement and customElements.
This meant that using Lit in frameworks such as Next did not work out-of-the-box even when only client-side rendering is needed.
This change
After this PR, Lit can be imported from Node without errors, and custom elements can be defined as no-ops.
This doesn't remove the need for loading the DOM shim shim when SSR is actually needed, but it does give us compatibility at least for client-side rendering in frameworks like Next, without the user needing to do anything special.
This works by adding a Node-specific Rollup build and a node export condition to lit-html and @lit/reactive-element. Neither lit-element nor lit need a Node build, because only the underlying 2 libraries currently need Node-specific behavior.
This only defines customElements as new globals. It does not define window, HTMLElement, or any other APIs. This way, we won't affect the behavior of libraries that detect whether they are in Node vs the browser by checking for a window global.
For testing, I've added a node-imports.ts file to each package, and added a new node:test script which executes that module with Node. This confirms that Node doesn't crash on import.
Fixes https://github.com/lit/lit/issues/3079
Followup
In a follow up, I will propose moving the actual functional dom-shim implementations of HTMLElement and customElements from @lit-labs/ssr into reactive-element (in this PR they are just no-ops) so that ultimately the user never has to load the dom-shim manually.
However, in order to do this, we will need to update the Lit SSR ModuleLoader to understand export conditions -- because currently it does not support them, so it will not find the Node build. We depend on the resolve package to do module resolution, which unfortunately does not yet support export conditions, so this work will be a little bit non-trivial.
Another followup will be to add an isSsr const or function. Rather than doing runtime detection, this can simply be true in the node build and false in the browser build.
Size
This actually slightly drops the size of the main lit-html.js and reactive-element.js production build files:

File
Old size (raw/gzip/brotli)
New size (raw/gzip/brotli)

lit-html.js
8.68 / 3.45 / 3.13
8.02 / 3.38 / 3.07

reactive-element.js
5.97 / 2.17 / 1.93
5.95 / 2.17 / 1.92

Just added a fix to the update-version-variables.js script to account for the fact that the code is sometimes now on global instead of globalThis.

Looks good.
Since we have to commit to this approach long-term, I'm mildly interested in how much code it would add if we just did the shimming in the normal codepath (and not have a separate node export condition and all the config file overhead that requires).

Below is a comparison of the sizes of the two main affected files. "Prod" is the minified browser build (NODE_MODE=false with unreachable code removed by terser). "Combined" is if we were to just have a single build (NODE_MODE = typeof window === 'undefined'):

Build
File name
Size
Gzipped
Brotli

Prod
lit-html.js
8.02 kB
3.38 kB
3.07 kB

Combined
lit-html.js
8.09 kB
3.41 kB
3.1 kB

Prod
reactive-element.js
5.95 kB
2.17 kB
1.92 kB

Combined
reactive-element.js
6.16 kB
2.26 kB
2 kB

So, a little more substantial on the reactive-element.js size.
Also I believe this difference will increase even further, because I plan to extend the Node customElement.define implementation in reactive-element.js to keep track of calls instead of being a no-op, so that they can be later replayed if the full fidelity shim is loaded later.

Cool thanks. The 90-100b on RE is definitely worth it, plus what you said about possibly expanding it a bit, so I think I'm sold.


Testing metrics via sqlmesh

One of two PRs testing different metrics setups

Intended this to be a draft. Do not merge please

Part of #1456


Unify verification messages between containers and blobs

Description
Currently, when verifying a container, you get a message saying what items were verified followed by a non-human readable string of text.
When you verify a blob, you get back the simple string "Verified OK".
The two verification processes should produce the same output, and probably should not spit out large volumes of text that are not supposed to be readable by humans.

I looked into the internal working of cosign verification for blob as well as image for key mode.

Image verification:

When we verify a image, we provide a public key and image(i.e. payload or kind of artifact). And internally cosign verifies the signature(stored in image layer) against the public key. That's it and return the output.

Blob verification:

When we verify a blob, we provide a public key, signature and artifact. Internally, cosign uses public key against the signature, which returns the hash value. And this hash value is check against the calculated hash value of a provided artifact. And if both hash matches, then artifact is verified.

Difference b/w Image verification and blob verification:

In image verification cosign doesn't matches the hash values, whereas in case of blob, it matches the hash values.
Just for additional info: In image, signature is present in image layer, i.e. within the image(i.e. artifact) itself,  whereas in blob it's not the case.

Now, come back to the issue: What we can do to make the o/p of verification of blob and image as the same:

In image verification, we can configure cosign to go one more step ahead for verification and that is to verify the hash value. We know when public key is check against the signature, it must returns the hash, which is actually the hash of payload. Cosign can internally calculate the hash of payload and match them. If they does, that means verification success and simple return the same o/p for blob i.e. Verified OK.

I tried to verify image like verifying blob by providing payload(i.e. artifact), public key, and signature:
$ cosign verify-blob --key cosign.pub --signature signature payload
Verified OK

NOTE: I didn't sign the image as like blob, i simply extracted those values during normal image signing process.
$ cosign sign --key cosign.key --output-payload=payload --output-signature=signature  viveksahu26/hi-cosign:main

Hey @haydentherapper @cpanato @hectorj2f need your suggestion on this. Would be great to hear from you all.

Rather than simply output Verified OK,

But that what an user want. Complete verification to be done internally and no machine-readable o/p to be thrown until and unless user explicit want that.  The blob verification whether using key or keyless follows the same simple o/p in one line Verified Ok. Therefore same,  could be applicable for image as well.

I'd rather look at structuring the output to be machine readable, and contain information about what steps were taken during verification (key vs cert, log verification, timestamp verification etc).

This seems to be heading towards inspect command. So, we can do one thing is, to add support  of a flag --inspect in verify command. If the flag is true then show the whole o/p in detail. Whether we can show the same o/p which currently image verification shows or add more detail to it  later that inspect supposed to show. And if flag is false, then simple print Verified OK.

I'd rather look at structuring the output to be machine readable. This relates to #2210, I could imagine similar output.

I didn't get about machine readable format. AFAIK, the machine o/p is similar to below o/p or something else you mean to.
{
    "critical": {
      "identity": {
        "docker-reference": "index.docker.io/viveksahu26/hi-cosign"
      },
      "image": {
        "docker-manifest-digest": "sha256:2a82a3d5bfda5c43d2e0e8880a7044985d6839e3c892f2b0c1fd0b010a0eedcc"
      },
      "type": "cosign container image signature"
    },
    "optional": {
      "<IP_ADDRESS>.4.1.57264.1.1": "https://accounts.google.com",
      "Bundle": {
        "SignedEntryTimestamp": "MEQCIG1746zc7KR8YOEArWLpHFEFI6hxLUZcBkSdWj8ahOO3AiBlLPsKOdZocMb5tUXgJFJLLJ3vgc3bGFCAJNM1ok8JUA==",
        "Payload": {
          "body": "eyJhcGlWZXJzaW9uIjoiMC4wLjEiLCJraW5kIjoiaGFzaGVkcmVrb3JkIiwic3BlYyI6eyJkYXRhIjp7Imhhc2giOnsiYWxnb3JpdGhtIjoic2hhMjU2IiwidmFsdWUiOiIzNTg2Y2ZiOTU0ZTgxYTU0Yzg3NzRiZWY1MjhhZWVhYTQ5OGExNWRkZDQxNjU2MWRjZjZlN2VlYmU2MTUzNjVhIn19LCJzaWduYXR1cmUiOnsiY29udGVudCI6Ik1FWUNJUUNZUjFCZlFmSi96c3FKeEgxZ2VTOFFlM2JQZWpiQ0xrRHN2ZDFzdjg2RUt3SWhBTGRINzcwdzdjNi9NN0hCd3pVZWhFck1iWFhJNUNCV2N1aG1RdnhEKzlMWCIsInB1YmxpY0tleSI6eyJjb250ZW50IjoiTFMwdExTMUNSVWRKVGlCRFJWSlVTVVpKUTBGVVJTMHRMUzB0Q2sxSlNVTXhWRU5EUVd4NVowRjNTVUpCWjBsVlNWUXJWbUpDTW01cFVHWTBaM0ZDWmpkMmRtNU9SRWxxYjNWQmQwTm5XVWxMYjFwSmVtb3dSVUYzVFhjS1RucEZWazFDVFVkQk1WVkZRMmhOVFdNeWJHNWpNMUoyWTIxVmRWcEhWakpOVWpSM1NFRlpSRlpSVVVSRmVGWjZZVmRrZW1SSE9YbGFVekZ3WW01U2JBcGpiVEZzV2tkc2FHUkhWWGRJYUdOT1RXcFJkMDFxUlhsTlJGbDVUVlJGZDFkb1kwNU5hbEYzVFdwRmVVMUVXWHBOVkVWM1YycEJRVTFHYTNkRmQxbElDa3R2V2tsNmFqQkRRVkZaU1V0dldrbDZhakJFUVZGalJGRm5RVVZSS3k5SmFHTlBRUzl4WnpsTE1VTnVTa1pzTkdSR1VXeFhkRkZUVEdReWRqWmpZMjhLWm1wWlJuQk5jRGRRV0hSS1ExQllaemN6VGtKTmNFUlhaakV2TjNsck1sRXpLMjlFTDNnNFZrOHdlVXBySzFGcWF6WlBRMEZZYzNkblowWXpUVUUwUndwQk1WVmtSSGRGUWk5M1VVVkJkMGxJWjBSQlZFSm5UbFpJVTFWRlJFUkJTMEpuWjNKQ1owVkdRbEZqUkVGNlFXUkNaMDVXU0ZFMFJVWm5VVlZ5U205MkNubHJjME0xYTFWUFVXVXlSMlV6VlZFeGIzSlFZbTh3ZDBoM1dVUldVakJxUWtKbmQwWnZRVlV6T1ZCd2VqRlphMFZhWWpWeFRtcHdTMFpYYVhocE5Ga0tXa1E0ZDB0bldVUldVakJTUVZGSUwwSkRRWGRJYjBWaldUSjRkbVJYVW5WWldGSndaRzFXYW1SWGVEQmtXRXBzVVVka2RGbFhiSE5NYlU1MllsUkJjQXBDWjI5eVFtZEZSVUZaVHk5TlFVVkNRa0owYjJSSVVuZGplbTkyVERKR2Fsa3lPVEZpYmxKNlRHMWtkbUl5WkhOYVV6VnFZakl3ZDB0M1dVdExkMWxDQ2tKQlIwUjJla0ZDUTBGUlpFUkNkRzlrU0ZKM1kzcHZka3d5Um1wWk1qa3hZbTVTZWt4dFpIWmlNbVJ6V2xNMWFtSXlNSGRuV1hOSFEybHpSMEZSVVVJS01XNXJRMEpCU1VWbVVWSTNRVWhyUVdSM1JHUlFWRUp4ZUhOalVrMXRUVnBJYUhsYVducGpRMjlyY0dWMVRqUTRjbVlyU0dsdVMwRk1lVzUxYW1kQlFRcEJXVEppSzJkVWVVRkJRVVZCZDBKSlRVVlpRMGxSUXpSTE1YazRTRlpOVkhaM056SlZjall4YmxCQlZrY3pNMGRCYlhFMVRHRmFVa3gwTWtSRFQySk5Da1pSU1doQlRteFJTbmN5VlVkdEwwaG9ibGM1V2tKUWVrSlphRmRJWTB0VWF5ODJlVlJhYVRGMFdVTnZiSEpPVWsxQmIwZERRM0ZIVTAwME9VSkJUVVFLUVRKalFVMUhVVU5OUldsemRUbEJUVTV1TkdvMlNtdDJlbkZwSzFWNE9HSktSM05qWWpkcVpsTnJLeTlhUXpKc1FtYzBSbmcwVkc5cGRuWjRjblpKT1FwalZXWndXamgwV1VkblNYZEJiR2RSVmpOdFVsTlZRU3RxYWxKa0t6QlBWVTVZYm1kRk9GQlpWVmRTWjNSbVNqWlFZMVZEVEZKUWJHeFhUMEpVYUhaMkNrMUtNa1JFZFZRd1lURnliUW90TFMwdExVVk9SQ0JEUlZKVVNVWkpRMEZVUlMwdExTMHRDZz09In19fX0=",
          "integratedTime":<PHONE_NUMBER>,
          "logIndex": 70787776,
          "logID": "c0d23d6ad406973f9559f3ba2d1ca01f84147d8ffc5b8445c224f98b9591801d"
        }
      },
      "Issuer": "https://accounts.google.com",
      "Subject"<EMAIL_ADDRESS>    }
  }
]

@haydentherapper :point_up:


Update pip.md

Added dbt-trino as a viable thing one might wish to install.
Description & motivation

To-do before merge

Prerelease docs
If this change is related to functionality in a prerelease version of dbt (delete if not applicable):

[ ] I've added versioning components, as described in "Versioning Docs"
[ ] I've added a note to the prerelease version's Migration Guide

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects
[ ] Run link testing to update the links that point to the deleted page

hi @MichaelTiemannOSC thanks for opening this pr up and improving our docs.  apologies for the delay here, we've gone ahead and merged this for you!


release: update manifest and helm charts for v1.4.0

Signed-off-by: Anish Ramasekar<EMAIL_ADDRESS>
Reason for Change:

Requirements

[ ] squashed commits
[ ] included documentation
[ ] added unit tests and e2e tests (if applicable).

Issue Fixed:

Does this change contain code from or inspired by another project?

[ ] Yes
[ ] No

If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers:

Codecov Report

Merging #1040 (b866e35) into master (d5c8c90) will not change coverage.
The diff coverage is n/a.


Blazor Working With Images Docs

Fixes: https://github.com/dotnet/aspnetcore/issues/36991
Fixes: https://github.com/dotnet/aspnetcore/issues/37016

@guardrex for your consideration. As always feel free to change / edit as you see fit 😄

I'll be OOF for a few hours this afternoon. If you're ready for me to proceed :ear:, I'll edit and ping you back this evening. I couldn't tell if you were done because of the suggestion up there for Line 187 :point_right:.

I'll be OOF for a few hours this afternoon. If you're ready for me to proceed 👂, I'll edit and ping you back this evening. I couldn't tell if you were done because of the suggestion up there for Line 187 ☝.

Yep all ready, committed the suggestion for 187 😄

Ok, thanks. I'll ping u back later today for the final look and updates.

@TanayParikh ... Let's start with this, but I'll edit it again in the morning.
btw --- You don't say where to call ...
URL.revokeObjectURL(url);

btw --- You don't say/show where to call ...
URL.revokeObjectURL(url);

There isn't really a set location to call URL.revokeObjectURL(url), it depends on when you're done with the image. For example, in a form, after the user "submits" the data, you may wish to revoke the object URL as it will no longer be needed.

@TanayParikh ... I ran into a few more nittish-new-topic things this morning on the edit, and some of this is just an FYI for future reference:

Fully working, cut-'n-paste examples are popular with readers. I go ahead here with the first (basic) example and set it up that way. With our built-in Copy code button in live docs, it's great. I've added this topic (and the download files topic) to my UE pass tracking issue for more work like this later.
I added a line about when to call URL.revokeObjectURL(), but it's still confusing: We don't say "object URL" or have a url var anywhere above this remark, so it's still not clear where/how/on what it should be called. btw ---- and along the lines of a fully-working example ---- do you have a fully working example of this component that you could paste into a comment here? Immediately before the revokeObjectURL remarks, I could add it ... "Here's the complete code for the Razor component:" ... and then show the full component that would show revokeObjectURL being called. Then, we could have it say, "In the preceding component, URL.revokeObjectURL() is called to ..... (proceed with our current text).
I moved the JS for the last two examples into a lead-in under an H2 heading. Again, this will facilitate (eventually) fully working/cut-'n-paste examples.

This looks great, thanks!
Agree copy+paste would help simplify things 😄

Do you have these last two components/JS bits in fully working form such that you could paste them into a comment here (or provide a link to a GH repo where you have them)? If not, no worries ... I'll get them set up later when I get back to this topic for more work. I'll ping u on that PR, which very likely wouldn't be until 22H1.

I had working bits aligned with my initial commit, but there's been some churn in the docs so my code isn't up-to-date anymore. I think this should be fine for now ✅


[Firestore] Google\ApiCore\Serializer fatal error

After #1210 I have come to another problem with Firestore client. I am creating a document with a bit more structure and I am getting the following error when I try to get the snapshot:
Fatal error: Call to undefined method Google\Protobuf\Timestamp::getValues() in /application/vendor/google/gax/src/ApiCore/Serializer.php on line 232
I reproduced the error with this snippet:
$firestore = new Google\Cloud\Firestore\FirestoreClient();

$firestore->collection("customCollection")
	->document("myFunkyDocument")
	->delete();

$firestore->collection("customCollection")
	->document("myFunkyDocument")
	->create([
		"my_string"       => "Some string",
		"my_number"       => 1337,
		"my_float"        => 42.42,
		"my_timestamp"    => new \Google\Cloud\Core\Timestamp(new DateTime('1989-03-31 12:13:14')),
		"my_int_array"    => [1, 2, 3, 4],
		"my_string_array" => ["a", "b", "c", "d"],
		"my_hasmap"       => [
			"hasmap_string"       => "Some string",
			"hasmap_number"       => 1337,
			"hasmap_float"        => 42.42,
			"hasmap_timestamp"    => new \Google\Cloud\Core\Timestamp(new DateTime('1989-03-31 12:13:14')),
			"hasmap_int_array"    => [1, 2, 3, 4],
			"hasmap_string_array" => ["a", "b", "c", "d"],
		],
	]);

$document01 = $firestore->collection("customCollection")->document("myFunkyDocument");

$document01Snap = $document01->snapshot(); // <= Error happens here

echo $document01Snap->exists() ? "EXISTS" : "DOES NOT EXIST -_-";
echo PHP_EOL;
var_dump($document01Snap->data());
echo PHP_EOL;

The error doesn't occur if I remove the my_int_array and my_string_array and my_hasmap from the create structure.

P.s.: Before I tested I removed my vendor directory and reinstalled the packages 🙂

I've added the print_r and it doesn't output nothing (it doesn't get executed). I also found out that if I run it multiple times I get different errors from the Serializer.
root@php-docker:/application# php index.php fire

Fatal error: Call to undefined method Google\Protobuf\Timestamp::getFields() in /application/vendor/vendor/google/gax/src/ApiCore/Serializer.php on line 232
root@php-docker:/application# php index.php cron fire loc

Fatal error: Call to undefined method Google\Protobuf\Timestamp::getFields() in /application/vendor/vendor/google/gax/src/ApiCore/Serializer.php on line 232
root@php-docker:/application# php index.php cron fire loc

Fatal error: Call to undefined method Google\Cloud\Firestore\V1beta1\ArrayValue::getFields() in /application/vendor/vendor/google/gax/src/ApiCore/Serializer.php on line 232

My apologies, I should have recognized that wouldn't work. I'll continue trying to reproduce the error and get back to you once I have an idea of what's going on. Thanks for providing the additional information.

That snippet was run with PHP 5.6.33. Currently having some problems running the snippet on the macOS due to segmentation fault. I will try to run under different versions and give some feedback.

OK I've tested on php version 5.6, 7.0 and 7.2. There were problems only on 5.6 version. I've created a Gist and docker containers where the tests were created.
I've also run the test.php on my local compuser and I get Segmentation fault. The same error is printed out when I bash into 5.6 docker container (from the gist) and run the script.
The problem is that I was unable to recreate the Fatal error on these new containers. Will try a bit more (I presume the problem is this Segmentation fault)

I was able to reproduce the error with the protobuf extension enabled on PHP 5.6. Can you confirm that you are using the protobuf PHP extension? If so, try disabling it (comment it out in the php configuration file) and see if that fixes the issue.

OK great! No problem I'm just glad that I can move further with my development 😄
Thank you for your help!

Moving this to P2 as we have a workaround :)

This issue should be fixed in the forthcoming release of protobuf 3.7.

Protobuf 3.7 has been released, closing this out.


OCM-9704 | feat: persist refreshed tokens

Modifies the behavior for token management in ROSA

Before: When the initial access token is expired, it would not be persisted to the config, ROSA would use the refresh token to get a new access token on every request
Now: New tokens are always persisted to the OCM configuration. This will allow the SDK to use valid tokens more often and prevent additional round-trips to SSO
On log polling: On every callback request from getting updated logs during a watch, tokens will now be refreshed. This is required for environments with short-lived refresh tokens such as FedRAMP.

Overall LGTM- one nitpick, and i would suggest some negative test cases but the errors that can occur seem to come from other funcs already made, or, from nil checks.
For sanity reasons, may be good to provide nil values and make sure things are fine

Fair enough! Added some additional coverage around invalid sdk connection and failure to talk to SSO.


setup github actions to help during the sprint (and after)

Hello there,
the sprint for the KB is going to see various contributions and changes, so it could be useful to have automated checks to make the KB consistent.
A few that comes to mind for markdown based project:

A spell check that works for markdown files https://github.com/marketplace/actions/github-spellcheck-action
A bad link reporter https://github.com/caltechlibrary/baler
A Markdown linter https://github.com/marketplace/actions/markdownlint-cli2-action
(optional) Adding alt text for image to improve accessibility https://github.com/marketplace/actions/markdown-accessibility

Let me know what you think,
Best,

I added the CI checks but made them optional since people still might want to force changes even if the linter fails. Feel free to modify any settings, I'll close the issue for now


docs: release notes for 202306.07.0

What type of PR is this?
/kind documentation

What this PR does / why we need it:
Add release notes for 202306.07.0 VHD
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
61.7%

Covered Lines:
2543

Relevant Lines:
4124

💛  - Coveralls


[SS] - Avoid using return inside CachedKafkaConsumer.get

During profiling of a structured streaming application with Kafka as the source, I came across this exception:

This is a 1 minute sample, which caused 106K NonLocalReturnControl exceptions to be thrown.
This happens because CachedKafkaConsumer.get is ran inside:
private def runUninterruptiblyIfPossible[T](body: => T): T
Where body: => T is the get method. Turning the method into a function means that in order to escape the while loop defined in get the runtime has to do dirty tricks which involve throwing the above exception.
What changes were proposed in this pull request?
Instead of using return (which is generally not recommended in Scala), we place the result of the fetchData method inside a local variable and use a boolean flag to indicate the status of fetching data, which we monitor as our predicate to the while loop.
How was this patch tested?
I've ran the KafkaSourceSuite to make sure regression passes. Since the exception isn't visible from user code, there is no way (at least that I could think of) to add this as a test to the existing suite.

Can one of the admins verify this patch?

Test build #3907 has started for PR 19059 at commit c20bd14.

Test build #3907 has finished for PR 19059 at commit c20bd14.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #3910 has started for PR 19059 at commit 18b9301.

Test build #3910 has finished for PR 19059 at commit 18b9301.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

By the way @YuvalItzchakov  could you make a JIRA and link it? this is small but it's a non-trivial improvement and think we should handle it as a normal issue.

https://issues.apache.org/jira/browse/SPARK-21873

Merged to master


high_volubility.py: output directory for .wav and .rttm

Is there a parameter that I am not aware of that allows the user to choose where the output wav and rttm files will be stored, and if not, would it be possible to add that parameter? Otherwise the output of one run with a set of parameters gets overwritten by the following run with another set of parameters (one fix is to have several copies of the same file in different directories to run different versions of the script on one wav file, but it is not viable for daylong files)

I added an "--output" switch


Fix Login for Newly Registered Users

Fix user's first login from after registering. Current version allows any password as valid.

Issue closed.


Parse pre-completion cells in proper sorts.

When generating the modified productions that allow incomplete cells
with features such as dots and omitting parent cells,
the modified productions for indvidual cells were places into the
generic Cell sort (used for flexibility such as allowing parent cells),
this instead keeps the original more specific FooCell sorts,
for which subsorts into Cell were already generated.
@daejunpark  this should fix #1748 #1747
I need to add tests, but you can check if this commit helps.

With this code there are no parse error on your test module, with necessary imports added as below:
require "domains.k"
module TEST
imports MAP
imports INT

configuration <T>
  <k> $PGM:K </k>
  <x>
    <a> 1 </a>
    <b> 2 </b>
    <c> 3 </c>
  </x>
</T>

syntax KItem ::= frag(XCellFragment) | "run1" | "run2"

rule frag(<x>-fragment <a> A </a> <b> B </b> C </x>-fragment) => A +Int B

rule <k> run1 => frag(X) ... </k>
     <x>
       X:XCellFragment
       <c> _:Int </c>
     </x>

rule <k> run2 => frag(<x>-fragment A B noCCell </x>-fragment) ... </k>
     <x>
       A:ACell
       B:BCell
       <c> _:Int </c>
     </x>

endmodule

I can vaguely recall trying this in the past and running into some kind of problem on the C semantics. Can you test to make sure that make test-build still passes?

I'm testing it now. I wouldn't be too surprised if the C semantics needs some work to move away from the old assumption that cell fragments are Bags.

@daejunpark please review.
It did turn out to be a one-line change in the code.

lgtm

@dwightguth A bunch of errors seem to all involve the <curr-scope> cell, starting with these two
[Error] Critical: Unexpected sort Scope for term
#SemanticCastToScope(#token(#KVariable,"OldScope")). Expected FileScope.
	Source(/home/brandon/code/k/c-semantics/semantics/././language/common/dynamic.k)
	Location(208,24,208,38)
[Error] Critical: Unexpected sort Scope for term Scope. Expected FileScope.
	Source(/home/brandon/code/k/c-semantics/semantics/././language/common/dynamic.k)
	Location(212,29,212,34)

and then many instances like these of parse errors on uses of that cell:
[Error] Inner Parser: Parse error: unexpected character '<'.
	Source(/home/brandon/code/k/c-semantics/semantics/././language/translation/decl/global.k)
	Location(111,32,111,33)
[Error] Inner Parser: Parse error: unexpected character '<'.
	Source(/home/brandon/code/k/c-semantics/semantics/././language/translation/decl/global.k)
	Location(123,32,123,33)
[Error] Inner Parser: Parse error: unexpected character '<'.
	Source(/home/brandon/code/k/c-semantics/semantics/././language/translation/decl/global.k)
	Location(53,32,53,33)
[Error] Inner Parser: Parse error: unexpected character '<'.
Source(/home/brandon/code/k/c-semantics/semantics/././language/translation/decl/global.k)
Location(61,32,61,33)

There seem to be a bunch of unrelated errors from not allowing ",," anymore.

@dwightguth Considering I don't change at all the items expected inside the generated production, are you sure test-build works correctly on the current master of K?

Oh, I guess maybe it doesn't. Try running it from the following pull
request: https://github.com/kframework/c-semantics/pull/120
On Wed, Sep 30, 2015 at 6:20 PM, bmmoore<EMAIL_ADDRESS>wrote:

@dwightguth https://github.com/dwightguth Considering I don't change at
all the items expected inside the generated production, are you sure
test-build works correctly on the current master of K?
—
Reply to this email directly or view it on GitHub
https://github.com/kframework/k/pull/1754#issuecomment-144572115.

Jenkins: test this please.

The errors seem to come from C still using .Bag and :Bag for the absence of cells and to annotate cell bags or fragments, which is an error in the C semantics that will have to be fixed.

This commit leaves one remaining parsing ambiguity in the thrd-join rule in the threads library of C, but it happens to be resovled in favor of #cells, so test-build passes even without #1791 after adding this commit.

What's the ambiguity? And will #1791 fix it?

The ambiguity is between parsing a pair of identical cells, like the <thread>...</thread><thread>...</thread> using either production
Bag ::= Bag Bag [klabel("#cells")]
from the loose front-end syntax, or
ThreadCellBag ::= ThreadCellBag ThreadCellBag [klabel("ThreadCellBag")]
from the more strict generated syntax.
The latest commit vastly reduces the scope of the ambiguity by making the rule syntax parse cell bodies as Bag rather than K, so the second production can't apply, but an ambiguity is still possible if this situation appears at the very top of a rule, which is parsed as K. This happens in the rule I mentioned above - and in the C semantics, only in that rule.
The problem #1791 fixes is that SortCells would fail with an exception on the second parse. It doesn't touch parsing, but it does fix SortCells to handle either parse equivalently.

@daejunpark Do you have any comments on the commits since your approval?

Jenkins: test this please.

lgtm.

looks fine to merge to me.

@bmmoore I cannot find this merge in the latest master. For example, there is no cell_arguments directory in the regression test:
https://github.com/kframework/k/tree/master/k-distribution/tests/regression/kore-cell-fragments
Do you know what happened?

The lost merge has been sorted out.


Implement feature request from #61

Adds the ability to print the descriptor id instead of the defaultMessage, see #61

Current coverage is 100% (diff: 100%)

Merging #71 into master will not change coverage

@@           master   #71   diff @@
===================================
  Files           2     2
  Lines          27    27
  Methods         7     6     -1
  Messages        0     0
  Branches        6     8     +2
===================================
  Hits           27    27
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update cc2fe74...3e3e82f


Unknown syntax error when trying to pass module as named parameter

Not sure if you want these issues but the compiler told me to add it.
Repro:
let maker ::SomeModule => SomeModule.make ();

What's the message?
This is a syntax error that we should guard against, yes.

:: is not supported in Reason, please use [hd, ...tl] instead

  We've found a bug for you!
  /home/ulrik/src/error-test/src/ErrorTest.re

  There's been an error running Reason's refmt parser on a file.
  This was the command:

  /home/ulrik/src/error-test/node_modules/bs-platform/bin/refmt.exe --print binary '/home/ulrik/src/error-test/src/ErrorTest.re' > /tmp/ocamlppc236cb

  Please file an issue on github.com/facebook/reason. Thanks!

ninja: error: rebuilding 'build.ninja': subcommand failed
>>>> Finish compiling

This is a weird error as I was expecting it to be a named parameter, was trying to pass a module as a parameter.

Error after #2480:
$ echo "let maker = (~SomeModule) => SomeModule.make ();" | ./_build/install/default/bin/refmt
File "", line 1, characters 14-24:
Error: variables and labels should be lowercased. Try `someModule'


Add lateinit fields for rds instance

Description of your changes
Add additional fields to aws_db_instance lateInit.
Fixes #723
I have:

[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Tested manually using the following manifest: after 11 hours still READY/SYNCED:
 kubectl get -f instance.yaml
NAME                                                               READY   SYNCED   EXTERNAL-NAME                          AGE
instance.rds.aws.upbound.io/example-dbinstance-replicate-primary   True    True     example-dbinstance-replicate-primary   11h

NAME                                                        READY   SYNCED   EXTERNAL-NAME                          AGE
key.kms.aws.upbound.io/sample-key-replicate-rds-replicate   True    True     163b26ec-dac3-4c54-80d5-4d80d31ded84   11h

NAME                                                               READY   SYNCED   EXTERNAL-NAME                          AGE
instance.rds.aws.upbound.io/example-dbinstance-replicate-replica   True    True     example-dbinstance-replicate-replica   11h

apiVersion: rds.aws.upbound.io/v1beta1
kind: Instance
metadata:
  annotations:
    meta.upbound.io/example-id: rds/v1beta1/instance
  labels:
    testing.upbound.io/example-name: example-dbinstance-replicate-primary
  name: example-dbinstance-replicate-primary
spec:
  forProvider:
    region: us-west-1
    allocatedStorage: 20
    autoMinorVersionUpgrade: true
    backupRetentionPeriod: 14
    instanceClass: db.t3.micro
    name: example
    engine: postgres
    engineVersion: "13.7"
    username: adminuser
    autoGeneratePassword: true
    passwordSecretRef:
      key: password
      name: example-dbinstance
      namespace: upbound-system
    backupWindow: "09:46-10:16"
    maintenanceWindow: "Mon:00:00-Mon:03:00"
    publiclyAccessible: false
    skipFinalSnapshot: true
    storageEncrypted: true
    storageType: gp2
    kmsKeyIdSelector:
      matchLabels:
        testing.upbound.io/example-name: sample-key-rds-replicate
  writeConnectionSecretToRef:
    name: example-dbinstance-replicate-primary
    namespace: default
---
apiVersion: kms.aws.upbound.io/v1beta1
kind: Key
metadata:
  annotations:
    meta.upbound.io/example-id: rds/v1beta1/instance
  labels:
    testing.upbound.io/example-name: sample-key-rds-replicate
  name: sample-key-replicate-rds-replicate
spec:
  forProvider:
    region: us-west-1
    description: Created with Crossplane
    deletionWindowInDays: 7
---
apiVersion: rds.aws.upbound.io/v1beta1
kind: Instance
metadata:
  annotations:
    meta.upbound.io/example-id: rds/v1beta1/instance
  labels:
    testing.upbound.io/example-name: example-dbinstance-replica
  name: example-dbinstance-replicate-replica
spec:
  forProvider:
    region: us-west-1
    instanceClass: db.t3.micro
    replicateSourceDbSelector:
      matchLabels:
          testing.upbound.io/example-name: example-dbinstance-replicate-primary
  writeConnectionSecretToRef:
    name: example-dbinstance-replicate-replica
    namespace: default

/test-examples="examples/rds/instance-with-replica.yaml"

/test-examples="examples/rds/instance-with-replica.yaml"

/test-examples="examples/rds/instance-with-replica.yaml"

/test-examples="examples/rds/instance-with-replica.yaml"

/test-examples="examples/rds/instance-with-replica.yaml"

Thanks @stevendborrelli!
I am a bit hesitant to move forward with this PR since this is disabling late initialization for fundamental parameters, and we're moving away from XRM for a particular condition. I believe, we need a more flexible configuration API for late initialization in upjet and should rather disable late-initialization of those fields only if replicateSourceDb is set.
Just opened an issue for that: https://github.com/upbound/upjet/issues/214

@turkenh, I am considering moving this PR forward to prevent users encountering this issue from being blocked.
I will add a condition for replicateSourceDb parameter when the interface you mentioned in the issue is provided.

With managementPolicies promoted to beta, hence on by default, one can disable late initialization for a given resource as follows:
apiVersion: rds.aws.upbound.io/v1beta1
kind: Instance
metadata:
  annotations:
    meta.upbound.io/example-id: rds/v1beta1/instance
  labels:
    testing.upbound.io/example-name: example-dbinstance-replicate-primary
  name: example-dbinstance-replicate-primary
spec:
  managementPolicies: ["Observe", "Create", "Update", "Delete"]
  forProvider:
    ...

So, without disabling most of the late-initialization for everyone with this change, one can disable it for an RDS instance with replicateSourceDb.
Did we consider this option already?

any update regarding this issue?

/test-examples="examples/rds/instance-with-replica.yaml"

Closing this in favor of using managementPolicies.


using stdlib in custom webapp

I love the programing model and excellent blog post.
would be great to have ability to compose a notebook on https://beta.observablehq.com/, but then be able to ""export"" results into self hosted environment, where access to larger datasets and runtime environments is easier.
In this model https://beta.observablehq.com/ is the "exploration and composition" platfrom, working on sample data, and the self hosted environment is the runtime.
The specific request here is:

demo / blog / instructions for leveraging stdlib in a self hosted env.

I like the way of thinking and writing notebooks using modern native web technologies also very much and would also like requesting the possibility to run such notebooks in a 'local' environment, especially possibly within a 'Browser Extension'!
So a serialized version of a notebook and an 'easy to use/install/start' local environment like a 'Browser Extension' would be a fantastic thing.

Yep, serializing/exporting notebooks is high on our list and we've already got a prototype working. I added some more details in https://github.com/observablehq/notebook-stdlib/issues/25 and I'm closing this one to centralize discussion there, but feel free to post there with any other questions.


add shear-flow MD parsing to H5MD

MD workflow is parsed directly according to NOMAD schema in H5MD!


Fetch Custom Buildpack Failing

Hi All!
Just started a new fresh Dokku VM.  Attempting to deploy a static site via Nginx Static Buildpack.
Seems to be hitting a hard stop here.  I have no idea how to read this trace, but maybe someone can see what's wrong?
This was working super nice on an old Dokku VM I had.  Unsure what has changed?

How are you setting the custom buildpack?

thanks @josegonzalez !
setting it with .env at the project root, as per the README above

Can you set it via the BUILDPACK_URL env var instead? Something like:
dokku config:set BUILDPACK_URL=https://github.com/florianheinemann/buildpack-nginx.git

Same issue:
-----> Cleaning up...
-----> Building test-static-buildpack from herokuish...
-----> Adding BUILD_ENV to build environment...
-----> Fetching custom buildpack
To dokku@sanctuarycomputer.link:test-static-buildpack
 ! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'dokku@sanctuarycomputer.link:test-static-buildpack

How long does it take for that to bail out? If it takes a while, might be that it's timing out on the curl call?

pretty much as soon as it hits -----> Fetching custom buildpack it dies.  less than a second, id say

Plus one to this issue.
Here are logs:
Counting objects: 65, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (61/61), done.
Writing objects: 100% (65/65), 684.90 KiB | 0 bytes/s, done.
Total 65 (delta 16), reused 0 (delta 0)
-----> Cleaning up...
-----> Building personal from herokuish...
-----> Adding BUILD_ENV to build environment...
-----> Fetching custom buildpack
To dokku@proitm.pro:personal
 ! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'dokku@proitm.pro:personal'

Here is .env:
export BUILDPACK_URL=https://github.com/yabawock/buildpack-nginx

I have fixed my problem by changing the buildpack to https://github.com/florianheinemann/buildpack-nginx. This might be helpful for people who search a fast solution. But the question, why my previous buildpack has been failing, is still open.

FWIW, we use https://github.com/yabawock/buildpack-nginx in our static app test

Seems like the issue cropped up again on irc. Not sure why custom buildpacks for nginx would fail...

@hhff the nginx static buildpacks all seem to require a .static file in the root of the repo. it will fail in this way if that's not the case.

yup - I'm using .static, and tried bother yabawock & florian's versions.

@hhff can you try this on the latest dokku (0.3.26) and adding TRACE=true to the build phase.
dokku docker-options:add $APP build "-e TRACE=true"

deploy again and let's see what the output is.

@hhff bump

been traveling the last few days!  haven't forgotten about this - will hopefully have a chance on the weekend

OK!  Trace is here
https://gist.github.com/hhff/4fb1085d533250fba101
Action starts around here: https://gist.github.com/hhff/4fb1085d533250fba101#file-gistfile1-txt-L96
Hope this helps guys!

detect fails on line 915. Have a look at the buildpack and see what detect tries to do.

@hhff any luck?

bump

Closing, I think it's just the custom buildpack's detect method doesn't work for your repo, so fixing the detect method will fix your deploys :)

ahh sorry guys - I moved off Dokku for this particular project :heart: will let you know how it goes in the future.  Would love if Dokku had an official static site Buildpack!

This is the one we use in tests: https://github.com/yabawock/buildpack-nginx
Probably would help if we added this to herokuish. @michaelshobbs thoughts?

:+1: that would be gnarly, i would use dokku for erryting in that case

Yeah why not? :shrug:

Added an issue over there https://github.com/gliderlabs/herokuish/issues/79

I had same problem with custom heroku buildpack. After dokku ps:restart the trouble was fixed. Amazing magic.


Mark Properties Not working in New Sketch 41

Spec not working but also Properties is unresponsive in the new Sketch Release.

fixed v2.3.1


Allow configuration of output directory via assemble.

Instead of requiring outputs like project/macos/Flutter/example/App.framework, allow some configurable output/App.framework path. This could be provided via command line flag or others. This would be distinct from the build directory, which is only a cache for intermediate artifacts.
cc @jmagman

I'd like it to be very explicit and to separate out the logic of assembling out from where the output it.  For example, any command that needs an App framework can instantiate or pass into a AotAssemblyBase subclass a new AotOutputLocations object with properties like:
appUniversalFrameworkDirectory
Map<DarwinArch, Directory> frameworkDirectoryByIOSArchs
etc
That object could inform the output sources?

In your example, does build framework (or equivalent) use `frameworkDirectoryByIOSArchs/appUniversalFrameworkDirectory to configure the assemble command?

In your example, does build framework (or equivalent) use `frameworkDirectoryByIOSArchs/appUniversalFrameworkDirectory to configure the assemble command?

I think so.  Right now I'm trying to leverage the logic in  AotAssemblyBase.build() to snapshot build and lipo to build the add-to-app iOS frameworks.  Ideally in my new command I could just add AotAssemblyRelease, etc as configured targets since they are close to what I need, or subclass AotAssemblyBase myself with the right inputs and output.  But there's too much opinionation (we really need to make that a word) in AotAssemblyBase.build() around what environment variables should be there, where the products get output related to the build directory, etc.
Ditto problems in DebugBundleFlutterAssets (though an iOS variant).
So specifying frameworkDirectoryByIOSArchs/appUniversalFrameworkDirectory could be one way to configure it.  Maybe the subclasses need to override it?  Or it gets passed into the target's initialization?
Similarly, the assemble targets seem opinionated about what variables are populated in the Environment (bitcode, target platform, build mode).  When building frameworks for add-to-app I want to build debug, profile, release build mode frameworks in the same command.  Could these values also be overridden by a subclass/passed in, then the subclass/caller could appropriately check and throw if the expected environment values aren't set?  Or is the expectation for the environment be repopulated between targets?
TL;DR assemble targets should uncouple build logic from output location and command flags to be more modular and re-usable.


[Feature] adding options class method

Hey guys,
Great Job on this one, a quick suggestion, do you think this pattern is a good idea:
class User
  extend Dry::Initializer::Mixin

  options :admin, :phone, :email, :address

  # the rest ...
end

If yes, let me know and I'll submit a PR ;)
Cheers,

Ouch! Sorry @rudyyazdi, I've somehow missed your issue
I'm not sure about this kind of DSL, finding it too strict. The method you've suggested builds several options, that are (1) all required (because they have neither default, nor optional keys); (2) has no type constraint.
I think using dry-initializer for this kind of staff is an overkill. Maybe for cases when you need a structure without type control you should use other solutions like Hashie::Mash


📖 REST API documentation consistency improvements

What happened?
I have identified the following possible instances of inconsistencies between Open API specification and Documentation:

🟩Found in Open API specification, 🟥Not found in Documentation

[ ] GET /app/devices
[ ] POST /message/{message_id}/reaction

🟥Not found in Open API specification, 🟩Found in Documentation

[ ] POST /message/:message_id/react
[ ] DELETE /group/participants
[ ] POST /group/participants/promote
[ ] POST /group/participants/demote

About
This is part of the evaluation of my Master's Project at Imperial College London. The section above is automatically generated and aims to expose API documentation inconsistencies in real-world GitHub repositories. The end goal is for the tool to be used as part of CI/CD, namely as a GitHub action.
Hopefully, this is a step towards easier maintenance of API documentation. If you find this helpful, please consider updating the documentation to keep it in sync with the source code. I am also happy to assist with it, if appropriate. If this has not been useful, consider updating this issue with an explanation, so I can improve my approach. Thank you!
Application Version
4.13.0
Verification

[X] I have checked for existing closed issues and discussions.

@alexvuka1 I think you can check in Readme that promote, demote, etc. haven't been released yet.

Thank you for the prompt response and for pointing out the false positives.
The tool currently flags any discrepancies between the documented endpoints and the Open API specification to ensure consistency. However, I realise this can include endpoints that are pending release, such as in your case. To improve the tool's accuracy, it would be useful to incorporate a way to identify and exclude such pre-release features from the checks.
Another approach that I can think of is to add the endpoints to the Open API specification but mark them as not implemented. That would also resolve the inconsistencies. It would be interesting to hear your thoughts on that.
Other than that, it flagged 3 more inconsistencies, are those valid?
Thank you again for your cooperation and understanding!


Dependencies ?

In trying to install this in my environment I find there are a number of missing dependencies. Fails early on in the script with not being able to find ./library/cli.sh:
RobWeaver:bin robweaver$ ./install-tools.sh
./install-tools.sh: line 44: /Users/robweaver/Downloads/gitprime-tools-1.0.1/bin/./library/common.sh: No such file or directory
./install-tools.sh: line 46: /Users/robweaver/Downloads/gitprime-tools-1.0.1/bin/./library/cli.sh: No such file or directory
./install-tools.sh: line 58: add_cli_argument: command not found
./install-tools.sh: line 59: add_cli_argument: command not found
./install-tools.sh: line 60: add_cli_argument: command not found
./install-tools.sh: line 65: parse_cli_arguments: command not found
./install-tools.sh: line 80: get_argument_value: command not found
./install-tools.sh: line 89: get_argument_value: command not found
./install-tools.sh: line 97: get_argument_value: command not found
log: Unknown subcommand 'About to install the GitPrime Development tools to /Users/robweaver/.gitprime-tools.'
usage:
    log <command>

global options:
    -?, --help
    -q, --quiet
    -v, --verbose

examples:
    log show
    log collect
    log erase --all
    log help stream

commands:
    collect, config, erase, show, stream, stats

further help:
    log help <command>
./install-tools.sh: line 131: react_to_exit_code: command not found
./install-tools.sh: line 137: update_environment_files: command not found
log: Unknown subcommand 'Installation completed.'
usage:
    log <command>

global options:
    -?, --help
    -q, --quiet
    -v, --verbose

examples:
    log show
    log collect
    log erase --all
    log help stream

commands:
    collect, config, erase, show, stream, stats

further help:
    log help <command>
usage:
    log <command>

examples:
    log show
    log collect
    log erase --all
    log help stream

commands:
    collect, config, erase, show, stream, stats

further help:
    log help <command>
./install-tools.sh: line 141: log.warn: command not found
./install-tools.sh: line 142: log.warn: command not found
./install-tools.sh: line 143: log.warn: command not found
./install-tools.sh: line 145: handle_exit: command not found

Ah nevermind, if i run ./bin/install-tools.sh it appears to work


Adds MobX

@tronicdata Please check this PR
Adds MobX so we can have a global state container ( so we don't have to pass some data down five components)

@tronicdata Merging


WIP: [Artwork View] - Adds AuctionPrice component

Adds AuctionPrice component to handle displaying Auction bid price and current status

WIP: needs tests

Fails

:no_entry_sign:
Missing Test Files:

src/lib/Scenes/Artwork/Components/__tests__/AuctionPrice-tests.tsx

If these files are supposed to not exist, please update your PR body to include "#skip_new_tests".

  Generated by :no_entry_sign: dangerJS


[eas cli][submit] Use GraphQL instead of REST endpoints

Checklist

[x] I've added an entry to CHANGELOG.md if necessary. (not necessary, no user-facing changes)

Why
We want to stop using REST endpoints in eas-cli. https://github.com/expo/universe/pull/7320 added GraphQL support for Submissions. This PR makes eas submit use GraphQL.
How

3b2ba48 - Added submission query and fragment
199d88d - Added submission mutation
8ca6bb5 - Replaced most of internal submission types with graphql types
67cd200 - Removed SubmissionService.types.ts - it had been left with only two lines, moved them to SubmissionService.ts
1c41b6f - Replaced all occurrences of SubmissionPlatform with `AppPlatform
2c3813d - Updated tests
fadedca - Autogenerated code and graphql schema

Test Plan
Tested by running eas submit for both platforms manually

Maybe add a changelog entry so we know when we switched to GraphQL without traversing git tree.


Lauching WebCrate creates redirect loop

🏷 WebCrate Version
v.0.11.0
⚙ Environment
Chrome
🐞 Describe the bug
When opening webcrate via deta.space, directly through an URL or via the browser extension, I am redirected from webcrate to deta.space in an endless loop.

📚 To Reproduce
Opening the webcrate website.
💡 Expected behavior
Launching into my application
📋 Additional context
No response

NVM, trying the same thing in a private window solves the issue, so I guess its a clientside problem.


Application and file icons not displayed under windows

Hello everyone,
I found (after a lot of trouble) your spotlight which is much more successful than what I have tested so far, excellent work continue like this!
Concerning the problem, I think you must already know it, but in doubt, no icon is displayed for applications and files under Windows.
If ever the OS version is important, I'm running Windows 7.

Thank you

It's being worked on here: https://github.com/KELiON/cerebro/pull/198
There's even a test build with this functionality implemented, you are welcome to try it and provide feedback on that page.

Oops, I had not seen, thank you for the info.


Patch to get collectd_elasticsearch to get the correct es_version

This patch should make the plugin wait till ES is up and running to get the right ES version

I might just do this as a patch in the brew2deb package rather than continuing to maintain our own repo for it; your call on that.


Dummy change to test main

All Submissions:

[X] Have you signed our CLA?

We found the issue, and the proper fix is slightly different. Will come in a separate PR.


[WARN] [getMetricData] ValidationError - The values for parameter id in MetricDataQueries are not unique.

We have upgraded from limit monitor to quota monitor. Used the hub and spoke model deployment solution using quota-monitor-hub template and its dependencies.
Currently trust advisor is sending notifications. However, service quota check is not sending any notifications. Changed the CW poller event rule scheduler to invoke QMCWPollerQMCWPollerLambda so that I can test the notifications are being sent out. There are valid service quota thresholds for VPC quota limit and Elastic IP limit in monitored account. Still notifications are not received. Verified Hub account DynamoDB tables, they show status as "OK" for these limits.
When checked the logs for QMCWPollerQMCWPollerLambda, I see constant warning messages like mentioned below. Is this anything to worry about?
[WARN] [getMetricData] ValidationError - The values for parameter id in MetricDataQueries are not unique.
Can you please guide me on how to test couple of notification scenarios for Service quota threshold? May be by tweaking spoke Service Quota DynamoDB table values?

Thanks. This approached worked. Actually, I changed the QMCWPollerQMCWPollerLambda function environment variable for threshold value rather than redeploying the spoke template for testing purpose only. Anyways this worked. Thank you


Allowing to search by email properly

Summary
Allowing to search by email properly on invitation modal
Ticket Link
MM-17257

The changes will be tested as part of 5.16 release Guest Account feature. Removing the QA Review label.


[Bug]: Browsing is not working

Is there an existing issue for the same bug?

[X] I have checked the troubleshooting document at https://github.com/OpenDevin/OpenDevin/blob/main/docs/guides/Troubleshooting.md
[X] I have checked the existing issues.

Describe the bug
When I ask a question that requires browsing the web to get the answer, OpenDevin does not use the "browsing" tab.
For instance, I asked
Please resolve this pull request: https://github.com/OpenDevin/OpenDevin/issues/1275

In trying to resolve the pull request, OpenDevin tried to install Playwright to browse the web instead of using built-in browsing capability, and the browsing tab said "no screenshot available".
Current Version
`ghcr.io/opendevin/opendevin:0.3.1`

Installation and Configuration
export LLM_API_KEY="sk-..."
export WORKSPACE_DIR=$(pwd)/workspace

Model and Agent
No response
Reproduction Steps

Ask OpenDevin: Please resolve this pull request: https://github.com/OpenDevin/OpenDevin/issues/1275

Logs, Errors, Screenshots, and Additional Context
It seems that this is a relevant error:
STEP 2

23:12:20 - opendevin:INFO: agent_controller.py:89
PLAN
Resolve this pull request: https://github.com/OpenDevin/OpenDevin/issues/1275
23:12:26 - opendevin:INFO: agent_controller.py:107
ACTION
BrowseURLAction(url='https://github.com/OpenDevin/OpenDevin/pull/1275', action=<ActionType.BROWSE: 'browse'>)
huggingface/tokenizers: The current process just got forked, after parallelism has already been used. Disabling parallelism to avoid deadlocks...
To disable this warning, you can either:
        - Avoid using `tokenizers` before the fork if possible
        - Explicitly set the environment variable TOKENIZERS_PARALLELISM=(true | false)
23:12:27 - opendevin:INFO: agent_controller.py:160
OBSERVATION
BrowserType.launch: Executable doesn't exist at /root/.cache/ms-playwright/chromium-1112/chrome-linux/chrome
╔════════════════════════════════════════════════════════════╗
║ Looks like Playwright was just installed or updated.       ║
║ Please run the following command to download new browsers: ║
║                                                            ║
║     playwright install                                     ║
║                                                            ║
║ <3 Playwright Team                                         ║
╚════════════════════════════════════════════════════════════╝

This may be duplicate with https://github.com/OpenDevin/OpenDevin/issues/1172
And solved by https://github.com/OpenDevin/OpenDevin/pull/1184
I'll try to take a look.

Should be fixed by #1184


[Bug] Doris include table pattern null

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
Doris flink connector: flink-doris-connector-1.17-1.5.0
What's Wrong?
When I use --single-sink flag to start my flink job, the task will report error:
Caused by: java.lang.IllegalArgumentException: Can't find any matched tables, please check your configured database-name: [svc_dw] and table-name: [((svc_dw)\.(null))(?!(svc_dw\.(upload_histories|supported_schemas))$)]

I try to figure out the question and I found the code that causes the question:
    protected String getSyncTableList(List<String> syncTables) {
        if (!singleSink) {
            return syncTables.stream()
                    .map(v -> getTableListPrefix() + "\\." + v)
                    .collect(Collectors.joining("|"));
        }

        // includingTablePattern and ^excludingPattern
        String includingPattern =
                String.format("(%s)\\.(%s)", getTableListPrefix(), includingTables);
        if (StringUtils.isNullOrWhitespaceOnly(excludingTables)) {
            return includingPattern;
        } else {
            String excludingPattern =
                    String.format("?!(%s\\.(%s))$", getTableListPrefix(), excludingTables);
            return String.format("(%s)(%s)", includingPattern, excludingPattern);
        }
    }

If I use --excluding-tables flag but not use --including-tables flag with --single-sink flag，the mysql flink cdc source will get the --table-name with value: ((svc_dw)\.(null))(?!(svc_dw\.(upload_histories|supported_schemas))$)
What You Expected?
Start the flink job successfully with --single-sink and --excluding-tables flags.
How to Reproduce?
Start a flink job  with --single-sink and --excluding-tables flags and without --including-tables flag.
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Can you try the latest version 1.6.0?


<EMAIL_ADDRESS>

The extra patch is required because I added some checks ensuring bazelisk is used in the test script to be able to efficiently define which Bazel version is used in the GitHub Actions based DWYU CI. Those extra checks where added after I discovered my CI jobs did did unexpected things on Windows workers.
It seems the BCR CI works by providing the Bazel binary in the desired version via some extra orchestration without bazelisk. I will adapt the test script and future DWYU releases will not need this extra patch for the Bazel binary.


feat: add command and extra_content to control plane

Motivation:
command allow custom command for the control plane image. extra_content alllow extra properties in control-plane.conf

@karimatwa why did you merge this PR?


Video embeds not working Discord

It only shows the preview image, even when enlarged.

Might be specifically a reels issue, I have the same issue with my reel link.

I know youtube-dl has been having issues getting reel URLs for me a couple/few weeks? I can tight-click, copy URL from vid in Safari on macOS using the Baking Soda plugin tho… Lé sigh. Meta tryin' to be smart is making us NORMAL NERDS sadge.

i noticed this one doesn't even load up.

I only just started seeing this issue, was fine earlier in the week.
Ah example link
https://www.ddinstagram.com/reel/C3-9hPNIUzS/?igsh=

It's all now working guys, the link given by the OP is also worked

doesnt work, the following video isn't loading
https://www.instagram.com/p/C1cKtgZtWaV/

It's all now working guys, the link given by the OP is also worked

Nope.

InstaFix get the video URL from Instagram embed, sometimes they are blocked by Instagram, for example:

https://www.instagram.com/p/C1cKtgZtWaV/embed/captioned, when the post is blocked you cant play this link from yout browser, and only shows its thumbnail and "Watch on Instagram"

There is no bypass for this except to create a dummy account and scrape from it, but it will be easily blocked by IG and hard to maintain

InstaFix get the video URL from Instagram embed, sometimes they are blocked by Instagram, for example:
* https://www.instagram.com/p/C1cKtgZtWaV/embed/captioned, when the post is blocked you cant play this link from your browser, and only shows its thumbnail and "Watch on Instagram"

There is no bypass for this except to create a dummy account and scrape from it, but it will be easily blocked by IG and hard to maintain

hmm this one right here,

is showing the video on discord embed, but if we open the embed caption on the browser, it shows "Watch on Instagram" too

is showing the video on discord embed, but if we open the embed caption on the browser, it shows "Watch on Instagram" too

There is other method used by InstaFix, sometimes IG does not block scraping directly from their graphql API (https://github.com/Wikidepia/InstaFix/blob/87e06bd9bfcf7b0df4760b8ad7b4eb6743684dfe/handlers/data/data.go#L325). Scraping in general is very fragile and IG change their anti bot stuff

yeah, https:/ddinstagram.com/reel/C50Yr1MRtXh/ this one didn't embed the video

(because I want to share it with my friend) I tried downloading it with cobalt.tools and it also didn't work

but using yt-dlp works fine tho....

yeah, https:/ddinstagram.com/reel/C50Yr1MRtXh/ this one didn't embed the video

(because I want to share it with my friend) I tried downloading it with cobalt.tools and it also didn't work

but using yt-dlp works fine tho....

It seems that most of the links that were "broken" before, will fix themselves if you wait long enough.
for example, the reels I sent 3d ago are now embedding properly

https://github.com/Wikidepia/InstaFix/issues/82#issuecomment-2142535246


Download from oreilly with special format video like 720p

Checklist

[ ] I'm asking a question
[ ] I've looked through the README and FAQ for similar questions
[ ] I've searched the bugtracker for similar questions including closed ones

Question

WRITE QUESTION HERE
Hello,
Is possible i download from oreilly  video with special format video like 720p or 1080p small
When try for this
youtube-dl -u "PRIVATE" -p "PRIVATE" -f 3215 -o "c:\video"
ERROR: requested format not available
[info] Available formats for 0_qv0os915:
format code  extension  resolution note
mp4-56       mp4        audio only   56k , isom container, 0fps, ~1.04MiB
hls-58       mp4        audio only   58k , mp4a.40.2
mp4-458      mp4        640x360     458k , isom container, avc1, 23fps, ~8.44MiB
hls-497      mp4        640x360     497k
mp4-649      mp4        854x480     649k , isom container, avc1, 23fps, ~12.00MiB
hls-697      mp4        854x480     697k
mp4-877      mp4        960x540     877k , isom container, avc1, 23fps, ~16.20MiB
hls-935      mp4        960x540     935k
mp4-1411     mp4        1280x720   1411k , isom container, avc1, 23fps, ~26.00MiB
mp4-1479     mp4        1280x720   1479k , isom container, avc1, 23fps, ~27.20MiB
hls-1494     mp4        1280x720   1494k
mp4-3215     mp4        1920x1080  3215k , isom container, avc1, 23fps, ~59.20MiB
mp4-16701    mp4        1920x1080  16701k , mp42 container, avc1, 23fps, ~300.00MiB (best)

Problem fixed by add mp4 code before 3215 like mp4-3215


[Bug] Gap Between Custom Cursor and Pointer

Description
The custom cursor has a noticeable gap from the actual mouse pointer. This gap needs to be fixed so the cursor follows the mouse pointer closely.
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

I'm interested in working on issue #1384. I've already reviewed the issue description and believe I have the necessary skills to contribute.
Please let me know if you're open to assigning this to me.

https://github.com/user-attachments/assets/747744a7-06b7-4c37-8c84-168f28acb423
Here's the solution that I've created for the reported issue.


Add redraw main snippetArea after reload

because i need globalFormStack at footer

WDYT @paveljanda?

I think that would redraw filter inputs, wouldn't it?

It definitely would. I dont think this is a good idea.

So I'm closing it now. Thank you @martinknor anyway.


Selected brush regions are unselected when changing tools

Describe the bug
When working with Brush Labels, the region/layer selection does not persist when toggling between tools like brush, erase, pan, etc. This makes it difficult to annotate, because each change to the tool requires an extra manual step to re-select the layer/region I was previously working with.
This seems to be a new behavior since 1.6.0, and is also present in 1.7.0. In version 1.5.0, this was not the behavior, though appear to be several changes to the brush regions since 1.5.0.
To Reproduce
Steps to reproduce the behavior:

Set up a project with Brush Labels
Start labeling part of the image
Change the tool (e.g. brush to pan, brush to erase, then back to brush)
See error - a new region will be created or the region selection will not persist with the change of tool.

Expected behavior
The layer selection persists regardless of what tool is being used, or if the tool is changed. Layers should only be deselected if:

the annotation is updated
another layer is selected by clicking
the layer is manually unselected (another click, or esc)

Screen Recording
Google drive file (too big for github): https://drive.google.com/file/d/1ST2QkjMMQnTb8ovkH8tWJPYGqvB1414x/view?usp=sharing
Environment (please complete the following information):

OS: hosted on Linux VM, viewing on MacOS/Chrome.
Label Studio Version: 1.7.0

@cameronbronstein Could you run LS with this env variable:
ff_front_1170_outliner_030222_short=false label-studio

and check the brush behaviour again?

@makseq I tried that out and noticed that the frontend changed back to the older style, with regions/layers appearing on the right of the images.

However, the brush behavior is similar. I select a layer first and then select the brush tool, the layer is immediately unselected. I can toggle from brush to erase or brush to pan while maintaining selection. So it seems that selecting the brush tool is paired with region deselection.

Broken:

Select the Brush tool, draw a brush, pan, then draw the brush again - Brush tool incorrectly adds a new region when it should have collapsed into existing region.
Select the Brush tool, draw a brush, erase, then draw the brush again - Same, breaks and adds a new brush region
Select a class, do the magic wand, then do brush, then do magic wand (this correctly stacks with the older magic wand), then do the brush again - brush incorrectly does not stack with earlier brush region.

My plan is to address these three problems, as well as add integration tests to prevent this from regressing again.


Use PC training results on Microcontroller

Hi dears.
At the first, appreciation ur Solutions for Reinforcement Learning.
One critical question for me !
Could I do my RLtools Model  training phase on the PC (Linux OS) & use these results as an acceptable format for Microcontroller , That could  use for offline Learning step& increase my learning process on the Micro?!
Realistically I saw ur example for ubuntu and this solution came to my mind.
I look forward hearing from u .
All the best.

Hi @ShHasanzadeh
yes, you could do that! The most common use case right now is training on a PC and then deploying the trained policy onto a microcontroller. On the other hand, we also have the example where we entirely train a policy on the Teensy microcontroller, hence you could combine these two. A first step would be to just load the actor checkpoint, after that you could also load the full training state (Replay Buffer, critics, etc.).
But to get started I would suggest training a policy on a PC and deploying it onto a microcontroller for inference

Dear sir.
Before any Things, Thx for ur response & attention.
refer to my issue, Which example do u recommend for this job?!
,That could First learns on the PC & uses the compiled  results on the Micro!
The output results need to be over what formst & how I could use them on the Micro , So if there is any Guid Documentation at yhe site, It's my appritiate  let me know🙏🙏🙏

Hi @ShHasanzadeh,
I think the easies way for you to get started would be to use the Python interface. I updated the documentation to also include how a trained policy from the Python interface could be integrated into a C++ program/firmware:
https://docs.rl.tools/09-Python%20Interface.html
You can also run it on Google Colab:

Hope this helps!

@jonas-eschmann Thank you for providing the C++ code to embed the exported checkpoint.h. However, I encountered some compilation errors. For instance, in the line:
using T = policy::MODEL::T; // Floating point type (use the one defined in the exported model)
it appears that the namespace policy is undefined. Could you please provide some guidance on how to resolve this issue?
Thank you for your help!

Hi @min-tang,
can you give more details?
When I run  and check the checkpoint.h after training in the file explorer, I can see that the namespace policy is defined

checkpoint.zip
Thanks for your quick response! I've attached the checkpoint.h file generated from training the ppo_pendulum-v1 model. Could you please help me locate the namespace policy within it?

checkpoint.zip Thanks for your quick response! I've attached the checkpoint.h file generated from training the ppo_pendulum-v1 model. Could you please help me locate the namespace policy within it?

@jonas-eschmann Could you go over it and provide some insight? Thanks.

Hi @min-tang,
I checked your file and it seems like the checkpoint comes from the zoo configuration (which uses the Loop interface and particularly the checkpoint step of the Loop interface). In this case the model is saved in the rl_tools::checkpoint::actor namespace, so if you are using https://docs.rl.tools/09-Python%20Interface.html#Embedding-Exported-Checkpoints to compile it for inference you should replace policy with rl_tools::checkpoint::actor in there and it should work.
Please let me know how it goes!

@jonas-eschmann It works now, thanks for your help!

Hi @min-tang,
I checked your file and it seems like the checkpoint comes from the zoo configuration (which uses the Loop interface and particularly the checkpoint step of the Loop interface). In this case the model is saved in the rl_tools::checkpoint::actor namespace, so if you are using https://docs.rl.tools/09-Python Interface.html#Embedding-Exported-Checkpoints to compile it for inference you should replace policy with rl_tools::checkpoint::actor in there and it should work.
Please let me know how it goes!


Document "how this works", please?

Question
The DEVELOPERS.md page could use a section describing how this extension actually achieves its filtering.
For instance, when playing with the settings, I saw there was a "files.exclude" setting.  If I knew how the actual filtering is done, I could write a patch supporting a "files.include" setting, to follow the rules: (1) if a file matches the files.include setting, show it, (2) else if a file matches the files.exclude settings or the checked filters, exclude it, (3) else show the file.

Thanks for the question 👍🏽
files.exclude is not a setting my extension introduces, it is a Native VS Code Setting.  All this extension is doing is giving VS Code users a visual panel to manage this native setting.
Adding a "files.include" won't really do anything in this case, as that logic, as mentioned, is not part of this extension.  VS Code is 100% responsible for handling this, and just uses the files.exclude setting as the rules to use.  This extension helps manage that specific setting, even creating a backup property so users can temporarily disable the setting for testing stuff out without deleting it, but at the end of the day, what actually does the excluding is VS Code specifically using that files.exclude setting.
It's also entirely possible to just manually edit your settings ( either in your user, folder or workspace settings ) and just manage files.exclude without this extension, as it is not required to make that files.exclude work.  This extension just makes it a little easier to "see" that list and manage it.


[Gluon] Support None argument in HybridBlock

Description
Solves https://github.com/apache/incubator-mxnet/issues/13247
We will later consider to support dict, OrderedDict, namedtuple and customized python classes in Gluon 2.0 @leezu @junrushao1994
Support the following way of writing the hybrid_forward:
class Foo(HybridBlock):
    def hybrid_forward(self, F, a, b):
        if a is None and b is not None:
            return b
        elif b is None and a is not None:
            return a
        elif a is not None and b is not None:
            return a + b
        else:
            raise NotImplementedError

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] HybridBlock argument as None, tests

@eric-haibin-lin FYI on the merge.


Ignore Whitelist

I'd like to ignore the fileupload extension whitelist.  I am aware of possible security implications and have taken appropriate steps.  Is there a way to do this?

Currently, there's no option to bypass the whitelist check. Possible solutions::

Build a custom release from sources with these lines removed: https://github.com/DevExpress/AjaxControlToolkit/blob/18.1.1/AjaxControlToolkit/AjaxFileUpload/AjaxFileUploadHelper.cs#L58-L59
Suggest end-users to upload ZIP archives (more secure).


Namespace Microsoft provider

@Fryguy @blomquisg @bronaghs @djberg96 Please review

@matthewd
In this PR SCVMM has been renamed to Microsoft, but soon we will add support for Azure which is also a Microsoft product. Seems like we need to stick with SCVMM. Thoughts?

@bronaghs I think SCVMM will become Microsoft::InfraProvider and Azure will become Microsoft::CloudProvider.  Keep me honest, @matthewd :-)

@chessbyte @matthewd - ok, that makes sense

There were some parts labelled SCVMM, but I just followed the main class: EmsMicrosoft becomes ManageIQ::Providers::Microsoft::InfraManager.
Whether Azure lives under the same provider is a bridge we'll need to cross in the very near future, but not today -- even if it ends up as ManageIQ::Providers::Azure (which I suspect it will), that will only be minimally confusing (and certainly no more than when this was called EmsMicrosoft).

Note that we've already identified ManageIQ::Providers::Redhat needs to be renamed, so this wouldn't be alone... but no matter what, once we've got everything re-shuffled into our desired tree structure, renaming the actual module should be trivial.

@matthewd This looks good...just needs a rebase.


ImageGlass: fix URL and SHA1 and regex

replaced the wrong URLs SHA1s and regex strings with correct ones

Closes #8611

Relates to #8611

[x] I have read the Contributing Guide.

/verify


makeStyles theme.spacing does not work on mui/material: 5.2.7 but works on 5.2.2

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I upgrade from @mui/material 5.2.2 to @mui/material 5.2.7 and suddenly I get
TypeError: theme.spacing is not a function
  35 |   },
  36 |   menuButton: {
> 37 |     marginRight: theme.spacing(2),
     |                       ^
  38 |   },
  39 |   title: {
  40 |     marginRight: 26,

Expected behavior 🤔
I expected that the import makeStyles from '@mui/styles/makeStyles'; would work at it did previously.
Steps to reproduce 🕹
Steps:

Upgrade @mui/material 5.2.2 to 5.2.7 and test makeStyles

Context 🔦
Currently it is not an option rewrite to the new style of MUI v5, so I would like to be able to continue to use the old method.
Your environment 🌎

`npx @mui/envinfo`
  System:
    OS: Windows 10 10.0.19044
  Browser:
    Google Chrome
  Binaries:
    Node: 14.17.6 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.15 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.15 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 97.0.4692.71
    Edge: Spartan (44.19041.1266.0), Chromium (97.0.1072.55)
  npmPackages:
    @emotion/react: 11.7.0 => 11.7.0
    @emotion/styled: 11.6.0 => 11.6.0
    @mui/base:  5.0.0-alpha.58
    @mui/icons-material: 5.2.5 => 5.2.5
    @mui/lab: 5.0.0-alpha.58 => 5.0.0-alpha.58
    @mui/material: 5.2.2 => 5.2.2
    @mui/private-theming:  5.2.3
    @mui/styled-engine:  5.2.6
    @mui/styles: 5.2.3 => 5.2.3
    @mui/system:  5.2.8
    @mui/types:  7.1.0
    @mui/utils:  5.2.3
    @types/react:  17.0.38
    react: 17.0.2 => 17.0.2
    react-dom: 17.0.2 => 17.0.2

You need to also upgrade @mui/styles to 5.2.3. Here is a working sandbox - https://codesandbox.io/s/admiring-cherry-zlimx?file=/src/App.tsx

@Drachmann-statu Did you manage to fix this? I am upgrading from 5.2.0 to 5.2.8, I have updated @mui/styles to 5.2.3 but I am still getting this error. If I debug the theme before it is passed to ThemeProvider all keys exist. But the second any child component tries to consume the theme, no keys are present.

Note that @mui/material and @mui/styles need to depend on the same version of @mui/private-theming otherwise the context would be different. Check if this is the case and update the versions accordingly.

Thank you @mnajdova I downgraded @mui/x-data-grid-pro to 5.2.0 which matches my other mui package versions and it is all working. When I get time I will upgrade all packages, but I just needed to get the license key installed.


MessageBodyType as default in the options

Hi!
We use the ServiceBusExplorer and I have loaded the latest version.
Here I have adapted that as a MessageBodyTyp "String" is enabled by default.
Because we only use „String“.
Is it possible to include a default MessageBodyType in the options for your next changes?

@NoNod not sure I follow you. Could you please elaborate in a bit more details? Thank you.

Hi!
Stream is selected by default. ( cboBodyType.SelectedIndex = 0; )
I want, that String is selected by default. ( cboBodyType.SelectedIndex = 1; )
My idea:
Define the default value in the options.
( cboBodyType.SelectedIndex = ServiceBusHelper.MessageBodyTypeSelectedIndex; )
Here in private void GetServiceBusNamespaceSettingsFromConfiguration()
Set the value from config:
        parameter = ConfigurationManager.AppSettings[ConfigurationParameters.MessageBodyTypeSelectedIndex];

        if (!string.IsNullOrWhiteSpace(parameter))

        {

            int messageBodyTypeSelectedIndex;

            if (int.TryParse(parameter, out messageBodyTypeSelectedIndex))

            {

                ServiceBusHelper.MessageBodyTypeSelectedIndex = messageBodyTypeSelectedIndex;

            }

        }

The ConfigurationParameters.MessageBodyTypeSelectedIndex bring in the Options-Dialog.
Have a funny day!
Grettings
Ralf
Von: Sean Feldman<EMAIL_ADDRESS>Gesendet: Dienstag, 31. Juli 2018 04:33
An: paolosalvatori/ServiceBusExplorer<EMAIL_ADDRESS>Cc: Ralf Gedrat<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Betreff: Re: [paolosalvatori/ServiceBusExplorer] MessageBodyType as default in the options (#242)
@NoNod https://github.com/NoNod  not sure I follow you. Could you please elaborate in a bit more details? Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/paolosalvatori/ServiceBusExplorer/issues/242#issuecomment-409091436 , or mute the thread https://github.com/notifications/unsubscribe-auth/AnwbScxdrRJSbe5qUnjNJgBijDsrZlSXks5uL94NgaJpZM4VmC0R .

Ok, now I see what you're after. Perhaps the issue should be titled something among the lines of "Allow default MessageBodyType to be user configured" rather than "MessageBodyType as default in the options"? 🙂
I see no problem with this. Feel free to send a PR if you'd like to speed it up.

Looks like this was added/fixed in #243

Closed since it was fixed in #243 as mentioned.


Spelling error in CheckAvailabilityResult

In https://github.com/Azure/azure-sdk-for-net/blob/3f736b5af3851ab99bbaa98483ae537de0d48cfb/src/SDKs/NotificationHubs/Management.NotificationHubs/Generated/Models/CheckAvailabilityResult.cs
The IsAvailiable property has one i too many.

https://github.com/Azure/azure-sdk-for-net/blob/3f736b5af3851ab99bbaa98483ae537de0d48cfb/src/SDKs/NotificationHubs/Management.NotificationHubs/Generated/Models/CheckAvailabilityResult.cs#L60

This is actually in the REST Swagger - see https://github.com/Azure/azure-rest-api-specs/blob/master/specification/notificationhubs/resource-manager/Microsoft.NotificationHubs/stable/2017-04-01/notificationhubs.json#L1589.
Since it's an upstream issue, we can't fix it in the SDK (which is autogen from the swagger) - it needs to be fixed upstream.


Added USB API (linux only)

This is a possible API to retrieve USB information.
/cc @larsgk

@larsgk
I've added an implementation for Linux based on sysfs instead of launching udevadm (anyway, I think that udevadm explores sysfs as well).
I've also slightly expanded the PortDetails structure with some more information.
IMHO this is ready to be merged (and expanded with implementation for other OSs through other PRs).

Cool stuff :)
On Oct 12, 2016 17:38, "Cristian Maglie"<EMAIL_ADDRESS>wrote:

@larsgk https://github.com/larsgk
I've added an implementation for Linux based on sysfs instead of launching
udevadm (anyway, I think that udevadm explores sysfs as well).
I've also slightly expanded the PortDetails structure with some more
information.
IMHO this is ready to be merged (and expanded with implementation for
other OSs through other PRs).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bugst/go-serial/pull/8#issuecomment-253239123, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA97loZzqL55ZXpJnmVkbV2d7ZCfaFYtks5qzPZJgaJpZM4KRrXM
.


[FLINK-8013] [table] Support aggregate functions with generic arrays

What is the purpose of the change
Adds supports for aggregate functions with generic arrays and fixes other bugs.
Brief change log

Changes to the type extraction at several locations.

Verifying this change
Complex aggregation function added.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Merging this...


ML 7.4: XGBoost (V)

Description
📌  Issues for Week 7
Welcome to 'ML' Team, good to see you here
:arrow_forward: This issue will helps readers in acquiring all the knowledge that one needs to know about ----
XGBoost.
:red_circle: To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
✔ Domain : Machine Learning
:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/Ensemble_based_Learning_&_Probabilistic_ML Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

:white_check_mark: To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Video

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Video
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Full name : Aditya Nikhil
Batch Number : 2
GitHub Profile Link : github.com/adityanikhil
Which type of Contribution you want to make :
Video

I would like to contribute to this issue. @Isha307
Name: Riya Srisailam Sailesh
Batch: 4
Profile link: https://github.com/riyasai22
Contribution type: video

Hello, I am Deepthi M with serial number:198, Batch-2. I would like to do video on this issue.my approach: a detailed explanation


No Mob spawns, but bats spawn Compact Machines version: 1.12.2-3.0.11-b207

Compact Machines version: 1.12.2-3.0.11-b207
Forge version: <IP_ADDRESS>03
I am not using Optifine: using optifine
Link to Crashlog: Have none
Screenshot (if possible):
Description of the problem including expected versus actual behavior:
Mobs fail to spawn in compact machines. Placing dirt then pure evil on the dirt to create cursed earth for a mob farm, but the mobs never spawn.
Steps to reproduce:
Place dirt down inside a compact machine. use pure evil on dirt to turn it into cursed earth. leave the machine but keep it loaded. return later to find no mobs. same happens if you stay in the same machine.

that worked, thank you.


⚠ Processors has degraded performance

In fd4e867, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: Processors performance has improved in 951811f.


sync code from dev into dev-jar

Codecov Report

Merging #117 into dev-jar will increase coverage by 0.16%.
The diff coverage is 11.11%.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 634a399...4caf563. Read the comment docs.


Application error on the domain https://shields.io/

Are you experiencing an issue with...
shields.io
🐞 Description

as soon as the https://shields.io/ website is open, we see an application error.
None of the icons is getting rendered on our GitHub profile page.
🔗 Link to the badge
https://img.shields.io/badge/platform-windows | macos | linux-lightgrey
💡 Possible Solution
Not sure

I am experiencing the same issue. Shields.io seems to be down. It seems to happen from time to time: https://github.com/badges/shields/issues?q=is%3Aissue+application+error

same

Same here.

Same thing, I just tried to create a license icon and ran into the same problem :)

The service is up for me now

Screenshot


Public category with access private still shows on other videos

I'm not sure why you can even set the access on public categories (shouldn't they always be public?), but if you set a public category to private, it will still show for other users. BUT the annotations created from that category will not show for other users. Annotations created by other users with this category will show until reload.

This will be closed with #547, where setting the access for public categories is not possible anymore

This has happened by now, so closing. :slightly_smiling_face:


add 2 Heart icons and fix fav axios to set favorite in DB

Modify Favorite function

Even navigate to somewhere, display contents and db condition are super fine.

Modified! thanks!