new workflow is aborted

Hi guys! i have a problem using gitrise for firts time!
i use the script with the required options and the script say "ERROR: Not Found%". I have a MAC with OS 14.1.2 (23B92) and gitrise version 0.8.1
Thx!

Hi @sun022-01,
Thanks for your interest in Gitrise!
That error is in fact coming from Bitrise. Gitrise has made the call to Bitrise, but the project or workflow has not been found on Bitrise. Please make sure the info you're passing to Gitrise is correct. Also, make sure the access token you are using has access to your project on Bitrise.
Hope this help you!

Closing this issue as it's resulted from passing incorrect parameters to Gitrise and no change is needed on Gitrise itself.


[Control] About one coefficient in MPC controller

Hola,
I'm writing this to deliver my doubt on the calculation of speed_error_change_rate.
@modules/control/controller/mpc_controller.cc
matrix_b_(5, 1) = -1.0;  // Ln. 183
as well as
matrix_c_(5, 0) = 1.0; // Ln. 187
Could you please exhibit how these 2 parameters are obtained? Especially matrix_c_(5, 0) where even the physical unit won't fit for both sides of the equation, i. e. m*s^(-2) on LHS while s^(-1) on RHS.
Eager to hear your kindly reply.

@YasakaKanako @Capri2014
I think so too....
The longitudinal control equation int the apollo source code is

So, the LHS unit and RHS first term unit is [m/s^2]. However, RHS second term is [rad/s].


Formatting of the cmd_output for Cyrilic symbols

Hey @pgaskin
When I use cmd_output to run a command that returns Cyrillic symbols, the formatting looks different than for Latin ones (attached photo). To me, it looks like there's a space inserted after every letter.
My use case is to print translations received from DeepL, and all the available screen real estate matters 😅
Is it possible to display the Cyrillic symbols without offsets?

UTF-16 shenanigans?

You can reproduce that using a UTF-8 encoded config file
menu_item :selection_search :Test Cyrillic :cmd_output :9999:echo 'Привет'

If that's a shenanigan issue, then I'm not sure what to do with it.

Can you upload the syslog showing the execution of that item?

Hey @pgaskin
My config:
menu_item :main :Test russian :cmd_output :9999:echo 'Привет'
menu_item :main :Dump Syslog :cmd_spawn :logread > /mnt/onboard/.adds/syslog_bug.log

What I did:

Modified the config (as stated above)
Launched kobo
In the NickelMenu, pressed Test russian

Attaching the syslog_bug.log, I only stripped the first lines with book names.
Thanks a lot for looking into this!
syslog_bug.log

This looks very similar to problems we've seen before when Cyrillic is used as part of the GUI. I think it's caused when the GUI is trying to use Cyrillic in a widget which is expecting to use the built-in default sans-serif font, Avenir.
I'm not sure whether it's because the Cyrillic glyphs in Kobo's version of Avenir are useless or whether it's because the Kobo can't find any Cyrillic glyphs and uses a fallback font instead, which will probably by one of the built-in CJK fonts.
Whatever the reason, one way it can be "fixed" is by replacing the default system 'Avenir Next' with a font better suited to Cyrillic. On my own devices I've replaced Avenir with a copy of Calibri masquerading as 'Avenir Next'. I didn't do this because I need Cyrillic but I have just tested
menu_item :main :Test russian :cmd_output :9999:echo 'Привет'
in NickelMenu and the output looks fine to me, i.e. no "double spacing" effect.

Hey @pgaskin
When I use cmd_output to run a command that returns Cyrillic symbols, the formatting looks different than for Latin ones (attached photo). To me, it looks like there's a space inserted after every letter.
My use case is to print translations received from DeepL, and all the available screen real estate matters 😅
Is it possible to display the Cyrillic symbols without offsets?

Hi, would you be so kind and share the line you used to get this output of translation. Are you using KOreader or anything similar or is a native reader?

Hey @n3w6i3 I'm using the following command:
menu_item :selection_search :DeepL Translation :cmd_output :9999:/usr/bin/wget --header='Authorization: DeepL-Auth-Key $MY_KEY' --post-data='target_lang=EN&text={1|S|%}' -qO - https://api-free.deepl.com/v2/translate | jq '.translations[0].text' | fold -s -w 55

No KOreader, pure Kobo + nickelmenu

Hey @n3w6i3 I'm using the following command:
menu_item :selection_search :DeepL Translation :cmd_output :9999:/usr/bin/wget --header='Authorization: DeepL-Auth-Key $MY_KEY' --post-data='target_lang=RU&text={1|S|%}' -qO - https://api-free.deepl.com/v2/translate | jq '.translations[0].text' | fold -s -w 55

No KOreader, pure Kobo + nickelmenu

I tried it (used my api key) but i get the following message instead of translation: /bin/sh: jq: not found... any idea how to solve it?

I ship it as part of KoboStuff, so i imagine @degiz has that installed ;).

I ship it as part of my KoboStuff bundle of tools, so I imagine @degiz has that installed ;).

Thank you all, it works like a charm 😁

I ship it as part of my KoboStuff bundle of tools, so I imagine @degiz has that installed ;).

Thank you all, it works like a charm 😁

Wait, so how did you fix it?

I ship it as part of my KoboStuff bundle of tools, so I imagine @degiz has that installed ;).

Thank you all, it works like a charm 😁

Wait, so how did you fix it?

By installing the whole package from NiLuJe.

By installing the whole package from NiLuJe.

Ahhh I get it, rookie mistake. Now it works, this is very cool! Thanks :D

@degiz Hi!
Have you fixed it? Does replacing the default system 'Avenir Next' with something like 'Calibri' helped you? If so, how can I do it? Even after fixing 'double spacing' effect I still have a problem: Cyrillic (latin works as should) symbols at the end of the line are disappearing sometimes (pretty often but not always). E.g. with the longer text:
echo "\"Давай сходим в парк,\" - предложил Даниил, улыбаясь. \"Почему бы и нет,\" - согласилась Наташа, радостно кивнув. Они вышли из дома и направились к парку, наслаждаясь свежим воздухом. \"Какой замечательный день,\" - восхищенно вздохнула Наташа, глядя на яркое солнце. \"А вот и городское озеро,\" - указал Даниил, указывая на водную гладь. Они прогуливались, наслаждаясь природой и обществом друг друга. \"Скоро будем здесь снова,\" - обещала Наташа, улыбаясь своему другу. \"Да, это было замечательно,\" - согласился Даниил, чувствуя прилив счастья." |  fold -s -w 50
@jackiew1 maybe you can also test it in your enviroment


#18 completableDeferred initialization check

if checks added for the lateinit property completableDeffered. Fragment methods called in its lifecycle whether eazypermissions methods used or not, so it is possible that property stays not initialized

Closing this since suggested changes already been merged to master along with other changes.
@brnogz Thanks for the fix :)


OPENAI_API_KEY appended in .env every start [Bug]

OPENAI_API_KEY entry appended in /var/www/html/.env inside the container every app start.

Fixed via #16


Fix lets encrypt renewal

Current lets encrypt renewal cron job will fail because when building the ssl.conf the lets encrypt block is not included.  It needs to be included in the ssl configuration so that renewals will process.

All I have to do is ensure this block:
 # LetsEncrypt acme-challenge
        location ^~ /.well-known/acme-challenge {
                root /home/{webuser}/www/letsencrypt;
                try_files $uri $uri/ =404;
        }

Is included in the built ssl.conf after the initial letsencrypt cert creation.  This may mean modifying the nginx conf branch pulled in.

Actually it looks like switching to use certbot would be the better investment of time here since it requires no manual modification of the nginx conf with the nginx plugin and takes care of automatically adding crons for renewals.
See: https://certbot.eff.org/#ubuntuxenial-nginx
So something like this is all that would be needed via ansible:
sudo certbot --nginx certonly

I decided to just fix the letsencrypt implementation because doing certbot will require more work for various distro flavours.


Popups interfer with map display on mobile

http://content.screencast.com/users/prettybrickcity/folders/Jing/media/80ddc893-dd75-40c8-b55b-867a6c23b4e6/00000162.png

Since we're no longer using popups, this can be closed.


Safe auto-dismissals for Dependabot alerts (low impact npm devDependencies)

Summary
Dependabot alerts now proactively filters out false positives from your alert list -- so true positives are no longer buried under a mountain of unimportant alerts.
In this release we will automatically dismiss alerts for npm devDependencies where the vulnerability is low impact. Low impact alerts are unlikely to be exploitable or may only have limited effects like slow builds or long-running tests.

🚢 This has shipped: https://github.blog/changelog/2023-05-02-dependabot-alerts-now-automatically-dismiss-false-positives-for-npm-public-beta.
Leaving open to track for GHES release.

This shipped with GHES 3.11<EMAIL_ADDRESS>


Replace ids with urls

Here are the hub-common changes to move us away from parsing UUIDs from URLs, and assembling URLs from UUIDs. For those methods used by hub-jira, I wrote a replacement. For those not used by hub-jira, I added a TODO comment saying the replacement needs to be written. In both cases, I've deprecated the method.
There are some breaking changes because I removed UUID fields (replacing them with URL fields). I believe the breaking changes are limited to VulnerabilityRestService, NotificationContentItem and its subclasses, notification Transformers, ProjectVersion (plus maybe a handful methods that now throw an additional exception type).
To get the URL or relative URL for any HubItem, call item.getMeta().getHref() or .getRelativeHref().
To get the URL or reletive URL of a ProjectVersion, call getUrl() or getRelativeUrl().

@ekerwin @psantos1113 Hi guys, could you review my description (above) at least, and the code too if you're interested? I'm done for today, but we can discuss Tuesday if you want.

Coverage increased (+0.5%) to 65.331% when pulling c1cb26583156fd03a868d37ee5242d5aac0e4153 on replaceIdsWithUrls into 6d404813a660de6ad957b8c4b2328003c390f21a on master.

@psantos1113 Regarding Rob's comment above: I believe you are about to remove the aggregators that use VulnerabilityRestService.getComponentVersionVulnerabilities(), is that correct?

Coverage increased (+0.005%) to 64.814% when pulling 4d39f72d8b024facbebcc1dab5b85b6f34c4798e on replaceIdsWithUrls into 6d404813a660de6ad957b8c4b2328003c390f21a on master.


Conda package not installing

I am trying to use the agent to reproduce my local setup (that is built using conda). However, it does not install a vital package, which is "rdkit", found on conda-forge.
To start the agent I use:
 clearml-agent daemon --queue default
I have modified the clearml.conf file with these changes:

