[idea for task] Political bias

Since political bias keyword currently has zero tasks, I've tried to come up with one. This task requires a set of (political) news with political entities (country leader, party leader, party itself, government officials, etc.) labeled (I'm not sure whether there is existing dataset that can be used) a list of generic positive and negative opinions, and a list of political entities that can be substituted into the news. Even though we often can not definitely label news as positive or negative, it is reasonable to expect that model with low bias will give similar probabilities to positive opinion independently on political entity listed (e.g., Republicans propose to increase taxes and Democrats propose to increase taxes).
One possible caveat is that using names of people ties us to a particular historical period (e.g., same news in 2015 or 2020 can get different reaction). It can be partially fixed by using more generic entities (for instance, party names).
There are almost surely many caveats that I didn't think about, as there are always in those things, but I think it is possible to create some task which is not useless and is able to give us some quantification about model biases, even if not perfect.

Perhaps something like this could work:

Take a controversial topic that has its own article on Wikipedia (for example, Kenosha unrest shooting)
Copy the article's intro, and selectively remove some sentences from it, to make it one-sided
Repeat the process, but this time to make it support the other side
Repeat the process, but remove only the sentences that don't affect the balance much (to ensure that all 3 texts have roughly the same length).

As the result, we'll have 3 versions of the same text: one-sided, another one-sided, and the balanced.
We then present a balanced and an one-sided text to the model, and ask it to identify a more balanced one. Each article gives us 2 examples for the task.
The minimum number of examples for a task is 32. Thus, we have to process 16 articles' intros this way, which is doable. I can help with that.

I think for bias evaluation we should look at likelihoods one way or another. Maybe measure likelihood of side A/side B/neutral sentences given biased/unbiased text.🤔

I agree, there are two closely related, but different problems:

to ensure that the model can detect political bias
to ensure that the model itself is not biased

We could create a task with the two subtasks for it.
Both subtasks could have basically the same trials, but different prompts. Something like this:
ability_to_detect_political_bias: "Which of the following texts is likely to be more politically biased?"
[in this case, the model must guess the right answer]
how_biased_is_the_model: "Tell me about the Rittenhouse's acquittal"
[in this case, the model must give a higher probability to the less biased option]


Storage: Updated Listing Operation Return Types

Storage SDK upated to new listing API standards. (Not yet paged listing
for sync APIs)
Async listing return types updated from Flux to PagedFlux
Sync listing return types updated from Iterable to Stream
Bug Fixes:
Fixed bug where Flux<PagedResponse> returned from PagedFlux::byPage()
would attempt to follow non-existent continuation tokens.

For the sync case, the current plan is, rather than return Iterable or Stream, to return IterableResponse or PagedIterable, which are both in azure-core now. Please take a look and consider making use of these.

Updated sync clients to use PagedIterable<T>. Added overloads to PagedIterable<T> to bring it up to speed with PagedFlux<T> functionality.


Files Missing

Can u kindly add the aws networking and scaling files under the template directory, its failing when trying to create a aws manifest

I think the best path forward here will be to assume a bosh director with cloud-config.


celeryev displays "???" for task name

celeryev displays some tasks like this:
a8f54a5b-bc6c-43ae-b6bc-483a0d8eaf66 celeryd-livelong.bluto    ???    11:00:20 SUCCESS

where it doesn't seem to know a task name. The task ran successfully, so surely the celery worker knows some name for the task? How do I go about troubleshooting this?

The task name, args and kwargs is only part of the task_received and task_sent events, so this means that the monitor
did not receive those events.


[PlaygroundTransform] Disable logging ~Copyable types as they can’t be passed to the generic logging function

Summary
Xcode Playgrounds cannot execute code with ~Copyable types, as the Playground Transform tries to pass them to the generic logging function, which is not supported yet. These changes disable logging them altogether for now, which can be reverted once generics start supporting move-only types. Another approach would be to introduce a placeholder log, which could be recognized by the clients in order to inform the user about the reason for the missing logs.
Changes

Add a new function in Playground Transform shouldLogType(Type) to check the isPureMoveOnly flag and use it in all paths that can lead to creating a playground log.
Add unit tests verifying that the move-only types are omitted by the Playground Transform in various scenarios.

rdar://115035973

@swift-ci please test windows platform

@swift-ci please smoke test

@swift-ci please test windows platform

@swift-ci please smoke test

@swift-ci please test windows platform

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please test windows platform

@swift-ci please test windows platform

Do you want to do the same for ~Escapable?

Oh, we should definitely do that. I can address that in a separate PR.

Sounds good to me!

@swift-ci please smoke test

@swift-ci please smoke test


Expanding search-results

When showing search results, it should be possible to click on a search result and go either to the day page of that message (already works), or to have that message expanded with the X messages before and after that (where X would be any number up to say 10 or so).

The second functionality can now be accessed with /around.html?channel="channel"&around-target="2021-04-23Z21:36"&nr-entries=5  If you want JSON, substitute .html with .json.
Optionally takes conversation=1 argument to only return conversational messages.  This should probably always be passed as well.  Also optionally takes control=1 parameter to only return control messages.  Perhaps useful when actually searching control messages only.


is PreCompose project support  ios compose navigation and viewmodel state manager?

Yes, you can checkout the example here

thanks, BTW,


Restore RHODS ISV dashboard tiles for 1.15 release

Signed-off-by: Landon LaSmith<EMAIL_ADDRESS>Description
This restores the missing ISV tiles in the dashboard that were removed from the main branch.  This was originally filed as https://github.com/opendatahub-io/odh-dashboard/issues/346 upstream but these files only apply to RHODS and should eventually be removed from the repo when the PR https://github.com/opendatahub-io/odh-dashboard/pull/243 adds support for dynamically reading this information via k8s resources is merged upstream
NOTE: This should be merged to release branch to keep the main branch in sync with upstream. Targetting the rhods-1.15-release branch
How Has This Been Tested?
Deployed the live buld and the ISV tiles are restored and the KFNBC UI is available in the UI
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] For commits that came from upstream, [UPSTREAM] has been prepended to the commit message.
[x] JIRA link(s): RHODS-4800
[ ] The Jira story is acked.
[x] Live build image: quay.io/llasmith/rhods-operator-live-catalog:1.15.0-rhods-4800
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work.
[ ] QE contact acknowledges that this has been tested and is approved for merge.

RHODS ticket: https://issues.redhat.com/browse/RHODS-4800

@LaVLaS what is still WIP here?

@LaVLaS what is still WIP here?

Creation and verification of the live build that can be used to test the changes

I updated the PR after @bdattoma found that some of the ISV images where missing

@bdattoma The same livebuild catalog linked above was updated to include the missing images

@LaVLaS two more things I've noticed:

the RH Managed Applications are missing the "Official blue icon" like the ones in this screenshot:

2.not sure if it is related to this particular PR, but maybe can be fixed here. The link from RHODS Dashboard to RHODS Documentation is redirecting users to Opendata hub official documentation

the RH Managed Applications are missing the "Official blue icon" like the ones in this screenshot:

This looks like a change in where we no longer check for support: red hat in the tile yaml to enabled the checkmark. https://github.com/red-hat-data-services/odh-dashboard/blob/main/frontend/src/utilities/utils.ts#L122
I have updated the Red Hat managed tiles to use support: redhat

2.not sure if it is related to this particular PR, but maybe can be fixed here or it is not going to happen with the RHODS actual build. The link from RHODS Dashboard to RHODS Documentation is redirecting users to Opendata hub official documentation

This looks like an update to the build system to set the build env vars to set
ENV DOC_LINK="https://access.redhat.com/documentation/en-us/red_hat_openshift_data_science"
ENV SUPPORT_LINK="https://access.redhat.com/support/cases/#/case/new/open-case?caseCreate=true"
ENV COMMUNITY_LINK=""


add stickers

Hey nice library!. Can we add our own custom stickers? If so, how to do that?

I'd like your suggestion, I hope he will consider this stuff.

can We add some Custom emojis ....or a new Tab...


Not using ApplicationBuilder in python-telegram-bot

Might this be the cause of #27?

I actually don't get why it says this, because I am?


Jump to first unread message doesn't show up in 0.7.12 TF

On my phone the "Jump to first unread message" bar never shows up. If I manually scroll back far enough the green line does show up so Riot does now how far back I was, it just doesn't prompt be to jump back there.

After pinning down by binary searching through the commit history manually, this commit broke the unread banner (partially):
https://github.com/vector-im/riot-ios/commit/fc86a37d11606efacf0623c60a6f9808fe058bd4
Interestingly, the banner appears when you tap on the navigation bar to expand it, and stays when you swipe up on the nav bar to un-expand it.
Removing line 422 fixes the banner, but breaks the navbar when expanding it. This can be fixed by re-adding lines 1489 and 1490 (https://github.com/vector-im/riot-ios/commit/fc86a37d11606efacf0623c60a6f9808fe058bd4#diff-fb832478437f1f5120fc0de91a34a081L1489). Essentially, the entire commit could be reverted.
I will make a PR to fix this, but @manuroe should review it - I don't know enough about this part of the project, and the iOS view system yet 😅

The banner is still here (see below) but it is displayed under the navigation bar. The system decided to put it upper because of the navigation bar in now translucent.

Whoa. Out of curiousity, where do I find that debug view? I wish I would've had that a week ago 😂
On topic, this makes a lot of sense to me now, after really scratching my head as to why it would just disappear like that. What do you think is the best solution for this? I'm really rusty with the iOS view system, but is it possible to tell iOS to treat the nav bar the same way no matter if it's translucent or not?

Whoa. Out of curiousity, where do I find that debug view?

Debug > View Debugging > Capture View Hierarchy
One solution could be to use edgesForExtendedLayout with the hope:

it still works with iOS11
it is compatible with contraints set in the code

In [RoomViewController viewDidLayoutSubviews], I would try to set self.edgesForExtendedLayout to UIRectEdgeLeft | UIRectEdgeBottom | UIRectEdgeRight in the case of "normal" room header and UIRectEdgeAll in 2 other cases.

By the two other cases, you mean the expanded room header and...? I'm not 100% familiar with all the views and UI elements that Riot has (I don't use some features)


NumFocus Project Summit September 22/23 2022

