File Operations on selected files

I'm loving that I can view a ton of SVG files at once, and select them, but currently I can't do anything with them. So delete/copy/move/mass rename might all be very useful features to have on a folder full of images.

@pepsakdoek Yes, the application already has the ability to select multiple files, and later a menu with some standard file actions will be added.

Hello @pepsakdoek!
Update <IP_ADDRESS>5 has been released. Now, if you select multiple (!) images in the gallery (using Ctrl+Click or Shift+Click), and then Right-Click on any of them, then a Windows Explorer will open, with all these multiple SVG files selected. So you can perform actions on multiple files in Explorer - Copy, Cut, Delete. etc.
Also, maybe later it will be possible to do file operations in the application itself, but turning an application into a file manager is a more difficult task than it seems.


 Also, I feel if these were all symlinks to ~/.cargo/bin/rustup, that would show more clearly (to someone figuring out what lives where and how which component of rust is called) that these are all wrappers.

They're already hardlinks

Oh, I guess I could have checked that.
I think symlinks would be clearer (because anyone doing ls -la would see what is going on, and that these are wrappers). But oh well.


Lumping support for multi-word prepositions?

e.g. 2 word prepositions:
according to
adjacent to
ahead of
apart from
as for
as of
as per ...
e.g. 3 word prepositions:
as well as
at the behest of
by means of
by virtue of
for the sake of ...
The single term prepositions are recognized but not the multi-word.

hi John! this is a good point. I think we should support a bunch of these examples. Do you have more? I think most of them are good candidates to drop into the lexicon straight-up.
I guess it's no longer a preposition if it conjugates? 'adjacent to' works, but 'close to' doesn't?
happy to add these!
cheers

Yes, I have more examples. Here is a snippet from my data showing the two and three word prepositions and also attached file list of prepositions 1, 2, and 3:
according to
adjacent to
ahead of
apart from
as for
as of
as per
as regards
aside from
astern of
back to
because of
close to
due to
except for
far from
inside of
instead of
left of
near to
next to
opposite of
opposite to
out from
out of
outside of
owing to
prior to
pursuant to
rather than
regardless of
right of
subsequent to
such as
thanks to
up to
as far as
as opposed to
as soon as
as well as
at the behest of
by means of
by virtue of
for the sake of
in accordance with
in addition to
in case of
in front of
in lieu of
in order to
in place of
in point of
in spite of
on account of
on behalf of
on top of
with regard to
with respect to
with a view to
prepositions.txt

thanks, i'm just working on a pretty big re-write. will add these in once things are stable.
in the meantime, you can augment the lexicon, as you wish.
cheers

It was easy to roll out my (less developed) NLP and use NLP-compromise.  As you have seen, I have a little GUI for interface so easy to test. It also works from speech to text and I am trying to integrate recognition of homonyms and oronyms to improve interpretation of results. As  you type it does an update of terms when you type a space or hit enter. Look forward to your next release and ready to test.

very neat!


sfc-init generated configuration with unused external imports

I recently created a project with sfc-init.
When building the project, the warning
(!) Unused external imports
withScopeId imported from external module "vue" but never used in "src\masonry-wall.vue?vue&type=template&id=257b3451&lang.js" and "src\masonry-wall.vue?vue&type=script&lang.ts"

is printed for every build format.
The configuration can be found at https://github.com/DerYeger/vue-masonry-wall/blob/master/build/rollup.config.js.

Seems like the warning is caused by adding
<style scoped>
/* Some style */
</style>

to the SFC.

Is this still an issue? I checked out your repo and ran a build, never saw the issue. It's likely upstream anyway, since none of the code coming from sfc-init directly uses withScopeId as reported, so it's likely something upstream...

Is this still an issue? I checked out your repo and ran a build, never saw the issue. It's likely upstream anyway, since none of the code coming from sfc-init directly uses withScopeId as reported, so it's likely something upstream...

I removed all <style scoped> occurrences, which fixed the issue. I agree that it is most likely an upstream issue after finding the cause.


test: remove eip712 tests for ethers-signers

Motivation
Removes cyclical dependency:
ethers-signers <-> ethers-contract(-derive)
CC @gakonst was this the one that messes up releases?

Solution

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

https://github.com/gakonst/ethers-rs/blob/master/ethers-contract/tests/it/contract.rs#L787-L911 this is the one that's causing the cyclical dep


ElectronicCats

Hello esteemed colleagues,
I have a PCB with an Arduino Mega and an RMF95, and I am working with 915MHz (868Mhz also, works perfect). Currently, I am attempting to join a ChirpStack server, which I was able to achieve only with the "lora.setDataRate(0)" function, which sets the data rate to 0, DR0, SF10, BW 125 KHz, for uplink. I confirmed this with the packet forwarder log of the gateway, which is a Tektelic Kona Enterprise with the latest version of packet forwarder and firmware.
My problem is that the join is random, and I am unable to send data from the endpoint to the gateway when the join is successful. Could someone please guide me?

Hi @Microbitz
Thank you for contacting the Electronic Cats support team :octocat: !
To help us to understand your situation, we would like to ask you for the following additional information:

What activation method are you using (e.g, OTAA, ABP)?
Are you using one of ours examples? If so, let me know which one.
What LoRaWAN class are you using?
Screenshot of the ChirpStack console and the Serial Monitor of the board.

Also teh version of lorawan in chirpstack is the 1.0.0


⚠ Samsung has degraded performance

In f91f018, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2659 ms

Resolved: Samsung performance has improved in 3f94894.


vlc: add ncurses support

This allows building the nvlc program, resolving #130119.
Motivation for this change
The nvlc wrapper is nice to have, and basically every distro packages it with vlc.
Things done

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 131639 at 72b73d39 run on x86_64-linux 1

2 packages failed to build:

libsForQt514.elisa
obs-studio-plugins.obs-ndi

2 packages skipped due to time constraints:

libsForQt512.elisa
strawberry

26 packages built successfully:

arcan.arcan
arcan.arcan-wrapped
arcan.durden-wrapped
arcan.everyone-wrapped
arcan.pipeworld-wrapped
arcan.prio-wrapped
arcan.xarcan
libsForQt5.elisa (libsForQt515.elisa ,plasma5Packages.elisa)
libsForQt5.phonon-backend-vlc (libsForQt515.phonon-backend-vlc ,plasma5Packages.phonon-backend-vlc)
libsForQt512.phonon-backend-vlc
libsForQt514.phonon-backend-vlc
libvlc
megaglest
minitube
obs-studio
obs-studio-plugins.looking-glass-obs
obs-studio-plugins.obs-gstreamer
obs-studio-plugins.obs-move-transition
obs-studio-plugins.obs-multi-rtmp
obs-studio-plugins.wlrobs
pympress
python38Packages.python-vlc
python39Packages.python-vlc
reaper
tribler
vlc (vlc_qt5)

1 suggestion:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/applications/video/vlc/default.nix:96:3:
   |
96 |   meta = with lib; {
   |   ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

ndi failure is unrelated (requires an account to download, no maintainers listed?), and looks like elisa is marked as broken when I run nixpkgs-review pr 131639 locally.


random.seed

tf.reset_default_graph()
np.random.seed(10)
rand_x = np.random.randn(1500)/50
np.random.seed(8)
rand_y = np.random.randn(1500)/50
Why there are two seed in the data initialization part in toydata exp.?

Hi, the seed here helps to generate random toy data. As long as the total training data construct the pattern region (the grey area in the readme ),  it will not affect the final outcome.


TLS environment variables for multi domain

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
I have 2 domains on Cloudflare (or could also be other provider) and want to 'host' them on a single Traefik instance.
So I have created 2 certResolvers but the corresponding api keys/tokens (environment variables) can not be linked to the respective certResolver...
What could be the best way to link the api keys/tokens to a specific CertResolver? A prefix, index,...?

I'll try to solve it temporary by using the httpChallenge for LetsEncrypt but would prefer to use dnsChallenge.

Duplicate of #5472


General: Fix display of user group selection

Checklist
General

[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple screenshots/screencasts of my UI changes.

Motivation and Context
Description

Steps for Testing

Prerequisites:

1 Administrator
1 Student
1 Course with user groups

Administrator: Navigate to Server Administration > User management > Edit
Adjust the user groups
Verify that the selection is easy to read and the background is not transparent

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress
Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Screenshots

Before (transparent, hard to read)
After (not transparent, easy to read)

Tested on TS2_Legacy
Your changes look good! The transparency is gone.


Default store location

We could define a location by default for the store, which would make us more zeroconf.

It's probably just a bad idea. Store can always be specified as a mandatory argument and that's much cleaner anyway.


[Bugfix] Block Editor: Block Definition Preview Update

Left out a parameter in a FactoryUtils function that broke the block definition preview.

This change is

LGTM

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Automatically run query when sorting is changed in sorting popover

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1875
Originally created by @DouweM on 2020-03-19 22:44:06

View 2 previous comments from the original issue on GitLab


Migrate to shared-kernel

Signed-off-by: Niels Thykier<EMAIL_ADDRESS>

Pending https://github.com/dcsaorg/DCSA-Event-Core/pull/304 + @FinnAtAsseco super pom branch to merged first.


fix: deployment priorityclassname

I apologize, but during the review process of the previous pr when I was making changes as requested, an error was made that ended up in the master branch. Please merge to fix it.
@d3adb5  @aslafy-z @rasheedamir

Thanks @rtest12


Using CUDA

How to run faster-whisper using CUDA ?

cudnn libs + --device cuda
on my LXC setup i followed this article https://sluijsjes.nl/2024/05/18/coral-and-nvidia-passthrough-for-proxmox-lxc-to-install-frigate-video-surveillance-server/
you just have to include the correct libs in the container, or map damn from the host to the container.
i ended up building the cointainer myself with this dockerfile:
FROM rhasspy/wyoming-whisper:2.1.0
RUN apt update && apt install -y wget software-properties-common
RUN wget https://developer.download.nvidia.com/compute/cudnn/<IP_ADDRESS>/local_installers/12.2/cudnn-local-repo-debian11-<IP_ADDRESS>_1.0-1_amd64.deb
RUN dpkg -i cudnn-local-repo-debian11-<IP_ADDRESS>_1.0-1_amd64.deb
RUN cp /var/cudnn-local-repo-debian11-<IP_ADDRESS>/cudnn-*-keyring.gpg /usr/share/keyrings/
RUN wget https://developer.download.nvidia.com/compute/cuda/repos/debian11/x86_64/cuda-keyring_1.1-1_all.deb
RUN dpkg -i cuda-keyring_1.1-1_all.deb
RUN add-apt-repository contrib && apt --allow-releaseinfo-change update
RUN apt -y install libcudnn8 libcublas-12-0
RUN rm cudnn-local-repo-debian11-<IP_ADDRESS>_1.0-1_amd64.deb

took some time to find the right versions to make it run.

👍 For official GPU support, would be nice to speed things up :)

:+1: for this feature. I already have a GPU in my server for other applications :)

Also, is it possible to make it unload model based on some --keep-alive param?

