Combine order_by() and mutate()

Description
We need to make this statement work .order_by(dist=cosine_distance(Column("emd"), REMOVAL_TARGET)) instead of two:
    .mutate(dist=cosine_distance(Column("emd"), REMOVAL_TARGET))
    .order_by(dist)

It seems straight forward to implement.
More detailed use case:
model = PaliGemmaForConditionalGeneration.from_pretrained("google/paligemma-3b-mix-224")
processor = AutoProcessor.from_pretrained("google/paligemma-3b-mix-224")

def process_image(file) -> str:
   image = file.get_value().convert("RGB")
   inputs = processor(text="caption", images=image, return_tensors="pt")
   generate_ids = model.generate(**inputs, max_new_tokens=100)
   return processor.batch_decode(generate_ids, skip_special_tokens=True,
                                 clean_up_tokenization_spaces=False)[0]

chain = (
      DataChain.from_storage("gs://datachain-demo/newyorker_caption_contest/images")
      .map(scene=process_image)
      .save("image_captions")
)

def get_embedding(text: str) -> list[float]:
    return openai.Embedding.create(input=text,
                                   model="text-embedding-ada-002")

REMOVAL_TARGET = get_embedding("green grass scene")
chain = DataChain.from_dataset("image_captions")

images_to_remove = (
    chain.map(emd=get_embedding("scene"))
    .order_by(dist=cosine_distance(Column("emd"), REMOVAL_TARGET))
    .limit(15)
)

cleansed = chain.subtract(images_to_remove)

Changing to P1 since it's not about the release anymore.

per @dmpetrov - need a bit more input here.

I assume that dist in the example must be preserved in the chain

No. User should not see the signal after the command is done. The signal name is optional here. That's a better code example: .order_by(cosine_distance(Column("emd"), REMOVAL_TARGET))

    DataChain.from_values(file=[File(name=name) for name in names])
    .order_by(length(C("file__name")))  # need to fix this to accept file.name instead

This looks good! Could you please check if this will work with .order_by(cosine_distance(Column("emd"), REMOVAL_TARGET))? If works - that's exactly what we need.

This looks good! Could you please check if this will work with .order_by(cosine_distance(Column("emd"), REMOVAL_TARGET))? If works - that's exactly what we need.

Yes, this works. Here is simple example with nested column names as function arguments:
from pydantic import BaseModel
from datachain.sql.functions.array import cosine_distance
from datachain.lib.dc import C, DataChain
from datachain.lib.data_model import DataModel

target = [0.1, 0.2]

class Embedding(BaseModel):
    id: int
    values: list[float]

DataModel.register(Embedding)

assert list(
    DataChain.from_values(
        embedding=[
            Embedding(id=1, values=[0.1, 0.3]),
            Embedding(id=2, values=[0.1, 0.5]),
            Embedding(id=3, values=[0.1, 0.4]),
            Embedding(id=4, values=[0.1, 0.7]),
            Embedding(id=5, values=[0.1, 0.6]),
        ],
    )
    .order_by(cosine_distance(C("embedding.values"), target))
    .collect("embedding.id")
) == [1, 3, 2, 5, 4]

Amazing! That's exactly what's needed 🙂
Closing.


The line hover legend currently gives too much information. Can it report only the x value, rounded to the nearest integer?

This is what it looked like in voila:


[webpack-cli] Error: spawn Unknown system error -86

Laravel Mix Version: 6.0.41 (npm list --depth=0)
Node Version (node -v): 16.13.2
NPM Version (npm -v): 8.1.2
OS: macOS Monterey 12.0.1

Description:
Mix throws weird error even in completely new Laravel project.
Steps To Reproduce:

Create new Laravel Project
run npm install
run npm run dev

