QS: show string encoding

Having UTF-16LE strings in the output by default is a great quality of life improvement. However, it would be nice if the output also indicated the string encoding that a particular string has.

via https://github.com/mandiant/flare-floss/issues/776
see https://github.com/getreu/stringsext for tool that already does this

i'll acknowledge that most tools today show the string encoding, so QS probably should too.
however, what is the value of this field, and is worth 6-8 characters per line to indicate the flavor? how does someone use this field? and, do they use the field in most invocations of strings/QS or just occasionally?
my feeling is that i rarely use this information, unless im crafting a yara rule or other signature that relies on the encoding. in this case, its certainly safer for me to use a hex editor, etc. to figure out the specific byte sequence. so, i wonder if it's a reasonable choice to not show the encoding in the common case?
thoughts @cxiao @mr-tz ?

alternatively, we could use color to indicate encoding; however, we should avoid using color as the only way to indicate something, so we'd still need some character/symbol to indicate the flavor.
we could a single character/emoji/unicode to indicate the flavor, rather than "#ascii" or whatever. this would save space and some visual noise. but, would it be easy enough to interpret?

I think a single character indication (A for ASCII, U for UTF-16LE, or A is default/implicit and U indicates a wide string) and maybe color would work well. With a dynamic UI we could also make this configurable.

One quick test, adding U for UTF16-LE strings and nothing for ASCII (assume default) before the offset:

With a dynamic UI we could also make this configurable.

this is a good point, i won't get too hung up on this now.

adding U for UTF16-LE strings and nothing for ASCII (assume default) before the offset

nice job. looks solid. only question is whether this is easy enough to interpret, which im inclined to believe, especially if we emit a footer with fields and recommended interpetation.

Explored to add a very basic and ugly header line. In general, I would prefer to move this to the documentation (a TLDR screenshot, for example) and to keep things as self-explanatory as much as possible.

added in 8d41bde


Connecting with android not works correctly

I like to build sth. like a captive portal (AP + Website on the host machine that is reachable via the AP).
Connecting with an android device results sometimes in connection loops and an authentication  erorr.
Here is the log of the host:
$ sudo create_ap -n wlan0 --redirect-to-localhost --no-virt --country DE ap password
Config dir: /tmp/create_ap.wlan0.conf.5VuN1Q2l
PID: 15934
Network Manager found, set wlan0 as unmanaged device... DONE
No Internet sharing
hostapd command-line interface: hostapd_cli -p /tmp/create_ap.wlan0.conf.5VuN1Q2l/hostapd_ctrl
Configuration file: /tmp/create_ap.wlan0.conf.5VuN1Q2l/hostapd.conf
wlan0: interface state UNINITIALIZED->COUNTRY_UPDATE
Using interface wlan0 with hwaddr b4:6d:83:96:52:49 and ssid "ap"
wlan0: interface state COUNTRY_UPDATE->ENABLED
wlan0: AP-ENABLED
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: authenticated
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: associated (aid 1)
wlan0: AP-STA-CONNECTED e8:50:8b:e3:84:fb
wlan0: STA e8:50:8b:e3:84:fb RADIUS: starting accounting session 56954B3F-00000000
wlan0: STA e8:50:8b:e3:84:fb WPA: pairwise key handshake completed (RSN)
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: authenticated
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: associated (aid 1)
wlan0: STA e8:50:8b:e3:84:fb RADIUS: starting accounting session 56954B3F-00000000
wlan0: STA e8:50:8b:e3:84:fb WPA: pairwise key handshake completed (RSN)
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: authenticated
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: associated (aid 1)
wlan0: STA e8:50:8b:e3:84:fb RADIUS: starting accounting session 56954B3F-00000000
wlan0: STA e8:50:8b:e3:84:fb WPA: pairwise key handshake completed (RSN)
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: authenticated
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: associated (aid 1)
wlan0: STA e8:50:8b:e3:84:fb RADIUS: starting accounting session 56954B3F-00000000
wlan0: STA e8:50:8b:e3:84:fb WPA: pairwise key handshake completed (RSN)
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: authenticated
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: associated (aid 1)
wlan0: AP-STA-DISCONNECTED e8:50:8b:e3:84:fb
WPA: wpa_sm_step() called recursively
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: authenticated
wlan0: STA e8:50:8b:e3:84:fb IEEE 802.11: associated (aid 1)
WPA: wpa_sm_step() called recursively

Usually this happens when you create an AP with different password from previous AP that has the same name. On Android press and hold the name of your AP, choose Forget Network and then try again. In create_ap also try -w 1 and -w 2 to choose WPA or WPA2 only.
If you still have a problem then maybe it's a hostapd bug or a bug in the driver of your adapter.

Check also the comments on #73


Copy and Paste broken from co-pilot chat (or other extension sidebar)

Environment data

VS Code version:1.84.2
Jupyter Extension version (available under the Extensions sidebar): v2023.10.1100000000
Python Extension version (available under the Extensions sidebar): v2023.22.1
OS (Windows | Mac | Linux distro) and version:OSX 12.6.3
Python and/or Anaconda version: any
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): any
Jupyter server running: Local | Remote | any

Expected behaviour
If you have a notebook open, you should be able to highlight text in the extension sidebar (For example co-pilot Chat window) copy it and paste it into the notebook code block
Actual behaviour
When you run copy, it seems to ignore the text selection and instead copy the current notebook block.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Open a notebook
Add a code block with "hello"
Open Co-pilot Chat window (or other extension sidebar where you can select text)
cmd-c
back in the notebook, click the line after hello
cmd-p

You'll now see hellohello.

Logs
Reproduction does not generate any log events

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.85.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Issue reproduced on v1.85.1


Support workingDir init on Windows

Previously, we used --shell-image (typically distroless/base) to invoke
mkdir -p <dirs> to initialize any working dirs that were needed by
steps in a TaskRun.
This isn't portable to Windows, since distroless/base doesn't provide
Windows images, and since invoking mkdir -p won't work on Windows.
Instead, with this change, we init working dirs using a very simple Go
binary that can be cross-compiled to run on both OSes.
Fixes https://github.com/tektoncd/pipeline/issues/4473
cc @lippertmarkus
/kind feature
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Docs included if any changes are user facing
[ ] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
Working dir initialization is now done using a Go binary in a new container image included in the release, which makes it portable to run on Windows nodes.

Very nice, cheers!
/lgtm

/lgtm


Changing profile photo not working and Friends Profiles Pictures not show

Hi I read the last bug closed for Changing profile photo not working but still that not work I have 2 installations in different domains with SSL and in both some users you add us friend not show the profile picture is only the icon of error picture because not found . That error is in both installations and in one installation one user can upload picture crop and setup us profile picture but the picture never change.
Test difference browsers and networks not make different

@royalterra This issue is closed because Redmatrix evolved into Hubzilla. When this is still an issue in the most recent version of Hubzilla, please recreate this issue over there. We're sorry for any inconvenience this will maybe give you.


Some strange logs...

That´s in my log by starting the adapter:

coronavirus-statistics.0
2020-03-23 16:43:00.881
info
(31228) Terminated (ADAPTER_REQUESTED_TERMINATION): Without reason

coronavirus-statistics.0
2020-03-23 16:43:00.407
warn
(31228) State attribute definition missing for + countryInfo

coronavirus-statistics.0
2020-03-23 16:42:59.980
warn
(31228) State attribute definition missing for + countryInfo

coronavirus-statistics.0
2020-03-23 16:42:59.729
warn
(31228) State attribute definition missing for + countryInfo

coronavirus-statistics.0
2020-03-23 16:42:59.433
warn
(31228) State attribute definition missing for + countryInfo

coronavirus-statistics.0
2020-03-23 16:42:59.095
warn
(31228) State attribute definition missing for + countryInfo

coronavirus-statistics.0
2020-03-23 16:42:58.792
warn
(31228) State attribute definition missing for + countryInfo

coronavirus-statistics.0
2020-03-23 16:42:58.557
warn
(31228) State attribute definition missing for + countryInfo

coronavirus-statistics.0
2020-03-23 16:42:58.451
warn
(31228) State attribute definition missing for + countryInfo

coronavirus-statistics.0
2020-03-23 16:42:58.450
warn
(31228) Turks_and_Caicos not found in lib! Must be added to the country name translator.

coronavirus-statistics.0
2020-03-23 16:42:58.250
warn
(31228) State attribute definition missing for + countryInfo

coronavirus-statistics.0
2020-03-23 16:42:58.000
warn
(31228) State attribute definition missing for + countryInfo

fixed 0.3.4 :)
https://github.com/iobroker-community-adapters/ioBroker.coronavirus-statistics/commit/2c7df7bb970cfae93dc61ef838b23eaa8258a50c


Doubt on some pieces of code

in function sortFinished in models/beam.py, when the parameter minimum is not None, we need add from beam, but during the iteration, the beam index i is not updated

if minimum is not None:
  i = 0
  # Add from beam until we have minimum outputs.
  while len(self.finished) < minimum:
    s = self.scores[i]
    self.finished.append((s, len(self.nextYs) - 1, i))
    # update i?
    # i += 1

in function sample in models.seq2seq.py line 125, is the variable alignmentsi misspelled?

Many thanks for pointing out our mistakes. We've already corrected them.

hello，谢谢作者的开源分享．有几点疑问想向您请教下：１.我看代码inception部分，BN放在conv和RELU后面而不是二者中间，请问这个是有什么特别的考量么？2.self-attention部分的操作和大众所知的self-attention貌似也有一些差别，也有特殊考量么？3.GLU部分，是用LSTM的正向output＊sigmoid(反向output)，有木有考虑过直接［正向，反向］＊sigmoid([正向，反向])呢？


Issue In Training with Ubuntu Corpus

My code
import logging
from chatterbot import ChatBot
from chatterbot.trainers import UbuntuCorpusTrainer
Enable info level logging
logging.basicConfig(level=logging.INFO)
chatbot = ChatBot('Example Bot')
trainer = UbuntuCorpusTrainer(chatbot)
Start by training our bot with the Ubuntu corpus data
trainer.train()
Now let's get a response to a greeting
response = chatbot.get_response('How are you doing today?')
print(response)
When I run this, I get following error
self._popen = self._Popen(self)

