Documentation - factories - usage not shown

The section on factories shows two alternatives on how to define a factory:
def taxCalculator(taxBase: Double) = wire[TaxCalculator]
    // or: lazy val taxCalculator = (taxBase: Double) => wire[TaxCalculator]

It does not show though, how a class can depend on such factory.
This answer on SO shows, how to inject a factory defined in a val (or lazy val) - second line in the documentation snippet
However, I have found no way on how to inject a factory, which is defined using a def - first line in the documentation snippet.

Depending on taxBase: Double => TaxCalculator (the function type) doesn't work? If not, than it's a bug.

Am I using it as intended ?
https://gist.github.com/ygor-sk/b8e64fc68627fe9a2930d78829147004

Yes, seems there's a bug or a missing feature :)


How to play the Landscape Video in the MpMovieplayer in Landscape mode.

I take a video in Landscape mode . i am playing the Video using MPMovieplayer in Landscape mode. the problem is video is playing in  another order. how to play the Landscape Video in the MPMovieplayer  Landscape mode .
Thank you

Hi Ajay , have you solved this issue?Please reply

Hi Trupti. i am still working on issue.

Thanks a lot Ajay fir replying . please let me know if u r getting any
solution.
On 06-Dec-2015 2:36 PM, "ajay2c9"<EMAIL_ADDRESS>wrote:

Hi Trupti. i am still working on issue.
—
Reply to this email directly or view it on GitHub
https://github.com/piemonte/PBJVision/issues/296#issuecomment-162289747.

ok

:paperclip: https://github.com/piemonte/PBJVision/issues/84


Is the README.md of sysvad project up to date?

I tried following the instructions in the README.md file, however the solution can not build, in fact huge amount of errors are shown
Could you please take a look and perhaps write a little about the setup of the project before an actual build of solution from inside VS2017

I suggest adding props file that have $(ProjectDir)Debug   in the VC++ Directories --> Include Directories
After adding this to almost all projects, then the solution could build


how to send markdown format with Request::sendMessage

https://sourceforge.net/p/telegram/wiki/markdown_syntax/
  \Longman\TelegramBot\Request::sendMessage([             'chat_id' => (int) $chat_id,             'text'    => $message,         ]);
$message ='
 * this shows up in italics: *a happy day*
* this shows the asterisks: \*a happy day\*
';

Check the API for available parameters:
https://core.telegram.org/bots/api#sendmessage
You need to add 'parse_mode' => 'markdown' to your parameters 👍

Nice


Workshop UI showing contract events

Simple UI on localhost to show a stream of marketplace contract events. Ideas to include:

contract events
uploaded images (moderated) or QR code to show image
stats / metrics

filled slots
started / finished contracts

Potentially show uploaded photos for newly created contracts (will need some way to moderate the images)

https://github.com/codex-storage/ethcc-demo


[Bug]: SQL execution timeout error during database change modification index modification

ODC version
odc424
OB version

What happened?

What did you expect to happen?
Prompt modified to 120 hours
How can we reproduce it (as minimally and precisely as possible)?

Anything else we need to know?
No response
Cloud
No response

Fixed In Commit 94477387e8bad409736b55be42058068ba858c76
------
陈朝俊
Wed Feb 28 19:47:44 2024 +0800

pass


How to use [ and ] characters in cells ?

style_parser.go implements a special feature where [ and ] characters are used to separate styled text.
https://github.com/gizak/termui/blob/f976fe697aa09b747f16fa6e08c36dde5fb16f27/style_parser.go#L19-L20
https://github.com/gizak/termui/blob/f976fe697aa09b747f16fa6e08c36dde5fb16f27/style_parser.go#L105-L112
Which ends up cropping the text:

Is there a way to disable this feature without having to edit the code ?