[webpack-cli] Error: spawn Unknown system error -86
    at ChildProcess.spawn (node:internal/child_process:412:11)
    at spawn (node:child_process:698:9)
    at Object.execFile (node:child_process:325:17)
    at Object.module.exports.fileCommandJson (/Users/alexanderhorner/Documents/GitHub/wgg-homepage/test/node_modules/node-notifier/lib/utils.js:88:13)
    at NotificationCenter.notifyRaw (/Users/alexanderhorner/Documents/GitHub/wgg-homepage/test/node_modules/node-notifier/notifiers/notificationcenter.js:81:11)
    at WebpackNotifierPlugin.compilationDone (/Users/alexanderhorner/Documents/GitHub/wgg-homepage/test/node_modules/webpack-notifier/index.js:129:14)
    at _next1 (eval at create (/Users/alexanderhorner/Documents/GitHub/wgg-homepage/test/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:39:1)
    at eval (eval at create (/Users/alexanderhorner/Documents/GitHub/wgg-homepage/test/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:63:1) {
  errno: -86,
  code: 'Unknown system error -86',
  syscall: 'spawn'
}

I'm getting the same error on a new Laravel/Breeze/Vue install.
Laravel Mix Version: 6.0.41
Node Version: 17.4.0
NPM Version 8.4.1
OS: macOS Monterey 12.2 (clean install)

Duplicate of #3027
We're gonna have to replace a dependency to fix things on ARM macOS it seems.

I installed Rosetta 2 and fixed the problem.


Allow normal/visual/replace modes in cmdline

Is your feature request related to a problem? Please describe.
I use vi-bindings in my shell. I'd like to be able to use vim motions/actions to edit the current command line text.
Describe the solution you'd like
Add a configuration option that allows me to enter normal/visual/replace/etc. modes without closing the cmdline (currently pressing <ESC> just closes it and pressing jk which I've mapped to <ESC> does nothing).
Describe alternatives you've considered
Not using vim actions/motions to edit command line 😭
Additional context
Add any other context or screenshots about the feature request here.

Not possible. I've been answering multiple related issues already, so please check the closed issues


[Question]: Pointers on how to have a CapsuleGeometry supported for set_object

Hi,
I got to the point that I needed to create Capsules for my simulation, and I was wondering what parts in the code should I change to support creating capsules. Could you give some pointers to what I should change to obtain this result?.
Thanks in advance

https://github.com/ami-iit/meshcat-cpp/blob/master/include/MeshcatCpp/Shape.h#L57

Oh, yes I followed that path, but it seems that there's an issue with the version of threejs bundled into the main.min.js. It doesn't recognize the CapsuleGeometry. I updated the version of threejs, but then got some issues with the packet of data sent to the meshcat server. It seems that we have to update some parts of meshcatcpp to make it work with a newer version of threejs. Did u run into a similar issue when trying to add this new geometry?. I think it's a whole process, as the folks in drake had a similar issue.

Could you link drake issue.

Sure.
https://github.com/RobotLocomotion/drake/issues/17149


Install deb package without using web installer?

I'm installing Dokku using the deb package, but looking at the source it seems I can't bypass the web installer (Have it not initialise/run). Am I overlooking something or is this not yet implemented?

The web installer:

Writes out or removes a VHOST file
Writes out a HOSTNAME file
Calls sshcommand acl-add dokku admin with your key on stdin
Removes the dokku-installer nginx conf and removes its init file.

You could do all of that to bypass a web installer. If you want to disable it straightaway, thats not implemented, though I would consider a pull request that created /etc/default/dokku with some env var to disable it.

Is there any reason why this is done using a web interface? as the same could be accomplished through a prompt during installation, which would also remove the dependency on ruby.
Not sure what we could do if we want to bypass the prompts, I don't know what the idiomatic Debian packaging way of handling that is.

It's done using the web interface as its more user friendly for the common user of dokku (seriously).
If you'd like to modify the debian package to prompt for this information, sure, but doing so probably means you want an automated way to do this, in which case a defaults file is the way to go...

It would be for just the Debian package yes, and a config option could be added to disable the web-based config.
As far as I can tell debconf would be the idiomatic way of handling this.
Values can be set before install, and can be prompted for during install.
What do you think? If it sounds like the right way to go, I'll look into it.

Go for it

Closed by #1171.


disable prefix for task

Disable prefix for task

closing due to inactivity, feel free to reopen if you'd like to work on it more


amplify.yml can't eval ssh-agent to add ssh key

Describe the bug
I am attempting to add an ssh key so that other git repos can be cloned in my Amplify build process. The very fist command in my build process is eval "$(ssh-agent -s)" and once this command is run the whole build process hangs and no other output is ever output in the the build phase that can be viewed in Amplify console. The build hangs and eventually times out. The only output in the Backend build is:
                                 # Starting phase: build

Is the 'eval' command no longer able to be run on the build containers?
My amplify.yml looks like:
version: 1
backend:
  phases:
    build:
      commands:
        - eval "$(ssh-agent -s)"
        - ssh-add <(echo "$DEPLOYMENT_KEY" | base64 -d)
        - update-alternatives --install /usr/bin/python3 python3 /usr/local/bin/python3.8 11
        - /usr/local/bin/pip3.8 install --user pipenv
        - '# Execute Amplify CLI with the helper script'
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - npm ci
    build:
      commands: []
  artifacts:
    # IMPORTANT - Please verify your build output directory
    baseDirectory: /
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

This is something that it looks like others widely use and rely on as see here:

https://github.com/aws-amplify/amplify-console/issues/150#issuecomment-545982853
https://forums.aws.amazon.com/message.jspa?messageID=924092#924092
https://sapanbodiwala.com/blog/deploying-to-aws-amplify

To Reproduce
Set the value of an Amplify Console env variable named "DEPLOYMENT_KEY" to the value of a base64 encoded private ssh key.
Kick of an Amplify Console deployment / build using the above amplify.yml build settings.
Expected behavior
Build does not hang and can clone other github repos as part of the build
Additional context
Amplify CLI version being used by Amplify Console:
2021-03-10T18:05:40.945Z [INFO]: # Ensuring NPM package '@aws-amplify/cli' is version: 'latest'
2021-03-10T18:05:46.970Z [INFO]: # Patching NPM package '@aws-amplify/cli' from 4.29.4 to 4.45.0...
2021-03-10T18:06:43.431Z [INFO]: # Done patching NPM package '@aws-amplify/cli'

Hello, if I am understanding correctly, you are trying to pull from a private package during your build.  it looks like your build commands are correct, could you please verify you followed these steps correctly here: https://github.com/aws-amplify/amplify-console/blob/master/FAQ.md#how-do-i-pull-private-packages-during-a-build, if you have please let me know so I can look into your issue further. Thank you

@brnmye I opened this ticket under a different github account by accident... but yes the steps you linked to in the FAQ are the exact steps I used. Yes your understanding is correct, I'm trying to pull from a private github repo during the build.

I am having the same issue, here is my amplify.yml:
version: 1 frontend:   phases:     preBuild:       commands:         - eval "$(ssh-agent -s)"         - ssh-add <(echo "$DEPLOY_KEY" | base64 -d)         - npm ci     build:       commands:         - if [ "${AWS_BRANCH}" = "master" ]; then npm run build-live; fi         - if [ "${AWS_BRANCH}" = "dev" ]; then npm run build; fi   artifacts:     # IMPORTANT - Please verify your build output directory     baseDirectory: /dist     files:       - '**/*'   cache:     paths:       - node_modules/**/*

I'm encountering the same issue - removing eval "$(ssh-agent -s)" results in an error but does not hang as mentioned above, indicating a problem specifically with this command.

For those running into the same issue, you have to make sure you remove the new line characters from your base64 encoded string before adding it to DEPLOY_KEY. That solved the problem for me.


Fixes to the proposals and referendum contracts based on main net int…

…egration testing.

Task linked: DEMZNE-1303 Fixes on referendum and Dacproposals for mainnet integration testing


Added Storage tests for Volume Check issue

I started the building blocks for this,
However I was not able to do `{Should begreaterthan .... }
https://github.com/Ticketmaster/poshspec/issues/14

I have no idea how to fix this, if you want to put aside some time to assist that would be awesome. I have no idea how to clean this up... :/

Closing this. Will merge #30.


TemplateParser should be able to report multiple errors

Currently the template parser code throws an argument exception with the first error it encounters in parsing a route template.
We should have a custom exception type for this and expose rich information including the original template and all errors that were found. This will be more useful to extenders and will be needed by us when we do richer diagnostics functionality

We should do this after preview 1 as it's not high priority, just polish


Non-urgent | Update Mozilla Firefox DOH instructions

Might trip up some non-tech people, fwiw:
This page:
https://github.com/cloudflare/cloudflare-docs/blob/production/products/<IP_ADDRESS>/src/content/dns-over-https/web-browser.md
Has instructions:
"Open Preferences: Click on the hamburger icon at the top right corner of your browser, and click on Preferences."
But, my copy of Firefox 83.0:
Refers to "Options" not 'Preferences'
New users might also benefit from advice in Step 2 to:
"Use search box by typing in Network"
And step 3:
"Use search box by typing in DNS."
Or similar, to take users to the actual link, helpfully highlighted in yellow, rather than visual searching dozens of items. Especially as both items are below-the-fold (?) or require scrolling to access. The need to scroll down may not be immediately apparent to techno-noobs i.e. the majority of users in my own noobish experience.
thanks!

hey, thanks for raising this! I've walked through the Firefox (85.0) flow and updated instructions accordingly. #702 should fix this.


Undercover message support for send_media_broadcast_close in tinychat.py

There should be the option to allow sending an undercover message, in the send_media_broadcast_close function, for a stated nick name as opposed to sending it to all the users.
This just needs to be the same as how send_media_broadcast_start deals with undercover messages.
I might submit a pull request another time.
EDIT: A pull request was submitted to deal with this.

send_media_broadcast_start, send_media_broadcast_close, send_media_broadcast_play, send_media_broadcast_pause and send_media_broadcast_skip now all have a private_nick parameter.

OK, thanks.


Scroll Down Programmatically?

Is there a way to scroll down programmatically or without dragging the scrollbar down?

Hi!
Yes, you can call yourhtmlframe.html.yview_scroll(value, "units"). The document will scroll down if value is positive and scroll up if negative. To move directly to a particular point on the page (i.e. scroll relative to the top of the document, and not the current position), instead use yourhtmlframe.html.yview_moveto(value).

Hi!
Yes, you can call yourhtmlframe.html.yview_scroll(value, "units"). The document will scroll down if value is positive and scroll up if negative. To move directly to a particular point on the page (i.e. scroll relative to the top of the document, and not the current position), instead use yourhtmlframe.html.yview_moveto(value).

Thank you for the info! Also, the reason that I've discovered your module is because I don't really like the Ttk "Treeview" because it is too limited as far as adding images & styling it goes. So I have another question...
Say that I use your browser to created a HTML table with css to style it, do you think it would be okay to use it to load possibly 2,000-3,000 rows or do you think that it would be too slow to replace the Treeview widget?
I'd prefer to use your browser because I'd get way better aesthetically-speaking. The table would have to keep appending/loading additional rows to it.
Do you think that this could work out or be too resource heavy? It would have to be fast enough to keep appending new rows on the table. Thanks again for your share & great work 👍👍


Leaving History independently

Leaving History independently so that each state machine can import it.

Please remember to set the Labels :-)

Sorry !!

@bryancall @zwoop Thanks for reviewing, and please review again.

Hi @jpeach , I've added the unit test. Can you please review again!

[approve ci autest]

Hi @jpeach updated, Can you please review again!

Sounds like a good idea. I'll updated later !!

Hi @jpeach , Can you please review again ?


push trouble

when push transaction Like Tthis :
eos --print-request  push transaction '{"expiration": "2018-07-05T04:13:48.000","ref_block_num": 336909,"ref_block_prefix":<PHONE_NUMBER>,"max_net_usage_words": 0,"max_cpu_usage_ms": 0,"delay_sec": 0,"context_free_actions": [],"actions": [{"account": "eosio.token","name": "transfer","authorization": [{"actor": "liyan","permission": "active"}],"data": "000000008069bc8b00a6823403ea3055400d0300000000000453595300000000056c6979616e"}],"transaction_extensions": [],"signatures": [],"context_free_data": []},["EOS8XsZF723pBgy1PWQyzzDY85qu5f5j4YfDeVf4pf87XnUWphjHa"],"cf057bbfb72640471fd910bcb67639c22df9f92470936cddc1ade0e2f2e7dc4f"'
I got this: It is sure that the wallet is unlocked
Error 3090003: provided keys, permissions, and delays do not satisfy declared authorizations Ensure that you have the related private keys inside your wallet and your wallet is unlocked. Error Details: transaction declares authority '{"actor":"liyan","permission":"active"}', but does not have signatures for it under a provided delay of 0 ms, provided permissions [], and provided keys []

eos --print-request => what is this tool you used to push the transaction? is that an alias of something else?

eos is a alias of cleos!


Change log levels to debug so they dont pollute CLIs.

https://github.com/SAP/open-ux-tools/issues/2270
Fix for CLI pollution (axios-extension):

Change info to debug as info is not appropriate for a low level networking module that is consumed by CLI prompts, and there is no way to change the default level or disable.

The correct approach is to support log configuration (log level, or a new logger instance) so consumers can set the log level to off or whatever is appropriate for their usage. But for now we need to quickly hide them.


Update lambda with md5 function

add option to use user-provided md5
fix environment variable naming convention
add timeout function (can be updated later to take action in case of timeout)

Leaving environment variables as they are while we explore options, but would like to update the md5 function


opt partition rollup support week and quarter (#39286)

pick: https://github.com/apache/doris/pull/39286

run buildall


Sandbox security?

Hi there,
I'm quite interested in Croncats. The ecosystem needs a decentralized cron service. One question I had was related to security and the notion of tasks running in a sandbox.
How can I assure that the parameters of my task aren't tampered with?

Hello @mweichert take a look here: https://github.com/Cron-Near/contracts/blob/main/manager/src/tasks.rs#L161
This method doesnt not allow execution with tampered parameters, and the agent has no way to dictate which task gets executed.


FAQ page

Many links to the old DataONE site on this page

Hi @aebudden . I'd like to work on this issue...... but before that, can you elaborate the issue! Thank you.


Sort order of entries in data file

This is rather a discussion point, and a somewhat difficult one...
Currently, there are some odd entries are at the top of the JSON data file - e.g. entries for glTF support in some browsers, which eventually have not really been implemented...
Yes, that's my fault - I just added the entries in the order in which they appeared at that point in the "Late Breaking" list...
Some questions that we might have to think about:

Should we edit the data file to move the "odd" entries to the bottom?
More generally: Should we apply some arbitrary order to the other projects, based on a vague gut feeling (a mix of subjective "importance", "activity", "maturity"...?)
Project authors who submit their project via PRs may (likely) try to add their projects prominently at the top...

For example, I'd rather like to see things like the glTF validator at the top when opening the project explorer, because it is an important and unique Khronos-owned project. Similarly, for the Khronos-owned converters or the reference viewer. But beyond that, I'd hesitate to dictate any sort order...
I think during the calls, we also considered some sort of "featuring/starring" for recommended projects, but we haven't really thought about an approach for this.

Considerations from the call:

Alphabetical order would be the most fair solution
Having a flag indicating the official Khronos projects?

Maybe even display them as an own category?

More generally: Any sort of priorization for the projects?
Having a server pull the "number of stars" for GitHub projects to use this as a sort order?

This could be a script that runs once per week - not for each request by clients

Are all resources really on GitHub?

Not all: Some proprietary applications or model repositories, but most applications/libraries are on GitHub

Suggestion: Two flags: Khronos (yes/no) and "Staff Pick" (yes/no) to use as primary order, and alphabetical order as secondary

So, I ran out of time today to actually fully implement this, but here's what I'm planning. I will try to wrap it up tomorrow or Friday. Now that I'm also feeling healthy again, I may try to do some additional work on Saturday as well.
For the Khronos official project flag, I'm going to key the boolean field khronosOfficial. For the Staff Pick, I'm thinking of going with a staffPick boolean field. In both cases I'm using camelCase just because it's easier to deal with in the browser JSON parsers and follows general industry standards for TypeScript/JavaScript.
On the UI side, I am going to add two "switch" elements that users can use to filter against these two new fields. Additionally, I'm going to make the "on" state of these switch elements exclusive; if you're filtering by "Khronos Official", you can't also choose "Staff Picks". Implementing the switch element is where I left off this evening.
@pjcozzi and @javagl, do we have an idea what we'd like to flag initially as "Staff Picks"?
Additionally, I'm going to configure the sorting to follow the pattern discussed, with the Khronos Official projects by alpha at top, then the Staff Picks by alpha, then the rest by alpha. For testing, I am going to flag Three.js and Babylon as Staff Picks so we at least have some content there until others can get back to me.

When it comes to questions like this, I'm always trying to extrapolate possible future demands and see whether they can be handled more generically. Very roughly speaking: When there are two possible flags, might there be a third one in the future? If so, shouldn't this be handled with something like a "tags" : [ "khronosOfficial", "staffPick", ... ] property?
On a far more technical level, and assuming that there will be several different sorting criteria and priorizations: How can this be structured in order to easily configure the sort order? But I'm not familiar enough with React/TypeScript/... to even guess what might be the best solution here - Pseudocode would involve lines like this.sorter = Sorters.createAlphabeticalWithPriorizedTags("khronosOfficial"); or so...
On the one hand, the issue about the boolean fields is critical: The current "schema" will eventually have to be cast into some sort of database schema, and preferably, there should very rarely be a reason to change this and migrate it to a different schema. On the other hand, we currently only have the JSON file, and thus, some leeway to try out what works best, and hopefully pick the right approach when this goes into a live database.

I rather like the "tags" approach. It fits with the rest of patterns we're already using and instead of using a switch we can add another filter element. I think I might start going back in that direction.
There are a lot of different approaches we can take here, but for the first pass I am likely going to add a step within store/results/Sagas.ts:*applyFIlters() to take the filtered results and sort them. My first pass is just going to provide a static list of tags to prioritize in priority form. Then I'm going to split the filtered results into buckets, sort each bucket by alpha in parallel, and then join everything together based on the said priority. Hard to describe but should work well for us for now.
When we move to having a backing database, I think the best approach is to still just spit out the data in a format similar to our JSON file. As long as it's not an unreasonably large output, continuing to perform filtering and searching on the client side generally performs better at the cost of upfront loading time. If we continue with a client-side approach, having the extra flexibility provided by a "tags" field won't significantly harm us and actually may make managing the database easier in the future. If we have a new boolean tag to add, we can just add it without having to modify the database.

Fixed via https://github.com/KhronosGroup/glTF-Project-Explorer/pull/24


[FEATURE REQUEST] Icon size

It would be great to have an option to change the icon size. The icons are smaller than the ones on the taskbar and it would not ba a problem if they where the same size, but the possibility would be nice. also the folder icon you set for the shortcut is also smaller than the taskbar icons.
Im not sure if its related but i have a 2560x1600 resolution and i use 110% scaling.

I think #35 is what you are asking for. I'll merge it in after I figure out DPI related issues with the application right now.

Merged in https://github.com/PikeNote/taskbar-groups-pike-beta/commit/68ecf909a8df6f63481795162885a57a40bd2d44


Segmenation Fault Code - Running on Kali

VS Code Version: 1.85.1
OS Version: Kali Linux 6.3.0-kali1-amd64

Steps to Reproduce:

Install VSCode
Install Python 3 extension
Install Python 3 Extension Pack (2 or 3rd on the list of extensions when searching for Python)
terminal: gh repo clone oxylabs/how-to-build-a-price-tracker (as a test file)
VSCode: 'Save As' file as price_tracker_test.py
Close and relaunch VSCode with crash and segmentation error.

Error:'
──(tnkbel㉿thinkmachine)-[~]
└─$ code
┏━(Message from Kali developers)
┃ code is not the binary you may be expecting.
┃ You are looking for "code-oss"
┃ Starting code-oss for you...
┗━
[main 2023-12-30T11:26:45.122Z] update#setState disabled
[main 2023-12-30T11:26:45.124Z] update#ctor - updates are disabled as there is no update URL
zsh: segmentation fault  code
`
I've seen this issue report in the past but mostly on Windows machines, so the suggested fixes weren't very helpful.
Looking for some help here. VSCode launches and after about 2-3 seconds closes.
Thanks in advance.

Hi, I'm obviously not a developer but I was just looking through issues for fun really. I wonder if you have tried reinstalling vscode - I know it's not ideal but it looks like your machine could have installed it incorrectly. You could also try installing it using the .deb package from Microsoft as it's possible the version installed by apt is broken somehow

Please attach the crash logs following the steps at https://github.com/microsoft/vscode/wiki/Native-Crash-Issues#creating-a-crash-report

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


fix: remove two_passes_id_search and query_params_key from providers …

…config
removes two_passes_id_search and query_params_key from the providers.yml file because they are not used anymore

query_params_key is still present in config.py
https://github.com/CS-SI/eodag/blob/0abfd6f953f3e3054cd7cf5550660eb0809d889e/eodag/config.py#L317
Is the intention of this PR to remove it only from providers.yml?

Good point. I remove it from config.py too.
NOTE: failing tests are not related to changes on this bracnh but to a general problem with shapely and python 3.12


Add C API function that returns all parameter names with their aliases

Fixed #2633

OK, I believe this PR is ready for reviews.

@jameslamb Thanks a lot for your review comments! I think I've addressed them all in the latest commit. Please take another look when have time.

/gha run r-valgrind

@shiyu1994 do you support this addition to c_api.cpp? I think changes to that file should probably have two approvals, just since it is the core public API of LightGBM.


PodDisruptionBudgets API version policy/v1beta1 removed in Kubernetes 1.25

Describe the bug
I was trying to install kuberhealthy  in my kubernetes cluster version 1.25, but I faced issue with kind: PodDisruptionBudget since in version v1.25  PodDisruptionBudgets API version policy/v1beta1 removed.
Steps To Reproduce
# If you don't use Prometheus:
kubectl create namespace kuberhealthy
kubectl apply -f https://raw.githubusercontent.com/kuberhealthy/kuberhealthy/master/deploy/kuberhealthy.yaml

Can you please add support to the latest version of kubernetes cluster v1.25 of PodDisruptionBudget

Hi @seifrajhi the static yaml files still need to be updated to account for the deprecation of policy/v1beta1.
The remaining components should install without issue and you should have a functional Kuberhealthy in your K8s cluster.

avoid stale

/do-not-close

avoid stale

@seifrajhi curious if you were able to get this working with v1.25/ if the team addressed your issue?

This should actually be working in the latest versions.  I see the proper version in the helm chart's PDB.  Maybe you are using an old version of the chart?
Current PDB template: https://github.com/kuberhealthy/kuberhealthy/blob/master/deploy/helm/kuberhealthy/templates/poddisruptionbudget.yaml


Remove cmarkgfm from CI

Or come up with a workaround for the wheel on py3.11.
Context: there is no wheel of cmarkgfm available for py3.11, so our tox invoked environments all crash on failure to asssemble the package.
This will be especially important to be finished before end of September, as the release of python 3.12 will likely increase the incidence rate with py311 crashing our tests.

Completed in sphinx upgrade PR #32066


elm make errors on any package that uses test-dependencies

✅ If I have an application with valid elm.json, elm make accepts it and works as expected, both when I do and when I do not specify any test-dependencies.
✅  If I have a package whose elm.json file specifies "test-dependencies": {}, that also works as expected.
However, if I have a package whose elm.json file specifies "test-dependencies": { "elm-explorations/test": "1.0.0" }, elm make gives this error message:
-- BAD JSON ----------------------------------------------------------- elm.json

The string at project['test-dependencies']['elm-explorations/test'] is causing
issues.

It is not a valid constraint. The standard format is "1.0.0 <= v < 2.0.0"

SSCCE
Put this elm.json in an empty directory and run elm make:
{
    "type": "package",
    "name": "repro/bug",
    "summary": "Reproduce a bug",
    "license": "BSD-3-Clause",
    "version": "1.0.0",
    "exposed-modules": [
    ],
    "elm-version": "0.19.0 <= v < 0.20.0",
    "dependencies": {
        "elm/core": "1.0.0 <= v < 2.0.0"
    },
    "test-dependencies": {
        "elm-explorations/test": "1.0.0"
    }
}

Extra Info
It appears to be looking for "direct" and "indirect" within test-dependencies, because it gets farther when I try this:
    "test-dependencies": {
        "direct": "1.0.0 <= v < 2.0.0",
        "indirect": "1.0.0 <= v < 2.0.0"
    }

Then the error changes to:
-- BAD JSON ----------------------------------------------------------- elm.json

The object at project['test-dependencies'] is causing issues.

The "direct" entry is not a valid package name. I recommend deleting it, finding
the package you want on the package website, and installing it with the
`elm install` command instead.

So it appears to be looking for a field named direct, but then tries to interpret direct as a package and gets stuck. I also tried this:
    "test-dependencies": {
        "direct": {
            "elm-explorations/test": "1.0.0"
        },
        "indirect": {}
    }

...which resulted in a different error:
-- BAD JSON ----------------------------------------------------------- elm.json

The object at project['test-dependencies'].direct is causing issues.

I was expecting a STRING.


fix(link): removed hover on mobile devices

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9363

Relevant Lines:
10247

💛  - Coveralls


Code review

Do the code review and create issues for:
Fix JSON req/rsp  #191
Guard present offer #192
cleanup init for UIs #193
Use standard debounce #194
copy resources from TikiSdk Package #195
Docs #183
Tests https://github.com/tiki/tiki-sdk-dart/issues/282
use md for learnmore #197
create story for fixing Bluetooth permissions #196

Code review updates added to #199


sharing a customized profile with friends

Hello,
I open this issue to get the Status of this old request:
Today, the use of a "customized profile" in the Brouter-web is not very easy. Many issues/requests exist, see
https://github.com/nrenner/brouter-web/issues/26
as exemple.
For my own, I have such a customized profile (base on fastbike.brf) which I liked to share with Friends....
To make the use of a custom profile easier, many solutions are possible:

Adding my profile in the list of the standards profiles (who / which concil can validate this?)
Extend the "profile box" in brouter-web (nearly as the nogo upload). As result, a custom profile saved on the PC could be uploaded and activated automatically by start)
or or or…
Hope a solution can be found soon, and the fantastic Features of brouter can be used by all
Regards

For sharing #26 is the intended solution, so closing as duplicate.
The status is that #26 and some form of #127 are planned for of the upcoming version, see milestone (from my part; there will probably be some intermediate release). But I keep planning this for quite some time now with always something else getting inbetween, so ...

Adding my profile in the list of the standards profiles (who / which concil can validate this?)

As @abrensch is responsible for the domain and hosting, I leave it to him to decide if or which additional profiles should be added. You could add an issue or pull request at https://github.com/abrensch/brouter (see e.g. abrensch/brouter#179).


[Feature request] Pass custom fetch function

Hello,
It would be nice to be able to pass a specific fetch function to use the adapter with, so that it could support peculiar environments such as SvelteKit's SSR load function.
Thanks

@cuibonobo 🙏

What do you propose the API for this should be?

How about axios.create({ adapter: fetchAdapter({ fetch }) }); ? Thanks

I was afraid you would say this! I have been testing out this API and it might be dangerous if Axios ever changes their config params to include "fetch". The fetchAdapter() function needs to return a Promise when it's used inside of Axios, but with the proposed API, it would need to return either a Promise or a function, depending on whether you pass a fetch function or not. If Axios changes their config in the future to include a "fetch" param, it will clash with this API so it will not be clear whether to return the Promise or the function.
That was a mouthful, so I'll illustrate with some code:
// Signature for the original `fetchAdapter` function. Axios uses this internally to make requests. It accepts
// an `AxiosRequestConfig` object and returns a `Promise`
function fetchAdapter(config: AxiosRequestConfig): Promise<AxiosResponse>;

// Proposed API, which *generates* a function with the above signature. Luckily `AxiosRequestConfig`
// from above does not contain a "fetch" key, but if that ever changed in the future, we wouldn't be
// able to distinguish the params between the top signature and the bottom signature.
function fetchAdapter(config: {fetch: any}): (config: AxiosRequestConfig) => Promise<AxiosResponse>;

Maybe an alternative could be a separate function for setting a custom fetch if that is ever needed? For example:
import axios from "axios";
import myFetch from "myFetch";
import fetchAdapter, { fetchAdapterFactory } from "@haverstack/axios-fetch-adapter";

// Current behavior
const defaultFetchClient = axios.create({
  adapter: fetchAdapter
});

// Proposal for custom `fetch` functions
const customFetchClient = axios.create({
  adapter: fetchAdapterFactory(myFetch)
});

Actually, I was implicitly suggesting that fetchAdapter would, from now on, permanently use the second signature, which, of course, would be a backwards-incompatible change.
The lib still being in prerelease, it shouldn't be a problem, right ?

I'm not willing to do that.

Why not ?

I don't want to introduce a backwards-incompatible change.

I don't see why these functions have to be called the same thing. What's wrong with using a factory function when you want to set your own fetch?

That's an option as well. I just find the first one more sexy, but if you choose the second one then I'll go with it.
But in that case, please drop the OOP vocabulary that JS doesn't need and just call it createFetchAdapter.
Also, here's an intermediate alternative : still using fetchAdapter(), but with { adapterConfig: { fetch } } as parameter, preventing any future ambiguity.

Oh and regarding the second option, I just noticed that you're proposing to pass fetch without putting it in an object : I don't recommend this, because it makes it less convenient to add future configuration options if needed.
Thanks

I'm not really interested in sexyness, but I appreciate the alternative name suggestion.

Fixed in 4bd2d4984b88679eb36ca22f20a256d7df0a0916. Deployed to NPM as v0.8.0.

Thanks !
However, I've got a new problem : it returns ERR_INVALID_URL when using an url that starts with /, which, using normal fetch, would indeed be incorrect, but using a custom one in an SSR context, is proper.
Thanks

Could setting the baseURL in the Axios config to http://localhost help with this?

No, because then it would make an actual request to localhost, which will obviously fail.

I'm honestly not sure how to accommodate this. The ERR_INVALID_URL is coming from Axios itself, not the code in this repo. But even if that weren't the case, this adapter works by creating a new Request object, which expects a fully-qualified URL. Does Svelte have a custom Request object as well? If you can get Axios to stop yelling at you, maybe the fix could be something like:
const myFetchAdapter = createFetchAdapter({ fetch: myFetch, request: myRequest });

Here's the stacktrace :
TypeError: Failed to parse URL from /some-endpoint
    at new Request (node_modules/undici/lib/fetch/request.js:87:15)
    at createRequest (node_modules/@haverstack/axios-fetch-adapter/lib/index.js:440:10)
    ... 5 lines matching cause stack trace ...
    at axiosAdapter (node_modules/@haverstack/axios-fetch-adapter/lib/index.js:336:12)
    at Axios.dispatchRequest (file://node_modules/axios/lib/core/dispatchRequest.js:51:10)
    at Axios.request (file://node_modules/axios/lib/core/Axios.js:148:33) {
  [cause]: TypeError [ERR_INVALID_URL]: Invalid URL
      at new NodeError (node:internal/errors:405:5)
      at new URL (node:internal/url:611:13)
      at new Request (node_modules/undici/lib/fetch/request.js:85:21)
      at createRequest (node_modules/@haverstack/axios-fetch-adapter/lib/index.js:440:10)
      at node_modules/@haverstack/axios-fetch-adapter/lib/index.js:337:23
      at Generator.next (<anonymous>)
      at node_modules/@haverstack/axios-fetch-adapter/lib/index.js:66:61
      at new Promise (<anonymous>)
      at __async (node_modules/@haverstack/axios-fetch-adapter/lib/index.js:50:10)
      at axiosAdapter (node_modules/@haverstack/axios-fetch-adapter/lib/index.js:336:12) {
    input: '/some-endpoint',
    code: 'ERR_INVALID_URL'
  }
}

Which seems to indicate that the issue comes from Request indeed.
Could the usage of Request be disabled when passing a custom fetch instance ?
Thanks

The Request object is useful for building up authentication headers and whatnot. Better to disable it explicitly if it's not desired, because of the possibly strange side-effects. Something like:
const myFetchAdapter = createFetchAdapter({ fetch: myFetch, disableRequest: true });

What do you think?

The Request object is useful for building up authentication headers and whatnot.

Why ? The official adapters don't need it, e.g. here's what their XHR one uses :
https://github.com/axios/axios/blob/v1.x/lib/adapters/xhr.js#L74-L79
Also, I made an adapter myself for another environment, so I used the same idea :
https://git.kaki87.net/KaKi87/axios-tauri-adapter/src/branch/master/index.js#L43

Better to disable it explicitly if it's not desired, because of the possibly strange side-effects.

I'm afraid such effects are unacceaptable.
Thanks

The idea is to pass the URL directly to the fetch function without creating a Request object first. An example of a side effect of this is that the AxiosResponse object will just have this URL in its request property. This means that accessing, for example, response.request.url won't work.

Well, if only response.requests is affected then it's fine.
But if config is affected then its not.

What's the verdict ?
Thanks

Hello ?

Hello,
Any news on this ?
Thanks

The disableRequest option is now available in version 0.11.0 on NPM.


Fix Maintain Webring Script

Fixed

Also please allow workflows to write repository or else this wont work

And please enable the workflow i think its disabled manually by someone cuz last time it ran was 2 weeks ago

-.-

yeah the branch is main not master

yeah the branch is main not master

Makes sense, I'll send a pr to try to fix


Fixed youtube errors in getting started page

Purpose
Currently, a number of youtube videos in the Getting Started Page have menu bars that disappear on mouseover and have non-functional buttons.
Closes Issue https://github.com/CenterForOpenScience/osf.io/issues/2640
Changes
Put youtube videos in separate containers.  Not sure what this changes but now all the videos work.
Side Effects
Fixes problem but makes code a little more lengthy than I would have wanted

@PatrickEGorman this bug exists on production, which means your branch should based on master branch. And when you do a hotfix, you need to create a new branch based on that instead of just directly create a pull request from your develop or master branch. Nice work on the comment part tho.

ping @caneruguz for review of the markup/CSS for this fix.

Thanks for taking a look at this @PatrickEGorman ,this is a good first step but before it can be merged there are a few things:

Row needs to be it's own div; as shown in the Bootstrap guidelines: http://getbootstrap.com/css/#grid
You don't need to add col-**-12 multiple times, Bootstrap goes from lowest to highest, so if "col-sm-12" is defined then every upper size is also 12. Also in this case "col-xs" is also 12 since 12 is the default for undefined sizes.
But more importantly you don't need to add column sizing classes at all since the mako files inside have those already.

I don't know why this is fixing the youtube issue but there is no clear reason why it should so that's very curious. Can you look into it a bit more?  It seems like adding a separate

 might be the thing doing the trick but everything else is redundant.

@chennan47 I will start working on this right away.
@caneruguz Sounds good.  I toyed with it a lot and though this is kind of ugly, it's the only thing I could get to work.  I'll work on it some more and see if I can't figure out a more efficient way of making them work.

I've tried

 and it's not working.  I'm going to try to simplify the solution as much as I can but it still might look a little strange....

Just a quick update:  I've managed to simplify some but the aesthetic of the code still isn't ideal.  It is in the format:

            <%include file="(template)"/>

I'm not very pleased with the appearance of this and I'm sure others won't be as well.  However, I've tried pretty much everything I could think of and this is the simplest structure that works.  One of the videos is being especially difficult as it is wedged in another template and doesn't respond the same way as the others.  Hopefully I'll figure out something that works soon.


reading LuaControl.force should only return LuaForce, not LuaForce | string

At the moment, reading LuaControl.force returns either a string or LuaForce, but it should only return a LuaForce.
  const player = game.players[playerIndex]

  const force : LuaForce = player.force
  // ERROR
  // TS2322: Type 'ForceIdentification' is not assignable to type 'LuaForce'.
  // Type 'string' is not assignable to type 'LuaForce'.

However, assigning a string should be possible.
I think this is probably true anywhere that ForceIdentification can be read, but that's a guess. At a minimum it's also documented as the case for

LuaControl.force
LuaEntity.render_to_forces
BaseEntity.render_to_forces

I also guess this is similar to #7
This is a really minor issue, it's not disrupting me, it's just something I noticed.
https://lua-api.factorio.com/latest/LuaControl.html#LuaControl.force

The force of this entity. Reading will always give a LuaForce, but it is possible to assign either string or LuaForce to this attribute to change the force.

Hi, thanks for the suggestion!
I verified that reading ForceIdentification always yields a LuaForce. A similar thing is true for ItemPrototypeSpecification and TechnologyIdentification.
This change was easy enough to implement, so I did just that.
Released in v0.18.1

Thanks for the quick fix! It's really nice using this library.


Beef up Server-API to facilitate more complex front-end queries

As mentioned in this PR on the front-end, the idea came up to have the server return more useful things.
For example:

individuals for only a specific package
packages could contain more information, as extracted from the YAML file, so the description, contributors, SNP sets, and things like that.

And also bib-information!


Update infra: v.1.2.25

Summary | Résumé
Bump to version 1.2.25
add new app_db_user to rds proxy (#720)

add new app_db_user to rds proxy
dont need that variable
I think I need more coffee

v1.2.24: Upping aws provider to ~4.0 in staging (#729)

Upping aws provider to ~4.0 in staging
Restoring provider block in ses terragrunt hcl since it loses the state when removed

v1.2.22

Drop '%' character from metric patterns #728

@whabanks @ben851 I'm a bit confused... why aren't the RDS changes in #720 reflected in the plan? And why does the plan for RDS have so much "changed in AWS but not in terraform" items?


Prefer opening local documentation for active connection if available

Fixes https://github.com/Microsoft/RTVS/issues/2553
Having an option for it seem overkill.
This will open the documentation from local disk when there's an active local connection, otherwise it will go online.
It would be possible to check all connections if there is no active connection, but that feels like it would be unpredictable behavior.

👍


Refactor valueFromAST

Pull Request Test Coverage Report for Build 925

54 of 113 (47.79%) changed or added relevant lines in 9 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.07%) to 82.203%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Type/Definition/EnumType.php
5
8
62.5%

src/Execution/ValuesResolver.php
14
18
77.78%

src/SchemaBuilder/DefinitionBuilder.php
4
10
40.0%

src/Util/ValueNodeCoercer.php
19
65
29.23%

Files with Coverage Reduction
New Missed Lines
%

src/SchemaBuilder/DefinitionBuilder.php
1
65.25%

Totals

Change from base Build 917:
0.07%

Covered Lines:
4374

Relevant Lines:
5321

💛  - Coveralls

Pull Request Test Coverage Report for Build 925

54 of 113 (47.79%) changed or added relevant lines in 9 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.07%) to 82.203%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Type/Definition/EnumType.php
5
8
62.5%

src/Execution/ValuesResolver.php
14
18
77.78%

src/SchemaBuilder/DefinitionBuilder.php
4
10
40.0%

src/Util/ValueNodeCoercer.php
19
65
29.23%

Files with Coverage Reduction
New Missed Lines
%

src/SchemaBuilder/DefinitionBuilder.php
1
65.25%

Totals

Change from base Build 917:
0.07%

Covered Lines:
4374

Relevant Lines:
5321

💛  - Coveralls


Ensure an empty osf_settings.API_PORT works properly

Currently, the API Port property is expecting a number. I expect that if it's empty, rather than going to the default port, it will append a : to the end of the URL with no port after it.

Furl or urlparse should be able to deal with this intelligently..?
>>> e = furl.furl('http://google.com'); e.port=None; print str(e)
http://google.com

:+1:

The setting is removed and unnecessary now.


Updates you might be interested in

I've changed the original script to suit my own needs. You may find some of the changes useful, but probably not all of them.
I've completely eliminated the app.js file in favour of Frida's python support, switched to argparse for argument handling, added a named output option, replaced the os.system calls with subprocess and use paramiko for ssh support. I've tested the app with hundreds of IPAs, so it should generally work well. I use the process.sh script to test all of the IPAs downloaded in the iTunes Mac client.

Thanks so much for a very helpful PR! 🎉


examples - input fields in forms not user friendly - cannot click in large part of the input area

https://getbootstrap.com/docs/5.3/examples/sign-in/
Say you click here:

It does not work, it makes no sense and is user hostile IMO. Many times I by mistake clicked on the non-clickable area in the input field.

Thanks for opening this issue @gregjotau.
It was linked to an issue of the floating labels.
It has been fixed recently in our main branch (see https://twbs-bootstrap.netlify.app/docs/5.3/examples/sign-in/) and will be available in the next v5.3.0-alpha3.


Azure API error

Hello,
I am using the Azure OpenAI service and have had the following error message, when initialising the model:
# Or, to use with Azure
tm = TopicGPT(
    api_key = <your-azure-openai-api-key>,
    azure_endpoint = {
         "endpoint": <your-azure-openai-endpoint-url>,
         "api_version": <api-version>
     },
    n_topics = 20
)

TypeError: TopicGPT.init() got an unexpected keyword argument 'azure_endpoint'
On another note, api_key needs to change to openai_api_key on this snippet in my testing.
Any clues to fix this error?
Thanks!

Hi, try this
tm = TopicGPT(     openai_api_key = api_key_openai,     n_topics = 20 )

Hi, I am also facing the same issue, is is not working for azure api?
TypeError: TopicGPT.init() got an unexpected keyword argument 'azure_endpoint'


Support for timers?

Hi. I am looking into profiling the performance of some Ajax http requests. Does this library support timers in a similar fashion to http://fiddler.wikidot.com/timers?

Fiddler is a web debugging proxy, and we are more focused on performance and being light weight.
So we don't support it. But you can accomplish some of the timer functionality but adding timers inside the request/response handlers


End to End test for upload file

Currently there is not an end to end test for upload file

End to end tests are not immediately feasible for this.


read lengths differ

R1 27; R2 125
R1 29; R2 99
R1 29; R2 92
Alignment crashes when the read length is less than 31.

[2024-10-19 14:41:05.213] [info] enable structural constraints : false
[2024-10-19 14:41:05.213] [info] No poison k-mer map exists, or it was requested not to be used
[2024-10-19 14:41:05.213] [info] loading index from /scratch/xm41/hg38_resources/scRNA/refdata-gex-GRCh38-2020-A/human-2020-A_splici/91/index/piscem_idx
[2024-10-19 14:41:08.690] [info] done loading index
Error: piscem mapping failed with exit status ExitStatus(unix_wait_status(134))

Should I create an index with simpleaf index -k 26?

Can you please say how you are running simpleaf (the invocation) and also what versions of simpleaf, piscem and alevin-fry you are using?  R1 is usually the technical read and only needs to be long enough to hold the barcode and UMI. Most commonly, it’s not even attempted to align it. Regardless, in the worst case reads shorter than the kmer length should be discarded ( not cause termination ). You shouldn’t have to change the index.
—Rob

I highly appreciate your quick response.
I am following the simpleaf online documentation.
That's how I created the index:
rlen=91
IDX_DIR="${REF_DIR}/human-2020-A_splici/${rlen}"

 simpleaf index \
--output $IDX_DIR \
--fasta $REF_DIR/fasta/genome.fa \
--gtf $REF_DIR/genes/genes.gtf \
--rlen ${rlen} \
--threads $(nproc) \
--use-piscem

simpleaf quant \
--reads1 ${workdir}/${sra_id}_1.fastq.gz \
--reads2 ${workdir}/${sra_id}_2.fastq.gz \
--threads $(nproc) \
--index $IDX_DIR/index \
--chemistry 10xv3 --resolution cr-like \
--expected-ori fw --unfiltered-pl \
--t2g-map $IDX_DIR/index/t2g_3col.tsv \
--output ${workdir}/alevin/${sra_id}

Hrmm, certainly it should not be the case that read 1 is being mapped in 10xv3 chemistry. You more that this happens when read 1 is < 31 nucleotides. Do you have data with longer R1 where it runs to completion?
If it is possible to share your index somehow, and a small sample of reads that reproduce the problem (a few hundred or few thousand should suffice), we’d be happy to take a look.

Sure. I am happy to. Where should I send the files to?

Hi @robertzeibich,
The read pairs (when gzipped) should be small enough to upload directly here. If you send me an e-mail (rob@cs.umd.edu), I can share a box folder with you where you can upload the index.
--Rob

I selected the first 5,000 rows:
SRR12905322_1.fastq_sub.gz
SRR12905322_2.fastq_sub.gz
I hope that this is enough. I will upload the index in a second.

Based on your question: Do you have data with longer R1 where it runs to completion? Yes, I have longer R1 where it runs to completion.

This issue can be closed due to @rob-p support.
If you should run into the same error (see first message), make sure that the FASTQ files were downloaded correctly.
Re different read lenght:

"R1 is usually the technical read and only needs to be long enough to hold the barcode and UMI"
R2 contains the sequence. From a given sequence the length can be extracted.

As I stated in my first message, I got read lengths of
R1 27; R2 125
R1 29; R2 99
R1 29; R2 92
At the end, I used the read lengths 98 and 91 to build the indices. However, I also tried 99 and 92 and the number of cells and genes were exactly the same.


Release 1.0.0

Breaking changes

Pull request #60: Upgrade to<EMAIL_ADDRESS>
See notes on https://github.com/alphagov/accessible-autocomplete/releases/tag/v2.0.0 due to breaking changes (now using GOV.UK Design System WCAG 2.1 compatible CSS)
Closes https://github.com/alphagov/govuk-country-and-territory-autocomplete/issues/22

@nickcolley Thanks for the switch to npm.


Time picker - set Time Slots : Unable to view 12:00 time when set through setSelectableTimes - 12 hour format.

Log :
11:20, 11:40, 12:20, 12:40, 14:20, 14:40, 15:00, 15:20, 15:40, 16:00, 16:20, 16:40, 17:00, 17:20, 17:40, 18:00, 18:20, 18:40, 19:00, 19:20, 19:40, 20:00, 20:20, 20:40
if (workingDays.containsKey(dateKey)) {
ArrayList timeSlots = workingDays.get(dateKey);
timepoints = new Timepoint[timeSlots.size()];
            for (String timeSlot : timeSlots) {
                Logger.d("TimeSlots",timeSlot);
                String[] timeSplit = timeSlot.split(":");
                int hour = Integer.parseInt(timeSplit[0]);
                int min = Integer.parseInt(timeSplit[1]);
                timepoints[timeIndex++] = new Timepoint(hour, min, 00);
            }

            if( timeSlots != null && timeSlots.size() > 0 ) {

                hour = Integer.parseInt(timeSlots.get(0).split(":")[0]);
                minutes = Integer.parseInt(timeSlots.get(0).split(":")[1]);
                timePickerDialog = com.wdullaer.materialdatetimepicker.time.TimePickerDialog.newInstance(
                        BookASlotActivity.this,
                        hour,
                        minutes,
                        false
                );
                timePickerDialog.setSelectableTimes( timepoints );
                timePickerDialog.setOnTimeSetListener(this);
                timePickerDialog.show(getFragmentManager(), "TimePickerDialog");
            }

        }

What is the actual issue here?
You are constructing an Array of TimePoints that doesn't contain 12:00. This does not seem to be related to the library.

Its not just an issue with 12:00, any slot which is available from 12 to 1 wouldn't render 12 as selected in time picker. This is the screen shot :

Time slots from Log :
[10:00, 10:20, 10:40, 11:00, 11:20, 11:40, 12:00, 12:20, 12:40, 14:00, 14:20, 14:40, 15:00, 15:20, 15:40, 16:00, 16:20, 16:40, 17:00, 17:20, 17:40, 18:00, 18:20, 18:40, 19:00, 19:20, 19:40, 20:00, 20:20, 20:40]

To add : The setMinTime and setSelectable times , when used - the TPD does not show 12 as highlighted white (selectable color) , one can select 12 and its min slots but not see the 12 highlighted as seen above.

Ok, I get it now.
I'll have a closer look.


Feature value object support

When a value is an object, try to do a __toString().
If that object does not implement that method, an exception will be thrown.

Please provide documentation.


Make tracker code work with the simulation

Really this shouldn't be too hard. Just subscribe to the simulation camera's topic, instead of what it's currently subscribed to. Also make it optional to make it flexible.

Working on this now. Shouldn't be too hard...

Done in 3fc65dd
I couldn't seem to figure out a good way to make it optional however, the topic is still hardcoded.


added verify option to util.openURL

ssl verification is enabled by default and can be disabled with verify=False. This flag can be used by wps, wms, etc ... when working with self-signed certificates.
http://docs.python-requests.org/en/latest/user/advanced/?highlight=ssl#ssl-cert-verification

@tomkralidis
Reviewed myself. Test are passing except for a timeout of a WMS service. It is only an optional parameter. The "real" changes need to happen in the WPS etc modules.


Flag to launch non-Byron-only testnet with custom activeSlotCoeff

The following shows that the genesis files get updated to have the provided activeSlotCoeff value:
$ (cd cardano-node-chairman; export KEEP_WORKSPACE=1; cabal run -- cardano-testnet shelley --active-slots-coeff 0.25)
$ for x in $(find /private/var/folders/zh/ln41q4zs52x2fd61rxccmq640000gn/T/chairman/test-2dbb537814cf6ce0/ -name 'g*.json'); do echo "== $x =="; cat $x | jq . | grep eff; done
== /private/var/folders/zh/ln41q4zs52x2fd61rxccmq640000gn/T/chairman/test-2dbb537814cf6ce0//genesis.json ==
  "activeSlotsCoeff": 0.25,
== /private/var/folders/zh/ln41q4zs52x2fd61rxccmq640000gn/T/chairman/test-2dbb537814cf6ce0//genesis.spec.json ==
  "activeSlotsCoeff": 0.25,

Change the text to use a slightly better command.  Note this just in the PR description and not in the git history.

Yes, but if you use github to merge it, it will be in the git history.

That's awkward.  Moved it into a comment:
The following shows that the genesis files get updated to have the provided activeSlotCoeff value:
$ (cd cardano-node-chairman; export KEEP_WORKSPACE=1; cabal run -- cardano-testnet shelley --active-slots-coeff 0.25)
$ for x in $(find /private/var/folders/zh/ln41q4zs52x2fd61rxccmq640000gn/T/chairman/test-2dbb537814cf6ce0/ -name 'g*.json'); do echo "== $x =="; cat $x | jq . | grep eff; done
== /private/var/folders/zh/ln41q4zs52x2fd61rxccmq640000gn/T/chairman/test-2dbb537814cf6ce0//genesis.json ==
  "activeSlotsCoeff": 0.25,
== /private/var/folders/zh/ln41q4zs52x2fd61rxccmq640000gn/T/chairman/test-2dbb537814cf6ce0//genesis.spec.json ==
  "activeSlotsCoeff": 0.25,

bors merge


Preprocess the malware analytic results

Preprocess data from the services below:

[x] PEinfo
[x] Objdump
[ ] YARA
[x] Cuckoo
[x] Richheader

Currently the yara service is no longer maintained.

#13 #14 #15 #16 #17


Fix POM

Modernize and standardize POM, concentrate version management into top-level POM

What are the changes (the diff shows only red and green) and we had one POM change request which unfortunately consistently failed on that -- will the maven publish still work afterwards?

Find patch attached. POM.patch
With regards to motivation for modifying the Maven config: Find log files attached, resulting from a compilation run against the current master. mustangproject-master-logs.zip
With regards to build issues:
As far as I see there are two reasons why the build fails:

Error:  Tests run: 14, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.568 s <<< FAILURE! -- in org.mustangproject.ZUGFeRD.ZF2PushTest
Error:  org.mustangproject.ZUGFeRD.ZF2PushTest.testRead -- Time elapsed: 0.002 s <<< ERROR!
org.mustangproject.ZUGFeRD.ZUGFeRDExportException: java.nio.file.NoSuchFileException: \Users\jstaerk\temp\1424413_anonymized.xml

This one has been fixed with PR#626. The issue has nothing to do with PR#623.

[INFO] Running org.mustangproject.ZUGFeRD.DeSerializationTest
Error:  Tests run: 9, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.567 s <<< FAILURE! -- in org.mustangproject.ZUGFeRD.DeSerializationTest
Error:  org.mustangproject.ZUGFeRD.DeSerializationTest.testFileSerialization -- Time elapsed: 0.021 s <<< FAILURE!
junit.framework.AssertionFailedError

For me it works:

[INFO] Running org.mustangproject.ZUGFeRD.DeSerializationTest
[INFO] Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.855 s -- in org.mustangproject.ZUGFeRD.DeSerializationTest

Find log files attached. So, this issue has nothing to do with PR#623 as well.


First time logged LDAP user (say mobla) can create a project, add other LDAP user to the new project that is created. Later that user loses admin rights to the project

Production harbor instance: (reg-xxxx)
Harbor Version: 0.4.6-test
Steps:
0. Ensure other users (say user1, user2 were logged in and user-profiles were created in Harbor)

Login as LDAP user (vmware ldap - mobla)
Create a project say mobla-project
At this point Harbor Admin UI lets mobla user to add other LDAP users to the mobla-project
Now, logout and re-login as mobla user
mobla user lost admin rights to the mobla-project and not able to add new user or modify user role to the existing users

Live setup is accessible to Harbor team (reg-xxxx)..
Thanks,
Murali

I can still see you are project admin. Can you provide a screenshoot?

After refresh the browser, it worked fine. Please check.

Refresh does not help in my case. Works fine on Incognito window or a new session. Browser cache issue with different users logged in with different roles?

please clean the browser history and refresh, yes it's cache issue, some part of the code is not latest on the browser and failed to work with the latest response data.

@mobla based on our chat I believe this is due to browser cache, can I close this issue?

Should we document this?

We have created a fix for this to clear the cache after upgrading to a newer version.

#1073 tracked the issue about having to clear browser cache after upgrade.
closing this issue.


Add type inference for stablehlo.convolution.

Request description
Follow docs/type_inference.md.

Still WIP and expect to finish EOW.

Delayed by MHLO integration work, expect to be finished in coming week.

Still in progress, as an issue in https://github.com/openxla/stablehlo/pull/360 still need further investigation/debugging.


add Chromium item to Edge troubleshooting

Docs Build status updates of commit 433f174:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 37b294b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 124879f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Tracking issue of RFC-0191: Async Streaming IO

Tracking of RFC-0191: Async Streaming IO
Checklist

[x] https://github.com/datafuselabs/opendal/pull/192
[ ] refactor write
[ ] make sure layer works as expected (callback and observer)
[ ] polish ReaderStream, improve the logic around file
[ ] benchmarks
[ ] add docs for stream and sink
[ ] error handling in stream

Demo benchmark:
read:
read_full/4.00 KiB      time:   [483.41 us 494.67 us 506.13 us]
                        thrpt:  [7.7179 MiB/s 7.8967 MiB/s 8.0806 MiB/s]
read_full/256 KiB       time:   [560.29 us 570.83 us 581.40 us]
                        thrpt:  [430.00 MiB/s 437.96 MiB/s 446.20 MiB/s]
read_full/4.00 MiB      time:   [1.7365 ms 1.7699 ms 1.7986 ms]
                        thrpt:  [2.1718 GiB/s 2.2071 GiB/s 2.2495 GiB/s]
read_full/16.0 MiB      time:   [6.2546 ms 6.2921 ms 6.3295 ms]
                        thrpt:  [2.4686 GiB/s 2.4833 GiB/s 2.4981 GiB/s]

read2:
read_full/4.00 KiB      time:   [484.17 us 497.62 us 511.67 us]
                        thrpt:  [7.6344 MiB/s 7.8499 MiB/s 8.0679 MiB/s]
                 change:
                        time:   [-1.8230% +1.3081% +4.6383%] (p = 0.40 > 0.05)
                        thrpt:  [-4.4327% -1.2912% +1.8569%]
                        No change in performance detected.
read_full/256 KiB       time:   [526.42 us 538.48 us 550.63 us]
                        thrpt:  [454.03 MiB/s 464.27 MiB/s 474.91 MiB/s]
                 change:
                        time:   [-8.3234% -5.7718% -2.9627%] (p = 0.00 < 0.05)
                        thrpt:  [+3.0532% +6.1253% +9.0791%]
                        Performance has improved.
read_full/4.00 MiB      time:   [1.5191 ms 1.5910 ms 1.6682 ms]
                        thrpt:  [2.3415 GiB/s 2.4552 GiB/s 2.5713 GiB/s]
                 change:
                        time:   [-7.6376% -4.5056% -1.3878%] (p = 0.01 < 0.05)
                        thrpt:  [+1.4073% +4.7182% +8.2691%]
                        Performance has improved.
read_full/16.0 MiB      time:   [5.4844 ms 5.5831 ms 5.6837 ms]
                        thrpt:  [2.7491 GiB/s 2.7986 GiB/s 2.8490 GiB/s]
                 change:
                        time:   [-13.085% -11.268% -9.6865%] (p = 0.00 < 0.05)
                        thrpt:  [+10.725% +12.699% +15.055%]
                        Performance has improved.

Problem: Stream works on well on object storage, but adds extra cost for local fs.


[CARBONDATA-2606] [Complex DataType Enhancements]Fix for ComplexDataType Projection PushDown

Fix for ComplexDataType Projection PushDown when Table Schema contains ColumnName in UpperCase

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6582/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5413/

Retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6601/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5485/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5423/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5441/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6650/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5477/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6706/

Retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6708/

Retest sdv please

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5536/

Retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6714/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5541/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5563/

Retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6720/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5547/

LGTM


Multiple routes

This appears to be locked in to one universal route (in Express) All traffic flows through render.js What if one wanted to do multiple routes? (not using React router) I'm considering pulling it all apart and playing with it. My app would have to do some configuration on startup - instead of doing all the work on req/res. I just didn't know if you might have some immediate thoughts.

@nobleach I'm not sure I fully understand your question but I'll try and point you in the right direction. render.js is compiled to build/server.js which is used as middleware in the express app defined in src/server/index.js. You can see this happening in bin/server.prod.js.
In short, if you'd like to add routes at the express level, the best place to do it would probably be src/server/index.js.
Hope this helps!

No problem, glad I could help clear things up.


Move requests import to runtime

Importing requests takes well over 100ms and is used only for resolving refs from remote URIs - move the import to runtime.
--
Hopefully the test case modification is ok too - mocking a runtime import was not something I've done before :) If there is more elegant or "industry standard" solution, please let me know, and I'll fix it.

hi @hmehta I'm quite interested in this PR. If I understand, moving the requests import inside resolve_remote() would shave ~100ms off each time a schema validation is run, correct?
I'm asking because we are using this library pretty heavily in request/response cycles; any small improvement would dramatically impact our general backend performances.
Is some perf unit testing is needed, can this PR move forward? It would great to have it merged :pray:

It's not per validation, this PR is for the one time cost that happens
during import (but still do want to move it along)
On Mon, Mar 19, 2018, 17:40 apiraino<EMAIL_ADDRESS>wrote:

hi @hmehta https://github.com/hmehta I'm quite interested in this PR.
If I understand, moving the requests import inside resolve_remote() would
shave ~100ms off each time a schema validation is run, correct?
I'm asking because we are using this library pretty heavily in
request/response cycles; any small improvement would dramatically impact
our general backend performances.
Is some perf unit testing is needed, can this PR move forward? It would
great to have it merged 🙏
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Julian/jsonschema/pull/388#issuecomment-374389413,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUIXtN_SOqiO3rrhBn2k-yXK14ogMnHks5tgCXJgaJpZM4RscDZ
.

Merged finally, apologies for the delay... really appreciated!


  Cuda is set up but I get:
E tensorflow/stream_executor/cuda/cuda_driver.cc:300] failed call to cuInit: CUDA_ERROR_NO_DEVICE: no CUDA-capable device is detected
And:
INFO:tensorflow:Saving checkpoints for 0 into ../bert/squad/output/model.ckpt. INFO:tensorflow:global_step/sec: 0.0131662 INFO:tensorflow:examples/sec: 0.157994 INFO:tensorflow:global_step/sec: 0.0128566 INFO:tensorflow:examples/sec: 0.154279
ls output:
checkpoint                               model.ckpt-0.data-00000-of-00001  train.tf_record events.out.tfevents.1542228741.dccxc040  model.ckpt-0.index graph.pbtxt                              model.ckpt-0.meta
But no predictions.json file in that directory.  Also the checkpoint isn't model.chkpt.
Is the CUDA error the problem?  Thanks!

The prediction gets generated at the end of training, which takes a few hours on a GPU.

If you're having trouble with CUDA then that's a general TensorFlow environment issue I wont be able to help with.

Hai, I'm trying to run the SQuAD2.0 in colab using bert.(with TPU support on google Cloud). The training stalled after some some. I don't know what to do. Can you help with this?
And how long is the training time is for SQuAD2.0 using BERT-LARGE.

Hai, I'm trying to run the SQuAD2.0 in colab using bert.(with TPU support on google Cloud). The training stalled after some some. I don't know what to do. Can you help with this?
And how long is the training time is for SQuAD2.0 using BERT-LARGE.

Can you post a link to your notebook? Or just copy and paste the code here.

Is it possible to use smaller train-v11.1.json file to test on CPU and get predictions.json in output?

The prediction gets generated at the end of training, which takes a few hours on a GPU.

why is it making more than one ckpt files(model.ckpt-2000, model.ckpt-3000,.... . . ) of same size(1.22GB each) in the destination folder(in google drive) while fine-tuning BERT base with SQuAD in colab with GPU?


client_reference_id is null in stripe webhook

Hi - I've got this running, but adding credits through stripe the webhook callback is missing the client_reference_id, hence raising an error.  Any suggestions ?

Hey, yes you have to pass this in the stripe pricing table component as another prop

thanks- sorted


Feature/multi task

Updates

Added basic multi-task trainer for linear AutoEncoder with RMSD prediction network.
Using uncertainty weighted loss functions from https://arxiv.org/pdf/1705.07115.pdf
Refactored pydantic config to have base class taking care of wandb config
Added wandb setup utility function
dict_to_device helper function to move dictionary of torch.Tensor's to device.
Small updates to ContactMapDataset: optional values interface update, optional requires_grad for scalars.

Updates

Added basic multi-task trainer for linear AutoEncoder with RMSD prediction network.
Using uncertainty weighted loss functions from https://arxiv.org/pdf/1705.07115.pdf
Refactored pydantic config to have base class taking care of wandb config
Added wandb setup utility function
dict_to_device helper function to move dictionary of torch.Tensor's to device.
Small updates to ContactMapDataset: optional values interface update, optional requires_grad for scalars.


Issues with selinux, unlabeled container fs files.

Issues:
On my system I seem to have the below from containers, however k3s-selinux seems to have <> for the snapshot files.
This results in pods not starting,
I manually added a rule to label the files as container_ro_file_t, and the issue was immediately resolved.
OS: Rocky Linux release 8.6 (using rhel8 rpm)
system containerd rules:
/var/lib/containerd(/.*)?                          all files          system_u:object_r:container_var_lib_t:s0
/var/lib/containerd/[^/]*/sandboxes(/.*)?          all files          system_u:object_r:container_ro_file_t:s0
/var/lib/containerd/[^/]*/snapshots(/.*)?          all files          system_u:object_r:container_ro_file_t:s0

k3s-selinux rules
/var/lib/rancher/k3s/agent/containerd/[^/]*/sandboxes(/.*)? all files          system_u:object_r:container_ro_file_t:s0
/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots directory          system_u:object_r:container_ro_file_t:s0
/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots/[^/]* directory          system_u:object_r:container_ro_file_t:s0
/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots/[^/]*/.* all files          <<None>>

Manual Rule:
/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots(/.*)? all files          system_u:object_r:container_ro_file_t:s0

Ask:
Label this rule.
/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots/[^/]*/.* all files          <<None>>

Hi,
want to bump this thread, because we had this issue on our k3s instances while upgrading selinux policy from 1.2 to 1.3
The weird thing is that only some nodes were affected.
OS: RHEL 8.8
Sympthoms: such log messages in audit log
type=AVC msg=audit(1686836924.829:903): avc:  denied  { read } for  pid=18301 comm="java" path="/lib/x86_64-linux-gnu/libz.so.1.2.11" dev="dm-0" ino=202766785 scontext=system_u:system_r:container_t:s0:c547,c785 tcontext=system_u:object_r:var_lib_t:s0 tclass=file permissive=0
Cause: after upgrade of policy for some reason wrong labels were placed on k3s files. It is var_lib_t instead of container_file_t
drwxr-xr-x. 7 root root system_u:object_r:var_lib_t:s0 84 Aug 23  2022 /var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/114/fs/lib/
Temporary quick fix: chcon -Rv system_u:object_r:container_file_t:s0 /var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/*
Long term fix: as tim mentioned above it would be nice to have default label for that, so simple auto-relabel will fix an issue

I've seen the behaviour @vands56 describes in one of my updates as well, but I couldn't reproduce it. It happened during the rhel 8.8 updates so it may be because of the issues observed in #47
I did upgrade other clusters, they seemed to work fine. But I do not not understand what the selinux context should be of that directory. In the rule the files should have "none" context, but when running k3s the files do have contexts set.
The set contexts seem to be a mixture of system_u:object_r:container_ro_file_t:s0, system_u:object_r:container_file_t:s0 and system_u:object_r:container_var_lib_t:s0
So I am guessing my main question is "if the selinux contexts get messed up, how do you properly get this aligned again?" restorecon would be the obvious answer but that would mean the files in there get a fixed selinux context. Any insights on this @rancher-max or @galal-hussein

Thanks for opening the issue, as far as I can see, this line:
/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots/[^/]*/.* <<none>>

which means that the policy doesnt relabel the file system files within the container, however this line has not been changed in the newer update for the policy and been there since the initial commit, the new policy simply remove the file transition pattern to not conflict with recent changes of container-selinux.
@Timdawson264 if you are still seeing the same issue, can you provide more info on the error you seeing
@vands56 if you have an older system with the older policy can you post the labels you saw on
/var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/114/fs/lib/ before the upgrade.

@Klaas- as for your question, yes the relabeling of the files should get the labels aligned again with the file contexts configured by the policy.

@Klaas- as for your question, yes the relabeling of the files should get the labels aligned again with the file contexts configured by the policy.

You misunderstood my question :) The policy just says "no selinux file context" for files inside that dir. The current reality is that the files inside those dirs are a mixture of 3 contexts (at least in my systems): system_u:object_r:container_ro_file_t:s0, system_u:object_r:container_file_t:s0 and system_u:object_r:container_var_lib_t:s0
So it means k3s/containerd(/somthing else?) sets those contexts at the moment using some kind of logic on the fly. Where is that logic hiding? :) Are there objectively correct selinux contexts? So that we can put them into a selinux rule that we can put into k3s-selinux instead of the <> line we have currently (which will then enable using restorecon/autorelabel to fix a broken state)

@Klaas- yeah sorry about the misunderstanding, so k3s indeed adds logic to label these files, so the k3s policy has the following fc rules:
/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots           -d  gen_context(system_u:object_r:container_file_t,s0)
/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots/[^/]*     -d  gen_context(system_u:object_r:container_file_t,s0)
/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots/[^/]*/.*      <<none>>

The reason that you might end up with different contexts like container_ro_t is because of a recent update that happened to container-selinux https://github.com/containers/container-selinux/commit/8ca4b89b82846d126000601f6b8530a962f0ecd3#diff-d87cfd6c86daf9204f212619e52210d06d7575d9b4ad27b41966ab3e1b80e2a8R536 where they basically converted the container_ro_t to container_file_t, which caused a conflict with our policy, thats why we had to change the file contexts in some distros to align with container-selinux, to give you more contexts, the container_file_t is only needed when using buildkit, so it really doesnt cause a current problem with our policy.
The line:
/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots/[^/]*/.*      <<none>>

specifically has been there from the early start of the k3s-selinux policy, and I really don't know how its different now with the new policy, I am currently investigating this to see if that line is actually needed or not.

@galal-hussein okay I now understand where part of this problem comes from. Have you thought about a different approach for the containerd dirs? I think you could save yourself a few headaches if you just set equivalencies.
So as an example: Do an equivalency of /var/lib/containerd to /var/lib/rancher/k3s/agent/containerd and you no longer need to think about upstream changes, you'll just follow container-selinux "blindly"
Of cause this only works as long as you do not need to modify anything from upstream, but that seems to be the case here.

My fix was to add an new selinux rule to the system.
I haven't had the issue since.
semanage fcontext -t container_ro_file_t -a '/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots(/.*)?'

My fix was to add an new selinux rule to the system. I haven't had the issue since.
semanage fcontext -t container_ro_file_t -a '/var/lib/rancher/k3s/agent/containerd/[^/]*/snapshots(/.*)?'

So I would say it should be container_file_t
https://github.com/containers/container-selinux/blob/e3d7d0b13381f00fb39181ea0bb3e329aeed4726/container.fc#L77
container_ro_file_t is for the sandboxes path:
https://github.com/containers/container-selinux/blob/e3d7d0b13381f00fb39181ea0bb3e329aeed4726/container.fc#L78
or this needs to be corrected upstream :) but as I said, I would guess for k3s the easiest thing is setting a equivalency /var/lib/rancher/k3s/agent/containerd to /var/lib/containerd and same for /var/run/k3s/containerd to /var/run/containerd

