The experiment setting for BMG

Hi,
Thanks so much for sharing your reproduction. I just wanna know what is your experiment setting for BMG. I got quite different results from the ones you on readme. The BMG now is worse than the baseline A2C.
One potential error that we found right now is in bmg.py line 24.
We think the code should be loss, _, _ = self.controller.rollout(self.inner_model, self.outer_model)

Thanks for pointing out the error. I will revise the code. Please check if the problem still exists. (btw, my time currently would not allow me to maintain the code as it has been conducted for a long time, but once I have available time I will re-check)

Thanks for replying. But I am still curious about what hyperparameter setting you used to generate the results you showed on the README.

The hyperparameters we use are the same as the script as we provided in the README, and the results of both MGRL and BMG only run once. It worth mentioning that our result is based on the initial release of the torchopt, whereas it has made a lot changes by far, so there may be some performance discrepancy.

As no further issue appears at this time, I will close it.


Some extra vSphere objects, JSON issue

Addresses #35 and #3.

Hi Austin, thanks for the pull request. I already have a potential solution for issue #3 so I am probably not going to merge that part of your pull request. That part that covers issue #33 is certainly going to get merged but it is going to take a few days. I'd also like to write a test case for issue #33, like the others in https://github.com/mjuenema/python-terrascript/tree/develop/tests. Feel free to add one yourself if you like.
Can you also leave a note in the CHANGELOG.md about your fix to issue #33 so I don't forget!!!

I merged this into develop.


Implement curry

I think there are many occasions when bind is used with null first argument. Does it need implementation? And what file should it place in?

But that's not currying. The first argument of Function.prototype.bind sets the [[BoundThis]] internal slot, which is effectively passed as this value when calling the wrapped function.

@aqrln I meant occasions when bind is used not for that purpose but has more arguments and doesn't use this. Under these conditions first argument looks redundant.

@DzyubSpirit so do you think curry(fn.bind, null).call(fn, ...args) is in any way more concise or readable than fn.bind(null, ...args)? I feel like I miss the point and don't understand you right.

@aqrln Why does curry(fn, ...args) not work?

@aqrln I think @DzyubSpirit  proposes:
function curry(fn, ...args) {
  return fn.bind(null, ...args);
}

So you can use it like this:
const multiply = (x, y) => x * y;
const double = curry(multiply, 2);

Am I right, @DzyubSpirit ?
Yet I see no reason in doing so because common.curry(multiply, 2) doesn't look much better then multiply.bind(null, 2) IMHO

@nechaido, exactly