+1 for this. Maybe we should make [ escape the significance of the [ character.

Ok I finally have a solution! Made a PR https://github.com/gizak/termui/pull/321 and there is no need to \ to escape. I made the logic look for ]( i.e. you have to have a ( to start the style after  the ending ] or it will ignore [ and ].


Add a test for the cmake installation, fix cmake bugs

There were quite a few things missing if someone used CMake to install this library.
This should fix a few of them. I also asked the cmake build script in the CI to install the library and then try to build another library and link against it, so hopefully we can catch more of these issues in the future.
The original CMakeLists file was written by @krasznaa, and he has far more experience with CMake than I do, so I'll wait to see if he has any suggestions.

This is one way of going about it. But if it was me, I would stop using PUBLIC_HEADER completely for the library.
Unfortunately that machinery only works correctly in CMake if all of the headers coming with your library are in the same directory. If you have headers put into multiple sub-directories, your best bet is to make the installation of the headers and the library independent of each other.
So I would remove all the lines having anything to do with PUBLIC_HEADER, and install all the headers with a single
install( DIRECTORY "include/" DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}" )

Notice that since I wrote the configuration for lwtnn, I got into the habit of relying on CMake's own settings for the default include/library/etc. install directories.
https://cmake.org/cmake/help/latest/module/GNUInstallDirs.html
But let's not do too many things at the same time. You can just write:
install( DIRECTORY "include/" DESTINATION "include" )

(Take note of how I use / in that code. I believe this is the correct formalism. But definitely check it yourself as well if things come out as they should...)

Hi @krasznaa, thanks for the quick look. I wasn't sure if there was some particular reason you'd done it that way, but this is indeed much cleaner. Unless you have further comments I'll tag a release from this version.


Collecting ideas

Do you guys prefer A, B, C or keep it as it is when line highlighter is enabled and when multiple lines are selected?
A:

B:

C:

Current:

I will go "C" if I don't get enough response.

Please add right click option

C looks nice to me.

C is more beauty and logical

A, C :T

I will stick with C and closing this thread for now.


zoom-us can't change pulseaudio sinks

Describe the bug
When I am in a Zoom conference call I cannot, using pavucontrol or pacmd change it's sink from my laptop's speakers to my headphones.
To Reproduce

Start Zoom and enter a meeting
Connect headphones
Attempt to set Zoom's sink using pavucontrol

Expected behavior
I'd expect to be able to move Zoom across sinks, like I can any other application (Firefox, for example, works fine.)
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.5.2, NixOS, 20.03pre212208.8130f3c1c2b (Markhor)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.2`
 - channels(root): `"nixos-20.03pre212208.8130f3c1c2b"`
 - channels(bemeurer): `"home-manager"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: zoom-us
# a list of nixos modules affected by the problem
module:

@lovesegfault I've had this happen before office hours once (basically panic), it seems there's settings inside zoom to change it that work. The view  Settings -> Audio .

Oh, If I set it to System Default then it seems to just obey PulseAudio!

Hmm, I see Same as system(maybe that's different in another version).
I don't understand why that's not a default.

Yeah, definitely bizarre behavior to override system defaults wrt. audio outputs

I think this is basically "works as intended" since you can specify the input/output in the app and then it tries to keep it that way per it's code

The view Settings -> Audio .

The arrows next to the microphone/speaker in the meeting view should reveal the same settings

Alright, closing this issue since I was just me not knowing how to use computers. Thanks for the help everyone!

I am also having this issue.

It seems to be application code releated AFAIK since it changes it back by itself... so "works as intended"?

I think this is basically "works as intended" since you can specify the input/output in the app and then it tries to keep it that way per it's code

I don't see it that way. The in-app settings should not be the only place where users can change the PA settings.
Also, if set to 'as system', this uses the system-default, which isn't necessarily the device you want for Zoom, and also that cannot be changed in pavucontrol, so at the moment, the only way to change the device is in Zoom itself, which is contrary to most other applications and limits the control people have over their system.

Wrt using a monitor (which Zoom doesn't allow by default), a workaround is creating a virtual source with module-remap-source:
pactl load-module module-remap-source source_name=Remap-Source master=Virtual-Speaker.monitor

Any update on this issue?


Submission language selector bug

Describe the bug
When configuring a submission field to contain a language selector, the language is lost when clicking the "Add" button
To Reproduce
Steps to reproduce the behavior:

Add <language value-pairs-name="common_iso_languages">true</language> in submission-forms.xml (I've tested with dc.title.alternative)
Enter a value and select a language
Click on the "Add" button
The language is lost, and has to be selected again
https://www.dropbox.com/s/2edxvgg4aqjovbw/Schermopname 2021-01-27 om 11.33.32.mov?dl=0

Expected behavior
After clicking add, the language should remain present

May be related to #971, which @atarix83 is working on.

May be related to #971, which @atarix83 is working on.

That sounds related indeed. It would be good to check again once 971 is ready to review

That sounds related indeed. It would be good to check again once 971 is ready to review

I've tested this with #1025 applied and I've verified that PR fixes this bug.


[Feature] Exported Content In Error Event

Request a new feature!
Is your proposal related to a problem?
No
Describe the solution you'd like
I think it would be pretty cool and beneficial to be able to access module exported content and information when a error occurs. This would be via the error event within the sern event type. This would allow us to create crash dumps for example that can return error solutions to the user via a new exported member.
Describe alternatives you've considered
I cannot think of any other solutions while remaining using the sern handler
Additional context
None

#115
This pull request will add ( module | undefined ) info to error events.


Update to Python 3

Had to change a bunch of print(), some itertool calls, and how bz2 is read.

@googlebot I signed it!
On Mon, Apr 6, 2020 at 3:12 PM googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot
I signed it! and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

ℹ Googlers: Go here
https://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fgoogle%2Fcategorybuilder%2Fpull%2F10
for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/categorybuilder/pull/10#issuecomment-610064129,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAMLPEUOYVWPS4DVZM6MGTRLJHUNANCNFSM4MCUME3A
.

Manually confirmed the author amahabal@ signed CLA


Adding missing platemap

The platemap C-7161-01-LM6-003.txt was missing from the original platemap metadata upload in #10
It is unclear why this platemap was missed. A couple of notes:

The metadata folder is nearly empty in ~/efs/2015_10_05_DrugRepurposing_AravindSubramanian_GolubLab_Broad/workspace/metadata
It is missing from the bucket folder

(base) ubuntu@ip-10-0-9-22:~/bucket/projects/2015_10_05_DrugRepurposing_AravindSubramanian_GolubLab_Broad/workspace/metadata/2016_04_01_a549_48hr_batch1/platemap$ ls
barcode_platemap.csv   C-7161-01-LM6-007.txt  C-7161-01-LM6-013.txt  C-7161-01-LM6-019.txt  C-7161-01-LM6-025.txt
C-7161-01-LM6-001.txt  C-7161-01-LM6-008.txt  C-7161-01-LM6-014.txt  C-7161-01-LM6-020.txt  C-7161-01-LM6-026.txt
C-7161-01-LM6-002.txt  C-7161-01-LM6-009.txt  C-7161-01-LM6-015.txt  C-7161-01-LM6-021.txt  C-7161-01-LM6-027.txt
C-7161-01-LM6-004.txt  C-7161-01-LM6-010.txt  C-7161-01-LM6-016.txt  C-7161-01-LM6-022.txt  C-7161-01-LM6-028.txt
C-7161-01-LM6-005.txt  C-7161-01-LM6-011.txt  C-7161-01-LM6-017.txt  C-7161-01-LM6-023.txt
C-7161-01-LM6-006.txt  C-7161-01-LM6-012.txt  C-7161-01-LM6-018.txt  C-7161-01-LM6-024.txt

I don't know the last time I checked either resource, but I did notice that the code in #21 failed whenever it was processing a plate with the C-7161-01-LM6-003.txt platemap.

thanks for quick review 🚀 - I realized when I was about to merge that I need to rerun all the metatadata notebooks after adding the platemap. I will simply rerun all the scripts and add the changes here. It would be good to have another quick look before finally merging, since there will be a few updated (and important files) Sorry for the inconvenience!
What changed in the mapping file:

Looking back, the number 1514 should have alerted my attention more. I am much more comfortable with the updated 1570!
Additional commits incoming

Note that the numbers in #20 are no longer accurate

alright @niranjchandrasekaran - PR ready for another quick look. Last time for this PR I swear!

Looking back, the number 1514 should have alerted my attention more. I am much more comfortable with the updated 1570!

@gwaygenomics Do we know what the actual number should be (the number of compounds that were profiled)? Is it 1570?

@gwaygenomics Do we know what the actual number should be (the number of compounds that were profiled)? Is it 1570?

This is the number I have always seen historically. I do not know what it should be officially (there is probably documentation somewhere...). I imagine that one way to get this official number is to look at all the platemaps, which are being added in this PR

Sounds good. This PR is ready for merge.

This is the number I have always seen historically

Yep

Source
Also confirmed by looking up metadata
cd ~/work/projects/2015_10_05_DrugRepurposing_AravindSubramanian_GolubLab_Broad/workspace/metadata/2016_04_01_a549_48hr_batch1_test/platemap
csvstack -t *.txt > /tmp/a
csvcut -c broad_sample /tmp/a|grep BRD|sort -u|wc -l
    1570


Extremely slow (>50 seconds) start-up with antigen

This started a while back but has gotten to the point where zsh with antigen is unusable for me. It takes nearly a minute for a new zsh shell to start up. Most of this time is spent in compdump. I tried to narrow it down to a particular plugin, but at this point I have a line to load antigen and antigen apply.
The output .zcompdump is huge, so that's probably what's causing the issue, but I don't really understand why it's so huge or what's going on with it.

.zcompdump
zprof.txt
.zshrc

I'm using zsh 5.2 and antigen 1.3.0 on OS X 10.12.1 with iTerm2 3.0.12.
Any help would be greatly appreciated. Please let me know if you need any more info to help analyze this issue. Thanks!

More than a second from version 1.1.0 and up is clearly a sign something is very wrong.
We've maintaining startup time below a second. There are some stats in the wiki.
It happened to me that zcompdump was appeding stuff to it on every startup. I don't remember the reason thou.
Did you tried to clear your compdup? antigen reset would do it. Otherwise  rm $HOME/.zcompdump*
I'll be taking a further look later today.

Not sure if it was the antigen reset or removing the zcompdump file(s) (although I'd deleted the zcompdump.HostName.xxxx files previously, I don't think I'd deleted .zcompdump itself), but that cleared it up. I now get:
num  calls                time                       self            name
-----------------------------------------------------------------------------------
 1)    1         878.57   878.57   43.84%    789.90   789.90   39.42%  zcache-load-cache
 2)    1         359.26   359.26   17.93%    359.26   359.26   17.93%  compdump
 3)    2         740.49   370.25   36.95%    210.20   105.10   10.49%  compinit
 4)    1         161.08   161.08    8.04%    161.08   161.08    8.04%  -zcache-unhook-antigen
 5)  752         130.28     0.17    6.50%    130.28     0.17    6.50%  compdef
...

That gets compdump down to a reasonable level. Thanks for the help and let me know if I can do anything to help figure out what causes that to happen.

Thank you for reporting. We'll keep an eye on the issue.
I'm closing this for now as the issue is solved.


All Pairs Shortest Paths Output

Even though the function for parsing APSP outputs writes a tab-separated csv, the text file doesn't have a tab separating the two nodes, but everything else does.
def parse_output(raw_pathway_file, standardized_pathway_file):         """         Convert a predicted pathway into the universal format         @param raw_pathway_file: pathway file produced by an algorithm's run function         @param standardized_pathway_file: the same pathway written in the universal format         """         df = pd.read_csv(raw_pathway_file, sep='\t', header=None)         df['Rank'] = 1  # add a rank column of 1s since the edges are not ranked.         df = readd_direction_col_undirected(df)         df.to_csv(standardized_pathway_file, header=False, index=False, sep='\t')

This is probably an easy fix, but I wrote it down so I don't forget.

I thought we modified it to output tab-separated output files. Could you possibly have a stale version of the Docker image locally? Does it work if you rebuild the latest Dockerfile?

I just updated my image, that was exactly what it was


ci: rename integration tests to platform tests

What this PR does / why we need it:
Some of our tests are in a directory called integration but in fact, they test Garden Linux on different platforms (such as AWS, Azure, chroot, KVM, etc.). Hence, we are already calling them platform tests internally and the name of the their location should reflect that.
Which issue(s) this PR fixes:
This Fixes #2303
Special notes for your reviewer:
tests/platformSetup cannot be named tests/platform as this clashes with python's platform module.

@Akendo @MrBatschner I verified that a pytest --iaas=kvm --configfile=/gardenlinux/config/kvm.yaml runs through after these changes.

pytest --iaas=chroot --configfile=/gardenlinux/config/chroot.yaml
============================================================================================================================ test session starts =============================================================================================================================
platform linux -- Python 3.11.2, pytest-8.3.2, pluggy-1.5.0
rootdir: /gardenlinux
collecting 67 items / 18 errors                                                                                Killed

I might have some trouble executing these tests?


[FEATURE] Help user to install dependencies when installing Longhorn

If we decide to install open-iscsi dependency, we need to do more investigation to ensure we have a way to workaround the potential Helm chart installation failure due to initContainer failed to install open-iscsi package.
For RWX volumes, we also require NFS client on the node. We can include it here as well.
Ref: https://github.com/longhorn/longhorn/issues/1741#issuecomment-724183156

I am still concerned about automated installation for dependencies. I don't want Helm installation to fail unnecessarily. For the first step, let's update the yaml to install nfs-common(NFS client) as well. Need to update the doc as well.

I am still concerned about automated installation for dependencies. I don't want Helm installation to fail unnecessarily. For the first step, let's update the yaml to install nfs-common(NFS client) as well. Need to update the doc as well.

Have update/add manifest for iscsi/nfs (#2210, #2211).  I am currently exploring using helm installation.

Have update/add manifest for iscsi/nfs (#2210, #2211).  I am currently exploring using helm installation.

@cclhsu Can you file out the checklist before moving the issue to Merged? Thanks.

The Yaml files for iscsi and nfs installation are validated with https://github.com/longhorn/longhorn/issues/2058 and https://github.com/longhorn/longhorn/issues/1741.
We should be good to close this.
@yasker @cclhsu anything specifically needs to be validated with this one?


vault ffi build failed

» rustc --version
rustc 1.47.0 (18bf6b4f0 2020-10-07)

» cargo build --features=ffi
warning: profiles for the non root package will be ignored, specify profiles at the workspace root:
package:   /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/channel/Cargo.toml
workspace: /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/Cargo.toml
warning: profiles for the non root package will be ignored, specify profiles at the workspace root:
package:   /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/message/Cargo.toml
workspace: /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/Cargo.toml
warning: profiles for the non root package will be ignored, specify profiles at the workspace root:
package:   /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/kex/Cargo.toml
workspace: /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/Cargo.toml
warning: profiles for the non root package will be ignored, specify profiles at the workspace root:
package:   /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/vault/Cargo.toml
workspace: /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/Cargo.toml
warning: profiles for the non root package will be ignored, specify profiles at the workspace root:
package:   /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/router/Cargo.toml
workspace: /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/Cargo.toml
warning: profiles for the non root package will be ignored, specify profiles at the workspace root:
package:   /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/transport/Cargo.toml
workspace: /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/Cargo.toml
warning: profiles for the non root package will be ignored, specify profiles at the workspace root:
package:   /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/worker/Cargo.toml
workspace: /Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/Cargo.toml
   Compiling log v0.4.11
   Compiling lazy_static v1.4.0
   Compiling ffi-support v0.4.2
   Compiling ockam-vault v0.1.0 (/Users/mrinal/Workspace/ockam-network/ockam/implementations/rust/vault)
error[E0061]: this function takes 4 arguments but 3 arguments were supplied
   --> vault/src/ffi/mod.rs:353:18
    |
353 |                   .hkdf_sha256(salt_ctx, Some(ikm_ctx), output_attributes)?
    |                    ^^^^^^^^^^^ --------  -------------  ----------------- supplied 3 arguments
    |                    |
    |                    expected 4 arguments
    |
   ::: vault/src/lib.rs:202:5
    |
202 | /     fn hkdf_sha256(
203 | |         &mut self,
204 | |         salt: SecretKeyContext,
205 | |         info: &[u8],
206 | |         ikm: Option<SecretKeyContext>,
207 | |         output_attributes: Vec<SecretKeyAttributes>,
208 | |     ) -> Result<Vec<SecretKeyContext>, VaultFailError>;
    | |_______________________________________________________- defined here

error: aborting due to previous error

For more information about this error, try `rustc --explain E0061`.
error: could not compile `ockam-vault`.

We need to run elixir ffi tests, otherwise this will happen all the time...


V1.4 Bugs UI

Closes #209 #210  #220  #223  #225  #229 #232 #236
Task List: (copied from issue):

[x] Buggy shortcut for deleting recipes #236

[x] Can't paste from clipboard using CTRL + V in add recipe form #220 (please check for bug vs new feature)

[x] Edit item window shows wrong ingredient input format #225 #223 #232

[x] Mouse hover takes priority over cardpane keyboard selector #229 #210

[x] Bug in exception handling #209 (refer to this link for discussion)

fix for #236: doesn't work on my end. even though no recipe card is selected (and highlighted in blue), pressing the "delete" key still prompts me to delete. if i confirm the deletion, the first recipe in the list is deleted
fix for #220: verified working
fix for #225 and duplicates: edit form ingredients still use a/, and add form ingredients are still empty. looking at the commit history, it looks like commit 9cdbdf9 is named wrongly? you may want to investigate

fix for [PE-D][Tester D] Buggy shortcut for deleting recipes #236: doesn't work on my end. even though no recipe card is selected (and highlighted in blue), pressing the "delete" key still prompts me to delete. if i confirm the deletion, the first recipe in the list is deleted
fix for [PE-D][Tester E] Can't paste from clipboard using CTRL + V in add recipe form #220: verified working
fix for [PE-D][Tester B] Edit item window shows wrong ingredient input format #225 and duplicates: edit form ingredients still use a/, and add form ingredients are still empty. looking at the commit history, it looks like commit 9cdbdf9 is named wrongly? you may want to investigate

Thanks. 3rd point is fixed. Could you test again for point 1? Right now on a fresh launch pressing 'D' does not trigger delete, unless the cursor is hovering over the card.

codecov for UI patches may be ignored

Is this go for merge?

all fixes are verified to be working, except for pre-filled text for add form.
for the add form, pre-filled text for ingredient and step fields are still not present (although they work for the edit form)

For this, since step and ingredient share the same textField, the prompt text has to be added via CSS @JamesLiuZX


RHTAPWATCH-322: Fluentd collectors are incompatible with Splunk ClusterLogForwarder configuration

Fluentd collectors are incompatible with Splunk ClusterLogForwarder configuration.
As detailed in
https://docs.openshift.com/container-platform/4.13/logging/cluster-logging-external.html#logging-forward-splunk_cluster-logging-external
fluentd is not supported with the Splunk Log Forwarder configuration.
Therefore, this commit swaps fluentd with vector.

/lgtm /approve

/lgtm

/approve

/retest-required

/retest-required


AL Language Extension 7.x Triggers Invalid Function ID Errors

1. Describe the bug
The following type of error is presented when invoking certain functions from extension/app code:
Function ID nnn was called.  The object with ID nnn does not have a member with that ID.

The error occurs when a call is made from code in one app (App A) to a function that exists in a separate app (App B).  App B is a dependency of App A.
Both apps compile and publish successfully from VS Code.  The error only appears within BC14, BC15, and BC16 sessions (Windows Client, Modern Client, and SOAP).  The apps have the same runtime version (for the targeted BC platform) specified in the app.json file.
The error surfaced after installing the AL Language extension for VS Code included in a container image with an Insider build of BC18.  The version of the language extension was 7.0.420929.  This week, in an effort to troubleshoot the cause, it was uninstalled and replaced with the current version from the Extension Marketplace (7.1.441584).  Unfortunately, that did not address the problem.  Removing that version of the language extension and installing an older one in its place -- in this case, one included with the RTM build of BC17 (6.0.338496) -- did away with the errors after re-publishing the apps.
2. To Reproduce
The error is triggered by statements similar to the one below, which call functions in a dependency:

Perhaps a similar scenario would be:
1/ Install version 7.x of the AL Language extension
2/ Create app: "App B"

create a codeunit with a function that requires the parameter: "Item: Record Item"

3/ Create app: "App A"

specify "App B" as a dependency
create a page extension that adds an action to the Items list page (P31) and call the function in the codeunit from "App B"

4/ Publish the apps to an on-prem instance of BC14, BC15, or BC16
3. Expected behavior
The function call should be completed without error.
4. Actual behavior
The following error is presented:
Function ID nnn was called.  The object with ID nnn does not have a member with that ID.

5. Versions:

AL Language: 7.1.441584
Visual Studio Code: 1.55.0
Business Central: NA Business Central 16.0 (Platform 16.0.11233.12061 + Application 16.0.12076.0)
List of Visual Studio Code extensions that you have installed:

AL Language
AL Object Designer
Cobalt2 Theme Official
GitLens — Git supercharged
Insert GUID

I referred to the following bug report:
https://github.com/microsoft/AL/issues/6448
The version of the AL Language extension that was mentioned as containing a fix (7.0.411812) is earlier than the versions that, in my case, resulted in a similar problem (7.0.420929 and 7.1.441584).

We'll take a look at this right away. Can you provide some sample code to reproduce the issue?

Hi, Alexandru.
Sorry, for the delay in getting back...  Included here are two basic apps; one has a dependency on the other.  It was possible to duplicate the error with them.
BrokenApp.zip
These are the extensions that are installed in my VS Code environment:

The version of the AL Language that was used to publish the apps:

The version of Business Central that the apps were published to:

The modified page object where one can invoke the function that triggers the error:

The resulting error:

After installing an older version of the language extension, the function presented the expected result:

After doing some fiddling around, I found that changing the parameter type for the function in App B from BigText to Text -- or removing it altogether -- also got rid of the problem.  It seems that function calls cannot include BigText values.
This problem occurs in BC14 through BC16.  I have not encountered it in BC17 or BC18... so far.

Same problem here! Any news?

Was it possible to verify and confirm that this is an issue?  The error is still raised with code compiled with the current version of the AL Language extension (v.7.2.475273).

Hi @bernalfr,
We have had fixes in this area, so please upgrade your server to the latest in the major version you are using and try publish + install the apps once again.

This issue has been closed because it needs more information and has not had recent activity. Please open a new issue if you believe this has not been resolved and reference the current issue.


can not use build-image-push

Describe the bug

apiVersion: core.oam.dev/v1beta1
kind: Application
metadata:
  name: build-push-image
spec:
  components:
  - name: my-web
    type: webservice
    properties:
      image: registry.cn-hangzhou.aliyuncs.com/hxpdocker/build-push-image:v1
      ports:
        - port: 80
          expose: true
  workflow:
    steps:
    - name: create-image-secret
      type: export2secret
      properties:
        secretName: image-secret
        kind: docker-registry
        dockerRegistry:
          username<EMAIL_ADDRESS>          password: xxx
    - name: build-push
      type: build-push-image
      properties:
        kanikoExecutor: registry.cn-hangzhou.aliyuncs.com/hxpdocker/kaniko-executor:v1.9.1
        context:
          git: gitee.com/hxpgit/build-push-image.git
          branch: master
        image: registry.cn-hangzhou.aliyuncs.com/hxpdocker/build-push-image:v1
        dockerfile: ./Dockerfile
        credentials:
          image:
            name: image-secret
        buildArgs:
        - BUILDKIT_INLINE_CACHE=1
        platform: linux/amd64
        verbosity: true
    - name: apply-comp
      type: apply-component
      properties:
        component: my-web

apply this got
      lastExecuteTime: "2023-03-09T07:52:25Z"
      message: "#do:       \"apply\"\n#provider: \"kube\"\n\n// +usage=The cluster
        to use\ncluster: *\"\" | string\n// +usage=The resource to apply\nvalue: {\n\tapiVersion:
        \"v1\"\n\tkind:       \"Pod\"\n\tmetadata: {\n\t\tname:      \"build-push-image-n3difdfdtz-kaniko\"\n\t\tnamespace:
        \"prod\"\n\t}\n\tspec: {\n\t\tcontainers: [{\n\t\t\targs: [\"--dockerfile=./Dockerfile\",
        _|_, \"--destination=registry.cn-hangzhou.aliyuncs.com/hxpdocker/build-push-image:v1\",
        \"--verbosity=true\", \"--customPlatform=linux/amd64\", \"--build-arg=BUILDKIT_INLINE_CACHE=1\"]\n\t\t\timage:
        \"registry.cn-hangzhou.aliyuncs.com/hxpdocker/kaniko-executor:v1.9.1\"\n\t\t\tvolumeMounts:
        [{\n\t\t\t\tmountPath: \"/kaniko/.docker/\"\n\t\t\t\tname:      \"image-secret\"\n\t\t\t}]\n\t\t\tname:
        \"kaniko\"\n\t\t}]\n\t\tvolumes: [{\n\t\t\tname: \"image-secret\"\n\t\t\tsecret:
        {\n\t\t\t\tdefaultMode: 420\n\t\t\t\titems: [{\n\t\t\t\t\tkey:  *\".dockerconfigjson\"
        | string\n\t\t\t\t\tpath: \"config.json\"\n\t\t\t\t}]\n\t\t\t\tsecretName:
        \"image-secret\"\n\t\t\t}\n\t\t}]\n\t\trestartPolicy: \"Never\"\n\t}\n}\n//
        +usage=The patcher that will be applied to the resource, you can define the
        strategy of list merge through comments. Reference doc here: https://kubevela.io/docs/platform-engineers/traits/patch-trait#patch-in-workflow-step\npatch?:
        {\n\t...\n}\n(bottom kind)"
      name: build-push

To Reproduce

Expected behavior

Screenshots

KubeVela Version

Cluster information

Additional context

We need to refine the error log to make it more clear \cc @FogDong

@13567436138 The verbosity's value is like verbosity: *"info" | "panic" | "fatal" | "error" | "warn" | "debug" | "trace", so true` is not a valid value of verbosity.

We need to refine the error log to make it more clear \cc @FogDong

This is a general problem of our cue-system currently, the error will be reported to both cli and VelaUX and it is a crucial problem to fix. Maybe we need to discuss it in another issue and make some designs first.

@13567436138 The verbosity's value is like verbosity: *"info" | "panic" | "fatal" | "error" | "warn" | "debug" | "trace", so true is not a valid value of verbosity.

How do you find the error comes from here? I didn't see any relevant information on the error message. @FogDong


Add a Wiki on Search and link it on Custom Search section on README

Inspired on our PHP SDK's search wiki, this document can specify features about search as the different methods exposed to the user (search() and rawSearch(), different constructors for this methods etc...
The repo's README can have a link to this WIKI in the Custom Search section as in this PHP README example

I agree with this idea, however I would rather not a Wiki. After using wiki, we found out this is not as easy to maintain as a "classic" file (like a README) in our repository: it's impossible to create PRs.
So, ok for this idea, but in the README 🙂

Hi @eskombro
Could you please assign this issue to me?

Hi @trangntt-016,
Thanks for asking. It's done

@eskombro , @alallema
Should I insert my documentation to the README's Custom Search or create another .md and link it to the README's search section?

This problem was written before the sample code was available. Now that it is, this issue is no longer required and would require too much maintenance to keep. I'm really sorry but I'll have to close this issue to maybe rewrite it and close your PR too. Even if your PR is really great.


Error when using natural from deno using esm.sh

error: The module's source code could not be parsed: `await` cannot be used as an identifier in an async context at https://esm.sh/v127/afinn-165@1.0.4/denonext/afinn-165.development.mjs:3650:5

If we go to that file we can see this:

This is preventing me from using natural in Deno Deploy atm. Is this a natural issue or an esm.sh issue?
https://github.com/esm-dev/esm.sh/issues/674

Natural does not support ESM.


[RFE] Include certs.d registry plugin in containerd config.toml

Current situation
I currently customize my containerd config.toml just to add the following settings:
[plugins."io.containerd.grpc.v1.cri".registry]
config_path = "/etc/containerd/certs.d"

My testing shows that containerd does not care if the directory does not exist, but I certainly would welcome further testing.
Impact
I have to add a custom containerd config that might not keep compatibility with upstream, simply so I can add custom registries or proxies.
Ideal future situation
To have a way of customizing a section of containerd's config.toml without replacing the whole file, which then also requires a dropin for the containerd service.
Either to have the plugin registry snippit added to the main config, or
**Implementation options
(tested) add the following snippet to config.toml, which does not seem to care if the directory does not exist:
[plugins."io.containerd.grpc.v1.cri".registry]
config_path = "/etc/containerd/certs.d"

(untested) to have an imports section that points to a custom config area such as:
imports = ["/etc/containerd/runtime_*.toml"]`

Hi,
good idea.
I agree that the imports directive could be a general good point for customization without "forking" the whole config.
We need to do some experiments to know how the import behaves. The filenames could be prefixed with custom_ or overwrite_ to know that it's not a file generated by the system.

I'll attempt to do some independent testing on imports before the holiday, and report my findings back.

My testing shows the following config works exactly as we would like, when /etc/containerd/custom_*.toml files exist and when they do not.
version = 2
imports = ["/etc/containerd/custom_*.toml"]
# persistent data location
root = "/var/lib/containerd"
# runtime state information
state = "/run/containerd"
# set containerd as a subreaper on linux when it is not running as PID 1
subreaper = true
# set containerd's OOM score
oom_score = -999
disabled_plugins = []

# grpc configuration
[grpc]
address = "/run/containerd/containerd.sock"
# socket uid
uid = 0
# socket gid
gid = 0

[plugins."containerd.runtime.v1.linux"]
# shim binary name/path
shim = "containerd-shim"
# runtime binary name/path
runtime = "runc"
# do not use a shim when starting containers, saves on memory but
# live restore is not supported
no_shim = false

[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.runc]
# setting runc.options unsets parent settings
runtime_type = "io.containerd.runc.v2"
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.runc.options]
SystemdCgroup = true


64-bit operating system requirement

Hey,
The frontpage for xcdat states:
The library considers a 64-bit operating system. The code has been tested only on Mac OS X and Linux. That is, this library considers only UNIX-compatible OS.

Is it a strict requirement? I would like to use the library on ARM32, so running under 32-bit OS.

@MrSherish

Is it a strict requirement?

Maybe yes. This is because the Rank dictionary (implemented in bit_vector.hpp) employs Vigna's Rank9 implementation that handles 64-bit words.
In Xcdat, however, all 64-bit words are handled using std::uint64_t. If std::uint64_t is simulated on your 32-bit environment, running Xcdat may be possible.

Maybe the requirements could be clarified to "requires std::uint64_t support" rather than "64-bit".


Do not wait any longer once quorum threshold is reached in handle_certificate calls

We continue the quorum round even after we have more than threshold stake with a reduced timeout which is probably because we don't want to terminate the connection causing validators to throw away whatever processing they have going on. This PR fixes the behavior. This may have been true before but we now spawn a tokio task on every server call which means the execution completes even if we terminate the connection. Added a test to confirm this behavior. There are other two places in authority_aggregator we use ContinueWithTimeout() which I am not fully certain if will benefit from this change (and hence not touching those) as we also seem to update authorities based on their response of object version they have present locally.

This was intended behavior - we want the certificate to be executed on as many validators as possible, so we continue past the point of quorum. This is in contrast to handle_transaction, where we want to stop immediately after quorum, since you don't get any bonus points for have extra signatures in your cert.

The certificate is still executed like the test proves but we don't need to wait for it on the client, no?

The certificate is still executed like the test proves but we don't need to wait for it on the client, no?

That's because all the authorities in your test immediately process the request, so the timeout never comes into play. But if there was any trouble sending the request in the first place, then other authorities may not even receive the request if we give up early.
That said, after thinking about it some more, this may be ok. I initially thought that your comment about running the handlers in separate tasks was a red herring, because when the authority aggregator code was written we hadn't even yet realized that early client disconnects would interrupt the request handler. So, that wasn't the original justification, but it may actually be pretty good justification after all.
I agree that it is the case that any validators that at least receive the request will finish, so waiting for more than 2f+1 to reply may not be necessary.
@lxfind what do you think?

Yeah I think this is the right thing to do.
My only question is whether this test is reliable? i.e. is one yield guaranteed for the server side to finish execution?

Yeah I think this is the right thing to do. My only question is whether this test is reliable? i.e. is one yield guaranteed for the server side to finish execution?

Maybe a reasonable sleep based delay will be better? I'm open to suggestions.

Yeah sleep should help. I wonder why this is causing checkpoint tests to fail

There are other two places in authority_aggregator we use ContinueWithTimeout() which I am not fully certain if will benefit from this change (and hence not touching those) as we also seem to update authorities based on their response of object version they have present locally.

IMO those functions are less relevant once gateway is gone, but let's keep their cleaning up for another PR

I also moved sync_certificate_to_authority to run in a tokio task so that we don't abort the syncing process mid way once we have enough votes. Note: this function is invoked when client identifies and decides to catch up a lagging authority. This may have been another reason why we had such a long post quorum timeout. Without this, we may never (or rarely) catch up lagging authorities (and just rely on checkpoints + gossip)


Use gcloud flow URL in documentation to enable gcloud settings

See an example here: https://github.com/google/voice-builder
Example flow: https://console.cloud.google.com/flows/enableapi?apiid=genomics,storage_component,storage_api,compute_component,containerregistry.googleapis.com

That flow is for enabling APIs, not creating a VM. That is done here: https://cloud.google.com/compute/docs/instances/create-start-instance#publicimage
It is a POST request, not what I was looking for.


Text Extraction (Too many / less spaces in text)

Link to PDF file (or attach file here):
http://dipbt.bundestag.de/dip21/btp/18/18145.pdf
Configuration:

Web browser and its version: Node 5.10.1
Operating system and its version: Mac OSX Latest
PDF.js version: Latest
Is an extension: ?

Steps to reproduce the problem:

Execute following script:

 * Created by velten on 13.05.16.
 */
"use strict";

require('pdfjs-dist');
const fs = require('fs');
const o_o = require('yield-yield');

const pdfPath = '/Users/velten/Downloads/18145.pdf';
const pdfUrl = "http://dipbt.bundestag.de/dip21/btp/18/18145.pdf";

o_o(function* () {
    try {
        var data = new Uint8Array(yield fs.readFile(pdfPath, yield));
        let pdfDocument = yield PDFJS.getDocument(data);
        let page = yield pdfDocument.getPage(1);
        let content = yield page.getTextContent();

        let stop = false;

        var strings = content.items.map(function (item) {
            if(stop) {
                console.log(item.str);
                exit();
            }
            if(item.str.indexOf('Dr') != -1) stop = true;
            return item.str;
        });
    }
    catch (err) {
        ServerError(err);
    }
})();

What is the expected behavior? (add screenshot)
Output = "." or ". "
What went wrong? (add screenshot)
Output is " . "

I saw that @speedplane and @yurydelendik did some work in this direction. Maybe they can tell me which functions are relevant?

The error is also directly inspectable in firefox:

Open http://dipbt.bundestag.de/dip21/btp/18/18145.pdf in Firefox
Inspect the Page: display:none the canvasWrapper and disable '.textLayer > div { color: transparent }
Look at Dr . Angela Merkel

Maybe I found a quick fix for this issue:
The elements "Dr" and " . " are overlapping.
So the x-position + width of "Dr" are greater then the x-position of " . ". So my code will include now:
if(item.str.startsWith(' ') && item.transform[4] < prev_item.transform[4]+prev_item.width) { return item.str.substring(1); }

I'm seeing the same issue with extra spaces in words. Example pdf available at https://github.com/jasonparallel/pdf.js-issues/blob/master/webSnapshot.pdf
For example at the bottom of the first page several spaces are inserted into n148584.rar  and a space is inserted into the word energy  right before that.

I'm seeing the same issue with extra spaces in words. Example pdf available at https://github.com/jasonparallel/pdf.js-issues/blob/master/webSnapshot.pdf
For example at the bottom of the first page several spaces are inserted into n148584.rar and a space is inserted into the word energy right before that.
This causes issues for copying text out of the pdf and for searching for text in the pdf.

Hi,
have you solve this problem, I have the same issue, my search query has one space in between each word, however, the pdf expect two spaces. do you have a solution ? thank you.


[Bugfix][Arith] Avoid flaky test with TryCombineSplitFromSameSource

This commit resolves a flaky test failure that was introduced in https://github.com/apache/tvm/pull/15081.  The unit test tests/python/unittest/test_meta_schedule_schedule_rule_mlt_tc.py::test_padded_matmul_relu failed approximately 30% of the time.
The error was due to changes in IterMapRewriter::NormalizeToIterWithOffset.  When attempting a simplfication with TryCombineSplitFromSameSource, if the returned Optional<IterSumExpr> is defined, but the args.size() < 1 check fails, then the expr argument is still overwritten, and presumably can cause a later TryFuseIters(expr) to fail.
This commit replaces expr = opt.value(); with auto combined = opt.value();, preserving the original argument.

No problem!  Once I had it bisected down, it was relatively quick to apply a fix as well.

And I spoke too soon.  The original "fix" also prevent several simplifications that are needed for index simplification.

I'm not sure at the moment which expression is failing to simplify on main.  For some of the choices of split factors, it resulted in the buffer C not being recognized as a region cover.  But I didn't quite get to the point of narrowing it down to a specific expression before the end of the day.
With the buggy fix, it broke simplification of n * (x//n) + x%n => x.  Closing this PR in favor of your fix.

For complete-ness, it looks like the failure can be traced back to the call to EstimateRegionLowerBound that occurs here, which returns NullOpt with the following arguments.
x = ax0_ax1_ax3_ax4_ax5_fused
y = ax0_0_0_ax1_0_0_fused

var_dom = {x: T.Range(0, 2048)}
indices = [x // 1024, y * 2 + x % 1024 // 512, ax2, x % 512 // 256, x % 256 // 16, x % 16]
predicate = ((x // 1024 * 64 + ax2 * 16 + x % 256 // 16 < 127) and
             (y * 64 + x % 1024 // 512 * 32 + x % 512 // 256 * 16 + x % 16 < 127))

Might be useful for us to send a regression


Signature for functions is showing as blank lines...

In version 3.29.0, the signature for functions do not show, but there is a blank line without a line number.
Another issue is that there appear a blank line above ALL 'let'  expressions:
let a = 1
let b = 2
let c = 3

I see the same issue (Ionide 3.29).
I also tried to enable LineLens (set the "FSharp.lineLens.enabled": "always") but they do not show up either (not even the "//" prefix).

Ok, I found what was causing this in my case. I have moved the F# code to new laptop and never built it. The project is .NET core so running dotnet run on the project immediately effected in showing the CodeLens signatures.

Blainne,thank you. Your last comment made me check every setup related to dotnet and/or net.
The Fsac Runtime flag has the value 'netcore'. Changing it to 'net' solved the problem.

@janpop2017 I think an issue should be created so that the notification contains [Fix it] & [Never notify again] buttons.


make codes executable and add source_env to dl1_dl2

make reorganize_dl1_output_files_hiperta_hipecta.py and hiperta_r0_to_dl1lstchain.py execuatable (change the 'main():' and how the arguments are passed)
add and update the source_env on train_pipe and dl1_dl2 to avoid that different environments could run consecutive jobs

Just given to the re-organizer script a consistent name reorganize_dl1hiperta_to_dl1lstchain.py (and change where it is invoked)


New pipeline header layout improvements

[x] Figure out layout
[ ] Add icons for other repo types
[ ] Fix smaller-screen layouts

Note: other repo provider icons have been added in #243!

The design work for this is in App UI > Pipeline Header Layout Improvements

Okay, this is ready to rumble. Thanks so much for your help @ticky, the new responsiveness is amaze.


Add support for time.Timer type and time.AfterFunc function

One of the implementations I'm working on uses time.AfterFunc, so I needed to add this to test that!

@aphistic Feel free to merge!

I don't think I have write access to the repo! :D


Export phrases - label and icon

https://trello.com/c/NfABaP9y/164-ux-fix-stahnout-fráze-ikona-exportovat-fráze

Hey! Looks good, I think we can merge this until we get a more specific design from UX. There is a small a conflict to resolve as well. If you feel like it you can also add the explanatory text to the export modal as described in the ticket. Just google translate the Ukrainian and Polish version, someone will look it over before publishing :) Thanks!


Fixed font configuration

Font-weights on Noto currently do not work, because they are not included in the GoogleFonts API call. This PR fixes that.

Before:

After:

Fixes #4613


Move container images to quay.io/konflux-ci

STONEBLD-2336
Signed-off-by: Adam Cmiel<EMAIL_ADDRESS>

Closing, we want to properly onboard to Konflux which will make this unnecessary


Filter not showing after loading more transactions

Describe the bug
When filtering transactions, if I click "Load more transactions", the filter is still in place, but the UI no longer shows it, which could be confusing.
To Reproduce
Steps to reproduce the behavior:

Go to https://explorer.hiro.so/transactions?chain=testnet&api=https://api.nakamoto.testnet.hiro.so
Click on 'Filters' and select 'Tenure change'
Click 'Load more transactions
Note that the filter is no longer showing

Expected behavior
The UI should continue to show the active filter.
Screenshots

Desktop (please complete the following information):

OS: macOS 14.6
Browser: Safari
Version: 18.0

https://github.com/hirosystems/explorer/pull/1835


Replace misleading error message when loading PEM

The error message "short header" when the end line
of PEM data cannot be identified is misleading.
Replace it with already existing "bad end line" error.
Fixes #8815

This pull request is ready to merge

Merged to master and 1.1.1.


Update to v0.22.0

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

closes #155

@conda-forge-admin, please rerender

I would appreciate at least one more set of eyes on:
https://github.com/SciTools/cartopy/blob/v0.22.0/pyproject.toml
to make sure I didn't mess anything up.
It would also be worth discussing if we want to make cartopy-with-* packages with any of the optional dependencies:
https://github.com/SciTools/cartopy/blob/v0.22.0/pyproject.toml#L52-L56
My current inclination is not to do that unless there is demand but it wouldn't be hard.

      LookupError: setuptools-scm was unable to detect version for /home/conda/feedstock_root/build_artifacts/cartopy_1691225778495/work.

      Make sure you're either building from a fully intact git repository or PyPI tarballs. Most other sources (such as GitHub's tarballs, a git checkout without the .git folder) don't contain the necessary metadata and will not work.

      For example, if you're using pip, instead of https://github.com/user/proj/archive/master.zip use git+https://github.com/user/proj.git#egg=proj
      [end of output]

I will try to remind myself what the workaround for this is but in the meantime I've submitted an issue upstream:
https://github.com/SciTools/cartopy/issues/2228

@conda-forge/cartopy, this is ready for review.

A question is whether we're okay with switching to GitHub for the source as done here or if we want to wait for source to be uploaded to PyPI, see https://github.com/SciTools/cartopy/issues/2228#issuecomment-1666504204

A question is whether we're okay with switching to GitHub for the source as done here or if we want to wait for source to be uploaded to PyPI, see SciTools/cartopy#2228 (comment)

I'm OK moving to GH and even moving back to PyPI after this release. Whatever you feel is easier. The only advantage to PyPI releases is that, some times, they are slimmer than the GH counterparts b/c authors remove some of the "fat."

@ocefpaf, I also feel like bypassing setuptools-scm isn't very clean so I would prefer to see if the source gets pushed in the next day or 2 as it seems it might.

I just uploaded the sdist to PyPI.

Thanks @dopplershift!

@conda-forge/cartopy, I have moved the source back to pypi and this is again ready for review (assuming tests pass).


The curly brace format is not supported

"$$\begin{cases}
...
\end{cases}$$"
latex like this , not working.

No. It works. Maybe try dropping the dollars?
\begin{cases}
	a & b\\
	c & d\\
	e & f
\end{cases}


CORE-17276 Refactor DB bootstrap into two containers

Refactor DB bootstrapping from 14 containers down to 2 so that a) they run faster as Kubernetes is not checking for updated images between every stage and b) it is easier to reason about the logic when making updates.

The bootstrap hasn't struck me as epecially slow, but that may be because I'm using local Docker Kubernetes.
This will conflict a bit with what @malachyb and I did recently if that hasn't been merged yet, but it's workable.
I think my preference is to have as little complexity in Helm as possible, and reduce it all to a Kotlin corda-cli Corda instlaler module which can either generate 10+ blocks of SQL content and maybe some commands to run (e.g. vault setup) or simply execute them directly.  I'd also like to align that with whatever local developers do.
To put it another way, let's have as much as possible in Kotlin and minimise the logic in Helm/Kubernetes since not everyone uses Kubernetes. Maybe in practice nearly all production deployments do, but that's less true of development of Corda itself, and perhaps development of Cordapps.
I think this may overlap a little with QA frameworks such as corda-non-function-test's eks.sh.

The bootstrap hasn't struck me as epecially slow, but that may be because I'm using local Docker Kubernetes.

If you're using the local developer overrides, then they set the image pull policy to IfNotPresent so you don't get the extra call to check the registry between each init container. We will only really see the benefits here when we reduce the number of times that we start a JVM (in line with what you suggest below), but I see this as a stepping stone towards that. The main driver at the moment is reducing the cognitive load on anyone who has to make changes in this area. It also makes it easier to find the relevant logs when something goes wrong: with the generation generally reliable, failures are more likely to be in the final container such that kubectl logs just works.

This will conflict a bit with what @malachyb and I did recently if that hasn't been merged yet, but it's workable.

I don't think it has merged yet, but happy to help out there if needed.

I think my preference is to have as little complexity in Helm as possible, and reduce it all to a Kotlin corda-cli Corda instlaler module which can either generate 10+ blocks of SQL content and maybe some commands to run (e.g. vault setup) or simply execute them directly.  I'd also like to align that with whatever local developers do.
To put it another way, let's have as much as possible in Kotlin and minimise the logic in Helm/Kubernetes since not everyone uses Kubernetes. Maybe in practice nearly all production deployments do, but that's less true of development of Corda itself, and perhaps development of Cordapps.
I think this may overlap a little with QA frameworks such as corda-non-function-test's eks.sh.

I don't disagree, but that's a bigger piece of work and impacts the product externals. I consider this PR a useful step in that direction, as it clarifies what commands are currently run as part of the bootstrapping.


Write the configuration of pkg repo type to right file

fix #13

I have an idea to check if the file has already exist, but if the project aim to fresh FreeBSD installation then the check is not necessary. The file should not exist on fresh installation.


Document the api

Complete application needs @moduledoc and @doc

Much of the application has been documented. Still some minor areas to complete.


Switch to skip ApiTrackingIdentifier for backward compatibility?

Hello, I am attempting to test the <IP_ADDRESS> version of the module due to some date issues I have experienced with version <IP_ADDRESS> but I am being prompted for an ApiTrackingIdentifier. We are in the process of testing our existing code with the api tracker in our dev environment, but have not transitioned to it in production. Is there no way to skip the ApiTrackingIdentifier for backward compatibility or does the <IP_ADDRESS> module not work with the 1.5 version of the API at all?
Thanks,
Jason

No, I am afraid it does not work with 1.5. Version <IP_ADDRESS> is the last version to work with 1.5.
And you are right. I have put quite an effort into finally getting the date handling right in the newest module version. I was not seeing the behavior I was lead to expect by the API documentation. I do believe this is caused by some unseen XML processing in the .NET code New-WebServiceProxy is based on. Date handling has been reliable in my tests since version <IP_ADDRESS>, but I would appreciate any feedback you could give me on how it works now.
For testing you could add another API user and activate an ApiTrackingIdentifier on the new user. If you activate an ApiTrackingIdentifier on your existing user you will have to use it to access API version 1.5 as well. A user with an ApiTrackingIdentifier MUST provide the tracker everytime, no matter which API version you try to access.
For what it is worth: In our tests all our old code worked with the new API version unmodified. The changes are mostly additive, new fields and entities have been added.

Thanks. I'm now using two different API accounts to continue testing both <IP_ADDRESS> and 1.6.2.x versions of the module.


Create README.md

Chassis Reboot Status and Reboot Cancellation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.374%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1314

Relevant Lines:
1841

💛  - Coveralls


createProjectCard: should content_id be number, not string?

Your doc: http://mikedeboer.github.io/node-github/#api-projects-createProjectCard
Github doc:  https://developer.github.com/v3/projects/cards/#create-a-project-card
When I attempt to create a project card with v9.2, I get an error like this:
 "Could not resolve to a node with the global id of 'MDExOlB1bGxSZXF1ZXN0Nw=='"

I am definitely passing in an issue id, in a fragment like this:
              let cardInfo = {
                column_id: BACKLOG_COLUMN,
                content_id: request.payload.issue.id,
                content_type: 'issue'
              }

Wondering if the "string" type I see here in index.d.ts is the issue?

hey, I'm having the same problem.
The content id should definitely be an integer according to the docs

@kaizensoze you moved on?  not whining, just wondering if you're actively maintaining.  Totally get it if you're not.

@zeke @gr2m are issues like this not meant to be in this project? Should these problems be addressed in the community, happy to open a discussion there if it is the case.
I am having the same problem and it seems like a bug that would be relevant to this project 😕

The issue was closed because it was fixed in 07fa82f back in April 2017, see the timeline event above

Bug reports such as this are always welcome.
The content_id parameter has type number, make sure you use the latest version of @octokit/rest. If you still think there is an issue, please open a new issue

My bad @gr2m! I read the issue backwards.
From my understanding the ID is meant to be a string. The ID you get from the columns and issues is a string like @gscott mentions above MDExOlB1bGxSZXF1ZXN0Nw==. Using these id's in a mutation function with graphql works but using the create card function here does not 😢
`mutation {
  addProjectCard( input: { contentId: "${ pullrequest.node_id }", projectColumnId: "${ column.id }" }) {
    clientMutationId
  }
}`

I feel like the documentation is maybe wrong for the GitHub api. I have tried using database_id and that also does not work.

The methods and types for @octokit/rest are automatically generated based on what is documented on https://developer.github.com/v3. In this case, the documentation for Create a project card says that content_id is a number, you set it to the (database) id of the issue or pull request that you want the project card to be associated with. You also have to set content_type accordingly.
Note that this is all related to the REST API only, not the GraphQL API. In GraphqlQL, the contentId parameter for the addProjectCard mutation is indeed a string. It’s the node ID which is a base64 encoded string of content type + database id. For example
query {
  resource(url:"https://github.com/octokit/rest.js/issues/531") {
    ... on Issue {
      id
	}
  }
}

returns MDU6SXNzdWUyMjI5Njk2MTM= as id, which is the encoded string of 05:Issue222969613. Not exactly sure what the 05: prefix is, but both information are included in the string, hence there is no second contentType parameter in GraphQL

@gr2m thank you so much 👍


Use the latest CDAP Ambari Service

This uses 4.0.0-1 of the cdap-ambari-service and also updates the attribute controlling ambari cookbook usage to something a little more sane. I've also moved ambari into depends rather than the unimplemented recommends in the cookbook metadata.

👍  lgtm


CMAKE Error (Unable to Generate makefile)

When trying to compile using
cmake -DCMAKE_BUILD_TYPE=Debug -DWITH_SSE2=ON .
on Raspbian 8 (Jessie)  I get the floowing Error:

cmake -DCMAKE_BUILD_TYPE=Debug -DWITH_SSE2=ON .
FREERDP_VERSION=2.0.0-dev2
-- Git Revision 2f15b80
-- GCC default symbol visibility: hidden
CMake Error at /usr/share/cmake-3.0/Modules/FindPackageHandleStandardArgs.cmake:136 (message):
Could NOT find Threads (missing: Threads_FOUND)
Call Stack (most recent call first):
/usr/share/cmake-3.0/Modules/FindPackageHandleStandardArgs.cmake:343 (_FPHSA_FAILURE_MESSAGE)
/usr/share/cmake-3.0/Modules/FindThreads.cmake:178 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
CMakeLists.txt:577 (find_package)

-- Configuring incomplete, errors occurred!
See also "/root/FreeRDP/CMakeFiles/CMakeOutput.log".
See also "/root/FreeRDP/CMakeFiles/CMakeError.log".

Please use IRC or the mailing list for support questions.


API: Added utilities to operate on blob streams as stdio FILEs

This lets you read and write blobs using standard C functions like fread, fseek, fwrite, similarly to the way other CBL platforms let you use the platform's stream classes.
The API is in src/CBLBlob+FILE.h. This is not in the include/cbl directory since it isn't (yet?) official API.
Unfortunately I was unable to find a way to implement this on Windows, since (apparently?) it does not have a function equivalent to funopen or fopencookie for creating a FILE* with custom I/O callbacks. Still, it seems a shame to leave out this feature, especially since the majority of CBL-C clients will probably be Linux-based.

This looks like it might be helpful -> https://github.com/Snaipe/fmem

fmem only provides a FILE that reads/writes a memory buffer. If we used that, the memory usage would be the same as our non-stream-based blob API, since the entire blob would have to exist in memory.

The build is failing now. Should be #include "CBLBlob+FILE.h" instead of "CBLBlob+File.h".
/Users/runner/work/couchbase-lite-C/couchbase-lite-C/test/BlobTest.cc:20:10: warning: non-portable path to file '"CBLBlob+FILE.h"'; specified path differs in case from file name on disk [-Wnonportable-include-path]
#include "CBLBlob+File.h"
         ^~~~~~~~~~~~~~~~
         "CBLBlob+FILE.h"
In file included from /Users/runner/work/couchbase-lite-C/couchbase-lite-C/test/BlobTest.cc:20:
/Users/runner/work/couchbase-lite-C/couchbase-lite-C/test/../src/CBLBlob+File.h:20:10: fatal error: 'CBLBlob.h' file not found
#include "CBLBlob.h"
         ^~~~~~~~~~~
1 warning and 1 error generated.
make[2]: *** [test/CMakeFiles/CBL_C_Tests.dir/BlobTest.cc.o] Error 1
make[1]: *** [test/CMakeFiles/CBL_C_Tests.dir/all] Error 2
make: *** [all] Error 2
Error: Process completed with exit code 2.

yes keep that work up as the WSL on windows allows that also i guess even cygwin allows that we also use that to binary pipe blobs into chrome on windows via the devtools protocol and this is a nice feat as it will allow my wasm implementation to take browser blobs directly in and out that makes streaming to the couchbase cluster via direct socket api experiment that we currently run much more nice.
Only for your interrest i am building a poc that directly uses current chromium dev the feat is already included to directly connect via the couchbase sdk to a couchbase cluster anyway i also build cblite to wasm and i am using then the binary in and output
it would help me if you could maybe make the fleece api run able more isolated so that i could create cblite with indexdb to run complet in the browser.

@snej Can you check the build failure?

@pasin please use the latest version and tell me if that happens with that also


scalafmt_native support?

Does the scalafmt integration include support for scalafmt_native? My understanding is no, since it relies on the persistent workers scheme. In our experience, scalafmt_native has significantly better memory usage when running large batch jobs.
If I wanted to add support for scalafmt_native, how would I go about that? Would the idea be to add some kind of integration to ext_scalafmt which specifies that the associated formatter is a scalafmt_native formatter, not a JVM-based scalafmt? I'm not sure, because it seems like ext_scalafmt is the way to configure formatting on an individual target basis, but most likely the user would want to configure it on a per-workspace basis.

maybe we can extract that into a toolchain? though maybe we should have two separate flavors of scalafmt? Not sure.
What are the advantages of the jvm based scalafmt?

JVM based is 2x faster on a large enough set of files, or once warmed up enough. At the cost of much higher memory usage.  Scalafmt is fast enough that I don't see a real world advantage of the jvm based scalafmt, even when formatting thousands of files at once I still find myself reaching for the native version.

@borkaehw as the original author wdyt? I'm thinking of just changing to the native one and that's it
@ShaneDelmore @arxanas when you say native one is it built somewhere and we retrieve it per platform or do we need to also build it from source?

@ittaiz Currently, you can grab pre-built versions per-platform from https://github.com/scalameta/scalafmt/releases/
Some implementation notes:

The executable is rather large (50+ MB) and can take a while to download, depending on your connection.
The format of releases changed between older and newer versions. The older versions used to package an archive, which made it faster to download but slightly harder to integrate into Bazel (you can't just use http_file anymore). This could be a problem for some use-cases, because the way to take a dependency on scalafmt_native changes depending on which version you want.
The native one must match the version in .scalafmt.conf exactly, or it will refuse to run.


pull request for e2e test at 01/22/2017 01:45:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Overworld generation fix

Fixes incomplete 1.9 migration (#239).
Next time please ask me instead of modifying the biomegrid, this is a sensible mechanic.

Hm, sorry. I tried to revert everything I removed after I figured out the problem, but I missed these.


Upgrade k8s.io/* to v0.25, sigs.k8s.io/controller-runtime to v0.13

How to categorize this PR?

/area open-source
/kind enhancement
What this PR does / why we need it:
Upgrade dependencies

k8s.io/* to v0.25
sigs.k8s.io/controller-runtime to v0.13

Which issue(s) this PR fixes:
Fixes #6661
Special notes for your reviewer:
Release note:

The following dependency is updated:
- k8s.io/* : v0.24.4 -> v0.25.0
- sigs.k8s.io/controller-runtime: v0.12.3 -> v0.13.0

@shafeeqes: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name	Commit	Details	Required	Rerun command
pull-gardener-check-vulnerabilities	868c507 link	false	/test pull-gardener-check-vulnerabilities
pull-gardener-integration	868c507 link	true	/test pull-gardener-integration
Full PR test history. Your PR dashboard. Command help for this repository. Please help us cut down on flakes by linking this test failure to an open flake report or filing a new flake report if you can't find an existing one. Also see our testing guideline for how to avoid and hunt flakes.

Failure is not related to this PR. Opened https://github.com/gardener/gardener/issues/6671
/test pull-gardener-integration


Provide ready to flash .bin files in github releases page

The README suggests to use M5Burner for ease of use.
However this only runs on Windows and MacOS. It would be useful if new users who just want to bootstrap a M5Stack with SD-Boot support could fetch a recompiled .bin file from the githup page and flash it with esptool.

M5Burner has a linux version

https://m5stack.com/pages/download
Also there's already a downloadable zip with the precompiled menu.bin in the releases page (Yolo edition)

Thanks. I made wrong assumptions.


feat: window.is_minimized()

Example use-case: A toggle-minimize command bound to a key that displays or minimizes a window. For example a popup terminal or "quake mode" terminal. This requires is robust method to tell if a window is already minimized, or a reliable event notification if (for example) the uses clicks a minimize button is a system titlebar.

A related issue: The documentation for is_visible doesn't state whether a minimized window is considered visible or not. I think it should. I haven't yet tested what the implementation does - it may depend on the platform.

yeah IIRC, macOS is differential than other platforms and we should document that.


Can the suggestions prop not be case-sensitive?

Is it possible to make the suggestions matching non-case-sensitive?
lowerCase prop and applying toLowerCase() to the suggestions has not worked.

@davidmednikov it is not easy to implement

I was able to implement it the following way:
In suggestions_textfield.dart, line 154 and 155:
matches =  _suggestions.where((String sgt) => sgt.startsWith(str)).toList();

I replaced this with:
_matches = _suggestions.where((String sgt) => sgt.toLowerCase().startsWith(str.toLowerCase())).toList();

__matches = _matches.map((match) => match.replaceFirst(match.substring(0, str.length), str)).toList();

Then, for the TagsTextField widget:
textField: TagsTextField(
    duplicates: true,
    ...
    onSubmitted: (newTagStr) {
        String tagName = suggestions.firstWhere((suggestion) => suggestion.toLowerCase() ==     newTagStr.toLowerCase(), orElse: () => null);
        if (tagName != null) {
            newTagStr = tagName;
        }
        if (!tags.map((tag) => tag.title.toLowerCase()).toList().contains(newTagStr.toLowerCase())) {
            tagsUpdated = true;
            setState(() {
                tags.add(Item(
                    index: tags.length,
                    title: newTagStr,
                    active: true,
                    customData: true
                ));
            });
        }
    }
}

@davidmednikov yours is a great job! I just wanted to avoid adding all this logic when instead a good practice would be to avoid upstream the use of contemporary uppercase and lowercase text in the suggestions. However, I will take your work into consideration, and carry out checks.


Improve reality tile purge and preloading

Review + integrate this PR into iTwin.js: https://github.com/iTwin/itwinjs-core/pull/3374

#4009

Need to revisit the changes.

removing from 3.4 due to other priorities


Integrate hyperlinks in citations

We have several references that contain a link to the doi. For example http://iopscience.iop.org/article/10.1088/0953-8984/27/45/455102/pdf
This is valuable information that ought to be extracted.

Hello!
Unfortunately I can't access this publication to test it.
In the current (master) version, the support for DOI and url should not be too bad (there was an effort on this last summer). They are explicitly extracted, with dedicated labels, and should be present in the result when they are recognized as such. It would be interesting to see why it fails with this paper - is it just with IOP?

Oh sorry. The pdf is available at https://www.researchgate.net/profile/Alberto_Fraile3/publication/282876717_Experimental_evidence_for_a_dynamical_crossover_in_liquid_aluminium/links/5623664c08ae70315b5b8e2b/Experimental-evidence-for-a-dynamical-crossover-in-liquid-aluminium.pdf

@renaud thanks ! I see the problem now, we have the url as annotation on the volume/page.
This case is definitively not working and this would be very valuable indeed. In the analysis process, all the URL annotations are recognized and an object PDFAnnotation is created, but it is not always outputted. We need to inspect this link to identify such common identifier and instantiate explicitly the identifier field so that it correctly appears in the TEI.

Added with commit f93e4ab4c074246214cf2d22ddfd1f283e1003de
@renaud you can test it on the demo server -> http://grobid.science-miner.com
(clicking on consolidate citations give really some nice additions)

thanks @kermitt2 , excellent!


ScopeRequirementMiddleware does not check for presence of access token

Current implementation of ScopeRequirementMiddleware has the following code:
// if no token was sent - no need to validate scopes
var principal = context.Authentication.User;
if (principal == null || principal.Identity == null || !principal.Identity.IsAuthenticated)
{
  await _next(env);
  return;
}

But this does not take into account that multiple authentication mechanisms could be used and principal could be already set by e.g Cookie Authentication beforehand. In this case ScopeRequirementMiddleware returns insufficient_scope even when there is no token.
The fix is to check for presence of idsrv:tokenvalidation:token in OWIN environment. If you agree I can fire a PR.
Having both Cookie and OAuth authentication schemes together might seem weird, but this still could be the case, for example when custom API (using IdentityServer3.AccessTokenValidation) is being a plugin of Identity Server (which relies on Cookie authentication).

Would that work for you? https://github.com/IdentityServer/IdentityServer3.AccessTokenValidation/pull/67
Nuget here: https://www.myget.org/F/identity/

Yeah, it works.
Btw, it has not been merged into DEV, so I could not use the Nuget you provided, had to build it on my own.

I will merge it now.

Thanks!


Enable custom metrics add more details

In the section "Emit custom metrics" it would be nice to add more details about the point
"2. Add "metrics":true to the payload, along with any other properties."
It was quite difficult to find out where to turn this feature on until I've come across this article.
Also some information about the pricing might be useful.

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

ID: c26c1996-853e-9a55-5802-8c4095d545b9
Version Independent ID: 437f5553-6914-c0ef-9370-bb11d3a0fa4b
Content: Integrate Azure API Management with Azure Application Insights - Azure API Management
Content Source: articles/api-management/api-management-howto-app-insights.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

@oskrabanek Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@oskrabanek - Thanks for letting us know about this issue. We'll add more information with a small doc update soon.

I have created a documentation work item to address this issue that we will assign to an upcoming doc sprint. According to our usual process, I will proceed to close this issue. Thank you again for pointing out this issue.
#please-close


Improving libdoc HTML output for mobile

Current libdoc html output does not work very well with mobile.
Here is an example from Robot Framework Browser library docs.
I propose the following changes: proof of concept docs

Fitting page width with the content
Adding horizontal scrolls to code examples, to prevent them from growing width of the page
Replacing keywords table with bordered boxes.

The reasoning behind the third change is that information in the current table layout tends to go to right side of the table. This makes most of the page uninformative - most focus should be on the Documentation part. Also headers of columns are hidden when scrolling.

Here is the example in proposed format:

I'm in general in favor of enhancing all out HTML outputs on mobile and otherwise. The proposed changes themselves sound pretty good too, although I'm not so certain about scrollbars. Do you propose doing these changes so that they affect all usages or only mobile or otherwise narrow layout? Do you have examples showing the differences?
A problem with this issue is that there are many different enhancements and there could be even more. It could be that we only want to include some of the enhancements and thus examples should also show only one change (unless changes are highly related). Especially with bigger changes we could also wait until we, hopefully, soon see face-to-face.

Sorry I meant Horizontal and not Vertical..
Anyway they are present in the example:

Pull request https://github.com/robotframework/robotframework/pull/3675

It would be nice if the css was all in a file that could be swapped out with a command-line option. For example, robot -m libdoc --css my-libdoc-styles.cssThis might make it easier for the community to contribute other styles, and make it easier for larger organizations to customize the look of their keyword documentation.

@boakley css controls now most of the look and feel of an html page. This means that it mostly builts what you see where as html only has some nodes of data.
Most of the work I have done so far is css.
I think we need to think carefully how this custom styling should actually happen. For instance overwriting colors might be easy or using company font or adding logo, but changing menu locations or link or list styles globally might have unwanted consequences.

Injecting css would be a very powerful feature and something I've been thinking would be a good idea also with logs and reports. Instead of replacing the existing css fully, it might be better to add new styles after it so that there's no need to recreate all styles but anything can be overwritten.
The main problem with this is that it would make the currently internal Libdoc/log/report HTML code part of the public API. Possible changes to HTML structure, ids, classes, etc. could break custom css badly.
Although that's a pretty big problem, the feature would be so powerful that I believe it would be worth it. We basically needed to decide and document which parts of the HTML code are concidered stable and in general avoid breaking changes outside major versions where they needed to be explicitly documented.
This enhancement would need its own issue where to discuss the design more. It ought to be pretty easy to implement and thus could be a possible addition to RF 4 scope.

I'll close this issue in favor of a better one related to LibDoc output html improvements

Um, do you have such ticket? I cannot find anything with https://github.com/robotframework/robotframework/issues?q=LibDoc+output+html+improvements

I think it's #3687.


fix(selector): makes component as uncontrolled component

Fix #365

This introduces a bug! If you press the back button, the value of the select won't be updated anymore.

@bobylito Can you tell us what is the difference between a controlled and uncontrolled component? I couldn't understand it from the docs.

So do we do something to solve what @bobylito said here?

We should open a new issue for tracking this bug, with a simple repro case (I did not get if it was pressing backspace in the input, or in the webpage -to go to previous page- that triggered the issue).
cc @bobylito


Ignore requirement groups with only undefined courses

Went with hybrid approach. Semester page now shows all courses that can be taken, ignoring undefined. But, it the does show any scheduling error that occurs because of an undefined course.
It is then up to the user to see if they can still take the course or not.


Theme for player controls menu bar

Coverage remained the same at 99.129% when pulling 983316f84a19c1ed057fb172f8fd8851fedc4214 on tf:player-controls-menu-bar into 2b2dbd6c0395410efaced87cf6f4cf61902736b9 on codevise:master.


Increase upgrade test timeout 30->45 mins

The upgrade tests have recently started being frequently canceled by
their 30min job timeouts.
github.com/submariner-io/submariner/actions?query=workflow%3AUpgrade
"The job running on runner GitHub Actions 2 has exceeded the maximum
execution time of 30 minutes."
It seems they are hitting a failure described by #1031 and then being
canceled before they have time to run their post mortem.
Signed-off-by: Daniel Farrell<EMAIL_ADDRESS>

@dfarrell07 a normal working pass is 19 min, that's 10min headroom which I suspect only happens when something is really going wrong.

@dfarrell07 a normal working pass is 19 min, that's 10min headroom which I suspect only happens when something is really going wrong.

The history here shows the old passing jobs at 24 mins.
https://github.com/submariner-io/submariner/actions?query=workflow%3AUpgrade


No result

Hi,
Is this code still working for Albert Heijn? I've installed the package and run the code. But I get no result, no error just nothing. The code I've used as test:
from supermarktconnector.ah import AHConnector connector = AHConnector() connector.get_categories()
Any help would be appreciated.
Greetings,
Jos

@josvink66 nope, alas, it used to work but since a couple of weeks, AH changed some things and this Python library no longer works.

It still works great. Only thing i needed to do, is change the jumbo API to v14 for now.

@rogierrijs
How did you do it?

@rogierrijs same question form me ... how to do? get the scraper to work? I am totally blind in python land, help is much appreciated
max

Hey, in the code, an url is used, which holds the version.. like /v12/ or something.  You should change that to the one that works.
Edit, just checked, that is for Jumbo. Not AH.
To be honest, havent used this in a long time, so maybe something else broke.


[BUG] jaxlib build fails on cuda 11.8 ubi9 container.

Error in fail: Repository command failed
Could not find any cudnn.h, cudnn_version.h matching version '8.9.2' in any subdirectory:
        ''
        'include'
        'include/cuda'
        'include/*-linux-gnu'
        'extras/CUPTI/include'
        'include/cuda/CUPTI'
        'local/cuda/extras/CUPTI/include'

jaxlib was not tagged correctly. Corrected the tag.


Add linux start command to docs

nohup ollama serve &

If the recommended way to install ollama is followed, ollama will be started automatically.
If the service is installed manually, ollama can be started on linux with sudo systemctl start ollama.

Thanks @rick-github ! As Rick mentioned, the default uses systemd. Other ways are possible as well, but we try to keep focused on ollama serve

In my case, I was using a container that did not support systemctl. Also, its a bit confusing what the difference between start and serve is?

start and serve are synonyms for ollama, they do the same thing.  If you are running ollama in a container, the act of starting the container (docker run or docker compose up) will start the server.

In my case, I don't have control over starting the container, as this was a vast ai instance. Also, I think it would make more sense to just have one start command?


THREAD WARNING: ['QRScanner'] took '468.501953' ms. Plugin should use a background thread.

Hi there,
I'm having issues using the QR Scanner with Ionic.
ERROR OUTPUT:
THREAD WARNING: ['QRScanner'] took '468.501953' ms. Plugin should use a background thread.
App Module:

/src/app/app.module.ts

import { ErrorHandler, NgModule } from '@angular/core';
import { IonicApp, IonicErrorHandler, IonicModule } from 'ionic-angular';
import { SplashScreen } from '@ionic-native/splash-screen';
import { StatusBar } from '@ionic-native/status-bar';
import { QRScanner } from '@ionic-native/qr-scanner';

import { MyApp } from './app.component';
import { HomePage } from '../pages/home/home';

@NgModule({
  declarations: [
    MyApp,
    HomePage
  ],
  imports: [
    BrowserModule,
    IonicModule.forRoot(MyApp)
  ],
  bootstrap: [IonicApp],
  entryComponents: [
    MyApp,
    HomePage
  ],
  providers: [
    StatusBar,
    SplashScreen,
    QRScanner,
    {provide: ErrorHandler, useClass: IonicErrorHandler}
  ]
})
export class AppModule {}

Markup

/pages/home/home.html

  <ion-navbar>
    <ion-title>
      App
    </ion-title>
  </ion-navbar>
</ion-header>

<ion-content padding>
  <button ion-button full (click)="scanTickets()">Scan Tickets</button>
</ion-content>

Home Typescript

/pages/home/home.ts

import { NavController, AlertController } from 'ionic-angular';
import { QRScanner, QRScannerStatus } from '@ionic-native/qr-scanner';

@Component({
  selector: 'page-home',
  templateUrl: 'home.html'
})
export class HomePage {

  constructor(
    public navCtrl: NavController,
    public alertCtrl: AlertController,
    private qrScanner: QRScanner) {

  }

  scanTickets() {
    // Optionally request the permission early
    this.qrScanner.prepare()
    .then((status: QRScannerStatus) => {
      if (status.authorized) {
        // camera permission was granted

        // start scanning
        let scanSub = this.qrScanner.scan().subscribe((text: string) => {
          console.log('Scanned something', text);

          this.qrScanner.hide(); // hide camera preview
          scanSub.unsubscribe(); // stop scanning
        });

        // show camera preview
        this.qrScanner.show();

        // wait for user to scan something, then the observable callback will be called

      } else if (status.denied) {
        // camera permission was permanently denied
        // you must use QRScanner.openSettings() method to guide the user to the settings page
        // then they can grant the permission from there
      } else {
        // permission was denied, but not permanently. You can ask for permission again at a later time.
      }
    })
    .catch((e: any) => console.log('Error is', e));
  }
}

+1

This happens for Cordova as well.
I got the message on an iPhone 5s
THREAD WARNING: ['QRScanner'] took '10.020996' ms. Plugin should use a background thread.

PRs welcome!
There are some changes in 2.6.0 which may help here. If you're still having trouble, please feel free to open a new issue.

Hi @bitjson , I have the same error and I have version 2.6.0 and for iOS it enables the background mode like that error comes out
THREAD WARNING: ['QRScanner'] took '68.887695' ms. Plugin should use a background thread.

Using Version 3.0.1 still the same error.
THREAD WARNING: ['QRScanner'] took '383.850830' ms. Plugin should use a background thread.

+1


Add QALayer results in drawing error message for Line Shape Minus attribute layer on SDE platforms

Workaround is to remove the double quotes around the fabric fields in the definition query to view the Line Shape Minus Attribute QA symbology layer.

fixed: https://github.com/Esri/parcel-fabric-desktop-addins/commit/e32fb114cf794792e6a248b34271b44152f41c88


Error when building the Remix blog example in Kitchensink

What version of Turborepo are you using?
1.0.17
Describe the Bug
The Remix blog example in Kitchensink does not build:

Expected Behavior
It should build without errors.
To Reproduce

Clone https://github.com/vercel/turborepo
Install deps (yarn/npm install)
Go to examples/kitchen-sink/apps/blog
yarn build

Should be fixed now


syntax error in active_corners

minor syntax error fix

Thanks, sorry this took so long to get merged.


systemlookup installation

Hi Katjahahn,
I have a problem while trying to install the tool using "pip install systemlookup". My system is using python 2.7.10 and this is the error that i got:
λ pip install systemlookup
Collecting systemlookup
Using cached systemlookup-0.0.3.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
File "c:\users\admins\appdata\local\temp\pip-build-tdnvmo\systemlookup\setup.py", line 32, in
long_description=open('README.md').read(),
IOError: [Errno 2] No such file or directory: 'README.md'
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in c:\users\admin\appdata\local\temp\pip-build-tdnvmo\systemlookup\
Would you please tell me how to fix it?

This should be fixed now. Please try again.

Thank Katjahan. The installation problem has been resolved. However this is the output when i run this command:
systemlookup -t name "explorer.exe"
Traceback (most recent call last):
File "C:\Python27\Scripts\systemlookup-script.py", line 11, in
load_entry_point('systemlookup==0.0.4', 'console_scripts', 'systemlookup')()
File "c:\python27\lib\site-packages\systemlookup\systemlookup.py", line 123, in main
data = syslookup(listnr, args.searchtype, args.searchitem)
File "c:\python27\lib\site-packages\systemlookup\systemlookup.py", line 96, in syslookup
return requests.get(base_url + sl_list + "&" + sl_type + "&" + sl_search + "&s=").text
File "c:\python27\lib\site-packages\requests\api.py", line 60, in get
return request('get', url, **kwargs)
File "c:\python27\lib\site-packages\requests\api.py", line 49, in request
return session.request(method=method, url=url, **kwargs)
File "c:\python27\lib\site-packages\requests\sessions.py", line 457, in request
resp = self.send(prep, **send_kwargs)
File "c:\python27\lib\site-packages\requests\sessions.py", line 569, in send
r = adapter.send(request, **kwargs)
File "c:\python27\lib\site-packages\requests\adapters.py", line 407, in send
raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', error(10060, 'A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond'))

Works for me, I cannot reproduce this error. Check your internet connection and availability of systemlookup.com

I will be closing this because the original issue has been resolved. Open a new issue if there are more.
Thank you for reporting to me! Very appreciated.


Update pace.js

Not work in webpack，Uses Node, AMD or browser globals to create a module.

They clearly use 2-space indentation.


indent: Wrong indentation in template literal expressions

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Describe the bug
Indentation seems to be off in template literal expressions in some cases. This was working correctly on version 2.6.3 (or maybe it didn't do any formatting checks at all)
Config options
stylistic.configs.customize({
  indent: 2,
  quotes: 'single',
  semi: true,
  jsx: false,
  braceStyle: '1tbs',
}),

Expected Behavior
Since the plugin won't parse the contents in template literal, I think expected behaviour would be not checking it at all (as on version 2.6.3) and leave as is. This could also be optional
`

`;

Actual Behavior
Automatically moves indentation of expression to the first line of literal
`

`;

Reproduction
https://stackblitz.com/edit/github-2gg16p
Contributes

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests

Upon further inspection, it looks like the problem occurs when the whole line doesn't start with ${
This is linted correctly
`

`;


Can we number the sections so that there is no ambiguity

In kubeflow-manifests/examples/aws/cognito-rds-s3/README.md can we number the subsections as 4.1 instead of just 1
so that there is no ambiguity

Also upto means including this step. @surajkota

addressed in #19


Method-namings for Entity-classes

@liach why? if something is named jockey, you probably think it is an object, not a bool :) isJockey is more descriptive IMO.