@galal-hussein okay I now understand where part of this problem comes from. Have you thought about a different approach for the containerd dirs? I think you could save yourself a few headaches if you just set equivalencies.
So as an example: Do an equivalency of /var/lib/containerd to /var/lib/rancher/k3s/agent/containerd and you no longer need to think about upstream changes, you'll just follow container-selinux "blindly"
Of cause this only works as long as you do not need to modify anything from upstream, but that seems to be the case here.

This is interesting approach, I will have to test it but this can save us a lot of headache with recent changes to the containerd dirs, I will look into it, thanks!

In general I can just recommend to use equivalencies where possible, that way you do not need to create your own logic, this is especially useful if you just relocate something. this containerd directory behaves exactly like the standard containerd directory. This container data dir behaves exactly like the normal  container data dir etc

@galal-hussein did you have time to think about this? :)


PagerDuty incident key arguments not working?

I have looked at the original commit for dynamic properties in incident keys and believe that my setup replicates what is in the tests.
My setup:
my-alert.yaml
...
pagerduty_incident_key: 'logstash/{0}'
pagerduty_incident_key_args: [ 'company-custom-event-id' ]
...

Pagerduty alert message:
{
  "client": "Elastasearch",
  "description": "my-alert",
  "event_type": "trigger",
  "incident_key": "logstash/{0}",
  ...
}