@nechaido, you forgot api., yeah? I think import path length is not critical. If you need several functions from common you probably already have had one of these statements:
const { ... } = app.common;
```javascript

const cmn = app.common;
With those qualified imports using common functions is really short.
I think common usages should be like that although sometimes `bind` is shorter


feat(material-experimental/mdc-radio): add test harness

Adds test harnesses for the MDC-based radio button and radio button group.

I talked to @mmalerba about this yesterday and the opinion was that we shouldn't be trying to reuse logic between the MDC and non-MDC components since it's mostly just calls into the internal harness APIs and the shared tests should ensure that the behavior is consistent.

Yeah, it's good that the shared tests ensure compatibility, though it seems like it would be still valuable to consider extending as that might help with code duplication. It seemed to work fine for the MDC-based dialog harness; and I can already see that there is quite a bit code duplicated in the radio harness.
Obviously there are cases where differences are expected, though that should not prevent us from sharing in other cases IMO. Most of the time it seems like only the locators would need to be updated. Since we ensure compatibility of these harnesses, we intend to keep logic in sync. Inheritance could make that easier, and also reduce code duplication.
To be clear: I'm perfectly fine with this change; I just wondered why we do not avoid the duplication.


Implement Optmized Failed Extrinsic and Nonce handling Mechanism.

Hm - where's the problem here? Could you pin some assosciated code or explain what needs improvement?


Code suggestion for async/dispose/setState no longer works in Flutter 3.0.2

I'm using flutter stable version 3.0.2 and dart 2.17.3. Every thing is ok as you see in below picture:

Code suggestion for async/initState/dispose/setState no works in android Studio.
below is the screenshots:

Please look into it and resolve it

In your doctor output, Android Studio is asking you to install the Flutter plugin. Please install the latest version of the plugin or update it and try again.

flutter and dart plugin is already updated as you see in screenshot

i try in vscode it works fine in vs code. But in Android Studio i still face this problem

This might be a regression. Try upgrading to 3.0.3 and see if it still persists. Anyhow, this issue is a duplicate of https://github.com/dart-lang/sdk/issues/49027. Please follow up there. I'm closing this issue since it's not a bug with Flutter.


Multifix

I know build still needs to be fixed. In theory, this should be everything, and is a compile test of all the commits from the mess of branches. If nobody sees anything wrong, I can try to figure out the best way to squash and merge any sub-parts. This is a giant PR.

I will go through, and re-cherry pick everything, I thought the point was to avoid having 100 branches.

@vladfrangu the different styles is because the commits are from many people and squashed.

@aakatz3 can this be closed?

@aakatz3 can this be closed?

Closing this as commits from here have been cherry picked into other pull requests.

Closing this as commits from here have been cherry picked into other pull requests.


All mxnet processes crash when lacking memory

When I used mxnet to train CNN, I found the memory(4*8G, 32 GB in total in my PC) it needed was much larger (about 10x) than GPU memory(Titan x, 12 GB in total).
For example, when I set batch size to 128, MXNet tended to allocate ~1GB GPU memory and 12 GB memory. If the actual memory does not meet such requirements, the following error would occur:

Furthermore, if one mxnet instance could not start when lacking memory, it would make all other mxnet processes crash too. But when I used caffe and no enough GPU memory (not memory) could be used for the caffe instance, other caffe processes would go on well.

This is because mxnet allocates memory during training. Using ccSGD instead of SGD might alleviate this problem.

Thanks for replying. I tried ccSGD but the memory MXNet allocated was still as high as SGD. Is it normal that MXNet tends to allocate such big memory during training as the screen shot shown below?

If you are training on big image dataset, it is possible for mxnet to allocate some memory for caching and there are multiple layers of caching and pre-fetching.
This is likely to be caused by ImageRecordIter and it was fixed a bit by latest version. Some configurations can be fine tuned to reduce the memory consumption.

You can try to set the following options to be smaller value (c.f. http://mxnet.readthedocs.org/en/latest/python/io.html#mxnet.io.ImageRecordIter)

prefetch_buffer
preprocess_threads

The URL gives 404

See here: http://mxnet-tqchen.readthedocs.io/en/latest/packages/python/io.html


[config] 'create_tarball_command:' vs. 'actions: create-archive:'

create_tarball_command: doc says: "a command which generates upstream tarball in the root of the upstream directory"
actions: create-archive: doc says: "when the archive needs to be created"
What's the difference (between tarball and archive)?
EDIT: Another point is that actions: create-archive: is supposed to be used in propose-update and srpm/copr-build, but looking at the code I see it only in API.create_srpm() not in API.sync_release()

so what's the work here?

The work here is to answer my question.
What's the difference between create_tarball_command and create-archive?

create_tarball_command: a command to use to create a tarball, instead of our git archive - this may be pretty tricky since people don't know what's the name of the archive meant to be
create-archive: an action which is suppose to replace more packit behaviour (and output the archive path). But when I look at the code right now, it doesn't make sense since both the action and the create_tarball_command are called - we should clean that up

we should clean that up

:+1:
The create_tarball_command was there before actions were supported.

What about moving this to packit repo if we want to deprecate that?
As @TomasTomecek wrote, they cannot be replaced as is. Since the behaviour differs a bit.

While updating the docs, I checked the Packit code and:

The workflow for creating an archive:

if there's defined the create-archive action, we run it, then go through the output, try to parse the archive path from it and possibly create a relative symlink to the archive from in the specfile directory
else we use the create_tarball_command; we construct the archive path as {package_name}-{version}.tar.gz and expect it to be created in the directory with the specfile and then run the command, which defaults to git archive -o "{specfile_dir}/{package_name}-{version}.tar.gz" --prefix "{package_name}-{version}/" HEAD, we don't check the output, just return the constructed archive path. So if I understand correctly we expect the create_tarball_command defined by user also to create archive {package_name}-{version}.tar.gz (but haven't found this mentioned in the docs)...

I would say this is confusing, IMHO it would make sense to have just create-archive  action and our git archive.. command as a fallback when no action defined.

EDIT: Another point is that actions: create-archive: is supposed to be used in propose-update and srpm/copr-build, but looking at the code I see it only in API.create_srpm() not in API.sync_release()

You're right, I will remove the create-archive from propose-update docs.

While updating the docs, I checked the Packit code and:

The workflow for creating an archive:

if there's defined the create-archive action, we run it, then go through the output, try to parse the archive path from it and possibly create a relative symlink to the archive from in the specfile directory
else we use the create_tarball_command; we construct the archive path as {package_name}-{version}.tar.gz and expect it to be created in the directory with the specfile and then run the command, which defaults to git archive -o "{specfile_dir}/{package_name}-{version}.tar.gz" --prefix "{package_name}-{version}/" HEAD, we don't check the output, just return the constructed archive path. So if I understand correctly we expect the create_tarball_command defined by user also to create archive {package_name}-{version}.tar.gz (but haven't found this mentioned in the docs)...

I would say this is confusing, IMHO it would make sense to have just create-archive  action and our git archive.. command as a fallback when no action defined.

EDIT: Another point is that actions: create-archive: is supposed to be used in propose-update and srpm/copr-build, but looking at the code I see it only in API.create_srpm() not in API.sync_release()

You're right, I will remove the create-archive from propose-update docs.

@lbarcziova agree. Thanks for looking into that. What about deprecating create_tarball_command then? (And removing a few versions later.)

@lbarcziova agree. Thanks for looking into that. What about deprecating create_tarball_command then? (And removing a few versions later.)

+1 for deprecating it, will create an issue for it...And now I found out there is a similar situation with the get-current-version action and current_version_command, can we do it as well (leaving just the action)?

+1 for deprecating it, will create an issue for it...And now I found out there is a similar situation with the get-current-version action and current_version_command, can we do it as well (leaving just the action)?

Still valid.

Isn't this fixed by deprecating create_tarball_command?

I think so, yes, thanks.


HTTP::Server#listen_fork

On Crystal 0.7, given
def fib(n)
  if n == 0
    return 0
  elsif n == 1
    return 1
  else
    return fib(n-1) + fib(n-2)
  end
end

server = HTTP::Server.new 8080, do |request|
  i = request.path[1..-1].to_i
  HTTP::Response.ok "text/plain", fib(i).to_s
end

puts "Listening on http://<IP_ADDRESS>:8080"
server.listen_fork

we get
Listening on http://<IP_ADDRESS>:8080
[warn] kevent: Bad file descriptor
[PCL] Stale coroutine called: curr=0x10834a000  exitto=0x7f9e92801200  caller=0x7f9e92801200
[warn] kevent: Bad file descriptor
[PCL] Stale coroutine called: curr=0x10834a000  exitto=0x7f9e92801200  caller=0x7f9e92801200
[warn] kevent: Bad file descriptor
[PCL] Stale coroutine called: curr=0x10834a000  exitto=0x7f9e92801200  caller=0x7f9e92801200
[warn] kevent: Bad file descriptor
[PCL] Stale coroutine called: curr=0x10834a000  exitto=0x7f9e92801200  caller=0x7f9e92801200
[warn] kevent: Bad file descriptor
[PCL] Stale coroutine called: curr=0x10834a000  exitto=0x7f9e92801200  caller=0x7f9e92801200
[warn] kevent: Bad file descriptor
[PCL] Stale coroutine called: curr=0x10834a000  exitto=0x7f9e92801200  caller=0x7f9e92801200
Ready
[warn] kevent: Bad file descriptor
[PCL] Stale coroutine called: curr=0x10834a000  exitto=0x7f9e92801200  caller=0x7f9e92801200
[warn] kevent: Bad file descriptor
[PCL] Stale coroutine called: curr=0x10834a000  exitto=0x7f9e92801200  caller=0x7f9e92801200

and it does not serve requests.
Even setting the number of workers to 1 server.listen_fork(1) is no help
Listening on http://<IP_ADDRESS>:8080
Ready
[warn] kevent: Bad file descriptor
[PCL] Stale coroutine called: curr=0x10f0f3000  exitto=0x7ff333801200  caller=0x7ff333801200

Instead of fork, we also tried to use threads in a similar way to the listen_fork:
class HTTP::Server
  def listen_thread(workers = 8)
    server = TCPServer.new(@port)
    workers.times do
      t = Thread.new do
        puts "starting worker"
          loop {
            puts "before handle_client"
            spawn handle_client(server.accept)
            puts "after handle_client"
          }
      end
    end

    puts "Ready"
    gets
  end
end

which gives
Listening on http://<IP_ADDRESS>:8080
starting worker
before handle_client
starting worker
before handle_client
starting worker
before handle_client
starting worker
before handle_client
starting worker
before handle_client
starting worker
before handle_client
starting worker
before handle_client
starting worker
before handle_client
Ready

By adding a call to t.join we can see a stacktrace
Listening on http://<IP_ADDRESS>:8080
starting worker
before handle_client
Could not get the current fiber (Exception)
*Exception#initialize<Exception, String, Nil>:Array(String) +21 [4352266693]
*Exception::new<String>:Exception +111 [4352266639]
*raise<String>:NoReturn +6 [4352237734]
*Fiber::current:Fiber +59 [4352269371]
*TCPServer#accept<TCPServer>:TCPSocket +145 [4352290065]
~fun_literal_12 +125 [4352240557]
~fun_literal_11 +49 [4352240417]
*Thread(Nil, NoReturn)@Thread(T, R)#start<Thread(Nil, NoReturn)>:Exception+ +100 [4352292084]
~fun_literal_9 +6 [4352240294]
GC_inner_start_routine +117 [4352690741]
GC_call_with_stack_base +59 [4352671835]
GC_start_routine +31 [4352697887]
_pthread_body +138 [140735574333593]
_pthread_struct_init +0 [140735574333226]
Program terminated abnormally with error code: 256

We also poked around by adding a fiber manually in the thread, but that did not help.
cc @fdr who looked into this with me earlier today

I think we'll remove that method now that #listen spawns and doesn't block. Can you use #listen instead?

I don't think that can use multiple threads.  @will and I were interested in trying out Crystal using multiple processors and seeing how that went.

Confirmed with Prax: it seems we can't spawn threads and have IO calls inside them anymore.
I guess we are in-between 2 solutions where the long term goal of Crystal is to have a multithreaded event loop (hence the choice of libevent over libuv), which is now limited to a single thread.

@waj I'm porting Prax to use the event loop (it works great). Yet, I'd like to keep some operations in their own thread, in order to not affect the main loop with something unrelated. For instance the monitor that kills unused spawned apps periodically, but I can't since just calling sleep will crash the thread (same error as above).
I can get rid of the thread and spawn a coroutine instead, but what if the operation was very CPU-bound and I really wanted to have it running on it's own to avoid blocking the loop?

So would the end goal here be for #listen to use multiple cores?
Really interested to see how that would look performance wise. Is there any way to getting something like that to work right now?
(I'm not knowledgeable enough to understand if this was already answered by @ysbaddaden mentioning the "event loop" above)

@dbackeus it seems that the end goal is to have a "Fiber pool" where a few Threads will handle, which will means you have Thread safety + Multi-core operations.

It looks like calling IO within a fork is also broken. I'm more annoyed by this problem than the lack of threads (no global process forking childs for never downtime).
I'm also having problems in signal handlers. They can happen when any thread (that crystal ran, like GC) is running, so handlers may fail randomly, and we can't use use a channel to postpone signal handling, since we can't call IO within a signal handler safely...

I think I'll just remove this method. I added it way before we think about what concurrency model was desired in Crystal.

@waj :+1:
I'm also thinking maybe fork is out of place in Crystal. I know Ruby tends to be very close to unix: errno, fork, etc., but most popular programming languages (for example: Java, C#, Go) tend to abstract that away, which I think makes sense. Programmers shouldn't have to deal with all those complexities.

in Java/Go/Objective-C runtime I don't think their implementations allow for fork, definitely true for JVM/ObjC runtime. why not remove when it's a problem? :)

Could fork create a new event loop? We obviously can't access it once we forked, but I guess we can create a new loop, can't we?

I didn't any research on the problem yet, but this seems to be related: http://www.wangafu.net/~nickm/libevent-book/Ref2_eventbase.html#_reinitializing_an_event_base_after_fork

I just did a quick fix to solve this issue. Still, I think the listen_fork method should go away. Specially once the scheduler supports multiple threads.

@ysbaddaden Still is not possible to create new threads and use IO inside them. So try to use spawn everywhere. Even though the performance will be far from ideal, eventually once we improve the scheduler, everything will start to perform better.
Although creating custom threads should be possible, the scheduler should be smart enough to continue processing IO on other threads while one is blocked in some CPU intensive task.

I'm not ranting on IO inside threads. This may either be fixed by multithreaded loop (I'm not sure) or not required anymore since there would be a better interface :)
I'm glad to see forkfixed! and yes, listen_fork should be removed. It's not monitoring child processes for instance, and it doesn't fit into the stdlib.

I'm just wondering about signals. How to react correctly to a SIGHUP or SIGTERM when they may happen anytime, like outside the main thread, which can't do any IO?
Maybe I should open another issue about this.

Yes please, open another issue and describe what's the problem with the signals, including an example if possible. I'm not sure to understand why signal handling might be problematic.


Config Contexts from Git Sync Should all Be Read Only

As ...
Austin - Network Automation Engineer
I want ...
To have all config context data that is synchronized via Git should be read only and that Git is the source and owner.
So that ...
Git controls all things when it is a Git sync.
I know this is done when...

I sync a Git repo that the data is not editable.

Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[X] Misc (including Data Sources)
[ ] Organization
[ ] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[ ] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
No response
External Dependencies
No response

Related: #1322, #3963


router: reduce per-packet allocations

Avoid allocating scionPacketProcessor; re-organized packet processing
functions. The scionPacketProcessor is now the main entry point for
packet processing and contains all the pre-allocated per-packet state.
Avoid allocation of message buffer for writing with WriteBatch.
We could just pre-allocate a buffer, but as we always write a batch of
size one, just use WriteTo instead.
For writing single packets, this is both more convenient and faster.
Additional benefit: Read/WriteBatch make rather large allocations
internally (see golang/go#26838) which is
avoided by using WriteTo.
The largest remaining allocations when processing SCION packets are:

ReadBatch allocates temporary storage internally (as mentioned above).
This is by far the biggest culprit, it accounts for almost 80% (after
the changes in this patch).
slayers/path.NewPath called during slayers.SCION.DecodeFromBytes:
Could be avoided by reusing Path objects, but requires larger changes.
slayers/path/scion.Raw.GetCurrentHopField and GetCurrentInfoField, called during parsePath:
Both return pointer types. Should be possible to change this to value
types instead, but requires larger changes.
DataPlane.resolveLocalDst: allocates IP and UDPAddr. Can only really
be fixed with better address types.
slayers/path.FullMAC. Allocates both input and output. The function
interface could be changed to allow reuse of the input buffer.
The output buffer cannot be reused with this CMAC library.

Depends on (or actually includes) #4029.

This change is

/rebase


Unchanged files with check annotations (beta)

Describe a bug
I'm unable to prevent coverage annotation on unchanged files. I've setup the action with annotations: coverage and expected to have annotations to changed files only, as stated in the docs:

coverage - Will annotate those sections of your code that test did not cover. Limited to changed lines when used on a Pull Request

Even so, GH is adding a entire section on the diff entitled "Unchanged files with check annotations (beta)" with a bunch of annotations.
As we are starting a long effort of covering a huge legacy application with tests, this is becoming an issue due to the extensive number of files without coverage.
I see this should be addressed already, per this  closed issue, but it's not working for me.
Expected behavior
To have coverage annotations on changed files only (changed lines if possible).
Details

Action version: v2.2.4

OS, where your action is running (windows, linux): linux (ubuntu)

action.yml file
```yml
name: Build and Test

