Do not create project folder by default

Currently, the extension forces the creation of a .vscode directory on every folder you open.  It'd be cool if there was a way to avoid this.
Possibilities:

Do not store colours permanently unless there're already project settings.
Make it configurable.

If colours are based on the root dir name you should get the same colour every time anyway.

Hi kAlvaro, thanks for the feedback!
I updated the extension to delete the .vscode/settings.json file as each window is closed.  As far as I know, it's not possible to change a specific window color without writing it to that file, but this should help keep the filesystem clean as you are opening and closing folders in VS Code.

Hi kAlvaro, thanks for the feedback!
I updated the extension to delete the .vscode/settings.json file as each window is closed. As far as I know, it's not possible to change a specific window color without writing it to that file, but this should help keep the filesystem clean as you are opening and closing folders in VS Code.

Has the change been published in Marketplace? In 1.0.5 it still keeps both file and directory.
It's a pity that you need permanent settings to change window colours. Considering that, it'd be nice to be able to restrict colour generation to projects that already have custom settings. But I guess that'd need to be a new ticket. Whatever, thanks for the addon!

I don't have a .vscode directory in the first place (that's what the ticket was about). No idea of what's happening then :-?

Is the extension modifying your colors without creating a .vscode folder?
I thought initially you said the problem was that it was making a .vscode folder in every window you opened which was cluttering your filesystem?
I have only tested it on a mac -- I'll try to test in on a PC soon.

Colour generation works as advertised. The problem is that it isn't removing neither the file nor the directory afterwards.
I may test in another PC later this week if I have the chance, just to discard stuff.

did you try manually deleting .vscode/settings.json?

I've just tried in my other computer (Windows 10 as well) and it happens too. I start Visual Studio Code in a folder that doesn't have a previous project and it creates the .vscode folder the settings.json file. The file only contains the settings added by the extension. I close the program and both the file and the directory remain intact. I can remove them manually but when I start VSCode again the cycle repeats.
Perhaps the issue is exclusive to Windows :-?

okay, i tried it on a Windows 10 EC2 instance.
one weird thing is that the first window i had open with the extension created a .vscode/extension directory.  but then i never saw it in vscode windows i opened later.  i never saw this folder on my mac, but it doesn't necessarily seem relevant here.
but i wasn't able to reproduce your issue.  .vscode/settings.json was deleted each time i closed a vscode window.
with vscode open:
PS C:\Users\Public\Documents> ls .vscode

    Directory: C:\Users\Public\Documents\.vscode

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----        12/6/2018   3:31 PM            194 settings.json