Versions:
If relevant, I couldn't work out what version of Elastalert we're using but from Logit.io:

Elastic Search version: 5.6
Logstash version: 5.6
Kibana versoin: 5.6

You're code looks fine. Try running $ pip freeze | grep elastalert to get elastalert's version. Looks like you need at least v0.1.14. The fact that you don't even get the "missing value" thing makes me think it's the version.

Yep. Sounds like we were on an older version - just checked with their support. Closing this issue, thanks.


Missing guide for the various log levels

We lack a guide to choose when to use each of the various log levels.

mentioned in commit ff11b1168cc538e572de559666fe82a2f1674df0

mentioned in commit b4324bab5ea81793a196c1c465518da0a8ac21e6

mentioned in commit 794e59dd2727a8c92261d0c644a73a43680643f3


git-svn rebase

There seems to be something missing in this section since it refers to itself by its name multiple times:

This accepts all options that git svn fetch and git rebase accept. However, --fetch-all only fetches from the current [svn-remote], and not all [svn-remote] definitions.
Like git rebase; this requires that the working tree be clean and have no uncommitted changes.

I'm not sure what you mean; the git svn rebase relies on git rebase to do some of its work, and passes the options along to it. The text looks correct to me.
At any rate, that text is in the git-svn manpage, which is pulled verbatim from the upstream Git project (this issue tracker is just for the git-scm.com website itself). If you want to discuss it further, you'll probably get more interest from the mailing list.