on:
  workflow_call:
    secrets: ...
    ...

 jobs:
   build-and-test:
   ...
     - name: Test
        id: test
        run:  npm run test:coverage:ci
        # jest --verbose --ci --coverage --json --testLocationInResults --outputFile=report.json

      - name: Find Current Pull Request
        id: findPr
        uses<EMAIL_ADDRESS>
      - name: Report Test Coverage
        uses<EMAIL_ADDRESS>        with:
          prnumber: ${{ steps.findPr.outputs.number }}
          annotations: coverage
          coverage-file: report.json
          base-coverage-file: report.json
```

Screenshots

Additional context

Hello @VictorComette, can you please provide minimal reproducible example of this issue? If not, maybe you can email your report.json file as well as PR diff to<EMAIL_ADDRESS>

I'm seeing this issue using the recommended default setup for this action on v2.3


Update documentation. Update add OpenCV MIL tracker tool

Related #4200
Motivation and context

This PR update OpenCV tools and AI tools sections.
preview:
https://tosmanov.github.io/cvat/docs/manual/advanced/opencv-tools/
https://tosmanov.github.io/cvat/docs/manual/advanced/ai-tools/
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added a description of my changes into CHANGELOG file
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2022 Intel Corporation
#
# SPDX-License-Identifier: MIT

@klakhov
Could you please approve or comment?


feat: Central Hierarchy + Environment design populated by Actions

The design is based on two central classes:

Hierarchywith two attributes _configurations and _templates https://github.com/NotANameServer/incipyt/blob/dc3fddef98fb89dc1da5b1633f171792e311b4a2/incipyt/system.py#L127-L143. They are dictionaries with instances of classes in incipyt/_internal/dumpers -- path-like classes with a specific  method to write them, e.g. for Toml https://github.com/NotANameServer/incipyt/blob/dc3fddef98fb89dc1da5b1633f171792e311b4a2/incipyt/_internal/dumpers.py#L71-L74 -- as keys. Each association key-value corresponds to a file path and its content to be written.  For _templates,  values are just Jinja template. _configurations is dedicated to all configuration files based on  key-value syntax (toml, cfg, ini, yaml, json, etc), values of this dictionary are then nested dictionary describing this configuration file.
Envrionment is basically a wrapper around a dictionary initiated with environment variables, https://github.com/NotANameServer/incipyt/blob/dc3fddef98fb89dc1da5b1633f171792e311b4a2/incipyt/system.py#L13-L65 its purpose is to deal with all variables, like {NAME} or {AUTHOR} that should be asked to the user. To do so, https://github.com/NotANameServer/incipyt/blob/dc3fddef98fb89dc1da5b1633f171792e311b4a2/incipyt/system.py#L67-L74 have to be implemented and ask the user for a specific variable. Then it behaves as a visitor for values of Hierarchy._configuration https://github.com/NotANameServer/incipyt/blob/dc3fddef98fb89dc1da5b1633f171792e311b4a2/incipyt/system.py#L97-L108 actually values can contain some callable instead of str, if so those callable are call and the value substituted. Typically, it can be utils.Requires("{NAME}") with https://github.com/NotANameServer/incipyt/blob/dc3fddef98fb89dc1da5b1633f171792e311b4a2/incipyt/_internal/utils.py#L12-L28 then it asks Environment for {NAME} : environment.pull(key).

There are similar substitution mechanisms for _template and also for using subprocess.run https://github.com/NotANameServer/incipyt/blob/dc3fddef98fb89dc1da5b1633f171792e311b4a2/incipyt/system.py#L110-L124
The main workflow is then to initialize a Hierarchy and let tool specific classes populate _configurations and _template https://github.com/NotANameServer/incipyt/blob/dc3fddef98fb89dc1da5b1633f171792e311b4a2/incipyt/system.py#L242-L245

An instance of Hierarchy stores all information about configurations files and templated files to write for the new project.
Then each tool is modeled by Action instances, they populate a hierarchy instance and define pre and post methods to run before and after the creation of the project files https://github.com/NotANameServer/incipyt/blob/dc3fddef98fb89dc1da5b1633f171792e311b4a2/incipyt/system.py#L250-L259. E.g. Git Action pre method runs git init, while post method runs git add --all and git commit.
Definitions of configurations and templates can be done with environment variables, like {NAME}. All those variables are stored in an Environment instance, it is initialized with environment variables system and can be modified by any Action using pre or post methods. Then values are used when needed and asked to user if missing.
When an Action is created, it can also register itself a hook, e.g.  Git Action register a VCSIgnore hook that defines how a pattern is ignored by git. Then any Action can used those hooks, e.g. Venv Action use VCSIgnore to make VCS system aware of ignoring .env folder.

Note about ECS: I did some tests where configurations_ and _templates are replaced by _components. i.e. each file to be written is a component of an single entity.
Finally, imo, it's not really useful as each component need its own system, so the decoupling is mainly artificial.

If you want to test
import logging

logging.basicConfig(level="INFO")

import subprocess

import incipyt

env = incipyt.Environment()
env.push("NAME", "its-a_test")

incipyt.process_actions(
    "./test_project",
    env,
    [incipyt.actions.Git(), incipyt.actions.Venv(), incipyt.actions.Setuptools()],
)


Collections : data tab

When clicking on a collection, there is a "Data" tab on the right side of the screen. This should show a default view of the data returned by the API.

This Data tab should allow editing of the data in the selected collection, simple CRUD interface.
In order for this feature to allow management of all data in the database, the UI will have to bypass default endpoint access restrictions, which can be done by specifying the secret parameter (either as a GET parameter or in the request body) and supplying the server`s secret as its value. Whenever the UI connects to a server using a secret, the secret is held in the window.localStorage.

Is this something like phpmyadmin, where you click on a table name and you can view/edit/delete data and remove table?

I would image it much simpler, simple data editing. When you are on this tab, you already have a table/collection selected. One should be able to list rows, edit values of cells, add/remove rows.
Like data editing in Deployd.


Is this gonna be similar to the other mailinabox?

The other mail in a box
https://mailinabox.email/

This will be a caddy plugin that allows caddy to interact with the APIs provided by that project.


Add action to set kubernetes size labels

What this PR does / why we need it:
This PR introduces a Github Action to set a label based on the size of the PR. This helps to keep an eye on the PR size to make the work of reviewers easier.

Fixes #906

Please reopen if you plan to push this further :)


⚠ Bofrost.lu has degraded performance

In c0ae55e, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 887 ms

Resolved: Bofrost.lu performance has improved in be6bb35 after 9 minutes.


Excluding Tables

Is it possible to exclude certain tables from being included in the dump?

Not possible in the current release.

Hi @nsaud01 ! Thank you for your suggestion. I keep your issue open since it could be considered an interesting feature for a future release.

This is now possible thanks to https://github.com/rap2hpoutre/pg-anonymizer/pull/26
Just append -T table_name to exclude the table from the dump.

How do I exclude multiple tables? I tried separating them with a comma, but it threw a
pg_dump command error: pg_dump:

How do I exclude multiple tables? I tried separating them with a comma, but it threw a pg_dump command error: pg_dump:

You can send multiple arguments: pg_dump [db] -T [table1] -T [table2]


fix: eth-p/bat-extras no longer removes the first '.' from the version string in the release zip file.

Cool, thanks!


result set clean up + fix mssql edge case

Describe your changes

cleaning up ResultSet
fixes an edge case when using SQL server
updates compatibility table
adds missing integration tests for bar and pie plots

Issue number
Closes #X
Checklist before requesting a review

[ ] Performed a self-review of my code
[ ] Formatted my code with pkgmt format
[ ] Added tests (when necessary).
[ ] Added docstring documentation and update the changelog (when needed)

:books: Documentation preview :books:: https://jupysql--781.org.readthedocs.build/en/781/

Running plots on taxi dataset on datetime field is a bit slow (tried on MSSQL) :
%sqlplot bar --table taxi --column passenger_count

This runs in 555ms:
CPU times: user 98 ms, sys: 38.1 ms, total: 136 ms
Wall time: 555 ms

Whereas on the tpep_pickup_datetime it takes 37.3secs:

Piechart is stuck even after running for a minute:

Should I open another issue for this? @edublancas

@neelasha23 yes, please open an issue. I think the problem is that histogram requires counting unique values and since those are timestamp, they're probably all unique. I don't think there's much to do here. perhaps we can show a warning depending on the data type (i.e., if the column they pass is a timestamp, we show the warning)


XFA - missing content on Ontario Lease

Attach (recommended) or Link to PDF file here:
http://www.forms.ssb.gov.on.ca/mbs/ssb/forms/ssbforms.nsf/GetFileAttach/047-2229E~1/$File/2229E.pdf
Configuration:

Web browser and its version: FF 91 (2021-06-21) with XFA enabled
Operating system and its version: macOS
PDF.js version: 2.10.146
Is a browser extension: Yes

Steps to reproduce the problem:

Load the PDF and compare to Adobe Reader or Chrome Canary with XFA turned on. (Chrome has lots of blank pages but seems to draw more stuff)

2229E.pdf

This looks like a dynamic form.

Yes it's a dynamic form but likely something is wrong in the layout: I'll have a look on it.

This PDF seems to draw correctly now. Most of the form controls are unusable though.

This seems to work well enough now. Certainly much better than Chrome Canary

Let's keep this open for the JS parts.

What's the status of the JS stuff?