File "D:\Users\Sarang_Jain\Anaconda3\lib\multiprocessing\context.py", line 322, in _Popen
return Popen(process_obj)
File "D:\Users\Sarang_Jain\Anaconda3\lib\multiprocessing\popen_spawn_win32.py", line 33, in init
prep_data = spawn.get_preparation_data(process_obj._name)
File "D:\Users\Sarang_Jain\Anaconda3\lib\multiprocessing\spawn.py", line 143, in get_preparation_data
_check_not_importing_main()
File "D:\Users\Sarang_Jain\Anaconda3\lib\multiprocessing\spawn.py", line 136, in _check_not_importing_main
is not going to be frozen to produce an executable.''')
RuntimeError:
An attempt has been made to start a new process before the
current process has finished its bootstrapping phase.
    This probably means that you are not using fork to start your
    child processes and you have forgotten to use the proper idiom
    in the main module:

        if __name__ == '__main__':
            freeze_support()
            ...

    The "freeze_support()" line can be omitted if the program
    is not going to be frozen to produce an executable.

Traceback (most recent call last):
File "chatbot2.py", line 17, in
trainer.train()
File "D:\Users\Sarang_Jain\Anaconda3\lib\site-packages\chat.
Please help me resolve this issue.
My chatterbot version is 1.0.4

sorry it seems its a windows issues. I googled and found out that it works fine on ubuntu. I am not an expert on windows so can anyone help me how to resolve this.

so i found a fix to the problem.
if name == 'main':
freeze_support()
trainer = UbuntuCorpusTrainer(chatbot)
# Start by training our bot with the Ubuntu corpus data
trainer.train()


[C4GT Community]:

Ticket Contents
Description
[Provide a brief description of the feature, including why it is needed and what it will accomplish.]
Goals
Goals

[ ] [Goal 1]
[ ] [Goal 2]
[ ] [Goal 3]
[ ] [Goal 4]
[ ] [Goal 5]

Expected Outcome
No response
Acceptance Criteria
No response
Implementation Details
Dummy
Mockups/Wireframes
No response
Product Name
Dummy
Organisation Name
SamagraX
Domain
No response
Tech Skills Needed
Babel
Mentor(s)
@omkar309
Complexity
Low
Category
Beginner Friendly

Hii @omkar309 I would like to work on this issue. can you assign this to me?


Mingw gcc doesn't define _MSC_VER, use _WIN32 instead.

Thank you for the pull request.
Sorry for not documenting this, but the code you are changing is actually part of another project that we automatically import. Can you please submit your pull request to https://github.com/google/cctz instead?

See google/cctz#81.


Crash/Freeze upon loading a world

Heya.  Looks like a really cool mod..  and I was really looking forward to using it's easy syntax.  Unfortunately, adding it to my modpack then loading an existing world, results in the game freezing up during world load.
I wish I could give you a crashlog, but when the game freezes, it fails to create the crashlog, maxing out one of my CPU cores and forces me to end task the game.  However, it does say this in the console.
[00:23:29] [Server thread/ERROR]: Encountered an unexpected exception
java.lang.NoSuchMethodError: 'void org.yaml.snakeyaml.constructor.Constructor.<init>(java.lang.Class)'
	at net.pcal.mobfilter.MFConfig.load(MFConfig.java:17) ~[transformed-mod-mobfilter.i0:0/:?]
	at net.pcal.mobfilter.MFService.loadConfig(MFService.java:115) ~[transformed-mod-mobfilter.i0:0/:?]
	at net.pcal.mobfilter.MFInitializer.lambda$onInitialize$0(MFInitializer.java:19) ~[transformed-mod-mobfilter.i0:0/:?]
	at org.quiltmc.qsl.lifecycle.api.event.ServerLifecycleEvents.lambda$static$0(ServerLifecycleEvents.java:50) ~[transformed-mod-quilt-lifecycle-events.i0:0/:?]
	at net.minecraft.server.MinecraftServer.handler$fmi000$quilt_lifecycle_events$serverStarting(MinecraftServer.java:3842) ~[transformed-mod-minecraft.i0:0/:?]
	at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:650) ~[transformed-mod-minecraft.i0:0/:?]
	at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:266) ~[transformed-mod-minecraft.i0:0/:?]
	at java.lang.Thread.run(Thread.java:833) ~[?:?]

Removing Mob Filter allows the modpack to load and run fine again.
This is what's uncommented in my mobfilter.yaml I was testing out -
rules:

 - name: No Surface Skeletons
   what: DISALLOW_SPAWN
   when:
     entityId : [minecraft:skeleton]
     dimensionId: ['minecraft:overworld']
     blockY: [0, MAX]

MC 1.19.4
Quilt 0.19.1
Mob Filter 0.2.1

Seems like same crash I just had.
24.08 00:37:24 [Server] Server thread/ERROR Encountered an unexpected exception
24.08 00:37:24 [Server] INFO java.lang.NoSuchMethodError: 'void org.yaml.snakeyaml.constructor.Constructor.(java.lang.Class)'
24.08 00:37:24 [Server] INFO at net.pcal.mobfilter.MFConfig.load(MFConfig.java:17) ~[mobfilter-0.4.0+1.20.1.jar:?]
24.08 00:37:24 [Server] INFO at net.pcal.mobfilter.MFService.loadConfig(MFService.java:115) ~[mobfilter-0.4.0+1.20.1.jar:?]
24.08 00:37:24 [Server] INFO at net.pcal.mobfilter.MFInitializer.lambda$onInitialize$0(MFInitializer.java:19) ~[mobfilter-0.4.0+1.20.1.jar:?]
24.08 00:37:24 [Server] INFO at net.fabricmc.fabric.api.event.lifecycle.v1.ServerLifecycleEvents.lambda$static$0(ServerLifecycleEvents.java:38) ~[fabric-lifecycle-events-v1-2.2.2+b3afc78b77-21c1963f26916f23.jar:?]
24.08 00:37:24 [Server] INFO at net.minecraft.server.MinecraftServer.handler$zeg000$fabric-lifecycle-events-v1$beforeSetupServer(MinecraftServer.java:2837) ~[server-intermediary.jar:?]
24.08 00:37:24 [Server] INFO at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:646) ~[server-intermediary.jar:?]
24.08 00:37:24 [Server] INFO at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:265) ~[server-intermediary.jar:?]
24.08 00:37:24 [Server] INFO at java.lang.Thread.run(Thread.java:833) ~[?:?]
Fabric Loader 0.14.21 My Server host is 1 version behind not sure if that affects things at all.
Fabric API 0.87.0

Crash Investigated by WORM in the mob-filter discord was found to be caused by ViaFabric. No known workaround though.


Folder combination program is too sensitive to filenames with commas and spaces

Noticed w/ @josephneos1010

B/c of this change: https://github.com/gerlichlab/looptrace/commit/7222ce2febc20373063f78d584e204ea975e0ffb#diff-53a6f50a981c928f4ed8802361e2e410ac03fc44b6222c1a2ce0d7c8d6f1439a

scala> preFields :+ res3 ++ postFields.tail
val res4: Array[String | IndexedSeq[Char | String]] = Array(20240313, 155456, 993, "", Vector(T, i, m, e, 0, 0, 0, 4, 2, Point0012, ChannelFar Red,Red, Seq0558.nd2))

scala> preFields ++ Array(res3) ++ postFields.tail
val res5: Array[String] = Array(20240313, 155456, 993, "", Time00042, Point0012, ChannelFar Red,Red, Seq0558.nd2)

^^ unfortunately, the compiler allows the first expression above, inferring an odd type that would be atypical to have as intended behavior in such a use context, but alas, we will wrap the value.


Job ENV variables are shared between init container & running container

Describe the bug: (a clear and concise description of what the bug is)
There is one config map created for both the init container and the running container. When you have the same values in the init as well as running container, it would store the value twice in the config map, making otomi/k8s complain
To Reproduce
Steps to reproduce the behavior:
Create a new Job:
{{- $v := .Environment.Values }}
{{- $c := $v.charts }}

type: Job
name: gitea
enabled: true
env:
  GITEA_URL: https://gitea.{{ $v.cluster.domain }}
  GITEA_PASSWORD: {{ $c | get "gitea.admin.password" "bladibla"}}
init:
  env:
    GITEA_URL: https://gitea.{{ $v.cluster.domain }}
    GITEA_PASSWORD: {{ $c | get "gitea.admin.password" "bladibla"}}
  image:
    repository: otomi/tools
    tag: v1.4.13
    pullPolicy: IfNotPresent
  shell: bash
  script: |
    {{- readFile "scripts/init/gitea.sh" | nindent 4 }}
image:
  repository: otomi/tools
  tag: v1.4.13
  pullPolicy: IfNotPresent
shell: bash
script: |
  {{- readFile "scripts/gitea.sh" | nindent 2 }}
runPolicy: Always

The config map created would be templated as follows:
# Source: jobs/templates/configmap.yaml
apiVersion: v1
kind: ConfigMap
metadata:
  name: job-gitea
  labels:
    helm.sh/chart: jobs-0.1.0
    app.kubernetes.io/name: jobs
    app.kubernetes.io/instance: job-gitea
    app.kubernetes.io/version: "0.1.0"
    app.kubernetes.io/managed-by: Helm
data:
    GITEA_PASSWORD: bladibla
    GITEA_URL: https://gitea.dev.eks.otomi.cloud
    GITEA_PASSWORD: bladibla
    GITEA_URL: https://gitea.dev.eks.otomi.cloud
---

Where data contains duplicate keys (and values)
Expected behavior: (a clear and concise description of what you expected to happen)
Not storing duplicate key/value pairs.
Versions: (seen in console's /settings)

Core Version - current master

Proposed solution:
Create two config maps, one for the init container, and one for the running container.
Would need to double check that the existing jobs move the ENV variables accordingly.

I like that you re-use our job chart, well done! Also a good catch of this use-case.
In your feature branch, do you think we can split it up like this:
apiVersion: v1
kind: ConfigMap
metadata:
  name: job-{{ $v.name }}
  labels: {{- include "jobs.labels" $ | nindent 4 }}
data:
  {{- with $v.env }}
    {{- toYaml . | nindent 4 }}
  {{- end }}
  {{- with $v.init.env  }}
    {{- toYaml . | nindent 4 }}
  {{- end }}

into
apiVersion: v1
kind: ConfigMap
metadata:
  name: job-{{ $v.name }}
  labels: {{- include "jobs.labels" $ | nindent 4 }}
data:
  {{- with $v.env }}
    {{- toYaml . | nindent 4 }}
  {{- end }}
---
apiVersion: v1
kind: ConfigMap
metadata:
  name: job-{{ $v.name }}
  labels: {{- include "jobs.labels" $ | nindent 4 }}
data:
  {{- with $v.init.env  }}
    {{- toYaml . | nindent 4 }}
  {{- end }}


feat(core): convert "/" in construct id to "--" and disallow tokens

Relax construct ID constraints to allow "/" (path seprator)
to be used in construct IDs, but convert it to "--" so it won't
collide with path strings.
It's quite rare for people to actually try to find a construct
by their ID (it's mostly "write-only") and the logical ID is eventually
mangled anyway when synthesized to CFN.
Fails if the construct ID contains a token. This won't work because
we mangle the IDs as strings when we generate the logical ID
and the construct's unique ID, and stringified tokens won't be
resolved.
Fixes #1351
Fixes #1374

Pull Request Checklist
Please check all boxes, including N/A items:
Testing

[ ] Unit test and/or integration test added
[ ] Toolkit change?: integration tests manually executed (paste output to the PR description)
[ ] Init template change?: coordinated update of integration tests (currently maintained in a private repo).

Documentation

[ ] README: README and/or documentation topic updated
[ ] jsdocs: All public APIs documented

Title and description

[ ] Change type: Title is prefixed with change type:

fix(module): <title> bug fix (patch)
feat(module): <title> feature/capability (minor)
chore(module): <title>  won't appear in changelog
build(module): <title> won't appear in changelog

[ ] Title format: Title uses lower case and doesn't end with a period
[ ] Breaking change?: Last paragraph of description is: BREAKING CHANGE: <describe exactly what changed and how to achieve similar behavior + link to documentation/gist/issue if more details are required>
[ ] References: Indicate issues fixed via: Fixes #xxx or Closes #xxx

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

@RomainMuller fully relaxing the constraints on construct IDs will make it easier to people to use any string that has the "locality" property for the ID without needing to think if this string is a valid ID, and sometimes (like #1351) those IDs are derived from user inputs. #1351 is a good example where @rix0rrr used the CloudWatch metric namespace for the construct ID, but didn't consider the option that the namespace may contain "/"s.
The risk for collision is very low, in which case users will get a very early error and they can address, so I don't think that's a real concern.

To solve @PaulMaddox's toString() issue, do you think we should build in a substring search for 'function()' and reject that as well?
I mean, it's probably not going to be common but it has happened to a knowledgeable individual, and the check for it isn't that hard.

@rix0rrr not sure... People can throw all bunch of garbage into the ID that is not a "good ID", but we can't catch all of that. I can even think of a legitimate use case where function() will be part of a construct ID, so I wouldn't want to start with this cat & mouse chase.

@RomainMuller I don't think we need to confuse people in the documentation of the escape hatch. The docs tell you to do L2.findChild(id) (not "path"), in which case the ID will likely be something like "Resource" or some other name for L1 that is a direct child of the L2. I will add some docs to the logical IDs topic though.


feat: LlookupAllResponseTransformer

What I did
Developed "LlookupAllResponseTransformer" which transforms the llookup:all response
How it works
AtClient._get(Keys.SelfKey) is a great illustration of how LlookupAllResponseTransformer is implemented.
In step 2, we fetch the llookup:all response. This response string looks something like {"key":"...", "data":"...", "metaData":{"createdBy": "...", ...}}
In step 3, we transform the String into a LlookupAllResponse object which is in org.atsign.common.response_models. See this commit to see what the model looks like. The model uses the @JsonProperty tag which comes from jackson's library. The Transformer essentially fills up the Model's fields. Now fields like model.data and model.metaData are defined.
private String _get(SelfKey key) throws AtException {
    // 1. build command
    String command = null;
    LlookupVerbBuilder builder = new LlookupVerbBuilder();
    builder.with(key, LlookupVerbBuilder.Type.ALL);
    command = builder.build();
    // 2. execute command
    Secondary.Response rawResponse = secondary.executeCommand(command, false);
    if (rawResponse.isError) {
        throw new AtException("Failed to " + command + " : " + rawResponse.error);
    }
    // 3. transform the data to a LlookupAllResponse object
    LlookupAllResponse model = null;
    LlookupAllResponseTransformer transformer = new LlookupAllResponseTransformer();
    model = transformer.transform(rawResponse); // contains variables for data we want from the `llookup:all:<fullKeyName>` command (e.g. model.
metaData.ttl)
    // 3. decrypt the value
    String decryptedValue = null;
    String encryptedValue = model.data;
    String selfEncryptionKey = keys.get(KeysUtil.selfEncryptionKeyName);
    try {
        decryptedValue = EncryptionUtil.aesDecryptFromBase64(encryptedValue, selfEncryptionKey);
    } catch (NoSuchPaddingException | NoSuchAlgorithmException | InvalidAlgorithmParameterException | InvalidKeyException | IllegalBlockSizeException |
BadPaddingException | NoSuchProviderException e) {
        throw new AtException("Failed to " + command + " : " + e.getMessage(), e);
    }
    // 4. update metadata. squash the fetchedMetadata with current key.metadata (fetchedMetadata has higher priority)
    Metadata fetchedMetadata = Metadata.fromModel(model.metaData);
    key.metadata = Metadata.squash(fetchedMetadata, key.metadata);
    return decryptedValue;
}

Other Notes

Tests for the Transformer can be found here
Metadata.fromModel creates a Keys.Metadata object for you. See here
Only _get(SelfKey) and _get(PublicKey) use the Transformer for now...
Added comments for the other _delete and _put methods

closes #33


When I try to ad a value to a Cell it prints undefined

Hi,
I'm trying to add some dynamic values to new cells like this
this._worksheet.cell("H7").value(dynVal);
I double check dynVal and I have a value but when I try to see the value inserted I got a 'undefined'
I made sure that dynVal was a string.
Any ideas?
Thanks

For some reason if I add a string to the var it will work :
let newDynVal = dynVal + ' ';
this._worksheet.cell("H7").value(newDynVal);

That's odd. Could you include a full code example that exhibits the issue?


Bump pathval from 1.1.0 to 1.1.1

Bumps pathval from 1.1.0 to 1.1.1.

Release notes
Changelog
Commits

updated-dependencies:

dependency-name: pathval dependency-type: indirect ...

⚠ ALL CHECKOBOXES MUST BE CHECKED BEFORE SUBBMITING A PULL REQUEST ⚠

[ ] I confirm that everything below is true
[ ] I understand that I'm not supposed to put solution to this repository
[ ] I confirm that It is not a solution to the task, but fix in task itself
[ ] In case if it is a solution I agree to get -100 points for the task

запров


The emitted file "script.asset.php" overwrites a previously emitted file of the same name.

When we have two files in different folder with the same name:
"script": ["file:./script.js", "file:./front/script.js"]
We have info after build:
The emitted file "script.asset.php" overwrites a previously emitted file of the same name.

Closed with 1738691f6ecc77578f7058ff1268baa956ec2c8d and release v1.3.2


what is the off-the-shelf detector used to label the synthetic images?

Can the detector that is trained with real + synthetic images achieve comparable performance to the off-the-shelf detector (or improve?) The logic here looks weird.

The off-the-shelf detectors are solely trained on images from a restricted set of base categories, resulting in their limited capacity to detect novel categories. In contrast, InstaGen has the capability to generate images across a wide range of categories and diversity, thus extending the scope of categories that the detector can recognize and enhancing its overall performance.

The off-the-shelf detectors are solely trained on images from a restricted set of base categories, resulting in their limited capacity to detect novel categories. In contrast, InstaGen has the capability to generate images across a wide range of categories and diversity, thus extending the scope of categories that the detector can recognize and enhancing its overall performance.

How do you create labels for the generated images in novel category? If you are using the off-the-shelf detector, isn't it not accurate for the novel category?

The off-the-shelf detectors are solely trained on images from a restricted set of base categories, resulting in their limited capacity to detect novel categories. In contrast, InstaGen has the capability to generate images across a wide range of categories and diversity, thus extending the scope of categories that the detector can recognize and enhancing its overall performance.

How do you create labels for the generated images in novel category? If you are using the off-the-shelf detector, isn't it not accurate for the novel category?

We trained a instance-level grounding head on the base categories, which can accurately predict bounding boxes for novel categories.


[Suggestion] Add a second account to scan the area

Like pokemon Go Map, a second account to scan the area and spot all pokemons, lured pokemons and pokestops, to say to the bot where search

Are you talking about web now?

Think he means something to scout ahead. Seems unnecessary for a bot.

This would actually make a lot of sense, since you could have a second account that could "look" anywhere without the risk of being banned, which populates a database for the bot to read to farm the area.
The best way of doing this would probably be to run PokemonGo-Map and have that populate a db, and have PokemonGo-Bot have the option to read this database to view the area without the risk of teleporting to scan and being banned.

A scout account would be a great way to avoid the main account getting banned for teleport/scan.

@surfaace yes and no, the second account will show all pokemons in the action area of the bot in the web map and telling the bot the location of all pokemons

#1043
Looks like these two are connected. This is might be a "killer feature".
Scan area with PockemonGo-Map and populate DB, then bot can quiery DB for name and location, teleport, wait for possible softban delay and catch poke. All you need is a populate "wanted list" and scan "nice spots".

Ok now here is an interesting find:
I have had a bot running on a dummy account, and the same account being used to populate a map. Interestingly the map scanner is in a different country to the bot worker (map in uk, bot in new york).
Not experienced any soft banning at all in the past 24 hours.
I'm wondering whether the bans only take place if you try to spin / fight / catch in two different areas at once - or even on the same login token.
Saying that, PTC accounts are very easy to come by, so not that relevant.

Good idea, also include an option to get first the more rare pokemons in the neighbourhood and which should last long enough to get there ( walk speed ). Some calculations should be made here but it's not impossible.


Avoiding passing the privateKey around in the code

I have seen in the documentation that there are functions that get the privateKey as an argument for signing (for example https://erebos.js.org/docs/api-bzz#signfeeddigest).
It would be better from a security standpoint to not expose the privateKey to libraries that you are using, because there might be dependencies in the code that can potentially leak it by mistake or by malicious purposes.
It's possible to use for example a function that can be called when your data needs to be signed. You can pass that function instead of the privateKey, and when the underlying code needs to sign the data, it can simply call the function with the data as parameter.
It enables the library user to have more control over what happens when the app requires access to the privateKey, for example it can show a confirmation dialog (for example fingerprint check on mobile) or possibly can use an external wallet API .
What do you think?

That sounds good to me. Let me know if I can help with something.

I'll try to get started on these changes this week, as part of the other feeds-related changes I'm working on, maybe you could review the PR I'll open before merging the branch if you don't mind?


Wrong port endianness on Windows

When I create a service with
op := dnssd.NewRegisterOp(c.Service.Name, c.Service.Type + "," + c.Service.SubType, 8050, RegisterCallback)

and after that start the service, the service will be announced on port 29215 (0x721f), which is a 8050 (0x1F72) converted to LittleEndian.
The same code works correctly when used on Mac OS X.
Is there a way to fix this from my side or does it have to be fixed on your library's side?

Oops. I thought I would have checked that.
I don't have access to a Windows machine at the minute so could you try out the branch atj-winregportfix? Also I've only ever used it on old 32-bit versions of Windows so if you have access to other releases please try it out.

I have tried it on 64 bit, it works :) I don't have access to any 32-bit Windows, sorry :-/

Good to know it works on 64-bit Windows :)
I've merged the fix and added a test. If you use this in something public please let me know.

Reopened as I realised resolve wasn't doing the right thing either. Fixed in e4c847a.


#195 skip tests requiring tensorflow if not found

I have tested this locally with and without tensorflow - if someone would like to double check either scenario that would be nice thanks - all CI has tensorflow installed so that's only 1 use case.


Compaction with MSQ

Adds support for compaction using MSQ.

Description
This PR adds logic to convert the parallel index specs generated by compact tasks into MSQ REPLACE statements.
It then uses the newly added RouterClient to submit these MSQ controllers to /druid/v2/sql/task

Release note

Add the following context to the compact task or auto-compaction config to enable compaction using MSQ.
"context": {
"useMSQCompaction": true
}

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Architecturally it seems weirdly complicated for the compact task to start a new top-level controller task. It will also probably lead to issues with locking.
One of these seems like a better approach to me. What do you think of these approaches?

 It would need to generate a native query rather than a SQL query for this to work, because it doesn't have direct access to the SQL planner.
Forget the compact task, focus on auto-compaction. Have the Coordinator submit SQL statements rather than compact tasks.

Adding a bit to the prior comment: personally I think the second approach, where we don't use compact tasks at all, is best. IMO, the ideal way to do it is something like this:
First, incorporate enough metadata into the metadata store, such that compaction doesn't need to fetch segments from deep storage to figure out what to do. There are a couple of approaches we could take here, including a catalog-based approaches (where metadata is explicitly specified) or a metadata-stashing approach where we save segment row signatures in the metadata store when segments are published. This latter idea would be useful for a bunch of other reasons.
Second, introduce a VACUUM [table] or COMPACT [table] command in SQL. It should take an optional interval and it should leverage the metadata from the first step in order to determine what to do.
Third, have auto-compaction at the Coordinator issue one of these SQL commands.

@AmatyaAvadhanula , I am closing this PR since the approach mentioned above would require patches unrelated to this PR.
Please feel free to reopen  the PR in-case the work is revived.


📝 PPSSPP guide

Proposed changes

Create a small guide for PPSSPP Emulator, should close #14

Should i squash ?

No, when I merge I'll squash. I still need to test this out myself; other than that, it should be ready to merge.


Migrate devicelab tasks a-f to null safety.

Part of #85995.
Migrate the first 3rd of the devicelab tasks to null safety.
This will require the core framework migration #85993 to land first in order for the tests to pass.

But it looks like the analyzer is failing due to importing other non-nullsafe stuff in flutter_devicelab.

Thanks for the review. Yeah, this will need the core framework from the device lab (#85993) to land first and then I will merge it with this before committing.


Code exemple is broken

ViewData["Me"] = me;
Cannot resolve symbol 'me'

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

ID: 15a27a6a-864a-a37e-e3fd-714f01ba7102
Version Independent ID: 4b14dfa1-7c7c-e727-b586-7f0abc9bcd57
Content: Call a web api from a web app - Microsoft identity platform
Content Source: articles/active-directory/develop/scenario-web-app-call-api-call-api.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

Thanks for the heads up @RaymsDev. I've proposed a PR to fix it.

#please-assign jmprieur

#assign-to jmprieur

#assign jmprieur


Cmake version error when trying to run idf.py menuconfig

I installed esp-idf following this guide: https://docs.espressif.com/projects/esp-idf/en/stable/esp32/get-started/
I successfully built and flashed the helloWorld example from the guide.
Now I want to use this repo micro_ros_espidf_component.
I tried to use examples/int32_publisher. So I created a folder inside examples/int32_publisher called components and i cloned this repo into the compononents folder.
I run . $IDF_PATH/export.sh and idf.py menuconfig. This produces an error:
`steffen@steffen-ubuntu:~/esp/int32_publisher$ idf.py menuconfig
Executing action: menuconfig
Running cmake in directory /home/steffen/esp/int32_publisher/build
Executing "cmake -G Ninja -DPYTHON_DEPS_CHECKED=1 -DESP_PLATFORM=1 -DCCACHE_ENABLE=0 /home/steffen/esp/int32_publisher"...
-- Found Git: /usr/bin/git (found version "2.25.1")
-- IDF_TARGET not set, using default target: esp32
-- The C compiler identification is GNU 8.4.0
-- The CXX compiler identification is GNU 8.4.0
-- The ASM compiler identification is GNU
-- Found assembler: /home/steffen/.espressif/tools/xtensa-esp32-elf/esp-2020r3-8.4.0/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /home/steffen/.espressif/tools/xtensa-esp32-elf/esp-2020r3-8.4.0/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /home/steffen/.espressif/tools/xtensa-esp32-elf/esp-2020r3-8.4.0/xtensa-esp32-elf/bin/xtensa-esp32-elf-g++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Project is not inside a git repository, or git repository has no commits; will not use 'git describe' to determine PROJECT_VER.
-- Building ESP-IDF components for target esp32
CMake Error at /home/steffen/esp/esp-idf/tools/cmake/component.cmake:221 (message):
CMake Deprecation Warning at /home/steffen/airwave/CMakeLists.txt:1
(cmake_minimum_required):
Compatibility with CMake < 2.8.12 will be removed from a future version of
CMake.

Update the VERSION argument <min> value or use a ...<max> suffix to tell
CMake that the project does not need compatibility with older versions.

Call Stack (most recent call first):
/home/steffen/esp/esp-idf/tools/cmake/scripts/component_get_requirements.cmake:60 (include)
/home/steffen/esp/esp-idf/tools/cmake/scripts/component_get_requirements.cmake:76 (__component_get_requirements)

CMake Error at /home/steffen/airwave/CMakeLists.txt:4 (project):
project command is not scriptable

Call Stack (most recent call first):
/home/steffen/esp/esp-idf/tools/cmake/scripts/component_get_requirements.cmake:60 (include)
/home/steffen/esp/esp-idf/tools/cmake/scripts/component_get_requirements.cmake:76 (__component_get_requirements)

Call Stack (most recent call first):
/home/steffen/esp/esp-idf/tools/cmake/build.cmake:405 (__component_get_requirements)
/home/steffen/esp/esp-idf/tools/cmake/project.cmake:395 (idf_build_process)
CMakeLists.txt:6 (project)
-- Configuring incomplete, errors occurred!
See also "/home/steffen/esp/int32_publisher/build/CMakeFiles/CMakeOutput.log".
cmake failed with exit code 1
`
It seems like my cmake is outdated?
When i run cmake --version i get:
steffen@steffen-ubuntu:~/esp/int32_publisher$ cmake --version cmake version 3.19.3
How can I solve this? Running idf.py menuconfig when I follow the guide from espressif works fine.

I'm running cmake version 3.16.3, so your issue is more likely to be project layout or component dependencies.
The errors you have suggest that the IDF is forwarding an error from
CMake Error at /home/steffen/airwave/CMakeLists.txt:4
and is suggesting you look at the version requirements in
CMake Deprecation Warning at /home/steffen/airwave/CMakeLists.txt:1
I have no idea what airwave is or why it's getting linked to your micro_ros build

I'm running cmake version 3.16.3, so your issue is more likely to be project layout or component dependencies.
The errors you have suggest that the IDF is forwarding an error from
CMake Error at /home/steffen/airwave/CMakeLists.txt:4
and is suggesting you look at the version requirements in
CMake Deprecation Warning at /home/steffen/airwave/CMakeLists.txt:1
I have no idea what airwave is or why it's getting linked to your micro_ros build

Could you provide steps for replicating this issue?
I just have checked this steps and everything works:
git clone https://github.com/micro-ROS/micro_ros_espidf_component
cd micro_ros_espidf_component
. $IDF_PATH/export.sh
cd examples/int32_publisher
idf.py menuconfig
# Set your micro-ROS configuration and WiFi credentials under micro-ROS Settings
idf.py build
idf.py flash
idf.py monitor

Could you provide steps for replicating this issue?
I just have checked this steps and everything works:
git clone https://github.com/micro-ROS/micro_ros_espidf_component
cd micro_ros_espidf_component
. $IDF_PATH/export.sh
cd examples/int32_publisher
idf.py menuconfig
# Set your micro-ROS configuration and WiFi credentials under micro-ROS Settings
idf.py build
idf.py flash
idf.py monitor

Hello, thank you for the responses. I have not tried to solve this problem any longer. I have followed this guide (https://micro-ros.github.io/docs/tutorials/core/first_application_rtos/freertos/) to create micro-ROS applications on my esp32 which works fine so far.


js webview is loaded multiple times

During startup of the app, the webview is loaded four times. Except for warnings that multiple @polkadot/api and others are detected, I did not see any bad behaviour.
I presume this was introduced with beb07ad054f37bace627a0eb2c29a96838370136

the mentioned commit deliberately loads it twice to make hot-restart work. Not sure why it is loaded four times though

@clangenb is this improved with PR#124?
Meaning: reduced to two times?

It is still observable in the PR.
I suspect this is due to the containing widget being built twice. It is still observable in the PR.
This might be a fix: https://flutterigniter.com/future-async-called-multiple-times/

I believe to have solved the issue in 2aa36291c17d793137dad301c42e6d42c92d1f6a. We still need to launch the webView twice. I currenctly don't see any way to circumvent this if we want to reload the webView with a hot-restart, which I would like to keep.
However, the behavior above, was in fact not the webView being loaded four times. It was related to open js-channels that were not closed during a hot-restart, as the webView was not closed. So, we ended up having multiple instances of the js-api, which were piling up every time a hot-restart was performed.


mobile responsive

Resolves #42
Just makes it more readable in mobile view and added a settings button for the menu to appear. The button settings placement could probably placed in a better position and be somewhat bigger.

Error: HTTP 400 happens when #50 isn't merged (using togetherai backend)

Nice idea, this looks very promising! However, I think these problems need to be solved before it can be merged:

Currently, it only works with the "Monospace Dark" theme.
The right margins are broken.

The bottom bar covers the text.

The text looks way too big.

I had an alterative implementation for the mobile responsive layout. Didn't push to any git repo but it should be simpler to implement.
@media (max-width: 767.98px) {
	body {
		flex-direction: column;
	}
	#prompt-container {
		width:100%;
		max-width: 100%;
	}

	#sidebar {
		width:auto;
		max-height: calc(30vh - 8px);
	}
}

@ihatenumbers Are you still planning to work on this PR?

Nice idea, this looks very promising! However, I think these problems need to be solved before it can be merged:

Currently, it only works with the "Monospace Dark" theme.
The right margins are broken.

The bottom bar covers the text.

The text looks way too big.

I think I ironed out all of those problems. Anyway, when I tried to edit padding in #prompt-overlay, it's being overridden by earlier padding: 2em 3em.

1, 2 and 4 seem fine now. I guess you were talking about 3? One alternative I can think of is dynamically setting a margin to prompt-container according to the height of the sidebar.

1, 2, and 4 seem fine now. I assume you were referring to issue 3, right? One alternative I can think of is dynamically setting a margin-bottom to prompt-container according to the height of the sidebar.
Also, it might be better for the sidebar to start hidden on mobile devices because it might not be intuitive to leave the sidebar. And it looks like if you're on the desktop and hide the sidebar and expand the window, the sidebar remains hidden

Only added some margin bottom to prompt-container and other things, is this fine now? Also, I tried merging with head without edits and it broke

Also, I tried pulling from head without edits and it broke

What's the error?

28px seems to be barely anything here, that's why I suggested dynamically changing the margin-bottom to the height of the sidebar so it would be enough for all fonts/sidebar sizes:
Maybe you could use ResizeObserver for this (example), unless you have a better idea.

took me longer than I'd admit, it's first time interacting with react. Is this fine now?

Also, I tried pulling from head without edits and it broke

What's the error?

POST http://<IP_ADDRESS>:3000/all 404 (Not Found)
Uncaught (in promise) 404

Sounds like there's an issue with the loadAllFromDatabase method of ServerDBAdapter, but that's unfortunately all I can say. The maintainer's gonna have to figure that one out.

Looks like the API SelectBox disappeared:

took me longer than I'd admit, it's first time interacting with react.
I hope the examples made it a bit easier to figure out :)

unhelpful errors.
POST http://<IP_ADDRESS>:3000/all 404 (Not Found)
Uncaught (in promise) 404

Sounds like you're using a web server, but it isn't the mikupad server, or you're using an old version of the mikupad server. I guess I will add a version check and that would fix both cases.

This looks fine to merge now, thank you for your contribution!


Added wheel control

Description
Added two new functions to the PythonAPI in order to change the direction (or rotation) of a vehicle wheel by using an index and also get the steer angle of any wheel of the vehicle.
Fixes #
Where has this been tested?

Platform(s):  Windows
Python version(s): 3.7
Unreal Engine version(s): 4.24

Possible Drawbacks
When the user asks for the steer angle of any vehicle wheel, it returns the steer angle calculated by the PhysX engine.

This change is

The include in EnvironmentObject.h was used because it was giving compile errors, not recognizing a certain class (FBoundingBox). I removed tutorial.py and the .uproject files from the commit.

CHANGELOG.md, line 19 at r7 (raw file):

  * API extensions:
  - Added `set_wheel_steer_direction()` function to change the bone angle of each wheel of a vehicle
	- Added `get_wheel_steer_angle()` function to get the steer angle of a vehicle whee

Fixed conflict with Changelog

CHANGELOG.md, line 19 at r7 (raw file):
Previously, bernatx (bernat) wrote…
Is there a TAB code?

Done.

Hello @algababr, @bernatx, @doterop & @DSantosO,
I want to apply individually torque to the wheels.
Unfortunately it doesnt work the way I did hope for, as seen in my Post #5215 .
Now I want to use the Function SetBrakeTorque from USimpleWheeledVehicleMovementComponent.
Are you able to possibly add this Function as well, or could describe to me how you have done it?
Currently I am using a Debian Package install of Carla (no build from Source Code).
Thank you very much :)


how to calculate the L1 Relative Error

Great work, but could you open-source how to calculate the L1 Relative Error?

Stay tuned for more updates: we will have some additional examples on how you can generate/calculate these schedules using the L1 Relative Error.


edit the name of the policy

#6584

Dear @JohanFreelancer9 ,
could you please review?
Thank you

@JohanFreelancer9 . Sir how can i see PR is  approved


Update Skia version to remove vulnerability

This PR updates following Skia version to remove security vulnerability.
  <ItemGroup>
    <PackageReference Include="Microsoft.Maui.Graphics" Version="$(MicrosoftMauiGraphicsVersion)" />
    <PackageReference Include="Microsoft.Maui.Graphics.Skia" Version="$(MicrosoftMauiGraphicsVersion)" />
  </ItemGroup>

Background
(https://github.com/dotnet/maui/pull/17558)
About 3 weeks ago, security vulnerability is detected for SkiaSharp.Views and any project referencing this package is warned for the security vulnerability.
The vulnerability is patched in SkiaSharp v2.88.6 and Microsoft.Maui.Graphics is now released with it on v8.0.0-rc.2.9373.
What to do next?

Release Tizen.UIExtensions (eg. v0.9.1)
Update Tizen.UIExtensions on MAUI.
Remove SkiaSharp.Views direct reference from CommunityToolkit.Maui which was the workaround for Tizen.

@rookiejava Absolutely! Issue is fixed on https://github.com/Samsung/Tizen.UIExtensions/pull/25.
I will rebase the PR when it is merged.

@rookiejava Absolutely! Issue is fixed on #25. I will rebase the PR when it is merged.

#25 has been merged.

SkiaSharp.Views direct reference is added because net6.0 Microsoft.Maui.Graphics is not released anymore.


Travis + Shellcheck

Attention The README.md contains links to Travis to include the status info. This links must be changed to user zMarch and to the master branch.
The free service https://travis-ci.org/ will run the ShellCheck program. A view steps are needed (login and activate) in Travis to enable the free service.

Some compatibility optimization inspired by shellcheck hints are also included in this pull request.
(My first plan was to separate the two items. But then I used the wrong branch.)

Fantastic changes


Fix available-at behavior for artifact only dependencies

When a first level dependency is marked non transitive and contains an
artifact reference, we ignore the available-at from Gradle Module
Metadata.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.


(SQUASH PLS) Include MAJOR change details in PR template

Category
What kind of change is this?

[x] Cosmetic change.
[ ] Documentation change by member.
[ ] Documentation change by Overture tech writer.
[ ] Material change.

Description
Brief description of the business purpose and effect of the pull request.
This change updates the pull request template to align with the schema versioning and stability wiki.
Reference
List of relevant links to GitHub issues, PRs, and other documentation.

Schema versioning and stability wiki on Overture Confluence

Testing
Brief description of the testing done for this change showing why you are confident it works as expected and does not introduce regressions. Provide sample output data where appropriate.
N/A - Not really testable.
Checklist
Checklist of tasks commonly-associated with schema pull requests. Please review the relevant checklists and ensure you do all the tasks that are required for the change you made.

[ ] Add relevant examples.
[ ] Add relevant counterexamples.
[ ] Update any counterexamples that became obsolete. For example, if a counterexample uses property A but is not intended to test property A's validity, and you made a schema change that invalidates property A in that counterexample, fix the counterexample to align it with your schema change.
[ ] Update in-schema documentation using plain English written in complete sentences, if an update is required.
[ ] Update Docusaurus documentation, if an update is required.
[ ] Review change with Overture technical writer to ensure any advanced documentation needs will be taken care of, unless the change is trivial and would not affect the documentation.

Documentation Website
Update the hyperlink below to put the pull request number in.
Docs preview for this PR.

Merging based on criteria for a cosmetic change: approval from 1 other steering member + 2 business days notice.


I was trying to get any valid cookies given in HTTP response header specifically via Set-Cookie header entry. https://go.dev/play/p/M8hiEzF_n97
What did you see happen?
In the example code, the Cookies method exclude utid because it has quotation mark chars inside cookie-value. Looking at the RFC in the comment, https://datatracker.ietf.org/doc/html/rfc6265#section-4.1.1
Quotation mark " OR in hex 0x22 is excluded, which is already expected behaviour:
 set-cookie-header = "Set-Cookie:" SP set-cookie-string
 set-cookie-string = cookie-pair *( ";" SP cookie-av )
 cookie-pair       = cookie-name "=" cookie-value
 cookie-name       = token
 cookie-value      = *cookie-octet / ( DQUOTE *cookie-octet DQUOTE )
 cookie-octet      = %x21 / %x23-2B / %x2D-3A / %x3C-5B / %x5D-7E
                       ; US-ASCII characters excluding CTLs,
                       ; whitespace DQUOTE, comma, semicolon,
                       ; and backslash

However, for web browsers that behaviour is totally fine.
What did you expect to see?
Interface for custom validation for valid cookies

(attn @neild)

Just adding an extra context here of what Chromium did, is basically allowing " and some others:

Expand

  // IsValidCookieValue() returns whether a string matches the following
  // grammar:
  //
  // cookie-value       = *cookie-value-octet
  // cookie-value-octet = %x20-3A / %x3C-7E / %x80-FF
  //                       ; octets excluding CTLs and ";"
  //
  // This can be used to determine whether cookie values contain any invalid
  // characters.
  //
  // Note that RFC6265bis section 4.1.1 suggests a stricter grammar for
  // parsing cookie values, but we choose to allow a wider range of characters
  // than what's allowed by that grammar (while still conforming to the
  // requirements of the parsing algorithm defined in section 5.2).
  //
  // For reference, see:
  //  - https://crbug.com/238041

Effectivly changing the grammar to:
cookie-value       = *cookie-value-octet
cookie-value-octet = %x20-3A / %x3C-7E / %x80-FF
                       ; octets excluding CTLs and ";"

https://chromium.googlesource.com/chromium/src/+/refs/heads/main/net/cookies/parsed_cookie.cc
a reference to the chromium implementation

Is this a duplicate of https://github.com/golang/go/issues/46443 ?

Is this a duplicate of #46443 ?

No, this one is expecting an interface for cookie validation for cookie-value grammar customability.

If I understand correctly this issue, Go also doesn't support any non-ascii values in the cookie value. E.g:
package main

import (
	"fmt"
	"net/http"
)

func main() {
	var resp http.Response

	resp.Header = map[string][]string{
		"Set-Cookie": {
			"emoji=å∫∂ƒ§πø¥†®€æ; path=/; secure",
			"text=hello world!; path=/; secure",
		},
	}

	for _, c := range resp.Cookies() {
		fmt.Printf("%+#v\n", c)
	}
}

will print only the second cookie.

I have experienced similar problem with backslashes. The backslashes are not allowed in cookie-value but web browser has no problem with it. You can check this behavior accessing the url httpbin-url-test wich sets a cookie value using backslases to scape unicode characters.  It will be nice if a custom validator for cookie values could be implemented.
Related code:
https://github.com/golang/go/blob/30b6fd60a63c738c2736e83b6a6886a032e6f269/src/net/http/cookie.go#L472


To get REAL center in graph.zoom you need to use window.innerWidth and window.innerHeight NOT graph.getViewPortCenterPoint()

Describe the bug
To get REAL center in graph.zoom you need to use window.innerWidth and window.innerHeight NOT graph.getViewPortCenterPoint()
graph.zoom(1.2, graph.getViewPortCenterPoint()) - WRONG
This call will zoom with move with offset
graph.zoom(1.2, {x: window.innerWidth / 2, y: window.innerHeight / 2} ) - CORRECT
This call will zoom right at the center.
Please state this fact in the documentation
Your Example Website or App

Steps to Reproduce the Bug or Issue

Create graph that takes up only part of the screen
Call graph.zoom(1.2, graph.getViewPortCenterPoint())

Expected behavior
As a user, I expected that graph.zoom(1.2, graph.getViewPortCenterPoint()) will zoom at the center of the camera view but i am seeing strange offset
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome

Additional context
No response

The implementation of getViewPortCenterPoint is:

Which means the API returns the point in rendering coordinate system for the center of canvas DOM. If the graph has initial zoom (1) and not translation, the point of rendering coordinate system is equal to the center point of canvas DOM in number. But after zooming and translating (maybe caused by fitView, fitCenter, mouse wheel, etc.), they are different. Refer to the doc:  https://g6.antv.antgroup.com/en/manual/advanced/coordinate-system , where the rendering coordinate system I mentioned before is the pointX/pointY coordinate system in the doc.


Upload source code on the repository itself

The add-on code is only contained within a ZIP on GitHub Releases right now. Could you also push the source code to the repository itself? This would make installation via git clone easier. Thanks in advance :slightly_smiling_face:
See https://github.com/Calinou/godot-addon-template for an example structure you can use.

The add-on code is only contained within a ZIP on GitHub Releases right now. Could you also push the source code to the repository itself? This would make installation via git clone easier. Thanks in advance 🙂
See https://github.com/Calinou/godot-addon-template for an example structure you can use.

thanks for warning calinou, i will provide the source code now


Handle MsgSnapshot sent before MsgAppend

What is changed and how it works?

Issue Number: Close #xxx
According to https://github.com/tikv/raft-rs/blob/2aefbf627f243dd261b7585ef1250d32efd9dfe7/src/raft.rs#L842,
if log is truncated in Leader, a MsgSnapshot may be sent directly before a MsgAppend.
If such MsgSnapshot is received when a FAP snapshot IS BUILDING, then it will be dropped.

What's Changed:

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

/retest


Fix script name - by README.md

Thanks for the suggestion. However, I'd rather change it in Readme :)


Chat icon on Windows 7 with tooltips

Tested this working on Windows 7 with tooltips :+1:
I do have one suggestion to change the chat-like :speech_balloon: icon that appears below the tooltip tip on W7 to the Atom logo. According the docs :memo: from Notifu this is possible.

@jerone Yep, and this should work, however, 512x512, probably, too big for Win7 tray area.
I made 16x16 icon for this and a little check for Windows NT 6.1.
Can you, please, verify if it works? If it does, probably 32x32 icon would work too.
Pushed 0.2.3 release few moments ago.

No success. Installed latest version (v0.2.3) and still have the same icon as screenshot above.

Seems related to the first issue mentioned here: https://github.com/mikaelbr/node-notifier/issues/38
A very small comment here indicates that icons for balloons aren't supported in the Cross-Platform code:

icon: path.join(__dirname, 'coulson.jpg'), // absolute path (not balloons)

So after some investigation it seems that Notifu only supports bitmaps, cursors and icons.
One solution is to use the appIcon property to differentiate from the normal icon property, and use an bitmap or .ico file.

@jerone So, reverting the changes for now.
I'll investigate behavior of balloon in Electron tomorrow. Looks like there is some NativeImage implementation for that. If this is what I think it is, I'll make a separate notifier for Windows 7.

@jerone Interesting. The option would still be called icon, as I understood. I guess, we can try to use .ico file there.
Published 0.2.5 patch.

Just tested. Seems fine with 16x16 .ico. :+1:
32x32 size and any .bmp failed, btw.

@jerone Thank you for your great support!

Just had the time to confirm the fix. Great work :+1

This issue appeared in latest (v0.4.0) version again  :(

@jerone Oops, forgot about proper path for .ico. Should be fixed in 0.4.1.

Yeah confirmed working again.


podman exec with heredoc results in read: connection reset by peer

/kind bug
Description
Recently I noticed our CI test failing when exec-ing into a running postgresql container and feeding it input using heredoc.
Steps to reproduce the issue:

podman run -e POSTGRESQL_ADMIN_PASSWORD=test -d --name postgresql registry.access.redhat.com/rhscl/postgresql-10-rhel7

podman exec -i postgresql cat <<EOF
testing input
EOF

Describe the results you received:
Error: read unixpacket @->/var/run/libpod/socket/c32a489545c3322cc24752af4af08df221aa2d2fa3e25a5affbd7dfe38bef8b6/attach: read: connection reset by peer
Describe the results you expected:
testing input
Additional information you deem important (e.g. issue happens only occasionally):
Container run as root.
When I hit it in the postgresql CI tests (feeding data into the db) even the error message did not get printed, the command succeeded, but data was not in the container.
Note that this does work with docker:
# docker-binary run -e POSTGRESQL_ADMIN_PASSWORD=test -d --name postgresql rhscl/postgresql-10-rhel7
904d31577c8bde39239cefcef357225bf561847b3a72944738d87bd46c8b701e
# docker-binary exec -i postgresql cat <<EOF
testing input
EOF
testing input

Output of podman version:
Version:            1.6.2
RemoteAPI Version:  1
Go Version:         go1.12.10
OS/Arch:            linux/amd64

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.12.10
  podman version: 1.6.2
host:
  BuildahVersion: 1.11.3
  CgroupVersion: v1
  Conmon:
    package: conmon-2.0.2-1.fc30.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.2, commit: a89d21975ee86e84e0b0e1c0f887687582f4b0e3'
  Distribution:
    distribution: fedora
    version: "30"
  IDMappings:
    gidmap:
    - container_id: 0
      host_id: 21722
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 21722
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
  MemFree:<PHONE_NUMBER>2
  MemTotal:<PHONE_NUMBER>8
  OCIRuntime:
    name: runc
    package: runc-1.0.0-95.rc9.gitc1485a1.fc30.x86_64
    path: /usr/bin/runc
    version: |-
      runc version 1.0.0-rc9+dev
      commit: 0840802d505e28d3adcbc7a275cbeadbdf64ddec
      spec: 1.0.1-dev
  SwapFree:<PHONE_NUMBER>6
  SwapTotal:<PHONE_NUMBER>6
  arch: amd64
  cpus: 8
  eventlogger: journald
  hostname: localhost.localdomain
  kernel: 5.3.7-200.fc30.x86_64
  os: linux
  rootless: true
  slirp4netns:
    Executable: /usr/bin/slirp4netns
    Package: slirp4netns-0.4.0-4.git19d199a.fc30.x86_64
    Version: |-
      slirp4netns version 0.4.0-beta.2
      commit: 19d199a6ca424fcf9516320a327cedad85cf4dfb
  uptime: 6h 40m 42.85s (Approximately 0.25 days)
registries:
  blocked: null
  search:
  - docker.io
  - registry.fedoraproject.org
  - quay.io
  - registry.access.redhat.com
  - registry.centos.org
store:
  ConfigFile: /home/pkubat/.config/containers/storage.conf
  ContainerStore:
    number: 12
  GraphDriverName: overlay
  GraphOptions:
    overlay.mount_program:
      Executable: /usr/bin/fuse-overlayfs
      Package: fuse-overlayfs-0.6.5-2.fc30.x86_64
      Version: |-
        fusermount3 version: 3.6.2
        fuse-overlayfs: version 0.6.5
        FUSE library version 3.6.2
        using FUSE kernel interface version 7.29
  GraphRoot: /home/pkubat/.local/share/containers/storage
  GraphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 23
  RunRoot: /run/user/21722
  VolumePath: /home/pkubat/.local/share/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
podman-1.6.2-2.fc30.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):

Ah sorry, I see its probably a dupe of  #4326. Should have searched before posting :/

Thanks a lot for reaching out, @pkubatrh! I also think that it's a dupe of #4326 which will be fixed with Podman v1.6.3 which will reach you soon on Fedora.

@vrothberg How do you know in which version it's going to be fixed? it's not fixed in master branch yet.

My bad. It's not yet fixed entirely. Thanks for pointing that out!


Test hangs on failure...

I have this task:
gulp.task('e2e',['start-rest','start-server','selenium'], function (done) {
     gulp.src('wdio.conf.js')
        .pipe(webdriver())
        .on('end', function(err){
            console.log(">>2>>"+err);
            done(err);
        })
        .on('error', function(err){
            console.log(">>>>"+err);
            done(err);
        });
});

It works fine when the tests pass, but hangs on failure:
0 passing (17.70s)
1 failing

1) Basic Test Should be able to login ..:
Failed: element (nsa-content) still not existing after 10000ms
running chrome
Error: element (nsa-content) still not existing after 10000ms
    at Page.value (test/e2e/spec/common/MainAssurancePage.js:27:18)
    at Object.<anonymous> (test/e2e/spec/testSpec.js:27:27)
    at elements("nsa-content") - isExisting.js:46:17
    at isExisting("nsa-content") - waitForExist.js:37:22

Wrote xunit report to [./target/test-res].
>>>>Error in plugin 'gulp-webdriver'
Message:
    wdio exited with code 1
[11:42:50] 'e2e' errored after 18 s
[11:42:50] Error in plugin 'gulp-webdriver'
Message:
    wdio exited with code 1

The versions of the plugins etc are:
<EMAIL_ADDRESS><EMAIL_ADDRESS>Any suggestions?

Should be fixed with latest version


Add Jekyll Feature

If you like~

Preview here: https://imgur.com/a/uudmMvV

Almost ok
LGTM👍

Why do you revert commit?

Ahh, I was trying it on another Repo before committing here,
so I purged it and force push it because it had a configuration left.

Ok. arogato

I configure it to GitHub Pages temporarily.
After confirmation, merge it into master.
https://wsldl-pg.github.io/docs

Yeah, it's better to check and then merge


Following the license change, modify the boilerplates in crypto/dh/

Merged.
e38873f5ce93b41929d8128a533cca7d9e3f0f5a Following the license change, modify the boilerplates in crypto/dh/


[Communication] - Azure.Communication.NetworkTraversal - Adding the ACS NetworkTraversal SDK

Communication Traversal/TURN.
Creating PR for Azure Communication Services - Relay Client (TURN) private preview.

/azp run js - core - ci


Render wheeled units

Quads, Trikes.
Acceptance criteria:

[ ] Unit is drawn on map
[ ] Unit is drawn within viewport
[ ] Unit has several facings (randomly shown)

@arjenvanderende looks like this is the first thing to do. So I'll start with it (will push branch soon)

Fixed in #42


#12196: Use split readers wherever possible in UNet Shallow

Ticket
Addresses all of the following:

#12196
#12195

What's changed

Enable split reader and activation double buffering on some convolutions
Fuse output layer bias with matmul
Use ttnn.reshard instead of ttnn.sharded_to_interleaved + ttnn.interleaved_to_sharded in bottleneck layer
Device performance is now 440 fps (including initial input sharding)
Adds device and end-to-end performance tests to CI

Checklist

[ ] Post commit CI passes
[ ] Model regression CI testing passes (if applicable)

any need to bump timeout w/ extra tests here?


Fixed typos in B-hooks.md

changed file extension of App and SearchParams from .js to .jsx

Thank you!


Connect issue, added ConnectWithContext to allow a context to be passed in, additional ChannelData and Bridge

Connect could become stuck if the listen did not successfully connect to the websocket. listen needed to call wg.Done if the context was cancelled.
Modified the native client to include a ConnectWithContext function. This allows the caller to pass in their context. The Connect then uses this Context for the cancel. If the caller/parent cancels the Context, the child Connect routine detects the Context is closed and cleans up. This allows for a more complex environment with multiple go routines running under a parent/client model.
ChannelData additions - Added ProtocolID to the ChannelData. For chan_sip and PJSIP this is the SIP header Call-ID value. Language was in the ChannelData protobuf, but was missing from the channel.go so applications could not retrieve the Language value.
Added AppName and AppData to the DialplanCEP. This allows the application to know the current dialplan application and the parameters (data) passed to the application. If ARI disconnects or is starting up, an application should retrieve a list of it's current channels. If the channels have stasis application name, we know we are in control of the call via ARI. The AppData field allows us to retrieve a list of any parameters that were passed to the stasis application. This is very important because Asterisk will keep the call in stasis control once it has done this. Without retrieving a list of channels, we will have calls stuck in Stasis control. PJSIP and chan_sip calls will be disconnected. However, local channels or ExternalMedia channels could become permanently stuck.
Updated bridge.go to include the CreationTime, VideoMode, and VideoSourceID fields that Asterisk has included for some time.
Updated the CONTRIBUTORS

This change is

There are good features in here, but this PR also has a bunch of package renames.

Probably best to make a clean fork, if you can do so without too much trouble.


are sections working properly in the example?

Downloaded source and just hit build.
Love the project tho, kudos :)

they are. if you go to the storyboard and set the tableview style to grouped, you'll be able to see the section action.


Election rake task to publish policy papers.

[ ] Code signoff
[x] Product signoff

These policy papers were generated from existing
policies to give them a historical presence on
GOV.UK.
This task is a simplified version of the final task,
adequate for our mock elections.
It:

force publishes the policy paper
publishes as a minor change
retains the first_published_at from the draft
sets the major change timestamp to match the first
published timestamp

Still to come:

Adding a change note
Setting dates appropriately for ordering the
content

https://trello.com/c/TnkdXGhp/214-0-5-create-task-to-publish-policy-publications

:+1:


Need way of indicating reason for people not showing up on check-in page

Currently on the check-in page, we can only say whether or not someone showed up. Would be useful to have a button that opened up a text field where we could indicate the reason for them not showing. That way, we can if people have a reason for not showing up, and how many are just no-shows

I like this idea.

@MVanderwood Add a notes field.


Used with Debian aarch64 gets wrong node tar URL

It tries to download node from https://nodejs.org/dist/v4.0.0/node-v4.0.0-linux-aarch64.tar.gz instead of https://nodejs.org/dist/v4.0.0/node-v4.0.0-linux-arm64.tar.gz , generating a   "gzip: stdin: not in gzip format" error

What does nvm debug and nvm_get_arch output?

nvm debug

$SHELL: /bin/bash
$NVM_DIR: '/usr/local/nvm'
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
nvm current: none
which node:
which iojs:
which npm:
npm config get prefix: $NVM_DIR/nvm.sh: line 1486: npm: command not found
npm root -g: $NVM_DIR/nvm.sh: line 1486: npm: command not found

nvm_get_arch

aarch64

In fact I think, this last output is the key of the problem, because node dist package is labeled as arm64. It's like line 1472 is not effective and is taking NVM_ARCH="$HOST_ARCH" instead
Also, I have to say that I'm running that on docker and it can be affecting in some way, but as you seen nvm_get_arch seems correct to have expected output.

hm, can you also provide the output of echo "|$(nvm_get_os)|" and echo "|$(command uname -m)|"?

echo "|$(nvm_get_os)|"

|linux|

echo "|$(command uname -m)|"

|aarch64|

OK - then can you provide the full output (in a gist, because it will be large) of set -x; nvm install 4.0.0?

https://gist.github.com/adomenech73/af5213a1a19f0f43ca304f745eb7c38a

What does nvm --version output? I think perhaps you want to update to the latest, and that will fix your problem.

Totally true, I didn't realize I had v0.29.0 harcoded on Dockerfile
Now it's installing just fine !!


turses: Fix build

Motivation for this change
ZHF #80379
Needs backport
Things done

[x] Built successfully on NixOS

Backported to 20.03 in 93626f5cd09eaf7dcae28e08b05acdc6248e42a6


[stable/graphite] Upgrading Graphite version to 1.1.4-3

Signed-off-by: deniszh<EMAIL_ADDRESS>
What this PR does / why we need it:
Graphite docker image 1.1.4-3 contains an important fix for Kubernetes - please see https://github.com/graphite-project/docker-graphite-statsd/releases/tag/v1.1.4-3
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
https://github.com/graphite-project/docker-graphite-statsd/issues/46
Special notes for your reviewer:
/cc @fabian-schlegel @monotek

/ok-to-test
/assign

/lgtm


cant login after the 1.9.2

keep getting the internal server error (500), and when i try to login offline i get this "[Error] Downloading stash for Standard, details logged to DebugInfo.log, please open a ticket at https://github.com/Stickymaddness/Procurement/issues"

Make sure that you have the account name field filled in, note that it is case sensitive!

Closing due to inactivity, please open a new ticket if this is still an issue.
To clarify the account name field:


It pauses the video in the background after 60 secs

In the new Firefox version 79.0.5 it pauses the video after 60 secs or less while playing in the background

Without this extension, I will use Edge or Chrome.

Change to Beta version or Nigthly Version, enter to about:config and change "dom.suspend_inactive.enabled" to false.

Change to Beta version or Nigthly Version, enter to about:config and change "dom.suspend_inactive.enabled" to false.

Yeah saw it on reddit as well, using Firefox beta for a week now.

Without this extension, I will use Edge or Chrome.

Why not browsers that include this like Brave or Bromite?

I switched now to Brave. Solved the Problem.

I think the original Firefox issues have since been resolved.

I think the original Firefox issues have since been resolved.


python3Packages.open-garage: init at 0.1.5

Motivation for this change
Python module to communicate with opengarage.io
https://github.com/Danielhiversen/pyOpenGarage
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 132821 run on x86_64-linux 1

2 packages built:

python38Packages.open-garage
python39Packages.open-garage

Result of nixpkgs-review pr 132821 at 5af9f944 run on aarch64-linux 1

2 packages built successfully:

python38Packages.open-garage
python39Packages.open-garage


Timeouts and deadlocked threads

My Environment

ArangoDB Version:        3.3.2
Storage Engine:             RocksDB
Deployment Mode:       Single Server
Deployment Strategy:   Windows service
Configuration:               Physical, 32-core (16 physical)
Infrastructure:               own
Operating System:        Windows Server 2012 R2
Used Package:              Windows installer

Component, Query & Data
Affected feature:
Using the HTTP REST interface directly, no driver.
Document collections only.
Steps to reproduce
Not simple to immediately reproduce

Create collections (say ~1000)
Populate with data (500 - 200,000 documents each)
Delete collections
Goto 1 until requests start timing out

Problem:
After the server has been running for sometime (initially over a month) we start to see some requests (POST, PUT, GET) timeout.
Investigating we noticed the ArangoDB server had a very large number of threads running (> 25,000).
A wait chain analysis suggested that a great many of these threads were deadlocked.
We restarted the server. After a few days timeouts started to happen again and we noticed that the number of threads had grown to above 1,500. Again these looked deadlocked.
Expected result:
No timeouts and no deadlocked threads.
We are upgrading to ArangoDB 3.3.19.
Will report if the problem is reproducible in that version as well.

Hi @not-sungard, thanks for the report. I have a few questions:

Are the collections created in parallel? If so, how many? All at once?
Do you first create all 1k collections and then start to populate them, or do you create collections and insert documents in an interleaving fashion?
Is the request to delete a collection issued while documents are still being inserted into this very collection, or is the collection possibly still busy (disk i/o, indexes etc.)?
Are new collections created with previously used names?

Created in parallel, but only a few at a time
Create collection then start to insert
Deletion is done much later as part of house keeping. Older data that isn't needed any more.
They shouldn't be, we are using UUIDs for collection names.

Our environment is a smallish grid, with a queue of "jobs" to run. The number that can run at any one time is constrained by the number of cores. Number of jobs at any moment in range 2 - 10. Each job creates and writes to its own collection.

@not-sungard can you create a minidump of the deadlocked process using one of the methods described here https://github.com/arangodb/arangodb/blob/devel/README_maintainers.md#windows-debugging ?

@not-sungard In addition to request for minidump by mpoeter , can you also share if this problem is still also seen in version 3.3.19 or 3.4 RC4? (and is occuring only on Windows OS?)

I have now seen the deadlock issue on my developer workstation.
I have attached a screenshot:

I also have a process dump of arangod.exe. Zipped this is ~208MB and too large to attach.
My environment:
ArangoDB Version: 3.3.19
Storage Engine: RocksDB
Deployment Mode: Single Server
Deployment Strategy: Windows service
Configuration: Physical, 24-core (12 physical)
Infrastructure: own
Operating System: Windows 10 Enterprise
Used Package: Windows installer

@not-sungard Did you close the issue on purpose or accidentally?

Oops! Accidentally!

Re-opening

Hi,
could you inspect the dump like this:
cdb -z ./a.dmp -y 'SRV*e:\symbol_cache*https://download.arangodb.com/symsrv_arangodb33;SRV*e:\symbol_cache\ms*http://msdl.microsoft'
and in cdb you may invoke it printing backtraces like this:
kp (for the backtrace of the current thread)
~*kb (for the backtraces of all threads)

in general it would be good if you could share the dump file with us via i.e. dropbox.

@not-sungard : Looking forward to receiving backtraces for the deadlocked process...
In the meantime, can you confirm that the storage engine in use is actually RockDB (asking because the default storage engine in the 3.3 branch is MMFiles), if the server still shows a very high number of threads, and if there is anything suspicious in the arangod.log log file?
Thanks!

Yes, RocksDB.
I have sent a link to a secure download the dump file via another channel.
There is nothing suspicious in the arangod.log file.

Ok, thanks. When the issue occurred last, do you know if the number of threads again was very high, or if it has been in the normal range?

Yes, on previous occasions we saw the timeouts the thread count was also very high.
I suspect that the reason for the timeout is that the communication thread gets deadlocked.

windows-dump.log

According to the dump here it seems that many threads (1955) are hanging in shutdown (~Thread()) and cannot make any progress. This also explains why the number of threads is very high.
Root cause still unknown, investigating further.

We have an idea what is causing this problem, and are working on a fix right now.

Fixed in 3.3.21

That's great news. Thanks to all.

Hi @not-sungard
I would like to inform you that we have released ArangoDB Version 3.3.21 that will hopefully fix your issue.
Please give it a try and let us know.
Best,

Hi OmarAyo,
Thanks. Could you confirm that this fix is also in 3.4.1?
All the best.

Hi @not-sungard,
yes, the fix is also included in 3.4.1, introduced by commit 9dbbd1cc9f3423c7e622b36a7b45dc10a6a9732d.

That's great, we'll get on with our testing.
Thanks.


Image classification: could not broadcast input array from shape into shape

Receiving this error trying to run a simple image classification example
  File "/Users/colinpetit/.pyenv/versions/3.6.5/lib/python3.6/site-packages/ludwig/features/image_feature.py", line 101, in add_feature_data
    data[feature['name']][i, :, :, :] = img
ValueError: could not broadcast input array from shape (743,620,3) into shape (383,442,4)

model_definition_file:
input_features:
  -
    name: img
    type: image
    encoder: stacked_cnn
output_features:
  -
    name: type
    type: category
training:
  epochs: 10

I have to specify this better in the docs: Either your images are all of the same size, or you have to resize them to the same size by specifying the resize_method, width and height parameters as explained here. Let me know if this solves your issue.

I've set a resize as follows
input_features:
    -
        name: image_path
        type: image
        encoder: stacked_cnn
        resize_method: crop_or_pad
        height: 500
        width: 500

output_features:
    -
        name: class
        type: category

However I still get an error.
ValueError: could not broadcast input array from shape (500,500,4) into shape (500,500,3)

Interesting, that 4 as the last dimension of your image, may mean images with an alpha channel, something I may have not considered. Can you send me an image form you dataset that has a fourth channel so that I can reproduce the issue?
@ydudin3 could you please try to take a look at this too?
Another remark: I would suggest not exceeding 224x224 as a image size, as othersize model training would require a lot of memory and it will be really slow. @ydudin3 maybe we can add something about it in the documentation, plus moving all parameters from the feature itself to preprocessing as it happens for other features and impose a defaul maximum size, so that if the dimension of images is above that size images are resized users get a warning printed on the console.

It is the Stanford dog breeds data set, I couldn't find the one or two with an alpha channel but I think that was the issue.
I just removed the alpha channel with this.
And yes, size was far too big! 112x112 was getting quick results.

Thanks @mattarderne for confirming it, will work on a patch that solves this issue, it will be pretty straightforward.

Hello,  @w4nderlust
Can I update the docs image-preprocessing  with your suggestion

Ludwig supports grayscale and color images, image type is automatically inferred. During preprocessing raw image files are transformed into numpy ndarrays and saved in the hdf5 format. Images should have the same size (keep the image size under 224x224 pixels). If they have different sizes they should be converted to the same size by specifing the resize_method, width,  and height parameters of image feature preprocessing.

Thanks for collecting this. Wait a bit as we are going to change a few things, including the support for images with alpha channels.

So all parameters have been moved to preprocessing, #141 . Now there is a PR for dealing with the 4th channel and image sizes too. Stay tuned.

I added a temporary workaround for images with a 4th channel. We will improve over this solution (which basically just ignores the alpha channel), but at the moment should fix this issue. @mattarderne and @ctpmp can you please confirm that with the code on master this issue is solved?


Migrate to NetStandard 1.4 for Xamarin support

Done

Removed support for various outdated framework targets that were not actually supported by the generated code.


Usage example?

I'm trying to use piral-cli-dotenv in a Piral app. I put my variables in .env.development and run piral debug --env development. In my app, I try console.log(process.env.MY_VAR) but it's undefined. Is there a step I'm missing?

What bundler are you using? I think this plugin only works with Parcel.

I'm using piral-cli-webpack. Is there a recommended way to use environment variables with webpack?

A combination of EnvironmentPlugin and plain dotenv could just work fine.
Best case is that I find a reliable way of supporting all / multiple bundlers here.

On second thoughts according to #3 it also works with Webpack. Maybe there is some other issue?
What exactly are you trying to do and how are you calling it (I assume you call, e.g., piral debug --env, right?)? Thanks @danbockapps

I have version 0.13.0 of both.

I have worked around this by uninstalling piral-cli-dotenv and creating a webpack.config.js file in the root of my piral project:
const path = require('path');
const webpack = require('webpack');
const dotenv = require('dotenv').config({
  path: path.join(__dirname, '.env.development'), // TODO also handle prod
});

module.exports = config => ({
  ...config,
  plugins: config.plugins.concat(
    new webpack.DefinePlugin({ 'process.env': JSON.stringify(dotenv.parsed) }),
  ),
});

Now I can refer to process.env in my TypeScript code to access the values from .env.development.

Actually, I've updated the plugin. It would have worked with  --env .env.development, but after the update (version is 0.2.0) it should also work as expected. Sorry for the inconvenience!


paginator and trail don't produce the same URLs

Config:
pagination:
  enabled: true
  per_page: 10
  permalink: '/page/:num'
  title: ':title - page :num'
  trail:
    before: 2
    after: 2

Input:
---
pagination:
  enabled: true
  category: blog
---
{% if paginator.total_pages > 1 %}

    {% if paginator.previous_page %}
      Previous
    {% endif %}
    {% if paginator.page_trail %}
      {% for trail in paginator.page_trail %}
        {% if trail.num == paginator.page %}
          {{ trail.num }}
        {% else %}
          {{ trail.num }}
        {% endif %}
      {% endfor %}
    {% endif %}
    {% if paginator.next_page %}
      Next
    {% endif %}

{% endif %}

Output:

  1
  2
  3
  4
  Next

You can see "Next" points to /blog/page/2 like the permalink would suggest but the trails point to /blog/page/2index. Additionally, /blog/page/2 404's.

Okay, I asked because from the source code below, it seems like extensionless urls were never part of the blueprint:
https://github.com/sverrirs/jekyll-paginate-v2/blob/c657c5f8e02580a5e13f336d5c34fd4c81e212f7/lib/jekyll-paginate-v2/generator/paginator.rb#L30-L33
https://github.com/sverrirs/jekyll-paginate-v2/blob/c657c5f8e02580a5e13f336d5c34fd4c81e212f7/lib/jekyll-paginate-v2/generator/utils.rb#L127-L136

Ok so would that mean that in order for this to work, I'd have to end the pages with a / or not set the custom naming?

in order for this to work, I'd have to end the pages with a /

Yes, that's correct.

I have the same problem, but I did not understand how to solve it: once the page paths were generated correctly, /page/num/index.html instead now it has been changed to /page/2index.html how can I restore it to the old method?

I have the same problem, but I did not understand how to solve it: once the page paths were generated correctly, /page/2/index.html instead now it has been changed to /page/2index.html how can I restore it to the old method?

ok fixed with custom indexpage:
pagination:
  enabled: true
  collection: blog
  indexpage: '/index'

Could you please upgrade to https://rubygems.org/gems/jekyll-paginate-v2/versions/3.0.0 and test again for me?

Could you please upgrade to https://rubygems.org/gems/jekyll-paginate-v2/versions/3.0.0 and test again for me?

Tested, now working but the link of first page have two trailing slash
Page 1:

                        1

                        2

                    Older Posts →

Page 2:

                    ← Newer Posts

                        1

                        2

I see similar issues with discrepancies between paginator and page_trail, missing slashes and redundant index.html.
This is my _config.yml:
title: Your awesome title
email<EMAIL_ADDRESS>future: true

plugins:
  - jekyll-paginate-v2

pagination:
  enabled: true
  permalink: /page:num
  per_page: 5
  trail:
    before: 2
    after: 2

sass:
  quiet_deps: true

index.md looks like this:
---
layout: home
permalink: /posts/
pagination:
  enabled: true
---

There are enough posts to generate five pages, and _layouts/home.html looks like
<!DOCTYPE html>

<title></title>

Home page

{% for post in paginator.posts %}

{{ post.title }}

{% endfor %}

paginator values:

First: {{ paginator.first_page_path }}

Previous: {{ paginator.previous_page_path }}

Current: {{ paginator.page_path }}

Next: {{ paginator.next_page_path }}

Last: {{ paginator.last_page_path }}

page_trail values:

{% for trail in paginator.page_trail %}

    {{ trail.num }}: {{ trail | jsonify }}

{% endfor %}

Now, for the three different options for pagination.permalink described here
https://github.com/sverrirs/jekyll-paginate-v2/blob/7bc2634c04c49120b674cc3506275f85c4389f5c/README-GENERATOR.md#L63-L65
I get the following results (I use no / before :num, but it makes no difference):
/page:num/
Generated pages:
posts/
├── index.html
├── page2
│   └── index.html
├── page3
│   └── index.html
├── page4
│   └── index.html
└── page5
    └── index.html

✅ Looks as expected
paginator values:

First: /posts/
Previous: /posts/page2/
Current: /posts/page3/index.html
Next: /posts/page4/
Last: /posts/page5/

:x: Redundant index.html for current page
page_trail values:

1: {"num":1,"path":"/posts/index.html","title":"Your awesome title"}
2: {"num":2,"path":"/posts/page2/index.html","title":"Your awesome title - page 2"}
3: {"num":3,"path":"/posts/page3/index.html","title":"Your awesome title - page 3"}
4: {"num":4,"path":"/posts/page4/index.html","title":"Your awesome title - page 4"}
5: {"num":5,"path":"/posts/page5/index.html","title":"Your awesome title - page 5"}

:x: Redundant index.html for every path

/page:num.html
posts/
├── index.html
├── page2.html
├── page3.html
├── page4.html
└── page5.html

✅ Looks as expected
paginator values:

First: /posts/
Previous: /posts/page2.html
Current: /posts/page3.html
Next: /posts/page4.html
Last: /posts/page5.html

✅ Looks as expected
page_trail values:

1: {"num":1,"path":"/posts/index.html","title":"Your awesome title"}
2: {"num":2,"path":"/posts/page2.html","title":"Your awesome title - page 2"}
3: {"num":3,"path":"/posts/page3.html","title":"Your awesome title - page 3"}
4: {"num":4,"path":"/posts/page4.html","title":"Your awesome title - page 4"}
5: {"num":5,"path":"/posts/page5.html","title":"Your awesome title - page 5"}

:x: Discrepancy between path for first page in paginator and page_trail (trail has extra index.html)

/page:num
posts/
├── index.html
├── page2index.html
├── page3index.html
├── page4index.html
└── page5index.html

:x: Pages beyond the first one seem to miss a / before index.html
paginator values:

First: /posts/
Previous: /posts/page2
Current: /posts/page3index
Next: /posts/page4
Last: /posts/page5

:x: Other than the first and current page, these are broken links
page_trail values:

1: {"num":1,"path":"/posts/index.html","title":"Your awesome title"}
2: {"num":2,"path":"/posts/page2index","title":"Your awesome title - page 2"}
3: {"num":3,"path":"/posts/page3index","title":"Your awesome title - page 3"}
4: {"num":4,"path":"/posts/page4index","title":"Your awesome title - page 4"}
5: {"num":5,"path":"/posts/page5index","title":"Your awesome title - page 5"}

:x: Different from paginator, but no broken links

Differences between paginator and page_trail aren't a huge deal, though it would be nice to not have them, but the behaviour for /page:num seems buggy (and that's exactly the behaviour I'd like to have, with links like posts/, posts/page2/ etc.)


Fixed readme props

Coverage remained the same at 99.648% when pulling e319f5ab78022f8d8f017085314619949a200041 on LasseRafn:master into d97b9c3bd17711e64b880d6150c54da369ae5aa5 on charliekassel:master.


What need to be change if generation added a new modal

Such as image, for both understanding and generation

In the current model architecture, incorporating image generation may not be easily achievable. Our model structure is currently more suited for speech generation. Perhaps in the future, we will explore how to achieve image generation.


SCM Graph: Incorrect display of empty commit message

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.93.1
OS Version: N/A

Steps to Reproduce:
git init test && cd test
git commit -m "Initial commit" --allow-empty --signoff
git commit -m "" --allow-empty --signoff --allow-empty-message
git commit -m "" --allow-empty --allow-empty-message

@xtqqczze, what would be your expectation to be shown in the graph for an empty commit?

@xtqqczze, what would be your expectation to be shown in the graph for an empty commit?

An empty string, not Signed-off-by: xtqqezze<EMAIL_ADDRESS>

The graph will always show the first line of the commit message which matches the behaviour of git log.

The graph will always show the first line of the commit message which matches the behaviour of git log.

% git log --oneline --no-decorate
f979a04
271a07c Signed-off-by: xtqqczze<EMAIL_ADDRESS>9616747 Initial commit


[Bug] Update-DbaInstance uses CredSSP even when Credential is not specified

Environmental information

#### PowerShell version:

Name                           Value
----                           -----
PSVersion                      6.2.3
PSEdition                      Core
GitCommitId                    6.2.3
OS                             Darwin 18.7.0 Darwin Kernel Version 18.7.0: Tue Aug 20 16:57:14 PDT 2019; root:xnu-4903.271.2~2/RELEASE_X86_64
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

#### dbatools Module version:

Name    : dbatools
Path    : /Users/kkravtsov/.local/share/powershell/Modules/dbatools/0.9.822/dbatools.psd1
Version : 0.9.822

SQL Server:

Not related

Report

Host used

[x] powershell.exe
[ ] ISE
[ ] VS Code
[ ] Other (please specify)

If anything other than powershell.exe was used, please confirm that you can duplicate the issue with powershell.exe

[ ] Still buggy in powershell.exe

Errors Received

WARNING: [10:03:40][Update-DbaInstance] Failed to enumerate files in -Path | The WinRM client cannot process the request. Requests must include user name and password when CredSSP authentication mechanism is used. Add the user name and password or change the authentication mechanism and try the request again.

Steps to Reproduce
Update-DbaInstance -ComputerName $server -Path $patchpath -ExtractPath $extractpath -KB $kb

Expected Behavior
No errors
Actual Behavior
Find-SqlInstanceUpdate errors out due to using CredSSP Authentication without Credentials

Tested on my environment as working without using -Credential