I made PR #44  to allow building it directly from this repository, and use CUDA.
It works perfectly with just docker compose up -d or using the build scripts in the README.
@synesthesiam will need to upload a gpu based image to rhasspy dockerhub if they want to allow people to pull without building locally.

cudnn libs + --device cuda
on my LXC setup i followed this article https://sluijsjes.nl/2024/05/18/coral-and-nvidia-passthrough-for-proxmox-lxc-to-install-frigate-video-surveillance-server/
you just have to include the correct libs in the container, or map damn from the host to the container.
i ended up building the cointainer myself with this dockerfile:
FROM rhasspy/wyoming-whisper:2.1.0
RUN apt update && apt install -y wget software-properties-common
RUN wget https://developer.download.nvidia.com/compute/cudnn/<IP_ADDRESS>/local_installers/12.2/cudnn-local-repo-debian11-<IP_ADDRESS>_1.0-1_amd64.deb
RUN dpkg -i cudnn-local-repo-debian11-<IP_ADDRESS>_1.0-1_amd64.deb
RUN cp /var/cudnn-local-repo-debian11-<IP_ADDRESS>/cudnn-*-keyring.gpg /usr/share/keyrings/
RUN wget https://developer.download.nvidia.com/compute/cuda/repos/debian11/x86_64/cuda-keyring_1.1-1_all.deb
RUN dpkg -i cuda-keyring_1.1-1_all.deb
RUN add-apt-repository contrib && apt --allow-releaseinfo-change update
RUN apt -y install libcudnn8 libcublas-12-0
RUN rm cudnn-local-repo-debian11-<IP_ADDRESS>_1.0-1_amd64.deb

took some time to find the right versions to make it run.

Did you ever keep this updated? I managed to get the container build but it throws a core dump when an assist comes in on whisper 2.4.0

Using CUDNN9.6 and 2.4.0 versions does not work properly. I hope someone can give me a hint
Unable to load any of {libcudnn_ops.so.9.1.0, libcudnn_ops.so.9.1, libcudnn_ops.so.9, libcudnn_ops.so}

Invalid handle. Cannot load symbol cudnnCreateTensorDescriptor


Add apps and permission groups to navigator

I want to merge this change because it adds navigator shortcut to apps and permission groups.
PR intended to be tested with API branch: master
Pull Request Checklist