For now: there are no plan to implement JS stuff but it's something we likely consider in the future.
From my pov, it will be a bit challenging because thanks JS, layout will become dynamic (it's possible to add pages, remove pages, add some text in a place which will move a lot of elements on the next page....).
So compared to what we did for classic Acroforms, my guess is that it'll be "slightly" harder.


Ability to ping from hosts list

Feature description
When on a server list (ssh, sftp, tunnel) and doing a contextual menu : ability to launch ping on the host
References

https://github.com/isontheline/pro.webssh.net/issues/874
https://github.com/isontheline/pro.webssh.net/discussions/873

Now available inside the latest beta release : https://testflight.apple.com/join/QSrBK59z


very poor performance

Hi, everyone!
I can run flownet2.0 now. However, the flownet2.0 performs very poorly and I can't get the same flow image in readme. What's more, the output image is random with same input images.
My env is CUDA 10 locally, and cudatoolkit 9.0, cudnn 7.3.1, and tensorflow-gpu 1.12 in conda env.
The log is as following:
(flownet2.0-gpu-v2) chenyang@chenyang-Inspiron-15-7000-Gaming:~/Downloads/flownet2-tf-master$ python -m src.flownet2.test --input_a data/samples/0img0.ppm --input_b data/samples/0img1.ppm --out ./
WARNING:tensorflow:From src/net.py:22: get_or_create_global_step (from tensorflow.contrib.framework.python.ops.variables) is deprecated and will be removed in a future version.
Instructions for updating:
Please switch to tf.train.get_or_create_global_step
WARNING:tensorflow:From src/flownet_cs/flownet_cs.py:26: calling reduce_sum (from tensorflow.python.ops.math_ops) with keep_dims is deprecated and will be removed in a future version.
Instructions for updating:
keep_dims is deprecated, use keepdims instead
2019-02-28 19:30:19.330567: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 FMA
2019-02-28 19:30:19.405113: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:964] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2019-02-28 19:30:19.405529: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1432] Found device 0 with properties:
name: GeForce GTX 1050 Ti major: 6 minor: 1 memoryClockRate(GHz): 1.62
pciBusID: 0000:01:00.0
totalMemory: 3.95GiB freeMemory: 3.64GiB
2019-02-28 19:30:19.405543: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1511] Adding visible gpu devices: 0
2019-02-28 19:30:19.629411: I tensorflow/core/common_runtime/gpu/gpu_device.cc:982] Device interconnect StreamExecutor with strength 1 edge matrix:
2019-02-28 19:30:19.629443: I tensorflow/core/common_runtime/gpu/gpu_device.cc:988] 0
2019-02-28 19:30:19.629449: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1001] 0: N
2019-02-28 19:30:19.629649: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1115] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 3364 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1050 Ti, pci bus id: 0000:01:00.0, compute capability: 6.1)
2019-02-28 19:30:24.964743: W tensorflow/core/common_runtime/bfc_allocator.cc:211] Allocator (GPU_0_bfc) ran out of memory trying to allocate 2.58GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory were available.
max flow: 153.2417
flow range:
u = -40.339 .. 22.779
v = -1.128 .. 152.825
(flownet2.0-gpu-v2) chenyang@chenyang-Inspiron-15-7000-Gaming:~/Downloads/flownet2-tf-master$ python -m src.flownet2.test --input_a data/samples/0img0.ppm --input_b data/samples/0img1.ppm --out ./
WARNING:tensorflow:From src/net.py:22: get_or_create_global_step (from tensorflow.contrib.framework.python.ops.variables) is deprecated and will be removed in a future version.
Instructions for updating:
Please switch to tf.train.get_or_create_global_step
WARNING:tensorflow:From src/flownet_cs/flownet_cs.py:26: calling reduce_sum (from tensorflow.python.ops.math_ops) with keep_dims is deprecated and will be removed in a future version.
Instructions for updating:
keep_dims is deprecated, use keepdims instead
2019-02-28 19:30:30.091185: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 FMA
2019-02-28 19:30:30.167325: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:964] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2019-02-28 19:30:30.167742: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1432] Found device 0 with properties:
name: GeForce GTX 1050 Ti major: 6 minor: 1 memoryClockRate(GHz): 1.62
pciBusID: 0000:01:00.0
totalMemory: 3.95GiB freeMemory: 3.64GiB
2019-02-28 19:30:30.167757: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1511] Adding visible gpu devices: 0
2019-02-28 19:30:30.397008: I tensorflow/core/common_runtime/gpu/gpu_device.cc:982] Device interconnect StreamExecutor with strength 1 edge matrix:
2019-02-28 19:30:30.397042: I tensorflow/core/common_runtime/gpu/gpu_device.cc:988] 0
2019-02-28 19:30:30.397065: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1001] 0: N
2019-02-28 19:30:30.397239: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1115] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 3364 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1050 Ti, pci bus id: 0000:01:00.0, compute capability: 6.1)
2019-02-28 19:30:35.757919: W tensorflow/core/common_runtime/bfc_allocator.cc:211] Allocator (GPU_0_bfc) ran out of memory trying to allocate 2.58GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory were available.
max flow: 37.8530
flow range:
u = -23.250 .. 24.644
v = -32.727 .. 33.248
You can see that it has two totally different results of 'max flow' and 'flow range' with same inputs.
I have set the hardcode as below, but this does not work for me.
GPUCFLAGS = -c -gencode=arch=compute_30,code=sm_30
Anyone can help?
Thanks.

arch=compute_30,code=sm_30 means you are targeting a NVIDIA Kepler devide (e.g. GTX770).

There was a warning:
W tensorflow/core/common_runtime/bfc_allocator.cc:211] Allocator (GPU_0_bfc) ran out of memory trying to allocate 2.58GiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory were available.
This may be related to poor memory.
Another guess: may be related to the OPs is not compiled correctly.

I can tell why your result was random, but:
arch=compute_30,code=sm_30 means you are targeting a NVIDIA Kepler devide (e.g. GTX 770), while you are using a GTX 1050 Ti.

Actually, I have tried with my Video Card's compute capability 6.1, and set 'arch=compute_61, code=sm_61',  but the output is still random.

solved by reinstall cuda 9.0, cudnn v7.2.1 locally.
Thanks everyone who give me help!


Wrong commands in this document

[Enter feedback here]
Please remove the section "Restore 'testserver2' to 'testserverNew', where 'testserverNew' is in a different resource group from 'testserver2'." and the command line below too because it is not supported in flexible server.
az postgres flexible-server restore --resource-group testGroup --name testserverNew
--source-server "/subscriptions/${SubID}/resourceGroups/${ResourceGroup}/providers/Microsoft.DBforPostgreSQL/servers/testserver2" \ --restore-time "2017-06-15T13:10:00Z"

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

ID: 065fdfa8-5ed2-e032-6ea7-99a1f5dc28b6
Version Independent ID: 71881d7f-c4b3-ce3c-1a1a-6c17dc533852
Content: az postgres flexible-server
Content Source: latest/docs-ref-autogen/postgres/flexible-server.yml
Service: postgresql
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to CXP team

@HamzaAqel Thank you for reaching out, we are looking into it.


Small fix in closure

A small fix forgotten in https://github.com/mirage/irmin/pull/920

Thanks. You know what I'm going to ask you now :wink: Is it possible to write a test that exposes the bug?

Thanks. You know what I'm going to ask you now 😉 Is it possible to write a test that exposes the bug?

Added the test, it fails without this fix.

Nice test!


Update signalr-blazor.md

There seems to be a missing code block at step 4

Thanks for pointing this out, @d4shm1r. That step shows the code below, which includes the line in question, along with the other line. The step is messy 🤮. I see what I'd like to do to fix up the whole step: I'm going to avoid individual code lines, leave the bullets, and highlight those two lines in the existing code block. I'll do that across all versions of the doc. I'll take care of this first thing Tuesday morning. Thanks again for pointing it out.
Moved to 👉 https://github.com/dotnet/AspNetCore.Docs/issues/27052


First time you create a shortcut, it will get stuck even on keyUp

Turns out it happens when you have the SwiftShift window open, as the global monitor doesn't trigger (should be local)


Created a blastoff program in cpp

@yuvimalhotra28 thanks for contributing.
Can you provide a brief description of the algorithm and add the same as comment in the file.


Remove pauli_group