after closing vscode:
PS C:\Users\Public\Documents> ls .vscode
ls : Cannot find path 'C:\Users\Public\Documents\.vscode' because it does not exist.`

what are the contents of your .vscode folder while your window is open?

I think I finally found a hint of what's going on. If I run a clean set up:
 code --user-data-dir . .

... the settings.json file has something like this:
{
    "workbench.colorCustomizations": {
        "activityBar.background": "#05360E",
        "titleBar.activeBackground": "#064B13",
        "titleBar.activeForeground": "#F2FEF4"
    }
}

... and gets removed when I close the file. However, if I launch the program normally:
code .

... I see:

{
    "workbench.colorCustomizations": {
        "tab.activeBorder": "#26A6F2",
        "tab.unfocusedActiveBorder": "#26A6F230",
        "activityBar.background": "#05360E",
        "titleBar.activeBackground": "#064B13",
        "titleBar.activeForeground": "#F2FEF4"
    }
}

That's because I've customised tab.activeBorder and tab.unfocusedActiveBorder in my global settings and when the extension creates project settings those two other directives get included too. Since the file has extra settings, it remains.

ah, great work!  i'll work on addressing this next week.  maybe i can check to see if any colorCustomizations are the same as global settings.  or maybe the extension is doing something weird to cause those tab colors to get written to the settings.json file.  either way thanks for rooting this out.

hey @kAlvaro - check it out!

setting Window Colors: 🌈 Delete Settings File Upon Exit to true will indiscriminately delete .vscode/settings.json when you close the window, regardless of file contents or modifications.
(needs to be selected in User Settings (vs Workspace Settings or else it will just delete itself 💥)
let me know how this latest version works for you

I've been playing with some local and network folders and it's been working as advertised—including the remove workspace customizations part ;-)
It's kind of scary and far from ideal but I guess there aren't many options left if Visual Studio Code doesn't provide an API to change colours programmatically. I have no idea if further refinements are possible or worth the time. Feel free to reopen the ticket.


Add UseSnakeCaseNamingConvention extension method

As mentioned in https://github.com/npgsql/npgsql/pull/1690.
I wanted to just open this up early :smile:
Had to update to the 3.0.0 CI package. I'll strip off that commit once a stable 3.0.0 is out.
I'll see if I can add some tests as well. Do you have any pointers on how you'd like them written?

Hey again @khellang!
There's nothing in this PR which is actually Npgsql-specific - it's just generic EF Core code which which modifies table/column names on the model, and could be used with SQL Server/Sqlite just as well (it just happens to use a public function exposed by Npgsql). So I'd see this more as a completely separate nuget (e.g. EntityFrameworkCore.SnakeCaseNaming?), rather than something done inside the Npgsql provider.
Note that EF Core supports a concept of conventions (unlike your extension method which happens to be named "Convention"), which may be a more appropriate way to implement what you want. You may want to look into that.

There's nothing in this PR which is actually Npgsql-specific - it's just generic EF Core code which which modifies table/column names on the model, and could be used with SQL Server/Sqlite just as well

Yes, I realize that, but since it's really common (idiomatic, even?) to use snake_case naming with Postgres, and the default PascalCase naming convention requires quotes everywhere, I figured it would be a nice addition in this package. It would be even better if EntityFramework supported different naming schemes out of the box (at which point, this method could be deprecated and delegate to the EF version).

it just happens to use a public function exposed by Npgsql

And that's not a coincidence, right? It's really handy for the reasons mentioned above.

So I'd see this more as a completely separate nuget (e.g. EntityFrameworkCore.SnakeCaseNaming?), rather than something done inside the Npgsql provider.

Installing a totally separate package and assembly just to get a single method to change the naming convention to snake_case sounds a bit overkill. At that point I'd rather just copy the code.

Note that EF Core supports a concept of conventions (unlike your extension method which happens to be named "Convention"), which may be a more appropriate way to implement what you want. You may want to look into that.

I don't think that's the case for naming. See
https://github.com/aspnet/EntityFrameworkCore/issues/214
https://github.com/aspnet/EntityFrameworkCore/issues/4445
https://github.com/aspnet/EntityFrameworkCore/issues/5159
https://stackoverflow.com/questions/37493095/entity-framework-core-rc2-table-name-pluralization
https://stackoverflow.com/questions/34700281/entity-framework-7-pluralize-table-names-with-code-first-approach
These are just from a quick Google search :wink:
All the official responses suggests this is the way to customize the names. I'd be happy to be proven wrong, though :smile:
Also related: https://github.com/npgsql/Npgsql.EntityFrameworkCore.PostgreSQL/issues/21

I definitely agree it would be good if EF supported pluggable naming schemes, and this has been requested several times (e.g. https://github.com/aspnet/EntityFrameworkCore/issues/5159). Even if snake case is common (but not universal) in PostgreSQL, it is also common in other databases.
So basically my main objection about including this in Npgsql is that this kind of thing either belongs in EF Core itself, or externally. Why not try submitting a PR to EF Core? If you show the complexity of correctly doing snakecase (including all the acronym work etc.), and argue that it's a shame to have to duplicate this code for every user, I'm hoping they may accept it (even if they might ask for a slightly different implementation.

So I'd see this more as a completely separate nuget (e.g. EntityFrameworkCore.SnakeCaseNaming?), rather than something done inside the Npgsql provider.

Installing a totally separate package and assembly just to get a single method to change the naming convention to snake_case sounds a bit overkill. At that point I'd rather just copy the code.

You're absolutely right that a nuget package with an assembly just for this one extension method is very heavy - but there's also the option of simply adding the a source file to your "content" dir. This removes the additional assembly, and is just a way to drop a source file (containing your extension) into the consumer's project. I still thing it's better for this to be in EF Core, but if not there's always this option.


Move repo under @risingstack on npm

Published alpha release is here: https://www.npmjs.com/package/@risingstack/react-easy-state
I published it manually without a release commit to not mess with future semantic release integration. Currently published version is 6.1.3 (tagged with alpha), we should bump to 6.2.0 with the first official release.

We should not merge this until the upcoming official release (because of all the references to @risingstack/react-easy-state in the readme).

Coverage increased (+0.01%) to 95.985% when pulling 7d21c97082abf507ecd97a023b785f5825f034f5 on rs-repo into b3240e8885460a59822a9f0dd8b29a8eae197837 on master.

I will redo this. I got into a huge git mess which I don't want to resolve 😄 Lessons learned, next time we will start with a release branch instead of these long none merged PRs.


Incorrect usage of _observerCallback

The globals.Document._setupMigrations method incorrectly uses _observerCallback from PeerDB. This results in errors similar to the following:
E20160226-16:06:59.653(1) (lib.coffee:1181) PeerDB exception: TypeError: Cannot call method 'apply' of undefined: [ {} ]
E20160226-16:06:59.654(1) (lib.coffee:1182) TypeError: Cannot call method 'apply' of undefined
    at packages/peerlibrary_peerdb/lib.coffee:1176:11
    at packages/mongo/observe_multiplex.js:183:1
    at Array.forEach (packages/es5-shim/.npm/package/node_modules/es5-shim/es5-shim.js:417:1)
    at Function._.each._.forEach (packages/underscore/underscore.js:105:1)
    at Object.task (packages/mongo/observe_multiplex.js:177:1)
    at [object Object]._.extend._run (packages/meteor/fiber_helpers.js:147:1)
    at packages/meteor/fiber_helpers.js:125:1

The cause is the following line in server.coffee:

https://github.com/peerlibrary/meteor-peerdb-migrations/blob/master/server.coffee#L555

The first argument should be something else, not a function.

Fixed. Thanks for reporting.


Code Reproduction Issues

Dear @flowersteam,
trying to reproduce your results for coursework.
I find there are a number of issues in running the code. Here is a list of what I found so far.
Importing
Several files within the experiments folder, e.g., experiments/agents/ppo/base_ppo_agent.py, call imports such as, from experiments.agents.etc import etc. These imports fail as the files themselves are already inside the experiments folder, as per my tests. Simply eliminating the experiments name at the beginning of the import call solves these issues.
Class Naming
Some classes fail to import as, I believe, have changed names over time. This is the case for, e.g., ValueModuleFn (imported in post-training_tests.py) which I assume has changed name to ValueHeadModuleFn.
Training Using Lamorel (on one GPU)
After fixing these minor issues, I managed to launch a training with the default single-GPU config, using the Flan-T5-small LLM and using Lamorel in two separate server-client instances as described in #11 -- obtaining weird results (reward flat to 0.0 for all episodes). I attach my logs
std.txt and final results log.csv for clarity.
Testing Using Lamorel (on one GPU)
I could not manage to run the post-training_tests.py script in the same way as the training script, as I find several issues with it. Using once again the standard Lamorel config for one GPU, two server-client instances and the previously trained model, errors occur when initializing the LLMPPOAgent named algo at line 390. Once again, I believe there was some change in the initialization process as there are several parameters that require specification and are missing, such as num_frames_per_proc (default is None, but if kept at default an error occurs when initializing self.obss in the BasePPOAgent superclass initialization). Required logging information (saving_path_logs, saving_path_model, id_expe) is not passed as well. Also, some optional parameters (reshape_reward, subgoals) are not explicitly named (ex.: LLMPPOAgent(..., reshape_reward=reshape_reward)), thus their values are stored in different variables when the object is initialized. Again, these were easy fixes. However, I later find myself stuck at an error occurring at line 310, which is reported below:
/Absolute/Path/To/Grounding_LLMs_with_online_RL/lamorel/lamorel/src/lamorel_launcher/launch.py:15: UserWarning:
The version_base parameter is not specified.
Please specify a compatability version level, or None.
Will assume defaults for version 1.1
  @hydra.main(config_path='', config_name='')
/Absolute/Path/To/miniconda3/envs/dlp/lib/python3.10/site-packages/hydra/_internal/hydra.py:119: UserWarning: Future Hydra versions will no longer change working directory at job runtime by default.
See https://hydra.cc/docs/1.2/upgrades/1.1_to_1.2/changes_to_job_working_dir/ for more information.
  ret = run_job(
[DATE TIME][root][INFO] - Using nproc_per_node=1.
/Absolute/Path/To/Grounding_LLMs_with_online_RL/experiments/train_language_agent.py:373: UserWarning:
The version_base parameter is not specified.
Please specify a compatability version level, or None.
Will assume defaults for version 1.1
  @hydra.main(config_path='config', config_name='config')
/Absolute/Path/To/Grounding_LLMs_with_online_RL/experiments/post-training_tests.py:281: UserWarning:
The version_base parameter is not specified.
Please specify a compatability version level, or None.
Will assume defaults for version 1.1
  @hydra.main(config_path='config', config_name='config')
/Absolute/Path/To/miniconda3/envs/dlp/lib/python3.10/site-packages/hydra/_internal/hydra.py:119: UserWarning: Future Hydra versions will no longer change working directory at job runtime by default.
See https://hydra.cc/docs/1.2/upgrades/1.1_to_1.2/changes_to_job_working_dir/ for more information.
  ret = run_job(
[DATE TIME][lamorel_logger][INFO] - Init rl group for process 1
[DATE TIME][lamorel_logger][INFO] - Init llm group for process 1
[DATE TIME][lamorel_logger][INFO] - Init rl-llm group for process 1
[DATE TIME][lamorel_logger][INFO] - Using CPU on process 1 (index 0)
Parallelising HF LLM on 1 devices
Loading model t5-small
/Absolute/Path/To/miniconda3/envs/dlp/lib/python3.10/site-packages/huggingface_hub/file_download.py:1132: FutureWarning: `resume_download` is deprecated and will be removed in version 1.0.0. Downloads always resume when possible. If you want to force a new download, use `force_download=True`.
  warnings.warn(
Error executing job with overrides: ['rl_script_args.path=/Absolute/Path/To/Grounding_LLMs_with_online_RL/experiments/post-training_tests.py', 'lamorel_args.accelerate_args.machine_rank=1']
Traceback (most recent call last):
  File "/Absolute/Path/To/Grounding_LLMs_with_online_RL/experiments/post-training_tests.py", line 310, in main
    lm_server = Caller(config_args.lamorel_args, custom_updater=LoadSpecificWeightsUpdater,
  File "/Absolute/Path/To/Grounding_LLMs_with_online_RL/lamorel/lamorel/src/lamorel/caller.py", line 53, in __init__
    Server(
  File "/Absolute/Path/To/Grounding_LLMs_with_online_RL/lamorel/lamorel/src/lamorel/server/server.py", line 60, in __init__
    self._updater.set_llm_module(
TypeError: BaseUpdater.set_llm_module() missing 1 required positional argument: 'llm_module'

As running using the Lamorel Launcher is advised, I find it difficult to solve this error with my usual procedure involving a debugger.
If you were able to reproduce the issue and help me solve it, I would be very thankful. I find your paper very insightful.
Thank you for your time!

Hi,
Thanks for reaching out and spotting all these issues!
I opened a PR.
Importing
It should be fixed now.
Class Naming
It should also be fixed.
Training Using Lamorel (on one GPU)
I fixed lamorel's initialization in train_language_agent.py so that the client and server could be launched together (see here).
Regarding your results, you seem to have performed 1k steps only. I am not surprised the agent has not solved a single episode, given our results in the paper (Figure 14), where Flan-T5 small reached a 0.2 success rate after approximately 10k steps. Secondly, we used 32 environments in our experiments (leading to much more diversity in batches).
Testing Using Lamorel (on one GPU)
It should also be fixed.
I hope this helps :)


Add period to Plotly docstring

Adds a period to the plotly docstring. In the docs it ends up looking like a typo:

Looks like convention is to end each param definition without a period, though, so I haven't added periods where the description is only one sentence.
I have read the CLA Document and I hereby sign the CLA

recheck


[Bug]: can't conda install via environment.yml on mac

liionpack Version
n/a
Python Version
3.9
Describe the bug
Install command suggested in the readme fails
$ conda env create -f environment.yml
Collecting package metadata (repodata.json): done
Solving environment: failed

ResolvePackageNotFound:
  - redis

Steps to Reproduce
No response
Expected behaviour
Installs
Relevant log output
No response
Additional context
Mac OS on M1
Possibly related to https://stackoverflow.com/questions/58219956/how-to-fix-resolvepackagenotfound-error-when-creating-conda-environment

This may be an issue with installing on the new Apple Silicon M1 Mac. Is Anaconda/Conda supported on M1 Macs? I have an Intel MacBook Pro and I don't have any liionpack installation issues with conda install using the environment.yml file.

I was able to install without any problems by first creating an empty environment then pip installing into it. Have never tried doing it with an environment.yml though, maybe that isn't supported. Also I have to use miniforge, don't know if that makes a difference

Did you create the empty environment using conda or pip?

conda

Can you try to install liionpack using the pip venv approach on your M1 Mac and see if you have any issues?
# Clone the repository
$ git clone https://github.com/pybamm-team/liionpack.git

# Create a virtual environment in the repository directory
$ cd liionpack
$ python -m venv .venv

# Activate the virtual environment and upgrade pip if venv installed an old version
$ source .venv/bin/activate
$ pip install --upgrade pip

# Install the required packages
$ pip install -r requirements.txt

# Install the liionpack package from within the repository
$ pip install -e .

Yeah that works

So this looks like a conda issue with M1 Macs and not an issue with liionpack itself. Also, I don't see an Anaconda installer for M1 Macs (https://www.anaconda.com/products/individual). I see a Miniconda installer for the M1 Mac that was released in November for Python 3.8.

Ok, thanks for following up


fix: local policy ID

--> Change the local policyID

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "local-id",
    "component": "enforcerd",
    "pr-id": "1330",
    "commit-sha": "61d6a5fe5f6d302e60f4586588985c3c7fe3f83a"
  },
  {
    "project": "local-id",
    "component": "trireme-lib",
    "pr-id": "855",
    "commit-sha": "a045a194a465ed7e16cf821837e955e663d7dc13"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "local-id",
    "component": "enforcerd",
    "pr-id": "1330",
    "commit-sha": "61d6a5fe5f6d302e60f4586588985c3c7fe3f83a"
  },
  {
    "project": "local-id",
    "component": "trireme-lib",
    "pr-id": "855",
    "commit-sha": "a045a194a465ed7e16cf821837e955e663d7dc13"
  },
  {
    "project": "local-id",
    "component": "clad",
    "pr-id": "1238",
    "commit-sha": "44a11eb2d9054ac844d621cf66a8bc1a591f5aa1"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "local-id",
    "component": "trireme-lib",
    "pr-id": "855",
    "commit-sha": "a045a194a465ed7e16cf821837e955e663d7dc13"
  },
  {
    "project": "local-id",
    "component": "clad",
    "pr-id": "1238",
    "commit-sha": "44a11eb2d9054ac844d621cf66a8bc1a591f5aa1"
  },
  {
    "project": "local-id",
    "component": "enforcerd",
    "pr-id": "1330",
    "commit-sha": "8858526f58797c4d5ed727d1ae1aad3203472ada"
  }
]


Henry Ecker: Blacklist<PHONE_NUMBER>3

Henry Ecker requests the blacklist of the number<PHONE_NUMBER>3. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by tripleee in Charcoal HQ


Update the DrawTextOptions in the Docs

Heii,
I'm currently using your Libary and wanted to add a Watermark.
First, I think this wouldn't work because there is no opacity parameter, but then I saw the Release Notes for v1.8.0.
Can you add this to the Documentation?

Hello @Psycc!
Looks like this is already in the documentation:

See https://pdf-lib.js.org/docs/api/classes/pdfpage#drawtext and https://pdf-lib.js.org/docs/api/interfaces/pdfpagedrawtextoptions#optional-opacity for one example.
If you have specific suggestions for how to improve the docs, I'd be happy to accept a PR!


Value error appears in graph classification task with MUTAG

Hi, thank you for your code, I've learned a lot from it.
Now I met a problem("ValueError: row, column, and data arrays must be 1-D") when I'm running train_IGNN.py in your graphclassification folder. The problem occurs because data.edge_attr is not None and it's a N*4 tensor, so adj_sp cannot be constructed correctly. Could you please tell me how to fix this?
Thank you and look forward to your reply:)

Hi,
For MUTAG, the edge attribute is the corresponding edge label, which can be referred here. I quote it here for your reference:
Edge labels:
  0  aromatic
  1  single
  2  double
  3  triple

Thus you can direct take the indices of the N*4 tensor as the labels for edges, and use it to construct your adjacency:
edge_weight = data.edge_attr.argmax(1)
Actually you can also ignore the edge attribute by treating it as unweighted graphs, as the examples provided in pyg.
I hope this answers your question.

Thank you so much for your answer. It helps me a lot:) BTW, could you please tell me that do you use the edge labels or just ignore them in your node classification experiments?

No problem :) We just ignore them and use the graphs as unweighted ones in experiments.


General: Implement switching Courses from a Course Icon in a Course

Checklist
General

[ ] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[ ] I chose a title conforming to the naming conventions for pull requests.

Client

[ ] I strictly followed the client coding and design guidelines.
[ ] I documented the TypeScript code using JSDoc style.

Motivation and Context
Many Applications like Slack allow users to navigate between areas just by clicking on the icon. This small feature is a nice little bonus for experienced users to navigate easily between recently accessed courses.
Description
Added the logic and dropdown box to a course icon after navigating to one course. The 3 recently accessed courses will be displayed or if a user is only enrolled in 1 or 2 courses, then this will be displayed.
Before:

After:

Steps for Testing

Navigate to a Course: Ensure you see a arrow under the Course Icon
Click on it and try to navigate to another Course: Check if the recently visited Courses are being listed (you might need to navigate first to 2 or 3 Courses)

Testserver States

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

Review Progress

Code Review

[x] Code Review 1
[x] Code Review 2

Manual Tests

[x] Test 1
[x] Test 2

Thanks @iyannsch 👍  and thank you for the review

https://github.com/ls1intum/Artemis/pull/8669#pullrequestreview-2079778040
The issue seems to be that when shrinking the width of the screen, the collapse function is not called.
It will work properly if you collapse the sidebar first and then shrink the page.

@muradium i fixed the issue 👍

The current selected course is also one of the recent courses right?
therefore the current selected course is displayed twice. I would propose to exclude the current selected course from the list

i would propose to get rid of the rounded corners on the top of the list so it looks more consistent with the current selected course.
additionally, let it fill the whole sidebar for the same reason.

@janthoXO I adjusted the box now to look like the other dropdown box on the same page :)

Thanks for your efforts and time, the code looks a lot more sophisticated (CourseAccessStorageService, ..) now and should be good to go. Also the new way of handling clicks on the dropdown is well thought of and takes advantage of Angular :)
Regarding the UI, however, I have to say I really don´t like the current way of accessing the dropdown! I was testing the feature and tried to find it but was not able to find the dropdown for a whole 3 minutes because it is not indicated by the "dropdown" arrow or anything else.. This fact together with the small action surface (only icon) makes this feature very unlikely to be ever discovered leave alone used :/
Please consider changing the UI to actually show the functionality - otherwise it might actually be useful not to merge this feature at all as it greatly increases the complexity of the course-overview 👀

I agree with Yanniks comment about the UI, I also find it very difficult to find  -  so maybe there is a better option to display it more intuitively :-)

Thanks for your efforts and time, the code looks a lot more sophisticated (CourseAccessStorageService, ..) now and should be good to go. Also the new way of handling clicks on the dropdown is well thought of and takes advantage of Angular :)
Regarding the UI, however, I have to say I really don´t like the current way of accessing the dropdown! I was testing the feature and tried to find it but was not able to find the dropdown for a whole 3 minutes because it is not indicated by the "dropdown" arrow or anything else.. This fact together with the small action surface (only icon) makes this feature very unlikely to be ever discovered leave alone used :/
Please consider changing the UI to actually show the functionality - otherwise it might actually be useful not to merge this feature at all as it greatly increases the complexity of the course-overview 👀

I agree with Yanniks comment about the UI, I also find it very difficult to find - so maybe there is a better option to display it more intuitively :-)

@iyannsch and @marlon-luca-bu, that's intended. This feature should not be to prominent to maintain a clear design. The idea of this feature is to be a shortcut — hidden from novice users — may speed up the interaction for the expert user (see Norman's Heuristics).

@az108 one client test consistently fails, please have a look and fix it

i fixed it 👍 @krusche

Thanks a lot for the suggestion @florian-glombik :)


[BUG] Discord limiting API response, causing messages to delete extremely slowly.

WARNING: Github Issues are publicly visible, do not post sensitive information here.
You may want to use the "Screenshot mode" before taking screenshots or copying logs.

Describe the bug
A clear and concise description of what the bug is.
Discord may be limiting the rate at what you may be able to delete at sadly, unless it's just a new error within the code, you should look into this.
To Reproduce
Steps to reproduce the behavior:

Go to a channel.
Click on everything to run the script
Scroll down the log, after a few minutes you'll see the a rate limit between 4000-6000, which impacts deletion speed.
See error

Screenshots / Logs
If applicable, add screenshots and or copy log to help explain your problem.

now the dates are messing up to 2085 and 2017 despite no messages being in 2019 or lower, only 2020.
Paste logs here
You're being rate limited to 5643ms.

Affects (Add an X to items that apply)

[x] DMs
[x] DM Groups
[x] Channel
[x] Channel (NSFW)
[x] Server (All Channels)
[x] Other (explain)

System:

OS: [CrOS]
Browser [newest Chrome version]

Yeah, I can confirm this bug. If one allows DDM to delete messages as fast as possible, it'll continously hit the rate limits and reach a delay somewhere in the 4-12 seconds range which is never lowered again. After a bit of testing, I've found that manually adjusting the delay to around 1500ms (1.5s) allows the script to work through thousands of messages without hitting Discord's rate limits.
@victornpb I think a potential solution would be to add a new form field where users can enter a custom delay to deal with server-side rate limit changes like this.

yeah probably, you can also set it to 600ms to get a few quick messages in it before the API limit gets automatically set to 1.5 or 1.6. And turning it off, refreshing or moving channels, and turning it back on brings it back up to a bit quicker to help it, sadly moving through 20k messages takes like 6 hours now. You could probably find a loophole to the API limit as well, though I haven't a clue what that would be. Maybe send an email to Discord support saying that deletion is too slow manually, and maybe they'll boost it back to what it was slightly, no clue. Your reply did help me though, whalehub! Thank you, or otherwise I wouldn't have thought of editing the code myself lol.

@victornpb please add a new form field for a custom delay.

I've been having the same issue as well. It's strange because it wasn't always the case.
I'm on Firefox 80.0.1 (x64) and my Discord version appears to be the one from August 10th (according to the Change Log).
I followed Whalehub's advice and changed the deleteDelay to about 1000ms and searchDelay to 500 (not sure if the latter is necessary). It's much slower now but at least it deletes at a constant pace without errors and having to stop/start the script several times.

After a bit of testing, I've found that manually adjusting the delay to around 1000ms (1s) allows the script to work through thousands of messages without hitting Discord's rate limits.

How can i add this? Unfortunately I am not very familiar with scripts.

@6kbyte
Open Discord on a webpage (the desktop app won't do). The tampermonkey extension icon will show up with a red 1, click on it and then right-click on the Undiscord script. This will open up a page with a bunch of parameter lines. Search for deleteDelay and change the value from 100 to 1000.
Click on File in the top left corner of the script and Save. Close the script page and close the Discord tab (for it to refresh and accept the new values).

@IvoryOwl92 thank you for your answer. I changed the value to 1000 but after that a saw a strange behavior: the script delete 10-20 messages, stops and automatically increase the waiting time... deletes 10-20 messages again, stops and automatically increase the waiting time... and so on. Result is that after a while you have to wait quite a long time until further deleting is done. Did i do something wrong?

@6kbyte If you're still hitting the rate limits with a value of 1000, you need to increase it a bit. Try it with a value of 1250 or 1500.
Discord changes the rate limits on their API from time to time, so a value that worked one day can stop working the next.

@6kbyte
That's the exact same issue we've been having and why this thread was made. Whalehub's advice was supposed to be a workaround. Why do you say it's strange behavior now? Wasn't that the reason that lead you here, wanting to change the script in the first place? In any case, on top of the previous advice, I also changed the searchDelay to 300. (I said 500 in my earlier post but 300 seems to be working as well.) Try doing that as well? Don't forget to save the script and close your web browser (to refresh). Then try mass deleting messages again.
If that doesn't work, then I'm out of ideas.

Currently there is only a mechanism to increase the delay once you get throttled, but it doesn't go back down.
I think I could implement some kind of decay, so it stays always at the optimum speed instead of just relying on a high delay number.
But testing will be required.
Ill try to get back to this later, I only have time to work this on my spare time on weekends.

i have been having this exact issue for a few weeks now. I've tried adjusting the delay to even 2000ms but i still get the same delay of around 70000ms or something similar after about 1 page of messages

@Broine1 Are you deleting your own messages or other people's messages as a mod in a server? The latter requires a delay of around 5000ms to work continuously because the rate limits are much stricter.

ah i see, yes i am deleting others messages (my own messages with an alt just in case) i will try using my own account

Same issue for me as well, using the older script method.
I will gladly help with testing.

Currently there is only a mechanism to increase the delay once you get throttled, but it doesn't go back down.
I think I could implement some kind of decay, so it stays always at the optimum speed instead of just relying on a high delay number.
But testing will be required.
Ill try to get back to this later, I only have time to work this on my spare time on weekends.

Any update regarding this issue? :)

Something causes the delay to lower itself from the default delay sometimes, for some reason. I've set my delay to 1000 to avoid rate limiting, but after a while it just starts deleting much quicker than the 1000 delay implies, and quickly gets stuck on the rate limiter which eventually rises to 4000+

You can't really see it in the screenshot, but I'm watching it go and for no reason it just suddenly ignores my 1000ms delay and starts deleting messages really fast when it shouldn't. It doesn't happen all the time, so I'm not sure it could even be reproduced reliably, and so far it's only happened when I've had has: link, has: file and include pinned checked.

Something causes the delay to lower itself from the default delay sometimes, for some reason.

Yeah this is an oversight when I accepted the PR that allowed the user to enter manually a delay. it didn't take into consideration that the code previously tried to auto adjust the delay based on the rate limit responses. So this part of the code should have been modified as well. Ty for reporting that screenshot

I think it's very likely that the constant rate of deletion is now being used as a factor used by Discord when choosing to limit user account's access to the API. I hacked together a quick and dirty fix, you can read about it / set it up here.
https://github.com/victornpb/deleteDiscordMessages/issues/155#issuecomment-757382923

I think it's very likely that the constant rate of deletion is now being used as a factor used by Discord when choosing to limit user account's access to the API. I hacked together a quick and dirty fix, you can read about it / set it up here.
https://github.com/victornpb/deleteDiscordMessages/issues/155#issuecomment-757382923

@victornpb please add this :) ^^^

@victornpb please add this :) ^^^

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

I'm currently dealing with a personal matter, so I can't test anything atm.
But I can take a look at your PR, and merge it if anyone can confirm it is working

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

I'm currently dealing with a personal matter, so I can't test anything atm.
But I can take a look at your PR, and merge it if anyone can confirm it is working

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

I'm currently dealing with a personal matter, so I can't test anything atm.
But I can take a look at your PR, and merge it if anyone can confirm it is working

There is no PR as of this moment because my hacked together Gist was written just for me and is

based on the less supported of the two codebases
uses defaults that may be way too high for some people

I'm going to start a new issue and see if I can't get any volunteers to try out the changes / see how widespread the new rate limiting is before I put in the effort to write good public facing code that plays nice with the existing codebase.

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

I'm currently dealing with a personal matter, so I can't test anything atm.
But I can take a look at your PR, and merge it if anyone can confirm it is working

There is no PR as of this moment because my hacked together Gist was written just for me and is

based on the less supported of the two codebases
uses defaults that may be way too high for some people

I'm going to start a new issue and see if I can't get any volunteers to try out the changes / see how widespread the new rate limiting is before I put in the effort to write good public facing code that plays nice with the existing codebase.

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

Yes, yours fixed my issues completely.

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

Yes, yours fixed my issues completely.

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

Yes, yours fixed my issues completely.

Awesome, I'll get started on a Pull Request this weekend

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

Yes, yours fixed my issues completely.

Awesome, I'll get started on a Pull Request this weekend

@victornpb please add this :) ^^^

Were you having issues before and did my gist fix it? I can submit my own pull request once I confirm that it's not just a placebo.

Yes, yours fixed my issues completely.

Awesome, I'll get started on a Pull Request this weekend

I wonder if the Quick and Dirty Fix previously mentioned turned out to be functional in the testing.
Did the PR turn out to create issues when implementing?


Added QR Code Generator Project

Type of change

[x] Added new project
[ ] Bug fix
[ ] New features
[ ] Enhanced documentation

Changes proposed in this pull request

Project name : QR Code Generator

Short description :  this is small python program to generate QR Code

Checklist

[x] The code is Python 3 compatible
[x] I have added relevant comments in my file to make code understandable
[x] I have added my project in a folder which includes all the necessary files

Looks good! Thank you. 🚀


Android Nougat - No information about rounded icons

Android Nougat introduced rounded launcher icons - I think this should be mentioned in this article.

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

ID: c7bf660f-3bbe-07d2-c24d-a67cdd45ee2b
Version Independent ID: ada428cb-4d42-af70-1cdd-cb2f8f04db9c
Content: Nougat Features - Xamarin
Content Source: docs/android/platform/nougat.md
Product: xamarin
GitHub Login: @conceptdev
Microsoft Alias: crdun

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


A tuple of uint64 (> 263 but < 264) as the key of numba Dict

Hello,
I'm trying to make a numba dict of type {(uint64, uint64}: int64}.
If I don't understand wrong, the element of the key can be within [0, 2 ** 64-1]. However, I get an OverflowError: int too big to convert, when the element is larger than  2**63-1.
Code to reproduce:
import numba as nb
key_type = nb.types.Tuple((nb.uint64, nb.uint64))
di = nb.typed.Dict.empty(key_type=key_type, value_type=nb.int64)
di[(nb.uint64(2**63), nb.uint64(0))] = 1
print(di)

Thanks for the report. This doesn't seem to reproduce on linux systems, was wondering if you are perhaps using Windows? The exception is probably coming from Numba's "unbox/box" code which is the part of Numba that handles the translation from Python objects to Numba's internal representation (or the reverse), I suspect a Python call to create an intermediate representation of the value is failing.

Could maybe be related (or possibly not, but worth a mention IMHO): #9576

Hello, thanks for the answer.
Sorry, I also cannot directly reproduce this error.
After retrying it a few times, the bug appear again.

Here is the code to reproduce:
import numba as nb
key_type = nb.types.Tuple((nb.uint64, nb.uint64))
di = nb.typed.Dict.empty(key_type=key_type, value_type=nb.int64)
di[(nb.uint64(2**63-1), nb.uint64(2**63+1))] = 1
print(di)

key_type = nb.types.Tuple((nb.uint64, nb.uint64))
di = nb.typed.Dict.empty(key_type=key_type, value_type=nb.int64)
di[(nb.uint64(2**63+1), nb.uint64(2**63+1))] = 1
print(di)

I'm using
macOS (14.4.1)
Python 3.12.3
numba (0.59.1)


[UI] Answers for Monitoring Disappear After Save (will return if UI is hard refreshed)

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Expand advanced options for cluster monitoring
Add an additional answer
Click Save

Result:
Answers disappear.
Other details that may be helpful:
If you do a full browser refresh the answers come back, but until the UI is forced to reload they will remain gone.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.3.0-rc2
Installation option (single install/HA): HA

Version - master 9/11
Checked and this is reproducible.  All Answers disappear on Save and don't come back until you refresh the browser.

@tfiduccia I see the answers persist after clicking save. But in the following scenario I'm left without user entered answers wiped:

Enter custom answers answers and save. UI will refresh and the answers persist in Advanced Options.  ✅
Stay in the same page do not refresh Disable monitoring. I still see the answers in the UI.
Do not refresh or navigate somewhere else. Click Enable
The user entered answers are not there anymore. ❌


Fix duplicate definition when using aws-sdk-cpp.

re: Issue https://github.com/Unidata/netcdf-c/issues/2927
The NC_s3sdkinitialize and NC_s3sdkfinalize functions were misplaced. They should have been moved from ds3util.c to ncs3sdk_h5.c.  When using ncs3sdl_aws.cpp, this resulted in a duplicate definition.
Other changes.

Factored out the common code of the ncs3sdk_xxx files.
Fixed a memory leak in the NCZarr S3 code.
Suppress a google storage test when using aws-sdk-cpp because google storage is not supported.

@czender Did this work for you? I'm trying to get v4.9.3 out asap so will test it myself shortly, but curious if it worked for you.

Yes, I should have let you know earlier that I tested it immediately and it worked fine for me.


Added documentation for translation support

please use proper commit messages (yarn commit or yarn commit:signed)

Can't do that when editing directly on github.

@BronzW rename the commit with the proper formatting. Just found about that today


Updated with new launchers

Updated the launcher binaries with changes from:
https://github.com/shartte/TerasologyJavaLauncher/commit/3667e739877b9295046b2c4a757bbf443df1affb

Man seeing that kind of a commit in C++ reminds me how happy I am to have gone Java, hehe. And glad you're handling it here!
I should be able to test this later today (along with the Index PR), will try my tablet for some Win8 flavor, less Java to uninstall there.

Interestingly the x64 works. But the x86 crashes immediately for me? I get the "Terasology.x86.exe has stopped working" popup. Not sure if it helps but here are the details:
Problem signature:
  Problem Event Name:	BEX
  Application Name:	Terasology.x86.exe
  Application Version:	<IP_ADDRESS>
  Application Timestamp:	54df506a
  Fault Module Name:	StackHash_0a9e
  Fault Module Version:	<IP_ADDRESS>
  Fault Module Timestamp:	00000000
  Exception Offset:	00000104
  Exception Code:	c0000005
  Exception Data:	00000008
  OS Version:	6.1.76<IP_ADDRESS>.256.1
  Locale ID:	1033
  Additional Information 1:	0a9e
  Additional Information 2:	0a9e372d3b4ad19135b953a78882e789
  Additional Information 3:	0a9e
  Additional Information 4:	0a9e372d3b4ad19135b953a78882e789

Worked fine previously and again if i revert to the prior x86

I think i fixed the issue (I ran into it myself). It was related to not specifying a calling convention.
Can you check again?


Issues with embedded videos when working with tabbed app.

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Current behavior:
When embedding a video say from YouTube, if you have the video playing in a tab and switched to another tab, the video keeps playing in the background. Also, when you play the video in full screen then click on the screen where the tabs buttons are (even through they are hidden) the app will switch to that tab.
Expected behavior:

The video should stop playing when switching to another tab.
When a video is playing in full screen, user interaction should be possible for the video not elements which are hidden. This only happen when running in mobile as full screen on desktop doesn't hide the tabs buttons.

Steps to reproduce:

Create a tabbed app, add the following to one of the tabs ion-content:
<iframe src="https://www.youtube.com/embed/O2WiI9QrS5s" frameborder="0" allowfullscreen></iframe>
Test

Other information:
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.5
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.0.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.9.4
Xcode version: Not installed

The video should stop playing when switching to another tab.
When a video is playing in full screen, user interaction should be possible for the video not elements which are hidden. This only happen when running in mobile as full screen on desktop doesn't hide the tabs buttons.

the video playing as the tabs are switched makes sense, since the DOM isn't getting removed, but just hidden. This would happen even if you end up just pushing a new page and not just switching tabs.

I think a decent solution would be to create a play were you can control the videos play/pause functionality. As we cannot expect every video to be the same, I think it's ok for the dev to do a bit of work and hook into the view events to control the video.

Not following what you are saying. Are you talking about how the tabs are still visible when the app is fullscreen?

For number 2, the tabs are invisible on my phone when I'm playing the video on full screen, however, they are clickable. So, when I click on the area where the about button is, it will go to the about page. even thought the tab buttons are invisible.
Not sure if this is a bug or not though.

Android or ios?

Android 7.1.1

Interesting. Ok, this second seems to be a bug.
Would you mind updating the title

Closing and merging with https://github.com/driftyco/ionic/issues/10132


Overworld camera should function properly during cutscenes

Currently the overworld camera is designed for 1-on-1 conversations. It uses a fixed zoom level (2.0x) and zooms out if the player is too far from the center.
This logic is unsuitable for cutscenes; the result is that when the player isn't in the center of the cutscene, the camera zooms out onto them so they can run around.
The cutscene camera should cover all of the active chat participants. The most robust way to fix this is have something like an "active camera zone" which encompasses all chat participants with a little buffer. The zoom factor is based on this calculated zone. If the player gets more than a few steps out of this zone, the camera should zooms out if and only if a cutscene is not playing.

-cutscene camera position
 -camera should zoom dynamically based on bounding box size
  -all characters are in frame
  -about 30% empty space around them
 -camera should only unzoom if the player leaes the bounding box by a certain amount
  -something like, bounding box + 15%?
 -camera should be lower; dialog shouldn't block characters
 -'zoom out logic' should be disabled for cutscenes
 -bug: sensei is excluded from camera calculations?

-cutscene camera position
 -camera should zoom dynamically based on bounding box size
  -all characters are in frame
  -about 30% empty space around them
 -camera should only unzoom if the player leaes the bounding box by a certain amount
  -something like, bounding box + 15%?
 -camera should be lower; dialog shouldn't block characters
 -'zoom out logic' should be disabled for cutscenes
 -bug: sensei is excluded from camera calculations?


Map Information in CARLA

Is there a method to access the map information such as static vehicles present in the map, lane information for all the lanes present in front of the ego vehicle, through PythonAPI's ?
Or is there there any other API for this ?

Hi,
We are currently developing an API for this, it's supposed to come out in the following releases. We'll keep you updated, in the meantime you can try to obtain this information yourself by using the current tools!

Thank you for your reply.
Could you specify the tools that are currently available for obtaining the map information ?
One way is to use Unreal editor to get info on static meshes present.
Another way is to use the semantic segmentation images to identify various objects present in the images.
Is there any other way ?


configure_instances playbook on GCE fails

following (ad litteram) the example at gce1.md, the configure instances playbook fails
TASK [Gathering Facts] ***********************************************************************************************************************************************************
fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh<EMAIL_ADDRESS>Permission denied (publickey,gssapi-keyex,gssapi-with-mic).\r\n", "unreachable": true}
fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh<EMAIL_ADDRESS>Permission denied (publickey,gssapi-keyex,gssapi-with-mic).\r\n", "unreachable": true}
fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh<EMAIL_ADDRESS>Permission denied (publickey,gssapi-keyex,gssapi-with-mic).\r\n", "unreachable": true}
fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh<EMAIL_ADDRESS>Permission denied (publickey,gssapi-keyex,gssapi-with-mic).\r\n", "unreachable": true}
fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh<EMAIL_ADDRESS>Permission denied (publickey,gssapi-keyex,gssapi-with-mic).\r\n", "unreachable": true}
fatal: [<IP_ADDRESS>]: UNREACHABLE! => {"changed": false, "msg": "Failed to connect to the host via ssh<EMAIL_ADDRESS>Permission denied (publickey,gssapi-keyex,gssapi-with-mic).\r\n", "unreachable": true}

provider_config:
  gce:
    service_account_email: [omitted]
    credentials_file: [omitted]/projects/contrail-ansible-deployer/vigilant-design-134823-6eb1d1d5e0eb.json
    project_id: [omitted]
    ssh_user: centos
    machine_type: n1-standard-4
    image: centos-7
    network: microservice-vn
    subnetwork: microservice-sn
    zone: us-west1-a
    disk_size: 50
instances:
  master1:
    provider: gce
    roles:
      config_database:
      config:
      control:
      analytics_database:
      analytics:
      webui:
      k8s_master:
  master2:
    provider: gce
    roles:
      config_database:
      config:
      control:
      analytics_database:
      analytics:
      webui:
      k8s_master:
  master3:
    provider: gce
    roles:
      config_database:
      config:
      control:
      analytics_database:
      analytics:
      webui:
      k8s_master:
  node1:
    provider: gce
    roles:
      vrouter:
      k8s_node:
  node2:
    provider: gce
    roles:
      vrouter:
      k8s_node:
  node3:
    provider: gce
    roles:
      vrouter:
      k8s_node:

BTW, ssh with private key works fine!


Yarn install fails on the specified anchor-cli version in package.json

error An unexpected error occurred: "https://github.com/kklas/anchor-nightly/releases/download/anchor-cli-latest/anchor-cli-0.25.0-7eb8ca8.tgz: Request failed "404 Not Found"".

anchor-cli is a dev dependency. Are you trying to build anchor-client-gen from this repo or just using it as a dependency in you project? If the former, then update to the master branch.


sequenceS, but for single properties

🚀 Feature request
Current Behavior
None
Desired Behavior
I don't have the terminology, I'll try to be as clear as possible
I find myself with an object with a monad inside it, I need to expand the effect of such monad to the whole monad
Let's call it sequenceKey
const obj = {
      a: O.some(1),
      b: O.none,
      c: true,
    }

const res1 = sequenceKey("a", O.Applicative)(obj) // O.some({ a: 1, b: O.none, c: true })
const res2 = sequenceKey("b", O.Applicative)(obj) // O.none

Suggested Solution
Basically something like this
export function sequenceKey<P extends string, F extends URIS4>(
  prop: P, F: Apply4<F>
): <S, R, E, NER extends Record<P, Kind4<F, S, R, E, any>>>(
  r: NER & Record<P, Kind4<F, S, R, E, any>>
) => Kind4<F, S, R, E, { [K in keyof NER]: K extends P ? ([NER[K]] extends [Kind4<F, any, any, any, infer A>] ? A : never) : NER[K] }>
export function sequenceKey<P extends string, F extends URIS3>(
  prop: P, F: Apply3<F>
): <R, E, NER extends Record<P, Kind3<F, R, E, any>>>(
  r: NER & Record<P, Kind3<F, R, E, any>>
) => Kind3<F, R, E, { [K in keyof NER]: K extends P ? ([NER[K]] extends [Kind3<F, any, any, infer A>] ? A : never) : NER[K] }>
export function sequenceKey<P extends string, F extends URIS3, E>(
  prop: P, F: Apply3C<F, E>
): <R, NER extends Record<P, Kind3<F, R, E, any>>>(
  r: NER & Record<P, Kind3<F, R, E, any>>
) => Kind3<F, R, E, { [K in keyof NER]: K extends P ? ([NER[K]] extends [Kind3<F, any, any, infer A>] ? A : never) : NER[K] }>
export function sequenceKey<P extends string, F extends URIS2>(
  prop: P, F: Apply2<F>
): <E, NER extends Record<P, Kind2<F, E, any>>>(
  r: NER & Record<P, Kind2<F, E, any>>
) => Kind2<F, E, { [K in keyof NER]: K extends P ? ([NER[K]] extends [Kind2<F, any, infer A>] ? A : never) : NER[K] }>
export function sequenceKey<P extends string, F extends URIS2, E>(
  prop: P, F: Apply2C<F, E>
): <NER extends Record<P, Kind2<F, E, any>>>(
  r: NER
) => Kind2<F, E, { [K in keyof NER]: K extends P ? ([NER[K]] extends [Kind2<F, any, infer A>] ? A : never) : NER[K] }>
export function sequenceKey<P extends string, F extends URIS>(
  prop: P, F: Apply1<F>
): <NER extends Record<P, Kind<F, any>>>(
  r: NER
) => Kind<F, { [K in keyof NER]: K extends P ? ([NER[K]] extends [Kind<F, infer A>] ? A : never) : NER[K] }>
export function sequenceKey<P extends string, F>(
  prop: P, F: Apply<F>
): <NER extends Record<P, HKT<F, any>>>(
  r: NER
) => HKT<F, { [K in keyof NER]: K extends P ? ([NER[K]] extends [HKT<F, infer A>] ? A : never) : NER[K] }>

export function sequenceKey<P extends string, F>(prop: P, F: Apply<F>) {
  return (obj: Record<P, HKT<F, any>>) => F.map(obj[prop], (value) => ({ ...obj, [prop]: value }))
}

And the potential traverseKey, for which I still have to find the type
There is also a fork https://github.com/DrTtnk/fp-ts
Who does this impact? Who is this for?
From beginners and above, every time there are consideration to make for a whole object, given a single property
Describe alternatives you've considered
Using boilerplate code with pipe, map, object destructuring... That has to be reimplemented in case there were other monadic types to manage
const obj = {
  a: O.some(1),
  b: "a",
  c: O.none
}

const result = pipe(
  obj,
  ({a, ...base}) => pipe(
    a,
    O.map((a) => ({...base, a}))
  )
)

Your environment

Software
Version(s)

fp-ts
2.21.1

TypeScript
4.7.4

The code you have posted implies that there is some computation down the road where b and c depend on a. I
think the do notation using apS and bindS works well for such cases and clearly shows the intent.

I wish I could do something like this:
const test = {
  a: 'a',
  b: 'b',
  c: O.some('c'),
}

const res = pipe(
  O.some(test),
  O.bind('c', (c) => O.some(c)),
  //     ^^^ TS2345: Argument of type 'string' is not assignable to parameter of type 'never'.
)

Unfortunately this is a typeerror

In that case, what about
‘
sequenceS(O.Applicative)({a:O.some(obj.a), b:O.some(obj.b),c})
’
Ultimately, that data structure has to go through some sort of transformation like that since as it stands(and I assume thats due to upstream api or whatever) it is sort of smelly. I think values in a product type should not have dependency on each other like this.


plugin-emotion still does not work uniformly

Describe the bug
This isn't a new issue. It's still https://github.com/swc-project/swc/issues/5453
Best guess is fix was done assuming there's a node_modules folder and anytime there isn't one the plugin doesn't work.
failed to get the node_modules path

Input code
No response
Config
No response
Playground link (or link to the minimal reproduction)
https://github.com/pietrop/cra-swc-emotion-plugin-example
SWC Info output
Operating System:
Platform: darwin
Arch: arm64
Machine Type: arm64
Version: Darwin Kernel Version 23.2.0: Wed Nov 15 21:54:05 PST 2023; root:xnu-10002.61.3~2/RELEASE_ARM64_T6031
CPU: (16 cores)
Models: Apple M3 Max
Binaries:
    Node: 18.19.0
    npm: 10.2.3
    Yarn: 3.7.0
    pnpm: N/A

Relevant Packages:
    @swc/core: N/A
    @swc/helpers: N/A
    @swc/types: N/A
    typescript: 4.4.4

SWC Config:
    output: N/A
    .swcrc path: N/A

Next.js info:
    output: N/A

Expected behavior
plugin-emotion should work as documented
Actual behavior
No response
Version
<EMAIL_ADDRESS>Additional context
While I added the swc info output, it's not really accurate since we are using .swcrc from within scoped packages for the company I work at since there's an internal tool that needs to share configs across many repos. There are .swcrc but only 3 flavors at the moment, CJS, ES, and TS. This is all to work around the fact that to replace babel-cli I have to use swc-cli but unlike babel having babel.config.js we have to create multiple rc files (and set them json and use require.resolve on those).
Tested on:
node:
v14
v16
v18
yarn:
3.2.0-rc.2
3.4.0
3.7.0

Ok I have more info to share as well as something I did to work around. I'll try to share as much as I can.
From what I can glean, the problems are related to how lifecycle scripts are being used in rust based projects like swc and yarn and pnpm. Because postinstall scripts allow downloading of binaries and those are being referenced plugins need to be listed in certain ways when we need to share references through other nodejs modules. This can also happen locally with different workspace setups but the gist of it is that there can't be a reliance on a reference just being @swc/plugin-emotion for example if it's not actually directly installed into that project.
To make things work properly I created a new package to house just the swc configs, added preferUnplugged: true in package.json for it to account for projects using newer versions of yarn along with a postinstall lifecycle script that creates a dynamic .swcrc per type of config that could be used.
The crux of the issue was that the plugin config needed to be generated as such:
const swcEmotionPluginPath = require.resolve('@swc/plugin-emotion');

...
experimental: {
      plugins: [
        [
          swcEmotionPluginPath,
          {
            sourceMap: true,
            autoLabel: 'dev-only',
            labelFormat: '[local]',
          },
        ],
      ],
    },
...

to look like this (example is with<EMAIL_ADDRESS>and 2.5.110 of @swc/plugin-emotion)
...
"experimental": {
      "plugins": [
        [
          "<ABSOLUTE_PATH_TO_PROJECT>/.yarn/unplugged/<EMAIL_ADDRESS>          {
            "sourceMap": true,
            "autoLabel": "dev-only",
            "labelFormat": "[local]"
          }
        ]
      ]
    }
...

Also a note that I could not verify this worked in other repos using linking either because lifecycle scripts don't work in linked projects for yarn. Here's the relevant output from running with RUST_BACKTRACE=1:

You may need to update @swc/plugin-emotion. See https://github.com/swc-project/plugins/pull/254

Oh ... You also need to update @swc/core because of AST-breaking changes

Your config was wrong. See https://github.com/kdy1/repro-swc-8529


Class auth does not exists

HI,
I installed your package, added the service provider and when i reload my webpage i get the following error:
ReflectionException (-1)
Class auth does not exist

local, sorry


Remove wlr_backend.events.{output_remove,device_remove}

Fixes #605

I'll do that include style change in another PR, since it's somewhat unrelated to this one.

LGTM.
The one thing I am wondering about is the performance inpact.
Do we want to replace every emit? Or should e.g. cursor movement emits be on the old style?
It would imply other semantics on what the handler is allowed to do, but if the signal usually emits to a low number of listeners very often (cursor movements, graphic tablets pen movement) this may be interesting from a performance perspective.

I don't think this presents significantly different performance compared to wl_signal_emit. They're almost the same process.

Yeah, the only difference is that it's updating a few pointers. Also libwayland's private, internal wl_priv_signal_emit has similar overhead and is used everywhere in libwayland's code.


Export trace key in logging-{winston,bunyan}

This makes the LOGGING_TRACE_KEY constant exported by the logging-{winston,bunyan} modules per conversation here: https://github.com/GoogleCloudPlatform/google-cloud-node/pull/2574#discussion_r136679515

Coverage remained the same at 100.0% when pulling 871ebd52e72772225330f4d4641bcf1365d11303 on draffensperger:export-logging-key into 5c7035b2cfedd1369d32727e57b8a6c3c297f3e7 on GoogleCloudPlatform:master.

Coverage remained the same at 100.0% when pulling 917f65f292675f36da0fb2bbc890d569650ef287 on draffensperger:export-logging-key into 5c7035b2cfedd1369d32727e57b8a6c3c297f3e7 on GoogleCloudPlatform:master.

Is this good to merge? @stephenplusplus, did you want to take a look?

Sorry I didn't catch this one. LGTM!


Android Basics: Viewmodel and State in Compose

ViewModel and State in Compose codelab, Slide 6

Even after bringing the following dependency in:
implementation 'androidx.lifecycle:lifecycle-viewmodel-compose:2.5.1'
Android studio cannot resolve the following import:
import androidx.lifecycle.viewmodel.compose.viewModel
unresolved reference: compose

Nevermind, after more investigation this is an issue on my end.


Fixed memory leak

Fixes #198.

test me

Build Passed :ok_hand:
You can safely merge 'hotfix-image-downscale-crash' into 'master'.

test me

Build Passed :ok_hand:
You can safely merge 'hotfix-image-downscale-crash' into 'master'.

👍

Thank you for the tests!


Lumira Designer 2 SP02 (Flow Container)

Hi Mike,
I checked the function surrounding the Flow Container.
Unfortunatly the main functions are broken:

registerParentComponent(component)
registerComponent(component)
unregisterComponent(component)

I guess classes have been changed / reorganized for Basis Containers (like PANEL).
Probably the similar issue with the new function PANEL.moveComponent(component) where I can not assign a panel. -> A PANEL does not belong to classtype component.
This combination works:
FLOW_LAYOUTER.registerComponent(CROSSTAB);
This combination does not work:
FLOW_LAYOUTER.registerComponent(PANEL);
Regards.
Nico

Hey Nico,
Thanks for reporting this one.  I'll take a look.

Hey Mike,
i created a simple example.
In the start routine (On StartUp) I added the code.
I saw additionally tha the rendering mode is not working.
Well, this component is quite similar to the new Adaptive Layout.
But the functionality .registerCompent / . unregisterCompent keeps a small dynamic advantage.
Regards.
Nico
SAMPLE.zip

Hey Nico,
I'm getting back to this one.  Looking now.

Hallo Mike,
great. That fits to my schedule right now. Will have time to check it on
time.
Regards
Nico
Am 13.10.2017 7:23 nachm. schrieb "Mike Howles"<EMAIL_ADDRESS>
Hey Nico,
I'm getting back to this one. Looking now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/org-scn-design-studio-community/lumiradesignercommunityext/issues/3#issuecomment-336515371,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWUwiaGRrF8Y6l_0mUTMCfKwP4yZp2dvks5sr5yAgaJpZM4PPUiT
.

Ok, I've updated this component with some new methods.  Basically it works the same as before, except if you need to register a Panel, you will need to call registerPanel and unregisterPanel.  Also there is a new registerParentPanel for the same reason.  The reason I had to make new methods, is like you said, the class types must have changed, and unlike real Java, in BIAL, we cannot have methods by the same name with different parameter signatures, unfortunately.  So to keep it simple and fast, this was quickest/easiest fix.
I've tested with a simple app with some panels and buttons registered and it seems to work as intended now.
Let me know if it works for you!  Thanks, sorry for the delay, I'm the only guy really working on this stuff any more.

And you're the guru as well !
We can't possibly compete with you ;)
On Fri, Oct 13, 2017 at 4:28 PM, Mike Howles<EMAIL_ADDRESS>wrote:

Ok, I've updated this component with some new methods. Basically it works
the same as before, except if you need to register a Panel, you will need
to call registerPanel and unregisterPanel. Also there is a new
registerParentPanel for the same reason. The reason I had to make new
methods, is like you said, the class types must have changed, and unlike
real Java, in BIAL, we cannot have methods by the same name with different
parameter signatures, unfortunately. So to keep it simple and fast, this
was quickest/easiest fix.
I've tested with a simple app with some panels and buttons registered and
it seems to work as intended now.
Let me know if it works for you! Thanks, sorry for the delay, I'm the only
guy really working on this stuff any more.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/org-scn-design-studio-community/lumiradesignercommunityext/issues/3#issuecomment-336559288,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJpiS0Rucnct6-hSdyEOnODCf7ZB7KfOks5sr8fxgaJpZM4PPUiT
.

--
Franck BLAIS

And you're the guru as well !
We can't possibly compete with you ;)

It's not a competition but I know you and Karol have your hands full these days with other important things :)


Make sccache never time out

This should fix https://github.com/servo/servo/issues/19495.

This change is

Are there docs for this somewhere? Please add a link to the commit message and also maybe a description of the problem (where is sccache timing out currently?) and/or link to a Buildbot log in this PR.

Commit message updated.

Thanks for the links. Two questions I have:

The Jenkins.md link recommends start the server beside Jenkins as a system service in addition to setting the environment variable in this PR. Do you know if this is required/what the effect is if we don't start sccache by ourselves as a system service (e.g. upstart or systemd)?
We will need to upgrade sccache from time to time (e.g. #755). What is the story for handling these kinds of upgrades: do we need to perform manual steps, and if so, what are they? (Stopping the existing sccache process(es), starting new one(s), clearing cache data from disk, etc.)

The first time a build is run with a timeout of 0, it will automatically start the long-lived server in the background. As part of upgrading, we should run sccache --stop-server before installing the new version. Otherwise we should not need to do anything else. This PR should be mergeable as-is.

I'm going to go ahead and merge this change because https://github.com/servo/servo/issues/19495 is really frustrating and I want to have enough time to be able to look into anything that might go wrong.
@bors-servo: r+

:pushpin: Commit bbf08ca has been approved by jdm

:hourglass: Testing commit bbf08ca5e980f59743d44356a13618cb1ccc1102 with merge 36dbe2c360abf2858921914a9ae01aa4eef17992...

:sunny: Test successful - status-travis
Approved by: jdm
Pushing 36dbe2c360abf2858921914a9ae01aa4eef17992 to master...


Script-Created Bones Can't Be Found

Tested versions
Tested in 4.2 and 4.3 snapshots
System information
Windows 10 - Godot 4.2
Issue description

every single bone made through script cannot be found.
animation mixer cant find a track matching it. when it uses the exact same name
node finder cant find it through the skeleton
skeleton doesn't play animation either.
im encountering this issue because im making an armor-core mech builder. it copys data from 4 different skeletons and merges it into one. names are kept nice and compatible through a bonemap.
i do see a few hypothetical issues, but i want to actually get through the saloon-door before i get to that bar fight.
my model isn't skinning correctly through the code too, but thats a tad off topic. and likely a modelling error.
Steps to reproduce

have animation using bone with specific name
use add_bone() with exact same specific name
watch your hopes and dreams burn.

Minimal reproduction project (MRP)
ProceduralAnimationError.zip

https://github.com/godotengine/godot/issues/39213#issuecomment-1826779241
the last sent message seems to have stumbled on the same issue as me, however-

it seems they named their nodes incorrectly, which could cause their issues.

The thing to remember about them is that mixing unique and non-unique paths can cause problems because they are considered to be different track paths when caching animations.
As a result, is the right essence of the issue “there is a problem when specifying a unique path for a packed scene on the Animation trackpath” or "is there a problem with the unique path regardless of the packed scene"?

neither seem to be the issue, it was just me not fully understanding the unique name :/ was right there too.
and because all the animations come from the same place i shouldn't have issues with differing tracks.
issue seems to be closed, better be ready for godot armorcore, thanks alot for the help!


Add x86_64 osx/gcc12 build to Azure CI

I tested with the changes from this PR on MSYS2/MINGW64 with gcc-12 at optimization level -O2.
The test program from https://github.com/msys2/MINGW-packages/issues/12857 that segfaulted without these changes, no longer crashes for me.
So, it looks like these changes also prevent the optimization issue on MINGW. 🎉

Thank you for the feedback. I will merge this later today after adding appropriate ifdefs around the pragma


Cannot prompt for [Are you sure you want to continue?] in a non-interactive shell

How did you install the Amplify CLI?
npm install -g @aws-amplify/cli
If applicable, what version of Node.js are you using?
v18.18.0
Amplify CLI Version
12.7.1
What operating system are you using?
Windows
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
Nope
Describe the bug
🛑 Cannot prompt for [Are you sure you want to continue?] in a non-interactive shell
Resolution: Please report this issue at https://github.com/aws-amplify/amplify-cli/issues and include the project identifier from: 'amplify diagnose --send-report'
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
Session Identifier: 86c4b04a-1587-4a95-99a2-c8620278d86f

Creating Zip

✅ Report saved: C:\Users\LPS85\AppData\Local\Temp\PROJECT_NAME\report-1699269479705.zip

Sending zip
√ Done

Project Identifier: a8002494b1b9b48b96a0bcb0fae0c733
Expected behavior
It should deploy
Reproduction steps
🛑 Cannot prompt for [Are you sure you want to continue?] in a non-interactive shell
Resolution: Please report this issue at https://github.com/aws-amplify/amplify-cli/issues and include the project identifier from: 'amplify diagnose --send-report'
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
Session Identifier: 86c4b04a-1587-4a95-99a2-c8620278d86f

Creating Zip

✅ Report saved: C:\Users\LPS85\AppData\Local\Temp\PROJECT_NAME\report-1699269479705.zip

Sending zip
√ Done

Project Identifier: a8002494b1b9b48b96a0bcb0fae0c733
Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @pratikbalochiya :wave: thanks for raising this! We've had similar reports in the past for shells like Git Bash on Windows. Are you using Git Bash by chance?

Hi @josefaidt , Thanks for your feedback, I checked in all prompts like git bash, command prompt & powershell, Everywhere getting same error

Glad to hear you were able to workaround this.
The issue is currently being tracked on https://github.com/aws-amplify/amplify-cli/issues/10552.
Closing this issue.

I ran into this same issue with git bash for windows but had a slightly different error. Instead of what this person got, it said, Cannot prompt for [choose the type of app that you're building] in a non-interactive shell
amplify-cli-2023-12-31.log
amplify-cli-2024-01-02.log
amplify-cli-2024-01-01.log
I don't know if this is helpful or not, but in Gitbash when you're selecting the options, I've noticed that instead of the selection your making corresponding to the option you selected, it is in fact selecting two options below the option your curser (the type prompt that blinks and looks like "|"). So when I was initially making my selection, I had two move the curser two lines up, into the actual text of the prompt, in order to make my first selection.
I was able to use the same workaround successfully. I attached two logs below.
The session identifier I was returned for my LAST failed attempt (which just took place minutes before I posted this) is c3fe04a5-9f1f-46d6-b814-d726730dddf6
Hope this is helfpul


Lines indented > 10 spaces are being wrapped in  tags

Confirming this is the expected behavior.
I have a block of code that contains Markdown and HTML. Some of the HTML is indented > 5 times (10 spaced, 2 spaces per indent).
When rendering, all of the code past this indentation level are wrapped in <pre><code>.
If this is the expected behavior, how can I prevent Showdown from converting this HTML?

Turns out this was an issue with unmatched closing tags.


Amend default ignored pattern to match .zshrc

.zshrc is not automatically ignored because the default pattern is too restrictive:
https://github.com/timonwong/vscode-shellcheck/blob/77b03afd1d8007b48eaa6351e3f9cc3b0c2be544/package.json#L102
Adding an extra * solves this problem:
"shellcheck.ignorePatterns": {
        "**/*.zsh*": true,
}

:tada: This issue has been resolved in version 0.11.0 :tada:
The release is available on:

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Evaluate using Profile-Guided Optimization (PGO) for optimizing the IKV performance

Hi!
I collect materials about applying Profile-Guided Optimization (PGO) to different applications in different software domains - all my results are available at https://github.com/zamazan4ik/awesome-pgo . According to my tests, PGO helps with achieving better performance in different areas, including many databases.
I think it will be interesting to see PGO results for IKV in benchmarks since according to my tests, even Redis performance can be improved with PGO (the results are available in the repo above).
Please do not treat the issue as a bug - it's an improvement idea.

Hi @zamazan4ik thanks for reaching out. What tooling do you suggest for analyzing IKV performance (read perf in particular). IKV users use Java/Go/Python client libraries to talk to IKV core (Rust). So there is some foreign func interface interaction, followed by some hashmap + mmap lookups in ikv core, then response preparation (some ser-deserialization).

As far as I see, you already have some benchmarks. You can try to use them to measure performance improvement from enabling PGO compared to usual Release build.

https://doc.rust-lang.org/rustc/profile-guided-optimization.html
Taking a look at this^.


proposal: redefine show tag keys, tag values, field keys api

In order to better understand and use APIs, we should have clear and simple API usage tutorials and tools to help users build their own applications. However, so far, the instructions for using the APIs ShowTagKeys, ShowTagValues, and ShowFieldKeys are vague, and users have no idea how to use the APIs. We need to inform users how to generate the command parameters of each API and what the return value means.
I have temporarily written comments for these APIs and created their builders to quickly generate simple query SQL.

@Chenxulin97 @shoothzj PTAL, thanks.

Great work! I prefer ShowTagValues(database, TagValuesQuery tvq) ([]string, error) than ShowTagValues(database, command string) ([]string, error). WDYT?

The builder may not be perfect, but we can improve it based on requirements and issues.


Deploy Nodejs through Reverse Proxy

problem with configuring /etc/nginx/nginx.conf.
configured my nodejs server private ip in /etc/nginx/nginx.conf file
even then nginx application was unable to load nodejs application .
launched 2 servers first one for nodejs app and second for nginx
opened all my ports for all protocols in aws security group and both the servers were using same vpc and SG.
followed this tutorials
https://www.digitalocean.com/community/tutorials/how-to-set-up-a-node-js-application-for-production-on-centos-7
attaching all required images.

After configuring private ip of nodejs i am getting  following error.

Please help

@Naveen
Please recheck IP Address format you have provided in nginx.conf file.
It should be http://<IP_ADDRESS>:8080
(There is an extra : in IP Address).

Modify conf file with correct IP
Restart Nginx service
Access Reverse proxy server

Regards
Arun Kumar.

that is not the issue ,i corrected it and tried but didn't worked

@Naveen
Create a user with password access, provide ssh access to your server.
Let me work on it.

have you tried doing use case from
https://github.com/asquarezone/ChefZone/blob/master/Workbook/nodejs/Workbooks.md
very first one was installing nginx on ubuntu but i tried on centos7.
using  following link.
https://www.digitalocean.com/community/tutorials/how-to-set-up-a-node-js-application-for-production-on-centos-7
if you have tried it and successful let me know

@naveenaws66
It is working good for me with two Centos 7 Instances. Check below screenshots
NodeJS application running

Nginx Server up & running

Server block in /etc/nginx/nginx.conf

Nginx as a reverse proxy accessing NodeJS app.

Check Nginx error log file /var/log/nginx/error.log
If you have error (13: Permission denied) while connecting to upstream, run below command
$ sudo setsebool -P httpd_can_network_connect 1
(Checked from https://stackoverflow.com/questions/23948527/13-permission-denied-while-connecting-to-upstreamnginx/24830777#24830777)
Regards
Arun Kumar

It's working fine for me with two Centos 7 instances. (one with NodeJS & other with Nginx)
NodeJS up & running.

Nginx up & running.

Server block in /etc/nginx/nginx.conf

Nginx as reverse proxy accessing NodeJS app.

Regards
Arun Kumar

hi @arunkumar127
destroyed my old servers created new servers.
this time its not showing any error page.its showing home page of nginx without any errors but its not redirecting to nodejs.

Is this resolved

yes, its resolved.


Demo master merge

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

from : https://github.com/hmcts/probate-orchestrator-service/pull/151/


api_object.present? does not work

An unexpected error is raised when calling api_object.present?, NameError ('@empty?' is not allowed as an instance variable name)

shopify_api version: 13.4.0
Ruby version: 2.7.7
Operating system: Mac OS 12.6 Monterey

Expected behavior
If the object is not nil it should return true, otherwise it should return false
Actual behavior
An non-relevant error was raised NameError ('@empty?' is not allowed as an instance variable name)
Steps to reproduce the problem

Instantiate an api object api_product = shop.with_shopify_session { ShopifyAPI::Product.new }
Invoke .present? on the object api_product.present?

Hi @jasonbuehler ,
Thank you for opening this issue, this is related to the one you opened here
https://github.com/Shopify/shopify-api-ruby/issues/1273
I'll close this issue for because it's a duplicate.
Si Le


[Improvement]: Add jetbrains annotations to the classpath dependencies for @NotNull and @Nullable annotations

Description
There is already some small usage of this dependency in the jballerina-unit-test module (here and here) but it is only via a transitive dependency of the kotlin std lib.
+--- project :ballerina-lang
|    +--- project :central-client
|    |    +--- com.squareup.okhttp3:okhttp -> 3.14.0
|    |    |    \--- com.squareup.okio:okio:1.17.2 -> 3.4.0
|    |    |         \--- com.squareup.okio:okio-jvm:3.4.0
|    |    |              +--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.0
|    |    |              |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.8.0
|    |    |              |    |    \--- org.jetbrains:annotations:13.0

It would be great to add this dependency directly to the javaProject.gradle compileOnly scope, so it can be used everywhere (without adding any runtime overhead or increasing the distribution size).
There are other options for @Nullable annotations, e.g.

Javax Annotation API
The Checker Framework
Findbugs Annotations

but since the JetBrains annotations are already used I think these should be adopted.
This would help will NPEs since nullable values can be annotated and therefore better documented.
Also IntelliJ and Spotbugs use these annotations in static analysis.
Describe your problem(s)
There is currently no standard defined to document nullable values in the project.
Describe your solution(s)
Add this dependency to the compileOnly scope of the javaProject.gradle file.
Related area
-> Other Area
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

This would also enable developers to write @Contracts which describes how methods behave to different inputs.
Examples:

@Contract("null -> null") if you provide a null to this method it will return null
@Contract("-> new") this method will create a new object and return it
@Contract("null -> fail") this method will throw an exception if you provide a null value.
@Contract("true -> fail") a typical assertFalse method which throws an exception if true is passed to it


0.6.0

Current dependencies on/for this PR:

main

PR #20   👈

This comment was auto-generated by Graphite.


MMM-PIR-Sensor-Lite not closing complete screen

First of all let me say i LOVE the app.  It was the easiest of all I tried to install and worked the first time...  Just amazing..
The one problem I am having is that it blanks the MagicMirror screen but doesn't blank the primary screen.  It still says "no signal" and has a square that the message is in.. I'm sure I missed something on the install but I looked through the options and didn't see anything that would help... Anyone know what I missed?
Thanks
Dennis

@DennisEJ
the command used vcgencmd  turns off the HDMI port. and then the energy-star compliant monitors/tv's MUST out up the no-input warning.. (and turn of after 15 minutes)..
NEWER Monitors provide support for the CEC command set (Consumer Electronics Communication)  which will allow programmatic  access to turn off/on the display .    it doesn't work on my pi3 , but does on my pi 4.. EXCEPT
my TV which says it DOES support cec commands I can turn on, but not off...!?
sudo apt install  cec-client
then see the cec-client --help for info.
IF your system and monitor support it,  then it could be used instead of vcgencmd


Organise test strings by lang, not glyphset

cc @yanone

Use this routine to pull supported languages out of a font dynamically.


Remove duplicate aws-sdk-go/$service import

Remove duplicate aws-sdk-go/$service import. It is already imported in L14 with the alias svcsdk
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Also in other files the svcsdkapi is used as alias for imports similar to  "github.com/aws/aws-sdk-go/service/{{ .ServiceIDClean }}/{{ .ServiceIDClean }}iface"

/lgtm


Added unit tests for Check Protection

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Added unit tests for Check Protection , addressing Checksum U8, U16, U32 and CRC-U32
Why should this Pull Request be merged?
The mentioned VIs had tests only for Add Protection and not for Check Protection.
What testing has been done?
All tests were ran with VI Tester

For all the tests with data corruption, you are missing the protection errors for all but the last iteration, because the input shift register is left unwired, inside the loop. Use an auto-indexing output tunnel, assert that all iterations have thrown the same error code, and then assert that the error code is -265.

Similar here, possibly missing errors in all but last iteration. You can use a shift register here:


Tetsuya Yamamoto: Blacklist healthcares.com.au

Tetsuya Yamamoto requests the blacklist of the website healthcares.com.au. See the Metasmoke search here

!!/approve


there is a call to write-colour

this doesnt cause the system to error, but it does cause the function call not to work.
https://github.com/UKHO/ADM-PSmodule/blob/cdd0f3e31a47dda117b3d52fe73889671a8bf15c/UKHO.ADM/Objects/Objects.ps1#L162

fixed by #28

merged #28


Upgrade to ES 7.7.1 and release version <IP_ADDRESS>

Hello!
Fixes #272
I just copied a previous PR #269  - wondering if the integration tests pass without any more changes

can we get this tested and merged?

Looks like https://gitlab.com/vvanholl/elasticsearch-prometheus-exporter/commits/master doesn't run any pipelines so I guess testing here is manual.

Unfortunately, atm the automatic branch PR testing does not kicks in if thee PR is not based on branch in this repo. As mentioned in https://github.com/vvanholl/elasticsearch-prometheus-exporter#testing you can just do
gradle clean check

to see if tests will pass.

Aha. Thanks and sorry for that.
I can see in a week or two after a short holiday if I can try out gradle :)

Hi @martbhell, Thank you for your PR. I am starting reviewing it on my side.

There are some slightly changes to do. I have created a branch with a working commit on my side : https://github.com/vvanholl/elasticsearch-prometheus-exporter/commit/13273d2c99980505964139f5f1f41063d39256c9
You can take the changes in your PR.

There are some slightly changes to do. I have created a branch with a working commit on my side : 13273d2
You can take the changes in your PR.

Cool! If you want to either push into my fork or just use your own branch in a new PR and close this. I am without a computer for a week now :)


exe crashes during flight

The exe crashes. That's all. No idea why.

No, not overly useful. I wasn't doing anything specific. Just flying one
place to another and the exe just quit. Crash report attached. Appreciate
your help.
Steve Reback
914.954.7401
@.***
On Tue, Apr 11, 2023 at 7:23 AM Jeff D. @.***> wrote:

Not overly useful information. You can look in eventviewer apps and look
for fs2ff crash reports. Also would help if you provided what you were
doing. Astenlund isn't actively working on t
—
Reply to this email directly, view it on GitHub
https://github.com/astenlund/fs2ff/issues/90#issuecomment-1503462277,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7CIR7IIBDZYCAS7PGQVTMDXAVSPZANCNFSM6AAAAAAWYMAAYY
.
You are receiving this because you authored the thread.Message ID:
@.***>

I was using Pilotedge at the time. Is there a known conflict?
Steve Reback
914.954.7401
@.***
On Wed, Apr 12, 2023 at 3:47 AM Steve Reback @.***> wrote:

No, not overly useful. I wasn't doing anything specific. Just flying one
place to another and the exe just quit. Crash report attached. Appreciate
your help.
Steve Reback
914.954.7401
@.***
On Tue, Apr 11, 2023 at 7:23 AM Jeff D. @.***> wrote:

Not overly useful information. You can look in eventviewer apps and look
for fs2ff crash reports. Also would help if you provided what you were
doing. Astenlund isn't actively working on t
—
Reply to this email directly, view it on GitHub
https://github.com/astenlund/fs2ff/issues/90#issuecomment-1503462277,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7CIR7IIBDZYCAS7PGQVTMDXAVSPZANCNFSM6AAAAAAWYMAAYY
.
You are receiving this because you authored the thread.Message ID:
@.***>

try the version of fs2ff.exe from here: https://github.com/jeffdamp-wave/fs2ff I just pulled in the latest SimConnect etc. See if you can repro again. There shouldn't be any incompatibility with PilotEdge.

Thanks Jeff. I'll try that later and let you know how it goes.
On Wed, Apr 12, 2023 at 07:58 Jeff D. @.***> wrote:

try the version of fs2ff.exe from here:
https://github.com/jeffdamp-wave/fs2ff I just pulled in the latest
SimConnect etc. See if you can repro again. There shouldn't be any
incompatibility with PilotEdge.
—
Reply to this email directly, view it on GitHub
https://github.com/astenlund/fs2ff/issues/90#issuecomment-1505427894,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7CIR7IEDAKP4P5KBYZYYOLXA27KTANCNFSM6AAAAAAWYMAAYY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Steve Reback
914.954.7401
@.***


Updated branding on header for documentation pages

yeah i was debating this, i thought it was a little weird to have the "MakeCode" text take you to the specific editor, but i'm willing to be convinced otherwise!! i think some of the docs (eg courses, etc) have links in the markdown itself but i definitely agree it's good to have something in the header


Remove trailing white spaces from skipped items

Closes #2037.
This PR changes rustfmt to remove trailing white spaces from skipped items. With file_lines option, this might be a problem, as it removes trailing white spaces from lines that are not specified (https://github.com/topecongiro/rustfmt/commit/b59992de2125fa11d62256ec482ba6d624271eeb). I am not sure if this is acceptable or not.
If this is not desired, I will add commits to fix it.

Hmm, I think when a node is skipped it really ought to be skipped completely. (I can't think of an example where trailing white space is significant, but I bet someone has one). This would be fine to have behind an option, but it should be off by default I think.
To fix 2037, I think we should have an option where skipped lines are not checked for either trailing whitespace or for being overlong.

To fix 2037, I think we should have an option where skipped lines are not checked for either trailing whitespace or for being overlong.

This is nontrivial as we currently check trailing white spaces or being overlong on char-by-char or line-by-line basis, not AST basis.

This is nontrivial ...

Hmm, true. Let me think what to do...

Since removing trailing white spaces from a skipped item is not a way to go, I am going close this PR.
I will work on skipped item issues when I come up with a better idea. Thanks for your review!

This is nontrivial as we currently check trailing white spaces or being overlong on char-by-char or line-by-line basis, not AST basis.

@topecongiro do you think we could track lines we skip in the filemap as we write them, then ignore these lines in 'by lines' pass?

Hmm, do you mean we keep track of skipped lines while formatting the code, like in rewrite() or visit_..()?

I was thinking we'd only need to track if we skip, so when we hit a skip attribute in the source we'd record the start and end of the emitted node, then figure out the line number some how (probably we'd want to keep count of newlines in FileMap).

I don't think that works when the input and the output changes after formatting. We can rustfmt twice and rely on the idempotency of rustfmt, but that seems a waste.

How can the output change after formatting? I'd have thought that if we format and record the position in the output as we go, then we'd be good?


Change the App Icon

Is your feature request related to a problem? Please describe.
Current App Icon is outdated and not relevant for our colors.
Describe the solution you'd like
Let's use this version
https://stackoverflow.com/questions/43928702/how-to-change-the-application-launcher-icon-on-flutter

Additional context
Add any other context or screenshots about the feature request here.

This looks exactly like passport icon, at least scaled down to icon size (I know it's not the same, but very similar)

@gguijarro-c-chwy is this completed?


Introducing "auto" mode.

The new evaluator.py is now available to evaluate if your prompt is worth searching online, or just a random chat.

Some mistakes


Add Codecov GitHub Action step

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

Addresses https://github.com/Homebrew/brew/issues/10506
Move from using the Codecov gem to the Codecov GitHub Action.

@MikeMcQuaid I think we'll have to add the Codecov Action to the allowed list of Actions as described here

@MikeMcQuaid looks like there is an issue with how we are parsing the coverage report. I'll look to fix on our end.

@thomasrockhu Great, thanks!

Just a quick update, we have found the source of the issue but are figuring out the best way to fix the issue. I'll update when we have a fix on hand.

Sorry again for the delay. The newest update is that we are trying a few vectors to prevent any negative impacts to other users. I'll update again here with any changes.

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

Apologies on the delay here, we still have an engineer working on the implementation now.

@thomasrockhu Any news here? We'd really like to get this CodeCov token out of our repository ASAP and want to avoid disabling CodeCov entirely but it might get to that stage 😭

@MikeMcQuaid I wanted to unblock this PR, but it requires using the cobertura formatted for SimpleCov. I know a big part of what we were trying to do was reduce the number of gems, but is this an acceptable midpoint while we work on our handlers?

@thomasrockhu Yes that's fine. Actually our only goals were:

split code coverage reporting into a separate test step
remove the hardcoded token from the repository

@MikeMcQuaid I think this is in a good state now. Am I missing anything?

@thomasrockhu Code and CI results look good! https://app.codecov.io/gh/Homebrew/brew/compare/10514 seems to be broken.

@MikeMcQuaid oof, this is an issue from our side that we are working on that typically affects users that aren't logged in. The fix for that issue is going to take some time for now, is this a blocker for you and your team?

Thanks so much for your first contribution!  You rock, @thomasrockhu!