Migrate from Jest to Uvu

I suggest to use Uvu instead of Jest. It's stable, many popular packages are using this utility: Browserslist, PostCSS, etc.
Jest vs. Uvu comparison
Installation speed
Jest:
$ npm i --force
...
added 900 packages, and audited 901 packages in 30s

Uvu:
$ npm i

added 508 packages, and audited 509 packages in 7s

node_modules size
Jest:
$ du -sh node_modules/
310M	node_modules/

Uvu:
$ du -sh node_modules
286M	node_modules

Testing time
Jest (2,5 sec.):
$ npm run test
...
Test Suites: 1 passed, 1 total
Tests:       13 passed, 13 total
Snapshots:   0 total
Time:        2.514 s, estimated 4 s
Ran all test suites.

Uvu (~1 ms, not second):
$ npm run test
...
  Total:     13
  Passed:    13
  Skipped:   0
  Duration:  1.05ms

I'm open to doing this as long as it works well with Quasar and doesn't burn any bridges. Quasar recommends Jest and I don't want to stray too far from their way of doing things because I do want to build mobile apps with Cordova eventually, and the promise of Quasar is that they make it easy to do so.

Also curious if package-lock.json it necessary for yarn to work? I didn't think it was, and I'd prefer to avoid checking in two different lockfiles.

I'm open to doing this as long as it works well with Quasar and doesn't burn any bridges. Quasar recommends Jest and I don't want to stray too far from their way of doing things because I do want to build mobile apps with Cordova eventually, and the promise of Quasar is that they make it easy to do so.

It will work, of course. It's just a unit testing framework. A very lightweight framework. For example, it doesn't have expect.arrayContaining.

const odd = [1, 3, 5, 7];

test('just an example', () => {
  expect(odd).toEqual(expect.arrayContaining([3, 5]));
});

// Could be rewritten as
test('just an example', () => {
  expect(
    [3, 5].every(n => odd.includes(n)),
    true,
  );
});