In December 2020 (https://github.com/Qiskit/qiskit-terra/pull/5261) the function pauli_group was deprecated. Time to remove it.

Thanks. But sorry. Duplicated with https://github.com/Qiskit/qiskit-terra/pull/8070.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
661 unchanged lines in 26 files lost coverage.
Overall coverage decreased (-1.0%) to 83.384%

Files with Coverage Reduction
New Missed Lines
%

qiskit/algorithms/minimum_eigen_solvers/vqe.py
1
93.33%

qiskit/opflow/list_ops/list_op.py
1
87.18%

qiskit/quantum_info/operators/symplectic/clifford_circuits.py
1
92.79%

qiskit/tools/parallel.py
1
75.81%

qiskit/version.py
1
77.88%

qiskit/quantum_info/operators/measures.py
2
61.72%

qiskit/result/postprocess.py
2
73.56%

qiskit/utils/arithmetic.py
2
75.32%

qiskit/init.py
3
73.53%

qiskit/opflow/expectations/expectation_factory.py
3
81.08%

Totals

Change from base Build<PHONE_NUMBER>:
-1.0%

Covered Lines:
53873

Relevant Lines:
64608

💛  - Coveralls

Closing in favor of https://github.com/Qiskit/qiskit-terra/pull/8070.


New function: Invoke-DbaDbIndexRebuild

Summary of new feature
Create a new function Invoke-DbaDbIndexRebuild which can rebuild one or more indexes on a collection of tables. Should allow for passing collections of tables, databases, index names and require -AllDatabases if no databases are passed. Should also allow for various index rebuild options like online rebuilds, etc.
Allow for reorgs as well.
This may eventually be called from Invoke-DbaDatabaseUpgrade (optionally) as index rebuilds are needed for some upgrades.
Proposed technical details (if applicable)
Brute force method:
get-dbadatabase -serverinstance YOURSERVER | foreach-object {$PSItem.Tables.Indexes|ForEach-Object {$PSItem.OnlineIndexOperation = $true; $PSItem.Rebuild()};}
Latest version of dbatools as of writing
0.9.382

@jpomfret and I were talking about this a few weeks ago - would be cool to see Ola's maintenance recreated as a PowerShell command instead of stored proc!

I was thinking a full recreation, not just invoking Ola's scripts. Tons of work, probably not much benefit other than not having to have Ola's maintenance installed everywhere.

at the very worst you can install, run, uninstall. But I really don't see the benefit of reinventing the wheel given there are no shortcomings on both ola's and tiger's tsql routines....

If I could suggest, please allow for rebuild of heaps as well as Ola's scripts do not support heaps.


Move contributing.md

Fixes
Fixes #2008 by @AetherUnbound
Description
For a good understanding of the task please read the description here https://github.com/WordPress/openverse/issues/2008.
Developer Certificate of Origin
Signed by Shivam Ganwani

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@WordPress/openverse-maintainers in the latest workflow, lint has failed, how to fix it?

@shivamganwani to make the Lint checks pass, you need to fix the problems it has found. Luckily lint is also capable of fixing most problems, by running lint locally. Inside the root of the repository, run
$ just lint

This should update the files with the required changes. You might face problems in linting because the workflow needs Docker installed and Node.js to be set up. If you face difficulties, let us know and we'll fix the lint problems from our side.

Thanks @dhruvkb for reviewing, please fix the lint issues for now, I am parallelly trying to do the setup too (for next time)

@dhruvkb addressed your comments, but you'll have to fix lint again I guess

No issues @dhruvkb, I also missed a few breaking links.
Guess you'll have to fix lint again

@dhruvkb what about this -> (https://docs.openverse.org/general/contributing.html). in CONTRIBUTING.md. Should we change to markdown?

No @shivamganwani, since that file is not inside the documentation project (that only includes content from the documentation/ directory), the rule for using Markdown instead of HTML does not apply to it.
The CONTRIBUTING.md file is targeted for readers using the GitHub UI so it should point to the full URL of the docs (with .html).

@AetherUnbound looks good?


Enable external DNS servers/search domains to be provided by network drivers

Proposal
Simplify the configuration of External DNS servers for to work in co-existence with the embedded DNS server offered in docker 1.10.
We propose offering the DNS options to the container config during container creation, as offered by DHCP Server. The proposed changes do not offer to disable embedded DNS as mentioned in Issue https://github.com/docker/docker/issues/19474 ([1.10] How to disable embedded DNS?), but improve the offering of basic pluggable DNS servers by --dns option.
If the remote network plugin does not offer the DNS options there is no change in behavior and embedded DNS would continue to provide the name resolution for the containers in the network. Hence, this is backward-compatible with the existing solution.
Details
The remote network drivers can pass any external DNS server(s) and DNS search domain(s) that are configured for that network and pass this information when container address is allocated (Similar to DNS options provided by DHCP). These options are configured as external DNS servers.
Also, if the /etc/resolv.conf is modified, the modified changes take precedence over the configured DNS which is in line with the current behavior.
The precedence for answering DNS queries:

Embedded DNS
Configured external DNS servers

Modifications to /etc/resolv.conf takes precedence
DNS servers configured at container level in the order they are provided

provided by --dns or by remote ipam driver. The --dns provided server(s) overrides the remote ipam driver server(s).

Tested scenarios
Scenario 1:
Embedded DNS coexisting with remote ipam provided DNS Server (<IP_ADDRESS>).
Container c1,c2 attached to epg.nw1.tenant1 [Query: "ping c1" from c2]
Mar  2 22:47:13 vagrant docker[2025]: level=info msg="Querying embedded dns udp for c1.[1]"
Scenario 2:
Embedded DNS coexisting with remote ipam provided DNS Server (<IP_ADDRESS>).
Container c1,c2 attached to epg.nw1.tenant1 [Query: "dig tenant1 SRV" from c1]
Mar  2 22:51:16 vagrant docker[2025]: level=info msg="Querying ext dns udp:<IP_ADDRESS> for tenant1.[33]"
Scenario 3:
Embedded DNS coexisting with remote ipam provided DNS Server (<IP_ADDRESS>). Container c3 is launched with --dns=<IP_ADDRESS> option.
Container c1,c2,c3 attached to epg.nw1.tenant1 [Query: "dig tenant1 SRV" from c3]
Mar  2 22:49:10 vagrant docker[2025]: level=info msg="Querying ext dns udp:<IP_ADDRESS> for tenant1.[33]"
Mar  2 22:49:12 vagrant docker[2025]: level=error msg="external resolution failed, read udp <IP_ADDRESS>:53840-><IP_ADDRESS>:53: i/o timeout"
Mar  2 22:49:12 vagrant docker[2025]: level=info msg="Querying ext dns udp:<IP_ADDRESS> for tenant1.[33]"
Note
These changes were tested with the fix in PR #895 (Source external DNS queries from container namespace)
Automating these test scenarios is a work-in-progress. Will send out a PR for it as well.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "dnsaddr_handling" git@github.com:DivyaVavili/libnetwork.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

/cc @mavenugo @sanimej @jainvipin @shaleman

@DivyaVavili thanks for the proposal. 1 question on the design though.
Is there a reason why we would want to have a different DNS and DNS Search-domain for every RequestAddress in a network / Pool ? Isn't it enough we return this in ReleasePool which will be called during Network creation events and all the endpoints in that network can share the same DNS and Search-domains ?
I understand that this is addressing one of a popular IPAM use-cases (using DHCP). Since we dont have the corresponding IPAM driver, it is hard for me to put the full picture together.

@mavenugo - modeling this PR to what DHCP would return a dhcp response. Though not likely, say we implement a DHCP ipam driver, a mismatch (different search domains for two containers in the same subnet) would create an issue. This also inline with the granularity of dns address/search domain is per container level today i.e. it can be specified differently at docker run differently for different containers.

Is there a reason why we would want to have a different DNS and DNS Search-domain for every RequestAddress in a network / Pool ? Isn't it enough we return this in ReleasePool which will be called during Network creation events and all the endpoints in that network can share the same DNS and Search-domains ?

@jainvipin fair enough. it would be great if we also have a DHCP driver :). but that is unrelated to this PR.

@mavenugo
yes a separate PR, coming...

it would be great if we also have a DHCP driver :). but that is unrelated to this PR.

@jainvipin @DivyaVavili
With the proposed changes, how will the IPAM driver determine what DNS server/options to give to a particular container?

@sanimej
As of today, the IPAM driver knows the subnet and pool-id i.e. subnet is rough indicator of the network (not docker network, but subnet itself is some indication of a local-area-network) and pool-id would approximately identify the tenancy of the container. For Contiv we plan to use those two.
It would be wonderful to allow passing more hints (aka labels equivalent) that can be configured in commercial IPAM software, so this can be automated between orchestrators and IPAM stacks. To be able to implement these functions, without breaking architectural boundaries is the trick. Any suggestions?

With the proposed changes, how will the IPAM driver determine what DNS server/options to give to a particular container?

I am :-1: on this design. We should not try to inject unrelated options (like DNS) through an unrelated extension point (like IPAM). If a particular ecosystem vendor want's it's users to use a particular DNS server, then they should guide their users to pass the preferred DNS server as part of the already existing --dns option. There is nothing more needed.

We should not try to inject unrelated options (like DNS) through an unrelated extension point (like IPAM).

@mrjana - we can agree to disagree, IMHO IPAM has always included search-domain and dns-server-ip allocation. Check out about 40 commercial and non commercial products today uses this wholesome definition of IPAM: https://en.wikipedia.org/wiki/IP_address_management

If a particular ecosystem vendor wants its users to use a particular DNS server, then they should guide their users to pass the preferred DNS server as part of the already existing --dns option. There is nothing more needed.

With this change, I believe we could enable all those eco-systems with existing (e.g. above mentioned wiki link) tools, rather than forcing them to take up IPAM themselves.

Looks like we would not be able to merge this PR because of merge conflicts. Please rebase, fix conflicts, and force push to your branch.

@mavenugo @aboch @sanimej : I have addressed the comments.

The return signature for RequestAddress is retained as is and the information is now passed as remote driver data.
The join handles setting up resolution only when remote driver provides DNS options

If everything looks OK, I can add these details in remote.md

@DivyaVavili It has been detected that this issue has not received any activity in over 6 months. Can you please let us know if it is still relevant:

For a bug: do you still experience the issue with the latest version?
For a feature request: was your request appropriately answered in a later version?

Thank you!
This issue will be automatically closed in 1 week unless it is commented on.
For more information please refer to https://github.com/docker/libnetwork/issues/1926


fix(core): render template type when used in expressionProperties

This broke my use of ngx-formly. In version 4 I had been emitting field json like this:
"type":"select", "template":null,
This worked fine, because in line 74 the check was for:
if (field.template) ...
So if the template were null, everything was fine.
But in this commit the check was changed to:
if (field.hasOwnProperty('template') ...)
which is true even if the value of "template" is null.
So now I'm getting the error:
NgxFormly: passing 'type' property is not allowed when 'template' is set.
In my opinion the error is not really correct ... if "template" is null it is not actually being "set" to anything.
Is there a reason you are no longer checking to see if the template has been set?
I don't know about the larger issue you were trying to fix, but if you have no objection I could do a pull request with line 77 changed to either:
(field.hasOwnProperty('template') && field.template)

I opened issue #2121


go.dev: README.md shows non-breaking space and non-breaking hyphen as   and ‑ strings

This makes correct text look like it has typos when viewed by prospective employers, etc. on go.dev.
GitHub doesn't have any problems and you can see the difference in the screenshots.
Here's a screenshot from go.dev, from github.com, and the markdown code.
/cc @julieqiu @dmitshur
What did you see? (go.dev screenshot)
Problems in the screenshot include:

Preferred serialization
Well‑formedness checks
RFC 7049 Section 8 Security Considerations.

What did you expect to see? (github.com screenshot)

Reproducer (markdown snippet)
This is from fxamacker/cbor.  The   and ‑ allow the text to avoid messy line breaks in the "CBOR Feature" column. The last row of "Description" is also a problem on go.dev.
|     | CBOR Feature | Description |
| --- | :--- | :--- |
| ✔ | CBOR tags | API supports built-in and user-defined tags. |
| ✔ | Preferred serialization | Integers encode to fewest bytes. Optional float64→float32→float16 if value fits. |
| ✔ | Sort map keys | Unsorted, length-first (Canonical CBOR), and bytewise-lexicographic (CTAP2) |
| ✔ | Duplicate map keys | Always forbid for encoding, option to allow/forbid for for decoding. |
| ✔ | Indefinite length data | Option to allow/forbid for encoding and decoding. |
| ✔ | Well‑formedness checks | Always checked and enforced. |
| ✔ | Basic validity checks | UTF-8 validity, etc. checked after well-formedness. |
| ✔ | Security considerations | Prevent integer overflow and resource exhaustion described in RFC 7049 Section 8 Security Considerations. |