Following the instructions gives me different results.

Using dotenv new console doesn't create a file with the results that follow the article text: "... Open Program.cs. It should look like this:"
Following the instruction: "Create a new file using the File:New command. Name it BankAccount.cs. Add the following code to your BankAccount.cs file:" creates a file but not in the same directory. Where should we save it?
When pasting the code into Program.cs I get an error line under BankAccount in this line: var account = new BankAccount("jimmy", 1000);. The error reads: CS0246	The type or namespace name 'BankAccount' could not be found (are you missing a using directive or an assembly reference?)

These are trivial issues if you've some coding experience, but these are aimed at beginners so I would imagine are unusable for many new-starters

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

ID: 24c8353f-69e7-9068-9b18-df0198a1b122
Version Independent ID: 823031fe-c8a4-69cd-2b1a-9faacd282aab
Content: Classes and objects  - C# Fundamentals tutorial
Content Source: docs/csharp/fundamentals/tutorials/classes.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for writing this up @jimmythecode
This tutorial needs to be updated for the changes added in C# 11. I've added it to the current sprint.


TypeError: Cannot read property 'includes' of undefined

Hello,
Thanks for your plugin.
I just installed it, and when I try to add my yeelight bulb, I get this error:

My config:
{
"name": "Yeelighter",
"timeout": 5000,
"interval": 60000,
"ctforcolor": true,
"blocking": false,
"split": true,
"manual": [
{
"id": "64:90:C1:B4:5B:E2",
"address": "<IP_ADDRESS>",
"name": "Ampoule",
"model": "yeelink.light.color5",
"log": true,
"color": true,
"backgroundLight": true,
"nightLight": true,
"separateAmbient": true,
"colorTemperature": {
"min": 5000
}
}
],
"platform": "Yeelighter"
}
What can I do ?