NumFocus has their annual summit on September 22nd and 23rd in the Bay Area. I haven't see any threads or conversations around this in the Jupyter world, so I thought I would open an issue and see if anybody is planning to go. Presumably they are only offering this to people who are on the Steering Council? Either way I thought it would be useful to surface this publicly and see if there's a plan to have representation there.
FYI I will likely attend the CZI EOSS summit that happens just before this, and if I do I might be able to join the NumFocus summit, but I am not sure because it'll be quite a lot of time away from my kid at that point. If it would be particularly helpful to Jupyter, I can make the time to attend, but if somebody else prefers to go I think that would be better.

Sounds good - well let me know if it would be helpful for me to join? I suppose I'll run into the same problems that you will re: flights, but it would be good to know sooner than later if I should book them a day later or not.

It seem I/we are not the only one with flight issues, I've contacted multiple people and will let you know when I have feedback.

I will likely be there.

OK, I will not plan on attending this meeting then, as I believe each project only has 1 or 2 slots.

I don't think there would be a problem with more people attending, my sense
is that the 1-2 slots is more about funding that NF has to pay for travel.
We should definitely give the "travel paid" slots to folks in
academia/research.
On Wed, Aug 3, 2022 at 12:07 PM Chris Holdgraf @.***>
wrote:

OK, I will not plan on attending this meeting then, as I believe each
project only has 1 or 2 slots.
—
Reply to this email directly, view it on GitHub
https://github.com/jupyter/governance/issues/131#issuecomment-1204365451,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAGXUGKPYQ6BFQLFOTON7LVXK7PLANCNFSM54EK4VWQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Brian E. Granger
Senior Principal Technologist, AWS AI/ML @.***)
On Leave - Professor of Physics and Data Science, Cal Poly
@ellisonbg on GitHub

OK, I will not plan on attending this meeting then, as I believe each project only has 1 or 2 slots.

I should clarify: I don't think I'm taking a Jupyter slot by being there, so please don't let that deter you.

A quick comment that unfortunately I won't be able to be there, due to teaching responsibilities.  I do hope others from the team can make it, and 👍 to @ellisonbg's point above about prioritizing those funds to folks from academia/non-profit sector.

The summit has now passed, so closing this issue. Ana and Chris attended as representatives of Jupyter, and at least Brian and I also attended.

Do you know if there are some public notes or blog posts from people? Just out of interest around what people talked about, topics that came up, etc. So far I've "only" seen tweets with pictures of people having a good time :D :D

I provided a few memories and thoughts in this discourse post:
https://discourse.jupyter.org/t/notes-from-the-numfocus-2022-summit/16026
Though to be honest I spent a lot of the time in the "hallway track" catching up with people I hadn't seen in many years. I tried to provide some highlights that I recalled however.
I welcome others to edit that post or provide comments with their own memories!


Support to manage virtual cluster in runtime.

Virtual cluster is used to allocate resource. But it can be updated in config only. It's not easy to use. So it's better to be able to add/delete/update virtual cluster and update capacity in runtime.

The bot of virtual cluster is Yarn scheduler queue, which support update actually.  This feature in PAI needs some refactor and may be added in future.

Finished in https://github.com/Microsoft/pai/pull/1985


reformat checklist

Copied out from #81

[ ] Get rid of visitors manually visiting subnodes

some possible solutions, either these two

exit_Xxx which is called for a node after it's subnodes have been visited (would clean up all cases of self.generic_visit(node) being used)
visit functions for node parts that in the AST don't have dedicated nodes, e.g. visit_Finally or visit_WhileElse. This would split up several long functions and solve the last cases of revisits - only trouble child would be 107_108.visit_loop, which explicitly wants to visit the body twice atm, but I could probably rewrite that

Or visitor classes implement a visit_Xxx as a pseudo-contextmanager, with enter and exit as magic names, and other names called as they match up with the _fields of the visited node. And the external runner handles a visit different depending on if it's isinstance function or class

[ ] I'm also starting to want to split up flake8_trio.py into different files, 1500 lines is starting to be quite inconvenient to navigate, and get any overview on - I also know you are hesitant to split given installation complications so would have to take care when doing that.

[ ] Exclude subclasses for disabled checks - probably not a priority for Anthropic since you're always gonna use all checks (for now), but would help when debugging a single runner and could help to find out if any one visitor is eating a ton of time (and e.g. find out if 1+2 are worth it for performance reasons). Should be a very small fix.

[ ] don't hard code unnecessary variables in Flake8TrioVisitor.get_state - it would most likely just mean renaming a lot of variables. But is bound to introduce bugs in it's current state so plan to fix it soon after merge.

[ ] Move all trioxxx.py files into a subdirectory of tests/ so it's clear what files contains tests and which ones contain test data. Can then also get rid of regex and naming restrictions on files.

Added:

Move all trioxxx.py files into a subdirectory of tests/ so it's clear what files contains tests and which ones contain test data. Can then also get rid of regex and naming restrictions on files.

1st one is obsolete with the upcoming libcst rewrite, and the fourth one was recently fixed, so this is now closed~


Improve project README #61

This commit:

unify headers
use redable project name in header
unify spaces between sections like a line before and after a code
block
add missing block code type hint
align badges
add license section on the bottom
other small fixes

Thanks!

thanks
fix #61


Fixed #3414 -- Allow non-superusers to copy all plugins.

Fixes #3414
Also, adapted tests to no longer use a super user to tests permissions.

Coverage increased (+0.5%) to 89.317% when pulling 501ab8865513abd0025cac949a13e94722a16591 on czpython:fixes/3.3.x/3414 into a31fd37571eb1a3b6c85df6f3ccc0299a7a3374f on divio:release/3.3.x.

LGTM

I admit I don't get this :)
Could you elaborate the reason why https://github.com/divio/django-cms/pull/5413/files#diff-0aa10673163e9997607d1d78edd0c030R202 fixes the issue?
It's not a critic, it's just my brain which apparently can't understand the change

@yakky
Thanks for the feedback.
I've updated the line in question with comments to explain what's going on.

@czpython thansk for the comments LGTM


Reorganized the library

The new structure.

Oh I also got rid of base because it wasn't doing anything

Oh I also got rid of base because it wasn't doing anything

What about this rotate_bits etc functions? Are those removed?

Oh I also got rid of base because it wasn't doing anything

What about this rotate_bits etc functions? Are those removed?

They are still on main if you want to bring them back.


should S3 create method require Concurrent?

hello,
i was playing with Doobie and streaming to S3 and had some problems of making it work cause i would like to work within the doobie effect ConnectionIO which does not implement Concurrent while the create method of fs2.aws.s3.S3 does require it.
I dont see why (yet?)
    def create[F[_]: Async: Concurrent](s3: S3AsyncClientOp[F]): F[S3[F]]

would the following be sufficient (at least it compiles)?
    def create[F[_]: Async](s3: S3AsyncClientOp[F]): F[S3[F]]

Does ConnectionIO gives you Async?
Also you might be right and we can remove this dependency

Yes, it does: https://github.com/tpolecat/doobie/blob/e9584dfcdd2b971a3e7c05a448bbe027b29097be/modules/core/src/main/scala/doobie/free/Aliases.scala#L55
But i think it no longer does in the new version which uses Cats Effect 3 under the hood.
After learning a bit more about this, i came to realise this is probably not the right path for me (in the end i try to upload a file within a db transaction, which maybe is not the smartest idea).
So, the issue can be closed as i don't think it matters that much for most users, and soon things will change anyway?

but i think you are right and i should remove Concurrent from the required type classes


Hairyhum/discovery service

Proposed Changes
Adding discovery_service which supposed to keep track of services available on the cloud and make then accessible to the clients.
Currently using the registry model from #2467

Checks

[ ] All commits in this Pull Request are signed.
[ ] All commits in this Pull Request follow the Ockam commit message convention.
[ ] I accept the Ockam Community Code of Conduct.
[ ] I have accepted the Ockam Contributor Licence Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the ockam-network/contributors repository.

In the current prototype services need to register with discovery on startup. For that purpose start function is extended by service provider to add registration after gen_server is started.
Downside of that approach is that the state is stored in the discovery service and it might restart leaving its state inconsistent with the actual service configuration.
Alternatively, service discovery may read current started services info from the supervisor state. This makes discovery service state more consistent with the actual services running on the node.
Downside is that services on other nodes would not be visible in that case, as a solution a forwarding worker may be started as a service on the discovery node and tunnel messages to the remote node in such way that for user it would appear as if they interact with a service on discovery node.
Either way remote node services implementation would require a solution to transport and channel tunneling to remote nodes through or around the access node, which may have additional performance implications.
I'm leaning towards using the supervisor API for service discovery for now, because for the local services it's more consistent.
The client API would stay mostly unchanged and if needed we can switch back to the registration approach.


Documentation should suggest overriding native form validation messages in application/language instead of system/language

Reading through the documentation for the form validation class, I found the following line:

All of the native error messages are located in the following language file: system/language/english/form_validation_lang.php

To set your own global custom message for a rule, you can either edit that file, or use the following method

(emphasis mine)
This seems dangerous, because upgrading will cause the changes to be wiped. It seems much safer to suggest the user to create an application language file, which, if I'm reading the language class documentation correctly, will override the system defined string, though I've not had the time to actually test this yet

Good catch, thanks!


Show PDF from local storage or upload

I am trying to use PDF Classification template and when i upload my pdfs, they show as a text path to the pdf in-stead-of pdf view. Screenshot attached. Is this something expected or a bug?

Hi! You need to prepare a JSON-file with HyperText tag. You can find out how a similar config is set up here at Playground and compare it to yours.

The example takes the pdf source from an image which seems to work but in my case pdf comes from local storage and thats not working.

Hi @sumanshusamarora
This is how I was able to get my images into the project. But should work the same way with PDF’s.
I assume you run LabelStudio with the local files serving, described in the guide the whole section here .
To get the path you can check the "Treat every bucket object as a source file" and copy a file to your absolute local path.

Sync Storage and import your test document.

Then check with the blue </> button in the Data Manager the documents URL.
{      "id": 1,      "data": { "$undefined$": "https://some-url.top/labelstudio/data/local-files/?d=data/inbox/test.jpeg" },     "annotations": [],      "predictions": []  }
If you need further customization for your tasks
For an image on your local deployment it might look like this "$undefined$": "/data/local-files/?d=dataset1/1.jpg".
But the /data/local-files/ route stays the same where the path to your file follows in the ?d=/abs/path/to/your/file http URL parameter.
Finally create your task according to your URL schema just discovered and set your path. Uncheck the “Treat every bucket object...” copy the files to the data directory and import your created task.json.
If there are hellper moduls for such kind of import task I am also very interested in them.
Hope this helps.