To mock & spy functions nanospy will perfectly fit (it's also used in projects like PostCSS or Browserslist). For example, @vue/test-utils will work fine with Uvu.

About Cordova, from the official website:

Cordova wraps your HTML/JavaScript app into a native container which can access the device functions of several platforms.

The application stays the same web-application (not native) with some additional APIs provided by Cordova. That's it.
P.S. It will be very laggy on cheap smartphones. That's why React Native or Flutter exists. Just warning you.

Also curious if package-lock.json it necessary for yarn to work? I didn't think it was, and I'd prefer to avoid checking in two different lockfiles.

Will remove it.

P.S. It will be very laggy on cheap smartphones. That's why React Native or Flutter exists. Just warning you.

Until maplibre-gl-js works with React Native or Flutter, this is IMO the best way to build a single-codebase maps app. It won't be perfect but it's the best I can do I think.

I'm not sure about this, honestly. I read through the diff and I really do enjoy having fluent assertions. I'm more than willing to sacrifice a few megabytes on my hard drive and 1s per run for that. Are there other reasons to switch?

For PostCSS (25 test suits): time reduced from ~8.7s to ~250ms (3,380% faster), size of node_modules decreased from 150 mb to 114mb (31.578947% smaller).
I think if there's a chance to make software faster, we should take this chance :-)
P.S. https://github.com/headwaymaps/headway/pull/164#discussion_r974659228


Container won't start

I'm trying to get the container working as part of a portainer stack, adding in the containers one at a time, but it's throwing an nginx error and stopping
The full log file is attached but the problem line seems to be
[nginx       ] nginx: [emerg] open() "/var/tmp/nginx/default_site.conf" failed (13: Permission denied) in /opt/base/etc/nginx/nginx.conf:76
The stack entry for this container is
jdownloader:
<<: *common-keys-media # See EXTENSION FIELDS at the top
image: jlesage/jdownloader-2:latest
container_name: jdownloader
networks:
t2_proxy:
ports:
- "5800:5800"
volumes:
- /opt/ext-vol/downloads:/data/downloads
- /opt/docker/appdata/jdownloader:/config
environment:
USER_ID: 1000
GROUP_ID: 1000
TZ: $TZ
UMASK: 002
KEEP_APP_RUNNING: 1
CLEAN_TMP_DIR: 1
DISPLAY_WIDTH: 1600
DISPLAY_HEIGHT: 960
I've removed the container and images and started again with the same result.
I also get  the same result using docker compose with a yml file just with the container in
jdownloader.txt
Any ideas on this as I can't keep the container running long enough to console in to dig further?

Getting something similar with the latest version:
[supervisor  ] starting service 'nginx'...
[nginx       ] Listening for HTTP connections on port 5800.
[nginx       ] nginx: [alert] could not open error log file: open() "/config/log/nginx/error.log" failed (13: Permission denied)
[nginx       ] 2023/05/09 21:00:28 [emerg] 631#631: open() "/config/log/nginx/error.log" failed (13: Permission denied)
[supervisor  ] service 'nginx' failed to be started: not ready after 5000 msec, giving up.
[supervisor  ] stopping service 'nginx'...
[supervisor  ] service 'nginx' exited (with status 1).
[supervisor  ] stopping service 'xvnc'...

in my case, nginx fails to start somehow in the container, but only when binding a volume to /config.
with no volumes bound, the container doesn't crash at least, but the drawback is obvious...

Just solved my issue by changing rights of the folder to mount.
was d---------+ before and is drwxrwxrwx now after running chmod 777 ./Jdownloader2/config where config is the folder to bind to the volume.
777 may be overkill, but hope this helps you too.

@EBS-DarkD, maybe you could share the output of docker inspect jdownloader ?

Apologies, I've been swamped with work and not had a chance to look at this for a while. Turns out I had a permissions problem on the host that was causing all sorts of crazy issues!

User error!


Integrated Terminal opens to automatically scroll few lines and prompt is invisible

Issue Type: Bug

Launch Integrated Terminal, (the issue is reproducable for both PowerShell and cmd.exe.)

VS Code version: Code 1.51.1 (e5a624b788d92b8d34d1392e4c4d9789406efe8f, 2020-11-10T23:34:32.027Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-6006U CPU @ 2.00GHz (4 x 1992)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.89GB (3.90GB free)

Process Argv
--crash-reporter-id 07176252-0d46-44ab-8953-bcbbaafb357a

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

atlascode
atl
2.8.4

rest-client
hum
0.24.4

java
red
0.70.0

vscode-yaml
red
0.12.0

LiveServer
rit
5.6.1

sonarlint-vscode
Son
1.18.0

vscodeintellicode
Vis
1.2.10

vscode-java-debug
vsc
0.29.0

vscode-java-pack
vsc
0.10.0

Please provide a a gif illustrating the problem. See this wiki page for more general info on how to report actionable terminal issues.
/label "needs more info" integrated-terminal

I'm also running into this bug on my Windows 10 with and without all the extensions disabled.

Note that my VSCode version and OS version are exactly the same as paritosh149.
It seems that the bug has nothing to do with the recent release since I often encounter it.

This looks like an issue with our old terminal backend "winpty", going forward we are focusing our efforts on the new backend "conpty" which is build by the Console team at Microsoft and should fix many issues with the terminal under Windows. You can read more about getting ConPTY here.
/duplicate https://github.com/Microsoft/vscode/issues/45693

Just tested with settings changed to "terminal.integrated.windowsEnableConpty": false. and the issue persists.

Oh no, turning this setting off won't help at all.
If I wasn't wrong, changing this setting to false will cause the newer version of Windows 10 to use the old terminal backend winpty instead of conpty, while on our windows 10 version 1809, there is no conpty support and this setting is ignored, so it is expected that the issue persists.
... and if I wasn't wrong, the vscode team will no longer care about the bug. The only way to fix this is to upgrade the windows 10 to at least versoin 1903, and leave this setting on.

You are correct @Zjl37 and we do care, but are unable to fix the windows problem.


Update docs and fix bugs

Fixes a couple of bugs for setting timestamps and adds basic documentation.

+1


Remove deprecated APIs

resolves #319

Coverage increased (+0.3%) to 81.656% when pulling 7a23d3e614b8c6e8abea3a78dff60b16607af9f5 on deprecated into 8ef2bbfbca4751a1b2627fdfcd6b063efee73783 on 2.0.0-RC.


[UWP] Background URI test file causes a crash

Platform
What platform is your issue or question related to? (Delete other platforms).

UWP

Details
The v1.2/Elements/AdaptiveCard.BackgroundImage.DataUri.json sample file crashes with a null pointer deref when loaded in the visualizer. This seems to have been introduced with #4225.

Duping this to #4651.


Export Layer Group as Animated GIF

Would it be possible to add an option to export each layer in a layer group as an animated gif?
I know I can already export a layer group as one still image but when the 'Export Image as GIF' dialog pops up the 'As animation' box is greyed out for some reason as well as all the other animated gif options.
example, I have a layer structure like this
├── image.jpg
├── image.jpg
├── animated image.gif
│   ├── frame 1
│   ├── frame 2
│   ├── frame 3

and that would three images with the third one being animated.
Is this already achievable or am I requesting a new feature?

Try if the following works for you:

Add the "Export" procedure. The folder explorer and the file extension in the main dialog will now be ignored.
Adjust the output folder as needed.
Type gif next to File extension.
Set Perform export: to For each top-level layer or group.

Once you start exporting and your first item is a layer group, you should be able to check As animation. However, if the first item is a layer, it seems the option cannot be checked and you will not be able to adjust this for subsequent layers. You could rearrange your image or add a dummy layer group to work around this.
If each layer within a layer group has different dimensions, this could be a problem. As a workaround, you could uncheck "Use layer size" to use the image size, but not this will use the same size for all layers within the image.

Thanks! Your solution worked. However, the two non-animated layers I showed in my example are made into .gif files also insted of .jpg when using the "Export" procedure. I checked "Use file extension in layer name" but that didn't seem to do anything. It might be a bug, I dont know.

Type [layer name, %e] in the main dialog next to Save as:. This should preserve the file extension.

Okay, that worked. Thanks!

This is definitely a bug, and I can confirm it occurs on my end as well.
In the meantime, you could work around this by performing export twice - first with the layer group(s) only and then with the standalone layers.
To export the layer group only, uncheck the Layers constraint and add the Layer groups constraint.
To export the regular layers only, uncheck the Layer groups constraint and check the Layers constraint.

I'll try. Still, I appreciate your support anyways.

I just released version 4.1, which should fix the export.

OK, I tried it, and everything seems to come out how I want it. Thanks for the help!


brew install homebrew/apache/mod_python does not install the associated python modules

brew install homebrew/apache/mod_python installs mod_python.so, but does not install any of the python modules under the lib directory in the source distribution.   Those are very necessary for any reasonable mod_python code.
lib
└── python
    └── mod_python
        ├── Cookie.py
        ├── Session.py
        ├── __init__.py
        ├── apache.py
        ├── cache.py
        ├── cgihandler.py
        ├── httpdconf.py
        ├── psp.py
        ├── publisher.py
        ├── python22.py
        ├── testhandler.py
        ├── util.py
        └── wsgi.py

Is there some other way I'm supposed to use to install that code, or was it just forgotten?

(apologies on the delay, I am planning on investing time into homebrew-apache issues and PRs this weekend)

:+1:

👍

Looking at the code it only seems to install the .so so you are probably correct that it is not reflected in the Formula at the moment. Can you have a look at the Formula and see if you can complement it in any way ?

This tap is now deprecated, sorry.


Emacs-style comment break state file processing when non-default environment used

How to reproduce
~/tmp/helmfile ❯ helmfile --version
helmfile version v0.139.9

MacOS 11.4, x86_64, helmfile installed via brew
Create helmfile.yaml file:
# -*- mode: yaml; mode; view -*-
---
environments:
  bootstrap:
releases:
  - name: k8s-iae
    chart: sb/k8s-iae

and try to apply/render YAML/render values
Actual result
~/tmp/helmfile ❯ helmfile write-values
Writing values file helmfile-0c6885e8/k8s-iae.yaml
~/tmp/helmfile ❯ helmfile -e bootstrap write-values
err: no releases found that matches specified selector() and environment(bootstrap), in any helmfile

Expected result
~/tmp/helmfile ❯ helmfile write-values
Writing values file helmfile-0c6885e8/k8s-iae.yaml
~/tmp/helmfile ❯ helmfile -e bootstrap write-values
Writing values file helmfile-0c6885e8/k8s-iae.yaml

Workarounds found so far
---
environments:
  bootstrap:
releases:
  - name: k8s-iae
    chart: sb/k8s-iae

or
# -*- mode: yaml; mode; view -*-
environments:
  bootstrap:
releases:
  - name: k8s-iae
    chart: sb/k8s-iae

or
environments:
  bootstrap:
releases:
  - name: k8s-iae
    chart: sb/k8s-iae

# any comment
---
environments:
 example

I believe the example above results in two YAML documents, and the first is empty, which obviously contains no release specs for helmfile to parse.
Perhaps you could use a local variables list near the near the end of the file instead.

The YAML specification example 2.7 implies that
# any comment
---
environments:
 example

would be only 1 YAML document.

It might be more about that helmfile doesn't allow empty helmfile.yaml part, as it can't decide if the user mistakenly created an empty YAML doc part in helmfile.yaml or it's intended.

I'd consider this a bug. Consider the following helmfile.
# A comment.
---
environments:
  default:
    values:
      - label: "I was parsed from the default environment."
  other:
    values:
      - label: "I was parsed from the other environment."

releases:
  - name: ingress-nginx
    chart: ingress-nginx/ingress-nginx
    namespace: ingress-nginx
    values:
      - commonLabels:
          label: {{ .Environment.Values.label }}

repositories:
  - name: ingress-nginx
    url: https://kubernetes.github.io/ingress-nginx

The command helmfile --environment default template will parse the values label from the default environment and add it to commonLabels in the ingress-nginx release.
However, the command helmfile --environment other template will result in error err: no releases found that matches specified selector() and environment(other), in any helmfile.
Thus helmfile does allow the empty helmfile YAML part only for the default environment, but will not allow the empty helmfile YAML part for other environments.
P.S. I understand that Helmfile development has migrated to https://github.com/helmfile/helmfile. If you prefer, I can file this as an issue there.

@scruffaluff Thanks for clarifying!
The intention here is helmfile should fail like err: no releases found that matches specified selector() and environment(other), in any helmfile on empty helmfile part and helmfile.yaml, regardless of the user chosen the default env or the named env(like other).
So, if it was a bug, we'd need to "fix" it to also fail for the former case, which is still not what you want.


config to enable ACH file batch merging based on the batch header

What were you trying to do?
Currently there is no way to configure merging of entry records under the same batch if the batch headers are the same.
What did you expect to see?
A configuration option to merge batches if the batch headers are the same.

We can do this. When testing out that linked PR I ran into https://github.com/moov-io/ach/issues/863 and https://github.com/moov-io/ach/issues/867 which caused files to not be uploaded. https://github.com/moov-io/ach/issues/866 might be a fix we implement as well.


Rewritten DSN parsing to use parse_url and parse_str to comply with rfc3986 (fixes #218)

See issue #218 for more info. This change makes sure the URL is parsed properly to just extract the password.

After the tests failed it seems that the current implementation is the best solution when dealing with sockets as well, I'll try again with a simple fix to extract the password correctly.


SW-6059 Workflow refactor Part 2 - Templates

New template structure with sub-directories
New layout template for a "tab" page
Simplified the way templates are defined and executed from Go files
Simplified the client-tasks template by moving certain elements to a higher level template

Codecov Report
Patch coverage: 2.63% and project coverage change: -0.13 :warning:

Comparison is base (ecfa752) 65.19% compared to head (bef5b77) 65.07%.

:exclamation: Current head bef5b77 differs from pull request most recent head c493040. Consider uploading reports for the commit c493040 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #336      +/-   ##
==========================================
- Coverage   65.19%   65.07%   -0.13%
==========================================
  Files          14       14
  Lines        1063     1065       +2
==========================================
  Hits          693      693
- Misses        330      332       +2
  Partials       40       40

Impacted Files
Coverage Δ

internal/server/server.go
35.29% <ø> (ø)

internal/server/wrap_handler.go
0.00% <0.00%> (ø)

main.go
0.00% <0.00%> (ø)

internal/server/client_tasks.go
81.65% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


⚠ Jenkins Plugins Artifactory has degraded performance

In 8a74077, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1259 ms

Resolved: Jenkins Plugins Artifactory performance has improved in cb06b98 after 12 minutes.


Unnecessary notifications after a (re-)start of the phone

Since some weeks I face the following issue when starting my Android phone. After a (re-)start I will be notified in the notification bar about a significant number of tasks with no clear reason why.
Due dates have not been reached yet for them. I never set start dates. I am running about altogether 60 open tasks and about 20 to 30 tasks will appear in such cases. I am not able to determine which are shown here (due dates vary from in a few days to in a some weeks).
Expected and existing behavior of the App (as I have seen in the past) was no such unnecessary notifications after a (re-)start of the phone.
I am using DAVx5 to sync my CALDAV calendars.
OpenTasks Version: 1.1.18
Phone : Google Pixel 3
Android Version: 9

This could be a duplicate of #800. Do they reappear on reboot when you pin and un-pin the task once?

If have checked this now with several tasks. When pin and un-pin them before a reboot, such tasks will not reappear.

Ok, in this case I'll close this as a duplicate of #800.


ValueError: time data '2019-09-04T16:52:42+01:0.000000Z' does not match format '%Y-%m-%dT%H:%M:%S.%fZ'

Although when modifying a task to have a due date and then syncing I got the following error, however not sure if this is a generic bug or an OS specific bug.
Traceback (most recent call last):
  File "/USERS/intern1/Library/Python/3.7/bin/tw_gcal_sync", line 61, in <module>
    main()
  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.7/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/USERS/intern1/Library/Python/3.7/bin/tw_gcal_sync", line 53, in main
    aggregator.register_items(gcal_items, "gcal")
  File "/Users/intern1/Library/Python/3.7/lib/python/site-packages/taskw_gcal_sync/TWGCalAggregator.py", line 249, in register_items
    other_item_new = convert_fun(item)
  File "/Users/intern1/Library/Python/3.7/lib/python/site-packages/taskw_gcal_sync/TWGCalAggregator.py", line 512, in convert_gcal_to_tw
    tw_item['due'] = GCalSide.get_event_time(gcal_item, t='end')
  File "/Users/intern1/Library/Python/3.7/lib/python/site-packages/taskw_gcal_sync/GCalSide.py", line 223, in get_event_time
    item[t][GCalSide.get_date_key(item[t])])
  File "/Users/intern1/Library/Python/3.7/lib/python/site-packages/taskw_gcal_sync/GCalSide.py", line 271, in parse_datetime
    dt_out = datetime.datetime.strptime(dt2, _format)
  File "/usr/local/Cellar/python/3.7.4/Frameworks/Python.framework/Versions/3.7/lib/python3.7/_strptime.py", line 577, in _strptime_datetime
    tt, fraction, gmtoff_fraction = _strptime(data_string, format)
  File "/usr/local/Cellar/python/3.7.4/Frameworks/Python.framework/Versions/3.7/lib/python3.7/_strptime.py", line 359, in _strptime
    (data_string, format))
ValueError: time data '2019-09-04T16:52:42+01:0.000000Z' does not match format '%Y-%m-%dT%H:%M:%S.%fZ'

Originally posted by @alessio-b-zak in https://github.com/bergercookie/taskw_gcal_sync/issues/11#issuecomment-527947377

[ ] Solve datetime issue
[ ] Consider switching from timedate usage to arrow

@alessio-b-zak, unfortunately I can't reproduce it.
I fixed a push to the GCalSide class though so that it can handle the format that was breaking.
Let me know if this still doesn't work.


Provide access to labelled meshes via python and api, to support new Julich-Brain version and freesurfer

related card: https://trello.com/c/69JRHucS/65-provide-access-to-labelled-meshes-via-python-and-api-to-support-new-julich-brain-version-and-freesurfer

depends on https://github.com/FZJ-INM1-BDA/siibra-python/issues/13


An error when running scDML.

Hello,
Recently, an error is bothering me when I use scDML to intergrate my sc-RNA data.
It appears when I running the following codes:
if os.path.isfile(os.path.join(scdml.save_dir,"scDML_model.pkl")):
os.remove(os.path.join(scdml.save_dir,"scDML_model.pkl"))
features=scdml.train(expect_num_cluster=None)
adata.obsm["X_emb"]=features
adata.obs["reassign_cluster"]=scdml.train_label.astype(int).astype(str)
adata.obs["reassign_cluster"]=adata.obs["reassign_cluster"].astype("category")
And the error is:
2023-10-02 13:45:41,594 - root - INFO - Optimal number of clusters [ 5 10 13 17 26]
2023-10-02 13:45:41,595 - root - INFO - scDML can't find the mering result of cluster=5 ,you can run merge_cluster(fixed_ncluster=5) function to get this
Sometimes I restart the jupyter nootbook and re-run the code solves this error. But sometimes it keep going wrong.
I'd like to know what is going on when this appear and what should I do to solve it.
Thanks.

I also tried to run the following code as the error information noticed
scdml.merge_cluster(fixed_ncluster=5)
However, this gives a new error:
TypeError: merge_cluster() got an unexpected keyword argument 'fixed_ncluster'

Hello, Recently, an error is bothering me when I use scDML to intergrate my sc-RNA data. It appears when I running the following codes:
if os.path.isfile(os.path.join(scdml.save_dir,"scDML_model.pkl")): os.remove(os.path.join(scdml.save_dir,"scDML_model.pkl"))
features=scdml.train(expect_num_cluster=None) adata.obsm["X_emb"]=features adata.obs["reassign_cluster"]=scdml.train_label.astype(int).astype(str) adata.obs["reassign_cluster"]=adata.obs["reassign_cluster"].astype("category")
And the error is:
2023-10-02 13:45:41,594 - root - INFO - Optimal number of clusters [ 5 10 13 17 26] 2023-10-02 13:45:41,595 - root - INFO - scDML can't find the mering result of cluster=5 ,you can run merge_cluster(fixed_ncluster=5) function to get this
Sometimes I restart the jupyter nootbook and re-run the code solves this error. But sometimes it keep going wrong. I'd like to know what is going on when this appear and what should I do to solve it. Thanks.

what about running with python script(.py)


Rpm fixes fedora

several fixes to get rpm building again on fedora 33

Coverage remained the same at 62.378% when pulling 29b2488541d9d9e96ca38afe923a41de027b07ef on trixirt:rpm-fixes-fedora into c4c9efbb3a47b68a97c5ef53baabdb09a08e1b83 on OPAE:master.

rebased to master with no changes

No changes to previous commits
rebase to master
fix packager problem
builds pass again on fedora 33


I want modify source code in Element.java

In org.jsoup.node.Element.java on line 1045, "sb.append(elementData);" should be changed to "sb.append(element)"

This will modify the behavior in a way so data() no longer does what it's documented to do.
What are you trying to accomplish? You didn't provide any explanation whatsoever.

I wrote an issue because it seemed that the test case value did not come out correctly during checking the code.

Document document = Jsoup.parse("OneTwoThree");
Element p = document.select("p").first();
assertEquals("One", p.data()); //return false..
In this case, 'p' returns a null value. it seems some mistake.
Data function output of the Element  seems the desired value("One").

The code sample in your comment is not displaying properly. Check out https://help.github.com/articles/creating-and-highlighting-code-blocks/ to learn how to format code blocks on GitHub.
I'm assuming you posted a variation of this test method from your PR #856:
@Test
public void TestDataFunctionInElement() {
    Document doc2 = Jsoup.parse("

One
Two
Three
");
    Element p2 = doc2.select("p").first();
    assertEquals("One", p2.data()); //return false..
}

The thing is that data() only returns the "data" of an element. That's mainly to get the data between


Controllers specs

Create a Request spec file for all controllers

Add posts_spec.rb and users_spec.rb

Add RSpec as a gem to the project

Run linters for errors

For each action you check:

[x] If response status was correct.

[x] If a correct template was rendered.

[x] If the response body includes correct placeholder text.

@TadesseDev Thank you1

You did a great job @Chaseknis 🙌

@TadesseDev my dashboard is still showing REVIEW PENDING, is there something wrong?


collapse: improve type definition. remove invalid onOpened and onClosed

[x] Bug fix
[ ] New feature
[ ] Chore
[ ] Breaking change
[ ] There is an open issue which this change addresses
[x] I have read the CONTRIBUTING document.
[ ] My commits follow the Git Commit Guidelines
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.

[ ] I have updated the documentation accordingly.

[ ] My change requires a change to Typescript typings.

[ ] I have updated the typings accordingly.

[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Could not find  onOpened and onClosed in source code. also the other properties have node and boolean in the listener from react-transition-group

looks like nobody cares


mcumgr: Add latency control functionality

This change adds a posibility to enable low latency connection
parameters for BT when SMP commands are handled.
Support for this functionality is disabled by the default and
can be enabled by CONFIG_MCUMGR_SMP_BT_LATENCY_CONTROL=y option.
Signed-off-by: Emil Obalski<EMAIL_ADDRESS>

Rebased due to some weird doc build error. @de-nordic @joerchan please have a look.

@alwa-nordic Could you please have a look in this PR? This change adds a very handy feature for our samples.

@de-nordic please review from the point of view of mcumgr.
@alwa-nordic please take a look at the BLE functionality.


Recommended method of revoking users?

Thanks for providing the cookbook to create a VPN server.  I don't see anything in the documentation about how to revoke users.  I also don't see the revoke-full command on the server to do it according to the "OpenVPN way".
What is the recommended way of removing users access to the VPN?
Ideally I'd like to remove the user, manually run chef on the server and have it boot that user from the connection.
Thanks!

I believe we are still looking for contribution in this area.
There is some related history:

https://github.com/xhost-cookbooks/openvpn/pull/25
https://github.com/xhost-cookbooks/openvpn/pull/82
https://github.com/xhost-cookbooks/openvpn/issues/66

Would love to see a pull request including documentation in the readme on usage.

https://github.com/xhost-cookbooks/openvpn/issues/55 is also related.

revoke-full is in easy-rsa (https://github.com/OpenVPN/easy-rsa-old/blob/master/easy-rsa/2.0/revoke-full). Created https://github.com/xhost-cookbooks/openvpn/issues/90 to begin.


validates health-check-port-index even if ports is not explicitly provided

Changes proposed in this PR

validates health-check-port-index even if ports is not explicitly provided

Why are we making these changes?
Disallow invalid configurations from being accepted and run as services.

Closing.
This is working as designed to allow on-the-fly to specify the a valid ports value.


Star ratings are displayed inconsistently when clicking through multiple games

NOTE: Please submit website issues here
Describe the bug
When clicking through multiple games with different star ratings, the star ratings get mixed up and are displayed inconsistently.
To Reproduce
Steps to reproduce the behavior:

Open Phoenix
Click through a few games which have different star ratings
You should notice that the ratings are not correct, and are often different each time you click on the same game

Expected behavior
Star ratings should consistently reflect the rating stored in games.json
Screenshots
https://github.com/user-attachments/assets/2a6c5162-408b-481e-864d-2415af01e909
Logs
Find the logs by pressing CMD+Shift+O in Phoenix, or going to ~/Library/Application Support/Phoenix
log_2024-08-23 18:13:05.448000.log
Desktop (please complete the following information):

macOS Version: macOS 14.6.1
Phoenix Version: Phoenix 0.1.10 (33)

Additional context
n/a

Fixed in #183


Infinite Scroll Plugin

This could be a great feature; if I can get it hacked together on my end I will gladly share the code in case it wants to get implemented

i don't know whether it is late. But i would love to see it


Fix a bug when adding new partitions in FULL AUTO

Summary
When adding a new partition to a new host in FULL AUTO, we would wait go through helix state transition to create a bootstrap replica before we update the property store to add replicas to data node config.  When creating bootstrap replica, we would put the partition name to partitionNameToAmbryParitition map, so later when the property store is updated, we would ignore adding partition to partitionMap. That's why we encountered a Partition Id from stream is unknown error.
This pr fixes that.

Codecov Report
Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 31.81%. Comparing base (52ba813) to head (9c15c92).
Report is 25 commits behind head on master.

Files
Patch %
Lines

...m/github/ambry/clustermap/HelixClusterManager.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2800       +/-   ##
=============================================
- Coverage     64.24%   31.81%   -32.43%
+ Complexity    10398     5372     -5026
=============================================
  Files           840      841        +1
  Lines         71755    72155      +400
  Branches       8611     8684       +73
=============================================
- Hits          46099    22959    -23140
- Misses        23004    47420    +24416
+ Partials       2652     1776      -876

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update http4s-blaze-client, ... to 0.23.0

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-dsl
org.http4s:http4s-scala-xml

from 0.21.25 to 0.23.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Migrations

github:http4s/http4s/v0_22?sha=v0.22.0

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-minor, scalafix-migrations

Superseded by #224.


Godot Android Editor's pan and scale half-works

Godot version
Godot 4.0 RC 4
System information
Xiaomi Pad 5, Adreno 640
Issue description
Good afternoon,
I have this problem with godot where selecting nodes using a touchscreen is basically impossible, because for some reason it starts to pan instead of selecting.
And using the two-fingers gesture to pan produces a weird effect, here's a video for a more visual example
https://user-images.githubusercontent.com/34092413/221162971-a7fa2205-5405-47d8-a434-5d5bf101b2f2.mp4
I tried to toggle the pan-and-scale gesture from the Editor's settings, but no effect is seen
Steps to reproduce
1- Install Android Godot Editor 4 RC 4
2- Try to select stuff with a touchscreen
Minimal reproduction project
N/A

Fixed by https://github.com/godotengine/godot/pull/75718


DONOT merge: update mixer version for testing

Jenkins job istio/presubmit passed

Jenkins job istio/presubmit passed


tdp init --overiddes fails if one of the overrides file is empty

Reproduce:
tdp init --overrides tdp_vars_overrides

If one of the file in the override is empty (e.g. tdp_vars_overrides/hive/hive.yml) init crashes with error
raise EmptyCommit(
tdp.core.repository.repository.EmptyCommit: validating these changes would produce no difference

Would it be better to raise an error like EmptyFile or to just ignore it?

A Warning explicity saying that the file is empty and no commit has been made is appropriate imo


Support setting nullable / non-nullable constraints in pipedag.Table()

Since pipedag does the materialization, we want to add the feature to specify which columns should be non-nullable. Offering both nullable and non_nullable as optional parameters allows for specifying a positive or negative list. If both lists are given, they together must comprise all columns.

I was looking into the option to run the SELECT statement with LIMIT 0 and to retrieve types from the resultset object.
This somewhat works with dbapi for a database which is not duckdb:
x = cur.execute("SELECT TOP 0 1 as x, 'aba' as y, cast(1.0 as numeric(15,2)) as z")
x.description
(('x', <class 'int'>, None, 10, 10, 0, False), ('y', <class 'str'>, None, 3, 3, 0, False), ('z', <class 'decimal.Decimal'>, None, 15, 15, 2, True))

However, I don't feel comfortable to assemble database types based on this output to produce a complete CREATE TABLE statement with given types. Thus I suggest to submit the query twice: 1st: CREATE TABLE x AS SELECT ... LIMIT 10. Then adjust the nullable/non-nullable columns. Finally, run a INSERT INTO x SELECT statement.

Duckdb simply creates NoneType columns for SELECT 1 as a. Thus it is not well suited for testing:
cur = connection.connection.cursor()
x = cur.execute("SELECT 1 as x")
x.description
(('x', None, None, None, None, None, None),)

Another attempt with SQLAlchemy:
Table('tmp', MetaData(), Column('x', NullType(), table=<tmp>), Column('y', NullType(), table=<tmp>), schema=None)

For INSERT INTO AS SELECT appending, we might have to do table level locking properly:
Postgres:

ACCESS EXCLUSIVE (AccessExclusiveLock)

Conflicts with locks of all modes (ACCESS SHARE, ROW SHARE, ROW EXCLUSIVE, SHARE UPDATE EXCLUSIVE, SHARE, SHARE ROW EXCLUSIVE, EXCLUSIVE, and ACCESS EXCLUSIVE). This mode guarantees that the holder is the only transaction accessing the table in any way.

MSSQL: INSERT INTO x WITH(TABLOCKX) AS SELECT ...
DB2: we already perform locking

Unfortunately, adding the LIMIT 0 / TOP 0 is non-trivial for sa.text() handwritten queries. Here is an attempt:
sql = sa.text("SELECT 1 as x, 'test' as y, cast('2021-01-01' as date) as z, cast('2021-01-01' as datetime) as w, cast(1.0 as Numeric(15,6)) as v")
if isinstance(sql, sa.sql.expression.TextClause):
    parts = str(sql).split("SELECT")
    limit_sql = sa.select(sa.text("SELECT".join(parts[1:]).strip())).limit(0)
else:
    limit_sql = sql.limit(0)

It seems to work for mssql and postgres.
> str(limit_sql.compile(dialect=postgresql.dialect(), compile_kwargs={"literal_binds": True}))
"SELECT 1 as x, 'test' as y, cast('2021-01-01' as date) as z, cast('2021-01-01' as datetime) as w, cast(1.0 as Numeric(15,6)) as v \n LIMIT 0"
> str(limit_sql.compile(engine, compile_kwargs={"literal_binds": True}))
"SELECT TOP 0 1 as x, 'test' as y, cast('2021-01-01' as date) as z, cast('2021-01-01' as datetime) as w, cast(1.0 as Numeric(15,6)) as v"

We can add safety checks and in case something goes wrong, we can still fallback to issuing the Non-Nullable calls after loading of full table. This should be correct. Just slower.

I found a more reliable way to hack the LIMIT clause with a subquery (performance should not matter for the limit 0 query):
    def get_limit_query(
        self,
        query: sa.sql.expression.Selectable | sa.sql.expression.TextClause,
        rows: int,
    ) -> sa.sql.expression.Select:
        if isinstance(query, sa.sql.expression.TextClause):
            # This is quite a hack. We look for the first SELECT and then use
            # sqlalchemy to put the limit/top clause in the correct position.
            # Furthermore, we search for a LIMIT n or TOP n clause to replace it.
            query_str = str(query)
            return (
                sa.select(sa.text("*"))
                .limit(rows)
                .select_from(sa.text("(" + query_str + ") AS A"))
            )
        else:
            return sa.select(sa.text("*")).limit(rows).select_from(query.alias("A"))


Repository GPG keys are not used for image builds via Cloud API

When images are built via Cloud API, the list of repositories to be used for the image build is part of the ImageRequest. However the Repository object does not include GPGKey property:
https://github.com/osbuild/osbuild-composer/blob/54a458af5c4440dc9c96092976743c39c3221e55/internal/cloudapi/openapi.gen.go#L171-L176
As a result, no GPG key is copied over to rpmmd.RepoConfig structures when a new Cloud API compose request is processed:
https://github.com/osbuild/osbuild-composer/blob/54a458af5c4440dc9c96092976743c39c3221e55/internal/cloudapi/server.go#L113-L127
Such rpmmd.RepoConfig list is then passed to imageType.Manifest(), which produces Manifest to be used by osbuild. As a result, no GPG keys are used in the resulting org.osbuild.rpm stage of the Manifest and the GPG signatures of the installed RPMs are not checked at all when osbuild builds the image. Also no GPG keys are imported into the RPM database during the image build.

After a discussion on team's Telegram, the conclusion was that we should aim for verification of RPM repository metadata signatures. However repo metadata are currently not signed for all of the distributions supported by composer.
The API could include just a single list of GPG keys, which can be used for verification of RPM and repo metadata. This is compliant with what DNF does. A RPM repository definition can contain only a single gpgkey item, which may be a list of URLs described as URLs of a GPG key files that can be used for signing metadata and packages of this repository .... Therefore if we change the Cloud API in a similar way, it should be prepared also for a case when we start verifying repo metadata in addition (or instead) of RPM signatures.
A note from @teg:

In the future, we could introduce an option to select between metadata or rpm verification. We only ever need one, and the choice should be explicit, so a Boolean would be fine.
The Boolean should default to verifying repo metadata, and the rpm verification should be opt-in. To nudge people in the right direction.


Error building Eudico on Mac M1

While running the quickstart guide, I'm getting the following error when running:
make build && make install-infra at the step
 [lotus-builder 7/7] RUN make spacenet:
#15 57.53 # github.com/filecoin-project/lotus/cmd/eudico
#15 57.53 /usr/local/go/pkg/tool/linux_arm64/link: running gcc failed: exit status 1
#15 57.53 /usr/bin/ld: /opt/filecoin/extern/filecoin-ffi/libfilcrypto.a(filcrypto-b101665716a8d1f4.filcrypto.4b299269-cgu.10.rcgu.o): Relocations in generic ELF (EM: 62)

The last line repeats a hundred or so times.
I appreciate the quickstart guide is targeted at Ubuntu, so feel free to close this if Mac M1 isn't in scope, but I'd love to get it running to try and build a subnet!
I see there is also a repo for eudico itself, so let me know if I should raise the issue there instead of there.

Hey @CluEleSsUK.
Try this
docker build -t eudico --build-arg FFI_BUILD_FROM_SOURCE=1 .

instead of this
docker build -t eudico .

in install_infra.sh.
Or wait for the fix.
Thank you for reporting this bug.

This has been fixed in the infra scripts, unless there's any outstanding issue closing it for now. Thank you both


xesmf doesn't work in pangeo-notebook

As reported on discourse: https://discourse.pangeo.io/t/regridding-error-while-using-xesmf-mpi-module-missing/749

You have to do something special to get the right environment with MPI. @ocefpaf helped me with this on the pangeo gitter a while ago.

You need to request the mpi variant you want with "mpi=*=openmpi". Conda is not smart enough, or maybe it is to allow multiple mpis?
conda create --name TEST xesmf "mpi=*=openmpi"

If you want mpich you can do,
conda create --name TEST xesmf "mpi=*=mpich"

Note that the solver may choke on packages that are not built with the desired mpi implementation.

Thanks @ocefpaf these two variants seem to work:

conda create --name TEST "mpi==openmpi" "esmpy==mpi_openmpi*" xesmf
conda create --name TEST "mpi=*=mpich" esmpy xesmf

@rabernat or @dcherian which do you recommend? I've not used xesmf before...

i got mpich working on my laptop but that's the extent of my knowledge :)

conda create --name TEST "mpi==openmpi" "esmpy==mpi_openmpi*" xesmf

There is probably a mistake somewhere in the package that made this extra string required. I'll look into it.

If anyone wants to open a PR with the your favorite xesmf variant we can definitely get it merged.

Hi everyone, from this issue reply (https://github.com/JiaweiZhuang/xESMF/issues/47#issuecomment-665516640), it seems that we can get MPI to install with esmpy last version, therefore, adding esmpy explicitly into the environmet.yml file list may solve this problem. I've tried it and it works for my pangeo binder, but I am not sure if this also works for pangeo cloud.

Hi all, any chance one of you who understands the xesmf issue better than I could help me with @scottyhq's suggestion to open a PR for fixing this?  I would really like to be able to use xesmf again on the us-central1-b.gcp.pangeo.io cluster

@amv10070 if I understand https://github.com/pangeo-data/pangeo-docker-images/issues/101#issuecomment-670292539 correctly, it sounds like adding esmpy to https://github.com/pangeo-data/pangeo-docker-images/blob/master/pangeo-notebook/environment.yml will be all we need. Then bots will rebuild everything else.

@amv10070 and @shunzlu - please do open a PR with the instructions above ! Then we can get new images on the hubs ASAP.

closed by #122 so xesmf should work with pangeo/pangeo-notebook:2020.09.01 and later tags


Add sharp

When running the build, the following warning can be seen:
Warning: For production Image Optimization with Next.js, the optional 'sharp' package is strongly recommended. Run 'yarn add sharp', and Next.js will use it automatically for Image Optimization.
Read more: https://nextjs.org/docs/messages/sharp-missing-in-production

https://github.com/ReckfulArchive/twitter-website/commit/a4656cab990e20c96ebd9651723254d9abd74446


docs: 更新文档高亮

:tada: This PR is included in version 1.1.0-dev.1 :tada:
The release is available on:

npm package (@dev dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support for AWS Elasticache Autodiscovery on FactoryBean

can you help to release new version for this to maven central @killme2008 ?
thanks in advance


Plugin Development: Received Message to use python manage.py migrate

Environment

Python version: 3.7.7
Nautobot version: 1.0.0b3

Steps to Reproduce

Create a plugin with some model changes
Execute makemigrations step
Restart Nautobot (no migrate)

Expected Behavior
Expected the response to be run nautobot-server migrate to apply them

Observed Behavior
Got a message:
You have 1 unapplied migration(s). Your project may not work properly until you apply the migrations for app(s): <app_name.>.
Run 'python manage.py migrate' to apply them.

Sounds like we have two options for now:

Add documentation on this, optionally adding a nautobot-server command as a wrapper.
What @jathanism documented above overloading those two classes

Sorry about not being clear before. I should have stated that I didn't think the juice was worth the squeeze on this one, but do appreciate the effort spent on it nonetheless.

Yeah, so marking it as a feature and adding it to the MIlestone made me think that this was accepted.


Refactor to use Bottom Navigation View from Android Design Support Library

It might be wise to look into the opportunity for removing the dependency on the BottomBar component and use the official Bottom Navigation View from the Design Support Library 💃

@henningms Have you tried below package? It looks like it uses the new BottomNavigationView. If yes, pls let me know if its worth to switch from BottomNavigationBarXF to this one
https://github.com/NAXAM/bottomtabbedpage-xamarin-forms


[MONITOR-SPEC-PR] .Net Automation Failure

Description:
error CS1061: 'PlaywrightTestingQuotaData' does not contain a definition for 'FreeTrial' and no accessible extension method 'FreeTrial' accepting a first argument of type 'PlaywrightTestingQuotaData' could be found.
Additional context:
link1: https://github.com/Azure/azure-rest-api-specs/pull/31795/checks?check_run_id=34025726202
link2: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=4391014&view=logs&j=03afb3bb-7296-55ad-aa07-ceee610c73b2&t=0f320c29-1c90-5304-d2d3-c58a6bb76437&l=311

@jianingwang123 .net test failure indicates breaking changes, please comment in pr to ask whether this is expected in the future

@jianingwang123  the test failure indicates breaking changes, this package is not GA for stable version, so it's ok: check https://azure.github.io/azure-sdk/releases/latest/index.html to search for released packages


ResponseError: unknown command '502'

Hello, not sure if this is a properly placed issue report. I'm getting failed responses to 'GET' command like these (even though redis server is running):
value = self._client.get(key)

File "/srv/ask/node1/env/local/lib/python2.7/site-packages/redis/client.py", line 616, in get
return self.execute_command('GET', name)
File "/srv/ask/node1/env/local/lib/python2.7/site-packages/redis/client.py", line 394, in execute_command
return self.parse_response(connection, command_name, **options)
File "/srv/ask/node1/env/local/lib/python2.7/site-packages/redis/client.py", line 404, in parse_response
response = connection.read_response()
File "/srv/ask/node1/env/local/lib/python2.7/site-packages/redis/connection.py", line 316, in read_response
raise response
ResponseError: unknown command '502'

@andymccurdy Opt to close this issue. No provided additional information for 4+ years now. No provided way of reproducing the bug so no need to keep it open.


Algorithm change?

This morning I received this message
$python3 ~/tmp/zippyshare_downloader/zippyshare_downloader.py links
Starting Zippyshare Downloader for http://www66.zippyshare.com/v/VlNZgnnb/file.html...
Traceback (most recent call last):
  File "/home/debuti/tmp/zippyshare_downloader/zippyshare_downloader.py", line 74, in <module>
    dl(line.rstrip())
  File "/home/debuti/tmp/zippyshare_downloader/zippyshare_downloader.py", line 43, in dl
    auth_number = eval(re.findall('\(.*\)', download_link[0])[0])
  File "<string>", line 1, in <module>
NameError: name 'a' is not defined

Seems like the algorithm changed, I will take a quick look to broaden the report
Thanks

To solve this once and for all I'm going to implement a little phantomjs script to fetch the dl link, solving the involved JS of the page. Ill let you know when its ready

Hi!
Thank you for improve this handy tool!

Hey, i found a way to always interpret the correct algorithm using selenium:
from selenium import webdriver
driver = webdriver.Firefox()
driver.get(URL)
element = driver.find_element_by_id('dlbutton')
url_download = element.get_attribute('href')
print(url_download)

Nice! So you wont need any dependency again? Is it faster than phantomjs? The last one is very likely :/

I've made some tests, and using PhantomJS is better, because you don't need a browser window opening everytime :P
I have changed the main code.
Some things still need more work, buut, now it works without worry about algorithm change.
And another thing, i've seen some links from Zippy in this format: zippyshare.com/pd...
I think the URL validation need to use another way :(
Buut, works! :D


Fix the cookiecutter's tests

Oops: the DB option is called postgres not db, and there's no services option.

This works, but it worked in main too, but I suppose it wasn't testing what we thought it was. It seems we ignore invalid / unexpected arguments, so this was setting up the templated test environments incorrectly.

I think you can use a pre_gen_project.py hook to validate options, created an issue for that: https://github.com/hypothesis/cookiecutters/issues/142


[Batch] Data plane SDK v11.0.0

Updates the SDK to use the 2021-06-01 API version.

Note: The doc string changes here are caused by a regression in autorest.python and are out of our control unless we change our swagger not to use the title property. See the discussion at https://github.com/Azure/autorest.python/issues/978

Regenerated the models to fix an issue we found where switching some ContainerRegistry properties to optional re-ordered constructor arguments. We should be good to release!

the reddick haved changing opulate conflit of existence with debarquement of valkirye vendetta of my nursing haved changing by the conflit reddick he was den go assasinate by me and he was dead by nursing consulat of maitre assasin of illuminatie of my dauther neffretiti sheayrheaynhervha and is attesded by consulat of kevin pare my name and his was interessed by my profilence of git hub

Rebased on main to pick up the latest management client and reran the tests using it. Will merge this assuming CI passes.


Fix zero byte files

Fixes an issue where data transport errors that are retried could result in zero byte files being created in the cloud if the local file size is smaller than 4 MB. This is caused but not rewinding to the beginning of the file before retrying the upload.
Note that chunked uploads for files > 4 MB are already handled correctly.
Fixes #925 and #926.

Hope it's OK to comment here too (I'm new to GitHub, so not sure about etiquette). I'm not sure this issue has been fully fixed — I've been experiencing #925 in the latest versions, and several times since this fix. I've posted logs in my comment here: https://github.com/samschott/maestral/issues/925#issuecomment-2271967043


Update to node20

Fixes https://github.com/teunmooij/yaml/issues/1
Node 16 is deprecated, see https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/

Thanks for your contribution


Parse - "Missing push configuration" error on Heroku dashboard

I'm trying to implement push notifications in my android app. However, I am not sure why I am getting this error.
I did some research and saw that I had to configure stuff in index.js. But because I deployed to Heroku, my project doesn't contain an index.js file.
All I was able to to was to get an "Installation" class on my parse-server. But this installation doesn't send as a notification.

Hey there,
Did you deploy to heroic using their guide? If so, you can configure using heroku's Config Vars.
Let me know if it went alright
Kind Regards,
Ramon

I deployed to heroku by pushing the purple button @ https://github.com/ParsePlatform/parse-server-example.
I created my app and everything works, my only problem is trying to implement push notifications because I can't find an index.js file in my android studio project. Without index.js. I can't even use mailgun either.
I understand where to view/change my config variables. But how/where do I use them to implement push notifications?
@rvitorper

Hi again,
It is a great idea to use parse-server-example: a very easy setup, isn't it? There are, though, a few details you might want to pay attention: every configuration necessary you should pass as a Config Var from Heroku.
Knowing that you already understand that, I'll give you a nice tip that will help you with that. At the directory of the parse-server, there is a folder known as 'src', which inside is another folder named 'cli', which inside there exists another one named 'definitions'. Inside this last folder lies a file named 'parse-server.js' - or you can just click here.
The purpose of this file is to map configuration parameters to Config Vars names. So, in your case you need to provide a parameter named "PARSE_SERVER_PUSH". This has to be a stringified JSON in this format:
{
    "android": {
        "senderId": "Your GCM sender ID",
        "apiKey": "Your API key"
    },
    "ios": {
        "pfx": "path to P12 file",
        "passphrase": "optional password here",
        "bundleId": "bundleID here",
        "production": <true or false here>
    }
}

Make the json in a single line and feed that to your Config Vars and you should be set. In case you are having some trouble setting up some certificates from Google, this page might help. You'll find a way to find the GCM sender ID and the API Key.
Hope it helps,
Ramon

My project doesn't contain "parse-server-example:" thus, doesn't contain anything called 'parse-server.js'

Can I copy the file from here into my project directly?  @rvitorper

Hey,
I just updated the answer, for the last one I got wrong. This one will require though that you have done GitHub integration with Heroku
@grantespo

Thank You for your quick replies,

How/Where do I redeploy to my dyno?

How do I get to the file where I add the parameter?

PLEASE explain everything clearly STEP BY STEP to fix my issue because I am new to Heroku @rvitorper

Since this is not an issue with Parse Server itself, I'm closing this issue. Feel free to keep getting help from @rvitorper and other contributors in this thread.

Hi there,
Hope the modifying index.js and pushing to GitHub part went alright.
At Heroku, go to the Deploy tab:

Then, scroll down and locate Manual deploy. Select the branch you're working with and click Deploy Branch:

Any problems, let me know @grantespo
Ramon

Hello, I don't see "Manual Deploy",

At the bottom of the attached picture, it hints "repo-name". Do I put something in there? What exactly do I put?
Is there a way I can private chat you to figure this out? Because All I need to do is implement these push notifications so I can publish my app.
@rvitorper

I just want to send Push Notifications, can you tell me how step by step? I don't know how to get into my cloud code. I don't where my cloud code is. And I don't know what to put in my cloud code to make push notifications works. Can you please tell me everything step by step? There is no clear documentation on this. @rvitorper

Hi again,
You didn't integrate with GitHub, that's why it didn't work. In case you don't want to have so much trouble dealing with a Parse Server, you can use a BaaS solution, such as the ones described in this article. However, if you want to use parse-server, please remember to integrate your GitHub with Heroku.
Ramon


Fix panic under race condition (send on closed channel)

There's a race condition that usually happens under high load when the response time of SMPP server is equal to the go-smpp timeout.
When the SubmitSM is sent, go-smpp waits for SubmitSMResp for RespTimeout duration.
If the SubmitSMResp comes in the exact moment t.cl.respTimeout() kicks in, then the channel in t.tx.inflight is already closed, but HandlePDU already got the channel from this map in parallel and tries to send SubmitSMResp there. And here we get "panic: send on closed channel".
Let's just skip the channel close and let HandlePDU write there and then, after the channel entry is deleted from the t.tx.inflight map, the channel will be garbage collected.

Hey, thanks for this. Good catch.


[bug]: Input component can't handle type tel and pattern

Describe the bug
Input component can't handle type tel and pattern. I can type letters on it even type is tel.

Phone number

Affected component/components
import { Input } from "@/app/components/shadcn/ui/input";
How to reproduce
Use input component on page like this template then write letter on it.

Phone number

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
OS Name	Microsoft Windows 11 Pro
Version	10.0.22631 Build 22631

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

@algorithm-developer Input type pattern is not supposed to block the user on entering invald pattern text. On submitting the value in a form, it will pop up validation error.
Your issue for input type tel is valid, though!

I can take this up.

Raised PR for the issue
https://github.com/shadcn-ui/ui/pull/3560

@algorithm-developer Input type pattern is not supposed to block the user on entering invald pattern text. On submitting the value in a form, it will pop up validation error. Your issue for input type tel is valid, though!

Example:
I want to take phone numbers from clients (users). I must put input type to "tel" because without that users can type letters on it. After set type and pattern its still taking letter inputs on it even type is set to "tel" and even pattern set.


Add new server for reading your Apple Notes

Thanks for reaching out about this list, making it and an awesome post you wrote to help me understand how to even build it.
This adds a link to my MCP server for reading your apple notes locally.

Thank you!


Sparse array iterators include index position.

Iterators over the sparse array structures have gained an initial argument which indicates the index into the array of the element.  This can be used, e.g., to delete or modify the associated value.

[x] documentation is added or updated
[x] tests are added or updated

Merged to master.  Thanks for the feedback.


modules/flower-power improvements [v2]

Changelog:

drop changes on json parser and handle errors on module
cancel pending http requests
renames on init macro

+1

Yeah, on i2c I would name it "handle" or "operation".
Naming a struct with an adjective instead of a noun doesn't seems clear to me, specially with one that is a temporary state (on some cases the connection won't be pending anymore, as it's the case of the finish_cb).
Also you're not sure what is pending. For http-client I was able to guess it was a connection, but for i2c I had to read the documentation.

so go ahead, you're working closer this api than I. We still can change the i2c api soon :)

merged with proposed change


Doesn't work with Pandas 2.0 and SQLAlchemy 2.0

Zipline doesn't work with Pandas 2.0 ("Algorithm should have a utc datetime") and SQLAlchemy 2.0 ("The 'future' parameter passed to create_engine() may only be set to True").
The latest pre-2.0 versions seem to work, at least I can run a simple test program:
import pandas as pd
import pandas_datareader.data as web
from zipline import run_algorithm
from zipline.api import order, record, symbol

def initialize(context):
    pass

def handle_data(context, data):
    order(symbol('AAPL'), 10)
    record(AAPL=data.current(symbol('AAPL'), 'price'))

start = pd.Timestamp('2014')
end = pd.Timestamp('2018')

sp500 = web.DataReader('SP500', 'fred', start, end).SP500
benchmark_returns = sp500.pct_change()

result = run_algorithm(start=start,
                       end=end,
                       initialize=initialize,
                       handle_data=handle_data,
                       capital_base=100000,
                       benchmark_returns=benchmark_returns,
                       bundle='quandl',
                       data_frequency='daily')

Environment:

Kubuntu 22.04
Python 3.10
pip
zipline-reloaded 2.4

Hi, did you solve this ["The 'future' parameter passed to create_engine() may only be set to True"] ? I have the same mistake here.

pandas = 2 ans sqlalchemy >= 2 , introduce non-backward compatible changes. You need to downgrade pandas and sqlalchemy

Please retry with latest, just released version.


How to get the ID ACC

Hello, I don't know how to get the ID ACC based on your code.

As discussed in our paper and replied in email. The ID acc is get through linear probing on our trained feature extractor using PALM


Fix params referencing

Pull Request Test Coverage Report for Build 27

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.541%

Totals

Change from base Build 23:
0.0%

Covered Lines:
100

Relevant Lines:
102

💛  - Coveralls


AIMA4e reimplement Fig 4.24 LRTA*-Agent from AIMA3e branch #133

As a design decision , AbstractAgent class was dropped in preference to using action and percept as parameter making UI clean and code easier to understand.
@ctjoreilly

Have made necessary changes in related classes to make it compatible , improved documentation to make code easier to understand. Is is upto expectations sir?
@ctjoreilly

Missed out adding some casses. Will be adding them soon.

Have tried to use Optional class added by java 8. Moreover i would recommend using Optional class for implementation of all NoOp in AIMA4e branch. We can also use Lambda Expression for anonymous classes to make their implementation more clear. Stream can be used inplace of ArrayList at some places. If selected for GSoC 2016 i would like to do these and more and would love to continue beyond GSoC as a contributor to this project helping new community members.
@ctjoreilly
Regards
Subham Mishra


Command: "enableSharedFolder", error: the virtual machine is not powered on

Let me start off with saying that DetectionLab is really awesome work and loving it ever since I discovered it!

Operating System Version: Windows 11 - 21H2 OS BUILD 22000.376
Deploying via (VirtualBox/VMWare/AWS/Azure/ESXi): VMWare
Vagrant Version (if applicable): Vagrant 2.2.19

Please verify that you are building from an updated Master branch before filing an issue. ✅
Description of the issue:
While bringing any box up (logger, dc, wef or win10) the following error is presented:
An error occurred while executing `vmrun`, a utility for controlling
VMware machines. The command and output are below:

Command: ["enableSharedFolders", "D:\\DetectionLab\\Vagrant\\.vagrant\\machines\\win10\\vmware_desktop\\d929dfab-247c-470f-9772-8300c26661d6\\windows_10.vmx", {:notify=>[:stdout, :stderr]}]

Stdout: Error: The virtual machine is not powered on: D:\DetectionLab\Vagrant\.vagrant\machines\win10\vmware_desktop\d929dfab-247c-470f-9772-8300c26661d6\windows_10.vmx

Stderr:

I can verify that the machine is in fact up and running:
> "C:\Program Files (x86)\VMware\VMware Workstation\vmrun.exe" -T ws list
Total running VMs: 4
D:\DetectionLab\Vagrant\.vagrant\machines\logger\vmware_desktop\4d1820ef-86a7-43e5-8da5-0e08f4478556\ubuntu-20.04-amd64.vmx
D:\DetectionLab\Vagrant\.vagrant\machines\dc\vmware_desktop\b2ca2ca9-c264-461f-8d9b-5342290ac81c\WindowsServer2016.vmx
D:\DetectionLab\Vagrant\.vagrant\machines\wef\vmware_desktop\c3b7afe4-cf0d-45f9-9d8f-c483642c96ef\WindowsServer2016.vmx
D:\DetectionLab\Vagrant\.vagrant\machines\win10\vmware_desktop\d929dfab-247c-470f-9772-8300c26661d6\windows_10.vmx

This also happened a few times when bringing up the machines after completely removing them. After a few runs of vagrant reload [host] --provision eventually it  completed the sequence without this error. Some things are working: given that the logs are coming into Splunk with no problem (did not test all functionality).
However some things seem to be a bit flaky, e.g. pinging hosts from the DC (also can't reach the c$ share of win10.windomain.local, neither works via IP):

Any idea what might go wrong here? Everything destroying all VMs and or repeatedly reprovisioning them doesn't sound like the correct solution

I see that the issue is already described before in: https://github.com/clong/DetectionLab/issues/289 and https://github.com/clong/DetectionLab/issues/447.
Let me try those solutions and report back if the issue is persistent, is there something that might have reintroduced this behaviour?

What did work for me is as soon as the script has reached:
==> win10: Enabling and configuring shared folders...

You reinstall the VMware tools manually, after that it will pick up automatically and continue the provisioning.

I had the same error. And what i have done to solve this is:

when it raises a err like: "Command: ["enableSharedFolders", ....", log on to DC machine (VMWare in my case), and install VMtools manually (VM -> Install VMWare tools)

try vagrant reload dc --provision to reload DC machine, again!

Its worked! modern problems require modern solutions! :) I think UAC (when) is the main problems to cause this issue (correct me if im wrong)

Hi folks, there more context here (https://github.com/clong/DetectionLab/issues/720) but the long story short is there's a race condition when the hostname is getting set. When the VM boots for the first time, it also has to configure VMware tools to re-configure itself (due to sysprep having run). If VMware tools setup completes before the hostname change, everything will work fine. If not, you'll run into this issue. I still haven't found a solid fix for this yet.

This might be solvable by disabling the Vagrant hostname setting and just using powershell to do it via Rename-Computer

Closing as this is now being tracked in #720


README: Added dependencies for Fedora

Pull Request Overview
Adding Fedora deps and instructions required to cross compile
libtock-c apps for arm/risc V.
Testing Strategy
make RISCV=1 and make on Fedora 35
Documentation Updated
README updated
Signed-off-by: Wilfred Mallawa<EMAIL_ADDRESS>

I have not tested this, but always helpful to have build instructions for additional distros, thank you for the contribution!

I have not tested this, but always helpful to have build instructions for additional distros, thank you for the contribution!

No worries!

You probably want to split some of that into the next enumerated item below. This one is installation for arm, while the next is for RISC-v.

Updated, let me know if that's not what you meant.

Thanks so much!


Load test plutus playground

We will be monitoring the playground during the summit however we should carry out some load testing against a full environment to see what the effects of load are, what we should alert on and how we should scale.

I’ve spent most of today doing load testing on plutus playground and meadow. Since these machines are basically GHC as a service they suffer CPU saturation as the number of concurrent requests increases. I have tried increasing instance size using the high compute instances and this type of scaling works very well. Our problem is that we have no idea what the levels of traffic will be :disappointed:
Given this I suggest that on the morning or evening before the conference we up the machines to something beefy like a c5.9xlarge and keep an eye on CPU use through the day, towards the end of the day we can hopefully ramp down considerably. Under normal circumstances this adjustment can be made slowly as traffic will be smoother and its easy to judge from average CPU use whether we need to scale up or down.
Scaling on the fly only gives about a minute or so of down time anyway so in the worst case scenario we could go up even higher without too many user issues however I imagine that in reality traffic will end up being very low, I just want to give us a safety net during the conference.

Additionally, in the future we could move to cheaper high-compute bare metal machines.


Use Kueue API types in FlavorAssignerTest

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
We update the FlavorAssignerTest to use Kueue API types, rather than construct CQ/Cohort snapshot manually. This will allow us to change CQ/Cohort Snapshot logic without updating the tests each time.
Preparation for main changes in #79
Does this PR introduce a user-facing change?
NONE

@macsko

Looks good, two small comments

thanks for the review

/lgtm
/approve
/hold
in case @alculquicondor wants to give a pass

LGTM, just one nit to rename the test

LGTM, just one nit to rename the test

Done. Thanks for the review!

/lgtm

/hold cancel
I think we are good to merge as all comments are addressed. We can follow up in case @alculquicondor has another comment.


Investigate Renaming the Drill Method

From a comment on the reviews, someone suggested that we rename the drill method so that it is more apparent what it does.

We determined that this would go beyond the scope of the project given the way the drill method is implemented.


[Custom request] enable analytics on docs

https://squidfunk.github.io/mkdocs-material/setup/setting-up-site-analytics/


FDCAN: Fix offset issue preventing CAN2 and CAN3 from working.

Fix for https://github.com/embassy-rs/embassy/issues/2702
Tested for CAN2 but should fix CAN3 on devices that support it also.

+Trusted please?


[DO NOT MERGE] Send first_published_at to GA when publishing

For: Trello card
This will send a GA hit whenever an edition is published for the first time.
The information will be sent along as an event, using the trackerId currently in
use in govuk_admin_template (tid: UA-26179049-6).
The client id we're sending (cid: 660ad712-9753-4cb9-97a7-c9e9f13c318e) is
randomly generated and can be used to identify that the hits are coming from the
publishing-api rather than govuk_admin_template or govuk_frontend_toolkit (both
apps are normally used to send analytics information).

This feels like something that belongs in the sidekiq worker for downstreaming live content: https://github.com/alphagov/publishing-api/blob/master/app/workers/downstream_live_worker.rb
Ideally it'd be something that listened to the Publishing API message queue rather than integrating directly - but not sure how much that messes with your scope. We have email alert api and search that listen to the message queue currently.

@kevindew I've moved the sending of the event to the DownstreamLiveWorker

Following a conversation with Tim from analytics, it seems that sending this info via the publishing-api won't provide the analytics team with enough information, so I'm closing this PR.


get_dummies uses the prefix parameter whose type is dict return KeyError

Use get_dummies demo with prefix  and columns

pandas result like this:

cc @itholic can you take a look when you're available?

@HyukjinKwon Sure, leave it to me. I'll take it.

This is because It didn't consider about dict as a prefix type.
I just created a PR to fix this.


Patch page model method

Patch required. The file was copied into these projects from a modified template without reverting the method name in the template.

@guardrex Please resolve the conflict, and then this PR is good to merge.


chore: sunday-daniel to 0.0.1

chore: Promote sunday-daniel to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


hyper-parameters

I notice that:
# Based on translation qualities, choose different weights here.
# For AG News: German: 1, Russian: 0, ori: 1
# For DBPedia: German: 1, Russian: 1, ori: 1
# For IMDB: German: 0, Russian: 0, ori: 1
# For Yahoo Answers: German: 1, Russian: 0, ori: 1 / German: 0, Russian: 0, ori: 1
You give the setting of hyper-parameters for AG News, DBPedia, IMDB, and Yahoo Answers dataset.
It means German: 1, Russian: 0, ori: 1 and German: 0, Russian: 0, ori: 1achieves similar results?
In other words, can I use any setting to get the results reported in the paper?

You could use the setting mentioned in the code to get the results reported in the paper.


kubectl label output from file input with multiple objects does not include a document separator

What happened:
When running "kubectl label" with file input containing multiple objects no document separator is included in the output. This means only the last object will be picked up to be labelled on the cluster (or passed on to an apply command etc).
What you expected to happen:
Multiple objects labelled with a document separator in between.
How to reproduce it (as minimally and precisely as possible):
cat <<EOF > /tmp/cm_test.yaml
apiVersion: v1
items:
- apiVersion: v1
  kind: ConfigMap
  metadata:
    name: test1
- apiVersion: v1
  kind: ConfigMap
  metadata:
    name: test2
kind: List
metadata:
  resourceVersion: ""
  selfLink: ""
EOF
cat /tmp/cm_test.yaml | kubectl label testlabel=foo -o yaml -f - --local

Output:
apiVersion: v1
kind: ConfigMap
metadata:
  labels:
    testlabel: foo
  name: test1
apiVersion: v1  # No record separator here
kind: ConfigMap
metadata:
  labels:
    testlabel: foo
  name: test2

Anything else we need to know?:
The same process and data but using the "annotate" command instead of "label" works perfectly and includes document separators. as per output below.
apiVersion: v1
kind: ConfigMap
metadata:
  annotations:
    testlabel: foo
  name: test1
---
apiVersion: v1
kind: ConfigMap
metadata:
  annotations:
    testlabel: foo
  name: test2

Environment:

Kubernetes client and server versions (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.0", GitCommit:"4ce5a8954017644c5420bae81d72b09b735c21f0", GitTreeState:"clean", BuildDate:"2022-05-03T13:46:05Z", GoVersion:"go1.18.1", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.4
Server Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.5+70fb84c", GitCommit:"3c28e7a79b58e78b4c1dc1ab7e5f6c6c2d3aedd3", GitTreeState:"clean", BuildDate:"2022-04-25T15:58:12Z", GoVersion:"go1.17.5", Compiler:"gc", Platform:"linux/amd64"}

Problem stems from that label initializes new printer for each object https://github.com/kubernetes/kubernetes/blob/10eb7092f854c71122c03752465e868bce23c0b6/staging/src/k8s.io/kubectl/pkg/cmd/label/label.go#L390 unlike to annotate.
/triage accepted
/assign

Currently YAMLPrinter only adds document separator(---) if the object count is more than 1 for one YAMLPrinter object https://github.com/kubernetes/kubernetes/blob/e4fca6469022309bc0ace863fce73054a0219464/staging/src/k8s.io/cli-runtime/pkg/printers/yaml.go#L48.
However, for the command label(and for a couple of more commands), there is no one YAMLPrinter, instead to manage the message in printer object, YAMLPrinter is initialized per object https://github.com/kubernetes/kubernetes/blob/10eb7092f854c71122c03752465e868bce23c0b6/staging/src/k8s.io/kubectl/pkg/cmd/label/label.go#L390.
That causes each YAMLPrinter has it's count to 1. Therefore, they are not adding document separator.
This simply works for annotate command because print message is not being changed for annotate command(there is also an issue for that problem https://github.com/kubernetes/kubernetes/issues/110123) and printer is initialized once in completed.
I wonder would it be possible to add document separator in any case without checking the counter in here https://github.com/kubernetes/kubernetes/blob/e4fca6469022309bc0ace863fce73054a0219464/staging/src/k8s.io/cli-runtime/pkg/printers/yaml.go#L48
@eddiezane @brianpursley @soltysh
/unassign

Hi! I would like to take a look at this issue and see if I can help with it.
/assign

/remove-lifecycle stale

It looks like PR #110124 is already addressing this issue and has been hardened and reviewed for a while now.
I'm not really contributing to sig-cli anymore so I'll not look further into this.
Unassigning and closing my PR so another person can tackle it!
/unassign


refactor: histogram creation logic

factoring out the check for whether a histogram is needed

Codecov Report

Merging #32 into master will increase coverage by 0.21%.
The diff coverage is 10.52%.

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
+ Coverage   50.00%   50.21%   +0.21%
==========================================
  Files           9        9
  Lines         454      456       +2
  Branches       66       66
==========================================
+ Hits          227      229       +2
  Misses        222      222
  Partials        5        5

Impacted Files
Coverage Δ

src/cabinetry/template_builder.py
45.16% <0.00%> (+7.82%)
:arrow_up:

src/cabinetry/workspace.py
29.71% <0.00%> (+0.42%)
:arrow_up:

src/cabinetry/configuration.py
70.17% <11.76%> (-24.83%)
:arrow_down:

Continue to review full report at Codecov.

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


lower visibility_timeout for send-email-high queue

Summary | Résumé
Analogous to the work in #976, set the visibility timeout for the send-email-high queue to 26 seconds.
Test instructions | Instructions pour tester la modification
Once in staging, do a high priority soak with a low priority bulk and ensure that the send-email-high delay doesn't get too big.
Release Instructions
THIS RELEASE REQUIRES AN IMPORT FIRST
Find the sqs queue url  from the console
cd env/production/common
terragrunt import -var-file <prod.tfvars> aws_sqs_queue.eks_notification_canada_ca_email_high_queue <sqs queue url - found in AWS Console>

resource imported


DevUI: Add sorting to table columns

I think this is right @phillip-kruger I looked at some other examples.  We should probably add filtering too since there can be a ton of loggers.

Thanks @melloware !


[WIP] Openshift/release reconciler

TODO:

[  ] Image mirroring

Signed-off-by: Pierangelo Di Pilato<EMAIL_ADDRESS>

I might be missing something obvious (as a python noob). Is there any benefit writing it in .py instead of .go? go install/go run seems like a nice tool to distribute and reuse such tool.

Not a strict requirement but some improvements can be done to make code more Python-like as pylint complains for several stuff.

I think this looks good now...

/unhold


Add support for itemProperties in schemas

These edits allow you to use itemProperties in a schema.js file, like:
input: 'textarea',
itemProperties: {
  layout: 'inputOnly'
},
...

See Slack discussion at https://vulcanjs.slack.com/archives/C02L990AF/p1574895832425900 thru https://vulcanjs.slack.com/archives/C02L990AF/p1574900591428300

Thanks!


how to uncheck all checkbox by default?

Is there an option to uncheck all checkbox by default?

Sorry. There is no option. Please try to set: checked=false :(


Map.replace() fixed

Current coverage is 91.52%

Merging #820 into master will not affect coverage as of b4ab58d

@@            master    #820   diff @@
======================================
  Files           77      77
  Stmts         7895    7895
  Branches      1585    1585
  Methods          0       0
======================================
  Hit           7226    7226
  Partial        201     201
  Missed         468     468

Review entire Coverage Diff as of b4ab58d

Powered by Codecov. Updated on successful CI builds.

thx!


new version

Hi guys!
Thanks for this conda package. The current version of texttable is by now  1.1.0, while this conda package is still at 0.9.1. It is possible to update the package in conda-forge?
Thanks!

Fixed by #3, thanks for the heads up.


Use evaluate_function for CallStack::PackageRun; Sample response for CallStack::CheckDeployment

Motivation
If we evaluate_function during deployment, we risk hitting false positive failed constraints on our random assignment.
So instead, this PR proposes to sample a response. After sampling, construction of Response is handled the same way evaluate_function.
Test Plan
In the future we will test a wider range of programs.
Related PRs
https://github.com/AleoHQ/snarkVM/pull/2176
https://github.com/AleoHQ/snarkVM/pull/2224
https://github.com/AleoHQ/snarkVM/pull/2226

Did another walk through the code, running on a local devnet and deploying and executing max nested programs succeeds.


MSRV is 1.71 and not 1.70

Steps to reproduce the bug with the above code
Since I use the code as a subdmodule in a different project, I can't write the reproduction steps here. But I assume, running the CLI would be enough.
All I can say is that local rust version is 1.70
Actual Behaviour
rustdoc format v24 for file ....target/semver-checks/target/doc/clap.json is not supported

Expected Behaviour
Tool to work
Generated System Information
Used the code from the commit 438f6d80a368e9d119159ce44e27ca1bf692b9ab which is a few days ago.
Build Configuration
No response
Additional Context
No response

Yes, you're totally right — sorry for finding this out the hard way. I accidentally failed to include this in the release notes for v0.25. I have now updated the release notes.
Rustdoc prior to Rust 1.73 has a bug where re-exports of #[doc(hidden)] items are incorrectly omitted from the JSON output, which can cause false-positive lints to be reported. Rust 1.71-1.73 all use the same rustdoc JSON format so technically the MSRV is 1.71, but 1.73+ is strongly recommended. I dropped Rust 1.70 which used an older rustdoc JSON format, and would have dropped 1.71-1.72 as well if that were feasible, but it was not.
Unfortunately I failed to include this in the release notes, and instead it remained in my draft release. Sorry about that. I've since updated the release.
(Sidenote: cargo-semver-checks has two different flavors of MSRV: the minimum version which it requires to compile, and the minimum version whose rustdoc it can check. This affects only the latter (can't check Rust 1.70), not the former (it can still compile and work fine on 1.70, as you found). The "compile MSRV" should always be <= the "check MSRV.")

Thanks for highlighting these. I should add more info to the README.

There is a reference to 1.70.0 in README.md and Cargo.toml. I assume that you want them to also represent the check MSRV.

Those are both the compile MSRV, since Cargo.toml is what cargo install will read.

I don't think there's a need to ever talk about compile MSRV at all, because we can always treat the check MSRV as the only important one.

This is tricky. In principle, it should be true. In practice, we've had (rare!) situations where the compile MSRV was actually higher than the check MSRV since our dependencies' MSRV got bumped up but the rustdoc JSON format remained the same.
Rust 1.70 was such a case: lots of libraries bumped to MSRV 1.70 but rustdoc v24 was in 1.68, 1.69, and 1.70 — so cargo-semver-checks needed to be compiled on 1.70 but could then check output as old as 1.68. This is the only time in ~a year and a half this has happened, but I think it could happen again in the future so I'm hesitant to make the change you suggest.

Opened #598 to add more detail in the MSRV section of the README. If you have a chance to check it out, I'd love your input (but no pressure).


no save buttons in portal v2.0.2

Hi Joe,
Upgraded system to new v2.0.2 with lite options.
Had problems initially with saving settings when running ip_of_device/install
Deleted file /var/www/html/classes/settings.class.php and ran
ip_of_device/install    one more time
no save buttons to save user settings in web portal so for example I cannot save wlan0 for monitoring, keeps reverting back to default eth0.

Fixed in commit d86c7e1028943698882d42bd5e61531136e8d35b.
You grabbed this version a little bit early doing final testing now on a new image.


⚠ Thumbnails API Endpoint has degraded performance

In 3e7ae1e, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=1427869852&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1367 ms

Resolved: Thumbnails API Endpoint performance has improved in 0bd5b24 after 18 minutes.


QoL fixes

close #665
close #666

To test:

cant select biomarker button in quickmaps sidebar
single plausible script tag added instead of multiple (console breathes a sigh of relief)
plausible registering page changes for analytics

the surplus script tag is not visible when use view source as before but you can see it is there when ciewing elements in chrome debug tools. The plausible stats page is also showing realtime data to 'follow' you about, so it looks to be still working as before.


python3Packages.keyring: 23.2.1 -> 23.3.0

Motivation for this change
https://github.com/jaraco/keyring/blob/v23.3.0/CHANGES.rst
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg build python38.pkgs.keyring python39.pkgs.keyring

@ofborg build python38.pkgs.keyring python39.pkgs.keyring

@ofborg build python38.pkgs.keyring python39.pkgs.keyring

If you name your commit python39Packages.keyring then ofborg will build the package automatically.

If you name your commit python39Packages.keyring then ofborg will build the package automatically.

This is true of anything before the :.
I think @dotlambda wanted to just test it on both 38 and 39.

This is true of anything before the :.

Not really. It needs to be a package that's changed by the PR I think, which python3Packages.keyring "isn't" cause python3Packages isn't recursed into.

I think @dotlambda wanted to just test it on both 38 and 39. Which ofborg should get a cache hit on the 39

I mostly wanted to test Darwin, but I don't like how a commit message with python39Packages.keyring looks.


SCP Workflow - Ex.03 -Trouble Getting the SAP Cloud Connector Started

@qmacro
Based on: SAP Cloud Platform Workflow - Ex.03 - Installing & configuring the SAP Cloud Connector
Having trouble getting SCC / localhost:8443 up and running with the following error  on a windows 10 machine after following the instructions to the T ( Only hint i've got is https://imgur.com/a/kyTCUdw ). Files: sapcc-2.12.4-linux-x64.zip, sapjvm-8.1.064-linux-x64.rpm.
Also raised it here:
https://github.com/nzamani/sap-cloud-connector-docker/issues/18

Hi @sammok83 let's see if we can get to the bottom of this. The screenshot you shared only shows some parts of the container build, but no errors relating to the running of the container.
Please describe in detail what's happening, and what's going wrong.

Hi @qmacro,
Here are the details:
Files used are as per SCP Tools:

sapcc-2.12.4-linux-x64.zip
sapjvm-8.1.064-linux-x64.rpm

Executed the commands as per steps 2 and 3 of the instructions and got this.
https://imgur.com/Z9GtTCe
Upon running https://localhost:8443:
Error
Don't see any errors but let me know what else I can do on my end to provide you with more info?
Thanks!

Ah, ok now I understand @sammok83 - it's not an error, in fact, it's described directly below where it tells you to open up https://localhost:8443, in step 4 of the exercise:
Your browser will likely warn you that the site is insecure, because the certificate that the site presents (via HTTPS) has not been signed by any authority it recognizes. This is OK for what we want to achieve in this Virtual Event, and you should proceed through any warning. It's possible to fix this by installing a signed certificate into the SAP Cloud Connector, but this is beyond the scope of this exercise.
The browser page should give you an option to proceed - if you're using Chrome, as directed in the exercise prerequisites, there should be an "Advanced" button you can use for this.
Good luck and have fun!

@qmacro
Thanks for responding.
I'm well aware of the advanced button shown in browsers.
However, there's no advanced button - only a reload button.
Please find a sample of what it looks like for https://localhost:8443 and a client site:
https://imgur.com/a/gmiUMRF

Can you please show a screenshot of the entire browser window?

https://imgur.com/a/IhLDZnq

Hi @sammok83, @qmacro
please check my response here https://github.com/nzamani/sap-cloud-connector-docker/issues/18

@sammok83 Did you get this to work? I have tried multiple things but failed!

I uninstalled and reinstalled docker and ran through the complete tutorial again. The "Failed D-bus connection" disappeared, but was still not able to access localhost from Google Chrome but at least it is working from Internet Explorer. Could be some firewall issue, not sure!

I have no issues on Safari on my macOS to access the SAPCC by an additional click:

On macOS Chrome, however, the good old "access anyway" button is not displayed anymore - no clue why.
Access must work on any browser. And here is the trick for Chrome on your macOS (should work on Windows as well):

Focus the chrome pages which shows the warning by clicking somewhere on the page
Just type thisisunsafe (don't think about it, you'll see)
Here you go... :-)

So I'd call this a hidden feature. I've just documented this here.
ps The dockerfile was updated and should now run just fine. Plus it uses now the latest SAPCC 2.13.0

I have no issues on Safari on my macOS to access the SAPCC by an additional click:

On macOS Chrome, however, the good old "access anyway" button is not displayed anymore - no clue why.
Access must work on any browser. And here is the trick for Chrome on your macOS (should work on Windows as well):

Focus the chrome pages which shows the warning by clicking somewhere on the page
Just type thisisunsafe (don't think about it, you'll see)
Here you go... :-)

So I'd call this a hidden feature. I've just documented this here.
ps The dockerfile was updated and should now run just fine. Plus it uses now the latest SAPCC 2.13.0


Fix https://github.com/tbrand/which_is_the_fastest/pull/11

I'v fixed https://github.com/tbrand/which_is_the_fastest/pull/11.
If @audy is ok, I'll merge this.
Thanks:smile:

Thanks! I haven't had a chance to look at it yet.
--austin

On May 2, 2017, at 20:25, Taichiro Suzuki<EMAIL_ADDRESS>wrote:
Merged #14.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


pug nested template not work.

<template lang="pug">
    <template src='./_new.pug'></template>
</template>

This not gives error, but _new.pug not rendered.

Hey 😁! Is that internal <template> a pug feature? Any markup code inside your outer <template> must be in the language you defined on the lang or type attributes

no its not a pug feature.
I tried https://pugjs.org/language/includes.html and it works. thanks!


0.6.0 release fixes

Failed attempt at 0.6.0: https://github.com/opendp/opendp/actions/runs/3323457930/jobs/5493829582
Seems that Rust now needs GLIBC 2.17: https://github.com/rust-lang/rust/blob/master/RELEASES.md#compatibility-notes

Closed in #590, #591


Can't select correct backlight pin in Guition ESP32-S3-4848S040 firmware

In the firmware esp32-s3-4848s040_full_16MB_v0.7.0-rc12_cd34d89
it is not possible to select the correct pin.
The wrong pins are offered for selection, and the correct pin 38 cannot be selected

In the previous firmware version (0.7.0-rc12 82e4095) there was no such problem

Backlight GPIO is set at compile time to GPIO38. If you need to change it, you'll need to edit TFT_BCKL in user_setups/esp32s3/esp32-s3-4848S040.ini

It is clear that you can compile anything, but I wrote about an error in the already created firmware for a particular model, which in fact does not work.

I have the ESP32-S3-4848S040 display myself. The backlight GPIO pin is hardwired in to the board, so there is nothing to set as such. I had submitted a change recently that hid most of the hard wired GPIO (mostly related to TFT control) from the user. See commits c70dfe6 and 7a3b8d0
Perhaps if the backlight GPIO is hard coded, the option to change it should be blanked out so as to avoid confusion.

If you leave the field empty, the brightness control does not work, i.e. everything less than 255 is perceived as 0.
To be more precise, there is probably no direct connection, but firmware esp32-s3-4848s040_full_16MB_v0.7.0-rc12_cd34d89 has exactly this behavior.

Will have to do a build with the latest code (I'm running a two week old build here).
Issuing a backlight("state":"on","brightness":25} dims the backlight as expected. Won't have time to dig any deeper until the weekend.

The backlight pin should be changeable, TFT_BL is treated as a preference not hard-coded. Sp this shouldn't be checked here, as it prevents selecting the preferred pin:
#ifdef TFT_BCKL
       || (pin == TFT_BCKL)
#endif

@tdvtdv Does the above commit solve your issue ?

I can't check it yet, because I took a ready-made build from nightly.openhasp.com, and there is an old version there.

Just checked out and compiled - Backlight defaults to GPIO 38, and can be selected via the web interface.
Version is displayed as: 	0.7.0-rc12 bbd96b2
Latest version available on nightly.openhasp.com is 0.7.0-rc12 cd34d89 which is over a month old - Time to trigger a new nightly build ?

It's not automated to the point that it's just a "trigger" but still required manual work.
Do you have experience with GitHub automation and front-end building?
It would be nice to automate this indeed, but no time for that myself atm.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-source-version-target/pull/74 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-source-version-target.git

Docs Build status updates of commit 2919176:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel