Failing to index my /ideas url on my Obsidian Publish site

This is my idea page: https://pooriajr.com/ideas
Here's a gif showing how it doesn't work

I was about to open this same issue, except using a subdomain. In my case:

https://notes.samantha.wiki/about
https://notes.samantha.wiki/ideas
https://notes.samantha.wiki/now

It seems like Obsidian Publish requires JavaScript to render the content, and we currently don't support that in the crawler. Let me see if we can change this...

Third-ing this issue, utilizing Angular to render my personal site on my end (pages are stubs, for the time being) -
https://www.jeremycantu.com/about
https://www.jeremycantu.com/ideas
https://www.jeremycantu.com/now

Same problem here on my static site (js not required)
https://villapirorum.netlify.app/about
https://villapirorum.netlify.app/now
https://villapirorum.netlify.app/ideas

Same here. I enforce www redirects and this makes the adding fail it seems.


python: Use Python's logging through pyo3_log

This makes logging accessible and configurable through Python.

I've used this a lot with own added info messages; the trace messages emitted by lakers itself are not visible in this unless we touch one more knob -- might make sense to turn that on by default, because nothing we do is performance critical on the Python side.

Cool! Suggested a comment just for clarity.
Could you enable that knob so that we don't lose the from-Rust log functionality that we currently have? Is it possible to have it controlled by an env var like with env_logger?

I'll add some API expositure and some basicConfig reference.

I misunderstood how this can be configured – more API won't do us any good. Instead, given that we're not logging a whole lot, and are not performance critical in Python, I've changed the code to unconditionally log trace from Rust. (This is still not super slow, as all pyo3_log's caching etc is still in). There are module level docs now that explain the logging isutation.

Thanks for this PR! Merging


Add mailing list and calendar to website

Include how to join, etc.

there is https://opiproject.org/join/

there is https://opiproject.org/join/

The join page on the website does get you in the right place generally, but it specifically points you to the opi list and not the others.  Some instructions that point to the subgroups section which shows the rest would be useful.


Update Wiki page content

Adds methods to update a Wiki's page on ADO. The current page's version is required when updating so there are two API calls that need to be made here.
Example usage:
page_version = ado_client.wiki.get_page_version(page_id=1, wiki_id="wiki", project_id="project")
ado_client.wiki.update_page(page_id=1, wiki_id="wiki", project_id="project", content="Testing updates", current_version_etag=page_version)

@dalemyers PR ready for review if you wouldn't mind taking a look!

Before I review, the test failures are due to PAT rotation. I've not updated it here yet.


How to access pvtTable element by jquery ?

How to access two pvtTable element by jquery ?
I need to access pvtTable > thead > tr > th > pvtAxisLabel and
pvtTable > tbody > pvtRowLabel
it is a challenge for me as the target AxisLabel could be 1st or 2nd column. I am using this
$("pvtTable > thead > tr > th > thpvtAxisLabel")
it is not able to gain access. How should I code this ?
Thanks

This type of jQuery-selector-specific question is best suited for StackOverflow or something like this, as it's not really PivotTable.js-specific.

OK Thanks


graphcs are not being rendered

The article has sections that appear  to be graphics to be rendered inline, but only the hook to the graphics shows up instead.
Example:
 Apply the Production Checkpoint with Hyper-V Manager

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

ID: 371a9010-df75-2250-6e55-85bef8c0b9de
Version Independent ID: 9f61de64-c87e-39c9-901d-463e44df06b8
Content: Using checkpoints
Content Source: virtualization/hyper-v-on-windows/user-guide/checkpoints.md
Service: windows-10-hyperv
Product: windows-10-hyperv
GitHub Login: @scooley

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


uninitialized constant Slaw::ZA (NameError)

generator = Slaw::ZA::ByLawGenerator.new
This lines give error. trying to find the code but could not the definition any where.

Please check with the latest version and see if you still get this error.


Club time of death is set to 1969

This isn't just new clubs, check the history of this box for an example: agxzfm1haWxmb29nYWVyLAsSDE9yZ2FuaXphdGlvbiINemFjaGxhdHRhLmNvbQwLEgRDYXNlGJOxgx4M


[FLINK-24468][runtime] Wait for the channel activation before creating partition request client

What is the purpose of the change
This PR fixes NPE which happens when we try to use channel context inside of ClientHandler before the channel was activated
Brief change log

Wait for the channel activation before creating partition request client
Ignore the exception during the throughput calculation in order to avoid the failure of the further calculation

Verifying this change
This change added tests and can be verified as follows:

Added test for CreditBasedPartitionRequestClientHandler

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Another question. Why does it happen only for announcing the new buffer size? Shouldn't we have the same issue for all other methods of CreditBasedPartitionRequestClientHandler such as e.g. notifyCreditAvailable?

So I spent some time looking into the waitForChannelActivation thingy. I am a bit concern if the sync waiting does not prolong the startup too much. If I understand the code correctly, we will sequentially wait for each channel to become active before requesting a partition on another channel.
As the initial error shows it may take some time. I was rather thinking if we could return the future itself (I know we would've to pass it through a couple of layers). Then we could start buffer debloating once a future for all channels activation completes.
Something like:
        CompletableFuture.allOf(
                        Arrays.stream(environment.getAllInputGates())
                                .map(InputGate::getAllChannelsActivatedFuture)
                                .toArray(CompletableFuture[]::new))
                .whenComplete(
                        (r, ex) -> {
                            scheduleBufferDebloater();
                        });

Most likely we added notifyNewBufferSize to the ChannelHandler to follow the same path as notifyCreditAvailable. At least me myself, have not given much thought whether it should actually be implemented how you are proposing. Maybe you are right @dawidwys .
I don't know why notifyCreditAvailable was added the way it was added. Maybe that was also a wrong place to add it.
I haven't read the full thread, so I'm not sure. If we had changed the way you are proposing @dawidwys , would it solve/help solve the problems in this ticket?


Many tests failing

Describe the bug
What's the bug?
I'm packaging gdsfactory for NixOS. It compiles and simple examples from the tutorial do work. However, we normally prefer to run the tests of packages. When I run them, I find that many tests fail (287!), with a large fraction because they can't find GDS files.
Is the test suite supposed to pass all tests? (NixOS is unconventional and it could be that they are failing for some reason I don't currently understand.)
Here the the build log
gdsfactory.log
Expected behavior
I'd expect all tests to pass.
Suggested fix
How could we fix the bug? As an open-source project, we welcome your suggestions and PRs.
Environment (please complete the following information):

[x] I have reviewed the documentation, discussions and issues and found no relevant solution.
[x] I understand that this is an open-source project, and maintainers may not have the resources to address every issue. I am prepared to contribute by fixing the issue myself or hiring someone to do so, if needed, and accept that the issue may not be resolved otherwise.
[x] I am using the latest version of GDSFactory, with Python 3.10, 3.11, or 3.12.

Please provide the output for the following code:
import sys
print(sys.version)
print(sys.executable)

import gdsfactory as gf
gf.config.print_version_plugins()

3.12.7 (main, Oct  1 2024, 02:05:46) [GCC 13.3.0]
/nix/store/mzq6hm06nigc2p9n6xdnz99hxhqx86jv-python3-3.12.7-env/bin/python3.12
                                      Modules
┏━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃    Package ┃ version                          ┃                             Path ┃
┡━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┩
│     python │ 3.12.7 (main, Oct  1 2024,       │ /nix/store/mzq6hm06nigc2p9n6xdn… │
│            │ 02:05:46) [GCC 13.3.0]           │                                  │
│ gdsfactory │ 8.18.1                           │ /home/beffa/src/git/nixpkgs/res… │
│   gplugins │ not installed                    │                                  │
│        ray │ not installed                    │                                  │
│    femwell │ not installed                    │                                  │
│     devsim │ not installed                    │                                  │
│     tidy3d │ not installed                    │                                  │
│       meep │ not installed                    │                                  │
│       meow │ not installed                    │                                  │
│     lumapi │ not installed                    │                                  │
│        sax │ not installed                    │                                  │
└────────────┴──────────────────────────────────┴──────────────────────────────────┘

You need to have the reference gds files, otherwise the tests fail due to missing reference data.
The tests are doing it with the following lines from the Makefile
https://github.com/gdsfactory/gdsfactory/blob/main/Makefile#L22-L23

OK. thanks!


Multi-arch support for a nydus-snapshotter image

The project is currently utilized by confidential-containers. Presently, the CI system is set up exclusively for x86_64 architecture. As the project progresses, there's a need to expand the CI to include support for other platforms. However, the deployment relies on an image, ghcr.io/containerd/nydus-snapshotter, which is only compatible with x86_64 architecture. It would be advantageous to transform this into a multi-arch image using a GitHub Actions workflow.

@BbolroC :( It seems have a wrap line problem when handle latest and tag:
https://github.com/containerd/nydus-snapshotter/actions/runs/8325473739/job/22779362791

@imeoer Thanks for sharing the log. I missed a tagging part. I will update the PR. 😉


SerializerFieldNotFoundError: Could not generate an XPath for <stix.extensions.identity.ciq_identity_3_0.CIQIdentity3_0Instance object at 0x7fd6906b8400>

In [2]: from stix2slider import slide_file
...: from stix2slider.options import initialize_options as slider_options
...: from stix2validator import validate_file
...: slider_options()
...: result=validate_file(file_name)
...: print("Valid :",result.is_valid)
...: if result.is_valid:
...:     results = slide_file(file_name)
...:     print(results)
...:
Valid : True
[stixmarx.fields] [INFO   ] [2018-05-18 14:47:43,608] No compatible cybox <IP_ADDRESS> mappings found. Loaded latest unchanged <IP_ADDRESS> field mappings.
[stix2slider.options] [502] [WARNING] [2018-05-18 14:47:43,829] labels not representable in a STIX 1.x ToolInformation.  Found in tool--ce45f721-af14-4fc0-938c-000c16186418
[stix2slider.options] [502] [WARNING] [2018-05-18 14:47:43,831] labels not representable in a STIX 1.x ToolInformation.  Found in tool--e9778c42-bc2f-4eda-9fb4-6a931834f68c
[stix2slider.options] [502] [WARNING] [2018-05-18 14:47:43,832] labels not representable in a STIX 1.x ToolInformation.  Found in tool--1cf6a3b8-be43-4c1a-b042-546a890c31b2
[stix2slider.options] [502] [WARNING] [2018-05-18 14:47:43,833] labels not representable in a STIX 1.x ToolInformation.  Found in tool--4d82bd3e-24a3-4f9d-b8f3-b57267fe06a9
[stix2slider.options] [502] [WARNING] [2018-05-18 14:47:43,834] labels not representable in a STIX 1.x ToolInformation.  Found in tool--7de5dfcc-6809-4772-9f11-cf26c2be53aa
[stix2slider.options] [502] [WARNING] [2018-05-18 14:47:43,835] labels not representable in a STIX 1.x ToolInformation.  Found in tool--266b12f2-aa16-4607-809e-f2d33eebb52e
[stix2slider.options] [502] [WARNING] [2018-05-18 14:47:43,836] labels not representable in a STIX 1.x ToolInformation.  Found in tool--98fd8dc1-6cc7-4908-899f-07473f55149a
[stix2slider.options] [502] [WARNING] [2018-05-18 14:47:43,838] labels not representable in a STIX 1.x ToolInformation.  Found in tool--4215b0e5-928e-4b2a-9b5f-64819f287f48
[stix2slider.options] [502] [WARNING] [2018-05-18 14:47:43,839] labels not representable in a STIX 1.x ToolInformation.  Found in tool--a6dd62d0-9683-48bf-a9cd-61e7eceae57e
[stix2slider.options] [502] [WARNING] [2018-05-18 14:47:43,840] labels not representable in a STIX 1.x ToolInformation.  Found in tool--806a8f83-4913-4216-bb19-02b48ae25da5

SerializerFieldNotFoundError              Traceback (most recent call last)
 in ()
6 print("Valid :",result.is_valid)
7 if result.is_valid:
----> 8     results = slide_file(file_name)
9     print(results)
10
~/Cyware/source-code/CTIX-S/ctixserver/stix2slider/init.py in slide_file(fn, encoding)
17     obj = stix2.parse(json_content)
18     # TODO: validate STIX 2.0 content - what to do if it is invalid??
---> 19     stix_package = convert_bundle(obj)
20
21     if stix_package:
~/Cyware/source-code/CTIX-S/ctixserver/stix2slider/convert_stix.py in convert_bundle(bundle_obj)
960     for k, v in _KILL_CHAINS.items():
961         pkg.ttps.kill_chains.append(v["kill_chain"])
--> 962     CONTAINER.flush()
963     CONTAINER = None
964     return pkg
~/Cyware/source-code/CTIX-S/venv/lib/python3.5/site-packages/stixmarx/container.py in flush(self)
319         """
320         writer = serializer.MarkingSerializer(marking_container=self)
--> 321         writer._apply_markings()
322
323         # Reset the collections so we don't return duplicates
~/Cyware/source-code/CTIX-S/venv/lib/python3.5/site-packages/stixmarx/serializer.py in _apply_markings(self)
96     def _apply_markings(self):
97         self._apply_global_markings()
---> 98         self._apply_field_markings()
99         self._apply_null_markings()
100
~/Cyware/source-code/CTIX-S/venv/lib/python3.5/site-packages/stixmarx/serializer.py in _apply_field_markings(self)
79
80         for field, markings_info in iteritems(field_markings):
---> 81             self._apply_markings_to_field(field, markings_info)
82
83     def _apply_null_markings(self):
~/Cyware/source-code/CTIX-S/venv/lib/python3.5/site-packages/stixmarx/serializer.py in _apply_markings_to_field(self, field, marking_info)
68             marking = copy.deepcopy(marking)
69             marking.controlled_structure, handling =
---> 70                 self._find_path_and_handling(field, descendants)
71
72             handling.add_marking(marking)
~/Cyware/source-code/CTIX-S/venv/lib/python3.5/site-packages/stixmarx/serializer.py in _find_path_and_handling(self, field, descendants)
210             error = "Could not generate an XPath for {0}".format(field)
211             raise errors.SerializerFieldNotFoundError(entity=field,
--> 212                                                       message=error)
213
214         xpath = xml.XPATH_SELECT_OPERATOR.join(xpath)
SerializerFieldNotFoundError: Could not generate an XPath for <stix.extensions.identity.ciq_identity_3_0.CIQIdentity3_0Instance object at 0x7fd6906b8400>

Hi @ajnovice,
I think we have a fix for this which will be merged in soon.  Basically, data markings in STIX 1.x do not have an information_source property, so we can't find the CIQ object when it is trying to create the XPath.

@rpiazza @emmanvg what is the status of this?

Fixed in 7fe85e744c4bf6f9c2e1ec9190a9c689f124431d


[WIP] Allow random cluster selection when no override

TL;DR
In case there is no override in resource table, we should allow the code to go through and apply non-label based weighted random selection.
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[ ] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
As commented, it makes more sense to let the code go through and apply different selection.
BTW these unit tests relies on GOPATH and we should fix that.
Tracking Issue
Will create later
https://github.com/lyft/flyte/issues/
Follow-up issue
NA

Codecov Report

Merging #117 into master will increase coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
+ Coverage   62.32%   62.39%   +0.07%
==========================================
  Files         104      104
  Lines        7753     7752       -1
==========================================
+ Hits         4832     4837       +5
+ Misses       2351     2345       -6
  Partials      570      570

Flag
Coverage Δ

#unittests
62.39% <ø> (+0.07%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...g/executioncluster/impl/random_cluster_selector.go
80.39% <ø> (+5.63%)
:arrow_up:

Continue to review full report at Codecov.

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


Names of subdirectories for checkpoint and logging

❓ Questions and Help
First question
The directories where checkpoints are saved are always something like
$DIR/$SUBDIR/checkpoint/

I have $DIR controlled: it can be set by default_save_path argument in Trainer, and it is also a member of the class with the same name.
$SUBDIR seems randomly generated. Where is it generated? Is there any way where I can access it after training?
My use case is: I am training a LightningModule, and by default it is saving the best model (checkpoint) during training (the one with lowest val_loss). I want to load the best model just after training and, for example run a test run.
Second question:
A similar question about wandb logger. Is there any way that I can see which logger directory and subdirectory it is pointing to?
My goal would be to select the best, for example, 5 models out of 20 runs and load them by combining the metrics stored in the logger and the weights stored in the checkpoints. Alternatively, what would be the best practice here?
What have you tried?
I went through the code looking for where $SUBDIR is generated and I did not find it. I could not find anything relevant as members of the classes Trainer or LightningModule either.

Hi, it seems that I found part of the answer: trainer.checkpoint_callback.best_k_models is a dictionary with the path and the metric (in my case validation loss). By default it only contains an element, which is the best model, so this works for me:
path_to_best_model =next(iter(trainer.checkpoint_callback.best_k_models.keys()))
model.load_from_checkpoint(path_to_best_model)

If you have more than one model saved, you can use the following snippet for getting the best model:
op = min if checkpoint_callback.mode == 'min' else max
best_model_path = op(
    checkpoint_callback.best_k_models,
    key=checkpoint_callback.best_k_models.get
)

BTW, this snippet comes from #1799 , this PR will make it even easier to get back the best model =)

btw, #1799 hs landed, so I assume this can be also closed... feel free to reopen if needed :raccoon:


[BUG]: "could not read Password" error in checkout task when fetchFilter is provided

What happened?
If you add a fetchFilter (from #4693) to the checkout task (as well as enabling the AGENT_USE_FETCH_FILTER_IN_CHECKOUT_TASK knob), then the checkout manages to fetch objects from the remotes, but fails on the final git checkout task with an authentication issue:
There is a warning in the result suggesting changing fetchDepth, however I get the same issue regardless of whether I set fetch depth
Repo pipeline:
job: demo
displayName: 🏗 Build
steps:
  - checkout: self
    fetchFilter: tree:0
    #fetchDepth: 0

Versions
Current agent version: '3.241.0'
Operating System
Ubuntu
22.04.4
LTS
Runner Image
Image: ubuntu-22.04
Version: 20240616.1.0
Included Software: https://github.com/actions/runner-images/blob/ubuntu22/20240616.1/images/ubuntu/Ubuntu2204-Readme.md
Image Release: https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20240616.1
Runner Image Provisioner
2.0.370.1
Current image version: '20240616.1.0'
Environment type (Please select at least one enviroment where you face this issue)

[ ] Self-Hosted
[X] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Ubuntu 22.04.04 LTS
Version controll system
Git
Relevant log output
Starting: Checkout REPO@main to s
==============================================================================
Task         : Get sources
Description  : Get sources from a repository. Supports Git, TfsVC, and SVN repositories.
Version      : 1.0.0
Author       : Microsoft
Help         : [More Information](https://go.microsoft.com/fwlink/?LinkId=798199)
==============================================================================
Syncing repository: REPO (Git)
git version
git version 2.45.2
git lfs version
git-lfs/3.5.1 (GitHub; linux amd64; go 1.21.8)
git init "/home/vsts/work/1/s"
hint: Using 'master' as the name for the initial branch. This default branch name
hint: is subject to change. To configure the initial branch name to use in all
hint: of your new repositories, which will suppress this warning, call:
hint:
hint: 	git config --global init.defaultBranch <name>
hint:
hint: Names commonly chosen instead of 'master' are 'main', 'trunk' and
hint: 'development'. The just-created branch can be renamed via this command:
hint:
hint: 	git branch -m <name>
Initialized empty Git repository in /home/vsts/work/1/s/.git/
git remote add origin<EMAIL_ADDRESS>git config gc.auto 0
git config core.longpaths true
git config --get-all<EMAIL_ADDRESS>git config --get-all http.extraheader
git config --get-regexp .*extraheader
git config --get-all http.proxy
git config http.version HTTP/1.1
git --config-env=http.extraheader=env_var_http.extraheader fetch --force --tags --prune --prune-tags --progress --no-recurse-submodules origin --depth=1 --filter=tree:0 +edb49c62b56ceb393542ebb413c58b39130fd07e:refs/remotes/origin/edb49c62b56ceb393542ebb413c58b39130fd07e
remote: Azure Repos
remote:
remote: Found 1 objects to send. (10 ms)
Receiving objects: 100% (1/1)
Receiving objects: 100% (1/1), 184 bytes | 184.00 KiB/s, done.
From https://dev.azure.com/ORG/PROJECT/_git/REPO
 * [new ref]         edb49c62b56ceb393542ebb413c58b39130fd07e -> origin/edb49c62b56ceb393542ebb413c58b39130fd07e
git --config-env=http.extraheader=env_var_http.extraheader fetch --force --tags --prune --prune-tags --progress --no-recurse-submodules origin --depth=1 --filter=tree:0 +edb49c62b56ceb393542ebb413c58b39130fd07e
remote: Azure Repos
remote:
remote: Found 0 objects to send. (19 ms)
From https://dev.azure.com/ORG/PROJECT/_git/REPO
 * branch            edb49c62b56ceb393542ebb413c58b39130fd07e -> FETCH_HEAD
git checkout --progress --force refs/remotes/origin/edb49c62b56ceb393542ebb413c58b39130fd07e
fatal: could not read Password for<EMAIL_ADDRESS>terminal prompts disabled
fatal: could not fetch 7f12c57ea65125ee8da773d3de56ea12bc338daa from promisor remote
##[warning]Git checkout failed on shallow repository, this might because of git fetch with depth '1' doesn't include the checkout commit 'refs/remotes/origin/edb49c62b56ceb393542ebb413c58b39130fd07e'. Please reference documentation (http://go.microsoft.com/fwlink/?LinkId=829603)
##[error]Git checkout failed with exit code: 128
Finishing: Checkout REPO@main to s

Setting AGENT_USE_FETCH_FILTER_IN_CHECKOUT_TASK  no longer seems to be required, but the issue still occurs.
I believe the problem is the final checkout call is missing credentials (presumably --config-env=http.extraheader=env_var_http.extraheader)
git checkout --progress --force refs/remotes/origin/edb49c62b56ceb393542ebb413c58b39130fd07e

Hi @afscrome, thanks for reporting.
I have reproduced the issue.
It arises when the checkout command is executed on an agent that either lacks access to the credential manager or does not have the necessary credentials for the requested repository in credential-manager.
I will keep you updated as I make further progress.

@afscrome, I've linked the PR that addresses the issue and is currently under review.

@afscrome, could you kindly verify if the issue with your pipeline has been addressed?

We are still seeing this issue @afscrome.
We are on agent version 3.246.0. Our situation is a little bit complex - we are doing comparisons between source and target commits during a PR to determine impacted projects for testing purposes.
To do this we first check out the target branch, get the commit id, then the source branch, do a comparison between them and get a file list to check against our projects. We have a very large history and therefore are taking advantage of the support for fetch filtering using blob:none to avoid the lengthy download.
When we get to checking out the source commit, if there are blobs that are not already present in the local repository, there is a fetch to the promisor remote (origin) that i believe is still missing the credentials. Please see the screenshot below:

This doesn't happen all the time, only when there is gap that has to be fetched. In these cases having the developer rebase onto the target usually resolves the issue (avoiding the promisor remote fetch). We have also added powershell script to perform some of the git commands directly which seems to work with the persisted credentials just fine.

@richshadman, could you please provide a reproducible sample of the pipeline?


Parse CSV to String Array

We have a use case where we are dynamically generating csv columns and don't know what columns are available until runtime. Therefore, we can't define a CSVMapping object  in order to instantiate a CSVParser object.
I would like the ability to define and CSVParser without a mapping object and parse a  file/string into a String Array. Something like...
String[]  sArray = new CsvParser(csvParserOptions).ParseToStringArray(filePath);

Right now it's not possible out of the box. You could reproduce it by copying the CsvParser implementation and leave the last mapping step out. Then you could return the tokenized rows. Please note, that you will also need to copy some of the CsvParserExtension methods (ReadFromFile) to have File Reading for your custom parser.
If you need assistance doing it, let me know.

Here is an example:
// Copyright (c) Philipp Wagner. All rights reserved.
// Licensed under the MIT license. See LICENSE file in the project root for full license information.

using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Text;
using NUnit.Framework;
using TinyCsvParser.Mapping;
using TinyCsvParser.Model;

namespace TinyCsvParser.Test.Examples
{
    public class CustomCsvParser
    {
        private readonly CsvParserOptions options;

        public CustomCsvParser(CsvParserOptions options)
        {
            this.options = options;
        }

        public ParallelQuery<TokenizedRow> Parse(IEnumerable<Row> csvData)
        {
            if (csvData == null)
            {
                throw new ArgumentNullException(nameof(csvData));
            }

            var query = csvData
                .Skip(options.SkipHeader ? 1 : 0)
                .AsParallel();

            // If you want to get the same order as in the CSV file, this option needs to be set:
            if (options.KeepOrder)
            {
                query = query.AsOrdered();
            }

            query = query
                .WithDegreeOfParallelism(options.DegreeOfParallelism)
                .Where(row => !string.IsNullOrWhiteSpace(row.Data));

            // Ignore Lines, that start with a comment character:
            if (!string.IsNullOrWhiteSpace(options.CommentCharacter))
            {
                query = query.Where(line => !line.Data.StartsWith(options.CommentCharacter));
            }

            return query
                .Select(line => new TokenizedRow(line.Index, options.Tokenizer.Tokenize(line.Data)));
        }

        public override string ToString()
        {
            return $"CsvParser (Options = {options})";
        }
    }

    public static class CustomCsvParserExtensions
    {
        public static ParallelQuery<TokenizedRow> ReadFromFile(this CustomCsvParser csvParser, string fileName, Encoding encoding)
        {
            if (fileName == null)
            {
                throw new ArgumentNullException(nameof(fileName));
            }

            var lines = File
                .ReadLines(fileName, encoding)
                .Select((line, index) => new Row(index, line));

            return csvParser.Parse(lines);
        }
    }

    [TestFixture]
    public class CustomCsvParserTest
    {
        [Test]
        public void CustomCsvParserReadFromFile()
        {
            CsvParserOptions options = new CsvParserOptions(true, ';');

            CustomCsvParser customCsvParser = new CustomCsvParser(options);

            var stringBuilder = new StringBuilder()
                .AppendLine("FirstName;LastName;BirthDate")
                .AppendLine("     Philipp;Wagner;1986/05/12       ")
                .AppendLine("Max;Mustermann;2014/01/01");
#if NETCOREAPP1_1
            var basePath = AppContext.BaseDirectory;
#else
            var basePath = AppDomain.CurrentDomain.BaseDirectory;
#endif
            var filePath = Path.Combine(basePath, "test_file.txt");

            File.WriteAllText(filePath, stringBuilder.ToString(), Encoding.UTF8);

            var result = customCsvParser
                .ReadFromFile(filePath, Encoding.UTF8)
                .ToList();

            Assert.AreEqual(2, result.Count);

            Assert.AreEqual("Philipp", result[0].Tokens[0]);
            Assert.AreEqual("Wagner", result[0].Tokens[1]);

            Assert.AreEqual("Max", result[1].Tokens[0]);
            Assert.AreEqual("Mustermann", result[1].Tokens[1]);
        }

        [Test]
        public void CustomCsvParserGetAsStringArrays()
        {
            CsvParserOptions options = new CsvParserOptions(true, ';');

            CustomCsvParser customCsvParser = new CustomCsvParser(options);

            var stringBuilder = new StringBuilder()
                .AppendLine("FirstName;LastName;BirthDate")
                .AppendLine("     Philipp;Wagner;1986/05/12       ")
                .AppendLine("Max;Mustermann;2014/01/01");
#if NETCOREAPP1_1
            var basePath = AppContext.BaseDirectory;
#else
            var basePath = AppDomain.CurrentDomain.BaseDirectory;
#endif
            var filePath = Path.Combine(basePath, "test_file.txt");

            File.WriteAllText(filePath, stringBuilder.ToString(), Encoding.UTF8);

            var result = customCsvParser
                .ReadFromFile(filePath, Encoding.UTF8)
                .Select(x => x.Tokens)
                .ToList();

            Assert.AreEqual(2, result.Count);

            Assert.AreEqual("Philipp", result[0][0]);
            Assert.AreEqual("Wagner", result[0][1]);

            Assert.AreEqual("Max", result[1][0]);
            Assert.AreEqual("Mustermann", result[1][1]);
        }
    }
}

Just wanted to note that this scenario may be pretty common and in my project I also need to parse and forward data with dynamic model. It would be nice if this will eventually be supported by library.
To keeping things DRY, I tried to inherit CSVParser or CsvMapping but unfortunately their methods is not virtual and they are dependent on classes instead of interfaces.

Could you create a short example of using your "Dream API"... I mean how an API would look for your use case in a perfect world. So we can see how to do it with the library or modify the library. 😎

Thank you for your question 😄
I'm not ready to discuss "Dream API" yet, but for this specific case I see the simplest approach as:

make CsvMapping.Map virtual;
create custom StringsCsvMapping that simply forwards tokenized strings from TokenizedRow.Tokens;
allow to pass entityFactory to CsvParser to remove new() limitation
then we can use CsvParser<string[]> and pass our StringsCsvMapping as mapping to parse csv to sequence of strings[] rows.

I can create a PR for this in a day or two if you don't mind.

Yes, you could make a Pull Request. But I first want to have a look, which changes would be breaking. To get a list of Tokens you can use the parser shown above. I guess the simplest way however will be to make the CsvMapping.Map virtual like you mentioned above.
The new() limitation can be removed I think. It should be the responsibility of the mapping class to constraint the types, because after all that's where the objects are created. It makes sense to refactor this and it wouldn't be a breaking change anyway.
Please give me some moments to play around with the library. ☕

@alexb5dh @hthroweriii I have released a Version, which now allows to parse to String Arrays and use the existing infrastructure. Many, many thanks to you two for the input! And sorry to @hthroweriii, I didn't really understand your initial request and closed the issue too early.
You can now use it like this:
[Test]
public void StringArrayMappingTest()
{
    CsvParserOptions csvParserOptions = new CsvParserOptions(false, ';');
    CsvReaderOptions csvReaderOptions = new CsvReaderOptions(new[] { Environment.NewLine });
    CsvStringArrayMapping csvMapper = new CsvStringArrayMapping();
    CsvParser<string[]> csvParser = new CsvParser<string[]>(csvParserOptions, csvMapper);

    var stringBuilder = new StringBuilder()
        .AppendLine("Philipp;Wagner;1986/05/12")
        .AppendLine("Max;Mustermann;2014/01/01");

    var result = csvParser
        .ReadFromString(csvReaderOptions, stringBuilder.ToString())
        .ToList();

    Assert.AreEqual(2, result.Count);

    Assert.IsTrue(result.All(x => x.IsValid));

    Assert.AreEqual("Philipp", result[0].Result[0]);
    Assert.AreEqual("Wagner", result[0].Result[1]);
    Assert.AreEqual("1986/05/12", result[0].Result[2]);

    Assert.AreEqual("Max", result[1].Result[0]);
    Assert.AreEqual("Mustermann", result[1].Result[1]);
    Assert.AreEqual("2014/01/01", result[1].Result[2]);
}

@hthroweriii @alexb5dh Is there anything else, that we should add to the library?

Thanks @bytefish, looks great!

Cool! 😎 Let me know, if there is anything you would like to see. 🎅

I have released a Version, which now allows to parse to String Arrays and the existing infrastructure.
Thank you very much, this is exactly that I wanted
Btw it's good to place it to FAQ section as well, because this scenario is quite common

I will add if as soon as possible.


macOS Support

It looks like App Attest has support for macOS. Can you create a shared implementation for iOS and macOS for this plugin please? https://docs.flutter.dev/packages-and-plugins/developing-packages#shared-ios-and-macos-implementations

Hello, Alex!
Sorry for taking too long on replying you.
Nice! I will create it ;)


It only downloads a 1kb mp3 file

I tried the script hoping to fix the redirection issue, but when it downloads the mp3 file, this is a 1kb file.

Hey there! I'll investigate on it right now and fix this bug.

Hello there, Upon testing, doing right click save as seemed to have worked correctly. Could you link to me the album or game and track you've tried to download?

Sure, here it is :)
https://downloads.khinsider.com/game-soundtracks/album/starcraft-2-wings-of-liberty-gamerip
It works if you, indeed, use right-click on the save button and click on Save target as, but clicking the icon does what I said earlier.

Oh right, so the icon is not downloading or is downloading an incorrect file correct?
That could be caused by cross site content policy. I could make a patch for it that uses elevated tampermonkey requesting privileges if needed (like i do for my other scripts).

Is downloading an incorrect file, that's right.
Surely is trying to download an mp3 file, but as I said, it's only 1kb.

https://github.com/realcoloride/khinsiderantiredirect/assets/108619637/d24b1d37-92b5-4925-b178-bd58b4a5f114
Hey there. Here is a short demo of what it looks like now. I will do some QA and bug testing and i'll publish.

Excellent! Thank you very much, very glad the script is very much alive :)

No problem! It is a pleasure to make these scripts.
I have commited https://github.com/realcoloride/khinsiderantiredirect/commit/e0b599f3cb405c1335da1a797e96146c26d90702. Please download the version and test it out and let me know if it works correctly for you.

Yes, now it's working properly.
Once again, thank you very much <3

No problem! Thanks for the contribution to the project. Make sure to star this repository if you like it and share it!
Feel also free to check my other scripts here.
The update will be pushed and ready to use for others soon. If you have other questions or another problem to report, feel free to re-open an issue.


Support Guillemets

According to the Wikipedia, guillemets are used to indicate book or album
titles.
Reference:
https://en.wikipedia.org/wiki/Guillemet#Uses

Whoops, sorry for the noise.


Mathjax appears to the right

This is the math, but it appears to be a lot to the right...Like something is pushing it...
I added this:
MathJax.Hub.Config({         tex2jax: {         inlineMath: [['$','$'], ['\\(','\\)']],         skipTags: ["script","noscript","style","textarea"]},         displayAlign: "left"});, but it didn't work so I added this too:
.math {             margin-left: -50px;         } but with no success

Please share a link to a live, minimal example that exhibits the problem.

Closed as abandoned.


lint: Fix violation in the GitHub webhook view.

This was apparently making master's backend tests fail. Now everything should be alright.

Nice, thanks for fixing tonight @YagoGG!!


otc-auth: init at 2.0.3

Description of changes

Open Source CLI for the Open Telekom Cloud written in go.
GitHub
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Please make the "add yourself to maintainers" and "new package" parts into two separate commits like this:

maintainers: add fabius
otc-auth: init at 2.0.3

Apart from that, I think you're good to go!

Done, thanks!

bump, can this be merged?


Update mdo-portal-permissions.md

Align capitalisation of product names to user interfaces in Entra ID

@microsoft-github-policy-service agree


Add an optional rootElementProps prop

This PR aims to add support for props that are assigned to the root element.
In my particular case, I would need to add support for the id property. My use-case is that I use the Intersection Observer API to measure the offset relative to some parent, and the parent is identified using the id property. In practice, this is used to lazily load content (for example images) within a scrollable container, which is a fairly common use-case.
The prepublish and build:docs scripts run just fine but I wasn't able to run the tests locally, seemingly because of issues with the webpack configuration. If I can get some pointers on that, I'll happily add a few test cases as well.
This is the output I get when attempting to run the tests:
 rc-scrollbars git:(support-htmldivelement-props) ✗ yarn test
yarn run v1.22.4
$ cross-env NODE_ENV=test karma start
WebpackOptionsValidationError: Invalid configuration object. Webpack has been initialised using a configuration object that does not match the API schema.
 - configuration.module has an unknown property 'loaders'. These properties are valid:
   object { exprContextCritical?, exprContextRecursive?, exprContextRegExp?, exprContextRequest?, noParse?, rules?, defaultRules?, unknownContextCritical?, unknownContextRecursive?, unknownContextRegExp?, unknownContextRequest?, unsafeCache?, wrappedContextCritical?, wrappedContextRecursive?, wrappedContextRegExp?, strictExportPresence?, strictThisContextOnImports? }
   -> Options affecting the normal modules (`NormalModuleFactory`).
    at webpack (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/webpack/lib/webpack.js:23:9)
    at Plugin (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/karma-webpack/dist/karma-webpack.js:128:16)
    at invoke (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:75:15)
    at Array.instantiate (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:59:20)
    at get (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:48:43)
    at /Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:71:14
    at Array.map (<anonymous>)
    at Array.invoke (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:70:31)
    at Injector.get (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:48:43)
    at instantiatePreprocessor (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/karma/lib/preprocessor.js:84:20)
25 02 2021 11:00:42.442:ERROR [preprocess]: Can not load "webpack"!
  WebpackOptionsValidationError: Invalid configuration object. Webpack has been initialised using a configuration object that does not match the API schema.
 - configuration.module has an unknown property 'loaders'. These properties are valid:
   object { exprContextCritical?, exprContextRecursive?, exprContextRegExp?, exprContextRequest?, noParse?, rules?, defaultRules?, unknownContextCritical?, unknownContextRecursive?, unknownContextRegExp?, unknownContextRequest?, unsafeCache?, wrappedContextCritical?, wrappedContextRecursive?, wrappedContextRegExp?, strictExportPresence?, strictThisContextOnImports? }
   -> Options affecting the normal modules (`NormalModuleFactory`).
    at webpack (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/webpack/lib/webpack.js:23:9)
    at Plugin (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/karma-webpack/dist/karma-webpack.js:128:16)
    at invoke (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:75:15)
    at Array.instantiate (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:59:20)
    at get (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:48:43)
    at /Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:71:14
    at Array.map (<anonymous>)
    at Array.invoke (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:70:31)
    at Injector.get (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/di/lib/injector.js:48:43)
    at instantiatePreprocessor (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/karma/lib/preprocessor.js:84:20)

START:
25 02 2021 11:00:42.476:ERROR [karma-server]: Error during file loading or preprocessing
TypeError: process is not a function
    at executeProcessor (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/karma/lib/preprocessor.js:47:11)
    at runProcessors (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/karma/lib/preprocessor.js:60:23)
    at FileList.preprocess [as _preprocess] (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/karma/lib/preprocessor.js:134:11)
25 02 2021 11:00:42.477:INFO [karma-server]: Karma v5.2.3 server started at http://localhost:9876/
25 02 2021 11:00:42.477:INFO [launcher]: Launching browsers Chrome with concurrency unlimited
25 02 2021 11:00:42.478:ERROR [karma-server]: Error: Found 1 load error
    at Server.webServer.listen (/Users/zakke/sources/mafy/rc-scrollbars/node_modules/karma/lib/server.js:213:27)
    at Object.onceWrapper (events.js:286:20)
    at Server.emit (events.js:203:15)
    at emitListeningNT (net.js:1314:10)
    at process._tickCallback (internal/process/next_tick.js:63:19)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Updated PR to include only id prop after discussion in the related issue.


Avatar Proxy allows requesting any resource from literally every origin

Description
I just noticed that the avatar proxy allows proxying/loading literally any request - no matter the original ressource's origin or filetype.
Version
2.0.0
Steps to Reproduce
Try requesting (e. g.):
https://{YOUR-JELLYSEER-HOST}/avatarproxy/https://ash-speed.hetzner.com/100MB.bin
Screenshots
No response
Logs
No response
Platform
desktop
Device
MacBook Pro (doesn't matter tho)
Operating System
macOS 15.0.1
Browser
Zen Browser (Firefox)
Additional Context
The avatar proxy should only allow loading ressources from the configured Jellyfin host (or it's external URL) and maybe also check MIME types
Code of Conduct

[X] I agree to follow Jellyseerr's Code of Conduct

Hi, same as https://github.com/Fallenbagel/jellyseerr/issues/1012

Hi, yep I'm fixing it. A patch will be available very soon

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

v2.0.1
GitHub release

Your semantic-release bot :package::rocket:


Feature/add tfvarspaths spec2

New PR without conflicts.
Fix of PR #463
Fix Issue #338
Copy of old PR comment:
With this PR, we can add more .tfvars files to the Terraform object.
those files are only files located in the Terraform.spec.sourceRef and the path is relative to its root.

Also, I upgraded the docker files base image to version 3.16.2 in order for it to build correctly.

Works the same as terraform's -var-file: https://developer.hashicorp.com/terraform/language/values/variables#variable-definition-precedence
Example:
apiVersion: infra.contrib.fluxcd.io/v1alpha1
kind: Terraform
metadata:
  name:
spec:
    path: deployments/eks/platform
    fileMappings: # File mappings will also be in the tfVarsFiles
        - location: "workspace"
           path: "another.tfvars"
           secretRef:
               name: ...
               key: ...
    tfVarsFiles: # This is the new field. It holds the precedence + paths
        - deployments/eks/platform/default.tfvars # From sourceRef
        - deployments/eks/platform/another.tfvars # From fileMappings
        - environments/platform/env.tfvars # From sourceRef
    sourceRef: # GitRepository as source
  	kind: GitRepository
        name: helloworld
        namespace: flux-system

Thank you, I'll try to review in the following day. I'm a bit busy these days, but I'll try to finish as soon as possible.

@svengreb FYI
@yitsushi Thanks for reviewing this!

Thanks both of you, for implementing this feature and picking the review back up so quickly 🚀

Hi guys,
Is this feature already available?
It will be very useful

It's available on main, but we don't have a release yet that contains this patch.

Thanks @yitsushi


Add --pypi to pixi add subcommand

We want the users to have the ability to add pypi-dependencies to the pixi.toml.
pixi add --pypi haystack[inference]
# Result: haystack = { version = "==x.y.z", extra = ["inference"] }
pixi add --pypi "pytest==7.4.1"
# Result: pytest = "==7.4.1"
pixi add --pypi numpy
# Result: numpy = "==x.y.z"

For nice UX, do we want add to follow on to pypi if no matches found in conda-forge or similar? For example, for haystack. Or be explicit? (currently i believe we have to explicitly add to pypi-dependencies in pixi.toml...

I think you are right that that would be very helpful, but first lets go the explicit route and then see how we can make it more user friendly. Especially while the PyPI integration is in alpha mode.

@rahuldave I came here to say the same thing. I'm kinda lazy and just wanna "pixi add" without config stuff. @ruben-arts might have a good point on wanting the dust to settle first.

I think "pixi add -pypi" is a great start! I am a bit confused about resolution though. Dependencies for a pypi install are pypi satisfied or fail , or will the resolution go back to conda channels?


[atlmfc] build failure

Host Environment

OS: Windows 10
Compiler: VS2019

To Reproduce
Steps to reproduce the behavior:
./vcpkg install atlmfc:x64-windows
Failure logs
C:\dev\vcpkg>vcpkg.exe install atlmfc:x64-windows                                                                       Computing installation plan...                                                                                10:57The following packages will be built and installed:                                                                         atlmfc[core]:x64-windows -> 0#1                                                                                Detecting compiler hash for triplet x64-windows...                                                                      Restored 0 package(s) from C:\Users\Sergey\AppData\Local\vcpkg\archives in 391.3 us. Use --debug to see more details.   Installing 1/1 atlmfc:x64-windows...                                                                                Building atlmfc[core]:x64-windows...                                                                                CMake Error at ports/atlmfc/portfile.cmake:7 (message):                                                                   Unable to locate 'afxres.h'.  Ensure you have installed the ATL/MFC                                                     component of Visual Studio.                                                                                Call Stack (most recent call first):                                                                                scripts/ports.cmake:147 (include)
stdout-x64-windows.log

Could you please make sure you downloaded the install ATL/MFC component?  :)

See issue https://github.com/microsoft/vcpkg/issues/10460

It turns out that you should have installed ATL/MFC for Visual Studio Build Tools, not for Visual Studio Community.


what if noise of Microsoft.Network/networkInterfaces and Microsoft.Network/privateEndpoints

Describe the noise
Resource type (i.e. Microsoft.Storage/storageAccounts)
Microsoft.Network/networkInterfaces@2022-01-01
Microsoft.Network/privateEndpoints@2022-01-01
apiVersion (i.e. 2019-04-01)
2022-01-01
Client (PowerShell, Azure CLI, or API)
az cli
Relevant ARM Template code (we only need the resource object for the above resourceType and apiVersion, but if it's easier you can include the entire template
#1
param networkInterfaces_yangshenvm0012632_name string = 'yangshenvm0012632'
param publicIPAddresses_yangshenvm0012_ip_externalid string = '/subscriptions/79ed831f-c7b8-402e-a226-de8aa5f4764a/resourceGroups/yangshentest001/providers/Microsoft.Network/publicIPAddresses/yangshenvm0012-ip'
param virtualNetworks_yangshenrg001_vnet_externalid string = '/subscriptions/79ed831f-c7b8-402e-a226-de8aa5f4764a/resourceGroups/yangshenrg001/providers/Microsoft.Network/virtualNetworks/yangshenrg001-vnet'
resource networkInterfaces_yangshenvm0012632_name_resource 'Microsoft.Network/networkInterfaces@2022-01-01' = {
name: networkInterfaces_yangshenvm0012632_name
location: 'eastus'
kind: 'Regular'
properties: {
ipConfigurations: [
{
name: 'ipconfig1'
properties: {
privateIPAddress: '<IP_ADDRESS>'
privateIPAllocationMethod: 'Dynamic'
publicIPAddress: {
id: publicIPAddresses_yangshenvm0012_ip_externalid
}
subnet: {
id: '${virtualNetworks_yangshenrg001_vnet_externalid}/subnets/default'
}
primary: true
privateIPAddressVersion: 'IPv4'
}
}
]
allowPort25Out: true
dnsSettings: {
dnsServers: []
}
enableAcceleratedNetworking: true
enableIPForwarding: false
}
}
#2
param privateEndpoints_yangshenpe001_name string = 'yangshenpe001'
param storageAccounts_yangshendls005_externalid string = '/subscriptions/79ed831f-c7b8-402e-a226-de8aa5f4764a/resourceGroups/yangshentest001/providers/Microsoft.Storage/storageAccounts/yangshendls005'
param virtualNetworks_yangshentest001_vnet_externalid string = '/subscriptions/79ed831f-c7b8-402e-a226-de8aa5f4764a/resourceGroups/yangshentest001/providers/Microsoft.Network/virtualNetworks/yangshentest001-vnet'
param privateDnsZones_privatelink_blob_core_windows_net_externalid string = '/subscriptions/79ed831f-c7b8-402e-a226-de8aa5f4764a/resourceGroups/yangshentest001/providers/Microsoft.Network/privateDnsZones/privatelink.blob.core.windows.net'
resource privateEndpoints_yangshenpe001_name_resource 'Microsoft.Network/privateEndpoints@2022-01-01' = {
name: privateEndpoints_yangshenpe001_name
location: 'eastus'
properties: {
privateLinkServiceConnections: [
{
name: privateEndpoints_yangshenpe001_name
properties: {
privateLinkServiceId: storageAccounts_yangshendls005_externalid
groupIds: [
'blob'
]
privateLinkServiceConnectionState: {
status: 'Approved'
description: 'Auto-Approved'
actionsRequired: 'None'
}
}
}
]
manualPrivateLinkServiceConnections: []
subnet: {
id: '${virtualNetworks_yangshentest001_vnet_externalid}/subnets/default'
}
customDnsConfigs: []
}
}
resource privateEndpoints_yangshenpe001_name_default 'Microsoft.Network/privateEndpoints/privateDnsZoneGroups@2022-01-01' = {
parent: privateEndpoints_yangshenpe001_name_resource
name: 'default'
properties: {
privateDnsZoneConfigs: [
{
name: 'privatelink-blob-core-windows-net'
properties: {
privateDnsZoneId: privateDnsZones_privatelink_blob_core_windows_net_externalid
}
}
]
}
}
Expected response (i.e. "I expected no noise since the template has not been modified since the resources were deployed)
It showed deletion on fields but the deployment won't delete them.
Current (noisy) response (either include a screenshot of the what-if output, or copy/paste the text)

Additional context
Add any other context about the problem here.

Any update about the PrivateDNSzoneGroup noise which comes for any private endpoint. Its very difficult to proceed with What-If output since if we have 15 PE then for 15 times this modification would be displayed. Please provide us some update on the ETA to fix this.

I have the same issue. It would be nice to be able to somehow state that this is a false positive somewhere.
 ~ Microsoft.Network/networkInterfaces/nic02 [2022-07-01]     - kind:                                                                      "Regular"     - properties.allowPort25Out:                                                 true     + properties.auxiliaryMode:                                                  "None"

Same problem. Every What-If deployment shows that my Microsoft.Network/networkInterfaces kind and properties.allowPort25Out values will be deleted. These seem like internal properties, and there is no way to set them using a template.


YARD-style directives

Is your feature request related to a problem? Please describe.
YARD doesn't provide any way to ignore comments. So a common case of wrapping a method in rubocop: disable...rubocop: enable ends up with the directive in the docs.
Describe the solution you'd like
I propose to add YARD-style directive syntax.
# Method docs go here.
# @rubocop disable Metrics/PerceivedComplexity
def create_apple_pie_from_scratch
  # ...
end
# @rubocop enable Metrics/PerceivedComplexity

This way YARD would treat it as a tag. By default the tag and its content wouldn't rendered.
We can keep the colon. It would be enough to only add an optional @ before rubocop to make it work.
Considered alternative
# @rubocop
#   rubocop: disable Metrics/PerceivedComplexity

This seem to work but looks like an obvious hack.

Although I find this idea reasonable, I want to mention a couple more alternatives for your consideration.

Disable above the documentation comment.

# rubocop disable Metrics/PerceivedComplexity

# Method docs go here.
def create_apple_pie_from_scratch
  # ...
end
# rubocop enable Metrics/PerceivedComplexity

Use an end-of-line comment to disable.

# Method docs go here.
def create_apple_pie_from_scratch # rubocop disable Metrics/PerceivedComplexity
  # ...
end

Each of these has its own drawbacks, but maybe they are good enough to motivate not changing anything in the implementation of disable comments?

Update: Apparently, Style/CommentedKeyword doesn't allow a space after colon. So even within RuboCop alone there's no coherent concept of a directive. 😔


Create reusable schemas for complex types

I would like to define only once an enum or an object and reuse it as a type.
Example
{
  "@context": "dtmi:dtdl:context;2",
  "@id": "dtmi:loglevel;1",
  "@type":"???ComplexType???",
  "schema": {
    "@type": "Enum",
    "valueSchema": "integer",
    "enumValues": [
      {
        "displayName": "Trace",
        "enumValue": 0,
        "name": "Trace"
      },
      {
        "displayName": "Debug",
        "enumValue": 1,
        "name": "Debug"
      },
      {
        "displayName": "Information",
        "enumValue": 2,
        "name": "Information"
      },
      {
        "displayName": "Warning",
        "enumValue": 3,
        "name": "Warning"
      },
      {
        "displayName": "Error",
        "enumValue": 4,
        "name": "Error"
      },
      {
        "displayName": "Critical",
        "enumValue": 5,
        "name": "Critical"
      }
    ]
  }
},

And then have an interface that uses the enum:
Example:
  {
    "@context": "dtmi:dtdl:context;2",
    "@id": "dtmi:devicehealth;1",
    "@type": "Interface",
    "displayName": {
      "en": "Device Health"
    },
    "contents": [
      {
        "@type": "Property",
        "displayName": "Log Level",
        "name": "LogLevel",
        "schema": "dtmi:loglevel;1"
    },
      {
        "@type": "Telemetry",
        "displayName": "Trace",
        "name": "trace",
            "schema": {
              "@type": "Object",
              "fields": [
                {
                  "name": "TimeStamp",
                  "schema": "dateTime"
                },
                {
                  "name": "level",
                  "schema": "dtmi:loglevel;1"
                },
    },

Thanks for bringing up this issue. Currently, DTDL v2 supports reusable types within an interface definition (by using the schemas property on Interface). See this section in the DTDL v2 documentation: https://github.com/Azure/opendigitaltwins-dtdl/blob/master/DTDL/v2/dtdlv2.md#interface-schemas.
However, this does not support "globally" reusable types or types defined outside of any interface, which is what you may be asking for. This is a feature we are interested in providing. We're not quite ready to share a design proposal on this, so I will leave this issue open until we get a proposal together to share.

Theoretically, the schemas are reusable, but actually they are not. When I tried to use it with Azure Digital Twins, my models cannot be uploaded because it says that the Enum Schema is defined in another interface. So they propose to copy the schema in every interface that needs it.
This is not useful like that, it is necessary to allow the creation of global schemas that could be reused by all the interfaces or schemas available to the descendants.

Thanks for bringing up this issue. Currently, DTDL v2 supports reusable types within an interface definition (by using the schemas property on Interface). See this section in the DTDL v2 documentation: https://github.com/Azure/opendigitaltwins-dtdl/blob/master/DTDL/v2/dtdlv2.md#interface-schemas.
However, this does not support "globally" reusable types or types defined outside of any interface, which is what you may be asking for. This is a feature we are interested in providing. We're not quite ready to share a design proposal on this, so I will leave this issue open until we get a proposal together to share.

Any update on this?

Wow, this is an old thread!  I'm sorry to report that DTDL v3, which was released earlier this year, does not include this feature.  It is a fairly straightforward addition to the language, but it requires extensive service changes to support storing, indexing, and retrieving definitions that are not Interfaces, which breaks a lot of assumptions throughout service code.  This is recognized as an important use case, but it unfortunately did not make the cut relative to other service features that were in demand.


Spinner broken after settings display

Software Versions

Python: 3.9.7
OS: RedHat 8u2
Kivy: 2.1.0
Kivy installation method: pip install kivy[base]

Describe the bug
I'm using the demo/showcase provided in kivy-example.
The ActionSpinner is working fine until I click F1 to get the Kivy settings and close.
Then the ActionSpinner is not refreshed anymore when selecting another item from the DropDown list
I cannot repeat the issue with a Python 3.6.8.
Expected behavior
The ActionSpinner should still be working after the Kivy settings are displayed.
To Reproduce
the kivy-examples demo/showcase

I'm hitting this too with kivy 2.2.1.  Is forcing me to write my own settings panel.

Fixed via #8535


doc: change mailing list references to github

Do we want to completely remove references to the mailing list?

Do we want to completely remove references to the mailing list?

Good question. Imho, the answer is yes. Existing mailing list users can continue to use the list, but especially new users should probably be directed to GitHub.
(just for clarity, this PR replaces all references to the mailing list except for 1 reference in configure.ac, which requires an email address)


Registration

add registration process with :

Email as username
password
email verification process

And add profile picture stuff

Done


Orchestrator server doesn't start Using installation script

Steps
use installation script https://github.com/zero-os/0-orchestrator/blob/master/scripts/install-orchestrator.sh to install orchestrator inside js9 docker
Issue
After installation finished ,orchestrator server didn't start as shown

software version
master : 851d41fa


[WFLY-8418] Enhance the way licenses are presented and fix inconsistencies

First part - fix inconsistencies in licenses.xml and dependencies in feature-pack/pom.xml
issue: https://issues.jboss.org/browse/WFLY-8418

retest this please

retest this please


[Bug] Param "outFileName" doesn't work

When I'm running the command:
npx fetch-appwrite-types outDir ./src/lib/database outFileName \"appwrite.d\" hardTypes
it just creates the file but writes types to the default appwrite.ts file in a specified directory.

Hello @cogscides,
Found the issue, will release a fix this day.

That should be fixed in 1.8.1. Thanks for the feedback!


Canvas vs HTML+CSS

Have you considered using HTML+CSS to render data instead of canvas? Canvas has the downside that it doesn't scale well, and ends up looking a little fuzzy. I have a working demo for the renderer40k.ts, happy to clean it up and send a pull request if you'd like to take a look. LMK

Yes. I just started to move the 40k renderer to straight html.  I have already redone the Warcry and Age of Sigmar renderers.
For 40k my plan is mimic the look and feel of the 9th edition codex.

Merged and pushed the changes to the main release.  Thank you for your work on this.


chore(sequencere_infra): add component client trait for local client

Stack:

#2079
#2078 ⬅

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is


Update package's declared licensing values

The scancode package model has changed to include declared_license from declared_licensing.
ABCM should be updated to handle this new value.

#319 Handles the initial changes. Will update the package model itself on the next schema-change milestone.


Refactor (nano conventions, query helpers, views)

Checklist

[x] Tick to sign-off your agreement to the Developer Certificate of Origin (DCO) 1.1
[x] Added tests for code changes or test/build only changes
[x] Updated the change log file (CHANGES.md|CHANGELOG.md) or test/build only changes
[x] Completed the PR template below:

Description
Hello, this PR contains a substantial redesign and rewrite of the package. As such I understand if it's seen as too opinionated, yet I'd like the PR to be considered and welcome suggestions please.
Changes come from the forked repo of the master barshociaj branch - release v0.3.3.
They need to be split into two PRs:

This PR - main code without 3 additional examples (just with fixed references to cloudant-labs instead of barshociaj)
Once released, I'd raise another PR containing the rest of the examples referencing the new cloudant-labs release.

Approach
The fork I'd like to merge has been used in a production IBM SaaS application for several months now. It contains:

New naming conventions, closer to the popular NodeJS nano library
Refactored query helpers
Added convenience (optional) interface implementation and new examples of usage (examples will follow in subsequent PR)
Added support for views
Bug fix of a leak when running with multiple Cloudant instances

Schema & API Changes
File name changes

follow.go to changes.go
database.go to dbs.go & doc.go & changes.go & view.go (moved functions by owning concept)
alldbs_query & test to dbs_query - aligns with dbs.go
get_query & tests to doc_query - aligns with doc.go
alldocs_query to view_query - because alldocs is a view

API name changes

cloudant.CreateClient & cloudant.CreateClientWithRetry to cloudant.NewClient
client.GetOrCreate to client.Use & client.UseOrCreate - per nano convention
db.Set to db.Insert - per nano convention
db.Delete to db.Destroy - per nano convention
db.All to db.List - per nano convention

Client struct changes

Renamed CouchClient to Client
cloudant.NewClient accepts functional options

E.g. cloudant.ClientConcurrency(20), cloudant.ClientRetryCountMax(5), cloudant.ClientRetryDelayMin(10), cloudant.ClientRetryDelayMax(60)

Query helper changes

Switched all queries to output URLValues and refactored them from interface to a struct helper

Removes the need to .Build() query, yet keeps the support for convenient chaining
Note I considered merging all query options together and leave the choice to the developer as nano library does, but at the end realized that the original concept of a query per type is stronger. The URLValues concept now gives the necessary freedom to developers.

Repurposed/renamed cloudant.NewAllDocsQuery to cloudant.NewViewQuery

Extended the helper according to Cloudant specification with EndKeyDocID, Group, GroupLevel, Reduce, Stable, Stale, StaleUpdateAfter, StartKeyDocID, DoNotUpdate, UpdateLazy

Renamed cloudant.NewAllDBsQuery to cloudant.NewDBsQuery
Renamed cloudant.NewGetQuery to cloudant.NewDocQuery

Additions

client.Info - retrieve database info
db.InsertEscaped - see Security section
db.InsertRaw - enables inserting design documents or otherwise use raw JSON
db.View - enable querying views
db.ViewRaw - enable querying custom views
cloudant.NewViewQuery - new query helper to support views

Bug fixes

pool.go - resolved a leak cross Cloudant instances (added workerFunc to the w *worker scope)

Security and Privacy
Switch to unescaped HTML processing

GO by default escapes HTML when marshalling JSON which caused a major incompatibility. This behaviour is too opinionated, the PR uses unescaped HTML by default - as nano does - and adds the option to escape HTML using db.InsertEscaped

Testing
Travis is fixed by using official couch db docker instead of the couchdb Travis extension. This is because Travis extension enforces admin mode and we don't support it.
Added new optional interface (interface/*)

Includes convenient mock functions for quick local testing without Cloudant

Extended examples to follow in subsequent PR (examples/*)

Added simple API example and showed usage with and without interface

Modified existing tests to suit new API names. Added testing for new functionality (view_test, interface/*, examples/*)
Monitoring and Logging
No change
Documentation
Testing instructions are moved to CONTRIBUTING.md. This separates consumers from contributors.
Major rewrite of README. Reflects new API endpoints and usage.
Expanded or modified many comments in the code

Thanks for opening this, but we're not going to accept API changes here. The reason is that our plans for Go are oriented around producing an IBM Cloud style SDK, to replace this best-effort library and move to an officially supported status. These changes will be coming soon to https://github.com/ibm/cloudant-go-sdk which you can follow for updates. As you can imagine we are reluctant to ask people to change APIs and doing so twice in quick succession would be even more painful.
I guess you are already running a fork so hopefully your effort has not been wasted.

Thanks for the quick answer, no worries, makes sense. We'll continue using the fork until the SDK is production ready. Looking forward to the upcoming Cloudant upgrades too.


[BUG] Slider and RangeSlider compressed to a point when align=center in a dmc.Stack.

If you try to center align a Slider or RangeSlider within a dmc.Stack, it will be completely compressed to a single point and is not usable. The issue doesn't arise with other align types. Here is an example:
import dash_mantine_components as dmc
from dash import callback, Output, Input, Dash

app = Dash(
    __name__,
)

app.layout = dmc.Stack(
    [
        dmc.RangeSlider(
            id="range-slider-callback",
            value=[26, 50],
            marks=[
                {"value": 20, "label": "20%"},
                {"value": 50, "label": "50%"},
                {"value": 80, "label": "80%"},
            ],
            mb=35,
        ),
        dmc.Text(id="range-slider-output"),
    ],
    align="center"
)

@callback(
    Output("range-slider-output", "children"), Input("range-slider-callback", "value")
)
def update_value(value):
    return f"You have selected: [{value[0]}, {value[1]}]"

if __name__ == "__main__":
    app.run_server(debug=True)

This is how the upstream Mantine library works. You should take car that the RangeSlider has a width, e.g. by supplying a specific width (like 20rem, 300px or 100% of the container).


Exponential lag on auto-updating dashboards

Starting with the 3.x releases of grafana and zabbix-grafana plugin, running zabbix continuously has become impossible for me. Setting up simple charts monitoring a couple interfaces is enough to crash the browser after a few minutes.
The setup is simple. setup a chart watching a couple network interfaces. Set the auto refresh to 1 minute. The first few refreshes are fine, but after 10 minutes things start to slow down. After 15 minutes, the reload time for the page can exceed a full minute. The browser pegs the CPU, and the fan kick in.
I have tested this on safari, chrome, and firefox on both OS X and windows. Same results.
running grafana beta 3, plugin b5, zabbix 3.0.1
New dashboard-1460490084828.txt

I'll try to reproduce this bug, but would be great if you provide some info like performance profiling results from browser.

Hi, i have the same issue, that is preventing me from moving to Grafana 3.
Zabbix 2.4.3
grafana-zabbix beta 3
Grafana 3.0.0-beta1
Here is some profiling, i hope it helps. if not please contact me
CPU-20160413T112130.zip

Thanks!

@vpuech Do you see graph from Zabbix or dashboard can't load?

I see that v.onload function take most of time. Strange, it's Grafana function. Can you try another datasource (not Zabbix)?

@alexanderzobnin Yes, everything seams to work just fine except the lag.
i will try to test without zabbix datasource. i'll keep you posted

Added multiple graph from Elasticsearch and no issue so far

Ok, I see the same problem. Now I trying to collect CPU and memory profiles for detailed analyzing.

It's a great news, if you can reproduce the issue. Thanks you for you're great work. poke me if you need any help.

Can you get your full Grafana version (with commit hash - it's available at login page)?

@alexanderzobnin : Grafana version: 3.0.0-beta2, commit: v3.0-beta2, build date: 2016-04-04 13:52:41

Ok, thanks.

Seems, I found a reason... API response caching - cache grows and insert time also grows. I will try to fix it.

Thank you very much Alexander

Fix available in version 3.0.0-beta6. I've sent PR to grafana plugin repo so it will be available via grafana-cli soon. Or check out it from https://github.com/alexanderzobnin/grafana-zabbix-app

Now CPU profiles looks much better :+1:

Awesome! Thanks a ton!
On Wed, Apr 13, 2016 at 1:59 PM, Alexander Zobnin<EMAIL_ADDRESS>wrote:

Now CPU profiles looks much better [image: :+1:]
[image: screenshot from 2016-04-13 23 58 00]
https://cloud.githubusercontent.com/assets/4932851/14509168/a0aef422-01d3-11e6-8c6a-f48235a8a30a.png
[image: screenshot from 2016-04-13 23 57 52]
https://cloud.githubusercontent.com/assets/4932851/14509172/a34e0c54-01d3-11e6-8583-e520a7b27b27.png
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/alexanderzobnin/grafana-zabbix/issues/196#issuecomment-209645369


PR: Wait for autoformat before returning on 'save' method (Editor)

Description of Changes

Since the autoformat option can take time we need to wait before returning on the editor save method to prevent lossing changes when closing an unsaved file
Issue(s) Resolved

Fixes #17836
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: dalthviz

Hello @dalthviz! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/editor/widgets/tests/conftest.py:

Line 96:5: E303 too many blank lines (2)

I think the test on conda are failing due to the PyQt 5.15.4 release on conda-forge. Maybe we are missing https://anaconda.org/conda-forge/pyqtwebengine as a dependency for the tests?


PR: Remove if/else/try/for statements from the outline explorer tree

Fixes #5821

[x] Fixes issue in the code
[ ] Write tests

The goal of this PR is to prevent if/else/try/for statements from showing in the outline explorer tree.

For comparison, here is a screenshot for the same file before the PR:

Hello @jnsebgosselin! Thanks for updating the PR.

In the file spyder/widgets/editortools.py, following are the PEP8 issues :

Line 395:1: W293 blank line contains whitespace

@ccordoba12 Ok, I added the blank spaces back, sorry for that.
I just need to write a test now. I would like to add a docstring to the method populate_branch also. Do you have a suggestion?

Do you have a suggestion?

Nop. Whatever you decided is fine.

@ccordoba12 I've tested this change for a few days and I like it. The only thing though is I'm missing when if __name__ == '__main__':  was displayed in the outline explorer tree. I'm tempted to let this one pass through. What do you think?
Also, I'll try to work on the tests soon.

@ccordoba12 I've tested this change for a few days and I like it. The only thing though is I'm missing when if name == 'main': was displayed in the outline explorer tree. I'm tempted to let this one pass through. What do you think?

Ok forget it, I've changed my mind. If one wants a link to if __name__ == '__main__': , just add a # ---- comment before it. I'll start working on the tests now.

Ok, great, so just the test is missing :-)

@jnsebgosselin, is this ready?

Yes, it is ready.


Compatibility question.

I cant seem to find any post on how well this (awesome) plugin works on ie9?

Don't expect much from IE9 support here. If you need IE 9 support you may use Swiper 2.x branch

Thanks. Yeah I ended up just having no slideshow on ie9, instead just showing the first slide.

A official list of compatible browser would be very much appreciated.

A list of which features in a browser are used would be better. Testing against browsers is not a solid style.

This features should be supported for full Swiper support: CSS Flexbox, CSS 3D Transforms, CSS Transitions, JS classList


update website photo to a 2021 photo

pray 4 me not to have cache invalidation issues in my code that show people the old picture forever

(success!)


Add options to pin setuptools  version used

Some packages rely on older versions of setuptools to be built. Using whatever default version we pick up can thus lead to build failures.
We now exposes a setuptools_version option to allow users to pin the specific version they need.

It seems there is an issue with the Linux version. I'm looking into it right now.

It seems there is an issue with the Linux version. I'm looking into it right now.

Turns out this was something unrelated in my test setup.


Try to setup URL redirection for disappearing server nmag.soton.ac.uk

But to where?
For now, it would have to be https://fangohr.github.io/nmag.github.io/index.html
Or we could consider a maybe better URL (see #14).

I have put a redirection on the current site (see https://github.com/fangohr/nmag/issues/14#issuecomment-914465690) as a first step.

I suggest we use the following code to redirect the site instead:
 <meta http-equiv="Refresh" content="0; url=https://nmag-project.github.io/index.html" />

I used 0; instead of 60; so that the page would redirect right away instead of waiting a minute to do so.
It would be good to include a message in the body that links to the new site as well so that if a browser doesn't redirect automatically, the user can still find the new site:

The nmag documentation has permanently moved to https://nmag-project.github.io/index.html

...It's worth noting that the best way to do this is with an HTTP Redirect header via the server configuration, but that's not quite as easy. It would help the new site rank higher with search engines, though.

I can update the redirect message (and reduce the interval to 0 seconds) once we have the new site up and running at the final location.
The .htaccess route we can explore, but the machine is likely to be taken down in a few days. More important would be to have a redirect at University level. I'll try for this, but can't promise it will be possible.

The site is up now. The redirect should point to https://nmag-project.github.io/

That's done.
I also added the 'permanently moved' statement.

The redirection takes a long time for me (don't know why) but eventually (after 10 seconds?) works.

I also created a .hcaccess entry but it doesn't seem to be recognised. Maybe we can talk briefly tonight.
$ cat /var/www/.htaccess  Redirect 301 /nmag https://nmag-project.github.io

I also created a .hcaccess entry but it doesn't seem to be recognised. Maybe we can talk briefly tonight.
It's probably not enabled on the server. There would need to be an AllowOverrideAll directive in the base apache config for /nmag.
https://askubuntu.com/questions/429869/is-this-a-correct-way-to-enable-htaccess-in-apache-2-4-7

Yeah, we can talk about it later.

I have requested that the university sets up some forwarding.

Redirection is now in place. :)


Missing .sass-lint.yml file

A rules/config file for the sass linter should be provided.
See here.

Will look at this once I have some time. Probably after VS 2017 upgrade.

I've archived the MVC boilerplate project due to the size and complexity of moving it to ASP.NET Core 2.0 with the latest and greatest web front end stuff like Bootstrap 4, Webpack, Workbox etc. I also had to learn all that stuff so needed time for that. I'm focusing my efforts on the API Boilerplate project and will bring back the MVC project better than ever before once I have the courage to attempt it.


Release 2.2.1

/assign @yevgeny-shnaidman
/cc @TomerNewman
Cherry-picking 2 bug fixes for release-2.2

/override pull-kernel-module-management-check-commits-count
This is a release PR with multiple commits in it. It is designed to have more than 1 commit.

/lgtm


Enlarge click area of scroll legend page icons

What problem does this feature solve?
On mobile, the page buttons on a scroll legend can be quite small and difficult to press. I would like to keep them as the same size, but make them easier for users to press on their phone.
Is there a way to add a click target element overtop of the page buttons to increase the size of the click area? This would help a lot with user experience on mobile.
What does the proposed API look like?
Not sure at the moment. Want to check if it's already possible, but if not I'd be happy to brainstorm some potential ways to do it.

Maybe we should support pageIconPadding. @hughess Are you interested in making a pull request? Please checkout the wiki to learn more.

@Ovilia I can definitely make a pull request at some point. This isn't my top issue at the moment, but I'll come back to this when it is.


Pathways widget not loading

Reactome have updated their API, while the old one has been deprecated: https://reactome.org/about/news/124-deprecated-support-for-reactome-restful-api
This breaks the Pathways widget in production.
This is a webapp frontend issue.

Updated call to Reactome API.


sfPropelORMPlugin doesnt fit in after update symfony1

After updating symfony1 (v1.5.11 => v1.5.15) there is a problem to embed propel/sf-propel-o-r-m-plugin
# ./symfony cc PHP Fatal error:  Cannot declare class XMLElement, because the name is already in use in /var/symfony/vendor/propel/propel1/generator/lib/model/XMLElement.php on line 20
only with >= PHP8.0

If you create a repo for testing, I'll try to figure out what's going on here.

Not sure if relevant, but propel1 (and not necessarily propel/sf-propel-o-r-m-plugin) has limited PHP 8 support. There's some good work for that at https://github.com/tworzenieweb/Propel/tree/feature/php8.0 and I have done some additional changes at https://github.com/mentalstring/Propel/tree/feature/php8.1 for PHP 8.1 and I've been using that with PHP 8.2 with no issues — YMMV.
Having said that, my guess is that this issue isn't specific to symfony1 and maybe should be created in https://github.com/propelorm/sfPropelORMPlugin or https://github.com/propelorm/Propel.
@devmuc Are you still hitting this? Did you try manually clearing the cache folder by removing the files (instead of symfony cc)?

@mentalstring I have never used propel. Do you actively use it? I've looked at your modifications and I see little BC like, but it doesn't seem too bad. Should we bring it under the fos1 organization as well?

While my changes are enough for my needs to use it with PHP 8.2, I don't think it's ready for general availability due to the BC changes (mostly due to strftime() being deprecated, which I can live without).

Should we bring it under the fos1 organization as well?

Would need to see more people interested in this and helping cover the gaps I missed/don't use. That said, strftime() parity seems hard so probably it will never be perfect.

@mentalstring I didn't know there were such problems with propel. I looked at some of the more active forks, but I didn't really see anyone doing anything about this problem. In general, I think this could be solved by making a tool to migrate data that relies on the strftime function.

vendor/bin/phpunit
PHP Fatal error:  Cannot declare class XMLElement, because the name is already in use in /Users/connor/Work/opensource/propel/generator/lib/model/XMLElement.php on line 20

I was able to reproduce it. php 8.3 is indeed problematic with the basic repo.


New version: RealVNC.VNCViewer version 7.11.0

Pull request has been created with komac v2.2.1 :rocket:
Microsoft Reviewers: Open in CodeFlow
Resolves: https://github.com/microsoft/winget-pkgs/issues/148773, https://github.com/microsoft/winget-pkgs/issues/150613, https://github.com/microsoft/winget-pkgs/issues/143419

Hi @vikingnope,
This manifest removes Apps and Features entries that are present in previous PR versions. These entries should be added to this version, to maintain version matching, and prevent the 'upgrade always available' situation with this package.
(Automated response - build 877.)

Verify:

@vikingnope Could you please submit the new version for <IP_ADDRESS>?

@vikingnope Could you please submit the new version for <IP_ADDRESS>?

And for server too?


Admin rights to fstar-mode repo?

I'd like to set up travis integration for the fstar-mode repo. Is that ok? I need admin access to that particular repo to do that though.

You should now have admin rights for that repo.

Thanks!


Example test harness html is misleading.

I understand that the example test harness file is just a starting place but it was confusing to me when trying to make things work for the first time.  A more stripped down version with links to a cdn for the resources might make this example much clearer in where it needs to get its resources from.
I also think it might be broken as I had to use mocha.setup('bdd') instead of mocha.ui. For those of us struggling to get something started being able to copy the example and have it just work would be helpful!

I had to use mocha.setup('bdd') instead of mocha.ui

It's right here and been there for a while.

A more stripped down version with links to a cdn for the resources might make this example much clearer in where it needs to get its resources from.

I don't want to promote a CDN usage when using npm or another package manager makes more sense. You can do that if you want though. That's really the point of an example. It's more of fill in the blank.

For those of us struggling to get something started being able to copy the example and have it just work would be helpful!

The biggest goal of mocha-phantomjs is that it complements your workflow when testing in a regular browser. If it's not working with an example it's because you don't have tests working at all. You start with your browser tests and tack this on. You can also work just directly from the command line from the start of this if you want.


Question - Overriding the Enter Key with the Tab Key

We love the angular ui select plugin.  I have been working on trying to override the enter key functionality with the tab key.  The easiest way was to modify the uiSelect controller itself.  Any other way to do this? (best practice to use maintained bower version than my own version).
https://github.com/angular-ui/ui-select/blob/master/src/uiSelectController.js#L386
Change Line 397
    function _handleDropDownSelection(key) {
          var processed = true;
          switch (key) {
            case KEY.DOWN:
              if (!ctrl.open && ctrl.multiple) ctrl.activate(false, true); //In case its the search input in 'multiple' mode
              else if (ctrl.activeIndex < ctrl.items.length - 1) { ctrl.activeIndex++; }
              break;
            case KEY.UP:
              if (!ctrl.open && ctrl.multiple) ctrl.activate(false, true); //In case its the search input in 'multiple' mode
              else if (ctrl.activeIndex > 0 || (ctrl.search.length === 0 && ctrl.tagging.isActivated && ctrl.activeIndex > -1)) { ctrl.activeIndex--; }
              break;
            case KEY.ENTER:
            case KEY.TAB:
              if (!ctrl.multiple || ctrl.open) ctrl.select(ctrl.items[ctrl.activeIndex], true);
              break;

Can you explain your use case? This sounds like a very bad idea for accessibility reasons.

It was a customer user case.  Thanks for responding! I ended up modifying to my needs. Thanks again for checking in.


Rancher2.2-rc Monitoring won't install in air-gapped environments.

Now that monitoring has been upgraded to more than just a Catalog App/Charts it is still using the rancher catalog to deploy which doesn't work in air-gapped environments.
Even if I add an internal catalog with rancher-monitoring catalog, it won't install correctly as it needs many internal elements. Can this be packaged/documented in a such a way to allow air-gapped installation?

@JohnDavidLarsen could you post the exact RC version you've hit the issue on?

Currently testing 2.2.0-rc4 released yesterday, hadn't tested previous 2.2.0-rc in airgrapped environment.
Currently error I get is
failed to ensure catalog "catalog://?catalog=system-library&template=rancher-monitoring&version=0.0.2" failed to find by catalog by ID "catalog://?catalog=system-library&template=rancher-monitoring&version=0.0.2"
Alternatively if I install through the custom catalog from  https://github.com/rancher/system-charts/tree/dev/charts/rancher-monitoring/v0.0.2
it get stuck giving me the Error: no objects visited" in the log.

HI @JohnDavidLarsen the v2.2.0-rc4 is good for the monitoring feature, you are almost there but there are two more steps to make it happen:

Clone the system-charts repo and run the git server in your air-gapped environment such that Rancher server can reach the repo to use those charts. You need to make sure the master branch exists in your lcoal repo, the link you provided points to dev not master, so it clones the devbranch not master
add a new global catalog named  system-library and set its catalog URL to your local git server.
Save and wait its state to be active.

Now you should be able to enable monitoring.

Thanks I get an error that there are already is a system-library though

@JohnDavidLarsen  Could you try this:

replace <your-server> with your real address and open it in the browser

https://<your-server>/v3/catalogs/system-library

click on edit on the upper right corner -> change the URL in the editing page -> show request -> send request -> follow self link
switch back to your Rancher UI, and now the system-library should show up in the list of catalogs.

Awesome, thanks that worked. So mostly a documentation issue. Glad to know I can customize those core catalogs now.

Possibly related, should I put in another ticket. The Global-DNS system doesn't even appear in the Tools dropdown in the air-gapped system. I'm assuming it couldn't find the library on first boot, but I don't see a way of retriggering it, I've tried to restarting the rancher/rancher container.

global dns is not supported in a couple scenarios:

if you are in a single node rancher server install
youve disabled the local cluster
this is because we need to deploy the controller for it into the local cluster. if we detect youre in one of those setups, we hide the option

It won't really work in an airgap env regardless bc we only support dns providers that are on the public internet at this time

Hi,
I have the same issue and I wonder if there is a way to use monitoring without requiring git server.
After I saw the problem, I disabled monitoring. However, The error message <failed to ensure catalog "catalog://?catalog=system-library&template=rancher-monitoring&version=0.0.2" failed to find by catalog by ID "catalog://?catalog=system-library&template=rancher-monitoring&version=0.0.2"> didn't disappear. How would I get rid of that message?

where is it appearing?

This is showing up on my HA local cluster on the cluster list. Active is in RED but everything else looks ok.

@ekristen when you clicked save did you change any configs? This did not help me

Close in favor of https://github.com/rancher/rancher/issues/20387


[Concurrency] Avoid importing 'async' overloads.

When importing a method as async, look whether it would overload a
synchronous method. If so, append the Async suffix to the base name
to keep it out of the way of the original function.

@swift-ci please test

@swift-ci please test source compatibility

Build failed
Swift Test Linux Platform
Git Sha - 0be8fd2e910f56d1364f08a9a18fb1a5dd3b0a52

@compnerd Windows CI out of disk space

@swift-ci please smoke test Linux

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please smoke test

This experiment did not work out


Merge again and use random keys instead of autoincrement integers

This merges 0.11.0-alpha up again w/ master.
It also adds a change to use a random key for entity keys instead of an auto-increment integer. The auto-increment integer can cause problems with entity keys being always reused if you are serializing content state including the entity map and reloading it.
This doesn't remove the possibility of duplicate keys, but it dramatically lowers the chance, since the first entity created isn't always a 0!

Lemme get rid of this PR and split it up so that its easier to merge.


Update README.md

fix minor grammar errors.
"and of course, as always, no CCU limits ever." -> "and, as always, it has no CCU limits whatsoever." Of course and always seem to mean the same thing in this context.
"Once it was working, we made it so that Unity could use it." -> "Once it was working, we added Unity integration." less verbose.
Remove nominalization. "easy for you to do serialization of data" -> "easy for you to serialize data"
"make the security of your project as secure as you want" -> "make your project as secure as you want"

Looks good to me.


migrate: ValueError: Related model 'filer.Image' cannot be resolved

Hi, I've been following the docs. Everything works fine, I can add and remove files via the admin and the files get added to the media directory. After updating a model, and running $ python manage.py migrate I get this raise ValueError('Related model %r cannot be resolved' % self.remote_field.model) ValueError: Related model 'filer.Image' cannot be resolved
... Just as a side-note, I started this project using Heroku's starter template.
models.py
from django.db import models

from filer.fields.image import FilerImageField

class About(models.Model):
    image = FilerImageField(related_name="billede")

migrations
# -*- coding: utf-8 -*-
# Generated by Django 1.9.12 on 2017-01-06 07:24
from __future__ import unicode_literals

from django.db import migrations
import django.db.models.deletion
import filer.fields.image

class Migration(migrations.Migration):

    dependencies = [
        ('about', '0002_auto_20170105_1049'),
    ]

    operations = [
        migrations.AlterField(
            model_name='about',
            name='image',
            field=filer.fields.image.FilerImageField(on_delete=django.db.models.deletion.CASCADE, related_name='billede', to='filer.Image'),
        ),
    ]

settings.py
INSTALLED_APPS = [
    ...
    'whitenoise.runserver_nostatic',
    'django.contrib.staticfiles',
    'easy_thumbnails',
    'filer',
    'mptt',
    ...
]

STATIC_URL = '/static/'

# Extra places for collectstatic to find static files.
STATICFILES_DIRS = [
        os.path.join(PROJECT_ROOT, 'static'),
]

# Simplified static file serving.
# https://warehouse.python.org/project/whitenoise/
STATICFILES_STORAGE = 'whitenoise.storage.CompressedManifestStaticFilesStorage'

# Media files (user uploaded files)
MEDIA_DIR = os.path.join(BASE_DIR, 'media')
MEDIA_ROOT = MEDIA_DIR
MEDIA_URL = '/media/'

# easy_thumbnails retina support
# https://django-filer.readthedocs.io/en/latest/installation.html#configuration
THUMBNAIL_HIGH_RESOLUTION = True

THUMBNAIL_PROCESSORS = (
    'easy_thumbnails.processors.colorspace',
    'easy_thumbnails.processors.autocrop',
    #'easy_thumbnails.processors.scale_and_crop',
    'filer.thumbnail_processors.scale_and_crop_with_subject_location',
    'easy_thumbnails.processors.filters',
)

dependencies/requirements
$ pip freeze
dj-database-url==0.4.1
Django==1.9.12
django-filer==1.2.5
django-mptt==0.8.7
django-polymorphic==1.0.2
easy-thumbnails==2.3
gunicorn==19.6.0
Pillow==4.0.0
psycopg2==2.6.2
pytz==2016.10
whitenoise==3.2

I realise that this issue is quite old, but for anyone else that lands here after adding django-filer to an existing project:
A workaround for this is to include the most recent django-filer migration in the list of dependencies for your migration that adds the filer-related fields.
dependencies = [
        # ...,
        ('filer', '0007_auto_20161016_1055'),
    ]


drivers: gpio: sifive: cleanup irq initialzation code

Clean up irq initialization code with macro definitions.
Signed-off-by: Song Qiang<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@34a2630). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #13907   +/-   ##
=========================================
  Coverage          ?   52.25%
=========================================
  Files             ?      307
  Lines             ?    45429
  Branches          ?    10506
=========================================
  Hits              ?    23740
  Misses            ?    16899
  Partials          ?     4790

Continue to review full report at Codecov.

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


How to restrict future date.

is it possible to restrict future date?

Just use setMaxDate(Calendar calendar).