The JavaScript version that your node.js is running is too old. It doesn't support some of the features used by yeelighter. You can fix this by updating your node.js.
https://github.com/homebridge/homebridge/wiki/How-To-Update-Node.js
You will need node v. 16 or above


Ignore tag for acceptance tests that fails

Description
Ignore tag for acceptance tests that fails

Changes Unknown when pulling c7bf83a88a8333b89aefded23d46092fa5c35e07 on hdominguez1989:master into ** on Stratio:master**.


XMLProperty "championMasteryScore" value: "46" not implemented yet!

I get the information: "XMLProperty "championMasteryScore" value: "46" not implemented yet!"!

Solved this issue in the pull-request #30. Have to wait until it is merged.

Thank you


⚠ Bofrost.be has degraded performance

In d1e8e77, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1752 ms

Resolved: Bofrost.be performance has improved in ae17297.


Observer of Date property fires even when there is no change

Tracking down a cause of recursion in my app, I wrote this little test harness.  You will notice I write the same value back to the property that I take out of it, for both a number and a date.  With the number the observer doesn't fire, for the date it does.
<html>
<head>
  <script src="components/webcomponentsjs/webcomponents-lite.js"></script>
  <link rel="import" href="components/polymer/polymer.html">
  <script type="text/javascript">
    Polymer({
      is: 'date-tester',
      properties: {
        date: {
          type: Date,
          value: function() {
            var now = new Date();
            return new Date(
              now.getFullYear(),
              now.getMonth() - 1,
              1,
              6,0,0
            );
          },
          observer: 'dateChanged'
        },
        num: {
          type: Number,
          value: 200,
          observer: 'numberChanged'
        }
      },
      attached: function() {
        this.async(function() {
          var value = this.date.getTime();
          var n = this.num;
          this.async(function() {
            this.date = new Date(value);
            this.num = n;
          },10);
        },10);
      },
      dateChanged: function(newDate,oldDate) {
        if (typeof oldDate !== 'undefined') {
          console.log('Old Date: ' + oldDate.toUTCString());
          console.log('new Date: ' + newDate.toUTCString());
          console.log('Millisecond Diff: ' + (newDate.getTime() - oldDate.getTime()).toString());
        }
      },
      numberChanged: function(newNo,oldNo) {
        if(typeof oldNo !== 'undefined') {
          console.log('Old Number ' + oldNo.toString());
          console.log('New Number ' + newNo.toString());
        }
      }
    })
  </script>
  </head>
  <body>
    <template id="app" is="dom-bind">
      <date-tester></date-tester>
    </template>
  </body>