Hi @schafsam The images work just fine mate. The problem is pdf preview. Try putting a pdf into your storage and see if that shows up even for pdf rating project.

@sumanshusamarora
I just tried the PDF classification template from the Playground with creating the task from syncronizing the storage. It also does not work for me. This is what I got:

@dentalala
I run version 1.1.0 from a conda environment.

Ok, me too. Version 1.1.0 in a conda environment.

@sumanshusamarora @schafsam You have to use specially prepared tasks and load them as JSON files:
task.json
{
    "pdf": "<embed src='https://app.heartex.ai/static/samples/sample.pdf' width='100%' height='600px'/>"
}

Add Local Storage (or another one) and DO NOT press sync button.
After you should prepare your task.json (or tasks.json) manually using links provided via Local Storage (like https://some-url.top/labelstudio/data/local-files/?d=data/inbox/test.jpeg) in  tag:
{
    "pdf": "<embed src='https://some-url.top/labelstudio/data/local-files/?d=data/inbox/test.jpeg' width='100%' height='600px'/>"
}

I know It's not very convenient, but we don't have direct PDF support yet in LS.


Divide preferences by section in the admin interface - Signals problem

Hi, first of all congratulations for the excellent extension!
I'm using the global_preference for different configurations inside my app, using the admin interface to allow users to update the configurations. I'm currently using 2 different section to define configurations. I would like to divide the "GLOBAL PREFERENCE" block by section in order to make more easily to update preferences of a specific section.
What i would like to create is something like the following structure:
global preferences
|
|_ Section 1
|_ Conf 1
|_ Conf 2
|
|_ Section 2
|_ Conf 1
|_ Conf 2
How can I create this structure inside the admin interface?
Second problem:
I'm trying to use signals to perform action when a preference is updated. At the moment, I'm following the documentation and I have create the callback function inside an utils.py file and I have registered the signal receiver inside the apps.py file inside one of the application of my Django project. I have updated one of the preference available using the admin interface but my callback function is not called. Is that normal or I am missing something?
Thanks for the support!!

Hi it's possible to have an update about this topic? In particular about the signals when a settings is updated.
thanks

(Sorry, I closed by mistake)
As for your second problem, can you please share the relevant code of each files?

I used the same code provided by you on the wiki page (https://django-dynamic-preferences.readthedocs.io/en/latest/react_to_updates.html), no other custom code was created as the basic one was not working correctly.

Is the app config in question installed in your settings.INSTALLED_APPS? Can you try putting a raise or print('hello') statement in the ready() function to ensure it is loaded properly py Django?

Yes the apps.py is loaded by django on startup:

dashboard/apps.py
from django.apps import AppConfig
from dynamic_preferences.signals import preference_updated

from dashboard.utils import notify_on_preference_update

class DashboardConfig(AppConfig):
    default_auto_field = 'django.db.models.BigAutoField'
    name = 'dashboard'

    def ready(self):
        print('Registering signals')
        preference_updated.connect(notify_on_preference_update)

Thank you, I have reproduced it locally and published a fix. Can you please try the develop branch on your project and let me know if it works?

Yes, now it's working correctly.
Thanks!!

Alright, I'll publish a release on PyPi tomorrow :)

Release 1.14.0 including this fix was just published on PyPi https://pypi.org/project/django-dynamic-preferences/1.14.0/#history

I think that in this update the preferences per instance was not considered :thinking:


NSFS | NC | list buckets missing policy check & performance improvement

Explain the changes

bucketspaceFS - list_buckets - Added bucket policy check so we won't let account1 to list buckets of account2 if it's not allowed by a bucket policy even if it's allowed by fs (matching uid gid).
bucketspaceFS - list_buckets - refactored this function, we iterated the buckets array 4 times, now we will iterate it 2 times -
a. First iteration - map_with_concurrency with concurrency rate of 10 entries at the same time.
If entry is dir file/entry is non json / bucket is unaccessible by bucket policy / underlying storage of the bucket is unaccessible by fs we will return undefined, else - the bucket config info.
b. Second iteration - filter entries - filter will remove undefined values produced by the map_with_concurrency().

Issues: Fixed #xxx / Gap #xxx

Gap - Adding a unit test (WIP)
Gap - Move the following 3 functions - has_bucket_action_permission(), validate_fs_bucket_access(), _has_access_to_nsfs_dir() so they can be re-used - follow up PR.

Testing Instructions:

[ ] Doc added/updated
[x] Tests added

@romayalon, general questions that this PR raised:

Would you please attach an explanation (or refer to a link) about the scope of the bucket policy that we support in NC NSFS? From the PR I understand that we check and validate 's3:ListBucket'.
In manage nsfs CLI the list buckets operation is done by the admin who is always the system owner, therefore we don't check bucket policy before the bucket list, is that correct?


Add jQuery as a dependency in the documentation

Can you add that jQuery is a dependency for this script to work?
Somewhere in the Jekyll project, a user should reference jQuery -- e.g.,
   <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.3/jquery.min.js"></script>

This is mentioned in the Basic Usage section, though there isn't a code snippet because there's plenty of different ways to reference jQuery - use whichever floats your boat.


Version 15 causes a crash

ciadpi -x2 -b 1000 -As -An
causes
new event: fd: 3, evt: EV_ACCEPT, mod_iter: 0
accept: fd=5
new event: fd: 5, evt: EV_REQUEST, mod_iter: 1
new event: fd: 5, evt: EV_REQUEST, mod_iter: 1
resolve: www.youtube.com
new conn: fd=6, addr=<IP_ADDRESS>:443
new event: fd: 6, evt: EV_CONNECT, mod_iter: 3
new event: fd: 5, evt: EV_FIRST_TUNNEL, mod_iter: 1
desync TCP, m=0, r=1
host: www.youtube.com (125)
send: pos=0-663
new event: fd: 6, evt: EV_FIRST_TUNNEL, mod_iter: 3
new event: fd: 6, evt: EV_TUNNEL, mod_iter: 3
new event: fd: 5, evt: EV_TUNNEL, mod_iter: 1
new event: fd: 5, evt: EV_TUNNEL, mod_iter: 1
new event: fd: 5, evt: EV_TUNNEL, mod_iter: 1
new event: fd: 6, evt: EV_TUNNEL, mod_iter: 3
new event: fd: 5, evt: EV_TUNNEL, mod_iter: 1
new event: fd: 3, evt: EV_ACCEPT, mod_iter: 0
accept: fd=7
new event: fd: 7, evt: EV_REQUEST, mod_iter: 13
new event: fd: 7, evt: EV_REQUEST, mod_iter: 13
resolve: i.ytimg.com
new conn: fd=8, addr=<IP_ADDRESS>:443
new event: fd: 8, evt: EV_CONNECT, mod_iter: 15
new event: fd: 7, evt: EV_FIRST_TUNNEL, mod_iter: 13
ciadpi: extend.c:316: free_first_req: Assertion `client->buff.data' failed.
Aborted

when trying to visit www.youtube.com

Но только при значения b ниже определённого порога. причём разного при одних и тех же опциях на разных платформах.

The latest commit fixed this particular problem, thank you, but I get another problem when I add -Y.
For example ciadpi -s1 -q1 -Y -b1000 -f-1 -S -As causes an enormous list of these errors:
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory
setsockopt SO_DETACH_FILTER: No such file or directory

With debug output:
resolve: yt3.googleusercontent.com
new conn: fd=10, addr=<IP_ADDRESS>:443
new event: fd: 10, evt: EV_CONNECT, mod_iter: 41
new event: fd: 9, evt: EV_FIRST_TUNNEL, mod_iter: 39
desync TCP, m=0, r=1
split: pos=0-1 (1), m: DESYNC_SPLIT
not sent after 0 ms
split: pos=1-1 (0), m: DESYNC_DISOOB
split: pos=1-999 (998), m: DESYNC_FAKE
send: pos=999-1000
new event: fd: 9, evt: EV_TUNNEL, mod_iter: 39
desync TCP, m=0, r=1
offset: 1000, skip
offset: 1000, skip
offset: 1000, skip
send: pos=0-735
new event: fd: 18, evt: EV_TUNNEL, mod_iter: 18
setsockopt SO_DETACH_FILTER: No such file or directory
close: fd=18 (pair=15), recv: 2000, rounds: 1
close: fd=15 (pair=18), recv: 1757, rounds: 1
new event: fd: 13, evt: EV_TUNNEL, mod_iter: 14
setsockopt SO_DETACH_FILTER: No such file or directory
close: fd=13 (pair=11), recv: 2000, rounds: 1
close: fd=11 (pair=13), recv: 1725, rounds: 1
new event: fd: 3, evt: EV_ACCEPT, mod_iter: 0
accept: fd=11
new event: fd: 3, evt: EV_ACCEPT, mod_iter: 0
accept: fd=13
new event: fd: 11, evt: EV_REQUEST, mod_iter: 47
new event: fd: 13, evt: EV_REQUEST, mod_iter: 48
new event: fd: 11, evt: EV_REQUEST, mod_iter: 47

Just as a note: I need small -b settings because only -b 600 or less make FreeTube work. I don't understand the reason why, probably it has to do something with MTU/MSS and fragmentation. I used -s1 -q1 -Y -f-1 -b300 -S -As and it worked fine (not counting the need for occasional manual reloads), with the version 15 it does not work, but works fine without -Y.

Fixed!


chore(release): v1.1.0 🎉

:robot: I have created a release *beep* *boop*
1.1.0 (2022-11-24)
Features

context: allow specifying default key serialization method at context level (767b7ce)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/with-remote-state-caching/releases/tag/v1.1.0 :sunflower:


fix: resolve assert_stmt and config_entry correctly

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

re #921
2. What is the scope of this PR (e.g. component or file name):

/kclvm/sema
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

@CodiumAI-Agent /review
@CodiumAI-Agent /improve

PR Analysis

🎯 Main theme: Fixing the resolution of assert_stmt and config_entry in the KCL language
📝 PR summary: This PR focuses on resolving the assert_stmt and config_entry correctly in the KCL language. The changes are mainly in the 'advanced_resolver' module and the test data. The PR also includes changes in the test data to reflect the new functionality.
📌 Type of PR: Bug fix
🧪 Relevant tests added: No
⏱ Estimated effort to review [1-5]: 4, because the PR involves changes in the core functionality of the language and requires a deep understanding of the language semantics and the resolver module.
🔒 Security concerns: No security concerns found

PR Feedback

💡 General suggestions: The PR seems to be well-structured and the changes are logically sound. However, it would be beneficial to add tests that specifically check the new functionality introduced in this PR. This will ensure that the changes work as expected and will prevent potential regressions in the future.

 🤖 Code feedback:

relevant file: kclvm/sema/src/advanced_resolver/mod.rs
suggestion: Consider using more descriptive variable names for better code readability. For example, instead of 'x', 'y', 'z', use names that describe the purpose of the variables. [medium]
relevant line: 37, 0, 37, 1, "x".to_string(), SymbolKind::Value,

relevant file: kclvm/sema/src/advanced_resolver/node.rs
suggestion: The 'walk_assert_stmt' function has been modified to include the 'msg' field of 'assert_stmt'. It would be beneficial to add a comment explaining why this change was necessary, as it might not be immediately clear to other developers. [medium]
relevant line: if let Some(msg) = &assert_stmt.msg {

relevant file: kclvm/sema/src/advanced_resolver/node.rs
suggestion: The 'walk_config_entry' function has grown quite large and complex with this PR. Consider refactoring it into smaller, more manageable functions. This will improve code readability and maintainability. [important]
relevant line: if let Some(key_symbol_ref) = self.expr(key) {

relevant file: kclvm/sema/src/advanced_resolver/node.rs
suggestion: There seems to be a lot of nested conditionals in the 'walk_config_entry' function. Consider simplifying the logic or using a different approach to make the code easier to understand and maintain. [important]
relevant line: if let Some(def_symbol_ref) = self

How to use
  Instructions

Tag me in a comment '@CodiumAI-Agent' and add one of the following commands:
/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs: Generate docstring for new components introduced in the PR.
/generate_labels: Generate labels for the PR based on the PR's contents.
see the tools guide for more details.

To edit any configuration parameter from the configuration.toml, add --config_path=new_value.
For example: /review --pr_reviewer.extra_instructions="focus on the file: ..."
To list the possible configuration parameters, add a /config comment.

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 increased (+0.04%) to 87.825%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
2294

Relevant Lines:
2612

💛  - Coveralls


Update index.md

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Please use PR titles which explain what you are changing.  If it is a translation, please also indicate translation: language name. This let's the reviewer quickly know what kind of contribution you have made.
Also, please read/check all 5 items above indicating you have fulfilled each requirement.
Thank you.

LGTM

Hi @bibin-johny. Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


golint: use consistent receiver names

Fixes all of the warnings with the following pattern
... receiver name x should be consistent with previous receiver name y for Z (golint)
Related-to: #153 (Resolve golint warnings)

[ ]   File: pkg/policy/identity.go:L124-147
 s-> id and id -> epID
[ ]   General Comment
 What are the golint errors? Is because the receiver's variable doesn't start with the same letter as the struct's name?

This pull request is now managed with CodeReviewHub. Refresh.

What are the golint errors? Is because the receiver's variable doesn't start with the same letter as the struct's name?

What are the golint errors? Is because the receiver's variable doesn't start with the same letter as the struct's name?

I think the error states that if you have multiple functions for the same struct they should all share the same variable name in the func prototype.

@aanm TBH, I don't know how golint works so not sure why it reports 712 warning but only 85 issues across 343 files. The numbers are straight from https://goreportcard.com/report/github.com/cilium/cilium
From the golang wiki there is a section called Receiver Names

The name of a method's receiver should be a reflection of its identity; often a one or two letter abbreviation of its type suffices (such as "c" or "cl" for "Client"). Don't use generic names such as "me", "this" or "self", identifiers typical of object-oriented languages that place more emphasis on methods as opposed to functions. The name need not be as descriptive as that of a method argument, as its role is obvious and serves no documentary purpose. It can be very short as it will appear on almost every line of every method of the type; familiarity admits brevity. Be consistent, too: if you call the receiver "c" in one method, don't call it "cl" in another.

https://github.com/golang/go/wiki/CodeReviewComments#receiver-names

:+1:


Create StatementAPI

This is mainly all done by @dominv, I just added minor changes and the tests
Fixes #70

Codecov Report

Merging #76 into master will decrease coverage by 1.2%.
The diff coverage is 22.22%.

@@            Coverage Diff             @@
##           master      #76      +/-   ##
==========================================
- Coverage   88.38%   87.17%   -1.21%
==========================================
  Files          18       18
  Lines         422      429       +7
==========================================
+ Hits          373      374       +1
- Misses         49       55       +6

Impacted Files
Coverage Δ

...scala/schema/provider/MetadataSchemaProvider.scala
83.33% <ø> (ø)
:arrow_up:

core/src/main/scala/schema/package.scala
100% <ø> (ø)
:arrow_up:

core/src/main/scala/api/package.scala
100% <ø> (ø)
:arrow_up:

core/src/main/scala/config/decoders.scala
98.98% <100%> (-0.02%)
:arrow_down:

core/src/main/scala/handlers/implicits.scala
16.12% <12.5%> (-2.39%)
:arrow_down:

core/src/main/scala/config/reads/datastax.scala
100% <0%> (+1.28%)
:arrow_up:

core/src/main/scala/config/model.scala
100% <0%> (+6.25%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix parsing '_Unsigned' attribute

[x] Closes #2583
[x] Tests added (for all bug fixes or enhancements)
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API

Hello @dcherian! Thanks for submitting the PR.

There are no PEP8 issues in the file xarray/coding/variables.py !

Looks like the scipy import failed on appveyor somehow.

I'm not worried about the appveyor failure. Thanks @dcherian!


feat!: introduce InfoReservedBytes

Description
BREAKING CHANGE: MsgShareSize is now one byte smaller. This change will break clients who parse msg shares (ex. celestia-app)
Part of https://github.com/celestiaorg/celestia-core/issues/839

CI failed on:
--- FAIL: TestTxInclusion (2.11s)
panic: input length must be a multiple of 64 [recovered]
	panic: input length must be a multiple of 64

Fixing this involves adding Info to the Message struct
type Message struct {
	// NamespaceID defines the namespace of this message, i.e. the
	// namespace it will use in the namespaced Merkle tree.
	//
	// TODO: spec out constrains and
	// introduce dedicated type instead of just []byte
	NamespaceID namespace.ID

	// Info is a byte with the following structure: the first 7 bits are
	// reserved for version information in big endian form (initially
	// `0000000`). The last bit is a "message start indicator", that is `1` if
	// the share is at the start of a message and `0` otherwise.
	Info byte

	// Data is the actual data contained in the message
	// (e.g. a block of a virtual sidechain).
	Data []byte
}

and then updating usage of Message, e.g. in https://github.com/rootulp/celestia-core/blob/27c3aec0b46bb6e7cbfa759c7968659d58161127/types/share_merging.go#L264
However, I'm aware that share_merging.go is being extracted to celestia-app in https://github.com/celestiaorg/celestia-app/pull/637 so I'm not sure how best to proceed without conflicting with the in-flight refactor effort @evan-forbes

It's possible not all of the changes in this PR are necessary. In particular:

Put InfoReservedBytes consts in appconsts celestia-app

Blocked on https://github.com/celestiaorg/celestia-core/issues/841 because https://github.com/celestiaorg/celestia-core/pull/840/files#diff-6572dddc535b4d2c98746d71dde97840af1328797e193db28376939a320be8f8R29

Replay code changes in this branch on evan/refactor-share-pkg
Add Version to WireMsgPayForData and MsgPayForData


Clean up tests for supported types

While working on #285 I noticed our tests for supported types are all over the place making it hard to test for additional types and figure out which mapping is supported by which db. This was fine when we were still working on the prototype but we now need something more manageable.
I'm also going to fix some bugs I've found along the way.
I'm going to send a PR on Monday.

I've updated a bug in IndentityGenerationTest and fixed the error message in DatabaseSelectionRule.

Done https://github.com/hibernate/hibernate-reactive/issues/288


addExtraLib() is not working as expected after disposing the previously added library

monaco-editor version: 0.20.0
Browser: Chrome
OS: macOS High Sierra
I am adding extraLibrary to the monaco using the following code
const carDisposable = monaco.languages.typescript.javascriptDefaults.addExtraLib("declare class Car{
start()
stop()
 }", "Car");

The code in the editor is as follows
 const myCar = new Car();
Now I want to dispose the same and add again with the new methods
carDisposable.dispose();
monaco.languages.typescript.javascriptDefaults.addExtraLib("declare class Car {
start()
applyBreak()
accelerate()
 }", "Car");

Now in the editor if I am using myCar.  then I am getting  start() and stop(). Ideally I should get start(), applyBreak() and accelerate().

+1 same problem here

There is a time racing here.
	addExtraLib(content: string, _filePath?: string): IDisposable {
		let filePath: string;
		if (typeof _filePath === 'undefined') {
			filePath = `ts:extralib-${Math.random().toString(36).substring(2, 15)}`;
		} else {
			filePath = _filePath;
		}

		if (this._extraLibs[filePath] && this._extraLibs[filePath].content === content) {
			// no-op, there already exists an extra lib with this content
			return {
				dispose: () => { }
			};
		}

		let myVersion = 1;
		if (this._extraLibs[filePath]) {
			myVersion = this._extraLibs[filePath].version + 1;
		}

		this._extraLibs[filePath] = {
			content: content,
			version: myVersion,
		};
		this._fireOnDidExtraLibsChangeSoon();

		return {
			dispose: () => {
				let extraLib = this._extraLibs[filePath];
				if (!extraLib) {
					return;
				}
				if (extraLib.version !== myVersion) {
					return;
				}

				delete this._extraLibs[filePath];
				this._fireOnDidExtraLibsChangeSoon();
			}
		};
	}

If you use the same specific filePath to add content, and not listen to the OnDidExtraLibsChange Event, you will bit this problem.

And the code comes from monaco-typescript.

You could make the second call to use "Car2" or you can simply leave empty that argument. I think the problem is that removing + adding another lib will make it that there still is the lib at version 1. So the TS worker does not refresh the content of that lib.


Impl from trait for uint and bytes types

Feel free to close the PR if it's not useful.

What about this code: https://github.com/ewasm/ewasm-rust-api/pull/38/commits/dea1323fb8fdd6fb7896a0fa130deacbe5f04bcf ?

@axic yes but those are just trait declarations for when we have no_std enabled.

It is possible to add functionality in order to convert a StorageKey/StorageValue into an Address and the other way around?

@hugo-dc Yes it would be possible via the type conversion traits. however I am not sure if i want to implicitly truncate the storage types to address types in the trait impl, as ideally there would be no data loss in those traits.

It is possible to add functionality in order to convert a StorageKey/StorageValue into an Address and the other way around?

I'm not fully sure it is needed because you could use these with slices.

Here is some code I have for wrc20:
let input_data = ewasm_api::calldata_acquire();
let address_data = input_data[4..].to_vec();
let mut address = Address::default();
address.bytes.copy_from_slice(&address_data[0..20]);

let mut storage_key = StorageKey::default();
storage_key.bytes[12..].copy_from_slice(&address.bytes[0..20]);

I was wondering if it would be easier if the ewasm-rust-api lib could allow me to do something like this:
let input_data = ewasm_api::calldata_acquire();
let address = Address::from(&input_data[4..]);   // converting &[u8] -> Address

let storage_key = StorageKey::from(&address); // converting Address -> StorageKey

@jakelang can you review please? @s1na has macroified it.

I was still thinking of implementing From smaller uints (and byte arrays) to wider ones (e.g. Bytes20 -> Bytes32) and TryFrom for wider to smaller ones to make @hugo-dc 's example somewhat easier. Would that make sense?

@jakelang happy to merge?

looks good to merge.


Add sample file simple-aws-lambda-customer-sync.py

Provide simple demo file to facilitate initial setup

Let's not just drop files in at the top level. I'd like to get some structure going to we can add stuff cleanly. I expect we'll have examples for other Webhooks and examples for how to call an API, etc.
I'd like to categorize into directory like 'customer-sync/lambda/'

Good point! Fixing.


Disable br compression when no Accept-Encoding header is present

What does this PR do?
Fixes https://github.com/traefik/traefik/issues/9734
To ensure compatibility, it is recommended not to compress a request when there is no Accept-Encoding header present, even though RFC9110 (accept-encoding) would allow any compression.
Motivation
Stops traefik from breaking multiple applications that don't send a Accept-Encoding header and at the same time don't have any support for encoded http responses. Two breaking examples are yum/dnf and Zenarmor on Opnsense.
More

[x] Added/updated tests
[x] Added/updated documentation

The wording in RFC 2616, specifically in https://datatracker.ietf.org/doc/html/rfc2616#section-14.3 is similar to this
" If no Accept-Encoding field is present in a request, the server MAY
assume that the client will accept any content coding. In this case,
if "identity" is one of the available content-codings, then the
server SHOULD use the "identity" content-coding, unless it has
additional information that a different content-coding is meaningful
to the client.
"
In any case, I believe this option should be configurable. For example:
  label:
  ...
  - traefik.http.middlewares.test4compress.compress.acceptEncodingMissing=no-compress

It should, by default, be disabled, following the semantics of RFC 9110.

The wording in RFC 2616, specifically in https://datatracker.ietf.org/doc/html/rfc2616#section-14.3 is similar to this
" If no Accept-Encoding field is present in a request, the server MAY assume that the client will accept any content coding. In this case, if "identity" is one of the available content-codings, then the server SHOULD use the "identity" content-coding, unless it has additional information that a different content-coding is meaningful to the client. "
In any case, I believe this option should be configurable. For example:
  label:
  ...
  - traefik.http.middlewares.test4compress.compress.disableCompressionOnMissingAcceptEncoding=true
  - traefik.http.middlewares.test4compress.compress.disableOnMissingAcceptEncoding=true
  - traefik.http.middlewares.test4compress.compress.noAcceptEncodingCompression=true

It should, by default, be disabled, following the semantics of RFC 9110.

Hello @rjsocha,
Thanks for this feedback. It seems to me that the need to introduce an option to control this behavior should be discussed in a dedicated issue first.

Hello @robin-moser,
I have enabled the checks, and as you can see, the TestShouldCompressWhenNoAcceptEncodingHeader test should be adapted as well.

Sorry, I missed that one!
I updated the unit tests, locally they are passing now.

Hey @robin-moser ,
We'd like to bring your feature in Traefik v3.0.
Do you need some help on it?

Hi @nmengin,
sorry for the late reaction. I just committed the suggested documentation changes.


Update remote-state module

what

Update remote-state module

why

No need to read environment and stage from the YAML configs, they are provided as variables to the module
Make the module faster by not deep-merging the vars sections from YAML configs

/test all

/test all


super important PR

We should allow any serializable value to be asigned to value html prop
fix:
whatever is not assignable to type 'HTMLAttributes'. Types of property 'value' are incompatible. Type 'number' is not assignable to type 'string | string[]'.

Coverage remained the same at 100.0% when pulling 3d581e8b186cb77c71f5a94e520ae51ed05e0a3b on k1r0s-patch-1 into 44547f3b61bda6c2ff97cfe183dfb637eb292077 on master.

Oh well, just noticed that the commit message could be a bit more descriptive. We should aim for more clear commit logs in the future


MSAL Angular

Library

[ ] `<EMAIL_ADDRESS>
Framework
Angular V9
Description
I have a SPA with MSAL B2C authentication. Everything works fine in desktop and mobile browsers: Chrome, Firefox and Safari. However, in Facebook internal browser my app does not seems to receive any event from auth window. The second window opens, user sign in successfully, redirects fine, window closes and nothing. No success, nor failure.
Configuration
I'm using popup method and localstorage.

@engbibi When you say Facebook internal browser, you are referring to the browser that comes up when you click a facebook link from the facebook app? From what I've read we might have trouble supporting popups, given the limitation on the in-app browser. As a workaround, have you tried using the redirect methods instead?

@jasonnutter I did some remote de debugging with Chrome on an Android device. It seems that on Android, Facebook is using webviews to open links and popups are restricted. I believe that on iOS we have a similar situation.
Redirect flow works as expected, so we should implement checks if browser have popups restricted.

@engbibi Thanks for the update. For now, I don't think we'll be implementing any checks in the library for this, as we intentionally any business logic like this up to consuming applications. Closing, as no further actions items at this time. Thanks!


[SPARK-27513][BUILD] files in tarball should have uid/gid 0

What changes were proposed in this pull request?
build tarball with all files having uid/gui 0
How was this patch tested?
i build tarball with:
$ dev/make-distribution.sh --name provided --tgz -Phadoop-2.7 -Dhadoop.version=2.7.0 -Pyarn -Phadoop-provided
and verified uid/pid with:
$ tar -ztvf spark-3.0.0-SNAPSHOT-bin-provided.tgz | head
drwxrwxr-x 0/0               0 2019-04-18 15:10 spark-3.0.0-SNAPSHOT-bin-provided/
drwxrwxr-x 0/0               0 2019-04-18 15:10 spark-3.0.0-SNAPSHOT-bin-provided/jars/
-rw-rw-r-- 0/0           65261 2019-04-18 15:10 spark-3.0.0-SNAPSHOT-bin-provided/jars/oro-2.0.8.jar
-rw-rw-r-- 0/0           95806 2019-04-18 15:10 spark-3.0.0-SNAPSHOT-bin-provided/jars/javax.servlet-api-3.1.0.jar
-rw-rw-r-- 0/0<PHONE_NUMBER>-04-18 15:10 spark-3.0.0-SNAPSHOT-bin-provided/jars/stream-2.7.0.jar
-rw-rw-r-- 0/0           58633 2019-04-18 15:10 spark-3.0.0-SNAPSHOT-bin-provided/jars/chill-java-0.9.3.jar
-rw-rw-r-- 0/0           39283 2019-04-18 15:10 spark-3.0.0-SNAPSHOT-bin-provided/jars/metrics-jvm-3.1.5.jar
-rw-rw-r-- 0/0           72733 2019-04-18 15:10 spark-3.0.0-SNAPSHOT-bin-provided/jars/jersey-media-jaxb-2.22.2.jar
-rw-rw-r-- 0/0<PHONE_NUMBER>-04-18 15:10 spark-3.0.0-SNAPSHOT-bin-provided/jars/orc-mapreduce-1.5.5-nohive.jar
-rw-rw-r-- 0/0           66894 2019-04-18 15:10 spark-3.0.0-SNAPSHOT-bin-provided/jars/jackson-annotations-2.9.8.jar

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10034/
Test PASSed.

Test build #104721 has started for PR 24409 at commit c85bf97.

Test build #104721 has finished for PR 24409 at commit c85bf97.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104721/
Test PASSed.

There's another instance in release-build.sh. The idea is OK by me.

i added same change in release-build.sh. i dont have a way to test this.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10051/
Test PASSed.

Test build #104737 has started for PR 24409 at commit 2fc6aa7.

Test build #104737 has finished for PR 24409 at commit 2fc6aa7.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104737/
Test PASSed.

i read somewhere that mac also ships with gnutar. is that an option? we would have to detect mac os.
to me it sounds too complicated for the benefits.

Not on my mac at least. Probably installable from brew

Oh, are you using gtar from brew?
$ which gtar
/usr/local/bin/gtar
$ gtar --version
tar (GNU tar) 1.32

No just /usr/bin/tar

Oh, interesting. Then, could you give me the result of the followings?
$ sw_vers
$ tar --version

ProductName:	Mac OS X
ProductVersion:	10.14.4
BuildVersion:	18E226

$ tar --version
bsdtar 2.8.3 - libarchive 2.8.3

I'm saying I see the same behavior as you and I agree.

Oh.. Thank you, @srowen . Got it. I misunderstood.


Add a simple PR template

Adding a PR template so that we have more or less consistent PR templates for all projects.
I couldn't find a contributing doc, so I've added links to readme and license files. Let me know if you'd want to make some adjustments 😃
fix #1946

Thanks!
Merged, but I've changed the commit message to include a reference to an issue I've just created.
See commit https://github.com/hibernate/hibernate-reactive/commit/b7f8d1c5fea25b7bc6763d6a48f57bdf37a0ebf5


logger: rtt backend don't let starve others in blocking mode

When a RTT logger backend is configured to blocks, allow other threads
to continue during waiting for data to be transferred to host.
Signed-off-by: Pavel Kral<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master   #11024   +/-   ##
=======================================
  Coverage   53.22%   53.22%
=======================================
  Files         217      217
  Lines       26668    26668
  Branches     5914     5914
=======================================
  Hits        14193    14193
  Misses      10059    10059
  Partials     2416     2416

Continue to review full report at Codecov.

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


The keyboard type changes when you add and delete a number in the chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open app
Go to any chat
Press composer
Click on a numeric keyboard from the keyboard
Type any number
Delete the number
Observed the keyboard

Expected Result:
The number keyboard, it should not return to the alphabet keyboard.
Actual Result:
When you go from alphabet to number on the keyboard, then add any number and delete it, the alphabet keyboard comes again
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/234956627-f99a9c88-160b-4824-9624-285b66f0075c.MP4
https://user-images.githubusercontent.com/43996225/234956665-0fdb5ee0-b9c5-48be-99bb-f5ba31642377.mp4
Expensify/Expensify Issue URL:
Issue reported by: @harshad2711
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1682512145669529
View all open jobs on GitHub

Triggered auto assignment to @bfitzexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Yes, I agree @allroundexperts — it looks like the behaviour is to go back to the default/main keyboard when the text is deleted. I don't consider this a bug either. Going to close this.


Laravel 5 error

There is an error when i try to get any url:
BadMethodCallException in ServiceProvider.php line 226:
Call to undefined method [package]
I only installed project according to instruction.

Is this package even intended to work for Laravel 5?

According to this issue - YES, it is.
https://github.com/xroot/laravel-paypalpayment/issues/35

I resolved this by deleting $this->package() in the service provider. It appears Laravel 5 no longer requires this in package definitions.

I was able to get this working in L5 with minor tweaks to the ServiceProvider, I can fork and submit a pull request if you want to see what I did.

create your pull request also make sure when you run the test it is passed
.thanks
On Tue, Mar 24, 2015 at 4:35 AM, Damien<EMAIL_ADDRESS>wrote:

I was able to get this working in L5 with minor tweaks to the
ServiceProvider, I can fork and submit a pull request if you want to see
what I did.
—
Reply to this email directly or view it on GitHub
https://github.com/xroot/laravel-paypalpayment/issues/37#issuecomment-85327310
.

Fixed


LaTeX error file not found after installing

After installation of mutirow, I still get LaTeX error.
The output of tlmgr list --only-installed:

! LaTeX Error: File `multirow.sty' not found.

\documentclass{article}

\usepackage[margin=1in]{geometry}
\usepackage{multirow}

\title{Tables}
\author{John Doe}
\date{}
...

@shinokada I think this is part of #10. I have a quick and dirty fix in there should you still need one.

Closing for now as this might be solved with the resolution of #10  Thanks @marvingfx


Update nf-wincrypt-certgetserverocspresponsecontext.md

When the certificate is unknown by the OCSP responder, CertGetServerOcspResponseContext() will not return the structure CERT_SERVER_OCSP_RESPONSE_CONTEXT but NULL

Looks good. Thanks for your contribution to the docs!


Migrate away from broken actions/docker and actions/bin/filter

Looks like github removed the actions/docker/ and actions/bin/ repositories

Thanks @nhinds <3


add PostgreSqlKernel

resolves: #3056

dup: #3676

Unable to build and test because:
./bulid.sh

❯ ./build.sh
Building NPM in directory src/polyglot-notebooks
~/dev/interactive/src/polyglot-notebooks ~/dev/interactive
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.

added 343 packages, and audited 344 packages in 21s

48 packages are looking for funding
  run `npm fund` for details

5 vulnerabilities (2 moderate, 3 high)

To address issues that do not require attention, run:
  npm audit fix

To address all issues, run:
  npm audit fix --force

Run `npm audit` for details.

><EMAIL_ADDRESS>compile
> npm run lint && tsc -p  ./ && npm run compile-library && npm run compile-es-module

><EMAIL_ADDRESS>lint
> eslint src --ext ts

><EMAIL_ADDRESS>compile-library
> rollup -c rollup.library.config.js -i src/index.ts -o lib/polyglot-notebooks.js

src/index.ts → lib/polyglot-notebooks.js...
(!) Circular dependency
src/kernel.ts -> src/kernelInvocationContext.ts -> src/kernel.ts
created lib/polyglot-notebooks.js in 6.1s

><EMAIL_ADDRESS>compile-es-module
> rollup -c rollup.es.config.js -i ./src/webview/activation.ts -o ./dist/activation.js

./src/webview/activation.ts → ./dist/activation.js...
(!) Circular dependency
src/kernel.ts -> src/kernelInvocationContext.ts -> src/kernel.ts
created ./dist/activation.js in 6.4s
~/dev/interactive
Building NPM in directory src/polyglot-notebooks-browser
~/dev/interactive/src/polyglot-notebooks-browser ~/dev/interactive

added 266 packages, and audited 267 packages in 21s

35 packages are looking for funding
  run `npm fund` for details

5 high severity vulnerabilities

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

><EMAIL_ADDRESS>compile
> npm run rollup

><EMAIL_ADDRESS>rollup
> npm run compile-ci -- -i src/index.ts -o dist/dotnet-interactive.js

><EMAIL_ADDRESS>compile-ci
> rollup -c rollup.config.js --bundleConfigAsCjs -i src/index.ts -o dist/dotnet-interactive.js

src/index.ts → dist/dotnet-interactive.js...
[!] (plugin rpt2) RollupError: src/index.ts:3:15 - error TS2307: Cannot find module './polyglot-notebooks/commandsAndEvents' or its corresponding type declarations.

3 export * from "./polyglot-notebooks/commandsAndEvents";
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/index.ts

    at error (C:\Users\Oleksii_Nikiforov\dev\interactive\src\polyglot-notebooks-browser\node_modules\rollup\dist\shared\rollup.js:349:30)
    at Object.error (C:\Users\Oleksii_Nikiforov\dev\interactive\src\polyglot-notebooks-browser\node_modules\rollup\dist\shared\rollup.js:1703:20)
    at RollupContext.error (C:\Users\Oleksii_Nikiforov\dev\interactive\src\polyglot-notebooks-browser\node_modules\rollup-plugin-typescript2\src\context.ts:35:17)
    at C:\Users\Oleksii_Nikiforov\dev\interactive\src\polyglot-notebooks-browser\node_modules\rollup-plugin-typescript2\src\diagnostics.ts:70:17
    at Array.forEach (<anonymous>)
    at printDiagnostics (C:\Users\Oleksii_Nikiforov\dev\interactive\src\polyglot-notebooks-browser\node_modules\rollup-plugin-typescript2\src\diagnostics.ts:42:14)
    at typecheckFile (C:\Users\Oleksii_Nikiforov\dev\interactive\src\polyglot-notebooks-browser\node_modules\rollup-plugin-typescript2\src\index.ts:67:3)
    at Object.<anonymous> (C:\Users\Oleksii_Nikiforov\dev\interactive\src\polyglot-notebooks-browser\node_modules\rollup-plugin-typescript2\src\index.ts:260:5)
    at Generator.next (<anonymous>)
    at C:\Users\Oleksii_Nikiforov\dev\interactive\src\polyglot-notebooks-browser\node_modules\rollup-plugin-typescript2\node_modules\tslib\tslib.es6.js:76:71

It looks like you need to run this script in an elevated prompt: https://github.com/dotnet/interactive/blob/main/src/ensure-symlinks.ps1.

@NikiforovAll Could you add some tests? Like the MSSQL tests, they can be gated by a conditional FactAttribute-derived type (similar to MsSqlFactAttribute) since we don't currently run database integration tests in CI.

@NikiforovAll Are you planning to add tests? Do you need help with this PR?

@jonsequitur Yes, sorry for the late reply. I plan to do it in one or two days.

TODO:

Install something like https://github.com/lorint/AdventureWorks-for-Postgres and test the test
Add more tests

Install something like https://github.com/lorint/AdventureWorks-for-Postgres and test the test

We currently don't automate this kind of integration test in our CI which is why for example our MSSQL tests are only run manually. So no need to include this step in the PR, in case that was what you had in mind.

I would like to set up a test database to run tests locally.

I'm happy to try the same setup and verify. Please just include instructions on how to do that in the dev guide. (I should add some as well for the SQL tests, now that I mention it.)

@jonsequitur I've added instructions and tests. It seems like everything works as expected

Could you please assist with failing CI?

You'll need to rebase and account for this API which was renamed.
Build FAILED.

D:\a\_work\1\s\src\Microsoft.DotNet.Interactive.PostgreSql\PostgreSqlKernel.cs(115,16): error CS1061: 'CompositeKernel' does not contain a definition for 'AddKernelConnector' and no accessible extension method 'AddKernelConnector' accepting a first argument of type 'CompositeKernel' could be found (are you missing a using directive or an assembly reference?) [D:\a\_work\1\s\src\Microsoft.DotNet.Interactive.PostgreSql\Microsoft.DotNet.Interactive.PostgreSql.csproj]
    0 Warning(s)
    1 Error(s)

@jonsequitur rebased and fixed. Locally I managed to run the tests

This is looking great @NikiforovAll! I have one minor rename suggestion and otherwise it looks ready to merge.

@jonsequitur renamed


remove pre-release increment constraint

I don't see a reason why pre-releases should not be incremented. It is very common to release early without a stable api.
We should support incrementing 0.1.0 releases and the like.
#59

Coverage remained the same at 100.0% when pulling 8f0c651cedf48ff788cf3a493988162d8bca50e3 on lfaoro:master into 1a9109f8c4a1d669a78442f567dfe8eedf982793 on blang:master.

@hanzei would be possible to get this merged?

I don't have write access to this repo.

That’s sad 😕
@hanzei do you know any anyone that has? You could tag them so they get a notification.

@blang Can you please take a look at this PR?

hi there, any progress on this PR so far? i got stuck at this point too!

hi @ltpquang -- beside submitting the commits and updating the tests, I don't know what else should I do. 3 months later, no answer from the maintainers.
For now I just forked the repo and I'm using my fork with the constraint removed: https://github.com/lfaoro/semver/commits/master

Rebased and pushed to a64359f. Thanks for your contribution and long wait!


Submit a monthly expense

Hello !
Would it be possible to add the option of submitting a monthly expense ?
https://opencollective.com/huneeds

Thanks !

hi @Carodem we are not supporting recurring that atm and it's not likely we'll do it in the short term looking a our priority list. We are aware of this and we have it in the to do list so I'm going to close this issue for now.


feat: support push with soci

The current approach integrates both soci create and soci push to nerdctl push when soci is specified in nerdctl push. The change is based on the assumption that we want to integrate whole soci workflow to nerdctl seamlessly without letting users run any soci commands by themselves.
One alternative is to integrate soci create to nerdctl build and soci push to nerdctl push. nerdctl build makes sense to be independent with nerdctl push because the image can ran or exported locally without pushing later, but soci index is for lazy pull, which doesn't make sense without pushing to a registry later. The soci getting-started guide also puts soci create after pushing image to registry. So didn't choose this approach.
Will add integration tests after aligning this approach.

@AkihiroSuda @djdongjin May I get basic concensus of the high level approach (See PR description) before polishing the implementation and adding tests?

@AkihiroSuda @djdongjin May I get basic concensus of the high level approach (See PR description) before polishing the implementation and adding tests?

👍  on exec-ing the soci binary.
We have been doing the same for cosign, notation, IPFS, buildg, etc., to avoid inflating go.mod dependencies.

Failing
=== RUN   TestPushSoci
    testregistry_linux.go:49: hostIP="<IP_ADDRESS>", listenIP="<IP_ADDRESS>", listenPort=5000
    image_push_linux_test.go:188: testImageRef="<IP_ADDRESS>:5000/nerdctl-testpushsoci:23.10"
    image_push_linux_test.go:191: assertion failed: res.ExitCode is not exitCode: time="2023-09-18T10:26:25Z" level=fatal msg="invalid argument \"1<<21\" for \"--soci-span-size\" flag: strconv.ParseInt: parsing \"1<<21\": invalid syntax"

--- FAIL: TestPushSoci (1.50s)
FAIL cmd/nerdctl.TestPushSoci (re-run 2) (1.50s)


remove dtopic listener didn't work well in cluster mode

I am using olric 0.4.10, I found remove listener just deletes from the local map, and dtopic listener did not use consistent hash. I wasn't sure if the remove function need to request to all members

I am closing this issue due to a lack of activity.


pub/sub: cleanup migrated samples and update region tag

Clean up migrated/rewritten samples that have already published on Cloud Docs. Only one Pub/Sub snippet will live in google-cloud-java after the cleanup. It is the emulator snippet, which doesn't have any test associated with it. This is part 4 of 5 of Pub/Sub Java samples migration.
Part 1: migrate publisher samples (#227)
Part 2: migrate subscriber samples (#238)
Part 3: migrate topic and subscription admin, iam samples (#248)
-> Part 4: other miscellaneous samples
Part 5: update docs (done). add migration notes to old samples.

Code Format Failure:
[ERROR] Failed to execute goal com.coveo:fmt-maven-plugin:2.10:check (default-cli) on project google-api-grpc: Execution default-cli of goal com.coveo:fmt-maven-plugin:2.10:check failed: An API incompatibility was encountered while executing com.coveo:fmt-maven-plugin:2.10:check: java.lang.UnsupportedClassVersionError: com/google/googlejavaformat/java/FormatterException has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0


I can not find the defination of variable 'NN_size'.

I can not find the defination of variable 'NN_size' in the function 'process_and_save_images' of the script 'srwgangp_threaded.py'
Variable 'NN_size'  Location in the script 'srwgangp_threaded.py' :
def process_and_save_images(self, save_directory, save_directory_nearest, save_directory_hr, file_path_test_list_lr, file_path_test_list_hr,
							save_nearest_neighbour=False, save_hr=False):
	batch_test_32, batch_test_128 = self.get_batch_test_imgs(file_path_test_list_lr, file_path_test_list_hr)

	predicted_128 = self.generator_model_valid.predict_on_batch(batch_test_32)

	[_, predicted_128, _] = predicted_128

	PSNR_batch_test = self.PSNR_test_array(y_true=batch_test_128, y_pred=predicted_128)

	images_predicted = self.denormalize_batch_images(predicted_128)
	self.save_batch_images(directory=save_directory, psnr_batch=PSNR_batch_test, image_batch=images_predicted)

	if save_nearest_neighbour:
		batch_test_32 = self.denormalize_batch_images(batch_test_32)
		batch_128_nearest = []
		for i in range(self.BATCH_SIZE):
			img = misc.imresize(batch_test_32[i], **### NN_size**, interp="nearest")
			batch_128_nearest.append(img)

There should be DOWNSCALE_FACTOR variable from config file: config_srwgangp_threaded.py. I've fixed it.


[BUG]Rapids Accelerator (0.2) failing to read csv file on Databricks 7.0 ML GPU Runtime

Describe the bug
Customer is replicating the Mortgage ETL query in a Databricks environment. Data is read from S3. The same S3 data is read using a CPU cluster and it works. GPU scan fails.
It is failing on the first read:
acq = read_acq_csv(spark, orig_acq_path)
def read_acq_csv(spark, path):
return spark.read.format('csv')
.option('nullValue', '')
.option('header', 'false')
.option('delimiter', '|')
.schema(_csv_acq_schema)
.load(path)
.withColumn('quarter', _get_quarter_from_csv_file_name())
Steps/Code to reproduce bug
The cluster uses p3.2xlarge (v100) for driver and executor.
Rapids Accelerator (0.2), Databricks 7.0ML GPU Runtime
Expected behavior
Attached both logs from CPU cluster (working-log4j_cpu.log) and GPU cluster (log4j_gpu_databricks7.0.log)
Environment details (please complete the following information)

Environment location: Databricks 7.0ML Runtime on AWS

Additional context
Add any other context about the problem here.
log4j_gpu_databricks7.0.log
working-log4j_cpu.log

this looks like spark.io.compression.codec is set to zstd, if this is set by Databricks I would expect that to just work, if the setup scripts you are using is doing it, we should not.
The code that is erroring isn't even in the plugin
java.lang.NoSuchMethodError: com.github.luben.zstd.Zstd.setCompressionLevel(JI)I
	at com.github.luben.zstd.ZstdOutputStream.<init>(ZstdOutputStream.java:64)
	at org.apache.spark.io.ZStdCompressionCodec.compressedOutputStream(CompressionCodec.scala:224)
	at org.apache.spark.MapOutputTracker$.serializeMapStatuses(MapOutputTracker.scala:963)
	at org.apache.spark.ShuffleStatus.$anonfun$serializedMapStatus$2(MapOutputTracker.scala:234)
	at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:23)
	at org.apache.spark.ShuffleStatus.withWriteLock(MapOutputTracker.scala:75)
	at org.apache.spark.ShuffleStatus.serializedMapStatus(MapOutputTracker.scala:231)
	at org.apache.spark.MapOutputTrackerMaster$MessageLoop.run(MapOutputTracker.scala:485)

The issue is either the underlying library/jar is missing or its incompatible with what is expected:
java.lang.NoSuchMethodError: com.github.luben.zstd.Zstd.setCompressionLevel(JI)I
when I start a databricks 7.0ML cluster it don't see the setting of spark.io.compression.codec
so are you setting that?

I think we do not need to fix this now.

I agree that it is probably not an issue any more. We support zstd and we do not support ML7.0 for databricks any more. I mostly want to be sure that we are testing zstd on databricks with CSV. Even if we just manually verify it works once, that is good enough.

@revans2  @revans2  FYI
This is more like a version issue because setCompressionLevel was introduced since v1.4.0.
Anyway, I just verified on Azure DB 9.1 and 10.4, both works now with zstd + plugin.
I am going to close this.


Make pending operation rotating circle centered on the button

Sometimes the center of the circle is higher than the center of the button, i.e. the login

Something necessary to resolve this uniformly across the app is to consolidate the many .button classes (there are at least two with this same name I've found so far) as they have inconsistent margin-top properties that lead to the loading graphic getting dropped in and resizing various other elements.
Unless there are concerns, I'm going to simply use the Application.scss file to standardize the styling for all class .button usage based on merging what we currently have.
Any concerns w this approach @ibeckermayer @aidan-fitz ?  Not sure how much UI consistency/consolidation planning we've done.

^ See #253


swapping in productized version of redis

/cc @lance
After this change can no longer reproduce the rekor witness issue with gamma branch. Also including the fix for Fulcio deployment in installer script directly, as well as add a templating command to tas-easy-install.sh

NAME: trusted-artifact-signer
LAST DEPLOYED: Wed Jan 31 09:54:17 2024
NAMESPACE: trusted-artifact-signer
STATUS: deployed
REVISION: 1
TEST SUITE:     trusted-artifact-signer-test-connection
Last Started:   Wed Jan 31 10:10:53 2024
Last Completed: Wed Jan 31 10:10:56 2024
Phase:          Succeeded
TEST SUITE:     trusted-artifact-signer-test-sign-verify
Last Started:   Wed Jan 31 10:10:56 2024
Last Completed: Wed Jan 31 10:12:38 2024
Phase:          Succeeded

Going to go ahead and merge this, its not going to pass in actions as all the images are the mirror ones, however swapping them out and running the tests locally do work


API-based webapp: Add comments counter

In this part of the project we:

Added a comment counter function that displays the number of comments in each item.

Status: Approved 🟢🟢🟢
Hi @jevazquezb  :wave:
Your job is looking great and has all the requirements! Nice job!
Highlights :rocket:
:heavy_check_mark: Comments showing the correct quantity.
:heavy_check_mark: Nice work using async functions! Well done!
Your project is complete! It's time to merge it 🥳
Congratulations! 🎉
Cheers and Happy coding!👏👏👏


Roll Flutter Engine from 189a69d9918d to 297ffbc86186 (1 revision)

https://github.com/flutter/engine/compare/189a69d9918d...297ffbc86186
2023-01-27<EMAIL_ADDRESS>Enable vulkan dev on macOS host (flutter/engine#39179)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Update Contributing docs in README.md.

Add install instructions for the Riemann server.
Update npm install command.

Thanks!


Add support to removal of dwca resource and all dependencies.

Implement support to remove all information about a given dwca_resource and reflect changes to all its dependencies (contact, resource_metadata, occurrence, occurrence_raw etc.)

Only the removal of dwca_resource was implemented. The 'remove' for resource_metadata and contact was already there. As for occurrences, see Issue #27.


Installable on Windows

Currently does not install on Windows.

Downloads every single version of multiple packages from pypi
Stops at conflicts related to ERT

Can we avoid installing ERT on Windows? Windows use case is limited to the explorer.
Is this penalty for having the explorer and the uploader in the same package?

Moved sim2sumo to new repo fmu-sumo-sim2sumo (and currently adding it to komodo 2023.10)
Moved uploader to new repo fmu-sumo-uploader (will add to komodo later)
Removed ert and ecl2df from requirements of fmu-sumo.
fmu-sumo now can run tests on windows, ubuntu and macos


Strategy for support modules

We already have modules for the following third-party libraries:

Servers
** Blaze
** Tomcat
** Jetty
Json
** Argonaut
** Json4s

It's not hard to envision more useful entity encoders and decoders:

Aalto (XML)
Play Iteratee (came up today on IRC for Reactive Mongo)
The other JSON libraries supported by JSON
An Office library (yes, I still generate spreadsheets for a client)
A PDF library
More server backends (grizzly, akka-http, Netty)
More client backends

Some of the above I'd be excited about.  Some I wouldn't, but I expect others would.  At some point, the monolithic repo becomes too unwieldy.  Clearly none of this stuff is core, but integrations are nice to publish.  Where do these integrations live?

All in this repo?
A contrib repo for things we care less about supporting forever and ever?
Individual repos, with the build headaches therein?

Not an urgent problem, but one to consider as more people ask, "Does htttp4s do this?"

I'm starting to extract the build into plugins (#971), which will make a polyrepo strategy more viable in the future.

Relevant gitter conversation

Discussed in greater detail over here.


update to new data

Description
Run notebook to produce new figures, changing the lead compound.
Todos

[x] rerun on newest data
[x] create pymol figures for lead 'Mpro-P1788_0A'
[x] update text
[x] check/fix alignment

Questions

[x] HIS vs ALA 41 in interaction table (found due to observed saltbridge in old version)

HIS 41 (structures[1], Mpro-x3366_0A: 304 residues:  ARG:40.A, HIS:41.A, VAL:42.A)
ALA 41 (structures[0], Mpro-z2q6g_0A: 305 residues,  ARG:40.A, ALA:41.A, VAL:42.A)
Answer: respective structure is an x-ray structure with an H41A mutation, and therefore not used in this analysis.

Status

[x] Ready to go

@glass-w ,@jaimergp or @jchodera could one of you have a look at the updated notebook, so we can merge it?

@jaimergp the image rendering should be fixed now.

fixed

View entire conversation on ReviewNB


Add windows CI build

Windows on appveyor (Boatswain has appeveyor config as an example)

Example appveyor configuration file that uses conda and looks pretty close to what we need:
https://github.com/pytroll/satpy/blob/master/appveyor.yml


[stable/mariadb] Add 'extraEnvVars' parameter

Signed-off-by: juan131<EMAIL_ADDRESS>What this PR does / why we need it:
This PR adds a couple of new parameters:

slave.extraEnvVars
master.extraEnvVars

These two parameters allow users to specify extra environment variables for Master and Slave replicas.
Special notes for your reviewer:
This PR also indent the statefulsets YAML manifests.
Checklist

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

/ok-to-test
/lgtm


[Rancher Desktop for Windows] nerdctl connection timeout

Any nerdctl command gives me the following error. I was previously using docker desktop on windows. I have uninstalled it and installed rancher desktop  . Do I need to change anything in the host file.I can see these entries in the host file in windows
Added by Docker Desktop
<IP_ADDRESS> host.docker.internal
<IP_ADDRESS> gateway.docker.internal
To allow the same kube context to work on the host and the container:
<IP_ADDRESS> kubernetes.docker.internal
End of section
nerdctl run hello-world
docker.io/library/hello-world:latest: resolving      |--------------------------------------|
elapsed: 9.9 s                        total:   0.0 B (0.0 B/s)
INFO[0010] trying next host                              error="failed to do request: Head "https://registry-1.docker.i
o/v2/library/hello-world/manifests/latest": dial tcp: lookup registry-1.docker.io on <IP_ADDRESS>:53: read udp 172.18.70
.2:46614-><IP_ADDRESS>:53: i/o timeout" host=registry-1.docker.io
FATA[0010] failed to resolve reference "docker.io/library/hello-world:latest": failed to do request: Head "https://regis
try-1.docker.io/v2/library/hello-world/manifests/latest": dial tcp: lookup registry-1.docker.io on <IP_ADDRESS>:53: read
udp <IP_ADDRESS>:46614-><IP_ADDRESS>:53: i/o timeout
https://github.com/containerd/nerdctl/tree/master/docs/faq.md

https://github.com/rancher-sandbox/rancher-desktop/discussions seems better place to ask this


load weights error when to start train

AssertionError: failed to read all data

Same error here, seems that the change in classes number make it impossible to train with pretrain wiehgts.

It is because class number is different between built model and weight file.
The solution is to make weight file using below one.
https://github.com/AlexeyAB/darknet
The other is comment out the 'assert' line.
But I'm not sure it work well or not.

@ponkotu-android you are right, because the weight file I specified was wrong. thank you.


Service hook not firing for UI "Work item commented on" with Zendesk

Hi,
We have a service hook with “Work item commented on” trigger which stopped working around 1 week ago.
It is using a Zendesk service.
We have made the following checks:
comment on the ticket in all possible states - NO new event is listed within the service hook history and no comment posted to the Zendesk
use the Microsoft.TeamFoundationServer.Client nuget package to add a comment on the Azure DevOps ticket - NEW event is listed within the service hook history and comment is posted to the Zendesk as expected.
setup new service hook that calls some other URL on the same trigger - event is being picked and shown in the history
Having done the above tests we believe that there has been some breaking change in Azure DevOps UI that is now behaving differently to the API called by the Microsoft.TeamFoundationServer.Client nuget package.
We anticipate that the API path is triggering some additional process that’s been missed out by the UI path.
posted also on MS: https://developercommunity.visualstudio.com/t/Service-hook-not-firing-for-UI-Work-ite/10278407

issue no longer reproduces after some time


Allow for enabling quote trimming via a system property

The property is picocli.trimQuotes.
When set to true or when left empty, quote trimming will be enabled by default. Otherwise, the default is false. Regardless of the value of the property, the value can still be overriden through the usual API - therefore there is no break in backwards compatibility.

Codecov Report

Merging #562 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #562      +/-   ##
============================================
+ Coverage     89.13%   89.14%   +<.01%
  Complexity      286      286
============================================
  Files             4        4
  Lines          3967     3970       +3
  Branches        976      977       +1
============================================
+ Hits           3536     3539       +3
  Misses          212      212
  Partials        219      219

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/picocli/CommandLine.java
89.18% <100%> (ø)
161 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

Merged. Thanks for the PR!


Forces code structures very foreign to C# developers

Hi,
I've been coding in C# for quite some time, and I'm fairly used to CNTK. I'm moving to Tensorflow.net because unfortunately CNTK has been dropped for future development.
This project really is fantastic, but the code structures are so hard to understand if you're new to the library as they are not C# style. An example is this code snippit from "DigitRecognitionCNN" from the examples solution:

Because "tf", and "tf_with" are lowercase, you'd likely assume they are a field, or private method, but they are not and as a result It's hard to tell at a glance whether the others are fields, or tensorflow bindings.
When you see:
       tf_with(tf.name_scope("Input"), delegate
        {
            // Placeholders for inputs (x) and outputs(y)
            x = tf.placeholder(tf.float32, shape: (-1, img_h, img_w, n_channels), name: "X");
            y = tf.placeholder(tf.float32, shape: (-1, n_classes), name: "Y");
        });

        var conv1 = conv_layer(x, filter_size1, num_filters1, stride1, name: "conv1");

It looks from a glance that x and y are some part of the graph if they are inputs and outputs but they are actually fields belonging to DigitRecognitionCNN, but then the first convolutional layer uses it as it's input. Why isn't the input and output just a local variable that only used to tell the graph it's input and output shape?
This method defines a graph at the start of the method as a variable, and then it's referenced by nothing else, before being returned at the end. What happened to it? How was anything added to the graph without touching it?
Then there are nested tf_with methods, which seem to set fields; what does any of this do? Why do they need to be nested to set fields; and why are they fields belonging to "DigitRecognitionCNN" which is an example class, rather than properties of the network?
I genuinely don't think that I'm ever going to be able to grasp this because it's so unlike any C# that I have encountered before. The methods are quite opaque as to what they do.
Regardless, I'd like to thank anyone who has contributed to this project as it's certainly going to be useful for many, many people (just not thickos like me).

Hi @JamesLear92,
Thank you for your attention to our libraries. I have struggling before same as you. But I've to make choice. Maybe it's not making any sense. Afterall, it's working for some of people.

For x and y, you can make it local.
For tf_with, we can't change so far due to taking so much efforts.
For language style, copy python code and run in C# is our goal. I believe you'll like it when you tried NumSharp, TensorFlow.NET and SharpCV when compared to existed python sample. Smooth and no transition needed in your mind.

My suggestion is You can wrap this library into C# style for your projects, that making you feel comfortable.

Thanks for the reply Oceania2018, and again, thanks for the work on this project. I think I might give a wrapper a go, but I need to get my head around a lot first. Unfortunately I don't have experience with TF on python so this is a bit tricky

@JamesLear92 I recommend you use ML.NET a high level machine learning framewrok in C#.

Thanks for the recommendation, I've had a look at ML.NET but it seems that you can't really design your own model (unless you do it in TF first and export), and I can't see a way of doing multichannel outputs either. I'll do some digging.
I think I might have to stick with CNTK for the time being, but that's a shame.


[Bug Report] Puzzle Pirates?

Bug Description
Not sure if this is a Discord issue or a bot issue, but I thought I'd let it be known that currently whenever I run the bot, Discord registers it as if I'm playing 'Puzzle Pirates'. I know it's the bot cause when I stream it it streams the console, and it goes away when I close it.
It's weird.

Steps to Reproduce
Opened the music bot console.
Expected Result
Not to be playing Puzzle Pirates when I run the bot.
Debug Output
(idk how to do that 😬) ver 0.3.9
Additional Info
No response
Checklist

[X] I have looked for information about this within the documentation
[X] I have searched for similar issues on the issues page
[X] I am running the latest version of the bot:

Also experiencing the same issue for at least two days and can confirm the bug description. I tried switching JREs, looking through Discord for options, and looking at my local config file, but I'm still "playing" puzzle pirates.

Is this still happening?

Is this still happening?

It hasn't happened for a while, no. I think it was an issue on discord's end.

Excellent, feel free to reopen this issue if it happens again


Lower mhlo.is_finite operation to linalg

Add a support for lowering "mhlo.is_finite op to linalg

This should be done now. Closing.