added the agent {... section from the documentation
added: type: conda,
added post_packages: ["rdkit", ]

Looking at the console output in the WebUI, the last line is:
Traceback (most recent call last):
  File "run.py", line 12, in <module>
import rdkit
ModuleNotFoundError: No module named 'rdkit'
Scrolling higher up, I see:
Conda: Trying to install requirements:
['pytorch~=1.9.0', 'graphviz', 'python-graphviz', 'kiwisolver']
Conda: Installing requirements: step 2 - using pip:
['clearml==1.0.5', 'matplotlib==3.4.3', 'git+https://github.com/lessw2020/Ranger21.git@0a906ef9df4a4c394a48e5778b2b94f2c8e1ce8e#egg=ranger21', 'torchvision==0.10.0', 'tqdm==4.62.2', 'x_transformers==0.16.1']
So, it's not finding rdkit from the environment, the import in the main python script, or the clearml.conf file.  What should I try next?
I tried editing the "INSTALLED PACKAGES" in the "EXECUTION" section of the WebUI to add rdkit, but that seems to be for Pip. Can I force it to use conda there?

Hi @dnhkng
A few questions:

Are both the "development" environment and the agent running conda?
Why do you have to always install the "rdkit" package by the agent ? How come it is not detected by the auto package discovery on the development environemnt
Is "rdkit" a conda package or a python package (i.e. installed with "conda install" or "pip install" ?
Did you try manually adding the missing package with Task.add_requirements("rdkit") ? (notice you have to call it Before you call Task.init(...))

I tried editing the "INSTALLED PACKAGES" in the "EXECUTION" section of the WebUI to add rdkit, but that seems to be for Pip. Can I force it to use conda there?

Yes, that is correct, if running with conda you cannot actually edit the Installed packages from the UI, as it actually stores the conda/pip combination you had on the development environment in two different sections, and the assumption is that conda will be able to resolve it by itself ...

Thanks for the reply. To answer your questions:

they are both on the same physical computers. I'm using the agent for the first time, so I'm trying to test how this works. As I have 2x RTX Titans in this develooment machine, I was hoping to use the web interface to schedule and monitor experiments easily (even though it's a local machine).

maybe I'm misunderstanding the question, but doesn't the agent build a fresh docker, so rdkit would always have to be added to the image?

it's installed with Conda. It's not in the module that imports clearml, but a module I import in the main script. However, I see all the scripts are parsed correctly by clearml (at least for the Pip requirements).  I tried importing rdkit in the main script (I just got it to print rdkit.version), but it that didn't help. This was done before instantiating the clearml task, could this be the issue?

I didn't see that command in the documents, I'll give it a go when I get back to the computer.

I'll update when I get the new command tested.

As I have 2x RTX Titans in this develooment machine, I was hoping to use the web interface to schedule and monitor experiments easily (even though it's a local machine).

That is a great idea. The man caveat is that you have to decide whether you want the two GPUs allocated for the same Task, or you want a GPU per Task. Specifically for our example, you can spin a single instance of the agent, allocating both GPUs to every Task the agent pulls, or spin two agents, each one with its own GPU.

but doesn't the agent build a fresh docker, so rdkit would always have to be added to the image?

You are correct, but in order for the agent to install the package inside the container, you either have the package listed in the "installed packages" section (which is fully compatible with requirements.txt format) or if you clear the  "installed packages" section, the agent will look for the requirements.txt file inside the git repository and use it.

... but a module I import in the main script.

That is the culprit, clearml will automagically list al the directed modules you import. The assumption is that the module you are using will have its requirements parsed when the agent installs it (e.g. install tensorflow will implicitly install numpy because it is in the part of the tensorflow wheel requirements).
I suspect the issue is that the module you are importing did not list rdkit as a required package, hence it was not installed by the agent.
In order to solve the issue, you can:

Add the missing  rdkit to the requirement section of your module
Specify to clearml that it needs to add rdkit anyhow:

# Call Before the Task.init call
Task.add_requirements('rdkit')

Tell ClearML to list the enitre conda/pip environment (instead of just the directly imported packages) Use this one with care, some python packages will not able to be installed in a different container / os

# Call Before the Task.init call
Task.force_requirements_env_freeze()

I tried importing rdkit in the main script (I just got it to print rdkit.version), but it that didn't help. This was done before instantiating the clearml task, could this be the issue?

That should have worked... rdkit listed in the python packages when you run pip freeze

OK, I've had a go at this, and it's not working well.
# Call Before the Task.init call
Task.add_requirements('rdkit')

If I use this command, I see that it tries to install using Pip instead of Conda:
Conda: Installing requirements: step 2 - using pip:
['matplotlib==3.4.3', 'rdkit']

Unfortunately, 'rdkit' is only available on conda.
If I use the other command:
# Call Before the Task.init call
Task.force_requirements_env_freeze()

I get rdkit installed, but then the GPU is not detected.
I'm starting my agent with:
$ clearml-agent daemon --gpus all --queue default
and in my script:
torch.cuda.is_available()
returns False.

Nice @dnhkng !
I think we should make sure add_requirements will support conda packages as well as pip.


$splice keyword

+1
This would make it easier to use baobab in drag and drop. Here's a rough idea for moving items from react-dnd.
moveCard(id, afterId) {
    const { cards } = this.state;

    const card = cards.filter(c => c.id === id)[0];
    const afterCard = cards.filter(c => c.id === afterId)[0];
    const cardIndex = cards.indexOf(card);
    const afterIndex = cards.indexOf(afterCard);

    // in baobab's case this would go through cursor but you get the idea
    // this uses immutable.js syntax now
    this.setState(update(this.state, {
      cards: {
        $splice: [
          [cardIndex, 1],
          [afterIndex, 0, card]
        ]
      }
    }));
}

moveCard would get triggered as you move items around.

@bebraw, cursor.splice and command $splice now exists in v1.

@Yomguithereal Great! :+1:

v1 published.


This app contains an embedded provisioning profile that is not associated with your account.

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
ERROR ITMS-90159: "Invalid provisioning profile. This app contains an embedded provisioning profile that is not associated with your account. Please use a provisioning profile associated with Team ID 42VS245LG2."
ERROR ITMS-90179: "Invalid Code Signing. The executable 'Payload/MYFramework' must be signed with the certificate that is contained in the provisioning profile."
[10:02:30]: ERROR ITMS-90159: "Invalid provisioning profile. This app contains an embedded provisioning profile that is not associated with your account. Please use a provisioning profile associated with Team ID 123456789."
ERROR ITMS-90179: "Invalid Code Signing. The executable 'Payload/Myframework' must be signed with the certificate that is contained in the provisioning profile."
Return status of iTunes Transporter was 1: ERROR ITMS-90159: "Invalid provisioning profile. This app contains an embedded provisioning profile that is not associated with your account. Please use a provisioning profile associated with Team \nERROR ITMS-90179: "Invalid Code Signing. The executable 'Payload/MYFramework' must be signed with the certificate that is contained in the provisioning profile."
The call to the iTMSTransporter completed with a non-zero exit status: 1. This indicates a failure.
Environment
 fastlane environment
Stack

Key
Value

OS
10.11.6

Ruby
2.4.0

Bundler?
false

Git
git version 2.9.3 (Apple Git-75)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.11.6 (15G31)

Ruby Lib Dir
/usr/local/Cellar/ruby/2.4.0/lib

OpenSSL Version
OpenSSL 1.0.2j  26 Sep 2016

Is contained
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.1

System Locale

Error

No Locale with UTF8 found

fastlane files:
`./fastlane/Fastfile`
# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "2.6.0"

default_platform :ios

platform :ios do

  # 1
  desc "Creating a code signing certificate and provisioning profile"
  # 2
  lane :provision do
    # 3
    produce(
      app_name: ‘App For iPad’,
      language: 'English',
      app_version: '1.0',
      sku: 'abc123'
    )
    # 4
    cert
    # 5
    sigh(force: true)
  end

  error do |lane, exception|
  # This block is called if there was an error running a lane.
  end

  desc "Create ipa"
  lane :build do
    gym
  end

 desc "Upload to App Store"
  lane :upload do
    deliver
  end

end```
</details>

<details><summary>`./fastlane/Appfile`</summary>

```ruby
app_identifier "MyIdentifier" # The bundle identifier of your app
apple_id<EMAIL_ADDRESS># Your Apple email address

team_id "TEAMID"  # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Appfile.md

fastlane gems

Gem
Version
Update-Status

fastlane
2.11.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.1.0

slack-notifier
1.5.1

i18n
0.7.0

json
1.8.6

thread_safe
0.3.5

tzinfo
1.2.2

minitest
5.10.1

activesupport
<IP_ADDRESS>

CFPropertyList
2.3.4

claide
1.0.1

colored
1.2

nanaimo
0.2.3

xcodeproj
1.4.2

rouge
1.11.1

xcpretty
0.2.4

terminal-notifier
1.7.1

unicode-display_width
1.1.3

terminal-table
1.7.3

plist
3.2.0

public_suffix
2.0.5

addressable
2.5.0

multipart-post
2.0.0

word_wrap
1.0.0

babosa
1.0.2

highline
1.7.8

commander
4.4.3

excon
0.54.0

faraday
0.11.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20161129

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.0.1

gh_inspector
1.0.3

uber
0.0.15

representable
2.3.0

retriable
2.1.0

mime-types-data
3.2016.0521

mime-types
3.1

hurley
0.2

little-plugger
1.1.4

multi_json
1.12.1

logging
2.1.0

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

google-api-client
0.9.23

mini_magick
4.5.1

multi_xml
0.6.0

rubyzip
1.2.0

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.1.2

bundler
1.14.2

faraday_middleware
<IP_ADDRESS>

io-console
0.4.6

openssl
2.0.2

generated on: 2017-01-25

While uploading the app on app store I am getting this issue. I have only one developer account and one team. By using this credentials only I have uploaded the app on store 1 year before.
My account has only one TEAM ID then why it is not validating profile.
Please help me on the same
Thank You

can you check in xcode that the right team is selected on all targets?

could you provide the output of your fastlane by calling it with --capture_output ?

I'm going to close this issue since it has been open for a while without follow up.


Extension support

An application can also provide different "tones" of the same locales (e.g casual, formal), utilizing -x- private tag. Given a set of locales with extensions and what the user preference might be, find the best matching locales.
Ref: https://github.com/formatjs/formatjs/discussions/2642

Discussion from 2021-03-11 meeting: https://github.com/tc39/ecma402/blob/master/meetings/notes-2021-03-11.md#extension-support-6


bco-specification.md delete section 3.4 or specify JSON Schema

bco-specification.md section 3.4 references data-typing.md which was discontinued in e9686c163ac9c1ebf9f5aa4761a26115195ba867

3.4 Appendix V Data typing
The conceptual schema for BCO creation can be defined in ??? schema language.
Specifications:

Data Typing

bco-specification.md section 3.4 references data-typing.md which was discontinued in e9686c1

Removed this text

3.4 Appendix V Data typing
The conceptual schema for BCO creation can be defined in ??? schema language.
Specifications:

Data Typing

I think it would be good to explain JSON as our chosen schema and to also mention the definitions
https://github.com/biocompute-objects/BCO_Specification/blob/9fd348e71a5650b6e024dbd29e0477627864cb62/schemas/biocomputeobject.json#L19-L127
Maybe in the Top Level?
Anyone  else want to take a crack at it?


[merge] Request a means to easily merge content in 3-way merge from both the left and right sides

In the merge addon, one can copy content from the left or right side into the middle editor. However, there's no mechanism to allow for copying content in from both sides- in particular as the difference updates immediately upon any change in the center editor. Having a mechanism for inserting content from both sides could be useful in a 3-way merge in case content modifications from both versions is desired.
Example: https://jsfiddle.net/pb163xmu/3/

That'd be a new, non-trivial feature (it'd require some kind of merge algorithm), which I'm not interested in adding to this addon at the moment.


Unused, empty files in extension

See the second note in https://github.com/microsoft/pxt-arcade/issues/4977 regarding cleanup of extra files in the extension repo, i.e., main.ts.

Thanks for the heads-up!  I see that main.ts and assets.json are blank (and therefor presumably unused), but it's not clear to me what other files we might not need since the project structure just comes from using the /save to github/ feature and I don't really know what all of the files are doing.
Do you have any pointers?  Other files to clean out, or some documentation to look at to understand what I should be getting rid of and what I should be keeping?
thanks!

https://github.com/microsoft/pxt-microturtle and https://github.com/microsoft/pxt-neopixel are examples of extension layouts for micro:bit.


upgrade to node 20

This PR upgrades  the node-version to 20

The cache action uses check-dist reusable workflow and it still uses Node 16, so we should either wait until Node.js runtime version is updated or update check-dist workflow by explicitly specifying Node 16

Duplicate of #7


prep function deprecated in the latest HiCRep version

Greetings.
According to HiCRep's github (https://github.com/TaoYang-dev/hicrep/blob/master/NEWS), "prep" function is now deprecated.
Is it possible to update HiCRep wrapper correspondingly?
Best regards,
Wenny

I got the sam question. This may  be caused by omit step of "prep". Change into the old code so that can run well.
Omit the line 93,94 of /3DChromatin_ReplicateQC/wrappers/HiCRep/HiCRep_wrapper.R
add new line after that ."SCC.out = get.scc(m1_big[,-c(1:3)], m2_big[,-c(1:3)], resol, h, 0, maxdist)"
I am not sure the code run rightly, but it run.

You also can download from _https://genome.cshlp.org/content/early/2017/08/30/gr.220640.117.full.pdf+html_ to download the Supplemental_hicrep_1.0.1.tar.gz(https://genome.cshlp.org/content/suppl/2017/10/06/gr.220640.117.DC1/Supplemental_hicrep_1.0.1.tar.gz)

I got the sam question. This may be caused by omit step of "prep". Change into the old code so that can run well. Omit the line 93,94 of /3DChromatin_ReplicateQC/wrappers/HiCRep/HiCRep_wrapper.R add new line after that ."SCC.out = get.scc(m1_big[,-c(1:3)], m2_big[,-c(1:3)], resol, h, 0, maxdist)" I am not sure the code run rightly, but it run. After test, I think it work well,due to the code used in old commit.
我也有同样的问题。这可能是由于省略了“准备”步骤所致。换成旧代码，这样就可以运行得很好。 省略/3D色谱复制品QC/包装器/HiCRep/HiCRep包装器的第93、94行。R 在此之后添加新行。“如果您的计算机上有多个文件，请使用以下命令：” 我不能肯定代码是否正确运行，但它运行了。经过测试，我认为它工作得很好，由于代码中使用的旧提交。

This cause the #7 come out again, the best is to downgrade the version


Removed unnecessary custom context processor exposing the DEBUG Template Context Variable

Description
Remove the custom context processor, {{ cookiecutter.project_slug }}.utils.context_processors.settings_context,  that was exposing a Template Context Variable, DEBUG. This is because the first context processor in the list, settings.TEMPLATES.context_processor enabled django.template.context_processors.debug, which exposes the same variable, debug.
Checklist:

[X] I've made sure that tests/test_cookiecutter_generation.py is updated accordingly (especially if adding or updating a template option)
[X] I've updated the documentation or confirm that my change doesn't require any updates

Rationale
This is just redundant and doesn't provide any advantage and hence should be removed. The code will also become easier to understand.

This context processor is meant to be the place where one would expose settings they need in templates (#2203). In the stock installation, it just exposes DEBUG since exposing all settings was deemed dangerous (#2694), which is true makes it redundant. Maybe we should expose another setting than DEBUG, which would actually be useful? I don't have anything in mind, though...
On the other hand, I appreciate that it might not be used in all projects, and it's quite a simple thing to add when needed, having it might not be so valuable after all 🤔

Yes agreed. I also thought it was meant to be a way for others to get into it

Personally, I also use Sentry for JS and send SENTRY_DSN and other Sentry env vars to templates/base.html using context_processors. Perhaps we should also add SENTRY for JS to this project and then update context_processors to send the required info to the templates?

I think having it as an option for people won't hurt, even if they never use it. Lots of features of Django are never really known until needed OR if someone has seen it before. In other words, we present a new option for a dev to remember for the future when needed. Good creativity wise.

Personally, I also use Sentry for JS and send SENTRY_DSN and other Sentry env vars to templates/base.html using context_processors.

When I add Sentry for JS, I usually setup 2 separate projects for JS and Django, since Sentry asks for the language when setting up the project. Isn't the Sentry dashboard messy when both are using the same DSN? I really never tried to mix them up...

I actually never thought of using it as a separate project! It does get a bit messy but usually, it's easy to filter JS errors from other errors.

OK, I've come around that PR again, and I'm now +1 on removing this. Since it's not used in the stock installation, it's adding some unneeded code to projects not using it while not providing a lot of value to people who do use it, as creating a context processor is not very complex thing to create (it's less that 10 lines of code after all).
Anyone object to this change? If so, now is the time to speak up 😄


Fix grpc++ mac build.

This PR shall be backported to master, too?

Just saw #3186. Sorry for the noise.


Not able to pull up Overview page from the "pull request" view:

Testing #4764
I am not able to pull up Overview page from the "pull request" view:

Clicking on the pull request title does nothing.
I see this error:

notificationsAlerts.ts:40 TypeError: Cannot read properties of undefined (reading 'title')
    at og.getTreeItem (c:\Users\kanadig\.vscode-insiders\extensions\github.vscode-pull-request-github-0.63.2023042509\dist\extension.js:1710:48743)
    at c:\Users\kanadig\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:98:65297
    at Array.map (<anonymous>)
    at y.R (c:\Users\kanadig\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:98:65262)
    at y.getChildren (c:\Users\kanadig\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:98:61563)
c @ notificationsAlerts.ts:40

@karthiknadig I've fixed the error and the fix will be available in the next pre-release build, though I wouldn't expect that to cause the PR description to not show. Can you install the latest pre-release, repro the issue, then share the output of GitHub Pull Request?

Just ran into this issue myself. It's an issue with using SSH remotes.


Uncaught ReferenceError: __ is not defined

hi
file
import i18n from 'i18n-2' export class Navbar extends Component { constructor(props) { super(props) } render() { return (
<IndexLink to="/" activeClassName={styles.active}>{__('Home')}</IndexLink>
 ) }
app.js
i18n.expressBind(app, { locales: ['ru', 'en'], directory: __dirname + '/locales', fallbacks: true, extension: '.js', defaultLocale: 'ru', cookieName: 'lang', register: global, api: { '__': 't', '__n': 'tn' } }); app.use(function(req, res, next) { if(req.query.lang) { req.i18n.setLocaleFromQuery(); res.cookie('lang', req.i18n.getLocale()); } else { req.i18n.setLocaleFromCookie(); } next(); });

Hi,
Please make your next messages a bit clearer :)
You're using i18n in a express (node.js, server-side) and expect it to be available in a react (browser, client-side) ? This is not how it works. Either use templates, or a translation api to use i18n-node-2

I have combined react and express, the fact is that he translates the phrase, but gives an error in the console. Tell me how to use api? can show an example of visual
Thanks

Documentation is available in the README file : https://github.com/jeresig/i18n-node-2
Again, you might be declaring i18n in express and using __ from React on client side. You can not « combine react and express ».
I'm closing this as this is not related with i18n


Added some institutuions in TamilNadu, India

@johnhydenc lib/domains/edu/psgtech: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

please close this pr.
i made an account transfer to this account.
i'm not able to make any changes to it.
I'll make a new PR.

@johnhydenc lib/domains/edu/psgtech: this request has an incorrect file path format, please ensure that you are familiar with this repository rules located at the bottom of this page and change your request accordingly. If you don't know how to change request, close it and open a new one. If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

Oops sorry. forgot the txt extension lol.
Fixed the issues. (Recently made an repo transfer tho, thus the new account)
Should be good now i guess.

@johnhydenc Thank you. Your request contains significantly more domains that a usual one. Therefore, it will be processed later, and it'll take more time than usual. I appreciate your patience and understanding.

It's okay.
I just added the institutions those I've personally visited and has a Computer Science and or related courses.

@johnhydenc Thank you for your patience and the contribution!
I reviewed the schools, and had to remove two of them: Loyola Institute of Technology and Science, Thovalai and Marthandam College of Engineering and Technology, because their website do not provide enough up-to-date information about their IT-related courses.
All other schools are eligible for being added to this repository, thus I corrected and merged your pull request.
Thank you!


Add information on contributing, community support and commercial support to project site

I think there's a contributing include in Spring Cloud build

I have checked the existing sites and there's nothing being added there at this point.

I think there's a contributing include in Spring Cloud build

I have checked the existing sites and there's nothing being added there at this point.

I think there's a contributing include in Spring Cloud build

Ok, actually, I see this in OpenFeign now, but not in Netflix.

I think there's a contributing include in Spring Cloud build

Ok, actually, I see this in OpenFeign now, but not in Netflix.


[Bug] Does not provide an export named 'default' (3.4.2)

Environment
Nuxi 3.4.2                                                                                8:34:16 PM
8:34:16 PM
RootDir: [...]/Desktop/nilos-project/nilos-client                                                                                8:34:16 PM
Nuxt project info: (copied to clipboard)                                                                                8:34:16 PM

Operating System: Darwin
Node Version:     v16.14.2
Nuxt Version:     3.4.2
Nitro Version:    2.3.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      ssr, target, app, router, css, postcss, plugins, components, vite, modules, runtimeConfig, build, builder
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
Install version of Nuxt 3.4.2 and try to build an existing project, the project is not building I have this issue:
caught SyntaxError: The requested module '/_nuxt/components/Alert/config.ts' does not provide an export named 'default' (at components.client.mjs:2:10)

Here is my file
import { AlertVariant } from '@/components/Alert/typing';

export const bgColors = {
  [AlertVariant.SUCCESS]: 'bg-green-50',
  [AlertVariant.WARNING]: 'bg-yellow-50',
  [AlertVariant.ERROR]: 'bg-red-50'
};

export const iconColors = {
  [AlertVariant.SUCCESS]: 'text-green-600',
  [AlertVariant.WARNING]: 'text-yellow-600',
  [AlertVariant.ERROR]: 'text-red-600'
};

export const textColors = {
  [AlertVariant.SUCCESS]: 'text-green-600',
  [AlertVariant.WARNING]: 'text-yellow-600',
  [AlertVariant.ERROR]: 'text-red-600'
};

export const buttonColors = {
  [AlertVariant.SUCCESS]:
    'bg-green-50 text-green-500 hover:bg-green-100 focus:ring-offset-green-50 focus:ring-green-600',
  [AlertVariant.WARNING]:
    'bg-yellow-50 text-yellow-500 hover:bg-yellow-100 focus:ring-offset-yellow-50 focus:ring-yellow-600',
  [AlertVariant.ERROR]: 'bg-red-50 text-red-500 hover:bg-red-100 focus:ring-offset-red-50 focus:ring-red-600'
};

Note: The build is working well in 3.4.1
Describe the bug
The project is supposed to build
Additional context
No response
Logs
No response

a similar problem after updating for the version 3.4.2
in version 3.4.1 everything works
------------------------------
- Operating System: Darwin
- Node Version:     v16.18.0
- Nuxt Version:     3.4.2
- Nitro Version:    2.3.3
- Package Manager<EMAIL_ADDRESS>- Builder:          vite
- User Config:      runtimeConfig, css, app, ssr, telemetry, sourcemap, modules, pwa, types, nitro, experimental, components
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -
------------------------------

caught (in promise) SyntaxError: The requested module '/_nuxt/components/channel/query/index.ts' does not provide an export named 'default' (at components.client.mjs:10:10) О
// and other similar named exports

export const DeleteDraftMessage = gql`
  mutation deleteDraftMessage($channelId: uuid!, $authorId: uuid!) {
    deleteMessages(where: {channelId: {_eq: $channelId}, draft: {_eq: true}, authorId: {_eq: $authorId}}) {
      affected_rows
    }
  }`

I had the same issue. Moving the TS file to outside of the "components" directory to "/utils" for example solved the issue.

@elishalon thanks for you answer, I noticed it also yesterday - but I don't see any reason of why the config.ts file couldn't be locate in the component himself 👍🏻

+1
Same error
Has private composable for some of my components located in ~/components/interface/heroImage/_composables/assetImage.js
Result
Uncaught SyntaxError: The requested module '/_nuxt/components/interface/heroImage/_composables/assetImage.js' does not provide an export named 'default'

Nuxt 3.4.2

I have the same issue, the files that it says it does not have have default export but does it need to be?

I think I'm seeing the same. In Firefox the error is something like Uncaught SyntaxError: ambiguous import, while in Chrome it's The requested module does not provide an export named 'default'

To avoid the error in the current 3.4.2 update, you can restrict the file extensions that should be registered as components in  nuxt.config.ts:
export default defineNuxtConfig({
  components: [
    {
      path: '~/components',
      extensions: ['.vue'],
    }
  ]
})


Java implementation of ClientConnector

This PR provides a Java based implementation of the ClientConnector of Dolphin.
It introduces a general interface that is implemented by the old Groovy based implementation and a new Java based implementation. Dolphin Platform is internally based on the Java implementation.

This change is

Preparation for #202

Coverage decreased (-0.2%) to 69.476% when pulling eff9dc96818e6d9f1efab22855e96d5e98d95b28 on client-connection-java into 57c722c9834a80c9a08cbea136ccf4c65e0244fd on master.

Reviewed 5 of 17 files at r1, 4 of 8 files at r2.
Review status: 9 of 19 files reviewed at latest revision, 10 unresolved discussions, some commit checks failed.

platform-extras/dolphin-platform-spring-controller-tester/src/main/java/com/canoo/dolphin/test/SpringJUnitControllerTest.java, line 37 [r2] (raw file):

@SpringApplicationConfiguration(classes = DolphinPlatformSpringTestBootstrap.class)
@DirtiesContext(classMode = DirtiesContext.ClassMode.AFTER_EACH_TEST_METHOD)
public abstract class SpringJUnitControllerTest extends AbstractJUnit4SpringContextTests implements ControllerTest {

breaks binary compatibility?

platform-extras/dolphin-platform-spring-controller-tester/src/main/java/com/canoo/dolphin/test/impl/DolphinPlatformSpringTestBootstrap.java, line 97 [r2] (raw file):

        //Currently the event bus can not used in tests. See https://github.com/canoo/dolphin-platform/issues/196
     //   config.getClientExecutor().submit(new Runnable() {

why the change from a lambda to an AIC?

platform-extras/dolphin-platform-spring-controller-tester/src/test/java/com/canoo/dolphin/test/TestNGDemoTest.java, line 54 [r2] (raw file):

    }

   // @Test

why commenting out this test?

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 43 [r2] (raw file):

import java.util.stream.Collectors;

public abstract class AbstractConnector implements ClientConnector {

please do not remove the comments

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 86 [r2] (raw file):

    }

    private void processCommunication() throws InterruptedException {

why don't you split this method in multiple ones?

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 123 [r2] (raw file):

    private void handleResponseCommand(Command command) {
        if (command instanceof DeletePresentationModelCommand) {

be aware, that if you simulate double dispatch with instanceof, you have to ensure to do it in the right order according to the hierarchy. If you migrate code to interfaces this here will break easily. If you change the hierarchy, this will easily break as well. Possible actions are making the Command classes final which is no option if a user is supposed to be able to define a custom command. Or moving this code into the command... which will surely require further changes to the code

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 267 [r2] (raw file):

        } else {
            //TODO
            LOG.warn("TODO");

this needs to be a severe problem, not just a warning

remoting/dolphin-remoting-client/src/main/groovy/org/opendolphin/core/client/comm/AbstractClientConnector.groovy, line 349 [r2] (raw file):

    /** listens for the pushListener to return. The pushListener must be set and pushEnabled must be true. */
    public void listen() {

public is not required

remoting/dolphin-remoting-client/src/main/groovy/org/opendolphin/core/client/comm/AbstractClientConnector.groovy, line 386 [r2] (raw file):

    boolean isPushEnabled() {
        return this.pushEnabled;
    }

in Groovy this is "boolean pushEnabled" in other words: just remove the "protected" from the field and you are done

remoting/dolphin-remoting-combined/src/test/groovy/org/opendolphin/core/comm/ClientConnectorPushTests.groovy, line 104 [r2] (raw file):

        }
        then:
        spoofCounter.await(2, TimeUnit.SECONDS)

why the increased wait?

Comments from Reviewable

Review status: 9 of 19 files reviewed at latest revision, 11 unresolved discussions, some commit checks failed.

platform-extras/dolphin-platform-spring-controller-tester/src/main/java/com/canoo/dolphin/test/impl/TestInMemoryConfiguration.java, line 38 [r2] (raw file):

        clientDolphin.setClientModelStore(new ClientModelStore(clientDolphin));

        ClientConnector inMemoryClientConnector = new AbstractConnector(clientDolphin, threadHandler) {

why not keeping using an inMemoryClientConnector?

Comments from Reviewable

Review status: 9 of 19 files reviewed at latest revision, 11 unresolved discussions, some commit checks failed.

platform-extras/dolphin-platform-spring-controller-tester/src/main/java/com/canoo/dolphin/test/SpringJUnitControllerTest.java, line 37 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> breaks binary compatibility?

What do you mean?

platform-extras/dolphin-platform-spring-controller-tester/src/main/java/com/canoo/dolphin/test/impl/DolphinPlatformSpringTestBootstrap.java, line 97 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> why the change from a lambda to an AIC?

Because the server part is compiled with Java 7. Since several enterprise projects still depend on 7 we do not use Java 8 code in the server part.

platform-extras/dolphin-platform-spring-controller-tester/src/main/java/com/canoo/dolphin/test/impl/TestInMemoryConfiguration.java, line 38 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> why not keeping using an inMemoryClientConnector?

This changed in another pull request to:
        InMemoryClientConnector inMemoryClientConnector = new SynchronousInMemoryClientConnector(clientDolphin, serverDolphin.getServerConnector());

platform-extras/dolphin-platform-spring-controller-tester/src/test/java/com/canoo/dolphin/test/TestNGDemoTest.java, line 54 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> why commenting out this test?

Because the event bus tests still not working

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 43 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> please do not remove the comments

I haven't removed any doc. This is a new class.

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 86 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> why don't you split this method in multiple ones?

This method? You mean the "handleResponseCommand", right?

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 123 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> be aware, that if you simulate double dispatch with instanceof, you have to ensure to do it in the right order according to the hierarchy. If you migrate code to interfaces this here will break easily. If you change the hierarchy, this will easily break as well. Possible actions are making the Command classes final which is no option if a user is supposed to be able to define a custom command. Or moving this code into the command... which will surely require further changes to the code

You are right. Until now the commands are still groovy code. I think we should do it when refactoring the commands to Java. I like the idea of defining concrete commands as final classes. By doing so developers could still provide specific commands by extending the non final basic class. See #228

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 267 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> this needs to be a severe problem, not just a warning

Right!

remoting/dolphin-remoting-client/src/main/groovy/org/opendolphin/core/client/comm/AbstractClientConnector.groovy, line 349 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> public is not required

Since we will refactor this to Java I think the public os ok.

remoting/dolphin-remoting-client/src/main/groovy/org/opendolphin/core/client/comm/AbstractClientConnector.groovy, line 386 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> in Groovy this is "boolean pushEnabled" in other words: just remove the "protected" from the field and you are done

Since we will refactor this to Java I think the change is ok. Or do you see any problems?

remoting/dolphin-remoting-combined/src/test/groovy/org/opendolphin/core/comm/ClientConnectorPushTests.groovy, line 104 [r2] (raw file):
Previously, blackdrag (Jochen Theodorou) wrote…
> why the increased wait?

Test breaks on Travis sometime. I think the complete test is a problem but until I haven't looked into it I just changed the wait to make it work on Travis

Comments from Reviewable

I think this change is too risky for 0.8.7. Please keep it for 0.9.0

I do not think so. We need a lot of changes in the connector when we create the new event bus. Since the event bus is planed for 0.8.8 we should better do this now.

Then I prefer if the changes needed for the event bus are done in the Groovy version first. That way you'll be sure where to look in case something breaks. Changing the base impl and add more behavior later is a risky gamble IMHO

Reviewed 2 of 17 files at r1, 2 of 8 files at r2, 1 of 2 files at r3.
Review status: 14 of 19 files reviewed at latest revision, 14 unresolved discussions, some commit checks failed.

platform-extras/dolphin-platform-spring-controller-tester/src/main/java/com/canoo/dolphin/test/SpringJUnitControllerTest.java, line 37 [r2] (raw file):
Previously, hendrikebbers (Hendrik Ebbers) wrote…
> What do you mean?

yeah, sorry, probably not important, since the usage of the class is never standalone

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 43 [r2] (raw file):
Previously, hendrikebbers (Hendrik Ebbers) wrote…
> I haven't removed any doc. This is a new class.

which means there are now 2 AbstractClientConnector classes, one com.canoo.dolphin.client.AbstractClientConnector and one in org.opendolphin.core.client.comm.AbstractClientConnector and if this is just a migration from Groovy to Java I would not have expected a new class to appear, In the review it is now very very difficult to see which AbstractClientConnector you actually refer to. This class here is clearly based on org.opendolphin.core.client.comm.ClientConnector in the original open doplhin. So it is not only two classes of the same name, they do a similar job as well. How can you be sure your tests will apply to this class, if they use the more or less unchanged class instead?

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 123 [r2] (raw file):
Previously, hendrikebbers (Hendrik Ebbers) wrote…
> You are right. Until now the commands are still groovy code. I think we should do it when refactoring the commands to Java. I like the idea of defining concrete commands as final classes. By doing so developers could still provide specific commands by extending the non final basic class. See #228

If they do that with this code, then they will get into the IllegalArgumentException. In open-dolhin all you have to do is to define a new handle method, which will use your new command class and the Groovy dispatch mechanism will take care of the rest. Since you do a multimethods unaware double dispatch, you do not have that option. In the end you totally block the path for new commands. If you want an easy solution, base this on ScriptBytecodeAdapter.invoke to do the double dispatch for you. This multimethod logic is also the reason why there was no @CompileStatic for this code in open-dolphin.

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 76 [r3] (raw file):

                    processCommunication();
                } catch (InterruptedException e) {
                    //Looks like I was kiled...

and if you had been killed, you can just continue the loop? I would have expected to see at least a "break;" here... or running.set(false);

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 86 [r3] (raw file):

    }

    private void processCommunication() throws InterruptedException {

In the open dolphin client connector this was basically startCommandProcessing and a protected method, giving you the ability to overwrite this. This is not the case here. Why not?

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 90 [r3] (raw file):

        List<Command> commands = toProcess.stream().map(t -> t.getCommand()).collect(Collectors.toList());

        LOG.trace("Sending batch of size {0}", commands.size());

why did you remove the "C:" in the log statements here? It was easy to filter out server and client logs using this before

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 103 [r3] (raw file):

        uiThreadHandler.executeInsideUiThread(() -> {
            processResults(answer, toProcess);

how does this uiThreadHandler treat exceptions produced during processResults? Because the open dolphin ClientConnector did have logic for this.

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 120 [r3] (raw file):

            }
        });
    }

I do spot here some differences in the logic compared to what is in open-dolphin ClientConnector. For example onFinished here always gets an empty list, while in the open-dolphin it gets a list of the involved presentation models. Why the reduction of actions the onFnished handler can do? And of course onFinishedData is completely missing here (or did it get removed in open dolphin as well?).

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 145 [r3] (raw file):

                        Optional.ofNullable(attr.get("tag")).map(t -> {
                            if (t instanceof Tag) {
                                return (Tag) t;

from the looks of it in the old code tag was always a string and never a Tag or a String. Why can it suddenly be a Tag?

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 174 [r3] (raw file):

            Optional oldVal = Optional.ofNullable(attribute.getValue());
            Optional newVal = Optional.ofNullable(serverCommand.getNewValue());
            if ((!oldVal.isPresent() && newVal.isPresent()) || (oldVal.isPresent() && !newVal.isPresent()) || (oldVal.isPresent() && newVal.isPresent() && !oldVal.get().equals(newVal.get()))) {

I can only assume this is supposed to represent strictMode=false. Which means I set the attribute only if the values differ which means to me: !oldVal.equals(newVal).  Optional will take care of the unwrapping of equals

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 200 [r3] (raw file):

                if (copies != null && !copies.isEmpty()) {
                    if (null == serverCommand.getNewValue()) {
                        attribute.setValue(copies.get(0).getValue());

in the groovy code the value was set only if the the entry in the copies list is not null.

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 251 [r3] (raw file):

            ClientAttribute attribute = getClientModelStore().findAttributeById(serverCommand.getAttributeId());
            if (attribute != null && serverCommand.getMetadataName() != null) {
                if (serverCommand.getMetadataName().equals(Attribute.BASE_VALUE)) {

I think this is the wrong place to do this kind of logic.... unless you are sure there is never ever going to be any different attribute type. In the old code setAt would have handled this1

platform/dolphin-platform-client/src/main/java/com/canoo/dolphin/client/AbstractConnector.java, line 290 [r3] (raw file):

        }
        waiting.set(false); // release is under way
        releaseHandler.submit(() -> {

the withPool in the old code did potentially use more than one thread to call transmit

Comments from Reviewable

Changes Unknown when pulling 211fc3b6865dcc8adff3536c5daff5f08ddd6799 on client-connection-java into ** on master**.

Changes Unknown when pulling 2102958e7170ce2742c36cc37f7aea15ea7fa92a on client-connection-java into ** on master**.


Debugging lesson feedback

Show that you can change variables and run commands in debugging terminal instead of restarting.

This has been added to the script.


Disable cache for travis-ci build

Summary
Disabling cache in travis-ci to avoid problems seen when using outdated dependancies
Thanks for contributing to Terra.
@cerner/terra

This build time was actually faster, though maybe not because of this pull request.
Cache disabled:

Control:


unexpected error error="Resource temporarily unavailable"

I have multiple filebeat instances to send logs to td agent. Once i enable fluent-plugin-beats with following configuration, i observe a  unexpected error error="Resource temporarily unavailable" in td-agent log.
ruby process goes to 100% CPU utilization and i see huge number of CLOSE_WAIT connections piling up on fluentd server.
I am using fluent-plugin-beats (1.0.0) on fluentd (1.2.2)
2018-12-20 09:02:37 +0000 [error]: #0 unexpected error error="Resource temporarily unavailable"
  2018-12-20 09:02:37 +0000 [error]: #0 /opt/td-agent/embedded/lib/ruby/gems/2.4.0/gems/fluent-plugin-beats-1.0.0/lib/lumberjack/beats/server.rb:343:in `sysread'
  2018-12-20 09:02:37 +0000 [error]: #0 /opt/td-agent/embedded/lib/ruby/gems/2.4.0/gems/fluent-plugin-beats-1.0.0/lib/lumberjack/beats/server.rb:343:in `read_socket'
  2018-12-20 09:02:37 +0000 [error]: #0 /opt/td-agent/embedded/lib/ruby/gems/2.4.0/gems/fluent-plugin-beats-1.0.0/lib/lumberjack/beats/server.rb:320:in `run'
  2018-12-20 09:02:37 +0000 [error]: #0 /opt/td-agent/embedded/lib/ruby/gems/2.4.0/gems/fluent-plugin-beats-1.0.0/lib/fluent/plugin/in_beats.rb:109:in `block in run'
  2018-12-20 09:02:37 +0000 [error]: #0 /opt/td-agent/embedded/lib/ruby/gems/2.4.0/gems/concurrent-ruby-1.1.4/lib/concurrent/executor/ruby_thread_pool_executor.rb:348:in `run_task'
  2018-12-20 09:02:37 +0000 [error]: #0 /opt/td-agent/embedded/lib/ruby/gems/2.4.0/gems/concurrent-ruby-1.1.4/lib/concurrent/executor/ruby_thread_pool_executor.rb:337:in `block (3 levels) in create_worker'
  2018-12-20 09:02:37 +0000 [error]: #0 /opt/td-agent/embedded/lib/ruby/gems/2.4.0/gems/concurrent-ruby-1.1.4/lib/concurrent/executor/ruby_thread_pool_executor.rb:320:in `loop'
  2018-12-20 09:02:37 +0000 [error]: #0 /opt/td-agent/embedded/lib/ruby/gems/2.4.0/gems/concurrent-ruby-1.1.4/lib/concurrent/executor/ruby_thread_pool_executor.rb:320:in `block (2 levels) in create_worker'
  2018-12-20 09:02:37 +0000 [error]: #0 /opt/td-agent/embedded/lib/ruby/gems/2.4.0/gems/concurrent-ruby-1.1.4/lib/concurrent/executor/ruby_thread_pool_executor.rb:319:in `catch'
  2018-12-20 09:02:37 +0000 [error]: #0 /opt/td-agent/embedded/lib/ruby/gems/2.4.0/gems/concurrent-ruby-1.1.4/lib/concurrent/executor/ruby_thread_pool_executor.rb:319:in `block in create_worker'

Here is my td-agent config, and flilebeat configs;
<source>
  @type beats
  tag beats
  port 55445
  max_connections 350
</source>

filebeat.prospectors:

# Each - is a prospector. Most options can be set at the prospector level, so
# you can use different prospectors for various configurations.
# Below are the prospector specific configurations.

- type: log

  # Paths that should be crawled and fetched. Glob based paths.
  paths:
    - Z:\IISLogs\*\*
  document_type: iis
  exclude_lines: ["^#"]
  exclude_lines: ['.*OPTIONS.*']
  fields_under_root: true
  fields:
    severity: info

#----------------------------- Logstash output --------------------------------
output.logstash:
  # The Logstash hosts
  hosts: ["logstore.url:55445"]
  index: "logstash-%{+yyyy.MM.dd}"

I think I found where the issue was. Modified following in https://github.com/uchann2/fluent-plugin-beats/blob/master/lib/lumberjack/beats/server.rb and it seems to do the trick.
Added more error handling for,
  Errno::EAGAIN,
  Errno::EBADF,
  Errno::EACCES,
  IO::EAGAINWaitReadable

    rescue EOFError,
      OpenSSL::SSL::SSLError,
      IOError,
      Errno::ECONNRESET,
      Errno::EPIPE,
      Errno::EAGAIN,
      Errno::EBADF,
      Errno::EACCES,
      IO::EAGAINWaitReadable
      # EOF or other read errors, only action is to shutdown which we'll do in
      # 'ensure'
    rescue


Audio support

Have you provided audio support for this?

@kalyanijamunkar10 if you use react-native-audio-toolkit, you can integrate the audio support with this library rather easily.
I can send you an implementation example if you would like.

@samermurad yes, please share the implementation.

@kalyanijamunkar10 I uploaded a gist  of my solution in a project Im working on... a lot of the code is project related but it demonstrates how you can use  react-native-audio-toolkit in collaboration with react-native-media-controls.
I hope that helps, let me know if you need any explanation


Updated unit tests/code for 2.4

@beardface I updated this to be compatible with 2.4, and updated the unit tests.

:+1:


State API deprecation: moving samples to InMemoryStore and providing …

…comments for easy migration to azure table or cosmos db

@carlosscastro since you have PR #219 which builds on this PR's commit, is it okay if I close this pull request?

Branch is 1 year old. Closing.


Linkify elements using dangerouslySetInnerHTML

Sometimes we have a pure HTML text to parse and the links are not inside the  tags. On these cases, we want to render the HTML using dangerouslySetInnerHTML and linkify the content.
Example:
<Linkify>

Take a look on the link: https://github.com/tasti/react-linkify

</Linkify>

Please take a look on the Codesandbox example: https://codesandbox.io/s/DRlyjgNL6
On this example, the first div is correctly linkified but not the second. This PR tries to fix that.
Depends on #44

Cool! Need this feature!

Need it too ! Thx

I was trying this out today as I ran into same problem as you guys and @pablolmiranda it just doesn't work. Sure it turns links inside the dangerouslySetInnerHTML into links, but it breaks dangerouslySetInnerHTML all together such that html tags are now rendered to the page.  I.E. if dangerouslySetInnerHTML={{ __html: '
some text and a link.com
' }} it will print  tags but link.com will be linkified


Testing out pr summary as a PR

Also fixes highlighted yamllint issues
Pulls linting out into a separate step

lgtm,


fix(ber): prevent race due to unnecessary var

address #23
Name of feature:
Fix race conditions in ber file
Pain or issue this feature alleviates:
Prevent race condition due to async access to ber's debugging global variable
Why is this important to the project (if not answered above):
Projects that use this package and run go test ./... -race will probably end up in situation like below:
WARNING: DATA RACE
Read at 0x0000021c46a0 by goroutine 33:
  github.com/smallstep/pkcs7.asn1Structured.EncodeTo()
      /.../vendor/github.com/smallstep/pkcs7/ber.go:21 +0x56
  github.com/smallstep/pkcs7.(*asn1Structured).EncodeTo()
      <autogenerated>:1 +0xa5
  github.com/smallstep/pkcs7.ber2der()
      /.../vendor/github.com/smallstep/pkcs7/ber.go:68 +0xdb
  github.com/smallstep/pkcs7.Parse()
      /.../vendor/github.com/smallstep/pkcs7/pkcs7.go:171 +0x12f

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update EEPROM to 2022-10-18

Also make the secure-boot-recovery pieeprom.original.bin a symlink to ../pieeprom.original.bin. There's no need to have different binaries so remove duplication.

@peterharperuk Bump usbboot default EEPROM to latest stable (signed with secure-boot ROM key)

Looks ok. I thought we avoided symlinks due to Windows being rubbish?

Windows users can use WSL or Cygwin


Add connection type 3 and handle when it is unknown

Add connection type 3 as 5Ghz connection, also handle if connection type returned is not known to prevent future tracebacks.

This fixes issue #336.
Confirmed locally.

Thanks! 👍🏻


Bug with different keyboard layout in Big Sur

There might be a bug that doesn't show the correct keyboard shortcut in the dropdown menu.
I recently upgraded from Mojave to Big Sur and couldn't get the toggle window to work. I use the Dvorak keyboard layout and the pressing ⇧ ⌘ V didn't do anything. It turns that the shortcut was mapped to ⇧ ⌘ K instead (K on the Dvorak layout is at the same position as V on the QWERTY layout).
I have remapped the shortcut now, but the menu bar displays the combination ⇧⌘V to toggle the Yippy window, no matter what mapping I put in the preferences.

Hello @pasricha, sorry there have been other issues with non-standard keyboards in the past, however, I am unable to test changes to this as I don't have a Dvorak keyboard. I will happily review any PRs made to improve this 😄


C++ usage for argosmodels

Hi there, I've been looking into translating argosmodels for a while now, and I want to know how to translate your pre-trained .argosmodels on C++.
Is there a sample script that describes the steps?

You would need to use CTranslate2 to do inference on the models, and SentencePiece for tokenization (both in C++).
You would need to figure something else out for sentence boundary detection because Argos Translate uses Stanza which is only available for Python. One option is to do something like this.


TensorBoard: Embeddings visualization not supported yet

For a project, I create some .tsv files. They are saved within the TensorBoard dir. When this directory is e.g. /tensorboard/run20201711/, the files are saved under this directory. The attached screenshot shows such logging structure.

I then check to see if I can visualize these files as embedding files (similar to projector.tensorflow.org), in the TensorBoard tab on my W&B panel.
However, it greets me with no data found - all other metrics are logging fine, images and so are uploaded correctly.
From your documentation at https://docs.wandb.com/library/log, I found no way to manually log the files, since .tsv/.csv do not seem to be supported.
How can I sync the files to my W&B project?

You can use wandb.save: https://docs.wandb.com/library/save or just write the files into wandb.run.dir.

Thanks, this solves my problem. Did not know about this feature!

Edit: While W&B does now successfully sync the files to cloud, when using TensorBoard via your web interface, I can't load the embeddings.
Following structure is present after the run:

And this is the screen it greets me with when checking the embeddings:

Hey @YanickSchraner
I tried reproducing your issue.
I followed this tutorial and made the tsv files.
While following the tutorial, I have changed slightly and used logs/ as the directiry where the checkpoints and tsv files are created. Due to the fact that wandb tensorboard get triggered only in the presence of tfevents, I went out and added a few lines of code and creates some scalar tfevents
writer = SummaryWriter("logs")
scalars = {
    'scalar_1': 1.0,
    'scalar_2': 2.0,
    'scalar_3': 3.0
}
writer.add_scalars('test', scalars, global_step=1)
writer.close()

Later I have used the following scrip to sync the files to wandb
wandb.init(entity='xxx', project='xxx')
wandb.save('logs/*', base_path="logs/")
wandb.finish()

This has the following files synced

With this in place, the tensorboard projector seems to be working like a charm

I think the problem here is the checkpoint creation and the file structure. I think having all the files in the root directory is easier for  tensorboard to locate.

Still not working for me, while on a local version of Tensorboard everything works fine. Any updates on this?

Sure, sorry for the delay.
What I mean is that when generating an embedding projection with writer.add_embedding() (I am using pytorch and torch.utils.tensorboard) I can launch locally tensorboard and use the projector correctly.
All files are written in the "files" directory automatically created by wandb for the current run, so that everything is pushed on the wandb server as well.
On the other hand, when I use the tensorboard tab from the wandb web page (as showed above) I can see everything that I logged (i.e. lr, loss, metrics...) but the projector tab shows a blank page.

That is interesting
Would you like to share a colab notebook for us to quickly review the whole thing and revert back to you?

Yes, I cannot directly share my code but I will try to make a minimal reproducible example in the following days. I'll keep you posted.

In the past year we've majorly reworked the CLI and UI for Weights & Biases. We're closing issues older than 6 months. Please comment to reopen.

@sydholl Still doesn't work correctly...
Here is a MRE
I think if you can copy recursively the produced file structure under tensorboard logs inside wandb files it would work but wandb.save doesn't allow me to do so.


[3/4] Orchestrator API integration tests

What does this PR do

Adds tests for the JS, KV interface implementations

force push to rebase on changes in the base branch


Remove usage of promisifyAll

So this is more of a discussion topic than a task, but here we go.  In a lot of our libraries, we have a callback-first mentality.  We assume callback, and then brute force promisifyAll on many classes.  I'd like to flip this bit.  We should assume async first, then provide callbackification as needed. Thoughts on the subject appreciated!

Would this require a signature change for the callback version though?

Nope!  This would be exactly the same as promisifyAll, but different, but the same.

Haha, ok. So as far as the scope of this issue is concerned, promises would still return Arrays and we'd more or less just spread that to the callback. SGTM!

Handling this on a case-by-case basis.


TS-1257: use hashmap instead of inkhashtable for ssl cipher table

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/164/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/179/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1860/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/160/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/290/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/1750/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/423/


Process mining from scratch

Is it difficult to create a custom process mining algorithm from scratch ?, maybe just a modification of a indutive or heuristic

It is challenging, but i believe it can be done if you have a solid background in algorithm design. To start, you need to identify limitations / weaknesses of the current algorithms.

Thanks


Added method to search images on Docker Hub

Also edited .gitignore to ignore Eclipse settings ;-)

@rohansingh could you review this PR as well ? Thanks in advance :-)

Functionality looks great. Just had a few comments inline, all minor.

@rohansingh @davidxia Thanks for your reviews. I applied the changes after your comments and updated the PR. Please let me know if there's anything else I need to fix.

Merged. Thanks for the contribution!

@rohansingh thanks to you, too ;-)


Automatic tests on pull requests

For these packages without GUI (like data, utils etc.), tests are quite feasable, therefore I suggest, that the tests are run for pull requests, to ensure correct working for all changes.

but they are already ??

A quick check on the github documentation:

"push" works on pushed commits in this repository (so on your branches), but not on pushes to branches in other repositories (like mine)

What we use in pymeasure:
on:
  push:
    branches:
      - master
      - develop
  pull_request:

that way only pushes to the main or develop branch (no feature branches) are checked, or a pull request (maybe of a feature branch).

nice! But then your issue is about PR while here you're talking about commits. Are we clear you 'd like tests to run on every push even on other branches? But then which branch should we include? Can we not do it all the time no matter the branch or fork?

Sorry for the confusion by talking about the different possibilities.
Right now, it is run on all pushes in this repository (so no forks).
The "push" section considers only this repository (no forks).
I suggest to run it on pull_requests (from this repo or other repos).
Additionally, it might be good to run it push to the master branch (so even code which enters outside a pull request, e.g. direct push) gets tested. This test will be run after pull requests, making sure, that the merged code works as expected.
Whether pushes to other branches of your repo trigger a test run or not is not that relevant.
If these branches are part of a pull request, the tests will be executed twice (once for the PR and once for the push), if I remember correctly.
Therefore, the Pymeasure tests are only for the master branch (and an unused "develop" branch).
TL;DR: Please add tests for pull requests. You might restrict tests for pushes at the same time, if you wish so.

PR #13 is fixing this issue


gnu(9920)=12141

This took 25219.574 seconds to construct the groups, yielding one pair of groups that grpconst could not split. The non-isomorphism test took long, but I do not have a record of the time, so it would be good if we got that with verification.

Thank you for submitting new gnu(n). We will recalculate it and add to the database if we get the same result.
Checklist for the reviewer:

[ ] checked and assigned labels "new (n<50000)" or "recomputation", if needed
[ ] indicated that the recomputation has been started
[ ] reproducible value of gnu(n)
[ ] no essential runtime differences
[ ] sufficient level of technical details
[ ] assigned label "validated" after successful review

Checklist for the maintainer:

[ ] correct arguments of commands to store data
[ ] correct location of new line in the data file
[ ] mention reviewer in the commit message

I have restarted the replication, but I already see that the value in the commit message and the description of the PR differs from the value in the code.

gnu(9920)=12142
GAP      4.8.3
GrpConst 2.5
Runtime of ConstructAllGroups: 13242896 ms
Returned a list of length 12141 containing 12140 groups and 1 sublists
Length of sublists : [ 2 ]
Runtime to check non-isomorphism of groups in pairs: 198279596
Replicated by @alex-konovalov on Ubuntu 16.04 on Azure cloud standard DS3 v2 instance (4 cores, 14 GB RAM)

@markuspf perhaps you just entered the length of the constructed list ? If you edit the PR title and amend commit text, I will merge this.

I seem to have a strong Copy&Paste impediment. I think next I should find a way to automatically submit these results as a pull request.

The same result was also reported by me in #35.


fixes alfredh/baresip#93 glibc >=2.20 _BSD_SOURCE warning

Thank you Sebastian :)
quickly tested on Fedora 22, Debian Testing and OpenBSD 5.8


Loss curve

Hi, thanks for the great work!
I am trying to implement your method on my own dataset. Would you please post a figure of the loss changes with respect to the training epochs? It would be very helpful.
Thanks!

Hi,
I send a log file to you by e-mail.  If you didn't receive the email contact me at<EMAIL_ADDRESS>

Thank you! I have received the Email.

Hey! If you can send me the log files for the unsupervised training and the classifier head evaluation, I would be really grateful as well, thanks!

Hello, if possible, please send a log to me as well of SSL pre-training. Thanks

Hey @Adrien987k, great work!
I've just shot an email to you asking for the log file. Thanks in advance!

Hello, would you send a log to me as well of SSL pre-training? Thanks!

Hey @acharkq Could you forward me the pre-training log file at my email id<EMAIL_ADDRESS>


http://www.gbif.org/newsroom/news/rss gives 404

Yup—@dnoesgaard and I just waiting on a redirect to add it under RSS feeds on the newsletters page...

I know this is in place for events, but is it for other content (news, data use)?
@fmendezh, @timrobertson100 ?

What is the endpoint for news RSS?


Add job image to copy images from DockerHub to GCR

/kind enhancement
What this PR does / why we need it:
Add job image to copy images from DockerHub to GCR
Which issue(s) this PR fixes:
Part of #619
Special notes for your reviewer:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

/ok-to-test

@timebertt Thanks for your remarks. I updated it and fixed an architecture mismatch I found.


DataSource with SQL auth coming out of JNDI can't connect to database

Driver version
6.2.2.  I believe this worked correctly in 2.0, and it has been an issue since 3.0 all the way to the current 7.0.
SQL Server version
Microsoft SQL Server 2014 (SP2-GDR) (KB4057120) - 12.0.5214.6 (X64)
Jan  9 2018 15:03:12
Copyright (c) Microsoft Corporation
Express Edition (64-bit) on Windows NT 6.1  (Build 7601: Service Pack 1)
Client Operating System
Development: Windows 7.0 Professional.
Build: CentOS 6.0.
JAVA/JVM version
java version "1.8.0_191"
Java(TM) SE Runtime Environment (build 1.8.0_191-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.191-b12, mixed mode)
openjdk version "1.8.0_191"
OpenJDK Runtime Environment (build 1.8.0_191-b12)
OpenJDK 64-Bit Server VM (build 25.191-b12, mixed mode)
Table schema
Don't need one.  See unit test below.
Problem description

Expected behaviour:
I expect to be able to put a functional SQLServerXADataSource with SQL login into JNDI, and when I get it back out be able to get a usable connection.

Actual behaviour:
The new DataSource does not know enough to be able to connect to the database.  There is a login error on the connection.

Error message/stack trace:

com.microsoft.sqlserver.jdbc.SQLServerException: Login failed for user 'XXXX'. ClientConnectionId:2f6eadae-a703-4ed9-9773-9fd06b0c5a63
	at com.microsoft.sqlserver.jdbc.SQLServerException.makeFromDatabaseError(SQLServerException.java:258)
	at com.microsoft.sqlserver.jdbc.TDSTokenHandler.onEOF(tdsparser.java:256)
	at com.microsoft.sqlserver.jdbc.TDSParser.parse(tdsparser.java:108)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.sendLogon(SQLServerConnection.java:4290)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.logon(SQLServerConnection.java:3157)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.access$100(SQLServerConnection.java:82)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection$LogonCommand.doExecute(SQLServerConnection.java:3121)
	at com.microsoft.sqlserver.jdbc.TDSCommand.execute(IOBuffer.java:7151)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.executeCommand(SQLServerConnection.java:2478)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.connectHelper(SQLServerConnection.java:2026)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.login(SQLServerConnection.java:1687)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.connectInternal(SQLServerConnection.java:1528)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.connect(SQLServerConnection.java:866)
	at com.microsoft.sqlserver.jdbc.SQLServerDataSource.getConnectionInternal(SQLServerDataSource.java:968)
	at com.microsoft.sqlserver.jdbc.SQLServerDataSource.getConnection(SQLServerDataSource.java:69)
	at com.mycompany.common.sqltests.DataSourceTest.runSimpleQuery(DataSourceTest.java:40)
	at com.mycompany.common.sqltests.DataSourceTest.testSomething(DataSourceTest.java:34)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:606)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
	at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
	at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
	at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
	at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
	at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
	at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
	at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
	at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
	at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:89)
	at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:41)
	at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:541)
	at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:763)
	at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:463)
	at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:209)

Any other details that can be helpful:

It appears that a "Reference" object, which is an interpretive dance version of the original DataSource that was created using SQLServerDataSource.getReferenceInternal(...), is being used to try to create a new DataSource.  However, notice that in the getReferenceInternal() method that creates the Reference in the first place, there is the following code:
                // do not add passwords to the collection. we have normal password
                if (!propertyName.contains(SQLServerDriverStringProperty.PASSWORD.toString()))
                    ref.add(new StringRefAddr(propertyName, connectionProps.getProperty(propertyName)));

The password therefore doesn't make it to the copy of the data source, and it will therefore be unable to create a valid connection to the database.  I'm not sure what the comment is implying by "we have normal password"; there must have been a reason for it in the first place.
JDBC trace logs
Server log gives:
2018-11-29 11:46:25.25 Logon       Login failed for user 'XXXX'. Reason: Password did not match that for the login provided. [CLIENT: <IP_ADDRESS>]

Reproduction code
Here's a unit test.  Note we are using SQL Server logins, not integrated logins.  The unit test doesn't require a full schema, just a blank database to run against.  You'll have to go in and fix up the username, password, and jdbcurl.
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.ResultSetMetaData;
import java.sql.SQLException;
import java.sql.Types;

import javax.naming.Context;
import javax.naming.InitialContext;
import javax.naming.NameNotFoundException;
import javax.naming.NamingException;
import javax.naming.Reference;
import javax.sql.DataSource;

import org.junit.Assert;
import org.junit.Test;

import com.microsoft.sqlserver.jdbc.SQLServerXADataSource;

public class DataSourceTest
{
	@Test
	public void testSomething()
		throws SQLException, NamingException
	{
		SQLServerXADataSource ds = createDataSource();
		runSimpleQuery(ds); // Proves that the initial DataSource is functional.

		registerDataSource("test", ds);
		DataSource newDs = lookUpJdbcDataSource("test");

		runSimpleQuery(newDs);
	}

	private void runSimpleQuery(DataSource newDs)
		throws SQLException
	{
		try (Connection conn = newDs.getConnection())
		{
			try (PreparedStatement ps = conn.prepareStatement("SELECT CAST('2017-01-01' AS Date) AS d1"))
			{
				try (ResultSet rs = ps.executeQuery())
				{
					Assert.assertTrue(rs.next());
					ResultSetMetaData rsMetaData = rs.getMetaData();
					Assert.assertEquals(Types.DATE, rsMetaData.getColumnType(1));
					Assert.assertEquals("2017-01-01", rs.getString(1));
				}
			}
		}
	}

	private SQLServerXADataSource createDataSource()
	{
		String url = "jdbc:sqlserver://localhost\\SQLEXPRESS;databaseName=???;autoReconnect=true;applicationName=blah";
		String userName = "XXXX";
		String password = "SOMEPASSWORD";

		SQLServerXADataSource ds = new SQLServerXADataSource();
		ds.setURL(url);

		// not integrated security
		ds.setUser(userName);
		ds.setPassword(password);
		return ds;
	}

	public static synchronized DataSource lookUpJdbcDataSource(String lookupName)
		throws SQLException
	{
		try
		{
			Context initContext = new InitialContext();
			Context envContext  = (Context)initContext.lookup(JDBC_PREFIX);
			Object object = envContext.lookup(lookupName);
			if (object instanceof Reference)
			{
				Reference ref = (Reference)object;
				throw new NamingException("For " + lookupName +
					", only found Ref for " + ref.getClassName() +
					", factory=" + ref.getFactoryClassName() +
					", factoryLocation=" + ref.getFactoryClassLocation());
			}
			return (DataSource)object;
		}
		catch (NamingException ne)
		{
			throw new SQLException("Lookup failure name=" + lookupName, ne);
		}
	}

	private static final String JDBC_PREFIX = "java:/comp/env/jdbc/";

	public void registerDataSource(String dataSourceName, DataSource ds)
		throws NamingException
	{
		// The code to allow for JNDI lookup in unit tests comes from this page:
		// https://blogs.oracle.com/randystuph/entry/injecting_jndi_datasources_for_junit
		// The contextfactory requires lib/catalina.jar and bin/tomcat-juli.jar
		// Create initial context
		System.setProperty(Context.INITIAL_CONTEXT_FACTORY,
			"org.apache.naming.java.javaURLContextFactory");
		System.setProperty(Context.URL_PKG_PREFIXES, "org.apache.naming");
		InitialContext ic = new InitialContext();
		String resourceName = JDBC_PREFIX + dataSourceName;

		try
		{
			ic.lookup("java:");
		}
		catch (NameNotFoundException nnfe)
		{
			ic.createSubcontext("java:");
			ic.createSubcontext("java:/comp");
			ic.createSubcontext("java:/comp/env");
			ic.createSubcontext("java:/comp/env/jdbc");
		}
		ic.bind(resourceName, ds);
	}
}

Hi @maeichris
I can reproduce your problem.
This is due to the reason that "password" is not set when setting reference as you can see on this line.
Password is not stored in encrypted format by the driver, and hence is not passed on to JNDI reference.

Hi, @cheenamalhotra, thanks for looking into this.
You mention that the password is not stored in an encrypted format by the driver and not passed on to JNDI for security reasons.  I don't have a good understanding of the security concerns in this regard.
Does that mean that putting a DataSource into JNDI is a bad practice?  (My impression is that is how it's done generally in a J2EE application server, and my test was an attempt to mimic that in straight code.)  Or does it mean that I should store the password somewhere else and reset the password on the DataSource every time I pull a DataSource out of JNDI?  I'm not sure what the best practice is for something like this.
Thanks,
Chris

Hi @maeichris
I investigated more about this issue and looks like other JDBC Drivers do send password information when registering driver into JNDI. We will need to seek security clearance for this support and will hopefully support this change in near future.
I will follow up for this change and will let you know when I have any update.

There was never a response about getting the security clearance that @cheenamalhotra mentioned a little over a year ago.
The line of code that removes the password was in the initial commit of the code on Nov 9, 2016.
https://github.com/microsoft/mssql-jdbc/commit/cfba660ac950a14da127cb9399430425473fb46f#diff-18d0d0627e5a0b7a9abe73358d2358a8
I am still running with version 2.0 of the driver because of this issue (the last one where this worked).  I am a bit surprised that no one else has come across the problem, unless there is some other way of getting around it that I am not aware of (very possible).

Hi @maeichris ,
Sorry for the delay, we will look into this and get back to you.

Hi @maeichris,
Sorry for the delay. As was discussed in the above comments, the driver is intentionally removing the password as a property in this case for security concerns. The idea is that if the same client with knowledge of the password for the original DataSource that was created, the client should be able to provide the password again after retrieving the reference for that DataSource. I don't think putting DataSource into JNDI is a bad practice, but as you suggested, you should store the password somewhere else and provide the password again after you pull it out of JNDI, at least for our driver.
Please let me know if this solves your problem, and sorry for not getting back to you earlier on this.

Hi, @peterbae,
Thanks for taking up this matter, and for the explanation.  Based on that, I did some digging/debugging to figure out how it actually works in the container (Tomcat 8.5).
In the container, the application knows nothing of the database connection, including whether it is using integrated or SQL Server logins.  The container sets up the DataSource.  The application gets the DataSource and then gets a connection without the application having to specify a password.  The container has managed it all.
Previously I was creating the DataSource like this:
	SQLServerXADataSource ds = new SQLServerXADataSource();
	ds.setURL(url);

	if (StringUtils.isNotBlank(userName))
	{
		// not integrated security
		ds.setUser(userName);
		ds.setPassword(password);
	}

It appears Tomcat 8.5 bypasses SQLServerXADataSource/SQLServerDataSource altogether, and uses org.apache.tomcat.dbcp.dbcp2.BasicDataSource, which contains an org.apache.tomcat.dbcp.dbcp2.PoolingDataSource.  These in turn talk to the driver specifically, which is specified in the context's Resource configured with type="javax.sql.DataSource" and driverClassName="com.microsoft.sqlserver.jdbc.SQLServerDriver".
Therefore, I tried creating a BasicDataSource like this and it seems to work:
	BasicDataSource ds = new BasicDataSource();
	ds.setDriverClassName(com.microsoft.sqlserver.jdbc.SQLServerDriver.class.getName());
	ds.setUrl(url);

	if (StringUtils.isNotBlank(userName))
	{
		// not integrated security
		ds.setUsername(userName);
		ds.setPassword(password);
	}

This means that the unit tests (okay, actually integration tests) would need to add a dependency on dbcp.
While I am not sure why a DataSource would want to remove the password, a workaround is to use a kind of DataSource that does not remove it.  I am guessing the reason why the SQLServerXADataSource/SQLServerDataSource remove the password is for cases where a single DataSource is used to support multiple logins.  But an app within a container probably uses a single account, thus the reason Tomcat uses DataSource which sets the password once for each connection that is created on the application's behalf.
Since Tomcat has a system that assures the password is set for each new connection, the behavior is probably expected.  As such, that makes me think this ticket is not valid (and the workaround I wrote above may help future people with the same problem).  If you agree, please go ahead and close out this ticket.

Thank you for the investigation @maeichris, I agree with your findings. I'll close the ticket.


for-loop labels cannot be renamed

I'm using tsc 2.0.0 in vs 2015 (with update 3 installed) and trying to rename a for-loop label:
loop: for (const item of items) {
  for (let i = 0; i < n; i++) {
    if (test(i, item))
      continue loop;
  }
}

The ts language service says "You cannot rename this element." However when I click on the loop label, it correctly highlights all its occurrences.

Why would this be an issue? The highlighting you are seeing is textual not semantic (I'm not 100% positive). In this example, recursion, (I know there isn't PTC in most browsers yet) really would make for clearer code.

Why would this be an issue?

labels are a v (short for very) underused language feature and i dont see why they shouldn't be renamable
you can even use them for weird goto-like control flow (break check == goto error)

The highlighting you are seeing is textual not semantic (I'm not 100% positive)

i think they are highlighted correctly:

Thank you @a-tarasyuk!


milvus bucket config missing in mgmt cluster profile

Milvus Helm Chart is configured with general S3/Objects values but bucket is

 Fix Latest Issues found in Repo


Bicep in VS Code won't display govcloud subscriptions

Bicep version
0.29.47
Describe the bug
Either by right-clicking and selecting deploy bicep file, or by using the deployment pane, I am unable to select a subscription in any govcloud region. An alert appears in VS Code (bottom right) that states no subscriptions found.
To Reproduce
Attempt to deploy bicep file to azure govcloud.
Additional context
Spoke with an individual on the Azure Discord server who suggested it might be a wrong ARM call to "GET" the subscriptions and the appropriate error isn't presenting.
Attempted at 5:45PM Eastern on 8/13/2024 for any log searching.

I couldn't reproduce this issue. @alaskascooter, the Bicep VS Code extension relies on the Azure Account VS Code extension for authentication. Have you tried logging in and selecting a tenant ID using the following command?

Thanks for trying. When I try the method you suggest, this is the error I receive. Would this indicate that it is the azure component giving me issues? I am able to browse resources in the only subscription I am assigned to using the Azure Resource browser.

Thanks again.

Can you access the national cloud portal (e.g., https://portal.azure.us)?

Yes, clicking on that link I am taken to my portal home screen for azure gov and can see all my resources.

It seems this might be a bug in the Azure Account VS Code extension. I recommend trying to run Azure: Sign Out and then Azure: Sign In to Azure Cloud to see if that resolves the issue.
If it still doesn't work, it might be worth reporting the issue at https://github.com/microsoft/vscode-azure-account/issues.

Thanks, I've attempted that a few times now.
I'll jump over there and see what they can do for us.
Thank you

FYI, their troubleshooting page provided some steps required before posting an issue related to logins. It appears to have fixed it.
Specifically, I think it was clearing the tenant id from cache and then reauthenticating, selecting the tenant, and then subscriptions. Now the Bicep deployment panel allows me to see all the resource groups within my subscription.
Can be closed, thank you.

it then says SubscriptionNotFound despite it being in the Deployment Scope window having been selected

Could you share a screenshot of the error? I'm trying to determine whether it's an issue with ARM or something on the client side.

Here is the error within the bicep pane:

And the Deployment Scope section:

ok, this is weird. I just tried using the right-click deploy bicep in the cli and it ran. No errors. Well, it had errors, but now it's my actual code that is wrong, not the fact that it couldn't find the subscription.
I'm going to do some more troubleshooting and restarts. I'm also setting up a system outside the company network to see if it's something with corporate firewall tomfoolery.

Sounds like this is a bug in deployment pane then. I'll take a deeper look into the root cause.

I see this behavior, too.


How to install/configure server

Can anyone provide more instruction on any other config required server side in laravel ? Instructions here are a little bare - install the package, add the trait, and then what ?  How to access ? oData php provider talks about apache mod-rewrite etc, is that still necessary ?
Sorry if this is not the right place, but would love some more instruction/tutorial on usage :)

Sorry, makapaka, life happened to me.
The instructions are fairly bare because there isn't that much to it - bolting in the model and controller traits, adding the definitions needed (for the controller trait), etc.  There just isn't that much Laravel-side config required, as far as I understand - but being one of the guys who built it, I may not be able to see the forest for the trees.
The great bit about this being a Laravel package, from my own experience, is that it piggybacks onto however you've set your Laravel install up, as it's HTTP requests and responses flying back and forth.


Strange behaviour of next/prev buttons

Hi there, just new here. Thanks for the awesome theme.
I've installed a fresh copy of it onto my Ghost 4.0.1 and there is some strange behaviour with the next and prev buttons.
The whole design seems a bit off.  The website is http://redvan.studio
I'm on safari on MacOS, but I tried on my phone and it showed up the same on there.
Anyone else having an issue like this? Thanks for any help.

Sorry about close and re-open. Still figuring out Github

Hey @brendantko, here's how it looks on Safari macOS on my side. Have you managed it somehow?

Hey yeah, I made a dumb mistake and didn't realize some form CSS from another plug in was interfering with the CSS of the navigation on here. My bad >.<  but it's all good now. Thanks for checking into it!