</html>

If I am not mistaken this is by design.
The date property is an object and thus the observer will fire when you create a new instance (the equality check will return false), while numbers are primitives and the equality check is based on the value of the number.

I checked the developers guide to see what it said, and it didn't say that.  It says when the "value" of the property changes the Observer is fired.  That is why my test case deliberately created a new value

The "value" of the property actually has changed. Only the value of the "value" of the property is the same, but that isn't what Polymer checks.

When #3238 lands, you can make this work.

Yep, this is a dupe of #2719


Find My Mouse Activation

Description of the new feature / enhancement
I'd like to be able to set the activation keys for Find My Mouse as I can for the other mouse utilities.
Scenario when this would be used?
I'd like this so I can use a consistent base key (Apps/Menu in my case) for all 3 mouse functions.
Supporting information
No response

/dup https://github.com/microsoft/PowerToys/issues/14651


Difficulty URL parameter isn't working

Example map from readme:
https://supermedium.com/beatsaver-viewer/?id=811-535&difficulty=Easy
https://supermedium.com/beatsaver-viewer/?id=811-535&difficulty=Normal
https://supermedium.com/beatsaver-viewer/?id=811-535&difficulty=Hard
All links lead to Expert difficulty being selected.

Fixed, thanks!
https://github.com/supermedium/beatsaver-viewer/commit/f71705ec0d52dcffff255649bcdf00491c266202