[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Translated strings are extracted.
[ ] Number of API calls is optimized.
[ ] The changes are tested.
[ ] Data-test are added for new elements.
[ ] Type definitions are up to date.
[x] Changes are mentioned in the changelog.

Test environment config

API_URI=https://master.staging.saleor.rocks/graphql/

Task linked: CU-7eued7 Add permission groups to navigator


Update packages

In particular, to get material_color_utilities 0.2.0.
#108948

test-exempt: rolling dependency


ament_cmake clone failed

Issue template

Hardware description: RPi Pico, Ubuntu 22.04, git version 2.34.1
Installation type: Platformio
Version or commit hash: humble

Steps to reproduce the issue
New Platformio project in VS Code, select Pico board, and add
lib_deps = https://github.com/micro-ROS/micro_ros_platformio

to platformio.ini. Save to start auto-build.
Expected behavior
Build
Actual behavior
Configuring pico with transport serial
Downloading micro-ROS dev dependencies
ament_cmake clone failed:
fatal: Too many arguments.

Additional information
Same issue as the closed https://github.com/micro-ROS/micro_ros_platformio/issues/60
Platformio.ini:
[env:pico]
platform = raspberrypi
board = pico
framework = arduino
lib_deps = https://github.com/micro-ROS/micro_ros_platformio
lib_ldf_mode = chain+
upload_protocol = cmsis-dap

Hello,
I had the same issue. I was working with a git repository and the VS Code extension for platformio, but after disabling or uninstalling it, the "ament_cmake clone failed" bug appeared.
A solution was to move all my code to a totally new folder and to recreate a git repo from scratch (git init).
From this new folder, I don't get ament_cmake clone failed anymore.
Hope it works for other people as well.


Address Lighthouse Score And Suggestions

The site didn't score poorly but can be improved in many spots. Lighthouse is helpful in that it usually shows examples of what to change so I don't it will be hard to figure out how to address the sore spots.

[ ]
[ ]
[ ]
[ ]
[ ]
[ ]
[ ]

Bloop, bloop...I was going to try and improve these ratings but more trouble than it's worth, especially since all of the items could have been addressed during development time.


HStack Fill equally

Hello,
I have two label inside HStack and i want to make them fill equally (each label = 50% width of content)
How can i do that ? I tried align(.fill) but didn't work

By default HStack(or VStack) only supports items with intrinsic content size or fixed size. With one exception, align(.fill), which will stretch the cross axis to match the size of Stack (aka. vertical axis for HStack, horizontal axis for VStack).
If you want a proportional size for your items on main axis, you have to fall back to layout constraints.
view1.makeCons { $0.width.equal(view2).multiply(0.3) }
In your case, simply use:
label1.makeCons { $0.width.equal(label2) }


Is Django 1.8 supported?

The readme says this app supports Django 1.8 but setup.py define >=1.9. Can you please update the readme.

Sorry for the late reply, Yes It supports Django 1.8.


[1.18.1] Switching Saplings

Forge v39.0.59
Bonsai Trees v3.0.2
Switching the sapling will keep the fully grown tree instead of resetting the growth.
https://user-images.githubusercontent.com/12000593/152401571-2d3bd699-3a08-4a3c-b014-3517b1f8d79a.mp4

Oops. Will look into it soon.

Fixed, will be in the next release.

Thanks!


Check commands for compatibility before using them

AMAX panels don't support Request output text and more strangely, they don't support alarm memory summary CF01 but do support CF02.

Is there a more pythonic way to write these sections?
 if bitmask[7] & 0x08:
      self._area_text_supported_format = 3
 elif bitmask[7] & 0x20:
      self._area_text_supported_format = 1

Thank you!


Fix seek bar layout on iOS 17

Pull request
Description
This PR allows us to fix a layout issue with the player seek bar.
Changes made

The access to the AirPlay's image view is now asynchronous.

Checklist

[x] The branch should be rebased onto the develop branch for whole tests with nighties.
[x] The documentation has been updated (if relevant).
[x] The demo has been updated (if relevant).
[x] Issues are linked to the PR, if any.

I'm not really confident and happy with this approach because the issue seems to originate at a higher level, specifically in the Letterbox.


@conda-forge-admin, please add user @teo-tsirpanis

Additional comment:
No response

My bad: I mistook azure-identity-feedstock for azure-identity-cpp-feedstock. :facepalm:


Update to export file with 100,000 resource records.

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

File
Status
Preview URL
Details

articles/healthcare-apis/azure-api-for-fhir/export-data.md
: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 the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@mikaelweave

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

Learn Build status updates of commit 28bd2e6:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/99028/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
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.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


SPARKC-227: Fix Connection Caching, Change SSL EnabledAlgorithms to Set

The bug here is that an Array(enabledAlgorithms) will not match another
Array even if their contents match exactly. This will happen even if the
array is declared statically and finally. To fix this we need to change
the enabledAlgorithms variable to a Set which is more appropriate for
this field as well (Since order and duplicates do not matter). The
original decision to make this an Array was most likely done to match
the Java Driver's implementation, unfortunately this has some bad
implications for the underlying connection pool which we can avoid with
a Set.
Since the Hash of the CassandraConnectionConf is used by
the Cache to determine whether or not a new Cluster object needs to be
made, using an Array forces the creation of a new Cluster object on
every write. These cannot be reused. Changing this to a set allows the
cache to recognize that two CassandraConnectionConfs which contain SSL
EnabledAlgorithms which have the same contents are the same.

Build against Scala 2.10 finished with success

Build against Scala 2.11 finished with success

+1


Export Button

In the <List> view, an <Export> component should let the user download the related data:

with the current filters enabled
unpaginated (i.e. the whole data)
in CSV

Just like <Filters>, this component should let the developer choose the fields to export:
const PostExport = (props) => (
    <Export {...props}>
            <TextField source="id" />
            <ReferenceField label="User" source="userId" reference="users">
                <TextField source="name" />
            </ReferenceField>
            <TextField source="title" />
            <TextField source="body" />
    </Export>
);

export const PostList = (props) => (
    <List {...props} export={PostExport}>
        // ...
    </List>
);

Not sure if using fields makes sense here, or if we can just pass an object mapping source to formatted data:
const PostExport = (props) => (
    <Export {...props} fields={{
         Id: { source: 'id' },
         User: { source: 'userId', reference: 'users', referenceSource: 'name' },
         Title: { source: 'title', transform: data => data.toLowercase() },
         Body: { source: 'body' },
    }}>
);

cons: it makes us redevelop reference fetching.

Elaborating on my previous comment: using Field components is a bad idea as they're intended for HTML, and contain HTML tags (e.g. <TextField> outputs foo).
Besides, the Reference fields are asynchronous, and as such will be hard to include in a string for the user to download - we'll have to know when all the actions have been dealt with.
So we're left with:

rewrite all the Field components as simple functions, minus the HTML, and rewrite the Reference field logic to bypass Redux and look more synchronous, or
use an export function and leave the reference fetching as an exercise to the user

const PostExport = (props) => (
    <Export {...props} render={record => ({
         Id: record.id,
         UserId: record.userId,
         User: ???,
         Title: record.title.toLowercase(),
         Body: record.body,
    })}>
);

@fzaninotto I really need this a lot! Thanks for thinking about that, was any progress made to that, in 'next' for example?

No, we haven't implemented it in next. This is still an open issue.

Implemented in 2.2, see #2034


[ISSUE] Script issues while source oracle scripts against oracle 12c and 19c with 5.9.0 latest wum

Environment
wso2is-5.9.0+1585323544240.full
Steps to Reproduce
Source dbscripts/oracle.sql against oracle 12c and oracle 19c
Observation
Below errors were noticed
table REG_CLUSTER_LOCK created.
table REG_LOG created.
sequence REG_LOG_SEQUENCE created.
index REG_LOG_IND_BY_REGLOG created.
TRIGGER REG_LOG_TRIGGER compiled
table REG_PATH created.
Error starting at line 40 in command:
CREATE INDEX REG_PATH_IND_BY_PATH_VALUE ON REG_PATH(REG_PATH_VALUE, REG_TENANT_ID)
Error at Command Line:40 Column:53
Error report:
SQL Error: ORA-01408: such column list already indexed
01408. 00000 -  "such column list already indexed"
*Cause:
*Action:
index REG_PATH_IND_BY_PARENT_ID created.
sequence REG_PATH_SEQUENCE created.

TRIGGER UM_HYBRID_REMEMBER_ME_TRIGGER compiled
table UM_SYSTEM_ROLE created.
sequence UM_SYSTEM_ROLE_SEQUENCE created.
Error starting at line 784 in command:
CREATE INDEX SYSTEM_ROLE_IND_BY_RN_TI ON UM_SYSTEM_ROLE(UM_ROLE_NAME, UM_TENANT_ID)
Error at Command Line:784 Column:57
Error report:
SQL Error: ORA-01408: such column list already indexed
01408. 00000 -  "such column list already indexed"
*Cause:
*Action:
TRIGGER UM_SYSTEM_ROLE_TRIGGER compiled
table UM_SYSTEM_USER_ROLE created.
sequence UM_SYSTEM_USER_ROLE_SEQUENCE created.
TRIGGER UM_SYSTEM_USER_ROLE_TRIGGER compiled

Identity/uma/oracle.sql
While executing the uma oracle script noticed below errors
table IDN_UMA_RESOURCE created.
Error starting at line 14 in command:
CREATE SEQUENCE IDN_UMA_RESOURCE_SEQ START WITH 1 INCREMENT BY 1 NOCACHE
  /

CREATE OR REPLACE TRIGGER IDN_UMA_RESOURCE_TRIG
BEFORE INSERT
ON IDN_UMA_RESOURCE
REFERENCING NEW AS NEW
FOR EACH ROW
BEGIN
SELECT IDN_UMA_RESOURCE_SEQ.nextval INTO :NEW.ID FROM dual
Error at Command Line:15 Column:3
Error report:
SQL Error: ORA-00933: SQL command not properly ended
00933. 00000 -  "SQL command not properly ended"
*Cause:
*Action:

Error starting at line 24 in command:
END
Error report:
Unknown Command
index IDX_RID created.
index IDX_USER created.
index IDX_USER_RID created.
Error starting at line 36 in command:
CREATE TABLE IDN_UMA_RESOURCE_META_DATA (
  ID                INTEGER,
  RESOURCE_IDENTITY INTEGER                NOT NULL,
  PROPERTY_KEY      VARCHAR2(40),
  PROPERTY_VALUE    VARCHAR2(255),
  PRIMARY KEY (ID),
  FOREIGN KEY (RESOURCE_IDENTITY) REFERENCES IDN_UMA_RESOURCE (ID) ON DELETE CASCADE
)
  /

CREATE SEQUENCE IDN_UMA_RESOURCE_META_DATA_SEQ START WITH 1 INCREMENT BY 1 NOCACHE

Error at Command Line:44 Column:3
Error report:
SQL Error: ORA-00922: missing or invalid option
00922. 00000 -  "missing or invalid option"
*Cause:
*Action:
Error starting at line 49 in command:
CREATE OR REPLACE TRIGGER IDN_UMA_RESOURCE_METADATA_TRIG
BEFORE INSERT
ON IDN_UMA_RESOURCE_META_DATA
REFERENCING NEW AS NEW
FOR EACH ROW
BEGIN
SELECT IDN_UMA_RESOURCE_META_DATA_SEQ.nextval INTO :NEW.ID FROM dual;
END;
Error report:
ORA-00942: table or view does not exist
00942. 00000 -  "table or view does not exist"
*Cause:
*Action:
Error starting at line 59 in command:
CREATE TABLE IDN_UMA_RESOURCE_SCOPE (
  ID                INTEGER,
  RESOURCE_IDENTITY INTEGER                NOT NULL,
  SCOPE_NAME        VARCHAR2(255),
  PRIMARY KEY (ID),
  FOREIGN KEY (RESOURCE_IDENTITY) REFERENCES IDN_UMA_RESOURCE (ID) ON DELETE CASCADE
)
  /

CREATE SEQUENCE IDN_UMA_RESOURCE_SCOPE_SEQ START WITH 1 INCREMENT BY 1 NOCACHE

Error at Command Line:66 Column:3
Error report:
SQL Error: ORA-00922: missing or invalid option
00922. 00000 -  "missing or invalid option"
*Cause:
*Action:
Error starting at line 71 in command:
CREATE OR REPLACE TRIGGER IDN_UMA_RESOURCE_SCOPE_TRIG
BEFORE INSERT
ON IDN_UMA_RESOURCE_SCOPE
REFERENCING NEW AS NEW
FOR EACH ROW
BEGIN
SELECT IDN_UMA_RESOURCE_SCOPE_SEQ.nextval INTO :NEW.ID FROM dual;
END;
Error report:
ORA-00942: table or view does not exist
00942. 00000 -  "table or view does not exist"
*Cause:
*Action:
Error starting at line 81 in command:
CREATE INDEX IDX_RS ON IDN_UMA_RESOURCE_SCOPE (SCOPE_NAME)
Error at Command Line:81 Column:24
Error report:
SQL Error: ORA-00942: table or view does not exist
00942. 00000 -  "table or view does not exist"
*Cause:
*Action:
Error starting at line 84 in command:
CREATE TABLE IDN_UMA_PERMISSION_TICKET (
  ID              INTEGER,
  PT              VARCHAR2(255)           NOT NULL,
  TIME_CREATED    TIMESTAMP              NOT NULL,
  EXPIRY_TIME     TIMESTAMP              NOT NULL,
  TICKET_STATE    VARCHAR2(25) DEFAULT 'ACTIVE',
  TENANT_ID       INTEGER     DEFAULT -1234,
  TOKEN_ID        VARCHAR(255),
  PRIMARY KEY (ID)
)
  /

CREATE SEQUENCE IDN_UMA_PERMISSION_TICKET_SEQ START WITH 1 INCREMENT BY 1 NOCACHE

Error at Command Line:94 Column:3
Error report:
SQL Error: ORA-00922: missing or invalid option
00922. 00000 -  "missing or invalid option"
*Cause:
*Action:
Error starting at line 99 in command:
CREATE OR REPLACE TRIGGER IDN_UMA_PERMISSION_TICKET_TRIG
BEFORE INSERT
ON IDN_UMA_PERMISSION_TICKET
REFERENCING NEW AS NEW
FOR EACH ROW
BEGIN
SELECT IDN_UMA_PERMISSION_TICKET_SEQ.nextval INTO :NEW.ID FROM dual;
END;
Error report:
ORA-00942: table or view does not exist
00942. 00000 -  "table or view does not exist"
*Cause:
*Action:
Error starting at line 109 in command:
CREATE INDEX IDX_PT ON IDN_UMA_PERMISSION_TICKET (PT)
Error at Command Line:109 Column:24
Error report:
SQL Error: ORA-00942: table or view does not exist
00942. 00000 -  "table or view does not exist"
*Cause:
*Action:
Error starting at line 112 in command:
CREATE TABLE IDN_UMA_PT_RESOURCE (
  ID             INTEGER,
  PT_RESOURCE_ID INTEGER                NOT NULL,
  PT_ID          INTEGER                NOT NULL,
  PRIMARY KEY (ID),
  FOREIGN KEY (PT_ID) REFERENCES IDN_UMA_PERMISSION_TICKET (ID) ON DELETE CASCADE,
  FOREIGN KEY (PT_RESOURCE_ID) REFERENCES IDN_UMA_RESOURCE (ID) ON DELETE CASCADE
)
  /

CREATE SEQUENCE IDN_UMA_PT_RESOURCE_SEQ START WITH 1 INCREMENT BY 1 NOCACHE

Error at Command Line:120 Column:3
Error report:
SQL Error: ORA-00922: missing or invalid option
00922. 00000 -  "missing or invalid option"
*Cause:
*Action:
Error starting at line 125 in command:
CREATE OR REPLACE TRIGGER IDN_UMA_PT_RESOURCE_TRIG
BEFORE INSERT
ON IDN_UMA_PT_RESOURCE
REFERENCING NEW AS NEW
FOR EACH ROW
BEGIN
SELECT IDN_UMA_PT_RESOURCE_SEQ.nextval INTO :NEW.ID FROM dual;
END;
Error report:
ORA-00942: table or view does not exist
00942. 00000 -  "table or view does not exist"
*Cause:
*Action:
Error starting at line 135 in command:
CREATE TABLE IDN_UMA_PT_RESOURCE_SCOPE (
  ID             INTEGER,
  PT_RESOURCE_ID INTEGER                NOT NULL,
  PT_SCOPE_ID    INTEGER                NOT NULL,
  PRIMARY KEY (ID),
  FOREIGN KEY (PT_RESOURCE_ID) REFERENCES IDN_UMA_PT_RESOURCE (ID) ON DELETE CASCADE,
  FOREIGN KEY (PT_SCOPE_ID) REFERENCES IDN_UMA_RESOURCE_SCOPE (ID) ON DELETE CASCADE
)
  /

CREATE SEQUENCE IDN_UMA_PT_RESOURCE_SCOPE_SEQ START WITH 1 INCREMENT BY 1 NOCACHE

Error at Command Line:143 Column:3
Error report:
SQL Error: ORA-00922: missing or invalid option
00922. 00000 -  "missing or invalid option"
*Cause:
*Action:
Error starting at line 148 in command:
CREATE OR REPLACE TRIGGER IDN_UMA_PT_RESOURCE_SCOPE_TRIG
BEFORE INSERT
ON IDN_UMA_PT_RESOURCE_SCOPE
REFERENCING NEW AS NEW
FOR EACH ROW
BEGIN
SELECT IDN_UMA_PT_RESOURCE_SCOPE_SEQ.nextval INTO :NEW.ID FROM dual;
END;
Error report:
ORA-00942: table or view does not exist
00942. 00000 -  "table or view does not exist"
*Cause:
*Action:

Couldn't reproduce in 5.11.0-alpha hence closing the issue.


[BUG] When Dark Mode is enabled, ReadMe Text is not readable

When Dark Mode is enabled, some text in ReadMe is not possible to read.

Fixed the Issue.


Tilde character in file path in a shell  must expand to the absolute path of the home directory

Proposal
** Tilde symbol in a file path needs to expand to absolute  path of home directory of the user as per the *nix **
Bug Report
What did you do?
As per instructions given in https://prometheus.io/docs/introduction/first_steps/;  start prometheus by executing
./prometheus --config.file=~asathe/prometheus-2.3.0.linux-amd64/prometheus.yml
What did you expect to see?
In the  ~asathe/prometheus-2.3.0.linux-amd64/prometheus.yml ; I expected the ~asathe to expand to home directory of asathe ( in my case; it is /home/pnq/asathe )
What did you see instead? Under which circumstances?
prometheus reported
level=error ts=2018-06-08T13:41:36.291212386Z caller=main.go:597 err="Error loading config couldn't load configuration (--config.file=~asathe/prometheus-2.3.0.linux-amd64/prometheus.yml): open ~asathe/prometheus-2.3.0.linux-amd64/prometheus.yml: no such file or directory"
Environment

System information:

Linux 4.14.16-300.fc27.x86_64 x86_64  ```

Prometheus version:

./prometheus --version
prometheus, version 2.3.0 (branch: HEAD, revision: 290d71791a507a5057b9a099c9d48703d86dc941)
  build user:       root@d539e167976a
  build date:       20180607-08:46:54
  go version:       go1.10.2

Prometheus configuration file:

[asathe@localhost prometheus-2.3.0.linux-amd64]$ cat prometheus.yml
# my global config
global:
  scrape_interval:     15s # Set the scrape interval to every 15 seconds. Default is every 1 minute.
  evaluation_interval: 15s # Evaluate rules every 15 seconds. The default is every 1 minute.
  # scrape_timeout is set to the global default (10s).

# Alertmanager configuration
alerting:
  alertmanagers:
  - static_configs:
    - targets:
      # - alertmanager:9093

# Load rules once and periodically evaluate them according to the global 'evaluation_interval'.
rule_files:
  # - "first_rules.yml"
  # - "second_rules.yml"

# A scrape configuration containing exactly one endpoint to scrape:
# Here it's Prometheus itself.
scrape_configs:
  # The job name is added as a label `job=<job_name>` to any timeseries scraped from this config.
  - job_name: 'prometheus'

    # metrics_path defaults to '/metrics'
    # scheme defaults to 'http'.

    static_configs:
    - targets: ['localhost:9090']
[asathe@localhost prometheus-2.3.0.linux-amd64]$

Logs:

[root@localhost prometheus-2.3.0.linux-amd64]# ./prometheus --config.file=~asathe/prometheus-2.3.0.linux-amd64/prometheus.yml
level=info ts=2018-06-08T13:41:36.242555868Z caller=main.go:222 msg="Starting Prometheus" version="(version=2.3.0, branch=HEAD, revision=290d71791a507a5057b9a099c9d48703d86dc941)"
level=info ts=2018-06-08T13:41:36.242694699Z caller=main.go:223 build_context="(go=go1.10.2, user=root@d539e167976a, date=20180607-08:46:54)"
level=info ts=2018-06-08T13:41:36.2427562Z caller=main.go:224 host_details="(Linux 4.14.16-300.fc27.x86_64 #1 SMP Wed Jan 31 19:24:27 UTC 2018 x86_64 localhost.localdomain (none))"
level=info ts=2018-06-08T13:41:36.242800958Z caller=main.go:225 fd_limits="(soft=1024, hard=4096)"
level=info ts=2018-06-08T13:41:36.24340709Z caller=main.go:514 msg="Starting TSDB ..."
level=info ts=2018-06-08T13:41:36.249070562Z caller=web.go:426 component=web msg="Start listening for connections" address=<IP_ADDRESS>:9090
level=info ts=2018-06-08T13:41:36.272393842Z caller=main.go:524 msg="TSDB started"
level=info ts=2018-06-08T13:41:36.275782327Z caller=main.go:603 msg="Loading configuration file" filename=~asathe/prometheus-2.3.0.linux-amd64/prometheus.yml
level=info ts=2018-06-08T13:41:36.27587218Z caller=main.go:402 msg="Stopping scrape discovery manager..."
level=info ts=2018-06-08T13:41:36.275922809Z caller=main.go:416 msg="Stopping notify discovery manager..."
level=info ts=2018-06-08T13:41:36.275960298Z caller=main.go:438 msg="Stopping scrape manager..."
level=info ts=2018-06-08T13:41:36.276101975Z caller=main.go:412 msg="Notify discovery manager stopped"
level=info ts=2018-06-08T13:41:36.287078035Z caller=manager.go:464 component="rule manager" msg="Stopping rule manager..."
level=info ts=2018-06-08T13:41:36.287120767Z caller=manager.go:470 component="rule manager" msg="Rule manager stopped"
level=info ts=2018-06-08T13:41:36.287138091Z caller=notifier.go:512 component=notifier msg="Stopping notification manager..."
level=info ts=2018-06-08T13:41:36.290181893Z caller=main.go:398 msg="Scrape discovery manager stopped"
level=info ts=2018-06-08T13:41:36.29078386Z caller=main.go:432 msg="Scrape manager stopped"
level=info ts=2018-06-08T13:41:36.290835358Z caller=main.go:588 msg="Notifier manager stopped"
level=error ts=2018-06-08T13:41:36.291212386Z caller=main.go:597 err="Error loading config couldn't load configuration (--config.file=~asathe/prometheus-2.3.0.linux-amd64/prometheus.yml): open ~asathe/prometheus-2.3.0.linux-amd64/prometheus.yml: no such file or directory"
level=info ts=2018-06-08T13:41:36.291274804Z caller=main.go:599 msg="See you next time!"

Tilde expansion is the responsibility of your shell on Unix. Which shell are you using?
If your shell doesn't support it, $HOME always works.

Tilde expansion is the responsibility of your shell on Unix. Which shell are you using?

root@localhost ~]# file ~
/root: directory
[root@localhost ~]# file ~asathe/
/home/asathe/: directory
[root@localhost ~]# echo $HOME
/root
[root@localhost ~]# echo $SHELL
/bin/bash
[root@localhost ~]# rpm -q bash
bash-4.4.12-13.fc27.x86_64
[root@localhost ~]#

[asathe@localhost node_exporter-0.16.0.linux-amd64]$ j=~asathe
[asathe@localhost node_exporter-0.16.0.linux-amd64]$ echo $j
/home/asathe

It is not mentioned in the documents, but the './prometheus --configfile= ..' needs to be executed as root user.  In my scenario; I extracted  prometheus into normal user's directory  '~asathe/prometheus'.
I switched non-login/login-shell to root to execute prometheus binary.  But; after switching the variable $HOME would store the home directory of root user.  Hence; the only option is to use the tilde symbol.
Thus; this is still an issue.
In the scenario; 'prometheus' directory is present inside the home directory root user; $HOME gets expanded to its value ( /root ) in the following command:
./prometheus --config.file=$HOME/prometheus-2.3.0.linux-amd64/prometheus.yml

on a second thought  @amej do you have any immediate use for that? unless it is trivial change and someone really needs this maybe not worth adding any extra code for this.

don't think we have an immediate need for this so will revisit if needed.
feel free to reopen if you have a good use case that requires this.


Fix plotting memory leak

This PR resolves #9003 (and explains matplotlib/matplotlib#3892). The root cause of the memory leak is a reference cycle between MPLPlot objects and the AxesSubplot objects they create.
Specifically, a plotf function object is stored in ax._plot_data for the purposes of potentially redrawing if the data needs to be resampled. This would be fine if this were a top-level function; however, these are all nested functions that make use of self. This means that by plotf pulls in self.ax and self.axes, which point to the AxesSubplot that plotf is being attached to. We therefore have a reference cycle:
AxesSubplot -> AxesSubplot._plot_data -> plotf -> self -> self.ax -> AxesSubplot

In order to make the objects collectable, we need to either explicitly break a link or replace it with a weakref. Weakrefs don't work as AxesSubplot and MPLPlot have the same lifetime. Just not using _plot_data prevents the leak but breaks functionality.
The final option as I see it is to change plotf to not depend on self. This works but involves a fair amount of modifications. I elected to make each of the plotfs top-level functions to make the lack of self-dependency explicit. This also required making several other functions classmethods and moving some data from MPLPlot to the AxesSubplot object.
The key assumption being made by this change is that either MPLPlot objects are discarded immediately after use or we don't want any modifications to the MPLPlot (e.g., adding errors post-plotting) to be reflected if redrawing. I believe both cases are true but this patch has the potential for behavioral changes if MPLPlot objects are regularly being retained and modified.
I've also added a memory-leak test to prevent a regression; the test fails as expected if applied without the other commits in this patch.

@jreback I think this patch is ready for review. Please let me know if you have any questions or suggestions.

cc @TomAugspurger @sinhrks

Specifically, a plotf function object is stored in ax._plot_data

I understand you're referring to this. If so, only line and area are affected and no need to change others?

@sinhrks Yep, it's a subtle bug so I thought it'd be better to convert everything to a safer approach to prevent a reoccurrence.

Understood. What I'm caring is current impl makes it more difficult to understand the logic...
How about:

Change closure to normal function (not return a function, but uses a function defined in the module directly)
Make mapping from class to each plot function and unify separated _get_plot_function.

class LinePlot(MPLPlot):
    plotf = _lineplot_plotf
# or
_plotf_mapping = {LinePlot: _lineplot_plotf, ...}

@sinhrks @TomAugspurger can you have a quick look?

@qwhelan can you squash to a single commit? and pls add a release note referncing the bug?

@TomAugspurger can you have a look?

@TomAugspurger can you have a look?

I've looked at this a bit. Haven't had a chance to fully digest it since I've never had to deal with this kind of problem before. I'll look more and let you know. It may be something to push for another release.

On Mar 10, 2015, at 18:20, jreback<EMAIL_ADDRESS>wrote:
@TomAugspurger can you have a look?
—
Reply to this email directly or view it on GitHub.

@jreback Hey, sorry about the delay - made the mistake of letting Inbox decide what emails I see. I'll squash it now. Let me know what version I should put it under for the release notes.

@jreback Given we're at a rc1, I'm going to wait on adding to the docs until 0.16.1 is in development. As @sinhrks pointed out, it'd be possible to trim this down to just LinePlot and AreaPlot if that makes including this in a point-release more palatable.

whats new is now available for 0.16.1 https://github.com/pydata/pandas/commit/5ebf521eac528f5a4b772e4677f70b689b8b1826

Closing due to better version available in #9814


Plugin crop/fix safety checks

What's Changing and Why

safety checks for the width and height of remainingPixels are moved behind the pixelsToCrop sanitization, otherwise they lead to wrong values if the pixelsToCrop are negative

I encountered this in the following use case:

when calling the autocrop function on an image, I specified a leaveBorder value that was larger than the number of pixels that were cropped from the image. This lead to incorrect values for widthOfRemainingPixels and heightOfRemainingPixels, as they use the negative values of pixelsToCrop instead of the sanitized ones.

leaveBorder is applied here:
https://github.com/oliver-moran/jimp/blob/736054ef1bea6b6fb03db3e75f05cd922a9c104f/packages/plugin-crop/src/index.js#L209-L213
safety checks are applied here:
https://github.com/oliver-moran/jimp/blob/736054ef1bea6b6fb03db3e75f05cd922a9c104f/packages/plugin-crop/src/index.js#L224-L234
Moving the safety checks behind the sanitization leads to correct values.
Note: I was not able to build and test locally, due to this issue.
What else might be affected

only the crop plugin is affected

Tasks

[x] Add tests
[x] Update Documentation
[x] Update jimp.d.ts
[ ] Add SemVer Label

is this repo dead? 😢

@jagaSto  please rebase so the CI runs. Getting this project set up with good CI/CD has been an issue. but now we have circleci setup (I think) so it should be easier to mange the project!

thx for the hint @hipstersmoothie, will fix the tests once I find the time. Do you have any insights into https://github.com/oliver-moran/jimp/issues/742 ? Running the tests locally is not working for me, which will make the process of debugging a bit more painful 🙃

@hipstersmoothie this is fixed now!

:rocket: PR was released in v0.6.7 :rocket:


Before added geometry disabled editing.

Hello there . I have a geojson file and I get it with ajax. but I just want to prevent the editing of the geometries I added. how can I do that ?
i used generate geojson value  = http://geojson.io/#map=15/40.9933/29.0527

I solved.
 L.geoJSON(data, { style: polystyle, pmIgnore: true }).addTo(map)

PmIgnore does not work in 'erase mode'.

@VCodepp what data are you adding? Polygons? Circles?

In my tests, it works fine. Can you provide a JSfiddle where it's not working?

I'm sorry for the late reply.
https://jsfiddle.net/vcode/acgebzj5/41/
Click any layer in the delete mode. you'll see that you've removed them all.

Ah thank you. This is a duplicate of https://github.com/codeofsumit/leaflet.pm/issues/401 and will be fixed in the next release 👍


Update frontend .env / config

For the ESLint config with Create-React-App there was a recent adjustement to add custom ESLint rules, which requires EXTEND_ESLINT=true in the client/.env file.
This file is not in added in version control since it's in .gitignore because it contains REACT_APP_AIRTABLE_API_KEY and REACT_APP_AIRTABLE_BASE.
This might cause issues if newcomers setup the repository and forget to add this variable.
While these 2 settings will probably be removed eventually and replaced by our own mongo, we can move them to client/config.js and add client/.env to version control to use for the CRA setting.  Unless someone comes up with a better solution.

@robinv85 Do you want to add specifically the .env file?
I noticed that the .env.example file is already included there like this ->

Perhaps add somewhere on Eng onboarding to request for the Airtable API Key and the baseId. or I can make an update to just have the baseId, and have new Engs request the Airtable API KEY if it is better?

@idjevm for now, these are in .env but airtable will be removed from the frontend eventually
otherwise we could move them to the client/config.js too, but i think it's not needed
so the only thing really needed here is when we remove airtable just add .env to git and then the problem is solved i think :)
I don't know if there is a ticket to remove airtable or where it's used currently, if you want you can work on that too

see #342 i think that depends on geolocation backend update first which is now highest priority

@robinv85 @idjevm - from our product/engineering meeting today: we need to add feature flags so we can hide/unhide certain features if they are not ready for launch. Robin's comment on how to do that:
"i wonder, feature flags always need the .env file i guess? because that ESLint stuff for frontend would require to put the .env file in git so might be related"

yeah I don't see another way, I think @mannykary also suggested we could use multiple .env files so for example one that we add to git and one that can contain things like keys. but in the frontend I don't think we will need that if we remove the airtable keys here.
Only other configs is currently in config.js the password min and max length but that isn't really needed either I think

This will be resolved with #649

Will be resolved by #649

@vikjoshi and @joshmorel If I understand then this issue is resolved so the /client/.env is now in git version control
This doesn't provide feature flags functionality, but if we add that functionality, it will probably use this .env file to set them, at least for development
Also I think @MohamedEl1 asked about setting the GTM id in the frontend, I suppose this is also related, not sure if we want to add that in .env which is in git?

I think we should not add the GTM key (or any other secrets) to the .env file committed to git. We should store secrets/keys used by the client container either Github Secrets (in repo settings), or in an encrypted S3 bucket. Since we don't have a lot of secrets for the client, we can store them in Github Secrets for now. We would just pull the keys from Github Secrets and append them to the .env file while building the container.

@robinv85 I see now this is only partially resolved. For some reason I missed the part not related to the REACT_APP_AIRTABLE* stuff. These are all removed from F/E resolved part.
But committing the client/.env file with ESLINT=true - that wasn't done it's still in .gitignore. Is the reason we want to commit because people aren't always copying the .env.example file? Could we add a RUN command into Dockerfile.env to add this line to .env if it doesn't exist (because there may be other stuff in .env like GTM we don't want to commit to git).


add oop handson

Added hands on exercises for OOP both demo and exercise folder
5 demo packages to introduce the OOP concepts
5 blank training exercises to let participants practice

@yh-sm @khs-sm
Made some changes based on your comments. Kindly check. Thanks.


improve the documentation about stack system

Functions caller and callee lake for specification.
Here is a deep explanation:
Stack system in Massa

* ExecuteSC
    When an account A send an ExecuteSC operation, the stack at the beginning of that execution is:
        bottom [ A ] top

* CallSC
    When an account A sends a CallSC operation to call a function in a smart contract B, the stack at the beginning of the execution of that function is:
        bottom [ A, B ] top
    Note: A and B can be the same

* Call from one smart contract to another
    When a function F from smart contract C is being executed with the stack [A, B, C] and calls a function on a smart contract D, the stack at the beginning of the execution of D's function becomes:
        bottom [A, B, C, D] top
    When D's function finishes, the stack becomes
        bottom [A, B, C] top
    and the execution of F resumes

* Autonomous SC: a message sent at a moment when the stack was [A, B, C] and calling a target function F of a smart contrtact D will yield the following stack at the beginning of the execution of the target function:
        bottom [C, D] top
  note: C and D can be the same

* Local execution (not yet implemented, see https://github.com/massalabs/massa-sc-runtime/issues/170 )
    Local executions don't change the stack: they allow executing foreign code in the current context

ABIs (I'm not a big fan of caller, callee, transactionCreator, I find them misleading):
addressStack() returns the full call stack as a list, bottom to top
caller() returns the stack element just below the top
callee() returns the stack element at the top of the stack
transactionCreator() returns the stack element at the bottom of the stack

@Thykof your comment should be in the code.
Furthermore, if you have some suggestion on how we can improve the name of some functions, feel free to share them :-)

We have created a page https://docs.massa.net/en/latest/web3-dev/smart-contracts/address-stack.html do you still want to put all of this in the code?

I would prefer to. Code is law ;-)

@longgamma-massa for QA, you can look into the code review and see if the documentation is clear enough.

Q/A to finish, I need the ability to leave you fixes for the English language of this document. Can you assign me as a reviewer for the whole page?

I have updated the issue description


Revert "Add verbose logging for customer testing shard"

Reverts flutter/flutter#154356
The issue is resolved in https://github.com/flutter/tests/pull/406 so we can remove this verbose logging.
Related to https://github.com/flutter/flutter/issues/154251

Can we also remove bringup: true from the .ci.yaml target?

I have that staged in https://github.com/flutter/flutter/pull/154372

Should we also move some of the timing related prints from being guarded by verbose mode? Maybe specifically this print? https://github.com/christopherfujino/flutter/blob/slim-the-binary-test/dev/customer_testing/lib/runner.dart#L173-L175

Great idea! I will add this tomorrow. Marking this as a draft.

Can we also remove bringup: true from the .ci.yaml target?

I have that staged in #154372

nice

Closing in lieu of https://github.com/flutter/flutter/pull/154577


Expose reactions in EnrichedActivity

This allow the user access all reactions fields in EnrichedActivity

Fixed in version 3.0.1

Thank you very much!


Correctly escape style attribute in SSR output

Fixes the behavior that you can observe here: https://stackblitz.com/edit/solid-ssr-vite-juconn?file=src%2Fpages%2FHome.tsx where if you put a string that contains a double quote into the style attribute of an element, the server side rendered version isn't correctly escaped. Example: <main data-hk="<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.0.1">
Home!

Edit: fix wasn't as easy as I thought, gotta improve this a lil

@ryansolid I just noticed this happens with every attribute set in the style of attribute={something "with double quotes"}
Is this enough of a bug report for you? Would appreciate if you could take over this since the issue seems so widespread and I'm unsure how to solve it without further investigation

The dots on the hydration key indicates an outdated version.
But I guess it's true that quotes doesn't seem to get escaped.

Yeah I had the issues in solid 1.8.7, the reproduction was just something I pulled out of google to not have to set up an env

Fix should be somewhere here: https://github.com/ryansolid/dom-expressions/blob/b1c5b9d7d9c406070c30d0772eaf9b0e8266f033/packages/babel-plugin-jsx-dom-expressions/src/ssr/element.js#L128
escapeHTML could possibly replaced with escapeExpression however I'm not sure yet how this stuff goes

I'm fine merging this one, and then we could just check on the remaining issue with attributes

Sure, sounds good. I'm running this change with yarn patch rn and it fixes the issue with the style attribute for me

Just remembered this - did you make sure to fix the escaping of the attributes for other attributes too or should I open an issue to make sure you don't forget it?


Set macOS deployment target in Makefile

The idea is to avoid warnings like:
ld: warning: object file XX was built for newer macOS
version (14.4) than being linked (14.0)

@derrix060 good job!


fetch_symbols(): Reqwest Error: error decoding response body: unknown variant Pending

When running this code:
//no problems with this part
let mut api = http::Client::new(
	http::MAINNET_BYBIT,
	bybit_pub_key,
	bybit_priv_key,
)
.unwrap();

//erroring part
println!("{:#?}", api.fetch_symbols().await?);

I get this error (no need for unwrap by my pr but this also is an error on your git repo):
Reqwest Error: error decoding response body: unknown variant `Pending`, expected one of `Trading`, `Settling`, `Closed` at line 1 column 45977

Hmm good catch! The API docs don't seem to mention the Pending status anywhere.
I will add a Pending case to ContractStatus


chore: add go package to version control for release branches

What this PR does / why we need it:
chore: add go package to version control for release branches
Add go package to packageRules so we are able to control which go version is in which branch
Release note:
NONE

LGTM


Feat: table widget label type columns

Description
Label styles for column in table widget
Fixes #4333
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: feature/table-widget-label-type-columns

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
54.94 (0)
36.72 (-0.09)
33.71 (0)
55.48 (-0.01)

:red_circle:
app/client/src/components/ads/ColorPickerComponent.tsx
28.57 (0)
43.75 (-14.58)
0 (0)
31.82 (0)

:green_circle:
app/client/src/components/designSystems/appsmith/TableComponent/TableStyledWrappers.tsx
68.18 (0.36)
22.43 (0.58)
50 (0)
68.18 (0.36)

:red_circle:
app/client/src/components/designSystems/appsmith/TableComponent/TableUtilities.tsx
36.36 (-2.8)
15.45 (-3)
13.95 (-1.84)
34.9 (-2.79)

:sparkles: :new:
app/client/src/components/designSystems/blueprint/TagComponent.tsx
64.71
14.29
16.67
60

:sparkles: :new:
app/client/src/components/propertyControls/LabelColorPickerControl.tsx
40.54
14.29
18.18
40

:green_circle:
app/client/src/components/propertyControls/index.ts
85.29 (0.44)
100 (0)
0 (0)
84.85 (0.47)

:red_circle:
app/client/src/utils/helpers.tsx
63.87 (-1.58)
37.89 (-3.16)
45.16 (-3.23)
60.93 (-1.32)

:green_circle:
app/client/src/widgets/TableWidget/TablePropertyPaneConfig.ts
48.21 (9.48)
10.2 (0)
73.17 (7.54)
49.69 (9.54)

:red_circle:
app/client/src/widgets/TableWidget/index.tsx
27.01 (-0.13)
5.88 (-0.04)
25 (0)
27.13 (-0.14)

@vicky-primathon Observations so far:

Column type name is Label. As per the design it should be Tag
Other properties are not coming in property pane - I am not able to test other flows due to this.
After selecting Label for an column, the column data is coming as +3

@vicky-primathon Observations so far:

Column type name is Label. As per the design it should be Tag
Other properties are not coming in property pane - I am not able to test other flows due to this.
After selecting Label for an column, the column data is coming as +3

Issue is fixed
If the column size is smaller than we show the tag with tooltip. To see the tags, increase the size of column.

@vicky-primathon lets plan a meeting around this with the stakeholders so that we have what we need, Kindly let me know what help you might need here

Closed this for now


oneko, xearth, xskat, xsok, x11_ssh_askpass: unbreak on aarch64-darwin

Description of changes
Upstreamed: https://gitlab.freedesktop.org/xorg/util/cf/-/merge_requests/11.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] 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/)
23.05 Release Notes (or backporting 22.11 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 eval

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/762


fix type stability of clipping methods

Closes #73
Seems the problem is less complicated that we thought. target should be a constructed trait rather than a trait type mostly GeoInterfaces.jl does this too for the same reason.

I can't see any problems with tuples, the red block is just array allocation.

Should we do the same for apply?

One reason not to do this in apply is you can't use Union or abstract types. I think that's why I wrote it like that.
That might actually be a problem here too?

Hmm, that's a good point.  How about this:
struct Joint{T}
end

Joint(args...) = Joint{Union{typeof.(args)...}()

and then you pass in a Joint(GI.PointTrait(), GI.LineStringTrait()) to the type?  We can bikeshed the name, but that would be the general principle.
If better for general adoption we could even put this in GeoInterface.

@skygering do you have any thoughts on changing this argument?

Okay, so I originally had that. My only concern relates to @asinghvi17's comments in #66. In my opinion, this is more painful. However, I don't see it being that big of a problem given that we eventually will have default MultiTraits and people will only specify if they need one specific type out (or at least that is what I imagine being the biggest use case). We can also have some pre-defined MultiTraits that are the most common and useful.
Basically, I don't love it, but if it improves performance, then I don't think it is that bad. Does it improve runtime given the type instability doesn't propagate?

Hmm, that's a good point. How about this:
struct Joint{T} <: GeoInterface.AbstractTrait
end

Joint(args...) = Joint{Union{typeof.(args)...}()

and then you pass in a Joint(GI.PointTrait(), GI.LineStringTrait()) to the type? We can bikeshed the name, but that would be the general principle.
If better for general adoption we could even put this in GeoInterface.
Edit: an interesting name might be MultiTrait for this.

I like the name MultiTrait for this much more than Joint.

We could always convert a tuple input at the top level, and type info should propagate through that, so I don't see it being too large of a burden?  It's also not that much different than typing Union{...} and quite a bit cleaner IMP

I dont think MultiTraits will work in practice because we lose dispatch, that we keep with Unions

Summary from Slack:

Acceptable forms of input to target are trait instances (PointTrait()), tuples of trait instances, union types, and multitraits.
These are all converted to multitraits internally and propagated down.
MultiTraits are what we dispatch on even if only one trait is needed, by necessity.

is that the correct understanding of what we just discussed?

Ok, this PR is now implementing TraitTarget wrapper everywhere applicable. Its basically Val with a name. Has the same effect as switching to constructed traits.

I haven't tried this out yet, but did you notice a performance speedup?

The same one as swapping to constructed traits, shown in the profile.
apply is still not 100% type stable in all cases but I assume there is something else going on there.


Fix sorting

Deploy preview for angry-elion-ffa09b ready!
Built with commit 8a9aad53012fc31b975f977ded4acd181a064f8f
https://deploy-preview-10--angry-elion-ffa09b.netlify.com


Stop analysis button no longer works

Should the new icon be a solid square (for stop), rather than a pause symbol? Otherwise, this works now.

Good eye, @AlistairNWard.  Yeah, I think you are right.  I thought the pause button might be more recognizable to the general audience.

I changed this to the 'stop' glyph.  Good eye @AlistairNWard!

I just looked (on stage) and I just saw a gray circle with no symbol.

Hi @AlistairNWard .  Would you mind checking again?  It should look like this:

Now we can add movies!
https://user-images.githubusercontent.com/523111/103294120-33b4cc00-49bf-11eb-8ca4-3c54619475a6.mov

I also cleared out my cache, just in case, but I still just see a gray circle.

This is so strange, @AlistairNWard.  How in the world could we show different glyphs?  I tried in Firefox and Chrome.  What browser are you running in?

This leaves me weary.  I am going to change back to the original glyph.  Now if you tell me you see a smiley face, I'm quitting.

Does it look like this now @AlistairNWard?

Yes, now I see it as just the square.

@AlistairNWard sees a square.  Tonight, I sleep well.

I take my role as bringer of bad news seriously!


[TS] Fix codegen for endpoint returning enums

Fix #4426

@andreaTP I'm not sure this is the right thing to do here. By the time this function is called, codeclass instances have been converted to interfaces and methods to functions.

@andrueastman I was a little surprised too, but it seems like that this phase of the refiner happens before the transformation you mention.
The it test that I added demonstrates the fix is effective, but I'm not sure it's the most correct approach and/or if I'm missing cases (e.g. should we include method parameters in addition to return types?)

@andreaTP You're right, I've confirmed that the interface generation for the request builders happens after the codeEnumObject creation. So modifying the code class object should be right thing to do here.
I've added @koros as a reviewer as he introduced the codeEnumObject to comment on whether the usings these should be added for method parameters as well.
Otherwise, this looks good from my end.

to comment on whether the usings these should be added for method parameters as well.

🙏 yes please, and for any other case you can think about.


Negative Rules

Was asked for in the support group, seems useful enough (and easy to boot :-D ) I'm implementing this for the rewrite branch.
Dunno if it's useful to add this to the master branch for the moment, as the rewrite branch should be workable within a week (I hope), and it's useless to keep fixing and adding features to a (soon to be) deprecated branch.

"rejected" for the 1.x.x series,

Hmm, yeah, good idea to jump the version number like that, definitely if we're going to dump user prefs and such


Projection Pushdown in PhysicalPlan

Which issue does this PR close?

Closes #.
Rationale for this change

Pushing down the ProjectionExec is generally beneficial for execution. Hence, whenever feasible and advantageous, we should aim to swap it with its input. This PR introduces a rule for this purpose. While a similar rule exists in the logical planning stage, some cases may emerge for further optimization after some optimizer rules have worked.
What changes are included in this PR?

This PR introduces a PhysicalOptimizerRule ProjectionPushdown implemented at the final optimization step. The rule initially checks if the operation is a ProjectionExec. If it is, the rule attempts to eliminate it if it's redundant. If not, it examines the input of the projection. Each operator has its own conditions for swapping with a projection. If the conditions are satisfied, the plan tree Projection <-- X <-- Y evolves to X <-- Projection <-- Y. Two projections can always be combined into one, and in some scenarios, projections can be removed from the plan if they can be propagated to the source providers.
Are these changes tested?

Yes, unit tests have been added to cover each operator. Additionally, various .slt test changes show successful optimizations. Benchmark results are as follows:
┏━━━━━━━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━┓
┃ Query        ┃     main ┃ feature_optimize_projections ┃        Change ┃
┡━━━━━━━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━┩
│ QQuery 1     │ 421.93ms │                     421.60ms │     no change │
│ QQuery 2     │  59.86ms │                      59.83ms │     no change │
│ QQuery 3     │ 141.49ms │                     140.42ms │     no change │
│ QQuery 4     │  89.99ms │                      88.31ms │     no change │
│ QQuery 5     │ 173.94ms │                     171.95ms │     no change │
│ QQuery 6     │  84.61ms │                      84.30ms │     no change │
│ QQuery 7     │ 265.81ms │                     251.49ms │ +1.06x faster │
│ QQuery 8     │ 203.04ms │                     200.24ms │     no change │
│ QQuery 9     │ 285.64ms │                     285.41ms │     no change │
│ QQuery 10    │ 224.73ms │                     220.80ms │     no change │
│ QQuery 11    │  57.97ms │                      58.64ms │     no change │
│ QQuery 12    │ 158.46ms │                     158.17ms │     no change │
│ QQuery 13    │ 231.70ms │                     221.08ms │     no change │
│ QQuery 14    │ 119.61ms │                     119.42ms │     no change │
│ QQuery 15    │  82.82ms │                      83.33ms │     no change │
│ QQuery 16    │  51.07ms │                      50.90ms │     no change │
│ QQuery 17    │ 222.45ms │                     204.10ms │ +1.09x faster │
│ QQuery 18    │ 428.66ms │                     436.40ms │     no change │
│ QQuery 19    │ 239.63ms │                     241.73ms │     no change │
│ QQuery 20    │ 137.80ms │                     141.81ms │     no change │
│ QQuery 21    │ 354.14ms │                     362.21ms │     no change │
│ QQuery 22    │  52.01ms │                      53.07ms │     no change │
└──────────────┴──────────┴──────────────────────────────┴───────────────┘

Are there any user-facing changes?

@crepererum  implemented something similar to this in IOx -- can you please review this PR as well @crepererum ? Maybe we can contribute some of IOx's tests cases back upstream?

I have a follow on PR with some proposed simplifications: https://github.com/apache/arrow-datafusion/pull/8109#


why it's dependent on active_support?

From the gemspec there is no dependency for active_support, but in resource_finder.rb, there is  require 'active_support/concern'.
I also suggest to set up a ci tool for this project :)

Yes, because this gem strongly relies on Rails. However Rails has already required active_support/concern, then It's not necessary to require one more time in resource_finder.


Add autoupdate feature

Can you add this feature to the container image?
That way the tailscale app internally can auto-update itself.
This is how I do currently:

Since it's non package manager installation, I would suggest in the update.rsc you add the command from the image and an option to stop and start the container after update is complete, in order to get tailscaled to restart properly

Hi,
Thanks for the suggestion. I think this would be possible if the docker image is built using the Alpine package to enable the auto-update feature to work.
Then we would add an environment variable, e.g. AUTO_UPDATE to run tailscale set --auto-update in tailscale.sh

Yes, that would be better

It seems as though tailscale relies on packages provided via Alpine - see here, so I don't think auto-update will be possible.
The next best option as you suggest is to add an environment variable UPDATE_TAILSCALE that will run tailscale update --yes prior to starting tailscaled & tailscale.
In this way if a new tailscale version was available, starting and stopping the container would apply the update.

See 0.1.26


How to support LFS?

Configure Git Large File Storage

I take it this answered your question? Can you close this issue or did you have more questions?


Add an XCode build

Coverage remained the same at 98.515% when pulling 7203c2a2d3f497454759853fa92b4e61d489c6e7 on thetic:xcode into a521a90d35ba267089f6bb1fd28c754f9f4a04b9 on cpputest:master.


set global varibal for internal API

Hi, How can i declare global variable in src/api/index.js ?
I has declare my variable as "var myItem = []" at the top of file. But every time a route triggerd, it's value always be reseted to initial ([] in this case)

@lamvananh you cannot do it like that. Please consider using Redux instead.

Hi, could you show me how to dispatch and consume a redux action in this project ?. I  don't find this in the document, it's talking about the config step only

@lamvananh , you can checkout https://github.com/Atyantik/example-pawjs-redux


Fixed service selector (Kubernetes)

Closes #2709

Damn!
I'll always refresh the page 🤣
Fixed by #2723


Gyro drifting

Gyro drift has almost been eliminated by first calibrating the jackal robot by following the instructions here:
https://www.clearpathrobotics.com/assets/guides/jackal/calibration.html
After the calibration, there was some noise, which caused the robot world frame to oscillate in a rotational way around z-axis.
The above oscillation was reduced to some acceptable range by setting the 'gain' parameter of the imu_filter node from 0.1 to 0.01. This was changed in this file on the robot:
/etc/ros/jackal_base (will update this later)
rosrun dynamic_reconfigure dynparam set imu_filter gain 0.01 (temporary command line fix - to be done after every boot)

Keeping this issue open  for a couple more weeks to ensure this does not reoccur

Issue still present for outdoor data. Recomend to use a homing beacon by absolute GPS coordinates. (1711_1)


Update README.md

Fix readme.md

Coverage remained the same at 99.15% when pulling a3c48c31010669ea7f0eb31a463b33fc73a5b391 on MatayoshiMariano:patch-1 into 3b52ad07fb6283ae3170407d795a06c34da2cca1 on inf0rmer:master.

Thanks!


unbound not doing upstream tcp connectivity

Describe the bug
A clear and concise description of what the bug is.
When running unbound with loglevel 3 or above, I see lots of these messages:
outnettcp got tcp error -1
and the occasional (but much less often):
tcp error for address <IP_ADDRESS> port 853
which is configured as a DoH upstream server.
When looking at the stats, the TCP usage is always 0.
To reproduce
Steps to reproduce the behavior:

set loglevel to 3 or above
restart unbound
do some random DNS queries

Expected behavior
A clear and concise description of what you expected to happen.
Unbound being able to successfully use tcp transfers.
System:

Unbound version: 1.13.1
OS: OPNsense 21.7.1 (FreeBSD 12.1-RELEASE-p19-HBSD)
unbound -V output:

Version 1.13.1
Configure line: --with-ssl=/usr/local --with-libexpat=/usr/local --disable-dnscrypt --disable-dnstap --with-libnghttp2 --enable-ecdsa --disable-event-api --enable-gost --with-libevent --with-pyunbound=yes --with-pythonmodule=yes LDFLAGS=-L/usr/local/lib ac_cv_path_SWIG=/usr/local/bin/swig --disable-subnet --disable-tfo-client --disable-tfo-server --with-pthreads --prefix=/usr/local --localstatedir=/var --mandir=/usr/local/man --infodir=/usr/local/share/info/ --build=amd64-portbld-freebsd12.1
Linked libs: libevent 2.1.12-stable (it uses kqueue), OpenSSL 1.1.1k  25 Mar 2021
Linked modules: dns64 python respip validator iterator
BSD licensed, see LICENSE in source package for details.
Report bugs to<EMAIL_ADDRESS>or https://github.com/NLnetLabs/unbound/issues
Additional information
Add any other information that you may have gathered about the issue here.
I have double-checked the firewall and NAT configuration, and nothing seems to be blocking DNS related TCP traffic, or any traffic originating from the firewall itself.

Can you show what the tcp error is? I thought it would also log the error description with the tcp error?
These errors are not shown a lower verbosity, and they show because you ask you see them. This is what happens to the socket connections. And it is logged for you.

It doesn't log anything else unfortunately, so I'm not sure where to start
looking next. Can you give any pointers? Or should I run with a higher
loglevel?
Met vriendelijke groet,
Derkjan de Haan
Op wo 25 aug. 2021 om 09:48 schreef Wouter Wijngaards <
@.***>:

Can you show what the tcp error is? I thought it would also log the error
description with the tcp error?
These errors are not shown a lower verbosity, and they show because you
ask you see them. This is what happens to the socket connections. And it is
logged for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NLnetLabs/unbound/issues/535#issuecomment-905267162,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMA2GFUZSJSOFZYHZ2EYTSTT6SN6RANCNFSM5CXLUVIA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Can you give some context, preferably on verbosity 4 or more, around these log lines? For a couple examples? I think this may also get logged for like, TCP resets, host down, and maybe stuff like timeouts.
The query num tcp stat is downstream queries, not upstream ones. So I guess your clients use UDP not TCP.

The values are called 'num.query.tcp' for incoming tcp and 'num.query.tcpout' for outgoing tcp. The man page for unbound-control also lists the statistics output as incoming connections for the tcp usage, which is about the number of buffers in use.


logging: add template for log configuration

Added Kconfig template file to be used when log configuration
in a module is added. Added example usage of kconfig template
in logger sample.
It is using new kconfig feature added in: #9797 (issue #9761)
Signed-off-by: Krzysztof Chruscinski<EMAIL_ADDRESS>

fyi @Olivier-ProGlove @jhedberg @jukkar see the example how log configuration can be established in each module:
https://github.com/zephyrproject-rtos/zephyr/blob/1d8b49eb592a13e966949d0603b9e36d4bf2ea12/samples/subsys/logging/logger/Kconfig#L13

@jukkar done.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #9933   +/-   ##
=======================================
  Coverage   52.51%   52.51%
=======================================
  Files         213      213
  Lines       26082    26082
  Branches     5624     5624
=======================================
  Hits        13696    13696
  Misses      10135    10135
  Partials     2251     2251

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 0329ec7...f40bac7. Read the comment docs.

I have been porting this to networking logging code and was wondering if something like this would be possible (ping @ulfalizer) :
Kconfig:
module=FOOBAR
module-str=FOOBAR component
module-help=Enables FOOBAR debug messages.
source "Kconfig.log_level_template"

Kconfig.log_level_template:
choice
	prompt "Log level for $(module-str)"
	default $(module)_LOG_LEVEL_WRN
	help
	  $(module-help)

Current code outputs the help text just plain "$(module-help)" instead of expected help text.

Current code outputs the help text just plain "$(module-help)" instead of expected help text.

No macro expansion is done in help texts, so not currently possible. I wonder if it would break stuff in case help texts talk about Make variables, though that's unlikely in Zephyr.
Do you think making a reference to the prompt would be possible in the meantime, or making it generic ("enables some-feature for the subsystem" or the like)?

good point. it is work updating the documentation.

This is great. Just a minor comment, should we settle on an extension for Kconfig templates? I would perhaps suggest “.ktemplate”

@carlescufi would you propose renaming file to log_config.ktemplate ?

@carlescufi would you propose renaming file to log_config.ktemplate ?

Actually looking at what we do today:
Kconfig.defconfig.nrf51822_QFAC
I think the consistent naming scheme would be:
Kconfig.template.logging

@carlescufi maybe Kconfig.template.log_config since in future we might want to add another template (all must be in separate files) so Kconfig.template.logging is to wide. Don't you think?

@nordic-krch
Sounds good to me.


disable no-shadowed-variable with unused parameter (_)

Feature request
A code example:

// config.get = (key, (err, value) => void);

config.get("someFeature.available", (_, available) => {
  if (!available) return;
  config.get("someFeature.enabled", (_, enabled) => {
    if (!enabled) return;
    config.get("someFeature.url" (_, url) => {
      service.call(url, ...);
    });
  });
});

In this case, config.get is provided by 3rd party, and I want just ignored the error. So I used _ as a placeholder for err parameter.
But TSLint warn me that Shadowed name '_'.
I think it would be perfect if we can except the _ variable for this rule.

💀 It's time! 💀
TSLint is deprecated and no longer accepting pull requests other than security fixes. See #4534. ☠
We recommend you instead use typescript-eslint to lint your TypeScript code with ESLint. ✅
👋 It was a pleasure open sourcing with you!

@JoshuaKGoldberg Thank you, it's a Angular project, it's not support ESLint now.
I will continue to focus on the migration progress.
Ref:
https://github.com/angular/angular-cli/issues/13732
https://github.com/mgechev/codelyzer/issues/763

🤖 Beep boop! 👉  TSLint is deprecated 👈  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


Added colors to the debug output sections

Small contributions from Dublin Scala Sprees:
I'm working on #338
I'm trying to prettify logs

Hi @muxanick,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:
http://www.lightbend.com/contribute/cla

To validate this pull request against other sbt modules, please visit this link.

To validate this pull request against other sbt modules, please visit this link.

To validate this pull request against other sbt modules, please visit this link.

To validate this pull request against other sbt modules, please visit this link.

To validate this pull request against other sbt modules, please visit this link.

A validation involving this pull request is in progress...

The validator has checked the following projects against Scala 2.12,
tested using dbuild, projects built on top of each other.

Project
Reference
Commit

sbt
1.x
sbt/sbt@34945af6314dee3c9ea55a76c882007f44819509

zinc
pull/407/head
sbt/zinc@7d1db6b1b668be98150d0243fc3e2b61f58a8f15

io
1.x
sbt/io@62004b28dfcdc33273ebb07a563e78178f6eb21d

librarymanagement
1.x
sbt/librarymanagement@04193716da1d7454d1d7c5935b396ebeefc8c2d4

util
1.x
sbt/util@cba6f83447b84cc1e967f25a930d4f411f6f0664

website
1.x

:white_check_mark: The result is: SUCCESS
(restart)

Hey @muxanick, you did a great job here! @eed3si9n did some work on this area some time ago, and this PR is no longer updated.  I'll add your change in the future and credit it! :smile:. Thank you for doing this.


Parse causing crash on android.  Cannot catch.

What is the issue you have?

Running and android app on android 10.  Sometimes, when using nlohmann::json::parse my application will crash.  I am parsing simple strings like {"stringy":{"value":"a string"}}.  Sometimes it works and I can use the resulting object.  Mostly it crashes with the error:
0xebadde09 skipped times: 0
A/libc: Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x227245ef88a0 ...
A try catch block will not help at all.  It doesn't catch anything.  The log message before the parse prints and then the app crashes immediately.
I know there is some information about 0xebadde09 being related to hardware acceleration but I have disabled that as well (this is running in a background service anyway).  Any hints on what to do or where to look would be appreciated.  I've been trying many permutations with no luck.

Please describe the steps to reproduce the issue. Can you provide a small but working code example?

nlohmann::json::parse a valid std::string.

What is the expected behavior?

Successful parse

And what is the actual behavior instead?

Crash

Which compiler and operating system are you using? Is it a supported compiler?

Using android studio with the -std=c++11 flag set.

Did you use a released version of the library or the version from the develop branch?

Released 3.6.1 and 3.7.3

If you experience a compilation error: can you compile and run the unit tests?

N/A

Do you have more information like a stack trace or example code including the input. Did you try to run the code with address sanitizer enabled or using Valgrind?

This is the stack:
art_sigsegv_fault 0x0000007be641f864
art::FaultManager::HandleFault(int, siginfo*, void*) 0x0000007be641fd74
___lldb_unnamed_symbol24$$app_process64 0x0000005a4cba0dfc
<unknown> 0x0000007c6dcd1690
nlohmann::detail::json_sax_dom_parser<nlohmann::basic_json<std::__ndk1::map, std::__ndk1::vector, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, bool, long, unsigned long, double, std::__ndk1::allocator, nlohmann::adl_serializer> >::key(std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >&) json.h:4530
bool nlohmann::detail::parser<nlohmann::basic_json<std::__ndk1::map, std::__ndk1::vector, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, bool, long, unsigned long, double, std::__ndk1::allocator, nlohmann::adl_serializer> >::sax_parse_internal<nlohmann::detail::json_sax_dom_parser<nlohmann::basic_json<std::__ndk1::map, std::__ndk1::vector, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, bool, long, unsigned long, double, std::__ndk1::allocator, nlohmann::adl_serializer> > >(nlohmann::detail::json_sax_dom_parser<nlohmann::basic_json<std::__ndk1::map, std::__ndk1::vector, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, bool, long, unsigned long, double, std::__ndk1::allocator, nlohmann::adl_serializer> >*) json.h:8862
nlohmann::detail::parser<nlohmann::basic_json<std::__ndk1::map, std::__ndk1::vector, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, bool, long, unsigned long, double, std::__ndk1::allocator, nlohmann::adl_serializer> >::parse(bool, nlohmann::basic_json<std::__ndk1::map, std::__ndk1::vector, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, bool, long, unsigned long, double, std::__ndk1::allocator, nlohmann::adl_serializer>&) json.h:8767
nlohmann::basic_json<std::__ndk1::map, std::__ndk1::vector, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, bool, long, unsigned long, double, std::__ndk1::allocator, nlohmann::adl_serializer>::parse(nlohmann::detail::input_adapter&&, std::__ndk1::function<bool (int, nlohmann::detail::parser<nlohmann::basic_json<std::__ndk1::map, std::__ndk1::vector, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, bool, long, unsigned long, double, std::__ndk1::allocator, nlohmann::adl_serializer> >::parse_event_t, nlohmann::basic_json<std::__ndk1::map, std::__ndk1::vector, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >, bool, long, unsigned long, double, std::__ndk1::allocator, nlohmann::adl_serializer>&)>, bool) json.h:20750
..... (going into our stuff)

These are some of the values in the top level of the stack.  Not really sure what is happening.

The string going in: "{"logically":{"value":"True"}}"
void Parameters::updateParameters(std::string newParams) {
    try {
    Log::v("PARAMETERS", "Parameter update %s, %s", newParams.c_str(), params.dump().c_str());
    // this will just be the variable name and the new value
    nlohmann::json newParamObject = nlohmann::json::parse(newParams);
...

Is the input being corrupted somehow?  Not really sure if this is a json library problem but it keeps happening here.

Hi, we managed to isolate the json out of the crash.  Something is happening at the same time as the json call and ruining the memory.  Sorry for wasting time!

Hi how did you manage to resolve this issue? I am experiencing same issue. App crash is happening on specific network while parsing JSON. Exception not caught. Not entering in rescue block as well.

@YaminiT-94 The information was being sent via udp and being received and sent down to c++ through jni.  Once the data was processed it was sent off via an internal messaging system and then the local variables were deleted.  There was an extra delete of a variable that just happened to always happen at the same time as the json data getting parsed as they were being executed on different threads.
Basically it was completely unrelated to the json and had to do with an extra delete causing a crash.

Hi, in my case application is crashing due to.poor network. In watchdog crash report I received excpetionas below:
Exception Type: EXC_CRASH (SIGKILL)
Exception Codes: 0x0000000000000000, 0x0000000000000000
Exception Note: EXC_CORPSE_NOTIFY
Termination Reason: Namespace SPRINGBOARD, Code 0x8badf00d
Termination Description: SPRINGBOARD, scene-update
As given solution in Apple documentation I cannot convert that synchronous call to asynchronous, neither can put on secondary thread. No use of increasing network timeout. Watchdog timeout is less than network timeout.
Any suggesstion/solution on this?


Add information about reserved places to the checkin result views.

Pull Request Test Coverage Report for Build 427

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 99.735%

Files with Coverage Reduction
New Missed Lines
%

attendee/views.py
7
97.37%

Totals

Change from base Build 426:
-0.3%

Covered Lines:
3440

Relevant Lines:
3447

💛  - Coveralls


Does not create a directory when saving a file

I get an error when I try to save a file in a non-existent directory.
New filename: /home/dios/public/files/image/2019-05-24/thumbnail/JDOM0IvWtFomoHLvRZ4go0Hd5_thumbnail_tiny.jpeg
Existing path: /home/dios/public/files/image/2019-05-24/
Need to create the directory 'resize': /home/dios/public/files/image/2019-05-24/resize/
When saving, I have to use additional functions to create directories.
$dir = dirname($filename);

if (! is_dir($dir)) {
       mkdir($dir, 0777, true);
}

yes, i don't think this is a bug. I believe that creating directories is "out of scope" for Intervention. Intervention is only for manipulating images. Manipulating the OS filesystem is up to the user and their framework or OS commands.

I can understand that automatically creating directories would be very convenient. However, I see the danger of unintentional creation of folders and that the file system becomes cluttered as a result. I prefer to create the folder structure manually.
Alternatively, packages that are specifically created for managing file systems can be used for this task.
Abstract example with imaginary filesystem package
$encoded = $image->toJpeg();
$filesystem->write('foo/bar/baz/example.jpg', (string) $encoded);


fix: breadcrumb not jump

General

✏ Mark the necessary items without changing the structure of the PR template.

[x] Pull request template structure not broken

Type

ℹ What types of changes does your code introduce?

👉 Put an x in the boxes that apply

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

ℹ Check all checkboxes - this will indicate that you have done everything in accordance with the rules in CONTRIBUTING.

👉 Put an x in the boxes that apply.

[ ] My code follows the style guidelines of this project
[ ] Is the code format correct
[ ] Is the git submission information standard?
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

其实改成这样主要是为了消灭 直接import router from '/@/router'的写法，这个写法热更新会出问题。这个bug我在看看怎么弄

ok

在useGo和useRedo传人两个参数，第二个参数是useRouter是不是可以避免这个问题呢

应该是可以。router从外部传入


Exit controller service with error when not configured

Otherwise, the init system will not know that the start failed.

https://ocd.appscale.com:8080/job/Daily Build/3133/
The Hawkeye failure was due to a rabbitmq cookie issue.


Update to vuestic-ui 1.5.0

[x] at least no error when update vuestic-ui
[x] app.use(createVuestic({ config: vuesticGlobalConfig })) error with colors property in config
need to move colors value to {colors: { variables: <prev colros value>}}
[x] buttons not rounded
add      VaButton: {       round: true,     } to components config
[x] text in sidebar still dark when section is selected
just remove VaSidebar config pfrom globalConfig and it is fixed?
[ ] dropdowns in admin header (colors, countries) are deadб popover also dead
[x] breakpoints has been changed(?) so now classes like xl6 have some problems, breakpoints hardcoded in scss
[x] helper classes renamed, -- -> -, gutter--xl -> va-gutter, spacing classes must changed, display-2 -> va-h2
[ ] accordion multiply prop renamed to multiple
[ ] tree view dead

@m0ksem multiply prop accordion prop doesn't was renamed, do we need to do that?

Yeah


Currently it allows entering alpha characters but maybe restricting these on a type event would be ideal

Hey, just following up whether you have plans to implement this.
Thanks !!

Not really sold on this idea; never was a big fan of typing and being restricted and not seeing any feedback.
Could provide hooks for it tho. I've stopped using this component and I'm maintaining for the sake of it atm, if you have suggestions on how hooks could solve this let me know and I could spend some time on it.


fix doc

1、修正了树莓派文档中--model_dir 参数说明不正确的问题

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wangsiyuan06 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


dep init to hang indefinitely

What version of dep are you using (dep version)?
$ dep version
dep:
version     : v0.4.1
build date  : 2018-01-27
git hash    : 37d9ea0
go version  : go1.9.3
go compiler : gc
platform    : darwin/amd64
What dep command did you run?
dep init -v
What did you expect to see?
Success with new files
*Gopkg.toml
*Gopkg.lock
/vendor/
What did you see instead?
Getting direct dependencies...
Checked 8 directories for packages.
Found 6 direct dependencies.
Root project is "gitlab.dmall.com/arch/fisher"
6 transitively valid internal packages
6 external packages imported from 6 projects
(0)   ✓ select (root)
(1)     ? attempt github.com/lestrrat-go/file-rotatelogs with 1 pkgs; 4 versions to try
(1)         try<EMAIL_ADDRESS>(1)     ✓ select<EMAIL_ADDRESS>w/1 pkgs
(2)     ? attempt github.com/lestrrat/go-strftime with 1 pkgs; 1 versions to try
(2)         try github.com/lestrrat/go-strftime@master
(2)     ✓ select github.com/lestrrat/go-strftime@master w/1 pkgs
(3)     ? attempt github.com/howeyc/fsnotify with 1 pkgs; 16 versions to try
(3)         try<EMAIL_ADDRESS>(3)     ✓ select<EMAIL_ADDRESS>w/1 pkgs
(4)     ? attempt github.com/spf13/cobra with 1 pkgs; 4 versions to try
(4)         try<EMAIL_ADDRESS>(4)     ✓ select<EMAIL_ADDRESS>w/1 pkgs
(5)     ? attempt github.com/inconshreveable/mousetrap with 1 pkgs; 2 versions to try
(5)         try<EMAIL_ADDRESS>(5)     ✓ select<EMAIL_ADDRESS>w/1 pkgs
(6)     ? attempt github.com/rifflock/lfshook with 1 pkgs; 16 versions to try
(6)         try<EMAIL_ADDRESS>(6)     ✓ select<EMAIL_ADDRESS>w/1 pkgs
(7)     ? attempt github.com/spf13/pflag with 1 pkgs; 4 versions to try
(7)         try<EMAIL_ADDRESS>(7)     ✓ select<EMAIL_ADDRESS>w/1 pkgs
(8)     ? attempt github.com/pkg/errors with 1 pkgs; 11 versions to try
(8)         try<EMAIL_ADDRESS>(8)     ✓ select<EMAIL_ADDRESS>w/1 pkgs
(9)     ? attempt github.com/sirupsen/logrus with 1 pkgs; 48 versions to try
(9)         try<EMAIL_ADDRESS>... hang here idefinitely

Hey @reallax, could you provide a smaller example that recreates the issue? Something like this:
// $(go env GOPATH)/src/dep-1825/main.go
package main

import (
	_ "package/that/causes/hang"
	// Add more packages if needed
)

func main() {}

I should be able to run dep init -v inside the $(go env GOPATH)/src/dep-1825 package and hang indefinitely. That'll make it a lot easier to figure out what's going on. Thanks!

Thanks for your reply @mattayes, the example is like this:
`package main
import (
"fmt"
"github.com/sirupsen/logrus"
"os"
)
var log = logrus.New()
func main() {
fmt.Println("init dep demo...")
log.Out = os.Stdout

log.WithFields(logrus.Fields{
	"animal": "walrus",
	"size":   10,
}).Info("A group of walrus emerges from the ocean")

}after exec dep init -v, the output is as follow:Getting direct dependencies...
Checked 1 directories for packages.
Found 1 direct dependencies.
Root project is "awesomeProject"
1 transitively valid internal packages
1 external packages imported from 1 projects
(0)   ✓ select (root)
(1)     ? attempt github.com/sirupsen/logrus with 1 pkgs; 48 versions to try
(1)         try<EMAIL_ADDRESS>`
maybe that is helpful.