Thanks for reporting! We're working on fixes for this and other README rendering issues.

Julie, I appreciate your reply and your work trying to fix this.
I'm just sharing this with you in case there's anyone on your team that's under the impression there's a workaround available.
There is no workaround for this issue and #37284 because go.dev shows all past releases.
I don't think there's anything maintainers can do to make fxamacker/cbor v2.2.0 and prior versions render correctly on go.dev.  All maintainers can do is continue converting a dozen markdown tables to svg, and etc. so that v2.3.0 can look nice on go.dev.
I hope you're having a great day and know that lots of people outside of Google also appreciate & value your work!

@x448 - Thanks for the feedback, and I am happy to hear that you are enjoying using pkg.go.dev!
I understand the README issues have been frustrating - I have a fix in review for #37284, which I hope to have live next week, and am currently looking into this issue.
We are also working towards open sourcing the site (https://groups.google.com/g/golang-dev/c/mfiPCtJ1BGU/m/OZ9cU3SgBgAJ), so that in the future we can accept contributions for issues like this.
I'll update this issue once the fix has been merged, and also when it is live on the site. Thanks for being patient, and hope this helps!

Hey Julie, this is fantastic news!  Thanks so much for letting me know and sharing as much info as you did!  I think @fxamacker and others will appreciate it as well.
Open sourcing pkg.go.dev is a really great decision.  I heard a rumor that might be happening so this confirmation is really nice.  Revenue generating features could simply build on top of it so it's smart.
Thanks again for coming to the rescue on the two issues.  Seeing dmitshur give you a thumbs up on your comment made me happy too.  Have a fantastic weekend!

Hi!  I discovered this problem is not limited to tables.
This can be seen by bringing up the README of fxamacker/cbor v2.1.0 (not latest version) on go.dev and searching for nbsp (it's in the Comparisons section).
FYI - (you probably know about this) GitHub Flavored Markdown Spec highlights where they differ from regular markdown and also includes 650+ examples/tests.
p.s.
Julie's fix for #37284 not only worked but it's is beautiful!

@x448 - this should now be fixed.


Swift Package Manager dependency syntax

Following the README and ran into an error with regard to dependency syntax
it seems it was simply a capitalization issue - that package just needs to be all lowercase

Hi @nyx could you create a PR and I will be happy to merge it, thanks!

Closing this. I'm not making a PR for a simple typo sorry.


How to use themeGet with breakpoints?

Using themeGet, I am trying to achieve this:
const Box = styled.div`
    ${typography};
`;

const Test = () => <Box fontSize={["2xl", "xl", "base"]}>Box</Box>;

But using array in themeGet don't work:
const Box = styled.div`
  font-size: ${themeGet(["fontSizes.2xl", "fontSizes.xl", "fontSizes.base"])}px;
`;

const Test = () => <Box>Box</Box>;

themeGet can only get one key at a time. The only ways to use responsive styles in tagged template literals is to write out the actual media queries. You can define media query constants in your theme object for reuse, but you still have to use them.

themeGet can only get one key at a time. The only ways to use responsive styles in tagged template literals is to write out the actual media queries. You can define media query constants in your theme object for reuse, but you still have to use them.

I wrote a tiny extension that can achieve (hopefully) the result you're looking for here https://www.npmjs.com/package/better-theme-get


Add binance-btc extension

Description

Screencast

Checklist

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

Is this doing something that's not possible e.g. with this extension


No tabs in fullscreen, focus mode or not

Windows Terminal version (or Windows build number)
1.11.2421.0
Other Software
No response
Steps to reproduce
Use F11 to switch between full screen and windowed mode. In the latter, tabs appear fine. In the former, there are never tabs shown. Setting a keybind for focus toggle will only affect the windowed mode, and I cannot get tabs in full screen mode under any circumstances.
I've tried opening multiple tabs to make sure it wasn't a "auto-hide the tabs if there's only one session open" case.
Expected Behavior
I would expect full screen mode to honor the focus mode.
Actual Behavior
Focus mode is not honored and tabs will never appear in full screen mode.

That explains why I didn't find it when I did a search. If I'd known, I wouldn't have opened this issue. Feel free to close.

Additionally consider having the ability to show the tabs when you move your mouse to the top similar to many other applications.

I mean, yea, there's a chance - someone just needs to write the code now 😉

Any chance to add the option to show tabs in full screen mode? would be nice to have that option to toggle on/off

It would be better if the tabs show up when you hover over the top instead of having it always in full-screen mode.

Wadda ya know, that's also on the backlog, see #5677

+1 for always show tabs making it so that it always shows tabs


An animated button added

Hover to see the animation

The file name should be style.css not styles.css 😦

@Sapna127


Certain buttons are not recognized in the YouTube UI

I use styleboot mainly to make my UIs cleaner, especially after having already learned shortcuts for certain websites.
On YouTube for example, I removed most of the player buttons but apparently only some are removed while others are not, in particular I can't remove the "picture in picture" button (button.ytp-pip-button). The code is the same as the other keys but that one remains visible no matter what I do.
Can anyone help me? Maybe I'm doing something wrong?

Chrome 120.0.6099.225
Win11 23H2 22635.2915

The PiP button is from an extension, it's not something from youtube.
I use the Enhancer for YouTube™ extension, which has this button, and uses a bit of weird styling to make the button.
Image

Because it has display: inline-block !important;, and uses a more specific selector, your display: none; is being overridden.
To fix this, you need to also have a specific css selector (stylebot automatically adds !important, so you don't need to add it).
This css works for me, and should work for you.
#movie_player div.ytp-chrome-controls button.ytp-pip-button.ytp-button {
  display: none !important;
}

@Obsidianninja11
Copied, pasted and it works flawlessly.
Thanks a lot!


fix: BluetoothCharacteristicWindows

You need to override the super properties of the parent class, otherwise you cannot get the correct properties properties because fromProto does not use the properties property

Can I get an example code which make the bug related with fromProto?
In general case, user does not need to define BluetoothCharacteristic.fromProto(...)  in Flutter Blue Plus Windows.


Debugging .NET Core C# code using Console: "The handle is invalid"

Issue Type: Bug
Run the following C# code (.NET Core) under the debugger:
var row = Console.CursorTop;
Receive the exception:
Exception has occurred: CLR/System.IO.IOException
Exception thrown: 'System.IO.IOException' in System.Console.dll: 'The handle is invalid'
   at System.ConsolePal.GetBufferInfo(Boolean throwOnNoConsole, Boolean& succeeded)
   at System.Console.get_CursorTop()

This only happens when using the VS Code debug console (when debugging).
VS Code version: Code 1.23.1 (d0182c3417d225529c6d5ad24b7572815d0de9ac, 2018-05-10T17:11:17.614Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz (8 x 2494)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.88GB (9.69GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

vscode-new-file
dku
4.0.2

docomment
k--
0.1.0

azure-account
ms-
0.4.0

csharp
ms-
1.15.2

PowerShell
ms-
1.7.0

vscode-docker
Pet
0.0.27

Ok, I immediately then thought to check launch.json and found console setting.
Using externalTerminal for that prop fixes this.
Is there anything that could be done to make it more obvious/easier/the internal VS code debug console  work with it?  Running in the integrated terminal (using powershell in VS Code) works fine..

This issue was moved to OmniSharp/omnisharp-vscode#2336


Railsbump down? Cloudflare SSL handshake fails

Thanks for the ping, @dpaola2! Seems to be a problem with Cloudflare...
I'm trying to find out what happened.

Seems like it's working again! Thanks 😀

Finally! The SSL certs from Cloudflare and Fly.io were battling each other for a while... ⚔

Unfortunately it looks like the site is down again for the same issue.

Sounds good! I saw it was down, waited a few days, then saw that it was still down so I figured I would make you aware. Hopefully it doesn't take too much to get it back up, its a great app and I thank you for it!

Finally back up and not using Cloudflare SSL anymore, so this issue should (hopefully) not reappear.


Translation preprocessing/postprocessing

It would be great to have some kind of preprocessing/postprocessing functions as configuration options in $translateProvider config for preprocessing/postprocessing translations.
For example — I prefer uppercase notation in my translation tables and HTML templates:

But look at the typical situation I usually have — translation code in uppercase + some $scope variable in lowercase:
$scope.someVariable = 'baz';

I often have to use uppercase filter to make all translation code in uppercase. So, it would be great to have some preprocessing function in $translateProvider config to do any preprocessing stuff with translation table, e.g.:
angular.module('app').config(function($translateProvider) {
    $translateProvider.preferredLanguage('en');
    $translateProvider.preprocessTranslationKey(function(key) {
        return key.toUpperCase();
    });
});

@artuska Interesting idea, this would be optional functions for the service and have to be invoked in right places. Contributions are welcome.

Hello,
in part of a "Spring Cleaning 2017", we are closing this issue as part of several others. The meta discussion is available here for full transparency.
 Said this, we must keep the focus and scope on the project.
We are grateful and appreciate to everyone for each made contribution (regardless being code, idea, documentation, and support).
We are deprecating/closing all issues which fulfills a specific set of criteria: mainly outdated, nice-to-have, or just out-of-scope.

Although the list has been curated to the best of our's knowledge, there will be some lack of understanding.
Okay, there a different stories behind all open issues, but we do not have the time for everything:

Support: Because we have only limited resources, we cannot handle support questions any more. Please use appropriate boards like Stackoverflow.
Feature requests or ehancements: Altough it is probalbly a great idea (otherwise it would be closed already), we can not giving more resources to things out of scope or which are useful for too specific scenarios only.
Bugs: Filed a long time ago, this bug either has a reasonable workaround, no effective solution without harming all others, or is lacking additional information/use-case.
Docs: All documentation efforts not matching English will be abonded.

Thank you.
angular-translate team

FAQ
"But this is really important!"
Well, we are all humans. Maybe we have overseen an important issue. If we missed something, we hope you are reaching as again filing a new issue with the reference to the old one. But please be aware: A bug without further details and a demo included will be rejected again.
"But this would a great feature!"
Reconsider: Are we really required to add something in order to make this feature happen? Or could you also create a project on your own utilizing this project?


Typo in componentProps

What changes are proposed in this pull request?
fo.operators.types.View refers to componentProps but this property is not used in the frontend.
The correct property name is componentsProps.
How is this patch tested? If it is not, please explain why.
If the user sets types.Notice(.., description="", componentsProps={"description": ""}) it will work because all arguments to Notice are passed to the frontend when calling to_json.
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[x] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

Thank you for the contribution @marijnl! 🎉


[Question] Ipynb for ONNX conversion?

Could you please share the code you're using to convert models to onnx? I know you say in your cards you're using Optimum, but when I try to do it myself, I get much larger onnx files (talking about disk space here) and I don't know what I'm doing wrong.

Probably the easiest way is to clone the repo and use our conversion script (in the scripts folder).
python -m scripts.convert --quantize --model_id <model_name_or_path>

Check out the docs for more info.

@xenova This is what I did, but I ended up with onnx files that are similar in disk space size with the initial model sizez while your onnx models are smaller.

Did you use the `--quantize`` flag?

Oh, sorry, you pointed me to some other docs than the ones I was using for Optimum conversion.
My bad and thank you very much!
Will retry today with your script. Closing.


SPARKC-686 Port to Scala 2.13

Description
How did the Spark Cassandra Connector Work or Not Work Before this Patch
Spark Cassandra Connector did not work with Scala 2.13 before this patch.
General Design of the patch
No real API changes. In general, just migrating from scala.collection.JavaConversions to scala.jdk.CollectionConverters. Some usages of Seq needed to be migrated to immutable.Seq.
I apologize ahead of time if there's anything that is suboptimal here - I'm not a Scala developer so I'm not entirely familiar with best practices and how to do this migration. However, I think this stuff was all pretty straightforward with these two migrations.
I'm also not sure exactly how you would want to release this and how you'd like to branch it. Please let me know if you'd like me to do a PR to a different branch or something.
How Has This Been Tested?
All tests pass when run on my machine with sbt/sbt test.
Checklist:

[X] I have a ticket in the OSS JIRA
[X] I have performed a self-review of my own code
[X] Locally all tests pass (make sure tests fail without your patch)

@marquiswang please see the merge strategy here -> https://github.com/datastax/spark-cassandra-connector/blob/master/doc/developers.md#merge-path

Any update on this pull?

Hi there, when do you reckon this release will become available? I was looking to open an issue for you and I came across this one. In the only repo I'm using in this library, everything is failing for a while now and I come across more updates that I cannot do anymore: https://github.com/jesperancinha/bridges-surveillance-story.  An example of an indirect limitation is that the Glassfish containers that my modules are implicitly using have suffered an update and now they are Jakarta containers and not Javax containers. And if I update this to use Javax then I get other conflicts. I'm also getting serialization issues but long story short, I need the new spark cassandra connector. Please let me know. Thanks in advance!

Hi there. Does anybody know of an estimated timespan for when a Scala 2.13 version of the connector will be released? Thank you in advance

No real API changes. In general, just migrating from scala.collection.JavaConversions to scala.jdk.CollectionConverters. Some usages of Seq needed to be migrated to immutable.Seq.

Hm, is this really advisable? There maybe memory and performance impacts due to the use of immutable collections.

Hi there! How is the progress going on this issue? I just want to know a bit about how this is going. I think I might be looking for alternatives soon. My whole project can't get updated as it is, to Scala 2.13 solely because of this issue, so I definitely need to think about other alternatives soon.

Another attempt to finish this work has been made here https://github.com/datastax/spark-cassandra-connector/pull/1361

I can' really follow that well the work you are doing because I thought this branch was going to be the one to determine the next version supporting scala 2.13. I noticed that a release was already made at the 3rd quarter of last year. I noticed it just now. I updated my project and after struggling a bit with it everythng works! So thank YOU so much for this!  Great work!


chore(deps): update dependency jsdom to v24

This PR contains the following updates:

Package
Type
Update
Change

jsdom
devDependencies
major
^23.0.1 -> ^24.0.0

Release Notes

jsdom/jsdom (jsdom)
v24.0.0
Compare Source
This release reverts our selector engine back to nwsapi. As discussed in #3659, the performance regressions from @asamuzakjp/dom-selector turned out to be higher than anticipated. In the future, we can revisit @asamuzakjp/dom-selector after it reaches nwsapi's performance on the two real-world benchmarks provided by the community.
Since reverting to nwsapi causes several functionality regressions, e.g. removing :has() support, we've decided to make this a major version.
Additionally:

Small fixes to edge-case behavior of the following properties: input.maxLength, input.minLength, input.size, progress.max, tableCell.colSpan, tableCell.rowSpan, tableCol.span, textArea.cols, textArea.maxLength, textArea.minLength, textArea.rows.

Configuration
📅 Schedule: Branch creation - "after 10pm every weekday,before 5am every weekday,every weekend" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Thanks for the PR!
Deployments, as required, will be available below:

Frontend

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future 24.x releases. But if you manually upgrade to 24.x then Renovate will re-enable minor and patch updates automatically.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Fix pagination issue

The issue is we need to stop paginating if we recieve an empty string for
a next token marker. Before, if we did not have a check to see that the next
token was not the same as the previous token and throw an error, we would have
hit an infinite loop.

Coverage decreased (-0.02%) to 94.5% when pulling 24a133627bcc0997de9c46633b2a2246ce848c95 on kyleknap:fix-pagination into dbe6fbef4e20184c05ac552b265ea9ba0241c8b5 on boto:develop.

Coverage decreased (-0.02%) to 94.5% when pulling 24a133627bcc0997de9c46633b2a2246ce848c95 on kyleknap:fix-pagination into dbe6fbef4e20184c05ac552b265ea9ba0241c8b5 on boto:develop.

Coverage increased (+0.0%) to 94.52% when pulling 2e0a5f72cd6ce33fa679638d29dd535ecd588f6c on kyleknap:fix-pagination into dbe6fbef4e20184c05ac552b265ea9ba0241c8b5 on boto:develop.


Tiktok WARNING: [TikTok] Expecting value in

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
USA
Provide a description that is worded well enough to be understood
WARNING: [TikTok] Expecting value in '': line 1 column 1 (char 0). Retrying
Video downloads without issue.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
C:\Users\Desktop\Downloads\yt-dlp>yt-dlp --verbose -s https://www.tiktok.com/@foodnetwork/video/7346630599861390634?lang=en
[debug] Command-line config: ['--verbose', '-s', 'https://www.tiktok.com/@foodnetwork/video/7346630599861390634?lang=en']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [615a84447] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 2023-12-07-git-f89cff96d0-essentials_build-www.gyan.dev (setts), ffprobe 2023-12-07-git-f89cff96d0-essentials_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.02.02, mutagen-1.47.0, requests-2.31.0, sqlite3-3.35.5, urllib3-2.2.1, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1803 extractors
[TikTok] Extracting URL: https://www.tiktok.com/@foodnetwork/video/7346630599861390634?lang=en
[TikTok]<PHONE_NUMBER>861390634: Downloading video feed
WARNING: [TikTok] Expecting value in '': line 1 column 1 (char 0). Retrying... (attempt 1 of 4)
[TikTok]<PHONE_NUMBER>861390634: Downloading video feed
WARNING: [TikTok] Expecting value in '': line 1 column 1 (char 0). Retrying... (attempt 2 of 4)
[TikTok]<PHONE_NUMBER>861390634: Downloading video feed
WARNING: [TikTok] Expecting value in '': line 1 column 1 (char 0). Retrying... (attempt 3 of 4)
[TikTok]<PHONE_NUMBER>861390634: Downloading video feed
WARNING: [TikTok]<PHONE_NUMBER>861390634: Failed to parse JSON (caused by JSONDecodeError("Expecting value in '': line 1 column 1 (char 0)")); trying with webpage
[TikTok]<PHONE_NUMBER>861390634: Downloading webpage
[debug] [TikTok] Found universal data for rehydration
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info]<PHONE_NUMBER>861390634: Downloading 1 format(s): download

for the fix, run this: yt-dlp --update-to nightly
duplicate of pinned issue


Changed xWebsite LogPath to directory attribute. Fixes #256

Changed VerboseTestTargetFalseLogPath: The error message did not contain 'not'
Changed instances of $website.logfile.LogPath to $website.logfile.directory
Added extra unit tests for xWebsite.

Fixes #256

This change is

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, 9 unresolved discussions.

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 514 at r1 (raw file):

                $MockLogOutput.directory = $MockParameters.LogPath

                Mock -CommandName Test-Path -MockWith {Return $true}

{ return $true }

space around braces and 'return' in lowercase

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 516 at r1 (raw file):

                Mock -CommandName Test-Path -MockWith {Return $true}

                Mock -CommandName Get-Website -MockWith {return $MockWebsite}

space around curly braces

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 519 at r1 (raw file):

                Mock -CommandName Get-WebConfigurationProperty `
                    -MockWith {return $MockLogOutput.logExtFileFlags }

space after '{'

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 521 at r1 (raw file):

                    -MockWith {return $MockLogOutput.logExtFileFlags }

                $Result = Test-TargetResource -Ensure $MockParameters.Ensure `

$result

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 1033 at r1 (raw file):

                }

                Mock -CommandName Get-Website -MockWith {return $MockWebsite}

spaces around curly braces

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 1069 at r1 (raw file):

                    -ParameterFilter { ($Type -eq 'Windows') }

                $Result = Set-TargetResource @MockParameters

$result

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 1071 at r1 (raw file):

                $Result = Set-TargetResource @MockParameters

                It 'should call all the mocks' {

It 'Should...'

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 1103 at r1 (raw file):

                }

                Mock -CommandName Get-Website -MockWith {return $MockWebsite}

spaces around braces

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 1141 at r1 (raw file):

                $Result = Set-TargetResource @MockParameters

                It 'should call all the mocks' {

It 'Should....'

Comments from Reviewable

Review status: 2 of 3 files reviewed at latest revision, 9 unresolved discussions.

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 514 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
> ```PowerShell
> { return $true }
> ```
>
> space around braces and 'return' in lowercase

Done.

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 516 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
> space around curly braces

Done.

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 519 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
> space after '{'

Done.

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 521 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
> $result

Done.

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 1033 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
> spaces around curly braces

Done.

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 1069 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
> $result

Done.

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 1071 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
> It 'Should...'

Done.

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 1103 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
> spaces around braces

Done.

Tests/Unit/MSFT_xWebsite.Tests.ps1, line 1141 at r1 (raw file):
Previously, mbreakey3 (Mariah) wrote…
> It 'Should....'

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Update README.md

I already had my rails server running, and couldn't figure out for a second why the method was undefined.

I bumped the version.rb to 0.0.11 - when you bundle the gem as-is, it grabs version 0.0.10 and the config/schedulable.rb file references the older and incorrect config.max_count and config.max_period
These values should be config.max_build_count and config.max_build_period - when you grab master's HEAD the template is correct.  So the only change is the version number to grab the updated tag.


How do we run a simple node application with EbbRT

Hi All,
I read the paper on EbbRT and wanted to give it a try. Specifically, I would like to run a node application on it. How do I go about doing it? I looked in code/docs but did not any clue.
Any pointers would be appreciated.

Hey you can take a look at the tutorial notes here:
https://github.com/SESA/EbbRT/wiki/Build-Tutorial-Notes


Feature/add sync connection

This is the implementation for sync and its tests. Async is WIP.
After this PR is merged, replace with async connection

There remains still ordering service's server. But PR is a little fat, so I'll stop.
Also, suddenly block_index_test failed. The cause is unclear.


Feature/Luma Ai Video Generation Driver

[ ] I have read and agree to the contributing guidelines for submitting new pull requests.

Describe your changes
Added Luma AI as a driver, tool, task, to generate videos.
Issue ticket number and link
Add Luma Ai as a driver

This functionality is experimental enough where it should be initially implemented as a griptape extension once https://github.com/griptape-ai/tool-template/pull/3 goes in


stable/jenkins: Solve warning when supplying sidecar resources

What this PR does / why we need it:
The resources value is currently nil, but requires a map to be supplied. When overriding the value from a parent chart, a warning is produced because a map is applied to a value which was originally nil.
  Merging destination map for chart 'jenkins'. The destination item 'resources' is a table and ignoring the source 'resources' as it has a non-table value of: <nil>

Adding an empty map solves the warning and ensures the original intent.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

@lachie83 @viglesiasce @maorfr @torstenwalter @mogaal

/lgtm


Apps aren't loaded yet

Hi,
I am just starting on using ClickHouse with Django 3.2.18 ,and Python 3.8.7.But it not worked.
I added this in my Database setting:
DATABASES = {
    'default': {
        'ENGINE': 'django.db.backends.mysql',
        'NAME': APP_CODE + '-dev',
        'USER': 'root',
        'PASSWORD': 'jlpay@159357',
        'HOST': '<IP_ADDRESS>',
        'PORT': 13306,
        "init_command": "SET foreign_key_checks = 0;"
    },
    'clickhouse': {
        'ENGINE': 'clickhouse_backend.backend',
        'NAME': 'default',
        'HOST': '<IP_ADDRESS>',
        'USER': 'admin',
        'PASSWORD': 'xxxxxx',
        'TEST': {
            'fake_transaction': True
        }
    }
}
DATABASE_ROUTERS = ['dbrouters.ClickHouseRouter']
DEFAULT_AUTO_FIELD = 'django.db.models.BigAutoField'

and when I import in databaseroutes.py :
from clickhouse_backend.models import ClickhouseModel

same errors was happened:
Exception has occurred: AppRegistryNotReady
Apps aren't loaded yet.
  File "D:\vscode\opsmanage\opsmanage\conf\dev.py", line 41, in <module>
    from clickhouse_backend.models import ClickhouseModel
  File "D:\vscode\opsmanage\opsmanage\settings.py", line 13, in <module>
    _module = __import__(DJANGO_CONF_MODULE, globals(), locals(), ['*'])
  File "D:\vscode\opsmanage\manage.py", line 27, in <module>
    execute_from_command_line(sys.argv)

expect your's
thanks

DO NOT past your DB username and password.
It seems that this line caused the problem.
  File "D:\vscode\opsmanage\opsmanage\settings.py", line 13, in <module>
    _module = __import__(DJANGO_CONF_MODULE, globals(), locals(), ['*'])

DO NOT post your DB username and password.
You can not use models in your app before your project setup. Please move and other relavant code from to other place.from clickhouse_backend.models import ClickhouseModel``D:\vscode\opsmanage\opsmanage\conf\dev.py

oh,it is working,thanks!


Collection - Categorizing/Tagging games

Feedback

One thing that would be nice is to be able categorize and search my owned games by mechanisms.

it will be nice to have tags or something like that, so I may see which games I have in my language, not in English, games I have home, games I have in my cottage house, games I've borrow to someone, maybe some list of games I want to sell..

AC

[?] Add ability to categorize owned games
Add ability to filter / search collections by mechanism

There's a separate taks to handle filtering / searching by category #106


test(e2e): add balance check before running e2e tests

Context
There is a race condition between starting the local test network and running the e2e tests, sometimes the e2e test can start before the state of the local test network is fully indexed by dbSync, which causes the tests to fail since the wallet balance is still not available.
Proposed Solution
Make the e2e test wallet for balance to be available at the beginning of the test.

It would be good to define a test util, considering it looks like each test needs:
await firstValueFrom(wallet.syncStatus.isSettled$.pipe(filter((isSettled) => isSettled)));
await firstValueFrom(wallet.balance.utxo.total$.pipe(filter(({ coins }) => coins > 0)));

Maybe walletReady()

addressed here: https://github.com/input-output-hk/cardano-js-sdk/pull/435/commits/0875c29000e43ce9ca04a77f259b1ce0d46f72af


Replace store statics

This patch moves the statics storing the various storage components into
the local resources for the init tasks.  This makes them sound and safe
to use.  Unfortunately, this leads to a significant increase in binary
size.

Based on:

https://github.com/Nitrokey/nitrokey-3-firmware/pull/481

Using MaybeUninit instead of Option already improves the binary size a lot.  Maybe I can find some more optimizations.

Of course this breaks the provisioner app which wants a mutable reference to the storage so that it can format the filesystem.  Maybe we can just drop that feature.  I don’t think we use it during provisioning, and for development devices, we have other means to format the IFS.

I think it should be fine to drop support for it - that's usually anyways only needed during provisioning - for development we have the debugger to do this job

No significant changes.

Insignifcant changes

metric
value

change

binary-size-nk3am
1431243
:red_circle:
+7973 (+0.01%)

binary-size-nk3am-test
2032675
:red_circle:
+10501 (+0.01%)

binary-size-nk3xn
513076
:red_circle:
+2804 (+0.01%)

binary-size-nk3xn-test
550416
:red_circle:
+3260 (+0.01%)

binary-size-nkpk
742941
:red_circle:
+6520 (+0.01%)


feat: Context is now an object instead of a boolean

Closes #
Description of the changes

Checklist

[x] I didn't over-scope my PR
[x] My PR title matches the commit convention
[x] I did not include breaking changes
[x] I made my own code-review before requesting one

I included unit tests that cover my changes

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

I added/updated the documentation about my changes

[ ] 📜 README.md
[ ] 📕 docs/*.md
[ ] 📓 docs.lenra.io
[x] 🙅 no documentation needed

Technical highlight/advice

This was supposed to be done directly in the components-api library.


Feature Request: A button to automatically cop the untranslated phrase at translated box

Feature Request: A button to automatically cop the untranslated phrase at translated box
User Case: Having 50 phrases starting as "expecting error at.......

By pressing a copy button to transfer the phrase below, i can quickly highlight it and paste the translated phrase.

Ideally, poedit must allow a type of dictionary that you must define that some phrases be substituted automatically. Like copy/paste in writer.

So by copy/paste, "expecting error..." can be susbstituted very quickly.

This will speed up the translation very much!

Feature Request: A button to automatically cop the untranslated phrase at translated box

See the Edit menu.
If you mean something else (you're not exactly comprehensible and should read http://www.chiark.greenend.org.uk/~sgtatham/bugs.html), submit a PR.


Docs: Add real thumbnail img

Note: Please transform - [ ] into - (NA) in the description when things are not applicable
Related issues

Description

Replace current svg square yellow and blue with a thumbnail image
Motivation & Context

Doc enhancement
Types of change

Refactoring (non-breaking change)

Live previews

https://deploy-preview-2034--boosted.netlify.app/docs/5.3/content/tables/#with-icons-or-thumbnails

Checklist

Contribution

[x] I have read the contributing guidelines

Accessibility

[x] My change follows accessibility good practices; I have at least run axe

Design

[x] My change respects the design guidelines defined in Orange Design System
[x] My change is compatible with responsive display

Development

[x] My change follows the developer guide
[ ] I have added JavaScript unit tests to cover my changes
[ ] I have added SCSS unit tests to cover my changes

Documentation

[x] My change introduces changes to the documentation and/or I have updated the documentation accordingly

Checklist (for Core Team only)

[ ] My change introduces changes to the migration guide
(NA) My new component is added in Storybook
(NA) My new component is compatible with RTL
(NA) Manually run BrowserStack tests
(NA) Manually test browser compatibility with BrowserStack (Chrome >= 60, Firefox >= 60 (+ ESR), Edge, Safari >= 12, iOS Safari, Chrome & Firefox on Android)
[ ] Code review
[ ] Design review
[ ] A11y review

After the merge

[ ] Manually launch Percy tests

The thumbnail looks good. No issues.


Encode/Decode for FIPS module

Most of ASN.1 functions in ec_asn1.c are below #ifndef FIPS_MODULE compilation condition.
(https://github.com/openssl/openssl/blob/master/crypto/ec/ec_asn1.c#L24).
Where is the relevant implementation of encoders/decoders for FIPS module?

There are no encoders/decoders in the FIPS module. This is outside the FIPS boundary. Typically you will load the "base" provider at the same time as "fips" to ensure you have the encoders/decoders available without having to have the whole "default" provider loaded.

So it should use the same code from ec_asn1?

So it should use the same code from ec_asn1?

The base provider does - yes.