If onChange passed to Field component returns truthy value, then component don't get field value updated

Are you submitting a bug report or a feature request?
bug report
What is the current behavior?
If onChange passed to Field component returns truthy value, then component don't get field value updated.
A component from the sandbox below (MyForm.js file)
<Field
  name="name"
  component={({ input: { value, onChange } }) => (

 onChange(Math.random())}>click me {value}

  )}
  placeholder="Name"
  onChange={() => true}
/>

If you'd click on "click me" text you'll not get a new random value. But if you'd replace onChange={() => true} by onChange={() => false} you'll get it functioning.
At my specific case onChange returned setTimeout value to get redux sagas notified after store is updated (setTimeout as you know returns timeout ID). It was really challenging to understand what's going on after I decided to finally update react-redux and redux-form.
What is the expected behavior?
Component updates shouldn't depend on what onChange returns.
Sandbox Link
All core dependencies got updated https://codesandbox.io/s/reduxform-template-87mjk
What's your environment?
No matter. The bug is reproduce-able on any environment.
Other information
Didn't find any information about this bug.

I'm also a victim of this bug, moreover the bug doesn't exist up to at least version 7.4.2. Is there any chance for a fix?

@finom Any solution you got?

@anupalhub nope, just make all functions not return anything: onChange={() => { onChange() }}


False unused/orphaned positive with polymorphic responses

components:
  schemas:
    ErrSomething:
      # ...
paths:
  /fish/paste:
    get:
      responses:
        default:
          content:
            application/json:
              schema:
                oneOf:
                  - $ref: "#/components/schemas/ErrSomething"
                  # ...
                discriminator:
                  # ...

Here, I get a warning
#/components/schemas/ErrSomething` is potentially unused or has been orphaned

This specific one seems to occur for polymorphic responses, i.e. components that are schema alternatives in a oneOf only in the doc, i.e. not as direct $refs below response schema.
I haven't checked if this happens similarly for polymorphic requests, but I guess it might.
https://github.com/OAI/OpenAPI-Specification/blob/3.1.0/versions/3.1.0.md#discriminator-object

This was a bug, essentially two of the three allOf, oneOf and anyOf polymorphic collections available in the index, was not being checked as part of the unused_definition function.
Thank you for your help in making vacuum better!